Closed
Bug 888336
Opened 11 years ago
Closed 11 years ago
chat title not displayed after tearing off and reattaching a chat window
Categories
(Firefox Graveyard :: SocialAPI, defect)
Firefox Graveyard
SocialAPI
Tracking
(firefox24 verified, firefox25 verified)
VERIFIED
FIXED
Firefox 25
People
(Reporter: florian, Assigned: florian)
References
Details
Attachments
(3 files, 1 obsolete file)
(deleted),
patch
|
mixedpuppy
:
review+
bajaj
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details |
At first this looked like a trivial missing setAttribute call, but it turned out there were several issues:
- first, the swapDocShells method wasn't called with the right parameter and 'this' values when reattaching.
- then, the title was displayed correctly, but the window from which we were moving the docShell didn't close itself any more because of an exception when removing a webProgressListener (in socialErrorListener.remove).
- and finally, when I wanted to test this, my test failed intermittently because the "got-chatbox-visibility" message is sometimes received once, sometimes twice (I guess the order in which the events related to destroying the old window, and loading the new one aren't fired in a deterministic order). I wonder if this (especially the swap.click() call in there) could have been causing other less visible intermittent failures.
Attachment #768990 -
Flags: review?(mixedpuppy)
Assignee | ||
Updated•11 years ago
|
status-firefox24:
--- → affected
status-firefox25:
--- → affected
Assignee | ||
Comment 1•11 years ago
|
||
(bug 886086 had changed browser_chat_tearoff.js since my last mozilla-central update.)
Attachment #768990 -
Attachment is obsolete: true
Attachment #768990 -
Flags: review?(mixedpuppy)
Attachment #769667 -
Flags: review?(mixedpuppy)
Updated•11 years ago
|
Attachment #769667 -
Flags: review?(mixedpuppy) → review+
Assignee | ||
Comment 2•11 years ago
|
||
Comment 3•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 25
Assignee | ||
Comment 4•11 years ago
|
||
Comment on attachment 769667 [details] [diff] [review]
Unbitrotted patch
[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 880911
User impact if declined: Some brokenness on detached and then reattached SocialAPI chat windows. The most visible one is the lack of title in the floating window's title bar.
Testing completed (on m-c, etc.): Test included in the patch; landed on m-c a few days ago.
Risk to taking this patch (and alternatives if risky): low.
String or IDL/UUID changes made by this patch: none.
Attachment #769667 -
Flags: approval-mozilla-aurora?
Updated•11 years ago
|
Comment 5•11 years ago
|
||
Comment on attachment 769667 [details] [diff] [review]
Unbitrotted patch
low risk patch that helps fix a user visible bug when user is trying to use the new feature (tearing-off of chat windows in Social). Approving on aurora.
Attachment #769667 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 6•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/938ff744e73e
The patch I landed on aurora is actually attachment 768990 [details] [diff] [review], as bug 886086 has not landed on aurora.
Samvedana, can you please verify this is fixed in Firefox 24 and 25?
QA Contact: samvedana.gohil
Comment 8•11 years ago
|
||
User Agent :Mozilla/5.0 (Windows NT 6.2; Win64; x64; rv:25.0) Gecko/20130723 Firefox/25.0
Build ID: 20130723030205
Tested this on Windows 8 using latest nightly. I don't see any issues. Both windows were showing chat title.
See attached screen shots.
Updated•11 years ago
|
Comment 9•11 years ago
|
||
Comment 10•11 years ago
|
||
User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:24.0) Gecko/20130723 Firefox/24.0
Build ID: 20130723004004
Tested this on Windows 8 using latest Aurora. I don't see any issues. Both windows were showing chat title.
Comment 11•11 years ago
|
||
I backed this out and re-landed it on beta so that I could more easily uplift the fixes for bug 886072 and bug 886086. There should be no functional change - it's just reordering a couple commits.
https://hg.mozilla.org/releases/mozilla-beta/rev/8fb54e7478ef
Updated•6 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•