Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(11)

Issue 1654813003: Remove old dead perf dashboard pages and js (Closed)

Created:
4 years, 10 months ago by agable
Modified:
4 years, 10 months ago
Reviewers:
qyearsley
CC:
chromium-reviews, cmp-cc_chromium.org, ilevy-cc_chromium.org
Project:
tools
Visibility:
Public.

Description

Remove old dead perf dashboard pages and js The below pages have historically been served on build.chromium.org/f/chromium/perf/dashboard/ui/*. This was poorly known, and even more poorly maintained. I sent out a survey[1] asking which pages under that path were still used. The ones being removed here are no longer used, and mostly broken to boot. [1] https://docs.google.com/a/chromium.org/forms/d/1f1yx-_WmMfjVqtsPylzHUWn8Q8k6e7UWAVEz8KrvKKA/viewanalytics R=qyearsley@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=298507

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -5514 lines) Patch
D perf/dashboard/ui/chrome_report.html View 1 chunk +0 lines, -202 lines 0 comments Download
D perf/dashboard/ui/details.html View 1 chunk +0 lines, -175 lines 0 comments Download
D perf/dashboard/ui/endure_js/common.js View 1 chunk +0 lines, -87 lines 0 comments Download
D perf/dashboard/ui/endure_js/coordinates.js View 1 chunk +0 lines, -196 lines 0 comments Download
D perf/dashboard/ui/endure_js/dom_utils.js View 1 chunk +0 lines, -43 lines 0 comments Download
D perf/dashboard/ui/endure_js/endure_plotter.js View 1 chunk +0 lines, -553 lines 0 comments Download
D perf/dashboard/ui/endure_js/graph_utils.js View 1 chunk +0 lines, -141 lines 0 comments Download
D perf/dashboard/ui/endure_js/plotter.js View 1 chunk +0 lines, -1199 lines 0 comments Download
D perf/dashboard/ui/endure_plotter.html View 1 chunk +0 lines, -116 lines 0 comments Download
D perf/dashboard/ui/generic_plotter.html View 1 chunk +0 lines, -533 lines 0 comments Download
D perf/dashboard/ui/js/coordinates.js View 1 chunk +0 lines, -136 lines 0 comments Download
D perf/dashboard/ui/js/graph.js View 1 chunk +0 lines, -645 lines 0 comments Download
D perf/dashboard/ui/js/plotter.js View 1 chunk +0 lines, -608 lines 0 comments Download
D perf/dashboard/ui/pagecycler_report.html View 1 chunk +0 lines, -260 lines 0 comments Download
D perf/dashboard/ui/playback_report.html View 1 chunk +0 lines, -341 lines 0 comments Download
D perf/dashboard/ui/sunspider_report.html View 1 chunk +0 lines, -279 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
agable
Oh man, I had to upload this change with `gcl`. That's been a while. PTAL!
4 years, 10 months ago (2016-02-02 00:07:20 UTC) #2
qyearsley
On 2016/02/02 at 00:07:20, agable wrote: > Oh man, I had to upload this change ...
4 years, 10 months ago (2016-02-02 00:20:12 UTC) #3
agable
4 years, 10 months ago (2016-02-02 01:15:25 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as r298507 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698