OLD | NEW |
---|---|
(Empty) | |
1 // Copyright (c) 2016 The Chromium Authors. All rights reserved. | |
kylechar
2016/12/08 02:02:46
No (c) in new copyright headers.
kylechar
2016/12/08 02:02:46
File should have same name as class.
| |
2 // Use of this source code is governed by a BSD-style license that can be | |
3 // found in the LICENSE file. | |
4 | |
5 #ifndef CC_SURFACES_SURFACE_EMBEDDING_H_ | |
6 #define CC_SURFACES_SURFACE_EMBEDDING_H_ | |
7 | |
8 #include "cc/output/compositor_frame_metadata.h" | |
9 #include "cc/surfaces/surface_embedding_base.h" | |
10 #include "cc/surfaces/surfaces_export.h" | |
11 | |
12 namespace cc { | |
13 | |
14 class CC_SURFACES_EXPORT SurfaceEmbeddingUsingSequence | |
Fady Samuel
2016/12/07 23:29:05
How about SurfaceSequenceSurfaceEmbedding?
| |
15 : public NON_EXPORTED_BASE(SurfaceEmbedding) { | |
16 public: | |
17 explicit SurfaceEmbeddingUsingSequence(const SurfaceInfo& info) | |
18 : SurfaceEmbedding(info) {} | |
19 | |
20 private: | |
21 void DelegateReferenceImpl(CompositorFrameMetadata* metadata) override; | |
22 void AddReferenceImpl(SurfaceEmbeddingOwner*) override; | |
kylechar
2016/12/08 02:02:46
SurfaceEmbeddingOwners* owner
| |
23 void RemoveReferenceImpl() override; | |
24 virtual void RequireSequence(const SurfaceSequence&) = 0; | |
kylechar
2016/12/08 02:02:45
You should have a virtual destructor for an abstra
| |
25 virtual void SatisfySequence(const SurfaceSequence&) = 0; | |
26 SurfaceSequence seq_; | |
27 }; | |
kylechar
2016/12/08 02:02:46
Again, should this be copyable?
| |
28 | |
29 } // namespace cc | |
30 | |
31 #endif // CC_SURFACES_SURFACE_EMBEDDING_H_ | |
OLD | NEW |