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

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

Issue 18805: Correct sqlite wrapper behavior on systems where wchar_t is UTF-32, (Closed)
Patch Set: minor fixes Created 11 years, 11 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
« no previous file with comments | « chrome/browser/meta_table_helper.cc ('k') | chrome/common/sqlite_utils.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/webdata/web_database.cc
diff --git a/chrome/browser/webdata/web_database.cc b/chrome/browser/webdata/web_database.cc
index 7d5b524e15b52d886668a8fd1fd972ff91b57a29..7ee4f79d1913c94bdf50bf2bd316bc281925d262 100755
--- a/chrome/browser/webdata/web_database.cc
+++ b/chrome/browser/webdata/web_database.cc
@@ -541,23 +541,23 @@ bool WebDatabase::GetKeywords(std::vector<TemplateURL*>* urls) {
std::wstring tmp;
template_url->set_id(s.column_int64(0));
- s.column_string16(1, &tmp);
+ s.column_wstring(1, &tmp);
DCHECK(!tmp.empty());
template_url->set_short_name(tmp);
- s.column_string16(2, &tmp);
+ s.column_wstring(2, &tmp);
template_url->set_keyword(tmp);
- s.column_string16(3, &tmp);
+ s.column_wstring(3, &tmp);
if (!tmp.empty())
template_url->SetFavIconURL(GURL(WideToUTF8(tmp)));
- s.column_string16(4, &tmp);
+ s.column_wstring(4, &tmp);
template_url->SetURL(tmp, 0, 0);
template_url->set_safe_for_autoreplace(s.column_int(5) == 1);
- s.column_string16(6, &tmp);
+ s.column_wstring(6, &tmp);
if (!tmp.empty())
template_url->set_originating_url(GURL(WideToUTF8(tmp)));
@@ -571,7 +571,7 @@ bool WebDatabase::GetKeywords(std::vector<TemplateURL*>* urls) {
template_url->set_show_in_default_list(s.column_int(10) == 1);
- s.column_string16(11, &tmp);
+ s.column_wstring(11, &tmp);
template_url->SetSuggestionsURL(tmp, 0, 0);
template_url->set_prepopulate_id(s.column_int(12));
@@ -761,12 +761,12 @@ static void InitPasswordFormFromStatement(PasswordForm* form,
form->origin = GURL(tmp);
s->column_string(1, &tmp);
form->action = GURL(tmp);
- s->column_string16(2, &form->username_element);
- s->column_string16(3, &form->username_value);
- s->column_string16(4, &form->password_element);
+ s->column_wstring(2, &form->username_element);
+ s->column_wstring(3, &form->username_value);
+ s->column_wstring(4, &form->password_element);
s->column_blob_as_string(5, &encrypted_password);
Encryptor::DecryptWideString(encrypted_password, &form->password_value);
- s->column_string16(6, &form->submit_element);
+ s->column_wstring(6, &form->submit_element);
s->column_string(7, &tmp);
form->signon_realm = tmp;
form->ssl_valid = (s->column_int(8) > 0);
@@ -1020,7 +1020,7 @@ bool WebDatabase::GetFormValuesForElementName(const std::wstring& name,
values->clear();
int result;
while ((result = s.step()) == SQLITE_ROW)
- values->push_back(s.column_string16(0));
+ values->push_back(s.column_wstring(0));
return result == SQLITE_DONE;
}
« no previous file with comments | « chrome/browser/meta_table_helper.cc ('k') | chrome/common/sqlite_utils.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698