DevTools: use v8::Isolate::RequestInterrupt instead of v8::Debug::DebugBreakForCommand

DevTools: use v8::Isolate::RequestInterrupt instead of v8::Debug::DebugBreakForCommand

RequestInterrupt provides necessary primitive for implementing command queue on embedder's side. A big advantage of this approach is that we don't have to enable debugger in order to dispatch such commands and can manage them on embedder's side.

ScriptDebugServer now uses thread safe queue to store pending tasks. Although the queue is thread safe it is still caller's responsibility to make sure that ScriptDebugServer stays alive while the queue is being accessed.

BUG=468735
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=192543

[yurys, 2015-03-25 09:00:13]

yurys@chromium.org changed reviewers:
+ alph@chromium.org, pfeldman@chromium.org, sergeyv@chromium.org

[yurys, 2015-03-25 09:00:40]

yurys@chromium.org changed reviewers:
+ loislo@chromium.org

[alph, 2015-03-25 09:10:03]

https://codereview.chromium.org/1038603002/diff/20001/Source/bindings/core/v8...
File Source/bindings/core/v8/ScriptDebugServer.cpp (right):

https://codereview.chromium.org/1038603002/diff/20001/Source/bindings/core/v8...
Source/bindings/core/v8/ScriptDebugServer.cpp:485: m_taskQueue->append(task);
Hmm, what makes a call to InterruptRequest?

[yurys, 2015-03-25 09:38:49]

https://codereview.chromium.org/1038603002/diff/20001/Source/bindings/core/v8...
File Source/bindings/core/v8/ScriptDebugServer.cpp (right):

https://codereview.chromium.org/1038603002/diff/20001/Source/bindings/core/v8...
Source/bindings/core/v8/ScriptDebugServer.cpp:485: m_taskQueue->append(task);
On 2015/03/25 09:10:03, alph wrote:
> Hmm, what makes a call to InterruptRequest?

Done.

[yurys, 2015-03-25 09:40:21]

Patchset #2 (id:20001) has been deleted

[alph, 2015-03-25 09:47:04]

Cool. lgtm

[yurys, 2015-03-25 13:45:27]

The CQ bit was checked by yurys@chromium.org

[commit-bot: I haz the power, 2015-03-25 13:45:37]

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

[commit-bot: I haz the power, 2015-03-25 15:22:45]

Committed patchset #3 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=192543