Chrome OS highlighting: show only focus or caret, never both.

chrome/browser/chromeos/accessibility/accessibility_highlight_manager.cc

 // Copyright (c) 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/shell.h"
 #include "chrome/browser/chromeos/accessibility/accessibility_highlight_manager.h"
 #include "chrome/browser/chromeos/ui/accessibility_focus_ring_controller.h"
 #include "content/public/browser/focused_node_details.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_types.h"
@@ skipping 50 matching lines @@
         OffscreenPoint());
 
     aura::Window* root_window = shell->GetPrimaryRootWindow();
     ui::InputMethod* input_method = GetInputMethod(root_window);
     input_method->RemoveObserver(this);
   }
 }
 
 void AccessibilityHighlightManager::HighlightFocus(bool focus) {
   focus_ = focus;
-
-  std::vector<gfx::Rect> rects;
-  rects.push_back(focus_ ? focus_rect_ : OffscreenRect());
-  AccessibilityFocusRingController::GetInstance()->SetFocusRing(rects);
+  UpdateFocusAndCaretHighlights();
 }
 
 void AccessibilityHighlightManager::HighlightCursor(bool cursor) {
   cursor_ = cursor;
 
   AccessibilityFocusRingController::GetInstance()->SetCursorRing(
       cursor_ ? cursor_point_ : OffscreenPoint());
 }
 
 void AccessibilityHighlightManager::HighlightCaret(bool caret) {
   caret_ = caret;
-
-  AccessibilityFocusRingController::GetInstance()->SetCaretRing(
-      caret_ ? caret_point_ : OffscreenPoint());
+  UpdateFocusAndCaretHighlights();
 }
 
 void AccessibilityHighlightManager::OnMouseEvent(ui::MouseEvent* event) {
   if (event->type() == ui::ET_MOUSE_MOVED) {
     cursor_point_ = event->root_location();
     AccessibilityFocusRingController::GetInstance()->SetCursorRing(
         cursor_ ? cursor_point_ : OffscreenPoint());
   }
 }
 
 void AccessibilityHighlightManager::Observe(
     int type,
     const content::NotificationSource& source,
     const content::NotificationDetails& details) {
   DCHECK_EQ(type, content::NOTIFICATION_FOCUS_CHANGED_IN_PAGE);
   content::FocusedNodeDetails* node_details =
       content::Details<content::FocusedNodeDetails>(details).ptr();
   focus_rect_ = node_details->node_bounds_in_screen;
-
-  if (focus_) {
-    std::vector<gfx::Rect> rects;
-    rects.push_back(focus_rect_);
-    AccessibilityFocusRingController::GetInstance()->SetFocusRing(rects);
-  }
+  UpdateFocusAndCaretHighlights();
 }
 
 void AccessibilityHighlightManager::OnTextInputStateChanged(
     const ui::TextInputClient* client) {
   if (!client || client->GetTextInputType() == ui::TEXT_INPUT_TYPE_NONE) {
-    caret_point_ = OffscreenPoint();
-    if (caret_) {
-      AccessibilityFocusRingController::GetInstance()->SetCaretRing(
-          caret_point_);
-    }
+    caret_visible_ = false;
+    UpdateFocusAndCaretHighlights();
   }
 }
 
 void AccessibilityHighlightManager::OnCaretBoundsChanged(
     const ui::TextInputClient* client) {
   gfx::Rect caret_bounds = client->GetCaretBounds();
-  if (caret_bounds.width() == 0 && caret_bounds.height() == 0)
-    caret_bounds = OffscreenRect();
   caret_point_ = caret_bounds.CenterPoint();
+  caret_visible_ = client->GetTextInputType() != ui::TEXT_INPUT_TYPE_NONE &&
+                   (caret_bounds.width() || caret_bounds.height());
+  UpdateFocusAndCaretHighlights();
+}
 
-  if (caret_)
-    AccessibilityFocusRingController::GetInstance()->SetCaretRing(caret_point_);
+void AccessibilityHighlightManager::UpdateFocusAndCaretHighlights() {
+  auto controller = AccessibilityFocusRingController::GetInstance();
+
+  // The caret highlight takes precedence over the focus highlight if
+  // both are visible.
+  if (caret_ && caret_visible_) {
+    controller->SetCaretRing(caret_point_);
+    controller->SetFocusRing(std::vector<gfx::Rect>());
+  } else if (focus_) {
+    controller->SetCaretRing(OffscreenPoint());
+    std::vector<gfx::Rect> rects;
+    if (!focus_rect_.IsEmpty())
+      rects.push_back(focus_rect_);
+    controller->SetFocusRing(rects);
+  } else {
+    controller->SetCaretRing(OffscreenPoint());
+    controller->SetFocusRing(std::vector<gfx::Rect>());
+  }
 }
 
 }  // namespace chromeos