Apply subpixel rendering to labels in cros system menu.

ash/common/system/tray/tray_popup_utils.cc

 // Copyright 2016 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 "ash/common/system/tray/tray_popup_utils.h"
 
 #include "ash/common/ash_constants.h"
 #include "ash/common/ash_view_ids.h"
 #include "ash/common/material_design/material_design_controller.h"
 #include "ash/common/session/session_state_delegate.h"
@@ skipping 203 matching lines @@
                                base::MakeUnique<views::FillLayout>());
 
   return tri_view;
 }
 
 views::Label* TrayPopupUtils::CreateDefaultLabel() {
   views::Label* label = new views::Label();
   label->SetHorizontalAlignment(gfx::ALIGN_LEFT);
   label->SetBorder(
       views::CreateEmptyBorder(0, 0, 0, kTrayPopupLabelRightPadding));
-
-  // TODO(bruthig): Fix this so that |label| uses the kBackgroundColor to
-  // perform subpixel rendering instead of disabling subpixel rendering.
-  //
-  // Text rendered on a non-opaque background looks ugly and it is possible for
-  // labels to given a a clear canvas at paint time when an ink drop is visible.
-  // See http://crbug.com/661714.
-  label->SetSubpixelRenderingEnabled(false);
-
+  // Frequently the label will paint to a layer that's non-opaque, so subpixel
+  // rendering won't work unless we explicitly set a background. See
+  // crbug.com/686363
+  label->set_background(
+      views::Background::CreateSolidBackground(kBackgroundColor));
+  label->SetBackgroundColor(kBackgroundColor);
   return label;
 }
 
 views::ImageView* TrayPopupUtils::CreateMainImageView() {
   return new FixedSizedImageView(
       GetTrayConstant(TRAY_POPUP_ITEM_MAIN_IMAGE_CONTAINER_WIDTH),
       GetTrayConstant(TRAY_POPUP_ITEM_MIN_HEIGHT));
 }
 
 views::ImageView* TrayPopupUtils::CreateMoreImageView() {
@@ skipping 226 matching lines @@
 bool TrayPopupUtils::CanOpenWebUISettings(LoginStatus status) {
   // TODO(tdanderson): Consider moving this into WmShell, or introduce a
   // CanShowSettings() method in each delegate type that has a
   // ShowSettings() method.
   return status != LoginStatus::NOT_LOGGED_IN &&
          status != LoginStatus::LOCKED &&
          !WmShell::Get()->GetSessionStateDelegate()->IsInSecondaryLoginScreen();
 }
 
 }  // namespace ash