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

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

Issue 2583483002: [cc] Adds source_id and sequence_number to BeginFrameArgs. (Closed)
Patch Set: fix field ordering Created 4 years 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') | cc/test/fake_layer_tree_host_impl.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 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
58 client_->OnRemoveObserver(obs); 58 client_->OnRemoveObserver(obs);
59 } 59 }
60 60
61 bool FakeExternalBeginFrameSource::IsThrottled() const { 61 bool FakeExternalBeginFrameSource::IsThrottled() const {
62 return true; 62 return true;
63 } 63 }
64 64
65 void FakeExternalBeginFrameSource::TestOnBeginFrame( 65 void FakeExternalBeginFrameSource::TestOnBeginFrame(
66 const BeginFrameArgs& args) { 66 const BeginFrameArgs& args) {
67 DCHECK(CalledOnValidThread()); 67 DCHECK(CalledOnValidThread());
68 BeginFrameArgs modified_args = args;
69 modified_args.source_id = source_id();
70 modified_args.sequence_number = next_begin_frame_number_++;
68 std::set<BeginFrameObserver*> observers(observers_); 71 std::set<BeginFrameObserver*> observers(observers_);
69 for (auto* obs : observers) 72 for (auto* obs : observers)
70 obs->OnBeginFrame(args); 73 obs->OnBeginFrame(modified_args);
71 if (tick_automatically_) 74 if (tick_automatically_)
72 PostTestOnBeginFrame(); 75 PostTestOnBeginFrame();
73 } 76 }
74 77
75 void FakeExternalBeginFrameSource::PostTestOnBeginFrame() { 78 void FakeExternalBeginFrameSource::PostTestOnBeginFrame() {
76 begin_frame_task_.Reset( 79 begin_frame_task_.Reset(
77 base::Bind(&FakeExternalBeginFrameSource::TestOnBeginFrame, 80 base::Bind(&FakeExternalBeginFrameSource::TestOnBeginFrame,
78 weak_ptr_factory_.GetWeakPtr())); 81 weak_ptr_factory_.GetWeakPtr()));
79 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask( 82 base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
80 FROM_HERE, 83 FROM_HERE, base::Bind(begin_frame_task_.callback(),
81 base::Bind(begin_frame_task_.callback(), 84 CreateBeginFrameArgsForTesting(
82 CreateBeginFrameArgsForTesting(BEGINFRAME_FROM_HERE)), 85 BEGINFRAME_FROM_HERE, source_id(),
86 next_begin_frame_number_)),
83 base::TimeDelta::FromMilliseconds(milliseconds_per_frame_)); 87 base::TimeDelta::FromMilliseconds(milliseconds_per_frame_));
88 next_begin_frame_number_++;
84 } 89 }
85 90
86 } // namespace cc 91 } // namespace cc
OLDNEW
« no previous file with comments | « cc/test/fake_external_begin_frame_source.h ('k') | cc/test/fake_layer_tree_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698