Commit 300d3fce authored by Ian Prest's avatar Ian Prest Committed by Commit Bot

UIA: Prevent AT from duplicating min/max/close

Chrome implements its own Minimize, Maximize, and Close buttons, and the
default buttons that UIA adds for all windows are therefore unnecessary.

Adding the `RefuseNonClientSupport` flag to ProviderOptions prevents UIA
from duplicating the window controls.

Bug: 928811
Change-Id: I1341b57b97a9f5a7131394bacb75dbec9f296676
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1747738
Commit-Queue: Ian Prest <iapres@microsoft.com>
Reviewed-by: default avatarKevin Babbitt <kbabbitt@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#686208}
parent 92add81d
...@@ -4096,7 +4096,8 @@ IFACEMETHODIMP AXPlatformNodeWin::get_ProviderOptions(ProviderOptions* ret) { ...@@ -4096,7 +4096,8 @@ IFACEMETHODIMP AXPlatformNodeWin::get_ProviderOptions(ProviderOptions* ret) {
WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_PROVIDER_OPTIONS); WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_PROVIDER_OPTIONS);
UIA_VALIDATE_CALL_1_ARG(ret); UIA_VALIDATE_CALL_1_ARG(ret);
*ret = ProviderOptions_ServerSideProvider | ProviderOptions_UseComThreading; *ret = ProviderOptions_ServerSideProvider | ProviderOptions_UseComThreading |
ProviderOptions_RefuseNonClientSupport;
return S_OK; return S_OK;
} }
......
...@@ -4119,9 +4119,10 @@ TEST_F(AXPlatformNodeWinTest, TestUIAGetProviderOptions) { ...@@ -4119,9 +4119,10 @@ TEST_F(AXPlatformNodeWinTest, TestUIAGetProviderOptions) {
ProviderOptions provider_options = static_cast<ProviderOptions>(0); ProviderOptions provider_options = static_cast<ProviderOptions>(0);
EXPECT_HRESULT_SUCCEEDED(root_node->get_ProviderOptions(&provider_options)); EXPECT_HRESULT_SUCCEEDED(root_node->get_ProviderOptions(&provider_options));
EXPECT_EQ( EXPECT_EQ(ProviderOptions_ServerSideProvider |
ProviderOptions_ServerSideProvider | ProviderOptions_UseComThreading, ProviderOptions_UseComThreading |
provider_options); ProviderOptions_RefuseNonClientSupport,
provider_options);
} }
TEST_F(AXPlatformNodeWinTest, TestUIAGetHostRawElementProvider) { TEST_F(AXPlatformNodeWinTest, TestUIAGetHostRawElementProvider) {
......
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