commit fee56e9c7b39cf1d48102e2ece517110435ada71
author Hiroshige Hayashizaki <hiroshige@chromium.org> Wed Feb 20 22:27:16 2019
committer Commit Bot <commit-bot@chromium.org> Wed Feb 20 22:27:16 2019
tree 05bfca9b8c4f5ce88e54cb8f3c345b2e97989572
parent 3ffcfe44e98be7ffc84656f4bd76486223c0b085

Remove ExecutionContext dependency in WillSendRequest() for replay XHR

In off-the-main-thread worker script fetch, worker scripts are
fetched on the worker thread, but WorkerGlobalScope shouldn't be
referenced there as it is not yet fully initialized and
it is not the fetch client's settings object.

To reduce dependency to WorkerGlobalScope=ExecutionContext
during fetch, this CL removes ExecutionContext usage from
InspectorNetworkAgent::WillSendRequestInternal()
and instead supplies ExecutionContext for replayXHR from
WillLoadXHR(), and stores ExecutionContext
in XHRReplayData instead of NetworkResourceData.

The ExecutionContext Member was moved from XHRReplayData
to NetworkResourceData by
https://chromium-review.googlesource.com/c/chromium/src/+/833344/
but after
https://chromium-review.googlesource.com/c/chromium/src/+/950304/
we no longer need ExecutionContext in NetworkResourceData.

This CL also makes the reference to ExecutionContext weak.

This CL also removes unused parameters of WillLoadXHR().

Bug: 880027
Change-Id: I173a8a0e3d66ec26572e2ae738c29d62f8d76d39
Reviewed-on: https://chromium-review.googlesource.com/c/1383908
Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#633913}