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

Side by Side Diff: content/browser/accessibility/browser_accessibility_state_impl.cc

Issue 2694413006: Scope and clean up uses of AccessibilityMode. (Closed)
Patch Set: #6 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/accessibility/browser_accessibility_state_impl.h" 5 #include "content/browser/accessibility/browser_accessibility_state_impl.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/metrics/histogram_macros.h" 10 #include "base/metrics/histogram_macros.h"
11 #include "build/build_config.h" 11 #include "build/build_config.h"
12 #include "content/browser/renderer_host/render_widget_host_impl.h" 12 #include "content/browser/renderer_host/render_widget_host_impl.h"
13 #include "content/browser/web_contents/web_contents_impl.h" 13 #include "content/browser/web_contents/web_contents_impl.h"
14 #include "content/public/browser/browser_thread.h" 14 #include "content/public/browser/browser_thread.h"
15 #include "content/public/common/content_switches.h" 15 #include "content/public/common/content_switches.h"
16 #include "ui/gfx/color_utils.h" 16 #include "ui/gfx/color_utils.h"
17 17
18 namespace content { 18 namespace content {
19 19
20 // IMPORTANT! 20 // IMPORTANT!
21 // These values are written to logs. Do not renumber or delete 21 // These values are written to logs. Do not renumber or delete
22 // existing items; add new entries to the end of the list. 22 // existing items; add new entries to the end of the list.
23 enum ModeFlagHistogramValue { 23 enum ModeFlagHistogramValue {
24 UMA_AX_MODE_FLAG_NATIVE_APIS = 0, 24 UMA_AX_MODE_NATIVE_APIS = 0,
25 UMA_AX_MODE_FLAG_WEB_CONTENTS = 1, 25 UMA_AX_MODE_WEB_CONTENTS = 1,
26 UMA_AX_MODE_FLAG_INLINE_TEXT_BOXES = 2, 26 UMA_AX_MODE_INLINE_TEXT_BOXES = 2,
27 UMA_AX_MODE_FLAG_SCREEN_READER = 3, 27 UMA_AX_MODE_SCREEN_READER = 3,
28 UMA_AX_MODE_FLAG_HTML = 4, 28 UMA_AX_MODE_HTML = 4,
29 29
30 // This must always be the last enum. It's okay for its value to 30 // This must always be the last enum. It's okay for its value to
31 // increase, but none of the other enum values may change. 31 // increase, but none of the other enum values may change.
32 UMA_AX_MODE_FLAG_MAX 32 UMA_AX_MODE_MAX
33 }; 33 };
34 34
35 // Record a histograms for an accessibility mode when it's enabled. 35 // Record a histograms for an accessibility mode when it's enabled.
36 void RecordNewAccessibilityModeFlags(ModeFlagHistogramValue mode_flag) { 36 void RecordNewAccessibilityModeFlags(ModeFlagHistogramValue mode_flag) {
37 UMA_HISTOGRAM_ENUMERATION("Accessibility.ModeFlag", 37 UMA_HISTOGRAM_ENUMERATION("Accessibility.ModeFlag", mode_flag,
38 mode_flag, 38 UMA_AX_MODE_MAX);
39 UMA_AX_MODE_FLAG_MAX);
40 } 39 }
41 40
42 // Update the accessibility histogram 45 seconds after initialization. 41 // Update the accessibility histogram 45 seconds after initialization.
43 static const int ACCESSIBILITY_HISTOGRAM_DELAY_SECS = 45; 42 static const int ACCESSIBILITY_HISTOGRAM_DELAY_SECS = 45;
44 43
45 // static 44 // static
46 BrowserAccessibilityState* BrowserAccessibilityState::GetInstance() { 45 BrowserAccessibilityState* BrowserAccessibilityState::GetInstance() {
47 return BrowserAccessibilityStateImpl::GetInstance(); 46 return BrowserAccessibilityStateImpl::GetInstance();
48 } 47 }
49 48
50 // static 49 // static
51 BrowserAccessibilityStateImpl* BrowserAccessibilityStateImpl::GetInstance() { 50 BrowserAccessibilityStateImpl* BrowserAccessibilityStateImpl::GetInstance() {
52 return base::Singleton< 51 return base::Singleton<
53 BrowserAccessibilityStateImpl, 52 BrowserAccessibilityStateImpl,
54 base::LeakySingletonTraits<BrowserAccessibilityStateImpl>>::get(); 53 base::LeakySingletonTraits<BrowserAccessibilityStateImpl>>::get();
55 } 54 }
56 55
57 BrowserAccessibilityStateImpl::BrowserAccessibilityStateImpl() 56 BrowserAccessibilityStateImpl::BrowserAccessibilityStateImpl()
58 : BrowserAccessibilityState(), 57 : BrowserAccessibilityState(),
59 accessibility_mode_(AccessibilityModeOff), 58 accessibility_mode_(AccessibilityMode::kOff),
60 disable_hot_tracking_(false) { 59 disable_hot_tracking_(false) {
61 ResetAccessibilityModeValue(); 60 ResetAccessibilityModeValue();
62 #if defined(OS_WIN) 61 #if defined(OS_WIN)
63 // On Windows, UpdateHistograms calls some system functions with unknown 62 // On Windows, UpdateHistograms calls some system functions with unknown
64 // runtime, so call it on the file thread to ensure there's no jank. 63 // runtime, so call it on the file thread to ensure there's no jank.
65 // Everything in that method must be safe to call on another thread. 64 // Everything in that method must be safe to call on another thread.
66 BrowserThread::ID update_histogram_thread = BrowserThread::FILE; 65 BrowserThread::ID update_histogram_thread = BrowserThread::FILE;
67 #else 66 #else
68 // On all other platforms, UpdateHistograms should be called on the main 67 // On all other platforms, UpdateHistograms should be called on the main
69 // thread. 68 // thread.
(...skipping 14 matching lines...) Expand all
84 83
85 void BrowserAccessibilityStateImpl::OnScreenReaderDetected() { 84 void BrowserAccessibilityStateImpl::OnScreenReaderDetected() {
86 if (base::CommandLine::ForCurrentProcess()->HasSwitch( 85 if (base::CommandLine::ForCurrentProcess()->HasSwitch(
87 switches::kDisableRendererAccessibility)) { 86 switches::kDisableRendererAccessibility)) {
88 return; 87 return;
89 } 88 }
90 EnableAccessibility(); 89 EnableAccessibility();
91 } 90 }
92 91
93 void BrowserAccessibilityStateImpl::EnableAccessibility() { 92 void BrowserAccessibilityStateImpl::EnableAccessibility() {
94 AddAccessibilityModeFlags(ACCESSIBILITY_MODE_COMPLETE); 93 AccessibilityMode complete_mode(
94 AccessibilityMode::kNativeAPIs | AccessibilityMode::kWebContents |
95 AccessibilityMode::kInlineTextBoxes | AccessibilityMode::kScreenReader |
96 AccessibilityMode::kHTML);
97 AddAccessibilityModeFlags(complete_mode);
95 } 98 }
96 99
97 void BrowserAccessibilityStateImpl::DisableAccessibility() { 100 void BrowserAccessibilityStateImpl::DisableAccessibility() {
98 ResetAccessibilityMode(); 101 ResetAccessibilityMode();
99 } 102 }
100 103
101 void BrowserAccessibilityStateImpl::ResetAccessibilityModeValue() { 104 void BrowserAccessibilityStateImpl::ResetAccessibilityModeValue() {
102 accessibility_mode_ = AccessibilityModeOff; 105 accessibility_mode_ = AccessibilityMode::kOff;
103 if (base::CommandLine::ForCurrentProcess()->HasSwitch( 106 if (base::CommandLine::ForCurrentProcess()->HasSwitch(
104 switches::kForceRendererAccessibility)) { 107 switches::kForceRendererAccessibility)) {
105 accessibility_mode_ = ACCESSIBILITY_MODE_COMPLETE; 108 accessibility_mode_.set_mode(
109 AccessibilityMode::kNativeAPIs | AccessibilityMode::kWebContents |
110 AccessibilityMode::kInlineTextBoxes |
111 AccessibilityMode::kScreenReader | AccessibilityMode::kHTML,
112 true);
106 } 113 }
107 } 114 }
108 115
109 void BrowserAccessibilityStateImpl::ResetAccessibilityMode() { 116 void BrowserAccessibilityStateImpl::ResetAccessibilityMode() {
110 ResetAccessibilityModeValue(); 117 ResetAccessibilityModeValue();
111 118
112 std::vector<WebContentsImpl*> web_contents_vector = 119 std::vector<WebContentsImpl*> web_contents_vector =
113 WebContentsImpl::GetAllWebContents(); 120 WebContentsImpl::GetAllWebContents();
114 for (size_t i = 0; i < web_contents_vector.size(); ++i) 121 for (size_t i = 0; i < web_contents_vector.size(); ++i)
115 web_contents_vector[i]->SetAccessibilityMode(accessibility_mode()); 122 web_contents_vector[i]->SetAccessibilityMode(accessibility_mode());
116 } 123 }
117 124
118 bool BrowserAccessibilityStateImpl::IsAccessibleBrowser() { 125 bool BrowserAccessibilityStateImpl::IsAccessibleBrowser() {
119 return ((accessibility_mode_ & ACCESSIBILITY_MODE_COMPLETE) == 126 return accessibility_mode_.has_mode(
120 ACCESSIBILITY_MODE_COMPLETE); 127 AccessibilityMode::kNativeAPIs | AccessibilityMode::kWebContents |
128 AccessibilityMode::kInlineTextBoxes | AccessibilityMode::kScreenReader |
129 AccessibilityMode::kHTML);
121 } 130 }
122 131
123 void BrowserAccessibilityStateImpl::AddHistogramCallback( 132 void BrowserAccessibilityStateImpl::AddHistogramCallback(
124 base::Closure callback) { 133 base::Closure callback) {
125 histogram_callbacks_.push_back(callback); 134 histogram_callbacks_.push_back(callback);
126 } 135 }
127 136
128 void BrowserAccessibilityStateImpl::UpdateHistogramsForTesting() { 137 void BrowserAccessibilityStateImpl::UpdateHistogramsForTesting() {
129 UpdateHistograms(); 138 UpdateHistograms();
130 } 139 }
(...skipping 26 matching lines...) Expand all
157 switches::kDisableRendererAccessibility)) { 166 switches::kDisableRendererAccessibility)) {
158 return; 167 return;
159 } 168 }
160 169
161 AccessibilityMode previous_mode = accessibility_mode_; 170 AccessibilityMode previous_mode = accessibility_mode_;
162 accessibility_mode_ |= mode; 171 accessibility_mode_ |= mode;
163 if (accessibility_mode_ == previous_mode) 172 if (accessibility_mode_ == previous_mode)
164 return; 173 return;
165 174
166 // Retrieve only newly added modes for the purposes of logging. 175 // Retrieve only newly added modes for the purposes of logging.
167 AccessibilityMode new_mode_flags = accessibility_mode_ & (~previous_mode); 176 int new_mode_flags = mode.get_mode() & (~previous_mode.get_mode());
168 if (new_mode_flags & ACCESSIBILITY_MODE_FLAG_NATIVE_APIS) 177 if (new_mode_flags & AccessibilityMode::kNativeAPIs)
169 RecordNewAccessibilityModeFlags(UMA_AX_MODE_FLAG_NATIVE_APIS); 178 RecordNewAccessibilityModeFlags(UMA_AX_MODE_NATIVE_APIS);
170 if (new_mode_flags & ACCESSIBILITY_MODE_FLAG_WEB_CONTENTS) 179 if (new_mode_flags & AccessibilityMode::kWebContents)
171 RecordNewAccessibilityModeFlags(UMA_AX_MODE_FLAG_WEB_CONTENTS); 180 RecordNewAccessibilityModeFlags(UMA_AX_MODE_WEB_CONTENTS);
172 if (new_mode_flags & ACCESSIBILITY_MODE_FLAG_INLINE_TEXT_BOXES) 181 if (new_mode_flags & AccessibilityMode::kInlineTextBoxes)
173 RecordNewAccessibilityModeFlags(UMA_AX_MODE_FLAG_INLINE_TEXT_BOXES); 182 RecordNewAccessibilityModeFlags(UMA_AX_MODE_INLINE_TEXT_BOXES);
174 if (new_mode_flags & ACCESSIBILITY_MODE_FLAG_SCREEN_READER) 183 if (new_mode_flags & AccessibilityMode::kScreenReader)
175 RecordNewAccessibilityModeFlags(UMA_AX_MODE_FLAG_SCREEN_READER); 184 RecordNewAccessibilityModeFlags(UMA_AX_MODE_SCREEN_READER);
176 if (new_mode_flags & ACCESSIBILITY_MODE_FLAG_HTML) 185 if (new_mode_flags & AccessibilityMode::kHTML)
177 RecordNewAccessibilityModeFlags(UMA_AX_MODE_FLAG_HTML); 186 RecordNewAccessibilityModeFlags(UMA_AX_MODE_HTML);
178 187
179 std::vector<WebContentsImpl*> web_contents_vector = 188 std::vector<WebContentsImpl*> web_contents_vector =
180 WebContentsImpl::GetAllWebContents(); 189 WebContentsImpl::GetAllWebContents();
181 for (size_t i = 0; i < web_contents_vector.size(); ++i) 190 for (size_t i = 0; i < web_contents_vector.size(); ++i)
182 web_contents_vector[i]->AddAccessibilityMode(accessibility_mode_); 191 web_contents_vector[i]->AddAccessibilityMode(accessibility_mode_);
183 } 192 }
184 193
185 void BrowserAccessibilityStateImpl::RemoveAccessibilityModeFlags( 194 void BrowserAccessibilityStateImpl::RemoveAccessibilityModeFlags(
186 AccessibilityMode mode) { 195 AccessibilityMode mode) {
187 if (base::CommandLine::ForCurrentProcess()->HasSwitch( 196 if (base::CommandLine::ForCurrentProcess()->HasSwitch(
188 switches::kForceRendererAccessibility) && 197 switches::kForceRendererAccessibility) &&
189 mode == ACCESSIBILITY_MODE_COMPLETE) { 198 mode.has_mode(
199 AccessibilityMode::kNativeAPIs | AccessibilityMode::kWebContents |
200 AccessibilityMode::kInlineTextBoxes |
201 AccessibilityMode::kScreenReader | AccessibilityMode::kHTML)) {
190 return; 202 return;
191 } 203 }
192 204
193 accessibility_mode_ = accessibility_mode_ ^ (mode & accessibility_mode_); 205 int raw_flags = accessibility_mode_.get_mode() ^
206 (mode.get_mode() & accessibility_mode_.get_mode());
207 accessibility_mode_ = raw_flags;
208
194 std::vector<WebContentsImpl*> web_contents_vector = 209 std::vector<WebContentsImpl*> web_contents_vector =
195 WebContentsImpl::GetAllWebContents(); 210 WebContentsImpl::GetAllWebContents();
196 for (size_t i = 0; i < web_contents_vector.size(); ++i) 211 for (size_t i = 0; i < web_contents_vector.size(); ++i)
197 web_contents_vector[i]->SetAccessibilityMode(accessibility_mode()); 212 web_contents_vector[i]->SetAccessibilityMode(accessibility_mode());
198 } 213 }
199 214
200 } // namespace content 215 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698