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

Unified Diff: chrome/browser/chromeos/ui/accessibility_focus_ring_controller.h

Issue 2789583004: Add a color argument to accessibilityPrivate.setFocusRing (Closed)
Patch Set: Created 3 years, 9 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
Index: chrome/browser/chromeos/ui/accessibility_focus_ring_controller.h
diff --git a/chrome/browser/chromeos/ui/accessibility_focus_ring_controller.h b/chrome/browser/chromeos/ui/accessibility_focus_ring_controller.h
index 6eb71726fac4fb9caeb89c6bcd17257c282e0b29..a46e02954fda52af894f9f9ccd8389787cf0dfcd 100644
--- a/chrome/browser/chromeos/ui/accessibility_focus_ring_controller.h
+++ b/chrome/browser/chromeos/ui/accessibility_focus_ring_controller.h
@@ -27,6 +27,10 @@ class AccessibilityFocusRingController : public FocusRingLayerDelegate {
enum FocusRingBehavior { FADE_OUT_FOCUS_RING, PERSIST_FOCUS_RING };
+ // Set the focus ring color, or reset it back to the default.
+ void SetFocusRingColor(SkColor color);
xiyuan 2017/03/30 21:27:38 nit: #include "third_party/skia/include/core/SkCol
dmazzoni 2017/03/30 21:34:52 Done.
+ void ResetFocusRingColor();
+
// Draw a focus ring around the given set of rects, in global screen
// coordinates. Use |focus_ring_behavior| to specify whether the focus
// ring should persist or fade out.
@@ -100,6 +104,8 @@ class AccessibilityFocusRingController : public FocusRingLayerDelegate {
std::vector<AccessibilityFocusRing> focus_rings_;
std::vector<std::unique_ptr<AccessibilityFocusRingLayer>> focus_layers_;
FocusRingBehavior focus_ring_behavior_ = FADE_OUT_FOCUS_RING;
+ bool has_focus_ring_color_ = false;
+ SkColor focus_ring_color_ = SK_ColorBLACK;
LayerAnimationInfo cursor_animation_info_;
gfx::Point cursor_location_;

Powered by Google App Engine
This is Rietveld 408576698