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

Unified Diff: Source/devtools/front_end/sdk/ServiceWorkerManager.js

Issue 1164583002: [4/5 blink] Shows the clients which are controlled by ServiceWorker in DevTools. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: incorporated pfeldman's comment Created 5 years, 6 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
« no previous file with comments | « Source/devtools/front_end/resources/serviceWorkersView.css ('k') | Source/devtools/protocol.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/devtools/front_end/sdk/ServiceWorkerManager.js
diff --git a/Source/devtools/front_end/sdk/ServiceWorkerManager.js b/Source/devtools/front_end/sdk/ServiceWorkerManager.js
index 2602308d39c450ab8d53c72fc3dbe039a9607377..bc01e4fddbede223754e8fceb44bb077db4f4fe9 100644
--- a/Source/devtools/front_end/sdk/ServiceWorkerManager.js
+++ b/Source/devtools/front_end/sdk/ServiceWorkerManager.js
@@ -162,6 +162,14 @@ WebInspector.ServiceWorkerManager.prototype = {
},
/**
+ * @param {!ServiceWorkerAgent.TargetID} targetId
+ */
+ activateTarget: function(targetId)
+ {
+ this._agent.activateTarget(targetId);
+ },
+
+ /**
* @param {string} scope
*/
_unregister: function(scope)
@@ -202,6 +210,31 @@ WebInspector.ServiceWorkerManager.prototype = {
},
/**
+ * @param {!ServiceWorkerAgent.TargetID} targetId
+ * @param {function(?WebInspector.TargetInfo)=} callback
+ */
+ getTargetInfo: function(targetId, callback)
+ {
+ /**
+ * @param {?Protocol.Error} error
+ * @param {?ServiceWorkerAgent.TargetInfo} targetInfo
+ */
+ function innerCallback(error, targetInfo)
+ {
+ if (error) {
+ console.error(error);
+ callback(null);
+ return;
+ }
+ if (targetInfo)
+ callback(new WebInspector.TargetInfo(targetInfo));
+ else
+ callback(null)
+ }
+ this._agent.getTargetInfo(targetId, innerCallback);
+ },
+
+ /**
* @param {string} workerId
* @param {string} url
*/
@@ -502,6 +535,35 @@ WebInspector.ServiceWorkerConnection.prototype = {
/**
* @constructor
+ * @param {!ServiceWorkerAgent.TargetInfo} payload
+ */
+WebInspector.TargetInfo = function(payload)
+{
+ this.id = payload.id;
+ this.type = payload.type;
+ this.title = payload.title;
+ this.url = payload.url;
+}
+
+WebInspector.TargetInfo.prototype = {
+ /**
+ * @return {boolean}
+ */
+ isWebContents: function()
+ {
+ return this.type == "web_contents";
+ },
+ /**
+ * @return {boolean}
+ */
+ isFrame: function()
+ {
+ return this.type == "frame";
+ },
+}
+
+/**
+ * @constructor
* @param {!ServiceWorkerAgent.ServiceWorkerErrorMessage} payload
*/
WebInspector.ServiceWorkerErrorMessage = function(payload)
@@ -537,6 +599,10 @@ WebInspector.ServiceWorkerVersion.prototype = {
this.status = payload.status;
this.scriptLastModified = payload.scriptLastModified;
this.scriptResponseTime = payload.scriptResponseTime;
+ this.controlledClients = []
+ for (var i = 0; i < payload.controlledClients.length; ++i) {
+ this.controlledClients.push(payload.controlledClients[i]);
+ }
},
/**
« no previous file with comments | « Source/devtools/front_end/resources/serviceWorkersView.css ('k') | Source/devtools/protocol.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698