Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(678)

Unified Diff: mojo/public/js/router.js

Issue 2405093003: [WIP] Mojo native bindings interface.
Patch Set: fixes webui tests Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: mojo/public/js/router.js
diff --git a/mojo/public/js/router.js b/mojo/public/js/router.js
index e94c5eb50f1ef281580a3d468373ff2abbdedd35..e10a63c2aca91d7f65d0ec3ed47d66c8618d4011 100644
--- a/mojo/public/js/router.js
+++ b/mojo/public/js/router.js
@@ -5,11 +5,10 @@
define("mojo/public/js/router", [
"console",
"mojo/public/js/codec",
- "mojo/public/js/core",
"mojo/public/js/connector",
"mojo/public/js/lib/control_message_handler",
"mojo/public/js/validator",
-], function(console, codec, core, connector, controlMessageHandler, validator) {
+], function(console, codec, connector, controlMessageHandler, validator) {
var Connector = connector.Connector;
var MessageReader = codec.MessageReader;
@@ -17,8 +16,9 @@ define("mojo/public/js/router", [
var ControlMessageHandler = controlMessageHandler.ControlMessageHandler;
function Router(handle, interface_version, connectorFactory) {
- if (!core.isHandle(handle))
+ if (!(handle instanceof MojoHandle))
throw new Error("Router constructor: Not a handle");
+
if (connectorFactory === undefined)
connectorFactory = Connector;
this.connector_ = new connectorFactory(handle);
« no previous file with comments | « mojo/public/js/interface_types.js ('k') | mojo/public/tools/bindings/generators/js_templates/module.amd.tmpl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698