Reviewers: rossberg,
Message:
I could add a regression test for this to the "messages" test suite, not
sure
whether it's worth it though. Let me know what you think.
Description:
Fix error message about read-only symbol properties.
R=rossb...@chromium.org
BUG=v8:3441
Please review this at https://codereview.chromium.org/390783003/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+1, -0 lines):
M src/messages.js
Index: src/messages.js
diff --git a/src/messages.js b/src/messages.js
index
4f786b590fbca91513b4944117c2b467a2deba37..5aa4111b5d617714c2bd2bc03a81b68b8174f618
100644
--- a/src/messages.js
+++ b/src/messages.js
@@ -211,6 +211,7 @@ function NoSideEffectToString(obj) {
}
return str;
}
+ if (IS_SYMBOL(obj)) return %_CallFunction(obj, SymbolToString);
if (IS_OBJECT(obj) && %GetDataProperty(obj, "toString") ===
ObjectToString) {
var constructor = %GetDataProperty(obj, "constructor");
if (typeof constructor == "function") {
--
--
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.
