Commit d4d10513 authored by Sorin Jianu's avatar Sorin Jianu Committed by Commit Bot

Fix unit test dependencies for //chrome/updater.

chromium-review.googlesource.com/c/chromium/src/+/2018150 introduced a unit test
dependency on chrome/common, which breaks gn check:

c:\src\chromium\src>gn check out\Default chrome/updater/*
ERROR at //chrome/updater/run_all_unittests.cc:8:11: Can't include this header from here.
#include "chrome/common/chrome_paths.h"
          ^---------------------------
The target:
  //chrome/updater:updater_tests
is including a file from the target:
  //chrome/common:constants

Bug: 1031552
Change-Id: I03c62163df096a5a6ab89acc290b61238844b299
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2036370
Commit-Queue: Sorin Jianu <sorin@chromium.org>
Reviewed-by: default avatarJoshua Pawlicki <waffles@chromium.org>
Reviewed-by: default avatarNico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#738023}
parent 2ba496e5
......@@ -93,6 +93,7 @@ if (is_win || is_mac) {
":updater",
"//base",
"//base/test:test_support",
"//chrome/common:constants",
"//testing/gtest",
]
......
include_rules = [
"+chrome/common",
"+chrome/installer/util",
"+components/crash/core/common",
"+components/crx_file",
......
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