Revert 96912 - Always call setActive() in WebViewImpl::setFocus(),

Source/WebKit/chromium/tests/WebFrameTest.cpp

Index: Source/WebKit/chromium/tests/WebFrameTest.cpp
===================================================================
--- Source/WebKit/chromium/tests/WebFrameTest.cpp	(revision 99235)
+++ Source/WebKit/chromium/tests/WebFrameTest.cpp	(working copy)
@@ -30,7 +30,6 @@
 
 #include "config.h"
 
-#include "FrameTestHelpers.h"
 #include "ResourceError.h"
 #include "WebDocument.h"
 #include "WebFormElement.h"
@@ -40,8 +39,14 @@
 #include "WebSearchableFormData.h"
 #include "WebSecurityPolicy.h"
 #include "WebSettings.h"
+#include "WebString.h"
+#include "WebURL.h"
+#include "WebURLRequest.h"
+#include "WebURLResponse.h"
+#include "WebViewClient.h"
 #include "WebViewImpl.h"
 #include "v8.h"
+#include <googleurl/src/gurl.h>
 #include <gtest/gtest.h>
 #include <webkit/support/webkit_support.h>
 
@@ -52,8 +57,8 @@
 class WebFrameTest : public testing::Test {
 public:
     WebFrameTest()
-        : m_baseURL("http://www.test.com/"),
-          m_chromeURL("chrome://")
+        : baseURL("http://www.test.com/"),
+          chromeURL("chrome://")
     {
     }
 
@@ -64,19 +69,61 @@
 
     void registerMockedHttpURLLoad(const std::string& fileName)
     {
-        FrameTestHelpers::registerMockedURLLoad(m_baseURL, fileName);
+        registerMockedURLLoad(baseURL, fileName);
     }
 
     void registerMockedChromeURLLoad(const std::string& fileName)
     {
-        FrameTestHelpers::registerMockedURLLoad(m_chromeURL, fileName);
+        registerMockedURLLoad(chromeURL, fileName);
     }
 
+    void serveRequests()
+    {
+        webkit_support::ServeAsynchronousMockedRequests();
+    }
+
+    void loadHttpFrame(WebFrame* frame, const std::string& fileName)
+    {
+        loadFrame(frame, baseURL, fileName);
+    }
+
+    void loadChromeFrame(WebFrame* frame, const std::string& fileName)
+    {
+        loadFrame(frame, chromeURL, fileName);
+    }
+
+    void registerMockedURLLoad(const std::string& base, const std::string& fileName)
+    {
+        WebURLResponse response;
+        response.initialize();
+        response.setMIMEType("text/html");
+
+        std::string filePath = webkit_support::GetWebKitRootDir().utf8();
+        filePath += "/Source/WebKit/chromium/tests/data/";
+        filePath += fileName;
+
+        webkit_support::RegisterMockedURL(WebURL(GURL(base + fileName)), response, WebString::fromUTF8(filePath));
+    }
+
+    void loadFrame(WebFrame* frame, const std::string& base, const std::string& fileName)
+    {
+        WebURLRequest urlRequest;
+        urlRequest.initialize();
+        urlRequest.setURL(WebURL(GURL(base + fileName)));
+        frame->loadRequest(urlRequest);
+    }
+
 protected:
-    std::string m_baseURL;
-    std::string m_chromeURL;
+    std::string baseURL;
+    std::string chromeURL;
 };
 
+class TestWebFrameClient : public WebFrameClient {
+};
+
+class TestWebViewClient : public WebViewClient {
+};
+
 TEST_F(WebFrameTest, ContentText)
 {
     registerMockedHttpURLLoad("iframes_test.html");
@@ -84,8 +131,14 @@
     registerMockedHttpURLLoad("invisible_iframe.html");
     registerMockedHttpURLLoad("zero_sized_iframe.html");
 
-    WebView* webView = FrameTestHelpers::createWebViewAndLoad(m_baseURL + "iframes_test.html");
+    // Create and initialize the WebView.
+    TestWebFrameClient webFrameClient;
+    WebView* webView = WebView::create(0);
+    webView->initializeMainFrame(&webFrameClient);
 
+    loadHttpFrame(webView->mainFrame(), "iframes_test.html");
+    serveRequests();
+
     // Now retrieve the frames text and test it only includes visible elements.
     std::string content = webView->mainFrame()->contentAsText(1024).utf8();
     EXPECT_NE(std::string::npos, content.find(" visible paragraph"));
@@ -104,8 +157,15 @@
     registerMockedHttpURLLoad("invisible_iframe.html");
     registerMockedHttpURLLoad("zero_sized_iframe.html");
 
-    WebView* webView = FrameTestHelpers::createWebViewAndLoad(m_baseURL + "iframes_test.html", true);
+    // Create and initialize the WebView.
+    TestWebFrameClient webFrameClient;
+    WebView* webView = WebView::create(0);
+    webView->settings()->setJavaScriptEnabled(true);
+    webView->initializeMainFrame(&webFrameClient);
 
+    loadHttpFrame(webView->mainFrame(), "iframes_test.html");
+    serveRequests();
+
     v8::HandleScope scope;
     EXPECT_EQ(webView->mainFrame(),
               WebFrame::frameForContext(
@@ -121,8 +181,13 @@
 {
     registerMockedHttpURLLoad("form.html");
 
-    WebView* webView = FrameTestHelpers::createWebViewAndLoad(m_baseURL + "form.html");
+    TestWebFrameClient webFrameClient;
+    WebView* webView = WebView::create(0);
+    webView->initializeMainFrame(&webFrameClient);
 
+    loadHttpFrame(webView->mainFrame(), "form.html");
+    serveRequests();
+
     WebVector<WebFormElement> forms;
     webView->mainFrame()->document().forms(forms);
     webView->close();
@@ -137,11 +202,18 @@
 {
     registerMockedChromeURLLoad("history.html");
 
-    WebView* webView = FrameTestHelpers::createWebViewAndLoad(m_chromeURL + "history.html", true);
+    // Create and initialize the WebView.
+    TestWebFrameClient webFrameClient;
+    WebView* webView = WebView::create(0);
+    webView->settings()->setJavaScriptEnabled(true);
+    webView->initializeMainFrame(&webFrameClient);
 
+    loadChromeFrame(webView->mainFrame(), "history.html");
+    serveRequests();
+
     // Try to run JS against the chrome-style URL.
     WebSecurityPolicy::registerURLSchemeAsNotAllowingJavascriptURLs("chrome");
-    FrameTestHelpers::loadFrame(webView->mainFrame(), "javascript:document.body.appendChild(document.createTextNode('Clobbered'))");
+    loadFrame(webView->mainFrame(), "javascript:", "document.body.appendChild(document.createTextNode('Clobbered'))");
 
     // Now retrieve the frames text and see if it was clobbered.
     std::string content = webView->mainFrame()->contentAsText(1024).utf8();
@@ -175,12 +247,17 @@
     registerMockedHttpURLLoad("form.html");
 
     TestReloadDoesntRedirectWebFrameClient webFrameClient;
-    WebView* webView = FrameTestHelpers::createWebViewAndLoad(m_baseURL + "form.html", false, &webFrameClient);
+    WebView* webView = WebView::create(0);
+    webView->initializeMainFrame(&webFrameClient);
 
+    loadHttpFrame(webView->mainFrame(), "form.html");
+    serveRequests();
+    // Frame is loaded.
+
     webView->mainFrame()->reload(true);
     // start reload before request is delivered.
     webView->mainFrame()->reload(true);
-    webkit_support::ServeAsynchronousMockedRequests();
+    serveRequests();
 }
 
 TEST_F(WebFrameTest, ClearFocusedNodeTest)
@@ -188,8 +265,16 @@
     registerMockedHttpURLLoad("iframe_clear_focused_node_test.html");
     registerMockedHttpURLLoad("autofocus_input_field_iframe.html");
 
-    WebViewImpl* webViewImpl = static_cast<WebViewImpl*>(FrameTestHelpers::createWebViewAndLoad(m_baseURL + "iframe_clear_focused_node_test.html", true));
+    // Create and initialize the WebView.
+    TestWebFrameClient webFrameClient;
+    TestWebViewClient webviewClient;
+    WebViewImpl* webViewImpl = static_cast<WebViewImpl*>(WebView::create(&webviewClient));
+    webViewImpl->settings()->setJavaScriptEnabled(true);
+    webViewImpl->initializeMainFrame(&webFrameClient);
 
+    loadHttpFrame(webViewImpl->mainFrame(), "iframe_clear_focused_node_test.html");
+    serveRequests();
+
     // Clear the focused node.
     webViewImpl->clearFocusedNode();
 
@@ -268,7 +353,11 @@
 
     // Load a frame with an iframe, make sure we get the right create notifications.
     ContextLifetimeTestWebFrameClient webFrameClient;
-    WebView* webView = FrameTestHelpers::createWebViewAndLoad(m_baseURL + "context_notifications_test.html", true, &webFrameClient);
+    WebView* webView = WebView::create(0);
+    webView->settings()->setJavaScriptEnabled(true);
+    webView->initializeMainFrame(&webFrameClient);
+    loadHttpFrame(webView->mainFrame(), "context_notifications_test.html");
+    serveRequests();
 
     WebFrame* mainFrame = webView->mainFrame();
     WebFrame* childFrame = mainFrame->firstChild();
@@ -306,11 +395,15 @@
     registerMockedHttpURLLoad("context_notifications_test_frame.html");
 
     ContextLifetimeTestWebFrameClient webFrameClient;
-    WebView* webView = FrameTestHelpers::createWebViewAndLoad(m_baseURL + "context_notifications_test.html", true, &webFrameClient);
+    WebView* webView = WebView::create(0);
+    webView->settings()->setJavaScriptEnabled(true);
+    webView->initializeMainFrame(&webFrameClient);
+    loadHttpFrame(webView->mainFrame(), "context_notifications_test.html");
+    serveRequests();
 
     // Refresh, we should get two release notifications and two more create notifications.
     webView->mainFrame()->reload(false);
-    webkit_support::ServeAsynchronousMockedRequests();
+    serveRequests();
     ASSERT_EQ(4u, webFrameClient.createNotifications.size());
     ASSERT_EQ(2u, webFrameClient.releaseNotifications.size());
 
@@ -345,7 +438,11 @@
     registerMockedHttpURLLoad("context_notifications_test_frame.html");
 
     ContextLifetimeTestWebFrameClient webFrameClient;
-    WebView* webView = FrameTestHelpers::createWebViewAndLoad(m_baseURL + "context_notifications_test.html", true, &webFrameClient);
+    WebView* webView = WebView::create(0);
+    webView->settings()->setJavaScriptEnabled(true);
+    webView->initializeMainFrame(&webFrameClient);
+    loadHttpFrame(webView->mainFrame(), "context_notifications_test.html");
+    serveRequests();
 
     // Add an isolated world.
     webFrameClient.reset();