Chromium extension API - written as part of course in Open Software Development at Reykjavik Univer…

chrome/browser/extensions/api/time/time_api.h

Index: chrome/browser/extensions/api/time/time_api.h
diff --git a/chrome/browser/extensions/api/time/time_api.h b/chrome/browser/extensions/api/time/time_api.h
new file mode 100644
index 0000000000000000000000000000000000000000..5fcc51122dc9fa552b991b61825086b7b02d6b9e
--- /dev/null
+++ b/chrome/browser/extensions/api/time/time_api.h
@@ -0,0 +1,21 @@
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.

[Jói, 2013/06/28 10:49:15]

2013

[hpjonsson, 2013/06/28 12:13:04]

Done.

+// 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_TIME_TIME_API_H_
+#define CHROME_BROWSER_EXTENSIONS_API_TIME_TIME_API_H_
+
+#include "chrome/common/extensions/api/time.h"
+#include "chrome/browser/extensions/extension_function.h"
+

[Jói, 2013/06/28 10:49:15]

One blank line is enough.

[hpjonsson, 2013/06/28 12:13:04]

Done.

+
+namespace extensions {

[Jói, 2013/06/28 10:49:15]

Usually we put a blank line after this.

[hpjonsson, 2013/06/28 12:13:04]

Done.

+class TimeGetTimeFunction : public AsyncExtensionFunction {
+  private:

[Jói, 2013/06/28 10:49:15]

indent -1

+  virtual ~TimeGetTimeFunction();

[Jói, 2013/06/28 10:49:15]

Not sure, but the destructor probably needs to be public. If it compiles like
this, fine.

[einartryggvi, 2013/06/28 10:57:49]

We got an error saying a virtual destructor has to be private, thought it was
weird though.

On 2013/06/28 10:49:15, Jói wrote:

+  DECLARE_EXTENSION_FUNCTION("time.getTime", TIME_GETTIME);
+  virtual bool RunImpl() OVERRIDE;
+  };

[Jói, 2013/06/28 10:49:15]

indent -2

[hpjonsson, 2013/06/28 12:13:04]

Done.

+}  // namespace extensions
+
+#endif  // CHROME_BROWSER_EXTENSIONS_API_TIME_TIME_API_H_
+