[wasm][test] Remove redundant function

The {setTableLength} method is redundant and has a single user. Remove
it, use {setTableBounds} instead.

Drive-by: Add default to the table max, to document that this can
actually be {undefined}, in which case the table has no maximum.

R=binji@chromium.org

Bug: v8:8238
Change-Id: I0d7a2f4d49d083f7adadbb4b6cd4933bcb1dc174
Reviewed-on: https://chromium-review.googlesource.com/c/1350126Reviewed-by: Ben Smith <binji@chromium.org>
Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#57860}

test/mjsunit/wasm/bulk-memory.js

@@ -20,7 +20,7 @@ load("test/mjsunit/wasm/wasm-module-builder.js");
 (function TestPassiveElementSegment() {
   const builder = new WasmModuleBuilder();
   builder.addFunction('f', kSig_v_v).addBody([]);
-  builder.setTableLength(1);
+  builder.setTableBounds(1, 1);
   builder.addPassiveElementSegment([0, 0, 0]);
   builder.addPassiveElementSegment([0, 0]);
 

test/mjsunit/wasm/wasm-module-builder.js

@@ -341,18 +341,12 @@ class WasmModuleBuilder {
     return this.addElementSegment(this.table_length_min, false, array);
   }
 
-  setTableBounds(min, max) {
+  setTableBounds(min, max = undefined) {
     this.table_length_min = min;
     this.table_length_max = max;
     return this;
   }
 
-  setTableLength(length) {
-    this.table_length_min = length;
-    this.table_length_max = length;
-    return this;
-  }
-
   setName(name) {
     this.name = name;
     return this;