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

Side by Side Diff: cc/test/fake_external_begin_frame_source.cc

Issue 2919103002: Replace deprecated base::NonThreadSafe in /cc in favor of SequenceChecker. (Closed)
Patch Set: Created 3 years, 6 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 | « cc/test/fake_external_begin_frame_source.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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"
11 #include "cc/test/begin_frame_args_test.h" 11 #include "cc/test/begin_frame_args_test.h"
12 12
13 namespace cc { 13 namespace cc {
14 14
15 FakeExternalBeginFrameSource::FakeExternalBeginFrameSource( 15 FakeExternalBeginFrameSource::FakeExternalBeginFrameSource(
16 double refresh_rate, 16 double refresh_rate,
17 bool tick_automatically) 17 bool tick_automatically)
18 : tick_automatically_(tick_automatically), 18 : tick_automatically_(tick_automatically),
19 milliseconds_per_frame_(1000.0 / refresh_rate), 19 milliseconds_per_frame_(1000.0 / refresh_rate),
20 weak_ptr_factory_(this) { 20 weak_ptr_factory_(this) {
21 DetachFromThread(); 21 DETACH_FROM_SEQUENCE(sequence_checker_);
22 } 22 }
23 23
24 FakeExternalBeginFrameSource::~FakeExternalBeginFrameSource() { 24 FakeExternalBeginFrameSource::~FakeExternalBeginFrameSource() {
25 DCHECK(CalledOnValidThread()); 25 DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
26 } 26 }
27 27
28 void FakeExternalBeginFrameSource::SetPaused(bool paused) { 28 void FakeExternalBeginFrameSource::SetPaused(bool paused) {
29 if (paused != paused_) { 29 if (paused != paused_) {
30 paused_ = paused; 30 paused_ = paused;
31 std::set<BeginFrameObserver*> observers(observers_); 31 std::set<BeginFrameObserver*> observers(observers_);
32 for (auto* obs : observers) 32 for (auto* obs : observers)
33 obs->OnBeginFrameSourcePausedChanged(paused_); 33 obs->OnBeginFrameSourcePausedChanged(paused_);
34 } 34 }
35 } 35 }
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
80 } 80 }
81 81
82 BeginFrameArgs FakeExternalBeginFrameSource::CreateBeginFrameArgs( 82 BeginFrameArgs FakeExternalBeginFrameSource::CreateBeginFrameArgs(
83 BeginFrameArgs::CreationLocation location) { 83 BeginFrameArgs::CreationLocation location) {
84 return CreateBeginFrameArgsForTesting(location, source_id(), 84 return CreateBeginFrameArgsForTesting(location, source_id(),
85 next_begin_frame_number_++); 85 next_begin_frame_number_++);
86 } 86 }
87 87
88 void FakeExternalBeginFrameSource::TestOnBeginFrame( 88 void FakeExternalBeginFrameSource::TestOnBeginFrame(
89 const BeginFrameArgs& args) { 89 const BeginFrameArgs& args) {
90 DCHECK(CalledOnValidThread()); 90 DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
91 current_args_ = args; 91 current_args_ = args;
92 std::set<BeginFrameObserver*> observers(observers_); 92 std::set<BeginFrameObserver*> observers(observers_);
93 for (auto* obs : observers) 93 for (auto* obs : observers)
94 obs->OnBeginFrame(current_args_); 94 obs->OnBeginFrame(current_args_);
95 if (tick_automatically_) 95 if (tick_automatically_)
96 PostTestOnBeginFrame(); 96 PostTestOnBeginFrame();
97 } 97 }
98 98
99 void FakeExternalBeginFrameSource::PostTestOnBeginFrame() { 99 void FakeExternalBeginFrameSource::PostTestOnBeginFrame() {
100 begin_frame_task_.Reset( 100 begin_frame_task_.Reset(
101 base::Bind(&FakeExternalBeginFrameSource::TestOnBeginFrame, 101 base::Bind(&FakeExternalBeginFrameSource::TestOnBeginFrame,
102 weak_ptr_factory_.GetWeakPtr())); 102 weak_ptr_factory_.GetWeakPtr()));
103 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask( 103 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
104 FROM_HERE, 104 FROM_HERE,
105 base::BindOnce(begin_frame_task_.callback(), 105 base::BindOnce(begin_frame_task_.callback(),
106 CreateBeginFrameArgs(BEGINFRAME_FROM_HERE)), 106 CreateBeginFrameArgs(BEGINFRAME_FROM_HERE)),
107 base::TimeDelta::FromMilliseconds(milliseconds_per_frame_)); 107 base::TimeDelta::FromMilliseconds(milliseconds_per_frame_));
108 next_begin_frame_number_++; 108 next_begin_frame_number_++;
109 } 109 }
110 110
111 } // namespace cc 111 } // namespace cc
OLDNEW
« no previous file with comments | « cc/test/fake_external_begin_frame_source.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698