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

Side by Side Diff: cc/surfaces/direct_surface_reference_factory.cc

Issue 2614423003: Adding SatisfySequence and RequireSequence to SurfaceManager (Closed)
Patch Set: 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 #include "cc/surfaces/direct_surface_reference_factory.h" 5 #include "cc/surfaces/direct_surface_reference_factory.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "cc/surfaces/surface.h" 9 #include "cc/surfaces/surface.h"
10 10
11 namespace cc { 11 namespace cc {
12 12
13 DirectSurfaceReferenceFactory::DirectSurfaceReferenceFactory( 13 DirectSurfaceReferenceFactory::DirectSurfaceReferenceFactory(
14 base::WeakPtr<SurfaceManager> manager) 14 base::WeakPtr<SurfaceManager> manager)
15 : manager_(manager) {} 15 : manager_(manager) {}
16 16
17 DirectSurfaceReferenceFactory::~DirectSurfaceReferenceFactory() = default; 17 DirectSurfaceReferenceFactory::~DirectSurfaceReferenceFactory() = default;
18 void DirectSurfaceReferenceFactory::SatisfySequence( 18 void DirectSurfaceReferenceFactory::SatisfySequence(
19 const SurfaceSequence& sequence) const { 19 const SurfaceSequence& sequence) const {
20 if (!manager_) 20 if (!manager_)
21 return; 21 return;
22 std::vector<uint32_t> sequences; 22 manager_->SatisfySequence(sequence);
23 sequences.push_back(sequence.sequence);
24 manager_->DidSatisfySequences(sequence.frame_sink_id, &sequences);
25 } 23 }
26 24
27 void DirectSurfaceReferenceFactory::RequireSequence( 25 void DirectSurfaceReferenceFactory::RequireSequence(
28 const SurfaceId& surface_id, 26 const SurfaceId& surface_id,
29 const SurfaceSequence& sequence) const { 27 const SurfaceSequence& sequence) const {
30 auto* surface = manager_->GetSurfaceForId(surface_id); 28 manager_->RequireSequence(surface_id, sequence);
31 if (!surface) {
32 LOG(ERROR) << "Attempting to require callback on nonexistent surface";
33 return;
34 }
35 surface->AddDestructionDependency(sequence);
36 } 29 }
37 30
38 } // namespace cc 31 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698