Revert of Move ScriptState::GetExecutionContext (Part 5)

Revert of Move ScriptState::GetExecutionContext (Part 5) (patchset #3 id:40001 of https://codereview.chromium.org/2816543002/ )

Reason for revert:
Suspected culprit of webkit_tests failing on chromium.webkit/WebKit Linux Trusty MSAN.

BUG=711250
Original issue's description:
> Move ScriptState::GetExecutionContext (Part 5)
> 
> This CL replaces all uses of ScriptState::GetExecutionContext in Source/modules with ExecutionContext::From.
> 
> BUG=682322
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
> 
> Review-Url: https://codereview.chromium.org/2816543002
> Cr-Commit-Position: refs/heads/master@{#464056}
> Committed: https://chromium.googlesource.com/chromium/src/+/d2d9a34e2fd74fd45026a8fed6f4715de18df747

TBR=jbroman@chromium.org,haraken@chromium.org,adithyas@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=682322
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

[jdoerrie, 2017-04-13 11:10:36]

The CQ bit was checked by jdoerrie@chromium.org

[jdoerrie, 2017-04-13 11:10:37]

Created Revert of Move ScriptState::GetExecutionContext (Part 5)

[commit-bot: I haz the power, 2017-04-13 11:10:51]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-13 11:12:00]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-13 11:12:02]

Failed to apply patch for
third_party/WebKit/Source/modules/mediasession/MediaMetadata.cpp:
While running git apply --index -3 -p1;
  error: patch failed:
third_party/WebKit/Source/modules/mediasession/MediaMetadata.cpp:7
  Falling back to three-way merge...
  Applied patch to
'third_party/WebKit/Source/modules/mediasession/MediaMetadata.cpp' with
conflicts.
  U third_party/WebKit/Source/modules/mediasession/MediaMetadata.cpp

Patch:       third_party/WebKit/Source/modules/mediasession/MediaMetadata.cpp
Index: third_party/WebKit/Source/modules/mediasession/MediaMetadata.cpp
diff --git a/third_party/WebKit/Source/modules/mediasession/MediaMetadata.cpp
b/third_party/WebKit/Source/modules/mediasession/MediaMetadata.cpp
index
ca8750ad8d4eb7760f7e18832325c9cdfd561ab3..0b4f1514d062de61c067e6f875eea0625541d2bf
100644
--- a/third_party/WebKit/Source/modules/mediasession/MediaMetadata.cpp
+++ b/third_party/WebKit/Source/modules/mediasession/MediaMetadata.cpp
@@ -7,7 +7,6 @@
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/ScriptState.h"
 #include "bindings/core/v8/ToV8.h"
-#include "core/dom/ExecutionContext.h"
 #include "core/dom/TaskRunnerHelper.h"
 #include "modules/mediasession/MediaImage.h"
 #include "modules/mediasession/MediaMetadataInit.h"
