rephrase some language in code and comments

This rephrases some code and comments to avoid unclear and
non-inclusive wording. For the remaining terms in unblocked_terms.txt,
comments are added to explain why these terms are present in the code.

BUG=chromium:1099035
TEST=repo hooks

Change-Id: I985613d811836b6fb6351eadcf3e08a3290db97f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2419656
Tested-by: Bob Haarman <inglorion@chromium.org>
Reviewed-by: Manoj Gupta (OoO) <manojgupta@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>

4 files changed, 7 insertions, 6 deletions

diff --git a/crosperf/experiment_runner.py b/crosperf/experiment_runner.py
index 21fa3ea0..6a46adfc 100644
--- a/crosperf/experiment_runner.py
+++ b/crosperf/experiment_runner.py
@@ -173,7 +173,7 @@ class ExperimentRunner(object):
     try:
       # We should not lease machines if tests are launched via `skylab
       # create-test`. This is because leasing DUT in skylab will create a
-      # dummy task on the DUT and new test created will be hanging there.
+      # no-op task on the DUT and new test created will be hanging there.
       # TODO(zhizhouy): Need to check whether machine is ready or not before
       # assigning a test to it.
       if not experiment.skylab:
diff --git a/crosperf/generate_report_unittest.py b/crosperf/generate_report_unittest.py
index e19d4695..8c3510a9 100755
--- a/crosperf/generate_report_unittest.py
+++ b/crosperf/generate_report_unittest.py
@@ -109,9 +109,9 @@ class GenerateReportTests(unittest.TestCase):
     }
     results = generate_report.CutResultsInPlace(
         bench_data, max_keys=0, complain_on_update=False)
-    # Just reach into results assuming we know it otherwise outputs things
-    # sanely. If it doesn't, testCutResultsInPlace should give an indication as
-    # to what, exactly, is broken.
+    # Just reach into results assuming we know it otherwise outputs things in
+    # the expected way. If it doesn't, testCutResultsInPlace should give an
+    # indication as to what, exactly, is broken.
     self.assertEqual(list(results['foo'][0][0].items()), [('retval', 0)])
     self.assertEqual(list(results['bar'][0][0].items()), [('retval', 1)])
     self.assertEqual(list(results['baz'][0][0].items()), [])
diff --git a/crosperf/results_report.py b/crosperf/results_report.py
index ff6c4f96..ff8c119d 100644
--- a/crosperf/results_report.py
+++ b/crosperf/results_report.py
@@ -505,7 +505,7 @@ class HTMLResultsReport(ResultsReport):
     experiment_file = ''
     if self.experiment is not None:
       experiment_file = self.experiment.experiment_file
-    # Use kwargs for sanity, and so that testing is a bit easier.
+    # Use kwargs for code readability, and so that testing is a bit easier.
     return templates.GenerateHTMLPage(
         perf_table=perf_table,
         chart_js=chart_javascript,
diff --git a/crosperf/results_report_unittest.py b/crosperf/results_report_unittest.py
index e03ea431..1e96ef97 100755
--- a/crosperf/results_report_unittest.py
+++ b/crosperf/results_report_unittest.py
@@ -57,7 +57,8 @@ class FreeFunctionsTest(unittest.TestCase):
         ParseChromeosImage(os.path.dirname(buildbot_case)),
         ('', os.path.dirname(buildbot_img)))
 
-    # Ensure we don't act completely insanely given a few mildly insane paths.
+    # Ensure we do something reasonable when giving paths that don't quite
+    # match the expected pattern.
     fun_case = '/chromiumos_test_image.bin'
     self.assertEqual(ParseChromeosImage(fun_case), ('', fun_case))