[inspector] speculative avoid using debugger context as current in blink

[inspector] speculative avoid using debugger context as current in blink

Now we run nested message loop with debugger context as current one.

BUG=chromium:676749
R=yangguo@chromium.org,dgozman@chromium.org

Review-Url: https://codereview.chromium.org/2604553002
Cr-Commit-Position: refs/heads/master@{#41937}
Committed: https://chromium.googlesource.com/v8/v8/+/b571c6d1cea5b0513dfa748a6330c4cb8f4ca89e

[kozy, 2016-12-23 09:20:45]

Yang, please take a look.
Do we really need to run all code on pause with debug context?

[Yang, 2016-12-23 09:39:32]

On 2016/12/23 09:20:45, kozyatinskiy (OOO) wrote:
> Yang, please take a look.
> Do we really need to run all code on pause with debug context?

No. I don't think we need to. In fact, it's probably safer not to.

[Yang, 2016-12-23 09:39:40]

On 2016/12/23 09:39:32, Yang wrote:
> On 2016/12/23 09:20:45, kozyatinskiy (OOO) wrote:
> > Yang, please take a look.
> > Do we really need to run all code on pause with debug context?
> 
> No. I don't think we need to. In fact, it's probably safer not to.

LGTM.

[kozy, 2016-12-23 09:42:14]

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

[commit-bot: I haz the power, 2016-12-23 09:42:21]

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

[kozy, 2016-12-23 09:53:33]

The CQ bit was unchecked by kozyatinskiy@chromium.org

[kozy, 2016-12-23 09:53:38]

The CQ bit was checked by kozyatinskiy@chromium.org

[commit-bot: I haz the power, 2016-12-23 09:53:41]

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-23 10:06:25]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482486818549620, "parent_rev":
"833d2c4b05cef909a157da90a4ff6db27a09849a", "commit_rev":
"b571c6d1cea5b0513dfa748a6330c4cb8f4ca89e"}

[commit-bot: I haz the power, 2016-12-23 10:06:29]

Description was changed from

==========
[inspector] speculative avoid using debugger context as current in blink

Now we run nested message loop with debugger context as current one.

BUG=chromium:676749
R=yangguo@chromium.org,dgozman@chromium.org
==========

to

==========
[inspector] speculative avoid using debugger context as current in blink

Now we run nested message loop with debugger context as current one.

BUG=chromium:676749
R=yangguo@chromium.org,dgozman@chromium.org

Review-Url: https://codereview.chromium.org/2604553002
Cr-Commit-Position: refs/heads/master@{#41937}
Committed:
https://chromium.googlesource.com/v8/v8/+/b571c6d1cea5b0513dfa748a6330c4cb8f4...
==========

[commit-bot: I haz the power, 2016-12-23 10:06:30]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/b571c6d1cea5b0513dfa748a6330c4cb8f4...