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

Unified Diff: content/browser/android/synchronous_compositor_host.h

Issue 2174203002: OnDrawHardware() implementation with async messages (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added flag to branch old and new implementation Created 4 years, 4 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/android/synchronous_compositor_host.h
diff --git a/content/browser/android/synchronous_compositor_host.h b/content/browser/android/synchronous_compositor_host.h
index c864122979a50527f0af860010f4c0cc03d1d807..03dc711e72e3553075ef8e3bdcbecad22a650266 100644
--- a/content/browser/android/synchronous_compositor_host.h
+++ b/content/browser/android/synchronous_compositor_host.h
@@ -58,6 +58,10 @@ class SynchronousCompositorHost : public SynchronousCompositor {
const gfx::Size& viewport_size,
const gfx::Rect& viewport_rect_for_tile_priority,
const gfx::Transform& transform_for_tile_priority) override;
+ SynchronousCompositor::Frame DemandDrawHw_Async(
boliu 2016/08/11 19:23:39 this should not have a return value
ojars 2016/08/23 02:15:42 Done.
+ const gfx::Size& viewport_size,
+ const gfx::Rect& viewport_rect_for_tile_priority,
+ const gfx::Transform& transform_for_tile_priority) override;
bool DemandDrawSw(SkCanvas* canvas) override;
void ReturnResources(uint32_t output_surface_id,
const cc::ReturnedResourceArray& resources) override;
@@ -70,6 +74,8 @@ class SynchronousCompositorHost : public SynchronousCompositor {
void DidOverscroll(const DidOverscrollParams& over_scroll_params);
void DidSendBeginFrame(ui::WindowAndroid* window_android);
bool OnMessageReceived(const IPC::Message& message);
+ void DemandDrawHwReceiveFrame(uint32_t output_surface_id,
boliu 2016/08/11 19:23:39 doesn't need to be public
ojars 2016/08/23 02:15:42 Done.
+ const cc::CompositorFrame&);
boliu 2016/08/11 19:23:39 variable name required, by style guide
ojars 2016/08/23 02:15:42 Done.
// Called by SynchronousCompositorObserver.
int routing_id() const { return routing_id_; }

Powered by Google App Engine
This is Rietveld 408576698