Remove X-Chrome-Exponential-Throttling header

Remove X-Chrome-Exponential-Throttling header

This CL removes X-Chrome-Exponential-Throttling header support from net/ .

BUG=352259
Committed: https://crrev.com/041e939081191e9cdf29bbce2964c54add9de771
Cr-Commit-Position: refs/heads/master@{#330619}

[xunjieli, 2015-05-19 19:05:45]

xunjieli@chromium.org changed reviewers:
+ mmenke@chromium.org

[mmenke, 2015-05-19 19:18:37]

https://codereview.chromium.org/1148603003/diff/30001/net/url_request/url_req...
File net/url_request/url_request_throttler_entry.cc (right):

https://codereview.chromium.org/1148603003/diff/30001/net/url_request/url_req...
net/url_request/url_request_throttler_entry.cc:220: void
URLRequestThrottlerEntry::UpdateWithResponse(const std::string& host,
This method no longer needs the "host" argument.

https://codereview.chromium.org/1148603003/diff/30001/net/url_request/url_req...
net/url_request/url_request_throttler_entry.cc:226: }
nit:  Can just collapse this block to:

GetBackoffEntry()->InformOfRequest(!IsConsideredError(status_code));

Optionally could rename/modify !IsConsideredError -> IsConsideredSuccess.  Not
an error is a weird way of saying something is a success.

[xunjieli, 2015-05-19 20:06:44]

Thanks for the review! PTAL.

https://codereview.chromium.org/1148603003/diff/30001/net/url_request/url_req...
File net/url_request/url_request_throttler_entry.cc (right):

https://codereview.chromium.org/1148603003/diff/30001/net/url_request/url_req...
net/url_request/url_request_throttler_entry.cc:220: void
URLRequestThrottlerEntry::UpdateWithResponse(const std::string& host,
On 2015/05/19 19:18:37, mmenke wrote:
> This method no longer needs the "host" argument.

Done. thanks!

https://codereview.chromium.org/1148603003/diff/30001/net/url_request/url_req...
net/url_request/url_request_throttler_entry.cc:226: }
On 2015/05/19 19:18:37, mmenke wrote:
> nit:  Can just collapse this block to:
> 
> GetBackoffEntry()->InformOfRequest(!IsConsideredError(status_code));
> 
> Optionally could rename/modify !IsConsideredError -> IsConsideredSuccess.  Not
> an error is a weird way of saying something is a success.

Done.

[mmenke, 2015-05-19 20:31:24]

LGTM

https://codereview.chromium.org/1148603003/diff/50001/net/url_request/url_req...
File net/url_request/url_request_throttler_unittest.cc (right):

https://codereview.chromium.org/1148603003/diff/50001/net/url_request/url_req...
net/url_request/url_request_throttler_unittest.cc:454: // Host doesn't really
matter in this scenario so we skip it.
Obsolete comment.

https://codereview.chromium.org/1148603003/diff/50001/net/url_request/url_req...
net/url_request/url_request_throttler_unittest.cc:474: // Host doesn't really
matter in this scenario so we skip it.
Obsolete comment.

[xunjieli, 2015-05-19 20:54:16]

Thanks!

https://codereview.chromium.org/1148603003/diff/50001/net/url_request/url_req...
File net/url_request/url_request_throttler_unittest.cc (right):

https://codereview.chromium.org/1148603003/diff/50001/net/url_request/url_req...
net/url_request/url_request_throttler_unittest.cc:454: // Host doesn't really
matter in this scenario so we skip it.
On 2015/05/19 20:31:24, mmenke wrote:
> Obsolete comment.

Done.

https://codereview.chromium.org/1148603003/diff/50001/net/url_request/url_req...
net/url_request/url_request_throttler_unittest.cc:474: // Host doesn't really
matter in this scenario so we skip it.
On 2015/05/19 20:31:24, mmenke wrote:
> Obsolete comment.

Done.

[xunjieli, 2015-05-19 20:54:20]

The CQ bit was checked by xunjieli@chromium.org

[xunjieli, 2015-05-19 20:54:20]

The patchset sent to the CQ was uploaded after l-g-t-m from mmenke@chromium.org
Link to the patchset: https://codereview.chromium.org/1148603003/#ps70001
(title: "Comments")

[commit-bot: I haz the power, 2015-05-19 20:55:23]

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

[commit-bot: I haz the power, 2015-05-19 21:51:46]

Committed patchset #5 (id:70001)

[commit-bot: I haz the power, 2015-05-19 21:52:46]

Patchset 5 (id:??) landed as
https://crrev.com/041e939081191e9cdf29bbce2964c54add9de771
Cr-Commit-Position: refs/heads/master@{#330619}