commit 02ffb6f47a212714e68e34028d6bc23235f43f3c
author Andrew Grieve <agrieve@chromium.org> Tue Sep 22 00:49:16 2020 +0000
committer LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Sep 22 00:49:16 2020 +0000
tree 610a8eaaf7015d60ce6ba5ff5a4b92651a5b5c55
parent cc29098042868c6b833efa40c09762f8c4be326b

CheckDirMetadataFormat: LocalPath() -> AbsoluteLocalPath()

LocalPath() should be used in error messages, but never when actually
trying to access a path.

Bug: 1127507
Change-Id: I03ec5421891ac67a8620cd3c7e88c2de1811e914
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2418914
Reviewed-by: Josip Sokcevic <sokcevic@google.com>
Commit-Queue: Josip Sokcevic <sokcevic@google.com>
Auto-Submit: Andrew Grieve <agrieve@chromium.org>

presubmit_canned_checks.py

diff --git a/presubmit_canned_checks.py b/presubmit_canned_checks.py
index f4fe28b..50d3f7a 100644
--- a/presubmit_canned_checks.py
+++ b/presubmit_canned_checks.py
@@ -1051,7 +1051,7 @@
   file_filter = lambda f: (
       input_api.basename(f.LocalPath()) in ('DIR_METADATA', 'OWNERS'))
   affected_files = set([
-      f.LocalPath()
+      f.AbsoluteLocalPath()
       for f in input_api.change.AffectedFiles(
           include_deletes=False, file_filter=file_filter)
   ])