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

Unified Diff: ios/public/provider/chrome/browser/chrome_browser_provider.cc

Issue 1547403002: Move ownership of ChromeBrowserStateManager to ApplicationContext. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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
« no previous file with comments | « ios/public/provider/chrome/browser/chrome_browser_provider.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/public/provider/chrome/browser/chrome_browser_provider.cc
diff --git a/ios/public/provider/chrome/browser/chrome_browser_provider.cc b/ios/public/provider/chrome/browser/chrome_browser_provider.cc
index 560bec385409fa0a9c23b837e15dea30110af48a..287fc8137ef5cc69baa7409c4f621c00ca37ff03 100644
--- a/ios/public/provider/chrome/browser/chrome_browser_provider.cc
+++ b/ios/public/provider/chrome/browser/chrome_browser_provider.cc
@@ -10,6 +10,7 @@
#include "components/metrics/metrics_provider.h"
#include "components/sync_sessions/local_session_event_router.h"
#include "components/sync_sessions/synced_window_delegates_getter.h"
+#include "ios/public/provider/chrome/browser/browser_state/chrome_browser_state_manager.h"
namespace ios {
@@ -25,20 +26,16 @@ ChromeBrowserProvider* GetChromeBrowserProvider() {
return g_chrome_browser_provider;
}
-ChromeBrowserProvider::~ChromeBrowserProvider() {
-}
+ChromeBrowserProvider::~ChromeBrowserProvider() {}
// A dummy implementation of ChromeBrowserProvider.
-ChromeBrowserProvider::ChromeBrowserProvider() {
-}
+ChromeBrowserProvider::ChromeBrowserProvider() {}
-void ChromeBrowserProvider::AssertBrowserContextKeyedFactoriesBuilt() {
-}
+void ChromeBrowserProvider::AssertBrowserContextKeyedFactoriesBuilt() {}
void ChromeBrowserProvider::RegisterProfilePrefs(
- user_prefs::PrefRegistrySyncable* registry) {
-}
+ user_prefs::PrefRegistrySyncable* registry) {}
ProfileOAuth2TokenServiceIOSProvider*
ChromeBrowserProvider::GetProfileOAuth2TokenServiceIOSProvider() {
@@ -50,9 +47,9 @@ ChromeBrowserProvider::GetUpdatableResourceProvider() {
return nullptr;
}
-ChromeBrowserStateManager*
-ChromeBrowserProvider::GetChromeBrowserStateManager() {
- return nullptr;
+scoped_ptr<ChromeBrowserStateManager>
+ChromeBrowserProvider::CreateChromeBrowserStateManager() {
+ return scoped_ptr<ChromeBrowserStateManager>();
}
InfoBarViewPlaceholder ChromeBrowserProvider::CreateInfoBarView(
@@ -91,8 +88,7 @@ std::string ChromeBrowserProvider::GetDistributionBrandCode() {
}
void ChromeBrowserProvider::SetUIViewAlphaWithAnimation(UIView* view,
- float alpha) {
-}
+ float alpha) {}
autofill::CardUnmaskPromptView*
ChromeBrowserProvider::CreateCardUnmaskPromptView(
« no previous file with comments | « ios/public/provider/chrome/browser/chrome_browser_provider.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698