CLeanup: remove unnecessary call to parseParameters.
authorjparent@chromium.org <jparent@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 14 Mar 2013 03:33:47 +0000 (03:33 +0000)
committerjparent@chromium.org <jparent@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 14 Mar 2013 03:33:47 +0000 (03:33 +0000)
https://bugs.webkit.org/show_bug.cgi?id=112309

Reviewed by Ojan Vafai.

We call parseCrossDashboardParameters when the dashboard initially
loads to set the state required for loadBuildersList, and then we
call it again once we are fully done with loading, so there is no
reason to call it in the middle of the loading process here.

* TestResultServer/static-dashboards/loader.js:
(.):

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

Tools/ChangeLog
Tools/TestResultServer/static-dashboards/loader.js

index 54dee0b..a0075bd 100644 (file)
@@ -1,3 +1,18 @@
+2013-03-13  Julie Parent  <jparent@chromium.org>
+
+        CLeanup: remove unnecessary call to parseParameters.
+        https://bugs.webkit.org/show_bug.cgi?id=112309
+
+        Reviewed by Ojan Vafai.
+        
+        We call parseCrossDashboardParameters when the dashboard initially
+        loads to set the state required for loadBuildersList, and then we
+        call it again once we are fully done with loading, so there is no
+        reason to call it in the middle of the loading process here.
+
+        * TestResultServer/static-dashboards/loader.js:
+        (.):
+
 2013-03-13  Elliott Sprehn  <esprehn@chromium.org>
 
         Unreviewed. Now I'm a reviewer :)
index 7686875..be018a5 100644 (file)
@@ -123,8 +123,6 @@ loader.Loader.prototype = {
     },
     _loadResultsFiles: function()
     {
-        parseParameters();
-
         for (var builderName in currentBuilders())
             this._loadResultsFileForBuilder(builderName);
     },