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

Unified Diff: Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/checks/DisallowedGlobalPropertiesChecker.java

Issue 727823002: [DevTools] Ban getSelection, requestAnimationFrame, cancelAnimationFrame global functions. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 1 month 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: Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/checks/DisallowedGlobalPropertiesChecker.java
diff --git a/Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/checks/DisallowedGlobalPropertiesChecker.java b/Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/checks/DisallowedGlobalPropertiesChecker.java
index 43938adabef539cf4a0b1af445c9906b47121eec..ad4dc770d3435859bd0e76ec3df194745d11777e 100644
--- a/Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/checks/DisallowedGlobalPropertiesChecker.java
+++ b/Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/checks/DisallowedGlobalPropertiesChecker.java
@@ -22,6 +22,9 @@ public final class DisallowedGlobalPropertiesChecker extends ContextTrackingChec
DISALLOWED_PROPERTIES.add("document");
DISALLOWED_PROPERTIES.add("addEventListener");
DISALLOWED_PROPERTIES.add("removeEventListener");
+ DISALLOWED_PROPERTIES.add("requestAnimationFrame");
+ DISALLOWED_PROPERTIES.add("cancelAnimationFrame");
+ DISALLOWED_PROPERTIES.add("getSelection");
}
private static final FunctionRecord TOP_LEVEL_FUNCTION = new FunctionRecord();

Powered by Google App Engine
This is Rietveld 408576698