Level up Dali::Internal::FreeList as Dali::FreeList
[platform/core/uifw/dali-core.git] / dali / internal / update / render-tasks / scene-graph-render-task.h
index 698abe6..7a28016 100644 (file)
@@ -1,8 +1,8 @@
-#ifndef __DALI_INTERNAL_SCENE_GRAPH_RENDER_TASK_H__
-#define __DALI_INTERNAL_SCENE_GRAPH_RENDER_TASK_H__
+#ifndef DALI_INTERNAL_SCENE_GRAPH_RENDER_TASK_H
+#define DALI_INTERNAL_SCENE_GRAPH_RENDER_TASK_H
 
 /*
- * Copyright (c) 2014 Samsung Electronics Co., Ltd.
+ * Copyright (c) 2021 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.
  */
 
 // INTERNAL INCLUDES
-#include <dali/public-api/math/viewport.h>
-#include <dali/public-api/render-tasks/render-task.h>
 #include <dali/internal/common/buffer-index.h>
 #include <dali/internal/common/message.h>
 #include <dali/internal/event/common/event-thread-services.h>
-#include <dali/internal/update/common/property-owner.h>
+#include <dali/internal/render/common/render-instruction.h>
+#include <dali/internal/render/renderers/render-frame-buffer.h>
 #include <dali/internal/update/common/animatable-property.h>
