Ping watchdog thread during GpuChannel destruction

gpu/command_buffer/service/program_manager.cc

 // Copyright (c) 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 "gpu/command_buffer/service/program_manager.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <algorithm>
@@ skipping 10 matching lines @@
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/time/time.h"
 #include "gpu/command_buffer/common/gles2_cmd_format.h"
 #include "gpu/command_buffer/common/gles2_cmd_utils.h"
 #include "gpu/command_buffer/service/feature_info.h"
 #include "gpu/command_buffer/service/gles2_cmd_decoder.h"
 #include "gpu/command_buffer/service/gpu_preferences.h"
 #include "gpu/command_buffer/service/program_cache.h"
+#include "gpu/command_buffer/service/progress_reporter.h"
 #include "gpu/command_buffer/service/shader_manager.h"
 #include "third_party/re2/src/re2/re2.h"
 #include "ui/gl/gl_version_info.h"
 
 using base::TimeDelta;
 using base::TimeTicks;
 
 namespace gpu {
 namespace gles2 {
 
@@ skipping 2426 matching lines @@
       max_draw_buffers_(max_draw_buffers),
       max_dual_source_draw_buffers_(max_dual_source_draw_buffers),
       max_vertex_attribs_(max_vertex_attribs),
       gpu_preferences_(gpu_preferences),
       feature_info_(feature_info) {}
 
 ProgramManager::~ProgramManager() {
   DCHECK(programs_.empty());
 }
 
-void ProgramManager::Destroy(bool have_context) {
+void ProgramManager::Destroy(bool have_context,
+                             ProgressReporter* progress_reporter) {
   have_context_ = have_context;
 
   ProgramDeletionScopedUmaTimeAndRate scoped_histogram(
       base::saturated_cast<int32_t>(programs_.size()));
-  programs_.clear();
+  while (!programs_.empty()) {
+    programs_.erase(programs_.begin());
+    if (progress_reporter)

[danakj, 2016/10/05 01:53:48]

same, when is it null?

[ericrk, 2016/10/05 18:07:31]

refactored/removed.

+      progress_reporter->ReportProgress();
+  }
 }
 
 void ProgramManager::StartTracking(Program* /* program */) {
   ++program_count_;
 }
 
 void ProgramManager::StopTracking(Program* /* program */) {
   --program_count_;
 }
 
@@ skipping 85 matching lines @@
   DCHECK(program);
   program->ClearUniforms(&zero_);
 }
 
 int32_t ProgramManager::MakeFakeLocation(int32_t index, int32_t element) {
   return index + element * 0x10000;
 }
 
 }  // namespace gles2
 }  // namespace gpu