cc : Stop calculating visible rects on main thread

cc/debug/rasterize_and_record_benchmark.cc

 // Copyright 2013 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/debug/rasterize_and_record_benchmark.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <limits>
@@ skipping 96 matching lines @@
       base::Bind(&RasterizeAndRecordBenchmark::RecordRasterResults,
                  weak_ptr_factory_.GetWeakPtr())));
 }
 
 void RasterizeAndRecordBenchmark::RunOnLayer(PictureLayer* layer) {
   DCHECK(host_);
 
   if (!layer->DrawsContent())
     return;
 
-  gfx::Rect visible_layer_rect = layer->visible_layer_rect_for_testing();
-  if (visible_layer_rect.IsEmpty())
-    return;

[ajuma, 2016/04/04 20:47:17]

This seems like the most realistic thing to do (since cc doesn't actually skip
recording layers with empty visible rects). This might change benchmark results
though, so be sure to mention this in the description.

[jaydasika, 2016/04/04 21:29:16]

Done.

-
   ContentLayerClient* painter = layer->client();
 
   for (int mode_index = 0; mode_index < RecordingSource::RECORDING_MODE_COUNT;
        mode_index++) {
     ContentLayerClient::PaintingControlSetting painting_control =
         ContentLayerClient::PAINTING_BEHAVIOR_NORMAL_FOR_TEST;
     switch (static_cast<RecordingSource::RecordingMode>(mode_index)) {
       case RecordingSource::RECORD_NORMALLY:
         // Already setup for normal recording.
         break;
@@ skipping 60 matching lines @@
   }
 }
 
 RasterizeAndRecordBenchmark::RecordResults::RecordResults()
     : pixels_recorded(0), bytes_used(0) {
 }
 
 RasterizeAndRecordBenchmark::RecordResults::~RecordResults() {}
 
 }  // namespace cc