Revision: 16881
Author:   [email protected]
Date:     Mon Sep 23 11:53:24 2013 UTC
Log: Revert "Representation of interceptor / handler properties are tagged."

This reverts part of the back merge in r16811 that includes r16471.

[email protected]
BUG=295423

Review URL: https://codereview.chromium.org/24235008
http://code.google.com/p/v8/source/detail?r=16881

Modified:
 /branches/3.20/src/property.h
 /branches/3.20/src/version.cc
 /branches/3.20/test/cctest/test-api.cc

=======================================
--- /branches/3.20/src/property.h       Thu Sep 19 08:18:03 2013 UTC
+++ /branches/3.20/src/property.h       Mon Sep 23 11:53:24 2013 UTC
@@ -225,14 +225,14 @@
   void HandlerResult(JSProxy* proxy) {
     lookup_type_ = HANDLER_TYPE;
     holder_ = proxy;
-    details_ = PropertyDetails(NONE, HANDLER, Representation::Tagged());
+    details_ = PropertyDetails(NONE, HANDLER, Representation::None());
     cacheable_ = false;
   }

   void InterceptorResult(JSObject* holder) {
     lookup_type_ = INTERCEPTOR_TYPE;
     holder_ = holder;
- details_ = PropertyDetails(NONE, INTERCEPTOR, Representation::Tagged());
+    details_ = PropertyDetails(NONE, INTERCEPTOR, Representation::None());
   }

   void NotFound() {
=======================================
--- /branches/3.20/src/version.cc       Mon Sep 23 11:26:03 2013 UTC
+++ /branches/3.20/src/version.cc       Mon Sep 23 11:53:24 2013 UTC
@@ -35,7 +35,7 @@
 #define MAJOR_VERSION     3
 #define MINOR_VERSION     20
 #define BUILD_NUMBER      17
-#define PATCH_LEVEL       9
+#define PATCH_LEVEL       10
 // Use 1 for candidates and 0 otherwise.
 // (Boolean macro values are not supported by all preprocessors.)
 #define IS_CANDIDATE_VERSION 0
=======================================
--- /branches/3.20/test/cctest/test-api.cc      Thu Sep 19 08:18:03 2013 UTC
+++ /branches/3.20/test/cctest/test-api.cc      Mon Sep 23 11:53:24 2013 UTC
@@ -20172,22 +20172,6 @@
   CheckCorrectThrow("%DefineOrRedefineAccessorProperty("
                         "other, 'x', null, null, 1)");
 }
-
-
-THREADED_TEST(Regress256330) {
-  i::FLAG_allow_natives_syntax = true;
-  LocalContext context;
-  v8::HandleScope scope(context->GetIsolate());
-  Handle<FunctionTemplate> templ = FunctionTemplate::New();
-  AddInterceptor(templ, EmptyInterceptorGetter, EmptyInterceptorSetter);
-  context->Global()->Set(v8_str("Bug"), templ->GetFunction());
-  CompileRun("\"use strict\"; var o = new Bug;"
-             "function f(o) { o.x = 10; };"
-             "f(o); f(o); f(o);"
-             "%OptimizeFunctionOnNextCall(f);"
-             "f(o);");
-  ExpectBoolean("%GetOptimizationStatus(f) != 2", true);
-}


 #endif  // WIN32

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to