summaryrefslogtreecommitdiffstats
path: root/chrome/browser/webdata
diff options
context:
space:
mode:
authorestade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-06-29 20:35:19 +0000
committerestade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-06-29 20:35:19 +0000
commitddd231eaa03153f6d04894cceb0b4480755e1277 (patch)
treefaafed02945c1f8934e6e393ac0c6b094c303cdf /chrome/browser/webdata
parentc5e30d8572ffae1e0d4fbb2fff765f9a8cbace77 (diff)
downloadchromium_src-ddd231eaa03153f6d04894cceb0b4480755e1277.zip
chromium_src-ddd231eaa03153f6d04894cceb0b4480755e1277.tar.gz
chromium_src-ddd231eaa03153f6d04894cceb0b4480755e1277.tar.bz2
Change a bunch of string types.
Started out just trying to change PrefService::GetString and ::SetString. This snowballed a little bit. Had to change a bunch of url strings in search_engines/ from wstring to string (some of them may be better off as GURLs, but UTF-8 is a step in the right direction, since that's what GURL uses internally, as well as externally via its setters/getters). TODO (later patch): things that ask for accepted languages should use std::string, not std::wstring. BUG=none TEST=try bots Review URL: http://codereview.chromium.org/2854015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@51157 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/webdata')
-rw-r--r--chrome/browser/webdata/web_database.cc8
-rw-r--r--chrome/browser/webdata/web_database_unittest.cc8
2 files changed, 8 insertions, 8 deletions
diff --git a/chrome/browser/webdata/web_database.cc b/chrome/browser/webdata/web_database.cc
index 02ac222..870d2d7 100644
--- a/chrome/browser/webdata/web_database.cc
+++ b/chrome/browser/webdata/web_database.cc
@@ -546,7 +546,7 @@ static void BindURLToStatement(const TemplateURL& url, sql::Statement* s) {
url.GetFavIconURL()));
}
if (url.url())
- s->BindString(3, WideToUTF8(url.url()->url()));
+ s->BindString(3, url.url()->url());
else
s->BindString(3, std::string());
s->BindInt(4, url.safe_for_autoreplace() ? 1 : 0);
@@ -561,7 +561,7 @@ static void BindURLToStatement(const TemplateURL& url, sql::Statement* s) {
s->BindString(8, JoinStrings(";", url.input_encodings()));
s->BindInt(9, url.show_in_default_list() ? 1 : 0);
if (url.suggestions_url())
- s->BindString(10, WideToUTF8(url.suggestions_url()->url()));
+ s->BindString(10, url.suggestions_url()->url());
else
s->BindString(10, std::string());
s->BindInt(11, url.prepopulate_id());
@@ -629,7 +629,7 @@ bool WebDatabase::GetKeywords(std::vector<TemplateURL*>* urls) {
template_url->SetFavIconURL(GURL(tmp));
tmp = s.ColumnString(4);
- template_url->SetURL(UTF8ToWide(tmp), 0, 0);
+ template_url->SetURL(tmp, 0, 0);
template_url->set_safe_for_autoreplace(s.ColumnInt(5) == 1);
@@ -648,7 +648,7 @@ bool WebDatabase::GetKeywords(std::vector<TemplateURL*>* urls) {
template_url->set_show_in_default_list(s.ColumnInt(10) == 1);
tmp = s.ColumnString(11);
- template_url->SetSuggestionsURL(UTF8ToWide(tmp), 0, 0);
+ template_url->SetSuggestionsURL(tmp, 0, 0);
template_url->set_prepopulate_id(s.ColumnInt(12));
diff --git a/chrome/browser/webdata/web_database_unittest.cc b/chrome/browser/webdata/web_database_unittest.cc
index 0b9af58..3d3d1cf9 100644
--- a/chrome/browser/webdata/web_database_unittest.cc
+++ b/chrome/browser/webdata/web_database_unittest.cc
@@ -177,7 +177,7 @@ TEST_F(WebDatabaseTest, Keywords) {
GURL favicon_url("http://favicon.url/");
GURL originating_url("http://google.com/");
template_url.SetFavIconURL(favicon_url);
- template_url.SetURL(L"http://url/", 0, 0);
+ template_url.SetURL("http://url/", 0, 0);
template_url.set_safe_for_autoreplace(true);
template_url.set_show_in_default_list(true);
template_url.set_originating_url(originating_url);
@@ -253,10 +253,10 @@ TEST_F(WebDatabaseTest, UpdateKeyword) {
GURL favicon_url("http://favicon.url/");
GURL originating_url("http://originating.url/");
template_url.SetFavIconURL(favicon_url);
- template_url.SetURL(L"http://url/", 0, 0);
+ template_url.SetURL("http://url/", 0, 0);
template_url.set_safe_for_autoreplace(true);
template_url.set_show_in_default_list(true);
- template_url.SetSuggestionsURL(L"url2", 0, 0);
+ template_url.SetSuggestionsURL("url2", 0, 0);
SetID(1, &template_url);
EXPECT_TRUE(db.AddKeyword(template_url));
@@ -312,7 +312,7 @@ TEST_F(WebDatabaseTest, KeywordWithNoFavicon) {
TemplateURL template_url;
template_url.set_short_name(L"short_name");
template_url.set_keyword(L"keyword");
- template_url.SetURL(L"http://url/", 0, 0);
+ template_url.SetURL("http://url/", 0, 0);
template_url.set_safe_for_autoreplace(true);
SetID(-100, &template_url);