[webkit-changes] [122850] trunk/Tools

Tue, 17 Jul 2012 10:27:10 -0700

Title: [122850] trunk/Tools
Revision
122850
Author
[email protected]
Date
2012-07-17 10:19:18 -0700 (Tue, 17 Jul 2012)

Log Message

Unreviewed, rolling out r122828.
http://trac.webkit.org/changeset/122828
https://bugs.webkit.org/show_bug.cgi?id=91516

DumpRenderTree crashes after printing test results. (Requested
by vsevik on #webkit).

Patch by Sheriff Bot <[email protected]> on 2012-07-17

* DumpRenderTree/chromium/TestRunner/TestInterfaces.cpp:
(TestInterfaces::TestInterfaces):
(TestInterfaces::~TestInterfaces):
(TestInterfaces::bindTo):
(TestInterfaces::resetAll):
* DumpRenderTree/chromium/TestRunner/TestInterfaces.h:
(TestInterfaces):
* DumpRenderTree/chromium/TestShell.cpp:
(TestShell::initialize):
(TestShell::createMainWindow):
(TestShell::~TestShell):
(TestShell::resetTestController):
(TestShell::bindJSObjectsToWindow):
* DumpRenderTree/chromium/TestShell.h:
(TestShell::accessibilityController):
(TestShell):

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (122849 => 122850)


--- trunk/Tools/ChangeLog	2012-07-17 17:14:36 UTC (rev 122849)
+++ trunk/Tools/ChangeLog	2012-07-17 17:19:18 UTC (rev 122850)
@@ -1,3 +1,29 @@
+2012-07-17  Sheriff Bot  <[email protected]>
+
+        Unreviewed, rolling out r122828.
+        http://trac.webkit.org/changeset/122828
+        https://bugs.webkit.org/show_bug.cgi?id=91516
+
+        DumpRenderTree crashes after printing test results. (Requested
+        by vsevik on #webkit).
+
+        * DumpRenderTree/chromium/TestRunner/TestInterfaces.cpp:
+        (TestInterfaces::TestInterfaces):
+        (TestInterfaces::~TestInterfaces):
+        (TestInterfaces::bindTo):
+        (TestInterfaces::resetAll):
+        * DumpRenderTree/chromium/TestRunner/TestInterfaces.h:
+        (TestInterfaces):
+        * DumpRenderTree/chromium/TestShell.cpp:
+        (TestShell::initialize):
+        (TestShell::createMainWindow):
+        (TestShell::~TestShell):
+        (TestShell::resetTestController):
+        (TestShell::bindJSObjectsToWindow):
+        * DumpRenderTree/chromium/TestShell.h:
+        (TestShell::accessibilityController):
+        (TestShell):
+
 2012-07-17  Gabor Ballabas  <[email protected]>
 
         [Qt][V8] Remove the V8 related codepaths and configuration

Modified: trunk/Tools/DumpRenderTree/chromium/TestRunner/TestInterfaces.cpp (122849 => 122850)


--- trunk/Tools/DumpRenderTree/chromium/TestRunner/TestInterfaces.cpp	2012-07-17 17:14:36 UTC (rev 122849)
+++ trunk/Tools/DumpRenderTree/chromium/TestRunner/TestInterfaces.cpp	2012-07-17 17:19:18 UTC (rev 122850)
@@ -31,47 +31,27 @@
 #include "config.h"
 #include "TestInterfaces.h"
 
-#include "AccessibilityController.h"
 #include "GamepadController.h"
-#include "TextInputController.h"
 #include "platform/WebString.h"
 
 using WebKit::WebFrame;
 using WebKit::WebString;
-using WebKit::WebView;
 
 TestInterfaces::TestInterfaces()
 {
-    m_accessibilityController = adoptPtr(new AccessibilityController());
     m_gamepadController = adoptPtr(new GamepadController());
-    m_textInputController = adoptPtr(new TextInputController());
 }
 
 TestInterfaces::~TestInterfaces()
 {
-    m_accessibilityController->setWebView(0);
-    // m_gamepadController doesn't depend on WebView.
-    m_textInputController->setWebView(0);
 }
 
-void TestInterfaces::setWebView(WebView* webView)
-{
-    m_accessibilityController->setWebView(webView);
-    // m_gamepadController doesn't depend on WebView.
-    m_textInputController->setWebView(webView);
-}
-
-
 void TestInterfaces::bindTo(WebFrame* frame)
 {
-    m_accessibilityController->bindToJavascript(frame, WebString::fromUTF8("accessibilityController"));
     m_gamepadController->bindToJavascript(frame, WebString::fromUTF8("gamepadController"));
-    m_textInputController->bindToJavascript(frame, WebString::fromUTF8("textInputController"));
 }
 
 void TestInterfaces::resetAll()
 {
-    m_accessibilityController->reset();
     m_gamepadController->reset();
-    // m_textInputController doesn't have any state to reset.
 }

Modified: trunk/Tools/DumpRenderTree/chromium/TestRunner/TestInterfaces.h (122849 => 122850)


--- trunk/Tools/DumpRenderTree/chromium/TestRunner/TestInterfaces.h	2012-07-17 17:14:36 UTC (rev 122849)
+++ trunk/Tools/DumpRenderTree/chromium/TestRunner/TestInterfaces.h	2012-07-17 17:19:18 UTC (rev 122850)
@@ -35,29 +35,20 @@
 
 namespace WebKit {
 class WebFrame;
-class WebView;
 }
 
-class AccessibilityController;
 class GamepadController;
-class TextInputController;
 
 class TestInterfaces {
 public:
     TestInterfaces();
     ~TestInterfaces();
 
-    void setWebView(WebKit::WebView*);
-
     void bindTo(WebKit::WebFrame*);
     void resetAll();
 
-    AccessibilityController* accessibilityController() { return m_accessibilityController.get(); }
-
 private:
-    OwnPtr<AccessibilityController> m_accessibilityController;
     OwnPtr<GamepadController> m_gamepadController;
-    OwnPtr<TextInputController> m_textInputController;
 };
 
 #endif // TestInterfaces_h

Modified: trunk/Tools/DumpRenderTree/chromium/TestShell.cpp (122849 => 122850)


--- trunk/Tools/DumpRenderTree/chromium/TestShell.cpp	2012-07-17 17:14:36 UTC (rev 122849)
+++ trunk/Tools/DumpRenderTree/chromium/TestShell.cpp	2012-07-17 17:19:18 UTC (rev 122850)
@@ -146,9 +146,11 @@
 void TestShell::initialize()
 {
     m_webPermissions = adoptPtr(new WebPermissions(this));
+    m_accessibilityController = adoptPtr(new AccessibilityController());
     m_testInterfaces = adoptPtr(new TestInterfaces());
     m_layoutTestController = adoptPtr(new LayoutTestController(this));
     m_eventSender = adoptPtr(new EventSender(this));
+    m_textInputController = adoptPtr(new TextInputController());
 #if ENABLE(NOTIFICATIONS) || ENABLE(LEGACY_NOTIFICATIONS)
     m_notificationPresenter = adoptPtr(new NotificationPresenter(this));
 #endif
@@ -173,13 +175,15 @@
     m_drtDevToolsAgent = adoptPtr(new DRTDevToolsAgent);
     m_webViewHost = adoptPtr(createNewWindow(WebURL(), m_drtDevToolsAgent.get()));
     m_webView = m_webViewHost->webView();
-    m_testInterfaces->setWebView(m_webView);
+    m_accessibilityController->setWebView(m_webView);
+    m_textInputController->setWebView(m_webView);
     m_drtDevToolsAgent->setWebView(m_webView);
 }
 
 TestShell::~TestShell()
 {
-    m_testInterfaces.clear();
+    m_accessibilityController->setWebView(0);
+    m_textInputController->setWebView(0);
     m_drtDevToolsAgent->setWebView(0);
 }
 
@@ -290,6 +294,7 @@
 {
     resetWebSettings(*webView());
     m_webPermissions->reset();
+    m_accessibilityController->reset();
     m_testInterfaces->resetAll();
     m_layoutTestController->reset();
     m_eventSender->reset();
@@ -723,10 +728,12 @@
 void TestShell::bindJSObjectsToWindow(WebFrame* frame)
 {
     WebTestingSupport::injectInternalsObject(frame);
+    m_accessibilityController->bindToJavascript(frame, WebString::fromUTF8("accessibilityController"));
     m_testInterfaces->bindTo(frame);
     m_layoutTestController->bindToJavascript(frame, WebString::fromUTF8("layoutTestController"));
     m_layoutTestController->bindToJavascript(frame, WebString::fromUTF8("testRunner"));
     m_eventSender->bindToJavascript(frame, WebString::fromUTF8("eventSender"));
+    m_textInputController->bindToJavascript(frame, WebString::fromUTF8("textInputController"));
 }
 
 WebViewHost* TestShell::createNewWindow(const WebKit::WebURL& url)

Modified: trunk/Tools/DumpRenderTree/chromium/TestShell.h (122849 => 122850)


--- trunk/Tools/DumpRenderTree/chromium/TestShell.h	2012-07-17 17:14:36 UTC (rev 122849)
+++ trunk/Tools/DumpRenderTree/chromium/TestShell.h	2012-07-17 17:19:18 UTC (rev 122850)
@@ -38,6 +38,7 @@
 #include "NotificationPresenter.h"
 #include "TestEventPrinter.h"
 #include "TestInterfaces.h"
+#include "TextInputController.h"
 #include "WebPreferences.h"
 #include "WebViewHost.h"
 #include <string>
@@ -94,7 +95,7 @@
     WebViewHost* webViewHost() const { return m_webViewHost.get(); }
     LayoutTestController* layoutTestController() const { return m_layoutTestController.get(); }
     EventSender* eventSender() const { return m_eventSender.get(); }
-    AccessibilityController* accessibilityController() const { return m_testInterfaces->accessibilityController(); }
+    AccessibilityController* accessibilityController() const { return m_accessibilityController.get(); }
 #if ENABLE(NOTIFICATIONS)
     NotificationPresenter* notificationPresenter() const { return m_notificationPresenter.get(); }
 #endif
@@ -214,9 +215,11 @@
     OwnPtr<WebPermissions> m_webPermissions;
     OwnPtr<DRTDevToolsAgent> m_drtDevToolsAgent;
     OwnPtr<DRTDevToolsClient> m_drtDevToolsClient;
+    OwnPtr<AccessibilityController> m_accessibilityController;
     OwnPtr<TestInterfaces> m_testInterfaces;
     OwnPtr<EventSender> m_eventSender;
     OwnPtr<LayoutTestController> m_layoutTestController;
+    OwnPtr<TextInputController> m_textInputController;
 #if ENABLE(NOTIFICATIONS) || ENABLE(LEGACY_NOTIFICATIONS)
     OwnPtr<NotificationPresenter> m_notificationPresenter;
 #endif

_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to