[Extensions] Use document url (not top url) for tab-specific permissions

extensions/renderer/extension_injection_host.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef EXTENSIONS_RENDERER_EXTENSION_INJECTION_HOST_H_
 #define EXTENSIONS_RENDERER_EXTENSION_INJECTION_HOST_H_
 
 #include "base/memory/ref_counted.h"
 #include "extensions/common/extension.h"
 #include "extensions/renderer/injection_host.h"
@@ skipping 15 matching lines @@
   static scoped_ptr<const InjectionHost> Create(const std::string& extension_id,
                                                 const ExtensionSet* extensions);
 
  private:
   // InjectionHost:
   std::string GetContentSecurityPolicy() const override;
   const GURL& url() const override;
   const std::string& name() const override;
   PermissionsData::AccessType CanExecuteOnFrame(
       const GURL& document_url,
-      const GURL& top_frame_url,
+      blink::WebFrame* web_frame,

[not at google - send to devlin, 2015/06/03 20:15:45]

Perhaps begin the trend of preferring RenderFrame over WebFrame, since you're
changing call sites anyway? If it's not too much bother.

[Devlin, 2015/06/03 20:28:47]

Sure, done.

       int tab_id,
       bool is_declarative) const override;
   bool ShouldNotifyBrowserOfInjection() const override;
 
   const Extension* extension_;
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionInjectionHost);
 };
 
 }  // namespace extesions
 
 #endif  // EXTENSIONS_RENDERER_EXTENSION_INJECTION_HOST_H_