Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(855)

Unified Diff: base/linux_util.cc

Issue 155792: Try again: Add proxy config (using gnome-network-preferences) (Closed)
Patch Set: nits fixed Created 11 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: base/linux_util.cc
diff --git a/base/linux_util.cc b/base/linux_util.cc
index 604980b2b51abc73dcfb8a03a7d544b07850c2df..bc99d44a92bbee130a4922e39a7a46383877f5a2 100644
--- a/base/linux_util.cc
+++ b/base/linux_util.cc
@@ -10,6 +10,41 @@
#include "base/command_line.h"
#include "base/process_util.h"
+#include "base/string_util.h"
+
+namespace {
+
+class EnvironmentVariableGetterImpl
+ : public base::EnvironmentVariableGetter {
+ public:
+ virtual bool Getenv(const char* variable_name, std::string* result) {
+ const char* env_value = ::getenv(variable_name);
+ if (env_value) {
+ // Note that the variable may be defined but empty.
+ *result = env_value;
+ return true;
+ }
+ // Some commonly used variable names are uppercase while others
+ // are lowercase, which is inconsistent. Let's try to be helpful
+ // and look for a variable name with the reverse case.
+ char first_char = variable_name[0];
+ std::string alternate_case_var;
+ if (first_char >= 'a' && first_char <= 'z')
+ alternate_case_var = StringToUpperASCII(std::string(variable_name));
+ else if (first_char >= 'A' && first_char <= 'Z')
+ alternate_case_var = StringToLowerASCII(std::string(variable_name));
+ else
+ return false;
+ env_value = ::getenv(alternate_case_var.c_str());
+ if (env_value) {
+ *result = env_value;
+ return true;
+ }
+ return false;
+ }
+};
+
+} // anonymous namespace
namespace base {
@@ -59,4 +94,19 @@ std::string GetLinuxDistro() {
return linux_distro;
}
+// static
+EnvironmentVariableGetter* EnvironmentVariableGetter::Create() {
+ return new EnvironmentVariableGetterImpl();
+}
+
+bool UseGnomeForSettings(EnvironmentVariableGetter* env_var_getter) {
+ // GNOME_DESKTOP_SESSION_ID being defined is a good indication that
+ // we are probably running under GNOME.
+ // Note: KDE_FULL_SESSION is a corresponding env var to recognize KDE.
+ std::string dummy, desktop_session;
+ return env_var_getter->Getenv("GNOME_DESKTOP_SESSION_ID", &dummy)
+ || (env_var_getter->Getenv("DESKTOP_SESSION", &desktop_session)
+ && desktop_session == "gnome");
+}
+
} // namespace base

Powered by Google App Engine
This is Rietveld 408576698