Devtools/CC: Remove continuous repainting feature

cc/layers/heads_up_display_layer_impl.cc

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/layers/heads_up_display_layer_impl.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/numerics/safe_conversions.h"
@@ skipping 227 matching lines @@
   base::TimeTicks now = layer_tree_impl()->CurrentBeginFrameArgs().frame_time;
   if (base::TimeDelta(now - time_of_last_graph_update_).InSecondsF() > 0.25f) {
     time_of_last_graph_update_ = now;
 
     if (debug_state.show_fps_counter) {
       FrameRateCounter* fps_counter = layer_tree_impl()->frame_rate_counter();
       fps_graph_.value = fps_counter->GetAverageFPS();
       fps_counter->GetMinAndMaxFPS(&fps_graph_.min, &fps_graph_.max);
     }
 
-    if (debug_state.continuous_painting) {
-      PaintTimeCounter* paint_time_counter =
-          layer_tree_impl()->paint_time_counter();
-      base::TimeDelta latest, min, max;
-
-      if (paint_time_counter->End())
-        latest = **paint_time_counter->End();
-      paint_time_counter->GetMinAndMaxPaintTime(&min, &max);
-
-      paint_time_graph_.value = latest.InMillisecondsF();
-      paint_time_graph_.min = min.InMillisecondsF();
-      paint_time_graph_.max = max.InMillisecondsF();
-    }
-
     if (debug_state.ShowMemoryStats()) {
       MemoryHistory* memory_history = layer_tree_impl()->memory_history();
       if (memory_history->End())
         memory_entry_ = **memory_history->End();
       else
         memory_entry_ = MemoryHistory::Entry();
     }
   }
 
   fps_graph_.UpdateUpperBound();
   paint_time_graph_.UpdateUpperBound();
 }
 
 void HeadsUpDisplayLayerImpl::DrawHudContents(SkCanvas* canvas) {
   const LayerTreeDebugState& debug_state = layer_tree_impl()->debug_state();
 
   if (debug_state.ShowHudRects()) {
     DrawDebugRects(canvas, layer_tree_impl()->debug_rect_history());
     if (IsAnimatingHUDContents()) {
       layer_tree_impl()->SetNeedsRedraw();
     }
   }
 
   SkRect area = SkRect::MakeEmpty();
-  if (debug_state.continuous_painting) {
-    area = DrawPaintTimeDisplay(
-        canvas, layer_tree_impl()->paint_time_counter(), 0, 0);
-  } else if (debug_state.show_fps_counter) {
+  if (debug_state.show_fps_counter) {

[danakj, 2015/08/21 18:56:09]

you can just early out if !show_fps_counter and don't need to have these two if
statements and the MakeEmpty()?

[samli, 2015/08/26 21:08:24]

Done.

     // Don't show the FPS display when continuous painting is enabled, because

[danakj, 2015/08/21 18:56:09]

This comment should go

[samli, 2015/08/26 21:08:24]

Done.

     // it would show misleading numbers.
     area =
         DrawFPSDisplay(canvas, layer_tree_impl()->frame_rate_counter(), 0, 0);
   }
 
-  if (debug_state.show_fps_counter || debug_state.continuous_painting) {
+  if (debug_state.show_fps_counter) {
     area = DrawGpuRasterizationStatus(canvas, 0, area.bottom(),
                                       SkMaxScalar(area.width(), 150));
   }
 
   if (debug_state.ShowMemoryStats())
     DrawMemoryDisplay(canvas, 0, area.bottom(), SkMaxScalar(area.width(), 150));
 }
 int HeadsUpDisplayLayerImpl::MeasureText(SkPaint* paint,
                                          const std::string& text,
                                          int size) const {
@@ skipping 559 matching lines @@
   return "cc::HeadsUpDisplayLayerImpl";
 }
 
 void HeadsUpDisplayLayerImpl::AsValueInto(
     base::trace_event::TracedValue* dict) const {
   LayerImpl::AsValueInto(dict);
   dict->SetString("layer_name", "Heads Up Display Layer");
 }
 
 }  // namespace cc