Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6)

Unified Diff: extensions/browser/api/async_api_function.h

Issue 177003015: Pull AsyncApiFunction out of src/chrome (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: extensions/browser/api/async_api_function.h
diff --git a/chrome/browser/extensions/api/api_function.h b/extensions/browser/api/async_api_function.h
similarity index 70%
rename from chrome/browser/extensions/api/api_function.h
rename to extensions/browser/api/async_api_function.h
index 5fb32a9ae2274afed9edbf94c4feb32636a2d2ec..dd4b500e72b324955e99bfd2e490610d51b22b22 100644
--- a/chrome/browser/extensions/api/api_function.h
+++ b/extensions/browser/api/async_api_function.h
@@ -1,27 +1,18 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright 2014 The Chromium Authors. All rights reserved.
James Cook 2014/02/26 17:57:21 nit: I don't think we update the copyright date fo
Ken Rockot(use gerrit already) 2014/02/26 18:20:44 I was considering it to be a new file, but I guess
// 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_API_FUNCTION_H_
-#define CHROME_BROWSER_EXTENSIONS_API_API_FUNCTION_H_
+#ifndef EXTENSIONS_BROWSER_API_ASYNC_API_FUCTION_H_
+#define EXTENSIONS_BROWSER_API_ASYNC_API_FUCTION_H_
-#include "chrome/browser/extensions/chrome_extension_function.h"
#include "content/public/browser/browser_thread.h"
+#include "extensions/browser/extension_function.h"
namespace extensions {
-// Base class for API functions. TODO(miket): there isn't much here anymore
-// since the removal of ApiResourceEventRouter. Should we promote all its
-// subclasses to UIThreadExtensionFunctions?
-class ApiFunction : public ChromeAsyncExtensionFunction {
- protected:
- ApiFunction();
- virtual ~ApiFunction();
-};
-
// AsyncApiFunction provides convenient thread management for APIs that need to
// do essentially all their work on a thread other than the UI thread.
-class AsyncApiFunction : public ApiFunction {
+class AsyncApiFunction : public UIThreadExtensionFunction {
protected:
AsyncApiFunction();
virtual ~AsyncApiFunction();
@@ -66,4 +57,4 @@ class AsyncApiFunction : public ApiFunction {
} // namespace extensions
-#endif // CHROME_BROWSER_EXTENSIONS_API_API_FUNCTION_H_
+#endif // EXTENSIONS_BROWSER_API_ASYNC_API_FUCTION_H_

Powered by Google App Engine
This is Rietveld 408576698