@@ -107,7 +106,7 @@
   HeapVector<MediaImage> processed_artwork(artwork);
 
   for (MediaImage& image : processed_artwork) {
-    KURL url = ExecutionContext::From(script_state)->CompleteURL(image.src());
+    KURL url = script_state->GetExecutionContext()->CompleteURL(image.src());
     if (!url.IsValid()) {
       exception_state.ThrowTypeError("'" + image.src() +
                                      "' can't be resolved to a valid URL.");

[jdoerrie, 2017-04-13 11:28:02]

Description was changed from

==========
Revert of Move ScriptState::GetExecutionContext (Part 5) (patchset #3 id:40001
of https://codereview.chromium.org/2816543002/ )

Reason for revert:
Suspected culprit of webkit_tests failing on chromium.webkit/WebKit Linux Trusty
MSAN.

BUG=711250

Original issue's description:
> Move ScriptState::GetExecutionContext (Part 5)
> 
> This CL replaces all uses of ScriptState::GetExecutionContext in
Source/modules with ExecutionContext::From.
> 
> BUG=682322
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
> 
> Review-Url: https://codereview.chromium.org/2816543002
> Cr-Commit-Position: refs/heads/master@{#464056}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d2d9a34e2fd74fd45026a8fed6f4...

TBR=jbroman@chromium.org,haraken@chromium.org,adithyas@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=682322
==========

to

==========
Revert of Move ScriptState::GetExecutionContext (Part 5) (patchset #3 id:40001
of https://codereview.chromium.org/2816543002/ )

Reason for revert:
Suspected culprit of webkit_tests failing on chromium.webkit/WebKit Linux Trusty
MSAN.

BUG=711250

Original issue's description:
> Move ScriptState::GetExecutionContext (Part 5)
> 
> This CL replaces all uses of ScriptState::GetExecutionContext in
Source/modules with ExecutionContext::From.
> 
> BUG=682322
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
> 
> Review-Url: https://codereview.chromium.org/2816543002
> Cr-Commit-Position: refs/heads/master@{#464056}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d2d9a34e2fd74fd45026a8fed6f4...

TBR=jbroman@chromium.org,haraken@chromium.org,adithyas@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=682322
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[jdoerrie, 2017-04-13 11:29:22]

The CQ bit was checked by jdoerrie@chromium.org

[commit-bot: I haz the power, 2017-04-13 11:29:33]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-13 11:31:24]

CQ is committing da patch.

Bot data: {"patchset_id": 300001, "attempt_start_ts": 1492082962117660,
"parent_rev": "d3916f29a4b748b20d9de39d8655dc2c8c29e190", "commit_rev":
"322a025409d5e5897720f0a818426a3e37cc4a96"}

[commit-bot: I haz the power, 2017-04-13 11:36:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-13 11:36:38]

Prior attempt to commit was detected, but we were not able to check whether the
issue was successfully committed. Please check Git history manually and re-check
CQ or close this issue as needed.

[jdoerrie, 2017-04-13 11:40:10]

Description was changed from

==========
Revert of Move ScriptState::GetExecutionContext (Part 5) (patchset #3 id:40001
of https://codereview.chromium.org/2816543002/ )

Reason for revert:
Suspected culprit of webkit_tests failing on chromium.webkit/WebKit Linux Trusty
MSAN.

BUG=711250

Original issue's description:
> Move ScriptState::GetExecutionContext (Part 5)
> 
> This CL replaces all uses of ScriptState::GetExecutionContext in
Source/modules with ExecutionContext::From.
> 
> BUG=682322
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
> 
> Review-Url: https://codereview.chromium.org/2816543002
> Cr-Commit-Position: refs/heads/master@{#464056}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d2d9a34e2fd74fd45026a8fed6f4...

TBR=jbroman@chromium.org,haraken@chromium.org,adithyas@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=682322
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Revert of Move ScriptState::GetExecutionContext (Part 5) (patchset #3 id:40001
of https://codereview.chromium.org/2816543002/ )

Reason for revert:
Suspected culprit of webkit_tests failing on chromium.webkit/WebKit Linux Trusty
MSAN.

BUG=711250

Original issue's description:
> Move ScriptState::GetExecutionContext (Part 5)
> 
> This CL replaces all uses of ScriptState::GetExecutionContext in
Source/modules with ExecutionContext::From.
> 
> BUG=682322
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
> 
> Review-Url: https://codereview.chromium.org/2816543002
> Cr-Commit-Position: refs/heads/master@{#464056}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d2d9a34e2fd74fd45026a8fed6f4...

TBR=jbroman@chromium.org,haraken@chromium.org,adithyas@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=682322
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[jdoerrie, 2017-04-13 11:40:47]

A revert of this CL (patchset #2 id:300001) has been created in
https://codereview.chromium.org/2815313002/ by jdoerrie@chromium.org.

The reason for reverting is: Broke Build.

[findit-for-me, 2017-04-13 11:57:31]

Findit(https://goo.gl/kROfz5) identified this CL at revision 464373 as the
culprit for
failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3Itb...