Show stack traces in console for unhandled promise rejection messages.

Show stack traces in console for unhandled promise rejection messages.

BUG=393913
R=pfeldman@chromium.org, yurys@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183810

[yurys, 2014-10-16 07:43:45]

lgtm

https://codereview.chromium.org/657063002/diff/1/LayoutTests/inspector/consol...
File LayoutTests/inspector/console/console-uncaught-promise.html (right):

https://codereview.chromium.org/657063002/diff/1/LayoutTests/inspector/consol...
LayoutTests/inspector/console/console-uncaught-promise.html:39:
setTimeout(function timeout() {
style: { should go on the next line.

https://codereview.chromium.org/657063002/diff/1/LayoutTests/inspector/consol...
LayoutTests/inspector/console/console-uncaught-promise.html:47:
WebInspector.console.showPromise().done();
Why do we need to call done here?

https://codereview.chromium.org/657063002/diff/1/LayoutTests/inspector/source...
File
LayoutTests/inspector/sources/debugger/debugger-uncaught-promise-on-pause.html
(right):

https://codereview.chromium.org/657063002/diff/1/LayoutTests/inspector/source...
LayoutTests/inspector/sources/debugger/debugger-uncaught-promise-on-pause.html:40:
setTimeout(function timeout() {
style: { should go on the next line.

[aandrey, 2014-10-16 09:01:38]

https://codereview.chromium.org/657063002/diff/1/LayoutTests/inspector/consol...
File LayoutTests/inspector/console/console-uncaught-promise.html (right):

https://codereview.chromium.org/657063002/diff/1/LayoutTests/inspector/consol...
LayoutTests/inspector/console/console-uncaught-promise.html:39:
setTimeout(function timeout() {
On 2014/10/16 07:43:44, yurys wrote:
> style: { should go on the next line.

This is a function argument, so it should not. I verified using codesearch.

https://codereview.chromium.org/657063002/diff/1/LayoutTests/inspector/consol...
LayoutTests/inspector/console/console-uncaught-promise.html:47:
WebInspector.console.showPromise().done();
On 2014/10/16 07:43:44, yurys wrote:
> Why do we need to call done here?

It will report error to console, should the promise fail.

https://codereview.chromium.org/657063002/diff/1/LayoutTests/inspector/source...
File
LayoutTests/inspector/sources/debugger/debugger-uncaught-promise-on-pause.html
(right):

https://codereview.chromium.org/657063002/diff/1/LayoutTests/inspector/source...
LayoutTests/inspector/sources/debugger/debugger-uncaught-promise-on-pause.html:40:
setTimeout(function timeout() {
On 2014/10/16 07:43:44, yurys wrote:
> style: { should go on the next line.

Acknowledged.

[aandrey, 2014-10-16 09:01:41]

The CQ bit was checked by aandrey@chromium.org

[commit-bot: I haz the power, 2014-10-16 09:02:36]

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

[commit-bot: I haz the power, 2014-10-16 10:02:06]

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

[commit-bot: I haz the power, 2014-10-16 10:02:07]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/2...)

[aandrey, 2014-10-16 11:18:22]

The CQ bit was checked by aandrey@chromium.org

[commit-bot: I haz the power, 2014-10-16 11:19:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/657063002/20001

[commit-bot: I haz the power, 2014-10-16 12:45:25]

Committed patchset #2 (id:20001) as 183810