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

Unified Diff: content/browser/renderer_host/delegated_frame_host.cc

Issue 2834553002: Replace CompositorFrameSinkSupport::WillDrawSurface With RepeatingCallback (Closed)
Patch Set: Address Comments On Documentation Created 3 years, 8 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: content/browser/renderer_host/delegated_frame_host.cc
diff --git a/content/browser/renderer_host/delegated_frame_host.cc b/content/browser/renderer_host/delegated_frame_host.cc
index 2db813487f12ff86f0473fb0bb74568365a24df5..2e3ac0068767a5a20f2be4b1fcb57243bcd773e3 100644
--- a/content/browser/renderer_host/delegated_frame_host.cc
+++ b/content/browser/renderer_host/delegated_frame_host.cc
@@ -505,11 +505,7 @@ void DelegatedFrameHost::ReclaimResources(
void DelegatedFrameHost::WillDrawSurface(const cc::LocalSurfaceId& id,
const gfx::Rect& damage_rect) {
- // Frame subscribers are only interested in changes to the target surface, so
- // do not attempt capture if |damage_rect| is empty. This prevents the draws
- // of parent surfaces from triggering extra frame captures, which can affect
- // smoothness.
- if (id != local_surface_id_ || damage_rect.IsEmpty())
+ if (id != local_surface_id_)
return;
AttemptFrameSubscriberCapture(damage_rect);
}
« no previous file with comments | « cc/surfaces/surface_factory_unittest.cc ('k') | content/browser/renderer_host/render_widget_host_view_aura_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698