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

Unified Diff: content/browser/accessibility/accessibility_ui.cc

Issue 14473011: Adds new experimental accessibility extension api to enable or disable native accessibility. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Address Dominic's comments. Created 7 years, 8 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/accessibility_ui.cc
diff --git a/content/browser/accessibility/accessibility_ui.cc b/content/browser/accessibility/accessibility_ui.cc
index 0208214c0fc428d77d0e3a27c0e3d4dcfe6eaa4f..dfc7343f9b8add32c4cfea19b925d171b0b82729 100644
--- a/content/browser/accessibility/accessibility_ui.cc
+++ b/content/browser/accessibility/accessibility_ui.cc
@@ -121,7 +121,7 @@ void SendTargetsData(
scoped_ptr<DictionaryValue> data(new DictionaryValue());
data->Set("list", rvh_list.release());
scoped_ptr<FundamentalValue> a11y_mode(new FundamentalValue(
- BrowserAccessibilityStateImpl::GetInstance()->GetAccessibilityMode()));
+ BrowserAccessibilityStateImpl::GetInstance()->accessibility_mode()));
data->Set("global_a11y_mode", a11y_mode.release());
std::string json_string;
@@ -205,32 +205,11 @@ void AccessibilityUI::ToggleAccessibility(const base::ListValue* args) {
void AccessibilityUI::ToggleGlobalAccessibility(const base::ListValue* args) {
BrowserAccessibilityStateImpl* state =
BrowserAccessibilityStateImpl::GetInstance();
- AccessibilityMode mode = state->GetAccessibilityMode();
+ AccessibilityMode mode = state->accessibility_mode();
AccessibilityMode new_mode = (mode == AccessibilityModeOff
? AccessibilityModeComplete
: AccessibilityModeOff);
state->SetAccessibilityMode(new_mode);
- for (RenderProcessHost::iterator it(RenderProcessHost::AllHostsIterator());
- !it.IsAtEnd(); it.Advance()) {
- RenderProcessHost* render_process_host = it.GetCurrentValue();
- DCHECK(render_process_host);
-
- // Ignore processes that don't have a connection, such as crashed tabs.
- if (!render_process_host->HasConnection())
- continue;
-
- RenderProcessHost::RenderWidgetHostsIterator rwit(
- render_process_host->GetRenderWidgetHostsIterator());
- for (; !rwit.IsAtEnd(); rwit.Advance()) {
- RenderWidgetHost* rwh = const_cast<RenderWidgetHost*>(
- rwit.GetCurrentValue());
- DCHECK(rwh);
- if (!rwh || !rwh->IsRenderView())
- continue;
- RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(rwh);
- rwhi->SetAccessibilityMode(new_mode);
- }
- }
}
void AccessibilityUI::RequestAccessibilityTree(const base::ListValue* args) {

Powered by Google App Engine
This is Rietveld 408576698