Background

@apaskulin unearthed the following TODO while reviewing the Metrics Platform documentation in T372682: De-duplicate Metrics Platform API docs:

JavaScript:

• Currently, a number of other methods are attached to the core object and thereby exported via mw.eventLog. These include streamInSample(), and all methods under mw.eventLog.id and mw.eventLog.storage. These should not be invoked by external callers and will be removed from the de facto public interface in a future refactor.

AC

• Remove mw.eventLog.storage
  • At the time of writing, this is only referenced in the corresponding unit tests: https://codesearch.wmcloud.org/search/?q=mw.eventLog.storage&files=&excludeFiles=&repos=
• Remove mw.eventLog.id
  • At the time of writing, only mw.eventLog.id.getSessionId() is referenced: https://codesearch.wmcloud.org/things/?q=mw%5C.eventLog%5C.id&files=&excludeFiles=test&repos=

Notes

1. All uses of mw.eventLog.id.getSessionId() have now been removed – Experiment Platform decommissioned the Wikistories instrumentation and the performer_active_browsing_session_token contextual attribute