Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3973)

Unified Diff: cc/surfaces/surface_aggregator.cc

Issue 2676373004: Implement service-side surface synchronization (Closed)
Patch Set: Better unit test name Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: cc/surfaces/surface_aggregator.cc
diff --git a/cc/surfaces/surface_aggregator.cc b/cc/surfaces/surface_aggregator.cc
index 21e85ca0f610df0fef493a40c9884507e98d21ed..82bc5003a195841ab83949557d06f72502e2b0d3 100644
--- a/cc/surfaces/surface_aggregator.cc
+++ b/cc/surfaces/surface_aggregator.cc
@@ -168,9 +168,9 @@ void SurfaceAggregator::HandleSurfaceQuad(
Surface* surface = manager_->GetSurfaceForId(surface_id);
if (!surface)
return;
- if (!surface->HasFrame())
+ if (!surface->HasActiveFrame())
return;
- const CompositorFrame& frame = surface->GetEligibleFrame();
+ const CompositorFrame& frame = surface->GetActiveFrame();
// A map keyed by RenderPass id.
std::multimap<int, std::unique_ptr<CopyOutputRequest>> copy_requests;
@@ -536,9 +536,9 @@ gfx::Rect SurfaceAggregator::PrewalkTree(const SurfaceId& surface_id,
return gfx::Rect();
}
contained_surfaces_[surface_id] = surface->frame_index();
- if (!surface->HasFrame())
+ if (!surface->HasActiveFrame())
return gfx::Rect();
- const CompositorFrame& frame = surface->GetEligibleFrame();
+ const CompositorFrame& frame = surface->GetActiveFrame();
int child_id = 0;
// TODO(jbauman): hack for unit tests that don't set up rp
if (provider_) {
@@ -719,9 +719,9 @@ void SurfaceAggregator::CopyUndrawnSurfaces(PrewalkResult* prewalk_result) {
Surface* surface = manager_->GetSurfaceForId(surface_id);
if (!surface)
continue;
- if (!surface->HasFrame())
+ if (!surface->HasActiveFrame())
continue;
- const CompositorFrame& frame = surface->GetEligibleFrame();
+ const CompositorFrame& frame = surface->GetActiveFrame();
bool surface_has_copy_requests = false;
for (const auto& render_pass : frame.render_pass_list) {
surface_has_copy_requests |= !render_pass->copy_requests.empty();
@@ -769,10 +769,10 @@ CompositorFrame SurfaceAggregator::Aggregate(const SurfaceId& surface_id) {
DCHECK(surface);
contained_surfaces_[surface_id] = surface->frame_index();
- if (!surface->HasFrame())
+ if (!surface->HasActiveFrame())
return CompositorFrame();
- const CompositorFrame& root_surface_frame = surface->GetEligibleFrame();
+ const CompositorFrame& root_surface_frame = surface->GetActiveFrame();
TRACE_EVENT0("cc", "SurfaceAggregator::Aggregate");
CompositorFrame frame;

Powered by Google App Engine
This is Rietveld 408576698