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

Unified Diff: chrome/browser/resources/md_extensions/manager.js

Issue 2974583003: MD Extensions: Show messages for no extensions, and for no search results (Closed)
Patch Set: add md_extensions_strings.grdp=* in OWNER Created 3 years, 5 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/md_extensions/manager.js
diff --git a/chrome/browser/resources/md_extensions/manager.js b/chrome/browser/resources/md_extensions/manager.js
index 4bac6d83e6b8bb722028adb6af38bd5418a394b3..f35cad70cdf2d8fc2be012187b9efb1600d5aef8 100644
--- a/chrome/browser/resources/md_extensions/manager.js
+++ b/chrome/browser/resources/md_extensions/manager.js
@@ -98,6 +98,15 @@ cr.define('extensions', function() {
return [];
},
},
+
+ /**
+ * Prevents page content from showing before data is first loaded.
+ * @private
+ */
+ didInitPage_: {
+ type: Boolean,
+ value: false,
+ },
},
listeners: {
@@ -171,6 +180,7 @@ cr.define('extensions', function() {
* the user visits chrome://extensions/?id=..., we land on the proper page.
*/
initPage: function() {
+ this.didInitPage_ = true;
this.changePage(this.navigationHelper_.getCurrentPage(), true);
},
@@ -235,14 +245,6 @@ cr.define('extensions', function() {
this.apps[this.getIndexInList_('apps', id)];
},
- /**
- * @return {boolean} Whether the list should be visible.
- * @private
- */
- computeListHidden_: function() {
- return this.$['items-list'].items.length == 0;
- },
-
/**
* Creates and adds a new extensions-item element to the list, inserting it
* into its sorted position in the relevant section.
« no previous file with comments | « chrome/browser/resources/md_extensions/manager.html ('k') | chrome/browser/ui/webui/extensions/extensions_ui.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698