Closed
Bug 130620
Opened 23 years ago
Closed 22 years ago
Moz 0.9.9 misses mouseout events if the background color is changed
Categories
(Core :: DOM: Events, defect, P2)
Core
DOM: Events
Tracking
()
VERIFIED
FIXED
People
(Reporter: ron, Assigned: roc)
References
()
Details
(Keywords: regression, testcase, topembed+, Whiteboard: [bugscape 16562])
Attachments
(3 files)
Using IMP webmail application. They make extensive use of "onmouseover" and
"onmouseout" directives to highlight the email message the mouse is hovering
over. With the newest mozilla (0.9.9), the onmouseout events are getting missed
- causing several lines on the page to remain highlighted.
Machine is a Toshiba Laptop (P3-700, 256 meg ram).
This problem did not exist on Moz 0.9.8.
Comment 1•23 years ago
|
||
-> Dom Events
We need a testcase...
Assignee: asa → joki
Component: Browser-General → DOM Events
QA Contact: doronr → vladimire
Comment 2•23 years ago
|
||
I am having the same problem, and can confirm this. I have not developed a small
testcase but you can observe this on an internal site...
http://bubblegum/ngdriver/results/results.html
This used to work not long ago, so it is a regression.
Updated•23 years ago
|
Keywords: regression
Comment 3•23 years ago
|
||
This seems to be a related issue -
When moving the mouse over and out on the top grey box, mouseout only seems to
fire on every second expected mouseout event.
The second grey box shows some uglier, but possibly related issues with tables
and resizing, along with mouseout handling.
This is tested without missed events on:
Mozilla/5.0 (X11; U; Linux 2.4.17 i686; en-US; rv:0.9.1) Gecko/20010610
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0rc2) Gecko/20020510
About attachment id=80178:
I configrm problem when node.style.overflow is changed 'visible'<->'hidden'.
Table cell width grow when node.style.overflow is changed 'visible'<->'hidden'.
This is second problem.
Comment 5•22 years ago
|
||
*** Bug 137700 has been marked as a duplicate of this bug. ***
Comment 6•22 years ago
|
||
better URL from a duplicate
Comment 7•22 years ago
|
||
*** Bug 146616 has been marked as a duplicate of this bug. ***
Comment 8•22 years ago
|
||
*** Bug 145323 has been marked as a duplicate of this bug. ***
Comment 9•22 years ago
|
||
another url: http://www.microsoft.com/germany/
OS: Windows 2000 → All
Priority: -- → P2
Comment 10•22 years ago
|
||
Comment 11•22 years ago
|
||
This happens all over the web and is making our gecko qa automation test result
page hard to use.
Here's another url to reproduce the problem: http://boxfly.free.fr/test2/
Keywords: nsbeta1
Comment 12•22 years ago
|
||
This breaks a major AOL Partner site, adidas.com
(http://bugscape.netscape.com/show_bug.cgi?id=16554)
Marking topembed.
Is there any chance to have this for machv/m1.0.1?
Keywords: topembed
Comment 13•22 years ago
|
||
Being tracked at http://bugscape.netscape.com/show_bug.cgi?id=16562
Whiteboard: [bugscape 16562]
Updated•22 years ago
|
Attachment #83919 -
Attachment mime type: text/plain → text/html
Updated•22 years ago
|
Keywords: mozilla1.1,
testcase
Updated•22 years ago
|
Summary: Moz 0.9.9 misses mouseout events → Moz 0.9.9 misses mouseout events if the background color is changed
Comment 14•22 years ago
|
||
*** Bug 152233 has been marked as a duplicate of this bug. ***
Comment 16•22 years ago
|
||
this is a regression from bug 33601 (discovered by backing out patches from
ancient CVS)
cc'ing Robert O'Callahan
Comment 17•22 years ago
|
||
*** Bug 146790 has been marked as a duplicate of this bug. ***
Comment 18•22 years ago
|
||
*** Bug 152985 has been marked as a duplicate of this bug. ***
Comment 20•22 years ago
|
||
*** Bug 131244 has been marked as a duplicate of this bug. ***
Updated•22 years ago
|
Hardware: PC → All
Comment 21•22 years ago
|
||
*** Bug 157877 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 22•22 years ago
|
||
I'm not sure what the bug is here. The first testcase is weird because two
elements have the same ID, so it's not really valid. The table testcase seems to
work fine as far as I can tell.
Comment 23•22 years ago
|
||
for the table, if your mouse hits the text, then the bug will not show up.
you might try attachment 87883 [details] from bug 152233. it's simpler to trigger there.
Assignee | ||
Comment 24•22 years ago
|
||
Sorry, I was using the wrong version of Mozilla. I see it now.
Assignee | ||
Comment 25•22 years ago
|
||
What's happening here is that setting the "background" property on the table
cell is setting "background-attachment" to "scroll" which causes a FRAMECHANGE
to fire in nsCSSParser::AppendValue. There are actually two problems here:
1) The FRAMECHANGE causes the event state manager (I presume) to miss the
mouse-out event. Presumably the fact that the mouse entered the element is lost
during the FRAMECHANGE. This must be fixed because there are many style changes
that require a FRAMECHANGE. Thus cc'ing jst.
2) In this case, changing the background shouldn't actually require a
FRAMECHANGE. The reason we get a FRAMECHANGE is because the rule for the table
cell changes from not specifying a background-attachement to specifying
"scroll". nsCSSParser::AppendValue generates its change hint by looking at the
change in the rule alone, rather than by looking at the change in the computed
style for the particular element. This could be, and probably should be, made
smarter, but I'm not sure how best to do that. CC'ing dbaron.
Comment 26•22 years ago
|
||
Regarding (2). I've been thinking we might want to get rid of the hints that
are in nsCSSPropList.h (which is the relevant hint that you're talking about).
I'm not sure if any of the optimizations they make are useful anymore. (Were
they ever?) The best way to do that is probably to eliminate one caller at a
time. I just filed bug 158713 to examine doing this.
roc, which testcase where you analyzing in your previous comment?
Assignee | ||
Comment 27•22 years ago
|
||
Attachment 85477 [details] is the testcase I was looking at.
Doing reresolution and getting the change hint from there is exactly what we
need here.
Assignee | ||
Comment 28•22 years ago
|
||
joki's the event state manager man. I always confuse him with jst!
Comment 29•22 years ago
|
||
*** Bug 151893 has been marked as a duplicate of this bug. ***
Updated•22 years ago
|
Keywords: mozilla1.2
Comment 30•22 years ago
|
||
*** Bug 173535 has been marked as a duplicate of this bug. ***
Comment 31•22 years ago
|
||
Another example: Lefthand buttons on http://www.microsoft.com/Australia/
Comment 32•22 years ago
|
||
This is effecting top sites in Australia and Germany. Marking as nsbeta1+
Assignee | ||
Comment 33•22 years ago
|
||
bz's inline style fix should help with this kind of bug.
Another thing that I'm thinking of doing that will help is to make it so when a
style change demands a view, instead of reframing, we'll create a view for the
existing frame and jam it directly into the view hierarchy (then reflow the
existing frame to make sure the view is set up properly). This shouldn't be much
work and will improve performance in certain cases.
The frame-based ESM code still needs to be fixed up, though.
Updated•22 years ago
|
Keywords: mozilla1.1
Comment 34•22 years ago
|
||
changing background property to backgroundColor is a trivial workabout for this
bug.
Comment 35•22 years ago
|
||
Does this need to be reassigned??
Comment 36•22 years ago
|
||
*** Bug 146808 has been marked as a duplicate of this bug. ***
Comment 37•22 years ago
|
||
*** Bug 184445 has been marked as a duplicate of this bug. ***
Comment 38•22 years ago
|
||
*** Bug 184727 has been marked as a duplicate of this bug. ***
Comment 39•22 years ago
|
||
It looks to me like bug 103055 fixed this (in my debug build at least). Could
you guys retest with tomorrow's build?
Depends on: 103055
Comment 40•22 years ago
|
||
Oh yeah. John, You rock! This is fixed in build 2002-12-17-08-trunk
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Comment 41•22 years ago
|
||
can you check TB15208182M ? I just crashed with build 2002121708 on Linux
browing http://wfrv_new.webtest.netnet.net/ and hovering the menus.
Comment 42•22 years ago
|
||
Wow, that was a hard crash! Talkback has no stack trace for it:
Incident ID 15208182
Stack Signature 0x0000000b f475f2f3
Email Address cahagn_o@epita.fr
Product ID MozillaTrunk
Build ID 2002121708
Trigger Time 2002-12-17 11:23:02
Platform LinuxIntel
Operating System Linux 2.4.17-10mdk
Module
URL visited http://wfrv_new.webtest.netnet.net/
User Comments bug 130620, loading URL
Trigger Reason SIGSEGV: Segmentation Fault: (signal 11)
Source File Name
Trigger Line No.
Stack Trace
0x0000000b
Comment 43•22 years ago
|
||
0x0000000b is par for the course on linux. see bug 176886
Comment 44•22 years ago
|
||
http://wfrv_new.webtest.netnet.net/ crashes in windows xp also with build 2002121704
Comment 45•22 years ago
|
||
Update: looks like the crash is a regression due to the fix for bug 103055.
Bug 185850 has been filed: "Crash on http://wfrv_new.webtest.netnet.net"
You need to log in
before you can comment on or make changes to this bug.
Description
•