Reland of Switch V8HiddenValue from hidden values to privates

Reland of Switch V8HiddenValue from hidden values to privates (patchset #1 id:1 of https://codereview.chromium.org/1466733006/ )

Reason for revert:
Underlying oilpan issues were addressed. Esp. the hidden values are no longer touched in V8AbstractEventListener's dtor

Original issue's description:
> Revert of Switch V8HiddenValue from hidden values to privates (patchset #7 id:120001 of https://codereview.chromium.org/1416053012/ )
>
> Reason for revert:
> This CL broke tests in oilpan builds.
>
> https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Oilpan_ASAN/5177/layout-test-results/results.html
>
> Original issue's description:
> > Switch V8HiddenValue from hidden values to privates
> >
> > Privates are like ES6 symbols, but are not accesible by JavaScript.
> > Hidden values on the other hand will be deleted form the API soon.
> >
> > BUG=none
> > R=haraken@chromium.org
> >
> > Committed: https://crrev.com/b1802c0fc28635a50e7f55e40c71e69281cc85f3
> > Cr-Commit-Position: refs/heads/master@{#360786}
>
> TBR=vogelheim@chromium.org,sigbjornf@opera.com,jochen@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=none
>
> Committed: https://crrev.com/dfb67af2fb65ecb0daec0498140ad6d7a9a0ea59
> Cr-Commit-Position: refs/heads/master@{#361016}

R=vogelheim@chromium.org,sigbjornf@opera.com,haraken@chromium.org

Committed: https://crrev.com/7fc3f2ba0e94feba4116f64c72b653c1554794c7
Cr-Commit-Position: refs/heads/master@{#361417}

[jochen (gone - plz use gerrit), 2015-11-24 16:52:58]

Created Reland of Switch V8HiddenValue from hidden values to privates

[jochen (gone - plz use gerrit), 2015-11-24 16:53:20]

Description was changed from

==========
Reland of Switch V8HiddenValue from hidden values to privates (patchset #1 id:1
of https://codereview.chromium.org/1466733006/ )

Reason for revert:
Underlying oilpan issues were addressed. Esp. the hidden values are no longer
touched in V8AbstractEventListener's dtor

Original issue's description:
> Revert of Switch V8HiddenValue from hidden values to privates (patchset #7
id:120001 of https://codereview.chromium.org/1416053012/ )
> 
> Reason for revert:
> This CL broke tests in oilpan builds.
> 
>
https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Oil...
> 
> Original issue's description:
> > Switch V8HiddenValue from hidden values to privates
> >
> > Privates are like ES6 symbols, but are not accesible by JavaScript.
> > Hidden values on the other hand will be deleted form the API soon.
> >
> > BUG=none
> > R=haraken@chromium.org
> >
> > Committed: https://crrev.com/b1802c0fc28635a50e7f55e40c71e69281cc85f3
> > Cr-Commit-Position: refs/heads/master@{#360786}
> 
> TBR=vogelheim@chromium.org,sigbjornf@opera.com,jochen@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=none
> 
> Committed: https://crrev.com/dfb67af2fb65ecb0daec0498140ad6d7a9a0ea59
> Cr-Commit-Position: refs/heads/master@{#361016}

TBR=vogelheim@chromium.org,sigbjornf@opera.com,haraken@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=none
==========

to

==========
Reland of Switch V8HiddenValue from hidden values to privates (patchset #1 id:1
of https://codereview.chromium.org/1466733006/ )

Reason for revert:
Underlying oilpan issues were addressed. Esp. the hidden values are no longer
touched in V8AbstractEventListener's dtor

Original issue's description:
> Revert of Switch V8HiddenValue from hidden values to privates (patchset #7
id:120001 of https://codereview.chromium.org/1416053012/ )
> 
> Reason for revert:
> This CL broke tests in oilpan builds.
> 
>
https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Oil...
> 
> Original issue's description:
> > Switch V8HiddenValue from hidden values to privates
> >
> > Privates are like ES6 symbols, but are not accesible by JavaScript.
> > Hidden values on the other hand will be deleted form the API soon.
> >
> > BUG=none
> > R=haraken@chromium.org
> >
> > Committed: https://crrev.com/b1802c0fc28635a50e7f55e40c71e69281cc85f3
> > Cr-Commit-Position: refs/heads/master@{#360786}
> 
> TBR=vogelheim@chromium.org,sigbjornf@opera.com,jochen@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=none
> 
> Committed: https://crrev.com/dfb67af2fb65ecb0daec0498140ad6d7a9a0ea59
> Cr-Commit-Position: refs/heads/master@{#361016}

R=vogelheim@chromium.org,sigbjornf@opera.com,haraken@chromium.org
==========

[jochen (gone - plz use gerrit), 2015-11-24 16:53:33]

ptal (not sending automatically to CQ)

[jochen (gone - plz use gerrit), 2015-11-24 16:53:36]

The CQ bit was checked by jochen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-11-24 16:55:09]

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

[vogelheim, 2015-11-24 17:48:09]

lgtm

[commit-bot: I haz the power, 2015-11-24 18:14:43]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-11-24 18:14:43]

Dry run: This issue passed the CQ dry run.

[sof, 2015-11-24 18:26:42]

lgtm to retry

[jochen (gone - plz use gerrit), 2015-11-24 19:42:11]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2015-11-24 19:43:11]

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

[commit-bot: I haz the power, 2015-11-24 19:55:30]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-24 19:56:22]

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

[haraken, 2015-11-24 23:32:04]

LGTM

[sof, 2015-11-25 06:38:09]

Oilpan bots not picking up anything bad here either.

(That leaves https://codereview.chromium.org/1467493002/ , where no revert is in
order, really. )