[handles] Sanitize Handle and friends.

[handles] Sanitize Handle and friends.

Bunch of cleanups to allow us to get rid of handles-inl.h at some
point (in the not so far future); but more importantly to sanitize uses
of handles and prepare for handle canonicalization support.

R=yangguo@chromium.org

Committed: https://crrev.com/3283195d0408333cce552cf4087577e6f41054e5
Cr-Commit-Position: refs/heads/master@{#28222}

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

Committed: https://crrev.com/199e30d36fe8a360797b723ea19cbe106504dc43
Cr-Commit-Position: refs/heads/master@{#29675}

[Benedikt Meurer, 2015-05-05 10:23:21]

Hey Yang,

As discussed offline. Please take a look.

Thanks,
Benedikt

[Yang, 2015-05-05 11:01:31]

lgtm with comment

https://codereview.chromium.org/1128533002/diff/1/src/handles.h
File src/handles.h (left):

https://codereview.chromium.org/1128533002/diff/1/src/handles.h#oldcode270
src/handles.h:270: class SealHandleScope BASE_EMBEDDED {
Why are we getting rid of BASE_EMBEDDED here?

[Benedikt Meurer, 2015-05-05 12:11:16]

https://codereview.chromium.org/1128533002/diff/1/src/handles.h
File src/handles.h (left):

https://codereview.chromium.org/1128533002/diff/1/src/handles.h#oldcode270
src/handles.h:270: class SealHandleScope BASE_EMBEDDED {
Because it's crap and doesn't work as intended, as outlined in the Scott Meyers
book.

[Benedikt Meurer, 2015-05-05 12:11:24]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-05-05 12:11:25]

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

[commit-bot: I haz the power, 2015-05-05 12:11:36]

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

[commit-bot: I haz the power, 2015-05-05 12:48:21]

Committed patchset #4 (id:50001)

[commit-bot: I haz the power, 2015-05-05 12:48:36]

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

[Michael Achenbach, 2015-05-05 15:44:32]

A revert of this CL (patchset #4 id:50001) has been created in
https://codereview.chromium.org/1128543004/ by machenbach@chromium.org.

The reason for reverting is: [Sheriff] Breaks mac asan:
http://build.chromium.org/p/client.v8/builders/V8%20Mac64%20ASAN/builds/1198.

[Benedikt Meurer, 2015-07-14 05:27:41]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-07-14 05:27:42]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/1128533002/#ps100001
(title: "More fixing.")

[commit-bot: I haz the power, 2015-07-14 05:27:53]

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

[commit-bot: I haz the power, 2015-07-14 05:53:44]

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

[commit-bot: I haz the power, 2015-07-14 05:53:45]

Try jobs failed on following builders:
  v8_win64_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win64_rel/builds/6641)

[Benedikt Meurer, 2015-07-14 11:36:49]

Patchset #5 (id:80001) has been deleted

[Benedikt Meurer, 2015-07-14 11:36:56]

Patchset #5 (id:100001) has been deleted

[Benedikt Meurer, 2015-07-15 05:47:50]

Patchset #6 (id:140001) has been deleted

[Benedikt Meurer, 2015-07-15 05:47:56]

Patchset #5 (id:120001) has been deleted

[Benedikt Meurer, 2015-07-15 05:49:24]

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

[commit-bot: I haz the power, 2015-07-15 05:49:36]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1128533002/160001

[commit-bot: I haz the power, 2015-07-15 06:29:33]

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

[commit-bot: I haz the power, 2015-07-15 06:29:34]

Dry run: This issue passed the CQ dry run.

[Benedikt Meurer, 2015-07-15 06:34:51]

Patchset #5 (id:160001) has been deleted

[Benedikt Meurer, 2015-07-15 06:36:47]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-07-15 06:36:47]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/1128533002/#ps180001
(title: "Moar simplification.")

[commit-bot: I haz the power, 2015-07-15 06:36:57]

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

[commit-bot: I haz the power, 2015-07-15 07:13:56]

Committed patchset #5 (id:180001)

[commit-bot: I haz the power, 2015-07-15 07:14:15]

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

[Michael Achenbach, 2015-07-15 08:05:24]

A revert of this CL (patchset #5 id:180001) has been created in
https://codereview.chromium.org/1235253007/ by machenbach@chromium.org.

The reason for reverting is: [Sheriff] Still breaks mac asan:
http://build.chromium.org/p/client.v8/builders/V8%20Mac64%20ASAN/builds/2066.

[Benedikt Meurer, 2015-07-15 10:38:43]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-07-15 10:38:43]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/1128533002/#ps200001
(title: "Lower kTargetRecursionDepth.")

[commit-bot: I haz the power, 2015-07-15 10:38:49]

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

[commit-bot: I haz the power, 2015-07-15 11:05:08]

Committed patchset #6 (id:200001)

[commit-bot: I haz the power, 2015-07-15 11:05:16]

Patchset 6 (id:??) landed as
https://crrev.com/199e30d36fe8a360797b723ea19cbe106504dc43
Cr-Commit-Position: refs/heads/master@{#29675}