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

Unified Diff: ui/compositor/test/in_process_context_factory.cc

Issue 2096493002: Make cc::CompositorFrames movable [Part 1 of 2] (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed Dana's nits Created 4 years, 6 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
« no previous file with comments | « content/test/test_render_view_host.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/compositor/test/in_process_context_factory.cc
diff --git a/ui/compositor/test/in_process_context_factory.cc b/ui/compositor/test/in_process_context_factory.cc
index f396c941be4de6ee578e8284706b6b662640784a..f923cd385d483e136a72e1643b86763f86ac8fda 100644
--- a/ui/compositor/test/in_process_context_factory.cc
+++ b/ui/compositor/test/in_process_context_factory.cc
@@ -65,15 +65,15 @@ class DirectOutputSurface : public cc::OutputSurface {
return false;
return true;
}
- void SwapBuffers(cc::CompositorFrame* frame) override {
+ void SwapBuffers(cc::CompositorFrame frame) override {
DCHECK(context_provider_.get());
- DCHECK(frame->gl_frame_data);
- if (frame->gl_frame_data->sub_buffer_rect ==
- gfx::Rect(frame->gl_frame_data->size)) {
+ DCHECK(frame.gl_frame_data);
+ if (frame.gl_frame_data->sub_buffer_rect ==
+ gfx::Rect(frame.gl_frame_data->size)) {
context_provider_->ContextSupport()->Swap();
} else {
context_provider_->ContextSupport()->PartialSwapBuffers(
- frame->gl_frame_data->sub_buffer_rect);
+ frame.gl_frame_data->sub_buffer_rect);
}
gpu::gles2::GLES2Interface* gl = context_provider_->ContextGL();
const uint64_t fence_sync = gl->InsertFenceSyncCHROMIUM();
« no previous file with comments | « content/test/test_render_view_host.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698