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

Unified Diff: third_party/WebKit/Source/devtools/front_end/resources/ResourcesPanel.js

Issue 2623063003: DevTools: Fix getCookies to report for all resources (Closed)
Patch Set: rebase master Created 3 years, 11 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: third_party/WebKit/Source/devtools/front_end/resources/ResourcesPanel.js
diff --git a/third_party/WebKit/Source/devtools/front_end/resources/ResourcesPanel.js b/third_party/WebKit/Source/devtools/front_end/resources/ResourcesPanel.js
index 65a73cf6bdfaa0bae5eb008a23aa361f250f6fd8..58a3e642cdab996bb9782eb1741aa93a34e3d2c1 100644
--- a/third_party/WebKit/Source/devtools/front_end/resources/ResourcesPanel.js
+++ b/third_party/WebKit/Source/devtools/front_end/resources/ResourcesPanel.js
@@ -390,15 +390,18 @@ Resources.ResourcesPanel = class extends UI.PanelWithSidebar {
this.databasesListTreeElement.appendChild(databaseTreeElement);
}
- addDocumentURL(url) {
- var parsedURL = url.asParsedURL();
+ /**
+ * @param {!SDK.ResourceTreeFrame} frame
+ */
+ addCookieDocument(frame) {
+ var parsedURL = frame.url.asParsedURL();
if (!parsedURL)
return;
var domain = parsedURL.securityOrigin();
if (!this._domains[domain]) {
this._domains[domain] = true;
- var cookieDomainTreeElement = new Resources.CookieTreeElement(this, domain);
+ var cookieDomainTreeElement = new Resources.CookieTreeElement(this, frame, domain);
this.cookieListTreeElement.appendChild(cookieDomainTreeElement);
}
}
@@ -582,11 +585,12 @@ Resources.ResourcesPanel = class extends UI.PanelWithSidebar {
/**
* @param {!Resources.CookieTreeElement} treeElement
* @param {string} cookieDomain
+ * @param {!SDK.ResourceTreeFrame} cookieFrameTarget
*/
- showCookies(treeElement, cookieDomain) {
+ showCookies(treeElement, cookieDomain, cookieFrameTarget) {
var view = this._cookieViews[cookieDomain];
if (!view) {
- view = new Resources.CookieItemsView(treeElement, cookieDomain);
+ view = new Resources.CookieItemsView(treeElement, cookieFrameTarget, cookieDomain);
this._cookieViews[cookieDomain] = view;
}
@@ -945,7 +949,7 @@ Resources.FrameTreeElement = class extends Resources.BaseStorageTreeElement {
this._categoryElements = {};
this._treeElementForResource = {};
- this._storagePanel.addDocumentURL(frame.url);
+ this._storagePanel.addCookieDocument(frame);
}
get itemURL() {
@@ -1922,10 +1926,11 @@ Resources.DOMStorageTreeElement = class extends Resources.BaseStorageTreeElement
* @unrestricted
*/
Resources.CookieTreeElement = class extends Resources.BaseStorageTreeElement {
- constructor(storagePanel, cookieDomain) {
+ constructor(storagePanel, frame, cookieDomain) {
super(
storagePanel, cookieDomain ? cookieDomain : Common.UIString('Local Files'),
['cookie-tree-item', 'resource-tree-item']);
+ this._frame = frame;
this._cookieDomain = cookieDomain;
}
@@ -1963,7 +1968,7 @@ Resources.CookieTreeElement = class extends Resources.BaseStorageTreeElement {
*/
onselect(selectedByUser) {
super.onselect(selectedByUser);
- this._storagePanel.showCookies(this, this._cookieDomain);
+ this._storagePanel.showCookies(this, this._cookieDomain, this._frame.target());
return false;
}
};

Powered by Google App Engine
This is Rietveld 408576698