DevTools: show alternate title onexpand of object in console

third_party/WebKit/Source/devtools/front_end/components/ObjectPropertiesSection.js

 /*
  * Copyright (C) 2008 Apple Inc. All Rights Reserved.
  * Copyright (C) 2009 Joseph Pecoraro
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 11 matching lines @@
  * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 /**
  * @constructor
  * @extends {TreeOutlineInShadow}
  * @param {!WebInspector.RemoteObject} object
  * @param {?string|!Element=} title
+ * @param {?Element=} expandedTitle
  * @param {!WebInspector.Linkifier=} linkifier
  * @param {?string=} emptyPlaceholder
  * @param {boolean=} ignoreHasOwnProperty
  * @param {!Array.<!WebInspector.RemoteObjectProperty>=} extraProperties
  */
-WebInspector.ObjectPropertiesSection = function(object, title, linkifier, emptyPlaceholder, ignoreHasOwnProperty, extraProperties)
+WebInspector.ObjectPropertiesSection = function(object, title, expandedTitle, linkifier, emptyPlaceholder, ignoreHasOwnProperty, extraProperties)

[lushnikov, 2016/07/20 23:54:26]

Let's look if we can reduce amount of arguments here before adding the new, 8th,
argument

There's a good chance that ObjectPropertiesSection could be simplified at first
before complicating it
even further

[luoe, 2016/07/22 22:30:58]

Good call, by moving needsAlternateTitle() into OPS, we can avoid adding in a
this argument.

