Re: [PATCH v2 0/4] MR2523: uiautomationcore: Miscellaneous MSAA provider patches.

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/uiautomationcore/tests/uiautomation.c:
> - /* Accessible now outside of its window's bounds. */
> - set_accessible_props(&Accessible, 0, ~STATE_SYSTEM_OFFSCREEN, 0, L"Accessible", rect[0].right, rect[0].bottom,
> -
>        10, 10);
> - SET_EXPECT(Accessible_get_accState);
> - SET_EXPECT(Accessible_accLocation);
> - hr = IRawElementProviderSimple_GetPropertyValue(elprov, UIA_IsOffscreenPropertyId, &v);
> - ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
> - ok(V_VT(&v) == VT_BOOL, "V_VT(&v) = %d\n", V_VT(&v));
> - ok(check_variant_bool(&v, TRUE), "Unexpected BOOL %#x\n", V_BOOL(&v));
> - CHECK_CALLED(Accessible_get_accState);
> - CHECK_CALLED(Accessible_accLocation);
> -
> - /* Accessible within window bounds, but not client area bounds. */
> - ok(GetWindowRect(Accessible.ow_hwnd, &rect[1]), "GetClientRect returned FALSE\n");
> - set_accessible_props(&Accessible, 0, ~STATE_SYSTEM_OFFSCREEN, 0, L"Accessible", rect[1].left, rect[1].top,
> -
>        rect[0].left - 1, rect[0].top - 1);

This seems to be assuming the last 2 arguments are right/bottom rather than width/height.