[ES6 modules] Introduce ModulatorImpl

[ES6 modules] Introduce ModulatorImpl

This CL introduces ModulatorImpl, which is an implementation of
"settings object" spec concept + serves as a backplane for tieing
all ES6 module algorithm components together.

See below diagram for where it stands in the stack:
https://docs.google.com/document/d/1vjiWxwhg9D0GNNOYgw3AxMG0iKOC9I3jlID4GTgZsac/edit#heading=h.47x0qrpzjbj4

BUG=594639
Review-Url: https://codereview.chromium.org/2826803004
Cr-Commit-Position: refs/heads/master@{#465506}
Committed: https://chromium.googlesource.com/chromium/src/+/521525f8507b6525eceed9724bd9a5f66f078eda

[kouhei (in TOK), 2017-04-19 01:29:37]

kouhei@chromium.org changed reviewers:
+ hiroshige@chromium.org, japhet@chromium.org, yhirano@chromium.org

[kouhei (in TOK), 2017-04-19 01:29:41]

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

[commit-bot: I haz the power, 2017-04-19 01:30:01]

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

[nhiroki, 2017-04-19 03:15:46]

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

[nhiroki, 2017-04-19 03:15:47]

lgtm

https://codereview.chromium.org/2826803004/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/dom/ModulatorImpl.cpp (right):

https://codereview.chromium.org/2826803004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/ModulatorImpl.cpp:30: ExecutionContext*
execution_context,
We might not have to pass |execution_context| here because we can get it
by|ExecutionContext::From(script_state)|.

https://codereview.chromium.org/2826803004/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/dom/ModulatorImpl.h (right):

https://codereview.chromium.org/2826803004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/ModulatorImpl.h:65: private:
We are already in a private section (see line 37)

[kouhei (in TOK), 2017-04-19 03:38:53]

https://codereview.chromium.org/2826803004/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/dom/ModulatorImpl.cpp (right):

https://codereview.chromium.org/2826803004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/ModulatorImpl.cpp:30: ExecutionContext*
execution_context,
On 2017/04/19 03:15:47, nhiroki wrote:
> We might not have to pass |execution_context| here because we can get it
> by|ExecutionContext::From(script_state)|.

Done.

https://codereview.chromium.org/2826803004/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/dom/ModulatorImpl.h (right):

https://codereview.chromium.org/2826803004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/ModulatorImpl.h:65: private:
On 2017/04/19 03:15:47, nhiroki wrote:
> We are already in a private section (see line 37)

Done.

[kouhei (in TOK), 2017-04-19 03:44:42]

The CQ bit was checked by kouhei@chromium.org

[kouhei (in TOK), 2017-04-19 03:44:42]

The patchset sent to the CQ was uploaded after l-g-t-m from nhiroki@chromium.org
Link to the patchset: https://codereview.chromium.org/2826803004/#ps60001
(title: "no double private")

[commit-bot: I haz the power, 2017-04-19 03:44:59]

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-19 05:57:18]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1492573482110230,
"parent_rev": "20784f77e32bcfea4e305ec28e532ed3e43c2116", "commit_rev":
"521525f8507b6525eceed9724bd9a5f66f078eda"}

[commit-bot: I haz the power, 2017-04-19 05:57:30]

Description was changed from

==========
[ES6 modules] Introduce ModulatorImpl

This CL introduces ModulatorImpl, which is an implementation of
"settings object" spec concept + serves as a backplane for tieing
all ES6 module algorithm components together.

See below diagram for where it stands in the stack:
https://docs.google.com/document/d/1vjiWxwhg9D0GNNOYgw3AxMG0iKOC9I3jlID4GTgZs...

BUG=594639
==========

to

==========
[ES6 modules] Introduce ModulatorImpl

This CL introduces ModulatorImpl, which is an implementation of
"settings object" spec concept + serves as a backplane for tieing
all ES6 module algorithm components together.

See below diagram for where it stands in the stack:
https://docs.google.com/document/d/1vjiWxwhg9D0GNNOYgw3AxMG0iKOC9I3jlID4GTgZs...

BUG=594639

Review-Url: https://codereview.chromium.org/2826803004
Cr-Commit-Position: refs/heads/master@{#465506}
Committed:
https://chromium.googlesource.com/chromium/src/+/521525f8507b6525eceed9724bd9...
==========

[commit-bot: I haz the power, 2017-04-19 05:57:31]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/521525f8507b6525eceed9724bd9...

[hiroshige, 2017-04-19 21:26:26]

On 2017/04/19 05:57:31, commit-bot: I haz the power wrote:
> Committed patchset #4 (id:60001) as
>
https://chromium.googlesource.com/chromium/src/+/521525f8507b6525eceed9724bd9...

lgtm