Improve GPU process URL crash reporting

content/renderer/browser_plugin/old/guest_to_embedder_channel.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/browser_plugin/old/guest_to_embedder_channel.h"
 
 #include "base/process_util.h"
 #include "content/common/browser_plugin_messages.h"
 #include "content/common/child_process.h"
 #include "content/renderer/browser_plugin/old/browser_plugin_channel_manager.h"
 #include "content/renderer/browser_plugin/old/browser_plugin_var_serialization_rules.h"
 #include "content/renderer/render_thread_impl.h"
 #include "content/renderer/render_view_impl.h"
 #include "ppapi/c/pp_bool.h"
 #include "ppapi/c/pp_graphics_3d.h"
 #include "ppapi/proxy/ppapi_command_buffer_proxy.h"
 #include "ppapi/proxy/ppapi_messages.h"
 #include "ppapi/shared_impl/api_id.h"
 #include "ppapi/shared_impl/ppapi_globals.h"
 #include "ppapi/shared_impl/var.h"
+#include "third_party/WebKit/Source/WebKit/chromium/public/WebDocument.h"
+#include "third_party/WebKit/Source/WebKit/chromium/public/WebFrame.h"
 #include "webkit/plugins/ppapi/event_conversion.h"
 
 namespace content {
 
 GuestToEmbedderChannel::GuestToEmbedderChannel(
     const std::string& embedder_channel_name,
     const IPC::ChannelHandle& embedder_channel_handle)
     : Dispatcher(NULL),
       embedder_channel_name_(embedder_channel_name),
       embedder_channel_handle_(embedder_channel_handle) {
@@ skipping 55 matching lines @@
 
 bool GuestToEmbedderChannel::IsPlugin() const {
   return true;
 }
 
 WebGraphicsContext3DCommandBufferImpl*
     GuestToEmbedderChannel::CreateWebGraphicsContext3D(
         RenderViewImpl* render_view,
         const WebKit::WebGraphicsContext3D::Attributes& attributes,
         bool offscreen) {
+  GURL url;
+  if (render_view->webview()->mainFrame())

[Ken Russell (switch to Gerrit), 2012/07/10 23:38:01]

Under what conditions would this not be the case? (iframes?) Is there any
fallback URL we could use?

[jbates, 2012/07/11 00:07:18]

Not sure, this was copied from render_view_impl.cc.

(iframes?) Is there any
At least we can fall back on the method name - updated this instance as well as
render_view_impl.cc.

+    url = GURL(render_view->webview()->mainFrame()->document().url());
   scoped_ptr<WebGraphicsContext3DCommandBufferImpl> context(
       new WebGraphicsContext3DCommandBufferImpl(
-          0, GURL(), NULL,
+          0, url, NULL,
           render_view->AsWeakPtr()));
 
   // Special case: RenderView initialization has not yet completed.
   if (!render_view->guest_pp_instance())
     return context.release();
 
   if (CreateGraphicsContext(context.get(),
                             attributes,
                             offscreen,
                             render_view))
@@ skipping 188 matching lines @@
   render_view->GetWebView()->loseCompositorContext(1);
 }
 
 void GuestToEmbedderChannel::OnGuestReady(PP_Instance instance,
                                           int embedder_container_id) {
   RenderThreadImpl::current()->browser_plugin_channel_manager()->
       GuestReady(instance, embedder_channel_name(), embedder_container_id);
 }
 
 }  // namespace content