Prevent duplicate navigation to debug URLs from Telemetry.

content/browser/gpu/gpu_data_manager_impl_private.cc

Index: content/browser/gpu/gpu_data_manager_impl_private.cc
diff --git a/content/browser/gpu/gpu_data_manager_impl_private.cc b/content/browser/gpu/gpu_data_manager_impl_private.cc
index 95e0c47963c8396a7ac060dec5c376480c950f75..ccd583f65deea31406ef7ce4e076c488c1881a6d 100644
--- a/content/browser/gpu/gpu_data_manager_impl_private.cc
+++ b/content/browser/gpu/gpu_data_manager_impl_private.cc
@@ -758,6 +758,7 @@ void GpuDataManagerImplPrivate::ProcessCrashed(
   }
   {
     GpuDataManagerImpl::UnlockedSession session(owner_);
+    gpu_info_.process_crash_count++;

[Ken Russell (switch to Gerrit), 2014/07/25 01:08:26]

I'm sorry to be pedantic but could you please move the static variables in
GpuProcessHost::~GpuProcessHost into static class members and provide a static
method to query the gpu_crash_count? This should get the crash count from the
GpuProcessHost instead of keeping its own copy. Also, this should be moved up
one line. At this point the GpuDataManagerImpl can't be accessed in a thread
safe manner. (The above object is an "unlocker", not a locker.)

[vmiura, 2014/07/25 19:56:03]

Done.

     observer_list_->Notify(
         &GpuDataManagerObserver::OnGpuProcessCrashed, exit_code);
   }