Closed Bug 1544709 Opened 2 years ago Closed 2 years ago

Global DevTools shortcuts are broken when running in frame with type=content


(DevTools :: General, enhancement, P2)



(firefox68 fixed)

Firefox 68
Tracking Status
firefox68 --- fixed


(Reporter: jdescottes, Assigned: jdescottes)




(3 files, 1 obsolete file)

Split from Bug 1543940.

Fixing keyboard shortcuts in frames with type=content can be done independently of the work on context menu.


  • set content="type" for devtools frames (in toolbox-hosts::createDevToolsFrame() add frame.setAttribute("type", "content");)
  • some shortcuts are no longer working. eg:
    • select a node in markup-view
    • press search shortcut (cmd+F on osx)
      ER: Inspector search is focused
      AR: Browser search is started and focused

We need to rely on the chromeEventHandler from the toolbox to fix this.

Depends on D27672

Will be used both for shortcuts and contextmenu events in the toolbox.

Blocks: 1543940
Depends on: 1544749

Depends on D27683
As we will move shortcuts to use chromeEventHandler as their target, we need an easy way to add and remove them on host switch.
The chromeEventHandler will change when switching from window to docked hosts (and vice versa)

Depends on D27686
Toolbox, zoom keys and inspector search should use the chromeeventhandler as event target.
They need to capture events from inner frames.

Type: defect → enhancement
Attachment #9058570 - Attachment is obsolete: true
Pushed by
Move all shortcuts registration for toolbox in addShortcuts method r=ochameau
Use chromeEventHandler for splitConsole events r=ochameau
Register global keyboard shortcuts on chromeEventHandler r=ochameau
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 68
You need to log in before you can comment on or make changes to this bug.