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

Unified Diff: ui/compositor/compositor.h

Issue 11195011: Send vsync timebase updates to the browser compositor (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Use the existing output surface mechanism to feed vsync info into the compositor Created 8 years, 2 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: ui/compositor/compositor.h
diff --git a/ui/compositor/compositor.h b/ui/compositor/compositor.h
index c5dc3ba076a19d948fe0e2a811291ea65d0cd9d9..71ddecbcdf2eaca6d50f81c41d864ce7fa78ba7d 100644
--- a/ui/compositor/compositor.h
+++ b/ui/compositor/compositor.h
@@ -54,6 +54,12 @@ class COMPOSITOR_EXPORT ContextFactory {
virtual WebKit::WebGraphicsContext3D* CreateContext(
Compositor* compositor) = 0;
+ // Creates an output surface for the given compositor. The factory may keep
+ // per-compositor data (e.g. a shared context), that needs to be cleaned up
+ // by calling RemoveCompositor when the compositor gets destroyed.
+ virtual WebKit::WebCompositorOutputSurface* CreateOutputSurface(
+ Compositor* compositor) = 0;
+
// Creates a context used for offscreen rendering. This context can be shared
// with all compositors.
virtual WebKit::WebGraphicsContext3D* CreateOffscreenContext() = 0;
@@ -71,6 +77,8 @@ class COMPOSITOR_EXPORT DefaultContextFactory : public ContextFactory {
// ContextFactory implementation
virtual WebKit::WebGraphicsContext3D* CreateContext(
Compositor* compositor) OVERRIDE;
+ virtual WebKit::WebCompositorOutputSurface* CreateOutputSurface(
+ Compositor* compositor) OVERRIDE;
virtual WebKit::WebGraphicsContext3D* CreateOffscreenContext() OVERRIDE;
virtual void RemoveCompositor(Compositor* compositor) OVERRIDE;

Powered by Google App Engine
This is Rietveld 408576698