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

Unified Diff: content/browser/frame_host/render_frame_host_impl.cc

Issue 2558933002: Add more fine-grained accessibility modes. (Closed)
Patch Set: Reformat enums as uppercase Created 4 years 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/frame_host/render_frame_host_impl.cc
diff --git a/content/browser/frame_host/render_frame_host_impl.cc b/content/browser/frame_host/render_frame_host_impl.cc
index c8d66dc67f8bb41762bb38acbeee63283063152a..564fd6961dd4b6c53cd1415747fb030fcbcff19c 100644
--- a/content/browser/frame_host/render_frame_host_impl.cc
+++ b/content/browser/frame_host/render_frame_host_impl.cc
@@ -16,7 +16,6 @@
#include "base/process/kill.h"
#include "base/time/time.h"
#include "build/build_config.h"
-#include "content/browser/accessibility/accessibility_mode_helper.h"
#include "content/browser/accessibility/ax_tree_id_registry.h"
#include "content/browser/accessibility/browser_accessibility_manager.h"
#include "content/browser/accessibility/browser_accessibility_state_impl.h"
@@ -1909,7 +1908,7 @@ void RenderFrameHostImpl::OnAccessibilityEvents(
AccessibilityMode accessibility_mode = delegate_->GetAccessibilityMode();
if ((accessibility_mode != AccessibilityModeOff) && view && is_active()) {
- if (accessibility_mode & AccessibilityModeFlagPlatform)
+ if (accessibility_mode & ACCESSIBILITY_MODE_FLAG_NATIVE_APIS)
GetOrCreateBrowserAccessibilityManager();
std::vector<AXEventNotificationDetails> details;
@@ -1936,12 +1935,11 @@ void RenderFrameHostImpl::OnAccessibilityEvents(
details.push_back(detail);
}
- if (accessibility_mode & AccessibilityModeFlagPlatform) {
+ if (accessibility_mode & ACCESSIBILITY_MODE_FLAG_NATIVE_APIS) {
if (browser_accessibility_manager_)
browser_accessibility_manager_->OnAccessibilityEvents(details);
}
- // Send the updates to the automation extension API.
delegate_->AccessibilityEventReceived(details);
// For testing only.
@@ -1982,7 +1980,7 @@ void RenderFrameHostImpl::OnAccessibilityLocationChanges(
render_view_host_->GetWidget()->GetView());
if (view && is_active()) {
AccessibilityMode accessibility_mode = delegate_->GetAccessibilityMode();
- if (accessibility_mode & AccessibilityModeFlagPlatform) {
+ if (accessibility_mode & ACCESSIBILITY_MODE_FLAG_NATIVE_APIS) {
BrowserAccessibilityManager* manager =
GetOrCreateBrowserAccessibilityManager();
if (manager)
@@ -2007,7 +2005,7 @@ void RenderFrameHostImpl::OnAccessibilityLocationChanges(
void RenderFrameHostImpl::OnAccessibilityFindInPageResult(
const AccessibilityHostMsg_FindInPageResultParams& params) {
AccessibilityMode accessibility_mode = delegate_->GetAccessibilityMode();
- if (accessibility_mode & AccessibilityModeFlagPlatform) {
+ if (accessibility_mode & ACCESSIBILITY_MODE_FLAG_NATIVE_APIS) {
BrowserAccessibilityManager* manager =
GetOrCreateBrowserAccessibilityManager();
if (manager) {
@@ -2881,7 +2879,7 @@ BrowserAccessibilityManager*
void RenderFrameHostImpl::ActivateFindInPageResultForAccessibility(
int request_id) {
AccessibilityMode accessibility_mode = delegate_->GetAccessibilityMode();
- if (accessibility_mode & AccessibilityModeFlagPlatform) {
+ if (accessibility_mode & ACCESSIBILITY_MODE_FLAG_NATIVE_APIS) {
BrowserAccessibilityManager* manager =
GetOrCreateBrowserAccessibilityManager();
if (manager)
« no previous file with comments | « content/browser/frame_host/render_frame_host_delegate.h ('k') | content/browser/renderer_host/render_widget_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698