Closed Bug 1709826 Opened 3 years ago Closed 3 years ago

browser_memory_fission_switch_target.js doesn't wait correctly for its first navigation

Categories

(DevTools :: Memory, defect, P1)

defect

Tracking

(firefox90 fixed)

RESOLVED FIXED
90 Branch
Tracking Status
firefox90 --- fixed

People

(Reporter: ochameau, Assigned: ochameau)

References

Details

Attachments

(1 file)

This code should use navigateTo helper in order to wait for the panel to have processed the navigation:
https://searchfox.org/mozilla-central/source/devtools/client/memory/test/browser/browser_memory_fission_switch_target.js#22-23

The very first navigation wasn't using navigateTo and that caused some exception
because we were trying to use previous target's memory front which was already destroyed.

I also removed this now uncessary fork of navigateTo.

Assignee: nobody → poirot.alex
Status: NEW → ASSIGNED
Severity: -- → S3
Priority: -- → P1
Pushed by apoirot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/eabdca06d2b0
[devtools] Wait for panel to have processed this first page navigation in browser_memory_fission_switch_target.js. r=nchevobbe
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: