Fix Self-Referencing OOPIF Infinite Loop

third_party/WebKit/Source/core/html/HTMLPlugInElement.cpp

Index: third_party/WebKit/Source/core/html/HTMLPlugInElement.cpp
diff --git a/third_party/WebKit/Source/core/html/HTMLPlugInElement.cpp b/third_party/WebKit/Source/core/html/HTMLPlugInElement.cpp
index ed05618b07ae7bac1f96455b39d745f2e253b58f..62e35ac8948a52c3d2aad03ee6238deaf14ed3f5 100644
--- a/third_party/WebKit/Source/core/html/HTMLPlugInElement.cpp
+++ b/third_party/WebKit/Source/core/html/HTMLPlugInElement.cpp
@@ -469,7 +469,7 @@ bool HTMLPlugInElement::allowedToLoadFrameURL(const String& url) {
       !document().getSecurityOrigin()->canAccess(
           contentFrame()->securityContext()->getSecurityOrigin()))
     return false;
-  return document().frame()->isURLAllowed(completeURL);
+  return true;

[dcheng, 2017/02/12 08:34:59]

Nit: it's a really long return, but the usual convention is to just:

return !condition;

instead of

if (condition)
  return false;
return true;

[davidsac (gone - try alexmos), 2017/02/14 00:50:03]

Done.

 }
 
 // We don't use m_url, or m_serviceType as they may not be the final values