aboutsummaryrefslogtreecommitdiff
path: root/catapult/common/py_utils/py_utils/discover.py
diff options
context:
space:
mode:
authorTreehugger Robot <treehugger-gerrit@google.com>2020-08-05 23:05:44 +0000
committerAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>2020-08-05 23:05:44 +0000
commit548f841b745404224698a3eafc6d4cb190d0ca8c (patch)
tree8a0d6fd78eb610653f12ded770595c80f1e89a61 /catapult/common/py_utils/py_utils/discover.py
parent1d3aa42aef5ab74bf9b13fb5a7e18c73e137f58c (diff)
parenta5e4f4d2969520f4563ec0d66cbe469c49ee38bc (diff)
downloadchromium-trace-548f841b745404224698a3eafc6d4cb190d0ca8c.tar.gz
Merge changes Ie9f01eed,I3acb8a0d am: fb769a1607 am: 316160b179 am: 3fd616f492 am: 5338649baa am: a5e4f4d296
Original change: https://android-review.googlesource.com/c/platform/external/chromium-trace/+/1392438 Change-Id: I7a428aa7cb0cdad8a449e25867e8c4bb514c2d2d
Diffstat (limited to 'catapult/common/py_utils/py_utils/discover.py')
-rw-r--r--catapult/common/py_utils/py_utils/discover.py6
1 files changed, 2 insertions, 4 deletions
diff --git a/catapult/common/py_utils/py_utils/discover.py b/catapult/common/py_utils/py_utils/discover.py
index ae8ba87d..a9333e2d 100644
--- a/catapult/common/py_utils/py_utils/discover.py
+++ b/catapult/common/py_utils/py_utils/discover.py
@@ -101,20 +101,18 @@ def DiscoverClasses(start_dir,
for module in modules:
new_classes = DiscoverClassesInModule(
module, base_class, index_by_class_name, directly_constructable)
- # TODO(nednguyen): we should remove index_by_class_name once
+ # TODO(crbug.com/548652): we should remove index_by_class_name once
# benchmark_smoke_unittest in chromium/src/tools/perf no longer relied
# naming collisions to reduce the number of smoked benchmark tests.
- # crbug.com/548652
if index_by_class_name:
AssertNoKeyConflicts(classes, new_classes)
classes = dict(list(classes.items()) + list(new_classes.items()))
return classes
-# TODO(nednguyen): we should remove index_by_class_name once
+# TODO(crbug.com/548652): we should remove index_by_class_name once
# benchmark_smoke_unittest in chromium/src/tools/perf no longer relied
# naming collisions to reduce the number of smoked benchmark tests.
-# crbug.com/548652
def DiscoverClassesInModule(module,
base_class,
index_by_class_name=False,