Allow range to be specified using dates.

Allow range to be specified using dates.

e.g. range={2008-08-25}:{2008-08-26}

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=270306

[reveman, 2014-04-25 21:40:30]

This would be really useful in a chrome extension I'm writing. Wdyt?

[cmp, 2014-04-25 21:43:18]

+mmoss, agable, iannucci who can review

[reveman, 2014-04-25 21:46:28]

https://codereview.chromium.org/251013003/diff/1/dashboard/svn-log
File dashboard/svn-log (right):

https://codereview.chromium.org/251013003/diff/1/dashboard/svn-log#newcode20
dashboard/svn-log:20: ':{(\d{4}-(1[0-2]|0?[1-9])-(3[01]|[12][0-9]|0?[1-9]))}$',
fyi, this is not perfect. ie. it doesn't complain about 1980-02-31

[agable, 2014-04-26 18:43:45]

Is trunk/tools/perf/dashboard even used anymore? I thought all the perf
dashboard code was in google3 now. +qyearsley to confirm.

[qyearsley, 2014-04-27 03:09:52]

On 2014/04/26 18:43:45, agable wrote:
> Is trunk/tools/perf/dashboard even used anymore? I thought all the perf
> dashboard code was in google3 now. +qyearsley to confirm.

Yep, tools/perf/dashboard/ui/changelog.html (and the other changelog_*.html
files) are used by the new perf dashboard (chromeperf.appspot.com). When you
click a point on the graph, an iframe is put on the page to show the changelog,
which is loaded from build.chromium.org. e.g.:

https://chromeperf.appspot.com/revision_proxy?url_to_load=http%3A%2F%2Fbuild....

I believe that the main reason why this wasn't moved to the new dashboard is
because it it uses this svn-log script which runs "svn log"; the new dashboard
can't do this directly since it's an App Engine app.

However! AFAIK, some of the contents of tools/perf/dashboard/ui is not used by
anything any more, including:
 - *_report.html
 - *_plotter.html
 - endure_js/

[qyearsley, 2014-05-13 23:48:17]

By the way, this CL LGTM.

[reveman, 2014-05-14 02:56:30]

The CQ bit was checked by reveman@chromium.org

[commit-bot: I haz the power, 2014-05-14 02:56:42]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/reveman@chromium.org/251013003/1

[commit-bot: I haz the power, 2014-05-14 02:56:47]

Change committed as 270306