Make ModulatorImpl execution-context-agnostic

Make ModulatorImpl execution-context-agnostic

This CL enables Worklet to reuse ModulatorImpl for its script loading.

This is split from https://codereview.chromium.org/2826313003/

BUG=627945
Review-Url: https://codereview.chromium.org/2865003002
Cr-Commit-Position: refs/heads/master@{#469924}
Committed: https://chromium.googlesource.com/chromium/src/+/32ad6568b0726bf88a1cc0a2cd0e9eaff84d8292

[nhiroki, 2017-05-08 04:19:27]

The CQ bit was checked by nhiroki@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-08 04:19:54]

Dry run: CQ is trying da patch.

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

[nhiroki, 2017-05-08 04:20:26]

nhiroki@chromium.org changed reviewers:
+ kouhei@chromium.org

[nhiroki, 2017-05-08 04:20:27]

PTAL

[nhiroki, 2017-05-08 04:21:09]

Description was changed from

==========
Worklet: Make ModulatorImpl execution-context-agnostic

This CL enables Worklets reuse ModulatorImpl for its script loading.

This is split from https://codereview.chromium.org/2826313003/

BUG=627945
==========

to

==========
Worklet: Make ModulatorImpl execution-context-agnostic

This CL enables Worklets to reuse ModulatorImpl for its script loading.

This is split from https://codereview.chromium.org/2826313003/

BUG=627945
==========

[nhiroki, 2017-05-08 04:21:20]

Description was changed from

==========
Worklet: Make ModulatorImpl execution-context-agnostic

This CL enables Worklets to reuse ModulatorImpl for its script loading.

This is split from https://codereview.chromium.org/2826313003/

BUG=627945
==========

to

==========
Worklet: Make ModulatorImpl execution-context-agnostic

This CL enables Worklet to reuse ModulatorImpl for its script loading.

This is split from https://codereview.chromium.org/2826313003/

BUG=627945
==========

[kouhei (in TOK), 2017-05-08 04:41:35]

I'd remove "worklet: " from CL title.
otherwise lgtm

[nhiroki, 2017-05-08 04:44:53]

Description was changed from

==========
Worklet: Make ModulatorImpl execution-context-agnostic

This CL enables Worklet to reuse ModulatorImpl for its script loading.

This is split from https://codereview.chromium.org/2826313003/

BUG=627945
==========

to

==========
Make ModulatorImpl execution-context-agnostic

This CL enables Worklet to reuse ModulatorImpl for its script loading.

This is split from https://codereview.chromium.org/2826313003/

BUG=627945
==========

[nhiroki, 2017-05-08 04:45:04]

Thank you!

On 2017/05/08 04:41:35, kouhei (on transit) wrote:
> I'd remove "worklet: " from CL title.
> otherwise lgtm

DONE.

[nhiroki, 2017-05-08 04:45:09]

The CQ bit was unchecked by nhiroki@chromium.org

[nhiroki, 2017-05-08 04:45:13]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2017-05-08 04:45:22]

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-05-08 08:39:09]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1494218713570380, "parent_rev":
"31756b973bc0d13336b1aa3746213046bc7ed69e", "commit_rev":
"32ad6568b0726bf88a1cc0a2cd0e9eaff84d8292"}

[commit-bot: I haz the power, 2017-05-08 08:39:26]

Description was changed from

==========
Make ModulatorImpl execution-context-agnostic

This CL enables Worklet to reuse ModulatorImpl for its script loading.

This is split from https://codereview.chromium.org/2826313003/

BUG=627945
==========

to

==========
Make ModulatorImpl execution-context-agnostic

This CL enables Worklet to reuse ModulatorImpl for its script loading.

This is split from https://codereview.chromium.org/2826313003/

BUG=627945

Review-Url: https://codereview.chromium.org/2865003002
Cr-Commit-Position: refs/heads/master@{#469924}
Committed:
https://chromium.googlesource.com/chromium/src/+/32ad6568b0726bf88a1cc0a2cd0e...
==========

[commit-bot: I haz the power, 2017-05-08 08:39:28]

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