Remove temporary workaround TODOs from tools/tests/run.sh

tools/tests/run.sh

 #!/bin/bash
 
 # Tests for our tools.
 #
 # TODO: currently, this only passes on Linux (which is the platform that
 # the housekeeper bot runs on, e.g.
 # http://70.32.156.51:10117/builders/Skia_PerCommit_House_Keeping/builds/1415/steps/RunToolSelfTests/logs/stdio )
 # See https://code.google.com/p/skia/issues/detail?id=677
 # ('make tools/tests/run.sh work cross-platform')
 # Ideally, these tests should pass on all development platforms...
@@ skipping 130 matching lines @@
     exit 1
   fi
   COPY_EXPECTATIONS_FROM_DIR="$1"
   ARGS="$2"
   ACTUAL_OUTPUT_DIR="$3/output-actual"
   EXPECTED_OUTPUT_DIR="$3/output-expected"
 
   rm -rf $ACTUAL_OUTPUT_DIR
   mkdir -p $ACTUAL_OUTPUT_DIR
   EXPECTATIONS_TO_MODIFY_DIR="$ACTUAL_OUTPUT_DIR/gm-expectations"
-  # TODO(epoger): Temporarily exclude expectations subdirs with old base-* names,
-  # during a transition period (we need the svn rm of those subdirs to take
-  # effect)
-  BUILDERS=$(ls $COPY_EXPECTATIONS_FROM_DIR | grep -v ^base-)
+  BUILDERS=$(ls $COPY_EXPECTATIONS_FROM_DIR)
   for BUILDER in $BUILDERS; do
     mkdir -p $EXPECTATIONS_TO_MODIFY_DIR/$BUILDER
     cp $COPY_EXPECTATIONS_FROM_DIR/$BUILDER/expected-results.json \
        $EXPECTATIONS_TO_MODIFY_DIR/$BUILDER
   done
   COMMAND="python tools/rebaseline.py --expectations-root $EXPECTATIONS_TO_MODIFY_DIR $ARGS"
   echo "$COMMAND" >$ACTUAL_OUTPUT_DIR/command_line
   $COMMAND &>$ACTUAL_OUTPUT_DIR/stdout
   echo $? >$ACTUAL_OUTPUT_DIR/return_value
 
-  # TODO(epoger): Temporarily override compare_directories with a comparison
-  # that excludes expectations subdirs with old base-* names,
-  # during a transition period (we need the svn rm of those subdirs to take
-  # effect)
-  #
-  # WAS: compare_directories $EXPECTED_OUTPUT_DIR $ACTUAL_OUTPUT_DIR
-  diff --recursive --exclude=.* --exclude=base-* $EXPECTED_OUTPUT_DIR $ACTUAL_OUTPUT_DIR
-  if [ $? != 0 ]; then
-    echo "failed in: compare_directories $EXPECTED_OUTPUT_DIR $ACTUAL_OUTPUT_DIR"
-    exit 1
-  fi
+  compare_directories $EXPECTED_OUTPUT_DIR $ACTUAL_OUTPUT_DIR
 }
 
 # Run jsondiff.py with arguments in $1, recording its output.
 # Then compare that output to the content of $2/output-expected.
 function jsondiff_test {
   if [ $# != 2 ]; then
     echo "jsondiff_test requires exactly 2 parameters, got $#"
     exit 1
   fi
   ARGS="$1"
@@ skipping 74 matching lines @@
 #
 # Test jsondiff.py ...
 #
 
 JSONDIFF_INPUT=tools/tests/jsondiff/input
 JSONDIFF_OUTPUT=tools/tests/jsondiff/output
 jsondiff_test "$JSONDIFF_INPUT/old.json $JSONDIFF_INPUT/new.json" "$JSONDIFF_OUTPUT/old-vs-new"
 
 
 echo "All tests passed."