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

[ios] Fix uses of ios::ChromeBrowserState in /ios/chrome/browser/crash_report/breadcrumbs

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: I801d46c86d1ee02b69135224271f442595601a85
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2017349Reviewed-by: default avatarEugene But <eugenebut@chromium.org>
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#735297}
parent e4151601
...@@ -19,7 +19,7 @@ BreadcrumbManagerKeyedServiceFactory::GetInstance() { ...@@ -19,7 +19,7 @@ BreadcrumbManagerKeyedServiceFactory::GetInstance() {
// static // static
BreadcrumbManagerKeyedService* BreadcrumbManagerKeyedService*
BreadcrumbManagerKeyedServiceFactory::GetForBrowserState( BreadcrumbManagerKeyedServiceFactory::GetForBrowserState(
ios::ChromeBrowserState* browser_state) { ChromeBrowserState* browser_state) {
return static_cast<BreadcrumbManagerKeyedService*>( return static_cast<BreadcrumbManagerKeyedService*>(
GetInstance()->GetServiceForBrowserState(browser_state, /*create=*/true)); GetInstance()->GetServiceForBrowserState(browser_state, /*create=*/true));
} }
......
...@@ -7,16 +7,16 @@ ...@@ -7,16 +7,16 @@
#include "base/no_destructor.h" #include "base/no_destructor.h"
#include "components/keyed_service/ios/browser_state_keyed_service_factory.h" #include "components/keyed_service/ios/browser_state_keyed_service_factory.h"
#include "ios/chrome/browser/browser_state/chrome_browser_state_forward.h"
class BreadcrumbManagerKeyedService; class BreadcrumbManagerKeyedService;
class ChromeBrowserState;
class BreadcrumbManagerKeyedServiceFactory class BreadcrumbManagerKeyedServiceFactory
: public BrowserStateKeyedServiceFactory { : public BrowserStateKeyedServiceFactory {
public: public:
static BreadcrumbManagerKeyedServiceFactory* GetInstance(); static BreadcrumbManagerKeyedServiceFactory* GetInstance();
static BreadcrumbManagerKeyedService* GetForBrowserState( static BreadcrumbManagerKeyedService* GetForBrowserState(
ios::ChromeBrowserState* browser_state); ChromeBrowserState* browser_state);
private: private:
friend class base::NoDestructor<BreadcrumbManagerKeyedServiceFactory>; friend class base::NoDestructor<BreadcrumbManagerKeyedServiceFactory>;
......
...@@ -24,7 +24,7 @@ namespace { ...@@ -24,7 +24,7 @@ namespace {
std::unique_ptr<KeyedService> BuildBreadcrumbManagerKeyedService( std::unique_ptr<KeyedService> BuildBreadcrumbManagerKeyedService(
web::BrowserState* browser_state) { web::BrowserState* browser_state) {
return std::make_unique<BreadcrumbManagerKeyedService>( return std::make_unique<BreadcrumbManagerKeyedService>(
ios::ChromeBrowserState::FromBrowserState(browser_state)); ChromeBrowserState::FromBrowserState(browser_state));
} }
} }
...@@ -59,7 +59,7 @@ TEST_F(BreadcrumbManagerKeyedServiceTest, EventsLabeledWithBrowserState) { ...@@ -59,7 +59,7 @@ TEST_F(BreadcrumbManagerKeyedServiceTest, EventsLabeledWithBrowserState) {
breadcrumb_manager_service_->AddEvent("event"); breadcrumb_manager_service_->AddEvent("event");
std::string event = breadcrumb_manager_service_->GetEvents(0).front(); std::string event = breadcrumb_manager_service_->GetEvents(0).front();
ios::ChromeBrowserState* off_the_record_browser_state = ChromeBrowserState* off_the_record_browser_state =
chrome_browser_state_->GetOffTheRecordChromeBrowserState(); chrome_browser_state_->GetOffTheRecordChromeBrowserState();
BreadcrumbManagerKeyedService* otr_breadcrumb_manager_service = BreadcrumbManagerKeyedService* otr_breadcrumb_manager_service =
......
...@@ -25,8 +25,8 @@ BreadcrumbManagerTabHelper::BreadcrumbManagerTabHelper(web::WebState* web_state) ...@@ -25,8 +25,8 @@ BreadcrumbManagerTabHelper::BreadcrumbManagerTabHelper(web::WebState* web_state)
BreadcrumbManagerTabHelper::~BreadcrumbManagerTabHelper() = default; BreadcrumbManagerTabHelper::~BreadcrumbManagerTabHelper() = default;
void BreadcrumbManagerTabHelper::LogEvent(const std::string& event) { void BreadcrumbManagerTabHelper::LogEvent(const std::string& event) {
ios::ChromeBrowserState* chrome_browser_state = ChromeBrowserState* chrome_browser_state =
ios::ChromeBrowserState::FromBrowserState(web_state_->GetBrowserState()); ChromeBrowserState::FromBrowserState(web_state_->GetBrowserState());
std::string event_log = std::string event_log =
base::StringPrintf("Tab%d %s", unique_id_, event.c_str()); base::StringPrintf("Tab%d %s", unique_id_, event.c_str());
BreadcrumbManagerKeyedServiceFactory::GetForBrowserState(chrome_browser_state) BreadcrumbManagerKeyedServiceFactory::GetForBrowserState(chrome_browser_state)
......
...@@ -20,7 +20,7 @@ BreadcrumbPersistentStorageKeyedServiceFactory::GetInstance() { ...@@ -20,7 +20,7 @@ BreadcrumbPersistentStorageKeyedServiceFactory::GetInstance() {
// static // static
BreadcrumbPersistentStorageKeyedService* BreadcrumbPersistentStorageKeyedService*
BreadcrumbPersistentStorageKeyedServiceFactory::GetForBrowserState( BreadcrumbPersistentStorageKeyedServiceFactory::GetForBrowserState(
ios::ChromeBrowserState* browser_state) { ChromeBrowserState* browser_state) {
return static_cast<BreadcrumbPersistentStorageKeyedService*>( return static_cast<BreadcrumbPersistentStorageKeyedService*>(
GetInstance()->GetServiceForBrowserState(browser_state, true)); GetInstance()->GetServiceForBrowserState(browser_state, true));
} }
......
...@@ -7,16 +7,16 @@ ...@@ -7,16 +7,16 @@
#include "base/no_destructor.h" #include "base/no_destructor.h"
#include "components/keyed_service/ios/browser_state_keyed_service_factory.h" #include "components/keyed_service/ios/browser_state_keyed_service_factory.h"
#include "ios/chrome/browser/browser_state/chrome_browser_state_forward.h"
class BreadcrumbPersistentStorageKeyedService; class BreadcrumbPersistentStorageKeyedService;
class ChromeBrowserState;
class BreadcrumbPersistentStorageKeyedServiceFactory class BreadcrumbPersistentStorageKeyedServiceFactory
: public BrowserStateKeyedServiceFactory { : public BrowserStateKeyedServiceFactory {
public: public:
static BreadcrumbPersistentStorageKeyedServiceFactory* GetInstance(); static BreadcrumbPersistentStorageKeyedServiceFactory* GetInstance();
static BreadcrumbPersistentStorageKeyedService* GetForBrowserState( static BreadcrumbPersistentStorageKeyedService* GetForBrowserState(
ios::ChromeBrowserState* browser_state); ChromeBrowserState* browser_state);
private: private:
friend class base::NoDestructor< friend class base::NoDestructor<
......
...@@ -27,7 +27,7 @@ namespace { ...@@ -27,7 +27,7 @@ namespace {
std::unique_ptr<KeyedService> BuildBreadcrumbPersistentStorageKeyedService( std::unique_ptr<KeyedService> BuildBreadcrumbPersistentStorageKeyedService(
web::BrowserState* browser_state) { web::BrowserState* browser_state) {
return std::make_unique<BreadcrumbPersistentStorageKeyedService>( return std::make_unique<BreadcrumbPersistentStorageKeyedService>(
ios::ChromeBrowserState::FromBrowserState(browser_state)); ChromeBrowserState::FromBrowserState(browser_state));
} }
} }
......
...@@ -36,7 +36,7 @@ TEST_F(BreadcrumbPersistentStorageUtilTest, UniqueStorage) { ...@@ -36,7 +36,7 @@ TEST_F(BreadcrumbPersistentStorageUtilTest, UniqueStorage) {
TEST_F(BreadcrumbPersistentStorageUtilTest, UniqueIncognitoStorage) { TEST_F(BreadcrumbPersistentStorageUtilTest, UniqueIncognitoStorage) {
base::FilePath path1 = base::FilePath path1 =
GetBreadcrumbPersistentStorageFilePath(GetBrowserState()); GetBreadcrumbPersistentStorageFilePath(GetBrowserState());
ios::ChromeBrowserState* off_the_record_browser_state = ChromeBrowserState* off_the_record_browser_state =
chrome_browser_state_->GetOffTheRecordChromeBrowserState(); chrome_browser_state_->GetOffTheRecordChromeBrowserState();
base::FilePath path2 = base::FilePath path2 =
GetBreadcrumbPersistentStorageFilePath(off_the_record_browser_state); GetBreadcrumbPersistentStorageFilePath(off_the_record_browser_state);
......
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