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

Unified Diff: src/accessors.cc

Issue 1069883002: WIP SharedArrayBuffer implementation (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: update MakeTypeError calls Created 5 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
Index: src/accessors.cc
diff --git a/src/accessors.cc b/src/accessors.cc
index f220b829bf9dc0226e75bdfaa74bed46534520e4..caac40da015df3929752df41d9224bd343400899 100644
--- a/src/accessors.cc
+++ b/src/accessors.cc
@@ -79,6 +79,7 @@ bool Accessors::IsJSObjectFieldAccessor(Handle<Map> map, Handle<Name> name,
CheckForName(name, isolate->factory()->length_string(),
JSArray::kLengthOffset, object_offset);
case JS_ARRAY_BUFFER_TYPE:
+ case JS_SHARED_ARRAY_BUFFER_TYPE:
return CheckForName(name, isolate->factory()->byte_length_string(),
JSArrayBuffer::kByteLengthOffset, object_offset);
default:
@@ -99,6 +100,7 @@ bool Accessors::IsJSArrayBufferViewFieldAccessor(Handle<Map> map,
switch (map->instance_type()) {
case JS_TYPED_ARRAY_TYPE:
+ case JS_SHARED_TYPED_ARRAY_TYPE:
jochen (gone - plz use gerrit) 2015/04/28 18:31:46 can shared array buffers be neutered?
binji 2015/04/29 18:27:22 no, does that affect the code here?
// %TypedArray%.prototype is non-configurable, and so are the following
// named properties on %TypedArray%.prototype, so we can directly inline
// the field-load for typed array maps that still use their

Powered by Google App Engine
This is Rietveld 408576698