X-Git-Url: http://review.tizen.org/git/?a=blobdiff_plain;f=dali%2Finternal%2Fupdate%2Fmanager%2Fupdate-manager.cpp;h=344891ac1287683b5bfaf2817d7b13c8ea8998bd;hb=3dec6c3b4fe13f341a2bf130b0fc369e87f54450;hp=dae719408d94c83546eeb8ecc106e5e98e586dba;hpb=5ccda65a296046076deabf99b944f41eba28830a;p=platform%2Fcore%2Fuifw%2Fdali-core.git diff --git a/dali/internal/update/manager/update-manager.cpp b/dali/internal/update/manager/update-manager.cpp index dae7194..344891a 100644 --- a/dali/internal/update/manager/update-manager.cpp +++ b/dali/internal/update/manager/update-manager.cpp @@ -1,5 +1,5 @@ /* - * Copyright (c) 2016 Samsung Electronics Co., Ltd. + * Copyright (c) 2017 Samsung Electronics Co., Ltd. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -20,7 +20,6 @@ // INTERNAL INCLUDES #include -#include #include #include @@ -44,7 +43,6 @@ #include #include #include -#include #include #include #include @@ -55,7 +53,6 @@ #include #include #include -#include #include #include @@ -99,19 +96,64 @@ namespace Internal namespace SceneGraph { -typedef OwnerContainer< Shader* > ShaderContainer; -typedef ShaderContainer::Iterator ShaderIter; -typedef ShaderContainer::ConstIterator ShaderConstIter; +namespace +{ +/** + * Helper to reset animate-able objects to base values + * @param container to iterate over + * @param updateBufferIndex to use + */ +template< class T > +inline void ResetToBaseValues( OwnerContainer& container, BufferIndex updateBufferIndex ) +{ + // Reset animatable properties to base values + typename OwnerContainer::Iterator iter = container.Begin(); + const typename OwnerContainer::ConstIterator endIter = container.End(); + for ( ; iter != endIter; ++iter ) + { + (*iter)->ResetToBaseValues( updateBufferIndex ); + } +} + +/** + * Helper to Erase an object from OwnerContainer using discard queue + * @param container to remove from + * @param object to remove + * @param discardQueue to put the object to + * @param updateBufferIndex to use + */ +template < class T > +inline void EraseUsingDiscardQueue( OwnerContainer& container, T* object, DiscardQueue& discardQueue, BufferIndex updateBufferIndex ) +{ + DALI_ASSERT_DEBUG( object && "NULL object not allowed" ); + + typename OwnerContainer::Iterator iter = container.Begin(); + const typename OwnerContainer::ConstIterator endIter = container.End(); + for ( ; iter != endIter; ++iter ) + { + if ( *iter == object ) + { + // Transfer ownership to the discard queue, this keeps the object alive, until the render-thread has finished with it + discardQueue.Add( updateBufferIndex, container.Release( iter ) ); + return; + } + } +} +} + +typedef OwnerContainer< Shader* > ShaderOwner; +typedef ShaderOwner::Iterator ShaderIter; typedef std::vector ShaderDataBinaryQueue; -typedef OwnerContainer GestureContainer; -typedef GestureContainer::Iterator GestureIter; -typedef GestureContainer::ConstIterator GestureConstIter; +typedef OwnerContainer< TextureSet* > TextureSetOwner; +typedef TextureSetOwner::Iterator TextureSetIter; + +typedef OwnerContainer RendererOwner; +typedef RendererOwner::Iterator RendererIter; -typedef OwnerContainer< TextureSet* > TextureSetContainer; -typedef TextureSetContainer::Iterator TextureSetIter; -typedef TextureSetContainer::ConstIterator TextureSetConstIter; +typedef OwnerContainer< Camera* > CameraOwner; +typedef OwnerContainer< PropertyOwner* > CustomObjectOwner; /** * Structure to contain UpdateManager internal data @@ -145,20 +187,20 @@ struct UpdateManager::Impl systemLevelTaskList( renderMessageDispatcher ), root( NULL ), systemLevelRoot( NULL ), - renderers( sceneGraphBuffers, discardQueue ), + renderers(), textureSets(), + shaders(), + panGestureProcessor( NULL ), messageQueue( renderController, sceneGraphBuffers ), keepRenderingSeconds( 0.0f ), - animationFinishedDuringUpdate( false ), nodeDirtyFlags( TransformFlag ), // set to TransformFlag to ensure full update the first time through Update() - previousUpdateScene( false ), frameCounter( 0 ), + animationFinishedDuringUpdate( false ), + previousUpdateScene( false ), renderTaskWaiting( false ) { sceneController = new SceneControllerImpl( renderMessageDispatcher, renderQueue, discardQueue ); - renderers.SetSceneController( *sceneController ); - // create first 'dummy' node nodes.PushBack(0u); } @@ -235,16 +277,16 @@ struct UpdateManager::Impl SortedLayerPointers sortedLayers; ///< A container of Layer pointers sorted by depth SortedLayerPointers systemLevelSortedLayers; ///< A separate container of system-level Layers - OwnerContainer< Camera* > cameras; ///< A container of cameras - OwnerContainer< PropertyOwner* > customObjects; ///< A container of owned objects (with custom properties) + CameraOwner cameras; ///< A container of cameras + CustomObjectOwner customObjects; ///< A container of owned objects (with custom properties) AnimationContainer animations; ///< A container of owned animations PropertyNotificationContainer propertyNotifications; ///< A container of owner property notifications. - ObjectOwnerContainer renderers; - TextureSetContainer textureSets; ///< A container of texture sets - - ShaderContainer shaders; ///< A container of owned shaders + RendererOwner renderers; ///< A container of owned renderers + TextureSetOwner textureSets; ///< A container of owned texture sets + ShaderOwner shaders; ///< A container of owned shaders + OwnerPointer panGestureProcessor; ///< Owned pan gesture processor; it lives for the lifecycle of UpdateManager MessageQueue messageQueue; ///< The messages queued from the event-thread ShaderDataBinaryQueue renderCompiledShaders; ///< Shaders compiled on Render thread are inserted here for update thread to pass on to event thread. @@ -252,14 +294,11 @@ struct UpdateManager::Impl Mutex compiledShaderMutex; ///< lock to ensure no corruption on the renderCompiledShaders float keepRenderingSeconds; ///< Set via Dali::Stage::KeepRendering - bool animationFinishedDuringUpdate; ///< Flag whether any animations finished during the Update() - int nodeDirtyFlags; ///< cumulative node dirty flags from previous frame - bool previousUpdateScene; ///< True if the scene was updated in the previous frame (otherwise it was optimized out) - int frameCounter; ///< Frame counter used in debugging to choose which frame to debug and which to ignore. - GestureContainer gestures; ///< A container of owned gesture detectors + bool animationFinishedDuringUpdate; ///< Flag whether any animations finished during the Update() + bool previousUpdateScene; ///< True if the scene was updated in the previous frame (otherwise it was optimized out) bool renderTaskWaiting; ///< A REFRESH_ONCE render task is waiting to be rendered private: @@ -383,21 +422,8 @@ void UpdateManager::AddCamera( Camera* camera ) void UpdateManager::RemoveCamera( const Camera* camera ) { - // Find the camera - OwnerContainer::Iterator iter = mImpl->cameras.Begin(); - OwnerContainer::ConstIterator end = mImpl->cameras.End(); - for ( ; iter != end; ++iter ) - { - Camera* value = *iter; - if ( camera == value ) - { - // Transfer ownership to the discard queue - mImpl->discardQueue.Add( mSceneGraphBuffers.GetUpdateBufferIndex(), mImpl->cameras.Release( iter ) ); - - return; - } - } - + // Find the camera and destroy it + EraseUsingDiscardQueue( mImpl->cameras, const_cast( camera ), mImpl->discardQueue, mSceneGraphBuffers.GetUpdateBufferIndex() ); } void UpdateManager::AddObject( PropertyOwner* object ) @@ -409,23 +435,7 @@ void UpdateManager::AddObject( PropertyOwner* object ) void UpdateManager::RemoveObject( PropertyOwner* object ) { - DALI_ASSERT_DEBUG( NULL != object ); - - OwnerContainer< PropertyOwner* >& customObjects = mImpl->customObjects; - - // Find the object and destroy it - for ( OwnerContainer< PropertyOwner* >::Iterator iter = customObjects.Begin(); iter != customObjects.End(); ++iter ) - { - PropertyOwner* current = *iter; - if ( current == object ) - { - customObjects.Erase( iter ); - return; - } - } - - // Should not reach here - DALI_ASSERT_DEBUG(false); + mImpl->customObjects.EraseObject( object ); } void UpdateManager::AddAnimation( Animation* animation ) @@ -480,18 +490,7 @@ void UpdateManager::AddPropertyNotification( PropertyNotification* propertyNotif void UpdateManager::RemovePropertyNotification( PropertyNotification* propertyNotification ) { - PropertyNotificationContainer &propertyNotifications = mImpl->propertyNotifications; - PropertyNotificationIter iter = propertyNotifications.Begin(); - - while ( iter != propertyNotifications.End() ) - { - if( *iter == propertyNotification ) - { - propertyNotifications.Erase(iter); - break; - } - ++iter; - } + mImpl->propertyNotifications.EraseObject( propertyNotification ); } void UpdateManager::PropertyNotificationSetNotify( PropertyNotification* propertyNotification, PropertyNotification::NotifyMode notifyMode ) @@ -500,52 +499,17 @@ void UpdateManager::PropertyNotificationSetNotify( PropertyNotification* propert propertyNotification->SetNotifyMode( notifyMode ); } -ObjectOwnerContainer& UpdateManager::GetRendererOwner() -{ - return mImpl->renderers; -} - void UpdateManager::AddShader( Shader* shader ) { DALI_ASSERT_DEBUG( NULL != shader ); - if( mImpl->shaders.Count() == 0 ) - { - // the first added shader becomes our default shader - // Construct message in the render queue memory; note that delete should not be called on the return value - typedef MessageValue1< RenderManager, Shader* > DerivedType; - - // Reserve some memory inside the render queue - unsigned int* slot = mImpl->renderQueue.ReserveMessageSlot( mSceneGraphBuffers.GetUpdateBufferIndex(), sizeof( DerivedType ) ); - - // Construct message in the render queue memory; note that delete should not be called on the return value - new (slot) DerivedType( &mImpl->renderManager, &RenderManager::SetDefaultShader, shader ); - } - mImpl->shaders.PushBack( shader ); } void UpdateManager::RemoveShader( Shader* shader ) { - DALI_ASSERT_DEBUG(shader != NULL); - - ShaderContainer& shaders = mImpl->shaders; - // Find the shader and destroy it - for ( ShaderIter iter = shaders.Begin(); iter != shaders.End(); ++iter ) - { - Shader& current = **iter; - if ( ¤t == shader ) - { - // Transfer ownership to the discard queue - // This keeps the shader alive, until the render-thread has finished with it - mImpl->discardQueue.Add( mSceneGraphBuffers.GetUpdateBufferIndex(), shaders.Release( iter ) ); - - return; - } - } - // Should not reach here - DALI_ASSERT_DEBUG(false); + EraseUsingDiscardQueue( mImpl->shaders, shader, mImpl->discardQueue, mSceneGraphBuffers.GetUpdateBufferIndex() ); } void UpdateManager::SetShaderProgram( Shader* shader, @@ -575,67 +539,58 @@ void UpdateManager::SaveBinary( Internal::ShaderDataPtr shaderData ) } } -RenderTaskList* UpdateManager::GetRenderTaskList( bool systemLevel ) +void UpdateManager::SetShaderSaver( ShaderSaver& upstream ) { - if ( !systemLevel ) - { - // copy the list, this is only likely to happen once in application life cycle - return &(mImpl->taskList); - } - else - { - // copy the list, this is only likely to happen once in application life cycle - return &(mImpl->systemLevelTaskList); - } + mImpl->shaderSaver = &upstream; } -void UpdateManager::AddGesture( PanGesture* gesture ) +void UpdateManager::AddRenderer( Renderer* renderer ) { - DALI_ASSERT_DEBUG( NULL != gesture ); + DALI_ASSERT_DEBUG( renderer != NULL ); - mImpl->gestures.PushBack( gesture ); + mImpl->renderers.PushBack( renderer ); + + renderer->ConnectToSceneGraph( *mImpl->sceneController, mSceneGraphBuffers.GetUpdateBufferIndex() ); } -void UpdateManager::RemoveGesture( PanGesture* gesture ) +void UpdateManager::RemoveRenderer( Renderer* renderer ) { - DALI_ASSERT_DEBUG( gesture != NULL ); + // Find the renderer and destroy it + EraseUsingDiscardQueue( mImpl->renderers, renderer, mImpl->discardQueue, mSceneGraphBuffers.GetUpdateBufferIndex() ); + // Need to remove the render object as well + renderer->DisconnectFromSceneGraph( *mImpl->sceneController, mSceneGraphBuffers.GetUpdateBufferIndex() ); +} - GestureContainer& gestures = mImpl->gestures; +void UpdateManager::SetPanGestureProcessor( PanGesture* panGestureProcessor ) +{ + DALI_ASSERT_DEBUG( NULL != panGestureProcessor ); - // Find the gesture and destroy it - for ( GestureIter iter = gestures.Begin(), endIter = gestures.End(); iter != endIter; ++iter ) - { - PanGesture& current = **iter; - if ( ¤t == gesture ) - { - mImpl->gestures.Erase( iter ); - return; - } - } - // Should not reach here - DALI_ASSERT_DEBUG(false); + mImpl->panGestureProcessor = panGestureProcessor; } void UpdateManager::AddTextureSet( TextureSet* textureSet ) { DALI_ASSERT_DEBUG( NULL != textureSet ); + mImpl->textureSets.PushBack( textureSet ); } void UpdateManager::RemoveTextureSet( TextureSet* textureSet ) { - DALI_ASSERT_DEBUG(textureSet != NULL); - size_t textureSetCount( mImpl->textureSets.Size() ); - for( size_t i(0); itextureSets[i] ) - { - mImpl->textureSets.Remove( mImpl->textureSets.Begin() + i ); + mImpl->textureSets.EraseObject( textureSet ); +} - // Update manager has ownership of the TextureSet - delete textureSet; - return; - } +RenderTaskList* UpdateManager::GetRenderTaskList( bool systemLevel ) +{ + if ( !systemLevel ) + { + // copy the list, this is only likely to happen once in application life cycle + return &(mImpl->taskList); + } + else + { + // copy the list, this is only likely to happen once in application life cycle + return &(mImpl->systemLevelTaskList); } } @@ -674,54 +629,36 @@ void UpdateManager::ResetProperties( BufferIndex bufferIndex ) // Reset all the nodes Vector::Iterator iter = mImpl->nodes.Begin()+1; Vector::Iterator endIter = mImpl->nodes.End(); - for(;iter != endIter; ++iter) + for( ;iter != endIter; ++iter ) { (*iter)->ResetToBaseValues( bufferIndex ); } // Reset system-level render-task list properties to base values - const RenderTaskList::RenderTaskContainer& systemLevelTasks = mImpl->systemLevelTaskList.GetTasks(); - - for (RenderTaskList::RenderTaskContainer::ConstIterator iter = systemLevelTasks.Begin(); iter != systemLevelTasks.End(); ++iter) - { - (*iter)->ResetToBaseValues( bufferIndex ); - } + ResetToBaseValues( mImpl->systemLevelTaskList.GetTasks(), bufferIndex ); // Reset render-task list properties to base values. - const RenderTaskList::RenderTaskContainer& tasks = mImpl->taskList.GetTasks(); - - for (RenderTaskList::RenderTaskContainer::ConstIterator iter = tasks.Begin(); iter != tasks.End(); ++iter) - { - (*iter)->ResetToBaseValues( bufferIndex ); - } + ResetToBaseValues( mImpl->taskList.GetTasks(), bufferIndex ); // Reset custom object properties to base values - for (OwnerContainer::Iterator iter = mImpl->customObjects.Begin(); iter != mImpl->customObjects.End(); ++iter) - { - (*iter)->ResetToBaseValues( bufferIndex ); - } + ResetToBaseValues( mImpl->customObjects, bufferIndex ); - mImpl->renderers.ResetToBaseValues( bufferIndex ); + // Reset animatable renderer properties to base values + ResetToBaseValues( mImpl->renderers, bufferIndex ); // Reset animatable shader properties to base values - for (ShaderIter iter = mImpl->shaders.Begin(); iter != mImpl->shaders.End(); ++iter) - { - (*iter)->ResetToBaseValues( bufferIndex ); - } + ResetToBaseValues( mImpl->shaders, bufferIndex ); } bool UpdateManager::ProcessGestures( BufferIndex bufferIndex, unsigned int lastVSyncTimeMilliseconds, unsigned int nextVSyncTimeMilliseconds ) { bool gestureUpdated( false ); - // constrain gestures... (in construction order) - GestureContainer& gestures = mImpl->gestures; - - for ( GestureIter iter = gestures.Begin(), endIter = gestures.End(); iter != endIter; ++iter ) + if( mImpl->panGestureProcessor ) { - PanGesture& gesture = **iter; - gesture.ResetToBaseValues( bufferIndex ); // Needs to be done every time as gesture data is written directly to an update-buffer rather than via a message - gestureUpdated |= gesture.UpdateProperties( lastVSyncTimeMilliseconds, nextVSyncTimeMilliseconds ); + // gesture processor only supports default properties + mImpl->panGestureProcessor->ResetDefaultProperties( bufferIndex ); // Needs to be done every time as gesture data is written directly to an update-buffer rather than via a message + gestureUpdated |= mImpl->panGestureProcessor->UpdateProperties( lastVSyncTimeMilliseconds, nextVSyncTimeMilliseconds ); } return gestureUpdated; @@ -795,7 +732,7 @@ void UpdateManager::ConstrainRenderTasks( BufferIndex bufferIndex ) void UpdateManager::ConstrainShaders( BufferIndex bufferIndex ) { // constrain shaders... (in construction order) - ShaderContainer& shaders = mImpl->shaders; + ShaderOwner& shaders = mImpl->shaders; for ( ShaderIter iter = shaders.Begin(); iter != shaders.End(); ++iter ) { Shader& shader = **iter; @@ -849,14 +786,13 @@ void UpdateManager::ForwardCompiledShadersToEventThread() void UpdateManager::UpdateRenderers( BufferIndex bufferIndex ) { - const OwnerContainer& rendererContainer( mImpl->renderers.GetObjectContainer() ); - unsigned int rendererCount( rendererContainer.Size() ); - for( unsigned int i(0); irenderers.Count(); + for( unsigned int i = 0; i < rendererCount; ++i ) { //Apply constraints - ConstrainPropertyOwner( *rendererContainer[i], bufferIndex ); + ConstrainPropertyOwner( *mImpl->renderers[i], bufferIndex ); - rendererContainer[i]->PrepareRender( bufferIndex ); + mImpl->renderers[i]->PrepareRender( bufferIndex ); } } @@ -895,7 +831,7 @@ unsigned int UpdateManager::Update( float elapsedSeconds, //Process Touches & Gestures const bool gestureUpdated = ProcessGestures( bufferIndex, lastVSyncTimeMilliseconds, nextVSyncTimeMilliseconds ); - const bool updateScene = // The scene-graph requires an update if.. + bool updateScene = // The scene-graph requires an update if.. (mImpl->nodeDirtyFlags & RenderableUpdateFlags) || // ..nodes were dirty in previous frame OR IsAnimationRunning() || // ..at least one animation is running OR mImpl->messageQueue.IsSceneUpdateRequired() || // ..a message that modifies the scene graph node tree is queued OR @@ -911,8 +847,10 @@ unsigned int UpdateManager::Update( float elapsedSeconds, mImpl->transformManager.ResetToBaseValue(); } - //Process the queued scene messages - mImpl->messageQueue.ProcessMessages( bufferIndex ); + // Process the queued scene messages. Note, MessageQueue::FlushQueue may be called + // between calling IsSceneUpdateRequired() above and here, so updateScene should + // be set again + updateScene |= mImpl->messageQueue.ProcessMessages( bufferIndex ); //Forward compiled shader programs to event thread for saving ForwardCompiledShadersToEventThread(); @@ -1103,9 +1041,19 @@ void UpdateManager::SetLayerDepths( const SortedLayerPointers& layers, bool syst } } -void UpdateManager::SetShaderSaver( ShaderSaver& upstream ) +void UpdateManager::SetDepthIndices( NodeDepths* nodeDepths ) { - mImpl->shaderSaver = &upstream; + if( nodeDepths ) + { + // note,this vector is already in depth order. It could be used as-is to + // remove sorting in update algorithm. However, it lacks layer boundary markers. + for( std::vector::iterator iter = nodeDepths->nodeDepths.begin(), + end = nodeDepths->nodeDepths.end() ; + iter != end ; ++iter ) + { + iter->node->SetDepthIndex( iter->sortedDepth ); + } + } } void UpdateManager::AddSampler( Render::Sampler* sampler )