Remove wchar_t versions of StringPrintf from Android

chrome/common/metrics/experiments_helper.cc

Index: chrome/common/metrics/experiments_helper.cc
diff --git a/chrome/common/metrics/experiments_helper.cc b/chrome/common/metrics/experiments_helper.cc
index 27f0235c09a0f15df10821ab19df4f9bcb7a91bd..dfa3971829ee21e30dc3d925023964b5078bd9e9 100644
--- a/chrome/common/metrics/experiments_helper.cc
+++ b/chrome/common/metrics/experiments_helper.cc
@@ -156,9 +156,15 @@ void SetChildProcessLoggingExperimentList() {
   GetFieldTrialSelectedGroupIds(&name_group_ids);
   std::vector<string16> experiment_strings(name_group_ids.size());
   for (size_t i = 0; i < name_group_ids.size(); ++i) {
+#if defined(OS_ANDROID)
+    // Android doesn't have wchar_t StringPrintf.
+    experiment_strings[i] = UTF8ToUTF16(base::StringPrintf(

[sky, 2012/07/28 02:11:24]

Why the ifdef here? Can't we use this version every where?

[Iain Merrick, 2012/07/30 08:33:32]

I figured there might be a good reason for using wstrings in the original code,
so I was reluctant to change it for all platforms. But I'll go ahead and make
that change. Hopefully we have unit test coverage!

Done.

+        "%x-%x", name_group_ids[i].name, name_group_ids[i].group));
+#else
     // Wish there was a StringPrintf for string16... :-(
     experiment_strings[i] = WideToUTF16(base::StringPrintf(
         L"%x-%x", name_group_ids[i].name, name_group_ids[i].group));
+#endif
   }
   child_process_logging::SetExperimentList(experiment_strings);
 }