Remove ExtensionURLInfo, make security decisions in render process

chrome/renderer/extensions/app_bindings.cc

Index: chrome/renderer/extensions/app_bindings.cc
diff --git a/chrome/renderer/extensions/app_bindings.cc b/chrome/renderer/extensions/app_bindings.cc
index 8f07ed2d5730fe0cbc29774cfdd64b4a0e8d15fd..268727a2a7bbdbc22224bf38fffe39d9f6ef53e8 100644
--- a/chrome/renderer/extensions/app_bindings.cc
+++ b/chrome/renderer/extensions/app_bindings.cc
@@ -119,10 +119,13 @@ void AppBindings::GetDetailsForFrame(
 
 v8::Handle<v8::Value> AppBindings::GetDetailsForFrameImpl(
     WebFrame* frame) {
+  if (frame->document().securityOrigin().isUnique())
+    return v8::Null();

[not at google - send to devlin, 2013/07/15 22:09:12]

i don't think it makes sense to check isUnique here

+
   const Extension* extension =
       dispatcher_->extensions()->GetExtensionOrAppByURL(
-          ExtensionURLInfo(frame->document().securityOrigin(),
-                           frame->document().url()));
+          frame->document().url());
+
   if (!extension)
     return v8::Null();
 
@@ -166,11 +169,11 @@ void AppBindings::GetRunningState(
 
   // The app associated with the top level frame.
   const Extension* parent_app = extensions->GetHostedAppByURL(
-      ExtensionURLInfo(parent_frame->document().url()));
+      parent_frame->document().url());
 
   // The app associated with this frame.
-  const Extension* this_app = extensions->GetHostedAppByURL(ExtensionURLInfo(
-      context()->web_frame()->document().url()));
+  const Extension* this_app = extensions->GetHostedAppByURL(
+      context()->web_frame()->document().url());
 
   if (!this_app || !parent_app) {
     args.GetReturnValue().Set(