Changeset 57807 in webkit

Timestamp:

Apr 19, 2010 2:05:53 AM (14 years ago)

Author:

tkent@chromium.org

Message:

2010-04-19 Kent Tamura <​tkent@chromium.org>

Reviewed by Shinichiro Hamaji.

[DRT/Chromium] Fix a test initialization problem
​https://bugs.webkit.org/show_bug.cgi?id=37791

• DumpRenderTree/chromium/DumpRenderTree.cpp: (runTest): Call resetTestController() before runFileTest(). Some controllers initialize their fields in reset() and not in their constructors.

Location:

trunk/WebKitTools

Files:

• ChangeLog (modified) (1 diff)
• DumpRenderTree/chromium/DumpRenderTree.cpp (modified) (1 diff)

trunk/WebKitTools/ChangeLog

@@ +1 @@
+2010-04-19  Kent Tamura  <tkent@chromium.org>
+
+        Reviewed by Shinichiro Hamaji.
+
+        [DRT/Chromium] Fix a test initialization problem
+        https://bugs.webkit.org/show_bug.cgi?id=37791
+
+        * DumpRenderTree/chromium/DumpRenderTree.cpp:
+        (runTest): Call resetTestController() before runFileTest(). Some
+          controllers initialize their fields in reset() and not in their
+          constructors.
+
 2010-04-19  Kent Tamura  <tkent@chromium.org>
 

trunk/WebKitTools/DumpRenderTree/chromium/DumpRenderTree.cpp

@@ -56 +56 @@
     }
     params.testUrl = webkit_support::CreateURLForPathOrURL(pathOrURL);
+    shell.resetTestController();
     shell.runFileTest(params);
-    shell.resetTestController();
 }