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

Side by Side Diff: test/mjsunit/getter-in-prototype.js

Issue 6451004: Propagate exceptions thrown when setting elements. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 9 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2009 the V8 project authors. All rights reserved. 1 // Copyright 2009 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 30 matching lines...) Expand all
41 } 41 }
42 } 42 }
43 assertThrows("f()"); 43 assertThrows("f()");
44 44
45 __proto__ = p; 45 __proto__ = p;
46 function g() { 46 function g() {
47 eval('1'); 47 eval('1');
48 x = 42; 48 x = 42;
49 } 49 }
50 assertThrows("g()"); 50 assertThrows("g()");
51
52 p.__defineGetter__(0, function(){});
53
54 assertThrows("o[0] = 42");
55
56 function f2() {
57 with(o) {
58 this[0] = 42;
Lasse Reichstein 2011/02/08 14:17:56 I would think "this" here refers to the global obj
antonm 2011/02/08 14:30:21 Very good catch. That works as we set p to be a p
59 }
60 }
61 assertThrows("f2()");
62
63 __proto__ = p;
64 function g2() {
65 this[0] = 42;
66 }
67 assertThrows("g2()");
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698