2011-06-27 Kent Tamura <tkent@chromium.org>
authortkent@chromium.org <tkent@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Jun 2011 08:07:18 +0000 (08:07 +0000)
committertkent@chromium.org <tkent@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Jun 2011 08:07:18 +0000 (08:07 +0000)
        Reviewed by Hajime Morita.

        [Chromium] WebViewHost should be destructed before LayoutTestController, etc.
        https://bugs.webkit.org/show_bug.cgi?id=63427

        ~WebViewHost() navigates to about:blank, so it calls some
        callbacks and LayoutTestController functions.

        * DumpRenderTree/chromium/TestShell.h:
         Move m_printer upper and move m_webViewHost lower.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@89802 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/DumpRenderTree/chromium/TestShell.h

index c451404..8508c33 100644 (file)
@@ -1,3 +1,16 @@
+2011-06-27  Kent Tamura  <tkent@chromium.org>
+
+        Reviewed by Hajime Morita.
+
+        [Chromium] WebViewHost should be destructed before LayoutTestController, etc.
+        https://bugs.webkit.org/show_bug.cgi?id=63427
+
+        ~WebViewHost() navigates to about:blank, so it calls some
+        callbacks and LayoutTestController functions.
+
+        * DumpRenderTree/chromium/TestShell.h:
+         Move m_printer upper and move m_webViewHost lower.
+
 2011-06-27  Adam Barth  <abarth@webkit.org>
 
         Reviewed by Eric Seidel.
index 9a1fc19..dc4c6c8 100644 (file)
@@ -193,7 +193,9 @@ private:
     WebKit::WebWidget* m_focusedWidget;
     bool m_testShellMode;
     WebViewHost* m_devTools;
-    OwnPtr<WebViewHost> m_webViewHost;
+
+    // Be careful of the destruction order of the following objects.
+    OwnPtr<TestEventPrinter> m_printer;
     OwnPtr<WebPermissions> m_webPermissions;
     OwnPtr<DRTDevToolsAgent> m_drtDevToolsAgent;
     OwnPtr<DRTDevToolsClient> m_drtDevToolsClient;
@@ -203,7 +205,8 @@ private:
     OwnPtr<PlainTextController> m_plainTextController;
     OwnPtr<TextInputController> m_textInputController;
     OwnPtr<NotificationPresenter> m_notificationPresenter;
-    OwnPtr<TestEventPrinter> m_printer;
+    OwnPtr<WebViewHost> m_webViewHost;
+
     TestParams m_params;
     int m_timeout; // timeout value in millisecond
     bool m_allowExternalPages;