Tweaks to PPB_VarArrayBuffer in preperation for taking out of Dev.

ppapi/tests/test_post_message.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ppapi/tests/test_post_message.h"
 
 #include <algorithm>
 #include <sstream>
 
 #include "ppapi/c/dev/ppb_testing_dev.h"
@@ skipping 245 matching lines @@
   uint32_t sizes[] = { 0, 100, 1000, 10000, 100000 };
   for (size_t i = 0; i < sizeof(sizes)/sizeof(sizes[i]); ++i) {
     std::ostringstream size_stream;
     size_stream << sizes[i];
     const std::string kSizeAsString(size_stream.str());
 
     // Create an appropriately sized array buffer with test_data[i] == i.
     pp::VarArrayBuffer_Dev test_data(sizes[i]);
     if (sizes[i] > 0)
       ASSERT_NE(NULL, test_data.Map());
+    // Make sure we can Unmap/Map successfully (there's not really any way to
+    // detect if it's unmapped, so we just re-map before getting the pointer to
+    // the buffer).
+    test_data.Unmap();
+    test_data.Map();
     ASSERT_EQ(sizes[i], test_data.ByteLength());
     unsigned char* buff = static_cast<unsigned char*>(test_data.Map());
+    test_data.Unmap();
+    test_data.Map();
     const uint32_t kByteLength = test_data.ByteLength();
     for (size_t j = 0; j < kByteLength; ++j)
       buff[j] = static_cast<uint8_t>(j % 256u);
 
     // Have the listener test some properties of the ArrayBuffer.
     std::vector<std::string> properties_to_check;
     properties_to_check.push_back(
         "message_event.data.constructor.name === 'ArrayBuffer'");
     properties_to_check.push_back(
         std::string("message_event.data.byteLength === ") + kSizeAsString);
@@ skipping 199 matching lines @@
     ASSERT_TRUE(received_value <= kThreadsToRun);
     ++received_counts[received_value];
   }
   ASSERT_EQ(received_counts, expected_counts);
 
   message_data_.clear();
   ASSERT_TRUE(ClearListeners());
 
   PASS();
 }