Use int64 instead of int32 for response sizes.

Use int64 instead of int32 for response sizes.

Previously, the reported size of responses larger than max int32 would
overflow, and these overflowed sizes were even surfaced to UI for the
Data Reduction Proxy.

Verified by building Chrome with and without this patch, and using the
Data Saver Chrome extension to download a 2.6 GB file. Without this
patch, a negative number is shown in the Data Saver stats. With this
patch, 2.6 GB is shown in the Data Saver stats.

BUG=475081
Committed: https://crrev.com/b5c82196f9bb0b0033a29ff0b14553e4c2e7f41d
Cr-Commit-Position: refs/heads/master@{#324738}

[sclittle, 2015-04-10 20:01:24]

sclittle@chromium.org changed reviewers:
+ bengr@chromium.org, mmenke@chromium.org

[sclittle, 2015-04-10 20:01:25]

bengr: components/data_reduction_proxy/*
mmenke: net/url_request/url_request_job.h

Thanks in advance!

[mmenke, 2015-04-10 20:03:25]

On 2015/04/10 20:01:25, sclittle wrote:
> bengr: components/data_reduction_proxy/*
> mmenke: net/url_request/url_request_job.h
> 
> Thanks in advance!

This issue looks to be corrupt.

[sclittle, 2015-04-10 20:09:36]

On 2015/04/10 20:03:25, mmenke wrote:
> On 2015/04/10 20:01:25, sclittle wrote:
> > bengr: components/data_reduction_proxy/*
> > mmenke: net/url_request/url_request_job.h
> > 
> > Thanks in advance!
> 
> This issue looks to be corrupt.

OK, I'll create a new issue and see if that helps.

[sclittle, 2015-04-10 20:41:16]

On 2015/04/10 20:09:36, sclittle wrote:
> On 2015/04/10 20:03:25, mmenke wrote:
> > On 2015/04/10 20:01:25, sclittle wrote:
> > > bengr: components/data_reduction_proxy/*
> > > mmenke: net/url_request/url_request_job.h
> > > 
> > > Thanks in advance!
> > 
> > This issue looks to be corrupt.
> 
> OK, I'll create a new issue and see if that helps.

It seems to be fixed now, PTAL.

Looks the corruption was caused by
https://code.google.com/p/chromium/issues/detail?id=476045.

[mmenke, 2015-04-10 20:43:50]

https://codereview.chromium.org/1074293003/diff/60001/net/url_request/url_req...
File net/url_request/url_request_job.h (right):

https://codereview.chromium.org/1074293003/diff/60001/net/url_request/url_req...
net/url_request/url_request_job.h:401: int64 filter_input_byte_count_;
Erm...wait...filter_input_byte_count_ and prefilter_bytes_read_ look to be
identical now.  We should get rid of one or the other.

[sclittle, 2015-04-10 20:59:13]

https://codereview.chromium.org/1074293003/diff/60001/net/url_request/url_req...
File net/url_request/url_request_job.h (right):

https://codereview.chromium.org/1074293003/diff/60001/net/url_request/url_req...
net/url_request/url_request_job.h:401: int64 filter_input_byte_count_;
On 2015/04/10 20:43:50, mmenke wrote:
> Erm...wait...filter_input_byte_count_ and prefilter_bytes_read_ look to be
> identical now.  We should get rid of one or the other.

Looks like you're right; I want to merge this CL into M43 though, so removing
that redundancy seems outside the scope of this CL.

[mmenke, 2015-04-10 21:17:18]

On 2015/04/10 20:59:13, sclittle wrote:
>
https://codereview.chromium.org/1074293003/diff/60001/net/url_request/url_req...
> File net/url_request/url_request_job.h (right):
> 
>
https://codereview.chromium.org/1074293003/diff/60001/net/url_request/url_req...
> net/url_request/url_request_job.h:401: int64 filter_input_byte_count_;
> On 2015/04/10 20:43:50, mmenke wrote:
> > Erm...wait...filter_input_byte_count_ and prefilter_bytes_read_ look to be
> > identical now.  We should get rid of one or the other.
> 
> Looks like you're right; I want to merge this CL into M43 though, so removing
> that redundancy seems outside the scope of this CL.

net LGTM

[sclittle, 2015-04-10 23:04:35]

sclittle@chromium.org changed reviewers:
+ jeremyim@chromium.org

[sclittle, 2015-04-10 23:04:36]

jeremyim: PTAL

[jeremyim, 2015-04-10 23:07:01]

On 2015/04/10 23:04:36, sclittle wrote:
> jeremyim: PTAL

data_reduction_proxy LGTM

[sclittle, 2015-04-10 23:07:20]

The CQ bit was checked by sclittle@chromium.org

[commit-bot: I haz the power, 2015-04-10 23:09:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1074293003/60001

[commit-bot: I haz the power, 2015-04-11 01:14:09]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-04-11 01:15:00]

Patchset 4 (id:??) landed as
https://crrev.com/b5c82196f9bb0b0033a29ff0b14553e4c2e7f41d
Cr-Commit-Position: refs/heads/master@{#324738}