Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3312)

Unified Diff: chrome/browser/resources/inspect/inspect.js

Issue 568403002: Unify shared and service workers in chrome://inspect, remove service workers iframe. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebased, review fixes Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/inspect/inspect.js
diff --git a/chrome/browser/resources/inspect/inspect.js b/chrome/browser/resources/inspect/inspect.js
index a4a27587318407bca84ba7d7e64b920c474335df..18a2d1b06d9942ff7211ffb574607e7b3ceacef2 100644
--- a/chrome/browser/resources/inspect/inspect.js
+++ b/chrome/browser/resources/inspect/inspect.js
@@ -29,13 +29,6 @@ function sendTargetCommand(command, target) {
sendCommand(command, target.source, target.id);
}
-function sendServiceWorkerCommand(action, worker) {
- $('serviceworker-internals').contentWindow.postMessage({
- 'action': action,
- 'worker': worker
- },'chrome://serviceworker-internals');
-}
-
function removeChildren(element_id) {
var element = $(element_id);
element.textContent = '';
@@ -58,15 +51,6 @@ function onload() {
onHashChange();
initSettings();
sendCommand('init-ui');
- window.addEventListener('message', onMessage.bind(this), false);
-}
-
-function onMessage(event) {
- if (event.origin != 'chrome://serviceworker-internals') {
- return;
- }
- populateServiceWorkers(event.data.partition_id,
- event.data.workers);
}
function onHashChange() {
@@ -103,39 +87,6 @@ function selectTab(id) {
return true;
}
-function populateServiceWorkers(partition_id, workers) {
- var list = $('service-workers-list-' + partition_id);
- if (workers.length == 0) {
- if (list) {
- list.parentNode.removeChild(list);
- }
- return;
- }
- if (list) {
- list.textContent = '';
- } else {
- list = document.createElement('div');
- list.id = 'service-workers-list-' + partition_id;
- list.className = 'list';
- $('service-workers-list').appendChild(list);
- }
- for (var i = 0; i < workers.length; i++) {
- var worker = workers[i];
- worker.hasCustomInspectAction = true;
- var row = addTargetToList(worker, list, ['scope', 'url']);
- addActionLink(
- row,
- 'inspect',
- sendServiceWorkerCommand.bind(null, 'inspect', worker),
- false);
- addActionLink(
- row,
- 'terminate',
- sendServiceWorkerCommand.bind(null, 'stop', worker),
- false);
- }
-}
-
function populateTargets(source, data) {
if (source == 'local')
populateLocalTargets(data);
@@ -151,8 +102,9 @@ function populateLocalTargets(data) {
removeChildren('apps-list');
removeChildren('others-list');
removeChildren('workers-list');
+ removeChildren('service-workers-list');
- for (var i = 0; i < data.length; i++) {
+ for (var i = 0; i < data.length; i++) {
if (data[i].type === 'page')
addToPagesList(data[i]);
else if (data[i].type === 'background_page')
@@ -161,6 +113,8 @@ function populateLocalTargets(data) {
addToAppsList(data[i]);
else if (data[i].type === 'worker')
addToWorkersList(data[i]);
+ else if (data[i].type === 'service_worker')
+ addToServiceWorkersList(data[i]);
else
addToOthersList(data[i]);
}
@@ -422,6 +376,13 @@ function addToWorkersList(data) {
sendTargetCommand.bind(null, 'close', data), false);
}
+function addToServiceWorkersList(data) {
+ var row = addTargetToList(
+ data, $('service-workers-list'), ['name', 'description', 'url']);
+ addActionLink(row, 'terminate',
+ sendTargetCommand.bind(null, 'close', data), false);
+}
+
function addToOthersList(data) {
addTargetToList(data, $('others-list'), ['url']);
}

Powered by Google App Engine
This is Rietveld 408576698