Publish app shortcuts on Mac

chrome/browser/web_applications/web_app.h

 // Copyright (c) 2011 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_H_
 #define CHROME_BROWSER_WEB_APPLICATIONS_WEB_APP_H_
 #pragma once
 
 #include <string>
 #include <vector>
@@ skipping 49 matching lines @@
 #if defined(TOOLKIT_USES_GTK)
 // GTK+ windows that correspond to web apps need to have a deterministic (and
 // different) WMClass than normal chrome windows so the window manager groups
 // them as a separate application.
 std::string GetWMClassFromAppName(std::string app_name);
 #endif
 
 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
 
 #endif  // CHROME_BROWSER_WEB_APPLICATIONS_WEB_APP_H_