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

Unified Diff: content/browser/accessibility/browser_accessibility_state_impl.h

Issue 145283003: Switch AccessibilityMode to be a bitmap (Closed) Base URL: https://chromium.googlesource.com/chromium/src@enable
Patch Set: Removed AccessibilityModeFlagRenderer, etc. Created 6 years, 10 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: content/browser/accessibility/browser_accessibility_state_impl.h
diff --git a/content/browser/accessibility/browser_accessibility_state_impl.h b/content/browser/accessibility/browser_accessibility_state_impl.h
index 3098341ef25efeb46fef8762a7e1d3512e5fad86..f6cafa0ca87544ccd4a5c8c2881a25bb71d89ddf 100644
--- a/content/browser/accessibility/browser_accessibility_state_impl.h
+++ b/content/browser/accessibility/browser_accessibility_state_impl.h
@@ -42,14 +42,23 @@ class CONTENT_EXPORT BrowserAccessibilityStateImpl
virtual void EnableAccessibility() OVERRIDE;
virtual void DisableAccessibility() OVERRIDE;
+ virtual void ResetAccessibilityMode() OVERRIDE;
virtual void OnScreenReaderDetected() OVERRIDE;
virtual bool IsAccessibleBrowser() OVERRIDE;
virtual void AddHistogramCallback(base::Closure callback) OVERRIDE;
virtual void UpdateHistogramsForTesting() OVERRIDE;
- AccessibilityMode accessibility_mode() const { return accessibility_mode_; };
- void SetAccessibilityMode(AccessibilityMode mode);
+ unsigned int accessibility_mode() const { return accessibility_mode_; };
+
+ // Adds the given accessibility mode to the current accessibility mode bitmap.
+ void AddAccessibilityMode(AccessibilityMode mode);
+
+ // Removes the given accessibility mode from the current accessibility mode
+ // bitmap, managing the bits that are shared with other modes such that a
+ // bit will only be turned off when all modes that depend on it have been
+ // removed.
+ void RemoveAccessibilityMode(AccessibilityMode mode);
private:
friend class base::RefCountedThreadSafe<BrowserAccessibilityStateImpl>;
@@ -64,7 +73,11 @@ class CONTENT_EXPORT BrowserAccessibilityStateImpl
void UpdatePlatformSpecificHistograms();
- AccessibilityMode accessibility_mode_;
+ // Updates the accessibility mode of all render widgets, including swapped out
+ // ones. |add| specifies whether the mode should be added or removed.
+ void AddOrRemoveFromRenderWidgets(AccessibilityMode mode, bool add);
+
+ unsigned int accessibility_mode_;
std::vector<base::Closure> histogram_callbacks_;

Powered by Google App Engine
This is Rietveld 408576698