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

Unified Diff: chrome/common/mac/app_mode_common.h

Issue 9351014: Mac app mode: locate Chrome + refactor (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Path for landing1 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
« no previous file with comments | « chrome/common/mac/app_mode_chrome_locator_unittest.mm ('k') | chrome/common/mac/app_mode_common.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/mac/app_mode_common.h
diff --git a/chrome/common/mac/app_mode_common.h b/chrome/common/mac/app_mode_common.h
index bf43f3293daf3d78718f6dff6ecbad747510330f..5267cd495c35d29127089746c98bce1c41cd82be 100644
--- a/chrome/common/mac/app_mode_common.h
+++ b/chrome/common/mac/app_mode_common.h
@@ -8,6 +8,9 @@
#import <Foundation/Foundation.h>
+#include "base/file_path.h"
+#include "base/string16.h"
+
// This file contains constants, interfaces, etc. which are common to the
// browser application and the app mode loader (a.k.a. shim).
@@ -44,6 +47,10 @@ const unsigned kCurrentChromeAppModeInfoMinorVersion = 0;
// version number. It may refuse to load if the major version of the structure
// is different from the one it accepts.
struct ChromeAppModeInfo {
+ public:
+ ChromeAppModeInfo();
+ ~ChromeAppModeInfo();
+
// Major and minor version number of this structure.
unsigned major_version; // Required: all versions
unsigned minor_version; // Required: all versions
@@ -53,25 +60,25 @@ struct ChromeAppModeInfo {
char** argv; // Required: v1.0
// Versioned path to the browser which is being loaded.
- char* chrome_versioned_path; // Required: v1.0
+ FilePath chrome_versioned_path; // Required: v1.0
// Information about the App Mode shortcut:
// Path to the App Mode Loader application bundle originally run.
- char* app_mode_bundle_path; // Optional: v1.0
+ FilePath app_mode_bundle_path; // Optional: v1.0
// Short ID string, preferably derived from |app_mode_short_name|. Should be
// safe for the file system.
- char* app_mode_id; // Required: v1.0
+ std::string app_mode_id; // Required: v1.0
// Short (e.g., one-word) UTF8-encoded name for the shortcut.
- char* app_mode_short_name; // Optional: v1.0
+ string16 app_mode_short_name; // Optional: v1.0
// Unrestricted (e.g., several-word) UTF8-encoded name for the shortcut.
- char* app_mode_name; // Optional: v1.0
+ string16 app_mode_name; // Optional: v1.0
// URL for the shortcut. Must be a valid URL.
- char* app_mode_url; // Required: v1.0
+ std::string app_mode_url; // Required: v1.0
};
} // namespace app_mode
« no previous file with comments | « chrome/common/mac/app_mode_chrome_locator_unittest.mm ('k') | chrome/common/mac/app_mode_common.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698