Commit c64c0157 authored by Jacques Newman's avatar Jacques Newman Committed by Commit Bot

Add histogram logging for fragment root UIA entry points.

Bug: 1096223
AX-Relnotes: n/a
Change-Id: I7685e4e5b828a1d2ecc04398735506dde482dcf1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2250786Reviewed-by: default avatarKevin Babbitt <kbabbitt@microsoft.com>
Commit-Queue: Jacques Newman <janewman@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#780092}
parent e0a8ffdd
......@@ -695,6 +695,16 @@ uploading your change for review. These are checked by presubmit scripts.
<int value="245" label="UMA_API_WINDOW_GET_ISTOPMOST">
UIA Window.get_IsTopmost
</int>
<int value="246" label="UMA_API_ELEMENT_PROVIDER_FROM_POINT">
UIA FragmentRoot.ElementProviderFromPoint
</int>
<int value="247" label="UMA_API_GET_FOCUS">UIA FragmentRoot.GetFocus</int>
<int value="248" label="UMA_API_ADVISE_EVENT_ADDED">
UIA AdviseEvents.AdviseEventAdded
</int>
<int value="249" label="UMA_API_ADVISE_EVENT_REMOVED">
UIA AdviseEvents.AdviseEventRemoved
</int>
</enum>
<enum name="AccessibilityWinAPIGetPropertyValueEnum">
......@@ -43,6 +43,7 @@ class AXFragmentRootPlatformNodeWin : public AXPlatformNodeWin,
IFACEMETHODIMP get_HostRawElementProvider(
IRawElementProviderSimple** host_element_provider) override {
WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_HOST_RAW_ELEMENT_PROVIDER);
UIA_VALIDATE_CALL_1_ARG(host_element_provider);
HWND hwnd = GetDelegate()->GetTargetForNativeAccessibilityEvent();
......@@ -51,6 +52,7 @@ class AXFragmentRootPlatformNodeWin : public AXPlatformNodeWin,
IFACEMETHODIMP GetPropertyValue(PROPERTYID property_id,
VARIANT* result) override {
WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_PROPERTY_VALUE);
UIA_VALIDATE_CALL_1_ARG(result);
switch (property_id) {
......@@ -84,6 +86,7 @@ class AXFragmentRootPlatformNodeWin : public AXPlatformNodeWin,
IFACEMETHODIMP get_FragmentRoot(
IRawElementProviderFragmentRoot** fragment_root) override {
WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_FRAGMENTROOT);
UIA_VALIDATE_CALL_1_ARG(fragment_root);
QueryInterface(IID_PPV_ARGS(fragment_root));
......@@ -97,6 +100,7 @@ class AXFragmentRootPlatformNodeWin : public AXPlatformNodeWin,
double screen_physical_pixel_x,
double screen_physical_pixel_y,
IRawElementProviderFragment** element_provider) override {
WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_ELEMENT_PROVIDER_FROM_POINT);
UIA_VALIDATE_CALL_1_ARG(element_provider);
*element_provider = nullptr;
......@@ -124,6 +128,7 @@ class AXFragmentRootPlatformNodeWin : public AXPlatformNodeWin,
}
IFACEMETHODIMP GetFocus(IRawElementProviderFragment** focus) override {
WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_GET_FOCUS);
UIA_VALIDATE_CALL_1_ARG(focus);
*focus = nullptr;
......@@ -157,6 +162,7 @@ class AXFragmentRootPlatformNodeWin : public AXPlatformNodeWin,
//
IFACEMETHODIMP AdviseEventAdded(EVENTID event_id,
SAFEARRAY* property_ids) override {
WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_ADVISE_EVENT_ADDED);
if (event_id == UIA_LiveRegionChangedEventId) {
live_region_change_listeners_++;
......@@ -179,6 +185,7 @@ class AXFragmentRootPlatformNodeWin : public AXPlatformNodeWin,
IFACEMETHODIMP AdviseEventRemoved(EVENTID event_id,
SAFEARRAY* property_ids) override {
WIN_ACCESSIBILITY_API_HISTOGRAM(UMA_API_ADVISE_EVENT_REMOVED);
if (event_id == UIA_LiveRegionChangedEventId) {
DCHECK(live_region_change_listeners_ > 0);
live_region_change_listeners_--;
......
......@@ -281,6 +281,10 @@ enum {
UMA_API_WINDOW_GET_WINDOWVISUALSTATE = 243,
UMA_API_WINDOW_GET_WINDOWINTERACTIONSTATE = 244,
UMA_API_WINDOW_GET_ISTOPMOST = 245,
UMA_API_ELEMENT_PROVIDER_FROM_POINT = 246,
UMA_API_GET_FOCUS = 247,
UMA_API_ADVISE_EVENT_ADDED = 248,
UMA_API_ADVISE_EVENT_REMOVED = 249,
// This must always be the last enum. It's okay for its value to
// increase, but none of the other enum values may change.
......
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