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

Unified Diff: chrome/browser/automation/automation_provider_observers.cc

Issue 10238014: Fix search engine automation hooks to work with multiple profiles. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years, 8 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/automation/automation_provider_observers.cc
diff --git a/chrome/browser/automation/automation_provider_observers.cc b/chrome/browser/automation/automation_provider_observers.cc
index f7abc4176fcb7030a896a7d7dbaaa57be677348a..a6ad87c446db7f847bdd15896cd51b4a77782291 100644
--- a/chrome/browser/automation/automation_provider_observers.cc
+++ b/chrome/browser/automation/automation_provider_observers.cc
@@ -1529,8 +1529,10 @@ void AllDownloadsCompleteObserver::ReplyIfNecessary() {
AutomationProviderSearchEngineObserver::AutomationProviderSearchEngineObserver(
AutomationProvider* provider,
+ Profile* profile,
IPC::Message* reply_message)
: provider_(provider->AsWeakPtr()),
+ profile_(profile),
reply_message_(reply_message) {
}
@@ -1540,7 +1542,7 @@ AutomationProviderSearchEngineObserver::
void AutomationProviderSearchEngineObserver::OnTemplateURLServiceChanged() {
if (provider_) {
TemplateURLService* url_service =
- TemplateURLServiceFactory::GetForProfile(provider_->profile());
+ TemplateURLServiceFactory::GetForProfile(profile_);
url_service->RemoveObserver(this);
AutomationJSONReply(provider_, reply_message_.release()).SendSuccess(NULL);
}

Powered by Google App Engine
This is Rietveld 408576698