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

Unified Diff: third_party/WebKit/Source/devtools/front_end/sources/TabbedEditorContainer.js

Issue 2594883002: DevTools: [Persistence] introduce badged icons instead of checkmarks (Closed)
Patch Set: address comments Created 4 years 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/sources/TabbedEditorContainer.js
diff --git a/third_party/WebKit/Source/devtools/front_end/sources/TabbedEditorContainer.js b/third_party/WebKit/Source/devtools/front_end/sources/TabbedEditorContainer.js
index 1a8526d8fe07500c1734acd0ba6a79c024f6e34c..1410bfe8e381ad0b3b500df67ff83bc2c63fda28 100644
--- a/third_party/WebKit/Source/devtools/front_end/sources/TabbedEditorContainer.js
+++ b/third_party/WebKit/Source/devtools/front_end/sources/TabbedEditorContainer.js
@@ -537,10 +537,8 @@ Sources.TabbedEditorContainer = class extends Common.Object {
if (Persistence.persistence.hasUnsavedCommittedChanges(uiSourceCode)) {
icon = UI.Icon.create('smallicon-warning');
icon.title = Common.UIString('Changes to this file were not saved to file system.');
- } else if (Runtime.experiments.isEnabled('persistence2') && Persistence.persistence.binding(uiSourceCode)) {
- var binding = Persistence.persistence.binding(uiSourceCode);
- icon = UI.Icon.create('smallicon-green-checkmark');
- icon.title = Persistence.PersistenceUtils.tooltipForUISourceCode(binding.fileSystem);
+ } else {
+ icon = Persistence.PersistenceUtils.iconForUISourceCode(uiSourceCode);
}
this._tabbedPane.setTabIcon(tabId, icon);
}

Powered by Google App Engine
This is Rietveld 408576698