[proxies] Teach ToPropertyDescriptor to deal with Proxies

[proxies] Teach ToPropertyDescriptor to deal with Proxies

BUG=v8:1543
LOG=n

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

[Jakob Kummerow, 2015-11-12 13:44:45]

jkummerow@chromium.org changed reviewers:
+ cbruni@chromium.org

[Jakob Kummerow, 2015-11-12 13:44:46]

Camillo, PTAL.

Georg, FYI.

https://codereview.chromium.org/1438233002/diff/1/src/property-descriptor.cc
File src/property-descriptor.cc (right):

https://codereview.chromium.org/1438233002/diff/1/src/property-descriptor.cc#...
src/property-descriptor.cc:163: // enumerable?
This is the same as old lines 167 through 253 with different indentation.

https://codereview.chromium.org/1438233002/diff/1/src/property-descriptor.cc#...
src/property-descriptor.cc:258: void
PropertyDescriptor::CompletePropertyDescriptor(Isolate* isolate,
I'm not using this yet, but my next CL is going to need it.

[Camillo Bruni, 2015-11-12 13:52:19]

lgtm

https://codereview.chromium.org/1438233002/diff/1/src/property-descriptor.cc
File src/property-descriptor.cc (right):

https://codereview.chromium.org/1438233002/diff/1/src/property-descriptor.cc#...
src/property-descriptor.cc:19: bool GetPropertyIfPresent(Handle<Object> obj,
Handle<String> name,
Probably this will (as so many other methods) need a Maybe<bool> result.

[Jakob Kummerow, 2015-11-12 14:01:45]

Thanks, landing.

https://codereview.chromium.org/1438233002/diff/1/src/property-descriptor.cc
File src/property-descriptor.cc (right):

https://codereview.chromium.org/1438233002/diff/1/src/property-descriptor.cc#...
src/property-descriptor.cc:19: bool GetPropertyIfPresent(Handle<Object> obj,
Handle<String> name,
On 2015/11/12 13:52:19, cbruni wrote:
> Probably this will (as so many other methods) need a Maybe<bool> result.

Maybe. Let's introduce them as needed.

[Jakob Kummerow, 2015-11-12 14:01:48]

The CQ bit was checked by jkummerow@chromium.org

[commit-bot: I haz the power, 2015-11-12 14:01:51]

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

[commit-bot: I haz the power, 2015-11-12 14:17:41]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-14 23:20:45]

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