Mac: Generate App Mode Loader bundle + cleanup

chrome/browser/web_applications/web_app_mac.h

Index: chrome/browser/web_applications/web_app_mac.h
diff --git a/chrome/browser/web_applications/web_app_mac.h b/chrome/browser/web_applications/web_app_mac.h
index 261b9d724a4f00e48b7dc11546a6325f8aae60bd..6bf015c81cdcf44b9f5acaeec5662e894bd2f928 100644
--- a/chrome/browser/web_applications/web_app_mac.h
+++ b/chrome/browser/web_applications/web_app_mac.h
@@ -9,6 +9,8 @@
 #include "base/file_path.h"
 #include "chrome/browser/shell_integration.h"
 
+@class NSString;
+
 namespace web_app {
 
 // Creates a shortcut for a web application. The shortcut is a stub app
@@ -20,8 +22,10 @@ class WebAppShortcutCreator {
       const ShellIntegration::ShortcutInfo& shortcut_info);
   virtual ~WebAppShortcutCreator();
 
-  // Creates a shortcut.
-  bool CreateShortcut();
+  // Copies the app launcher template into place and fills in all relevant
+  // information.
+  // |chrome_bundle_id| - CFBundleIdentifier of the Chrome browser bundle.
+  bool CreateShortcut(NSString* chrome_bundle_id);

[sail, 2012/02/17 16:42:09]

Instead of passing the bundle ID to create shortcut it would be better to pass
it to the constructor and save it in a member variable. If you do that it would
also be better to use a string16 instead of a Objective-C object.

[jeremy, 2012/02/19 14:36:41]

Done.

 
  protected:
   // Returns a path to the app loader.
@@ -31,7 +35,7 @@ class WebAppShortcutCreator {
   virtual FilePath GetDestinationPath(const FilePath& app_file_name) const;
 
   // Updates the plist inside |app_path| with information about the app.
-  bool UpdatePlist(const FilePath& app_path) const;
+  bool UpdatePlist(const FilePath& app_path, NSString* chrome_bundle_id) const;
 
   // Updates the icon for the shortcut.
   bool UpdateIcon(const FilePath& app_path) const;