[wasm] W.validate should not throw on failure

src/wasm/wasm-result.h

 // 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.
 
 #ifndef V8_WASM_RESULT_H_
 #define V8_WASM_RESULT_H_
 
 #include <memory>
 
 #include "src/base/compiler-specific.h"
@@ skipping 94 matching lines @@
     CompileError("%s", str.str().c_str());
   }
 
   i::Handle<i::Object> Reify() {
     i::Handle<i::Object> result = exception_;
     exception_ = i::Handle<i::Object>::null();
     return result;
   }
 
   bool error() const { return !exception_.is_null(); }
+  bool wasm_error() { return wasm_error_; }
 
  private:
   void Format(i::Handle<i::JSFunction> constructor, const char* fmt, va_list);
 
   i::Isolate* isolate_;
   const char* context_;
   i::Handle<i::Object> exception_;
+  bool wasm_error_ = false;

[titzer, 2017/01/20 12:03:55]

Thinking of a better name for this. Looks like it is set whenever an error is
generated but never cleared, so many "any_error"?

[rossberg, 2017/01/20 12:17:22]

Nah, it's set only for {Compile,Link,Runtime}Error, and distinguishes these from
random JS errors; validate needs to return false for the former, but still throw
the latter.

 };
 }  // namespace wasm
 }  // namespace internal
 }  // namespace v8
 
 #endif