We often see the pattern ``for region in regions: blame.AddRegion(region)``. We should just make a method for that already.
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/00ccad6ac3e1f3ab5e4a17625dddddb89d5169f4 commit 00ccad6ac3e1f3ab5e4a17625dddddb89d5169f4 Author: wrengr <wrengr@chromium.org> Date: Tue Dec 20 20:06:12 2016 Added Blame.AddReagions(iterable) method BUG= 663445 TBR=stgao@chromium.org Review-Url: https://codereview.chromium.org/2590313002 [modify] https://crrev.com/00ccad6ac3e1f3ab5e4a17625dddddb89d5169f4/appengine/findit/crash/test/changelist_classifier_test.py [modify] https://crrev.com/00ccad6ac3e1f3ab5e4a17625dddddb89d5169f4/appengine/findit/crash/test/suspect_test.py [modify] https://crrev.com/00ccad6ac3e1f3ab5e4a17625dddddb89d5169f4/appengine/findit/libs/gitiles/blame.py [modify] https://crrev.com/00ccad6ac3e1f3ab5e4a17625dddddb89d5169f4/appengine/findit/libs/gitiles/test/blame_test.py [modify] https://crrev.com/00ccad6ac3e1f3ab5e4a17625dddddb89d5169f4/appengine/findit/util_scripts/git_checkout/test/local_git_parsers_test.py [modify] https://crrev.com/00ccad6ac3e1f3ab5e4a17625dddddb89d5169f4/appengine/findit/util_scripts/git_checkout/test/local_git_repository_test.py [modify] https://crrev.com/00ccad6ac3e1f3ab5e4a17625dddddb89d5169f4/appengine/findit/waterfall/test/build_failure_analysis_test.py
Comment 1 by wrengr@chromium.org
, Dec 20 2016Status: Started (was: Available)