Title: [150009] trunk/PerformanceTests
Revision
150009
Author
[email protected]
Date
2013-05-13 07:03:57 -0700 (Mon, 13 May 2013)

Log Message

Web Inspector: Get rid of native-memory-snapshot in performance test
https://bugs.webkit.org/show_bug.cgi?id=116013

Patch by Seokju Kwon <[email protected]> on 2013-05-13
Reviewed by Andreas Kling.

Remove native-memory-snapshot.html from PerformanceTests.

* inspector/native-memory-snapshot.html: Removed.
* inspector/performance-test.js:
(initialize_TimeTracker.Timer):
(initialize_TimeTracker.Timer.prototype._dump):
(initialize_TimeTracker.InspectorTest.runPerformanceTest):

Modified Paths

Removed Paths

Diff

Modified: trunk/PerformanceTests/ChangeLog (150008 => 150009)


--- trunk/PerformanceTests/ChangeLog	2013-05-13 13:55:31 UTC (rev 150008)
+++ trunk/PerformanceTests/ChangeLog	2013-05-13 14:03:57 UTC (rev 150009)
@@ -1,3 +1,18 @@
+2013-05-13  Seokju Kwon  <[email protected]>
+
+        Web Inspector: Get rid of native-memory-snapshot in performance test
+        https://bugs.webkit.org/show_bug.cgi?id=116013
+
+        Reviewed by Andreas Kling.
+
+        Remove native-memory-snapshot.html from PerformanceTests.
+
+        * inspector/native-memory-snapshot.html: Removed.
+        * inspector/performance-test.js:
+        (initialize_TimeTracker.Timer):
+        (initialize_TimeTracker.Timer.prototype._dump):
+        (initialize_TimeTracker.InspectorTest.runPerformanceTest):
+
 2013-04-10  Benjamin Poulain  <[email protected]>
 
         Mass remove all the empty directories

Deleted: trunk/PerformanceTests/inspector/native-memory-snapshot.html (150008 => 150009)


--- trunk/PerformanceTests/inspector/native-memory-snapshot.html	2013-05-13 13:55:31 UTC (rev 150008)
+++ trunk/PerformanceTests/inspector/native-memory-snapshot.html	2013-05-13 14:03:57 UTC (rev 150009)
@@ -1,76 +0,0 @@
-<html>
-<head>
-  <style type="text/css">
-    span {
-      color:red;
-      width: 100px;
-      height: 20px;
-    }
-    div {
-      color:blue;
-      width: 50px;
-      height: 10px;
-    }
-  </style>
-  <script src=""
-  <script src=""
-<script>
-
-function test()
-{
-    function performanceTest(timer)
-    {
-        var testName = /([^\/]+)\.html$/.exec(WebInspector.inspectedPageURL)[1];
-        var takeTimerCookie = timer.start("take");
-        MemoryAgent.getProcessMemoryDistribution(dump);
-
-        function dump(e, data)
-        {
-            timer.finish(takeTimerCookie);
-            function reportSize(data)
-            {
-                timer.reportSize(data.name, data.size);
-                if (data.children) {
-                    for (var i = 0; i < data.children.length; ++i)
-                        reportSize(data.children[i]);
-                }
-            }
-            reportSize(data);
-            var knownSize = 0;
-            for (var i = 0; i < data.children.length; ++i)
-                knownSize += data.children[i].size;
-            timer.reportSize("Unknown", data.size - knownSize);
-            timer.done(testName);
-        }
-    }
-
-    InspectorTest.runPerformanceTest(performanceTest, 2000);
-}
-
-function createDOMTree(elementsCount)
-{
-    var root = document.getElementById("testTreeRoot");
-
-    for (var i = 0; i < elementsCount; ++i) {
-        var span = document.createElement("span");
-        span.id = "span_" + i;
-        span.style.width = "10px";
-        root.appendChild(span);
-    }
-
-    for (var i = 0; i < elementsCount; ++i)
-        root.appendChild(document.createElement("div"));
-
-    for (var i = 0; i < elementsCount; ++i)
-        root.appendChild(document.createElement("a"));
-}
-
-setTimeout(createDOMTree.bind(null,1000), 0);
-
-</script>
-</head>
-<body _onload_="runTest()">
-  <div id="testTreeRoot">
-  </div>
-</body>
-</html>

Modified: trunk/PerformanceTests/inspector/performance-test.js (150008 => 150009)


--- trunk/PerformanceTests/inspector/performance-test.js	2013-05-13 13:55:31 UTC (rev 150008)
+++ trunk/PerformanceTests/inspector/performance-test.js	2013-05-13 14:03:57 UTC (rev 150009)
@@ -7,7 +7,6 @@
         this._callback = callback;
         this._test = test;
         this._times = {};
-        this._sizes = {};
         this._testStartTime = new Date();
         this._heapSizeDeltas = [];
         this._jsHeapSize = this._getJSHeapSize();
@@ -27,13 +26,6 @@
             this._times[cookie.name].push(endTime - cookie.startTime);
         },
 
-        reportSize: function(name, size)
-        {
-            if (!this._sizes[name])
-                this._sizes[name] = [];
-            this._sizes[name].push(size);
-        },
-
         _getJSHeapSize: function()
         {
             if (window.gc) {
@@ -83,9 +75,6 @@
             for (var testName in this._times)
                 InspectorTest.dumpTestStats(groupName, testName, this._times[testName], "ms");
 
-            for (var testName in this._sizes)
-                InspectorTest.dumpTestStats(groupName, testName, this._sizes[testName], "kB", 1024);
-
             var url = ""
             var regExp = /([^\/]+)\.html/;
             var matches = regExp.exec(url);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to