Summary: | Extend perf dashboard to support multiple pages. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | dewei_zhu | ||||||
Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, dewei_zhu, rniwa | ||||||
Priority: | P2 | ||||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
dewei_zhu
2016-09-26 22:26:27 PDT
Created attachment 289916 [details]
Patch
Comment on attachment 289916 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=289916&action=review > Websites/perf.webkit.org/public/v3/main.js:60 > else if (dashboardPages) There is a bug here. We should be checking dashboardPages.length here as well. > Websites/perf.webkit.org/public/v3/pages/summary-page.js:30 > - routeName() { return 'summary'; } > + routeName() { return this._route; } We should use `summary/${this._name}` instead just like we do in DashboardPage. Created attachment 290007 [details]
Patch for landing
Comment on attachment 290007 [details] Patch for landing Clearing flags on attachment: 290007 Committed r206465: <http://trac.webkit.org/changeset/206465> All reviewed patches have been landed. Closing bug. |