GitLab

They add a lot of redundant load, are flaky, and this is an unshipped configuration
Additionally, turns down android-mojo-webview-rel since that's the only thing it ran.

BUG=1169248

Change-Id: I11c7b5bd6a4b8965faa79dcec0ac12f2624ebb15
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2641119
Commit-Queue: Yaron Friedman <yfriedman@chromium.org>
Reviewed-by: Haiyang Pan <hypan@google.com>
Reviewed-by: Clark DuVall <cduvall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845844}

TBR=andypaicu@chromium.org

Bug: 1167452
Change-Id: I68155a0657feda06f713b9cf02f9b3d15d1dc01d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643756
Auto-Submit: David Trainor <dtrainor@chromium.org>
Reviewed-by: Xing Liu <xingliu@chromium.org>
Commit-Queue: David Trainor <dtrainor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845843}

We currently wait for an "I'm ready" signal from the Settings app before
we serve settings search results. Unfortunately this signal is flaky,
and leads to some users never seeing settings results. This CL removes
the check altogether as a temporary measure, while we investigate a
root-cause fix.

UX impact: there's a very short window of time after logging in where
settings results won't do anything, because the Settings app isn't ready
yet. This is short enough that we don't believe it's an issue.

Bug: 1167575
Change-Id: If63770453febbf48d2fe721706daee9ccedcd608
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2631822
Commit-Queue: Tony Yeoman <tby@chromium.org>
Reviewed-by: Rachel Wong <wrong@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845842}

https://dawn.googlesource.com/dawn.git/+log/001b499275e1..b5cfe3a7746c

2021-01-21 dawn-autoroll@skia-public.iam.gserviceaccount.com Roll Tint from afb8cfb2544b to 6761160dc149 (7 revisions)
2021-01-21 enga@chromium.org Add cron-linux-clang-rel-x64 for running cron jobs on CI

Also rolling transitive DEPS:
  https://dawn.googlesource.com/tint from afb8cfb2544b to 6761160dc149

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/dawn-chromium-autoroll
Please CC sarahmashay@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Cq-Include-Trybots: luci.chromium.try:dawn-linux-x64-deps-rel;luci.chromium.try:dawn-mac-x64-deps-rel;luci.chromium.try:dawn-win10-x64-deps-rel;luci.chromium.try:dawn-win10-x86-deps-rel
Bug: None
Tbr: sarahmashay@google.com
Change-Id: I6c3d0ba6b7068696a10fc05bac1b117028f8b015
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643122Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#845841}

Using wpt-import in Chromium bbb681f0.
With Chromium commits locally applied on WPT:
820c25f5 "Prevent underinvalidation of <pattern> with template"


Note to sheriffs: This CL imports external tests and adds
expectations for those tests; if this CL is large and causes
a few new failures, please fix the failures by adding new
lines to TestExpectations rather than reverting. See:
https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md

Directory owners for changes in this CL:
jsbell@chromium.org:
  external/wpt/resources

NOAUTOREVERT=true
TBR=lpz@google.com

No-Export: true
Cq-Include-Trybots: luci.chromium.try:linux-wpt-identity-fyi-rel,linux-wpt-input-fyi-rel
Change-Id: Ia665a4b9b4e60676b8153d3418d6d7f86112566c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643636Reviewed-by: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com>
Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#845840}

Change-Id: Ia3d620f9db46a3ba201c90495c9b8328d7bd52c6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643272Reviewed-by: Scott Little <sclittle@chromium.org>
Commit-Queue: Megan Jablonski <megjablon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845839}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/02e777d04e20..03fc9a3eb3f8

2021-01-21 chanli@chromium.org [depot_tools] Update rdb pin

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/depot-tools-chromium-autoroll
Please CC ajp@google.com,apolito@google.com,ehmaldonado@google.com,sokcevic@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Bug: chromium:1103287
Tbr: ajp@google.com,apolito@google.com,ehmaldonado@google.com,sokcevic@google.com
Change-Id: Ie3231b6c745be1629506020c43172289baeeb7e9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643603Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#845838}

This installs a homescreen shortcut for a webpage. It seems at least
some of the code for doing this, i.e. AddShortcutWithSkBitmap(), is
componentizable and could run in WebLayer. This may be desirable
specifically for the case where Chrome is not the default browser
(creating a shortcut in the default browser). However for now, that
decision and work is punted.

