Closed Bug 1706490 Opened 4 years ago Closed 4 years ago

Make browser_webconsole_network_messages_status_code.js work with native context menus

Categories

(Core :: Widget: Cocoa, task)

task
Points:
2

Tracking

()

RESOLVED FIXED
90 Branch
Iteration:
90.1 - Apr 19 - May 2
Tracking Status
firefox89 --- fixed
firefox90 --- fixed

People

(Reporter: bugzilla, Assigned: bugzilla)

References

Details

(Whiteboard: [proton-uplift])

Attachments

(1 file)

This test seems to be testing useless behaviour: a right Cmd click opening a context menu. Reading bug 1421213, where the test was introduced, it looks like it's the result of a miscommunication in the review process where the reviewer wanted to make sure that Ctrl+clicking on Mac also opened the context menu. The window object here also appears to be the hidden window. Which is all to say: the test is busted as-is. Even without native context menus, you can see it opens context menus over context menus, far away from the node its supposed to be clicking on. It seems to pass now sort of accidentally, with a subsequent test accidentally cleaning it up.

I'm going to remove the right Cmd+click case, and clean up the test.

The openContextMenu() helper does the same thing this test does now: dispatches a MouseEvent to a specified node. Using it allows us to clean up the test considerably.

Pushed by htwyford@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/30af39798ccf Make browser_webconsole_network_messages_status_code.js work with native context menus. r=mac-reviewers,mstange
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
Whiteboard: [proton-uplift]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: