Closed Bug 1173695 Opened 5 years ago Closed 5 years ago

Unable to import specific profiles which cause an error to be thrown by recording-utils.js:303:6

Categories

(DevTools :: Performance Tools (Profiler/Timeline), defect, P1)

40 Branch
defect

Tracking

(firefox40 affected, firefox41 verified)

VERIFIED FIXED
Firefox 41
Tracking Status
firefox40 --- affected
firefox41 --- verified

People

(Reporter: avaida, Assigned: jsantell)

References

Details

Attachments

(2 files)

Attached file profile
NOTE: this is a follow-up issue from Bug 1164130.

Reproducible on:
* Aurora 40.0a2 (2015-06-10)
* Nightly 41.0a1 (2015-06-10)

Affected platforms:
Windows 7 (x64), Mac OS X 10.9.5 and Ubuntu 14.04 (x64)

Steps to reproduce:
1. Launch Firefox.
2. Open the Performance tools.
3. Import the attached profile.
4. Check the Browser Console.

Expected result:
* The profile is successfully imported.
* There are no errors thrown in the Browser Console for this action.

Actual result:
The profile cannot be imported and the following error gets thrown in the Browser Console by recording-utils.js:303:6:
> TypeError: samples is undefined

Additional notes:
* This is the only profile sample I have that reproduces the issue, I'm not sure what went wrong but trying to import a 2nd profile made from the same browser version (Release 37.0.2) was successful.
Priority: -- → P1
Assignee: nobody → jsantell
Status: NEW → ASSIGNED
Not sure what causes this. But this recording is a v2 recording, and also has deflated profiler data. An extra thread is a string, and contains additional threads, like it's a full other recording. Anyways, this protects against that
Attachment #8621399 - Flags: review?(vporof)
Comment on attachment 8621399 [details] [diff] [review]
1173695-import-failure.patch

Review of attachment 8621399 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/devtools/performance/modules/logic/recording-utils.js
@@ +259,5 @@
>   */
>  function deflateProfile(profile) {
>    profile.threads = profile.threads.map((thread) => {
>      let uniqueStacks = new UniqueStacks();
> +

Nit: useless newline.
Attachment #8621399 - Flags: review?(vporof) → review+
https://hg.mozilla.org/mozilla-central/rev/10f378449181
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
Verified fixed on Nightly 41.0a1 (2015-06-23), using Windows 8.1 (x64), Ubuntu 14.04 (x86) and Mac OS X 10.9.5.
Status: RESOLVED → VERIFIED
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.