Title: [98412] trunk
Revision
98412
Author
aba...@webkit.org
Date
2011-10-25 16:43:33 -0700 (Tue, 25 Oct 2011)

Log Message

EventTargetFactory.in is not sorted
https://bugs.webkit.org/show_bug.cgi?id=70855

Reviewed by Eric Seidel.

PerformanceTests:

Add a microbenchmark for wrapping event target.

* Bindings: Added.
* Bindings/event-target-wrapper.html: Added.
* Parser/resources/runner.js:
(runLoop):
    - Let the user of the harness get a callback when the test is done.

Source/WebCore:

This patch sorts the file.  I was not able to measure a performance
difference using the microbenchmark included with this patch.

* dom/EventTargetFactory.in:

Modified Paths

Added Paths

Diff

Added: trunk/PerformanceTests/Bindings/event-target-wrapper.html (0 => 98412)


--- trunk/PerformanceTests/Bindings/event-target-wrapper.html	                        (rev 0)
+++ trunk/PerformanceTests/Bindings/event-target-wrapper.html	2011-10-25 23:43:33 UTC (rev 98412)
@@ -0,0 +1,40 @@
+<!DOCTYPE html>
+<body>
+<pre id="log"></pre>
+<script src=""
+<a href="" here to start</a>
+<script>
+var link = document.getElementsByTagName('a')[0];
+var evt = null;
+var kIteratonsPerTest = 1000000;
+
+link.addEventListener('click', function(event) {
+    link.textContent = '';
+    window.evt = event;
+    event.preventDefault();
+
+    start(20, function() {
+        var e = window.evt;
+        for (var x = 0; x < kIteratonsPerTest; x++) {
+            e.target;
+        }
+    }, 10, function() {
+        if (window.layoutTestController)
+            layoutTestController.notifyDone();
+    });
+
+    return false;
+}, false);
+
+if (window.layoutTestController) {
+    layoutTestController.waitUntilDone();
+    layoutTestController.dumpAsText();
+}
+
+if (window.eventSender) {
+    eventSender.mouseMoveTo(link.offsetLeft + 5, link.offsetTop + 5);
+    eventSender.mouseDown();
+    eventSender.mouseUp();
+}
+</script>
+</body>

Modified: trunk/PerformanceTests/ChangeLog (98411 => 98412)


--- trunk/PerformanceTests/ChangeLog	2011-10-25 23:42:57 UTC (rev 98411)
+++ trunk/PerformanceTests/ChangeLog	2011-10-25 23:43:33 UTC (rev 98412)
@@ -1,3 +1,18 @@
+2011-10-25  Adam Barth  <aba...@webkit.org>
+
+        EventTargetFactory.in is not sorted
+        https://bugs.webkit.org/show_bug.cgi?id=70855
+
+        Reviewed by Eric Seidel.
+
+        Add a microbenchmark for wrapping event target.
+
+        * Bindings: Added.
+        * Bindings/event-target-wrapper.html: Added.
+        * Parser/resources/runner.js:
+        (runLoop):
+            - Let the user of the harness get a callback when the test is done.
+
 2011-10-04  Eric Seidel  <e...@webkit.org>
 
         Update html5-full-render.html to load the HTML5 spec incrementally, closer to how the browser would

Modified: trunk/PerformanceTests/Parser/resources/runner.js (98411 => 98412)


--- trunk/PerformanceTests/Parser/resources/runner.js	2011-10-25 23:42:57 UTC (rev 98411)
+++ trunk/PerformanceTests/Parser/resources/runner.js	2011-10-25 23:43:33 UTC (rev 98412)
@@ -73,6 +73,7 @@
         window.setTimeout(run, 0);
     } else {
         logStatistics(times);
+        window.doneFunction();
     }
 }
 
@@ -91,10 +92,11 @@
     runLoop()
 }
 
-function start(runCount, runFunction, loopsPerRun) {
+function start(runCount, runFunction, loopsPerRun, doneFunction) {
     window.runCount = runCount;
     window.runFunction = runFunction;
     window.loopsPerRun = loopsPerRun || 10;
+    window.doneFunction = doneFunction || function() {};
 
     log("Running " + runCount + " times");
     runLoop();

Modified: trunk/Source/WebCore/ChangeLog (98411 => 98412)


--- trunk/Source/WebCore/ChangeLog	2011-10-25 23:42:57 UTC (rev 98411)
+++ trunk/Source/WebCore/ChangeLog	2011-10-25 23:43:33 UTC (rev 98412)
@@ -1,5 +1,17 @@
 2011-10-25  Adam Barth  <aba...@webkit.org>
 
+        EventTargetFactory.in is not sorted
+        https://bugs.webkit.org/show_bug.cgi?id=70855
+
+        Reviewed by Eric Seidel.
+
+        This patch sorts the file.  I was not able to measure a performance
+        difference using the microbenchmark included with this patch.
+
+        * dom/EventTargetFactory.in:
+
+2011-10-25  Adam Barth  <aba...@webkit.org>
+
         Attempt to fix the Windows build.  Windows mashes all these files into
         one, which doesn't respect the compilation unit rules of C++.
 

Modified: trunk/Source/WebCore/dom/EventTargetFactory.in (98411 => 98412)


--- trunk/Source/WebCore/dom/EventTargetFactory.in	2011-10-25 23:42:57 UTC (rev 98411)
+++ trunk/Source/WebCore/dom/EventTargetFactory.in	2011-10-25 23:43:33 UTC (rev 98412)
@@ -1,27 +1,27 @@
 namespace="EventTarget"
 
+AudioContext conditional=WEB_AUDIO
+DedicatedWorkerContext conditional=WORKERS
+DOMApplicationCache
+DOMWindow
 EventSource
-MessagePort
-Node
-DOMWindow
-XMLHttpRequest
-XMLHttpRequestUpload
-DOMApplicationCache
-SVGElementInstance conditional=SVG
-Worker conditional=WORKERS
-DedicatedWorkerContext conditional=WORKERS
-SharedWorker conditional=SHARED_WORKERS
-SharedWorkerContext conditional=SHARED_WORKERS
-AudioContext conditional=WEB_AUDIO
-_javascript_AudioNode conditional=WEB_AUDIO
-WebSocket conditional=WEB_SOCKETS
-Notification conditional=NOTIFICATIONS
 FileReader conditional=BLOB
 FileWriter conditional=FILE_SYSTEM
 IDBDatabase conditional=INDEXED_DATABASE
 IDBRequest conditional=INDEXED_DATABASE
 IDBTransaction conditional=INDEXED_DATABASE
 IDBVersionChangeRequest conditional=INDEXED_DATABASE
+_javascript_AudioNode conditional=WEB_AUDIO
+LocalMediaStream conditional=MEDIA_STREAM
 MediaStream conditional=MEDIA_STREAM
-LocalMediaStream conditional=MEDIA_STREAM
+MessagePort
+Node
+Notification conditional=NOTIFICATIONS
 PeerConnection conditional=MEDIA_STREAM
+SharedWorker conditional=SHARED_WORKERS
+SharedWorkerContext conditional=SHARED_WORKERS
+SVGElementInstance conditional=SVG
+WebSocket conditional=WEB_SOCKETS
+Worker conditional=WORKERS
+XMLHttpRequest
+XMLHttpRequestUpload
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to