Bug: 1164069
Change-Id: I8fbb789dc833a09358a821aba828d7a82ee9fd86
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2640935Reviewed-by: Dominick Ng <dominickn@chromium.org>
Commit-Queue: Evan Stade <estade@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845837}

This CL introduces a test helper method that refactors away common
checks used in multiple tests and moves those tests to use it.
It is in preparation of enabling subframe error page isolation.

Bug: 838161
Change-Id: Icb32d9a308dc5a8d5e4f93dbdf56e04bb1650b74
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2641085
Commit-Queue: Nasko Oskov <nasko@chromium.org>
Reviewed-by: Aaron Colwell <acolwell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845836}

This CL merges crostini_shared_usb_devices.{html,js} and plugin_vm_sh..,
as they are identical aside from one string and the VM name constant.

Bug: 1167980
Change-Id: Ie444a7d81b099d15d7ee04a65c4666231899c843
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2639715Reviewed-by: Jimmy Gong <jimmyxgong@chromium.org>
Commit-Queue: Timothy Loh <timloh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845835}

This CL is part of the Chrome OS source code directory migration:
https://docs.google.com/document/d/1g-98HpzA8XcoGBWUv1gQNr4rbnD5yfvbtYZyPDDbkaE.

Bug: 1101837, 1164001
Change-Id: I0108d52f30555e3945a324cf73038a0e39618d68
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2640353
Owners-Override: James Cook <jamescook@chromium.org>
Commit-Queue: Henrique Ferreiro <hferreiro@igalia.com>
Auto-Submit: Henrique Ferreiro <hferreiro@igalia.com>
Reviewed-by: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845834}

Replace kkhorimoto (left the team) and eugenebut (leaving the team)
with gambard and ajuma whose expertise is the closest to http
authentication.

Bug: None
Change-Id: I7044737e383a961c82fad44cf2f2c7352b26d0c7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2641132Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Reviewed-by: Ali Juma <ajuma@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845833}

Issue: Predictive completions in some IMEs are not working properly
when typing in a webpage.

Cause: Appkit <-(sync)-> RWHVCocoa <-(async)-> Blink

In an IME composition commit, RWHV commits the text to Blink
through an async mojo call. Later when Blink finishes commit, it updates
the text in RWHV through a separate mojo call.

For predictive completions to work, IMEs calls RWHV's
|attributedSubstringForProposedRange:| to retrieve some text
as the context for prediction. Oftentimes the call happens before
Blink's async update to RWHV, resulting in a wrong predictive context.

This issue troubles both Chromium and Firefox but not Safari/Webkit.
Webkit is using an undocumented async IME API.

Also after macOS 10.12+, IMEs expect |selectedRange:| to return a empty
range positioned at the end of the committed text. If not respected the
predictive completions won't work.

Solution:
- Temporarily insert the committed text to RWHV's available text
- Follow the |selectedRange:| requirement

Bug: 710101
Change-Id: I3e4267362ea3e0a6b9f7cebce23fe59d1a6ece6a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2606140Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Keren Zhu <kerenzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845832}

Previous CL split PasswordProtectionRequest into base and derived classes.
A weakptr_factory was added to the derived class, but this is not
necessary. We can use base::AsWeakPtr(&derived) to get a weak ptr to
the derived instance.

Bug: 1147967
Change-Id: I7df37dbe13448a7359707238fbfb8ca18b62ed65
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2642744Reviewed-by: Ali Juma <ajuma@chromium.org>
Reviewed-by: Bettina Dea <bdea@chromium.org>
Commit-Queue: edchin <edchin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845831}

https://swiftshader.googlesource.com/SwiftShader.git/+log/1d5174b9dc3b..5b36fbc5ed2b

2021-01-21 capn@google.com Target macOS 10.12 when building LLVM
2021-01-21 capn@google.com Use aligned allocation for classes with aligned members
2021-01-21 nicolascapens@google.com Enable and require C++17 compilation

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/swiftshader-chromium-autoroll
Please CC swiftshader-team+autoroll@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_chromium_msan_rel_ng;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:linux-swangle-try-x64;luci.chromium.try:win-swangle-try-x86
Bug: None
Tbr: swiftshader-team+autoroll@google.com
Change-Id: I986f61efb56c3c3ca18b0100ae6ca3dfee2fed7b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643117Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#845830}

Bug: 1159169
Change-Id: I9a23bed470b544d5da3938a6889b28fd2375201c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643700Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Ahmed Mehfooz <amehfooz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845829}

This reverts commit 6674e06c.

Reason for revert: Failing on android-pie-x86-rel:

PolicyTestWebXRImmersiveAR.CheckImmersiveARWorksWhenNotSet

PolicyTestWebXRImmersiveAR.CheckImmersiveARWorksWhenEnabled

Original change's description:
> WebXR: add enterprise policy for immersive-ar sessions
>
> This CL adds new enterprise policy that is used to gate access to
> immersive-ar sessions in WebXR Device API. The enterprise policy is
> per-profile, and is implemented by propagating the pref value from
> browser process to blink via WebPreferences.
>
> Additionally, the CL also introduces new AndroidBrowserTest that
> exercises the added glue code, with the assumption that the enterprise
> policy code that controls the pref is correct.
>
> Other changes:
> - add manifest entry in test jinja template in order for them to work on
>   newer Android versions
> - add DVLOGs in WebXR code paths to assist with debugging
>
> Bug: 1159067
> Change-Id: I27eba5fc0e7bae3dca2d47ca3fb2e935facbd37c
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2627778
> Commit-Queue: Piotr Bialecki <bialpio@chromium.org>
> Reviewed-by: Daniel Cheng <dcheng@chromium.org>
> Reviewed-by: Owen Min <zmin@chromium.org>
> Reviewed-by: Ted Choc <tedchoc@chromium.org>
> Reviewed-by: Alexander Cooper <alcooper@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#845521}

TBR=dcheng@chromium.org,tedchoc@chromium.org,zmin@chromium.org,bialpio@chromium.org,alcooper@chromium.org,chromium-scoped@luci-project-accounts.iam.gserviceaccount.com

Change-Id: Iac63a792c5d243d01bb3307983de25113cbe614d
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1159067
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643625Reviewed-by: David Trainor <dtrainor@chromium.org>
Commit-Queue: David Trainor <dtrainor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845828}

The CQ for infra/config changes often only run the chromium_presubmit
builder and when there are outages that affect the builders that share
machines with chromium_presubmit, it means that changes to attempt to
address the outage may experience long pending times and/or require
skipping the presubmit. Increasing the priority will allow us to keep
the safety provided by the presubmit while still being able to land
changes quickly to address outages.

Bug: 1168201
Change-Id: I828ffcf74a162a6ebdaba0075e6d66ffb31ab2c0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643407
Commit-Queue: Garrett Beaty <gbeaty@chromium.org>
Reviewed-by: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845827}

This cl changes the software backend used for
running viz_unittests and cc_unittests from
SwiftShader GL to SwANGLE (ANGLE on SwiftShader
Vulkan). Other tests suites may be affected.

From this cl, performance expectations for test
runs will likely change. For example, cc_unittests
appears to run faster, while viz_unittests and
views_unittests appear to run slower. Note that
this is still using the validating command decoder,
and once the OOP Raster tests can run on the
passthrough command decoder, we'll be able to use
it and gain some performance on all these test runs.

TBR=kbr@chromium.org
TBR=weiliangc@chromium.org
TBR=eugenis@chromium.org
Reason for TBR: These changes were already reviewed here https://chromium-review.googlesource.com/c/chromium/src/+/2532042

Bug: chromium:1060139
Change-Id: Idb0048d3a8e0dfe6855e1700ed26fb076be0c4f1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2580360
Commit-Queue: Alexis Hétu <sugoi@chromium.org>
Reviewed-by: Evgenii Stepanov <eugenis@chromium.org>
Reviewed-by: Alexis Hétu <sugoi@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845826}

The presubmit check uses the most recent version, which was resulting in
different output for cq-builders.md. It also ow warns about
crbug.com/1054172 being an unknown experiement.

Change-Id: I323a9474df1824bfbc61a892aad75899a0055945
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2642188
Auto-Submit: Garrett Beaty <gbeaty@chromium.org>
Reviewed-by: Ben Pastene <bpastene@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845825}

This reverts commit 1935a9b5.

Reason for revert: Strongly suspected cause of crbug.com/1167118

Original change's description:
> [Controls Refresh] Bug fix for Datalist uses wrong colors for HC black
> and HC white
>
> This CL fixes an issue in which, when in ForcedColors mode, the data
> list control uses an the default color scheme
>
> Bug: 1157185
> Change-Id: I5103d371fe69226be4ee495f237dd23c874f23f7
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2623150
> Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
> Commit-Queue: Sam Sebree <sasebree@microsoft.com>
> Cr-Commit-Position: refs/heads/master@{#842778}

TBR=ellyjones@chromium.org,chromium-scoped@luci-project-accounts.iam.gserviceaccount.com,sasebree@microsoft.com

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 1157185
Change-Id: I67167385980f3dca8a35e0b72f3ed6b00bc7d647
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643459Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845824}

This class updates the "Inhibited" property of the Cellular device.
When some SIM-related operations are performed, properties of the
Cellular device can change to a temporary value and then change back.
To prevent churn in these properties, Shill provides the "Inhibited"
property to inhibit any scans.

This class is intended to be used when performing such actions to ensure
that these transient states never occur. Because this operation is
expected to be performed often, we encapsulate it within this class.

Note that this CL does not add any clients for the new class; this will
occur as part of follow-up CLs.

Bug: 1093185
Change-Id: I5776c5c4a47db3645951d639b2917d8c5769c45f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2640314
Commit-Queue: Kyle Horimoto <khorimoto@chromium.org>
Reviewed-by: Azeem Arshad <azeemarshad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845823}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/af0dca3dc983..02e777d04e20

2021-01-21 vadimsh@chromium.org [lucicfg] Update v1.21.0 => v1.21.5.

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/depot-tools-chromium-autoroll
Please CC ajp@google.com,apolito@google.com,ehmaldonado@google.com,sokcevic@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Bug: None
Tbr: ajp@google.com,apolito@google.com,ehmaldonado@google.com,sokcevic@google.com
Change-Id: I206c84d499aa5e160aad385596687ec6e446ddd2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643461Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#845822}

The MessagePort conversion functions are no longer used outside of
WebEngine, now that previous clients have migrated to
use //components/cast/message_port instead.

Remove vestigial dep on now-removed 'message_port' target.

Bug: 1081525
Change-Id: Ie3629ff1542b96bfe9594cc5dcb1ef15c671c9c7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643398
Auto-Submit: Kevin Marshall <kmarshall@chromium.org>
Reviewed-by: Sharon Yang <yangsharon@chromium.org>
Commit-Queue: Sharon Yang <yangsharon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845821}

This reverts commit 3f26aff3.

Reason for revert:
This handles merged_json correctly in run_swarming_go.

Added test in https://crrev.com/c/2642838 detects previous breakage.

Original change's description:
> Revert "Reland "trigger_scripts: support go client in chromeos_device_trigger""
>
> This reverts commit 487e7412.
>
> Reason for revert:
> https://ci.chromium.org/p/chrome/builders/ci/Win%207%20Nvidia%20GPU%20Perf/17285
>
> Original change's description:
> > Reland "trigger_scripts: support go client in chromeos_device_trigger"
> >
> > This reverts commit 78d95924.
> >
> > Reason for revert:
> > This fixes previous failure and has a test
> >
> > Original change's description:
> > > Revert "trigger_scripts: support go client in chromeos_device_trigger"
> > >
> > > This reverts commit 3fd9ba25.
> > >
> > > Reason for revert:
> > > https://ci.chromium.org/p/chrome/builders/ci/android-go_webview-perf/23071
> > >
> > > Original change's description:
> > > > trigger_scripts: support go client in chromeos_device_trigger
> > > >
> > > > This scripts needs json conversion similar to
> > > > https://source.chromium.org/chromium/chromium/src/+/master:testing/trigger_scripts/base_test_triggerer.py;l=316-334;drc=e4b6e9349b783a53e3efcc45e54e6632f874ca62
> > > > when using go client.
> > > >
> > > > This is fix for breakage like
> > > > https://ci.chromium.org/p/chrome/builders/ci/chromeos-eve-chrome/12159
> > > >
> > > > Cq-Include-Trybots: luci.chromium.try:chromeos-kevin-rel
> > > > Cq-Include-Trybots: luci.chrome.try:chromeos-eve-chrome
> > > > Bug: 1127205
> > > > Change-Id: Ibf980d44b4b30855ed61e3d487bd683adf948003
> > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2635275
> > > > Reviewed-by: Ben Pastene <bpastene@chromium.org>
> > > > Reviewed-by: Wenbin Zhang <wenbinzhang@google.com>
> > > > Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
> > > > Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
> > > > Cr-Commit-Position: refs/heads/master@{#844979}
> > >
> > > TBR=tikuta@chromium.org,bpastene@chromium.org,chromium-scoped@luci-project-accounts.iam.gserviceaccount.com,wenbinzhang@google.com
> > >
> > > Change-Id: I96bf20546c714a40a53c44afb57bab5924da5d7e
> > > No-Presubmit: true
> > > No-Tree-Checks: true
> > > No-Try: true
> > > Bug: 1127205
> > > Cq-Include-Trybots: luci.chromium.try:chromeos-kevin-rel
> > > Cq-Include-Trybots: luci.chrome.try:chromeos-eve-chrome
> > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2638954
> > > Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
> > > Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
> > > Cr-Commit-Position: refs/heads/master@{#844987}
> >
> > # Not skipping CQ checks because this is a reland.
> >
> > Bug: 1127205
> > Change-Id: I6b408eed0dbf16608b6ee7084ec093ff4ab51233
> > Cq-Include-Trybots: luci.chrome.try:chromeos-eve-chrome
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2639441
> > Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
> > Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
> > Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#845499}
>
> TBR=kbr@chromium.org,tikuta@chromium.org,bsheedy@chromium.org,chromium-scoped@luci-project-accounts.iam.gserviceaccount.com,wenbinzhang@google.com
>
> Change-Id: I94525d422ea33245c3142696752cbf64f02e841a
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 1127205
> Cq-Include-Trybots: luci.chrome.try:chromeos-eve-chrome
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2639349
> Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
> Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#845511}


# Not skipping CQ checks because this is a reland.

Bug: 1127205
Change-Id: I8871ab029c1870e7edb7cf4e86350422d9beca62
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643676
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845820}

Bug: None
Change-Id: Ibe70a59070de6596e10620688c2daa5c35133e4a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2640402
Commit-Queue: Esmael Elmoslimany <aee@chromium.org>
Reviewed-by: Esmael Elmoslimany <aee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845819}

All of the testcases from the
IsPotentiallyTrustworthy.LegacyOriginUtilTests test were already covered
by the IsPotentiallyTrustworthy.Url test (which also covers some
additional testcases).

Bug: 1153336
Change-Id: I05f64324a40f81a7e8b52781b91946d6f859ee7d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2630297
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Reviewed-by: Frédéric Wang <fwang@igalia.com>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845818}

Empty size inputs for SetViewport* functions means the wp_viewport
cropping/scaling will not apply. |crop_rect_| and |display_size_px_|
should be empty and exit early.

Change-Id: I5f4ec85d89835daf5d9f7c46815a18dc4b6c279f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2641326
Commit-Queue: Kramer Ge <fangzhoug@chromium.org>
Reviewed-by: Nick Yamane <nickdiego@igalia.com>
Cr-Commit-Position: refs/heads/master@{#845817}

ANGLE performs it's own driver bug workaround flushing internally
resulting in 3 layers (command decoder, skia, ANGLE) all flushing
frequently.

This is a speculative fix to improve the crash rate seen in
bind_with_flush_on_mac when ANGLE is used on Mac.

Bug: chromium:1090584, chromium:1011460
Change-Id: I856d36ea48ad36a61603b726ea61e3e8943c8b6b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643056Reviewed-by: Jonathan Backer <backer@chromium.org>
Commit-Queue: Geoff Lang <geofflang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845816}

Tests insert a timeout on RunLoop in order to find slow or flaky tests
but the current error message only reports where the timeout is enabled
and not which RunLoop was currently running. Example:

../../content/public/test/browser_test_base.cc:701: Failure
Failed
RunLoop::Run() timed out.
Stack trace:

We will now include the Location where the RunLoop::Run() call happens
in order to report where the test is actually timing out. The new error
looks like:

../../content/test/content_browser_test_test.cc:341: Failure
Failed
RunLoop::Run() timed out. Timeout set at ../../content/public/test/browser_test_base.cc:701.
Stack trace:

Notably the file and line number where the "Failure" is reported
(in the first line) are where the RunLoop is Run().

R=gab@chromium.org

Bug: 1145280
Change-Id: Ia5e39dbbfb47fd4c9f831cb77889061c4cd3060f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2518125Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Bill Budge <bbudge@chromium.org>
Reviewed-by: Gabriel Charette <gab@chromium.org>
Commit-Queue: danakj <danakj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845815}

We were logging metrics to the wrong (non-existent) histogram.

This CL changes a constant so that we log to the correct histogram.

Bug: 1154341
Change-Id: I20bb0cc735e1f8e6d75e5f6d2c988c5188341509
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2642957Reviewed-by: Jon Mann <jonmann@chromium.org>
Commit-Queue: Claude van der Merwe <cvandermerwe@google.com>
Cr-Commit-Position: refs/heads/master@{#845814}

Before this CL, WidgetBase uses per-thread CompositorTaskRunner.

After this CL, WidgetBase will use per-ASG CompositorTaskRunner.

Bug: 1105403
Change-Id: I6809f79f69541c5a6d12a54ce39e49024bfdc96c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2626677Reviewed-by: Alexander Timin <altimin@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Commit-Queue: Minoru Chikamune <chikamune@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845813}

Navigation icons should be mirrored (resume, prev/next sentence/paragraph). Also left/right keyboard shortcuts should be reversed for sentence navigation.

Bug: 1168802
Change-Id: I03e54c5e98fb086cc86c2a8bd06455f35631a6e3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2640927
Commit-Queue: Joel Riley <joelriley@google.com>
Reviewed-by: Katie Dektar <katie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845812}

Bug: 1164416
Change-Id: Ie30d03a446c57f27af03aaccb8f199c9ce8c24cc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2623152
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Auto-Submit: Łukasz Anforowicz <lukasza@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845811}

Since r798292 NativeWidgetMacTest.WindowModalSheet has checked for
the presence of a special child window that macOS 11 splices into the
window tree to do the "blur" sheet visual effect. During one of the
later betas it appears that the system stopped adding this child
window, which broke this test.

Bug: 1169225
Change-Id: I8a31e8c6b9ab31c54c9f1a7f819510c1175ea23c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643517Reviewed-by: Leonard Grey <lgrey@chromium.org>
Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845810}

BUG=1168437
TEST=Manual

Change-Id: Ic9f719f660bea1f5d50c2ee77e0053e9186ec57b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2639800
Commit-Queue: Ahmed Fakhry <afakhry@chromium.org>
Reviewed-by: Sammie Quon <sammiequon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845809}

Bug: 1152279
Change-Id: I7512620397af04415592611083c4aba2b21e4a13
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2642372
Commit-Queue: David Bertoni <dbertoni@chromium.org>
Auto-Submit: Maksim Moskvitin <mmoskvitin@google.com>
Reviewed-by: David Bertoni <dbertoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845808}

This header uses size_t  and should include something to get it.
This happens to work in the default libc++ config, but  can
cause a build failure in builds with libstdc++.

Change-Id: I1fd3af74bc19b491ae9f4c3529a536f0b93735ad
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2640578
Auto-Submit: Tomasz Śniatowski <tsniatowski@vewd.com>
Commit-Queue: Matt Reynolds <mattreynolds@chromium.org>
Reviewed-by: Matt Reynolds <mattreynolds@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845807}

There's 2 things that were wrong. We were not initializing the
VAEncryptionParameters structure we were using, causing bad values to be
present.  We also had a regression error where we then were resubmitting
the VAEncryptionParameters during decode of full sample content. This
fixes both of those.

BUG=b:178039332
TEST=Full sample H264 protected content plays on volteer

Change-Id: I47b82a5f022292290151178cc26266c55b9dff42
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643536
Commit-Queue: Jeffrey Kardatzke <jkardatzke@google.com>
Commit-Queue: Jao-ke Chin-Lee <jchinlee@chromium.org>
Auto-Submit: Jeffrey Kardatzke <jkardatzke@google.com>
Reviewed-by: Jao-ke Chin-Lee <jchinlee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845806}

Previously we rasterized fully recorded tiles only, requiring that
blink must expand the recording rect by at least the tile size, which
was fragile.

CompositeAfterPaint calculates cull rect before painting and compositing
so we don't know the bounds of a composited layer and how it intersects
with the visible area, and we don't know which paint chunk will be
indirectly composited later, so we can't ensure we meet the recording
rect expansion requirement for every composited layer.

Allowing rasterization of partially recorded tiles will remove the
requirement, and this allows more flexibility of blink recording rect
expansion and cc tile size.

Bug: 1046544
Change-Id: Ibfe86eeecd46925deacc7a74afa2fee789e65867
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2628491Reviewed-by: vmpstr <vmpstr@chromium.org>
Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845805}