commit b3f26b9e4e579106209925df2790efede5b7b05b
author Jamie Madill <jmadill@chromium.org> Wed Jul 19 15:07:41 2017 -0400
committer Commit Bot <commit-bot@chromium.org> Wed Jul 19 19:58:23 2017 +0000
tree 575b708649081e6a0eb305541d41f2a916683e80
parent ab918821c28501967fc2aff5738578a6610e72b8

Add a zero-filled scratch buffer to Context.

We need this in a few places for handling resource init.
Centralize this in the context so we don't have to recreate
and re-fill a large zero buffer.

BUG=angleproject:2107

Change-Id: Icf9ce417e7ee3498f03e47741dfff89e81b49519
Reviewed-on: https://chromium-review.googlesource.com/576057
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Commit-Queue: Jamie Madill <jmadill@chromium.org>

src/common/MemoryBuffer.cpp

diff --git a/src/common/MemoryBuffer.cpp b/src/common/MemoryBuffer.cpp
index bc848fa..9ab2728 100644
--- a/src/common/MemoryBuffer.cpp
+++ b/src/common/MemoryBuffer.cpp
@@ -108,6 +108,20 @@
 
 bool ScratchBuffer::get(size_t requestedSize, MemoryBuffer **memoryBufferOut)
 {
+    return getImpl(requestedSize, memoryBufferOut, Optional<uint8_t>::Invalid());
+}
+
+bool ScratchBuffer::getInitialized(size_t requestedSize,
+                                   MemoryBuffer **memoryBufferOut,
+                                   uint8_t initValue)
+{
+    return getImpl(requestedSize, memoryBufferOut, Optional<uint8_t>(initValue));
+}
+
+bool ScratchBuffer::getImpl(size_t requestedSize,
+                            MemoryBuffer **memoryBufferOut,
+                            Optional<uint8_t> initValue)
+{
     if (mScratchMemory.size() == requestedSize)
     {
         mResetCounter    = mLifetime;
@@ -128,6 +142,10 @@
             return false;
         }
         mResetCounter = mLifetime;
+        if (initValue.valid())
+        {
+            mScratchMemory.fill(initValue.value());
+        }
     }
 
     ASSERT(mScratchMemory.size() >= requestedSize);