telemetry for IA2 deprecated methods

RESOLVED FIXED in mozilla10

Status

()

defect
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: surkov, Assigned: tbsaunde)

Tracking

({access})

unspecified
mozilla10
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug])

Attachments

(2 attachments)

similar to bug 679786
(In reply to alexander surkov from comment #0)
> similar to bug 679786

I'm not sure exactly what methods these are, and I suspect by the time you've listed them you'll hav written 75% of the patch ;)  so do you want to take this?
(In reply to Trevor Saunders (:tbsaunde) from comment #1)

> I'm not sure exactly what methods these are, and I suspect by the time
> you've listed them you'll hav written 75% of the patch ;)  so do you want to
> take this?

that's correct, not this minute, marking it as good first bug hoping that someone would willing to work on it

iirc only IAccessibleTable is deprecated now, so the patch should telemetry when this interface is queried
Whiteboard: [good first bug]
Posted patch patchSplinter Review
Attachment #562478 - Flags: review?(surkov.alexander)
Comment on attachment 562478 [details] [diff] [review]
patch

Review of attachment 562478 [details] [diff] [review]:
-----------------------------------------------------------------

r=me

::: accessible/src/base/Statistics.h
@@ +55,5 @@
>    inline void ISimpleDomUsed()
>    { Telemetry::Accumulate(Telemetry::ISIMPLE_DOM_USAGE, 1); }
>  
> +  /**
> +   * Report that IAccessibleTable has been used.

please add deprecated word

@@ +58,5 @@
> +  /**
> +   * Report that IAccessibleTable has been used.
> +   */
> +  inline void IAccessibleTableUsed()
> +  { Telemetry::Accumulate(Telemetry::IACCESSIBLE_TABLE_USAGE, 1); }

two space indentation of function body

::: toolkit/components/telemetry/TelemetryHistograms.h
@@ +54,4 @@
>   */
>  HISTOGRAM(A11Y_INSTANTIATED, 0, 1, 2, BOOLEAN, "has accessibility support been instantiated")
>  HISTOGRAM(ISIMPLE_DOM_USAGE, 0, 1, 2, BOOLEAN, "have the ISimpleDOM* accessibility interfaces been used")
> +HISTOGRAM(IACCESSIBLE_TABLE_USAGE, 0, 1, 2, BOOLEAN, "has the IAccessibleTable accessibility interface been used")

add deprecated word please
Attachment #562478 - Flags: review?(surkov.alexander) → review+
Assignee: nobody → trev.saunders
Posted patch patch v3Splinter Review
https://hg.mozilla.org/mozilla-central/rev/7019803376f1
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
You need to log in before you can comment on or make changes to this bug.