Bug 88003 – Chromium Console.time() and timeEnd() Should Support Async Traces

• Component: WebCore JavaScript
• Status: RESOLVED FIXED

[Kevin Greer]

When the Console.time/timerEnd() functions are called on the chromium platform, calls to chrome's TRACE_EVENT methods are called.
The versions being called only supported nested synchronous traces and should be switched to the versions that also support asynchronous un-nested traces.

[Kevin Greer]

Created attachment 145456 [details]
Patch

[Nat Duca]

Comment on attachment 145456 [details]
Patch

LGTM. +pfeldman for review.

[WebKit Review Bot]

Comment on attachment 145456 [details]
Patch

Clearing flags on attachment: 145456

Committed r119418: <http://trac.webkit.org/changeset/119418>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Dana Jansens]

This seems to have caused a failure in the browser_tests test GpuFeatureTest.RafNoDamage:

[12655:-1597528768:0604/134340:2263335561903:ERROR:trace_event_analyzer.cc(827)] Not enough events: 0
../../chrome/test/gpu/gpu_feature_browsertest.cc:425: Failure
Value of: trace_analyzer::GetRateStats(events, &stats, &stats_options)
  Actual: false
Expected: true
[12655:-1597528768:0604/134340:2263335777689:INFO:gpu_feature_browsertest.cc(427)] Number of RAFs: 0 Mean: 6.36069e-307 Min: -1.99068 Max: 2.56952e-256 StdDev: 2.57917e-256
../../chrome/test/gpu/gpu_feature_browsertest.cc:435: Failure
Expected: (stats.mean_us) > (15000.0), actual: 6.36069e-307 vs 15000


I bisected locally to find this change being the cause.

[Ryosuke Niwa]

Could you look into this issue? I'm going to roll out the patch I don't hear back from you in the next half an hour or so.

[James Robinson]

Looks like the chromium-side test should be updated to look for async trace events instead of normal matched ones.

Please don't revert, this patch is fine.

[Ryosuke Niwa]

(In reply to comment #7)
> Looks like the chromium-side test should be updated to look for async trace events instead of normal matched ones.
> 
> Please don't revert, this patch is fine.

Okay. Could you make that change? I have no idea what this test is testing, and don't know how to do that.

[John Bates]

Fix on the way:
https://code.google.com/p/chromium/issues/detail?id=131097

[Ryosuke Niwa]

(In reply to comment #9)
> Fix on the way:
> https://code.google.com/p/chromium/issues/detail?id=131097

Great! Thank you.