Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: qunit/common.js

Issue 5485735259930624: Issue 2066 - Get rid of non-standard __defineGetter__, __defineSetter__, and __lookupGetter__ (Closed)
Patch Set: Addressed comment Created March 2, 2015, 7:27 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « lib/utils.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: qunit/common.js
===================================================================
--- a/qunit/common.js
+++ b/qunit/common.js
@@ -44,8 +44,10 @@
{
this._pbackup = Object.create(null);
for (var pref in Prefs)
- if (Prefs.__lookupSetter__(pref))
+ {
+ if (typeof Prefs[pref] != "function")
this._pbackup[pref] = Prefs[pref];
+ }
Prefs.enabled = true;
}
« no previous file with comments | « lib/utils.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld