Add isPausedWorker command to Runtime and use it to auto attach DevTools to ServiceWorker.

Add isPausedWorker command to Runtime and use it to auto attach DevTools to ServiceWorker.

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

[horo, 2014-05-21 09:06:07]

pfeldman@
Could you please review this?

[yurys, 2014-05-21 10:23:48]

How can it happen that isPausedWorker is true at the moment we attach to a
worker? Are you going to have a separate page that allows to start it paused?

[horo, 2014-05-21 14:21:24]

On 2014/05/21 10:23:48, yurys wrote:
> How can it happen that isPausedWorker is true at the moment we attach to a
> worker? Are you going to have a separate page that allows to start it paused?

I'm planing to add "pause on start" check box in
chrome://serviceworker-internals.
https://codereview.chromium.org/299693002/
If the check box is checked ServiceWorker will start in paused state.
This will be very useful for ServiceWorker developers

[yurys, 2014-05-23 09:09:13]

LGTM.

Sorry for the delay. We discussed this with Pavel and it seems that reporting
current state of inspected target is a concept from a level above the protocol.
We didn't come up with a good generic solution yet though. For the time being
let's proceed with this solution.

https://codereview.chromium.org/299803003/diff/20001/Source/devtools/protocol...
File Source/devtools/protocol.json (right):

https://codereview.chromium.org/299803003/diff/20001/Source/devtools/protocol...
Source/devtools/protocol.json:827: "name": "isPausedWorker",
Let's rename this into isRunRequired to avoid confusion with debugger pause.

[horo, 2014-05-23 10:03:20]

https://codereview.chromium.org/299803003/diff/20001/Source/devtools/protocol...
File Source/devtools/protocol.json (right):

https://codereview.chromium.org/299803003/diff/20001/Source/devtools/protocol...
Source/devtools/protocol.json:827: "name": "isPausedWorker",
On 2014/05/23 09:09:14, yurys wrote:
> Let's rename this into isRunRequired to avoid confusion with debugger pause.

Done.

[horo, 2014-05-23 15:36:25]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-05-23 15:36:40]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/horo@chromium.org/299803003/40001

[commit-bot: I haz the power, 2014-05-23 17:11:36]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/8923)

[commit-bot: I haz the power, 2014-05-23 18:33:18]

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

[commit-bot: I haz the power, 2014-05-23 18:33:18]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/8940)

[horo, 2014-05-24 00:04:03]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-05-24 00:04:43]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/horo@chromium.org/299803003/40001

[commit-bot: I haz the power, 2014-05-24 00:54:56]

Change committed as 174735

[horo, 2014-05-24 02:02:17]

On 2014/05/24 00:54:56, I haz the power (commit-bot) wrote:
> Change committed as 174735

Oh, I forgot to change the description...