Add a Javascript wrapper around ServiceRegistry and expose it to WebUI.

chrome/browser/resources/omnibox/omnibox.js

Index: chrome/browser/resources/omnibox/omnibox.js
diff --git a/chrome/browser/resources/omnibox/omnibox.js b/chrome/browser/resources/omnibox/omnibox.js
index afde3e03477cd23c3b5e64fd1845d2d4b34669dc..992ad438b981ae4fc79d6cb16ea21a42c3446f79 100644
--- a/chrome/browser/resources/omnibox/omnibox.js
+++ b/chrome/browser/resources/omnibox/omnibox.js
@@ -19,7 +19,8 @@
 define('main', [
     'mojo/public/js/bindings/connection',
     'chrome/browser/ui/webui/omnibox/omnibox.mojom',
-], function(connector, browser) {
+    'content/public/renderer/service_registry',
+], function(connector, browser, serviceRegistry) {
   'use strict';
 
   var connection;
@@ -431,8 +432,10 @@ define('main', [
     refresh();
   };
 
-  return function(handle) {
-    connection = new connector.Connection(handle, OmniboxPageImpl,
-                                          browser.OmniboxUIHandlerMojoProxy);
+  return function() {
+    connection = new connector.Connection(
+        serviceRegistry.getInterface(browser.OmniboxUIHandlerMojoProxy.NAME_),

[darin (slow to review), 2014/06/21 04:44:15]

I think it would be nice to avoid having to use the NAME_ variable here. The
trailing "_" indicates a private field.

[Sam McNally, 2014/06/24 08:45:01]

Added a TODO.

+        OmniboxPageImpl,
+        browser.OmniboxUIHandlerMojoProxy);
   };
 });