Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe] Reporter
	Description • 10 years ago

We need to check how often the tools are opened using the context menu (bug 1076932 - remove Inspect Element from context menu for first time users). We need to add:

* DEVTOOLS_FIRST_LAUNCHED_VIA_CONTEXT_MENU_FLAG

* DEVTOOLS_FIRST_LAUNCHED_VIA_KEYBOARD_SHORTCUT

* DEVTOOLS_FIRST_LAUNCHED_VIA_DEVTOOLS_ICON_FLAG

* DEVTOOLS_FIRST_LAUNCHED_VIA_DEVELOPER_TOOLBAR_FLAG

If loaded via context menu we need:
DEVTOOLS_TIME_OPEN_AFTER_LAUNCHED_VIA_CONTEXT_MENU_EXPONENTIAL

=============================

I made a start on this as part of bug 1046234. Here are the code locations used by each launch type:
Context menu - nsContextMenu.js:512
  `inspectNode: function CM_inspectNode() {`

Keyboard Shortcut - gDevTools.jsm:708
  `selectToolCommand: function(gBrowser, toolId) {`

DevTools Icon - panelUI.js:302
  `showSubView: function(aViewId, aAnchor, aPlacementArea) {`

Developer Toolbar Icon - gDevTools.jsm:586
  `toggleToolboxCommand: function(gBrowser) {`

	Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe] Reporter
	Comment 1 • 9 years ago

Jeff, do you still want this?

	Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe] Reporter
	Comment 2 • 9 years ago

Filter on 1ff0543e-b501-4893-a72b-e4773c01e655

	Jeff Griffiths (:canuckistani) (:⚡︎)
	Comment 3 • 9 years ago

(In reply to Michael Ratcliffe [:miker] [:mratcliffe] from comment #1)
> Jeff, do you still want this?

Yes.