Move user_agent code from webkit/ to content/.

content/public/common/content_client.cc

Index: content/public/common/content_client.cc
diff --git a/content/public/common/content_client.cc b/content/public/common/content_client.cc
index 54db51778a46c3fd524de5c5257dff08f8378609..d0fc19e26f5c55cdbac435ace85a98edd0b5206a 100644
--- a/content/public/common/content_client.cc
+++ b/content/public/common/content_client.cc
@@ -6,8 +6,8 @@
 
 #include "base/logging.h"
 #include "base/strings/string_piece.h"
+#include "content/public/common/user_agent.h"

[jam, 2014/03/05 00:09:26]

nit: not needed

[tfarina, 2014/03/05 00:42:52]

Done.

 #include "ui/gfx/image/image.h"
-#include "webkit/common/user_agent/user_agent.h"
 
 namespace content {
 
@@ -36,13 +36,6 @@ class InternalTestInitializer {
 
 void SetContentClient(ContentClient* client) {
   g_client = client;
-
-  // Set the default user agent as provided by the client. We need to make
-  // sure this is done before webkit_glue::GetUserAgent() is called (so that
-  // the UA doesn't change).
-  if (client) {
-    webkit_glue::SetUserAgent(client->GetUserAgent(), false);

[jam, 2014/03/05 00:09:26]

i don't see any code which now calls ContentClient->GetUserAgent(). that still
needs to happen. it should just be lazily initialized in user_agent.cc's
GetUserAgent, i.e. a custom constructor for the lazy instance

[tfarina, 2014/03/05 00:22:11]

https://code.google.com/p/chromium/codesearch#search/&q=%22-%3Egetuseragent(%...

https://code.google.com/p/chromium/codesearch#search/&q=%22.getuseragent(%22&...

So looks like we have one in chrome_content_client_unittest.cc, one in
content/browser/accessibility/browser_accessibility_win.cc and this one here in
content_client.cc

I'm afraid that I'm unsure which change you want in user_agent.cc :(

[jam, 2014/03/05 08:34:56]

sorry I wasn't clear. I mean, user_agent.cc needs to call the equivalent of
SetUserAgent(content::GetContentClient()->GetUserAgent()) because otherwise the
user agent string from the embedder is never fetched by content itself for the
lazy instance. is that clearer?

btw looking at this some more, I think we can really simplify the user agent
code. The reason is that the microsoft/yahoo user agent changing code was there
for the first release. it's not needed anymore. the microsoft silverlight
download page still shows a download without that user agent override. the yahoo
downloads page for silverlight says that site will go away in june. The yahoo
gyao.yahoo.co.jp page doesn't have silverlight on it now.
promotion.shopping.yahoo.co.jp is dead.

[tfarina, 2014/03/05 13:31:55]

Sorry, I have no idea what you are asking me to do in user_agent.cc :(

-  }
 }
 
 ContentClient* GetContentClient() {
@@ -61,11 +54,6 @@ ContentUtilityClient* SetUtilityClientForTesting(ContentUtilityClient* u) {
   return InternalTestInitializer::SetUtility(u);
 }
 
-const std::string& GetUserAgent(const GURL& url) {
-  DCHECK(g_client);
-  return webkit_glue::GetUserAgent(url);
-}
-
 ContentClient::ContentClient()
     : browser_(NULL), plugin_(NULL), renderer_(NULL), utility_(NULL) {
 }