]> git.sesse.net Git - vlc/commit
skins2: remove unnecessary notifyLayout()
authorErwan Tulou <erwan10@videolan.org>
Thu, 14 Jan 2010 09:01:45 +0000 (10:01 +0100)
committerErwan Tulou <erwan10@videolan.org>
Thu, 14 Jan 2010 14:22:31 +0000 (15:22 +0100)
commit5cbb5d860cd870b552cf235e4ea385e3941dbc79
tree151765728c44ebeffb1afa2acd05e9338d071558
parente0674167fc97a82404526c9a08735517fe70732f
skins2: remove unnecessary notifyLayout()

In today's implementation, notifyLayout() for a control leads to a whole
 layout rebuilt. Therefore, this function must be called _only_ if needed.

- Never needed in onResize() since we are already in the process of rebuilding
the whole layout. Calling notifyLayout() means the job is done twice.
- Never needed in onPositionChange() for the same reason as onResize().
- Not needed for a checkbox since already called in setImage() two lines above.
modules/gui/skins2/controls/ctrl_checkbox.cpp
modules/gui/skins2/controls/ctrl_list.cpp
modules/gui/skins2/controls/ctrl_tree.cpp