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

Revert of DevTools: Fix console.log for arrays in some corner cases. (https://codereview.chromium.org/143263003/)

Reason for revert:
Console output got messy.

Original issue's description:
> DevTools: Fix console.log for arrays in some corner cases.
> 
> We didn't show array values defined on Array.prototype[], as well as non-index property names.
> 
> BUG=320632, 163817
> R=pfeldman
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=165713

TBR=JulianMRaya@gmail.com,aandrey@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=320632, 163817
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=166302

[pfeldman, 2014-02-01 10:08:57]

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

[commit-bot: I haz the power, 2014-02-01 10:09:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pfeldman@chromium.org/135973010/1

[commit-bot: I haz the power, 2014-02-01 10:09:10]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-02-01 10:09:11]

Failed to apply patch for
LayoutTests/inspector/console/console-format-expected.txt:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file LayoutTests/inspector/console/console-format-expected.txt
  Hunk #2 FAILED at 69.
  1 out of 2 hunks FAILED -- saving rejects to file
LayoutTests/inspector/console/console-format-expected.txt.rej

Patch:       LayoutTests/inspector/console/console-format-expected.txt
Index: LayoutTests/inspector/console/console-format-expected.txt
diff --git a/LayoutTests/inspector/console/console-format-expected.txt
b/LayoutTests/inspector/console/console-format-expected.txt
index
5fb667024e4d89907f8aafd68c5f60593a1bed56..7dcaedbdd7eb1c82ab6e399c80c69dc0b112b02f
100644
--- a/LayoutTests/inspector/console/console-format-expected.txt
+++ b/LayoutTests/inspector/console/console-format-expected.txt
@@ -49,8 +49,6 @@
 CONSOLE MESSAGE: line 13: -Infinity
 CONSOLE MESSAGE: line 12: test,test2,,,test4,,,,,
 CONSOLE MESSAGE: line 13: test,test2,,,test4,,,,,
-CONSOLE MESSAGE: line 12: 1,2,3,4
-CONSOLE MESSAGE: line 13: 1,2,3,4
 CONSOLE MESSAGE: line 12: [object Object]
 CONSOLE MESSAGE: line 13: [object Object]
 CONSOLE MESSAGE: line 12: function () {}
@@ -71,418 +69,133 @@
 CONSOLE MESSAGE: line 13: [object Object]
 Tests that console logging dumps proper messages.
 
- 
-
-Array[10]
-    0: "test"
-    1: "test2"
-    4: "test4"
-    foo: Object
-    length: 10
-    __proto__: Array[0] console-format.html:26
-
-Array[10]
-    0: "test"
-    1: "test2"
-    4: "test4"
-    foo: Object
-    length: 10
-    __proto__: Array[0] console-format.html:27
-
-Array[10]
-    0: "test"
-    1: "test2"
-    4: "test4"
-    foo: Object
-    length: 10
-    __proto__: Array[0] console-format.html:28
-
+ Array[10]0: "test"1: "test2"4: "test4"foo: Objectlength: 10__proto__: Array[0]
console-format.html:26
+Array[10]0: "test"1: "test2"4: "test4"foo: Objectlength: 10__proto__: Array[0]
console-format.html:27
+Array[10]0: "test"1: "test2"4: "test4"foo: Objectlength: 10__proto__: Array[0]
console-format.html:28
 Test for zero "0" in formatter console-format.html:29
-

/^url\(\s*(?:(?:"(?:[^\\\"]|(?:\\[\da-f]{1,6}\s?|\.))*"|'(?:[^\\\']|(?:\\[\da-f]{1,6}\s?|\.))*')|(?:[!#$%&*-~\w]|(?:\\[\da-f]{1,6}\s?|\.))*)\s*\)/i
console-format.html:12
-
-[/^url\(\s*(?:(?:"(?:[^\\\"]|(?:\\[\da-f]{1,6}\s?|\…?:[!#$%&*-~\w]|(?:\\[\da-f]{1,6}\s?|\.))*)\s*\)/i]
-    0:
/^url\(\s*(?:(?:"(?:[^\\\"]|(?:\\[\da-f]{1,6}\s?|\.))*"|'(?:[^\\\']|(?:\\[\da-f]{1,6}\s?|\.))*')|(?:[!#$%&*-~\w]|(?:\\[\da-f]{1,6}\s?|\.))*)\s*\)/i
-    length: 1
-    __proto__: Array[0] console-format.html:13
-
-> globals[0]
-
+[/^url\(\s*(?:(?:"(?:[^\\\"]|(?:\\[\da-f]{1,6}\s?|\…?:[!#$%&*-~\w]|(?:\\[\da-f]{1,6}\s?|\.))*)\s*\)/i]0:
/^url\(\s*(?:(?:"(?:[^\\\"]|(?:\\[\da-f]{1,6}\s?|\.))*"|'(?:[^\\\']|(?:\\[\da-f]{1,6}\s?|\.))*')|(?:[!#$%&*-~\w]|(?:\\[\da-f]{1,6}\s?|\.))*)\s*\)/ilength:
1__proto__: Array[0] console-format.html:13
+globals[0]

/^url\(\s*(?:(?:"(?:[^\\\"]|(?:\\[\da-f]{1,6}\s?|\.))*"|'(?:[^\\\']|(?:\\[\da-f]{1,6}\s?|\.))*')|(?:[!#$%&*-~\w]|(?:\\[\da-f]{1,6}\s?|\.))*)\s*\)/i
-
 /foo\\bar\sbaz/i console-format.html:12
-
-[/foo\\bar\sbaz/i]
-    0: /foo\\bar\sbaz/i
-    length: 1
-    __proto__: Array[0] console-format.html:13
-
-> globals[1]
-
+[/foo\\bar\sbaz/i]0: /foo\\bar\sbaz/ilength: 1__proto__: Array[0]
console-format.html:13
+globals[1]
 /foo\\bar\sbaz/i
-
 test console-format.html:12
-
 ["test"] console-format.html:13
-
-> globals[2]
-
+globals[2]
 "test"
-
 test named "test" console-format.html:12
-
 ["test named "test""] console-format.html:13
-
-> globals[3]
-
+globals[3]
 "test named "test""
-
-Error {stack: (...)}
-    stack: (...)
-    get stack: function () { [native code] }
-    set stack: function () { [native code] }
-    __proto__: d console-format.html:12
-
-[Error]
-    0: Error
-    length: 1
-    __proto__: Array[0] console-format.html:13
-
-> globals[4]
-
-Error {stack: (...)}
-    stack: (...)
-    get stack: function () { [native code] }
-    set stack: function () { [native code] }
-    __proto__: d
-
-Error {stack: (...), message: "my error message"}
-    message: "my error message"
-    stack: (...)
-    get stack: function () { [native code] }
-    set stack: function () { [native code] }
-    __proto__: d console-format.html:12
-
-[Error]
-    0: Error
-    length: 1
-    __proto__: Array[0] console-format.html:13
-
-> globals[5]
-
-Error {stack: (...), message: "my error message"}
-    message: "my error message"
-    stack: (...)
-    get stack: function () { [native code] }
-    set stack: function () { [native code] }
-    __proto__: d
-
-
-    <p id=​"p">​Tests that console logging dumps proper messages.​</p>​
console-format.html:12
-
-[p#p]
-    0: p#p
-    length: 1
-    __proto__: Array[0] console-format.html:13
-
-> globals[6]
-
-
-    <p id=​"p">​Tests that console logging dumps proper messages.​</p>​
-
+Error {stack: (...)}stack: (...)get stack: function () { [native code] }set
stack: function () { [native code] }__proto__: d console-format.html:12
+[Error]0: Errorlength: 1__proto__: Array[0] console-format.html:13
+globals[4]
+Error {stack: (...)}stack: (...)get stack: function () { [native code] }set
stack: function () { [native code] }__proto__: d
+Error {stack: (...), message: "my error message"}message: "my error
message"stack: (...)get stack: function () { [native code] }set stack: function
() { [native code] }__proto__: d console-format.html:12
+[Error]0: Errorlength: 1__proto__: Array[0] console-format.html:13
+globals[5]
+Error {stack: (...), message: "my error message"}message: "my error
message"stack: (...)get stack: function () { [native code] }set stack: function
() { [native code] }__proto__: d
+<p id="p">Tests that console logging dumps proper messages.</p>
console-format.html:12
+[p#p]0: p#plength: 1__proto__: Array[0] console-format.html:13
+globals[6]
+<p id="p">Tests that console logging dumps proper messages.</p>
 function () { return 1; } console-format.html:12
-
-[function]
-    0: function () { return 1; }
-    length: 1
-    __proto__: Array[0] console-format.html:13
-
-> globals[7]
-
+[function]0: function () { return 1; }length: 1__proto__: Array[0]
console-format.html:13
+globals[7]
 function () { return 1; } console-format.html:38
-
 function () {
         return 2;
     } console-format.html:12
-
-[function]
-    0: function () {
-    length: 1
-    __proto__: Array[0] console-format.html:13
-
-> globals[8]
-
+[function]0: function () {length: 1__proto__: Array[0] console-format.html:13
+globals[8]
 function () {
         return 2;
     } console-format.html:39
-
 0.12 console-format.html:12
-
 [0.12] console-format.html:13
-
-> globals[9]
-
+globals[9]
 0.12
-
 http://webkit.org/ console-format.html:12
-
 ["http://webkit.org/"] console-format.html:13
-
-> globals[10]
-
+globals[10]
 "http://webkit.org/"
-
 null console-format.html:12
-
 [null] console-format.html:13
-
-> globals[11]
-
+globals[11]
 null
-
 undefined console-format.html:12
-
 [undefined] console-format.html:13
-
-> globals[12]
-
+globals[12]
 undefined
-
-Attr.specified is deprecated. Its value is always true. VM:459
-
-
-    attr=​"" console-format.html:12
-
-[attr]
-    0: attr
-    length: 1
-    __proto__: Array[0] console-format.html:13
-
-> globals[13]
-
-
-    attr=​""
-
-
-    attr=​"value"
-    "value" console-format.html:12
-
-[attr]
-    0: attr
-    length: 1
-    __proto__: Array[0] console-format.html:13
-
-> globals[14]
-
-
-    attr=​"value"
-    "value"
-
-
-    id=​"x"
-    "x" console-format.html:12
-
-[id]
-    0: id
-    length: 1
-    __proto__: Array[0] console-format.html:13
-
-> globals[15]
-
-
-    id=​"x"
-    "x"
-
-Object {length: (...)}
-    length: (...)
-    get length: function length() { throw "Length called"; }
-    __proto__: Object console-format.html:12
-
-[Object]
-    0: Object
-    length: 1
-    __proto__: Array[0] console-format.html:13
-
-> globals[16]
-
-Object {length: (...)}
-    length: (...)
-    get length: function length() { throw "Length called"; }
-    __proto__: Object
-
+Attr.specified is deprecated. Its value is always true. VM:448
+attr="" console-format.html:12
+[attr]0: attrlength: 1__proto__: Array[0] console-format.html:13
+globals[13]
+attr=""
+attr="value""value" console-format.html:12
+[attr]0: attrlength: 1__proto__: Array[0] console-format.html:13
+globals[14]
+attr="value""value"
+id="x""x" console-format.html:12
+[id]0: idlength: 1__proto__: Array[0] console-format.html:13
+globals[15]
+id="x""x"
+Object {length: (...)}length: (...)get length: function length() { throw
"Length called"; }__proto__: Object console-format.html:12
+[Object]0: Objectlength: 1__proto__: Array[0] console-format.html:13
+globals[16]
+Object {length: (...)}length: (...)get length: function length() { throw
"Length called"; }__proto__: Object
 NaN console-format.html:12
-
 [NaN] console-format.html:13
-
-> globals[17]
-
+globals[17]
 NaN
-
 Infinity console-format.html:12
-
 [Infinity] console-format.html:13
-
-> globals[18]
-
+globals[18]
 Infinity
-
 -Infinity console-format.html:12
-
 [-Infinity] console-format.html:13
-
-> globals[19]
-
+globals[19]
 -Infinity
-
-["test", "test2", undefined × 2, "test4", undefined × 5, foo: Object]
-    0: "test"
-    1: "test2"
-    4: "test4"
-    foo: Object
-    length: 10
-    __proto__: Array[0] console-format.html:12
-
-[Array[10]]
-    0: Array[10]
-    length: 1
-    __proto__: Array[0] console-format.html:13
-
-> globals[20]
-
-["test", "test2", undefined × 2, "test4", undefined × 5, foo: Object
-    __proto__: Object]
-
-[1, 2, 3, 4, index0: 0, index1: 1, index2: 2, index3: 3, index4: 4…]
-    0: 1
-    1: 2
-    2: 3
-    3: 4
-    index0: 0
-    index1: 1
-    index2: 2
-    index3: 3
-    index4: 4
-    index5: 5
-    length: 4
-    __proto__: Array[0] console-format.html:12
-
-[Array[4]]
-    0: Array[4]
-    length: 1
-    __proto__: Array[0] con…
(message too large)

[commit-bot: I haz the power, 2014-02-01 10:09:13]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-02-01 10:09:15]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-02-01 10:09:20]

CQ bit was unchecked on CL. Ignoring.

[pfeldman, 2014-02-03 11:24:08]

The CQ bit was checked by pfeldman@chromium.org

[commit-bot: I haz the power, 2014-02-03 11:24:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pfeldman@chromium.org/135973010/190001

[commit-bot: I haz the power, 2014-02-03 11:25:18]

Change committed as 166302

[commit-bot: I haz the power, 2014-02-03 11:25:21]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-02-03 11:25:22]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-02-03 11:25:26]

CQ bit was unchecked on CL. Ignoring.