Cleanup to Resource11's reset method.

Only reset when the resource is valid. This will prevent a memory allocation that could happen when triggering suprious reset() calls. BUG=angleproject:1155 Change-Id: I3b9bc1f9e0542c14ed5fd4a03f82ad23c94b734b Reviewed-on: https://chromium-review.googlesource.com/659231 Commit-Queue: Jamie Madill <jmadill@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org>

Cleanup to Resource11's reset method.
2617eada · Jamie Madill · Commit Bot · 14bbb3f9 · 2617eada
Commit 2617eada authored Sep 12, 2017 by Jamie Madill Committed by Commit Bot Sep 12, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 1 deletion

ResourceManager11.h src/libANGLE/renderer/d3d/d3d11/ResourceManager11.h +5 -1

No files found.
--- a/src/libANGLE/renderer/d3d/d3d11/ResourceManager11.h
+++ b/src/libANGLE/renderer/d3d/d3d11/ResourceManager11.h
@@ -187,7 +187,11 @@ class Resource11Base : angle::NonCopyable

    bool valid() const { return (mData->object != nullptr); }

-    void reset() { mData.reset(new DataT()); }
+    void reset()
+    {
+        if (valid())
+            mData.reset(new DataT());
+    }

    ResourceSerial getSerial() const
    {