[wasm] Add a BytecodeIterator and use in non-performance-critical situations.

test/unittests/wasm/ast-decoder-unittest.cc

 // Copyright 2015 the V8 project 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 "test/unittests/test-utils.h"
 
 #include "src/v8.h"
 
 #include "test/cctest/wasm/test-signatures.h"
 
@@ skipping 2419 matching lines @@
   EXPECT_TRUE(result);
   EXPECT_EQ(5 + 1337 + 212, decls.total_local_count);
 
   LocalTypeMap map = Expand(decls);
   size_t pos = 0;
   pos = ExpectRun(map, pos, kAstF32, 5);
   pos = ExpectRun(map, pos, kAstI32, 1337);
   pos = ExpectRun(map, pos, kAstI64, 212);
 }
 
+class BytecodeIteratorTest : public TestWithZone {};
+
+TEST_F(BytecodeIteratorTest, SimpleForeach) {
+  byte code[] = {WASM_IF_ELSE(WASM_ZERO, WASM_ZERO, WASM_ZERO)};
+  BytecodeIterator iter(code, code + sizeof(code));
+  WasmOpcode expected[] = {kExprI8Const, kExprIf,      kExprI8Const,
+                           kExprElse,    kExprI8Const, kExprEnd};
+  size_t pos = 0;
+  for (WasmOpcode opcode : iter) {
+    if (pos >= arraysize(expected)) {
+      EXPECT_TRUE(false);

[ahaas, 2016/07/11 12:31:19]

why don't you say 
EXPECT_TRUE(pos < arraysize(expected));?

+      break;
+    }
+    EXPECT_EQ(expected[pos++], opcode);
+  }
+  EXPECT_EQ(arraysize(expected), pos);
+}
+
+TEST_F(BytecodeIteratorTest, ForeachTwice) {
+  byte code[] = {WASM_IF_ELSE(WASM_ZERO, WASM_ZERO, WASM_ZERO)};
+  BytecodeIterator iter(code, code + sizeof(code));
+  int count = 0;
+
+  count = 0;
+  for (WasmOpcode opcode : iter) {
+    USE(opcode);
+    count++;
+  }
+  EXPECT_EQ(6, count);
+
+  count = 0;
+  for (WasmOpcode opcode : iter) {
+    USE(opcode);
+    count++;
+  }
+  EXPECT_EQ(6, count);
+}
+
+TEST_F(BytecodeIteratorTest, WithAstDecls) {
+  byte code[] = {1, 1, kLocalI32, WASM_I8(9), WASM_I8(11)};
+  AstLocalDecls decls(zone());
+  BytecodeIterator iter(code, code + sizeof(code), &decls);
+
+  EXPECT_EQ(3, decls.decls_encoded_size);
+  EXPECT_EQ(3, iter.pc_offset());
+  EXPECT_TRUE(iter.has_next());
+  EXPECT_EQ(kExprI8Const, iter.current());
+  iter.next();
+  EXPECT_TRUE(iter.has_next());
+  EXPECT_EQ(kExprI8Const, iter.current());
+  iter.next();
+  EXPECT_FALSE(iter.has_next());
+}
+
 }  // namespace wasm
 }  // namespace internal
 }  // namespace v8