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

Unified Diff: cc/output/output_surface_frame.h

Issue 2693023002: Use SwapBuffersWithBounds on Chromecast (Closed)
Patch Set: Address danakj comments on PS1 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/output/output_surface_frame.h
diff --git a/cc/output/output_surface_frame.h b/cc/output/output_surface_frame.h
index 2306f2e5b4bbe686807d324dd438eee65266d245..a2437c2f51cd3bd59c051bdb7f5d9f5f50208170 100644
--- a/cc/output/output_surface_frame.h
+++ b/cc/output/output_surface_frame.h
@@ -26,7 +26,12 @@ class CC_EXPORT OutputSurfaceFrame {
OutputSurfaceFrame& operator=(OutputSurfaceFrame&& other);
gfx::Size size;
+ // |sub_buffer_rect| = swap rect for PartialSwap if != surface size.
+ // |content_bounds| = content area for SwapWithBounds if non-empty; rectangles
+ // may overlap. If neither is provided, use standard swap. Providing both
+ // |sub_buffer_rect| and |content_bounds| is not supported.
danakj 2017/02/16 18:54:19 I think this is a little misleading. sub_buffer_re
halliwell 2017/02/17 14:08:45 Yep, looked at this a little closer: * Optional do
halliwell 2017/02/23 18:01:46 Ok, comment updated, and using Optional now.
gfx::Rect sub_buffer_rect;
+ std::vector<gfx::Rect> content_bounds;
std::vector<ui::LatencyInfo> latency_info;
private:

Powered by Google App Engine
This is Rietveld 408576698