Commit 4869fbb4 authored by Nektarios Paisios's avatar Nektarios Paisios Committed by Commit Bot

Cleanup: Fixed a few nits in AXObjWrappers and Automation code

TBR: dmazzoni@chromium.org, dtseng@chromium.org
Change-Id: I8fb826b2993e0a032fa77f7d6281980e21fb5394
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2067414Reviewed-by: default avatarNektarios Paisios <nektar@chromium.org>
Commit-Queue: Nektarios Paisios <nektar@chromium.org>
Auto-Submit: Nektarios Paisios <nektar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743384}
parent 48cd818b
...@@ -26,7 +26,7 @@ class NoDestructor; ...@@ -26,7 +26,7 @@ class NoDestructor;
namespace ui { namespace ui {
class AXEventBundleSink; class AXEventBundleSink;
} } // namespace ui
namespace views { namespace views {
class AccessibilityAlertWindow; class AccessibilityAlertWindow;
......
...@@ -62,7 +62,7 @@ class AutomationEventRouter : public ui::AXEventBundleSink, ...@@ -62,7 +62,7 @@ class AutomationEventRouter : public ui::AXEventBundleSink,
const ExtensionMsg_AccessibilityLocationChangeParams& params) override; const ExtensionMsg_AccessibilityLocationChangeParams& params) override;
// Notify all automation extensions that an accessibility tree was // Notify all automation extensions that an accessibility tree was
// destroyed. If |browser_context| is null, // destroyed. If |browser_context| is null, use the currently active context.
void DispatchTreeDestroyedEvent( void DispatchTreeDestroyedEvent(
ui::AXTreeID tree_id, ui::AXTreeID tree_id,
content::BrowserContext* browser_context) override; content::BrowserContext* browser_context) override;
......
...@@ -40,7 +40,7 @@ class AutomationEventRouterInterface { ...@@ -40,7 +40,7 @@ class AutomationEventRouterInterface {
const ExtensionMsg_AccessibilityLocationChangeParams& params) = 0; const ExtensionMsg_AccessibilityLocationChangeParams& params) = 0;
// Notify all automation extensions that an accessibility tree was // Notify all automation extensions that an accessibility tree was
// destroyed. If |browser_context| is null, // destroyed. If |browser_context| is null, use the currently active context.
virtual void DispatchTreeDestroyedEvent( virtual void DispatchTreeDestroyedEvent(
ui::AXTreeID tree_id, ui::AXTreeID tree_id,
content::BrowserContext* browser_context) = 0; content::BrowserContext* browser_context) = 0;
......
...@@ -12,7 +12,7 @@ ...@@ -12,7 +12,7 @@
namespace gfx { namespace gfx {
class Point; class Point;
} } // namespace gfx
namespace ui { namespace ui {
......
...@@ -41,4 +41,5 @@ void AccessibilityAlertWindow::OnWillDestroyEnv() { ...@@ -41,4 +41,5 @@ void AccessibilityAlertWindow::OnWillDestroyEnv() {
observer_.RemoveAll(); observer_.RemoveAll();
alert_window_.reset(); alert_window_.reset();
} }
} // namespace views } // namespace views
...@@ -24,7 +24,7 @@ AXViewObjWrapper::AXViewObjWrapper(AXAuraObjCache* aura_obj_cache, View* view) ...@@ -24,7 +24,7 @@ AXViewObjWrapper::AXViewObjWrapper(AXAuraObjCache* aura_obj_cache, View* view)
AXViewObjWrapper::~AXViewObjWrapper() = default; AXViewObjWrapper::~AXViewObjWrapper() = default;
bool AXViewObjWrapper::IsIgnored() { bool AXViewObjWrapper::IsIgnored() {
return view_ ? view_->GetViewAccessibility().IsIgnored() : true; return !view_ || view_->GetViewAccessibility().IsIgnored();
} }
AXAuraObjWrapper* AXViewObjWrapper::GetParent() { AXAuraObjWrapper* AXViewObjWrapper::GetParent() {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment