Convert v8::Value::To* to use MaybeLocal

Convert v8::Value::To* to use MaybeLocal

BUG=v8:3929
LOG=y

Committed: https://crrev.com/2fc51d9cfa8d1b5e1a3a5b82b9af34994e286879
Cr-Commit-Position: refs/heads/master@{#26912}

[jochen (gone - plz use gerrit), 2015-02-27 09:42:48]

jochen@chromium.org changed reviewers:
+ jochen@chromium.org

[jochen (gone - plz use gerrit), 2015-02-27 09:42:49]

lgtm with comment

https://codereview.chromium.org/962983002/diff/40001/include/v8.h
File include/v8.h (right):

https://codereview.chromium.org/962983002/diff/40001/include/v8.h#newcode438
include/v8.h:438: V8_INLINE bool ToLocal(Local<S>* out) const {
can we mark this as must use result?

[dcarney, 2015-02-27 09:49:50]

https://codereview.chromium.org/962983002/diff/40001/include/v8.h
File include/v8.h (right):

https://codereview.chromium.org/962983002/diff/40001/include/v8.h#newcode438
include/v8.h:438: V8_INLINE bool ToLocal(Local<S>* out) const {
On 2015/02/27 09:42:49, jochen (slow) wrote:
> can we mark this as must use result?

Done.

[dcarney, 2015-02-27 10:16:02]

The CQ bit was checked by dcarney@chromium.org

[dcarney, 2015-02-27 10:16:03]

The patchset sent to the CQ was uploaded after l-g-t-m from jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/962983002/#ps60001 (title:
" ")

[commit-bot: I haz the power, 2015-02-27 10:16:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/962983002/60001

[commit-bot: I haz the power, 2015-02-27 10:39:30]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-02-27 10:39:56]

Patchset 4 (id:??) landed as
https://crrev.com/2fc51d9cfa8d1b5e1a3a5b82b9af34994e286879
Cr-Commit-Position: refs/heads/master@{#26912}