Add a content::GetUserAgent() to wrap webkit_glue::GetUserAgent()

content/common/content_client.h

 // Copyright (c) 2011 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.
 
 #ifndef CONTENT_COMMON_CONTENT_CLIENT_H_
 #define CONTENT_COMMON_CONTENT_CLIENT_H_
 #pragma once
 
 #include <string>
 #include <vector>
@@ skipping 26 matching lines @@
 class ContentClient;
 class ContentPluginClient;
 class ContentRendererClient;
 class ContentUtilityClient;
 
 // Setter and getter for the client.  The client should be set early, before any
 // content code is called.
 CONTENT_EXPORT void SetContentClient(ContentClient* client);
 CONTENT_EXPORT ContentClient* GetContentClient();
 
+// Returns the user agent string being used by the browser. SetContentClient()
+// must be called prior to calling this, and this routine must be used
+// instead of webkit_glue::GetUserAgent() in order to ensure things are
+// consistent.

[Peter Kasting, 2011/10/11 20:07:36]

Nit: "things are consistent" is pretty vague... it might be nice to be a little
clearer here about precisely what could go wrong when.

[Dirk Pranke, 2011/10/11 23:03:16]

Done.

+CONTENT_EXPORT const std::string& GetUserAgent(const GURL& url);
+
 // Interface that the embedder implements.
 class CONTENT_EXPORT ContentClient {
  public:
   ContentClient();
   virtual ~ContentClient();
 
   ContentBrowserClient* browser() { return browser_; }
   void set_browser(ContentBrowserClient* c) { browser_ = c; }
   ContentPluginClient* plugin() { return plugin_; }
   void set_plugin(ContentPluginClient* p) { plugin_ = p; }
@@ skipping 43 matching lines @@
   ContentPluginClient* plugin_;
   // The embedder API for participating in renderer logic.
   ContentRendererClient* renderer_;
   // The embedder API for participating in utility logic.
   ContentUtilityClient* utility_;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_COMMON_CONTENT_CLIENT_H_