[Mojo] Add support for HTML imports as well as AMD modules

[Mojo] Add support for HTML imports as well as AMD modules

This CL teaches the mojom generator how to package JavaScript generated from
mojom both as AMD modules as an HTML imports.

R=hansmuller@chromium.org

Committed: https://crrev.com/d1045a6133a372e40638457a5ab5189e139f07d5
Cr-Commit-Position: refs/heads/master@{#299310}

[hansmuller, 2014-10-10 22:12:55]

The delegating stub CL just landed - https://codereview.chromium.org/637373002/.
 The module template definitions will need to be folded into this.

[abarth-chromium, 2014-10-10 22:30:30]

On 2014/10/10 at 22:12:55, hansmuller wrote:
> The delegating stub CL just landed -
https://codereview.chromium.org/637373002/.  The module template definitions
will need to be folded into this.

I rebased this CL onto master, which now includes that CL.

[hansmuller, 2014-10-10 22:55:31]

Eventually we're going to need to a better scheme for naming modules. A
developer trying to combine a bunch of modules from different sources could run
into name collisions, and they'll probably have to get in the way of our loader.
I guess what we're implying, at least for now, is that paths like
"/mojo/public/js/bindings/codec.html" are a module's unique name. And for HTML
modules that name is just implicit, since it has to match the module's location.

I think we need a standard way to give modules a unique name. And a default way
to map module names to locations. And a straightforward way to change the
mapping.

https://codereview.chromium.org/643323002/diff/1/mojo/public/tools/bindings/g...
File mojo/public/tools/bindings/generators/js_templates/module.amd.tmpl (right):

https://codereview.chromium.org/643323002/diff/1/mojo/public/tools/bindings/g...
mojo/public/tools/bindings/generators/js_templates/module.amd.tmpl:11: ],
function(codec, validator
We haven't been completely consistent about this however in most cases the close
"]" has been left justified.

[abarth-chromium, 2014-10-10 22:57:55]

https://codereview.chromium.org/643323002/diff/1/mojo/public/tools/bindings/g...
File mojo/public/tools/bindings/generators/js_templates/module.amd.tmpl (right):

https://codereview.chromium.org/643323002/diff/1/mojo/public/tools/bindings/g...
mojo/public/tools/bindings/generators/js_templates/module.amd.tmpl:11: ],
function(codec, validator
On 2014/10/10 at 22:55:31, hansmuller wrote:
> We haven't been completely consistent about this however in most cases the
close "]" has been left justified.

Fixed.

[abarth-chromium, 2014-10-10 22:58:30]

On 2014/10/10 at 22:55:31, hansmuller wrote:
> Eventually we're going to need to a better scheme for naming modules.

Yes.  This CL doesn't solve that problem.  We'll probably need to solve it
around the time when the module system stops poking at the source tree to find
the code.  :)

[abarth-chromium, 2014-10-13 16:11:26]

@hansmuller: Ping?

[hansmuller, 2014-10-13 16:34:17]

On 2014/10/13 16:11:26, abarth wrote:
> @hansmuller: Ping?

This patch LGTM.

[abarth-chromium, 2014-10-13 16:38:19]

The CQ bit was checked by abarth@chromium.org

[commit-bot: I haz the power, 2014-10-13 16:39:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/643323002/300001

[commit-bot: I haz the power, 2014-10-13 17:02:33]

Committed patchset #3 (id:300001)

[commit-bot: I haz the power, 2014-10-13 17:03:25]

Patchset 3 (id:??) landed as
https://crrev.com/d1045a6133a372e40638457a5ab5189e139f07d5
Cr-Commit-Position: refs/heads/master@{#299310}