Commit d111cde8 authored by Sylvain Defresne's avatar Sylvain Defresne Committed by Commit Bot

[ios] Fix uses of ios::ChromeBrowserState in /ios/chrome/test

The ChromeBrowserState has been moved from the "ios" namespace
to the global namespace by http://crrev.com/c/2014913. Fix the
uses from ios::ChromeBrowserState to ChromeBrowserState.

Also remove includes of chrome_browser_state_forward.h and use
a simple forward declaration instead (since the old name is no
longer necessary).

This CL was uploaded by git cl split.

R=eugenebut@chromium.org

Bug: 1042208
Change-Id: I2ac990201f0de9332a204023c8ad2ebbf18a9407
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2017347
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: default avatarEugene But <eugenebut@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#734600}
parent 388ddae2
...@@ -22,7 +22,7 @@ bool BookmarksLoaded() { ...@@ -22,7 +22,7 @@ bool BookmarksLoaded() {
} }
bool ClearBookmarks() { bool ClearBookmarks() {
ios::ChromeBrowserState* browserState = GetOriginalBrowserState(); ChromeBrowserState* browserState = GetOriginalBrowserState();
bookmarks::BookmarkModel* bookmarkModel = bookmarks::BookmarkModel* bookmarkModel =
ios::BookmarkModelFactory::GetForBrowserState(browserState); ios::BookmarkModelFactory::GetForBrowserState(browserState);
BOOL removeSucceeded = RemoveAllUserBookmarksIOS(browserState); BOOL removeSucceeded = RemoveAllUserBookmarksIOS(browserState);
......
...@@ -28,7 +28,7 @@ using base::test::ios::WaitUntilConditionOrTimeout; ...@@ -28,7 +28,7 @@ using base::test::ios::WaitUntilConditionOrTimeout;
namespace { namespace {
bool ClearBrowsingData(bool off_the_record, BrowsingDataRemoveMask mask) { bool ClearBrowsingData(bool off_the_record, BrowsingDataRemoveMask mask) {
ios::ChromeBrowserState* browser_state = ChromeBrowserState* browser_state =
off_the_record ? chrome_test_util::GetCurrentIncognitoBrowserState() off_the_record ? chrome_test_util::GetCurrentIncognitoBrowserState()
: chrome_test_util::GetOriginalBrowserState(); : chrome_test_util::GetOriginalBrowserState();
...@@ -81,9 +81,9 @@ bool ClearAllWebStateBrowsingData() { ...@@ -81,9 +81,9 @@ bool ClearAllWebStateBrowsingData() {
} }
bool ClearCertificatePolicyCache(bool off_the_record) { bool ClearCertificatePolicyCache(bool off_the_record) {
ios::ChromeBrowserState* browser_state = ChromeBrowserState* browser_state = off_the_record
off_the_record ? GetCurrentIncognitoBrowserState() ? GetCurrentIncognitoBrowserState()
: GetOriginalBrowserState(); : GetOriginalBrowserState();
auto cache = web::BrowserState::GetCertificatePolicyCache(browser_state); auto cache = web::BrowserState::GetCertificatePolicyCache(browser_state);
__block BOOL policies_cleared = NO; __block BOOL policies_cleared = NO;
base::PostTask(FROM_HERE, {web::WebThread::IO}, base::BindOnce(^{ base::PostTask(FROM_HERE, {web::WebThread::IO}, base::BindOnce(^{
......
...@@ -8,10 +8,10 @@ ...@@ -8,10 +8,10 @@
#include "base/compiler_specific.h" #include "base/compiler_specific.h"
#import "base/ios/block_types.h" #import "base/ios/block_types.h"
#include "ios/chrome/browser/browser_state/chrome_browser_state_forward.h"
#import "ios/chrome/browser/ui/commands/browser_commands.h" #import "ios/chrome/browser/ui/commands/browser_commands.h"
@protocol ApplicationCommands; @protocol ApplicationCommands;
class ChromeBrowserState;
@class DeviceSharingManager; @class DeviceSharingManager;
@class MainController; @class MainController;
@class NewTabPageController; @class NewTabPageController;
...@@ -26,10 +26,10 @@ MainController* GetMainController(); ...@@ -26,10 +26,10 @@ MainController* GetMainController();
DeviceSharingManager* GetDeviceSharingManager(); DeviceSharingManager* GetDeviceSharingManager();
// Returns the current, non-incognito ChromeBrowserState. // Returns the current, non-incognito ChromeBrowserState.
ios::ChromeBrowserState* GetOriginalBrowserState(); ChromeBrowserState* GetOriginalBrowserState();
// Returns the current incognito ChromeBrowserState // Returns the current incognito ChromeBrowserState
ios::ChromeBrowserState* GetCurrentIncognitoBrowserState(); ChromeBrowserState* GetCurrentIncognitoBrowserState();
// Returns the dispatcher for the main BVC. // Returns the dispatcher for the main BVC.
// TODO(crbug.com/738881): Use DispatcherForActiveBrowserViewController() // TODO(crbug.com/738881): Use DispatcherForActiveBrowserViewController()
...@@ -57,7 +57,7 @@ void ClearPresentedState(); ...@@ -57,7 +57,7 @@ void ClearPresentedState();
void SetBooleanLocalStatePref(const char* pref_name, bool value); void SetBooleanLocalStatePref(const char* pref_name, bool value);
// Sets the value of a boolean user pref in the given browser state. // Sets the value of a boolean user pref in the given browser state.
void SetBooleanUserPref(ios::ChromeBrowserState* browser_state, void SetBooleanUserPref(ChromeBrowserState* browser_state,
const char* pref_name, const char* pref_name,
bool value); bool value);
......
...@@ -56,14 +56,14 @@ ...@@ -56,14 +56,14 @@
namespace { namespace {
// Returns the original ChromeBrowserState if |incognito| is false. If // Returns the original ChromeBrowserState if |incognito| is false. If
// |ingonito| is true, returns an off-the-record ChromeBrowserState. // |ingonito| is true, returns an off-the-record ChromeBrowserState.
ios::ChromeBrowserState* GetBrowserState(bool incognito) { ChromeBrowserState* GetBrowserState(bool incognito) {
std::vector<ios::ChromeBrowserState*> browser_states = std::vector<ChromeBrowserState*> browser_states =
GetApplicationContext() GetApplicationContext()
->GetChromeBrowserStateManager() ->GetChromeBrowserStateManager()
->GetLoadedBrowserStates(); ->GetLoadedBrowserStates();
DCHECK(!browser_states.empty()); DCHECK(!browser_states.empty());
ios::ChromeBrowserState* browser_state = browser_states.front(); ChromeBrowserState* browser_state = browser_states.front();
DCHECK(!browser_state->IsOffTheRecord()); DCHECK(!browser_state->IsOffTheRecord());
return incognito ? browser_state->GetOffTheRecordChromeBrowserState() return incognito ? browser_state->GetOffTheRecordChromeBrowserState()
...@@ -82,11 +82,11 @@ DeviceSharingManager* GetDeviceSharingManager() { ...@@ -82,11 +82,11 @@ DeviceSharingManager* GetDeviceSharingManager() {
return [GetMainController() deviceSharingManager]; return [GetMainController() deviceSharingManager];
} }
ios::ChromeBrowserState* GetOriginalBrowserState() { ChromeBrowserState* GetOriginalBrowserState() {
return GetBrowserState(false); return GetBrowserState(false);
} }
ios::ChromeBrowserState* GetCurrentIncognitoBrowserState() { ChromeBrowserState* GetCurrentIncognitoBrowserState() {
return GetBrowserState(true); return GetBrowserState(true);
} }
...@@ -156,7 +156,7 @@ void SetBooleanLocalStatePref(const char* pref_name, bool value) { ...@@ -156,7 +156,7 @@ void SetBooleanLocalStatePref(const char* pref_name, bool value) {
pref.SetValue(value); pref.SetValue(value);
} }
void SetBooleanUserPref(ios::ChromeBrowserState* browser_state, void SetBooleanUserPref(ChromeBrowserState* browser_state,
const char* pref_name, const char* pref_name,
bool value) { bool value) {
DCHECK(browser_state); DCHECK(browser_state);
......
...@@ -15,7 +15,7 @@ ...@@ -15,7 +15,7 @@
namespace chrome_test_util { namespace chrome_test_util {
void SetContentSettingsBlockPopups(ContentSetting setting) { void SetContentSettingsBlockPopups(ContentSetting setting) {
ios::ChromeBrowserState* browser_state = GetOriginalBrowserState(); ChromeBrowserState* browser_state = GetOriginalBrowserState();
HostContentSettingsMap* settings_map = HostContentSettingsMap* settings_map =
ios::HostContentSettingsMapFactory::GetForBrowserState(browser_state); ios::HostContentSettingsMapFactory::GetForBrowserState(browser_state);
settings_map->SetDefaultContentSetting(ContentSettingsType::POPUPS, setting); settings_map->SetDefaultContentSetting(ContentSettingsType::POPUPS, setting);
......
...@@ -97,7 +97,7 @@ bool SignOutAndClearAccounts() { ...@@ -97,7 +97,7 @@ bool SignOutAndClearAccounts() {
// important to autorelease all objects that make network requests to avoid // important to autorelease all objects that make network requests to avoid
// EarlGrey being confused about on-going network traffic.. // EarlGrey being confused about on-going network traffic..
@autoreleasepool { @autoreleasepool {
ios::ChromeBrowserState* browser_state = GetOriginalBrowserState(); ChromeBrowserState* browser_state = GetOriginalBrowserState();
DCHECK(browser_state); DCHECK(browser_state);
// Sign out current user. // Sign out current user.
...@@ -127,7 +127,7 @@ void ResetMockAuthentication() { ...@@ -127,7 +127,7 @@ void ResetMockAuthentication() {
} }
void ResetSigninPromoPreferences() { void ResetSigninPromoPreferences() {
ios::ChromeBrowserState* browser_state = GetOriginalBrowserState(); ChromeBrowserState* browser_state = GetOriginalBrowserState();
PrefService* prefs = browser_state->GetPrefs(); PrefService* prefs = browser_state->GetPrefs();
prefs->SetInteger(prefs::kIosBookmarkSigninPromoDisplayedCount, 0); prefs->SetInteger(prefs::kIosBookmarkSigninPromoDisplayedCount, 0);
prefs->SetBoolean(prefs::kIosBookmarkPromoAlreadySeen, false); prefs->SetBoolean(prefs::kIosBookmarkPromoAlreadySeen, false);
......
...@@ -50,7 +50,7 @@ NSString* const kSyncTestErrorDomain = @"SyncTestDomain"; ...@@ -50,7 +50,7 @@ NSString* const kSyncTestErrorDomain = @"SyncTestDomain";
// |create_http_post_provider_factory_cb|. // |create_http_post_provider_factory_cb|.
void OverrideSyncNetwork(const syncer::CreateHttpPostProviderFactory& void OverrideSyncNetwork(const syncer::CreateHttpPostProviderFactory&
create_http_post_provider_factory_cb) { create_http_post_provider_factory_cb) {
ios::ChromeBrowserState* browser_state = ChromeBrowserState* browser_state =
chrome_test_util::GetOriginalBrowserState(); chrome_test_util::GetOriginalBrowserState();
DCHECK(browser_state); DCHECK(browser_state);
syncer::ProfileSyncService* service = syncer::ProfileSyncService* service =
...@@ -79,7 +79,7 @@ void TearDownFakeSyncServer() { ...@@ -79,7 +79,7 @@ void TearDownFakeSyncServer() {
void StartSync() { void StartSync() {
DCHECK(!IsSyncInitialized()); DCHECK(!IsSyncInitialized());
ios::ChromeBrowserState* browser_state = ChromeBrowserState* browser_state =
chrome_test_util::GetOriginalBrowserState(); chrome_test_util::GetOriginalBrowserState();
SyncSetupService* sync_setup_service = SyncSetupService* sync_setup_service =
SyncSetupServiceFactory::GetForBrowserState(browser_state); SyncSetupServiceFactory::GetForBrowserState(browser_state);
...@@ -88,7 +88,7 @@ void StartSync() { ...@@ -88,7 +88,7 @@ void StartSync() {
void StopSync() { void StopSync() {
DCHECK(IsSyncInitialized()); DCHECK(IsSyncInitialized());
ios::ChromeBrowserState* browser_state = ChromeBrowserState* browser_state =
chrome_test_util::GetOriginalBrowserState(); chrome_test_util::GetOriginalBrowserState();
SyncSetupService* sync_setup_service = SyncSetupService* sync_setup_service =
SyncSetupServiceFactory::GetForBrowserState(browser_state); SyncSetupServiceFactory::GetForBrowserState(browser_state);
...@@ -96,7 +96,7 @@ void StopSync() { ...@@ -96,7 +96,7 @@ void StopSync() {
} }
void TriggerSyncCycle(syncer::ModelType type) { void TriggerSyncCycle(syncer::ModelType type) {
ios::ChromeBrowserState* browser_state = ChromeBrowserState* browser_state =
chrome_test_util::GetOriginalBrowserState(); chrome_test_util::GetOriginalBrowserState();
syncer::SyncService* sync_service = syncer::SyncService* sync_service =
ProfileSyncServiceFactory::GetForBrowserState(browser_state); ProfileSyncServiceFactory::GetForBrowserState(browser_state);
...@@ -150,7 +150,7 @@ void InjectBookmarkOnFakeSyncServer(std::string url, std::string title) { ...@@ -150,7 +150,7 @@ void InjectBookmarkOnFakeSyncServer(std::string url, std::string title) {
} }
bool IsSyncInitialized() { bool IsSyncInitialized() {
ios::ChromeBrowserState* browser_state = ChromeBrowserState* browser_state =
chrome_test_util::GetOriginalBrowserState(); chrome_test_util::GetOriginalBrowserState();
DCHECK(browser_state); DCHECK(browser_state);
syncer::SyncService* syncService = syncer::SyncService* syncService =
...@@ -160,7 +160,7 @@ bool IsSyncInitialized() { ...@@ -160,7 +160,7 @@ bool IsSyncInitialized() {
std::string GetSyncCacheGuid() { std::string GetSyncCacheGuid() {
DCHECK(IsSyncInitialized()); DCHECK(IsSyncInitialized());
ios::ChromeBrowserState* browser_state = ChromeBrowserState* browser_state =
chrome_test_util::GetOriginalBrowserState(); chrome_test_util::GetOriginalBrowserState();
syncer::DeviceInfoSyncService* service = syncer::DeviceInfoSyncService* service =
DeviceInfoSyncServiceFactory::GetForBrowserState(browser_state); DeviceInfoSyncServiceFactory::GetForBrowserState(browser_state);
...@@ -208,7 +208,7 @@ void DeleteAutofillProfileOnFakeSyncServer(std::string guid) { ...@@ -208,7 +208,7 @@ void DeleteAutofillProfileOnFakeSyncServer(std::string guid) {
} }
bool IsAutofillProfilePresent(std::string guid, std::string full_name) { bool IsAutofillProfilePresent(std::string guid, std::string full_name) {
ios::ChromeBrowserState* browser_state = ChromeBrowserState* browser_state =
chrome_test_util::GetOriginalBrowserState(); chrome_test_util::GetOriginalBrowserState();
autofill::PersonalDataManager* personal_data_manager = autofill::PersonalDataManager* personal_data_manager =
autofill::PersonalDataManagerFactory::GetForBrowserState(browser_state); autofill::PersonalDataManagerFactory::GetForBrowserState(browser_state);
...@@ -224,7 +224,7 @@ bool IsAutofillProfilePresent(std::string guid, std::string full_name) { ...@@ -224,7 +224,7 @@ bool IsAutofillProfilePresent(std::string guid, std::string full_name) {
} }
void ClearAutofillProfile(std::string guid) { void ClearAutofillProfile(std::string guid) {
ios::ChromeBrowserState* browser_state = ChromeBrowserState* browser_state =
chrome_test_util::GetOriginalBrowserState(); chrome_test_util::GetOriginalBrowserState();
autofill::PersonalDataManager* personal_data_manager = autofill::PersonalDataManager* personal_data_manager =
autofill::PersonalDataManagerFactory::GetForBrowserState(browser_state); autofill::PersonalDataManagerFactory::GetForBrowserState(browser_state);
...@@ -251,7 +251,7 @@ BOOL VerifySessionsOnSyncServer(const std::multiset<std::string>& expected_urls, ...@@ -251,7 +251,7 @@ BOOL VerifySessionsOnSyncServer(const std::multiset<std::string>& expected_urls,
} }
void AddTypedURLOnClient(const GURL& url) { void AddTypedURLOnClient(const GURL& url) {
ios::ChromeBrowserState* browser_state = ChromeBrowserState* browser_state =
chrome_test_util::GetOriginalBrowserState(); chrome_test_util::GetOriginalBrowserState();
history::HistoryService* historyService = history::HistoryService* historyService =
ios::HistoryServiceFactory::GetForBrowserState( ios::HistoryServiceFactory::GetForBrowserState(
...@@ -282,7 +282,7 @@ BOOL IsTypedUrlPresentOnClient(const GURL& url, ...@@ -282,7 +282,7 @@ BOOL IsTypedUrlPresentOnClient(const GURL& url,
BOOL expect_present, BOOL expect_present,
NSError** error) { NSError** error) {
// Call the history service. // Call the history service.
ios::ChromeBrowserState* browser_state = ChromeBrowserState* browser_state =
chrome_test_util::GetOriginalBrowserState(); chrome_test_util::GetOriginalBrowserState();
history::HistoryService* history_service = history::HistoryService* history_service =
ios::HistoryServiceFactory::GetForBrowserState( ios::HistoryServiceFactory::GetForBrowserState(
...@@ -330,7 +330,7 @@ BOOL IsTypedUrlPresentOnClient(const GURL& url, ...@@ -330,7 +330,7 @@ BOOL IsTypedUrlPresentOnClient(const GURL& url,
} }
void DeleteTypedUrlFromClient(const GURL& url) { void DeleteTypedUrlFromClient(const GURL& url) {
ios::ChromeBrowserState* browser_state = ChromeBrowserState* browser_state =
chrome_test_util::GetOriginalBrowserState(); chrome_test_util::GetOriginalBrowserState();
history::HistoryService* history_service = history::HistoryService* history_service =
ios::HistoryServiceFactory::GetForBrowserState( ios::HistoryServiceFactory::GetForBrowserState(
......
...@@ -215,7 +215,7 @@ void SaveSessionImmediately() { ...@@ -215,7 +215,7 @@ void SaveSessionImmediately() {
void EvictOtherTabModelTabs() { void EvictOtherTabModelTabs() {
id<BrowserInterfaceProvider> provider = GetMainController().interfaceProvider; id<BrowserInterfaceProvider> provider = GetMainController().interfaceProvider;
ios::ChromeBrowserState* otherBrowserState = ChromeBrowserState* otherBrowserState =
IsIncognitoMode() ? provider.mainInterface.browserState IsIncognitoMode() ? provider.mainInterface.browserState
: provider.incognitoInterface.browserState; : provider.incognitoInterface.browserState;
// Disabling and enabling web usage will evict all web views. // Disabling and enabling web usage will evict all web views.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment