Remove dependency on GURL from content/common/gpu

content/public/common/content_client.h

Index: content/public/common/content_client.h
diff --git a/content/public/common/content_client.h b/content/public/common/content_client.h
index 9d51d5ba83d4cf5a458f84137d963d39063c7c40..374f153a177f4dba3dbeb27903621d80ab15cbb4 100644
--- a/content/public/common/content_client.h
+++ b/content/public/common/content_client.h
@@ -14,10 +14,9 @@
 #include "build/build_config.h"
 #include "content/common/content_export.h"
 #include "ui/base/layout.h"
+#include "url/gurl.h"
 #include "url/url_util.h"
 
-class GURL;
-
 namespace base {
 class RefCountedStaticMemory;
 }
@@ -78,6 +77,7 @@ class CONTENT_EXPORT ContentClient {
 
   // Sets the currently active URL.  Use GURL() to clear the URL.
   virtual void SetActiveURL(const GURL& url) {}
+  void SetActiveURL(const std::string& url);

[piman, 2016/02/18 18:29:28]

nit: avoid overloads, especially with one method virtual and the other one not.
SetActiveURLFromString?

OTOH, since gpu_command_buffer_stub.cc is not supposed to access the
ContentClient directly (won't exist in mus), maybe the string->URL conversion
should be in the GpuChildThread (will stay in content/), accessed via the
GpuChannelManagerDelegate that Fady introduces in
https://codereview.chromium.org/1711533002 , so you wouldn't need this here.

[Mark Dittmer, 2016/02/19 14:58:45]

Refactored call into SetActiveURL to plumb through a delegate (same one as in
https://codereview.chromium.org/1711533002/). No more overload.

 
   // Sets the data on the current gpu.
   virtual void SetGpuInfo(const gpu::GPUInfo& gpu_info) {}