[3.0] (FrameBufferImage) Store ReleasePolicy to satisfy Tizen API policy

Only store the ReleasePolicy set as it's required satisfy Tizen API policy. However, it is not
used internally.

Change-Id: I768f7871d8110476a232f794acb787ed02e3a81b

diff --git a/automated-tests/src/dali/utc-Dali-FrameBufferImage.cpp b/automated-tests/src/dali/utc-Dali-FrameBufferImage.cpp
index b79619a..1f2b880 100644 (file)
--- a/automated-tests/src/dali/utc-Dali-FrameBufferImage.cpp
+++ b/automated-tests/src/dali/utc-Dali-FrameBufferImage.cpp
@@ -111,8 +111,8 @@ int UtcDaliFrameBufferImageNew03(void)
 
   DALI_TEST_CHECK( image );
 
-  //ReleasePolicy is always never for framebuffer images
-  DALI_TEST_EQUALS( image.GetReleasePolicy(), Image::NEVER, TEST_LOCATION );
+  // ReleasePolicy is deprecated. Only set, but not used internally.
+  DALI_TEST_EQUALS( image.GetReleasePolicy(), Image::UNUSED, TEST_LOCATION );
 
   // initialise handle with NEVER release policy
   image.Reset();
@@ -122,7 +122,7 @@ int UtcDaliFrameBufferImageNew03(void)
 
   DALI_TEST_CHECK( image );
 
-  //ReleasePolicy is always never for framebuffer images
+  // ReleasePolicy is deprecated. Only set, but not used internally.
   DALI_TEST_EQUALS( image.GetReleasePolicy(), Image::NEVER, TEST_LOCATION );
 
   END_TEST;

diff --git a/dali/internal/event/images/frame-buffer-image-impl.cpp b/dali/internal/event/images/frame-buffer-image-impl.cpp
index b7fda19..734aee0 100644 (file)
--- a/dali/internal/event/images/frame-buffer-image-impl.cpp
+++ b/dali/internal/event/images/frame-buffer-image-impl.cpp
@@ -122,6 +122,12 @@ bool FrameBufferImage::IsNativeFbo() const
 {
   return mIsNativeFbo;
 }
+
+void FrameBufferImage::SetReleasePolicy( Dali::Image::ReleasePolicy releasePolicy )
+{
+  mReleasePolicy = releasePolicy;
+}
+
 FrameBufferImage::~FrameBufferImage()
 {
 }

diff --git a/dali/internal/event/images/frame-buffer-image-impl.h b/dali/internal/event/images/frame-buffer-image-impl.h
index 42d5872..ffb778d 100644 (file)
--- a/dali/internal/event/images/frame-buffer-image-impl.h
+++ b/dali/internal/event/images/frame-buffer-image-impl.h
@@ -63,6 +63,12 @@ public:
    */
   bool IsNativeFbo() const;
 
+  /**
+   * @brief Required for backwards compatibility. Value is just stored but not used internally.
+   * @param[in] releasePolicy The release policy (ignored as not supported).
+   */
+  void SetReleasePolicy( Dali::Image::ReleasePolicy releasePolicy );
+
 public: // From Image
   /**
    * @copydoc Dali::Internal::Image::Connect()

diff --git a/dali/public-api/images/frame-buffer-image.cpp b/dali/public-api/images/frame-buffer-image.cpp
index e63aeda..028217d 100644 (file)
--- a/dali/public-api/images/frame-buffer-image.cpp
+++ b/dali/public-api/images/frame-buffer-image.cpp
@@ -63,7 +63,7 @@ FrameBufferImage FrameBufferImage::New( unsigned int width, unsigned int height,
   return FrameBufferImage(internal.Get());
 }
 
-FrameBufferImage FrameBufferImage::New( unsigned int width, unsigned int height, Pixel::Format pixelformat, ReleasePolicy /*releasePolicy*/, RenderBuffer::Format bufferformat )
+FrameBufferImage FrameBufferImage::New( unsigned int width, unsigned int height, Pixel::Format pixelformat, ReleasePolicy releasePolicy, RenderBuffer::Format bufferformat )
 {
   Dali::Vector2 stageSize = Stage::GetCurrent().GetSize();
   Internal::FrameBufferImagePtr internal = Internal::FrameBufferImage::New(
@@ -72,12 +72,14 @@ FrameBufferImage FrameBufferImage::New( unsigned int width, unsigned int height,
     pixelformat,
     bufferformat);
 
+  internal->SetReleasePolicy( releasePolicy );
   return FrameBufferImage(internal.Get());
 }
 
-FrameBufferImage FrameBufferImage::New( NativeImageInterface& image, ReleasePolicy /*releasePolicy*/ )
+FrameBufferImage FrameBufferImage::New( NativeImageInterface& image, ReleasePolicy releasePolicy )
 {
   Internal::FrameBufferImagePtr internal = Internal::FrameBufferImage::New( image );
+  internal->SetReleasePolicy( releasePolicy );
   return FrameBufferImage(internal.Get());
 }