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

Unified Diff: chrome/browser/resources/options/cookies_list.js

Issue 7491085: Revert 95607 - Adding usage entry to chrome://settings/cookies. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 4 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/options/cookies_list.js
===================================================================
--- chrome/browser/resources/options/cookies_list.js (revision 95777)
+++ chrome/browser/resources/options/cookies_list.js (working copy)
@@ -119,8 +119,6 @@
this.siteChild.className = 'cookie-site';
this.dataChild = this.ownerDocument.createElement('div');
this.dataChild.className = 'cookie-data';
- this.sizeChild = this.ownerDocument.createElement('div');
- this.sizeChild.className = 'cookie-size';
this.itemsChild = this.ownerDocument.createElement('div');
this.itemsChild.className = 'cookie-items';
this.infoChild = this.ownerDocument.createElement('div');
@@ -133,7 +131,6 @@
var content = this.contentElement;
content.appendChild(this.siteChild);
content.appendChild(this.dataChild);
- content.appendChild(this.sizeChild);
content.appendChild(this.itemsChild);
this.itemsChild.appendChild(this.infoChild);
if (this.origin && this.origin.data) {
@@ -253,10 +250,6 @@
else
text = list[i];
this.dataChild.textContent = text;
- if (info.quota && info.quota.totalUsage) {
- this.sizeChild.textContent = info.quota.totalUsage;
- }
-
if (this.expanded)
this.updateItems_();
},
@@ -437,21 +430,18 @@
for (var i = 0; i < this.children.length; ++i)
this.children[i].collectSummaryInfo(info);
} else if (this.data && !this.data.hasChildren) {
- if (this.data.type == 'cookie') {
+ if (this.data.type == 'cookie')
info.cookies++;
- } else if (this.data.type == 'database') {
+ else if (this.data.type == 'database')
info.database = true;
- } else if (this.data.type == 'local_storage') {
+ else if (this.data.type == 'local_storage')
info.localStorage = true;
- } else if (this.data.type == 'app_cache') {
+ else if (this.data.type == 'app_cache')
info.appCache = true;
- } else if (this.data.type == 'indexed_db') {
+ else if (this.data.type == 'indexed_db')
info.indexedDb = true;
- } else if (this.data.type == 'file_system') {
+ else if (this.data.type == 'file_system')
info.fileSystem = true;
- } else if (this.data.type == 'quota') {
- info.quota = this.data;
- }
}
},
@@ -484,8 +474,6 @@
text = localStrings.getString('cookie_file_system');
break;
}
- if (!text)
- return;
var div = item.ownerDocument.createElement('div');
div.className = 'cookie-item';
// Help out screen readers and such: this is a clickable thing.
« no previous file with comments | « chrome/browser/mock_browsing_data_quota_helper.cc ('k') | chrome/browser/resources/options/cookies_view.css » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698