[proxies] Use IsRevoked where possible, remove has_handler.

[proxies] Use IsRevoked where possible, remove has_handler.

R=jkummerow@chromium.org
BUG=

Committed: https://crrev.com/39296e679fc4d760087db784397a1ab93e92a5f4
Cr-Commit-Position: refs/heads/master@{#32394}

[Jakob Kummerow, 2015-11-30 08:47:55]

LGTM with comment.

https://codereview.chromium.org/1486553002/diff/1/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/1486553002/diff/1/src/objects.cc#newcode6921
src/objects.cc:6921: Handle<JSProxy> proxy =
Handle<JSProxy>::cast(it->GetHolder<Object>());
unnecessary double cast, just do:

... = it->GetHolder<JSProxy>();

[neis, 2015-11-30 09:17:55]

https://codereview.chromium.org/1486553002/diff/1/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/1486553002/diff/1/src/objects.cc#newcode6921
src/objects.cc:6921: Handle<JSProxy> proxy =
Handle<JSProxy>::cast(it->GetHolder<Object>());
On 2015/11/30 08:47:55, Jakob wrote:
> unnecessary double cast, just do:
> 
> ... = it->GetHolder<JSProxy>();

Done.

[neis, 2015-11-30 09:18:02]

The CQ bit was checked by neis@chromium.org

[neis, 2015-11-30 09:18:02]

The patchset sent to the CQ was uploaded after l-g-t-m from
jkummerow@chromium.org
Link to the patchset: https://codereview.chromium.org/1486553002/#ps20001
(title: "Address comment.")

[commit-bot: I haz the power, 2015-11-30 09:18:10]

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

[commit-bot: I haz the power, 2015-11-30 09:25:53]

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

[commit-bot: I haz the power, 2015-11-30 09:25:53]

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

[neis, 2015-11-30 09:37:11]

The CQ bit was checked by neis@chromium.org

[neis, 2015-11-30 09:37:12]

The patchset sent to the CQ was uploaded after l-g-t-m from
jkummerow@chromium.org
Link to the patchset: https://codereview.chromium.org/1486553002/#ps40001
(title: "Rebase.")

[commit-bot: I haz the power, 2015-11-30 09:37:20]

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

[commit-bot: I haz the power, 2015-11-30 09:59:24]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-11-30 09:59:59]

Description was changed from

==========
[proxies] Use IsRevoked where possible, remove has_handler.

R=jkummerow@chromium.org
BUG=
==========

to

==========
[proxies] Use IsRevoked where possible, remove has_handler.

R=jkummerow@chromium.org
BUG=

Committed: https://crrev.com/39296e679fc4d760087db784397a1ab93e92a5f4
Cr-Commit-Position: refs/heads/master@{#32394}
==========

[commit-bot: I haz the power, 2015-11-30 10:00:00]

Patchset 3 (id:??) landed as
https://crrev.com/39296e679fc4d760087db784397a1ab93e92a5f4
Cr-Commit-Position: refs/heads/master@{#32394}