Mailing-List: contact flex-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: flex-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of cframpto@adobe.com designates
 64.18.1.181 as permitted sender)
From: Carol Frampton <cframpto@adobe.com>
To: "flex-dev@incubator.apache.org" <flex-dev@incubator.apache.org>
Date: Thu, 5 Apr 2012 06:29:57 -0700
Subject: Re: performance issues of tab movement in s:DataGrid of Flex 4.6
Thread-Topic: performance issues of tab movement in s:DataGrid of Flex 4.6
Thread-Index: Ac0TMDNGg8mBewlESc+dk3NUGSe3Yw==
Message-ID: <CBA31675.124F3%cframpto@adobe.com>
In-Reply-To: <3A870C49-4444-4FC8-B625-623CB59D7DC8@classsoftware.com>
Accept-Language: en-US
Content-Language: en-US
user-agent: Microsoft-MacOutlook/14.14.0.111121
acceptlanguage: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

One of you is talking about s:DataGrid and one of you is talking about
mx:AdvancedDataGrid.  Not sure it is the same issue.

Carol

On 4/5/12 9 :15AM, "Justin Mclean" <justin@classsoftware.com> wrote:

>Hi,
>
>There a known issue with invalidateNow being call too often in the ADG.
>Once we have Mustella I have a patch that can be applied.
>
>Thanks,
>Justin
>
>> Hello
>>=20
>> I found an issue on item editors movement by TAB key.
>> In mx:DataGrid, it moves quickly. However, I feel it's slow in
>>s:DataGrid.
>> I investigated source of classes related with s:DataGrid a little.
>>=20
>> I found three factors.
>>=20
>>  1) s:GridItemEditor uses s:TextArea
>>   2) grid.focusManager.getNextFocusManagerComponent takes dozens of
>> milliseconds in editor_keyFocusChangeHandler function of
>> s:DataGridEditor
>>   3) it calls the validateNow method in do while loop at
>> ensureCellIsVisible function of s:Grid
>>=20
>> We can avoid 1) by using custom light-weight editor. But we can't
>> avoid 2) and 3).
>>=20
>> Why is 2) needed?
>>=20
>> It takes 10% of CPU resources at each TAB key movement of itemEditors
>> because the validateNow is called.
>> When I comment out 3), the focus moves incorrectly at rendererIsEditor
>>column.
>> However, there should be other ways without calling validateNow.
>>=20
>>=20
>> thanks
>>=20
>> Shigeru Nakagaki
>