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

Side by Side Diff: LayoutTests/fast/js/script-tests/delete-syntax.js

Issue 821303006: bindings: Fixes layouttests when moving attributes to prototype chains. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fixed performance memory tests and addressed review comments. Created 5 years, 11 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 description( 1 description(
2 "This test checks whether various forms of delete expression are allowed." 2 "This test checks whether various forms of delete expression are allowed."
3 ); 3 );
4 4
5 window.y = 0; 5 window.y = 0;
6 6
7 shouldBeTrue('delete x'); 7 shouldBeTrue('delete x');
8 window.x = 0; 8 window.x = 0;
9 window.y = 0; 9 window.y = 0;
10 shouldBeTrue('delete window.x'); 10 shouldBeTrue('delete window.x');
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 // Reverse order of delete & overwrite, tests delete after preventExtensions. 66 // Reverse order of delete & overwrite, tests delete after preventExtensions.
67 shouldBe("RegExp.prototype.compile", "regExpPrototypeCompile"); 67 shouldBe("RegExp.prototype.compile", "regExpPrototypeCompile");
68 shouldBe("RegExp.prototype.exec", "undefined"); 68 shouldBe("RegExp.prototype.exec", "undefined");
69 shouldBe("RegExp.prototype.test", "null"); 69 shouldBe("RegExp.prototype.test", "null");
70 70
71 // Check that once a property is deleted its name is removed from the property n ame array. 71 // Check that once a property is deleted its name is removed from the property n ame array.
72 delete Object.prototype.__defineSetter__; 72 delete Object.prototype.__defineSetter__;
73 shouldBe("Object.getOwnPropertyNames(Object.prototype).indexOf('__defineSetter__ ')", "-1"); 73 shouldBe("Object.getOwnPropertyNames(Object.prototype).indexOf('__defineSetter__ ')", "-1");
74 74
75 delete navigator.appCodeName; 75 delete navigator.appCodeName;
76 var navigatorPropertyNames = Object.getOwnPropertyNames(navigator); 76 delete Navigator.prototype.appCodeName;
77 var navigatorPropertyNames = getAllPropertyNames(navigator);
77 var expectedPropertyNames = [ 78 var expectedPropertyNames = [
78 "appName", 79 "appName",
79 "appVersion", 80 "appVersion",
80 "language", 81 "language",
81 "userAgent", 82 "userAgent",
82 "platform", 83 "platform",
83 "plugins", 84 "plugins",
84 "mimeTypes", 85 "mimeTypes",
85 "product", 86 "product",
86 "productSub", 87 "productSub",
87 "vendor", 88 "vendor",
88 "vendorSub", 89 "vendorSub",
89 "cookieEnabled", 90 "cookieEnabled",
90 "onLine" 91 "onLine"
91 ]; 92 ];
92 93
93 for (var i = 0; i < expectedPropertyNames.length; ++i) 94 for (var i = 0; i < expectedPropertyNames.length; ++i)
94 shouldBeTrue("navigatorPropertyNames.indexOf('" + expectedPropertyNames[i] + "') != -1"); 95 shouldBeTrue("navigatorPropertyNames.indexOf('" + expectedPropertyNames[i] + "') != -1");
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698