summaryrefslogtreecommitdiffstats
path: root/chrome
diff options
context:
space:
mode:
authortony@chromium.org <tony@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-03-08 00:49:16 +0000
committertony@chromium.org <tony@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-03-08 00:49:16 +0000
commit7f23bac00f99429e34198b12322c11de3097e0ae (patch)
treed6d5994d06ed4aaa1b11aa3bcaf55ebaa2fdcaa8 /chrome
parentb852b07bce576b50aae75cbdc6ebf8c53d226c7f (diff)
downloadchromium_src-7f23bac00f99429e34198b12322c11de3097e0ae.zip
chromium_src-7f23bac00f99429e34198b12322c11de3097e0ae.tar.gz
chromium_src-7f23bac00f99429e34198b12322c11de3097e0ae.tar.bz2
Remove wstrings from ui/base/text_elider.h functions.
The last param is for languages, which is already a std::string in most places. BUG=23581 Review URL: http://codereview.chromium.org/6627061 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77219 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
-rw-r--r--chrome/browser/possible_url_model.cc5
-rw-r--r--chrome/browser/ui/app_modal_dialogs/message_box_handler.cc3
-rw-r--r--chrome/browser/ui/cocoa/status_bubble_mac.mm4
-rw-r--r--chrome/browser/ui/gtk/status_bubble_gtk.cc2
-rw-r--r--chrome/browser/ui/status_bubble.h2
-rw-r--r--chrome/browser/ui/views/bookmarks/bookmark_bar_view.cc4
-rw-r--r--chrome/browser/ui/views/options/exceptions_page_view.cc4
-rw-r--r--chrome/browser/ui/views/options/passwords_page_view.cc4
-rw-r--r--chrome/browser/ui/views/status_bubble_views.cc4
9 files changed, 16 insertions, 16 deletions
diff --git a/chrome/browser/possible_url_model.cc b/chrome/browser/possible_url_model.cc
index c3e0815..73e9b46 100644
--- a/chrome/browser/possible_url_model.cc
+++ b/chrome/browser/possible_url_model.cc
@@ -80,9 +80,8 @@ void PossibleURLModel::Reload(Profile *profile) {
void PossibleURLModel::OnHistoryQueryComplete(HistoryService::Handle h,
history::QueryResults* result) {
results_.resize(result->size());
- std::wstring languages = profile_ ?
- UTF8ToWide(profile_->GetPrefs()->GetString(prefs::kAcceptLanguages)) :
- std::wstring();
+ std::string languages = profile_ ?
+ profile_->GetPrefs()->GetString(prefs::kAcceptLanguages) : std::string();
for (size_t i = 0; i < result->size(); ++i) {
results_[i].url = (*result)[i].url();
results_[i].index = i;
diff --git a/chrome/browser/ui/app_modal_dialogs/message_box_handler.cc b/chrome/browser/ui/app_modal_dialogs/message_box_handler.cc
index 5124ac1..0de1b9a 100644
--- a/chrome/browser/ui/app_modal_dialogs/message_box_handler.cc
+++ b/chrome/browser/ui/app_modal_dialogs/message_box_handler.cc
@@ -47,8 +47,7 @@ static std::wstring GetTitle(Profile* profile,
// TODO(brettw) it should be easier than this to do the correct language
// handling without getting the accept language from the profile.
string16 base_address = ui::ElideUrl(frame_url.GetOrigin(),
- gfx::Font(), 0,
- UTF8ToWide(profile->GetPrefs()->GetString(prefs::kAcceptLanguages)));
+ gfx::Font(), 0, profile->GetPrefs()->GetString(prefs::kAcceptLanguages));
// Force URL to have LTR directionality.
base_address = base::i18n::GetDisplayStringInLTRDirectionality(
diff --git a/chrome/browser/ui/cocoa/status_bubble_mac.mm b/chrome/browser/ui/cocoa/status_bubble_mac.mm
index f6fc627..aa77c16 100644
--- a/chrome/browser/ui/cocoa/status_bubble_mac.mm
+++ b/chrome/browser/ui/cocoa/status_bubble_mac.mm
@@ -151,7 +151,7 @@ void StatusBubbleMac::SetURL(const GURL& url, const string16& languages) {
string16 original_url_text = net::FormatUrl(url, UTF16ToUTF8(languages));
string16 status = ui::ElideUrl(url, font_chr, text_width,
- UTF16ToWideHack(languages));
+ UTF16ToUTF8(languages));
SetText(status, true);
@@ -591,7 +591,7 @@ void StatusBubbleMac::ExpandBubble() {
gfx::Font font_chr(base::SysNSStringToUTF16([font fontName]),
[font pointSize]);
string16 expanded_url = ui::ElideUrl(url_, font_chr,
- max_bubble_width, UTF16ToWideHack(languages_));
+ max_bubble_width, UTF16ToUTF8(languages_));
// Scale width from gfx::Font in view coordinates to window coordinates.
int required_width_for_string =
diff --git a/chrome/browser/ui/gtk/status_bubble_gtk.cc b/chrome/browser/ui/gtk/status_bubble_gtk.cc
index 98e08e2..de735bb 100644
--- a/chrome/browser/ui/gtk/status_bubble_gtk.cc
+++ b/chrome/browser/ui/gtk/status_bubble_gtk.cc
@@ -107,7 +107,7 @@ void StatusBubbleGtk::SetStatusTextToURL() {
// TODO(tc): We don't actually use gfx::Font as the font in the status
// bubble. We should extend ui::ElideUrl to take some sort of pango font.
url_text_ = UTF16ToUTF8(ui::ElideUrl(url_, gfx::Font(), desired_width,
- UTF16ToWideHack(languages_)));
+ UTF16ToUTF8(languages_)));
SetStatusTextTo(url_text_);
}
diff --git a/chrome/browser/ui/status_bubble.h b/chrome/browser/ui/status_bubble.h
index e7ddc15..43a30d2 100644
--- a/chrome/browser/ui/status_bubble.h
+++ b/chrome/browser/ui/status_bubble.h
@@ -36,6 +36,8 @@ class StatusBubble {
// the bubble to fade in and remain open until given an empty URL or until
// the Hide() method is called. languages is the value of Accept-Language
// to determine what characters are understood by a user.
+ // TODO(tc): |languages| should be std::string (ascii)-- it's current
+ // usage is as bad as a WideToUTF8Hack.
virtual void SetURL(const GURL& url, const string16& languages) = 0;
// Skip the fade and instant-hide the bubble.
diff --git a/chrome/browser/ui/views/bookmarks/bookmark_bar_view.cc b/chrome/browser/ui/views/bookmarks/bookmark_bar_view.cc
index 6fd2f90..aee3bf8 100644
--- a/chrome/browser/ui/views/bookmarks/bookmark_bar_view.cc
+++ b/chrome/browser/ui/views/bookmarks/bookmark_bar_view.cc
@@ -1575,8 +1575,8 @@ std::wstring BookmarkBarView::CreateToolTipForURLAndTitle(
// "/http://www.yahoo.com" when rendered, as is, in an RTL context since
// the Unicode BiDi algorithm puts certain characters on the left by
// default.
- std::wstring languages =
- UTF8ToWide(profile->GetPrefs()->GetString(prefs::kAcceptLanguages));
+ std::string languages = profile->GetPrefs()->GetString(
+ prefs::kAcceptLanguages);
string16 elided_url(ui::ElideUrl(url, tt_font, max_width, languages));
elided_url = base::i18n::GetDisplayStringInLTRDirectionality(elided_url);
result.append(UTF16ToWideHack(elided_url));
diff --git a/chrome/browser/ui/views/options/exceptions_page_view.cc b/chrome/browser/ui/views/options/exceptions_page_view.cc
index ab96ed8..363c3e8 100644
--- a/chrome/browser/ui/views/options/exceptions_page_view.cc
+++ b/chrome/browser/ui/views/options/exceptions_page_view.cc
@@ -52,8 +52,8 @@ void ExceptionsTableModel::OnPasswordStoreRequestDone(
pending_login_query_ = NULL;
STLDeleteElements<PasswordRows>(&saved_signons_);
- std::wstring languages =
- UTF8ToWide(profile_->GetPrefs()->GetString(prefs::kAcceptLanguages));
+ std::string languages = profile_->GetPrefs()->GetString(
+ prefs::kAcceptLanguages);
for (size_t i = 0; i < result.size(); ++i) {
saved_signons_.push_back(new PasswordRow(
ui::SortedDisplayURL(result[i]->origin, languages), result[i]));
diff --git a/chrome/browser/ui/views/options/passwords_page_view.cc b/chrome/browser/ui/views/options/passwords_page_view.cc
index 705c4d1..9b714e0 100644
--- a/chrome/browser/ui/views/options/passwords_page_view.cc
+++ b/chrome/browser/ui/views/options/passwords_page_view.cc
@@ -115,8 +115,8 @@ void PasswordsTableModel::OnPasswordStoreRequestDone(
STLDeleteElements<PasswordRows>(&saved_signons_);
saved_signons_.resize(result.size(), NULL);
- std::wstring languages =
- UTF8ToWide(profile_->GetPrefs()->GetString(prefs::kAcceptLanguages));
+ std::string languages = profile_->GetPrefs()->GetString(
+ prefs::kAcceptLanguages);
for (size_t i = 0; i < result.size(); ++i) {
saved_signons_[i] = new PasswordRow(
ui::SortedDisplayURL(result[i]->origin, languages), result[i]);
diff --git a/chrome/browser/ui/views/status_bubble_views.cc b/chrome/browser/ui/views/status_bubble_views.cc
index 6fa60c1..f7d2f97 100644
--- a/chrome/browser/ui/views/status_bubble_views.cc
+++ b/chrome/browser/ui/views/status_bubble_views.cc
@@ -647,7 +647,7 @@ void StatusBubbleViews::SetURL(const GURL& url, const string16& languages) {
int text_width = static_cast<int>(popup_bounds.width() -
(kShadowThickness * 2) - kTextPositionX - kTextHorizPadding - 1);
url_text_ = ui::ElideUrl(url, view_->Label::font(),
- text_width, UTF16ToWideHack(languages));
+ text_width, UTF16ToUTF8(languages));
std::wstring original_url_text =
UTF16ToWideHack(net::FormatUrl(url, UTF16ToUTF8(languages)));
@@ -801,7 +801,7 @@ void StatusBubbleViews::ExpandBubble() {
gfx::Rect popup_bounds = popup_->GetWindowScreenBounds();
int max_status_bubble_width = GetMaxStatusBubbleWidth();
url_text_ = ui::ElideUrl(url_, view_->Label::font(),
- max_status_bubble_width, UTF16ToWideHack(languages_));
+ max_status_bubble_width, UTF16ToUTF8(languages_));
int expanded_bubble_width =std::max(GetStandardStatusBubbleWidth(),
std::min(view_->Label::font().GetStringWidth(url_text_) +
(kShadowThickness * 2) + kTextPositionX +