Javadoc etc for @jtnord

core/src/main/java/hudson/security/csrf/CrumbFilter.java

@@ -98,7 +98,7 @@ public class CrumbFilter implements Filter {
             return false;
         }
 
-        return MultipartFormDataParser.isMultipart(request.getContentType());
+        return MultipartFormDataParser.isMultiPartForm(request.getContentType());
     }
 
     /**

core/src/main/java/hudson/util/MultipartFormDataParser.java

@@ -28,6 +28,7 @@ import org.apache.commons.fileupload.disk.DiskFileItemFactory;
 import org.apache.commons.fileupload.FileItem;
 import org.apache.commons.fileupload.FileUploadException;
 
+import javax.annotation.CheckForNull;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.ServletException;
 import java.util.List;
@@ -72,7 +73,15 @@ public class MultipartFormDataParser {
             item.delete();
     }
 
-    public static boolean isMultipart(String contentType) {
+    /**
+     * Checks a <a href="http://www.w3.org/Protocols/rfc1341/4_Content-Type.html">Content-Type</a> string
+     * to assert if it is "multipart/form-data".
+     *
+     * @param contentType <a href="http://www.w3.org/Protocols/rfc1341/4_Content-Type.html">Content-Type</a> string.
+     * @return {@code true} if the content type is "multipart/form-data", otherwise {@code false}.
+     * @since TODO
+     */
+    public static boolean isMultiPartForm(@CheckForNull String contentType) {
         if (contentType == null) {
             return false;
         }

core/src/test/java/hudson/util/MultipartFormDataParserTest.java

@@ -33,11 +33,11 @@ public class MultipartFormDataParserTest {
     
     @Test
     public void testIsMultipart() throws Exception {
-        Assert.assertEquals(false, MultipartFormDataParser.isMultipart(null));
-        Assert.assertEquals(false, MultipartFormDataParser.isMultipart("blah"));
+        Assert.assertEquals(false, MultipartFormDataParser.isMultiPartForm(null));
+        Assert.assertEquals(false, MultipartFormDataParser.isMultiPartForm("blah"));
 
-        Assert.assertEquals(true, MultipartFormDataParser.isMultipart("multipart/form-data"));
-        Assert.assertEquals(true, MultipartFormDataParser.isMultipart("multipart/form-data;"));
-        Assert.assertEquals(true, MultipartFormDataParser.isMultipart("multipart/form-data; boundary=----WebKitFormBoundary8OOwv1Xp4c5XkBmD"));
+        Assert.assertEquals(true, MultipartFormDataParser.isMultiPartForm("multipart/form-data"));
+        Assert.assertEquals(true, MultipartFormDataParser.isMultiPartForm("multipart/form-data;"));
+        Assert.assertEquals(true, MultipartFormDataParser.isMultiPartForm("multipart/form-data; boundary=----WebKitFormBoundary8OOwv1Xp4c5XkBmD"));
     }
 }