[DevTools] removed promise instrumentation from blink

[DevTools] removed promise instrumentation from blink

Since our new stacks using asyncTaskCreation we don't need blink instrumentation. asyncTaskCreated will be called as part of promise constructor and corresponded .then call.
Later we add meaningful description for asyncTaskCreation which will replace Async Call with something more specific.

BUG=v8:6189
R=dgozman@chromium.org

[kozy, 2017-05-08 20:11:11]

Description was changed from

==========
[DevTools] removed promise instrumentation from blink

Since we use per .then call instrumentation to build stacks we don't need blink
instrumentation.

BUG=v8:6189
==========

to

==========
[DevTools] removed promise instrumentation from blink

Since we use per .then call instrumentation to build stacks we don't need blink
instrumentation.

BUG=v8:6189
R=dgozman@chromium.org
==========

[kozy, 2017-05-08 20:11:12]

kozyatinskiy@chromium.org changed reviewers:
+ dgozman@chromium.org

[kozy, 2017-05-08 20:11:32]

Dmitry, please take a look.

[kozy, 2017-05-08 20:12:59]

Description was changed from

==========
[DevTools] removed promise instrumentation from blink

Since we use per .then call instrumentation to build stacks we don't need blink
instrumentation.

BUG=v8:6189
R=dgozman@chromium.org
==========

to

==========
[DevTools] removed promise instrumentation from blink

Since our new stacks using asyncTaskCreation we don't need blink
instrumentation. asyncTaskCreated will be called as part of promise constructor
and corresponded .then call.
Later we add meaningful description for asyncTaskCreation which will replace
Async Call with something more specific.

BUG=v8:6189
R=dgozman@chromium.org
==========

[dgozman, 2017-05-08 23:28:05]

Let's wait a bit with landing this, until we are sure about new stacks.

https://codereview.chromium.org/2872723002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/http/tests/inspector/sources/debugger/async-callstack-fetch-expected.txt
(right):

https://codereview.chromium.org/2872723002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/inspector/sources/debugger/async-callstack-fetch-expected.txt:7:
[Async Call]
Why this change?

https://codereview.chromium.org/2872723002/diff/1/third_party/WebKit/Source/b...
File third_party/WebKit/Source/bindings/core/v8/ScriptPromiseResolver.cpp
(left):

https://codereview.chromium.org/2872723002/diff/1/third_party/WebKit/Source/b...
third_party/WebKit/Source/bindings/core/v8/ScriptPromiseResolver.cpp:25:
probe::AsyncTaskScheduled(GetExecutionContext(), "Promise", this);
This also removes trace event.

[dgozman, 2017-07-15 05:52:54]

I think we are pretty confident in new stacks. Should we continue this?