Refactor away most of ExtensionRendererInfo

chrome/renderer/extensions/event_bindings.cc

Index: chrome/renderer/extensions/event_bindings.cc
diff --git a/chrome/renderer/extensions/event_bindings.cc b/chrome/renderer/extensions/event_bindings.cc
index 0e38dce3e8f4ae3e2148456e04d8d36997d3e278..2ebf5a0bf7e9758e38ce0bb40ff4b3c1e6ab5918 100644
--- a/chrome/renderer/extensions/event_bindings.cc
+++ b/chrome/renderer/extensions/event_bindings.cc
@@ -265,9 +265,10 @@ void EventBindings::HandleContextCreated(WebFrame* frame, bool content_script) {
   if (!ds)
     ds = frame->dataSource();
   GURL url = ds->request().url();
-  std::string extension_id = ExtensionRendererInfo::GetIdByURL(url);
+  ExtensionRendererInfo* extensions = RenderThread::current()->extensions();

[Matt Perry, 2011/01/24 20:08:34]

use the render_thread global instead, since RenderThread::current() is NULL in
unit tests. You'll have to make the extensions getter a virtual on
RenderThreadBase.

[Aaron Boodman, 2011/01/25 00:27:33]

Done.

+  std::string extension_id = extensions->GetIdByURL(url);
 
-  if (!ExtensionRendererInfo::ExtensionBindingsAllowed(url) &&
+  if (!extensions->ExtensionBindingsAllowed(url) &&
       !content_script) {
     // This context is a regular non-extension web page or an unprivileged
     // chrome app. Ignore it. We only care about content scripts and extension