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

Side by Side Diff: cc/trees/layer_tree_host_unittest.cc

Issue 2779633002: [cc] Remove remaining_frames from BeginFrameAck. (Closed)
Patch Set: Created 3 years, 8 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/trees/layer_tree_host_impl_unittest.cc ('k') | cc/trees/single_thread_proxy.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 2011 The Chromium Authors. All rights reserved. 1 // Copyright 2011 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/trees/layer_tree_host.h" 5 #include "cc/trees/layer_tree_host.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <algorithm> 10 #include <algorithm>
(...skipping 7077 matching lines...) Expand 10 before | Expand all | Expand 10 after
7088 frame_data_ = frame_data; 7088 frame_data_ = frame_data;
7089 return draw_result; 7089 return draw_result;
7090 } 7090 }
7091 7091
7092 void DisplayReceivedCompositorFrameOnThread( 7092 void DisplayReceivedCompositorFrameOnThread(
7093 const CompositorFrame& frame) override { 7093 const CompositorFrame& frame) override {
7094 if (compositor_frame_submitted_) 7094 if (compositor_frame_submitted_)
7095 return; 7095 return;
7096 compositor_frame_submitted_ = true; 7096 compositor_frame_submitted_ = true;
7097 7097
7098 EXPECT_EQ( 7098 EXPECT_EQ(BeginFrameAck(
7099 BeginFrameAck(current_begin_frame_args_.source_id, 7099 current_begin_frame_args_.source_id,
7100 current_begin_frame_args_.sequence_number, 7100 current_begin_frame_args_.sequence_number,
7101 current_begin_main_frame_args_on_impl_.sequence_number, 0, 7101 current_begin_main_frame_args_on_impl_.sequence_number, true),
7102 true), 7102 frame.metadata.begin_frame_ack);
7103 frame.metadata.begin_frame_ack);
7104 } 7103 }
7105 7104
7106 void DrawLayersOnThread(LayerTreeHostImpl* impl) override { 7105 void DrawLayersOnThread(LayerTreeHostImpl* impl) override {
7107 if (layers_drawn_) 7106 if (layers_drawn_)
7108 return; 7107 return;
7109 layers_drawn_ = true; 7108 layers_drawn_ = true;
7110 7109
7111 EXPECT_TRUE(frame_data_); 7110 EXPECT_TRUE(frame_data_);
7112 EXPECT_TRUE(compositor_frame_submitted_); 7111 EXPECT_TRUE(compositor_frame_submitted_);
7113 EXPECT_EQ( 7112 EXPECT_EQ(BeginFrameAck(
7114 BeginFrameAck(current_begin_frame_args_.source_id, 7113 current_begin_frame_args_.source_id,
7115 current_begin_frame_args_.sequence_number, 7114 current_begin_frame_args_.sequence_number,
7116 current_begin_main_frame_args_on_impl_.sequence_number, 0, 7115 current_begin_main_frame_args_on_impl_.sequence_number, true),
7117 true), 7116 frame_data_->begin_frame_ack);
7118 frame_data_->begin_frame_ack);
7119 EndTest(); 7117 EndTest();
7120 } 7118 }
7121 7119
7122 void AfterTest() override { EXPECT_TRUE(layers_drawn_); } 7120 void AfterTest() override { EXPECT_TRUE(layers_drawn_); }
7123 7121
7124 private: 7122 private:
7125 bool compositor_frame_submitted_ = false; 7123 bool compositor_frame_submitted_ = false;
7126 bool layers_drawn_ = false; 7124 bool layers_drawn_ = false;
7127 BeginFrameArgs current_begin_frame_args_; 7125 BeginFrameArgs current_begin_frame_args_;
7128 BeginFrameArgs current_begin_main_frame_args_; 7126 BeginFrameArgs current_begin_main_frame_args_;
(...skipping 100 matching lines...) Expand 10 before | Expand all | Expand 10 after
7229 DCHECK_EQ(hud->scroll_tree_index(), root_layer->scroll_tree_index()); 7227 DCHECK_EQ(hud->scroll_tree_index(), root_layer->scroll_tree_index());
7230 } 7228 }
7231 7229
7232 void AfterTest() override {} 7230 void AfterTest() override {}
7233 }; 7231 };
7234 7232
7235 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostTestHudLayerWithLayerLists); 7233 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostTestHudLayerWithLayerLists);
7236 7234
7237 } // namespace 7235 } // namespace
7238 } // namespace cc 7236 } // namespace cc
OLDNEW
« no previous file with comments | « cc/trees/layer_tree_host_impl_unittest.cc ('k') | cc/trees/single_thread_proxy.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698