cc : Remove unneeded TileManager function param

Don't ask another function to do conditionally something the caller
could just do.

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I2fc0d8cf581f57ece9051a0a85bf0c2ac65dfabf
Reviewed-on: https://chromium-review.googlesource.com/993155Reviewed-by: Khushal <khushalsagar@chromium.org>
Commit-Queue: enne <enne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547794}

cc/tiles/tile_manager.cc

@@ -536,7 +536,7 @@ void TileManager::CheckForCompletedTasks() {
   tile_task_manager_->CheckForCompletedTasks();
   did_check_for_completed_tasks_since_last_schedule_tasks_ = true;
 
-  CheckPendingGpuWorkTiles(true /* issue_signals */, false /* flush */);
+  CheckPendingGpuWorkTiles(true /* issue_signals */);
 
   TRACE_EVENT_INSTANT1(
       "cc", "TileManager::CheckForCompletedTasksFinished",
@@ -1357,7 +1357,8 @@ void TileManager::CheckAndIssueSignals() {
   tile_task_manager_->CheckForCompletedTasks();
   did_check_for_completed_tasks_since_last_schedule_tasks_ = true;
 
-  CheckPendingGpuWorkTiles(false /* issue_signals */, true /* flush */);
+  raster_buffer_provider_->Flush();
+  CheckPendingGpuWorkTiles(false /* issue_signals */);
 
   // Ready to activate.
   if (signals_.ready_to_activate && !signals_.did_notify_ready_to_activate) {
@@ -1539,15 +1540,12 @@ bool TileManager::UsePartialRaster() const {
          raster_buffer_provider_->CanPartialRasterIntoProvidedResource();
 }
 
-void TileManager::CheckPendingGpuWorkTiles(bool issue_signals, bool flush) {
+void TileManager::CheckPendingGpuWorkTiles(bool issue_signals) {
   TRACE_EVENT2("cc", "TileManager::CheckPendingGpuWorkTiles",
                "pending_gpu_work_tiles", pending_gpu_work_tiles_.size(),
                "tree_priority",
                TreePriorityToString(global_state_.tree_priority));
 
-  if (flush)
-    raster_buffer_provider_->Flush();
-
   std::vector<const ResourcePool::InUsePoolResource*> required_for_activation;
   std::vector<const ResourcePool::InUsePoolResource*> required_for_draw;
 
@@ -1587,7 +1585,7 @@ void TileManager::CheckPendingGpuWorkTiles(bool issue_signals, bool flush) {
             required_for_activation,
             base::Bind(&TileManager::CheckPendingGpuWorkTiles,
                        ready_to_draw_callback_weak_ptr_factory_.GetWeakPtr(),
-                       true /* issue_signals */, false /* flush */),
+                       true /* issue_signals */),
             pending_required_for_activation_callback_id_);
   }
 
@@ -1599,7 +1597,7 @@ void TileManager::CheckPendingGpuWorkTiles(bool issue_signals, bool flush) {
             required_for_draw,
             base::Bind(&TileManager::CheckPendingGpuWorkTiles,
                        ready_to_draw_callback_weak_ptr_factory_.GetWeakPtr(),
-                       true /* issue_signals */, false /* flush */),
+                       true /* issue_signals */),
             pending_required_for_draw_callback_id_);
   }
 

cc/tiles/tile_manager.h

@@ -391,7 +391,7 @@ class CC_EXPORT TileManager : CheckerImageTrackerClient {
 
   bool UsePartialRaster() const;
 
-  void CheckPendingGpuWorkTiles(bool issue_signals, bool flush);
+  void CheckPendingGpuWorkTiles(bool issue_signals);
 
   TileManagerClient* client_;
   base::SequencedTaskRunner* task_runner_;