V8 Bindings: Don't attempt to set an empty V8HiddenValue

V8 Bindings: Don't attempt to set an empty V8HiddenValue

After an execution context is stopped, IndexedDB returns an empty
ScriptValue() on property access. The bindings code should not
attempt to cache this using V8HiddenValue.

BUG=590730
R=haraken@chromium.org

Committed: https://crrev.com/96ad5a7ab9cea6a586e5df4acef371ef35d6c5ba
Cr-Commit-Position: refs/heads/master@{#380067}

[jsbell, 2016-03-09 00:44:43]

haraken@ - please take a look?

I don't know if the bots will like this or not, but the new test case passes
with/crashes without the CL.

[haraken, 2016-03-09 01:00:36]

LGTM

[haraken, 2016-03-09 03:58:11]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-03-09 03:58:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1780593002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1780593002/1

[commit-bot: I haz the power, 2016-03-09 04:42:24]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-09 04:44:51]

Description was changed from

==========
V8 Bindings: Don't attempt to set an empty V8HiddenValue

After an execution context is stopped, IndexedDB returns an empty
ScriptValue() on property access. The bindings code should not
attempt to cache this using V8HiddenValue.

BUG=590730
R=haraken@chromium.org
==========

to

==========
V8 Bindings: Don't attempt to set an empty V8HiddenValue

After an execution context is stopped, IndexedDB returns an empty
ScriptValue() on property access. The bindings code should not
attempt to cache this using V8HiddenValue.

BUG=590730
R=haraken@chromium.org

Committed: https://crrev.com/96ad5a7ab9cea6a586e5df4acef371ef35d6c5ba
Cr-Commit-Position: refs/heads/master@{#380067}
==========

[commit-bot: I haz the power, 2016-03-09 04:44:52]

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