|
Mojo JS bindings: change module loading solution.
This change takes place on the mojo/public/js/new_bindings copy so it doesn't
affect existing users.
- This change gets rid of AMD module loading. Now the bindings API is defined in
the "mojo" namespace. At build time, all bindings files are combined
into a single file "mojo_bindings.js". Users should use <script> tag to include
this file (as well as generated mojom.js files).
- Generated mojom.js files export their definitions under the same namespace as
the "module" statement in the corresponding mojom files.
- This change also adds a "use_new_js_bindings" option to the generator. It
duplicates the control message mojom files in order to generate two
different flavors of JS bindings.
- The new bindings use the Mojo system API defined by Web IDL.
BUG= 699569
Review-Url: https://codereview.chromium.org/2759563004
Cr-Commit-Position: refs/heads/master@{#459654}
Committed: https://chromium.googlesource.com/chromium/src/+/e6a5534bb3fe61b5224f1a22e43ba957190ad5d0
Total comments: 14
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+556 lines, -301 lines) |
Patch |
|
M |
BUILD.gn
|
View
|
1
2
3
4
5
6
7
|
2 chunks |
+17 lines, -0 lines |
0 comments
|
Download
|
|
M |
mojo/public/interfaces/bindings/BUILD.gn
|
View
|
|
1 chunk |
+12 lines, -0 lines |
0 comments
|
Download
|
|
A |
mojo/public/interfaces/bindings/new_bindings/OWNERS
|
View
|
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
|
A |
mojo/public/interfaces/bindings/new_bindings/interface_control_messages.mojom
|
View
|
1
2
|
1 chunk |
+67 lines, -0 lines |
0 comments
|
Download
|
|
A |
mojo/public/interfaces/bindings/new_bindings/pipe_control_messages.mojom
|
View
|
1
2
|
1 chunk |
+46 lines, -0 lines |
0 comments
|
Download
|
|
M |
mojo/public/interfaces/bindings/tests/BUILD.gn
|
View
|
1
2
3
4
5
6
7
|
2 chunks |
+10 lines, -0 lines |
0 comments
|
Download
|
|
A |
mojo/public/interfaces/bindings/tests/echo.mojom
|
View
|
1
2
3
4
|
1 chunk |
+12 lines, -0 lines |
0 comments
|
Download
|
|
A |
mojo/public/interfaces/bindings/tests/echo_import.mojom
|
View
|
1
2
3
4
|
1 chunk |
+10 lines, -0 lines |
0 comments
|
Download
|
|
M |
mojo/public/js/BUILD.gn
|
View
|
1
2
3
4
5
6
7
|
1 chunk |
+38 lines, -0 lines |
0 comments
|
Download
|
|
A |
mojo/public/js/new_bindings/base.js
|
View
|
1
2
3
4
5
6
7
|
1 chunk |
+33 lines, -0 lines |
0 comments
|
Download
|
|
M |
mojo/public/js/new_bindings/bindings.js
|
View
|
1
2
3
4
5
6
7
|
8 chunks |
+21 lines, -31 lines |
0 comments
|
Download
|
|
M |
mojo/public/js/new_bindings/buffer.js
|
View
|
1
2
3
4
5
6
7
|
2 chunks |
+4 lines, -5 lines |
0 comments
|
Download
|
|
M |
mojo/public/js/new_bindings/codec.js
|
View
|
1
2
3
4
5
6
7
|
10 chunks |
+51 lines, -56 lines |
0 comments
|
Download
|
|
M |
mojo/public/js/new_bindings/connector.js
|
View
|
1
2
3
4
5
6
7
|
4 chunks |
+20 lines, -30 lines |
0 comments
|
Download
|
|
M |
mojo/public/js/new_bindings/interface_types.js
|
View
|
1
2
3
4
5
6
7
|
3 chunks |
+8 lines, -14 lines |
0 comments
|
Download
|
|
M |
mojo/public/js/new_bindings/lib/control_message_handler.js
|
View
|
1
2
3
4
5
6
7
|
2 chunks |
+33 lines, -38 lines |
0 comments
|
Download
|
|
M |
mojo/public/js/new_bindings/lib/control_message_proxy.js
|
View
|
1
2
3
4
5
6
7
|
5 chunks |
+29 lines, -34 lines |
0 comments
|
Download
|
|
M |
mojo/public/js/new_bindings/router.js
|
View
|
1
2
3
4
5
6
7
|
8 chunks |
+13 lines, -26 lines |
0 comments
|
Download
|
|
M |
mojo/public/js/new_bindings/unicode.js
|
View
|
1
2
3
4
5
6
7
|
2 chunks |
+7 lines, -7 lines |
0 comments
|
Download
|
|
M |
mojo/public/js/new_bindings/validator.js
|
View
|
1
2
3
4
5
6
7
|
21 chunks |
+44 lines, -45 lines |
0 comments
|
Download
|
|
M |
mojo/public/tools/bindings/generators/js_templates/module.amd.tmpl
|
View
|
1
2
3
4
5
6
7
|
2 chunks |
+26 lines, -5 lines |
0 comments
|
Download
|
|
M |
mojo/public/tools/bindings/generators/js_templates/module_definition.tmpl
|
View
|
1
2
3
4
5
6
7
|
3 chunks |
+7 lines, -8 lines |
0 comments
|
Download
|
|
M |
mojo/public/tools/bindings/generators/mojom_js_generator.py
|
View
|
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
|
M |
mojo/public/tools/bindings/mojom.gni
|
View
|
|
2 chunks |
+12 lines, -0 lines |
0 comments
|
Download
|
|
M |
mojo/public/tools/bindings/mojom_bindings_generator.py
|
View
|
|
2 chunks |
+5 lines, -0 lines |
0 comments
|
Download
|
|
M |
mojo/public/tools/bindings/pylib/mojom/generate/generator.py
|
View
|
|
2 chunks |
+3 lines, -2 lines |
0 comments
|
Download
|
|
A |
third_party/WebKit/LayoutTests/mojo/module-loading.html
|
View
|
1
2
3
4
5
6
7
|
1 chunk |
+25 lines, -0 lines |
0 comments
|
Download
|
Dependent Patchsets:
Total messages: 68 (46 generated)
|