DevTools: [JsDocValidator] Refactor JsDoc annotation checkers

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

Index: Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/ValidationCheck.java
diff --git a/Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/checks/ValidationCheck.java b/Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/ValidationCheck.java
similarity index 62%
rename from Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/checks/ValidationCheck.java
rename to Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/ValidationCheck.java
index 6398e32e132dce1b4509e76c8408652b87e5546b..144658b672f9d6087cfc4f9b9a32bd810e859234 100644
--- a/Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/checks/ValidationCheck.java
+++ b/Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/ValidationCheck.java
@@ -1,7 +1,6 @@
-package org.chromium.devtools.jsdoc.checks;
+package org.chromium.devtools.jsdoc;
 
-import org.chromium.devtools.jsdoc.DoDidNodeVisitor;
-import org.chromium.devtools.jsdoc.ValidatorContext;
+import com.google.javascript.rhino.head.ast.AstNode;
 
 /**
  * A base class for all JSDoc validation checks.
@@ -10,13 +9,15 @@ public abstract class ValidationCheck implements DoDidNodeVisitor {
 
     private ValidatorContext context;
 
-    public void didTraverseTree() { }
-
     protected ValidatorContext getContext() {

[eustas, 2014/01/28 08:47:21]

unused?

[apavlov, 2014/01/28 10:40:17]

Done.

         return context;
     }
 
-    public void setContext(ValidatorContext context) {
+    protected String getNodeText(AstNode node) {
+        return context.getNodeText(node);
+    }
+
+    protected void setContext(ValidatorContext context) {
         if (this.context != null) {
             throw new RuntimeException("ValidatorContext already set");
         }