GitLab

This reverts commit d4e5f661.

Reason for revert: Still failing to build with deps errors:

https://ci.chromium.org/p/chrome/builders/ci/win64-chrome/10175

ERROR at //chrome/browser/win/conflicts/incompatible_applications_browsertest.cc:22:11: Can't include this header from here.
#include "chrome/browser/win/conflicts/incompatible_applications_updater.h"
          ^---------------------------------------------------------------
The target:
  //chrome/browser/win/conflicts:browser_tests
is including a file from the target:
  //chrome/browser:browser

It's usually best to depend directly on the destination target.
In some cases, the destination target is considered a subcomponent
of an intermediate target. In this case, the intermediate target
should depend publicly on the destination to forward the ability
to include headers.

Dependency chain (there may also be others):
  //chrome/browser/win/conflicts:browser_tests -->
  //chrome/test:test_support_ui -->
  //chrome/browser:test_support_ui --[private]-->
  //chrome/browser:browser

Original change's description:
> Reland "Extract browser_process.h/cc into own gn target"
>
> This is a reland of 86556051
>
> The original CL missed a dep requirement in the Chrome branded
> Windows build. This has been fixed:
> https://chromium-review.googlesource.com/c/chromium/src/+/2543873/2/chrome/browser/win/conflicts/BUILD.gn
>
> Original change's description:
> > Extract browser_process.h/cc into own gn target
> >
> > This CL pulls out browser_process.h and browser_process.cc into their
> > own gn target so that other gn targets can depend on it without creating
> > cyclic dependencies with //chrome/browser:browser.
> >
> > This is required for CL:
> > https://chromium-review.googlesource.com/c/chromium/src/+/2528519
> >
> > Bug: 1147410, 883570, 925153
> > Change-Id: I17ef1f0b1683c4abb7588a6ef24ec89f15647545
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2529309
> > Commit-Queue: Alan Cutter <alancutter@chromium.org>
> > Reviewed-by: Filip Gorski <fgorski@chromium.org>
> > Reviewed-by: Glen Robertson <glenrob@chromium.org>
> > Auto-Submit: Alan Cutter <alancutter@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#828077}
>
> Bug: 1147410
> Bug: 883570
> Bug: 925153
> Change-Id: I18faaddc3850ceafb924bda6d8e197f886e79433
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2543873
> Reviewed-by: Glen Robertson <glenrob@chromium.org>
> Reviewed-by: Filip Gorski <fgorski@chromium.org>
> Commit-Queue: Alan Cutter <alancutter@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#828442}

TBR=fgorski@chromium.org,alancutter@chromium.org,glenrob@chromium.org

Change-Id: I6c5dd2e541fc98622a24435610d16de5ac9ec75e
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1147410
Bug: 883570
Bug: 925153
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2545078Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#828463}