GitLab

https://skia.googlesource.com/skia.git/+log/db2e7641bed8..b6b7e7b6c8de

git log db2e7641bed8..b6b7e7b6c8de --date=short --first-parent --format='%ad %ae %s'
2019-12-03 mtklein@google.com Revert "replace SkColorSpaceXformSteps::Required()"
2019-12-03 fmalita@chromium.org [skottie] Remove defensive seek(0) from Animation ctor
2019-12-03 mtklein@google.com Fix some GCC warnings.
2019-12-03 reed@google.com remove legacy next from iter
2019-12-03 kjlubick@google.com [canvaskit] Add test and extern for MakeImage

Created with:
  gclient setdep -r src/third_party/skia@b6b7e7b6c8de

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/skia-autoroll
Please CC bsalomon@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/+/master/autoroll/README.md

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Bug: None
Tbr: bsalomon@google.com
Change-Id: I37739ac1c744a36fce9939d4052e6612c999d855
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1949023Reviewed-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@{#721102}

The CL adds some tests to ensure feature is enabled when checkbox
is checked and it can maintain its state when other theme is selected.

Bug: 1022210
Change-Id: I27ebe58d6cda4164c2ee5279bc1dac97cffec599
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1940609
Commit-Queue: Lijin Shen <lazzzis@google.com>
Reviewed-by: Theresa  <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721101}

Fixed: 1028827
Change-Id: I8d3d3b3134a9f91d86289f16c70d730f0ded34b5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948084
Commit-Queue: Collin Baker <collinbaker@chromium.org>
Auto-Submit: Collin Baker <collinbaker@chromium.org>
Reviewed-by: Peter Boström <pbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721100}

Test: No longer seeing warning at login screen on unsupported device
Test: Still seeing warning at login screen on unsupported device
Bug: b/144978786
Change-Id: I1c73fbeab03354a4dd3d43bde615fc242c5c46e4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946884Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Victor Hsieh <victorhsieh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721099}

Using wpt-import in Chromium 1d5a7f10.
With Chromium commits locally applied on WPT:
500a74f5 "Add a runtime flag to enforce strict MIME type checks for workers."
a2a91a99 "Allow another touch start happen when there are active touch points"
85b61e6b "[resource-timing] Ensure workerStart is protected by TAO"
a75430aa "Fix appcache tests."
a50e7e60 "Move `Sec-Fetch-User` to SetSecFetchMetadataHeaders."


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:
foolip@chromium.org, lpz@chromium.org, robertma@chromium.org:
  external/wpt/tools
foolip@chromium.org, robertma@chromium.org, smcgruer@chromium.org:
  external/wpt/infrastructure
jsbell@chromium.org:
  external/wpt/resources

NOAUTOREVERT=true
TBR=raphael.kubo.da.costa

