Speedometer 2.0: Angular v1 test only deletes the second item
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 19 May 2017 10:50:38 +0000 (10:50 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 19 May 2017 10:50:38 +0000 (10:50 +0000)
https://bugs.webkit.org/show_bug.cgi?id=172353

Reviewed by Antti Koivisto.

Like jQuery and Inferno, run querySelector after removing each todo item.

* Speedometer/InteractiveRunner.html: Tweaked CSS again.
* Speedometer/resources/tests.js:

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

PerformanceTests/ChangeLog
PerformanceTests/Speedometer/InteractiveRunner.html
PerformanceTests/Speedometer/resources/tests.js

index 3675be9..ec622f6 100644 (file)
@@ -1,5 +1,17 @@
 2017-05-19  Ryosuke Niwa  <rniwa@webkit.org>
 
+        Speedometer 2.0: Angular v1 test only deletes the second item
+        https://bugs.webkit.org/show_bug.cgi?id=172353
+
+        Reviewed by Antti Koivisto.
+
+        Like jQuery and Inferno, run querySelector after removing each todo item.
+
+        * Speedometer/InteractiveRunner.html: Tweaked CSS again.
+        * Speedometer/resources/tests.js:
+
+2017-05-19  Ryosuke Niwa  <rniwa@webkit.org>
+
         Speedometer 2.0: Elem test isn't updating DOM during the measurement
         https://bugs.webkit.org/show_bug.cgi?id=172343
 
index ff41620..6cc761e 100644 (file)
@@ -10,7 +10,7 @@ ol { list-style: none; margin: 0; padding: 0; }
 ol ol { margin-left: 2em; list-position: outside; }
 .running { text-decoration: underline; }
 .ran { color: grey; }
-nav { position: absolute; right: 10px; max-height: 600px; }
+nav { position: absolute; right: 10px; height: 600px; }
 nav > ol { height: 100%; overflow-y: scroll; }
 </style>
 </head>
index c43efe4..dd9a6fa 100644 (file)
@@ -250,9 +250,8 @@ Suites.push({
                 checkboxes[i].click();
         }),
         new BenchmarkTestStep('DeletingAllItems', function (newTodo, contentWindow, contentDocument) {
-            var deleteButtons = contentDocument.querySelectorAll('.destroy');
-            for (var i = 0; i < deleteButtons.length; i++)
-                deleteButtons[i].click();
+            for (var i = 0; i < numberOfItemsToAdd; i++)
+                contentDocument.querySelector('.destroy').click();
         }),
     ]
 });