crbug.com/127841 - Request Tablet Site on CB with touch screen.

webkit/glue/user_agent.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 "webkit/glue/user_agent.h"
 
 #if defined(OS_POSIX) && !defined(OS_MACOSX)
 #include <sys/utsname.h>
 #endif
 
@@ skipping 144 matching lines @@
       WEBKIT_VERSION_MINOR);
   return user_agent;
 }
 
 #if defined(OS_ANDROID)
 void SetUserAgentOSInfo(const std::string& os_info) {
   g_os_info.Get() = os_info;
 }
 #endif
 
+std::string BuildUserAgentOverrideForTabletSiteFromUserAgent(
+    const std::string& user_agent) {
+  //

[gone, 2012/08/03 00:08:26]

nit: Remove line

[sschmitz, 2012/08/03 15:36:58]

Done.

+  // Example: original user agent text:
+  //
+  //   Mozilla/5.0 (X11; CrOS x86_64 0.4.0)
+  //   AppleWebKit/537.2 (KHTML, like Gecko) Chrome/22.0.1211.0 Safari/537.2
+  //
+  // Copy & Edit to create user agent override text by replacing
+  //   the text within the first pair of "()". In this example:
+  //
+  //   Mozilla/5.0 (Linux; U; Android 4.0.2; en-us; Galaxy Nexus Build/ICL53F)
+  //   AppleWebKit/537.2 (KHTML, like Gecko) Chrome/22.0.1211.0 Safari/537.2
+  //

[gone, 2012/08/03 00:08:26]

Seems relatively fragile, but I don't have a better alternative at the moment. 
I filed http://crbug.com/128570 a while back because we're having trouble
spoofing a desktop UA for Android, too...  I could revisit this bit if I come up
with a good solution for both of us.

[sschmitz, 2012/08/03 15:36:58]

Yes. I agree. Thanks for looking out.

+  size_t p1 = user_agent.find("(");
+  if (p1 != std::string::npos) {
+    size_t p2 = user_agent.find(")", p1);
+    if (p2 != std::string::npos) {
+      std::string ua_override(user_agent);
+      ua_override.replace(
+          p1,p2-p1+1,
+          "(Linux; U; Android 4.0.2; en-us; Galaxy Nexus Build/ICL53F)");

[gone, 2012/08/03 00:08:26]

Yank this literal out of the replace() call and make it a constant somewhere
above it.  Also, the user agent format was updated a while ago; check:
https://developers.google.com/chrome/mobile/docs/user-agent

[sschmitz, 2012/08/03 15:36:58]

Done.

+      return ua_override;
+    }
+  }
+  return std::string();
+}
+
 }  // namespace webkit_glue