DevTools: Collect JS samples when recording timeline

DevTools: Collect JS samples when recording timeline

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

[yurys, 2014-08-01 15:29:18]

lgtm

https://codereview.chromium.org/431273005/diff/1/Source/devtools/front_end/ti...
File Source/devtools/front_end/timeline/TimelineFlameChart.js (right):

https://codereview.chromium.org/431273005/diff/1/Source/devtools/front_end/ti...
Source/devtools/front_end/timeline/TimelineFlameChart.js:260:
currentJSInvocationEndTime = e.endTime;
This doesn't seem to work well in case of nested JS invocations.

[alph, 2014-08-01 15:44:45]

https://codereview.chromium.org/431273005/diff/1/Source/devtools/front_end/ti...
File Source/devtools/front_end/timeline/TimelineFlameChart.js (right):

https://codereview.chromium.org/431273005/diff/1/Source/devtools/front_end/ti...
Source/devtools/front_end/timeline/TimelineFlameChart.js:260:
currentJSInvocationEndTime = e.endTime;
On 2014/08/01 15:29:17, yurys wrote:
> This doesn't seem to work well in case of nested JS invocations.

I plan to address nested invocations in a separate patch.

[alph, 2014-08-05 08:17:09]

The CQ bit was checked by alph@chromium.org

[commit-bot: I haz the power, 2014-08-05 08:18:33]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/alph@chromium.org/431273005/40001

[commit-bot: I haz the power, 2014-08-05 12:07:24]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_blink_compile_db...)
  android_chromium_gn_compile_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_chromium_gn_comp...)
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/1...)
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/1...)
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/20865)

[commit-bot: I haz the power, 2014-08-05 13:18:30]

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

[commit-bot: I haz the power, 2014-08-05 13:18:31]

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

[alph, 2014-08-11 14:06:19]

The CQ bit was checked by alph@chromium.org

[commit-bot: I haz the power, 2014-08-11 14:07:25]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/alph@chromium.org/431273005/40001

[commit-bot: I haz the power, 2014-08-11 14:41:05]

Change committed as 179956