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

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

Issue 10914278: Fix for non-sticky defaults bug. Added a bunch of tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase to head Created 8 years, 3 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_intents_table.cc
diff --git a/chrome/browser/webdata/web_intents_table.cc b/chrome/browser/webdata/web_intents_table.cc
index 90243112e2be6be5d6af888d6ce11148eecaa703..99709d770aa232b8e809d97ee9860d46deaf3108 100644
--- a/chrome/browser/webdata/web_intents_table.cc
+++ b/chrome/browser/webdata/web_intents_table.cc
@@ -300,14 +300,16 @@ bool WebIntentsTable::SetDefaultService(
const DefaultWebIntentService& default_service) {
sql::Statement s(db_->GetUniqueStatement(
"INSERT OR REPLACE INTO web_intents_defaults "
- "(action, type, url_pattern, user_date, suppression, service_url) "
- "VALUES (?, ?, ?, ?, ?, ?)"));
+ "(action, type, url_pattern, user_date, suppression,"
+ " service_url, scheme) "
+ "VALUES (?, ?, ?, ?, ?, ?, ?)"));
s.BindString16(0, default_service.action);
s.BindString16(1, default_service.type);
s.BindString(2, default_service.url_pattern.GetAsString());
s.BindInt(3, default_service.user_date);
s.BindInt64(4, default_service.suppression);
s.BindString(5, default_service.service_url);
+ s.BindString16(6, default_service.scheme);
return s.Run();
}
« no previous file with comments | « chrome/browser/webdata/web_data_service_unittest.cc ('k') | chrome/browser/webdata/web_intents_table_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698