Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(909)

Unified Diff: third_party/WebKit/Source/core/paint/MediaControlsPainter.cpp

Issue 2314403004: MediaControls Accessibility: update sliders grey color to a darker shade. (Closed)
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/WebKit/LayoutTests/platform/mac-mac10.9/media/video-zoom-controls-expected.png ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/paint/MediaControlsPainter.cpp
diff --git a/third_party/WebKit/Source/core/paint/MediaControlsPainter.cpp b/third_party/WebKit/Source/core/paint/MediaControlsPainter.cpp
index 53a1c8f774183999f1a1fb01c2aca72400796468..6646cbdf8fe60fc47447f5ee046c12c33db693d9 100644
--- a/third_party/WebKit/Source/core/paint/MediaControlsPainter.cpp
+++ b/third_party/WebKit/Source/core/paint/MediaControlsPainter.cpp
@@ -375,7 +375,7 @@ void MediaControlsPainter::paintMediaSliderInternal(const LayoutObject& object,
startColor = Color(60, 60, 60);
endColor = Color(76, 76, 76);
} else {
- startColor = endColor = Color(0x9f, 0x9f, 0x9f); // light grey.
+ startColor = endColor = Color(0x5a, 0x5a, 0x5a); // dark grey
}
if (endPosition > currentPosition)
@@ -441,7 +441,7 @@ bool MediaControlsPainter::paintMediaVolumeSlider(const LayoutObject& object, co
if (!RuntimeEnabledFeatures::newMediaPlaybackUiEnabled())
sliderBackgroundColor = Color(11, 11, 11);
else
- sliderBackgroundColor = Color(0x9f, 0x9f, 0x9f);
+ sliderBackgroundColor = Color(0x5a, 0x5a, 0x5a); // dark grey
paintRoundedSliderBackground(rect, style, context, sliderBackgroundColor);
// Calculate volume position for white background rectangle.
« no previous file with comments | « third_party/WebKit/LayoutTests/platform/mac-mac10.9/media/video-zoom-controls-expected.png ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698