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

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

Issue 2133203002: [Extensions] Code cleanup (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: update Created 4 years, 5 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/execute_code_function.h
diff --git a/extensions/browser/api/execute_code_function.h b/extensions/browser/api/execute_code_function.h
index d7d965f9e9ca027cb4f19da80d2b40a3043fd16d..a483e13ab3b57379e630d2d78efd4ffc311cc8da 100644
--- a/extensions/browser/api/execute_code_function.h
+++ b/extensions/browser/api/execute_code_function.h
@@ -5,6 +5,7 @@
#ifndef EXTENSIONS_BROWSER_API_EXECUTE_CODE_FUNCTION_H_
#define EXTENSIONS_BROWSER_API_EXECUTE_CODE_FUNCTION_H_
+#include "base/macros.h"
#include "extensions/browser/extension_function.h"
#include "extensions/browser/script_executor.h"
#include "extensions/common/api/extension_types.h"
@@ -45,9 +46,7 @@ class ExecuteCodeFunction : public AsyncExtensionFunction {
const std::string& data);
const HostID& host_id() const { return host_id_; }
- void set_host_id(HostID host_id) {
- host_id_ = host_id;
- }
+ void set_host_id(const HostID& host_id) { host_id_ = host_id; }
// The injection details.
std::unique_ptr<api::extension_types::InjectDetails> details_;
@@ -78,6 +77,8 @@ class ExecuteCodeFunction : public AsyncExtensionFunction {
// The ID of the injection host.
HostID host_id_;
+
+ DISALLOW_COPY_AND_ASSIGN(ExecuteCodeFunction);
};
} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698