diff options
author | horo <horo@chromium.org> | 2014-10-31 19:08:28 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-11-01 02:08:43 +0000 |
commit | e09b6c8616f4a8e45e91a171eedddefde9522ac2 (patch) | |
tree | a2bf42602693c66c8d5089e68aca52f7ae788d9a /printing/print_settings_conversion.h | |
parent | 2178ec8cf854f0996a2d66649f52841e1ed18795 (diff) | |
download | chromium_src-e09b6c8616f4a8e45e91a171eedddefde9522ac2.zip chromium_src-e09b6c8616f4a8e45e91a171eedddefde9522ac2.tar.gz chromium_src-e09b6c8616f4a8e45e91a171eedddefde9522ac2.tar.bz2 |
[ServiceWorker] Don't send the UMA related headers to the ServiceWorker.
Chrome add "X-Chrome-UMA-Enabled" (and sometimes "X-Client-Data") headers while accessing Google servers if UMA is enabled.
This header is added before sending the request to the ServiceWorker.
So the request which is passed to the ServiceWorker contains this header.
If event.request.mode is 'cors' and event.request.headers has "x-chrome-uma-enabled", fetch(event.request) send a CORS preflight request.
This preflight request contains "Access-Control-Request-Headers: x-chrome-uma-enabled" header.
But the response from the Google server doesn't contain "Access-Control-Allow-Headers: x-chrome-uma-enabled" header.
So this fetch fails.
To avoid this problem this patch removes these headers before sending the request to the ServiceWorker.
This patch introduces ServiceWorkerContext::AddExcludedHeadersForFetchEvent() method and changes DevToolsNetworkTransactionFactory to call this method.
BUG=425649
Review URL: https://codereview.chromium.org/666973003
Cr-Commit-Position: refs/heads/master@{#302379}
Diffstat (limited to 'printing/print_settings_conversion.h')
0 files changed, 0 insertions, 0 deletions