Use %DebugGetProperty in debug mirror to check for Promise.

Use %DebugGetProperty in debug mirror to check for Promise.

R=aandrey@chromium.org, amikhaylova@google.com

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

[Yang, 2014-05-15 13:56:00]

This CL is dependent on https://codereview.chromium.org/273653007/

What this changes:
- circumvent cross context access checks for PromiseMirror.
- return a mirror object from Promise.prototype.promiseValue instead of the raw
value, to maintain consistency to other mirror object types.

[Alexandra Mikhaylova, 2014-05-15 14:25:49]

On 2014/05/15 13:56:00, Yang wrote:
> This CL is dependent on https://codereview.chromium.org/273653007/
> 
> What this changes:
> - circumvent cross context access checks for PromiseMirror.
> - return a mirror object from Promise.prototype.promiseValue instead of the
raw
> value, to maintain consistency to other mirror object types.

lgtm

[Yang, 2014-05-15 14:26:33]

On 2014/05/15 14:25:49, Alexandra Mikhaylova wrote:
> On 2014/05/15 13:56:00, Yang wrote:
> > This CL is dependent on https://codereview.chromium.org/273653007/
> > 
> > What this changes:
> > - circumvent cross context access checks for PromiseMirror.
> > - return a mirror object from Promise.prototype.promiseValue instead of the
> raw
> > value, to maintain consistency to other mirror object types.
> 
> lgtm

Should I land this in combination of your CL? Will this fix the webkit layout
test failure we saw?

[Alexandra Mikhaylova, 2014-05-16 11:30:35]

On 2014/05/15 14:26:33, Yang wrote:
> On 2014/05/15 14:25:49, Alexandra Mikhaylova wrote:
> > On 2014/05/15 13:56:00, Yang wrote:
> > > This CL is dependent on https://codereview.chromium.org/273653007/
> > > 
> > > What this changes:
> > > - circumvent cross context access checks for PromiseMirror.
> > > - return a mirror object from Promise.prototype.promiseValue instead of
the
> > raw
> > > value, to maintain consistency to other mirror object types.
> > 
> > lgtm
> 
> Should I land this in combination of your CL? Will this fix the webkit layout
> test failure we saw?

Yes, it solves the problem with layout test failure, I think you should land it.

[aandrey, 2014-05-16 11:55:40]

lgtm

[Yang, 2014-05-16 13:06:27]

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