OLD | NEW |
1 // Copyright 2014 The Chromium Authors. All rights reserved. | 1 // Copyright 2014 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include "chrome/browser/ui/views/profiles/profile_chooser_view.h" | 5 #include "chrome/browser/ui/views/profiles/profile_chooser_view.h" |
6 | 6 |
7 #include "base/command_line.h" | 7 #include "base/command_line.h" |
8 #include "base/path_service.h" | 8 #include "base/path_service.h" |
9 #include "base/prefs/pref_service.h" | 9 #include "base/prefs/pref_service.h" |
10 #include "base/strings/utf_string_conversions.h" | 10 #include "base/strings/utf_string_conversions.h" |
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
76 #if defined(OS_CHROMEOS) || defined(OS_ANDROID) || defined(OS_IOS) | 76 #if defined(OS_CHROMEOS) || defined(OS_ANDROID) || defined(OS_IOS) |
77 // This test doesn't make sense for ChromeOS since it has a different | 77 // This test doesn't make sense for ChromeOS since it has a different |
78 // multi-profiles menu in the system tray instead. | 78 // multi-profiles menu in the system tray instead. |
79 // | 79 // |
80 // Mobile platforms are also excluded due to a lack of avatar menu. | 80 // Mobile platforms are also excluded due to a lack of avatar menu. |
81 #define MAYBE_ViewProfileUMA DISABLED_ViewProfileUMA | 81 #define MAYBE_ViewProfileUMA DISABLED_ViewProfileUMA |
82 #else | 82 #else |
83 #define MAYBE_ViewProfileUMA ViewProfileUMA | 83 #define MAYBE_ViewProfileUMA ViewProfileUMA |
84 #endif | 84 #endif |
85 | 85 |
86 // TODO(mlerman): Re-enable the test to MAYBE_ViewProfileUMA once there is a | 86 IN_PROC_BROWSER_TEST_F(ProfileChooserViewBrowserTest, MAYBE_ViewProfileUMA) { |
87 // launch plan for EnableAccountConsistency. | 87 UMAHistogramHelper histograms; |
88 IN_PROC_BROWSER_TEST_F(ProfileChooserViewBrowserTest, DISABLED_ViewProfileUMA) { | |
89 // If multiprofile mode is not enabled, you can't switch between profiles. | 88 // If multiprofile mode is not enabled, you can't switch between profiles. |
90 if (!profiles::IsMultipleProfilesEnabled()) | 89 if (!profiles::IsMultipleProfilesEnabled()) |
91 return; | 90 return; |
92 | 91 |
93 Profile* profile = browser()->profile(); | 92 Profile* profile = browser()->profile(); |
94 profile->GetPrefs()->SetInteger(prefs::kProfileAvatarTutorialShown, 0); | 93 profile->GetPrefs()->SetInteger(prefs::kProfileAvatarTutorialShown, 0); |
95 | 94 |
96 ASSERT_NO_FATAL_FAILURE(OpenProfileChooserView()); | 95 ASSERT_NO_FATAL_FAILURE(OpenProfileChooserView()); |
| 96 |
| 97 histograms.Fetch(); |
| 98 histograms.ExpectUniqueSample("Profile.NewAvatarMenu.Upgrade", |
| 99 ProfileMetrics::PROFILE_AVATAR_MENU_UPGRADE_VIEW, 1); |
97 } | 100 } |
OLD | NEW |