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

Unified Diff: src/accessors.h

Issue 232933003: Revert r20652 "Handlify and convert string.length to new API-style accessor." (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 6 years, 8 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 | « no previous file | src/accessors.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/accessors.h
diff --git a/src/accessors.h b/src/accessors.h
index db506cef976e7724b4da21fd6dbd632b3a24e119..83a847222b44e69a8341bbaac08bdcf95aaa5dac 100644
--- a/src/accessors.h
+++ b/src/accessors.h
@@ -43,6 +43,7 @@ namespace internal {
V(FunctionArguments) \
V(FunctionCaller) \
V(ArrayLength) \
+ V(StringLength) \
V(ScriptSource) \
V(ScriptName) \
V(ScriptId) \
@@ -56,9 +57,6 @@ namespace internal {
V(ScriptEvalFromScriptPosition) \
V(ScriptEvalFromFunctionName)
-#define ACCESSOR_INFO_LIST(V) \
- V(StringLength) \
-
// Accessors contains all predefined proxy accessors.
class Accessors : public AllStatic {
@@ -69,30 +67,11 @@ class Accessors : public AllStatic {
ACCESSOR_DESCRIPTOR_LIST(ACCESSOR_DESCRIPTOR_DECLARATION)
#undef ACCESSOR_DESCRIPTOR_DECLARATION
-#define ACCESSOR_INFO_DECLARATION(name) \
- static void name##Getter( \
- v8::Local<v8::String> name, \
- const v8::PropertyCallbackInfo<v8::Value>& info); \
- static void name##Setter( \
- v8::Local<v8::String> name, \
- v8::Local<v8::Value> value, \
- const v8::PropertyCallbackInfo<void>& info); \
- static Handle<AccessorInfo> name##Info( \
- Isolate* isolate, \
- PropertyAttributes attributes);
- ACCESSOR_INFO_LIST(ACCESSOR_INFO_DECLARATION)
-#undef ACCESSOR_INFO_DECLARATION
-
enum DescriptorId {
#define ACCESSOR_DESCRIPTOR_DECLARATION(name) \
k##name,
ACCESSOR_DESCRIPTOR_LIST(ACCESSOR_DESCRIPTOR_DECLARATION)
#undef ACCESSOR_DESCRIPTOR_DECLARATION
-#define ACCESSOR_INFO_DECLARATION(name) \
- k##name##Getter, \
- k##name##Setter,
- ACCESSOR_INFO_LIST(ACCESSOR_INFO_DECLARATION)
-#undef ACCESSOR_INFO_DECLARATION
descriptorCount
};
@@ -113,6 +92,7 @@ class Accessors : public AllStatic {
Handle<String> name,
int* object_offset);
+
private:
// Accessor functions only used through the descriptor.
static MaybeObject* FunctionSetPrototype(Isolate* isolate,
« no previous file with comments | « no previous file | src/accessors.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698