DevTools: Simplify isArrayLike() in injected script and extend InjectedScriptHost.type()

Source/core/inspector/InjectedScriptSource.js

 /*
  * Copyright (C) 2007 Apple Inc.  All rights reserved.
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * 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.
@@ skipping 134 matching lines @@
  * @return {boolean}
  */
 function isArrayLike(obj)
 {
     try {
         if (typeof obj !== "object")
             return false;
         if (typeof obj.splice === "function")
             return isFinite(obj.length);
         var str = InjectedScriptHost.callFunction(Object.prototype.toString, obj);
-        if (str === "[object Array]" ||
-            str === "[object Arguments]" ||
-            str === "[object HTMLCollection]" ||
-            str === "[object NodeList]" ||
-            str === "[object DOMTokenList]")
+        if (str === "[object Arguments]")
             return isFinite(obj.length);
     } catch (e) {
     }
     return false;
 }
 
 /**
  * @param {number} a
  * @param {number} b
  * @return {number}
@@ skipping 805 matching lines @@
      * @return {?string}
      */
     _subtype: function(obj)
     {
         if (obj === null)
             return "null";
 
         if (this.isPrimitiveValue(obj))
             return null;
 
-        if (this._isHTMLAllCollection(obj))
-            return "array";
-
         var preciseType = InjectedScriptHost.type(obj);
         if (preciseType)
             return preciseType;
 
         if (isArrayLike(obj))
             return "array";
 
         // If owning frame has navigated to somewhere else window properties will be undefined.
         return null;
     },
@@ skipping 34 matching lines @@
         if (subtype === "array") {
             if (typeof obj.length === "number")
                 className += "[" + obj.length + "]";
             return className;
         }
 
         // NodeList in JSC is a function, check for array prior to this.
         if (typeof obj === "function")
             return toString(obj);
 
-        if (isSymbol(obj)) {
+        if (subtype === "symbol") {
             try {
                 return InjectedScriptHost.callFunction(Symbol.prototype.toString, obj) || "Symbol";
             } catch (e) {
                 return "Symbol";
             }
         }
 
         if (obj instanceof Error && !!obj.message)
             return className + ": " + obj.message;
 
@@ skipping 648 matching lines @@
      */
     _logEvent: function(event)
     {
         inspectedWindow.console.log(event.type, event);
     }
 }
 
 injectedScript._commandLineAPIImpl = new CommandLineAPIImpl();
 return injectedScript;
 })