Switch AccessibilityMode to be a bitmap

content/public/browser/web_contents.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
 
 #include <set>
 
 #include "base/basictypes.h"
@@ skipping 222 matching lines @@
 
   // Allows overriding the user agent used for NavigationEntries it owns.
   virtual void SetUserAgentOverride(const std::string& override) = 0;
   virtual const std::string& GetUserAgentOverride() const = 0;
 
 #if defined(OS_WIN)
   virtual void SetParentNativeViewAccessible(
       gfx::NativeViewAccessible accessible_parent) = 0;
 #endif
 
+  virtual void SetRendererAccessibilityMode(bool on) = 0;

[David Tseng, 2014/01/27 22:16:06]

Why not expose this method inside of BrowserAccessibilityState?

[aboxhall, 2014/01/28 00:18:42]

I guess we could, although obviously that wouldn't replace this (since this
relates to a single WebContents). Should I do that?

[David Tseng, 2014/01/28 00:48:45]

I think we'll need both. BrowserAccessibilityState flips on everything.
I also think we'll need the generic version (SetAccessibilityMode).
Might be cleaner to just have SetAccessibilityMode.

[aboxhall, 2014/01/28 17:48:26]

On second thoughts, I might take this out for now - we will need something like
it eventually but it doesn't belong in this change. And I agree with you on just
having SetAccessibilityMode.

+
   // Tab navigation state ------------------------------------------------------
 
   // Returns the current navigation properties, which if a navigation is
   // pending may be provisional (e.g., the navigation could result in a
   // download, in which case the URL would revert to what it was previously).
   virtual const base::string16& GetTitle() const = 0;
 
   // The max page ID for any page that the current SiteInstance has loaded in
   // this WebContents.  Page IDs are specific to a given SiteInstance and
   // WebContents, corresponding to a specific RenderView in the renderer.
@@ skipping 255 matching lines @@
 
  private:
   // This interface should only be implemented inside content.
   friend class WebContentsImpl;
   WebContents() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_