Alternative GDrive promo for the File Manager

chrome/browser/chromeos/system/statistics_provider.cc

Index: chrome/browser/chromeos/system/statistics_provider.cc
diff --git a/chrome/browser/chromeos/system/statistics_provider.cc b/chrome/browser/chromeos/system/statistics_provider.cc
index 28118635c75ba22ebcadc78211511a041f5f2bff..81bf2981efed803234e2e49286beb6f0326051c7 100644
--- a/chrome/browser/chromeos/system/statistics_provider.cc
+++ b/chrome/browser/chromeos/system/statistics_provider.cc
@@ -6,6 +6,7 @@
 
 #include "base/bind.h"
 #include "base/chromeos/chromeos_version.h"
+#include "base/command_line.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/logging.h"
@@ -16,6 +17,7 @@
 #include "base/chromeos/chromeos_version.h"
 #include "chrome/browser/chromeos/system/name_value_pairs_parser.h"
 #include "chrome/common/child_process_logging.h"
+#include "chrome/common/chrome_switches.h"
 #include "chrome/common/chrome_version_info.h"
 #include "content/public/browser/browser_thread.h"
 
@@ -210,6 +212,14 @@ class StatisticsProviderStubImpl : public StatisticsProvider {
   // StatisticsProvider implementation:
   virtual bool GetMachineStatistic(const std::string& name,
                                    std::string* result) OVERRIDE {
+    if (name == "CHROMEOS_RELEASE_BOARD") {

[Dmitry Zvorygin, 2012/07/25 15:15:46]

I don't like that you use this sting in two places, and with little different
meaning. Why not to make a constant at least?

[Vladislav Kaznacheev, 2012/07/25 16:45:13]

The meaning in C++ is exactly the same. This name is also used in JS but there
is no way to declare a constant across the two languages.
On 2012/07/25 15:15:46, Dmitry Zvorygin wrote:

+      const CommandLine* command_line = CommandLine::ForCurrentProcess();
+      if (command_line->HasSwitch(switches::kChromeOSReleaseBoard)) {
+        *result = command_line->
+            GetSwitchValueASCII(switches::kChromeOSReleaseBoard);
+        return true;
+      }
+    }
     return false;
   }