The display ad pipeline on desktop was already disabled. This PR removes the
remaining frontend code and portions of the mojom code that no longer had any
effect. The remaining mojom code is still required for Android builds and will be
removed in a follow-up when the dead code for Brave News inline content ads
on Android is cleaned up.
This change moves the Brave Rewards code in "content" and "engine"
behind the ENABLE_BRAVE_REWARDS buildflag. In order to simplify
dependencies, it also moves the buildflags defined in "engine" to
"core/buildflags".
[CodeHealth][cr146] `base::Contains` dissallowed when `.contains` suffices it
This is being enforced by the compiler through a concept, and it only
applies for cases where `base::Contains` is used without a projection.
This change also carries out IWYU for `base::Contains`.
Chromium changes:
https://chromium.googlesource.com/chromium/src/+/4a17af8403373b7f7e7546ef61f30dfbe883d617
```
commit 4a17af8403373b7f7e7546ef61f30dfbe883d617
Author: Victor Hugo Vianna Silva <victorvianna@google.com>
Date: Tue Jan 13 07:30:34 2026 -0800
Disallow base::Contains() when contains() or find() methods available
This is a temporary step towards deleting base::Contains().
Trying to call base::Contains(foo, bar) will now cause a static
assertion failure if foo has contains() or find() methods. This
effectively makes base::Contains() an alias to std::ranges::contains().
The result is we can replace instances of 'base::Contains' with
'std::ranges::contains' in upcoming CLs while being sure to not
regress performance.
Bug: 470391351
Change-Id: I8778f435ff8f5b52fb3bf336724f2dfef388907a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7415207
Commit-Queue: Francois Pierre Doray <fdoray@chromium.org>
Reviewed-by: Francois Pierre Doray <fdoray@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1568392}
```
Bug: https://github.com/brave/brave-browser/issues/42557
This PR is the run of `gn format` on all `gn` files. This is a
mechanical change done with:
```sh
git ls-files -- "*.gn" | xargs gn format
git ls-files -- "*.gni" | xargs gn format
```
This change has been motivated primarily by an improvement to the
formatters privided by `gn` correcting cases of redundant target naming,
i.e cases where `//foo:foo` is used, and should just be `//foo`.
For this particular `gn` change, see:
https://chromium.googlesource.com/chromium/src/+/c822490a82cdb6ad479159683a92858f7c6f0a58
Resolves https://github.com/brave/brave-browser/issues/48161
This PR does IWYU for the following files:
- base/strings/utf_string_conversions.h
- base/i18n/number_formatting.h
This is a mechanical change, and for more details, check the issue
description.
Resolves https://github.com/brave/brave-browser/issues/47060
Aleksei Seren