[wasm] Add tests for breakpoints

src/wasm/wasm-debug.cc

Index: src/wasm/wasm-debug.cc
diff --git a/src/wasm/wasm-debug.cc b/src/wasm/wasm-debug.cc
index bb9bb07c06e418855c9abe983b29c6e24d0332b3..3b372e042ba1ba9ca8d223e604637439ff850c3e 100644
--- a/src/wasm/wasm-debug.cc
+++ b/src/wasm/wasm-debug.cc
@@ -34,16 +34,15 @@ class InterpreterHandle {
       : instance_(debug_info->wasm_instance()->compiled_module()->module()),
         interpreter_(GetBytesEnv(&instance_, debug_info), &allocator_),
         isolate_(isolate) {
-    Handle<JSArrayBuffer> mem_buffer =
-        handle(debug_info->wasm_instance()->memory_buffer(), isolate);
-    if (mem_buffer->IsUndefined(isolate)) {
-      DCHECK_EQ(0, instance_.module->min_mem_pages);
-      instance_.mem_start = nullptr;
-      instance_.mem_size = 0;
-    } else {
+    if (debug_info->wasm_instance()->has_memory_buffer()) {
+      JSArrayBuffer *mem_buffer = debug_info->wasm_instance()->memory_buffer();
       instance_.mem_start =
           reinterpret_cast<byte *>(mem_buffer->backing_store());

[titzer, 2017/01/16 13:55:27]

byte* instead of byte *

(I guess clang-format is accepting both these days?)

[Clemens Hammacher, 2017/01/16 18:51:30]

Fixed with http://crrev.com/2635003002 and http://crrev.com/2631183002.

       CHECK(mem_buffer->byte_length()->ToUint32(&instance_.mem_size));
+    } else {
+      DCHECK_EQ(0, instance_.module->min_mem_pages);
+      instance_.mem_start = nullptr;
+      instance_.mem_size = 0;
     }
   }