Publish app shortcuts on Mac

chrome/browser/web_applications/web_app.h

Index: chrome/browser/web_applications/web_app.h
diff --git a/chrome/browser/web_applications/web_app.h b/chrome/browser/web_applications/web_app.h
index 16175fc0f43a8460798b961be16526661a2bee2c..542f5dbecc8353ac0bad5bfacc424b839a8bad0b 100644
--- a/chrome/browser/web_applications/web_app.h
+++ b/chrome/browser/web_applications/web_app.h
@@ -67,13 +67,21 @@ std::string GetWMClassFromAppName(std::string app_name);
 namespace internals {
 
 #if defined(OS_WIN)
-FilePath GetSanitizedFileName(const string16& name);
-
 bool CheckAndSaveIcon(const FilePath& icon_file, const SkBitmap& image);
 #endif
 
 FilePath GetWebAppDataDirectory(const FilePath& root_dir,
                                 const ShellIntegration::ShortcutInfo& info);
+
+// Does the actual job of creating a shortcut (see CreateShortcut above).

[jeremy, 2012/02/08 09:10:45]

nit: "CreateShortcut()"

[---DO-NOT-USE---rsesek1, 2012/02/08 12:24:33]

Does this have to only be called on the FILE thread?

[sail, 2012/02/08 19:09:15]

Done.

[sail, 2012/02/08 19:09:15]

Yea. Ahh I forgot to add the DCHECK() on the Mac version. fixed.

[sail, 2012/02/08 19:09:15]

Yea. Ahh I forgot to add the DCHECK() on the Mac version. fixed.

+// This must be called on the file thread.
+void CreateShortcutTask(const FilePath& web_app_path,
+                        const FilePath& profile_path,
+                        const ShellIntegration::ShortcutInfo& shortcut_info);
+
+// Returns sanitized name that could be used as a file name

[jeremy, 2012/02/08 09:10:45]

How about:
Sanitizes |name| and returns a version of it that is safe to use as an on-disk
file name .

[sail, 2012/02/08 19:09:15]

Done.

+FilePath GetSanitizedFileName(const string16& name);
+
 }  // namespace internals
 
 }  // namespace web_app