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

Side by Side Diff: services/ui/surfaces/gpu_compositor_frame_sink.h

Issue 2632273003: Move ReferencedSurfaceTracker into GpuCompositorFrameSink. (Closed)
Patch Set: Fix iterator invalidation. Created 3 years, 11 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef SERVICES_UI_SURFACES_GPU_COMPOSITOR_FRAME_SINK_H_ 5 #ifndef SERVICES_UI_SURFACES_GPU_COMPOSITOR_FRAME_SINK_H_
6 #define SERVICES_UI_SURFACES_GPU_COMPOSITOR_FRAME_SINK_H_ 6 #define SERVICES_UI_SURFACES_GPU_COMPOSITOR_FRAME_SINK_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <vector> 9 #include <vector>
10 10
11 #include "base/macros.h" 11 #include "base/macros.h"
12 #include "base/memory/weak_ptr.h" 12 #include "base/memory/weak_ptr.h"
13 #include "cc/ipc/display_compositor.mojom.h" 13 #include "cc/ipc/display_compositor.mojom.h"
14 #include "cc/ipc/mojo_compositor_frame_sink.mojom.h" 14 #include "cc/ipc/mojo_compositor_frame_sink.mojom.h"
15 #include "cc/surfaces/compositor_frame_sink_support.h" 15 #include "cc/surfaces/compositor_frame_sink_support.h"
16 #include "cc/surfaces/compositor_frame_sink_support_client.h" 16 #include "cc/surfaces/compositor_frame_sink_support_client.h"
17 #include "cc/surfaces/referenced_surface_tracker.h"
17 #include "mojo/public/cpp/bindings/binding.h" 18 #include "mojo/public/cpp/bindings/binding.h"
18 19
19 namespace cc { 20 namespace cc {
20 class Display; 21 class Display;
21 } 22 }
22 23
23 namespace gfx { 24 namespace gfx {
24 class Size; 25 class Size;
25 class ColorSpace; 26 class ColorSpace;
26 } 27 }
(...skipping 18 matching lines...) Expand all
45 cc::mojom::MojoCompositorFrameSinkClientPtr client, 46 cc::mojom::MojoCompositorFrameSinkClientPtr client,
46 cc::mojom::DisplayPrivateRequest display_private_request); 47 cc::mojom::DisplayPrivateRequest display_private_request);
47 48
48 ~GpuCompositorFrameSink() override; 49 ~GpuCompositorFrameSink() override;
49 50
50 // cc::mojom::MojoCompositorFrameSink: 51 // cc::mojom::MojoCompositorFrameSink:
51 void EvictFrame() override; 52 void EvictFrame() override;
52 void SetNeedsBeginFrame(bool needs_begin_frame) override; 53 void SetNeedsBeginFrame(bool needs_begin_frame) override;
53 void SubmitCompositorFrame(const cc::LocalFrameId& local_frame_id, 54 void SubmitCompositorFrame(const cc::LocalFrameId& local_frame_id,
54 cc::CompositorFrame frame) override; 55 cc::CompositorFrame frame) override;
55 void AddSurfaceReferences(
56 const std::vector<cc::SurfaceReference>& references) override;
57 void RemoveSurfaceReferences(
58 const std::vector<cc::SurfaceReference>& references) override;
59 void Require(const cc::LocalFrameId& local_frame_id, 56 void Require(const cc::LocalFrameId& local_frame_id,
60 const cc::SurfaceSequence& sequence) override; 57 const cc::SurfaceSequence& sequence) override;
61 void Satisfy(const cc::SurfaceSequence& sequence) override; 58 void Satisfy(const cc::SurfaceSequence& sequence) override;
62 59
63 // cc::mojom::MojoCompositorFrameSinkPrivate: 60 // cc::mojom::MojoCompositorFrameSinkPrivate:
64 void AddChildFrameSink(const cc::FrameSinkId& child_frame_sink_id) override; 61 void AddChildFrameSink(const cc::FrameSinkId& child_frame_sink_id) override;
65 void RemoveChildFrameSink( 62 void RemoveChildFrameSink(
66 const cc::FrameSinkId& child_frame_sink_id) override; 63 const cc::FrameSinkId& child_frame_sink_id) override;
67 64
68 // cc::mojom::DisplayPrivate: 65 // cc::mojom::DisplayPrivate:
69 void SetDisplayVisible(bool visible) override; 66 void SetDisplayVisible(bool visible) override;
70 void ResizeDisplay(const gfx::Size& size) override; 67 void ResizeDisplay(const gfx::Size& size) override;
71 void SetDisplayColorSpace(const gfx::ColorSpace& color_space) override; 68 void SetDisplayColorSpace(const gfx::ColorSpace& color_space) override;
72 void SetOutputIsSecure(bool secure) override; 69 void SetOutputIsSecure(bool secure) override;
73 70
74 private: 71 private:
75 // cc::CompositorFrameSinkSupportClient implementation: 72 // cc::CompositorFrameSinkSupportClient implementation:
76 void DidReceiveCompositorFrameAck() override; 73 void DidReceiveCompositorFrameAck() override;
77 void OnBeginFrame(const cc::BeginFrameArgs& args) override; 74 void OnBeginFrame(const cc::BeginFrameArgs& args) override;
78 void ReclaimResources(const cc::ReturnedResourceArray& resources) override; 75 void ReclaimResources(const cc::ReturnedResourceArray& resources) override;
79 void WillDrawSurface() override; 76 void WillDrawSurface() override;
80 77
81 void OnClientConnectionLost(); 78 void OnClientConnectionLost();
82 void OnPrivateConnectionLost(); 79 void OnPrivateConnectionLost();
83 80
84 DisplayCompositor* const display_compositor_; 81 DisplayCompositor* const display_compositor_;
85 82
86 cc::CompositorFrameSinkSupport support_; 83 cc::CompositorFrameSinkSupport support_;
87 84
85 // Track the surface references for the surface corresponding to this
86 // compositor frame sink.
87 cc::ReferencedSurfaceTracker surface_tracker_;
88
88 bool client_connection_lost_ = false; 89 bool client_connection_lost_ = false;
89 bool private_connection_lost_ = false; 90 bool private_connection_lost_ = false;
90 91
91 cc::mojom::MojoCompositorFrameSinkClientPtr client_; 92 cc::mojom::MojoCompositorFrameSinkClientPtr client_;
92 mojo::Binding<cc::mojom::MojoCompositorFrameSink> binding_; 93 mojo::Binding<cc::mojom::MojoCompositorFrameSink> binding_;
93 mojo::Binding<cc::mojom::MojoCompositorFrameSinkPrivate> 94 mojo::Binding<cc::mojom::MojoCompositorFrameSinkPrivate>
94 compositor_frame_sink_private_binding_; 95 compositor_frame_sink_private_binding_;
95 mojo::Binding<cc::mojom::DisplayPrivate> display_private_binding_; 96 mojo::Binding<cc::mojom::DisplayPrivate> display_private_binding_;
96 97
97 DISALLOW_COPY_AND_ASSIGN(GpuCompositorFrameSink); 98 DISALLOW_COPY_AND_ASSIGN(GpuCompositorFrameSink);
98 }; 99 };
99 100
100 } // namespace ui 101 } // namespace ui
101 102
102 #endif // SERVICES_UI_SURFACES_GPU_COMPOSITOR_FRAME_SINK_H_ 103 #endif // SERVICES_UI_SURFACES_GPU_COMPOSITOR_FRAME_SINK_H_
OLDNEW
« no previous file with comments | « services/ui/surfaces/display_compositor.cc ('k') | services/ui/surfaces/gpu_compositor_frame_sink.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698