Profile_Metrics integration with Keystone

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..10272487179eb46f5a761d3d970d44f5b0e27942 100644
--- a/chrome/browser/mac/keystone_glue.h
+++ b/chrome/browser/mac/keystone_glue.h
@@ -101,6 +101,10 @@ enum BrandFileType {
 
   // YES if an update was ever successfully installed by -installUpdate.
   BOOL updateSuccessfullyInstalled_;
+
+  // Profile count information.
+  size_t numProfiles_;

[Mark Mentovai, 2014/10/09 21:26:20]

Just use uint32_t, since that’s what we have on both sides of the interface now.

size_t is wider than uint32_t in the 64-bit environment, what’s the point of
expanding a uint32_t to fill a size_t only to truncate it back down to uint32_t
when you give it to Keystone?

But see the “other option” comment in profile_metrics.cc.

[Mike Lerman, 2014/10/10 15:32:29]

Back to uint32_t for everything in keystone.

+  size_t numSignedInProfiles_;
 }
 
 // Return the default Keystone Glue object.
@@ -172,6 +176,10 @@ enum BrandFileType {
 // at the installed copy.
 - (void)setAppPath:(NSString*)appPath;
 
+// Sets the total number of profiles and the number of signed in profiles.
+- (void)updateProfileCountsWithNumProfiles:(uint32_t)numProfiles
+                       numSignedInProfiles:(uint32_t)numSignedInProfiles;
+
 @end  // @interface KeystoneGlue
 
 @interface KeystoneGlue(ExposedForTesting)