200331 - [AxPlugin] IOleContainer implementation on plugin does nothing

Status: RESOLVED INVALID

(Core Graveyard :: Embedding: ActiveX Wrapper, defect)

Description

[Adam Lock]

The document object in XPCDocument.cpp that implements IOleContainer returns
E_NOTIMPL for each of its methods. This prevents controls from asking the
container for an enumeration of other controls contained in the page.

The implementation should, at a bare minimum implement EnumObjects to return the
caller control but preferably return the list of other controls contained in the
same frame too or whatever behaviour IE uses.

Comment 1

[Adam Lock]

Attachment: Patch

Patch implements IOleContainer::EnumObjects. Impl is a modified version of that
found in ATL and has been verified using a test harness control.

Comment 2

[Juan José Mata]

Is this something we want to land soon?  What is the target milestone for it?

Comment 3

[Adam Lock]

It's not something I've seen any control in the wild ask for, though I've been
asked about it for 4.x. and it should go in for completeness.

I'll put future on it for the time being, though the patch could probably go in
soon enough if it has no detrimental effects.

Note to self, fix tabbing issues in patch

Comment 4

[Ed Morley [:emorley]]

The ActiveX embedding API was removed in bug 662023 and friends, making this INVALID.

[Filter bugspam on activexinvalid]