730198 - telemetry for YouDao Dictionary and Kazaguru

Status: RESOLVED DUPLICATE of bug 728905

(Core :: Disability Access APIs, defect)

Description

[alexander :surkov (:asurkov)]

spun off bug 729861 comment #2:

TextExtractorImpl32.dll and TextExtractorImpl64.dll are parts of YouDao Dictionary (Chinese).
Kazahook.dll is part of Kazaguru (Japanese) (see http://www.staticflower.net/software/kazaguru.html).

refer to bug 728905 to get instruction how to fix this one

Comment 1

[Mark Capella [:capella]]

(Added Trevor to cc:List ... wasn't sure if bugzilla was emailing him via whiteboard tag)

This is similar to bug728905 ... I could have rolled them together if I'd noticed earlier ... one special twist here is the use of either/or dll for YouDao win32 vs win64.

Is it right to add a consumer for each:

  if (::GetModuleHandleW(L"TextExtractorImpl32"))
    statistics::A11yConsumers(TEXTEXTRACTOR32);
  if (::GetModuleHandleW(L"TextExtractorImpl64"))
    statistics::A11yConsumers(TEXTEXTRACTOR64);

or to have both roll to the same consumer?

  if (::GetModuleHandleW(L"TextExtractorImpl32"))
    statistics::A11yConsumers(TEXTEXTRACTOR);
  if (::GetModuleHandleW(L"TextExtractorImpl64"))
    statistics::A11yConsumers(TEXTEXTRACTOR);

Comment 2

[alexander :surkov (:asurkov)]

keep both as one consumer

Comment 3

[Mark Capella [:capella]]

Ok then, how about if I add to and re-post the patch in bug#728905 ?

Comment 4

[alexander :surkov (:asurkov)]

(In reply to Mark Capella [:capella] from comment #3)
> Ok then, how about if I add to and re-post the patch in bug#728905 ?

fine with me.

Comment 5

[alexander :surkov (:asurkov)]

not fixed, but can mark as dupe of bug 728905 if you're going to address this bug there, or you could mark if fixed after you fixed it in bug 728905.