No-Export: true
Change-Id: Iace936c0121fa354939268551b418795c60c5f67
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948325Reviewed-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@{#721098}

https://chromium.googlesource.com/angle/angle.git/+log/ca1b0fb895ca..fc2dd45054bf

git log ca1b0fb895ca..fc2dd45054bf --date=short --first-parent --format='%ad %ae %s'
2019-12-03 geofflang@chromium.org Link to the autoroller configuration directory in the wrangling doc.
2019-12-03 thakis@chromium.org Remove an unused .gn file.
2019-12-03 flibitijibibo@gmail.com Fix UWP build
2019-12-03 geofflang@chromium.org GL: Make GL_EXT_texture_format_BGRA8888 enableable.
2019-12-03 jmadill@chromium.org Use SwiftShader-based ICD generation.
2019-12-03 angle-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/glslang/src 83af46951202..a4af7676feb0 (1 commits)
2019-12-03 angle-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/vulkan-tools/src 979b531e9fff..0a0625a3dca6 (1 commits)
2019-12-03 angle-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/spirv-tools/src 47f3eb42641b..3ed4586044f2 (1 commits)
2019-12-02 jmadill@chromium.org Add new test runner harness.
2019-12-02 syoussefi@chromium.org Vulkan: implement external semaphore barriers
2019-12-02 syoussefi@chromium.org Fix RenderingLimitToDefaultFBOSizeWithNoAttachments test
2019-12-02 syoussefi@chromium.org Add test for multithreaded shared-context resource
2019-12-02 geofflang@chromium.org D3D: Log why a renderer fails to initialize.

Created with:
  gclient setdep -r src/third_party/angle@fc2dd45054bf

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/angle-chromium-autoroll
Please CC geofflang@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/+/master/autoroll/README.md

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Bug: chromium:1026673,chromium:1029091
Tbr: geofflang@google.com
Change-Id: Ied89ac010fd38136e3d7b498491d4b5736e7326e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948797Reviewed-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@{#721097}

This reverts commit 094d246a.

Reason for revert: I believe this is causing flaky failures in BackgroundBehaviorTestInstances/WebMediaPlayerImplBackgroundBehaviorTest.AudioVideo:
https://ci.chromium.org/p/chromium/builders/ci/Linux%20Tests/84561
https://ci.chromium.org/p/chromium/builders/ci/linux-chromeos-rel/31980

Original change's description:
> Entering Picture-In-Picture re-enables disabled video track if needed.
> 
> This CL makes sure video track disabled by video background track
> optimization is re-enabled when entering Picture-in-Picture.
> 
> Bug: 949265
> Change-Id: Id0cca141f262537672f49f8cdc8bc6db5c10e3dc
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1556837
> Reviewed-by: Dan Sanders <sandersd@chromium.org>
> Commit-Queue: Mounir Lamouri <mlamouri@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#721018}

TBR=beaufort.francois@gmail.com,mlamouri@chromium.org,sandersd@chromium.org

Change-Id: I3aea0364af64ffa231dfafde3b519bdb65c4dd33
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 949265
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1949144Reviewed-by: Takumi Fujimoto <takumif@chromium.org>
Commit-Queue: Takumi Fujimoto <takumif@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721096}

Disabling because of flakiness.

TBR=sergeyu,sammc

Bug: 1030332
Change-Id: Id29f3e5d27d92bb87a003e3afac49d51ad3f3fbf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948672Reviewed-by: Takumi Fujimoto <takumif@chromium.org>
Commit-Queue: Takumi Fujimoto <takumif@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721095}

Bug: none
Test: none
Change-Id: If93f4f443906595b9303898f901cde491fc2140f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932699
Auto-Submit: James Hawkins <jhawkins@chromium.org>
Reviewed-by: Ryan Hansberry <hansberry@chromium.org>
Reviewed-by: Alexander Alekseev <alemate@chromium.org>
Reviewed-by: Roman Aleksandrov <raleksandrov@google.com>
Reviewed-by: Alex Gough <ajgo@chromium.org>
Commit-Queue: James Hawkins <jhawkins@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721094}

The background is inset visually to not necessarily fill the bounds of
the view. These insets are non-zero in touch mode and at the edges of
the screen. This splits the insets of the border into painted insets and
additional insets for positioning of elements.

Bug: None
Change-Id: Ia290aa0948e24c40f3fccb80861973f3d53981c0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948490Reviewed-by: Collin Baker <collinbaker@chromium.org>
Commit-Queue: Collin Baker <collinbaker@chromium.org>
Commit-Queue: Peter Boström <pbos@chromium.org>
Auto-Submit: Peter Boström <pbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721093}

In the Tab UI, we shouldn't go back to the pre-translating state on
confirming advanced option when the languages stayed the same. We should
stay in the after translate state.

In addition, this cl fixes a bug for the button to confirm advanced
options. In the Tab UI, both the source and target language views own
their own done buttons. They both need to be stored and updated with
'Translate' and 'Done' in UpdateAdvancedView. Previously just the most
recently created button that is stored in advanced_done_button_ was
getting updated, since advanced_done_button_ was overridden on the
second call to CreateViewAdvancedTabUi.

