Closed
Bug 576055
Opened 14 years ago
Closed 14 years ago
MayHavePaintEventListener fails for e10s message manager
Categories
(Core :: IPC, defect)
Tracking
()
VERIFIED
FIXED
Tracking | Status | |
---|---|---|
fennec | 2.0+ | --- |
People
(Reporter: mfinkle, Assigned: smaug)
References
Details
Attachments
(1 file)
871 bytes,
patch
|
dougt
:
review+
|
Details | Diff | Splinter Review |
Seems similar to bug 571438, but the event fails to be generated for remote content. test-ipc.xul also appears to fail.
Reporter | ||
Updated•14 years ago
|
tracking-fennec: --- → 2.0+
Comment 4•14 years ago
|
||
Olli, can you help the fennec guys out here? If not, please speak up.
Assignee: nobody → Olli.Pettay
Assignee | ||
Comment 5•14 years ago
|
||
This seems like merge problem. MayHavePaintEventListener was fixed to work with e10s in https://bug549223.bugzilla.mozilla.org/attachment.cgi?id=429426.
Assignee | ||
Comment 6•14 years ago
|
||
Something like this should work. I'll test tomorrow. - return PR_FALSE + nsCOMPtr<nsPIWindowRoot> root = do_QueryInterface(parentTarget); + nsPIDOMEventTarget* tabChildGlobal; + return root && + (tabChildGlobal = root->GetParentTarget()) && + (manager = tabChildGlobal->GetListenerManager(PR_FALSE)) && + manager->MayHavePaintEventListener(); } I still don't understand how this code has just disappeared from e10s.
Comment 7•14 years ago
|
||
could have been a bad merge. :(
Reporter | ||
Comment 8•14 years ago
|
||
(In reply to comment #6) > Something like this should work. I'll test tomorrow. > > - return PR_FALSE > + nsCOMPtr<nsPIWindowRoot> root = do_QueryInterface(parentTarget); > + nsPIDOMEventTarget* tabChildGlobal; > + return root && > + (tabChildGlobal = root->GetParentTarget()) && > + (manager = tabChildGlobal->GetListenerManager(PR_FALSE)) && > + manager->MayHavePaintEventListener(); > } > > I still don't understand how this code has just disappeared from e10s. This fixes the MozAfterPaint issues for me. www.webhamsters.com show dancing hamsters!
Comment 9•14 years ago
|
||
Attachment #455350 -
Flags: review?(Olli.Pettay)
Comment 10•14 years ago
|
||
works fine for me too
Comment 11•14 years ago
|
||
(In reply to comment #7) > could have been a bad merge. :( don't think it is bad merge... m-c also don't have this
Comment 12•14 years ago
|
||
Comment on attachment 455350 [details] [diff] [review] ok, then just fix it this was my bustage.
Attachment #455350 -
Flags: review?(Olli.Pettay) → review+
Comment 13•14 years ago
|
||
http://hg.mozilla.org/projects/electrolysis/rev/5149a5980945
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Comment 14•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/15675470aa2d
Assignee | ||
Comment 15•14 years ago
|
||
(In reply to comment #11) > (In reply to comment #7) > > could have been a bad merge. :( > > don't think it is bad merge... m-c also don't have this m-c didn't have it because m-c didn't have the remote out-of-process TabChildGlobal/messageManager.
Comment 16•14 years ago
|
||
verified FIXED on builds: Mozilla/5.0 (X11; U; Linux armv71; Nokia N900; en-US; rv:2.0b2pre) Gecko/2010701 Namoroka/4.0b2pre Fennec/2.0a1pre fennec-electrolysis-maemo5-gtk
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•