DevTools: add entry points for command menu

third_party/WebKit/Source/devtools/front_end/sources/SourcesView.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 /**
  * @implements {Sources.TabbedEditorContainerDelegate}
  * @implements {UI.Searchable}
  * @implements {UI.Replaceable}
  * @unrestricted
  */
 Sources.SourcesView = class extends UI.VBox {
   /**
    * @suppressGlobalPropertiesCheck
    */
   constructor() {
     super();
     this.registerRequiredCSS('sources/sourcesView.css');
     this.element.id = 'sources-panel-sources-view';
     this.setMinimumAndPreferredSizes(80, 52, 150, 100);
 
     var workspace = Workspace.workspace;
 
     this._searchableView = new UI.SearchableView(this, 'sourcesViewSearchConfig');
     this._searchableView.setMinimalSearchQuerySize(0);
     this._searchableView.show(this.element);
 
     /** @type {!Map.<!Workspace.UISourceCode, !UI.Widget>} */
     this._sourceViewByUISourceCode = new Map();
 
-    var tabbedEditorPlaceholderText =
-        Host.isMac() ? Common.UIString('Hit \u2318+P to open a file') : Common.UIString('Hit Ctrl+P to open a file');
-    if (Runtime.experiments.isEnabled('persistence2'))
-      tabbedEditorPlaceholderText += '\n\n' + Common.UIString('Drop in a folder to add to workspace');
     this._editorContainer = new Sources.TabbedEditorContainer(
-        this, Common.settings.createLocalSetting('previouslyViewedFiles', []), tabbedEditorPlaceholderText);
+        this, Common.settings.createLocalSetting('previouslyViewedFiles', []), this._placeholderElement());
     this._editorContainer.show(this._searchableView.element);
     this._editorContainer.addEventListener(
         Sources.TabbedEditorContainer.Events.EditorSelected, this._editorSelected, this);
     this._editorContainer.addEventListener(Sources.TabbedEditorContainer.Events.EditorClosed, this._editorClosed, this);
 
     this._historyManager = new Sources.EditingLocationHistoryManager(this, this.currentSourceFrame.bind(this));
 
     this._toolbarContainerElement = this.element.createChild('div', 'sources-toolbar');
     this._toolbarEditorActions = new UI.Toolbar('', this._toolbarContainerElement);
 
@@ skipping 43 matching lines @@
     }
 
     if (!window.opener)
       window.addEventListener('beforeunload', handleBeforeUnload, true);
 
     this._shortcuts = {};
     this.element.addEventListener('keydown', this._handleKeyDown.bind(this), false);
   }
 
   /**
+   * @return {!Element}
+   */
+  _placeholderElement() {
+    var shortcuts = [
+      {
+        defaultShortcut: Common.UIString('Ctrl+P'),

[pfeldman, 2017/04/24 18:29:25]

Since shortcuts are defined in quickopen and could be changed, you should do:

UI.shortcutRegistry.shortcutTitleForAction('') instead. Hardcoding id here seems
necessary.

[luoe, 2017/04/25 23:19:52]

Done.

+        macShortcut: Common.UIString('\u2318+P'),
+        description: Common.UIString('Open a file')
+      },
+      {
+        defaultShortcut: Common.UIString('Ctrl+Shift+P'),
+        macShortcut: Common.UIString('\u2318+Shift+P'),
+        description: Common.UIString('Run a command')
+      }
+    ];
+
+    var element = createElementWithClass('span', 'tabbed-pane-placeholder');
+    for (var shortcut of shortcuts) {
+      var shortcutKeyText = Host.isMac() ? shortcut.macShortcut : shortcut.defaultShortcut;
+      var row = element.createChild('div', 'tabbed-pane-placeholder-row');
+      row.createChild('div', 'tabbed-pane-placeholder-key').textContent = shortcutKeyText;
+      row.createChild('div', 'tabbed-pane-placeholder-value').textContent = shortcut.description;
+    }
+    if (Runtime.experiments.isEnabled('persistence2'))
+      element.createChild('div').textContent = Common.UIString('Drop in a folder to add to workspace');
+
+    return element;
+  }
+
+  /**
    * @return {!Map.<!Workspace.UISourceCode, number>}
    */
   static defaultUISourceCodeScores() {
     /** @type {!Map.<!Workspace.UISourceCode, number>} */
     var defaultScores = new Map();
     var sourcesView = UI.context.flavor(Sources.SourcesView);
     if (sourcesView) {
       var uiSourceCodes = sourcesView._editorContainer.historyUISourceCodes();
       for (var i = 1; i < uiSourceCodes.length; ++i)  // Skip current element
         defaultScores.set(uiSourceCodes[i], uiSourceCodes.length - i);
@@ skipping 647 matching lines @@
    * @return {boolean}
    */
   handleAction(context, actionId) {
     var sourcesView = UI.context.flavor(Sources.SourcesView);
     if (!sourcesView)
       return false;
     sourcesView._editorContainer.closeAllFiles();
     return true;
   }
 };