Bug: 976352
Change-Id: Ie16534717f5f8fdf0a608bd654b9e26db82d3a51
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1935725
Commit-Queue: Megan Jablonski <megjablon@chromium.org>
Reviewed-by: anthonyvd <anthonyvd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721092}

Uses multiple overlays to display rectangles.

BUG=none
TEST=ozone_demo --enable-overlay=2

Change-Id: Ic6cb683dbc043744597261fe15c711e09390ef46
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1922705
Commit-Queue: Fritz Koenig <frkoenig@chromium.org>
Auto-Submit: Fritz Koenig <frkoenig@chromium.org>
Reviewed-by: Daniele Castagna <dcastagna@chromium.org>
Reviewed-by: Michael Spang <spang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721091}

Bug: 925072
Change-Id: I8929e0bcdd89009320a35cb3c95d4daa349cb6ed
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946458Reviewed-by: Tommy Li <tommycli@chromium.org>
Commit-Queue: Ce Chen <cch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721090}

This cl removes stubs from PaymentApp and PaymentInstrument classes as
well as payment_handler_host.mojom

This should get landed after the dependnet Clank cl gets merged:
https://chrome-internal-review.googlesource.com/c/clank/internal/apps/+/2255260

Bug: 984694
Change-Id: I03972ab75d7c4f33b644ebf77a9ef2618ef07921
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946457Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Reviewed-by: Rouslan Solomakhin <rouslan@chromium.org>
Commit-Queue: Sahel Sharify <sahel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721089}

BUG=1028382
R=dtrainor@chromium.org,mdjones@chromium.org

Change-Id: I241391133850a23b2dd63193a4949889d45c9ddd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1938027
Commit-Queue: Pavel Yatsuk <pavely@chromium.org>
Reviewed-by: Matthew Jones <mdjones@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Reviewed-by: Pavel Yatsuk <pavely@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721088}

This is what chrome/content do to ensure the keyboard doesn't occlude any web
content.

BUG=1029720
TEST=test only change

Change-Id: Ib12f5b5fbd5a9cca6415451413ab400c7f03aeb9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948323Reviewed-by: Clark DuVall <cduvall@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721087}

Bug: 1027246
Change-Id: Ia5f47b53b502dc98ef65d4f65f9e933ab73b2330
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1928204Reviewed-by: Markus Heintz <markusheintz@chromium.org>
Commit-Queue: Sky Malice <skym@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721086}

Change-Id: I5b1132e596225950efc889494c8221ed8cce45ba
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948229
Commit-Queue: Dominique Fauteux-Chapleau <domfc@chromium.org>
Reviewed-by: Daniel Rubery <drubery@chromium.org>
Reviewed-by: Roger Tawa <rogerta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721085}

Changes the highlight of the indicator when a promo is shown, to mimic
being focused/hovered rather than clicked.

This is done according to recommendation from UX to achieve a lighter shade
of the background color (promo is showing) in contrast to a darker shade
(when the bubble opens). This change provides the user with subtle visual
feedback.

Bug: 986737
Change-Id: I6238548c74fcb0027c0f9a105fc9c769a23c052a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948929Reviewed-by: Balazs Engedy <engedy@chromium.org>
Reviewed-by: Tommy Li <tommycli@chromium.org>
Commit-Queue: Kamila Hasanbega <hkamila@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721084}

The proactive suggestions entry point times out after a fixed interval.
When the entry point is hovered over, timeout is paused.

ProactiveSuggestionsRichView is not currently pausing timeout on hover
because the embedded web contents consume the events causing mouse
enter/exit to not be received by the embedding view.

To fix this, we'll use an EventMonitor on our views native window to
make sure we get the events we need to detect hover state.

