mac: Prevent Address Book permissions dialog from erroneously appearing.

chrome/browser/mac/keystone_glue.h

Index: chrome/browser/mac/keystone_glue.h
diff --git a/chrome/browser/mac/keystone_glue.h b/chrome/browser/mac/keystone_glue.h
index 5c21c369e80294ede452810b37885491d48c7f79..6be62ba2aef25739ccba1251b7e50437fb897efa 100644
--- a/chrome/browser/mac/keystone_glue.h
+++ b/chrome/browser/mac/keystone_glue.h
@@ -8,11 +8,13 @@
 #include "base/strings/string16.h"
 
 #if defined(__OBJC__)
-
 #import <Foundation/Foundation.h>
 
 #include "base/mac/scoped_authorizationref.h"
 #import "base/mac/scoped_nsobject.h"
+#endif  // __OBJC__
+
+namespace keystone_glue {
 
 // Possible outcomes of various operations.  A version may accompany some of
 // these, but beware: a version is never required.  For statuses that can be
@@ -37,13 +39,18 @@ enum AutoupdateStatus {
   kAutoupdateNeedsPromotion,  // no version
 };
 
+}  // namespace keystone_glue

[Ilya Sherman, 2014/06/17 19:55:01]

I would prefer not to add this namespace as part of this CL.  If you're really
keen on adding this, please split that change off to a separate CL, and ask an
owner of //chrome/browser/mac to review it.  In general, though, it's uncommon
to use namespaces around enums in //chrome.  Instead, enums are typically named
in a wacky verbose style.  For this enum, the Chrome convention would be
something like this:

enum KeystoneAutoupdateStatus {
  KEYSTONE_AUTOUPDATE_STATUS_NONE = 0,
  KEYSTONE_AUTOUPDATE_STATUS_REGISTERED,
  // etc.
}

Specifically, the style is to pick a verbose name for the enum, so that the enum
name is clear and does not conflict with anything, and then to have each
enumerated constant's name contain the entire enum name as a prefix.  I assume
that this is preferred over namespaces because most other code isn't namespaced,
and its a little awkward to use namespaces just for enums -- especially given
that Objective-C doesn't support namespaces.

[erikchen, 2014/06/17 22:36:06]

I have reverted the namespace change, and well as the DCHECK in
keystone_glue.mm, and the additional comment. I was trying to do the "right"
thing, but I'm happy to write less code and have fewer reviewers.

+
+#if defined(__OBJC__)
+
 // kAutoupdateStatusNotification is the name of the notification posted when
-// -checkForUpdate and -installUpdate complete.  This notification will be
-// sent with with its sender object set to the KeystoneGlue instance sending
-// the notification.  Its userInfo dictionary will contain an AutoupdateStatus
+// -checkForUpdate and -installUpdate complete.  This notification will be sent
+// with with its sender object set to the KeystoneGlue instance sending the
+// notification.  Its userInfo dictionary will contain an AutoupdateStatus
 // value as an intValue at key kAutoupdateStatusStatus.  If a version is
 // available (see AutoupdateStatus), it will be present at key
-// kAutoupdateStatusVersion.
+// kAutoupdateStatusVersion. This notification is always posted on the main
+// thread.
 extern NSString* const kAutoupdateStatusNotification;
 extern NSString* const kAutoupdateStatusStatus;
 extern NSString* const kAutoupdateStatusVersion;
@@ -122,7 +129,7 @@ enum BrandFileType {
 
 // Accessor for the kAutoupdateStatusStatus field of recentNotification_'s
 // userInfo dictionary.
-- (AutoupdateStatus)recentStatus;
+- (keystone_glue::AutoupdateStatus)recentStatus;
 
 // Returns YES if an asynchronous operation is pending: if an update check or
 // installation attempt is currently in progress.