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

Side by Side Diff: chrome/browser/extensions/api/automation_internal/automation_internal_api.cc

Issue 595413002: Draw an accessibility focus ring around the focused object in ChromeVox Next. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix automation browser test Created 6 years, 2 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "chrome/browser/extensions/api/automation_internal/automation_internal_ api.h" 5 #include "chrome/browser/extensions/api/automation_internal/automation_internal_ api.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "base/strings/string_number_conversions.h" 9 #include "base/strings/string_number_conversions.h"
10 #include "chrome/browser/extensions/api/automation_internal/automation_action_ad apter.h" 10 #include "chrome/browser/extensions/api/automation_internal/automation_action_ad apter.h"
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
69 : public content::WebContentsObserver, 69 : public content::WebContentsObserver,
70 public content::WebContentsUserData<AutomationWebContentsObserver> { 70 public content::WebContentsUserData<AutomationWebContentsObserver> {
71 public: 71 public:
72 virtual ~AutomationWebContentsObserver() {} 72 virtual ~AutomationWebContentsObserver() {}
73 73
74 // content::WebContentsObserver overrides. 74 // content::WebContentsObserver overrides.
75 virtual void AccessibilityEventReceived( 75 virtual void AccessibilityEventReceived(
76 const std::vector<content::AXEventNotificationDetails>& details) 76 const std::vector<content::AXEventNotificationDetails>& details)
77 OVERRIDE { 77 OVERRIDE {
78 automation_util::DispatchAccessibilityEventsToAutomation( 78 automation_util::DispatchAccessibilityEventsToAutomation(
79 details, browser_context_); 79 details, browser_context_,
80 web_contents()->GetContainerBounds().OffsetFromOrigin());
80 } 81 }
81 82
82 virtual void RenderFrameDeleted( 83 virtual void RenderFrameDeleted(
83 content::RenderFrameHost* render_frame_host) OVERRIDE { 84 content::RenderFrameHost* render_frame_host) OVERRIDE {
84 automation_util::DispatchTreeDestroyedEventToAutomation( 85 automation_util::DispatchTreeDestroyedEventToAutomation(
85 render_frame_host->GetProcess()->GetID(), 86 render_frame_host->GetProcess()->GetID(),
86 render_frame_host->GetRoutingID(), 87 render_frame_host->GetRoutingID(),
87 browser_context_); 88 browser_context_);
88 } 89 }
89 90
(...skipping 158 matching lines...) Expand 10 before | Expand all | Expand 10 after
248 return RespondNow(Error("desktop permission must be requested")); 249 return RespondNow(Error("desktop permission must be requested"));
249 250
250 AutomationManagerAsh::GetInstance()->Enable(browser_context()); 251 AutomationManagerAsh::GetInstance()->Enable(browser_context());
251 return RespondNow(NoArguments()); 252 return RespondNow(NoArguments());
252 #else 253 #else
253 return RespondNow(Error("getDesktop is unsupported by this platform")); 254 return RespondNow(Error("getDesktop is unsupported by this platform"));
254 #endif // defined(OS_CHROMEOS) 255 #endif // defined(OS_CHROMEOS)
255 } 256 }
256 257
257 } // namespace extensions 258 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698