Closed Bug 1384286 Opened 7 years ago Closed 2 years ago

come up with a labeling strategy for PContent::Msg_ParentActivated

Categories

(Core :: DOM: Content Processes, enhancement, P3)

enhancement

Tracking

()

RESOLVED WONTFIX

People

(Reporter: froydnj, Unassigned)

References

Details

This is a reasonably common runnable, but we can't send it to SystemGroup, because the message actually pertains to a particular TabChild: we should be sending it to the TabChild's group instead.  Unfortunately, extracting the information on *which* TabChild we need from the message is...not easy.  Extracting a raw integer ID would be a little easier, and we could map that to a particular TabChild when determining the event target for the message itself.

Does that sound reasonable, or can you think of a better way?
Flags: needinfo?(wmccloskey)
This message used to be on PBrowser instead of PContent. I moved it to PContent in bug 1334572 because I was afraid that activating one window would also deactivate another window. I know that happened with the Activate/Deactivate messages. I don't know if it happens for ParentActivated. (I don't really know the specifics of what all these messages do.)

Maybe we could get away with moving just this message back to PBrowser? You'll know if it works based on whether we assert.

If not, then we'll have to somehow split the activation and deactivation into separate runnables. Bug 1334572 has more details on that.
Flags: needinfo?(wmccloskey)
Priority: -- → P3
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.