Switch AccessibilityMode to be a bitmap

content/browser/accessibility/browser_accessibility_state_impl.cc

 // Copyright (c) 2012 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 "content/browser/accessibility/browser_accessibility_state_impl.h"
 
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
 #include "base/timer/timer.h"
+#include "content/browser/accessibility/accessibility_mode_helper.h"
 #include "content/browser/renderer_host/render_widget_host_impl.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/render_widget_host_iterator.h"
 #include "content/public/common/content_switches.h"
 #include "ui/gfx/sys_color_change_listener.h"
 
 #if defined(OS_WIN)
 #include "base/win/windows_version.h"
 #endif
@@ skipping 53 matching lines @@
 }
 
 BrowserAccessibilityStateImpl::~BrowserAccessibilityStateImpl() {
 }
 
 void BrowserAccessibilityStateImpl::OnScreenReaderDetected() {
   if (CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kDisableRendererAccessibility)) {
     return;
   }
-  SetAccessibilityMode(AccessibilityModeComplete);
+  AddAccessibilityMode(AccessibilityModeComplete);
 }
 
 void BrowserAccessibilityStateImpl::EnableAccessibility() {
   // We may want to do something different with this later.
-  SetAccessibilityMode(AccessibilityModeComplete);
+  AddAccessibilityMode(AccessibilityModeComplete);
 }
 
 void BrowserAccessibilityStateImpl::DisableAccessibility() {
-  SetAccessibilityMode(AccessibilityModeOff);
+  RemoveAccessibilityMode(AccessibilityModeAll);
 }
 
 bool BrowserAccessibilityStateImpl::IsAccessibleBrowser() {
-  return (accessibility_mode_ == AccessibilityModeComplete);
+  return ((accessibility_mode_ & AccessibilityModeComplete) ==
+          AccessibilityModeComplete);
 }
 
 void BrowserAccessibilityStateImpl::AddHistogramCallback(
     base::Closure callback) {
   histogram_callbacks_.push_back(callback);
 }
 
 void BrowserAccessibilityStateImpl::UpdateHistogramsForTesting() {
   UpdateHistograms();
 }
@@ skipping 10 matching lines @@
   UMA_HISTOGRAM_BOOLEAN("Accessibility.ManuallyEnabled",
                         CommandLine::ForCurrentProcess()->HasSwitch(
                             switches::kForceRendererAccessibility));
 }
 
 #if !defined(OS_WIN)
 void BrowserAccessibilityStateImpl::UpdatePlatformSpecificHistograms() {
 }
 #endif
 
-void BrowserAccessibilityStateImpl::SetAccessibilityMode(
+void BrowserAccessibilityStateImpl::AddAccessibilityMode(
     AccessibilityMode mode) {
-  if (accessibility_mode_ == mode)
+  if ((accessibility_mode_ & mode) == mode)
     return;
-  accessibility_mode_ = mode;
 
+  accessibility_mode_ =
+      content::AddAccessibilityMode(accessibility_mode_, mode);

[David Tseng, 2014/02/04 20:25:51]

By setting text only, then complete, I would end up adding both text only and
complete, which is a functional change from the current code. (see my comments
about enforcement on add).

[aboxhall, 2014/02/05 16:02:47]

Complete + text only should be considered to be text only, IMO.

+
+  AddOrRemoveFromRenderWidgets(mode, true);
+}
+
+void BrowserAccessibilityStateImpl::RemoveAccessibilityMode(
+    AccessibilityMode mode) {
+  if ((accessibility_mode_ ^ mode) == mode)
+    return;
+
+  accessibility_mode_ =
+      content::RemoveAccessibilityMode(accessibility_mode_, mode);
+
+  AddOrRemoveFromRenderWidgets(mode, false);
+}
+
+void BrowserAccessibilityStateImpl::AddOrRemoveFromRenderWidgets(
+    AccessibilityMode mode,
+    bool add) {
   // Iterate over all RenderWidgetHosts, even swapped out ones in case
   // they become active again.
   scoped_ptr<RenderWidgetHostIterator> widgets(
       RenderWidgetHostImpl::GetAllRenderWidgetHosts());
   while (RenderWidgetHost* widget = widgets->GetNextHost()) {
     // Ignore processes that don't have a connection, such as crashed tabs.
     if (!widget->GetProcess()->HasConnection())
       continue;
     if (!widget->IsRenderView())
       continue;
 
-    RenderWidgetHostImpl::From(widget)->SetAccessibilityMode(mode);
+    RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(widget);
+    if (add)
+      rwhi->AddAccessibilityMode(mode);
+    else
+      rwhi->RemoveAccessibilityMode(mode);
   }
 }
 
 }  // namespace content