Closed Bug 1578979 Opened 5 years ago Closed 5 years ago

Fix test that are intermittents when Toolbox is in content frame

Categories

(DevTools :: Framework, task, P1)

task

Tracking

(firefox71 fixed)

RESOLVED FIXED
Firefox 71
Tracking Status
firefox71 --- fixed

People

(Reporter: jdescottes, Assigned: jdescottes)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

(Whiteboard: dt-fission-m1)

Attachments

(2 files)

Tests that force devtools.toolbox.content-frame to false:

  • devtools/client/webconsole/test/mochitest/browser_jsterm_screenshot_command_clipboard.js
  • devtools/client/shared/test/browser_html_tooltip_consecutive-show.js
  • devtools/client/shared/test/browser_treeWidget_keyboard_interaction.js
  • devtools/client/shared/test/browser_treeWidget_mouse_interaction.js

They should be fixed so that we can remove the preference.

Blocks: 1578980
Whiteboard: dt-fission → dt-fission-reserve
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Priority: P3 → P1

Even though we still need to fix those awaits, this removes the need to use the pref in 3 tests.
We should ultimately investigate why our switchHost/createHost can sometimes resolve too early and make the tests fail.

Not exactly sure why this fixes the issue.
Whether we run with a content frame or not, if you run the test on a slow platform,
and if you run the fullpage test a lot of times, you will sometimes see the toolbox become huge very briefly.

When the toolbox runs in a content frame, the screenshot will then not match the content page size.
I will investigate a bit more to get a better understanding, but in the meantime this should fix the issue.

Whiteboard: dt-fission-reserve → dt-fission
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4f9a8eba6f73 Wait after createHost to fix tests failing with devtools running in a content frame r=ochameau https://hg.mozilla.org/integration/autoland/rev/9529fc074e39 Fix fullpage screenshot test when devtools run in a content frame r=nchevobbe
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 71
Whiteboard: dt-fission → dt-fission dt-fission-m1
Whiteboard: dt-fission dt-fission-m1 → dt-fission-m1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: