Chromium Code Reviews| 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 676b20872badfb171ce62fa197b24a3fd63d1a50..282b03fe8a52adb59884d4fd29c5d154466e500b 100644 |
| --- a/chrome/browser/web_applications/web_app_mac.h |
| +++ b/chrome/browser/web_applications/web_app_mac.h |
| @@ -33,10 +33,10 @@ void MaybeLaunchShortcut(const ShellIntegration::ShortcutInfo& shortcut_info); |
| 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 ea
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); |
| @@ -69,11 +69,6 @@ class WebAppShortcutCreator { |
| // 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. |
| @@ -83,6 +78,10 @@ class WebAppShortcutCreator { |
| 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_; |