[promise] Test IsPromise() early in Promise.prototype.then()

[promise] Test IsPromise() early in Promise.prototype.then()

Addresses TODO by Dan --- simply by moving the check and exception
earlier in the function, before calling NewPromiseCapability() or
loading the constructor.

BUG=v8:4633
LOG=N
R=adamk@chromium.org, littledan@chromium.org, cbruni@chromium.org

Fixes 'test262/built-ins/Promise/prototype/then/context-check-on-entry'

Committed: https://crrev.com/6cd8535c3b369cf07b4c0eae8075820f1c229df5
Cr-Commit-Position: refs/heads/master@{#33137}

[caitp (gmail), 2016-01-06 16:52:49]

another quick fix, PTAL

[Dan Ehrenberg, 2016-01-06 17:19:53]

The CQ bit was checked by littledan@chromium.org

[Dan Ehrenberg, 2016-01-06 17:19:54]

lgtm

[commit-bot: I haz the power, 2016-01-06 17:20:00]

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

[commit-bot: I haz the power, 2016-01-06 17:30:55]

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

[commit-bot: I haz the power, 2016-01-06 17:30:56]

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

[adamk, 2016-01-06 18:49:31]

The CQ bit was checked by adamk@chromium.org

[adamk, 2016-01-06 18:49:32]

lgtm

[commit-bot: I haz the power, 2016-01-06 18:49:36]

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

[commit-bot: I haz the power, 2016-01-06 18:50:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-06 18:51:36]

Description was changed from

==========
[promise] Test IsPromise() early in Promise.prototype.then()

Addresses TODO by Dan --- simply by moving the check and exception
earlier in the function, before calling NewPromiseCapability() or
loading the constructor.

BUG=v8:4633
LOG=N
R=adamk@chromium.org, littledan@chromium.org, cbruni@chromium.org

Fixes 'test262/built-ins/Promise/prototype/then/context-check-on-entry'
==========

to

==========
[promise] Test IsPromise() early in Promise.prototype.then()

Addresses TODO by Dan --- simply by moving the check and exception
earlier in the function, before calling NewPromiseCapability() or
loading the constructor.

BUG=v8:4633
LOG=N
R=adamk@chromium.org, littledan@chromium.org, cbruni@chromium.org

Fixes 'test262/built-ins/Promise/prototype/then/context-check-on-entry'

Committed: https://crrev.com/6cd8535c3b369cf07b4c0eae8075820f1c229df5
Cr-Commit-Position: refs/heads/master@{#33137}
==========

[commit-bot: I haz the power, 2016-01-06 18:51:37]

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