DevTools: minor cleanup of ConsoleViewMessage, use switch for formatters

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

 /*
  * Copyright (C) 2011 Google Inc.  All rights reserved.
  * Copyright (C) 2007, 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
@@ skipping 27 matching lines @@
 WebInspector.ConsoleViewMessage = function(consoleMessage, linkifier, nestingLevel)
 {
     this._message = consoleMessage;
     this._linkifier = linkifier;
     this._repeatCount = 1;
     this._closeGroupDecorationCount = 0;
     this._nestingLevel = nestingLevel;
 
     /** @type {?WebInspector.DataGrid} */
     this._dataGrid = null;
-
-    /** @type {!Object.<string, function(!WebInspector.RemoteObject, !Element, boolean=)>} */
-    this._customFormatters = {
-        "array": this._formatParameterAsArray,
-        "typedarray": this._formatParameterAsArray,
-        "error": this._formatParameterAsError,
-        "function": this._formatParameterAsFunction,
-        "generator": this._formatParameterAsObject,
-        "iterator": this._formatParameterAsObject,
-        "map": this._formatParameterAsObject,
-        "node": this._formatParameterAsNode,
-        "object": this._formatParameterAsObject,
-        "promise": this._formatParameterAsObject,
-        "proxy": this._formatParameterAsObject,
-        "set": this._formatParameterAsObject,
-        "string": this._formatParameterAsString
-    };
     this._previewFormatter = new WebInspector.RemoteObjectPreviewFormatter();
     this._searchRegex = null;
 }
 
 WebInspector.ConsoleViewMessage.prototype = {
     /**
      * @return {?WebInspector.Target}
      */
     _target: function()
     {
@@ skipping 216 matching lines @@
     },
 
     /**
      * @param {!WebInspector.ConsoleMessage} consoleMessage
      * @return {?Element}
      */
     _buildMessageAnchor: function(consoleMessage)
     {
         var anchorElement = null;
         if (consoleMessage.source !== WebInspector.ConsoleMessage.MessageSource.Network || consoleMessage.request) {
-            if (consoleMessage.scriptId) {
+            if (consoleMessage.scriptId)
                 anchorElement = this._linkifyScriptId(consoleMessage.scriptId, consoleMessage.url || "", consoleMessage.line, consoleMessage.column);
-            } else {
-                if (consoleMessage.stackTrace && consoleMessage.stackTrace.callFrames.length)
-                    anchorElement = this._linkifyStackTraceTopFrame(consoleMessage.stackTrace);
-                else if (consoleMessage.url && consoleMessage.url !== "undefined")
-                    anchorElement = this._linkifyLocation(consoleMessage.url, consoleMessage.line, consoleMessage.column);
-            }
+            else if (consoleMessage.stackTrace && consoleMessage.stackTrace.callFrames.length)
+                anchorElement = this._linkifyStackTraceTopFrame(consoleMessage.stackTrace);
+            else if (consoleMessage.url && consoleMessage.url !== "undefined")
+                anchorElement = this._linkifyLocation(consoleMessage.url, consoleMessage.line, consoleMessage.column);
         } else if (consoleMessage.url) {
             var url = consoleMessage.url;
             var isExternal = !WebInspector.resourceForURL(url) && !WebInspector.networkMapping.uiSourceCodeForURLForAnyTarget(url);
             anchorElement = WebInspector.linkifyURLAsNode(url, url, "console-message-url", isExternal);
         }
 
         // Append a space to prevent the anchor text from being glued to the console message when the user selects and copies the console messages.
         if (anchorElement)
             anchorElement.appendChild(createTextNode(" "));
         return anchorElement;
@@ skipping 147 matching lines @@
     },
 
     /**
      * @param {!WebInspector.RemoteObject} output
      * @param {boolean=} forceObjectFormat
      * @param {boolean=} includePreview
      * @return {!Element}
      */
     _formatParameter: function(output, forceObjectFormat, includePreview)
     {
-        if (output.customPreview()) {
+        if (output.customPreview())
             return (new WebInspector.CustomPreviewComponent(output)).element;
-        }
 
         var type = forceObjectFormat ? "object" : (output.subtype || output.type);
-        var formatter = this._customFormatters[type] || this._formatParameterAsValue;
         var span = createElement("span");
         span.className = "object-value-" + type + " source-code";
-        formatter.call(this, output, span, includePreview);
+        switch (type) {
+        case "array":
+        case "typedarray":
+            this._formatParameterAsArray(output, span);
+            break;
+        case "error":
+            this._formatParameterAsError(output, span);
+            break;
+        case "function":
+        case "generator":
+            this._formatParameterAsFunction(output, span, includePreview);
+            break;
+        case "iterator":
+        case "map":
+        case "object":
+        case "promise":
+        case "proxy":
+        case "set":
+            this._formatParameterAsObject(output, span, includePreview);
+            break;
+        case "node":
+            this._formatParameterAsNode(output, span);
+            break;
+        case "string":
+            this._formatParameterAsString(output, span);
+            break;
+        case "boolean":
+        case "date":
+        case "null":
+        case "number":
+        case "regexp":
+        case "symbol":
+        case "undefined":
+            this._formatParameterAsValue(output, span);
+            break;
+        default:
+            this._formatParameterAsValue(output, span);
+            console.error("Tried to format remote object of unknown type.");
+        }
         return span;
     },
 
     /**
      * @param {!WebInspector.RemoteObject} obj
      * @param {!Element} elem
      */
     _formatParameterAsValue: function(obj, elem)
     {
         elem.createTextChild(obj.description || "");
         if (obj.objectId)
             elem.addEventListener("contextmenu", this._contextMenuEventFired.bind(this, obj), false);
     },
 
     /**
      * @param {!WebInspector.RemoteObject} obj
      * @param {!Element} elem
      * @param {boolean=} includePreview
      */
     _formatParameterAsObject: function(obj, elem, includePreview)
     {
-        this._formatParameterAsArrayOrObject(obj, elem, includePreview);
-    },
-
-    /**
-     * @param {!WebInspector.RemoteObject} obj
-     * @param {!Element} elem
-     * @param {boolean=} includePreview
-     */
-    _formatParameterAsArrayOrObject: function(obj, elem, includePreview)
-    {
         var titleElement = createElement("span");
         if (includePreview && obj.preview) {
             titleElement.classList.add("console-object-preview");
             this._previewFormatter.appendObjectPreview(titleElement, obj.preview);
+        } else if (obj.type === "function") {
+            WebInspector.ObjectPropertiesSection.formatObjectAsFunction(obj, titleElement, false);
+            titleElement.classList.add("object-value-function");
         } else {
-            if (obj.type === "function") {
-                WebInspector.ObjectPropertiesSection.formatObjectAsFunction(obj, titleElement, false);
-                titleElement.classList.add("object-value-function");
-            } else {
-                titleElement.createTextChild(obj.description || "");
-            }
+            titleElement.createTextChild(obj.description || "");
         }
 
         var section = new WebInspector.ObjectPropertiesSection(obj, titleElement, this._linkifier);
         section.element.classList.add("console-view-object-properties-section");
         section.enableContextMenu();
         elem.appendChild(section.element);
     },
 
     /**
      * @param {!WebInspector.RemoteObject} func
@@ skipping 69 matching lines @@
         {
             this._formatParameterAsObject(object, elem, false);
         }
     },
 
     _formattedParameterAsNodeForTest: function()
     {
     },
 
     /**
-     * @return {boolean}
-     */
-    _usePrintedArrayFormatter: function()
-    {
-        return this._message.type !== WebInspector.ConsoleMessage.MessageType.DirXML && this._message.type !== WebInspector.ConsoleMessage.MessageType.Result;
-    },
-
-    /**
      * @param {!WebInspector.RemoteObject} array
      * @param {!Element} elem
      */
     _formatParameterAsArray: function(array, elem)
     {
-        var maxFlatArrayLength = 100;
-        if (this._usePrintedArrayFormatter() || array.arrayLength() > maxFlatArrayLength)
-            this._formatParameterAsArrayOrObject(array, elem, this._usePrintedArrayFormatter() || array.arrayLength() <= maxFlatArrayLength);
+        var usePrintedArrayFormat = this._message.type !== WebInspector.ConsoleMessage.MessageType.DirXML && this._message.type !== WebInspector.ConsoleMessage.MessageType.Result;
+        var isLongArray = array.arrayLength() > 100;
+        if (usePrintedArrayFormat || isLongArray)
+            this._formatParameterAsObject(array, elem, usePrintedArrayFormat || !isLongArray);
         else
-            array.getAllProperties(false, this._printArrayResult.bind(this, array, elem));
+            array.getAllProperties(false, printArrayResult.bind(this));
+
+        /**
+         * @param {?Array.<!WebInspector.RemoteObjectProperty>} properties
+         * @this {!WebInspector.ConsoleViewMessage}
+         */
+        function printArrayResult(properties)
+        {
+            if (!properties) {
+                this._formatParameterAsObject(array, elem, false);
+                return;
+            }
+
+            var titleElement = createElement("span");
+            var elements = {};
+            for (var i = 0; i < properties.length; ++i) {
+                var property = properties[i];
+                var name = property.name;
+                if (isNaN(name))
+                    continue;
+                if (property.getter)
+                    elements[name] = this._formatAsAccessorProperty(array, [name], true);
+                else if (property.value)
+                    elements[name] = this._formatAsArrayEntry(property.value);
+            }
+
+            titleElement.createTextChild("[");
+            var lastNonEmptyIndex = -1;
+
+            function appendUndefined(titleElement, index)
+            {
+                if (index - lastNonEmptyIndex <= 1)
+                    return;
+                var span = titleElement.createChild("span", "object-value-undefined");
+                span.textContent = WebInspector.UIString("undefined × %d", index - lastNonEmptyIndex - 1);
+            }
+
+            var length = array.arrayLength();
+            for (var i = 0; i < length; ++i) {
+                var element = elements[i];
+                if (!element)
+                    continue;
+
+                if (i - lastNonEmptyIndex > 1) {
+                    appendUndefined(titleElement, i);
+                    titleElement.createTextChild(", ");
+                }
+
+                titleElement.appendChild(element);
+                lastNonEmptyIndex = i;
+                if (i < length - 1)
+                    titleElement.createTextChild(", ");
+            }
+            appendUndefined(titleElement, length);
+
+            titleElement.createTextChild("]");
+
+            var section = new WebInspector.ObjectPropertiesSection(array, titleElement, this._linkifier);
+            section.element.classList.add("console-view-object-properties-section");
+            section.enableContextMenu();
+            elem.appendChild(section.element);
+        }
     },
 
     /**
      * @param {!WebInspector.RemoteObject} output
      * @param {!Element} elem
      */
     _formatParameterAsString: function(output, elem)
     {
         var span = createElement("span");
         span.className = "object-value-string source-code";
@@ skipping 11 matching lines @@
      * @param {!Element} elem
      */
     _formatParameterAsError: function(output, elem)
     {
         var span = elem.createChild("span", "object-value-error source-code");
         var errorSpan = this._tryFormatAsError(output.description || "");
         span.appendChild(errorSpan ? errorSpan : WebInspector.linkifyStringAsFragment(output.description || ""));
     },
 
     /**
-     * @param {!WebInspector.RemoteObject} array
-     * @param {!Element} elem
-     * @param {?Array.<!WebInspector.RemoteObjectProperty>} properties
-     */
-    _printArrayResult: function(array, elem, properties)
-    {
-        if (!properties) {
-            this._formatParameterAsObject(array, elem, false);
-            return;
-        }
-
-        var titleElement = createElement("span");
-        var elements = {};
-        for (var i = 0; i < properties.length; ++i) {
-            var property = properties[i];
-            var name = property.name;
-            if (isNaN(name))
-                continue;
-            if (property.getter)
-                elements[name] = this._formatAsAccessorProperty(array, [name], true);
-            else if (property.value)
-                elements[name] = this._formatAsArrayEntry(property.value);
-        }
-
-        titleElement.createTextChild("[");
-        var lastNonEmptyIndex = -1;
-
-        function appendUndefined(titleElement, index)
-        {
-            if (index - lastNonEmptyIndex <= 1)
-                return;
-            var span = titleElement.createChild("span", "object-value-undefined");
-            span.textContent = WebInspector.UIString("undefined × %d", index - lastNonEmptyIndex - 1);
-        }
-
-        var length = array.arrayLength();
-        for (var i = 0; i < length; ++i) {
-            var element = elements[i];
-            if (!element)
-                continue;
-
-            if (i - lastNonEmptyIndex > 1) {
-                appendUndefined(titleElement, i);
-                titleElement.createTextChild(", ");
-            }
-
-            titleElement.appendChild(element);
-            lastNonEmptyIndex = i;
-            if (i < length - 1)
-                titleElement.createTextChild(", ");
-        }
-        appendUndefined(titleElement, length);
-
-        titleElement.createTextChild("]");
-
-        var section = new WebInspector.ObjectPropertiesSection(array, titleElement, this._linkifier);
-        section.element.classList.add("console-view-object-properties-section");
-        section.enableContextMenu();
-        elem.appendChild(section.element);
-    },
-
-    /**
      * @param {!WebInspector.RemoteObject} output
      * @return {!Element}
      */
     _formatAsArrayEntry: function(output)
     {
         return this._previewFormatter.renderPropertyPreview(output.type, output.subtype, output.description);
     },
 
     /**
      * @param {?WebInspector.RemoteObject} object
@@ skipping 525 matching lines @@
     {
         if (!this._element) {
             WebInspector.ConsoleViewMessage.prototype.toMessageElement.call(this);
             this._element.classList.toggle("collapsed", this._collapsed);
         }
         return this._element;
     },
 
     __proto__: WebInspector.ConsoleViewMessage.prototype
 }