Reland of Move ScriptState::GetExecutionContext (Part 5)

Reland of Move ScriptState::GetExecutionContext (Part 5) (patchset #2 id:300001 of https://codereview.chromium.org/2821443002/ )

Reason for revert:
Broke Build

Original issue's description:
> 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
> 

TBR=jbroman@chromium.org,haraken@chromium.org,adithyas@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=711250
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review-Url: https://codereview.chromium.org/2815313002
Cr-Commit-Position: refs/heads/master@{#464376}
Committed: https://chromium.googlesource.com/chromium/src/+/e87bbc88306d3bbac6249887d846077c8d0f923b

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

The CQ bit was checked by jdoerrie@chromium.org

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

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

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

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

[jdoerrie, 2017-04-13 11:46:14]

Description was changed from

==========
Reland of Move ScriptState::GetExecutionContext (Part 5) (patchset #2 id:300001
of https://codereview.chromium.org/2821443002/ )

Reason for revert:
Broke Build

Original issue's description:
> 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
> 

TBR=jbroman@chromium.org,haraken@chromium.org,adithyas@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=711250
==========

to

==========
Reland of Move ScriptState::GetExecutionContext (Part 5) (patchset #2 id:300001
of https://codereview.chromium.org/2821443002/ )

Reason for revert:
Broke Build

Original issue's description:
> 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
> 

TBR=jbroman@chromium.org,haraken@chromium.org,adithyas@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=711250
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

[jdoerrie, 2017-04-13 11:46:25]

The CQ bit was unchecked by jdoerrie@chromium.org

[jdoerrie, 2017-04-13 11:46:31]

The CQ bit was checked by jdoerrie@chromium.org

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

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:48:26]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492083991539180, "parent_rev":
"a98c2132ee5335e39d6d83f70ed37678237637e9", "commit_rev":
"e87bbc88306d3bbac6249887d846077c8d0f923b"}

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

Description was changed from

==========
Reland of Move ScriptState::GetExecutionContext (Part 5) (patchset #2 id:300001
of https://codereview.chromium.org/2821443002/ )

Reason for revert:
Broke Build

Original issue's description:
> 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
> 

TBR=jbroman@chromium.org,haraken@chromium.org,adithyas@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=711250
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

to

==========
Reland of Move ScriptState::GetExecutionContext (Part 5) (patchset #2 id:300001
of https://codereview.chromium.org/2821443002/ )

Reason for revert:
Broke Build

Original issue's description:
> 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
> 

TBR=jbroman@chromium.org,haraken@chromium.org,adithyas@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=711250
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review-Url: https://codereview.chromium.org/2815313002
Cr-Commit-Position: refs/heads/master@{#464376}
Committed:
https://chromium.googlesource.com/chromium/src/+/e87bbc88306d3bbac6249887d846...
==========

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

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/e87bbc88306d3bbac6249887d846...

[haraken, 2017-04-13 12:35:09]

LGTM. Yeah, this CL shouldn't be related.