Chromium Code Reviews| Index: chrome/browser/resources/ntp4/apps_page.js |
| diff --git a/chrome/browser/resources/ntp4/apps_page.js b/chrome/browser/resources/ntp4/apps_page.js |
| index 852887d79efd671f1b2e53c2e230620d9d6811b6..f7f076a8276d032765ebae8f18f572d80a51855e 100644 |
| --- a/chrome/browser/resources/ntp4/apps_page.js |
| +++ b/chrome/browser/resources/ntp4/apps_page.js |
| @@ -39,6 +39,12 @@ cr.define('ntp', function() { |
| cr.addSingletonGetter(AppContextMenu); |
| AppContextMenu.prototype = { |
| + /** |
| + * If true, the "Create shortcuts" context menu entry will be hidden. |
| + * @type {boolean} |
| + */ |
| + disableCreateAppShortcut: false, |
|
tapted
2013/05/30 05:32:24
pretty sure this should be in initialize(), with a
|
| + |
| initialize: function() { |
| var menu = new cr.ui.Menu; |
| cr.ui.decorate(menu, cr.ui.Menu); |
| @@ -73,7 +79,7 @@ cr.define('ntp', function() { |
| this.uninstall_.addEventListener('activate', |
| this.onUninstall_.bind(this)); |
| - if (!(cr.isChromeOS || cr.isMac)) { |
| + if (!cr.isChromeOS) { |
| this.createShortcutSeparator_ = |
| menu.appendChild(cr.ui.MenuItem.createSeparator()); |
| this.createShortcut_ = this.appendMenuItem_('appcreateshortcut'); |
| @@ -140,12 +146,12 @@ cr.define('ntp', function() { |
| this.details_.disabled = !app.appData.detailsUrl; |
| this.uninstall_.disabled = !app.appData.mayDisable; |
| - if (this.createShortcut_ && cr.isMac) { |
| + if (cr.isMac) { |
| // On Windows and Linux, these should always be visible. On ChromeOS, |
| // they are never created. On Mac, shortcuts can only be created for |
| // new-style packaged apps, so hide the menu item. |
| this.createShortcutSeparator_.hidden = this.createShortcut_.hidden = |
| - !app.appData.packagedApp; |
| + !app.appData.packagedApp || this.disableCreateAppShortcut; |
| } |
| }, |
| @@ -549,6 +555,13 @@ cr.define('ntp', function() { |
| }, |
| /** |
| + * Disable the "Create shortcut" context menu item. |
| + */ |
| + disableCreateAppShortcut: function() { |
| + AppContextMenu.getInstance().disableCreateAppShortcut = true; |
| + }, |
| + |
| + /** |
| * Highlight a newly installed app as it's added to the NTP. |
| * @param {Object} appData The data object that describes the app. |
| */ |