Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

FrameTiming build start timestamp fix and add vsync start timestamp #20229

Merged
fluttergithubbot merged 8 commits into from
Aug 8, 2020
Merged

FrameTiming build start timestamp fix and add vsync start timestamp #20229

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
01ace70
Change frame build time start timestamp
CareF Jul 30, 2020
84a11d6
add vsync start to layer_tree
CareF Aug 4, 2020
0ec79db
add test
CareF Aug 4, 2020
35d1bd8
formatting
CareF Aug 4, 2020
87d0982
move tracing
CareF Aug 4, 2020
8c7178c
remove vsync_time method
CareF Aug 4, 2020
3d14526
bug fix and formatting
CareF Aug 4, 2020
ecca65e
update constructor
CareF Aug 6, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
add test
  • Loading branch information
@CareF
CareF committed Aug 6, 2020
commit 0ec79db69cd3181dc30926ab651b4e9c4dd43bf7
2 changes: 1 addition & 1 deletion lib/ui/window.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,7 @@ class FrameTiming {

@override
String toString() {
return '$runtimeType(buildDuration: ${_formatMS(buildDuration)}, rasterDuration: ${_formatMS(rasterDuration)}, totalSpan: ${_formatMS(totalSpan)})';
return '$runtimeType(buildDuration: ${_formatMS(buildDuration)}, rasterDuration: ${_formatMS(rasterDuration)}, vsyncOverhead: ${_formatMS(vsyncOverhead)}, totalSpan: ${_formatMS(totalSpan)})';
}
}

Expand Down
2 changes: 1 addition & 1 deletion lib/web_ui/lib/src/ui/window.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1121,7 +1121,7 @@ class FrameTiming {

@override
String toString() {
return '$runtimeType(buildDuration: ${_formatMS(buildDuration)}, rasterDuration: ${_formatMS(rasterDuration)}, totalSpan: ${_formatMS(totalSpan)})';
return '$runtimeType(buildDuration: ${_formatMS(buildDuration)}, rasterDuration: ${_formatMS(rasterDuration)}, vsyncOverhead: ${_formatMS(vsyncOverhead)}, totalSpan: ${_formatMS(totalSpan)})';
}
}

Expand Down
4 changes: 2 additions & 2 deletions testing/dart/window_test.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,8 @@ void main() {
});

test('FrameTiming.toString has the correct format', () {
final FrameTiming timing = FrameTiming(<int>[1000, 8000, 9000, 19500]);
expect(timing.toString(), 'FrameTiming(buildDuration: 7.0ms, rasterDuration: 10.5ms, totalSpan: 18.5ms)');
final FrameTiming timing = FrameTiming(<int>[500, 1000, 8000, 9000, 19500]);
expect(timing.toString(), 'FrameTiming(buildDuration: 6.5ms, rasterDuration: 10.5ms, vsyncOverhead: 0.5ms, totalSpan: 18.5ms)');
});

test('computePlatformResolvedLocale basic', () {
Expand Down