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

Unified Diff: chrome/browser/webdata/web_data_service.cc

Issue 8342049: Added Protector, hooked up DSE verification with error bubble. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Moved check to WebDataService initialization. Created 9 years, 2 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/webdata/web_data_service.cc
diff --git a/chrome/browser/webdata/web_data_service.cc b/chrome/browser/webdata/web_data_service.cc
index 5d4a09cc9ca8c619c4b9a817cc549234bfe9e085..7ab4bc5b232bd8b1a9828a5cd2e47d746fd572aa 100644
--- a/chrome/browser/webdata/web_data_service.cc
+++ b/chrome/browser/webdata/web_data_service.cc
@@ -11,6 +11,8 @@
#include "chrome/browser/autofill/autofill_profile.h"
#include "chrome/browser/autofill/credit_card.h"
#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/protector/protector.h"
+#include "chrome/browser/protector/setting_change.h"
#include "chrome/browser/search_engines/template_url.h"
#include "chrome/browser/ui/profile_error_dialog.h"
#include "chrome/browser/webdata/autofill_change.h"
@@ -624,6 +626,18 @@ void WebDataService::InitializeDatabaseIfNecessary() {
return;
}
+ // Verify the default search provider id and notify UI if it's wrong.
+ KeywordTable* keyword_table = db->GetKeywordTable();
+ if (keyword_table && !keyword_table->IsDefaultSearchProviderIDVerified()) {
+ protector::SettingChange* change =
+ protector::CreateDefaultSearchProviderChange(keyword_table);
+ BrowserThread::PostTask(
+ BrowserThread::UI, FROM_HERE,
+ NewRunnableFunction(&protector::NotifyUserOfChange, change));
+ keyword_table->SetDefaultSearchProviderID(
+ keyword_table->GetDefaultSearchProviderIDBackup());
whywhat 2011/10/20 23:02:39 Scott, when the ID is set to 0, there's no default
sky 2011/10/20 23:40:59 Don't expose the keyword table. Doing so breaks th
whywhat 2011/10/21 16:48:18 Done.
+ }
+
BrowserThread::PostTask(
BrowserThread::UI, FROM_HERE,
NewRunnableMethod(this, &WebDataService::NotifyDatabaseLoadedOnUIThread));

Powered by Google App Engine
This is Rietveld 408576698