Remove COLOR_STATUS_BAR_TEXT.

chrome/browser/ui/views/status_bubble_views.cc

Index: chrome/browser/ui/views/status_bubble_views.cc
diff --git a/chrome/browser/ui/views/status_bubble_views.cc b/chrome/browser/ui/views/status_bubble_views.cc
index e2b21bd82bed9934fee28ac7f62c701ab3796094..826659a2f26e8282ffff0837101ae30764425c60 100644
--- a/chrome/browser/ui/views/status_bubble_views.cc
+++ b/chrome/browser/ui/views/status_bubble_views.cc
@@ -420,22 +420,25 @@ void StatusBubbleViews::StatusView::OnPaint(gfx::Canvas* canvas) {
   rrect.setRectRadii(RectToSkRect(rect), (const SkVector*)rad);
   canvas->sk_canvas()->drawRRect(rrect, paint);
 
-  // Draw highlight text and then the text body. In order to make sure the text
-  // is aligned to the right on RTL UIs, we mirror the text bounds if the
-  // locale is RTL.
+  // Compute text bounds.
   const gfx::FontList font_list;
   int text_width =
       std::min(gfx::GetStringWidth(text_, font_list),
                width - shadow_size - kTextPositionX - kTextHorizPadding);
   int text_height = height - shadow_size;
-  gfx::Rect body_bounds(kShadowThickness + kTextPositionX,
+  gfx::Rect text_bounds(kShadowThickness + kTextPositionX,
                         kShadowThickness,
                         std::max(0, text_width),
                         std::max(0, text_height));
-  body_bounds.set_x(GetMirroredXForRect(body_bounds));
+  // Make sure the text is aligned to the right on RTL UIs.
+  text_bounds.set_x(GetMirroredXForRect(text_bounds));
+
+  // Text color is the foreground tab text color at 50% alpha.
   SkColor text_color =
-      theme_provider_->GetColor(ThemeProperties::COLOR_STATUS_BAR_TEXT);
-  canvas->DrawStringRect(text_, font_list, text_color, body_bounds);
+      theme_provider_->GetColor(ThemeProperties::COLOR_TAB_TEXT);

[Evan Stade, 2016/03/04 04:22:16]

I tried this out on GTK theme mode and yea, it looks fine, if you assume the
classic theme also looks fine. But honestly I don't know why we don't just use
tab text color full stop (100% alpha). Can we ask a designer if that's OK?

I'm somewhat worried that this calculation will leave us with unreadable text in
some cases (not enough contrast). At the very least we should use
color_utils::GetReadableColor() but it would be easier (and imo looks fine) to
just use normal tab text color.

[Peter Kasting, 2016/03/04 05:53:46]

We use this intentionally because we're trying to avoid drawing attention to the
status bar text.  We found early on that the status bar was super distracting
and we want to at least minimize that effect.  It's been this was since at least
2009 on all non-Linux platforms, so I think it's fine; Linux just went under the
radar.

+  canvas->DrawStringRect(text_, font_list,
+                         SkColorSetA(text_color, SkColorGetA(text_color) / 2),
+                         text_bounds);
 }