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
remove vsync_time method
  • Loading branch information
@CareF
CareF committed Aug 6, 2020
commit 8c7178cc4a5598807534a3700a18a71f682780d0
1 change: 0 additions & 1 deletion flow/layers/layer_tree.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,6 @@ class LayerTree {
fml::TimePoint target_time);
fml::TimePoint vsync_start() const { return vsync_start_; }
fml::TimeDelta vsync_overhead() const { return build_start_ - vsync_start_; }
fml::TimeDelta vsync_time() const { return build_finish_ - vsync_start_; }
fml::TimePoint build_start() const { return build_start_; }
fml::TimePoint build_finish() const { return build_finish_; }
fml::TimeDelta build_time() const { return build_finish_ - build_start_; }
Expand Down
2 changes: 1 addition & 1 deletion shell/common/rasterizer.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -383,7 +383,7 @@ RasterStatus Rasterizer::DrawToSurface(flutter::LayerTree& layer_tree) {
// There is no way for the compositor to know how long the layer tree
// construction took. Fortunately, the layer tree does. Grab that time
// for instrumentation.
compositor_context_->ui_time().SetLapTime(layer_tree.vsync_time());
compositor_context_->ui_time().SetLapTime(layer_tree.build_time());

auto* external_view_embedder = surface_->GetExternalViewEmbedder();

Expand Down