Create a copy of each app's shim bundle in the app's app_data_path.

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 "base/files/file_path.h"
 #include "base/gtest_prod_util.h"
 #include "base/strings/string16.h"
@@ skipping 15 matching lines @@
     const ShellIntegration::ShortcutInfo& shortcut_info);
 
 // If necessary, launch the shortcut for an app.
 void MaybeLaunchShortcut(const ShellIntegration::ShortcutInfo& shortcut_info);
 
 // 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|.
+  // The shortcut stores its user data directory in |web_app_path|.

[benwells, 2013/06/13 00:13:31]

This comment doesn't make sense, it was from an earlier era where we ran apps in
their own user data directory.

Update it to describe what web_app_path is now used for.

[jackhou1, 2013/06/13 01:33:43]

Done.

   // |chrome_bundle_id| is the CFBundleIdentifier of the Chrome browser bundle.
   WebAppShortcutCreator(
-      const base::FilePath& user_data_dir,
+      const base::FilePath& web_app_path,
       const ShellIntegration::ShortcutInfo& shortcut_info,
       const string16& chrome_bundle_id);
 
   virtual ~WebAppShortcutCreator();
 
   // Returns a path to the destination where the app should be written to.
   base::FilePath GetShortcutPath() const;
 
   // Copies the app launcher template into place and fills in all relevant
   // information.
@@ skipping 12 matching lines @@
   // Updates the icon for the shortcut.
   bool UpdateIcon(const base::FilePath& app_path) const;
 
  private:
   FRIEND_TEST_ALL_PREFIXES(WebAppShortcutCreatorTest, UpdateIcon);
 
   // 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_;
-
   // 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;
 
   // Show the bundle we just generated in the Finder.
   virtual void RevealGeneratedBundleInFinder(
       const base::FilePath& generated_bundle) const;
 
+  // Path to the app's user data directory. For example:

[benwells, 2013/06/13 00:13:31]

Ditto. This isn't the app's user data directory.

[jackhou1, 2013/06/13 01:33:43]

Done.

+  // ~/Library/Application Support/Chromium/Default/Web Applications/_crx_abc/
+  base::FilePath web_app_path_;
+
   // Information about the app.
   ShellIntegration::ShortcutInfo info_;
 
   // The CFBundleIdentifier of the Chrome browser bundle.
   string16 chrome_bundle_id_;
 };
 
 }  // namespace web_app
 
 #endif  // CHROME_BROWSER_WEB_APPLICATIONS_WEB_APP_MAC_H_