The others seem to have a valid reason to be there, except maybe linkifier.  We
could have OPS expose a setLinkifier(linkifier) function, but that should be a
separate CL.

 {
     this._object = object;
     this._editable = true;
     TreeOutlineInShadow.call(this);
     this.hideOverflow();
     this.setFocusable(false);
     this._objectTreeElement = new WebInspector.ObjectPropertiesSection.RootElement(object, linkifier, emptyPlaceholder, ignoreHasOwnProperty, extraProperties);
     this.appendChild(this._objectTreeElement);
-    if (typeof title === "string" || !title)
-        this.element.createChild("span").textContent = title || "";
-    else
+    if (typeof title === "string" || !title) {
+        this.titleElement = this.element.createChild("span");
+        this.titleElement.textContent = title || "";
+    } else {
+        this.titleElement = title;
         this.element.appendChild(title);
+    }
+
+    if (expandedTitle)
+        this.expandedTitleElement = expandedTitle;
 
     this.element._section = this;
     this.registerRequiredCSS("components/objectValue.css");
     this.registerRequiredCSS("components/objectPropertiesSection.css");
     this.rootElement().childrenListElement.classList.add("source-code", "object-properties-section");
 }
 
 /** @const */
 WebInspector.ObjectPropertiesSection._arrayLoadThreshold = 100;
 
 /**
  * @param {!WebInspector.RemoteObject} object
  * @param {!WebInspector.Linkifier=} linkifier
  * @param {boolean=} skipProto
  * @return {!Element}
  */
 WebInspector.ObjectPropertiesSection.defaultObjectPresentation = function(object, linkifier, skipProto)
 {
     var componentRoot = createElementWithClass("span", "source-code");
     var shadowRoot = WebInspector.createShadowRootWithCoreStyles(componentRoot, "components/objectValue.css");
     shadowRoot.appendChild(WebInspector.ObjectPropertiesSection.createValueElement(object, false));
     if (!object.hasChildren)
         return componentRoot;
 
-    var objectPropertiesSection = new WebInspector.ObjectPropertiesSection(object, componentRoot, linkifier);
+    var objectPropertiesSection = new WebInspector.ObjectPropertiesSection(object, componentRoot, null, linkifier);
     objectPropertiesSection.editable = false;
     if (skipProto)
         objectPropertiesSection.skipProto();
 
     return objectPropertiesSection.element;
 }
 
 WebInspector.ObjectPropertiesSection.prototype = {
     skipProto: function()
     {
@@ skipping 82 matching lines @@
     TreeElement.call(this, contentElement);
     this.setExpandable(true);
     this.selectable = false;
     this.toggleOnClick = true;
     this.listItemElement.classList.add("object-properties-section-root-element");
     this._linkifier = linkifier;
 }
 
 WebInspector.ObjectPropertiesSection.RootElement.prototype = {
 
+    /**
+     * @override
+     */
     onexpand: function()
     {
-        if (this.treeOutline)
+        if (this.treeOutline) {
             this.treeOutline.element.classList.add("expanded");
-    },
-
-    oncollapse: function()
-    {
-        if (this.treeOutline)
-            this.treeOutline.element.classList.remove("expanded");
+            if (this.treeOutline.expandedTitleElement)
+                this.treeOutline.titleElement.swapChildren(this.treeOutline.expandedTitleElement);
+        }
     },
 
     /**
+     * @override
+     */
+    oncollapse: function()
+    {
+        if (this.treeOutline) {
+            this.treeOutline.element.classList.remove("expanded");
+            if (this.treeOutline.expandedTitleElement)
+                this.treeOutline.titleElement.swapChildren(this.treeOutline.expandedTitleElement);
+        }
+    },
+
+    /**
      * @override
      * @param {!Event} e
      * @return {boolean}
      */
     ondblclick: function(e)
     {
         return true;
     },
 
     onpopulate: function()
@@ skipping 76 matching lines @@
         var targetValue = this.property.name !== "__proto__" ? propertyValue : this.property.parentObject;
         WebInspector.ObjectPropertyTreeElement._populate(this, propertyValue, skipProto, this._linkifier, undefined, undefined, undefined, targetValue);
     },
 
     /**
      * @override
      * @return {boolean}
      */
     ondblclick: function(event)
     {
-        var editableElement = this.valueElement;
-        if (!this.property.value.customPreview() && (this.property.writable || this.property.setter) && event.target.isSelfOrDescendant(editableElement))
+        var inEditableElement = event.target.isSelfOrDescendant(this.valueElement) || (this.expandedValueElement && event.target.isSelfOrDescendant(this.expandedValueElement));
+        if (!this.property.value.customPreview() && inEditableElement && (this.property.writable || this.property.setter))
             this._startEditing();
         return false;
     },
 
     /**
      * @override
      */
     onattach: function()
     {
         this.update();
         this._updateExpandable();
     },
 
+    /**
+     * @override
+     */
+    onexpand: function()
+    {
+        if (this.expandedValueElement)
+            this.valueElement.swapChildren(this.expandedValueElement);
+    },
+
+    /**
+     * @override
+     */
+    oncollapse: function()
+    {
+        if (this.expandedValueElement)
+            this.valueElement.swapChildren(this.expandedValueElement);
+    },
+
     update: function()
     {
         this.nameElement = WebInspector.ObjectPropertiesSection.createNameElement(this.property.name);
         if (!this.property.enumerable)
             this.nameElement.classList.add("object-properties-section-dimmed");
         if (this.property.isAccessorProperty())
             this.nameElement.classList.add("properties-accessor-property-name");
         if (this.property.synthetic)
             this.nameElement.classList.add("synthetic-property");
 
         this._updatePropertyPath();
         this.nameElement.addEventListener("contextmenu", this._contextMenuFired.bind(this, this.property), false);
 
         var separatorElement = createElementWithClass("span", "object-properties-section-separator");
         separatorElement.textContent = ": ";
 
         if (this.property.value) {
             this.valueElement = WebInspector.ObjectPropertiesSection.createValueElementWithCustomSupport(this.property.value, this.property.wasThrown, this.listItemElement, this._linkifier);
             this.valueElement.addEventListener("contextmenu", this._contextMenuFired.bind(this, this.property), false);
         } else if (this.property.getter) {
             this.valueElement = WebInspector.ObjectPropertyTreeElement.createRemoteObjectAccessorPropertySpan(this.property.parentObject, [this.property.name], this._onInvokeGetterClick.bind(this));
         } else {
             this.valueElement = createElementWithClass("span", "object-value-undefined");
             this.valueElement.textContent = WebInspector.UIString("<unreadable>");
             this.valueElement.title = WebInspector.UIString("No property getter");
         }
 
+        var valueText = this.valueElement.textContent;
+        if (this.property.value && valueText && !this.property.wasThrown)
+            this.expandedValueElement = WebInspector.ObjectPropertiesSection.createExpandableValueElement(this.property.value);
+
         this.listItemElement.removeChildren();
         this.listItemElement.appendChildren(this.nameElement, separatorElement, this.valueElement);
     },
 
     _updatePropertyPath: function()
     {
         if (this.nameElement.title)
             return;
 
         var useDotNotation = /^(_|\$|[A-Z])(_|\$|[A-Z]|\d)*$/i;
@@ skipping 31 matching lines @@
 
         this._editableDiv = this.listItemElement.createChild("span");
 
         var text = this.property.value.description;
         if (this.property.value.type === "string" && typeof text === "string")
             text = "\"" + text + "\"";
 
         this._editableDiv.setTextContentTruncatedIfNeeded(text, WebInspector.UIString("<string is too large to edit>"));
         var originalContent = this._editableDiv.textContent;
 
-        this.valueElement.classList.add("hidden");
-
         // Lie about our children to prevent expanding on double click and to collapse subproperties.
         this.setExpandable(false);
         this.listItemElement.classList.add("editing-sub-part");
+        this.valueElement.classList.add("hidden");
 
         this._prompt = new WebInspector.ObjectPropertyPrompt();
 
         var proxyElement = this._prompt.attachAndStartEditing(this._editableDiv, this._editingCommitted.bind(this, originalContent));
         this.listItemElement.getComponentSelection().setBaseAndExtent(this._editableDiv, 0, this._editableDiv, 1);
         proxyElement.addEventListener("keydown", this._promptKeyDown.bind(this, originalContent), false);
     },
 
     _editingEnded: function()
     {
@@ skipping 67 matching lines @@
                 return;
             }
 
             if (!expression) {
                 // The property was deleted, so remove this tree element.
                 this.parent.removeChild(this);
             } else {
                 // Call updateSiblings since their value might be based on the value that just changed.
                 var parent = this.parent;
                 parent.invalidateChildren();
-                parent.expand();
+                parent.onpopulate();
             }
         };
     },
 
     /**
      * @param {?WebInspector.RemoteObject} result
      * @param {boolean=} wasThrown
      */
     _onInvokeGetterClick: function(result, wasThrown)
     {
@@ skipping 729 matching lines @@
     function mouseClick(event)
     {
         WebInspector.Revealer.reveal(value);
         event.consume(true);
     }
 
     return valueElement;
 }
 
 /**
+ * @param {!WebInspector.RemoteObject} value
+ * @return {boolean}
+ */
+WebInspector.ObjectPropertiesSection.needsAlternateTitle = function(value)
+{
+    return value && !value.customPreview() && value.hasChildren && value.subtype !== "node" && value.type !== "function";
+}
+
+/**
+ * @param {!WebInspector.RemoteObject} value
+ * @return {?Element}
+ */
+WebInspector.ObjectPropertiesSection.createExpandableValueElement = function(value)
+{
+    if (!WebInspector.ObjectPropertiesSection.needsAlternateTitle(value))
+        return null;
+
+    var valueElement = createElementWithClass("span", "value");
+    valueElement.setTextContentTruncatedIfNeeded(value.description || "");
+    valueElement.classList.add("object-value-" + (value.subtype || value.type));
+    valueElement.title = value.description || "";
+    return valueElement;
+}
+
+/**
  * @param {!WebInspector.RemoteObject} func
  * @param {!Element} element
  * @param {boolean} linkify
  * @param {boolean=} includePreview
  */
 WebInspector.ObjectPropertiesSection.formatObjectAsFunction = function(func, element, linkify, includePreview)
 {
     func.functionDetails(didGetDetails);
 
     /**
@@ skipping 153 matching lines @@
 
             result = currentName + (result ? "." + result : "");
             current = current.parent;
         }
         var treeOutlineId = treeElement.treeOutline[WebInspector.ObjectPropertiesSectionExpandController._treeOutlineId];
         result = treeOutlineId + (result ? ":" + result : "");
         treeElement[WebInspector.ObjectPropertiesSectionExpandController._cachedPathSymbol] = result;
         return result;
     }
 }