Shouldn’t the event be triggered by rollover rather than mouseover?

That seems like the issue to me…

> On Jul 11, 2017, at 10:13 AM, Alex Harui <aha...@adobe.com> wrote:
> 
> This seems like "just-in-case" code.  I could see that it might be needed
> in a different bead for Express though.
> 
> Without this change, is there an infinite loop somewhere?
> 
> Thanks,
> -Alex
> 
> On 7/10/17, 10:40 PM, "jmcl...@apache.org" <jmcl...@apache.org> wrote:
> 
>> FLEX-35338 stop cursor from spinning / CPU from spiking
>> 
>> 
>> Project: 
>> https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fgit-wip-us
>> .apache.org%2Frepos%2Fasf%2Fflex-asjs%2Frepo&data=02%7C01%7C%7C8e2e2d5b3b3
>> 54f54848508d4c81f5768%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C6363534
>> 84318029070&sdata=32hEJB%2BTNsWKG8MY%2FdPr3d0RWT5Xr1tECWWpVTlM270%3D&reser
>> ved=0
>> Commit: 
>> https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fgit-wip-us
>> .apache.org%2Frepos%2Fasf%2Fflex-asjs%2Fcommit%2F549a10d8&data=02%7C01%7C%
>> 7C8e2e2d5b3b354f54848508d4c81f5768%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%
>> 7C0%7C636353484318029070&sdata=dcNhAGu8WspafZBZCRxb%2FN0z7D%2Fym3tuv%2BAaq
>> G280z0%3D&reserved=0
>> Tree: 
>> https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fgit-wip-us
>> .apache.org%2Frepos%2Fasf%2Fflex-asjs%2Ftree%2F549a10d8&data=02%7C01%7C%7C
>> 8e2e2d5b3b354f54848508d4c81f5768%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C
>> 0%7C636353484318029070&sdata=0p89CGQLSv0Pp0yscOBNjKRJ1HUMhHrDngm4xxixalc%3
>> D&reserved=0
>> Diff: 
>> https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fgit-wip-us
>> .apache.org%2Frepos%2Fasf%2Fflex-asjs%2Fdiff%2F549a10d8&data=02%7C01%7C%7C
>> 8e2e2d5b3b354f54848508d4c81f5768%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C
>> 0%7C636353484318029070&sdata=%2FclUp4scfr0z%2B%2FTPjBMlVw2kScADTcLPMwop5J4
>> 7cIU%3D&reserved=0
>> 
>> Branch: refs/heads/develop
>> Commit: 549a10d82da5835c61a2b1247b65f7bfbc2f9a36
>> Parents: 3891820
>> Author: Justin Mclean <jmcl...@apache.org>
>> Authored: Tue Jul 11 14:57:04 2017 +1000
>> Committer: Justin Mclean <jmcl...@apache.org>
>> Committed: Tue Jul 11 14:57:04 2017 +1000
>> 
>> ----------------------------------------------------------------------
>> .../main/flex/org/apache/flex/html/accessories/ToolTipBead.as   | 5 ++++-
>> 1 file changed, 4 insertions(+), 1 deletion(-)
>> ----------------------------------------------------------------------
>> 
>> 
>> https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fgit-wip-us
>> .apache.org%2Frepos%2Fasf%2Fflex-asjs%2Fblob%2F549a10d8%2Fframeworks%2Fpro
>> jects%2FBasic%2Fsrc%2Fmain%2Fflex%2Forg%2Fapache%2Fflex%2Fhtml%2Faccessori
>> es%2FToolTipBead.as&data=02%7C01%7C%7C8e2e2d5b3b354f54848508d4c81f5768%7Cf
>> a7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636353484318029070&sdata=hifp5T8
>> xFlL4zov22P5qOUYtZqrYuPqTqOJg2x4X5lo%3D&reserved=0
>> ----------------------------------------------------------------------
>> diff --git 
>> a/frameworks/projects/Basic/src/main/flex/org/apache/flex/html/accessories
>> /ToolTipBead.as 
>> b/frameworks/projects/Basic/src/main/flex/org/apache/flex/html/accessories
>> /ToolTipBead.as
>> index 03ce21e..bdf0b68 100644
>> --- 
>> a/frameworks/projects/Basic/src/main/flex/org/apache/flex/html/accessories
>> /ToolTipBead.as
>> +++ 
>> b/frameworks/projects/Basic/src/main/flex/org/apache/flex/html/accessories
>> /ToolTipBead.as
>> @@ -100,7 +100,10 @@ package org.apache.flex.html.accessories
>>               * @private
>>               */
>>              protected function rollOverHandler( event:MouseEvent ):void
>> -            {       
>> +            {
>> +                    if (!toolTip)
>> +                            return;
>> +
>> 
>> IEventDispatcher(_strand).addEventListener(MouseEvent.MOUSE_OUT,
>> rollOutHandler, false);
>> 
>>            var comp:IUIBase = _strand as IUIBase
>> 
> 

Reply via email to