Move the paragraph mentioning the need to use the
`com.apple.developer.kernel.extended-virtual-addressing` entitlement to
the section that mentions other required entitlements.
Change-Id: I5de1176dbabd9137c5468f6b258064b44a116279
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5398223
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Henrique Ferreiro <hferreiro@igalia.com>
Cr-Commit-Position: refs/heads/main@{#1279333}
Bump the required version of Xcode to be in line with the
requirements in the code, change the wording of the macOS
version required (to avoid to have to update it each time
the Xcode version is updated) and mention that installing
the iOS simulator is required to build.
Bug: none
Change-Id: Ia823198e652a6731089b0bc111b7afb53324deab
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5071155
Commit-Queue: Chris Lu <thegreenfrog@chromium.org>
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Chris Lu <thegreenfrog@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1230714}
Crashpad is not yet updated for the new location, so leave a
forwarding header to be removed later.
Skipping trybots. This is a Mac change, and it’s passing all
trybots except for android_optional_gpu_tests_rel that is
repeatedly failing on an unrelated issue.
NOTRY=true
Bug: 1444927
Cq-Include-Trybots: luci.chrome.try:mac-chrome
Change-Id: Iad0c903187b0e1e5584c68f2eb00b5b026085596
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4790744
Owners-Override: Avi Drissman <avi@chromium.org>
Reviewed-by: Mark Mentovai <mark@chromium.org>
Commit-Queue: Avi Drissman <avi@chromium.org>
Auto-Submit: Avi Drissman <avi@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1185029}
The table of contents for the Chromium build instructions omits "git
status" meaning that the entry just says "Improving performance of",
which is a bit cryptic.
Also, the title isn't really accurate because the suggestions in that
section actually improve the performance of many git commands, such as
"git reset --hard" and many others.
Bug: None
Change-Id: I509cf4eefd830c5e26d9cbc2e7478ecdccea3a69
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4641872
Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Avi Drissman <avi@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1162142}
This change removes references to git versions since Chromium developers
should be using git from depot_tools, and since it isn't completely
clear whether 2.39 is greater or lower than 2.8.
This change also makes it clear that fsmonitor should be enabled on a
per-repo basis rather than globally, for the best cost/benefit ratio.
This change was inspired by the realization that enabling fsmonitor
globally on Windows caused 179 fsmonitor process to be spawned when
"gclient sync" was run. These processes commit an average of 86 MB
on Windows leading to 15 GB of commit (only a few hundred MB of
physical memory is used, but the commit usage is large enough to
matter).
Change-Id: I21c042462022088e11883bbf827276ee585fdaf3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4622567
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1159338}
Earl Grey has changed the file name of a file we widely import across the codebase. To prepare for that roll of EG2 ios/testing/earl_grey/earl_grey_test.h will be used as an umbrella header for the EG2 test support library within ios code. ios/testing/earl_grey/earl_grey_test.h will import the EG2 test's umbrella header.
Dependencies on "//ios/third_party/earl_grey2:test_lib" are replaced with "//ios/testing/earl_grey:eg_test_support+eg2" if that dependency is not already present.
Bug: 1420367
Change-Id: I1fd8fbc49d4a2aeb36fe504a23701376662f28b3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4303486
Reviewed-by: Lindsay Pasricha <lindsayw@chromium.org>
Commit-Queue: Cameron Higgins <cameronhiggins@google.com>
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1120040}
As Chromium is no longer build as universal binary, update the infra
documentation. Also mention that the environment must be set via the
target_environment (this is required because target_cpu cannot be
used to decide whether the build is for simulator or device).
Bug: 1337780
Change-Id: I38a9f20231514d0f60a0ffc55e983b76fcb265dd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4343311
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Olivier Robin <olivierrobin@chromium.org>
Reviewed-by: Olivier Robin <olivierrobin@chromium.org>
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1118018}
This change:
1. Moves all overfow-menu-carousel-related matchers into one location:
chrome_matchers.
2. Updates carousel matchers' names to use new "...Destination..."
naming convention.
3. Removes unnecessary, duplicate construction of matchers throughout
codebase, instead using the newly-refactored, localized set of matchers
in chrome_matchers.
Bug: 1339367
Change-Id: I87c2a0a6063e88e2697772c32e962772bd16881e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3722366
Reviewed-by: Mark Cogan <marq@chromium.org>
Reviewed-by: Olivier Robin <olivierrobin@chromium.org>
Reviewed-by: Robbie Gibson <rkgibson@google.com>
Commit-Queue: Benjamin Williams <bwwilliams@google.com>
Cr-Commit-Position: refs/heads/main@{#1018208}
Xcode 13.1 or higher is required to build Chrome on iOS (required
for the catalyst environment). Bump the version in the documentation
and the version check in the code.
JDK is no longer used for the build, remove it from the list of
required software.
Bug: none
Change-Id: If4ddf544b70e2c032ac3e4af95b40a02865e3b9a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3487908
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Olivier Robin <olivierrobin@chromium.org>
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Mike Pinkerton <pinkerton@chromium.org>
Commit-Queue: Mike Pinkerton <pinkerton@chromium.org>
Cr-Commit-Position: refs/heads/main@{#974670}
I suspect we actually never want this type of quote. However,
correcting them everywhere seems a lot of useless work. In the case of
code snippet however, it seems important to have it right.
Bug: 1294776
Change-Id: I71d63c902a959a3bede5aeb48bc2500a232097f6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3443425
Reviewed-by: Jérôme Lebel <jlebel@chromium.org>
Auto-Submit: Arthur Milchior <arthurmilchior@google.com>
Commit-Queue: Arthur Milchior <arthurmilchior@google.com>
Cr-Commit-Position: refs/heads/main@{#971228}
Test modules needs to have a stub version of the interface class
available to satisfy the linker. This stub is declared with the
macro GREY_STUB_CLASS_IN_APP_MAIN_QUEUE.
Xcode code completion is broken if the macro is not followed by
a semi-colon, but clang complains about trailing semi-colon in a
place where it is not required.
To bypass this issue, for each foo_interface.h file, create a
foo_interface_stub.mm file containing just the macro call. It
also avoid putting the stub declaration in a random source file.
Fix some naming inconsistency in file/target names.
Bug: 1015113
Change-Id: I6596575e8b7f8c0fac42fc8aace1568cecd69dd6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3222908
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Ewann Pellé <ewannpv@chromium.org>
Reviewed-by: Ewann Pellé <ewannpv@chromium.org>
Cr-Commit-Position: refs/heads/main@{#931916}
Building Chromium on iOS now requires access to iOS 15.0 SDK, so
update the documentation and the assert that enforces a correct
version of Xcode is used.
Bug: none
Change-Id: I5d77bf8682a19d7f3b01e7a2319646c57a7ebf3d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3193013
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/main@{#926262}
