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..1a02d2cbdd05d31054718028f1adf3e2ed98159f 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|. |
| + // A copy of the shortcut is placed in |web_app_path|. |
| // |chrome_bundle_id| is the CFBundleIdentifier of the Chrome browser bundle. |
| WebAppShortcutCreator( |
| - const base::FilePath& user_data_dir, |
| + const base::FilePath& web_app_path, |
|
benwells
2013/06/13 04:40:33
nit: web_app_path is a bad name. These aren't web
jackhou1
2013/06/14 03:31:27
Done.
|
| 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 data directory for this app. For example: |
| + // ~/Library/Application Support/Chromium/Default/Web Applications/_crx_abc/ |
| + base::FilePath web_app_path_; |
| + |
| // Information about the app. |
| ShellIntegration::ShortcutInfo info_; |