DevTools: Create extensible QuickOpen control

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 {
@@ skipping 584 matching lines @@
     if (uiSourceCode.contentType().isStyleSheet()) {
       Sources.StyleSheetOutlineDialog.show(uiSourceCode, this.showSourceLocation.bind(this, uiSourceCode));
       return true;
     }
 
     // We don't want default browser shortcut to be executed, so pretend to handle this event.
     return true;
   }
 
   /**
-   * @param {string=} query
-   */
-  showOpenResourceDialog(query) {
-    var uiSourceCodes = this._editorContainer.historyUISourceCodes();
-    /** @type {!Map.<!Workspace.UISourceCode, number>} */
-    var defaultScores = new Map();
-    for (var i = 1; i < uiSourceCodes.length; ++i)  // Skip current element
-      defaultScores.set(uiSourceCodes[i], uiSourceCodes.length - i);
-    if (!this._openResourceDialogHistory)
-      this._openResourceDialogHistory = [];
-    Sources.OpenResourceDialog.show(this, query || '', defaultScores, this._openResourceDialogHistory);
-  }
-
-  /**
    * @param {!Event=} event
    * @return {boolean}
    */
   _showGoToLineDialog(event) {

[pfeldman, 2017/02/07 02:10:03]

Extract this into an action in its own class (Sources.SourcesPanel.GoToHandler?)
where ever provider for ":" is.

[einbinder, 2017/02/28 23:59:07]

Would it use flavor(SourcesView) to check if the editor has a current file?

     if (this._editorContainer.currentFile())
-      this.showOpenResourceDialog(':');
+      QuickOpen.QuickOpen.show(':');
     return true;
   }
 
   /**
    * @return {boolean}
    */
   _save() {
     this._saveSourceFrame(this.currentSourceFrame());
     return true;
   }
@@ skipping 32 matching lines @@
     }
     return false;
   }
 
   /**
    * @param {boolean} active
    */
   toggleBreakpointsActiveState(active) {
     this._editorContainer.view.element.classList.toggle('breakpoints-deactivated', !active);
   }
+
+  /**
+   * @return {!Map.<!Workspace.UISourceCode, number>}
+   */
+  static defaultUISourceCodeScores() {

[pfeldman, 2017/02/07 02:10:03]

Declare static members first

[einbinder, 2017/02/28 23:59:07]

Done.

+    /** @type {!Map.<!Workspace.UISourceCode, number>} */
+    var defaultScores = new Map();
+    var sourcesView = UI.context.flavor(Sources.SourcesView);

[pfeldman, 2017/02/07 02:10:03]

Don't abuse context - all setters and getters of this flavor are in this class.
Just make it a singleton for now.

[einbinder, 2017/02/28 23:59:07]

A singleton feels weird because it operates on a visible SourcesView. But would
you want every user of SourcesView flavor to switch to the singleton, or just
this one? GoToLine uses it to get the current UISourceCode.

+    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);
+    }
+    return defaultScores;
+  }
 };
 
 /** @enum {symbol} */
 Sources.SourcesView.Events = {
   EditorClosed: Symbol('EditorClosed'),
   EditorSelected: Symbol('EditorSelected'),
 };
 
 /**
  * @interface
@@ skipping 79 matching lines @@
    * @return {boolean}
    */
   handleAction(context, actionId) {
     var sourcesView = UI.context.flavor(Sources.SourcesView);
     if (!sourcesView)
       return false;
     sourcesView._editorContainer.closeAllFiles();
     return true;
   }
 };