Intorduce a separate preference for 'proxy server mode'

chrome/browser/prefs/proxy_prefs.cc

Index: chrome/browser/prefs/proxy_prefs.cc
diff --git a/chrome/browser/prefs/proxy_prefs.cc b/chrome/browser/prefs/proxy_prefs.cc
new file mode 100644
index 0000000000000000000000000000000000000000..f89a5a56cbfb30dafff217ac1032b54e9d123966
--- /dev/null
+++ b/chrome/browser/prefs/proxy_prefs.cc
@@ -0,0 +1,45 @@
+// Copyright (c) 2010 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 "chrome/browser/prefs/proxy_prefs.h"
+
+#include "base/basictypes.h"
+#include "base/logging.h"
+
+namespace {
+  // These names are exposed to the proxy extension API. They must be in sync
+  // with the constants of ProxyPrefs.

[danno, 2010/12/17 14:21:11]

nit: you should also have a corresponding comment in the enum, e.g. "if you add
an enum constant, add a string to the mode names".

[gfeher, 2010/12/17 15:44:17]

Done.

+  const char* kProxyModeNames[] = { "disabled",
+                                    "auto_detect",
+                                    "manual",
+                                    "system" };
+}  // namespace
+
+COMPILE_ASSERT(arraysize(kProxyModeNames) == ProxyPrefs::NUM_MODES,
+               kProxyModeNames_must_have_size_of_NUM_MODES);
+
+// static
+bool ProxyPrefs::IntToProxyMode(
+    int in_value,
+    ProxyPrefs::ProxyServerMode* out_value) {
+  DCHECK(out_value);
+  if (in_value < 0 || in_value >= ProxyPrefs::NUM_MODES) {
+    return false;
+  }
+  *out_value = static_cast<ProxyPrefs::ProxyServerMode> (in_value);

[danno, 2010/12/17 14:21:11]

nit: extra space after >

[gfeher, 2010/12/17 15:44:17]

Done.

+  return true;
+}
+
+// static
+bool ProxyPrefs::StringToProxyMode(
+    const std::string& in_value,
+    ProxyPrefs::ProxyServerMode* out_value) {
+  DCHECK(out_value);
+  for (int i = 0; i < ProxyPrefs::NUM_MODES; i++) {
+    if (in_value == kProxyModeNames[i]) {
+      return IntToProxyMode(i, out_value);
+    }
+  }
+  return false;
+}