You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flex.apache.org by "Christoph (JIRA)" <ji...@apache.org> on 2013/11/12 14:14:17 UTC

[jira] [Commented] (FLEX-33062) mx.managers.PopUpManagerImpl.mx_internal::hideModalWindow() ignores destroy flag if CSS style modalTransparencyDuration is set to 0

    [ https://issues.apache.org/jira/browse/FLEX-33062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13820071#comment-13820071 ] 

Christoph commented on FLEX-33062:
----------------------------------

Phew! Took me a while to find this bug description. I was hunting popup leaks, and this was a kind of side quest. At least this one is easily worked around:

Always use at least 1 millisecond in modalTransparencyDuration!

> mx.managers.PopUpManagerImpl.mx_internal::hideModalWindow() ignores destroy flag if CSS style modalTransparencyDuration is set to 0
> -----------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: FLEX-33062
>                 URL: https://issues.apache.org/jira/browse/FLEX-33062
>             Project: Apache Flex
>          Issue Type: Bug
>         Environment: Flex 4.5.1; Flex 4.6.0
>            Reporter: Alex Harui
>              Labels: memory_leak
>   Original Estimate: 48h
>  Remaining Estimate: 48h
>
> mx.managers.PopUpManagerImpl.mx_internal::hideModalWindow() ignores the destroy flag if the CSS style modalTransparencyDuration is set to 0. In this case, the popup manager creates a memory leak. When destroy is set to true, it only sets the visibility of the modalWindow to false instead of removing it from its parent.
> The algorithm for removing the modal window from its parent is implemented in mx.managers.PopUpManagerImpl.fadeOutDestroyEffectEndHandler(). This code probably has to be moved into a separate function and either be called directly from hideModalWindow() when destroy is true and modalTransparencyDuration is 0, otherwise from fadeOutDestroyEffectEndHandler().



--
This message was sent by Atlassian JIRA
(v6.1#6144)