+#include <dali/internal/update/common/property-owner.h>
+#include <dali/public-api/math/viewport.h>
+#include <dali/public-api/render-tasks/render-task.h>
 
 namespace Dali
 {
-
 namespace Internal
 {
-class FrameBufferTexture;
-class ResourceManager;
-
 namespace Render
 {
 class RenderTracker;
@@ -43,18 +41,17 @@ class RenderTracker;
 namespace SceneGraph
 {
 class Node;
-class CameraAttachment;
+class Camera;
 class RenderInstruction;
 class RenderMessageDispatcher;
 
 /**
  * RenderTasks describe how the Dali scene should be rendered.
  */
-class RenderTask : public PropertyOwner
+class RenderTask : public PropertyOwner, public PropertyOwner::Observer
 {
 public:
-
-  enum State
+  enum State : uint8_t
   {
     RENDER_CONTINUOUSLY,               ///< mRefreshRate > 0
     RENDER_ONCE_WAITING_FOR_RESOURCES, ///< mRefreshRate = REFRESH_ONCE
@@ -70,20 +67,19 @@ public:
   /**
    * Virtual destructor
    */
-  virtual ~RenderTask();
+  ~RenderTask() override;
 
   /**
    * Initialize the render task. Called in update thread
    * @param[in] renderMessageDispatcher to send messages to render thread
-   * @param[in] resourceManager to check and update status of FBOs
    */
-  void Initialize( RenderMessageDispatcher& renderMessageDispatcher, ResourceManager& resourceManager );
+  void Initialize(RenderMessageDispatcher& renderMessageDispatcher);
 
   /**
    * Set the nodes to be rendered.
    * @param[in] node This node and its children will be rendered.
    */
-  void SetSourceNode( Node* node );
+  void SetSourceNode(Node* node);
 
   /**
    * Retrieve the source node.
@@ -95,7 +91,7 @@ public:
    * Set whether the RenderTask has exclusive access to the source nodes.
    * @param[in] exclusive True if the source nodes will only be rendered by this render-task.
    */
-  void SetExclusive( bool exclusive );
+  void SetExclusive(bool exclusive);
 
   /**
    * Query whether the RenderTask has exclusive access to the source actors.
@@ -104,23 +100,23 @@ public:
   bool IsExclusive() const;
 
   /**
-   * Set the node from which the scene is viewed.
-   * @param[in] node The scene is viewed from the perspective of this node.
+   * Set the camera from which the scene is viewed.
+   * @param[in] cameraNode that camera is connected with
+   * @param[in] camera to use.
    */
-  void SetCameraNode( Node* node );
+  void SetCamera(Node* cameraNode, Camera* camera);
 
   /**
    * Set the frame-buffer used as a render target.
-   * @param[in] resourceId The resource ID of the frame-buffer, or zero if not rendering off-screen.
-   * @param[in] isNativeFBO if this render task is targeting a native FBO
+   * @param[in] frameBuffer The framebuffer
    */
-  void SetFrameBufferId( unsigned int resourceId, bool isNativeFBO );
+  void SetFrameBuffer(Render::FrameBuffer* frameBuffer);
 
   /**
    * Retrieve the resource ID of the frame-buffer.
    * @return The resource ID, or zero if not rendering off-screen.
    */
-  unsigned int GetFrameBufferId() const;
+  Render::FrameBuffer* GetFrameBuffer();
 
   /**
    * Set the value of property viewportPosition
@@ -128,7 +124,7 @@ public:
    * @param[in] updateBufferIndex The current update buffer index.
    * @param[in] value The value of the property
    */
-  void SetViewportPosition( BufferIndex updateBufferIndex, const Vector2& value );
+  void SetViewportPosition(BufferIndex updateBufferIndex, const Vector2& value);
 
   /**
    * Get the value of property viewportPosition
@@ -136,7 +132,7 @@ public:
    * @param[in] bufferIndex The buffer to read from.
    * @return the value of the property.
    */
-  const Vector2& GetViewportPosition( BufferIndex bufferIndex ) const;
+  const Vector2& GetViewportPosition(BufferIndex bufferIndex) const;
 
   /**
    * Bake the value of the property viewportPosition
@@ -144,7 +140,7 @@ public:
    * @param[in] updateBufferIndex The current update buffer index.
    * @param[in] value The new value for property.
    */
-  void BakeViewportPosition( BufferIndex updateBufferIndex, const Vector2& value );
+  void BakeViewportPosition(BufferIndex updateBufferIndex, const Vector2& value);
 
   /**
    * Set the value of property viewportSize
@@ -152,7 +148,7 @@ public:
    * @param[in] updateBufferIndex The current update buffer index.
    * @param[in] value The value of the property
    */
-  void SetViewportSize( BufferIndex updateBufferIndex, const Vector2& value );
+  void SetViewportSize(BufferIndex updateBufferIndex, const Vector2& value);
 
   /**
    * Get the value of property viewportSize
@@ -160,7 +156,7 @@ public:
    * @param[in] bufferIndex The buffer to read from.
    * @return the value of the property.
    */
-  const Vector2& GetViewportSize( BufferIndex bufferIndex ) const;
+  const Vector2& GetViewportSize(BufferIndex bufferIndex) const;
 
   /**
    * Bake the value of the property viewportSize
@@ -168,7 +164,7 @@ public:
    * @param[in] updateBufferIndex The current update buffer index.
    * @param[in] value The new value for property.
    */
-  void BakeViewportSize( BufferIndex updateBufferIndex, const Vector2& value );
+  void BakeViewportSize(BufferIndex updateBufferIndex, const Vector2& value);
 
   /**
    * Get the value of property viewportEnabled
@@ -176,7 +172,7 @@ public:
    * @param[in] bufferIndex The buffer to read from.
    * @return the value of the property.
    */
-  bool GetViewportEnabled( BufferIndex bufferIndex ) const;
+  bool GetViewportEnabled(BufferIndex bufferIndex) const;
 
   /**
    * Query whether the optional viewport is set.
@@ -184,7 +180,7 @@ public:
    * @param[out] viewport The viewport position and size is populated.
    * @return true if the viewport has been set
    */
-  bool QueryViewport( BufferIndex bufferIndex, Viewport& viewport ) const;
+  bool QueryViewport(BufferIndex bufferIndex, Viewport& viewport) const;
 
   /**
    * Set the value of property clearColor
@@ -192,7 +188,7 @@ public:
    * @param[in] updateBufferIndex The current update buffer index.
    * @param[in] value The value of the property
    */
-  void SetClearColor( BufferIndex updateBufferIndex, const Vector4& value );
+  void SetClearColor(BufferIndex updateBufferIndex, const Vector4& value);
 
   /**
    * Get the value of property clearColor
@@ -200,7 +196,7 @@ public:
    * @param[in] bufferIndex The buffer to read from.
    * @return the value of the property.
    */
-  const Vector4& GetClearColor( BufferIndex bufferIndex ) const;
+  const Vector4& GetClearColor(BufferIndex bufferIndex) const;
 
   /**
    * Bake the value of the property clearColor
@@ -208,12 +204,12 @@ public:
    * @param[in] updateBufferIndex The current update buffer index.
    * @param[in] value The new value for property.
    */
-  void BakeClearColor( BufferIndex updateBufferIndex, const Vector4& value );
+  void BakeClearColor(BufferIndex updateBufferIndex, const Vector4& value);
 
   /**
    * @copydoc Dali::RenderTask::SetClearEnabled()
    */
-  void SetClearEnabled( bool enabled );
+  void SetClearEnabled(bool enabled);
 
   /**
    * @copydoc Dali::RenderTask::GetClearEnabled()
@@ -223,7 +219,7 @@ public:
   /**
    * @copydoc Dali::RenderTask::SetCullMode()
    */
-  void SetCullMode( bool mode );
+  void SetCullMode(bool mode);
 
   /**
    * @copydoc Dali::RenderTask::GetCullMode()
@@ -234,20 +230,20 @@ public:
    * Set the refresh-rate of the RenderTask.
    * @param[in] refreshRate The new refresh rate.
    */
-  void SetRefreshRate( unsigned int refreshRate );
+  void SetRefreshRate(uint32_t refreshRate);
 
   /**
    * Retrieve the refresh-rate of the RenderTask.
    * @return The refresh rate.
    */
-  unsigned int GetRefreshRate() const;
+  uint32_t GetRefreshRate() const;
 
   /**
    * Check if the render task is ready for rendering.
    * @param[in] updateBufferIndex The current update buffer index.
    * @return True if the render-task is ready for rendering.
    */
-  bool ReadyToRender( BufferIndex updateBufferIndex );
+  bool ReadyToRender(BufferIndex updateBufferIndex);
 
   /**
    * True if a render is required. If the current state is RENDER_CONTINUOUSLY, then
@@ -257,12 +253,6 @@ public:
   bool IsRenderRequired();
 
   /**
-   * Set whether all resources were available when the render-task was processed
-   * @param[in] resourcesComplete True if the resources of the source tree are completely loaded.
-   */
-  void SetResourcesFinished( bool resourcesFinished );
-
-  /**
    * Process a frame. This method is called each frame for every ready render task, regardless
    * of whether it needs to render (so that the frame counter can be updated).
    */
@@ -285,31 +275,31 @@ public:
   /**
    * @return The number of times we have transited from RENDERED_ONCE to RENDERED_ONCE_AND_NOTIFIED state.
    */
-  unsigned int GetRenderedOnceCounter() const;
+  uint32_t GetRenderedOnceCounter() const;
 
   /**
    * Retrieve the view-matrix; this is double buffered for input handling.
-   * @pre GetCameraNode() returns a node with valid CameraAttachment.
+   * @pre GetCameraNode() returns a node with valid Camera.
    * @param[in] bufferIndex The buffer to read from.
    * @return The view-matrix.
    */
-  const Matrix& GetViewMatrix( BufferIndex bufferIndex ) const;
+  const Matrix& GetViewMatrix(BufferIndex bufferIndex) const;
 
   /**
-   * @brief Retrieve the camera attachment.
-   * @pre GetCameraNode() returns a node with valid CameraAttachment.
+   * @brief Retrieve the camera.
+   * @pre GetCameraNode() returns a node with valid Camera.
    *
-   * @return The camera attachment.
+   * @return The camera.
    */
-  SceneGraph::CameraAttachment& GetCameraAttachment() const;
+  SceneGraph::Camera& GetCamera() const;
 
   /**
    * Retrieve the projection-matrix; this is double buffered for input handling.
-   * @pre GetCameraNode() returns a node with valid CameraAttachment.
+   * @pre GetCameraNode() returns a node with valid Camera.
    * @param[in] bufferIndex The buffer to read from.
    * @return The projection-matrix.
    */
-  const Matrix& GetProjectionMatrix( BufferIndex bufferIndex ) const;
+  const Matrix& GetProjectionMatrix(BufferIndex bufferIndex) const;
 
   /**
    * Prepares the render-instruction buffer to be populated with instructions.
@@ -318,10 +308,10 @@ public:
    * then this method will ensure that a GL sync object is created to track
    * when the rendering has finished.
    *
-   * @param[out] instruction to prepare
    * @param[in] updateBufferIndex The current update buffer index.
+   * @return instruction to prepare
    */
-  void PrepareRenderInstruction( RenderInstruction& instruction, BufferIndex updateBufferIndex );
+  RenderInstruction& PrepareRenderInstruction(BufferIndex updateBufferIndex);
 
   /**
    * @return true if the view matrix has been updated during this or last frame
@@ -329,187 +319,215 @@ public:
   bool ViewMatrixUpdated();
 
   /**
-   * @return A pointer to the camera used by the RenderTask
+   * Indicate whether GL sync is required for native render target.
+   * @param[in] requiresSync whether GL sync is required for native render target
    */
-  Node* GetCamera() const;
+  void SetSyncRequired(bool requiresSync);
 
-private:
+  /**
+   * Retrieve the render instruction.
+   * @param[in] updateBufferIndex The current update buffer index.
+   * @return The render instruction
+   */
+  RenderInstruction& GetRenderInstruction(BufferIndex updateBufferIndex)
+  {
+    return mRenderInstruction[updateBufferIndex];
+  }
 
+private: // from PropertyOwner::Observer
   /**
-   * Protected constructor.
+   * @copydoc PropertyOwner::Observer::PropertyOwnerConnected( PropertyOwner& owner )
    */
-  RenderTask();
+  void PropertyOwnerConnected(PropertyOwner& owner) override;
 
-  // Undefined
-  RenderTask(const RenderTask&);
+  /**
+   * @copydoc PropertyOwner::Observer::PropertyOwnerDisconnected( BufferIndex updateBufferIndex, PropertyOwner& owner )
+   */
+  void PropertyOwnerDisconnected(BufferIndex updateBufferIndex, PropertyOwner& owner) override;
 
-  // Undefined
-  RenderTask& operator=(const RenderTask&);
+  /**
+   * @copydoc PropertyOwner::Observer::PropertyOwnerDestroyed( PropertyOwner& owner )
+   */
+  void PropertyOwnerDestroyed(PropertyOwner& owner) override;
 
-private: // PropertyOwner
+private:
+  void SetActiveStatus();
 
-  virtual void ResetDefaultProperties( BufferIndex currentBufferIndex );
+  /**
+   * Constructor.
+   */
+  RenderTask();
 
-public: // Animatable Properties
-  AnimatableProperty< Vector2 >   mViewportPosition;    ///< viewportPosition
-  AnimatableProperty< Vector2 >   mViewportSize;        ///< viewportSize
-  AnimatableProperty< Vector4 >   mClearColor;          ///< clearColor
+  // Undefined
+  RenderTask(const RenderTask&) = delete;
+  RenderTask& operator=(const RenderTask&) = delete;
+
+public:                                          // Animatable Properties
+  AnimatableProperty<Vector2> mViewportPosition; ///< viewportPosition
+  AnimatableProperty<Vector2> mViewportSize;     ///< viewportSize
+  AnimatableProperty<Vector4> mClearColor;       ///< clearColor
 
 private:
   RenderMessageDispatcher* mRenderMessageDispatcher;
-  ResourceManager* mResourceManager;
-  Render::RenderTracker* mRenderSyncTracker;
-  Node* mSourceNode;
-  Node* mCameraNode;
-  CameraAttachment* mCameraAttachment;
-  unsigned int mFrameBufferResourceId;
-
-  bool mResourcesFinished:1; ///< True if all resources were available when the render-task was processed
-  bool mWaitingToRender:1; ///< True when an render once to FBO is waiting
-  bool mNotifyTrigger:1; ///< True if a render once render task has finished renderering
-  bool mExclusive: 1; ///< Whether the render task has exclusive access to the source actor (node in the scene graph implementation).
-  bool mClearEnabled: 1; ///< Whether previous results are cleared.
-  bool mCullMode: 1; ///< Whether renderers should be frustum culled
-
-  FrameBufferTexture* mRenderTarget;
-  Viewport mViewport;
-
-  State mState;                     ///< Render state.
-  unsigned int mRefreshRate;        ///< REFRESH_ONCE, REFRESH_ALWAYS or render every N frames
-  unsigned int mFrameCounter;       ///< counter for rendering every N frames
-
-  unsigned int mRenderedOnceCounter;  ///< Incremented whenever state changes to RENDERED_ONCE_AND_NOTIFIED
-  bool mTargetIsNativeFramebuffer; ///< Tells if our target is a native framebuffer
-
+  Render::RenderTracker*   mRenderSyncTracker;
+  Node*                    mSourceNode;
+  Node*                    mCameraNode;
+  SceneGraph::Camera*      mCamera;
+  Render::FrameBuffer*     mFrameBuffer;
+
+  RenderInstruction mRenderInstruction[2]; ///< Owned double buffered render instruction. (Double buffered because this owns render commands for the currently drawn frame)
+
+  uint32_t mRefreshRate;         ///< REFRESH_ONCE, REFRESH_ALWAYS or render every N frames
+  uint32_t mFrameCounter;        ///< counter for rendering every N frames
+  uint32_t mRenderedOnceCounter; ///< Incremented whenever state changes to RENDERED_ONCE_AND_NOTIFIED
+
+  State mState; ///< Render state.
+
+  bool mRequiresSync : 1;    ///< Whether sync is needed to track the render
+  bool mActive : 1;          ///< True when the task is active, i.e. has valid source and camera
+  bool mWaitingToRender : 1; ///< True when an render once to FBO is waiting
+  bool mNotifyTrigger : 1;   ///< True if a render once render task has finished renderering
+  bool mExclusive : 1;       ///< Whether the render task has exclusive access to the source actor (node in the scene graph).
+  bool mClearEnabled : 1;    ///< Whether previous results are cleared.
+  bool mCullMode : 1;        ///< Whether renderers should be frustum culled
 };
 
 // Messages for RenderTask
-
-inline void SetFrameBufferIdMessage( EventThreadServices& eventThreadServices, RenderTask& task, unsigned int resourceId, bool isNativeFBO )
+inline void SetFrameBufferMessage(EventThreadServices& eventThreadServices, const RenderTask& task, Render::FrameBuffer* frameBuffer)
 {
-  typedef MessageValue2< RenderTask, unsigned int, bool > LocalType;
+  using LocalType = MessageValue1<RenderTask, Render::FrameBuffer*>;
 
   // Reserve some memory inside the message queue
-  unsigned int* slot = eventThreadServices.ReserveMessageSlot( sizeof( LocalType ) );
+  uint32_t* slot = eventThreadServices.ReserveMessageSlot(sizeof(LocalType));
 
   // Construct message in the message queue memory; note that delete should not be called on the return value
-  new (slot) LocalType( &task, &RenderTask::SetFrameBufferId, resourceId, isNativeFBO );
+  new(slot) LocalType(&task, &RenderTask::SetFrameBuffer, frameBuffer);
 }
 
-inline void SetClearColorMessage( EventThreadServices& eventThreadServices, RenderTask& task, const Vector4& value )
+inline void SetClearColorMessage(EventThreadServices& eventThreadServices, const RenderTask& task, const Vector4& value)
 {
-  typedef MessageDoubleBuffered1< RenderTask, Vector4 > LocalType;
+  using LocalType = MessageDoubleBuffered1<RenderTask, Vector4>;
 
   // Reserve some memory inside the message queue
-  unsigned int* slot = eventThreadServices.ReserveMessageSlot( sizeof( LocalType ) );
+  uint32_t* slot = eventThreadServices.ReserveMessageSlot(sizeof(LocalType));
 
   // Construct message in the message queue memory; note that delete should not be called on the return value
-  new (slot) LocalType( &task, &RenderTask::SetClearColor, value );
+  new(slot) LocalType(&task, &RenderTask::SetClearColor, value);
 }
 
-inline void BakeClearColorMessage( EventThreadServices& eventThreadServices, const RenderTask& task, const Vector4& value )
+inline void BakeClearColorMessage(EventThreadServices& eventThreadServices, const RenderTask& task, const Vector4& value)
 {
-  typedef MessageDoubleBuffered1< RenderTask, Vector4 > LocalType;
+  using LocalType = MessageDoubleBuffered1<RenderTask, Vector4>;
 
   // Reserve some memory inside the message queue
-  unsigned int* slot = eventThreadServices.ReserveMessageSlot( sizeof( LocalType ) );
+  uint32_t* slot = eventThreadServices.ReserveMessageSlot(sizeof(LocalType));
 
   // Construct message in the message queue memory; note that delete should not be called on the return value
-  new (slot) LocalType( &task, &RenderTask::BakeClearColor, value );
+  new(slot) LocalType(&task, &RenderTask::BakeClearColor, value);
 }
 
-inline void SetClearEnabledMessage( EventThreadServices& eventThreadServices, RenderTask& task, bool enabled )
+inline void SetClearEnabledMessage(EventThreadServices& eventThreadServices, const RenderTask& task, bool enabled)
 {
-  typedef MessageValue1< RenderTask, bool > LocalType;
+  using LocalType = MessageValue1<RenderTask, bool>;
 
   // Reserve some memory inside the message queue
-  unsigned int* slot = eventThreadServices.ReserveMessageSlot( sizeof( LocalType ) );
+  uint32_t* slot = eventThreadServices.ReserveMessageSlot(sizeof(LocalType));
 
   // Construct message in the message queue memory; note that delete should not be called on the return value
-  new (slot) LocalType( &task, &RenderTask::SetClearEnabled, enabled );
+  new(slot) LocalType(&task, &RenderTask::SetClearEnabled, enabled);
 }
 
-inline void SetCullModeMessage( EventThreadServices& eventThreadServices, RenderTask& task, bool mode )
+inline void SetCullModeMessage(EventThreadServices& eventThreadServices, const RenderTask& task, bool mode)
 {
-  typedef MessageValue1< RenderTask, bool > LocalType;
+  using LocalType = MessageValue1<RenderTask, bool>;
 
   // Reserve some memory inside the message queue
-  unsigned int* slot = eventThreadServices.ReserveMessageSlot( sizeof( LocalType ) );
+  uint32_t* slot = eventThreadServices.ReserveMessageSlot(sizeof(LocalType));
 
   // Construct message in the message queue memory; note that delete should not be called on the return value
-  new (slot) LocalType( &task, &RenderTask::SetCullMode, mode );
+  new(slot) LocalType(&task, &RenderTask::SetCullMode, mode);
 }
 
-inline void SetRefreshRateMessage( EventThreadServices& eventThreadServices, RenderTask& task, unsigned int refreshRate )
+inline void SetRefreshRateMessage(EventThreadServices& eventThreadServices, const RenderTask& task, uint32_t refreshRate)
 {
-  typedef MessageValue1< RenderTask, unsigned int > LocalType;
+  using LocalType = MessageValue1<RenderTask, uint32_t>;
 
   // Reserve some memory inside the message queue
-  unsigned int* slot = eventThreadServices.ReserveMessageSlot( sizeof( LocalType ) );
+  uint32_t* slot = eventThreadServices.ReserveMessageSlot(sizeof(LocalType));
 
   // Construct message in the message queue memory; note that delete should not be called on the return value
-  new (slot) LocalType( &task, &RenderTask::SetRefreshRate, refreshRate );
+  new(slot) LocalType(&task, &RenderTask::SetRefreshRate, refreshRate);
 }
 
-inline void SetSourceNodeMessage( EventThreadServices& eventThreadServices, RenderTask& task, const Node* constNode )
+inline void SetSourceNodeMessage(EventThreadServices& eventThreadServices, const RenderTask& task, const Node* constNode)
 {
   // Scene graph thread can destroy this object.
-  Node* node = const_cast< Node* >( constNode );
+  Node* node = const_cast<Node*>(constNode);
 
-  typedef MessageValue1< RenderTask, Node* > LocalType;
+  using LocalType = MessageValue1<RenderTask, Node*>;
 
   // Reserve some memory inside the message queue
-  unsigned int* slot = eventThreadServices.ReserveMessageSlot( sizeof( LocalType ) );
+  uint32_t* slot = eventThreadServices.ReserveMessageSlot(sizeof(LocalType));
 
   // Construct message in the message queue memory; note that delete should not be called on the return value
-  new (slot) LocalType( &task, &RenderTask::SetSourceNode, node );
+  new(slot) LocalType(&task, &RenderTask::SetSourceNode, node);
 }
 
-inline void SetCameraNodeMessage( EventThreadServices& eventThreadServices, RenderTask& task, const Node* constNode )
+inline void SetCameraMessage(EventThreadServices& eventThreadServices, const RenderTask& task, const Node* constNode, const Camera* constCamera)
 {
-  // Scene graph thread can destroy this object.
-  Node* node = const_cast< Node* >( constNode );
+  using LocalType = MessageValue2<RenderTask, Node*, Camera*>;
+
+  Node*   node   = const_cast<Node*>(constNode);
+  Camera* camera = const_cast<Camera*>(constCamera);
+  // Reserve memory inside the message queue
+  uint32_t* slot = eventThreadServices.ReserveMessageSlot(sizeof(LocalType));
 
-  typedef MessageValue1< RenderTask, Node* > LocalType;
+  // Construct message in the message queue memory; note that delete should not be called on the return value
+  new(slot) LocalType(&task, &RenderTask::SetCamera, node, camera);
+}
+
+inline void SetExclusiveMessage(EventThreadServices& eventThreadServices, const RenderTask& task, bool exclusive)
+{
+  using LocalType = MessageValue1<RenderTask, bool>;
 
   // Reserve some memory inside the message queue
-  unsigned int* slot = eventThreadServices.ReserveMessageSlot( sizeof( LocalType ) );
+  uint32_t* slot = eventThreadServices.ReserveMessageSlot(sizeof(LocalType));
 
   // Construct message in the message queue memory; note that delete should not be called on the return value
-  new (slot) LocalType( &task, &RenderTask::SetCameraNode, node );
+  new(slot) LocalType(&task, &RenderTask::SetExclusive, exclusive);
 }
 
-inline void SetExclusiveMessage( EventThreadServices& eventThreadServices, RenderTask& task, bool exclusive )
+inline void SetSyncRequiredMessage(EventThreadServices& eventThreadServices, const RenderTask& task, bool requiresSync)
 {
-  typedef MessageValue1< RenderTask, bool > LocalType;
+  using LocalType = MessageValue1<RenderTask, bool>;
 
   // Reserve some memory inside the message queue
-  unsigned int* slot = eventThreadServices.ReserveMessageSlot( sizeof( LocalType ) );
+  uint32_t* slot = eventThreadServices.ReserveMessageSlot(sizeof(LocalType));
 
   // Construct message in the message queue memory; note that delete should not be called on the return value
-  new (slot) LocalType( &task, &RenderTask::SetExclusive, exclusive );
+  new(slot) LocalType(&task, &RenderTask::SetSyncRequired, requiresSync);
 }
 
-inline void BakeViewportPositionMessage( EventThreadServices& eventThreadServices, const RenderTask& task, const Vector2& value )
+inline void BakeViewportPositionMessage(EventThreadServices& eventThreadServices, const RenderTask& task, const Vector2& value)
 {
-  typedef MessageDoubleBuffered1< RenderTask, Vector2 > LocalType;
+  using LocalType = MessageDoubleBuffered1<RenderTask, Vector2>;
 
   // Reserve some memory inside the message queue
-  unsigned int* slot = eventThreadServices.ReserveMessageSlot( sizeof( LocalType ) );
+  uint32_t* slot = eventThreadServices.ReserveMessageSlot(sizeof(LocalType));
 
   // Construct message in the message queue memory; note that delete should not be called on the return value
-  new (slot) LocalType( &task, &RenderTask::BakeViewportPosition, value );
+  new(slot) LocalType(&task, &RenderTask::BakeViewportPosition, value);
 }
 
-inline void BakeViewportSizeMessage( EventThreadServices& eventThreadServices, const RenderTask& task, const Vector2& value )
+inline void BakeViewportSizeMessage(EventThreadServices& eventThreadServices, const RenderTask& task, const Vector2& value)
 {
-  typedef MessageDoubleBuffered1< RenderTask, Vector2 > LocalType;
+  using LocalType = MessageDoubleBuffered1<RenderTask, Vector2>;
 
   // Reserve some memory inside the message queue
-  unsigned int* slot = eventThreadServices.ReserveMessageSlot( sizeof( LocalType ) );
+  uint32_t* slot = eventThreadServices.ReserveMessageSlot(sizeof(LocalType));
 
   // Construct message in the message queue memory; note that delete should not be called on the return value
-  new (slot) LocalType( &task, &RenderTask::BakeViewportSize, value );
+  new(slot) LocalType(&task, &RenderTask::BakeViewportSize, value);
 }
 
 } // namespace SceneGraph
@@ -518,4 +536,4 @@ inline void BakeViewportSizeMessage( EventThreadServices& eventThreadServices, c
 
 } // namespace Dali
 
-#endif // __DALI_INTERNAL_SCENE_GRAPH_RENDER_TASK_H__
+#endif // DALI_INTERNAL_SCENE_GRAPH_RENDER_TASK_H