[DevTools] Disallow use of global property "document".

Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/checks/FunctionRecord.java

Index: Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/checks/FunctionRecord.java
diff --git a/Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/checks/FunctionRecord.java b/Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/checks/FunctionRecord.java
index a546ec5925161a65b50639f6a92b0e1b5f0d6f71..ea89f429254fa739626dabed363d0087c80b43bd 100644
--- a/Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/checks/FunctionRecord.java
+++ b/Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/checks/FunctionRecord.java
@@ -4,19 +4,23 @@ import com.google.javascript.jscomp.NodeUtil;
 import com.google.javascript.rhino.JSDocInfo;
 import com.google.javascript.rhino.Node;
 
+import java.util.Set;
+
 public class FunctionRecord {
     final Node functionNode;
     final JSDocInfo info;
     final String name;
+    final Set<String> parameterNames;
     final TypeRecord enclosingType;
     final FunctionRecord enclosingFunctionRecord;
 
     public FunctionRecord(Node functionNode, String name,
-            TypeRecord parentType,
+            Set<String> parameterNames, TypeRecord parentType,

[apavlov, 2014/10/17 11:59:41]

This should be a list not a set (and this can be reused in another check)

[dgozman, 2014/10/17 13:05:40]

Done.

             FunctionRecord enclosingFunctionRecord) {
         this.functionNode = functionNode;
         this.info = NodeUtil.getBestJSDocInfo(functionNode);
         this.name = name;
+        this.parameterNames = parameterNames;
         this.enclosingType = parentType;
         this.enclosingFunctionRecord = enclosingFunctionRecord;
     }
@@ -41,6 +45,11 @@ public class FunctionRecord {
         return info != null && info.getOriginalCommentString().contains("@suppressReceiverCheck");
     }
 
+    public boolean suppressesGlobalPropertiesCheck() {
+        return info != null
+            && info.getOriginalCommentString().contains("@suppressGlobalPropertiesCheck");
+    }
+
     @Override
     public String toString() {
         return (info == null ? "" : info.getOriginalCommentString() + "\n") +