Fix Self-Referencing OOPIF Infinite Loop

content/browser/frame_host/navigation_handle_impl.h

Index: content/browser/frame_host/navigation_handle_impl.h
diff --git a/content/browser/frame_host/navigation_handle_impl.h b/content/browser/frame_host/navigation_handle_impl.h
index 20a159af380b79c8a118d88c4fd31c9f2f599a1d..b2f9b27c42906cedfda49896f2b135b041359784 100644
--- a/content/browser/frame_host/navigation_handle_impl.h
+++ b/content/browser/frame_host/navigation_handle_impl.h
@@ -223,6 +223,10 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
       RequestContextType request_context_type,
       const ThrottleChecksFinishedCallback& callback);
 
+  // Prevents navigation to a page that is already referenced more than once in
+  // its ancestors

[alexmos, 2016/12/28 00:25:59]

nit: end with period.

[alexmos, 2016/12/28 00:25:59]

This comment isn't entirely accurate, as it only tells whether the navigation
URL is referenced in ancestors, but it doesn't actually prevent the navigation. 
Perhaps just tweak the wording to say that this is a helper function that's used
by WillStartRequest and WillRedirectRequest to prevent the navigation?

[davidsac (gone - try alexmos), 2017/01/06 00:44:57]

Done.

[davidsac (gone - try alexmos), 2017/01/06 00:44:57]

Done.

+  bool isURLBlocked();

[alexmos, 2016/12/28 00:25:59]

Let's use a name that conveys what specific check is done here.  For example,
IsSelfReferentialURL.

[alexmos, 2016/12/28 00:25:59]

Let's not insert this in between WillStartRequest and WillRedirectRequest, which
make sense to keep next to each other.  Also, can this be private, as it's not
really intended to be used outside of here?  So for example this could go just
after RegisterNavigationThrottles(), here and in in .cc to match header order.

[davidsac (gone - try alexmos), 2017/01/06 00:44:57]

Done.

[davidsac (gone - try alexmos), 2017/01/06 00:44:57]

Done.

+
   // Called when the URLRequest will be redirected in the network stack.
   // |callback| will be called when all throttles check have completed. This
   // will allow the caller to cancel the navigation or let it proceed.