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

Side by Side Diff: cc/output/compositor_frame_metadata.h

Issue 2712983003: [cc] Set BeginFrame sequence numbers on CompositorFrames from Scheduler. (Closed)
Patch Set: sync Created 3 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 unified diff | Download patch
« no previous file with comments | « no previous file | cc/scheduler/scheduler.h » ('j') | cc/scheduler/scheduler.h » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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_OUTPUT_COMPOSITOR_FRAME_METADATA_H_ 5 #ifndef CC_OUTPUT_COMPOSITOR_FRAME_METADATA_H_
6 #define CC_OUTPUT_COMPOSITOR_FRAME_METADATA_H_ 6 #define CC_OUTPUT_COMPOSITOR_FRAME_METADATA_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <vector> 10 #include <vector>
11 11
12 #include "cc/base/cc_export.h" 12 #include "cc/base/cc_export.h"
13 #include "cc/input/selection.h" 13 #include "cc/input/selection.h"
14 #include "cc/output/begin_frame_args.h"
14 #include "cc/surfaces/surface_id.h" 15 #include "cc/surfaces/surface_id.h"
15 #include "third_party/skia/include/core/SkColor.h" 16 #include "third_party/skia/include/core/SkColor.h"
16 #include "ui/events/latency_info.h" 17 #include "ui/events/latency_info.h"
17 #include "ui/gfx/geometry/size_f.h" 18 #include "ui/gfx/geometry/size_f.h"
18 #include "ui/gfx/geometry/vector2d_f.h" 19 #include "ui/gfx/geometry/vector2d_f.h"
19 #include "ui/gfx/selection_bound.h" 20 #include "ui/gfx/selection_bound.h"
20 21
21 namespace cc { 22 namespace cc {
22 23
23 class CC_EXPORT CompositorFrameMetadata { 24 class CC_EXPORT CompositorFrameMetadata {
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
75 76
76 std::vector<ui::LatencyInfo> latency_info; 77 std::vector<ui::LatencyInfo> latency_info;
77 78
78 // This is the set of Surfaces that are referenced by this frame. 79 // This is the set of Surfaces that are referenced by this frame.
79 std::vector<SurfaceId> referenced_surfaces; 80 std::vector<SurfaceId> referenced_surfaces;
80 81
81 // This indicates whether this CompositorFrame can be activated before 82 // This indicates whether this CompositorFrame can be activated before
82 // dependencies have been resolved. 83 // dependencies have been resolved.
83 bool can_activate_before_dependencies = true; 84 bool can_activate_before_dependencies = true;
84 85
86 // Source identifier and sequence number of the BeginFrame that this
87 // CompositorFrame is in response to and sequence number that it confirms.
88 uint32_t begin_frame_source_id = 0;
89 uint64_t begin_frame_sequence_number = BeginFrameArgs::kInvalidFrameNumber;
90 uint64_t latest_confirmed_begin_frame_sequence_number =
91 BeginFrameArgs::kInvalidFrameNumber;
brianderson 2017/02/24 01:08:36 Can this be an instance of the BeginFrameAck struc
Eric Seckler 2017/02/27 11:50:32 Done. Was trying to avoid overhead of the unused f
92
85 private: 93 private:
86 CompositorFrameMetadata(const CompositorFrameMetadata& other); 94 CompositorFrameMetadata(const CompositorFrameMetadata& other);
87 CompositorFrameMetadata operator=(const CompositorFrameMetadata&) = delete; 95 CompositorFrameMetadata operator=(const CompositorFrameMetadata&) = delete;
88 }; 96 };
89 97
90 } // namespace cc 98 } // namespace cc
91 99
92 #endif // CC_OUTPUT_COMPOSITOR_FRAME_METADATA_H_ 100 #endif // CC_OUTPUT_COMPOSITOR_FRAME_METADATA_H_
OLDNEW
« no previous file with comments | « no previous file | cc/scheduler/scheduler.h » ('j') | cc/scheduler/scheduler.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698