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

Unified Diff: components/mus/surfaces/display_compositor.h

Issue 1976663003: Pull parts of TopLevelDisplayClient into DisplayCompositor (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: RegisterSurfaceIdNamespace Created 4 years, 7 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 | « components/mus/surfaces/BUILD.gn ('k') | components/mus/surfaces/display_compositor.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/mus/surfaces/display_compositor.h
diff --git a/components/mus/surfaces/display_compositor.h b/components/mus/surfaces/display_compositor.h
new file mode 100644
index 0000000000000000000000000000000000000000..fc54e710b9f0597f995eef0b4c0ea7fe8ffe3af2
--- /dev/null
+++ b/components/mus/surfaces/display_compositor.h
@@ -0,0 +1,72 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef COMPONENTS_MUS_SURFACES_DISPLAY_COMPOSITOR_H_
+#define COMPONENTS_MUS_SURFACES_DISPLAY_COMPOSITOR_H_
+
+#include "cc/surfaces/surface.h"
+#include "cc/surfaces/surface_factory.h"
+#include "cc/surfaces/surface_factory_client.h"
+#include "cc/surfaces/surface_id_allocator.h"
+#include "components/mus/gles2/gpu_state.h"
+#include "components/mus/surfaces/surfaces_state.h"
+#include "ui/gfx/native_widget_types.h"
+
+namespace mus {
+
+class TopLevelDisplayClient;
+
+// TODO(fsamuel): This should become a mojo interface for the mus-gpu split.
+// TODO(fsamuel): This should not be a SurfaceFactoryClient.
+// The DisplayCompositor receives CompositorFrames from all sources,
+// creates a top-level CompositorFrame once per tick, and generates graphical
+// output.
+class DisplayCompositor : public cc::SurfaceFactoryClient {
+ public:
+ DisplayCompositor(scoped_refptr<base::SingleThreadTaskRunner> task_runner,
+ gfx::AcceleratedWidget widget,
+ const scoped_refptr<GpuState>& gpu_state,
+ const scoped_refptr<SurfacesState>& surfaces_state);
+ ~DisplayCompositor() override;
+
+ // DisplayCompositor embedders submit a CompositorFrame when content on the
+ // display should be changed. A well-behaving embedder should only submit
+ // a CompositorFrame once per BeginFrame tick. The callback is called the
+ // first time this frame is used to draw, or if the frame is discarded.
+ void SubmitCompositorFrame(
+ std::unique_ptr<cc::CompositorFrame> frame,
+ const base::Callback<void(cc::SurfaceDrawStatus)>& callback);
+
+ // TODO(fsamuel): This is used for surface hittesting and should not be
+ // exposed outside of DisplayCompositor.
+ const cc::SurfaceId& surface_id() const { return surface_id_; }
+
+ // This requests the display CompositorFrame be rendered and given to the
+ // callback within CopyOutputRequest.
+ void RequestCopyOfOutput(
+ std::unique_ptr<cc::CopyOutputRequest> output_request);
+
+ // TODO(fsamuel): Invent an async way to create a SurfaceNamespace
+ // A SurfaceNamespace can create CompositorFrameSinks where the client can
+ // make up the ID.
+
+ private:
+ // SurfaceFactoryClient implementation.
+ void ReturnResources(const cc::ReturnedResourceArray& resources) override;
+ void SetBeginFrameSource(cc::BeginFrameSource* begin_frame_source) override;
+
+ scoped_refptr<base::SingleThreadTaskRunner> task_runner_;
+ scoped_refptr<SurfacesState> surfaces_state_;
+ cc::SurfaceFactory factory_;
+ cc::SurfaceIdAllocator allocator_;
+ cc::SurfaceId surface_id_;
+
+ gfx::Size display_size_;
+ std::unique_ptr<TopLevelDisplayClient> display_client_;
+ DISALLOW_COPY_AND_ASSIGN(DisplayCompositor);
+};
+
+} // namespace mus
+
+#endif // COMPONENTS_MUS_SURFACES_DISPLAY_COMPOSITOR_H_
« no previous file with comments | « components/mus/surfaces/BUILD.gn ('k') | components/mus/surfaces/display_compositor.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698