Create hosted app shims on Mac.

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 1123 matching lines @@
   for (std::vector<base::FilePath>::const_iterator it = bundles.begin();
        it != bundles.end(); ++it) {
     web_app::ShortcutInfo shortcut_info =
         BuildShortcutInfoFromBundle(*it);
     WebAppShortcutCreator shortcut_creator(
         it->DirName(), shortcut_info, extensions::FileHandlersInfo());
     shortcut_creator.DeleteShortcuts();
   }
 }
 
+void RevealAppShimInFinderForApp(
+    Profile* profile,
+    const base::FilePath& app_data_path) {
+  web_app::ShortcutInfo shortcut_info =
+      BuildShortcutInfoFromBundle(app_data_path);

[jackhou1, 2014/12/03 05:12:05]

BuildShortcutInfoFromBundle actually reads the bundle from the file system,
which isn't ideal. Would web_app::ShortcutInfoForExtensionAndProfile not work?
It gets all the details you need on the UI thread.

Also, I think it would be neater if RevealAPpShimInFinderForApp can be called
from the UI thread, and posts to the FILE thread if it needs to.

[mitchellj, 2014/12/04 04:53:53]

Done. When using web_app::ShortcutInfoForExtensionAndProfile, turns out the
app_path didn't exist at creation time and so nothing was being revealed. The
RevealAppShimInFinder function has been modified to check if the app_path exists
first, otherwise reveals the directory above it.

+  web_app::WebAppShortcutCreator shortcut_creator(
+      app_data_path, shortcut_info, extensions::FileHandlersInfo());
+  shortcut_creator.RevealAppShimInFinder();
+}
+
 }  // namespace internals
 
 }  // namespace web_app
 
 namespace chrome {
 
 void ShowCreateChromeAppShortcutsDialog(
     gfx::NativeWindow /*parent_window*/,
     Profile* profile,
     const extensions::Extension* app,
     const base::Callback<void(bool)>& close_callback) {
   web_app::GetShortcutInfoForApp(
       app,
       profile,
       base::Bind(&web_app::CreateAppShortcutInfoLoaded,
                  profile,
                  app,
                  close_callback));
 }
 
 }  // namespace chrome