Bug: b:145056104
Change-Id: I2588fe119addfe9577963d4c29baf566e9168336
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1938448Reviewed-by: Tao Wu <wutao@chromium.org>
Commit-Queue: David Black <dmblack@google.com>
Cr-Commit-Position: refs/heads/master@{#721083}

Story tags were being mixed up into platforms.

Bug: 1029042
Change-Id: I33627111e3821c64b4f7a7970265337ae44d91cb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948834
Auto-Submit: Juan Antonio Navarro Pérez <perezju@chromium.org>
Commit-Queue: Caleb Rouleau <crouleau@chromium.org>
Reviewed-by: Caleb Rouleau <crouleau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721082}

Change-Id: I166725c025a6fa186ad011d0b542769c1d351b8a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1947421Reviewed-by: Sean Kau <skau@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721081}

This reverts commit 944d38d2.

Reason for revert: iOS build fails when RLZ is enabled. ios/chrome/browser/rlz/rlz_tracker_delegate_impl.h also needs to be updated to this new signature. crbug.com/1030223

Original change's description:
> Convert base::Bind and base::Callback in components/rlz to Once/Repeating
> 
> Bug: 1007725
> Change-Id: Ie9ce62d0775094cba94261d95853e4fe9dc19376
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1935649
> Commit-Queue: Ayu Ishii <ayui@chromium.org>
> Reviewed-by: Roger Tawa <rogerta@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#720626}

TBR=rogerta@chromium.org,ayui@chromium.org

Change-Id: Ia083d7ba2d272af221feaab1e415f786d2548e46
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1007725
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1949080Reviewed-by: Mike Dougherty <michaeldo@chromium.org>
Commit-Queue: Mike Dougherty <michaeldo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721080}

This change disables triggering of the heavy ad intervention on pages
that are the result of a user reload(referred to as reloads). This
also records in UMA whether we ignored a potential intervention due to
this logic.

This change simply disables the intervention on all reloads rather than
on reloads where the intervention occurred on the prior page. This type
of logic would require persisting information across page loads and adds
a lot of complexity for marginal gain, as only ~5% of pages are reloads.

UMA metrics, such as ComputedType are still recorded on reloads, even
though these ads are technically not eligible for intervention.

The added metrics, PageLoad.Clients.Ads.HeavysAds.IgnoredByReload is
recorded prior to querying the blocklist on a site, meaning that we may
record ignore in scenarios where the intervention may not have
triggered.

Bug: 1028248
Change-Id: I4b0abce4172c7ad295885a4ce82b1d928ae33f63
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932175Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Josh Karlin <jkarlin@chromium.org>
Commit-Queue: John Delaney <johnidel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721079}

This change makes accessible list item markers when using LayoutNG.

Before this change, Windows ATs would not read aloud the list item
markers but would read the list item content.
On windows, the AXTree looked something like this:
// <li>item 1</li>
--ListItem
----ListMarker"1."
----StaticText"item 1"
------InlineTextBox"item 1"

Now, we also read the list item markers. We expose a StaticText node
under the list marker node and made the list marker a generic container
instead of a text node.
On Windows, the AXTree now looks like this:
// <li>item 1</li>
--ListItem
----Group
------StaticText"1."
--------InlineTextBox"1."
----StaticText"item 1"
------InlineTextBox"item 1"

We intentionally don't support the Legacy layout.

Bug: 1015100
Change-Id: Ie9f4a254a55b459f59367d3a1512c12a57bfac1b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1880017
Commit-Queue: Benjamin Beaudry <benjamin.beaudry@microsoft.com>
Reviewed-by: Nektarios Paisios <nektar@chromium.org>
Reviewed-by: Adam Ettenberger <adettenb@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#721078}

This reverts commit 61668118.

Reason for revert: Still crashing flaky tests (All/ShelfLayoutManagerTest.NoShelfUpdateDuringOverviewAnimation/1)

