self.location should be the final redirected url rather than the original request url |
||||||
Issue descriptionReported by Ben Kelly: when you do new Worker(url) and url gets redirected, inside the Worker self.location should be the final redirected url rather than the original request url. See https://redirect-worker-test.glitch.me/
,
Jun 8 2018
,
Jun 19 2018
I'm now working on this for module workers. WIP CL: https://chromium-review.googlesource.com/c/chromium/src/+/1103433
,
Jun 22 2018
Let me hand this over to hiroshige@. Probably it's easier to fix classic workers first because there're several entangled issues in module workers (e.g., SettingsObject separation). Relevant issue 850662
,
Jul 8
,
Jul 27
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3e1228889eb3c291c86ba67afe791e2ec24f7f6d commit 3e1228889eb3c291c86ba67afe791e2ec24f7f6d Author: Hiroshige Hayashizaki <hiroshige@chromium.org> Date: Fri Jul 27 19:59:17 2018 Use response URL for classic dedicated worker global scope's URL Bug: 861564, 782981, 850662 Change-Id: Ib5c1dc874972aadd3223bd4851f0b5aeb695a727 Reviewed-on: https://chromium-review.googlesource.com/1128540 Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org> Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org> Reviewed-by: Kouhei Ueno <kouhei@chromium.org> Cr-Commit-Position: refs/heads/master@{#578760} [modify] https://crrev.com/3e1228889eb3c291c86ba67afe791e2ec24f7f6d/chrome/test/data/extensions/api_test/worker/background.js [modify] https://crrev.com/3e1228889eb3c291c86ba67afe791e2ec24f7f6d/third_party/WebKit/LayoutTests/TestExpectations [modify] https://crrev.com/3e1228889eb3c291c86ba67afe791e2ec24f7f6d/third_party/blink/renderer/core/workers/dedicated_worker.cc [modify] https://crrev.com/3e1228889eb3c291c86ba67afe791e2ec24f7f6d/third_party/blink/renderer/core/workers/dedicated_worker.h [modify] https://crrev.com/3e1228889eb3c291c86ba67afe791e2ec24f7f6d/third_party/blink/renderer/core/workers/global_scope_creation_params.h
,
Nov 15
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by falken@chromium.org
, Apr 5 2018