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

Unified Diff: services/surfaces/surfaces_impl.h

Issue 940293003: Add a Display and ContextProvider concept to mojom, use to recreate (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 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: services/surfaces/surfaces_impl.h
diff --git a/services/surfaces/surfaces_impl.h b/services/surfaces/surfaces_impl.h
index fe3c34dc604b7714e846bc27eaaa389494dd52d2..1c26f3422a14c323ceb5522b11918cc62e01fc74 100644
--- a/services/surfaces/surfaces_impl.h
+++ b/services/surfaces/surfaces_impl.h
@@ -25,18 +25,11 @@ class ApplicationManager;
namespace surfaces {
-class SurfacesImpl : public mojo::Surface,
- public mojo::ViewportParameterListener,
- public cc::SurfaceFactoryClient,
- public cc::DisplayClient {
+class SurfacesImpl : public mojo::Surface, public cc::SurfaceFactoryClient {
public:
class Client {
public:
- virtual void OnVSyncParametersUpdated(base::TimeTicks timebase,
- base::TimeDelta interval) = 0;
virtual void FrameSubmitted() = 0;
- virtual void SetDisplay(cc::Display* display) = 0;
- virtual void OnDisplayBeingDestroyed(cc::Display* display) = 0;
};
SurfacesImpl(cc::SurfaceManager* manager,
@@ -59,28 +52,10 @@ class SurfacesImpl : public mojo::Surface,
mojo::FramePtr frame,
const mojo::Closure& callback) override;
void DestroySurface(uint32_t local_id) override;
- void CreateGLES2BoundSurface(
- mojo::CommandBufferPtr gles2_client,
- uint32_t local_id,
- mojo::SizePtr size,
- mojo::InterfaceRequest<mojo::ViewportParameterListener> listener_request)
- override;
// SurfaceFactoryClient implementation.
void ReturnResources(const cc::ReturnedResourceArray& resources) override;
- // DisplayClient implementation.
- void DisplayDamaged() override;
- void DidSwapBuffers() override;
- void DidSwapBuffersComplete() override;
- void CommitVSyncParameters(base::TimeTicks timebase,
- base::TimeDelta interval) override;
- void OutputSurfaceLost() override;
- void SetMemoryPolicy(const cc::ManagedMemoryPolicy& policy) override;
-
- // ViewportParameterListener
- void OnVSyncParametersUpdated(int64_t timebase, int64_t interval) override;
-
cc::SurfaceFactory* factory() { return &factory_; }
private:
@@ -94,10 +69,7 @@ class SurfacesImpl : public mojo::Surface,
cc::SurfaceFactory factory_;
const uint32_t id_namespace_;
Client* client_;
- uint32_t displayed_surface_;
- scoped_ptr<cc::Display> display_;
mojo::ScopedMessagePipeHandle command_buffer_handle_;
- mojo::WeakBindingSet<ViewportParameterListener> parameter_listeners_;
mojo::ResourceReturnerPtr returner_;
mojo::StrongBinding<Surface> binding_;

Powered by Google App Engine
This is Rietveld 408576698