Original change's description:
> Cros: Reland "Add metrics for transitions in the hotseat"
> 
> Report animation smoothness for the transition into the hotseat in the
> HomeLauncher.
> 
> The original change caused use-of-uninitialized-value flaky tests. This
> is a possible fix for the issue by explicitly resetting the
> |animation_metrics_reporter_|
> 
> Bug: 1022178, 1022177, 1028255
> Change-Id: I9abe9532e4f8b8cbc8c3700b21db48144f76f767
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1940562
> Reviewed-by: Manu Cornet <manucornet@chromium.org>
> Commit-Queue: Ana Salazar <anasalazar@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#720619}

TBR=manucornet@chromium.org,anasalazar@chromium.org

Change-Id: I0989a28c010ad5c28c6d6a45c49d3a924e8acdb1
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1022178, 1022177, 1028255, 1030222
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1949070Reviewed-by: Manu Cornet <manucornet@chromium.org>
Commit-Queue: Ana Salazar <anasalazar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721077}

This change adds a status field to the malware deep scanning verdicts, as
well as the logic needed to use this status field.

Fixed: 1028256
Change-Id: I1d9cab0ea002dd5c03ff7063a472e904abce66ef
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1935042Reviewed-by: Bettina Dea <bdea@chromium.org>
Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Commit-Queue: Daniel Rubery <drubery@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721076}

Add a little delay between creating <div> and sending touch events,
to avoid flakiness.

Bug: 1011225, 1030260
Change-Id: I2075ff32ef8324194d1d8e2a6d6b68e644290f9e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948492
Auto-Submit: John Chen <johnchen@chromium.org>
Reviewed-by: Lan Wei <lanwei@chromium.org>
Commit-Queue: John Chen <johnchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721075}

When a new window is opened in touch mode, or any windows switch to
touch mode, the tab strip starts open.

Fixed: 1028384
Change-Id: Ie50c4df7725ebe4841d29ba84382638418df2f2d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1941057
Commit-Queue: Collin Baker <collinbaker@chromium.org>
Commit-Queue: Peter Boström <pbos@chromium.org>
Reviewed-by: Peter Boström <pbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721074}

The test suite has 10 minutes of real work, and 4 minutes of per-shard
overhead. By using 2 shards, we keep total runtime at 9 minutes,
without wasting too many resources on overhead.

Change-Id: Idcf5acc332845547229caf2cf2005c2db6fc0d2b
Bug: 1030341
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1933384
Commit-Queue: Erik Chen <erikchen@chromium.org>
Auto-Submit: Erik Chen <erikchen@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721073}

Right now when Duet and GridTabSwitcher are both turned on, the bottom
toolbar shows at the top of the screen in landscape mode. This CL
changes this behavior so that in landscape mode, we keep showing the
original top toolbar for GridTabSwitcher instead of the Duet bottom
toolbar.

So far the change is only visible when Duet and GTS are both turned on
and TabGroups is turned off.

Bug: 1012014
Change-Id: Ia6c6ed5dddfccb33c4b51374f483e60dda99fb84
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1929352
Commit-Queue: Yue Zhang <yuezhanggg@chromium.org>
Reviewed-by: Theresa  <twellington@chromium.org>
Reviewed-by: Matthew Jones <mdjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721072}

The previous solution can only capture logs after the child process
creates a ScopedLogging object. We see flaky crashes in sandbox creation
that happen before that so this will let us see the child process stack
traces.

Also moves some functions that were exported from ipc_test_util.h to be
private to proto_chrome_prompt_ipc_unittest.cc, which is the only test
that still uses them.

Bug: 1030333
Change-Id: Ie30eed3b16631721e81a420b3b70f1ec5f0d9a0c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1948483Reviewed-by: proberge <proberge@chromium.org>
Commit-Queue: Joe Mason <joenotcharles@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721071}

Bug: 999871
Change-Id: I04f7bcff1ed4ff7c17573980301647a64b29836d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1947474
Auto-Submit: Nico Weber <thakis@chromium.org>
Reviewed-by: Robert Sesek <rsesek@chromium.org>
Reviewed-by: Max Moroz <mmoroz@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721070}

