Recreate shortcuts on app update.

chrome/browser/web_applications/web_app_mac.h

 // 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.
 
 #ifndef CHROME_BROWSER_WEB_APPLICATIONS_WEB_APP_MAC_H_
 #define CHROME_BROWSER_WEB_APPLICATIONS_WEB_APP_MAC_H_
 
+#include <vector>
+
 #include "base/files/file_path.h"
 #include "base/gtest_prod_util.h"
 #include "base/strings/string16.h"
 #include "chrome/browser/shell_integration.h"
 
 #ifdef __OBJC__
 @class NSDictionary;
 @class NSString;
 #else  // __OBJC__
 class NSDictionary;
@@ skipping 13 matching lines @@
 // Creates a shortcut for a web application. The shortcut is a stub app
 // that simply loads the browser framework and runs the given app.
 class WebAppShortcutCreator {
  public:
   // Creates a new shortcut based on information in |shortcut_info|.
   // The shortcut stores its user data directory in |user_data_dir|.
   // |chrome_bundle_id| is the CFBundleIdentifier of the Chrome browser bundle.
   WebAppShortcutCreator(
       const base::FilePath& user_data_dir,
       const ShellIntegration::ShortcutInfo& shortcut_info,
-      const string16& chrome_bundle_id);
+      const std::string& chrome_bundle_id);
 
   virtual ~WebAppShortcutCreator();
 
-  // Returns a path to the destination where the app should be written to.
-  base::FilePath GetShortcutPath() const;
+  // Returns the base name for the shortcut.
+  base::FilePath GetShortcutName() const;
 
-  // Copies the app launcher template into place and fills in all relevant
-  // information.
+  // Returns a path to the Chrome Apps folder.

[tapted, 2013/06/18 06:54:59]

maybe mention that it's a subfolder of /Applications or ~/Applications since
there will be multiple destination paths

[jackhou1, 2013/06/18 08:32:40]

Done.

+  virtual base::FilePath GetDestinationPath() const;
+
+  // Creates shortcuts for the given shortcut info.

[tapted, 2013/06/18 06:54:59]

I don't think these comments on FooShortcut[s]() are adding anything

[jackhou1, 2013/06/18 08:32:40]

Removed.

   bool CreateShortcut();

[tapted, 2013/06/18 06:54:59]

make this plural to be consistent? (also, it's true now ;)

[jackhou1, 2013/06/18 08:32:40]

Done.

 
+  // Deletes shortcuts for the given shortcut info.
+  void DeleteShortcuts();
+
+  // Updates shortcuts for the given shortcut info.
+  bool UpdateShortcuts();
+
  protected:
   // Returns a path to the app loader.
   base::FilePath GetAppLoaderPath() const;
 
-  // Returns a path to the destination where the app should be written to.
-  virtual base::FilePath GetDestinationPath() const;
-
   // Updates the plist inside |app_path| with information about the app.
   bool UpdatePlist(const base::FilePath& app_path) const;
 
   // Updates the icon for the shortcut.
   bool UpdateIcon(const base::FilePath& app_path) const;
 
+  // Returns a path to an app bundle with the given id. Or an empty path if no
+  // matching bundle was found.
+  // Here so it can be mocked out for testing.

[tapted, 2013/06/18 06:54:59]

nit: "Here" -> Protected and virtual

[jackhou1, 2013/06/18 08:32:40]

Done.

+  virtual base::FilePath GetAppBundleById(const std::string& bundle_id) const;
+
  private:
   FRIEND_TEST_ALL_PREFIXES(WebAppShortcutCreatorTest, UpdateIcon);
+  FRIEND_TEST_ALL_PREFIXES(WebAppShortcutCreatorTest, UpdateShortcuts);
+
+  // Copies the app loader template into a temporary directory and fills in all
+  // relevant information.
+  bool BuildShortcut(const base::FilePath& staging_path) const;
+
+  // Builds a shortcut and copies it into the given destination folders.
+  bool CreateShortcutsIn(const std::vector<base::FilePath>& folders) const;
 
   // Updates the InfoPlist.string inside |app_path| with the display name for
   // the app.
   bool UpdateDisplayName(const base::FilePath& app_path) const;
 
   // Path to the app's user data directory. For example:
   // ~/Library/Application Support/Chromium/Default/Web Applications/_crx_abc/
   // Note, the user data directory is the parent of the profile directory.
   base::FilePath user_data_dir_;

[tapted, 2013/06/18 06:54:59]

has the change that moved this landed?. rebase?

[jackhou1, 2013/06/18 08:32:40]

Done.

 
   // Returns the bundle identifier to use for this app bundle.
-  // |plist| is a dictionary containg a copy of the template plist file to
-  // be used for creating the app bundle.
-  NSString* GetBundleIdentifier(NSDictionary* plist) const;
+  std::string GetBundleIdentifier() const;
 
   // Show the bundle we just generated in the Finder.
-  virtual void RevealGeneratedBundleInFinder(
-      const base::FilePath& generated_bundle) const;
+  virtual void RevealAppShimInFinder() const;
 
   // Information about the app.
   ShellIntegration::ShortcutInfo info_;
 
   // The CFBundleIdentifier of the Chrome browser bundle.
-  string16 chrome_bundle_id_;
+  std::string chrome_bundle_id_;
 };
 
 }  // namespace web_app
 
 #endif  // CHROME_BROWSER_WEB_APPLICATIONS_WEB_APP_MAC_H_