commit 25301b603159d19eaadc3ceb5d6cc52de6a95d20
author Jamie Madill <jmadill@chromium.org> Sat Oct 28 20:59:31 2017 -0400
committer Commit Bot <commit-bot@chromium.org> Wed Nov 01 18:12:41 2017 +0000
tree c7ba9436eb8617d207b01ff2ace97332a07dbf2c
parent 4751aabb7614b9a4057db9409def5e91eb4cb275

Remove WrappedObject::retain.

With resources being mostly deleted by the Renderer along with a
serial number, the retain move semantics weren't very useful.

Refactoring change only.

BUG=angleproject:2200

Change-Id: I7b72b1decfa7604cdd767e7d9b5213b9383eb240
Reviewed-on: https://chromium-review.googlesource.com/742369
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Jamie Madill <jmadill@chromium.org>

src/libANGLE/renderer/vulkan/FramebufferVk.cpp

diff --git a/src/libANGLE/renderer/vulkan/FramebufferVk.cpp b/src/libANGLE/renderer/vulkan/FramebufferVk.cpp
index 3220747..c868ce9 100644
--- a/src/libANGLE/renderer/vulkan/FramebufferVk.cpp
+++ b/src/libANGLE/renderer/vulkan/FramebufferVk.cpp
@@ -479,10 +479,7 @@
     renderPassInfo.dependencyCount = 0;
     renderPassInfo.pDependencies   = nullptr;
 
-    vk::RenderPass renderPass;
-    ANGLE_TRY(renderPass.init(device, renderPassInfo));
-
-    mRenderPass.retain(device, std::move(renderPass));
+    ANGLE_TRY(mRenderPass.init(device, renderPassInfo));
 
     return &mRenderPass;
 }
@@ -549,10 +546,7 @@
     framebufferInfo.height          = static_cast<uint32_t>(attachmentsSize.height);
     framebufferInfo.layers          = 1;
 
-    vk::Framebuffer framebuffer;
-    ANGLE_TRY(framebuffer.init(device, framebufferInfo));
-
-    mFramebuffer.retain(device, std::move(framebuffer));
+    ANGLE_TRY(mFramebuffer.init(device, framebufferInfo));
 
     return &mFramebuffer;
 }