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

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

Issue 2789583004: Add a color argument to accessibilityPrivate.setFocusRing (Closed)
Patch Set: Rebase 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.cc
diff --git a/chrome/browser/chromeos/ui/accessibility_focus_ring_controller.cc b/chrome/browser/chromeos/ui/accessibility_focus_ring_controller.cc
index 62ac28a0beaadf7d660b3f505000b8aa97d61ec7..946e8652869d60b999222ec244a09bbf2f21cf69 100644
--- a/chrome/browser/chromeos/ui/accessibility_focus_ring_controller.cc
+++ b/chrome/browser/chromeos/ui/accessibility_focus_ring_controller.cc
@@ -80,6 +80,16 @@ AccessibilityFocusRingController::AccessibilityFocusRingController() {
AccessibilityFocusRingController::~AccessibilityFocusRingController() {
}
+void AccessibilityFocusRingController::SetFocusRingColor(SkColor color) {
+ focus_ring_color_ = color;
+ UpdateFocusRingsFromFocusRects();
+}
+
+void AccessibilityFocusRingController::ResetFocusRingColor() {
+ focus_ring_color_.reset();
+ UpdateFocusRingsFromFocusRects();
+}
+
void AccessibilityFocusRingController::SetFocusRing(
const std::vector<gfx::Rect>& rects,
AccessibilityFocusRingController::FocusRingBehavior focus_ring_behavior) {
@@ -118,6 +128,13 @@ void AccessibilityFocusRingController::UpdateFocusRingsFromFocusRects() {
for (size_t i = 0; i < focus_rings_.size(); ++i)
focus_layers_[i]->Set(focus_rings_[i]);
}
+
+ for (size_t i = 0; i < focus_rings_.size(); ++i) {
+ if (focus_ring_color_)
+ focus_layers_[i]->SetColor(*focus_ring_color_);
+ else
+ focus_layers_[i]->ResetColor();
+ }
}
void AccessibilityFocusRingController::OnLayerChange(

Powered by Google App Engine
This is Rietveld 408576698