DevTools: Fix console.log for arrays in some corner cases.

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 92 matching lines @@
  * @template T
  */
 function nullifyObjectProto(obj)
 {
     if (obj && typeof obj === "object")
         obj.__proto__ = null;
     return obj;
 }
 
 /**
+ * @param {string} name
+ * @return {boolean}
+ */
+function isArrayIndexPropertyName(name)
+{
+    // Array index check according to the ES5-15.4.
+    var index = name >>> 0;
+    return toString(index) === name && index !== 0xffffffff;
+}
+
+/**
  * @constructor
  */
 var InjectedScript = function()
 {
     /** @type {number} */
     this._lastBoundObjectId = 1;
     /** @type {!Object.<number, Object>} */
     this._idToWrappedObject = { __proto__: null };
     /** @type {!Object.<number, string>} */
     this._idToObjectGroupName = { __proto__: null };
@@ skipping 882 matching lines @@
                 var nameToDescriptors = { __proto__: null };
                 for (var i = 0; i < descriptors.length; ++i) {
                     var descriptor = descriptors[i];
                     nameToDescriptors["#" + descriptor.name] = descriptor;
                 }
                 descriptors = [];
                 for (var i = 0; i < firstLevelKeys.length; ++i)
                     descriptors.push(nameToDescriptors["#" + firstLevelKeys[i]]);
             }
 
+            var isArray = (this.subtype === "array");
             for (var i = 0; i < descriptors.length; ++i) {
-                if (propertiesThreshold.indexes < 0 || propertiesThreshold.properties < 0)
+                if (propertiesThreshold.indexes < 0 && propertiesThreshold.properties < 0)
                     break;
 
                 var descriptor = descriptors[i];
                 if (!descriptor)
                     continue;
                 if (descriptor.wasThrown) {
                     preview.lossless = false;
                     continue;
                 }
-                if (!descriptor.enumerable && !descriptor.isOwn)
-                    continue;
 
                 var name = descriptor.name;
                 if (name === "__proto__")
                     continue;
-                if (this.subtype === "array" && name === "length")
+                if (isArray && name === "length")
+                    continue;
+
+                var isArrayIndex = isArray && isArrayIndexPropertyName(name);
+                if (!descriptor.enumerable && !descriptor.isOwn && !isArrayIndex)
                     continue;
 
                 if (!("value" in descriptor)) {
                     preview.lossless = false;
                     this._appendPropertyPreview(preview, { name: name, type: "accessor", __proto__: null }, propertiesThreshold);
                     continue;
                 }
 
                 var value = descriptor.value;
                 if (value === null) {
@@ skipping 47 matching lines @@
         return preview;
     },
 
     /**
      * @param {!RuntimeAgent.ObjectPreview} preview
      * @param {!Object} property
      * @param {!Object} propertiesThreshold
      */
     _appendPropertyPreview: function(preview, property, propertiesThreshold)
     {
-        if (toString(property.name >>> 0) === property.name)
-            propertiesThreshold.indexes--;
+        var threshold;
+        if (isArrayIndexPropertyName(property.name))
+            threshold = --propertiesThreshold.indexes;
         else
-            propertiesThreshold.properties--;
-        if (propertiesThreshold.indexes < 0 || propertiesThreshold.properties < 0) {
+            threshold = --propertiesThreshold.properties;
+        if (threshold < 0) {
             preview.overflow = true;
             preview.lossless = false;
         } else {
             preview.properties.push(property);
         }
     },
 
     /**
      * @param {string} string
      * @param {number} maxLength
@@ skipping 404 matching lines @@
      */
     _logEvent: function(event)
     {
         inspectedWindow.console.log(event.type, event);
     }
 }
 
 injectedScript._commandLineAPIImpl = new CommandLineAPIImpl();
 return injectedScript;
 })