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

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

Issue 720223002: DevTools: only allow status bar items in status bars. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: rebaselined 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 51e29df1a1a40315153d673bb353481cf40ff1c8..e683875514594a7d402ff6a3b7cd9e0a500a9d6a 100644
--- a/Source/devtools/front_end/resources/ApplicationCacheItemsView.js
+++ b/Source/devtools/front_end/resources/ApplicationCacheItemsView.js
@@ -46,8 +46,7 @@ WebInspector.ApplicationCacheItemsView = function(model, frameId)
this.connectivityMessage.className = "storage-application-cache-connectivity";
this.connectivityMessage.textContent = "";
- this.divider = createElement("span");
- this.divider.className = "status-bar-item status-bar-divider";
+ this.divider = new WebInspector.StatusBarSeparator();
this.statusIcon = createElement("div");
this.statusMessage = createElement("span");
@@ -72,12 +71,18 @@ WebInspector.ApplicationCacheItemsView = function(model, frameId)
}
WebInspector.ApplicationCacheItemsView.prototype = {
- get statusBarItems()
+ /**
+ * @return {!Array.<!WebInspector.StatusBarItem>}
+ */
+ statusBarItems: function()
{
return [
- this.deleteButton.element,
- this.connectivityIcon, this.connectivityMessage, this.divider,
- this.statusIcon, this.statusMessage
+ this.deleteButton,
+ new WebInspector.StatusBarItem(this.connectivityIcon),
+ new WebInspector.StatusBarItem(this.connectivityMessage),
+ this.divider,
+ new WebInspector.StatusBarItem(this.statusIcon),
+ new WebInspector.StatusBarItem(this.statusMessage)
];
},
« 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