Forbid creating new wrapper from SetWrapperReferenceTo

Forbid creating new wrapper from SetWrapperReferenceTo

SetWrapperReferenceTo IDL attribute is a hack to use V8 wrapper reference
to avoid reference cycles in Blink. The |visitDOMWrapper| callback is
used to create a temporary strong reference between the wrappers in GC
prologue.

Before this patch, |visitDOMWrapper| was allowed to create a new wrapper
if the wrapper for target object didn't exist. However this is dangerous,
as the target object may be already destructed. The target wrapper must
already exist for this reference hack to work correctly.

This patch forbids creating a new wrapper from |visitDOMWrapper| callback,
and adds an ASSERT that checks that the target object wrapper already
exists.

BUG=None

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175998

[Nils Barth (inactive), 2014-06-12 03:10:03]

LGTM; haraken?

[haraken, 2014-06-12 03:48:30]

LGTM

[kouhei (in TOK), 2014-06-12 03:49:34]

The CQ bit was checked by kouhei@chromium.org

[commit-bot: I haz the power, 2014-06-12 03:50:16]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/kouhei@chromium.org/332643003/1

[commit-bot: I haz the power, 2014-06-12 06:24:39]

Change committed as 175998

[kouhei (in TOK), 2014-06-17 03:48:42]

A revert of this CL has been created in
https://codereview.chromium.org/332183008/ by kouhei@chromium.org.

The reason for reverting is: There seems to be an exiting bug which hits this
callpath. Reverting the change until the bug is fixed..