Commit 913934a4 authored by Weilun Shi's avatar Weilun Shi Committed by Commit Bot

Also Pretty print the split histogram_suffixes_list

The split histogram_suffies_list.xml under histograms_xml/ should also
be pretty-printed.

Bug: 1121852
Change-Id: Icbdc0891c214be95b51a94c0a8520b2eb2e6aaa2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2376885
Auto-Submit: Weilun Shi <sweilun@chromium.org>
Commit-Queue: Alexei Svitkine <asvitkine@chromium.org>
Reviewed-by: default avatarAlexei Svitkine <asvitkine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#801759}
parent 34f1244c
......@@ -92,8 +92,10 @@ def ValidateSingleFile(input_api, output_api, file_obj, cwd, results):
# change obsolete_histograms.xml.
return False
# If the changed file is histograms.xml, pretty-print and validate prefix it.
elif 'histograms.xml' in filepath:
# If the changed file is histograms.xml or histogram_suffixes_list.xml,
# pretty-print and validate prefix it.
elif ('histograms.xml' in filepath
or 'histogram_suffixes_list.xml' in filepath):
GetPrettyPrintErrors(input_api, output_api, cwd, filepath, results)
# TODO(crbug/1120229): Re-enable validate prefix check once all histograms
# are split.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment