Commit 13bfbc41 authored by jeremya@chromium.org's avatar jeremya@chromium.org

Disable 'Create Shortcut' menu item on Mac.

This isn't ready for public use yet, so disable the UI.

BUG=138733


Review URL: https://chromiumcodereview.appspot.com/13722009

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@192903 0039d316-1c4b-4281-b951-d872f2087c98
parent 53d3e902
...@@ -73,7 +73,7 @@ cr.define('ntp', function() { ...@@ -73,7 +73,7 @@ cr.define('ntp', function() {
this.uninstall_.addEventListener('activate', this.uninstall_.addEventListener('activate',
this.onUninstall_.bind(this)); this.onUninstall_.bind(this));
if (!cr.isChromeOS) { if (!(cr.isChromeOS || cr.isMac)) {
this.createShortcutSeparator_ = this.createShortcutSeparator_ =
menu.appendChild(cr.ui.MenuItem.createSeparator()); menu.appendChild(cr.ui.MenuItem.createSeparator());
this.createShortcut_ = this.appendMenuItem_('appcreateshortcut'); this.createShortcut_ = this.appendMenuItem_('appcreateshortcut');
...@@ -140,7 +140,7 @@ cr.define('ntp', function() { ...@@ -140,7 +140,7 @@ cr.define('ntp', function() {
this.details_.disabled = !app.appData.detailsUrl; this.details_.disabled = !app.appData.detailsUrl;
this.uninstall_.disabled = !app.appData.mayDisable; this.uninstall_.disabled = !app.appData.mayDisable;
if (cr.isMac) { if (this.createShortcut_ && cr.isMac) {
// On Windows and Linux, these should always be visible. On ChromeOS, // On Windows and Linux, these should always be visible. On ChromeOS,
// they are never created. On Mac, shortcuts can only be created for // they are never created. On Mac, shortcuts can only be created for
// new-style packaged apps, so hide the menu item. // new-style packaged apps, so hide the menu item.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment