Remove mscom stuff that was only used by non-caching legacy accessibility implementation
Categories
(Core :: IPC: MSCOM, task)
Tracking
()
Tracking | Status | |
---|---|---|
firefox115 | --- | fixed |
People
(Reporter: Jamie, Assigned: Jamie)
References
Details
(Whiteboard: [ctw-postship])
Attachments
(2 files)
Once we always use the accessibility cache, we can remove a lot of ipc/mscom, including Interceptor, InterceptorLog, Handler, MainThreadHandoff, DispatchForwarder, MainThreadInvoker, SpinEvent, FastMarshaler, StructStream, PassthruProxy, VTableBuilder, ProxyStream, RegistrationAnnotator, AsyncInvoker, and a bunch of dependencies of those things.
We do not want to remove ProfilerMarkers, as these are used to highlight uses of synchronous outgoing COM that should not be occurring on the main thread.
Assignee | ||
Updated•11 months ago
|
Assignee | ||
Comment 1•11 months ago
|
||
Regarding LazyInstantiator::ResolveDispatch, we already had code in MsaaAccessible to get the IAccessible typelib.
mscom::RegisterTypelib has a lot of dependencies and it doesn't make sense to keep it for this one use case we can already support another way.
Assignee | ||
Comment 2•11 months ago
|
||
Nothing else uses any of this or is likely to use it, so I don't think it makes sense to leave it as dead code.
Pushed by jteh@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ccace46d62f4 part 1: Remove/replace the few remaining ipc/mscom uses in a11y code. r=nlapre https://hg.mozilla.org/integration/autoland/rev/6376fec9febc part 2: Remove ipc/mscom code only used by the old accessibility architecture based on content process COM proxies. r=bobowen
Comment 4•10 months ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ccace46d62f4
https://hg.mozilla.org/mozilla-central/rev/6376fec9febc
Description
•