Get rid of all non-IC uses of LookupOwnRealNamedProperty

Get rid of all non-IC uses of LookupOwnRealNamedProperty

BUG=
R=yangguo@chromium.org

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

[Toon Verwaest, 2014-08-20 15:04:58]

PTAL

[Yang, 2014-08-21 06:08:20]

https://codereview.chromium.org/491023002/diff/1/test/cctest/test-api.cc
File test/cctest/test-api.cc (left):

https://codereview.chromium.org/491023002/diff/1/test/cctest/test-api.cc#oldc...
test/cctest/test-api.cc:21727: context0->Global()->Set(v8_str("x"), v8_num(42));
Is this intentional? Why?

[Toon Verwaest, 2014-08-21 07:02:55]

https://codereview.chromium.org/491023002/diff/1/test/cctest/test-api.cc
File test/cctest/test-api.cc (left):

https://codereview.chromium.org/491023002/diff/1/test/cctest/test-api.cc#oldc...
test/cctest/test-api.cc:21727: context0->Global()->Set(v8_str("x"), v8_num(42));
On 2014/08/21 06:08:20, Yang wrote:
> Is this intentional? Why?

Because below it's calling into %AddNamedProperty, which due to my changes in
the LookupIterator to look behind the JSGlobalProxy into the JSGlobalObject, now
fails with a runtime assert that the property shouldn't exist yet. Plus given
that all should result in AccessCheck failure, it doesn't really make a
difference whether or not the property is there.

[Yang, 2014-08-21 07:37:26]

LGTM

https://codereview.chromium.org/491023002/diff/1/test/cctest/test-api.cc
File test/cctest/test-api.cc (left):

https://codereview.chromium.org/491023002/diff/1/test/cctest/test-api.cc#oldc...
test/cctest/test-api.cc:21727: context0->Global()->Set(v8_str("x"), v8_num(42));
On 2014/08/21 07:02:55, Toon Verwaest wrote:
> On 2014/08/21 06:08:20, Yang wrote:
> > Is this intentional? Why?
> 
> Because below it's calling into %AddNamedProperty, which due to my changes in
> the LookupIterator to look behind the JSGlobalProxy into the JSGlobalObject,
now
> fails with a runtime assert that the property shouldn't exist yet. Plus given
> that all should result in AccessCheck failure, it doesn't really make a
> difference whether or not the property is there.

Acknowledged.

[Toon Verwaest, 2014-08-21 08:26:49]

Committed patchset #1 manually as 23262 (presubmit successful).