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

Side by Side Diff: src/objects.cc

Issue 896643003: Class methods should be non enumerable (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: formatting Created 5 years, 10 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 the V8 project authors. All rights reserved. 1 // Copyright 2013 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <sstream> 5 #include <sstream>
6 6
7 #include "src/v8.h" 7 #include "src/v8.h"
8 8
9 #include "src/accessors.h" 9 #include "src/accessors.h"
10 #include "src/allocation-site-scopes.h" 10 #include "src/allocation-site-scopes.h"
(...skipping 12707 matching lines...) Expand 10 before | Expand all | Expand 10 after
12718 Handle<JSProxy>::cast(object), object, index, value, strict_mode); 12718 Handle<JSProxy>::cast(object), object, index, value, strict_mode);
12719 } 12719 }
12720 return JSObject::SetElement( 12720 return JSObject::SetElement(
12721 Handle<JSObject>::cast(object), index, value, attributes, strict_mode); 12721 Handle<JSObject>::cast(object), index, value, attributes, strict_mode);
12722 } 12722 }
12723 12723
12724 12724
12725 MaybeHandle<Object> JSObject::SetOwnElement(Handle<JSObject> object, 12725 MaybeHandle<Object> JSObject::SetOwnElement(Handle<JSObject> object,
12726 uint32_t index, 12726 uint32_t index,
12727 Handle<Object> value, 12727 Handle<Object> value,
12728 PropertyAttributes attributes,
12728 StrictMode strict_mode) { 12729 StrictMode strict_mode) {
12729 DCHECK(!object->HasExternalArrayElements()); 12730 DCHECK(!object->HasExternalArrayElements());
12730 return JSObject::SetElement(object, index, value, NONE, strict_mode, false); 12731 return JSObject::SetElement(object, index, value, attributes, strict_mode,
12732 false);
12731 } 12733 }
12732 12734
12733 12735
12734 MaybeHandle<Object> JSObject::SetElement(Handle<JSObject> object, 12736 MaybeHandle<Object> JSObject::SetElement(Handle<JSObject> object,
12735 uint32_t index, 12737 uint32_t index,
12736 Handle<Object> value, 12738 Handle<Object> value,
12737 PropertyAttributes attributes, 12739 PropertyAttributes attributes,
12738 StrictMode strict_mode, 12740 StrictMode strict_mode,
12739 bool check_prototype, 12741 bool check_prototype,
12740 SetPropertyMode set_mode) { 12742 SetPropertyMode set_mode) {
(...skipping 4147 matching lines...) Expand 10 before | Expand all | Expand 10 after
16888 Handle<DependentCode> codes = 16890 Handle<DependentCode> codes =
16889 DependentCode::Insert(handle(cell->dependent_code(), info->isolate()), 16891 DependentCode::Insert(handle(cell->dependent_code(), info->isolate()),
16890 DependentCode::kPropertyCellChangedGroup, 16892 DependentCode::kPropertyCellChangedGroup,
16891 info->object_wrapper()); 16893 info->object_wrapper());
16892 if (*codes != cell->dependent_code()) cell->set_dependent_code(*codes); 16894 if (*codes != cell->dependent_code()) cell->set_dependent_code(*codes);
16893 info->dependencies(DependentCode::kPropertyCellChangedGroup)->Add( 16895 info->dependencies(DependentCode::kPropertyCellChangedGroup)->Add(
16894 cell, info->zone()); 16896 cell, info->zone());
16895 } 16897 }
16896 16898
16897 } } // namespace v8::internal 16899 } } // namespace v8::internal
OLDNEW
« no previous file with comments | « src/objects.h ('k') | src/objects-inl.h » ('j') | src/runtime/runtime-object.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698