Fixed a header name error in UMA statistics that record what servers accept range requests.

Fixed a header name error in UMA statistics that record what servers accept range requests.

Change the names of the statistics, so that there is no pollution of the data.

BUG=None
TEST=None


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=112302

[ahendrickson, 2011-11-30 16:31:33]

PTAL.

[Randy Smith (Not in Mondays), 2011-11-30 18:49:27]

LGTM, but I don't think it's safe to commit without Chris' ok; he knows the
headers much better than I.

[cbentzel, 2011-11-30 18:51:01]

LGTM

Have you tried this locally?

http://codereview.chromium.org/8746020/diff/1/content/browser/download/downlo...
File content/browser/download/download_resource_handler.cc (right):

http://codereview.chromium.org/8746020/diff/1/content/browser/download/downlo...
content/browser/download/download_resource_handler.cc:133: &accepts_ranges_)) {
Nit: accept_ranges_ for the member name?

[ahendrickson, 2011-11-30 19:33:51]

On 2011/11/30 18:51:01, cbentzel wrote:
> LGTM
> 
> Have you tried this locally?
> 

Yes.  I used both a local download server, and an external ISO download site.
I verified that 'Accept-Ranges' was used in some cases and not in others, and
that the uses matched the statistics that were gathered.

http://codereview.chromium.org/8746020/diff/1/content/browser/download/downlo...
File content/browser/download/download_resource_handler.cc (right):

http://codereview.chromium.org/8746020/diff/1/content/browser/download/downlo...
content/browser/download/download_resource_handler.cc:133: &accepts_ranges_)) {
On 2011/11/30 18:51:01, cbentzel wrote:
> Nit: accept_ranges_ for the member name?

Changed.

[commit-bot: I haz the power, 2011-11-30 20:07:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ahendrickson@chromium.org/8746020/1005

[commit-bot: I haz the power, 2011-11-30 21:59:08]

Change committed as 112302