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

Unified Diff: chrome/browser/ui/views/avatar_menu_bubble_view.cc

Issue 8142026: Revert 104076 - Change std::wstring to string16 for views::Link (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: 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/ui/views/avatar_menu_bubble_view.cc
===================================================================
--- chrome/browser/ui/views/avatar_menu_bubble_view.cc (revision 104083)
+++ chrome/browser/ui/views/avatar_menu_bubble_view.cc (working copy)
@@ -4,8 +4,6 @@
#include "chrome/browser/ui/views/avatar_menu_bubble_view.h"
-#include <algorithm>
-
#include "base/utf_string_conversions.h"
#include "chrome/browser/browser_process.h"
#include "chrome/browser/ui/browser.h"
@@ -60,7 +58,6 @@
// Delegate to callback when the highlight state of a control changes.
class HighlightDelegate {
public:
- virtual ~HighlightDelegate() {}
virtual void OnHighlightStateChanged() = 0;
};
@@ -71,7 +68,7 @@
public:
explicit EditProfileLink(const string16& title,
HighlightDelegate* delegate)
- : views::Link(title),
+ : views::Link(UTF16ToWideHack(title)),
delegate_(delegate),
state_(views::CustomButton::BS_NORMAL) {
}
@@ -124,7 +121,7 @@
AddChildView(image_view_);
// Add a label to show the profile name.
- name_label_ = new views::Label(item_.name);
+ name_label_ = new views::Label(UTF16ToWideHack(item_.name));
ResourceBundle& rb = ResourceBundle::GetSharedInstance();
gfx::Font base_font = rb.GetFont(ResourceBundle::BaseFont);
int style = item_.active ? gfx::Font::BOLD : 0;
@@ -134,7 +131,7 @@
AddChildView(name_label_);
// Add a label to show the sync state.
- sync_state_label_ = new views::Label(item_.sync_state);
+ sync_state_label_ = new views::Label(UTF16ToWideHack(item_.sync_state));
const int kStateFontDelta = -1;
sync_state_label_->SetFont(base_font.DeriveFont(kStateFontDelta));
sync_state_label_->SetHorizontalAlignment(views::Label::ALIGN_LEFT);
@@ -250,7 +247,7 @@
views::Label* sync_state_label_;
};
-} // namespace
+} // namespace
AvatarMenuBubbleView::AvatarMenuBubbleView(Browser* browser)
: add_profile_link_(NULL),
@@ -368,8 +365,8 @@
separator_ = new views::Separator();
AddChildView(separator_);
- add_profile_link_ = new views::Link(
- l10n_util::GetStringUTF16(IDS_PROFILES_CREATE_NEW_PROFILE_LINK));
+ add_profile_link_ = new views::Link(UTF16ToWide(
+ l10n_util::GetStringUTF16(IDS_PROFILES_CREATE_NEW_PROFILE_LINK)));
add_profile_link_->set_listener(this);
add_profile_link_->SetHorizontalAlignment(views::Label::ALIGN_LEFT);
add_profile_link_->SetNormalColor(SkColorSetRGB(0, 0x79, 0xda));

Powered by Google App Engine
This is Rietveld 408576698