Show ServiceWorkers in chrome://inspect.

Show ServiceWorkers in chrome://inspect.

To show the information about ServiceWorkers I added iframe of "chrome://serviceworker-internals/#iframe" to chrome://inspect.
And I used postMessage() to achieve the cross-domain messaging between chrome://inspect and chrome://serviceworker-internals.

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

[horo, 2014-05-22 15:24:22]

yurys@
Could you please review this?
Thank you.

[yurys, 2014-05-23 09:22:40]

Overall looks good. Vlad, could you please review inspect_ui changes?

https://codereview.chromium.org/298993003/diff/20001/content/browser/resource...
File content/browser/resources/service_worker/serviceworker_internals.js
(right):

https://codereview.chromium.org/298993003/diff/20001/content/browser/resource...
content/browser/resources/service_worker/serviceworker_internals.js:18: return;
style: wrong alignment

https://codereview.chromium.org/298993003/diff/20001/content/browser/resource...
content/browser/resources/service_worker/serviceworker_internals.js:71: var
registration = registrations[i];
style: wrong alignment here and below.

[horo, 2014-05-23 10:49:02]

https://codereview.chromium.org/298993003/diff/20001/content/browser/resource...
File content/browser/resources/service_worker/serviceworker_internals.js
(right):

https://codereview.chromium.org/298993003/diff/20001/content/browser/resource...
content/browser/resources/service_worker/serviceworker_internals.js:18: return;
On 2014/05/23 09:22:41, yurys wrote:
> style: wrong alignment

Done.

https://codereview.chromium.org/298993003/diff/20001/content/browser/resource...
content/browser/resources/service_worker/serviceworker_internals.js:71: var
registration = registrations[i];
On 2014/05/23 09:22:41, yurys wrote:
> style: wrong alignment here and below.

Done.

[Vladislav Kaznacheev, 2014-05-23 11:03:28]

https://codereview.chromium.org/298993003/diff/40001/chrome/browser/resources...
File chrome/browser/resources/inspect/inspect.js (right):

https://codereview.chromium.org/298993003/diff/40001/chrome/browser/resources...
chrome/browser/resources/inspect/inspect.js:108: var list =
$('service-workers-list-' + partition_id);
Do partitions ever go away? This code can only add partitions, so if they will
only accumulate in the UI.

[horo, 2014-05-23 11:21:33]

https://codereview.chromium.org/298993003/diff/40001/chrome/browser/resources...
File chrome/browser/resources/inspect/inspect.js (right):

https://codereview.chromium.org/298993003/diff/40001/chrome/browser/resources...
chrome/browser/resources/inspect/inspect.js:108: var list =
$('service-workers-list-' + partition_id);
On 2014/05/23 11:03:28, Vladislav Kaznacheev wrote:
> Do partitions ever go away? This code can only add partitions, so if they will
> only accumulate in the UI.

Done.
Added removeChild().

[Vladislav Kaznacheev, 2014-05-23 13:53:29]

lgtm

[horo, 2014-05-23 13:59:53]

kinuko@
Could you please review serviceworker_internals.js and
service_worker_internals_ui.cc?

[horo, 2014-05-23 14:01:00]

jochen@
Could you please review url_constants.*?

[jochen (gone - plz use gerrit), 2014-05-23 14:04:23]

content/public lgtm

[kinuko, 2014-05-23 14:19:33]

https://codereview.chromium.org/298993003/diff/60001/content/browser/resource...
File content/browser/resources/service_worker/serviceworker_internals.js
(right):

https://codereview.chromium.org/298993003/diff/60001/content/browser/resource...
content/browser/resources/service_worker/serviceworker_internals.js:73:
registration.active.running_status != 'RUNNING') {
How are we going to do for inspecting pending (in-waiting) service workers?

https://codereview.chromium.org/298993003/diff/60001/content/browser/resource...
content/browser/resources/service_worker/serviceworker_internals.js:79: 'url':
registration['script_url']
nit: mixing up registration.foo and registration['foo'] notation looks a little
inconsistent

[horo, 2014-05-23 14:37:09]

https://codereview.chromium.org/298993003/diff/60001/content/browser/resource...
File content/browser/resources/service_worker/serviceworker_internals.js
(right):

https://codereview.chromium.org/298993003/diff/60001/content/browser/resource...
content/browser/resources/service_worker/serviceworker_internals.js:73:
registration.active.running_status != 'RUNNING') {
On 2014/05/23 14:19:33, kinuko wrote:
> How are we going to do for inspecting pending (in-waiting) service workers?

I'm planing to make another cl to support inspecting pending workers.

https://codereview.chromium.org/298993003/diff/60001/content/browser/resource...
content/browser/resources/service_worker/serviceworker_internals.js:79: 'url':
registration['script_url']
On 2014/05/23 14:19:33, kinuko wrote:
> nit: mixing up registration.foo and registration['foo'] notation looks a
little
> inconsistent

Done.

[kinuko, 2014-05-23 14:55:41]

lgtm

[horo, 2014-05-23 15:27:24]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-05-23 15:28:17]

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

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

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:
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)

[commit-bot: I haz the power, 2014-05-23 21:04:24]

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

[commit-bot: I haz the power, 2014-05-23 21:04:24]

Try jobs failed on following builders:
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)

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

The CQ bit was checked by horo@chromium.org

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

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

[commit-bot: I haz the power, 2014-05-24 12:28:03]

Change committed as 272715