Revert of [wasm] instantiate expressed in terms of compile

Revert of [wasm] instantiate expressed in terms of compile (patchset #6 id:140001 of https://codereview.chromium.org/2806073002/ )

Reason for revert:
Roll blocker: https://bugs.chromium.org/p/chromium/issues/detail?id=710824

Original issue's description:
> [wasm] instantiate expressed in terms of compile
> 
> Today, the semantics of:
> 
> WebAssembly.instantiate
> 
> and
> 
> WebAssembly.compile().then(new WebAssemblyInstance)
> 
> are subtly different, to the point where attempting the proposed 
> change uncovered bugs. 
> 
> In the future, it's possible that .instantiate actually have different
> semantics - if we pre-specialized to the provided ffi, for example. 
> Right now that's not the case.
> 
> This CL:
> - gets our implementation closer to what developers may write using 
> the compile -> new Instance alternative, in particular wrt promise 
> creation. By reusing code paths, we uncover more bugs, and keep 
> maintenance cost lower.
> 
> - it gives us the response-based WebAssembly.instantiate implicitly. 
> Otherwise, we'd need that same implementation on the blink side. The 
> negative is maintenance: imagine if the bugs I mentioned could only be 
> found when running in Blink. 
> 
> BUG=chromium:697028
> 
> Review-Url: https://codereview.chromium.org/2806073002
> Cr-Commit-Position: refs/heads/master@{#44592}
> Committed: https://chromium.googlesource.com/v8/v8/+/7829af3275ff4644a2d0a1270abe1a1e4415e9fb

TBR=bradnelson@chromium.org,ahaas@chromium.org,adamk@chromium.org,mtrofin@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:697028
Review-Url: https://codereview.chromium.org/2810203002
Cr-Commit-Position: refs/heads/master@{#44614}
Committed: https://chromium.googlesource.com/v8/v8/+/d3f1d5c50cd6ada48371c1eb2560d2b0ec24533a

[Michael Hablich, 2017-04-12 13:27:37]

The CQ bit was checked by hablich@chromium.org

[Michael Hablich, 2017-04-12 13:27:37]

Created Revert of [wasm] instantiate expressed in terms of compile

[commit-bot: I haz the power, 2017-04-12 13:27:46]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-12 13:27:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492003657120430, "parent_rev":
"7ad0742799e3f48e35faf8c66810d478bbce66b6", "commit_rev":
"d3f1d5c50cd6ada48371c1eb2560d2b0ec24533a"}

[commit-bot: I haz the power, 2017-04-12 13:28:02]

Description was changed from

==========
Revert of [wasm] instantiate expressed in terms of compile (patchset #6
id:140001 of https://codereview.chromium.org/2806073002/ )

Reason for revert:
Roll blocker: https://bugs.chromium.org/p/chromium/issues/detail?id=710824

Original issue's description:
> [wasm] instantiate expressed in terms of compile
> 
> Today, the semantics of:
> 
> WebAssembly.instantiate
> 
> and
> 
> WebAssembly.compile().then(new WebAssemblyInstance)
> 
> are subtly different, to the point where attempting the proposed 
> change uncovered bugs. 
> 
> In the future, it's possible that .instantiate actually have different
> semantics - if we pre-specialized to the provided ffi, for example. 
> Right now that's not the case.
> 
> This CL:
> - gets our implementation closer to what developers may write using 
> the compile -> new Instance alternative, in particular wrt promise 
> creation. By reusing code paths, we uncover more bugs, and keep 
> maintenance cost lower.
> 
> - it gives us the response-based WebAssembly.instantiate implicitly. 
> Otherwise, we'd need that same implementation on the blink side. The 
> negative is maintenance: imagine if the bugs I mentioned could only be 
> found when running in Blink. 
> 
> BUG=chromium:697028
> 
> Review-Url: https://codereview.chromium.org/2806073002
> Cr-Commit-Position: refs/heads/master@{#44592}
> Committed:
https://chromium.googlesource.com/v8/v8/+/7829af3275ff4644a2d0a1270abe1a1e441...

TBR=bradnelson@chromium.org,ahaas@chromium.org,adamk@chromium.org,mtrofin@chr...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:697028
==========

to

==========
Revert of [wasm] instantiate expressed in terms of compile (patchset #6
id:140001 of https://codereview.chromium.org/2806073002/ )

Reason for revert:
Roll blocker: https://bugs.chromium.org/p/chromium/issues/detail?id=710824

Original issue's description:
> [wasm] instantiate expressed in terms of compile
> 
> Today, the semantics of:
> 
> WebAssembly.instantiate
> 
> and
> 
> WebAssembly.compile().then(new WebAssemblyInstance)
> 
> are subtly different, to the point where attempting the proposed 
> change uncovered bugs. 
> 
> In the future, it's possible that .instantiate actually have different
> semantics - if we pre-specialized to the provided ffi, for example. 
> Right now that's not the case.
> 
> This CL:
> - gets our implementation closer to what developers may write using 
> the compile -> new Instance alternative, in particular wrt promise 
> creation. By reusing code paths, we uncover more bugs, and keep 
> maintenance cost lower.
> 
> - it gives us the response-based WebAssembly.instantiate implicitly. 
> Otherwise, we'd need that same implementation on the blink side. The 
> negative is maintenance: imagine if the bugs I mentioned could only be 
> found when running in Blink. 
> 
> BUG=chromium:697028
> 
> Review-Url: https://codereview.chromium.org/2806073002
> Cr-Commit-Position: refs/heads/master@{#44592}
> Committed:
https://chromium.googlesource.com/v8/v8/+/7829af3275ff4644a2d0a1270abe1a1e441...

TBR=bradnelson@chromium.org,ahaas@chromium.org,adamk@chromium.org,mtrofin@chr...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:697028

Review-Url: https://codereview.chromium.org/2810203002
Cr-Commit-Position: refs/heads/master@{#44614}
Committed:
https://chromium.googlesource.com/v8/v8/+/d3f1d5c50cd6ada48371c1eb2560d2b0ec2...
==========

[commit-bot: I haz the power, 2017-04-12 13:28:02]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/d3f1d5c50cd6ada48371c1eb2560d2b0ec2...