Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(403)

Unified Diff: trunk/src/chrome/browser/profiles/avatar_menu_model_unittest.cc

Issue 17068004: Revert 207755 "Add device policies to control accessibility sett..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: trunk/src/chrome/browser/profiles/avatar_menu_model_unittest.cc
===================================================================
--- trunk/src/chrome/browser/profiles/avatar_menu_model_unittest.cc (revision 207797)
+++ trunk/src/chrome/browser/profiles/avatar_menu_model_unittest.cc (working copy)
@@ -4,11 +4,9 @@
#include "chrome/browser/profiles/avatar_menu_model.h"
-#include "base/memory/scoped_ptr.h"
#include "base/metrics/field_trial.h"
#include "base/strings/string16.h"
#include "base/strings/utf_string_conversions.h"
-#include "chrome/browser/prefs/pref_service_syncable.h"
#include "chrome/browser/profiles/avatar_menu_model_observer.h"
#include "chrome/browser/profiles/profile_info_cache.h"
#include "chrome/browser/profiles/profile_manager.h"
@@ -62,10 +60,8 @@
string16 name1(ASCIIToUTF16("Test 1"));
string16 name2(ASCIIToUTF16("Test 2"));
- manager()->CreateTestingProfile("p1", scoped_ptr<PrefServiceSyncable>(),
- name1, 0);
- manager()->CreateTestingProfile("p2", scoped_ptr<PrefServiceSyncable>(),
- name2, 0);
+ manager()->CreateTestingProfile("p1", name1, 0);
+ manager()->CreateTestingProfile("p2", name2, 0);
MockObserver observer;
EXPECT_EQ(0, observer.change_count());
@@ -88,10 +84,8 @@
string16 name1(ASCIIToUTF16("Test 1"));
string16 name2(ASCIIToUTF16("Test 2"));
- manager()->CreateTestingProfile("p1", scoped_ptr<PrefServiceSyncable>(),
- name1, 0);
- manager()->CreateTestingProfile("p2", scoped_ptr<PrefServiceSyncable>(),
- name2, 0);
+ manager()->CreateTestingProfile("p1", name1, 0);
+ manager()->CreateTestingProfile("p2", name2, 0);
MockObserver observer;
AvatarMenuModel model(manager()->profile_info_cache(), &observer, browser());
@@ -106,10 +100,8 @@
string16 name2(ASCIIToUTF16("Beta"));
string16 newname1(ASCIIToUTF16("Gamma"));
- manager()->CreateTestingProfile("p1", scoped_ptr<PrefServiceSyncable>(),
- name1, 0);
- manager()->CreateTestingProfile("p2", scoped_ptr<PrefServiceSyncable>(),
- name2, 0);
+ manager()->CreateTestingProfile("p1", name1, 0);
+ manager()->CreateTestingProfile("p2", name2, 0);
MockObserver observer;
AvatarMenuModel model(manager()->profile_info_cache(), &observer, browser());
@@ -142,10 +134,8 @@
string16 name1(ASCIIToUTF16("Test 1"));
string16 name2(ASCIIToUTF16("Test 2"));
- manager()->CreateTestingProfile("p1", scoped_ptr<PrefServiceSyncable>(),
- name1, 0);
- manager()->CreateTestingProfile("p2", scoped_ptr<PrefServiceSyncable>(),
- name2, 0);
+ manager()->CreateTestingProfile("p1", name1, 0);
+ manager()->CreateTestingProfile("p2", name2, 0);
MockObserver observer;
EXPECT_EQ(0, observer.change_count());
@@ -155,8 +145,7 @@
EXPECT_EQ(2U, model.GetNumberOfItems());
string16 name3(ASCIIToUTF16("Test 3"));
- manager()->CreateTestingProfile("p3", scoped_ptr<PrefServiceSyncable>(),
- name3, 0);
+ manager()->CreateTestingProfile("p3", name3, 0);
// Four changes happened via the call to CreateTestingProfile: adding the
// profile to the cache, setting the user name, rebuilding the list of
@@ -197,8 +186,7 @@
TEST_F(AvatarMenuModelTest, DontShowAvatarMenu) {
string16 name1(ASCIIToUTF16("Test 1"));
- manager()->CreateTestingProfile("p1", scoped_ptr<PrefServiceSyncable>(),
- name1, 0);
+ manager()->CreateTestingProfile("p1", name1, 0);
EXPECT_FALSE(AvatarMenuModel::ShouldShowAvatarMenu());
@@ -208,8 +196,7 @@
return;
string16 name2(ASCIIToUTF16("Test 2"));
- manager()->CreateTestingProfile("p2", scoped_ptr<PrefServiceSyncable>(),
- name2, 0);
+ manager()->CreateTestingProfile("p2", name2, 0);
EXPECT_FALSE(AvatarMenuModel::ShouldShowAvatarMenu());
}
@@ -222,10 +209,8 @@
string16 name1(ASCIIToUTF16("Test 1"));
string16 name2(ASCIIToUTF16("Test 2"));
- manager()->CreateTestingProfile("p1", scoped_ptr<PrefServiceSyncable>(),
- name1, 0);
- manager()->CreateTestingProfile("p2", scoped_ptr<PrefServiceSyncable>(),
- name2, 0);
+ manager()->CreateTestingProfile("p1", name1, 0);
+ manager()->CreateTestingProfile("p2", name2, 0);
#if defined(OS_CHROMEOS)
EXPECT_FALSE(AvatarMenuModel::ShouldShowAvatarMenu());
@@ -239,8 +224,7 @@
if (!ProfileManager::IsMultipleProfilesEnabled())
return;
- manager()->CreateTestingProfile("p1", scoped_ptr<PrefServiceSyncable>(),
- ASCIIToUTF16("Test 1"), 0);
+ manager()->CreateTestingProfile("p1", ASCIIToUTF16("Test 1"), 0);
// Add a managed user profile.
ProfileInfoCache* cache = manager()->profile_info_cache();

Powered by Google App Engine
This is Rietveld 408576698