[wasm] Do not unregister an ArrayBuffer if it is already external

src/wasm/wasm-js.cc

Index: src/wasm/wasm-js.cc
diff --git a/src/wasm/wasm-js.cc b/src/wasm/wasm-js.cc
index bd371c0c50819ad5def070357e54b9951ea8de11..d5294ce36e11d302498b7c5b443724f0f26b4542 100644
--- a/src/wasm/wasm-js.cc
+++ b/src/wasm/wasm-js.cc
@@ -740,7 +740,8 @@ void WebAssemblyMemoryGrow(const v8::FunctionCallbackInfo<v8::Value>& args) {
     thrower.RangeError("Unable to grow instance memory.");
     return;
   }
-  i::wasm::DetachWebAssemblyMemoryBuffer(i_isolate, old_buffer);
+  bool free_memory = delta_size != 0 ? true : false;

[Clemens Hammacher, 2017/05/10 16:00:10]

Why not just (delta_size != 0)?

[gdeepti, 2017/05/10 16:14:28]

Done.

+  i::wasm::DetachWebAssemblyMemoryBuffer(i_isolate, old_buffer, free_memory);
   v8::ReturnValue<v8::Value> return_value = args.GetReturnValue();
   return_value.Set(ret);
 }