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

Unified Diff: src/api-arguments.h

Issue 2039913002: Make api-arguments.h not include inline headers. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@local_cleanup-includes-type-feedback-vector
Patch Set: Addressed comments. Created 4 years, 6 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
« no previous file with comments | « BUILD.gn ('k') | src/api-arguments.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/api-arguments.h
diff --git a/src/api-arguments.h b/src/api-arguments.h
index f1573fe83fa469f276ad8832a9ee4b28bf0291fc..0dfe61824a982181e377b5775dc132af1952a8f6 100644
--- a/src/api-arguments.h
+++ b/src/api-arguments.h
@@ -7,8 +7,6 @@
#include "src/api.h"
#include "src/isolate.h"
-#include "src/tracing/trace-event.h"
-#include "src/vm-state-inl.h"
namespace v8 {
namespace internal {
@@ -108,92 +106,24 @@ class PropertyCallbackArguments
*/
Handle<JSObject> Call(IndexedPropertyEnumeratorCallback f);
-#define FOR_EACH_CALLBACK_TABLE_MAPPING_1_NAME(F) \
- F(AccessorNameGetterCallback, "get", v8::Value, Object) \
- F(GenericNamedPropertyQueryCallback, "has", v8::Integer, Object) \
- F(GenericNamedPropertyDeleterCallback, "delete", v8::Boolean, Object)
+ inline Handle<Object> Call(AccessorNameGetterCallback f, Handle<Name> name);
+ inline Handle<Object> Call(GenericNamedPropertyQueryCallback f,
+ Handle<Name> name);
+ inline Handle<Object> Call(GenericNamedPropertyDeleterCallback f,
+ Handle<Name> name);
-#define WRITE_CALL_1_NAME(Function, type, ApiReturn, InternalReturn) \
- Handle<InternalReturn> Call(Function f, Handle<Name> name) { \
- Isolate* isolate = this->isolate(); \
- RuntimeCallTimerScope timer(isolate, &RuntimeCallStats::Function); \
- VMState<EXTERNAL> state(isolate); \
- ExternalCallbackScope call_scope(isolate, FUNCTION_ADDR(f)); \
- PropertyCallbackInfo<ApiReturn> info(begin()); \
- LOG(isolate, \
- ApiNamedPropertyAccess("interceptor-named-" type, holder(), *name)); \
- f(v8::Utils::ToLocal(name), info); \
- return GetReturnValue<InternalReturn>(isolate); \
- }
-
- FOR_EACH_CALLBACK_TABLE_MAPPING_1_NAME(WRITE_CALL_1_NAME)
-
-#undef FOR_EACH_CALLBACK_TABLE_MAPPING_1_NAME
-#undef WRITE_CALL_1_NAME
-
-#define FOR_EACH_CALLBACK_TABLE_MAPPING_1_INDEX(F) \
- F(IndexedPropertyGetterCallback, "get", v8::Value, Object) \
- F(IndexedPropertyQueryCallback, "has", v8::Integer, Object) \
- F(IndexedPropertyDeleterCallback, "delete", v8::Boolean, Object)
-
-#define WRITE_CALL_1_INDEX(Function, type, ApiReturn, InternalReturn) \
- Handle<InternalReturn> Call(Function f, uint32_t index) { \
- Isolate* isolate = this->isolate(); \
- RuntimeCallTimerScope timer(isolate, &RuntimeCallStats::Function); \
- VMState<EXTERNAL> state(isolate); \
- ExternalCallbackScope call_scope(isolate, FUNCTION_ADDR(f)); \
- PropertyCallbackInfo<ApiReturn> info(begin()); \
- LOG(isolate, ApiIndexedPropertyAccess("interceptor-indexed-" type, \
- holder(), index)); \
- f(index, info); \
- return GetReturnValue<InternalReturn>(isolate); \
- }
+ inline Handle<Object> Call(IndexedPropertyGetterCallback f, uint32_t index);
+ inline Handle<Object> Call(IndexedPropertyQueryCallback f, uint32_t index);
+ inline Handle<Object> Call(IndexedPropertyDeleterCallback f, uint32_t index);
- FOR_EACH_CALLBACK_TABLE_MAPPING_1_INDEX(WRITE_CALL_1_INDEX)
+ inline Handle<Object> Call(GenericNamedPropertySetterCallback f,
+ Handle<Name> name, Handle<Object> value);
-#undef FOR_EACH_CALLBACK_TABLE_MAPPING_1_INDEX
-#undef WRITE_CALL_1_INDEX
+ inline Handle<Object> Call(IndexedPropertySetterCallback f, uint32_t index,
+ Handle<Object> value);
- Handle<Object> Call(GenericNamedPropertySetterCallback f, Handle<Name> name,
- Handle<Object> value) {
- Isolate* isolate = this->isolate();
- RuntimeCallTimerScope timer(
- isolate, &RuntimeCallStats::GenericNamedPropertySetterCallback);
- VMState<EXTERNAL> state(isolate);
- ExternalCallbackScope call_scope(isolate, FUNCTION_ADDR(f));
- PropertyCallbackInfo<v8::Value> info(begin());
- LOG(isolate,
- ApiNamedPropertyAccess("interceptor-named-set", holder(), *name));
- f(v8::Utils::ToLocal(name), v8::Utils::ToLocal(value), info);
- return GetReturnValue<Object>(isolate);
- }
-
- Handle<Object> Call(IndexedPropertySetterCallback f, uint32_t index,
- Handle<Object> value) {
- Isolate* isolate = this->isolate();
- RuntimeCallTimerScope timer(
- isolate, &RuntimeCallStats::IndexedPropertySetterCallback);
- VMState<EXTERNAL> state(isolate);
- ExternalCallbackScope call_scope(isolate, FUNCTION_ADDR(f));
- PropertyCallbackInfo<v8::Value> info(begin());
- LOG(isolate,
- ApiIndexedPropertyAccess("interceptor-indexed-set", holder(), index));
- f(index, v8::Utils::ToLocal(value), info);
- return GetReturnValue<Object>(isolate);
- }
-
- void Call(AccessorNameSetterCallback f, Handle<Name> name,
- Handle<Object> value) {
- Isolate* isolate = this->isolate();
- RuntimeCallTimerScope timer(isolate,
- &RuntimeCallStats::AccessorNameSetterCallback);
- VMState<EXTERNAL> state(isolate);
- ExternalCallbackScope call_scope(isolate, FUNCTION_ADDR(f));
- PropertyCallbackInfo<void> info(begin());
- LOG(isolate,
- ApiNamedPropertyAccess("interceptor-named-set", holder(), *name));
- f(v8::Utils::ToLocal(name), v8::Utils::ToLocal(value), info);
- }
+ inline void Call(AccessorNameSetterCallback f, Handle<Name> name,
+ Handle<Object> value);
private:
inline JSObject* holder() {
« no previous file with comments | « BUILD.gn ('k') | src/api-arguments.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698