Closed Bug 1897755 Opened 10 months ago Closed 6 months ago

Turn on and run debugger DAMP tests for Codemirror 6

Categories

(DevTools :: Debugger, task)

task

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bomsy, Assigned: bomsy)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

No description provided.
Assignee: nobody → hmanilla
Status: NEW → ASSIGNED

The Bugbug bot thinks this bug should belong to the 'DevTools::Debugger' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Component: General → Debugger
No longer blocks: 1773246

When dispatching mouseevents e.pageX/Y, e.screenX/Y are always not set. This patch removes its
usage to determine the token positions for toolip previews.

To test this, just make sure that tooltip breakpoints still works.

Attachment #9402733 - Attachment description: WIP: Bug 1897755 - [devtools] Run codemirror 6 on DAMP → Bug 1897755 - [devtools] Update DAMP tests to support Codemirror 6 r=#devtools-reviewers
Keywords: leave-open
Pushed by hmanilla@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9881aaa9dcd6 [devtools] Stop using e.pageX/pageY for mouse events r=devtools-reviewers,nchevobbe
Pushed by hmanilla@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/57b3111a5beb [devtools] Update DAMP tests to support Codemirror 6 r=perftest-reviewers,devtools-reviewers,nchevobbe,sparky
Keywords: leave-open
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: