Clean comments in manifestValidator.js

Author: Lucas Rojas

lib/manifestTools/manifestValidator.js

@@ -12,31 +12,24 @@ var constants = require('../constants'),
 
 var toStringFunction = Object.prototype.toString;
 
-// load the rule(s) from a folder or file
 function loadValidationRules(fileOrDir, platforms, callback) {
 
   var stat = Q.nfbind(fs.stat); 
 
-  // list contents of the validation rules folder
   return Q.nfcall(fs.readdir, fileOrDir).then(function (files) {
     return Q.allSettled(files.map(function (file) {
       var filePath = path.join(fileOrDir, file);
       return stat(filePath).then(function (info) {
-        // test if file system object is a directory or a file
         if (info.isDirectory()) {
-          // ignore any directory that doesn't  match one of the requested platforms 
           if (platforms.indexOf(file) < 0) {
               return Q.resolve();
           }
 
-          // process the rules in the platform folder
           return loadValidationRules(filePath, []);
         }
 
-        // load the rules defined in the file
         var rulePath = path.join(fileOrDir, file);
         try {
-          // load the rule from the file
           return require(rulePath);
         }
         catch (err) {
@@ -45,7 +38,6 @@ function loadValidationRules(fileOrDir, platforms, callback) {
       });
     }))
     .then (function (results) {
-      // verify the results and consolidate the loaded rules  
       return results.reduce(function (validationRules, result) {
         if (result.state === 'fulfilled') {
           if (result.value) {
@@ -70,7 +62,6 @@ function loadValidationRules(fileOrDir, platforms, callback) {
       throw err;
     }
 
-    // fileOrDir is a file
     var rules = require(fileOrDir);
     return Array.isArray(rules) ? rules : [rules];
   })  
@@ -114,7 +105,6 @@ function applyValidationRules(w3cManifestInfo, platformModules, platforms) {
   var allResults = [];
 
   function validateAllPlatforms() {
-    // load and run validation rules for "all platforms"
     var validationRulesDir = path.join(__dirname, 'validationRules');
     return loadValidationRules(validationRulesDir).then(function (rules) {
       return runValidationRules(w3cManifestInfo, rules).then(function (results) {
@@ -124,7 +114,6 @@ function applyValidationRules(w3cManifestInfo, platformModules, platforms) {
   }
 
   function validatePlatform() {
-    // run platform-specific validation rules 
     var platformTasks = platformModules.map(function (platform) {
       return platform.getValidationRules(platforms).then(function (rules) {
         return runValidationRules(w3cManifestInfo, rules).then(function (results) {
@@ -136,7 +125,6 @@ function applyValidationRules(w3cManifestInfo, platformModules, platforms) {
     return Q.allSettled(platformTasks);
   }
 
-  // Only run the "All Platform" validations for W3C manifests
   if (w3cManifestInfo.format === constants.BASE_MANIFEST_FORMAT) {
     return validateAllPlatforms()
       .then(validatePlatform)
@@ -256,7 +244,6 @@ function validateAndNormalizeStartUrl(siteUrl, manifestInfo, callback) {
     var parsedSiteUrl = url.parse(siteUrl);
     var parsedManifestStartUrl = url.parse(manifestInfo.content.start_url);
     if (parsedManifestStartUrl.hostname && parsedSiteUrl.hostname !== parsedManifestStartUrl.hostname) {
-       // issue #88 - bis
       var subDomainOfManifestStartUrlSplitted = parsedManifestStartUrl.hostname.split('.');
       var lengthSubDomain = subDomainOfManifestStartUrlSplitted.length;
       var subDomainOfManifestStartUrl = null;