Title: [96100] trunk/Source/WebKit/chromium
Revision
96100
Author
yu...@chromium.org
Date
2011-09-27 05:39:16 -0700 (Tue, 27 Sep 2011)

Log Message

[Chromium] Web Inspector: Bug with console.log and popstate/hashchange events
https://bugs.webkit.org/show_bug.cgi?id=67732

Added WebKit part of a new interactive UI test for the bug with duplicated console messages after navigation back.

Reviewed by Pavel Feldman.

* src/js/Tests.js:
(.TestSuite.prototype.testConsoleOnNavigateBack.firstConsoleMessageReceived):
(.TestSuite.prototype.testConsoleOnNavigateBack.didClickLink):
(.TestSuite.prototype.testConsoleOnNavigateBack.didNavigateBack):
(.TestSuite.prototype.testConsoleOnNavigateBack.didCompleteNavigation):
(.TestSuite.prototype.testConsoleOnNavigateBack):

Modified Paths

Diff

Modified: trunk/Source/WebKit/chromium/ChangeLog (96099 => 96100)


--- trunk/Source/WebKit/chromium/ChangeLog	2011-09-27 12:34:42 UTC (rev 96099)
+++ trunk/Source/WebKit/chromium/ChangeLog	2011-09-27 12:39:16 UTC (rev 96100)
@@ -1,3 +1,19 @@
+2011-09-27  Yury Semikhatsky  <yu...@chromium.org>
+
+        [Chromium] Web Inspector: Bug with console.log and popstate/hashchange events
+        https://bugs.webkit.org/show_bug.cgi?id=67732
+
+        Added WebKit part of a new interactive UI test for the bug with duplicated console messages after navigation back.
+
+        Reviewed by Pavel Feldman.
+
+        * src/js/Tests.js:
+        (.TestSuite.prototype.testConsoleOnNavigateBack.firstConsoleMessageReceived):
+        (.TestSuite.prototype.testConsoleOnNavigateBack.didClickLink):
+        (.TestSuite.prototype.testConsoleOnNavigateBack.didNavigateBack):
+        (.TestSuite.prototype.testConsoleOnNavigateBack.didCompleteNavigation):
+        (.TestSuite.prototype.testConsoleOnNavigateBack):
+
 2011-09-27  Sheriff Bot  <webkit.review....@gmail.com>
 
         Unreviewed.  Rolled DEPS.

Modified: trunk/Source/WebKit/chromium/src/js/Tests.js (96099 => 96100)


--- trunk/Source/WebKit/chromium/src/js/Tests.js	2011-09-27 12:34:42 UTC (rev 96099)
+++ trunk/Source/WebKit/chromium/src/js/Tests.js	2011-09-27 12:39:16 UTC (rev 96100)
@@ -536,6 +536,40 @@
 };
 
 
+TestSuite.prototype.testConsoleOnNavigateBack = function()
+{
+    if (WebInspector.console.messages.length === 1)
+        firstConsoleMessageReceived.call(this);
+    else
+        WebInspector.console.addEventListener(WebInspector.ConsoleModel.Events.MessageAdded, firstConsoleMessageReceived, this);
+
+    function firstConsoleMessageReceived() {
+        this.evaluateInConsole_("clickLink();", didClickLink.bind(this));
+    }
+
+    function didClickLink() {
+        // Check that there are no new messages(command is not a message).
+        this.assertEquals(1, WebInspector.console.messages.length);
+        this.assertEquals(1, WebInspector.console.messages[0].totalRepeatCount);
+        this.evaluateInConsole_("history.back();", didNavigateBack.bind(this));
+    }
+
+    function didNavigateBack()
+    {
+        // Make sure navigation completed and possible console messages were pushed.
+        this.evaluateInConsole_("void 0;", didCompleteNavigation.bind(this));
+    }
+
+    function didCompleteNavigation() {
+        this.assertEquals(1, WebInspector.console.messages.length);
+        this.assertEquals(1, WebInspector.console.messages[0].totalRepeatCount);
+        this.releaseControl();
+    }
+
+    this.takeControl();
+};
+
+
 TestSuite.prototype.testSharedWorker = function()
 {
     function didEvaluateInConsole(resultText) {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to