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

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

Issue 273423004: Migrate accessibility from RenderView to RenderFrame. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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 1f4df798f46b6224b0e9327559e17c65a625657c..83e330445724eb46d39f33f5ede09e4a4acdf2bd 100644
--- a/content/browser/accessibility/accessibility_ui.cc
+++ b/content/browser/accessibility/accessibility_ui.cc
@@ -15,6 +15,7 @@
#include "content/browser/accessibility/browser_accessibility_state_impl.h"
#include "content/browser/renderer_host/render_widget_host_impl.h"
#include "content/browser/renderer_host/render_widget_host_view_base.h"
+#include "content/browser/web_contents/web_contents_impl.h"
#include "content/common/view_message_enums.h"
#include "content/public/browser/favicon_status.h"
#include "content/public/browser/navigation_entry.h"
@@ -63,11 +64,11 @@ base::DictionaryValue* BuildTargetDescriptor(
}
base::DictionaryValue* BuildTargetDescriptor(RenderViewHost* rvh) {
- WebContents* web_contents = WebContents::FromRenderViewHost(rvh);
- std::string title;
- RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(rvh);
- AccessibilityMode accessibility_mode = rwhi->accessibility_mode();
+ WebContentsImpl* web_contents = static_cast<WebContentsImpl*>(
+ WebContents::FromRenderViewHost(rvh));
+ AccessibilityMode accessibility_mode = web_contents->accessibility_mode();
+ std::string title;
GURL url;
GURL favicon_url;
if (web_contents) {
@@ -178,14 +179,15 @@ void AccessibilityUI::ToggleAccessibility(const base::ListValue* args) {
RenderViewHost* rvh = RenderViewHost::FromID(process_id, route_id);
if (!rvh)
return;
- RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(rvh);
- if (!rwhi)
- return;
- AccessibilityMode mode = rwhi->accessibility_mode();
- if ((mode & AccessibilityModeComplete) != AccessibilityModeComplete)
- rwhi->AddAccessibilityMode(AccessibilityModeComplete);
- else
- rwhi->ResetAccessibilityMode();
+ WebContentsImpl* web_contents = static_cast<WebContentsImpl*>(
+ WebContents::FromRenderViewHost(rvh));
+ AccessibilityMode mode = web_contents->accessibility_mode();
+ if ((mode & AccessibilityModeComplete) != AccessibilityModeComplete) {
+ web_contents->AddAccessibilityMode(AccessibilityModeComplete);
+ } else {
+ web_contents->SetAccessibilityMode(
+ BrowserAccessibilityStateImpl::GetInstance()->accessibility_mode());
+ }
}
void AccessibilityUI::ToggleGlobalAccessibility(const base::ListValue* args) {

Powered by Google App Engine
This is Rietveld 408576698