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

Side by Side Diff: src/objects.h

Issue 1717893002: Remove the Proxy enumerate trap (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Fix tests Created 4 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
« no previous file with comments | « src/js/proxy.js ('k') | src/objects.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 the V8 project authors. All rights reserved. 1 // Copyright 2015 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 #ifndef V8_OBJECTS_H_ 5 #ifndef V8_OBJECTS_H_
6 #define V8_OBJECTS_H_ 6 #define V8_OBJECTS_H_
7 7
8 #include <iosfwd> 8 #include <iosfwd>
9 9
10 #include "src/assert-scope.h" 10 #include "src/assert-scope.h"
(...skipping 9729 matching lines...) Expand 10 before | Expand all | Expand 10 after
9740 MUST_USE_RESULT static Maybe<bool> SetProperty(Handle<JSProxy> proxy, 9740 MUST_USE_RESULT static Maybe<bool> SetProperty(Handle<JSProxy> proxy,
9741 Handle<Name> name, 9741 Handle<Name> name,
9742 Handle<Object> value, 9742 Handle<Object> value,
9743 Handle<Object> receiver, 9743 Handle<Object> receiver,
9744 LanguageMode language_mode); 9744 LanguageMode language_mode);
9745 9745
9746 // ES6 9.5.10 (when passed SLOPPY) 9746 // ES6 9.5.10 (when passed SLOPPY)
9747 MUST_USE_RESULT static Maybe<bool> DeletePropertyOrElement( 9747 MUST_USE_RESULT static Maybe<bool> DeletePropertyOrElement(
9748 Handle<JSProxy> proxy, Handle<Name> name, LanguageMode language_mode); 9748 Handle<JSProxy> proxy, Handle<Name> name, LanguageMode language_mode);
9749 9749
9750 // ES6 9.5.11
9751 MUST_USE_RESULT static Maybe<bool> Enumerate(Isolate* isolate,
9752 Handle<JSReceiver> receiver,
9753 Handle<JSProxy> proxy,
9754 KeyAccumulator* accumulator);
9755
9756 // ES6 9.5.12 9750 // ES6 9.5.12
9757 MUST_USE_RESULT static Maybe<bool> OwnPropertyKeys( 9751 MUST_USE_RESULT static Maybe<bool> OwnPropertyKeys(
9758 Isolate* isolate, Handle<JSReceiver> receiver, Handle<JSProxy> proxy, 9752 Isolate* isolate, Handle<JSReceiver> receiver, Handle<JSProxy> proxy,
9759 PropertyFilter filter, KeyAccumulator* accumulator); 9753 PropertyFilter filter, KeyAccumulator* accumulator);
9760 9754
9761 MUST_USE_RESULT static Maybe<PropertyAttributes> GetPropertyAttributes( 9755 MUST_USE_RESULT static Maybe<PropertyAttributes> GetPropertyAttributes(
9762 LookupIterator* it); 9756 LookupIterator* it);
9763 9757
9764 // Dispatched behavior. 9758 // Dispatched behavior.
9765 DECLARE_PRINTER(JSProxy) 9759 DECLARE_PRINTER(JSProxy)
(...skipping 1075 matching lines...) Expand 10 before | Expand all | Expand 10 after
10841 } 10835 }
10842 return value; 10836 return value;
10843 } 10837 }
10844 }; 10838 };
10845 10839
10846 10840
10847 } // NOLINT, false-positive due to second-order macros. 10841 } // NOLINT, false-positive due to second-order macros.
10848 } // NOLINT, false-positive due to second-order macros. 10842 } // NOLINT, false-positive due to second-order macros.
10849 10843
10850 #endif // V8_OBJECTS_H_ 10844 #endif // V8_OBJECTS_H_
OLDNEW
« no previous file with comments | « src/js/proxy.js ('k') | src/objects.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698