Recreate shortcuts on app update.

chrome/browser/web_applications/web_app_mac.mm

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "chrome/browser/web_applications/web_app_mac.h"
 
 #import <Carbon/Carbon.h>
 #import <Cocoa/Cocoa.h>
 
 #include "base/command_line.h"
@@ skipping 419 matching lines @@
 
   NSString* path_string = [base::mac::CFToNSCast(url.get()) path];
   return base::FilePath([path_string fileSystemRepresentation]);
 }
 
 bool CreatePlatformShortcuts(
     const base::FilePath& web_app_path,
     const ShellIntegration::ShortcutInfo& shortcut_info,
     const ShellIntegration::ShortcutLocations& /*creation_locations*/) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
-  string16 bundle_id = UTF8ToUTF16(base::mac::BaseBundleID());
-  WebAppShortcutCreator shortcut_creator(web_app_path, shortcut_info,
-                            bundle_id);
+  WebAppShortcutCreator shortcut_creator(
+      web_app_path, shortcut_info, UTF8ToUTF16(base::mac::BaseBundleID()));
   return shortcut_creator.CreateShortcut();
 }
 
 void DeletePlatformShortcuts(
     const base::FilePath& web_app_path,
-    const ShellIntegration::ShortcutInfo& info) {
+    const ShellIntegration::ShortcutInfo& shortcut_info) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
-
-  base::FilePath bundle_path = GetAppBundleByExtensionId(info.extension_id);
-  file_util::Delete(bundle_path, true);
+  WebAppShortcutCreator shortcut_creator(
+      web_app_path, shortcut_info, UTF8ToUTF16(base::mac::BaseBundleID()));
+  file_util::Delete(shortcut_creator.GetShortcutPath(), true);

[tapted, 2013/06/11 06:40:08]

web_app_win.cc also checks whether the `Chrome Apps` folder becomes empty. Let's
do the same.

[jackhou1, 2013/06/12 07:43:50]

Done.

 }
 
 void UpdatePlatformShortcuts(
     const base::FilePath& web_app_path,
-    const string16& old_app_title,
+    const string16& /*old_app_title*/,
     const ShellIntegration::ShortcutInfo& shortcut_info) {
-  // TODO(benwells): Implement this when shortcuts / weblings are enabled on
-  // mac.
+  DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
+  WebAppShortcutCreator shortcut_creator(
+      web_app_path, shortcut_info, UTF8ToUTF16(base::mac::BaseBundleID()));
+  shortcut_creator.CreateShortcut();

[tapted, 2013/06/11 06:40:08]

I wonder if there are any downsides to copying over it in-place. E.g.
web_app_win will delete things first. And Mac does complain sometimes. E.g. If I
try to rename an applications that's running, I get a "Are you sure you want to
change the name of foo while it's open" dialog.

I can't really think of a downside right now.. But if/when we switch to writing
multiple locales in the display name, then you can end up leaving stray files
around if an extension deletes a locale in an update. Maybe that's subtle enough
that we will forget.

How much uglier is it to delete-first? (and is OSX OK with that?). I think
chrome installs new versions to a subfolder of its .app and then atomically
switches over somehow.

[jackhou1, 2013/06/12 07:43:50]

Seems to be fine to delete first. The app is restarted on update, so the shim
shuts down anyway.

 }
 
 }  // namespace internals
 
 }  // namespace web_app