DevTools: insert console message decorations in order

third_party/WebKit/Source/devtools/front_end/console/ConsoleContextSelector.js

 // Copyright 2015 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 {SDK.TargetManager.Observer}
  * @unrestricted
  */
 Console.ConsoleContextSelector = class {
   /**
+   * @param {!SDK.ExecutionContext} executionContext
+   * @param {boolean} formatForSelector
+   * @return {string}
+   */
+  static titleForContext(executionContext, formatForSelector) {
+    var result = executionContext.label(formatForSelector /* showStatus */) || executionContext.origin;
+    if (executionContext.isDefault && executionContext.frameId) {
+      var resourceTreeModel = SDK.ResourceTreeModel.fromTarget(executionContext.target());
+      var frame = resourceTreeModel && resourceTreeModel.frameForId(executionContext.frameId);
+      result = frame ? frame.displayName() : result;
+    }
+    result = executionContext.target().decorateLabel(result);

[luoe, 2017/01/19 23:22:59]

It turns out that browser extensions can load dedicated workers, in which case,
the execution context is not default, but we still want the decoration (gear
icon) consistent with other workers.

+
+    if (formatForSelector) {
+      if (!executionContext.isDefault)
+        result = '\u00a0\u00a0\u00a0\u00a0' + result;
+      result = result.trimMiddle(50);
+    }

[luoe, 2017/01/19 23:22:59]

Since the presentation logic for titles in the selector dropdown is in multiple
places, it was more convenient to use the flag passed in to decide whether to
apply this.

+    return result;
+  }
+
+  /**
    * @param {!Element} selectElement
    */
   constructor(selectElement) {
     this._selectElement = selectElement;
     /**
      * @type {!Map.<!SDK.ExecutionContext, !Element>}
      */
     this._optionByExecutionContext = new Map();
 
     SDK.targetManager.observeTargets(this);
     SDK.targetManager.addModelListener(
         SDK.RuntimeModel, SDK.RuntimeModel.Events.ExecutionContextCreated, this._onExecutionContextCreated, this);
     SDK.targetManager.addModelListener(
         SDK.RuntimeModel, SDK.RuntimeModel.Events.ExecutionContextChanged, this._onExecutionContextChanged, this);
     SDK.targetManager.addModelListener(
         SDK.RuntimeModel, SDK.RuntimeModel.Events.ExecutionContextDestroyed, this._onExecutionContextDestroyed, this);
 
     this._selectElement.addEventListener('change', this._executionContextChanged.bind(this), false);
     UI.context.addFlavorChangeListener(SDK.ExecutionContext, this._executionContextChangedExternally, this);
   }
 
   /**
    * @param {!SDK.ExecutionContext} executionContext
-   * @return {string}
-   */
-  _titleFor(executionContext) {
-    var result;
-    if (executionContext.isDefault) {
-      if (executionContext.frameId) {
-        var resourceTreeModel = SDK.ResourceTreeModel.fromTarget(executionContext.target());
-        var frame = resourceTreeModel && resourceTreeModel.frameForId(executionContext.frameId);
-        result = frame ? frame.displayName() : executionContext.label();
-      } else {
-        result = executionContext.target().decorateLabel(executionContext.label());
-      }
-    } else {
-      result = '\u00a0\u00a0\u00a0\u00a0' + (executionContext.label() || executionContext.origin);
-    }
-
-    var maxLength = 50;
-    return result.trimMiddle(maxLength);
-  }
-
-  /**
-   * @param {!SDK.ExecutionContext} executionContext
    */
   _executionContextCreated(executionContext) {
     // FIXME(413886): We never want to show execution context for the main thread of shadow page in service/shared worker frontend.
     // This check could be removed once we do not send this context to frontend.
     if (!executionContext.target().hasJSCapability())
       return;
 
     var newOption = createElement('option');
     newOption.__executionContext = executionContext;
-    newOption.text = this._titleFor(executionContext);
+    newOption.text = Console.ConsoleContextSelector.titleForContext(executionContext, true /* formatForSelector */);
     this._optionByExecutionContext.set(executionContext, newOption);
     var options = this._selectElement.options;
     var contexts = Array.prototype.map.call(options, mapping);
     var index = contexts.lowerBound(executionContext, executionContext.runtimeModel.executionContextComparator());
     this._selectElement.insertBefore(newOption, options[index]);
 
     if (executionContext === UI.context.flavor(SDK.ExecutionContext))
       this._select(newOption);
 
     /**
@@ skipping 14 matching lines @@
     this._updateSelectionWarning();
   }
 
   /**
    * @param {!Common.Event} event
    */
   _onExecutionContextChanged(event) {
     var executionContext = /** @type {!SDK.ExecutionContext} */ (event.data);
     var option = this._optionByExecutionContext.get(executionContext);
     if (option)
-      option.text = this._titleFor(executionContext);
+      option.text = Console.ConsoleContextSelector.titleForContext(executionContext, true /* formatForSelector */);
     this._updateSelectionWarning();
   }
 
   /**
    * @param {!SDK.ExecutionContext} executionContext
    */
   _executionContextDestroyed(executionContext) {
     var option = this._optionByExecutionContext.remove(executionContext);
     option.remove();
   }
@@ skipping 91 matching lines @@
 
   /**
    * @return {?Element}
    */
   _selectedOption() {
     if (this._selectElement.selectedIndex >= 0)
       return this._selectElement[this._selectElement.selectedIndex];
     return null;
   }
 };