DevTools: move tracing based Timeline out of experiment

LayoutTests/inspector/timeline/timeline-coalescing-expected.txt

Index: LayoutTests/inspector/timeline/timeline-coalescing-expected.txt
diff --git a/LayoutTests/inspector/timeline/timeline-coalescing-expected.txt b/LayoutTests/inspector/timeline/timeline-coalescing-expected.txt
index 1cdc3b16b9547d489efa6f727882d5af4af789dd..9683193f33836063a3cb78fb77e999715f733710 100644
--- a/LayoutTests/inspector/timeline/timeline-coalescing-expected.txt
+++ b/LayoutTests/inspector/timeline/timeline-coalescing-expected.txt
@@ -6,22 +6,22 @@ Root
 --------> Layout duration: 99.89900:99.89900 (rendering: 99.89900)
 --------> Layout duration: 99.89900:99.89900 (rendering: 99.89900)
 ----> Layout duration: 94.89900:94.89900 (rendering: 94.89900)
-----> FunctionCall duration: 199.99700:199.99700 (painting: 199.79800, scripting: 0.19900)
---------> DecodeImage x 2 duration: 199.89800:199.89800 (painting: 199.79800)
-------------> DecodeImage duration: 99.89800:99.89800 (painting: 99.89800)
-------------> DecodeImage duration: 99.90000:99.90000 (painting: 99.90000)
-----> FunctionCall duration: 199.99700:199.99700 (painting: 199.79800, scripting: 0.19900)
---------> DecodeImage x 2 duration: 199.89800:199.89800 (painting: 199.79800)
-------------> DecodeImage duration: 99.89800:99.89800 (painting: 99.89800)
-------------> DecodeImage duration: 99.90000:99.90000 (painting: 99.90000)
-----> Rasterize x 3 duration: 299.89900:299.89900 (painting: 299.69700)
---------> Rasterize duration: 99.89900:99.89900 (painting: 99.89900)
---------> Rasterize duration: 99.89900:99.89900 (painting: 99.89900)
---------> Rasterize duration: 99.89900:99.89900 (painting: 99.89900)
+----> FunctionCall duration: 199.99700:199.99700 (scripting: 0.19900, painting: 199.79800)
+--------> Decode Image x 2 duration: 199.89800:199.89800 (painting: 199.79800)
+------------> Decode Image duration: 99.89800:99.89800 (painting: 99.89800)
+------------> Decode Image duration: 99.90000:99.90000 (painting: 99.90000)
+----> FunctionCall duration: 199.99700:199.99700 (scripting: 0.19900, painting: 199.79800)
+--------> Decode Image x 2 duration: 199.89800:199.89800 (painting: 199.79800)
+------------> Decode Image duration: 99.89800:99.89800 (painting: 99.89800)
+------------> Decode Image duration: 99.90000:99.90000 (painting: 99.90000)
+----> RasterTask x 3 duration: 299.89900:299.89900 (painting: 299.69700)
+--------> RasterTask duration: 99.89900:99.89900 (painting: 99.89900)
+--------> RasterTask duration: 99.89900:99.89900 (painting: 99.89900)
+--------> RasterTask duration: 99.89900:99.89900 (painting: 99.89900)
 ----> Paint x 3 duration: 299.89900:299.89900 (painting: 299.69700)
 --------> Paint duration: 99.89900:99.89900 (painting: 99.89900)
 --------> Paint duration: 99.89900:99.89900 (painting: 99.89900)
 --------> Paint duration: 99.89900:99.89900 (painting: 99.89900)
 coalesced record's hasWarnings(): false
-coalesced record's childHasWarnings(): true
+coalesced record's childHasWarnings(): undefined

[alph, 2014/08/21 13:13:39]

shouldn't it be true/false?

[yurys, 2014/08/21 14:18:42]

Good catch. Since the logic we detect synchronous events has changed the test
input data needed to be updated accordingly. We used to mark Layout event as
synchronous if it had non-empty stack-trace, current implementation checks if it
the Layout event is inside some script event as stack collection may be off.