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

Unified Diff: LayoutTests/fast/js/script-tests/global-constructors.js

Issue 14447006: Global constructors should be configurable and not enumerable (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Clean rebase on master Created 7 years, 8 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: LayoutTests/fast/js/script-tests/global-constructors.js
diff --git a/LayoutTests/fast/js/script-tests/global-constructors.js b/LayoutTests/fast/js/script-tests/global-constructors.js
index ae72ca20b369e4b1e35970a8e21082dc61aa3494..cb105df111feed55aea13fa632821f0ab0a5bf25 100644
--- a/LayoutTests/fast/js/script-tests/global-constructors.js
+++ b/LayoutTests/fast/js/script-tests/global-constructors.js
@@ -2,7 +2,9 @@ description("This test documents our set of global constructors we expose on the
var constructorNames = [];
-for (var name in window) {
+var windowProperties = Object.getOwnPropertyNames(window);
+for (var i = 0; i < windowProperties.length; i++) {
+ var name = windowProperties[i];
var value = window[name];
var re = new RegExp("Constructor]$");
var isConstructor = re.exec(value);

Powered by Google App Engine
This is Rietveld 408576698