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

Unified Diff: Source/devtools/front_end/resources/ApplicationCacheItemsView.js

Issue 722713002: DevTools: get rid of getters and setters in StatusBarButton. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 1 month 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: Source/devtools/front_end/resources/ApplicationCacheItemsView.js
diff --git a/Source/devtools/front_end/resources/ApplicationCacheItemsView.js b/Source/devtools/front_end/resources/ApplicationCacheItemsView.js
index be5caeddab31a9674d44c87d3bf7e50a7f5d5d12..51e29df1a1a40315153d673bb353481cf40ff1c8 100644
--- a/Source/devtools/front_end/resources/ApplicationCacheItemsView.js
+++ b/Source/devtools/front_end/resources/ApplicationCacheItemsView.js
@@ -38,7 +38,7 @@ WebInspector.ApplicationCacheItemsView = function(model, frameId)
// FIXME: Needs better tooltip. (Localized)
this.deleteButton = new WebInspector.StatusBarButton(WebInspector.UIString("Delete"), "delete-storage-status-bar-item");
- this.deleteButton.visible = false;
+ this.deleteButton.setVisible(false);
this.deleteButton.addEventListener("click", this._deleteButtonClicked, this);
this.connectivityIcon = createElement("div");
@@ -88,7 +88,7 @@ WebInspector.ApplicationCacheItemsView.prototype = {
willHide: function()
{
- this.deleteButton.visible = false;
+ this.deleteButton.setVisible(false);
},
_maybeUpdate: function()
@@ -164,7 +164,7 @@ WebInspector.ApplicationCacheItemsView.prototype = {
delete this._resources;
this._emptyView.show(this.element);
- this.deleteButton.visible = false;
+ this.deleteButton.setVisible(false);
if (this._dataGrid)
this._dataGrid.element.classList.add("hidden");
return;
@@ -183,7 +183,7 @@ WebInspector.ApplicationCacheItemsView.prototype = {
this._dataGrid.autoSizeColumns(20, 80);
this._dataGrid.element.classList.remove("hidden");
this._emptyView.detach();
- this.deleteButton.visible = true;
+ this.deleteButton.setVisible(true);
// FIXME: For Chrome, put creationTime and updateTime somewhere.
// NOTE: localizedString has not yet been added.
« no previous file with comments | « Source/devtools/front_end/promises/PromisePane.js ('k') | Source/devtools/front_end/resources/CookieItemsView.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698