Move blocking of top-level navigations to nested URLs with extension origins from non-extension pro…

content/public/browser/navigation_handle.h

Index: content/public/browser/navigation_handle.h
diff --git a/content/public/browser/navigation_handle.h b/content/public/browser/navigation_handle.h
index 8770777171da92bf15cb92d1d4a0117004c22115..78e2a2a3e38ddee55009a12fc41adc7907cf1ca8 100644
--- a/content/public/browser/navigation_handle.h
+++ b/content/public/browser/navigation_handle.h
@@ -23,6 +23,7 @@ namespace content {
 class NavigationData;
 class NavigationThrottle;
 class RenderFrameHost;
+class SiteInstance;
 class WebContents;
 
 // A NavigationHandle tracks information related to a single navigation.
@@ -48,6 +49,9 @@ class CONTENT_EXPORT NavigationHandle {
   // virtual URL is prefixed with "view-source:".
   virtual const GURL& GetURL() = 0;
 
+  // Returns the SiteInstance that created the request.

[alexmos, 2016/10/12 23:13:28]

To eliminate any possible confusion with the initiator, would it be worth adding
a concrete example to the comment, along the lines of: If a frame in
SiteInstance A navigates a frame in SiteInstance B to a URL in SiteInstance C,
then this returns B.

[jam, 2016/10/13 00:14:46]

Done.

+  virtual SiteInstance* GetCreatorSiteInstance() = 0;

[alexmos, 2016/10/12 23:13:28]

We had some more discussions about the name, and we think it might be better to
name this GetStartingSiteInstance().  WDYT?

[jam, 2016/10/13 00:14:46]

Done.

+
   // Whether the navigation is taking place in the main frame or in a subframe.
   // This remains constant over the navigation lifetime.
   virtual bool IsInMainFrame() = 0;