0

[Views AX] Prefer base::SingleThreadTaskRunner in BAM_mac.mm

In preparation of moving BrowserAccessibility* and
BrowserAccessibilityManager* to the ui/ layer, I am working to move
anything that can be done ahead of time.

This change removes yet another content layer dependency from
BrowserAccessibilityManagerMac.

I have tested locally and found no difference in VoiceOver's
announcements of live region changes.

Bug: 40672441
Change-Id: I4ed6a3df0c5afd00c415631d9806087682733230
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5522879
Commit-Queue: Jacques Newman <janewman@microsoft.com>
Reviewed-by: Benjamin Beaudry <benjamin.beaudry@microsoft.com>
Cr-Commit-Position: refs/heads/main@{#1299323}
This commit is contained in:
Jacques Newman
2024-05-10 17:02:47 +00:00
committed by Chromium LUCI CQ
parent 3d284a872f
commit 6bac3de27a

@ -10,13 +10,12 @@
#import "base/mac/mac_util.h"
#include "base/strings/sys_string_conversions.h"
#include "base/strings/utf_string_conversions.h"
#include "base/task/single_thread_task_runner.h"
#include "base/task/task_traits.h"
#include "base/time/time.h"
#import "content/browser/accessibility/browser_accessibility_cocoa.h"
#import "content/browser/accessibility/browser_accessibility_mac.h"
#include "content/browser/renderer_host/render_frame_host_impl.h"
#include "content/public/browser/browser_task_traits.h"
#include "content/public/browser/browser_thread.h"
#include "content/public/browser/web_contents.h"
#include "ui/accelerated_widget_mac/accelerated_widget_mac.h"
#include "ui/accessibility/ax_role_properties.h"
#include "ui/accessibility/platform/ax_platform_tree_manager_delegate.h"
@ -250,7 +249,7 @@ void BrowserAccessibilityManagerMac::FireGeneratedEvent(
// Use native VoiceOver support for live regions.
BrowserAccessibilityCocoa* retained_node = native_node;
GetUIThreadTaskRunner({})->PostDelayedTask(
base::SingleThreadTaskRunner::GetCurrentDefault()->PostDelayedTask(
FROM_HERE,
base::BindOnce(
[](BrowserAccessibilityCocoa* wrapper) {