ChromeOS status text halo polish.

chrome/browser/chromeos/status/status_area_button.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/status/status_area_button.h"
 
 #include "gfx/canvas.h"
 #include "gfx/skbitmap_operations.h"
 #include "grit/theme_resources.h"
 #include "views/border.h"
 #include "views/view.h"
 
 namespace chromeos {
 
 ////////////////////////////////////////////////////////////////////////////////
 // StatusAreaButton
 
 StatusAreaButton::StatusAreaButton(views::ViewMenuDelegate* menu_delegate)
     : MenuButton(NULL, std::wstring(), menu_delegate, false),
       use_menu_button_paint_(false), enabled_(true) {
   set_border(NULL);
 
   // Use an offset that is top aligned with toolbar.
   set_menu_offset(0, 2);
 
-  // Use a black halo for status text.
-  SetTextHaloColor(SK_ColorBLACK);
+  // Use a 0.70 alpha black halo for status text as the icons.
+  SetTextHaloColor(SkColorSetARGB(0xB3, 0, 0, 0));

[DaveMoore, 2010/12/04 00:08:02]

Could you make this a constant?

[xiyuan, 2010/12/04 00:35:21]

Done.

 }
 
 void StatusAreaButton::Paint(gfx::Canvas* canvas, bool for_drag) {
   if (state() == BS_PUSHED) {
     // Apply 10% white when pushed down.
     canvas->FillRectInt(SkColorSetARGB(0x19, 0xFF, 0xFF, 0xFF),
         0, 0, width(), height());
   }
 
   if (use_menu_button_paint_) {
@@ skipping 48 matching lines @@
 
 bool StatusAreaButton::Activate() {
   if (enabled_) {
     return views::MenuButton::Activate();
   } else {
     return true;
   }
 }
 
 }  // namespace chromeos