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

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

Issue 2441673003: Disable a remote LTH test, we don't deserialize FakePictureLayer. (Closed)
Patch Set: Created 4 years, 2 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 | « no previous file | 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 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 269 matching lines...) Expand 10 before | Expand all | Expand 10 after
280 EXPECT_TRUE(all_tiles_required_for_activation_are_ready_to_draw_); 280 EXPECT_TRUE(all_tiles_required_for_activation_are_ready_to_draw_);
281 EXPECT_LE(size_t(1), required_for_activation_count_); 281 EXPECT_LE(size_t(1), required_for_activation_count_);
282 } 282 }
283 283
284 private: 284 private:
285 FakeContentLayerClient client_; 285 FakeContentLayerClient client_;
286 }; 286 };
287 287
288 // No single thread test because the commit goes directly to the active tree in 288 // No single thread test because the commit goes directly to the active tree in
289 // single thread mode, so notify ready to activate is skipped. 289 // single thread mode, so notify ready to activate is skipped.
290 REMOTE_AND_MULTI_THREAD_TEST_F(LayerTreeHostTestReadyToActivateNonEmpty); 290 // No remote test because we currently don't deserialize FakePictureLayer,
291 // so on the impl side, PictureLayerImpl is created instead of
292 // FakePictureLayerImpl, see crbug/657871.
293 MULTI_THREAD_TEST_F(LayerTreeHostTestReadyToActivateNonEmpty);
291 294
292 // Test if the LTHI receives ReadyToDraw notifications from the TileManager when 295 // Test if the LTHI receives ReadyToDraw notifications from the TileManager when
293 // no raster tasks get scheduled. 296 // no raster tasks get scheduled.
294 class LayerTreeHostTestReadyToDrawEmpty : public LayerTreeHostTest { 297 class LayerTreeHostTestReadyToDrawEmpty : public LayerTreeHostTest {
295 public: 298 public:
296 LayerTreeHostTestReadyToDrawEmpty() 299 LayerTreeHostTestReadyToDrawEmpty()
297 : did_notify_ready_to_draw_(false), 300 : did_notify_ready_to_draw_(false),
298 all_tiles_required_for_draw_are_ready_to_draw_(false), 301 all_tiles_required_for_draw_are_ready_to_draw_(false),
299 required_for_draw_count_(0) {} 302 required_for_draw_count_(0) {}
300 303
(...skipping 6736 matching lines...) Expand 10 before | Expand all | Expand 10 after
7037 EndTest(); 7040 EndTest();
7038 } 7041 }
7039 7042
7040 void AfterTest() override {} 7043 void AfterTest() override {}
7041 }; 7044 };
7042 7045
7043 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostTestSubmitFrameResources); 7046 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostTestSubmitFrameResources);
7044 7047
7045 } // namespace 7048 } // namespace
7046 } // namespace cc 7049 } // namespace cc
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698