[Laszlo-dev] For Review: Change 20081208-maxcarlson-n Summary: Fix ordering of onmouseover/onmouseupoutside events in swf9

[André Bargull]

This changes the order of "onmouseup" and "onmouseupoutside" for swf9:

swf8:
onmouseup
onmouseupoutside

but in swf9:
onmouseupoutside
onmouseup

testcase:
<canvas debug="true">
   <view name="foo" width="100" height="100" bgcolor="blue" >
     <handler name="onmouseup" >
       Debug.write("onmouseup")
     </handler>
     <handler name="onmouseupoutside" >
       Debug.write("onmouseupoutside")
     </handler>
   </view>
</canvas>

Some thoughts which came to my mind while working on this bug:
Do we need to check for "__mouseoverInFront" in case of normal 
"onmouseup"-events (`normal` means the mouse was released over the 
mousedown-view)? Or do we just need to check this condition when the 
mouse went up on another view, so for "onmouseupoutside"-events?
I think it's only the latter, so I've moved the deferred "onmouseover" 
code to "__globalmouseup()" (all testcases from LPP-7300 and LPP-7335 
worked now the same in swf8 and swf9).
(There is an additional bugfix, the "__mouseoverInFront" object needs to 
be cleared for "onmouseout"-events.)

I've attached a new patch.


On 12/8/2008 2:15 PM, Max Carlson wrote:
>  Change 20081208-maxcarlson-n by maxcarlson at Bank.local on 2008-12-08 
> 05:09:44 PST
>     in /Users/maxcarlson/openlaszlo/trunk-clean
>     for http://svn.openlaszlo.org/openlaszlo/trunk
> 
> Summary: Fix ordering of onmouseover/onmouseupoutside events in swf9
> 
> Bugs Fixed: LPP-7335 - Fix DHTML mouseover events and swf9 event timing 
> (partial)
> 
> Technical Reviewer: promanik
> QA Reviewer: andre.bargull at udo.edu
> 
> Details: After looking in all the wrong places, this was really simple.
> 
> Tests: Testcase from LPP-7335 is now consistent between swf8 and swf9
> 
> Files:
> M      WEB-INF/lps/lfc/kernel/swf9/LzSprite.as
> 
> Changeset: 
> http://svn.openlaszlo.org/openlaszlo/patches/20081208-maxcarlson-n.tar
> 

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: LzSprite.as.patch
Url: http://www.openlaszlo.org/pipermail/laszlo-dev/attachments/20081208/f3564c3f/LzSprite.as-0001.pl