Set up V8 bindings for extension/app APIs when they're first used, not on

context creation. This should gives us a significant reduction in extension/app
startup time and slightly better memory usage.

It also gives us better error messages, the chance to complete the 
implementation of API features, and eventually the ability to expose select
extension APIs (e.g. extension.sendMessage) to web pages.

Resubmitting: changes made to resubmit this patch reviewed in: https://codereview.chromium.org/12378077/

BUG=163678,120070,55316,177163
TBR=ben@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=186643

Review URL: https://chromiumcodereview.appspot.com/11571014

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@187143 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 5 insertions, 3 deletions

diff --git a/chrome/renderer/extensions/extension_custom_bindings.cc b/chrome/renderer/extensions/extension_custom_bindings.cc
index 1cc5e099..2c290e52 100644
--- a/chrome/renderer/extensions/extension_custom_bindings.cc
+++ b/chrome/renderer/extensions/extension_custom_bindings.cc
@@ -26,8 +26,10 @@ namespace {
 
 }  // namespace
 
-ExtensionCustomBindings::ExtensionCustomBindings(Dispatcher* dispatcher)
-    : ChromeV8Extension(dispatcher) {
+ExtensionCustomBindings::ExtensionCustomBindings(
+    Dispatcher* dispatcher,
+    v8::Handle<v8::Context> context)
+    : ChromeV8Extension(dispatcher, context) {
   RouteStaticFunction("GetExtensionViews", &GetExtensionViews);
 }
 
@@ -72,7 +74,7 @@ v8::Handle<v8::Value> ExtensionCustomBindings::GetExtensionViews(
   ExtensionCustomBindings* v8_extension =
       GetFromArguments<ExtensionCustomBindings>(args);
   const Extension* extension =
-      v8_extension->GetExtensionForCurrentRenderView();
+      v8_extension->GetExtensionForRenderView();
   if (!extension)
     return v8::Undefined();