Closed Bug 372703 Opened 17 years ago Closed 17 years ago

No visual content in chat windows when using GTalk in Gmail.

Categories

(Core :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: fredrik.henrysson, Assigned: roc)

References

()

Details

(Keywords: regression)

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.9a3pre) Gecko/20070305 Minefield/3.0a3pre
Build Identifier: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.9a3pre) Gecko/20070305 Minefield/3.0a3pre

When using the GTalk feature of Gmail, there is no visual content in the popups that appear when chatting with somebody. If the chatpart is poped out of gmail, the content reappears.

Reproducible: Always

Steps to Reproduce:
1. Log into Gmail account
2. Start chat session using the Quick contact list
3. Write any thing in the input field 
4. Send message
Actual Results:  
No content is shown in chat history. Can't see replies either.

Expected Results:  
Chat history with the message sent and replies.

This seemed to happen sometime last week around end of Feb. (20070227).
Product: Firefox → Core
Version: unspecified → Trunk
QA Contact: general → general
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Mac OS X → All
Hardware: Macintosh → All
This is still present (Win32 2007040504).  Periodically the content will redraw (when the page is updated? but not every time!), but it makes gtalk pretty hard to use, and I suspect it points at a bad regression.
Flags: blocking1.9?
Keywords: regression
why still no one is in charge of this bug? This bug is annoying for 3.0a3, which makes me yield to use 3.0 for a month. I would like to test it and use it everyday, but I can't even chat with my friends by gtalk in gmail... And I just downloaded 3.0a4, the bug improves a little but still can't be used....
Ning Qu:

Nobody has taken this bug because nobody knows what the problem is.  GMail is a complex application, and figuring out why something is broken is hard.  If you can narrow down the problem into what we call a "reduced testcase", which would be a much smaller bit of HTML that shows the same problem, then it's much more likely that someone will be able to fix it.  Until that point, it's unlikely that the problem will be found.

Attached patch fixSplinter Review
This fixes it ... since abs-pos frames maybe no longer have views, we need to call PositionChildViews instead or descendant view coordinates could be wrong.
Assignee: nobody → roc
Status: NEW → ASSIGNED
Attachment #263221 - Flags: superreview?(dbaron)
Attachment #263221 - Flags: review?(sharparrow1)
That's great! Hope it is merged into official version soon :) Maybe 3.0a5?
Comment on attachment 263221 [details] [diff] [review]
fix

sr=dbaron.

Did you check for other places this might need to be fixed?
Attachment #263221 - Flags: superreview?(dbaron) → superreview+
Comment on attachment 263221 [details] [diff] [review]
fix

Looks fine; you should probably get rid of the call to PositionChildViews a bit further down, though.  (My fix to Bug 373433 wasn't thorough enough; views are really confusing...)
Attachment #263221 - Flags: review?(sharparrow1) → review+
As far as I can tell, the problem still exists
That's because the patch hasn't landed yet..
Checked in.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Flags: in-testsuite?
Blocks: 401821
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: