Improve GPU process URL crash reporting

content/renderer/render_thread_impl.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 "content/renderer/render_thread_impl.h"
 
 #include <algorithm>
 #include <limits>
 #include <map>
 #include <vector>
@@ skipping 725 matching lines @@
     if (compositor_thread()) {
       compositor_thread()->GetWebThread()->message_loop()->DeleteSoon(
           FROM_HERE, gpu_vda_context3d_.release());
     } else {
       gpu_vda_context3d_.reset();
     }
   }
   if (!gpu_vda_context3d_.get()) {
     gpu_vda_context3d_.reset(
         WebGraphicsContext3DCommandBufferImpl::CreateOffscreenContext(
-            this, WebKit::WebGraphicsContext3D::Attributes()));
+            this, WebKit::WebGraphicsContext3D::Attributes(),
+            GURL("RenderThreadImpl::GetGpuVDAContext3D")));

[brettw, 2012/07/12 14:16:43]

In the places you use these fake URLs, I'd prefer something that looks more like
a URL. As-is, it looks like a mistake. Maybe
"chrome://gpu/RenderThreadImpl::GetGpuVDAContext3D"?

[jbates, 2012/07/12 17:51:13]

Done.

   }
   return gpu_vda_context3d_->AsWeakPtr();
 }
 
 #if defined(OS_WIN)
 void RenderThreadImpl::PreCacheFont(const LOGFONT& log_font) {
   Send(new ChildProcessHostMsg_PreCacheFont(log_font));
 }
 
 void RenderThreadImpl::ReleaseCachedFonts() {
@@ skipping 267 matching lines @@
 
 scoped_refptr<base::MessageLoopProxy>
 RenderThreadImpl::GetFileThreadMessageLoopProxy() {
   DCHECK(message_loop() == MessageLoop::current());
   if (!file_thread_.get()) {
     file_thread_.reset(new base::Thread("Renderer::FILE"));
     file_thread_->Start();
   }
   return file_thread_->message_loop_proxy();
 }