Index: tools/telemetry/telemetry/value/summary.py |
diff --git a/tools/telemetry/telemetry/value/summary.py b/tools/telemetry/telemetry/value/summary.py |
index d4a318017a100335f794a6e0b86c7c43838ba102..82ba0965cd7dace81055fa19b8a4c56d613ed367 100644 |
--- a/tools/telemetry/telemetry/value/summary.py |
+++ b/tools/telemetry/telemetry/value/summary.py |
@@ -78,15 +78,15 @@ class Summary(object): |
# Now we have a bunch of values, but there is only one value_name per page. |
# Suppose page1 and page2 ran, producing values x and y. We want to print |
- # x_by_url for page1 |
- # x_by_url for page2 |
+ # x for page1 |
+ # x for page2 |
# x for page1, page2 combined |
# |
- # y_by_url for page1 |
- # y_by_url for page2 |
+ # y for page1 |
+ # y for page2 |
# y for page1, page2 combined |
# |
- # We already have the x_by_url values in the values array. But, we will need |
+ # We already have the x values in the values array. But, we will need |
# them indexable by the value name. |
# |
# The following dict maps value_name -> list of pages that have values of |
@@ -95,7 +95,7 @@ class Summary(object): |
for value in merged_page_values: |
per_page_values_by_value_name[value.name].append(value) |
- # We already have the x_by_url values in the values array. But, we also need |
+ # We already have the x values in the values array. But, we also need |
# the values merged across the pages. And, we will need them indexed by |
# value name so that we can find them when printing out value names in |
# alphabetical order. |
@@ -110,8 +110,8 @@ class Summary(object): |
value_names = set([v.name for v in merged_page_values]) |
sorted_value_names = sorted(value_names) |
- # Time to walk through the values by name, printing first the by_url values |
- # and then the merged_site value. |
+ # Time to walk through the values by name, printing first the page-specific |
+ # values and then the merged_site value. |
for value_name in sorted_value_names: |
per_page_values = per_page_values_by_value_name.get(value_name, []) |
@@ -120,7 +120,7 @@ class Summary(object): |
sorted_per_page_values.sort( |
key=lambda per_page_values: per_page_values.page.display_name) |
- # Output the _by_url results. |
+ # Output the page-specific results. |
num_successful_pages_for_this_value_name = ( |
num_successful_pages_for_value_name[value_name]) |
for per_page_value in sorted_per_page_values: |