We should only add a live event listener once, not once per TestSelector.
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 20 Mar 2012 02:20:01 +0000 (02:20 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 20 Mar 2012 02:20:01 +0000 (02:20 +0000)
Adding one per TestSelector would cause accordion('option', 'active') to
incorrectly be "false" if you examine multiple sets of tests.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/ui/results.js:

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

Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/ui/results.js
Tools/ChangeLog

index 515c814..9cff9e1 100644 (file)
@@ -165,6 +165,12 @@ ui.results.ResultsDetails = base.extends('div', {
     },
 });
 
+// jQuery's builtin accordion overrides mousedown, which means you can't select the header text
+// or click on the link to the flakiness dashboard.
+$('.ui-accordion-header').live('click', function() {
+    $(this).trigger('customaccordianclick');
+})
+
 ui.results.TestSelector = base.extends('div', {
     init: function(delegate, resultsByTest)
     {
@@ -197,11 +203,6 @@ ui.results.TestSelector = base.extends('div', {
         });
         $(this).accordion('activate', false);
 
-        // jQuery's builtin accordion overrides mousedown, which means you can't select the header text
-        // or click on the link to the flakiness dashboard.
-        $('.ui-accordion-header').live('click', function() {
-            $(this).trigger('customaccordianclick');
-        })
     },
     nextResult: function()
     {
index 7a9c2ba..c3f700f 100644 (file)
@@ -1,3 +1,11 @@
+2012-03-19  Ojan Vafai  <ojan@chromium.org>
+
+        We should only add a live event listener once, not once per TestSelector.
+        Adding one per TestSelector would cause accordion('option', 'active') to
+        incorrectly be "false" if you examine multiple sets of tests.
+
+        * BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/ui/results.js:
+
 2012-03-19  Dirk Pranke  <dpranke@chromium.org>
 
         Fix typo introduced in r111293.