Commit 30c7f73d authored by Nicolas Ouellet-payeur's avatar Nicolas Ouellet-payeur Committed by Commit Bot

[Traffic Annotations] check_network_annotations when XML changes

Previously, if a CL only modified annotations.xml, CQ wouldn't run
check_network_annotations because the file doesn't appear in the
dependency tree for shipped_binaries.

This CL adds annotations.xml to shipped_binaries, solving that issue.

TESTED=Created a dependent CL that updates only annotations.xml and
       checked that it fails on CQ:
   https://chromium-review.googlesource.com/c/chromium/src/+/1758433

Change-Id: Ib33099df057f1cef882c46858862ea5fca5b91e6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1757297Reviewed-by: default avatarDirk Pranke <dpranke@chromium.org>
Reviewed-by: default avatarTakuto Ikuta <tikuta@chromium.org>
Reviewed-by: default avatarRamin Halavati <rhalavati@chromium.org>
Commit-Queue: Nicolas Ouellet-Payeur <nicolaso@chromium.org>
Cr-Commit-Position: refs/heads/master@{#688482}
parent bff4c077
......@@ -806,6 +806,7 @@ if ((is_linux || is_win) && enable_remoting && !use_ozone) {
deps = [
"//chrome:chrome",
"//remoting/host:host",
"//tools/traffic_annotation/summary:annotations_xml",
]
}
}
......
# Copyright (c) 2019 The Chromium Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
copy("annotations_xml") {
sources = [
"annotations.xml",
]
outputs = [
"$target_out_dir/annotations.xml",
]
}
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