Apply upgrade_enrollment Profile metrics to Mac Desktop.

Apply upgrade_enrollment Profile metrics to Mac Desktop.

BUG=357693
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=272909

[Mike Lerman, 2014-05-20 15:43:35]

Hi Hui,

Can you please review this change to ensure correctness?

Shess,

Owner review please?

Thanks!

[Scott Hess - ex-Googler, 2014-05-20 17:52:19]

LGTM for owner.  But I haven't the slightest idea whether it accomplishes what
is desired :-).

[guohui, 2014-05-22 19:43:17]

https://codereview.chromium.org/296023003/diff/20001/chrome/browser/ui/cocoa/...
File chrome/browser/ui/cocoa/profiles/profile_chooser_controller.mm (right):

https://codereview.chromium.org/296023003/diff/20001/chrome/browser/ui/cocoa/...
chrome/browser/ui/cocoa/profiles/profile_chooser_controller.mm:1024:
ProfileMetrics::PROFILE_ENROLLMENT_SHOW_PREVIEW_PROMO);
i think this code will double count when the bubble view is refreshed, we need
to do something similar as in views code,

https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/...

[Mike Lerman, 2014-05-26 18:20:46]

https://codereview.chromium.org/296023003/diff/20001/chrome/browser/ui/cocoa/...
File chrome/browser/ui/cocoa/profiles/profile_chooser_controller.mm (right):

https://codereview.chromium.org/296023003/diff/20001/chrome/browser/ui/cocoa/...
chrome/browser/ui/cocoa/profiles/profile_chooser_controller.mm:1024:
ProfileMetrics::PROFILE_ENROLLMENT_SHOW_PREVIEW_PROMO);
On 2014/05/22 19:43:17, guohui wrote:
> i think this code will double count when the bubble view is refreshed, we need
> to do something similar as in views code,
> 
>
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/...

Done.

[guohui, 2014-05-26 18:49:04]

lgtm

[Mike Lerman, 2014-05-26 22:46:44]

The CQ bit was checked by mlerman@chromium.org

[commit-bot: I haz the power, 2014-05-26 22:47:12]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/mlerman@chromium.org/296023003/80001

[commit-bot: I haz the power, 2014-05-27 00:41:33]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/7...)
  linux_chromium_chromeos_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)

[commit-bot: I haz the power, 2014-05-27 01:54:30]

Change committed as 272909