Initial skeleton for System Indicator API

chrome/browser/extensions/api/system_indicator/system_indicator_api.cc

Index: chrome/browser/extensions/api/system_indicator/system_indicator_api.cc
diff --git a/chrome/browser/extensions/api/system_indicator/system_indicator_api.cc b/chrome/browser/extensions/api/system_indicator/system_indicator_api.cc
new file mode 100644
index 0000000000000000000000000000000000000000..192ecf6355a21f78ceae3d4fc4b3927c7b972448
--- /dev/null
+++ b/chrome/browser/extensions/api/system_indicator/system_indicator_api.cc
@@ -0,0 +1,25 @@
+// Copyright (c) 2012 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/system_indicator/system_indicator_api.h"
+
+#include "base/values.h"
+#include "chrome/browser/extensions/extension_service.h"
+#include "chrome/browser/extensions/extension_system.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/common/extensions/api/system_indicator.h"
+
+namespace extensions {
+
+bool SystemIndicatorEnableFunction::ValidateArguments() {
+  // No arguments besides the callback.
+  return true;

[not at google - send to devlin, 2012/11/21 23:39:23]

oh I didn't realise the reason for the change to ValidateArguments was in order
to override it here.

What ExtensionAction does is nearly good enough. They know how to deal with the
lack of a tab ID property, so that's fine.

I guess you just want to prevent APIs from adding a tab ID? In which case I'd
just do that as a little check inside extension_actions_api.cc (there are
already plenty of those for other things). That said, it's unlikely that the UI
implementation will care about the tab ID, so eh. This validation is already
being done on the renderer. The validation we do on the browser is only for
sanity and security-important stuff, which the tab ID isn't.

So yeah - I'd delete this stuff.

[dewittj, 2012/11/26 18:32:58]

Done.

+}
+
+bool SystemIndicatorDisableFunction::ValidateArguments() {
+  // No arguments besides the callback.
+  return true;
+}
+
+}  // namespace extensions