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

Unified Diff: src/mirror-debugger.js

Issue 108083005: ES6: Add Object.getOwnPropertySymbols (Closed) Base URL: http://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Do the filtering in the runtime function Created 7 years 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/mirror-debugger.js
diff --git a/src/mirror-debugger.js b/src/mirror-debugger.js
index 4277136b609265b8f64515c6842a778fb139fc56..62222052dd4f5729602e40b913e9f60d767f27a2 100644
--- a/src/mirror-debugger.js
+++ b/src/mirror-debugger.js
@@ -638,13 +638,16 @@ ObjectMirror.prototype.propertyNames = function(kind, limit) {
// Find all the named properties.
if (kind & PropertyKind.Named) {
// Get the local property names.
- propertyNames = %GetLocalPropertyNames(this.value_, true);
+ propertyNames =
+ %GetLocalPropertyNames(this.value_,
+ PROPERTY_KEY_STRING | PROPERTY_KEY_SYMBOL);
total += propertyNames.length;
// Get names for named interceptor properties if any.
if (this.hasNamedInterceptor() && (kind & PropertyKind.Named)) {
var namedInterceptorNames =
- %GetNamedInterceptorPropertyNames(this.value_);
+ %GetNamedInterceptorPropertyNames(this.value_,
+ PROPERTY_KEY_STRING | PROPERTY_KEY_SYMBOL);
if (namedInterceptorNames) {
propertyNames = propertyNames.concat(namedInterceptorNames);
total += namedInterceptorNames.length;

Powered by Google App Engine
This is Rietveld 408576698