Commit ea9f6b4c authored by David Tseng's avatar David Tseng Committed by Commit Bot

Reland "Enable ChromeVox ARC support for all apps"

This reverts commit fe043a76.

Reason for revert: re-enabling for m68

Original change's description:
> Revert "Enable ChromeVox ARC support for all apps"
> 
> This reverts commit bea32252.
> 
> Reason for revert: various apps breaking e.g. scrolling
> 
> Original change's description:
> > Enable ChromeVox ARC support for all apps
> > 
> > Change-Id: I130db573fcc2ccbda3ce58c9d2bac845218ab1d9
> > Reviewed-on: https://chromium-review.googlesource.com/961549
> > Commit-Queue: David Tseng <dtseng@chromium.org>
> > Reviewed-by: Yuki Awano <yawano@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#544283}
> 
> TBR=dtseng@chromium.org,yawano@chromium.org
> 
> # Not skipping CQ checks because original CL landed > 1 day ago.
> 
> Change-Id: I6a3c80ac74b3101a3ab756f4e0c5e74980019980
> Reviewed-on: https://chromium-review.googlesource.com/989053
> Commit-Queue: David Tseng <dtseng@chromium.org>
> Reviewed-by: David Tseng <dtseng@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#547379}

TBR=dtseng@chromium.org,yawano@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Change-Id: I8f7ba6cfe4d59df80309a49499c754768645d453
Reviewed-on: https://chromium-review.googlesource.com/1014016Reviewed-by: default avatarDavid Tseng <dtseng@chromium.org>
Commit-Queue: David Tseng <dtseng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550977}
parent b789651a
......@@ -89,14 +89,12 @@ arc::mojom::AccessibilityFilterType GetFilterTypeForProfile(Profile* profile) {
if (accessibility_manager->profile() != profile)
return arc::mojom::AccessibilityFilterType::OFF;
if (accessibility_manager->IsSelectToSpeakEnabled() ||
if (accessibility_manager->IsSpokenFeedbackEnabled() ||
accessibility_manager->IsSelectToSpeakEnabled() ||
accessibility_manager->IsSwitchAccessEnabled()) {
return arc::mojom::AccessibilityFilterType::ALL;
}
if (accessibility_manager->IsSpokenFeedbackEnabled())
return arc::mojom::AccessibilityFilterType::WHITELISTED_PACKAGE_NAME;
if (accessibility_manager->IsFocusHighlightEnabled())
return arc::mojom::AccessibilityFilterType::FOCUS;
......
......@@ -93,7 +93,7 @@ IN_PROC_BROWSER_TEST_F(ArcAccessibilityHelperBridgeBrowserTest,
aura::client::kAccessibilityTouchExplorationPassThrough));
chromeos::AccessibilityManager::Get()->EnableSpokenFeedback(true);
EXPECT_EQ(mojom::AccessibilityFilterType::WHITELISTED_PACKAGE_NAME,
EXPECT_EQ(mojom::AccessibilityFilterType::ALL,
fake_accessibility_helper_instance_->filter_type());
// Touch exploration pass through of test_window_1 (current active window)
......
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