Initial skeleton for System Indicator API

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

+// 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.
+
+#ifndef CHROME_BROWSER_EXTENSIONS_API_SYSTEM_INDICATOR_SYSTEM_INDICATOR_API_H__

[jianli, 2012/11/12 23:00:55]

nit: only one trailing underscore is needed.

[dewittj, 2012/11/13 06:58:54]

Fixed

+#define CHROME_BROWSER_EXTENSIONS_API_SYSTEM_INDICATOR_SYSTEM_INDICATOR_API_H__
+
+#include "chrome/browser/extensions/extension_function.h"
+
+namespace extensions {
+
+class SystemIndicatorSetURLFunction : public SyncExtensionFunction {
+ public:
+  SystemIndicatorSetURLFunction();
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.systemIndicator.setURL")
+
+ protected:
+  virtual ~SystemIndicatorSetURLFunction() {}
+
+  virtual bool RunImpl() OVERRIDE;
+};
+
+class SystemIndicatorSetImageDataFunction : public SyncExtensionFunction {
+ public:
+  SystemIndicatorSetImageDataFunction();
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.systemIndicator.setImageData")
+
+ protected:
+  virtual ~SystemIndicatorSetImageDataFunction() {}
+
+  virtual bool RunImpl() OVERRIDE;
+};
+
+class SystemIndicatorShowFunction : public SyncExtensionFunction {
+ public:
+  SystemIndicatorShowFunction();
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.systemIndicator.show")
+
+ protected:
+  virtual ~SystemIndicatorShowFunction() {}
+
+  virtual bool RunImpl() OVERRIDE;
+};
+
+class SystemIndicatorHideFunction : public SyncExtensionFunction {
+ public:
+  SystemIndicatorHideFunction();
+  DECLARE_EXTENSION_FUNCTION_NAME("experimental.systemIndicator.hide");
+
+ protected:
+  virtual ~SystemIndicatorHideFunction() {}
+
+  virtual bool RunImpl() OVERRIDE;
+ private:

[jianli, 2012/11/12 23:00:55]

nit: not needed

[dewittj, 2012/11/13 06:58:54]

Fixed

+};
+
+}  // namespace extensions
+#endif  // CHROME_BROWSER_EXTENSIONS_API_SYSTEM_INDICATOR_SYSTEM_INDICATOR_API_H__

[jianli, 2012/11/12 23:00:55]

nit: empty line before this line.

[jianli, 2012/11/12 23:00:55]

nit: inconsistent naming in the comment

[dewittj, 2012/11/13 06:58:54]

Fixed

[dewittj, 2012/11/13 06:58:54]

Fixed