[chromedriver] Implement proxy capability.

chrome/test/chromedriver/capabilities_parser.h

+// Copyright (c) 2013 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.
+
+#ifndef CHROME_TEST_CHROMEDRIVER_CAPABILITIES_PARSER_H_
+#define CHROME_TEST_CHROMEDRIVER_CAPABILITIES_PARSER_H_
+
+#include <string>
+
+#include "base/callback_forward.h"
+#include "base/files/file_path.h"
+#include "base/values.h"
+
+class Status;
+
+struct Capabilities {
+  Capabilities();
+  ~Capabilities();
+
+  bool HasAndroidPackage() const;
+
+  Status Parse(
+      const base::DictionaryValue& chrome_options,
+      const base::Callback<bool(const base::FilePath&)>& file_checker);
+
+  std::string android_package;
+
+  base::FilePath chrome_exe;
+  std::string log_path;
+  base::ListValue args;

[kkania, 2013/04/03 19:09:35]

how about a std::vector<string> instead or a CommandLine?

[chrisgao (Use stgao instead), 2013/04/06 01:11:25]

changed to a CommandLine.

+  const base::DictionaryValue* prefs;

[kkania, 2013/04/03 19:09:35]

i think the capabilities object should own these, either in scoped_ptr or a copy

[chrisgao (Use stgao instead), 2013/04/06 01:11:25]

Done.

+  const base::DictionaryValue* local_state;
+  const base::ListValue* extensions;

[kkania, 2013/04/03 19:09:35]

how about a std::vector<string>?

[chrisgao (Use stgao instead), 2013/04/06 01:11:25]

Done.

+};
+
+#endif  // CHROME_TEST_CHROMEDRIVER_CAPABILITIES_PARSER_H_