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..0c721009ceec51a1566209f54b979775a89e623b 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.
+  uint32_t numProfiles_;
+  uint32_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

[Mark Mentovai, 2015/02/20 21:54:18]

I don’t think you need “num” in any of these, same in the .mm file. It’s obvious
that a count is going to be a number.

[Mike Lerman, 2015/02/23 16:12:11]

Done - though the class' member variables (numProfiles_ and
numSignedInProfiles_) should probably keep the prefix since they're not
explicitly parameters to a method called "count" - their meaning needs to stand
alone a bit better.

+                       numSignedInProfiles:(uint32_t)numSignedInProfiles;
+
 @end  // @interface KeystoneGlue
 
 @interface KeystoneGlue(ExposedForTesting)