The RETURN_IF_XR_FAILED macro previously required a local variable to
already exist for it to function correctly. This change removes that
requirement.

Change-Id: Ib0ebb5855abfe8788f906235d2bec95b13eb6655
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1934394
Commit-Queue: Alexander Cooper <alcooper@chromium.org>
Reviewed-by: Alexander Cooper <alcooper@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721069}

This CL adds accessiblity strings for closing suggestion. In addition,
it updates description for other closing suggestion related strings.

Change-Id: Ie3c87ae4b91273a818b8b8950e59ca7b9c284709
Bug: 1004570
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1935045
Commit-Queue: Mei Liang <meiliang@chromium.org>
Reviewed-by: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Reviewed-by: Peter Wen <wnwen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721068}

The back gesture kSwipingFromLeftEdgeToGoBack feature is disabled by
default, and is only enabled on the devices that has hotseat feature
enabled (i.e., on Krane and on Dogfood devices).

Bug: 1030122, 1002733
Change-Id: Ie6fed45ff8df7113bc86923c23ac9e581da64937
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1947947Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Min Chen <minch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721067}

There is a bug that text selections cannot be expanded via keyboard
shortcuts (i.e. control+shift+down) with Narrator running. This is
because Narrator intercepts the keyboard shortcuts and instead performs
selections via UIA.

When GetSelection was implemented, we followed a restriction that is
visible in Edge and Inspect, where no selection was reported if it
spans multiple nodes. It turns out that this is not entirely true -
a selection will be reported that spans multiple nodes if the text
range originally spanned those nodes. This appears to be a
side-effect or bug that isn't anywhere in the UIA spec. Rather than
attempt to model this behavior more exactly, this change simply
removes the cross-node restrictions on GetSelection entirely, which
fixes Narrator's text selection issues.

Bug: 1029901
Change-Id: Ifad1ae4f4bd48122bffd44bfbd974e8de96b7493
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946983Reviewed-by: Ian Prest <iapres@microsoft.com>
Commit-Queue: Kurt Catti-Schmidt <kschmi@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#721066}

This reverts commit 9f51fb75.

Reason for revert: internal linux builds broken with "no member named 'start_hidden' in 'base::LaunchOptions'"
from https://ci.chromium.org/p/chrome/builders/ci/linux-google-rel
eg: https://ci.chromium.org/p/chrome/builders/ci/linux-google-rel/13653

Original change's description:
> Refactor the recovery component action handler for Windows.
> 
> This CL refactors the implementation of the recovery component action
> handler for Windows so that most of the code can be reused in the
> future for macOS recovery component. It also adds a unit test.
> 
> Bug: 1029953
> Change-Id: Iead0b562ae311560b3ca0eacd8d417bedd007331
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946020
> Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
> Commit-Queue: Sorin Jianu <sorin@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#721041}

TBR=sorin@chromium.org,waffles@chromium.org

Change-Id: I9cac86b0150217bc5e10915b4f54ce4a72365d08
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1029953
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1949071Reviewed-by: Ben Pastene <bpastene@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721065}

This change records the duration of successfully evaluating a prediction model.
No timing is recording if the evaluation fails or a cached result is used.

Bug: 1001194
Change-Id: I73b1f11cdfb7c9cbc4759e4de13b182c6e2e0b66
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946926Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Sophie Chang <sophiechang@chromium.org>
Commit-Queue: Michael Crouse <mcrouse@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721064}

These pipes were used by Inter-Module Comms (IMC) but this
was disabled some time ago. (see https://crbug.com/nativeclient/3864)

Bug: 1023583
Change-Id: I8c6c573d32d6a7fff2218b8a3739d66684665185
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1934867Reviewed-by: Will Harris <wfh@chromium.org>
Reviewed-by: Mark Seaborn <mseaborn@chromium.org>
Commit-Queue: Alex Gough <ajgo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721063}