Attachment 192637 Details for Bug 94733 – Initial performance test

[patch] Initial performance test

bug-94733-20130312141627.patch (text/plain), 1.77 KB, created by Nils Barth on 2013-03-11 22:20:28 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Nils Barth

Created: 2013-03-11 22:20:28 PDT

Size: 1.77 KB

patch

obsolete

>Subversion Revision: 145343
>diff --git a/PerformanceTests/ChangeLog b/PerformanceTests/ChangeLog
>index 073c4cfb27f6d799472a0ede6540dfcff34fe28d..472971fadb57a8de1dd9fee03139c7f98bac9891 100644
>--- a/PerformanceTests/ChangeLog
>+++ b/PerformanceTests/ChangeLog
>@@ -1,3 +1,14 @@
>+2013-03-11  Nils Barth  <nbarth@chromium.org>
>+
>+        Setting innerHTML can be O(n^2) when selection is inside the replaced content
>+        https://bugs.webkit.org/show_bug.cgi?id=94733
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        New performance test.
>+
>+        * Parser/innerHTML-setter-selection-long.html: Added.
>+
> 2013-03-05  Eric Seidel  <eric@webkit.org>
> 
>         Add html-parser-threaded perf test now that the threaded parser is enabled by default
>diff --git a/PerformanceTests/Parser/innerHTML-setter-selection-long.html b/PerformanceTests/Parser/innerHTML-setter-selection-long.html
>new file mode 100644
>index 0000000000000000000000000000000000000000..5b44754e4eb28b13218cbca8a5bfccd1b26f1cd1
>--- /dev/null
>+++ b/PerformanceTests/Parser/innerHTML-setter-selection-long.html
>@@ -0,0 +1,23 @@
>+<!DOCTYPE html>
>+<script src="../resources/runner.js"></script>
>+<div id=content contentEditable=true></div>
>+<script>
>+function generateLongBRChildList(length) {
>+    var contentArray = new Array(length);
>+    for (var i = 0; i < length; ++i) {
>+      contentArray[i] = "<br>";
>+    }
>+    return contentArray.join("\n");
>+}
>+
>+var div = document.getElementById('content');
>+var childList = generateLongBRChildList(5000);
>+window.getSelection().collapse(div, 0);
>+
>+PerfTestRunner.measureRunsPerSecond({
>+    description: 'Set then clear innerHTML on an element with a Caret-type selection and many children',
>+    run: function() {
>+        div.innerHTML = childList;
>+        div.innerHTML = "";
>+    }});
>+</script>

Actions: View | Formatted Diff | Diff

Attachments on bug 94733: 191895 | 192416 | 192635 | 192637 | 192669 | 192848 | 192850 | 197500 | 197504