Dashboard Cleanup: Do not generate the page if we are about to reload.
authorjparent@chromium.org <jparent@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 10 Jan 2013 19:52:33 +0000 (19:52 +0000)
committerjparent@chromium.org <jparent@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 10 Jan 2013 19:52:33 +0000 (19:52 +0000)
commit075566106bad4b00c57589100d97a7c5d25e5b32
tree418af8fb75a78453016ab97a3781e22b59d459d7
parente528b36b986955e52ebacd54c9a0aa8f6f2a4974
Dashboard Cleanup: Do not generate the page if we are about to reload.
https://bugs.webkit.org/show_bug.cgi?id=106584

Reviewed by Dirk Pranke.

Move the logic to determine if we should generate the page into
parseParameters() since that is where we have the knowledge to
determine it, and return that value to handleLocationChange,
rather than passing the changed parameters back to handleLocationChange
and having it make the decision.

This is logically equivalent, save one case: when we are about to do
a reload, we know that there is no reason to generate the page.  Old
code was causing an extra page generation in this case.

* TestResultServer/static-dashboards/dashboard_base.js:
(parseParameters):
(handleLocationChange):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@139348 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Tools/ChangeLog
Tools/TestResultServer/static-dashboards/dashboard_base.js