| commit | 9185c3fff1e6c5d66e0759a3979164a7ce9a58b7 | |
| author | mmenke <mmenke@chromium.org> | |
| Tue, 26 May 2015 16:04:05 +0000 (26 09:04 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Tue, 26 May 2015 16:04:46 +0000 (26 16:04 +0000) | ||
| tree | 4f36d4aebde73e0910f3850616ee197395169f21 | treesnapshot (tar.gz zip) |
| parent | 4406f2f401e4f4e8b84cad8c891bb179f5cc3d0b | commitdiff |
Remove UserPolicyRequestContext.
The request context mirrors the contents of the Profile's request
context, but uses "Accept-Language: *", which is not actually needed.
It also has a TODO about using the SystemURLRequestContext's proxy
service. This wouldn't actually have worked, since it shares the
Profile's context's HttpNetworkSession, which is where most of the proxy
logic lives, and the HttpNetworkSession has its own pointer to the
ProxyService it's using.
The main motivation for removing this class is that the Profile's
URLRequestContextGetter has recently been modified to safely shut down
URLFetchers that were use it during Profile shutdown, so just using
the Profile's URLRequestContextGetter directly will result in safer
shutdown.
BUG=488627
Review URL: https://codereview.chromium.org/1140263002
Cr-Commit-Position: refs/heads/master@{#331368}
The request context mirrors the contents of the Profile's request
context, but uses "Accept-Language: *", which is not actually needed.
It also has a TODO about using the SystemURLRequestContext's proxy
service. This wouldn't actually have worked, since it shares the
Profile's context's HttpNetworkSession, which is where most of the proxy
logic lives, and the HttpNetworkSession has its own pointer to the
ProxyService it's using.
The main motivation for removing this class is that the Profile's
URLRequestContextGetter has recently been modified to safely shut down
URLFetchers that were use it during Profile shutdown, so just using
the Profile's URLRequestContextGetter directly will result in safer
shutdown.
BUG=488627
Review URL: https://codereview.chromium.org/1140263002
Cr-Commit-Position: refs/heads/master@{#331368}