Tighten HAS_PRIVATE to own properties

Tighten HAS_PRIVATE to own properties

R=mstarzinger@chromium.org
BUG=

Committed: https://code.google.com/p/v8/source/detail?r=21379

[Michael Starzinger, 2014-05-19 21:21:01]

I am confused! Can you provide context?

https://codereview.chromium.org/290373002/diff/20001/src/macros.py
File src/macros.py (right):

https://codereview.chromium.org/290373002/diff/20001/src/macros.py#newcode174
src/macros.py:174: macro HAS_PRIVATE(obj, sym) = (%HasOwnProperty(obj, sym));
OK, maybe I am confused here, but IMHO the macro HAS_PRIVATE is not used
anywhere in the codebase. Which in turn explains why it is no problem that the
%HasOwnProperty runtime does not exists. I think I need more context here.

[rossberg, 2014-05-20 06:57:19]

https://codereview.chromium.org/290373002/diff/20001/src/macros.py
File src/macros.py (right):

https://codereview.chromium.org/290373002/diff/20001/src/macros.py#newcode174
src/macros.py:174: macro HAS_PRIVATE(obj, sym) = (%HasOwnProperty(obj, sym));
On 2014/05/19 21:21:02, Michael Starzinger wrote:
> OK, maybe I am confused here, but IMHO the macro HAS_PRIVATE is not used
> anywhere in the codebase. Which in turn explains why it is no problem that the
> %HasOwnProperty runtime does not exists. I think I need more context here.

Oops, how embarrassing, that should be HasLocalProperty, of course. (It will
come in useful for one of Arv's in-flight CLs.)

It's unfortunate that it's not possible to write tests for macros.py...

[Michael Starzinger, 2014-05-20 09:12:24]

LGTM.

[rossberg, 2014-05-20 09:13:32]

Committed patchset #3 manually as r21379 (presubmit successful).