DevTools: rebaseline tests after V8 roll

DevTools: rebaseline tests after V8 roll

Test expectations were modified for tests that included the old format for
descriptions 'Array[2]'. Since the V8 change landed that updated descriptions
to 'Array(2)', this CL manually rebaselines the tests that failed.

CLs that marked expectations:
https://codereview.chromium.org/2524913002/
https://codereview.chromium.org/2550513002/

BUG=405845
Committed: https://crrev.com/a464b6447319d296f677b4250c1b0d858eb3ed5e
Cr-Commit-Position: refs/heads/master@{#436036}

[luoe, 2016-12-02 19:31:05]

Description was changed from

==========
DevTools: rebaseline tests after V8 roll

BUG=405845
==========

to

==========
DevTools: rebaseline tests after V8 roll

Test expectations were modified for tests that included the old format for
descriptions 'Array[2]'. Since the V8 change landed that updated descriptions
to 'Array(2)', this CL manually rebaselines the tests that failed.

Corresponding CL that marked expectations:
https://codereview.chromium.org/2524913002/

BUG=405845
==========

[luoe, 2016-12-02 19:31:28]

Description was changed from

==========
DevTools: rebaseline tests after V8 roll

Test expectations were modified for tests that included the old format for
descriptions 'Array[2]'. Since the V8 change landed that updated descriptions
to 'Array(2)', this CL manually rebaselines the tests that failed.

Corresponding CL that marked expectations:
https://codereview.chromium.org/2524913002/

BUG=405845
==========

to

==========
DevTools: rebaseline tests after V8 roll

Test expectations were modified for tests that included the old format for
descriptions 'Array[2]'. Since the V8 change landed that updated descriptions
to 'Array(2)', this CL manually rebaselines the tests that failed.

CLs that marked expectations:
https://codereview.chromium.org/2524913002/
https://codereview.chromium.org/2550513002/

BUG=405845
==========

[luoe, 2016-12-02 19:31:37]

The CQ bit was checked by luoe@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-02 19:33:13]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[luoe, 2016-12-02 19:33:31]

The CQ bit was checked by luoe@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-02 19:34:41]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[luoe, 2016-12-02 19:39:36]

luoe@chromium.org changed reviewers:
+ dgozman@chromium.org, kozyatinskiy@chromium.org

[luoe, 2016-12-02 19:39:36]

@kozyatinskiy: In console-format-expected.html, there was a change to the
precision of digits in the CONSOLE MESSAGE section.  Do you know whether this is
ok?

@dgozman: I've added a comment to RemoteObject.js saying why we have two matches
for two formats.

https://codereview.chromium.org/2543303002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/console/console-format-expected.txt
(right):

https://codereview.chromium.org/2543303002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/console/console-format-expected.txt:84:
CONSOLE MESSAGE: line 13: 42.0000
I'm not sure why this changed.  This console test does not depend on the
precision of digits, so it should be safe to rebaseline this.

[dgozman, 2016-12-02 19:41:56]

lgtm

[commit-bot: I haz the power, 2016-12-02 21:14:49]

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

[commit-bot: I haz the power, 2016-12-02 21:14:50]

Dry run: This issue passed the CQ dry run.

[kozy, 2016-12-02 21:21:48]

lgtm

https://codereview.chromium.org/2543303002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/console/console-format-expected.txt
(right):

https://codereview.chromium.org/2543303002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/console/console-format-expected.txt:84:
CONSOLE MESSAGE: line 13: 42.0000
On 2016/12/02 19:39:36, luoe wrote:
> I'm not sure why this changed.  This console test does not depend on the
> precision of digits, so it should be safe to rebaseline this.

Way how we convert double to string was changed in one of recent V8 roll. It's
safe.

[luoe, 2016-12-02 21:24:47]

The CQ bit was checked by luoe@chromium.org

[commit-bot: I haz the power, 2016-12-02 21:25:54]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-02 21:34:39]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1480713887054370,
"parent_rev": "687342df9cdaeb996b327de064091149954edbd7", "commit_rev":
"72018ad7c583fec571e84c7742f056bf2d06f8fd"}

[commit-bot: I haz the power, 2016-12-02 21:35:06]

Description was changed from

==========
DevTools: rebaseline tests after V8 roll

Test expectations were modified for tests that included the old format for
descriptions 'Array[2]'. Since the V8 change landed that updated descriptions
to 'Array(2)', this CL manually rebaselines the tests that failed.

CLs that marked expectations:
https://codereview.chromium.org/2524913002/
https://codereview.chromium.org/2550513002/

BUG=405845
==========

to

==========
DevTools: rebaseline tests after V8 roll

Test expectations were modified for tests that included the old format for
descriptions 'Array[2]'. Since the V8 change landed that updated descriptions
to 'Array(2)', this CL manually rebaselines the tests that failed.

CLs that marked expectations:
https://codereview.chromium.org/2524913002/
https://codereview.chromium.org/2550513002/

BUG=405845
==========

[commit-bot: I haz the power, 2016-12-02 21:35:17]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-02 21:38:04]

Description was changed from

==========
DevTools: rebaseline tests after V8 roll

Test expectations were modified for tests that included the old format for
descriptions 'Array[2]'. Since the V8 change landed that updated descriptions
to 'Array(2)', this CL manually rebaselines the tests that failed.

CLs that marked expectations:
https://codereview.chromium.org/2524913002/
https://codereview.chromium.org/2550513002/

BUG=405845
==========

to

==========
DevTools: rebaseline tests after V8 roll

Test expectations were modified for tests that included the old format for
descriptions 'Array[2]'. Since the V8 change landed that updated descriptions
to 'Array(2)', this CL manually rebaselines the tests that failed.

CLs that marked expectations:
https://codereview.chromium.org/2524913002/
https://codereview.chromium.org/2550513002/

BUG=405845

Committed: https://crrev.com/a464b6447319d296f677b4250c1b0d858eb3ed5e
Cr-Commit-Position: refs/heads/master@{#436036}
==========

[commit-bot: I haz the power, 2016-12-02 21:38:05]

Patchset 2 (id:??) landed as
https://crrev.com/a464b6447319d296f677b4250c1b0d858eb3ed5e
Cr-Commit-Position: refs/heads/master@{#436036}