Remove URLRequestJob::filter_input_byte_count_

net/url_request/url_request_job.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef NET_URL_REQUEST_URL_REQUEST_JOB_H_
 #define NET_URL_REQUEST_URL_REQUEST_JOB_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 321 matching lines @@
 
   // The status of the job.
   const URLRequestStatus GetStatus();
 
   // Set the status of the job.
   void SetStatus(const URLRequestStatus& status);
 
   // Set the proxy server that was used, if any.
   void SetProxyServer(const HostPortPair& proxy_server);
 
-  // The number of bytes read before passing to the filter.
+  // The number of bytes read before passing to the filter.  Include bytes
+  // when there is no filter.

[Randy Smith (Not in Mondays), 2015/04/10 22:03:58]

nit: This doesn't feel like a complete sentence to me?  Maybe "This value
reflects bytes read even when there is no filter."?

[mmenke, 2015/04/10 22:36:49]

Done.

   int prefilter_bytes_read() const { return prefilter_bytes_read_; }
 
-  // The number of bytes read after passing through the filter.
+  // The number of bytes read after passing through the filter.  Include bytes
+  // when there is no filter.
   int postfilter_bytes_read() const { return postfilter_bytes_read_; }
 
-  // Total number of bytes read from network (or cache) and typically handed
-  // to filter to process.  Used to histogram compression ratios, and error
-  // recovery scenarios in filters.
-  int64 filter_input_byte_count() const { return filter_input_byte_count_; }
-
   // The request that initiated this job. This value MAY BE NULL if the
   // request was released by DetachRequest().
   URLRequest* request_;
 
  private:
   // When data filtering is enabled, this function is used to read data
   // for the filter.  Returns true if raw data was read.  Returns false if
   // an error occurred (or we are waiting for IO to complete).
   bool ReadRawDataForFilter(int *bytes_read);
 
@@ skipping 28 matching lines @@
   // |location| and |http_status_code|.
   RedirectInfo ComputeRedirectInfo(const GURL& location, int http_status_code);
 
   // Indicates that the job is done producing data, either it has completed
   // all the data or an error has been encountered. Set exclusively by
   // NotifyDone so that it is kept in sync with the request.
   bool done_;
 
   int prefilter_bytes_read_;
   int postfilter_bytes_read_;
-  int64 filter_input_byte_count_;
 
   // The data stream filter which is enabled on demand.
   scoped_ptr<Filter> filter_;
 
   // If the filter filled its output buffer, then there is a change that it
   // still has internal data to emit, and this flag is set.
   bool filter_needs_more_output_space_;
 
   // When we filter data, we receive data into the filter buffers.  After
   // processing the filtered data, we return the data in the caller's buffer.
@@ skipping 20 matching lines @@
   NetworkDelegate* network_delegate_;
 
   base::WeakPtrFactory<URLRequestJob> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(URLRequestJob);
 };
 
 }  // namespace net
 
 #endif  // NET_URL_REQUEST_URL_REQUEST_JOB_H_