From: Seoyeon Kim Date: Tue, 2 May 2017 02:00:10 +0000 (+0900) Subject: Revert "[Tizen] Do nothing when setting focus on same actor" X-Git-Tag: accepted/tizen/unified/20170502.165632~11 X-Git-Url: http://review.tizen.org/git/?p=platform%2Fcore%2Fuifw%2Fdali-toolkit.git;a=commitdiff_plain;h=484d8acdc7efb1074de38d39eee8723a91b827d7 Revert "[Tizen] Do nothing when setting focus on same actor" This reverts commit a7ea628b9eda4d254f5e4bfe0bd0a874c8c20f56. Change-Id: Id85f7739dee7270ee565e8afb6e2f12d1c108fb9 --- diff --git a/dali-toolkit/internal/focus-manager/keyboard-focus-manager-impl.cpp b/dali-toolkit/internal/focus-manager/keyboard-focus-manager-impl.cpp index 2ad1502..d58560d 100644 --- a/dali-toolkit/internal/focus-manager/keyboard-focus-manager-impl.cpp +++ b/dali-toolkit/internal/focus-manager/keyboard-focus-manager-impl.cpp @@ -143,18 +143,6 @@ bool KeyboardFocusManager::DoSetCurrentFocusActor( Actor actor ) { bool success = false; - Actor currentFocusedActor = GetCurrentFocusActor(); - - // If developer set focus on same actor, doing nothing - if( actor == currentFocusedActor ) - { - if( !actor ) - { - return false; - } - return true; - } - // Check whether the actor is in the stage and is keyboard focusable. if( actor && actor.IsKeyboardFocusable() ) { @@ -162,8 +150,9 @@ bool KeyboardFocusManager::DoSetCurrentFocusActor( Actor actor ) { actor.Add( GetFocusIndicatorActor() ); } - // Send notification for the change of focus actor + Actor currentFocusedActor = GetCurrentFocusActor(); + if( !mFocusChangedSignal.Empty() ) { mFocusChangedSignal.Emit(currentFocusedActor, actor);