Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(31)

Unified Diff: third_party/WebKit/LayoutTests/fast/wasm/wasm-limits-tests-common.js

Issue 2702953002: [wasm] Block compile/instantiate of large array buffers (Closed)
Patch Set: Updated after V8 side landed Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/fast/wasm/wasm-limits-tests-common.js
diff --git a/third_party/WebKit/LayoutTests/fast/wasm/wasm-limits-tests-common.js b/third_party/WebKit/LayoutTests/fast/wasm/wasm-limits-tests-common.js
new file mode 100644
index 0000000000000000000000000000000000000000..3f84c02b10d96e4ff4c343bc73d3b774f63c053e
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/fast/wasm/wasm-limits-tests-common.js
@@ -0,0 +1,28 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+function createTestBuffers(limit) {
+ let builder = new WasmModuleBuilder();
+ let body = [];
+ var fct = builder.addFunction("f", kSig_v_v);
+
+ var l = builder.toBuffer().byteLength;
+
+ // in the bare bones buffer, the size of the function f
+ // is 0, which is encoded in 1 byte. For the 2^12 case,
+ // we need 2 bytes. Then, the function ends in kExprEnd,
+ // so we need that accounted, too.
+ var remaining = limit - l - 3;
+
+ for (var i = 0; i < remaining; ++i) body.push(kExprNop);
+ fct.addBody(body);
+
+ var small_buffer = builder.toBuffer();
+ // body is now 1 larger than before, because it has the kExpEnd at the end.
+ // replace that with kExprNop, and generate a new buffer.
+ body[body.length-1] = kExprNop;
+ fct.addBody(body);
+ var large_buffer = builder.toBuffer();
+ return {small: small_buffer, large: large_buffer};
+}

Powered by Google App Engine
This is Rietveld 408576698