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

Unified Diff: ios/chrome/common/app_group/app_group_constants.mm

Issue 2570043002: [ios] Adds a buildflag for the ios_app_bundle_id_prefix. (Closed)
Patch Set: Created 4 years 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
« build/config/ios/BUILD.gn ('K') | « ios/chrome/common/app_group/BUILD.gn ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/common/app_group/app_group_constants.mm
diff --git a/ios/chrome/common/app_group/app_group_constants.mm b/ios/chrome/common/app_group/app_group_constants.mm
index e5911ba6de99fbd1f7fcd63c9801388a5724d279..bc2863170d7cc4258c059e495d7cf4f3e089662c 100644
--- a/ios/chrome/common/app_group/app_group_constants.mm
+++ b/ios/chrome/common/app_group/app_group_constants.mm
@@ -6,17 +6,13 @@
#include "base/logging.h"
#include "base/strings/sys_string_conversions.h"
+#include "build/config/ios/ios_app_bundle_id_prefix.h"
#include "components/version_info/version_info.h"
#if !defined(__has_feature) || !__has_feature(objc_arc)
#error "This file requires ARC support."
#endif
-namespace {
-NSString* const kChromeAppGroupIdentifier =
- @"group." IOS_BUNDLE_ID_PREFIX ".chrome";
-}
-
namespace app_group {
const char kChromeAppGroupXCallbackCommand[] = "app-group-command";
@@ -51,7 +47,8 @@ NSString* ApplicationGroup() {
NSBundle* bundle = [NSBundle mainBundle];
NSString* group = [bundle objectForInfoDictionaryKey:@"KSApplicationGroup"];
if (![group length]) {
- return kChromeAppGroupIdentifier;
+ return [NSString stringWithFormat:@"group.%s.chrome",
+ BUILDFLAG(IOS_APP_BUNDLE_ID_PREFIX), nil];
rohitrao (ping after 24h) 2016/12/13 14:14:30 Is there a better way to concatenate these strings
sdefresne 2016/12/13 15:18:02 Ah, BUILDFLAG(IOS_APP_BUNDLE_ID_PREFIX) expands to
}
return group;
}
« build/config/ios/BUILD.gn ('K') | « ios/chrome/common/app_group/BUILD.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698