Title: [104822] trunk
Revision
104822
Author
loi...@chromium.org
Date
2012-01-12 06:44:06 -0800 (Thu, 12 Jan 2012)

Log Message

Web Inspector: performance: restore 'log 300 messages into console' test.
https://bugs.webkit.org/show_bug.cgi?id=76170

It was removed in order of transition from layout tests to perf tests.

Reviewed by Yury Semikhatsky.

PerformanceTests:

* inspector/console-300-lines.html: Added.

Source/WebCore:

* inspector/front-end/ConsoleView.js:
(WebInspector.ConsoleView.prototype._scheduleScrollIntoView.scrollIntoView):
(WebInspector.ConsoleView.prototype._scheduleScrollIntoView):

Modified Paths

Added Paths

Diff

Modified: trunk/PerformanceTests/ChangeLog (104821 => 104822)


--- trunk/PerformanceTests/ChangeLog	2012-01-12 14:39:36 UTC (rev 104821)
+++ trunk/PerformanceTests/ChangeLog	2012-01-12 14:44:06 UTC (rev 104822)
@@ -1,5 +1,16 @@
 2012-01-12  Ilya Tikhonovsky  <loi...@chromium.org>
 
+        Web Inspector: performance: restore 'log 300 messages into console' test.
+        https://bugs.webkit.org/show_bug.cgi?id=76170
+
+        It was removed in order of transition from layout tests to perf tests.
+
+        Reviewed by Yury Semikhatsky.
+
+        * inspector/console-300-lines.html: Added.
+
+2012-01-12  Ilya Tikhonovsky  <loi...@chromium.org>
+
         Web Inspector: performance tests: fix memory leak in first-open-resources test.
         https://bugs.webkit.org/show_bug.cgi?id=76049
 

Added: trunk/PerformanceTests/inspector/console-300-lines.html (0 => 104822)


--- trunk/PerformanceTests/inspector/console-300-lines.html	                        (rev 0)
+++ trunk/PerformanceTests/inspector/console-300-lines.html	2012-01-12 14:44:06 UTC (rev 104822)
@@ -0,0 +1,80 @@
+<html>
+<head>
+<script src=""
+<script src=""
+<script>
+
+function makeConsoleEvents(count)
+{
+    for (var i = 0; i < count; ++i)
+        console.log("Console log: " + i);
+}
+
+function test()
+{
+    var message = {
+        "method":"Console.messageAdded",
+        "params":{
+            "message":{
+                "source":"console-api",
+                "level":"log",
+                "text":"Console log message",
+                "type":"log",
+                "line":10,
+                "url":"file://./console-300-lines.html",
+                "repeatCount":1,
+                "parameters":[{
+                    "type":"string",
+                    "value":"Console log: 42"
+                }],
+                "stackTrace":[
+                    {
+                        "functionName":"makeConsoleEvents",
+                        "url":"file://./console-300-lines.html",
+                        "lineNumber":10,
+                        "columnNumber":17
+                    },
+                    {
+                        "functionName":"",
+                        "url":"file://./console-300-lines.html",
+                        "lineNumber":37,
+                        "columnNumber":9
+                    }
+                ]
+            }
+        }
+    };
+
+    WebInspector.showPanel("console");
+
+    function test(timer)
+    {
+        WebInspector.console.clearMessages();
+        InspectorTest.addSniffer(WebInspector.consoleView.promptElement, "scrollIntoView", finish);
+
+        var cookie = timer.start("console-300-lines");
+        for (var i = 0; i < 300; ++i)
+            InspectorBackend.dispatch(message);
+
+        function finish()
+        {
+            timer.finish(cookie);
+            timer.done("panel-update");
+        }
+    }
+
+    InspectorTest.runPerformanceTest(test, 10000);
+}
+
+if (!window.layoutTestController) {
+    setTimeout(function() {
+        makeConsoleEvents(300);
+    }, 3000);
+}
+
+</script>
+</head>
+
+<body _onload_="runTest()">
+</body>
+</html>
Property changes on: trunk/PerformanceTests/inspector/console-300-lines.html
___________________________________________________________________

Added: svn:eol-style

Modified: trunk/Source/WebCore/ChangeLog (104821 => 104822)


--- trunk/Source/WebCore/ChangeLog	2012-01-12 14:39:36 UTC (rev 104821)
+++ trunk/Source/WebCore/ChangeLog	2012-01-12 14:44:06 UTC (rev 104822)
@@ -1,3 +1,16 @@
+2012-01-12  Ilya Tikhonovsky  <loi...@chromium.org>
+
+        Web Inspector: performance: restore 'log 300 messages into console' test.
+        https://bugs.webkit.org/show_bug.cgi?id=76170
+
+        It was removed in order of transition from layout tests to perf tests.
+
+        Reviewed by Yury Semikhatsky.
+
+        * inspector/front-end/ConsoleView.js:
+        (WebInspector.ConsoleView.prototype._scheduleScrollIntoView.scrollIntoView):
+        (WebInspector.ConsoleView.prototype._scheduleScrollIntoView):
+
 2012-01-12  Hans Wennborg  <h...@chromium.org>
 
         Speech input: Send text to correct element even if focus has changed

Modified: trunk/Source/WebCore/inspector/front-end/ConsoleView.js (104821 => 104822)


--- trunk/Source/WebCore/inspector/front-end/ConsoleView.js	2012-01-12 14:39:36 UTC (rev 104821)
+++ trunk/Source/WebCore/inspector/front-end/ConsoleView.js	2012-01-12 14:44:06 UTC (rev 104822)
@@ -277,8 +277,8 @@
 
         function scrollIntoView()
         {
+            delete this._scrollIntoViewTimer;
             this.promptElement.scrollIntoView(true);
-            delete this._scrollIntoViewTimer;
         }
         this._scrollIntoViewTimer = setTimeout(scrollIntoView.bind(this), 20);
     },
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to