[api] Restrict Template::Set to take templates or primitive values

[api] Restrict Template::Set to take templates or primitive values

Embedders that passed in non-primitive values should either use
getters/setters (templates) or install a native data property,
otherwise, the values would all result in cross-context leaks

BUG=none
R=verwaest@chromium.org
LOG=y

Committed: https://crrev.com/eebdee8eafa97849cc70c25f3fecf1b075bac248
Cr-Commit-Position: refs/heads/master@{#35244}

[Toon Verwaest, 2016-03-29 20:19:16]

lgtm

[jochen (gone - plz use gerrit), 2016-04-04 15:29:48]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2016-04-04 15:29:57]

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

[commit-bot: I haz the power, 2016-04-04 16:58:23]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-04 16:58:48]

Description was changed from

==========
[api] Restrict Template::Set to take templates or primitive values

Embedders that passed in non-primitive values should either use
getters/setters (templates) or install a native data property,
otherwise, the values would all result in cross-context leaks

BUG=none
R=verwaest@chromium.org
LOG=y
==========

to

==========
[api] Restrict Template::Set to take templates or primitive values

Embedders that passed in non-primitive values should either use
getters/setters (templates) or install a native data property,
otherwise, the values would all result in cross-context leaks

BUG=none
R=verwaest@chromium.org
LOG=y

Committed: https://crrev.com/eebdee8eafa97849cc70c25f3fecf1b075bac248
Cr-Commit-Position: refs/heads/master@{#35244}
==========

[commit-bot: I haz the power, 2016-04-04 16:58:50]

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

[jochen (gone - plz use gerrit), 2016-04-05 14:13:34]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1859993002/ by jochen@chromium.org.

The reason for reverting is: Breaks node.js.

[jochen (gone - plz use gerrit), 2016-04-05 14:13:37]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1860003002/ by jochen@chromium.org.

The reason for reverting is: Breaks node.js.