[wasm] Use handles in wasm::GetDebugInfo.

[wasm] Use handles in wasm::GetDebugInfo.

Handles are necessary in wasm::GetDebugInfo because a GC can be
triggered in this function.

R=titzer@chromium.org, mstarzinger@chromium.org

Committed: https://crrev.com/717b2eafdd9001b19e762f6dfa972eac4d5c31ea
Cr-Commit-Position: refs/heads/master@{#37601}

[titzer, 2016-07-08 07:38:06]

lgtm

[Michael Starzinger, 2016-07-08 08:05:07]

https://codereview.chromium.org/2132553002/diff/1/src/frames.cc
File src/frames.cc (right):

https://codereview.chromium.org/2132553002/diff/1/src/frames.cc#newcode1436
src/frames.cc:1436: Script* WasmFrame::script() const {
If {wasm::GetDebugInfo} can really trigger a GC then we will need to rework the
frame-walker to handlify this virtual method as well.

[Michael Starzinger, 2016-07-08 08:06:20]

... but this is still a good first step, so LGTM on this despite my comment.

[ahaas, 2016-07-08 08:12:57]

The CQ bit was checked by ahaas@chromium.org

[commit-bot: I haz the power, 2016-07-08 08:13:08]

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, 2016-07-08 08:23:13]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-08 08:24:12]

Description was changed from

==========
[wasm] Use handles in wasm::GetDebugInfo.

Handles are necessary in wasm::GetDebugInfo because a GC can be
triggered in this function.

R=titzer@chromium.org, mstarzinger@chromium.org
==========

to

==========
[wasm] Use handles in wasm::GetDebugInfo.

Handles are necessary in wasm::GetDebugInfo because a GC can be
triggered in this function.

R=titzer@chromium.org, mstarzinger@chromium.org

Committed: https://crrev.com/717b2eafdd9001b19e762f6dfa972eac4d5c31ea
Cr-Commit-Position: refs/heads/master@{#37601}
==========

[commit-bot: I haz the power, 2016-07-08 08:24:13]

Patchset 1 (id:??) landed as
https://crrev.com/717b2eafdd9001b19e762f6dfa972eac4d5c31ea
Cr-Commit-Position: refs/heads/master@{#37601}