Closed Bug 1164130 Opened 5 years ago Closed 5 years ago
[regression] Importing profiler v2 fails as Recording
Utils not included in io .js
No description provided.
via Bug 1154115
Assignee: nobody → jsantell
Status: NEW → ASSIGNED
Attachment #8604858 - Flags: review?(vporof)
Comment on attachment 8604858 [details] [diff] [review] 1164130-fix-import.patch Review of attachment 8604858 [details] [diff] [review]: ----------------------------------------------------------------- amaze
Attachment #8604858 - Flags: review?(vporof) → review+
remote: https://hg.mozilla.org/integration/fx-team/rev/7f901174dd44 remote: https://hg.mozilla.org/integration/fx-team/rev/32ad078a4b5f remote: https://hg.mozilla.org/integration/fx-team/rev/914dda9f4a1a
Whiteboard: [devedition-40] → [devedition-40][fixed-in-fx-team]
ni? jsantell for STR
STR: Import a profile (version 2) in the new performance tool. (To create a profile of this version, use Firefox 39 or below and record a profile, and save that). Expected Results: Profile loads correctly, no errors thrown, and the call tree and framerate graphs are displayed.
Comment on attachment 8604858 [details] [diff] [review] 1164130-fix-import.patch Approval Request Comment [Feature/regressing bug #]: 1167252, the new performance tool [User impact if declined]: Won't ship the performance tool [Describe test coverage new/current, TreeHerder]: There are try pushes in Bug 1167252 with all patches needing uplift [Risks and why]: Requesting uplift for the accumulated changes in the performance tool since the 40 merge date, so these changes haven't had the full 6 weeks to bake. Risks are generally contained within devtools, specifically within the performance panel. [String/UUID change made/needed]: None
Attachment #8604858 - Flags: approval-mozilla-aurora?
Note: I had verbal confirmation for these uplifts from Sylvestre even before he's flagged them as a+. See https://bugzilla.mozilla.org/show_bug.cgi?id=1167252#c26
Comment on attachment 8604858 [details] [diff] [review] 1164130-fix-import.patch Change approved to skip one train as part of the spring campaign.
Attachment #8604858 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
With the exception of Bug 1173695, this is verified fixed on on Aurora 40.0a2 (2015-06-10), using Windows 7 (x64), Ubuntu 14.04 (x64) and Mac OS X 10.9.5. The user is able to successfully import v2 profiles in the new Performance tool, which will only display the "Call Tree" and "Flame Chart" detailed views for them. Tested with Nightly 39.0a1 (2015-03-22) and Release 37.0.2.
Whiteboard: [devedition-40] → [polish-backlog]
You need to log in before you can comment on or make changes to this bug.