[DevTools] Improve MainThreadDebugger::memoryInfo

[DevTools] Improve MainThreadDebugger::memoryInfo

Use context->Global() instead of CreationContext in MainThreadDebugger::memoryInfo.

BUG=616225
R=adamk@chromium.org,dgozman@chromium.org

Committed: https://crrev.com/6c932f8010646f4cc13985d05e3cc75f29ea2abb
Cr-Commit-Position: refs/heads/master@{#397887}

[kozy, 2016-06-03 19:14:22]

Dmitry and Adam, please take a look!

[adamk, 2016-06-03 19:21:18]

I'll mostly defer to dgozman here, but it would be great to have a regression
test for this. I would expect it to be relatively easy to create a layout test
that exercises this code.

[dgozman, 2016-06-03 21:06:26]

Fix lgtm. Let's add a test. Let's also ask the author of
V8WrapperInstantiationScope::securityCheck.

[kozy, 2016-06-03 23:46:16]

The CQ bit was checked by kozyatinskiy@chromium.org

[kozy, 2016-06-03 23:46:16]

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/2034203002/#ps20001
(title: "added a test")

[commit-bot: I haz the power, 2016-06-03 23:47:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2034203002/20001

[dgozman, 2016-06-03 23:59:08]

https://codereview.chromium.org/2034203002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector-protocol/runtime/runtime-console-memory.html
(right):

https://codereview.chromium.org/2034203002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector-protocol/runtime/runtime-console-memory.html:15:
console.log(memory.constructor.constructor("return document.body.innerHTML")());
document.querySelector("#mydiv").textContent

[kozy, 2016-06-03 23:59:36]

The CQ bit was unchecked by kozyatinskiy@chromium.org

[kozy, 2016-06-04 00:13:27]

All done!

https://codereview.chromium.org/2034203002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector-protocol/runtime/runtime-console-memory.html
(right):

https://codereview.chromium.org/2034203002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector-protocol/runtime/runtime-console-memory.html:15:
console.log(memory.constructor.constructor("return document.body.innerHTML")());
On 2016/06/03 23:59:07, dgozman wrote:
> document.querySelector("#mydiv").textContent

Done.

[kozy, 2016-06-04 00:13:31]

The CQ bit was checked by kozyatinskiy@chromium.org

[kozy, 2016-06-04 00:13:31]

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/2034203002/#ps60001
(title: " ")

[commit-bot: I haz the power, 2016-06-04 00:14:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2034203002/60001

[commit-bot: I haz the power, 2016-06-04 01:52:09]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-06-04 01:55:03]

Description was changed from

==========
[DevTools] Improve MainThreadDebugger::memoryInfo

Use context->Global() instead of CreationContext in
MainThreadDebugger::memoryInfo.

BUG=616225
R=adamk@chromium.org,dgozman@chromium.org
==========

to

==========
[DevTools] Improve MainThreadDebugger::memoryInfo

Use context->Global() instead of CreationContext in
MainThreadDebugger::memoryInfo.

BUG=616225
R=adamk@chromium.org,dgozman@chromium.org

Committed: https://crrev.com/6c932f8010646f4cc13985d05e3cc75f29ea2abb
Cr-Commit-Position: refs/heads/master@{#397887}
==========

[commit-bot: I haz the power, 2016-06-04 01:55:05]

Patchset 4 (id:??) landed as
https://crrev.com/6c932f8010646f4cc13985d05e3cc75f29ea2abb
Cr-Commit-Position: refs/heads/master@{#397887}