Commit b8435d96 authored by minch's avatar minch Committed by Commit Bot

dark_mode: Delete unnecessary color setting in TimeView.

Color of |vertical_label_minutes_| will be set through
TimeView::SetTextColorBasedOnSession at
https://cs.chromium.org/chromium/src/ash/system/time/time_view.cc?rcl=3886244ecee4d53af5684b966f679ba9c8838de9&l=115

No need to SetEnabledColor for it anymore.

Bug: 982950
Change-Id: I2d867f9e55c47c42d1ced5da52895bbb6b774857
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1762698Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Min Chen <minch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#688737}
parent 0acad021
...@@ -39,9 +39,6 @@ namespace { ...@@ -39,9 +39,6 @@ namespace {
// when the timer goes off. // when the timer goes off.
const int kTimerSlopSeconds = 1; const int kTimerSlopSeconds = 1;
// Text color of the vertical clock minutes.
const SkColor kVerticalClockMinuteColor = SkColorSetRGB(0xBA, 0xBA, 0xBA);
// Padding between the left edge of the shelf and the left edge of the vertical // Padding between the left edge of the shelf and the left edge of the vertical
// clock. // clock.
const int kVerticalClockLeftPadding = 9; const int kVerticalClockLeftPadding = 9;
...@@ -223,9 +220,6 @@ void TimeView::SetupLabels() { ...@@ -223,9 +220,6 @@ void TimeView::SetupLabels() {
SetupLabel(vertical_label_hours_.get()); SetupLabel(vertical_label_hours_.get());
vertical_label_minutes_.reset(new views::Label()); vertical_label_minutes_.reset(new views::Label());
SetupLabel(vertical_label_minutes_.get()); SetupLabel(vertical_label_minutes_.get());
// TODO(estade): this should use the NativeTheme's secondary text color. See
// crbug.com/687791
vertical_label_minutes_->SetEnabledColor(kVerticalClockMinuteColor);
// Pull the minutes up closer to the hours by using a negative top border. // Pull the minutes up closer to the hours by using a negative top border.
vertical_label_minutes_->SetBorder( vertical_label_minutes_->SetBorder(
views::CreateEmptyBorder(kVerticalClockMinutesTopOffset, 0, 0, 0)); views::CreateEmptyBorder(kVerticalClockMinutesTopOffset, 0, 0, 0));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment