Cleanup code to keep flag showing "was cached" only in URLRequestHttpJob

r=wtc
Review URL: http://codereview.chromium.org/113535

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@16435 0039d316-1c4b-4281-b951-d872f2087c98

3 files changed, 6 insertions, 12 deletions

diff --git a/net/base/filter.cc b/net/base/filter.cc
index 73cafd4..d12124c 100644
--- a/net/base/filter.cc
+++ b/net/base/filter.cc
@@ -39,7 +39,7 @@ const char kTextHtml[]             = "text/html";
 
 Filter* Filter::Factory(const std::vector<FilterType>& filter_types,
                         const FilterContext& filter_context) {
-  DCHECK(filter_context.GetInputStreamBufferSize() > 0);
+  DCHECK_GT(filter_context.GetInputStreamBufferSize(), 0);
   if (filter_types.empty() || filter_context.GetInputStreamBufferSize() <= 0)
     return NULL;
 
@@ -263,7 +263,7 @@ Filter::~Filter() {}
 
 bool Filter::InitBuffer() {
   int buffer_size = filter_context_.GetInputStreamBufferSize();
-  DCHECK(buffer_size > 0);
+  DCHECK_GT(buffer_size, 0);
   if (buffer_size <= 0 || stream_buffer())
     return false;
 
diff --git a/net/base/sdch_filter.cc b/net/base/sdch_filter.cc
index 2c747bc..9ddac28 100644
--- a/net/base/sdch_filter.cc
+++ b/net/base/sdch_filter.cc
@@ -25,8 +25,7 @@ SdchFilter::SdchFilter(const FilterContext& filter_context)
       dest_buffer_excess_index_(0),
       source_bytes_(0),
       output_bytes_(0),
-      possible_pass_through_(false),
-      was_cached_(filter_context.IsCachedContent()) {
+      possible_pass_through_(false) {
   bool success = filter_context.GetMimeType(&mime_type_);
   DCHECK(success);
   success = filter_context.GetURL(&url_);
@@ -69,7 +68,7 @@ SdchFilter::~SdchFilter() {
     UMA_HISTOGRAM_COUNTS("Sdch3.UnflushedVcdiffOut", output_bytes_);
   }
 
-  if (was_cached_) {
+  if (filter_context().IsCachedContent()) {
     // Not a real error, but it is useful to have this tally.
     // TODO(jar): Remove this stat after SDCH stability is validated.
     SdchManager::SdchErrorRecovery(SdchManager::CACHE_DECODED);
@@ -196,7 +195,7 @@ Filter::FilterStatus SdchFilter::ReadFilteredData(char* dest_buffer,
           // Since we can't do a meta-refresh (along with an exponential
           // backoff), we'll just make sure this NEVER happens again.
           SdchManager::BlacklistDomainForever(url_);
-          if (was_cached_)
+          if (filter_context().IsCachedContent())
             SdchManager::SdchErrorRecovery(
                 SdchManager::CACHED_META_REFRESH_UNSUPPORTED);
           else
@@ -206,7 +205,7 @@ Filter::FilterStatus SdchFilter::ReadFilteredData(char* dest_buffer,
         }
         // HTML content means we can issue a meta-refresh, and get the content
         // again, perhaps without SDCH (to be safe).
-        if (was_cached_) {
+        if (filter_context().IsCachedContent()) {
           // Cached content is probably a startup tab, so we'll just get fresh
           // content and try again, without disabling sdch.
           SdchManager::SdchErrorRecovery(
diff --git a/net/base/sdch_filter.h b/net/base/sdch_filter.h
index 7255839..012de89 100644
--- a/net/base/sdch_filter.h
+++ b/net/base/sdch_filter.h
@@ -111,11 +111,6 @@ class SdchFilter : public Filter {
   // This is used to restrict use of a dictionary to a specific URL or path.
   GURL url_;
 
-  // To facilitate error recovery, we store whether this content came from a
-  // cache, as we then probably don't have the requsite dictionary, and will
-  // need to induce a meta-refresh.
-  const bool was_cached_;
-
   // To facilitate error recovery, allow filter to know if content is text/html
   // by checking within this mime type (we may do a meta-refresh via html).
   std::string mime_type_;