Unverified Commit aef7929f authored by Zachary Anderson's avatar Zachary Anderson Committed by GitHub

Report total_ui_gc_time to SkiaPerf (#125919)

Not sure why this wasn't already being reported.
parent 707261fa
...@@ -936,6 +936,7 @@ class PerfTest { ...@@ -936,6 +936,7 @@ class PerfTest {
this.timelineFileName, { this.timelineFileName, {
this.measureCpuGpu = true, this.measureCpuGpu = true,
this.measureMemory = true, this.measureMemory = true,
this.measureTotalGCTime = true,
this.saveTraceFile = false, this.saveTraceFile = false,
this.testDriver, this.testDriver,
this.needsFullTimeline = true, this.needsFullTimeline = true,
...@@ -953,6 +954,7 @@ class PerfTest { ...@@ -953,6 +954,7 @@ class PerfTest {
this.testTarget, { this.testTarget, {
this.measureCpuGpu = false, this.measureCpuGpu = false,
this.measureMemory = false, this.measureMemory = false,
this.measureTotalGCTime = false,
this.testDriver = 'test_driver/e2e_test.dart', this.testDriver = 'test_driver/e2e_test.dart',
this.needsFullTimeline = false, this.needsFullTimeline = false,
this.benchmarkScoreKeys = _kCommonScoreKeys, this.benchmarkScoreKeys = _kCommonScoreKeys,
...@@ -979,6 +981,8 @@ class PerfTest { ...@@ -979,6 +981,8 @@ class PerfTest {
final bool measureCpuGpu; final bool measureCpuGpu;
/// Whether to collect memory metrics. /// Whether to collect memory metrics.
final bool measureMemory; final bool measureMemory;
/// Whether to summarize total GC time on the UI thread from the timeline.
final bool measureTotalGCTime;
/// Whether to collect full timeline, meaning if `--trace-startup` flag is needed. /// Whether to collect full timeline, meaning if `--trace-startup` flag is needed.
final bool needsFullTimeline; final bool needsFullTimeline;
/// Whether to save the trace timeline file `*.timeline.json`. /// Whether to save the trace timeline file `*.timeline.json`.
...@@ -1114,6 +1118,7 @@ class PerfTest { ...@@ -1114,6 +1118,7 @@ class PerfTest {
if (data['90th_percentile_memory_usage'] != null) '90th_percentile_memory_usage', if (data['90th_percentile_memory_usage'] != null) '90th_percentile_memory_usage',
if (data['99th_percentile_memory_usage'] != null) '99th_percentile_memory_usage', if (data['99th_percentile_memory_usage'] != null) '99th_percentile_memory_usage',
], ],
if (measureTotalGCTime) 'total_ui_gc_time',
if (data['30hz_frame_percentage'] != null) '30hz_frame_percentage', if (data['30hz_frame_percentage'] != null) '30hz_frame_percentage',
if (data['60hz_frame_percentage'] != null) '60hz_frame_percentage', if (data['60hz_frame_percentage'] != null) '60hz_frame_percentage',
if (data['80hz_frame_percentage'] != null) '80hz_frame_percentage', if (data['80hz_frame_percentage'] != null) '80hz_frame_percentage',
......
...@@ -52,6 +52,7 @@ void main() { ...@@ -52,6 +52,7 @@ void main() {
'90th_percentile_picture_cache_memory': 1, '90th_percentile_picture_cache_memory': 1,
'99th_percentile_picture_cache_memory': 1, '99th_percentile_picture_cache_memory': 1,
'worst_picture_cache_memory': 1, 'worst_picture_cache_memory': 1,
'total_ui_gc_time': 1,
'new_gen_gc_count': 1, 'new_gen_gc_count': 1,
'old_gen_gc_count': 1, 'old_gen_gc_count': 1,
'average_vsync_transitions_missed': 1, 'average_vsync_transitions_missed': 1,
...@@ -95,6 +96,7 @@ void main() { ...@@ -95,6 +96,7 @@ void main() {
'90th_percentile_picture_cache_memory': 1, '90th_percentile_picture_cache_memory': 1,
'99th_percentile_picture_cache_memory': 1, '99th_percentile_picture_cache_memory': 1,
'worst_picture_cache_memory': 1, 'worst_picture_cache_memory': 1,
'total_ui_gc_time': 1,
'new_gen_gc_count': 1, 'new_gen_gc_count': 1,
'old_gen_gc_count': 1, 'old_gen_gc_count': 1,
'average_vsync_transitions_missed': 1, 'average_vsync_transitions_missed': 1,
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment