Closed
Bug 1333983
Opened 8 years ago
Closed 8 years ago
Label runnable for PAPZ::Msg_RequestContentRepaint message
Categories
(Core :: Panning and Zooming, defect, P3)
Tracking
()
RESOLVED
DUPLICATE
of bug 1343479
People
(Reporter: billm, Unassigned)
References
Details
(Whiteboard: [gfx-noted])
I think this will be similar to the DidComposite message (bug 1333968). We'll need to figure out which tab the message and then send the message to the event target for that TabGroup.
For the first part, I think we can get the right TabChild via the APZChild's mController->mBrowser (assuming it's a ContentProcessController).
For the second part, bug 1333968 describes the two possible solutions.
Updated•8 years ago
|
Priority: -- → P2
Updated•8 years ago
|
Component: DOM → Panning and Zooming
Priority: P2 → --
Updated•8 years ago
|
Whiteboard: [gfx-noted]
Comment 1•8 years ago
|
||
(In reply to Bill McCloskey (:billm) from comment #0)
> For the first part, I think we can get the right TabChild via the APZChild's
> mController->mBrowser (assuming it's a ContentProcessController).
Yes, for messages going to the content process, the mController on the APZChild side will be a ContentProcessController, and the mBrowser is the TabChild.
Is there something you need us ("APZ folks") to do here, or is this something you will implement? I haven't been following the TabGroup changes.
OS: Unspecified → All
Priority: -- → P3
Hardware: Unspecified → All
Version: unspecified → 54 Branch
Comment 2•8 years ago
|
||
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #1)
> (In reply to Bill McCloskey (:billm) from comment #0)
> > For the first part, I think we can get the right TabChild via the APZChild's
> > mController->mBrowser (assuming it's a ContentProcessController).
>
> Yes, for messages going to the content process, the mController on the
> APZChild side will be a ContentProcessController, and the mBrowser is the
> TabChild.
>
> Is there something you need us ("APZ folks") to do here, or is this
> something you will implement? I haven't been following the TabGroup changes.
Flagging on Bill for confirming this question.
Flags: needinfo?(wmccloskey)
Reporter | ||
Comment 3•8 years ago
|
||
I'll need to do some initial work for this and bug 1333968. Depending on how easy it is, I might just fix them both myself. There's no need for graphics people to do anything right now though.
Flags: needinfo?(wmccloskey)
Reporter | ||
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•