bug 373228 incorporate additional review comments after initial checkin, r=dwitte, a=release-drivers

netwerk/cookie/src/nsCookieService.cpp

@@ -2028,12 +2028,8 @@ nsCookieService::CheckPath(nsCookieAttributes &aCookieAttributes,
       }
     }
 
-  } else {
-    if (aCookieAttributes.path.Length() > kMaxBytesPerPath ||
-        aCookieAttributes.path.FindChar('\t') != kNotFound )
-      return PR_FALSE;
-
 #if 0
+  } else {
     /**
      * The following test is part of the RFC2109 spec.  Loosely speaking, it says that a site
      * cannot set a cookie for a path that it is not on.  See bug 155083.  However this patch
@@ -2049,6 +2045,10 @@ nsCookieService::CheckPath(nsCookieAttributes &aCookieAttributes,
 #endif
   }
 
+  if (aCookieAttributes.path.Length() > kMaxBytesPerPath ||
+      aCookieAttributes.path.FindChar('\t') != kNotFound )
+    return PR_FALSE;
+
   return PR_TRUE;
 }