From: Paul Wisbey Date: Wed, 8 Feb 2017 19:03:45 +0000 (+0000) Subject: [3.0] Fixed issue with Indicator when plugging/unplugging charger X-Git-Tag: accepted/tizen/3.0/common/20170210.172532~2 X-Git-Url: http://review.tizen.org/git/?p=platform%2Fcore%2Fuifw%2Fdali-adaptor.git;a=commitdiff_plain;h=21f9fd72bee0219a5471c62b13f89a2eeff9a180 [3.0] Fixed issue with Indicator when plugging/unplugging charger RequestUpdate() Update thread wakes up in UpdateRenderReady() * update & render * * Indicator is updated * RequestUpdateOnce() IsUpdateRenderThreadPaused() returns false ProcessSleepRequest() Update thread sleeps in UpdateRenderReady() In other words we're sleeping too soon, before the Indicator has been rendered. This is more likely to occur on mobile, since we get spammed with damage events. Change-Id: I587c80a6545c0f5623c02dd5443b8e2ea2319be4 --- diff --git a/adaptors/base/combined-update-render/combined-update-render-controller.cpp b/adaptors/base/combined-update-render/combined-update-render-controller.cpp index a1496db..738da28 100644 --- a/adaptors/base/combined-update-render/combined-update-render-controller.cpp +++ b/adaptors/base/combined-update-render/combined-update-render-controller.cpp @@ -245,6 +245,12 @@ void CombinedUpdateRenderController::RequestUpdate() void CombinedUpdateRenderController::RequestUpdateOnce() { + // Increment the update-request count to the maximum + if( mUpdateRequestCount < MAXIMUM_UPDATE_REQUESTS ) + { + ++mUpdateRequestCount; + } + if( IsUpdateRenderThreadPaused() ) { LOG_EVENT_TRACE;