[promise] revert error message change for Promise.resolve()

[promise] revert error message change for Promise.resolve()

Previously, it was `ToString(PromiseCast)`, which comes out as
"function resolve() { [native code] }". I had changed it to "Promise.resolve()",
but the other style is more consistently used.

LOG=N
R=cbruni@chromium.org, adamk@chromium.org, littledan@chromium.org

Committed: https://crrev.com/5fcfe05a3b0d3d14575cf6fe112335442381bd33
Cr-Commit-Position: refs/heads/master@{#33129}

[caitp (gmail), 2016-01-05 21:23:05]

just undoes a small part of an earlier fix. I originally thought the updated
error message was an improvement, but the original is better in some ways, and
is more consistent with the rest of the code.

Not urgent, but might be preferred

[adamk, 2016-01-05 21:28:09]

Can you add info about the message change to the CL description? That'd help me
review this.

[caitp (gmail), 2016-01-05 21:29:35]

On 2016/01/05 21:28:09, adamk wrote:
> Can you add info about the message change to the CL description? That'd help
me
> review this.

Previously, it was `ToString(PromiseCast)`, which comes out as "function
resolve() { [native code] }". I had changed it to "Promise.resolve()", but the
other style is more consistently used

[caitp (gmail), 2016-01-05 21:29:57]

On 2016/01/05 21:29:35, caitp wrote:
> On 2016/01/05 21:28:09, adamk wrote:
> > Can you add info about the message change to the CL description? That'd help
> me
> > review this.
> 
> Previously, it was `ToString(PromiseCast)`, which comes out as "function
> resolve() { [native code] }". I had changed it to "Promise.resolve()", but the
> other style is more consistently used

--- so, this CL just changes it back to how it was before, that's all.

[adamk, 2016-01-05 21:30:30]

Description was changed from

==========
[promise] revert error message change for Promise.resolve()

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

to

==========
[promise] revert error message change for Promise.resolve()

Previously, it was `ToString(PromiseCast)`, which comes out as
"function resolve() { [native code] }". I had changed it to "Promise.resolve()",
but the other style is more consistently used.

LOG=N
R=cbruni@chromium.org, adamk@chromium.org, littledan@chromium.org
==========

[adamk, 2016-01-05 21:30:44]

lgtm

I added your explanation to the CL description.

[adamk, 2016-01-05 21:30:52]

Description was changed from

==========
[promise] revert error message change for Promise.resolve()

Previously, it was `ToString(PromiseCast)`, which comes out as
"function resolve() { [native code] }". I had changed it to "Promise.resolve()",
but the other style is more consistently used.

LOG=N
R=cbruni@chromium.org, adamk@chromium.org, littledan@chromium.org
==========

to

==========
[promise] revert error message change for Promise.resolve()

Previously, it was `ToString(PromiseCast)`, which comes out as
"function resolve() { [native code] }". I had changed it to "Promise.resolve()",
but the other style is more consistently used.

LOG=N
R=cbruni@chromium.org, adamk@chromium.org, littledan@chromium.org
==========

[caitp (gmail), 2016-01-05 21:32:02]

The CQ bit was checked by caitpotter88@gmail.com

[commit-bot: I haz the power, 2016-01-05 21:32:41]

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

[commit-bot: I haz the power, 2016-01-05 22:19:50]

Description was changed from

==========
[promise] revert error message change for Promise.resolve()

Previously, it was `ToString(PromiseCast)`, which comes out as
"function resolve() { [native code] }". I had changed it to "Promise.resolve()",
but the other style is more consistently used.

LOG=N
R=cbruni@chromium.org, adamk@chromium.org, littledan@chromium.org
==========

to

==========
[promise] revert error message change for Promise.resolve()

Previously, it was `ToString(PromiseCast)`, which comes out as
"function resolve() { [native code] }". I had changed it to "Promise.resolve()",
but the other style is more consistently used.

LOG=N
R=cbruni@chromium.org, adamk@chromium.org, littledan@chromium.org
==========

[commit-bot: I haz the power, 2016-01-05 22:19:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-05 22:20:44]

Description was changed from

==========
[promise] revert error message change for Promise.resolve()

Previously, it was `ToString(PromiseCast)`, which comes out as
"function resolve() { [native code] }". I had changed it to "Promise.resolve()",
but the other style is more consistently used.

LOG=N
R=cbruni@chromium.org, adamk@chromium.org, littledan@chromium.org
==========

to

==========
[promise] revert error message change for Promise.resolve()

Previously, it was `ToString(PromiseCast)`, which comes out as
"function resolve() { [native code] }". I had changed it to "Promise.resolve()",
but the other style is more consistently used.

LOG=N
R=cbruni@chromium.org, adamk@chromium.org, littledan@chromium.org

Committed: https://crrev.com/5fcfe05a3b0d3d14575cf6fe112335442381bd33
Cr-Commit-Position: refs/heads/master@{#33129}
==========

[commit-bot: I haz the power, 2016-01-05 22:20:45]

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