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

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

Issue 1427543002: Modified old wait sync point functions to also accept new sync tokens. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase + autogen Created 5 years, 1 month 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
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 2012 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 <sstream> 7 #include <sstream>
8 8
9 #include "base/files/file_path.h" 9 #include "base/files/file_path.h"
10 #include "base/files/file_util.h" 10 #include "base/files/file_util.h"
(...skipping 242 matching lines...) Expand 10 before | Expand all | Expand 10 after
253 SetTestName("long_scrollable_page_threaded_impl_side"); 253 SetTestName("long_scrollable_page_threaded_impl_side");
254 ReadTestFile("long_scrollable_page"); 254 ReadTestFile("long_scrollable_page");
255 // TODO(vollick): Remove verify_property_trees setting after 255 // TODO(vollick): Remove verify_property_trees setting after
256 // crbug.com/444219 is fixed. 256 // crbug.com/444219 is fixed.
257 bool old_verify_property_trees = verify_property_trees(); 257 bool old_verify_property_trees = verify_property_trees();
258 set_verify_property_trees(false); 258 set_verify_property_trees(false);
259 RunTest(true, false); 259 RunTest(true, false);
260 set_verify_property_trees(old_verify_property_trees); 260 set_verify_property_trees(old_verify_property_trees);
261 } 261 }
262 262
263 static void EmptyReleaseCallback(uint32 sync_point, bool lost_resource) {} 263 static void EmptyReleaseCallback(const gpu::SyncToken& sync_token,
264 bool lost_resource) {}
264 265
265 // Simulates main-thread scrolling on each frame. 266 // Simulates main-thread scrolling on each frame.
266 class BrowserCompositorInvalidateLayerTreePerfTest 267 class BrowserCompositorInvalidateLayerTreePerfTest
267 : public LayerTreeHostPerfTestJsonReader { 268 : public LayerTreeHostPerfTestJsonReader {
268 public: 269 public:
269 BrowserCompositorInvalidateLayerTreePerfTest() 270 BrowserCompositorInvalidateLayerTreePerfTest()
270 : LayerTreeHostPerfTestJsonReader(), 271 : LayerTreeHostPerfTestJsonReader(),
271 next_sync_point_(1), 272 next_sync_point_(1),
272 clean_up_started_(false) {} 273 clean_up_started_(false) {}
273 274
(...skipping 11 matching lines...) Expand all
285 void WillCommit() override { 286 void WillCommit() override {
286 if (CleanUpStarted()) 287 if (CleanUpStarted())
287 return; 288 return;
288 gpu::Mailbox gpu_mailbox; 289 gpu::Mailbox gpu_mailbox;
289 std::ostringstream name_stream; 290 std::ostringstream name_stream;
290 name_stream << "name" << next_sync_point_; 291 name_stream << "name" << next_sync_point_;
291 gpu_mailbox.SetName( 292 gpu_mailbox.SetName(
292 reinterpret_cast<const int8*>(name_stream.str().c_str())); 293 reinterpret_cast<const int8*>(name_stream.str().c_str()));
293 scoped_ptr<SingleReleaseCallback> callback = SingleReleaseCallback::Create( 294 scoped_ptr<SingleReleaseCallback> callback = SingleReleaseCallback::Create(
294 base::Bind(&EmptyReleaseCallback)); 295 base::Bind(&EmptyReleaseCallback));
295 TextureMailbox mailbox(gpu_mailbox, GL_TEXTURE_2D, next_sync_point_); 296 TextureMailbox mailbox(gpu_mailbox, gpu::SyncToken(next_sync_point_),
297 GL_TEXTURE_2D);
296 next_sync_point_++; 298 next_sync_point_++;
297 299
298 tab_contents_->SetTextureMailbox(mailbox, callback.Pass()); 300 tab_contents_->SetTextureMailbox(mailbox, callback.Pass());
299 } 301 }
300 302
301 void DidCommit() override { 303 void DidCommit() override {
302 if (CleanUpStarted()) 304 if (CleanUpStarted())
303 return; 305 return;
304 layer_tree_host()->SetNeedsCommit(); 306 layer_tree_host()->SetNeedsCommit();
305 } 307 }
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
337 TEST_F(LayerTreeHostPerfTestJsonReader, HeavyPageThreaded) { 339 TEST_F(LayerTreeHostPerfTestJsonReader, HeavyPageThreaded) {
338 begin_frame_driven_drawing_ = true; 340 begin_frame_driven_drawing_ = true;
339 measure_commit_cost_ = true; 341 measure_commit_cost_ = true;
340 SetTestName("heavy_page"); 342 SetTestName("heavy_page");
341 ReadTestFile("heavy_layer_tree"); 343 ReadTestFile("heavy_layer_tree");
342 RunTest(true, false); 344 RunTest(true, false);
343 } 345 }
344 346
345 } // namespace 347 } // namespace
346 } // namespace cc 348 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698