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

Side by Side Diff: cc/trees/layer_tree_host_impl.h

Issue 914403003: cc: Add main frame timing info to frame timing tracker. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 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
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 #ifndef CC_TREES_LAYER_TREE_HOST_IMPL_H_ 5 #ifndef CC_TREES_LAYER_TREE_HOST_IMPL_H_
6 #define CC_TREES_LAYER_TREE_HOST_IMPL_H_ 6 #define CC_TREES_LAYER_TREE_HOST_IMPL_H_
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
(...skipping 506 matching lines...) Expand 10 before | Expand all | Expand 10 after
517 scoped_ptr<TileTaskWorkerPool>* tile_task_worker_pool, 517 scoped_ptr<TileTaskWorkerPool>* tile_task_worker_pool,
518 scoped_ptr<ResourcePool>* resource_pool, 518 scoped_ptr<ResourcePool>* resource_pool,
519 scoped_ptr<ResourcePool>* staging_resource_pool); 519 scoped_ptr<ResourcePool>* staging_resource_pool);
520 520
521 bool prepare_tiles_needed() const { return tile_priorities_dirty_; } 521 bool prepare_tiles_needed() const { return tile_priorities_dirty_; }
522 522
523 FrameTimingTracker* frame_timing_tracker() { 523 FrameTimingTracker* frame_timing_tracker() {
524 return frame_timing_tracker_.get(); 524 return frame_timing_tracker_.get();
525 } 525 }
526 526
527 // Record main frame timing information.
528 // |start_of_main_frame_args| is the BeginFrameArgs of the beginning of the
529 // main frame (ie the frame that kicked off the main frame).
530 // |expected_next_main_frame_args| is the BeginFrameArgs of the frame that
531 // follows the completion of the main frame (whether it is activation or some
532 // other completion, such as early out). Note that if there is a main frame
533 // scheduled in that frame, then this BeginFrameArgs will become the main
534 // frame args. However, if no such frame is scheduled, then this _would_ be
535 // the main frame args if it was scheduled.
536 void RecordMainFrameTiming(
537 const BeginFrameArgs& start_of_main_frame_args,
538 const BeginFrameArgs& expected_next_main_frame_args);
539
527 protected: 540 protected:
528 LayerTreeHostImpl( 541 LayerTreeHostImpl(
529 const LayerTreeSettings& settings, 542 const LayerTreeSettings& settings,
530 LayerTreeHostImplClient* client, 543 LayerTreeHostImplClient* client,
531 Proxy* proxy, 544 Proxy* proxy,
532 RenderingStatsInstrumentation* rendering_stats_instrumentation, 545 RenderingStatsInstrumentation* rendering_stats_instrumentation,
533 SharedBitmapManager* shared_bitmap_manager, 546 SharedBitmapManager* shared_bitmap_manager,
534 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager, 547 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager,
535 TaskGraphRunner* task_graph_runner, 548 TaskGraphRunner* task_graph_runner,
536 int id); 549 int id);
(...skipping 209 matching lines...) Expand 10 before | Expand all | Expand 10 after
746 bool is_likely_to_require_a_draw_; 759 bool is_likely_to_require_a_draw_;
747 760
748 scoped_ptr<FrameTimingTracker> frame_timing_tracker_; 761 scoped_ptr<FrameTimingTracker> frame_timing_tracker_;
749 762
750 DISALLOW_COPY_AND_ASSIGN(LayerTreeHostImpl); 763 DISALLOW_COPY_AND_ASSIGN(LayerTreeHostImpl);
751 }; 764 };
752 765
753 } // namespace cc 766 } // namespace cc
754 767
755 #endif // CC_TREES_LAYER_TREE_HOST_IMPL_H_ 768 #endif // CC_TREES_LAYER_TREE_HOST_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698