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: cc/test/fake_external_begin_frame_source.cc

Issue 2591013004: [cc] Add and use BeginFrameAck for DidFinishFrame. (Closed)
Patch Set: remove ipc struct traits for BeginFrameAck. 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 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 #include "cc/test/fake_external_begin_frame_source.h" 5 #include "cc/test/fake_external_begin_frame_source.h"
6 6
7 #include "base/location.h" 7 #include "base/location.h"
8 #include "base/single_thread_task_runner.h" 8 #include "base/single_thread_task_runner.h"
9 #include "base/threading/thread_task_runner_handle.h" 9 #include "base/threading/thread_task_runner_handle.h"
10 #include "base/time/time.h" 10 #include "base/time/time.h"
(...skipping 23 matching lines...) Expand all
34 } 34 }
35 } 35 }
36 36
37 void FakeExternalBeginFrameSource::AddObserver(BeginFrameObserver* obs) { 37 void FakeExternalBeginFrameSource::AddObserver(BeginFrameObserver* obs) {
38 DCHECK(obs); 38 DCHECK(obs);
39 DCHECK(observers_.find(obs) == observers_.end()); 39 DCHECK(observers_.find(obs) == observers_.end());
40 40
41 bool observers_was_empty = observers_.empty(); 41 bool observers_was_empty = observers_.empty();
42 observers_.insert(obs); 42 observers_.insert(obs);
43 obs->OnBeginFrameSourcePausedChanged(paused_); 43 obs->OnBeginFrameSourcePausedChanged(paused_);
44 if (observers_was_empty && tick_automatically_) 44 if (observers_was_empty && tick_automatically_) {
45 PostTestOnBeginFrame(); 45 PostTestOnBeginFrame();
46 } else if (current_args_.IsValid()) {
47 const BeginFrameArgs& last_args = obs->LastUsedBeginFrameArgs();
48 if (!last_args.IsValid() ||
49 last_args.frame_time < current_args_.frame_time) {
50 current_args_.type = BeginFrameArgs::MISSED;
51 obs->OnBeginFrame(current_args_);
52 }
53 }
46 if (client_) 54 if (client_)
47 client_->OnAddObserver(obs); 55 client_->OnAddObserver(obs);
48 } 56 }
49 57
50 void FakeExternalBeginFrameSource::RemoveObserver(BeginFrameObserver* obs) { 58 void FakeExternalBeginFrameSource::RemoveObserver(BeginFrameObserver* obs) {
51 DCHECK(obs); 59 DCHECK(obs);
52 DCHECK(observers_.find(obs) != observers_.end()); 60 DCHECK(observers_.find(obs) != observers_.end());
53 61
54 observers_.erase(obs); 62 observers_.erase(obs);
55 if (observers_.empty()) 63 if (observers_.empty())
56 begin_frame_task_.Cancel(); 64 begin_frame_task_.Cancel();
57 if (client_) 65 if (client_)
58 client_->OnRemoveObserver(obs); 66 client_->OnRemoveObserver(obs);
59 } 67 }
60 68
61 bool FakeExternalBeginFrameSource::IsThrottled() const { 69 bool FakeExternalBeginFrameSource::IsThrottled() const {
62 return true; 70 return true;
63 } 71 }
64 72
65 void FakeExternalBeginFrameSource::TestOnBeginFrame( 73 void FakeExternalBeginFrameSource::TestOnBeginFrame(
66 const BeginFrameArgs& args) { 74 const BeginFrameArgs& args) {
67 DCHECK(CalledOnValidThread()); 75 DCHECK(CalledOnValidThread());
68 BeginFrameArgs modified_args = args; 76 current_args_ = args;
69 modified_args.source_id = source_id(); 77 current_args_.source_id = source_id();
70 modified_args.sequence_number = next_begin_frame_number_++; 78 current_args_.sequence_number = next_begin_frame_number_++;
71 std::set<BeginFrameObserver*> observers(observers_); 79 std::set<BeginFrameObserver*> observers(observers_);
72 for (auto* obs : observers) 80 for (auto* obs : observers)
73 obs->OnBeginFrame(modified_args); 81 obs->OnBeginFrame(current_args_);
74 if (tick_automatically_) 82 if (tick_automatically_)
75 PostTestOnBeginFrame(); 83 PostTestOnBeginFrame();
76 } 84 }
77 85
78 void FakeExternalBeginFrameSource::PostTestOnBeginFrame() { 86 void FakeExternalBeginFrameSource::PostTestOnBeginFrame() {
79 begin_frame_task_.Reset( 87 begin_frame_task_.Reset(
80 base::Bind(&FakeExternalBeginFrameSource::TestOnBeginFrame, 88 base::Bind(&FakeExternalBeginFrameSource::TestOnBeginFrame,
81 weak_ptr_factory_.GetWeakPtr())); 89 weak_ptr_factory_.GetWeakPtr()));
82 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask( 90 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
83 FROM_HERE, base::Bind(begin_frame_task_.callback(), 91 FROM_HERE, base::Bind(begin_frame_task_.callback(),
84 CreateBeginFrameArgsForTesting( 92 CreateBeginFrameArgsForTesting(
85 BEGINFRAME_FROM_HERE, source_id(), 93 BEGINFRAME_FROM_HERE, source_id(),
86 next_begin_frame_number_)), 94 next_begin_frame_number_)),
87 base::TimeDelta::FromMilliseconds(milliseconds_per_frame_)); 95 base::TimeDelta::FromMilliseconds(milliseconds_per_frame_));
88 next_begin_frame_number_++; 96 next_begin_frame_number_++;
89 } 97 }
90 98
91 } // namespace cc 99 } // namespace cc
OLDNEW
« no previous file with comments | « cc/test/fake_external_begin_frame_source.h ('k') | content/renderer/gpu/compositor_external_begin_frame_source.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698