Simplify GetLinuxDistro() for Chrome OS.

base/linux_util.cc

 // Copyright (c) 2009 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 "base/linux_util.h"
 
 #include <stdlib.h>
 
 #include <vector>
 
@@ skipping 28 matching lines @@
       return false;
     env_value = ::getenv(alternate_case_var.c_str());
     if (env_value) {
       *result = env_value;
       return true;
     }
     return false;
   }
 };
 
+// Not needed for OS_CHROMEOS.
+#if defined(OS_LINUX)
 enum LinuxDistroState {
   STATE_DID_NOT_CHECK  = 0,
   STATE_CHECK_STARTED  = 1,
   STATE_CHECK_FINISHED = 2,
 };
 
 // Helper class for GetLinuxDistro().
 class LinuxDistroHelper {
  public:
   // Retrieves the Singleton.
@@ skipping 22 matching lines @@
   void CheckFinished() {
     AutoLock scoped_lock(lock_);
     DCHECK(state_ == STATE_CHECK_STARTED);
     state_ = STATE_CHECK_FINISHED;
   }
 
  private:
   Lock lock_;
   LinuxDistroState state_;
 };
+#endif  // if defined(OS_LINUX)
 
 }  // anonymous namespace
 
 namespace base {
 
 uint8_t* BGRAToRGBA(const uint8_t* pixels, int width, int height, int stride) {
   if (stride == 0)
     stride = width * 4;
 
   uint8_t* new_pixels = static_cast<uint8_t*>(malloc(height * stride));
 
   // We have to copy the pixels and swap from BGRA to RGBA.
   for (int i = 0; i < height; ++i) {
     for (int j = 0; j < width; ++j) {
       int idx = i * stride + j * 4;
       new_pixels[idx] = pixels[idx + 2];
       new_pixels[idx + 1] = pixels[idx + 1];
       new_pixels[idx + 2] = pixels[idx];
       new_pixels[idx + 3] = pixels[idx + 3];
     }
   }
 
   return new_pixels;
 }
 
 // We use this static string to hold the Linux distro info. If we
 // crash, the crash handler code will send this in the crash dump.
-std::string linux_distro = "Unknown";
+std::string linux_distro =
+#if defined(OS_CHROMEOS)
+    "CrOS";
+#else  // if defined(OS_LINUX)
+    "Unknown";
+#endif
 
 std::string GetLinuxDistro() {
+#if defined(OS_CHROMEOS)
+  return linux_distro;
+#else  // if defined(OS_LINUX)
   LinuxDistroHelper* distro_state_singleton = LinuxDistroHelper::Get();
   LinuxDistroState state = distro_state_singleton->State();
   if (STATE_DID_NOT_CHECK == state) {
-#if defined(OS_CHROMEOS)
-    linux_distro = "CrOS";
-#else // if defined(OS_LINUX)
     // We do this check only once per process. If it fails, there's
     // little reason to believe it will work if we attempt to run
     // lsb_release again.
     std::vector<std::string> argv;
     argv.push_back("lsb_release");
     argv.push_back("-d");
     std::string output;
     base::GetAppOutput(CommandLine(argv), &output);
     if (output.length() > 0) {
       // lsb_release -d should return: Description:<tab>Distro Info
       static const std::string field = "Description:\t";
       if (output.compare(0, field.length(), field) == 0)
         linux_distro = output.substr(field.length());
     }
-#endif
     distro_state_singleton->CheckFinished();
     return linux_distro;
   } else if (STATE_CHECK_STARTED == state) {
     // If the distro check above is in progress in some other thread, we're
     // not going to wait for the results.
     return "Unknown";
   } else {
     // In STATE_CHECK_FINISHED, no more writing to |linux_distro|.
     return linux_distro;
   }
+#endif
 }
 
 // static
 EnvironmentVariableGetter* EnvironmentVariableGetter::Create() {
   return new EnvironmentVariableGetterImpl();
 }
 
 DesktopEnvironment GetDesktopEnvironment(EnvironmentVariableGetter* env) {
   std::string desktop_session;
   if (env->Getenv("DESKTOP_SESSION", &desktop_session)) {
@@ skipping 31 matching lines @@
       return "KDE4";
   }
   return NULL;
 }
 
 const char* GetDesktopEnvironmentName(EnvironmentVariableGetter* env) {
   return GetDesktopEnvironmentName(GetDesktopEnvironment(env));
 }
 
 }  // namespace base