Add runtime flag for <script type="module">

Add runtime flag for <script type="module">

Also exercise it in a very basic way in ScriptLoader (at the moment,
<script type="module"> is treated the same as <script>).

See intent-to-implement:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/uba6pMr-jec/tXdg6YYPBAAJ

BUG=594639
Committed: https://crrev.com/7e4a661dd6f79f847d878be5651d8e7bd662a7f4
Cr-Commit-Position: refs/heads/master@{#382083}

[adamk, 2016-03-17 20:46:41]

adamk@chromium.org changed reviewers:
+ haraken@chromium.org

[adamk, 2016-03-17 20:46:41]

Greetings,

I'm sending this CL out in such a basic state for two reasons:

1) To add a runtime flag for this feature and
2) To find the right reviewer for future patches in this series. This will be a
relatively large feature with changes in various ScriptLoader code; who's the
right person to handle such reviews?

[haraken, 2016-03-17 23:23:02]

LGTM

On 2016/03/17 20:46:41, adamk wrote:
> Greetings,
> 
> I'm sending this CL out in such a basic state for two reasons:
> 
> 1) To add a runtime flag for this feature and
> 2) To find the right reviewer for future patches in this series. This will be
a
> relatively large feature with changes in various ScriptLoader code; who's the
> right person to handle such reviews?

Sigbjorn and me?

[haraken, 2016-03-17 23:23:22]

haraken@chromium.org changed reviewers:
+ sigbjornf@opera.se

[haraken, 2016-03-17 23:23:22]

+Sigbjorn

[adamk, 2016-03-17 23:26:20]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2016-03-17 23:27:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1807293004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1807293004/1

[commit-bot: I haz the power, 2016-03-18 02:39:52]

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

[commit-bot: I haz the power, 2016-03-18 02:39:53]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[adamk, 2016-03-18 17:19:17]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2016-03-18 17:19:42]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1807293004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1807293004/1

[commit-bot: I haz the power, 2016-03-18 20:55:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-18 20:57:27]

Description was changed from

==========
Add runtime flag for <script type="module">

Also exercise it in a very basic way in ScriptLoader (at the moment,
<script type="module"> is treated the same as <script>).

See intent-to-implement:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/uba6pMr-jec/tXdg6YYP...

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

to

==========
Add runtime flag for <script type="module">

Also exercise it in a very basic way in ScriptLoader (at the moment,
<script type="module"> is treated the same as <script>).

See intent-to-implement:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/uba6pMr-jec/tXdg6YYP...

BUG=594639

Committed: https://crrev.com/7e4a661dd6f79f847d878be5651d8e7bd662a7f4
Cr-Commit-Position: refs/heads/master@{#382083}
==========

[commit-bot: I haz the power, 2016-03-18 20:57:28]

Patchset 1 (id:??) landed as
https://crrev.com/7e4a661dd6f79f847d878be5651d8e7bd662a7f4
Cr-Commit-Position: refs/heads/master@{#382083}