Implement Command Line Private extension API

chrome/browser/extensions/api/command_line_private/command_line_private_api.cc

Index: chrome/browser/extensions/api/command_line_private/command_line_private_api.cc
===================================================================
--- chrome/browser/extensions/api/command_line_private/command_line_private_api.cc	(revision 0)
+++ chrome/browser/extensions/api/command_line_private/command_line_private_api.cc	(revision 0)
@@ -0,0 +1,32 @@
+// Copyright 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.
+
+#include "chrome/browser/extensions/api/command_line_private/command_line_private_api.h"
+
+#include <string>
+
+#include "base/command_line.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/values.h"
+#include "chrome/common/extensions/api/command_line_private.h"
+
+namespace command_line_private = extensions::api::command_line_private;

[not at google - send to devlin, 2013/04/15 06:08:45]

can go inside the extensions { namespace below

[方觉(Fang Jue), 2013/04/15 06:25:03]

Done.

+
+namespace extensions {
+
+bool CommandLinePrivateHasSwitchFunction::RunImpl() {
+  scoped_ptr<command_line_private::HasSwitch::Params> params(
+      command_line_private::HasSwitch::Params::Create(*args_));
+  EXTENSION_FUNCTION_VALIDATE(params.get());

[not at google - send to devlin, 2013/04/15 06:08:45]

don't need .get()

[方觉(Fang Jue), 2013/04/15 06:25:03]

I looked at other API implementations (such as alarms) and they all use
EXTENSION_FUNCTION_VALIDATE(params.get());

[not at google - send to devlin, 2013/04/15 06:26:52]

Nevertheless, you don't need .get().

+
+  if (params->name.empty())
+    return false;

[not at google - send to devlin, 2013/04/15 06:08:45]

should also set error_ here

[方觉(Fang Jue), 2013/04/15 06:25:03]

Done.

+
+  const CommandLine* command_line = CommandLine::ForCurrentProcess();
+  bool result = command_line->HasSwitch(params->name);
+  SetResult(base::Value::CreateBooleanValue(result));

[not at google - send to devlin, 2013/04/15 06:08:45]

typically I'd (a) inline most of this and (b) use
command_line_private::HasSwitch::Result::Create instead of CreateBooleanValue
(in which case you need to assign to result_ rather than use SetResult).

[方觉(Fang Jue), 2013/04/15 06:25:03]

Done.

+  return true;
+}
+
+}  // namespace extensions
Property changes on: chrome/browser/extensions/api/command_line_private/command_line_private_api.cc
___________________________________________________________________
Added: svn:eol-style
   + LF