Commit 0ad55749 authored by Sylvain Defresne's avatar Sylvain Defresne Committed by Commit Bot

[ios] Use chrome_browser_state_forward.h in /ios/chrome/browser/history

Instead of forward declaring ChromeBrowserState in the ios
namespace, use a header. This will allow moving the class
to the global namespace without having to change all the
forward declaration at the same time.

This CL was uploaded by git cl split.

R=sczs@chromium.org

Bug: 1042208
Change-Id: Ic94da0d49905274b80bf5c5e164dbb98f2936d2a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2007744
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Sergio Collazos <sczs@chromium.org>
Reviewed-by: default avatarSergio Collazos <sczs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#734032}
parent 021b5f90
...@@ -10,6 +10,7 @@ ...@@ -10,6 +10,7 @@
#include "base/macros.h" #include "base/macros.h"
#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"
enum class ServiceAccessType; enum class ServiceAccessType;
...@@ -18,9 +19,6 @@ class HistoryService; ...@@ -18,9 +19,6 @@ class HistoryService;
} }
namespace ios { namespace ios {
class ChromeBrowserState;
// Singleton that owns all HistoryServices and associates them with // Singleton that owns all HistoryServices and associates them with
// ios::ChromeBrowserState. // ios::ChromeBrowserState.
class HistoryServiceFactory : public BrowserStateKeyedServiceFactory { class HistoryServiceFactory : public BrowserStateKeyedServiceFactory {
......
...@@ -9,15 +9,13 @@ ...@@ -9,15 +9,13 @@
#include "base/memory/ref_counted.h" #include "base/memory/ref_counted.h"
#include "base/no_destructor.h" #include "base/no_destructor.h"
#include "components/keyed_service/ios/refcounted_browser_state_keyed_service_factory.h" #include "components/keyed_service/ios/refcounted_browser_state_keyed_service_factory.h"
#include "ios/chrome/browser/browser_state/chrome_browser_state_forward.h"
namespace history { namespace history {
class TopSites; class TopSites;
} }
namespace ios { namespace ios {
class ChromeBrowserState;
// TopSitesFactory is a singleton that associates history::TopSites instance to // TopSitesFactory is a singleton that associates history::TopSites instance to
// ChromeBrowserState. // ChromeBrowserState.
class TopSitesFactory : public RefcountedBrowserStateKeyedServiceFactory { class TopSitesFactory : public RefcountedBrowserStateKeyedServiceFactory {
......
...@@ -10,15 +10,13 @@ ...@@ -10,15 +10,13 @@
#include "base/macros.h" #include "base/macros.h"
#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"
namespace history { namespace history {
class WebHistoryService; class WebHistoryService;
} }
namespace ios { namespace ios {
class ChromeBrowserState;
// Singleton that owns all WebHistoryServices and associates them with // Singleton that owns all WebHistoryServices and associates them with
// ios::ChromeBrowserState. // ios::ChromeBrowserState.
class WebHistoryServiceFactory : public BrowserStateKeyedServiceFactory { class WebHistoryServiceFactory : public BrowserStateKeyedServiceFactory {
......
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