[NUI] Awake process controller when someone add ProcessEventOnce #5575
Conversation
If someone register once events during 1 index event invoke, that event will be invoked at next processing. But if nobody make some events after that situation, some flickering might happend. To void that case, Let we call ProcessEvents() as next idle automatically. Signed-off-by: Eunki, Hong <[email protected]>
hinohie
requested review from
hyunjushin,
dongsug-song,
Seoyeon2Kim,
taehyub,
jaehyun0cho,
everLEEst and
rabbitfor
as code owners
| // If once event added during 1 event invoke, request Process again as Idle. | ||
| if (internalProcessorOnceEvent[0] != null) | ||
| { | ||
| Awake(); |
There was a problem hiding this comment.
이거 동작이 되는건가요? 제가 테스트했을땐, Process callback안에서 호출하는 Awake()는 동작을 안하는것 같은데.
There was a problem hiding this comment.
https://review.tizen.org/gerrit/c/platform/core/uifw/dali-adaptor/+/299045
https://review.tizen.org/gerrit/c/platform/core/uifw/dali-csharp-binder/+/299044
위 두 패치가 반영되어야지 동작을 합니다. Description 에도 추가해놓겠습니다.
There was a problem hiding this comment.
Dali에 넣은 코드만으로 충분한것 아닌가요?
Dali의 변경 사항은 Awake가 Process callback중에 호출되었을때 동작하도록 수정한것 같은데,
기존에 ProcessorOnceEvent 에 이벤트를 등록 하는 행위가 Awake를 의미하지 않았고, 이벤트를 등록한 후에 명시적으로 Awake를 하도록 가이드가 되었었습니다.
그런데 지금 변경 사항은 ProcessOnceEvent 수행 도중에 ProcessOnceEvent에 핸들러를 추가가 발생 했을 때 자동으로 Awake를 호출해주는 상황이 되어 버립니다.
그래서 ProcessorOnceEvent를 ProcessorOnceEvent이벤트 처리 중에 추가 하는 것과 그렇지 않은 상황에 있어서 동작이 달라져 버립니다.
There was a problem hiding this comment.
승근님 의견이 맞는 듯 합니다. 이 PR 없이 달리쪽 수정사항 만으로도 현재 제가 원하는 동작을 할 수 있을 듯 합니다. 이 PR 은 close 해두겠습니다.
If someone register once events during 1 index event invoke, that event will be invoked at next processing.
But if nobody make some events after that situation, some flickering might happend.
To void that case, Let we call ProcessEvents() as next idle automatically.
requried dali patches :
https://review.tizen.org/gerrit/c/platform/core/uifw/dali-adaptor/+/299045
https://review.tizen.org/gerrit/c/platform/core/uifw/dali-csharp-binder/+/299044