[wasm] enable wasm structured cloning in specific cases

third_party/WebKit/LayoutTests/http/tests/wasm/wasm_indexeddb_test.js

Index: third_party/WebKit/LayoutTests/http/tests/wasm/wasm_indexeddb_test.js
diff --git a/third_party/WebKit/LayoutTests/http/tests/wasm/wasm_indexeddb_test.js b/third_party/WebKit/LayoutTests/http/tests/wasm/wasm_indexeddb_test.js
index 4b7cafe45ba38b5e3ee95bc5bb4fb6a20096819b..f8ca94c11b91609376e3297880747038e79d4ab9 100644
--- a/third_party/WebKit/LayoutTests/http/tests/wasm/wasm_indexeddb_test.js
+++ b/third_party/WebKit/LayoutTests/http/tests/wasm/wasm_indexeddb_test.js
@@ -32,8 +32,10 @@ function createAndSaveToIndexedDB() {
   });
 }
 
+var kErrorMsg = "failed to retrieve object";
+
 function loadFromIndexedDB(prev) {
-  return new Promise(resolve => {
+  return new Promise((resolve, reject) => {
     prev.then(() => {
       var open_request = indexedDB.open(db_name);
       open_request.onsuccess = function() {
@@ -43,16 +45,34 @@ function loadFromIndexedDB(prev) {
         var get_request = store.get(module_key);
         get_request.onsuccess = function() {
           var mod = get_request.result;
-          var instance = new WebAssembly.Instance(get_request.result);
-          resolve(instance.exports.increment(1));
+          if (mod instanceof WebAssembly.Module) {
+            var instance = new WebAssembly.Instance(mod);

[jsbell, 2017/03/22 17:41:06]

Since the constructor may throw, maybe wrap in a try/catch block where catch
calls reject() ?

(Not critical - it's just a difference between the test timing out or getting a
useful message. It'll fail either way.)

[Mircea Trofin, 2017/03/22 23:31:15]

Done.

+            resolve(instance.exports.increment(1));
+          } else {
+            reject(new Error(kErrorMsg));

[jsbell, 2017/03/22 17:41:07]

We should be asserting something about what `mod` is in this case, at the very
least.

[Mircea Trofin, 2017/03/22 23:31:15]

Done.

+          }
         };
       };
     });
   });
 }
 
-function TestIndexedDBLoadStore() {
-  return loadFromIndexedDB(createAndSaveToIndexedDB())
-    .then((res) => assert_equals(res, 2))
-    .catch(error => assert_unreached(error));
+function TestIndexedDBLoadStoreSecure() {
+  if (window.location.origin != get_host_info().AUTHENTICATED_ORIGIN) {
+    window.location = get_host_info().AUTHENTICATED_ORIGIN + window.location.pathname;
+  } else {
+    return loadFromIndexedDB(createAndSaveToIndexedDB())
+      .then((res) => assert_equals(res, 2))

[jsbell, 2017/03/22 17:41:07]

I don't think you want to use .then().catch() here because if the callback for
.then() throws it will propagate to the .catch().

You probably want the two-argument form of then(), i.e.

.then(
  result => assert_equals(result, 2),
  error => assert_unreached(error)
);

[Mircea Trofin, 2017/03/22 23:31:15]

Done.

+      .catch(error => assert_unreached(error));
+  }
+}
+
+function TestIndexedDBLoadStoreInsecure() {
+  if (window.location.origin != get_host_info().UNAUTHENTICATED_ORIGIN) {
+    window.location = get_host_info().UNAUTHENTICATED_ORIGIN + window.location.pathname;
+  } else {
+    return loadFromIndexedDB(createAndSaveToIndexedDB())
+      .then(assert_unreached)

[jsbell, 2017/03/22 17:41:07]

same as above re: two argument form of then()

[Mircea Trofin, 2017/03/22 23:31:15]

Done.

+      .catch(error => assert_equals(error.message, kErrorMsg));
+  }
 }