Implement externally_connectable! Web pages can now communicate directly with

chrome/renderer/extensions/dispatcher.cc

Index: chrome/renderer/extensions/dispatcher.cc
diff --git a/chrome/renderer/extensions/dispatcher.cc b/chrome/renderer/extensions/dispatcher.cc
index a55632f067440c3ac29ce89955defe6f01159700..39a7c26873787c4300c90401f5fa8928157c1868 100644
--- a/chrome/renderer/extensions/dispatcher.cc
+++ b/chrome/renderer/extensions/dispatcher.cc
@@ -995,9 +995,9 @@ void Dispatcher::DidCreateScriptContext(
   switch (context_type) {
     case Feature::UNSPECIFIED_CONTEXT:
     case Feature::WEB_PAGE_CONTEXT:
-      // TODO(kalman): see comment below about ExtensionAPI.
-      InstallBindings(module_system, v8_context, "app");
-      InstallBindings(module_system, v8_context, "webstore");
+      RegisterBinding("app", context);

[Jeffrey Yasskin, 2013/06/08 00:28:09]

This change is because the lazy binding loader works now?

[not at google - send to devlin, 2013/06/08 02:02:33]

"runtime" needs to be used via RegisterBinding, InstallBindings would be wrong
because (as I think you're saying) it would break laziness.

I changed the others to RegisterBinding to make it consistent, but the code path
is identical for these (until we finally convert app and webstore to use
bindings system).

+      RegisterBinding("runtime", context);  // for connect() and sendMessage()
+      RegisterBinding("webstore", context);
       break;
     case Feature::BLESSED_EXTENSION_CONTEXT:
     case Feature::UNBLESSED_EXTENSION_CONTEXT: