Colors in views::StyledLabel.

chrome/browser/chromeos/ui/echo_dialog_view.cc

Index: chrome/browser/chromeos/ui/echo_dialog_view.cc
diff --git a/chrome/browser/chromeos/ui/echo_dialog_view.cc b/chrome/browser/chromeos/ui/echo_dialog_view.cc
index 15b8ed8e1df8676a60c82082928a14f4bc44150d..fde9b23cb2c0ddac36db80eb94f395c5f2b14795 100644
--- a/chrome/browser/chromeos/ui/echo_dialog_view.cc
+++ b/chrome/browser/chromeos/ui/echo_dialog_view.cc
@@ -58,9 +58,9 @@ void EchoDialogView::InitForEnabledEcho(const string16& service_name,
       ui::Range(offsets[0], offsets[0] + service_name.length()),
       service_name_style);
 
-  views::StyledLabel::RangeStyleInfo link_style;
+  views::StyledLabel::RangeStyleInfo link_style =
+      views::StyledLabel::RangeStyleInfo::CreateForLink();

[Roger Tawa OOO till Jul 10th, 2013/06/26 15:16:29]

This changes |disable_line_wrapping| from false to true.  Is that OK?

[fdoray, 2013/06/26 17:06:38]

I think it's OK. We don't want a line wrap in the middle of a "Learn More" link.

[Roger Tawa OOO till Jul 10th, 2013/07/02 14:45:29]

OK.  This seems more generic than just the "learn more" link so that's why I was
asking.

   link_style.font_style = gfx::Font::NORMAL;
-  link_style.is_link = true;
   label_->AddStyleRange(ui::Range(offsets[1], offsets[1] + link.length()),
                         link_style);
 
@@ -82,9 +82,9 @@ void EchoDialogView::InitForDisabledEcho() {
 
   label_ = new views::StyledLabel(text, this);
 
-  views::StyledLabel::RangeStyleInfo link_style;
+  views::StyledLabel::RangeStyleInfo link_style =
+      views::StyledLabel::RangeStyleInfo::CreateForLink();
   link_style.font_style = gfx::Font::NORMAL;
-  link_style.is_link = true;
   label_->AddStyleRange(ui::Range(offset, offset + link.length()), link_style);
 
   SetLabelBorderAndBounds();