Move Android user agent generation to native code

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
 
 #include "base/lazy_instance.h"
 #include "base/string_util.h"
 #include "base/stringprintf.h"
 #include "base/sys_info.h"
 
 #if defined(OS_WIN)
 #include "base/win/windows_version.h"
 #endif
 
 // Generated
 #include "webkit_version.h"  // NOLINT
 
-#if defined(OS_ANDROID)
-namespace {
-
-base::LazyInstance<std::string>::Leaky g_os_info = LAZY_INSTANCE_INITIALIZER;
-
-}  // namespace
-#endif
-
 namespace webkit_glue {
 
 std::string GetWebKitVersion() {
   return base::StringPrintf("%d.%d (%s)",
                             WEBKIT_VERSION_MAJOR,
                             WEBKIT_VERSION_MINOR,
                             WEBKIT_SVN_REVISION);
 }
 
 std::string GetWebKitRevision() {
@@ skipping 55 matching lines @@
       os_bugfix_version
 #elif defined(OS_CHROMEOS)
       "CrOS "
       "%s %d.%d.%d",
       cputype.c_str(),   // e.g. i686
       os_major_version,
       os_minor_version,
       os_bugfix_version
 #elif defined(OS_ANDROID)
       "Android %s",
-      g_os_info.Get().c_str()
+      BuildOSCpuInfoForAndroid().c_str()
 #else
       "%s %s",
       unixinfo.sysname,  // e.g. Linux
       cputype.c_str()    // e.g. i686
 #endif
   );  // NOLINT
 
   return os_cpu;
 }
 
+std::string BuildOSCpuInfoForAndroid() {

[jar (doing other things), 2012/08/15 23:35:17]

Who would ever call this, if not in android?  ... so why not make this ifdef
based on OS_ANDROID?

[gone, 2012/08/16 00:29:31]

It's supposed to be used for spoofing the UA in a follow-up CL.  I can just
#ifdef it out until that one lands.

+  int32 version_major = base::SysInfo::kDefaultAndroidMajorVersion;
+  int32 version_minor = base::SysInfo::kDefaultAndroidMinorVersion;
+  int32 version_bugfix = base::SysInfo::kDefaultAndroidBugfixVersion;
+  std::string android_build_codename;
+  std::string android_build_id;
+  std::string android_device_name;
+#if defined(OS_ANDROID)
+  base::SysInfo::OperatingSystemVersionNumbers(&version_major, &version_minor,
+      &version_bugfix);
+  android_build_codename = base::SysInfo::GetAndroidBuildCodename();
+  android_build_id = base::SysInfo::GetAndroidBuildID();
+  android_device_name = base::SysInfo::GetDeviceName();
+#endif
+
+  bool semicolon_inserted = false;
+  std::string android_info_str;
+
+  // Append information about the OS version.
+  if (version_bugfix == 0) {
+    base::StringAppendF(&android_info_str, "%d.%d", version_major,
+        version_minor);
+  } else {
+    base::StringAppendF(&android_info_str, "%d.%d.%d", version_major,
+        version_minor, version_bugfix);
+  }
+
+  // Append information about the device.
+  if ("REL" == android_build_codename && android_device_name.size() > 0) {
+    if (!semicolon_inserted) {

[jar (doing other things), 2012/08/15 23:35:17]

Logic for this seems confused.
semicolon_inserted is always false here.

Did you plan to sometimes set it around lines 127-134?

[gone, 2012/08/16 00:29:31]

Good catch, moved it downward.

+      android_info_str += ";";
+      semicolon_inserted = true;
+    }
+    android_info_str += " " + android_device_name;
+  }
+
+  // Append the build ID.
+  if (android_build_id.size() > 0) {
+    if (!semicolon_inserted) {
+      android_info_str += ";";
+      semicolon_inserted = true;

[jar (doing other things), 2012/08/15 23:35:17]

Why are you setting this variable, given that you never use it again?  The
compiler will discard this setting... but the code  just seems larger for no
good reason.

[gone, 2012/08/16 00:29:31]

Done.

+    }
+    android_info_str += " Build/" + android_build_id;
+  }
+
+  return android_info_str;
+}
+
 int GetWebKitMajorVersion() {
   return WEBKIT_VERSION_MAJOR;
 }
 
 int GetWebKitMinorVersion() {
   return WEBKIT_VERSION_MINOR;
 }
 
 std::string BuildUserAgentFromProduct(const std::string& product) {
   const char kUserAgentPlatform[] =
@@ skipping 22 matching lines @@
       kUserAgentPlatform,
       webkit_glue::BuildOSCpuInfo().c_str(),
       WEBKIT_VERSION_MAJOR,
       WEBKIT_VERSION_MINOR,
       product.c_str(),
       WEBKIT_VERSION_MAJOR,
       WEBKIT_VERSION_MINOR);
   return user_agent;
 }
 
-#if defined(OS_ANDROID)
-void SetUserAgentOSInfo(const std::string& os_info) {
-  g_os_info.Get() = os_info;
-}
-#endif
-
 }  // namespace webkit_glue