Fix memory leaks with V8 startup data.

testing/embedder_test.cpp

Index: testing/embedder_test.cpp
diff --git a/testing/embedder_test.cpp b/testing/embedder_test.cpp
index 59190a7751de2cca2e9b9f19294688cb0248e0a1..ec05dd421e1b01669ac93325d0bc36eafcb74d6e 100644
--- a/testing/embedder_test.cpp
+++ b/testing/embedder_test.cpp
@@ -62,6 +62,11 @@ EmbedderTest::~EmbedderTest() {
 #ifdef PDF_ENABLE_V8
   v8::V8::ShutdownPlatform();
   delete platform_;
+
+#ifdef V8_USE_EXTERNAL_STARTUP_DATA
+  free(const_cast<char*>(natives_.data));

[Lei Zhang, 2016/06/20 18:57:07]

Not sure why I didn't notice the bot failure earlier / notice the failure
locally, but this doesn't work since v8::V8::SetNativesDataBlob() uses
base::CallOnce() internally. I think we'll have to tolerate at least 1 leak.

+  free(const_cast<char*>(snapshot_.data));
+#endif  // V8_USE_EXTERNAL_STARTUP_DATA
 #endif  // PDF_ENABLE_V8
 }