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

Side by Side Diff: test/mjsunit/regress/regress-1199637.js

Issue 351853005: Split SetProperty(...attributes, strictmode) into AddProperty(...attributes) and SetProperty(...… (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 6 years, 5 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 | Annotate | Revision Log
« no previous file with comments | « test/mjsunit/object-define-property.js ('k') | test/mjsunit/regress/regress-334.js » ('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 2008 the V8 project authors. All rights reserved. 1 // Copyright 2008 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 16 matching lines...) Expand all
27 27
28 // Flags: --allow-natives-syntax --es52_globals 28 // Flags: --allow-natives-syntax --es52_globals
29 29
30 // Make sure that we can introduce global variables (using 30 // Make sure that we can introduce global variables (using
31 // both var and const) that shadow even READ_ONLY variables 31 // both var and const) that shadow even READ_ONLY variables
32 // in the prototype chain. 32 // in the prototype chain.
33 const NONE = 0; 33 const NONE = 0;
34 const READ_ONLY = 1; 34 const READ_ONLY = 1;
35 35
36 // Use DeclareGlobal... 36 // Use DeclareGlobal...
37 %SetProperty(this.__proto__, "a", 1234, NONE); 37 %AddProperty(this.__proto__, "a", 1234, NONE);
38 assertEquals(1234, a); 38 assertEquals(1234, a);
39 eval("var a = 5678;"); 39 eval("var a = 5678;");
40 assertEquals(5678, a); 40 assertEquals(5678, a);
41 41
42 %SetProperty(this.__proto__, "b", 1234, NONE); 42 %AddProperty(this.__proto__, "b", 1234, NONE);
43 assertEquals(1234, b); 43 assertEquals(1234, b);
44 eval("const b = 5678;"); 44 eval("const b = 5678;");
45 assertEquals(5678, b); 45 assertEquals(5678, b);
46 46
47 %SetProperty(this.__proto__, "c", 1234, READ_ONLY); 47 %AddProperty(this.__proto__, "c", 1234, READ_ONLY);
48 assertEquals(1234, c); 48 assertEquals(1234, c);
49 eval("var c = 5678;"); 49 eval("var c = 5678;");
50 assertEquals(5678, c); 50 assertEquals(5678, c);
51 51
52 %SetProperty(this.__proto__, "d", 1234, READ_ONLY); 52 %AddProperty(this.__proto__, "d", 1234, READ_ONLY);
53 assertEquals(1234, d); 53 assertEquals(1234, d);
54 eval("const d = 5678;"); 54 eval("const d = 5678;");
55 assertEquals(5678, d); 55 assertEquals(5678, d);
56 56
57 // Use DeclareContextSlot... 57 // Use DeclareContextSlot...
58 %SetProperty(this.__proto__, "x", 1234, NONE); 58 %AddProperty(this.__proto__, "x", 1234, NONE);
59 assertEquals(1234, x); 59 assertEquals(1234, x);
60 eval("with({}) { var x = 5678; }"); 60 eval("with({}) { var x = 5678; }");
61 assertEquals(5678, x); 61 assertEquals(5678, x);
62 62
63 %SetProperty(this.__proto__, "y", 1234, NONE); 63 %AddProperty(this.__proto__, "y", 1234, NONE);
64 assertEquals(1234, y); 64 assertEquals(1234, y);
65 eval("with({}) { const y = 5678; }"); 65 eval("with({}) { const y = 5678; }");
66 assertEquals(5678, y); 66 assertEquals(5678, y);
67 67
68 %SetProperty(this.__proto__, "z", 1234, READ_ONLY); 68 %AddProperty(this.__proto__, "z", 1234, READ_ONLY);
69 assertEquals(1234, z); 69 assertEquals(1234, z);
70 eval("with({}) { var z = 5678; }"); 70 eval("with({}) { var z = 5678; }");
71 assertEquals(5678, z); 71 assertEquals(5678, z);
72 72
73 %SetProperty(this.__proto__, "w", 1234, READ_ONLY); 73 %AddProperty(this.__proto__, "w", 1234, READ_ONLY);
74 assertEquals(1234, w); 74 assertEquals(1234, w);
75 eval("with({}) { const w = 5678; }"); 75 eval("with({}) { const w = 5678; }");
76 assertEquals(5678, w); 76 assertEquals(5678, w);
OLDNEW
« no previous file with comments | « test/mjsunit/object-define-property.js ('k') | test/mjsunit/regress/regress-334.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698