Revision: 23117
Author:   joc...@chromium.org
Date:     Thu Aug 14 08:49:13 2014 UTC
Log:      Version 3.28.71.2 (merged r23115)

Revert 23058 - "Allow externalizing cons and sliced string."

TBR=yang...@chromium.org

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

Modified:
 /trunk/src/api.cc
 /trunk/src/version.cc

=======================================
--- /trunk/src/api.cc   Thu Aug 14 08:46:03 2014 UTC
+++ /trunk/src/api.cc   Thu Aug 14 08:49:13 2014 UTC
@@ -5545,6 +5545,11 @@
   i::Handle<i::String> obj = Utils::OpenHandle(this);
   i::Isolate* isolate = obj->GetIsolate();

+  // TODO(yangguo): Externalizing sliced/cons strings allocates.
+  // This rule can be removed when all code that can
+  // trigger an access check is handlified and therefore GC safe.
+  if (isolate->heap()->old_pointer_space()->Contains(*obj)) return false;
+
   if (isolate->string_tracker()->IsFreshUnusedString(obj)) return false;
   int size = obj->Size();  // Byte size of the original string.
   if (size < i::ExternalString::kShortSize) return false;
=======================================
--- /trunk/src/version.cc       Thu Aug 14 08:46:03 2014 UTC
+++ /trunk/src/version.cc       Thu Aug 14 08:49:13 2014 UTC
@@ -35,7 +35,7 @@
 #define MAJOR_VERSION     3
 #define MINOR_VERSION     28
 #define BUILD_NUMBER      71
-#define PATCH_LEVEL       1
+#define PATCH_LEVEL       2
 // Use 1 for candidates and 0 otherwise.
 // (Boolean macro values are not supported by all preprocessors.)
 #define IS_CANDIDATE_VERSION 0

--
--
v8-dev mailing list
v8-dev@googlegroups.com
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 v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to