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

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

Issue 2897053002: [cc] Remove BeginFrameAck from BFS::DidFinishFrame and update tests. (Closed)
Patch Set: sync for PrimaryBFS 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
« no previous file with comments | « cc/test/fake_external_begin_frame_source.h ('k') | components/exo/surface.cc » ('j') | 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"
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
59 DCHECK(obs); 59 DCHECK(obs);
60 DCHECK(observers_.find(obs) != observers_.end()); 60 DCHECK(observers_.find(obs) != observers_.end());
61 61
62 observers_.erase(obs); 62 observers_.erase(obs);
63 if (observers_.empty()) 63 if (observers_.empty())
64 begin_frame_task_.Cancel(); 64 begin_frame_task_.Cancel();
65 if (client_) 65 if (client_)
66 client_->OnRemoveObserver(obs); 66 client_->OnRemoveObserver(obs);
67 } 67 }
68 68
69 void FakeExternalBeginFrameSource::DidFinishFrame(BeginFrameObserver* obs, 69 void FakeExternalBeginFrameSource::DidFinishFrame(BeginFrameObserver* obs) {}
70 const BeginFrameAck& ack) {
71 last_acks_[obs] = ack;
72 }
73 70
74 bool FakeExternalBeginFrameSource::IsThrottled() const { 71 bool FakeExternalBeginFrameSource::IsThrottled() const {
75 return true; 72 return true;
76 } 73 }
77 74
78 BeginFrameArgs FakeExternalBeginFrameSource::CreateBeginFrameArgs( 75 BeginFrameArgs FakeExternalBeginFrameSource::CreateBeginFrameArgs(
79 BeginFrameArgs::CreationLocation location, 76 BeginFrameArgs::CreationLocation location,
80 base::SimpleTestTickClock* now_src) { 77 base::SimpleTestTickClock* now_src) {
81 return CreateBeginFrameArgsForTesting(location, source_id(), 78 return CreateBeginFrameArgsForTesting(location, source_id(),
82 next_begin_frame_number_++, now_src); 79 next_begin_frame_number_++, now_src);
(...skipping 22 matching lines...) Expand all
105 weak_ptr_factory_.GetWeakPtr())); 102 weak_ptr_factory_.GetWeakPtr()));
106 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask( 103 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
107 FROM_HERE, 104 FROM_HERE,
108 base::BindOnce(begin_frame_task_.callback(), 105 base::BindOnce(begin_frame_task_.callback(),
109 CreateBeginFrameArgs(BEGINFRAME_FROM_HERE)), 106 CreateBeginFrameArgs(BEGINFRAME_FROM_HERE)),
110 base::TimeDelta::FromMilliseconds(milliseconds_per_frame_)); 107 base::TimeDelta::FromMilliseconds(milliseconds_per_frame_));
111 next_begin_frame_number_++; 108 next_begin_frame_number_++;
112 } 109 }
113 110
114 } // namespace cc 111 } // namespace cc
OLDNEW
« no previous file with comments | « cc/test/fake_external_begin_frame_source.h ('k') | components/exo/surface.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698