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

Unified Diff: third_party/WebKit/LayoutTests/fast/dom/constructors-cached.html

Issue 2667393002: Stop using script-tests in fast/dom/. (Closed)
Patch Set: . Created 3 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/fast/dom/constructors-cached.html
diff --git a/third_party/WebKit/LayoutTests/fast/dom/constructors-cached.html b/third_party/WebKit/LayoutTests/fast/dom/constructors-cached.html
index c382f363d1c6613a354704a61b16f93647e9a6ff..09b0b59e3e1d4899946f767388de7ef7a10131bb 100644
--- a/third_party/WebKit/LayoutTests/fast/dom/constructors-cached.html
+++ b/third_party/WebKit/LayoutTests/fast/dom/constructors-cached.html
@@ -4,6 +4,25 @@
<script src="../../resources/js-test.js"></script>
</head>
<body>
-<script src="script-tests/constructors-cached.js"></script>
+<script>
+description("This test ensures that objects with security restrictions are cached correctly");
+
+var ctors = ["Image", "Option", "XMLHttpRequest", "Audio"];
+
+for (var i = 0; i < ctors.length; i++) {
+ var ctor = ctors[i];
+ try {
+ // Test retrieving the object twice results in the same object
+ shouldBe(ctor, ctor);
+
+ // Be paranoid -- make sure that setting a property results in that property
+ // stays
+ this[ctor].testProperty = "property set successfully";
+ shouldBe(ctor + ".testProperty", '"property set successfully"');
+ } catch (e) {
+ testFailed("Testing " + ctor + " threw " + e);
+ }
+}
+</script>
</body>
</html>

Powered by Google App Engine
This is Rietveld 408576698