Benmason@ explained that a recent submission of a png file as png.sha1
file resulted in all images on that day not to be uploaded to TC.
2nd attempt after https://crrev.com/c/4374003 was reverted due to
"Evaluation of CheckStrings failed: Access outside the repository root is denied."
This is now using input_api.AffectedFiles().NewContents instead of
opening the file directly.
Bug: 1428712
Change-Id: Ia1bcb5b132f5d92ce7378a986be74b9958175680
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4432832
Reviewed-by: Ben Mason <benmason@chromium.org>
Reviewed-by: Dominic Battre <battre@chromium.org>
Commit-Queue: Ben Mason <benmason@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1142068}
Add the autosharder service account email to KNOWN_ROBOTS so the
presubmits don't think that this is an external account trying to
commit to src.
* The autosharder builder is not currently configured to run
automatically, so this CL is a no-op.
Bug: 1418199
Change-Id: I1c47728ac9548dd22690b2fc1e411f0c376b8760
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4490683
Reviewed-by: Garrett Beaty <gbeaty@google.com>
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
Auto-Submit: Stephanie Kim <kimstephanie@google.com>
Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1137217}
size.
Currently, for generateBid() and joinAdInterestGroup(), the supported
ad size units are
* "px": pixel
* "sw": screen width
This CL adds the unit:
* "sh": screen height
The parser is also updated to support parsing pure numbers as pixels.
For example, "100" is parsed as 100 pixels.
A regular expression is used to match the input, and capture the
sub-patterns for value and unit.
^\s*((?:0|(?:[1-9][0-9]*))(?:\.[0-9]+)?)(px|sw|sh)?\s*$
It basically means we match
1. Zero or more leading spaces.
2. Numbers, with optionally decimal point and digits. No leading space, cannot begin with zero, non-negative. (Sub-pattern captured)
3. Immediately followed by an unit which can be one of "px", "sw" or "sh", or the unit can be ignored entirely.
entirely. (Sub-pattern captured)
4. Zero or more trailing spaces.
Bug: http://b/239866637
See Turtledove issue: https://github.com/WICG/turtledove/issues/312
See Turtledove PR: https://github.com/WICG/turtledove/pull/417
Change-Id: I336055ff3dd635dcdfd78999d0cb5972569f5ac1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4402817
Commit-Queue: Xiaochen Zhou <xiaochenzh@chromium.org>
Reviewed-by: Garrett Tanzer <gtanzer@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1137102}
libc++ filesystem is not permitted according to the Chromium C++
styleguide. However, we are now adding a new fuzzing framework
(centipede) which depends upon it. (Centipede chose the STL filesystem
APIs because, as a standalone open source project, it can't rely on
alternatives such as Chromium base.)
Change-Id: I6434861a053dfb7019965f78238bb7019d727746
Bug: 1369919
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4455491
Commit-Queue: Grace Park <pgrace@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1136028}
crrev.com/c/4124057 added a special file size limit for the HSTS preload
list because the file had grown to more than 20MB, which was the hard
limit for files in chromium git.
crrev.com/c/4455622 removed a bunch of domains from the list, leaving
the file size around 13MB. That's still quite big, but small enough to
remove the special case for this file.
Bug: 1394954
Change-Id: I87996ae7f076b956198355209722f6804c094053
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4456904
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
Auto-Submit: Joe DeBlasio <jdeblasio@chromium.org>
Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1134005}
With absl::string_view turned into std::string_view,
abseil_string_conversions.h is now misnamed. However, those conversions
themselves are only needed because Chromium has forked a fundamental C++
vocabulary type.
As an intermediate state, until we make base::StringPiece into
std::string_view itself, add implicit conversions from/to the standard
type. This simplifies boundaries between external code, where the safe
pattern is currently tedious and the unsafe pattern (decomposing into
ptr/len) is much shorter.
However, this is imperfect. You may still need std::string_view in
places. E.g. implementing std::string_view-based interfaces, APIs that
take or return pointers, or cases when multiple implicit conversions are
chained together.
To that end, this reshuffles PRESUBMITs and guidelines:
- In Chromium, absl::string_view is now always std::string_view.
Whenever you would have said absl::string_view, say
std::string_view instead.
- However, for now, base::StringPiece is still distinct from
std::string_view and the guidelines still prefer base::StringPiece
over it. So, when you need std::string_view to deal with an
external API, use std::string_view. Otherwise, use
base::StringPiece.
(This is the same as the status quo, except absl::string_view is
replaced with std::string_view, and using the more standard spelling
when needed for 3p APIs is spelled out more explicitly. In other
cases, our non-standard spelling remains the preference.)
Existing uses of absl::string_view are left alone, but as they're the
exact same type, references can be freely and incrementally converted
to std::string_view.
Having two vocabulary types is still costly as developers must keep
track of the two, but it's better than three. The only way to truly
remove this cost is to merge base::StringPiece into std::string_view,
now that they are at parity in safety checks. This CL leaves that
decision for later, though its author is strongly of the opinion that we
should do it. :-)
As part of this, this removes the existing uses of
abseil_string_conversions.h, and cleans up some conversions and base
dependencies in the to-be-extracted net/der and net/cert/pki that are
no longer necessary.
Bug: 691162, 1370553
Change-Id: I0629029c157819ce673598caeb0f106917244f6c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4427593
Reviewed-by: Yuwei Huang <yuweih@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Code-Coverage: Findit <findit-for-me@appspot.gserviceaccount.com>
Commit-Queue: David Benjamin <davidben@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1132183}
A different profile downstream will be used for what we actually ship,
the upstream profile is there to ensure the rules are tested by CQ.
Currently disabled by gn arg until bundletool and/or playstore support
bundles with baseline profiles in non-base splits.
This reverts commit cabd06a100.
Reason for reland: Landing disabled behind a gn arg
Bug: 1378676
Change-Id: Ie14607293d17f9baba469637dfc99b3943260108
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4395063
Commit-Queue: Mohamed Heikal <mheikal@chromium.org>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Auto-Submit: Mohamed Heikal <mheikal@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1132110}
Parses the "scope_patterns" field for the tab_strip.home_tab manifest
field. This field contains a list of URLPatterns.
Design doc: go/tabbed-pwas-scope
Bug: 1381374
Change-Id: I9b5d3cbfc940b8d31382a6d8de7b4bf53cc51da7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4179490
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Louise Brett <loubrett@google.com>
Cr-Commit-Position: refs/heads/main@{#1128459}
The java changes should be a noop since we automatically do this rewrite
in compile_java.py. Also replace
//third_party/android_support_test_runner targets with
//third_party/androidx ones.
Additionally fix non-java files eg: docs, presubmits, scripts
This affects files outside //android_webview, //chrome, //components and
//weblayer which are covered by other cls.
Bug: 1428304
Change-Id: I2906b7530127983e6110e9ab5773843df0dd3ba8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4375857
Reviewed-by: Peter Wen <wnwen@chromium.org>
Owners-Override: Peter Wen <wnwen@chromium.org>
Commit-Queue: Mohamed Heikal <mheikal@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1123104}
1. The 'explanation' is expected to be a tuple. I didn't know about it,
and tried to copy the previous rule, who used a string. When a string
is used, the 'explanation' is displayed with one letter per line.
This patches adds an assertion, and fix every errors. Only one was
found.
2. One rule was using a regexp, but forgot to tag it with a slash
prefix.
3. Fix indentation.
Bug: None
Change-Id: If9fa2d9f5d552a36a8e86eef6a54aa01e8660e6c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4370679
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1122440}
While "export" is a reserved keyword, "import" and "module" are only
conditionally reserved, and so can also be used for identifiers. Try to
detect the specific patterns used with modules, so we don't warn on e.g.
setting the value of a variable named "module".
Bug: 1284275
Change-Id: I9d6ad24d9e6b5ff1efc29cb4448f3751fd806417
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4354531
Auto-Submit: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1120753}
A different profile downstream will be used for what we actually ship,
the upstream profile is there to ensure the rules are tested by CQ.
This reverts commit c406a96831.
Reason for reland: fix issue when incremental_install = true
Binary-Size: Profiles take space unfortunately.
Bug: 1378676
Change-Id: Ic4b554d1a0a422f257ae0b4e5119733d12d2b101
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4350539
Commit-Queue: Mohamed Heikal <mheikal@chromium.org>
Reviewed-by: Peter Wen <wnwen@chromium.org>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1120017}
A different profile downstream will be used for what we actually ship,
the upstream profile is there to ensure the rules are tested by CQ.
Binary-Size: Profiles take space unfortunately.
Bug: 1378676
Change-Id: I854f1ebbabdf15c3b5f943083e8481a7f8be7747
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4294696
Reviewed-by: Peter Wen <wnwen@chromium.org>
Commit-Queue: Mohamed Heikal <mheikal@chromium.org>
Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1118307}
Exclude the search_widget_extension from the systemImageNamed rule.
The helper to replace +[UIImage systemImageNamed:] is not available
for the widget.
Bug: 1424351
Change-Id: I63d1564040ef10c7a3fe4a543349d9778e1d9f96
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4334347
Auto-Submit: Gauthier Ambard <gambard@chromium.org>
Reviewed-by: Dominic Battré <battre@chromium.org>
Commit-Queue: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1117989}
We intended to ban engines and generators, but not distributions, from
<random>. I recently changed the guide and PRESUBMIT scripts to ban it
wholesale because I didn't read the email threads closely enough. Oops.
Also shortens the regex (that already existed) that was detecting
engines and generators.
Bug: none
Change-Id: Iac6b0fa136fbd980ed7a11778ef78f9496df9d03
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4337769
Auto-Submit: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1117202}
The presubmit currently prevents moving a JavaScript file to a new
location. This CL allows such CLs by using a warning instead of an error
if a script of the same name was deleted and added in the examined diff.
Change-Id: I0c5761b5f774bea8a6733e78c2ef0a31d47eab9a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4335695
Reviewed-by: Dominic Battré <battre@chromium.org>
Commit-Queue: Mike Dougherty <michaeldo@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1117004}
When opening a text file it is important to specify encoding='utf-8'
since otherwise the behavior will depend on your system locale. This
change fixes the remaining omissions in PRESUBMIT.py files, found by
running "git cl presubmit --all" with crrev.com/c/4290775 patched in.
These are the last two known errors in PRESUBMIT.py files.
Bug: 1418846
Change-Id: I39e2f08887439594b637643e4ce782b5e4782450
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4296977
Auto-Submit: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1111831}
When opening a text file it is important to specify encoding='utf-8'
since otherwise the behavior will depend on your system locale. This
change fixes various encoding omissions in PRESUBMIT.py files, found by
running "git cl presubmit --all" with crrev.com/c/4290775 patched in.
This CL was uploaded by git cl split.
R=battre@chromium.org
Bug: 1418846
Change-Id: Ia14d9c87966b6e54bdd0adf3c446979bca97f77a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4292641
Commit-Queue: Dominic Battré <battre@chromium.org>
Auto-Submit: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: Dominic Battré <battre@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1110260}
VectorDrawableCompat Could take a long time to complete, so adding
a ban to let users use the alternative class with trace events.
BUG=b/242038130
Change-Id: I174c918e7dd5fef15a551eace33d1bdfb3d203e0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4266131
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Commit-Queue: Min Qin <qinmin@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1108408}
Also deletes the related classes in base/debug/activity_analyzer.* and
base/debug/activity_tracker.*.
Many files include base/debug/activity_tracker.h and do work iff a
GlobalActivityTracker object exists. To keep this patch self-contained
it replaces the classes in activity_tracker.h with no-op stubs instead
of deleting it, so that all files that include it will still compile.
A followup will delete activity_tracker.h and all uses of it.
This should not cause any behaviour change because GlobalActivityTracker
is only created when the ExtendedCrashReporting feature is enabled, so
all uses of activity_tracker.h are already no-ops in production.
Bug: 1415328
Change-Id: I940fb0e013a3ae7a939136099f34791dec914db9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4265975
Reviewed-by: Colin Blundell <blundell@chromium.org>
Commit-Queue: Joe Mason <joenotcharles@google.com>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1107725}
ArcCertInstaller puts crypto key value into remote command, and the
command is being logged by SYSLOG.
SYSLOG is always active, regardless of logging level and applied flags.
This CL changes the logging to DLOG, so the key value is only logged in
debug build, not release build.
Bug: b:263394136
Test: autoninja -C out_hatch/Release/ chrome chrome_sandbox nacl_helper
Change-Id: I3d662bc439988622e006cfb45a171f9e51020f4d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4237413
Reviewed-by: Muhammad Hasan Khan <mhasank@chromium.org>
Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: Miriam Polzer <mpolzer@google.com>
Reviewed-by: Igor <igorcov@chromium.org>
Commit-Queue: Yao Li <yaohuali@google.com>
Cr-Commit-Position: refs/heads/main@{#1105950}
The styleguide updates are mostly for consistency. The PRESUBMIT
updates are for both consistency and to ban various
won't-be-allowed-when-we-enable-C++20 constructs (we should ban in
advance of announcing C++20 availability, since all platforms now build
in C++20 mode and it's easy for people to sneak these in).
Bug: 1284275
Change-Id: I8b90c4bbd14dd2202486688220166225eb44cbec
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4246721
Owners-Override: danakj <danakj@chromium.org>
Auto-Submit: Peter Kasting <pkasting@chromium.org>
Reviewed-by: danakj <danakj@chromium.org>
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1105710}
Migrates the easier (e.g. already had an Profile reference) usages
to the new method that takes a Profile.
Adds a PRESUBMIT to try and prevent additional getLastUsedRegularProfile()
calls.
BUG=1410601
Change-Id: I65a63d02eacb3459367ad3e6cf056d346aac8044
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4241300
Reviewed-by: Donn Denman <donnd@chromium.org>
Reviewed-by: Tomasz Wiszkowski <ender@google.com>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Commit-Queue: Ted Choc <tedchoc@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1105179}
This reverts commit 7bc47d6300.
Reason for revert: Relanding now that https://fxrev.dev/798731 and
https://fxrev.dev/801490 have rolled in, and manual test runs of video playback performance tests pass.
Original change's description:
> Revert "[fuchsia] cast_streaming_shell and web_engine_shell attempt to PresentView with fuchsia.element.GraphicalPresenter."
>
> This reverts commit 0605b82197.
>
> Reason for revert: Causing failures in encrypted video playback tests.
>
> Original change's description:
> > [fuchsia] cast_streaming_shell and web_engine_shell attempt to PresentView with fuchsia.element.GraphicalPresenter.
> >
> > As a fallback, we still connect to fuchsia.ui.policy.Presenter. This is preferred to using config-data or an
> > alternative component manifest to explicitly configure which
> > presentation protocol to use for the following reasons:
> > * The fallback is a temporary measure while the two shells are
> > transitioned to the new presenter protocol. There is no intention of
> > making this permanent, as the removal of the fallback protocol is
> > scheduled soon after the safe migration onto the preferred protocol.
> > * If there are any issues in the migration, the shells will be resilient
> > against any reverts or relands by being compatible to whichever
> > protocol is available at the time.
> >
> > Bug: 1351007
> > Test: autoninja -C out/fuchsia telemetry_gpu_integration_test_fuchsia && content/test/gpu/run_gpu_integration_test_fuchsia.py cast_streaming --browser=cast-streaming-shell --out-dir=out/fuchsia
> >
> > Change-Id: I9910dc8fb4dd8e3c64bbaafdd7cc2a9b0da047fb
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4067808
> > Reviewed-by: Wez <wez@chromium.org>
> > Commit-Queue: Caroline Liu <carolineliu@google.com>
> > Reviewed-by: Greg Thompson <grt@chromium.org>
> > Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
> > Reviewed-by: Daniel Cheng <dcheng@chromium.org>
> > Cr-Commit-Position: refs/heads/main@{#1094482}
>
> Bug: 1351007
> Change-Id: Id4f0cdf0f2f17b71567d39bfaff5d5ad809c509f
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4204527
> Reviewed-by: Alex Gough <ajgo@chromium.org>
> Reviewed-by: Greg Thompson <grt@chromium.org>
> Reviewed-by: Daniel Cheng <dcheng@chromium.org>
> Commit-Queue: Daniel Cheng <dcheng@chromium.org>
> Auto-Submit: Rohan Pavone <rohpavone@chromium.org>
> Owners-Override: Daniel Cheng <dcheng@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1098865}
Bug: 1351007
Change-Id: I1a11c0498244dc437577ca4c896c466b672cb458
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4210272
Reviewed-by: David Dorwin <ddorwin@chromium.org>
Commit-Queue: Caroline Liu <carolineliu@google.com>
Cr-Commit-Position: refs/heads/main@{#1104714}
* Stop listing std::chrono_literals as banned, since <chrono> is already banned.
* Alphabetize C++17 feature lists.
* Allow the following TBD C++17 language features:
`constexpr` lambdas
Declaring non-type template parameters with `auto`
`__has_include`
Lambda capture `this` by value (i.e. allows capturing `*this`)
`using` declaration for attributes
* Allow the following TBD C++17 library features:
3D `std::hypot`
Searchers
`std::atomic::is_always_lock_free`
`std::exclusive_scan`
`std::gcd`
`std::has_unique_object_representations`
`std::inclusive_scan`
`std::is_aggregate`
`std::is_swappable`
`std::launder`
`std::lcm`
`std::make_from_tuple`
`std::map::extract`/`std::map::merge`
Uninitialized memory algorithms
* Temporarily ban the following TBD C++17 library features. In all cases, we actually want the feature, but there's a bug open on "convert uses of our existing replacement to the C++17 version and remove it", so the ban will be reversed as part of fixing that bug:
`std::aligned_alloc`
`std::invoke`
`std::not_fn`
`std::string_view`
* Update DEPS for Abseil #includes to match current guidance.
* Update PRESUBMIT.py to match banned features above.
Bug: none
Change-Id: If53b81779df8117a84260e69979d00381ad761b8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4219878
Auto-Submit: Peter Kasting <pkasting@chromium.org>
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Commit-Queue: danakj <danakj@chromium.org>
Owners-Override: danakj <danakj@chromium.org>
Reviewed-by: danakj <danakj@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1103511}
This reverts commit 0605b82197.
Reason for revert: Causing failures in encrypted video playback tests.
Original change's description:
> [fuchsia] cast_streaming_shell and web_engine_shell attempt to PresentView with fuchsia.element.GraphicalPresenter.
>
> As a fallback, we still connect to fuchsia.ui.policy.Presenter. This is preferred to using config-data or an
> alternative component manifest to explicitly configure which
> presentation protocol to use for the following reasons:
> * The fallback is a temporary measure while the two shells are
> transitioned to the new presenter protocol. There is no intention of
> making this permanent, as the removal of the fallback protocol is
> scheduled soon after the safe migration onto the preferred protocol.
> * If there are any issues in the migration, the shells will be resilient
> against any reverts or relands by being compatible to whichever
> protocol is available at the time.
>
> Bug: 1351007
> Test: autoninja -C out/fuchsia telemetry_gpu_integration_test_fuchsia && content/test/gpu/run_gpu_integration_test_fuchsia.py cast_streaming --browser=cast-streaming-shell --out-dir=out/fuchsia
>
> Change-Id: I9910dc8fb4dd8e3c64bbaafdd7cc2a9b0da047fb
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4067808
> Reviewed-by: Wez <wez@chromium.org>
> Commit-Queue: Caroline Liu <carolineliu@google.com>
> Reviewed-by: Greg Thompson <grt@chromium.org>
> Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
> Reviewed-by: Daniel Cheng <dcheng@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1094482}
Bug: 1351007
Change-Id: Id4f0cdf0f2f17b71567d39bfaff5d5ad809c509f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4204527
Reviewed-by: Alex Gough <ajgo@chromium.org>
Reviewed-by: Greg Thompson <grt@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Daniel Cheng <dcheng@chromium.org>
Auto-Submit: Rohan Pavone <rohpavone@chromium.org>
Owners-Override: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1098865}
This CL adds to the policy logger the functionality to add the logs to
the policy logger as well as the console. This allows us to simply
call LOG_POLICY(..) in policy related files instead of keeping the
original log lines and then calling LOG_POLICY with the same message
on the next line in code related to policy.
Bug: b/266480750
Change-Id: I9acee47b6d2c8d305678b5715f4cf4766b1549b3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4190186
Reviewed-by: Yann Dago <ydago@chromium.org>
Commit-Queue: Salma Elmahallawy <esalma@google.com>
Cr-Commit-Position: refs/heads/main@{#1097958}
std::atomic is much more widely used in the codebase vs base::subtle
atomics. Add a PRESUBMIT check and documentation in base/atomicops.h to
explain it.
Bug: 1194917
Change-Id: I73989f2b7ec3eb0a1f24649c33404a2010d0fddf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4197075
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Benoit Lize <lizeb@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1097834}
This CL adds initial build system support for compiling Kotlin code
alongside our existing support for compiling Java code. In order to
compile Kotlin code, we need to use the kotlinc binary (analogous to
javac), also added to DEPS in this CL.
Although kotlinc only compiles Kotlin code into .class files, for
targets that have a mix of Java and Kotlin source files, kotlinc needs
to also be passed the Java files in the same target. This is so that it
can resolve references from the Kotlin code to the Java code in the same
target. It does not compile the Java files passed to it. That is left to
a subsequent run of javac.
Similar to Bazel, Kotlin code is compiled first, a header .jar file for
the Kotlin code is created at the same time, then the header .jar file
is merged into the header .jar file for the Java files generated by
turbine, while the .class files from kotlinc are merged into the .class
files that are subsequently generated by javac. This way a single header
.jar file and a single .jar file with all .class files is the resulting
output of compilation steps.
Another change is that .sources files now contain both Java and Kotlin
files. This resulted in some updates to JaCoCo scripts (which are not
tested in this CL as no Kotlin code has been added, so JaCoCo should not
be affected). Android Lint should already support Kotlin code, so even
when Kotlin code is added Android Lint should continue to work.
Also add print_version.sh for kotlinc.
Bug: 1401205
Change-Id: Idbeda044a01ab58e819a98364b64ddcfbfc298d0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4179272
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Auto-Submit: Peter Wen <wnwen@chromium.org>
Commit-Queue: Peter Wen <wnwen@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1095170}
As a fallback, we still connect to fuchsia.ui.policy.Presenter. This is preferred to using config-data or an
alternative component manifest to explicitly configure which
presentation protocol to use for the following reasons:
* The fallback is a temporary measure while the two shells are
transitioned to the new presenter protocol. There is no intention of
making this permanent, as the removal of the fallback protocol is
scheduled soon after the safe migration onto the preferred protocol.
* If there are any issues in the migration, the shells will be resilient
against any reverts or relands by being compatible to whichever
protocol is available at the time.
Bug: 1351007
Test: autoninja -C out/fuchsia telemetry_gpu_integration_test_fuchsia && content/test/gpu/run_gpu_integration_test_fuchsia.py cast_streaming --browser=cast-streaming-shell --out-dir=out/fuchsia
Change-Id: I9910dc8fb4dd8e3c64bbaafdd7cc2a9b0da047fb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4067808
Reviewed-by: Wez <wez@chromium.org>
Commit-Queue: Caroline Liu <carolineliu@google.com>
Reviewed-by: Greg Thompson <grt@chromium.org>
Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1094482}
RenderViewHostChanged is an API that should no longer exist and was
removed in r1089075. It might have been the root cause of a regression
in Android WebView where some apps do not properly scale their content.
This CL does a partial restore of this API only in the case of WebView
as a speculative fix for the regression. The API will be removed
immediately after this regression is understood and properly fixed.
Bug: 1168562, 1394513, 1402440
Change-Id: I1a1738ac3e0a861436d28eb81cea77d068164dd3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4149717
Reviewed-by: Nate Fischer <ntfschr@chromium.org>
Commit-Queue: Nasko Oskov <nasko@chromium.org>
Owners-Override: Nasko Oskov <nasko@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1091042}
