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

Side by Side Diff: cc/surfaces/direct_compositor_frame_sink.h

Issue 2848223003: Enforce constant size and device scale factor for surfaces (Closed)
Patch Set: c Created 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 CC_SURFACES_DIRECT_COMPOSITOR_FRAME_SINK_H_ 5 #ifndef CC_SURFACES_DIRECT_COMPOSITOR_FRAME_SINK_H_
6 #define CC_SURFACES_DIRECT_COMPOSITOR_FRAME_SINK_H_ 6 #define CC_SURFACES_DIRECT_COMPOSITOR_FRAME_SINK_H_
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "base/threading/thread_checker.h" 9 #include "base/threading/thread_checker.h"
10 #include "cc/output/compositor_frame_sink.h" 10 #include "cc/output/compositor_frame_sink.h"
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
69 const gfx::Rect& damage_rect) override; 69 const gfx::Rect& damage_rect) override;
70 70
71 // ExternalBeginFrameSourceClient implementation: 71 // ExternalBeginFrameSourceClient implementation:
72 void OnNeedsBeginFrames(bool needs_begin_frame) override; 72 void OnNeedsBeginFrames(bool needs_begin_frame) override;
73 void OnDidFinishFrame(const BeginFrameAck& ack) override; 73 void OnDidFinishFrame(const BeginFrameAck& ack) override;
74 74
75 // This class is only meant to be used on a single thread. 75 // This class is only meant to be used on a single thread.
76 base::ThreadChecker thread_checker_; 76 base::ThreadChecker thread_checker_;
77 77
78 const FrameSinkId frame_sink_id_; 78 const FrameSinkId frame_sink_id_;
79 LocalSurfaceId delegated_local_surface_id_; 79 LocalSurfaceId local_surface_id_;
80 SurfaceManager* surface_manager_; 80 SurfaceManager* surface_manager_;
81 LocalSurfaceIdAllocator local_surface_id_allocator_; 81 LocalSurfaceIdAllocator local_surface_id_allocator_;
82 Display* display_; 82 Display* display_;
83 gfx::Size last_swap_frame_size_; 83 gfx::Size last_swap_frame_size_;
84 float device_scale_factor_ = 0.f;
Fady Samuel 2017/05/17 00:40:50 Hmm, so it's starting out invalid? Maybe start it
Saman Sami 2017/05/17 00:55:17 It'll be updated from the metadata though.
84 bool is_lost_ = false; 85 bool is_lost_ = false;
85 std::unique_ptr<ExternalBeginFrameSource> begin_frame_source_; 86 std::unique_ptr<ExternalBeginFrameSource> begin_frame_source_;
86 87
87 DISALLOW_COPY_AND_ASSIGN(DirectCompositorFrameSink); 88 DISALLOW_COPY_AND_ASSIGN(DirectCompositorFrameSink);
88 }; 89 };
89 90
90 } // namespace cc 91 } // namespace cc
91 92
92 #endif // CC_SURFACES_DIRECT_COMPOSITOR_FRAME_SINK_H_ 93 #endif // CC_SURFACES_DIRECT_COMPOSITOR_FRAME_SINK_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698