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

Unified Diff: chrome/browser/importer/external_process_importer_host.cc

Issue 2216713002: Use BookmarkModelFactory::GetForBrowserContext everywhere (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@bookmarks
Patch Set: Eliminate Profile::FromBrowserContext in some places Created 4 years, 4 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: chrome/browser/importer/external_process_importer_host.cc
diff --git a/chrome/browser/importer/external_process_importer_host.cc b/chrome/browser/importer/external_process_importer_host.cc
index 6552f7f268f8e8fbbbda7f0812216913e43979f8..479cdd4791bc3306218af6d8f16a901d69317198 100644
--- a/chrome/browser/importer/external_process_importer_host.cc
+++ b/chrome/browser/importer/external_process_importer_host.cc
@@ -94,7 +94,7 @@ void ExternalProcessImporterHost::NotifyImportEnded() {
ExternalProcessImporterHost::~ExternalProcessImporterHost() {
if (installed_bookmark_observer_) {
DCHECK(profile_);
- BookmarkModelFactory::GetForProfile(profile_)->RemoveObserver(this);
+ BookmarkModelFactory::GetForBrowserContext(profile_)->RemoveObserver(this);
}
}
@@ -192,7 +192,7 @@ void ExternalProcessImporterHost::CheckForLoadedModels(uint16_t items) {
// BookmarkModel should be loaded before adding IE favorites. So we observe
// the BookmarkModel if needed, and start the task after it has been loaded.
if ((items & importer::FAVORITES) && !writer_->BookmarkModelIsLoaded()) {
- BookmarkModelFactory::GetForProfile(profile_)->AddObserver(this);
+ BookmarkModelFactory::GetForBrowserContext(profile_)->AddObserver(this);
waiting_for_bookmarkbar_model_ = true;
installed_bookmark_observer_ = true;
}

Powered by Google App Engine
This is Rietveld 408576698