Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2939)

Unified Diff: chrome/browser/web_applications/web_app_mac.mm

Issue 9374009: Install platform apps into a separate data directory (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix test Created 8 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/web_applications/web_app_mac.mm
diff --git a/chrome/browser/web_applications/web_app_mac.mm b/chrome/browser/web_applications/web_app_mac.mm
index 8ccaa978f8029435947f4732d688a5b121637694..da2fdfce7bfb78228b06ac1fd6aa0e201642270f 100644
--- a/chrome/browser/web_applications/web_app_mac.mm
+++ b/chrome/browser/web_applications/web_app_mac.mm
@@ -20,8 +20,9 @@
namespace web_app {
WebAppShortcutCreator::WebAppShortcutCreator(
+ const FilePath& web_app_path,
const ShellIntegration::ShortcutInfo& shortcut_info)
- : info_(shortcut_info) {
+ : web_app_path_(web_app_path), info_(shortcut_info) {
}
WebAppShortcutCreator::~WebAppShortcutCreator() {
@@ -95,6 +96,8 @@ bool WebAppShortcutCreator::UpdatePlist(const FilePath& app_path) const {
forKey:app_mode::kCrAppModeShortcutNameKey];
[dict setObject:base::SysUTF8ToNSString(info_.url.spec())
forKey:app_mode::kCrAppModeShortcutURLKey];
+ [dict setObject:base::mac::FilePathToNSString(web_app_path_)
+ forKey:app_mode::kCrAppModeUserDataDirKey];
return [dict writeToFile:plist_path atomically:YES];
}
@@ -111,7 +114,7 @@ namespace internals {
void CreateShortcutTask(const FilePath& web_app_path,
const FilePath& profile_path,
const ShellIntegration::ShortcutInfo& shortcut_info) {
- WebAppShortcutCreator shortcut_creator(shortcut_info);
+ WebAppShortcutCreator shortcut_creator(web_app_path, shortcut_info);
shortcut_creator.CreateShortcut();
}

Powered by Google App Engine
This is Rietveld 408576698