Refactor mixed content checks against the top frame into MixedContentChecker.

Source/modules/websockets/MainThreadWebSocketChannel.cpp

Index: Source/modules/websockets/MainThreadWebSocketChannel.cpp
diff --git a/Source/modules/websockets/MainThreadWebSocketChannel.cpp b/Source/modules/websockets/MainThreadWebSocketChannel.cpp
index ed7adbdfb02a9d6de2518971728ff3db4aca3c4c..2d2b76a658776388112c4ae178c44a02964160df 100644
--- a/Source/modules/websockets/MainThreadWebSocketChannel.cpp
+++ b/Source/modules/websockets/MainThreadWebSocketChannel.cpp
@@ -97,9 +97,6 @@ bool MainThreadWebSocketChannel::connect(const KURL& url, const String& protocol
 
     if (m_document->frame() && !m_document->frame()->loader().mixedContentChecker()->canConnectInsecureWebSocket(m_document->securityOrigin(), url))
         return false;
-    Frame* top = m_document->frame()->tree().top();
-    if (top != m_document->frame() && !toLocalFrame(top)->loader().mixedContentChecker()->canConnectInsecureWebSocket(toLocalFrame(top)->document()->securityOrigin(), url))
-        return false;
     if (MixedContentChecker::isMixedContent(m_document->securityOrigin(), url)) {

[Mike West, 2014/06/30 12:24:37]

This is here in case the user has flipped the about:flag we're talking about in
the other CL. I'm not at all sure that it's necessary, since we'll still be
throwing the passive-content warning, but it's our current behavior.

         String message = "Connecting to a non-secure WebSocket server from a secure origin is deprecated.";
         m_document->addConsoleMessage(JSMessageSource, WarningMessageLevel, message);