Commit d435037c authored by aroben@apple.com's avatar aroben@apple.com

Strip off /results.html from results URLs before trying to load leaks files from them

r82734 changed build.webkit.org's "view results" URLs to point straight to the results.html
files, rather than pointing to the directory that contains them. This is more convenient for
people browsing build.webkit.org, but confused Leaks Viewer.

Fixes <http://webkit.org/b/57869> REGRESSION (r82734): Links in Leaks Viewer's Recent Builds
list don't work

Reviewed by Joseph Pecoraro.

* BuildSlaveSupport/build.webkit.org-config/public_html/LeaksViewer/RecentBuildsLoader.js:
(RecentBuildsLoader.prototype.start): Strip off "/results.html" from the results URL.

git-svn-id: svn://svn.chromium.org/blink/trunk@82967 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent 5c29ddca
......@@ -61,7 +61,8 @@ RecentBuildsLoader.prototype = {
return;
if (!("view results" in step.urls))
return;
buildInfo.url = self._buildbotBaseURL + step.urls["view results"] + "/";
var url = step.urls["view results"];
buildInfo.url = self._buildbotBaseURL + url.replace(/\/results\.html$/, "") + "/";
}
if (buildInfo.leakCount && buildInfo.url) {
......
2011-04-05 Adam Roben <aroben@apple.com>
Strip off /results.html from results URLs before trying to load leaks files from them
r82734 changed build.webkit.org's "view results" URLs to point straight to the results.html
files, rather than pointing to the directory that contains them. This is more convenient for
people browsing build.webkit.org, but confused Leaks Viewer.
Fixes <http://webkit.org/b/57869> REGRESSION (r82734): Links in Leaks Viewer's Recent Builds
list don't work
Reviewed by Joseph Pecoraro.
* BuildSlaveSupport/build.webkit.org-config/public_html/LeaksViewer/RecentBuildsLoader.js:
(RecentBuildsLoader.prototype.start): Strip off "/results.html" from the results URL.
2011-04-05 Carol Szabo <carol@webkit.org>
Unreviewed.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment