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

Unified Diff: cc/trees/layer_tree_host_impl.cc

Issue 26031002: cc: Remove unused metrics from RenderingStats. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed raster_worker_pool_perftest.cc Created 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/trees/layer_tree_host.cc ('k') | cc/trees/single_thread_proxy.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/layer_tree_host_impl.cc
diff --git a/cc/trees/layer_tree_host_impl.cc b/cc/trees/layer_tree_host_impl.cc
index d8ca19bb2f79ba5e1cf25bffecbf491b4ff3735d..a6144d3cb4eb2b1231699cdb07f6a5e53abe102e 100644
--- a/cc/trees/layer_tree_host_impl.cc
+++ b/cc/trees/layer_tree_host_impl.cc
@@ -805,8 +805,6 @@ bool LayerTreeHostImpl::CalculateRenderPasses(FrameData* frame) {
}
if (append_quads_data.num_missing_tiles) {
- rendering_stats_instrumentation_->AddMissingTiles(
- append_quads_data.num_missing_tiles);
bool layer_has_animating_transform =
it->screen_space_transform_is_animating() ||
it->draw_transform_is_animating();
@@ -824,8 +822,6 @@ bool LayerTreeHostImpl::CalculateRenderPasses(FrameData* frame) {
output_surface_->capabilities().draw_and_swap_full_viewport_every_frame)
draw_frame = true;
- rendering_stats_instrumentation_->AddLayersDrawn(layers_drawn);
-
#ifndef NDEBUG
for (size_t i = 0; i < frame->render_passes.size(); ++i) {
for (size_t j = 0; j < frame->render_passes[i]->quad_list.size(); ++j)
@@ -1271,15 +1267,12 @@ void LayerTreeHostImpl::DrawLayers(FrameData* frame,
DCHECK(!frame->render_passes.empty());
- int old_dropped_frame_count = fps_counter_->dropped_frame_count();
fps_counter_->SaveTimeStamp(frame_begin_time,
!output_surface_->context_provider());
bool on_main_thread = false;
- rendering_stats_instrumentation_->IncrementScreenFrameCount(
+ rendering_stats_instrumentation_->IncrementFrameCount(
1, on_main_thread);
- rendering_stats_instrumentation_->IncrementDroppedFrameCount(
- fps_counter_->dropped_frame_count() - old_dropped_frame_count);
if (tile_manager_) {
memory_history_->SaveEntry(
@@ -1529,9 +1522,9 @@ void LayerTreeHostImpl::ActivatePendingTree() {
if (debug_state_.continuous_painting) {
const RenderingStats& stats =
rendering_stats_instrumentation_->GetRenderingStats();
- paint_time_counter_->SavePaintTime(
- stats.main_stats.paint_time + stats.main_stats.record_time +
- stats.impl_stats.rasterize_time_for_now_bins_on_pending_tree);
+ paint_time_counter_->SavePaintTime(stats.main_stats.paint_time +
+ stats.main_stats.record_time +
+ stats.impl_stats.rasterize_time);
}
client_->DidActivatePendingTree();
@@ -1972,7 +1965,6 @@ InputHandler::ScrollStatus LayerTreeHostImpl::ScrollBegin(
layer_impl, &scroll_on_main_thread);
if (scroll_on_main_thread) {
- rendering_stats_instrumentation_->IncrementMainThreadScrolls();
UMA_HISTOGRAM_BOOLEAN("TryScroll.SlowScroll", true);
return ScrollOnMainThread;
}
@@ -1986,7 +1978,6 @@ InputHandler::ScrollStatus LayerTreeHostImpl::ScrollBegin(
potentially_scrolling_layer_impl);
should_bubble_scrolls_ = (type != NonBubblingGesture);
wheel_scrolling_ = (type == Wheel);
- rendering_stats_instrumentation_->IncrementImplThreadScrolls();
client_->RenewTreePriority();
UMA_HISTOGRAM_BOOLEAN("TryScroll.SlowScroll", false);
return ScrollStarted;
« no previous file with comments | « cc/trees/layer_tree_host.cc ('k') | cc/trees/single_thread_proxy.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698