Chromium Code Reviews| Index: chrome/browser/search_engines/search_provider_install_data.cc |
| =================================================================== |
| --- chrome/browser/search_engines/search_provider_install_data.cc (revision 131375) |
| +++ chrome/browser/search_engines/search_provider_install_data.cc (working copy) |
| @@ -129,8 +129,7 @@ |
| const content::NotificationDetails& details) { |
| if (type == chrome::NOTIFICATION_GOOGLE_URL_UPDATED) { |
| BrowserThread::PostTask(BrowserThread::IO, FROM_HERE, |
| - base::Bind(&GoogleURLChangeNotifier::OnChange, |
| - change_notifier_.get(), |
| + base::Bind(&GoogleURLChangeNotifier::OnChange, change_notifier_.get(), |
| UIThreadSearchTermsData().GoogleBaseURLValue())); |
| } else { |
| // This must be the death notification. |
| @@ -145,9 +144,8 @@ |
| const TemplateURL* template_url, |
| const SearchTermsData& search_terms_data) { |
| DCHECK(requested_origin == requested_origin.GetOrigin()); |
| - return template_url && requested_origin == |
|
Peter Kasting
2012/04/10 01:24:33
I don't think our lone caller will ever pass NULL
|
| - TemplateURLService::GenerateSearchURLUsingTermsData( |
| - template_url, |
| + return requested_origin == |
| + TemplateURLService::GenerateSearchURLUsingTermsData(template_url, |
| search_terms_data).GetOrigin(); |
| } |