[inspector] don't truncate [[Entries]] from Runtime.getProperties

[inspector] don't truncate [[Entries]] from Runtime.getProperties

BUG=chromium:650729
R=dgozman@chromium.org

Committed: https://crrev.com/bcedfb029c992f2527db0d9d7766528e356ffe12
Cr-Commit-Position: refs/heads/master@{#39843}

[kozy, 2016-09-27 23:27:12]

Dmitry, please take a look!

[kozy, 2016-09-27 23:28:02]

test is here: https://codereview.chromium.org/2376863002/

[kozy, 2016-09-28 19:33:29]

Dmitry, please take a look!

[kozy, 2016-09-28 19:42:19]

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

[commit-bot: I haz the power, 2016-09-28 19:42:30]

Dry run: 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-09-28 20:09:46]

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

[commit-bot: I haz the power, 2016-09-28 20:09:47]

Dry run: This issue passed the CQ dry run.

[dgozman, 2016-09-28 20:21:52]

Let's add a test!

https://codereview.chromium.org/2373753003/diff/20001/src/inspector/v8-debugg...
File src/inspector/v8-debugger.cc (right):

https://codereview.chromium.org/2373753003/diff/20001/src/inspector/v8-debugg...
src/inspector/v8-debugger.cc:794: CHECK(entriesValue->IsArray());
if (!array) return undefined

[dgozman, 2016-09-28 20:22:14]

lgtm with a comment

[kozy, 2016-09-28 20:32:37]

all done, thanks!

https://codereview.chromium.org/2373753003/diff/20001/src/inspector/v8-debugg...
File src/inspector/v8-debugger.cc (right):

https://codereview.chromium.org/2373753003/diff/20001/src/inspector/v8-debugg...
src/inspector/v8-debugger.cc:794: CHECK(entriesValue->IsArray());
On 2016/09/28 20:21:52, dgozman wrote:
> if (!array) return undefined

Done.

[kozy, 2016-09-28 20:32:42]

The CQ bit was checked by kozyatinskiy@chromium.org

[kozy, 2016-09-28 20:32:43]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/2373753003/#ps40001
(title: "addressed comments")

[commit-bot: I haz the power, 2016-09-28 20:32:48]

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-09-28 20:32:49]

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

[commit-bot: I haz the power, 2016-09-28 20:32:50]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-v8-committers".
Note that this has nothing to do with OWNERS files.

[kozy, 2016-09-28 20:43:48]

kozyatinskiy@chromium.org changed reviewers:
+ alph@chromium.org

[kozy, 2016-09-28 20:43:49]

Alexey, please take a look, I need committer lgtm.

[kozy, 2016-09-28 20:43:55]

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

[commit-bot: I haz the power, 2016-09-28 20:44:07]

Dry run: 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-09-28 21:32:12]

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

[commit-bot: I haz the power, 2016-09-28 21:32:13]

Dry run: This issue passed the CQ dry run.

[alph, 2016-09-29 00:03:34]

lgtm

[kozy, 2016-09-29 00:07:13]

The CQ bit was checked by kozyatinskiy@chromium.org

[commit-bot: I haz the power, 2016-09-29 00:07:20]

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-09-29 00:09:45]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-29 00:10:20]

Description was changed from

==========
[inspector] don't truncate [[Entries]] from Runtime.getProperties

BUG=chromium:650729
R=dgozman@chromium.org
==========

to

==========
[inspector] don't truncate [[Entries]] from Runtime.getProperties

BUG=chromium:650729
R=dgozman@chromium.org

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

[commit-bot: I haz the power, 2016-09-29 00:10:21]

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