GitLab

Now to run the WebLayer shell on Android all you have to do is:

  $ autoninja -C out/foo run_weblayer_shell
  $ out/foo/bin/run_weblayer_shell

Change-Id: Icc6e1d771a6e83387c4d7fec989ddbb043f4575c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787855
Commit-Queue: Clark DuVall <cduvall@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693881}

The action button is added to error notification if
it is due to internet connectivity issue. Try again does
what the name says. Manually tested for some conditions, works well :)
Also removed translation files for non-existing texts.

Bug: 996322
Change-Id: I64b15fdeaaaf95edf1a081d7f852e3f4ab167f9e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787549Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: Michael van Ouwerkerk <mvanouwerkerk@chromium.org>
Commit-Queue: Himanshu Jaju <himanshujaju@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693880}

https://android.googlesource.com/platform/external/perfetto.git/+log/1ed60ee797b6..6d991e313322

git log 1ed60ee797b6..6d991e313322 --date=short --no-merges --format='%ad %ae %s'
2019-09-05 treehugger-gerrit@google.com Merge "Fix presubmit warning about js files"
2019-09-05 treehugger-gerrit@google.com Merge "proto: Minor changes to TrackEvent tracks"
2019-09-05 treehugger-gerrit@google.com Merge "ui: Show grab pointer when mouse over track shell"
2019-09-05 primiano@google.com Merge "tools(tmux): support target devices"
2019-09-05 fmayer@google.com Merge "Better docs for ProfilePacket."
2019-09-05 taylori@google.com Merge "perfetto-ui: Ensure userspace 'busy' slices are accurate"

Created with:
  gclient setdep -r src/third_party/perfetto@6d991e313322

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

TBR=perfetto-bugs@google.com

Bug: None
Change-Id: Ifeae5725b000389f7e8ba690936a7be3f8a6a215
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787851Reviewed-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@{#693879}

The only thing in resource_coordinator_features is a flag for the
performance_manager Windows WorkingSetTrimmer. That feature flag
should really live in performance_manager features with the other
similar flags. This CL moves that flag and then cleans up the files
that are no longer needed after that move.

BUG=chromium:988602

Change-Id: I0531ead60d5d1117046607a38d8165936f008ef0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1783299Reviewed-by: François Doray <fdoray@chromium.org>
Reviewed-by: Alexei Filippov <alph@chromium.org>
Reviewed-by: Mike West <mkwst@chromium.org>
Reviewed-by: Sigurður Ásgeirsson <siggi@chromium.org>
Commit-Queue: Brian Geffon <bgeffon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693878}

Default argument values are banned on virtual functions.
https://google.github.io/styleguide/cppguide.html#Default_Arguments

Bug: none
Change-Id: Ie3786e31165317dc2c3a156fc9437949f2eb80f0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1785740
Commit-Queue: Evan Stade <estade@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693877}

These tests were previously migrated from single-threaded MessageLoop to
a multi-threaded TaskEnvironment (then named ScopedTaskEnvironment) as
part of crbug.com/891670.

//base OWNERS decided in retrospect that it was better to keep a
single-threaded option for TaskEnvironment and introduced
SingleThreadTaskEnvironment. This CL retrofits that decision for
/components/domain_reliability.

This CL is a no-op if it passes CQ.

This CL was uploaded by git cl split.

R=eroman@chromium.org

Bug: 891670
Change-Id: Ia403dd784ffbbee2f9ca96e39bd58af6072cbb52
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786856
Auto-Submit: Gabriel Charette <gab@chromium.org>
Reviewed-by: Eric Roman <eroman@chromium.org>
Commit-Queue: Eric Roman <eroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693876}

- Adds a reference in the streaming client to the cached module data,
  to keep it alive as long as streaming is ongoing.

Bug: chromium:1001027
Change-Id: I8eb18b1adda2cbb47e76673e160fc0d6e51fdab1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787462Reviewed-by: Andreas Haas <ahaas@chromium.org>
Commit-Queue: Bill Budge <bbudge@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693875}

The effects of animating properties affected by forced colors mode
should be overridden in forced colors mode if forced-color-adjust is
set to auto.

Bug: 970285
Change-Id: I5d0c86319204848ca7eea54bc5d68b4396700d34
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1783338Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Reviewed-by: Kevin Babbitt <kbabbitt@microsoft.com>
Commit-Queue: Alison Maher <almaher@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#693874}

These tests were previously migrated from single-threaded MessageLoop to
a multi-threaded TaskEnvironment (then named ScopedTaskEnvironment) as
part of crbug.com/891670.

//base OWNERS decided in retrospect that it was better to keep a
single-threaded option for TaskEnvironment and introduced
SingleThreadTaskEnvironment. This CL retrofits that decision for
/chrome/browser/chromeos.

This CL is a no-op if it passes CQ.

This CL was uploaded by git cl split.

R=jamescook@chromium.org

Bug: 891670
Change-Id: I32b1cc16d0b36bdb79c3c7668aca9c90cbf0a4ed
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787140
Auto-Submit: Gabriel Charette <gab@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Commit-Queue: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693873}

Bug: 1000660
Change-Id: I11175c2e13f9751bacf095011d1a4c792c284afa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1784723Reviewed-by: Yi Su <mrsuyi@chromium.org>
Commit-Queue: Javier Ernesto Flores Robles <javierrobles@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693872}

Bug: 1000932
Change-Id: Ia6ae52ff439c1d7e8d759e6f083fb04d08663b1f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786616
Commit-Queue: Staphany Park <staphany@chromium.org>
Auto-Submit: Staphany Park <staphany@chromium.org>
Reviewed-by: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693871}

https://chromium.googlesource.com/chromiumos/chromite.git/+log/df4a44889317..d3de8dfbbb59

git log df4a44889317..d3de8dfbbb59 --date=short --no-merges --format='%ad %ae %s'
2019-09-05 engeg@google.com Add a helper function to build_api to list services
2019-09-05 andrewlamb@chromium.org Disable hardware tests for all betty release builders.

Created with:
  gclient setdep -r src/third_party/chromite@d3de8dfbbb59

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/chromite-chromium-autoroll
Please CC chrome-os-gardeners@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:chromeos-kevin-rel
TBR=chrome-os-gardeners@google.com

Bug: None
Change-Id: I50c96e2bcce145a76ef8d50006ee958bb133d445
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786029Reviewed-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@{#693870}

This test is flaky only on Mac, just like CrElementsInputTest. Re-enable
it on other platforms and add a comment referencing the bug for
CrElementsInputTest. The flakes are likely due to the same cause, since
these tests are identical other than the version of Polymer.

Bug: 997944
Change-Id: Ife5e77c066496f702a3661457aa68d0f2af808c8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1785624Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693869}

Bug: 990885
Change-Id: I0f1d195cf150f003d535c93dc2dccd6057c2cf50
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1788097
Commit-Queue: John Budorick <jbudorick@chromium.org>
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Auto-Submit: John Budorick <jbudorick@chromium.org>
Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693868}

When testing, I had disabled the whitelist checks. But this test
doesn't actually do anything if the whitelist matches, so don't
match the whitelist.

Change-Id: I1e28b4deea59206dac3a2f99ccbc8aba8944fd99
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787160
Auto-Submit: Daniel Rubery <drubery@chromium.org>
Reviewed-by: Varun Khaneja <vakh@chromium.org>
Commit-Queue: Daniel Rubery <drubery@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693867}

This CL makes sure that the toggle is disabled and displayed as off
when the feature cannot be used and replaces the old strings.

Bug:986322

Change-Id: Idccdfb16755095a46ff5e05db5e3d2daaa7e6285
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787544
Commit-Queue: Ioana Pandele <ioanap@chromium.org>
Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org>
Reviewed-by: Friedrich [CET] <fhorschig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693866}

Bug: None
Change-Id: Ibd16148a8b641131b8d5c22e9a58f77453f09f69
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1784622Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Jamie Madill <jmadill@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693865}

This is expected to be used by Tast test cases like
https://chromium-review.googlesource.com/c/chromiumos/platform/tast-tests/+/1779242

Bug: 999365
Test: the new Tast test
Change-Id: I4ba163dcb585a5785cea3b0ba1286337371b9156
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1779147Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Xiaoqian Dai <xdai@chromium.org>
Commit-Queue: Jun Mukai <mukai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693864}

This CL does the following:
1. Updates infobar java code to assign a priority number to each
infobar, and high priority infobar is shown in front of low priority
infobar. If infobars have the same priorities, the most recently
added one is shown behind previous ones.
2. Update the SMS infobar to have USER_TRIGGERED priority.

Bug: 999222
Change-Id: I8f2b3ccb45dd8a6ec97ed4f0b246ac69ae60c7c2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1783720
Commit-Queue: Jun Cai <juncai@chromium.org>
Reviewed-by: Ayu Ishii <ayui@chromium.org>
Reviewed-by: Matthew Jones <mdjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693863}

Bug: 1000932
Change-Id: I0b745dc4a35b3998dfd036f6900dccf5edead399
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786973
Commit-Queue: Staphany Park <staphany@chromium.org>
Auto-Submit: Staphany Park <staphany@chromium.org>
Reviewed-by: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693862}

https://chromium.googlesource.com/angle/angle.git/+log/d0e56cc64289..66e0feec40dc

git log d0e56cc64289..66e0feec40dc --date=short --no-merges --format='%ad %ae %s'
2019-09-05 jmadill@chromium.org Remove default template args in ResourceMap.

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

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 jmadill@chromium.org,jmadill@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
TBR=jmadill@chromium.org,jmadill@google.com

Bug: None
Change-Id: I2f5e5b37a9eaba61157c8965157592d3fd4a66d8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787850Reviewed-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@{#693861}

Also, enable the flag for all channels except stable.

Bug: b:130798103
Test: android-sh -c getprop | grep ro.boot.arc_print_spooler
Change-Id: I4ca45c248737bbc0e5c90ed679bfe2b77ec9ed12
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1784006Reviewed-by: Elijah Taylor <elijahtaylor@chromium.org>
Commit-Queue: Jesse Schettler <jschettler@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693860}

Added following histograms recording for USS Nigori (same as we have
for Directory implementation):
Sync.PassphraseType
Sync.CryptographerReady
Sync.CryptographerPendingKeys
Sync.KeystoreDecryptionFailed
Sync.CustomEncryption

There are some remaining histograms, which we should add later:

Together with full SCRYPT support:
Sync.Crypto.CustomPassphraseKeyDerivationMethodOnNewPassphrase
Sync.Crypto.CustomPassphraseKeyDerivationMethodOnSuccessfulDecryption
Sync.Crypto.CustomPassphraseKeyDerivationMethodStateOnStartup

Together with ping-pong prevention:
Sync.AutoNigoriOverwrites

Together with conflict resolution:
Sync.ResolveSimpleConflict

Bug: 922900
Change-Id: Ia5d6886ea7dd2855cf2240ffd14780c6158b7550
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787600Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Commit-Queue: Maksim Moskvitin <mmoskvitin@google.com>
Cr-Commit-Position: refs/heads/master@{#693859}

Currently there are two methods, OnProfileAdded (to replace
NOTIFICATION_PROFILE_ADDED and NOTIFICATION_PROFILE_CREATED) and
OnProfileMarkedForPermanentDeletion (to replace
NOTIFICATION_PROFILE_DESTRUCTION_STARTED).

Use the new interface in NoteTakingHelper and ExtensionService.

Another option would be to have OnProfileMarkedForPermanentDeletion
be a part of a ProfileObserver interface, but it seems best to
minimize the number of observer interfaces.

Bug: 268984
Change-Id: I96ba72e558ee3da30729207adcd8e22e0755293a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1776682
Commit-Queue: Evan Stade <estade@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693858}

Also some slight refactoring.

Bug: 997768
Change-Id: I25c6f2815c12b6de5c737fd9f6831149a92b7675
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1783663Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Manu Cornet <manucornet@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693857}

These tests were previously migrated from single-threaded MessageLoop to
a multi-threaded TaskEnvironment (then named ScopedTaskEnvironment) as
part of crbug.com/891670.

//base OWNERS decided in retrospect that it was better to keep a
single-threaded option for TaskEnvironment and introduced
SingleThreadTaskEnvironment. This CL retrofits that decision for
/printing.

This CL is a no-op if it passes CQ.

This CL was uploaded by git cl split.

R=rbpotter@chromium.org

Bug: 891670
Change-Id: I6077f3d2fb31e4fa16a64be3cf209e25ef2998db
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787179
Auto-Submit: Gabriel Charette <gab@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693856}

Noticed this while testing triggers. :)

Change-Id: Ib12f289bdc3bbf9cd96516c4f32d16b257102345
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787323
Commit-Queue: Stephen Nusko <nuskos@chromium.org>
Reviewed-by: Stephen Nusko <nuskos@chromium.org>
Reviewed-by: Eric Seckler <eseckler@chromium.org>
Auto-Submit: Stephen Nusko <nuskos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693855}

This CL updates the strings to follow the latest spec.

Bug: 954228
Test: builds
Change-Id: Ibe49b3dea0bfecbb1413d78079ddf5d9c4d3d90f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787764Reviewed-by: Michael Giuffrida <michaelpg@chromium.org>
Commit-Queue: Naoki Fukino <fukino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693854}

Bug: 988276, 860467
Change-Id: Ic5f69b9b6216b3bf812769369cef26cf2828d794
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787325Reviewed-by: Theresa  <twellington@chromium.org>
Commit-Queue: Finnur Thorarinsson <finnur@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693853}

Bug: 856601
Change-Id: Ic16d0527736d4b0bc5aa05f4a168cbb857c67485
TBR: robertma@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1788137Reviewed-by: Ian Clelland <iclelland@chromium.org>
Commit-Queue: Ian Clelland <iclelland@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693852}

Some web sites (e.g., salesforce.com) modifies the prototype of
AsyncFunction object, causing errors in Execute Script endpoint.
The CL works around this issue by avoid directly using AsyncFunction
object.

Bug: chromedriver:3103
Change-Id: If328aab279bd818f5766381cc537d9612674599c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1785738Reviewed-by: Rohan Pavone <rohpavone@chromium.org>
Commit-Queue: John Chen <johnchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693851}

This reverts commit a9ef3160.

Reason for revert: Breaks right click in other contexts.

Original change's description:
> Allow a Frame Background Context Menu on Mac
>
> This brings the frame context menu available on other platforms to Mac
> and allows users to restore the previously closed tab within the
> tabstrip.
>
> BUG=997637
>
> Change-Id: Iaff4cbb6066e52a6ffc969d030b9351051a7a1f6
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1779019
> Commit-Queue: Robert Liao <robliao@chromium.org>
> Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#693229}

TBR=ellyjones@chromium.org,robliao@chromium.org

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

Bug: 997637, 1001021
Change-Id: I9a154b7ca387371f188a1790f9af1b7ac2910075
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787445
Commit-Queue: Robert Liao <robliao@chromium.org>
Reviewed-by: Robert Liao <robliao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693850}

This reverts commit 48327f50.

Reason for revert: https://crbug.com/1000270 (device policy should not affect non-affiliated user sessions, I will create a new policy that is scoped down to login-screen only)

Original change's description:
> Add DeviceWebUsbAllowDevicesForUrls policy
>
> This CL adds a new device policy to whitelist certain USB devices for
> certain urls in order to be used by the WebUSB API. We already have a
> user policy for that, so this device policy reuses the existing user
> policy's infrastructure.
>
> Bug: 968373, 854329
> Change-Id: I21a80f0b6557a1270a8fd6243f07e2fc8682004a
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1636350
> Commit-Queue: Alexander Hendrich <hendrich@chromium.org>
> Reviewed-by: Sergey Poromov <poromov@chromium.org>
> Reviewed-by: Ovidio de Jesús Ruiz-Henríquez <odejesush@chromium.org>
> Reviewed-by: Dominic Battré <battre@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#665844}

TBR=battre@chromium.org,poromov@chromium.org,odejesush@chromium.org,hendrich@chromium.org

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

Bug: 968373, 854329, 1000270
Change-Id: Idc6961d6119325be385b67809fb685ede70cacad
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787647
Commit-Queue: Alexander Hendrich <hendrich@chromium.org>
Reviewed-by: Sergey Poromov <poromov@chromium.org>
Reviewed-by: Alexander Hendrich <hendrich@chromium.org>
Reviewed-by: Ovidio de Jesús Ruiz-Henríquez <odejesush@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693849}

These tests were previously migrated from single-threaded MessageLoop to
a multi-threaded TaskEnvironment (then named ScopedTaskEnvironment) as
part of crbug.com/891670.

//base OWNERS decided in retrospect that it was better to keep a
single-threaded option for TaskEnvironment and introduced
SingleThreadTaskEnvironment. This CL retrofits that decision for
/content/browser/fileapi.

This CL is a no-op if it passes CQ.

This CL was uploaded by git cl split.

R=mek@chromium.org

Bug: 891670
Change-Id: Id40fcab691f3126d75b96599b381d622f6c6cc11
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786814
Auto-Submit: Gabriel Charette <gab@chromium.org>
Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693848}

The URLRequestContext is only being used by one internal legacy
consumer, which doesn't need either file or data URL support. We'll want
to remove the URLRequestContext when we can, but for now, removing these
two factories will let us remove the code in net/ to handle those two
types of URLs, as they're not handled by the network service.

Bug: 934009
Change-Id: I753f03bcc6e9f915cb64490d9bee9a8664a90695
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787401Reviewed-by: Junbo Ke <juke@chromium.org>
Reviewed-by: Luke Halliwell <halliwell@chromium.org>
Commit-Queue: Matt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693847}

These tests were previously migrated from single-threaded MessageLoop to
a multi-threaded TaskEnvironment (then named ScopedTaskEnvironment) as
part of crbug.com/891670.

//base OWNERS decided in retrospect that it was better to keep a
single-threaded option for TaskEnvironment and introduced
SingleThreadTaskEnvironment. This CL retrofits that decision for
/components/invalidation.

This CL is a no-op if it passes CQ.

This CL was uploaded by git cl split.

R=pavely@chromium.org

Bug: 891670
Change-Id: I7bd7daa3a93a1ff5166ee56bdff138cd098ab4f9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787098
Auto-Submit: Gabriel Charette <gab@chromium.org>
Reviewed-by: Pavel Yatsuk <pavely@chromium.org>
Commit-Queue: Pavel Yatsuk <pavely@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693846}

This CL converts WakeLockPtr and WakeLockRequest in chrome,
chromeos, components, content, extensions, services and blink to
the new Mojo types, and uses pending_receiver<WakeLock>
in wake_lock.mojom, wake_lock_context.mojom, wake_lock_provider.mojom
in services and wake_lock.mojom in blink.

Bug: 955171, 978694
Change-Id: Ia79726cbdbc97ffd5714e976f65c4339d9356ca4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1771371
Commit-Queue: Miyoung Shin <myid.shin@igalia.com>
Reviewed-by: Ken Rockot <rockot@google.com>
Reviewed-by: Hidehiko Abe <hidehiko@chromium.org>
Reviewed-by: Min Qin <qinmin@chromium.org>
Reviewed-by: Yuri Wiitala <miu@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Achuith Bhandarkar <achuith@chromium.org>
Reviewed-by: Bill Budge <bbudge@chromium.org>
Reviewed-by: Maksim Ivanov <emaxx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693845}

We were seeing out of memory errors on Windows ThinLTO bots. After
measuring memory consumption of linking chrome.dll and
chrome_child.dll, it became clear that the estimated memory usage of a
thin link was too low. Adjusting to a more realistic value.

R=dpranke,gbiv,thakis
BUG=967074

Change-Id: I7d43d38e28e351a2dc16d79abbf332972cf66e9c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1785680Reviewed-by: Nico Weber <thakis@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Commit-Queue: Bob Haarman <inglorion@chromium.org>
Auto-Submit: Bob Haarman <inglorion@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693844}

Use [RaisesException] for immediate promise rejections in /third_party/blink/renderer/modules/picture_in_picture.

The bindings layer implicitly converts thrown exceptions in promise-returning
functions to promise rejections, and using ExceptionState makes this more
similar to ordinarily throwing exceptions, notably by including the auto-
generated exception context that makes it easier to see what IDL operation
caused an exception to be thrown.

This CL was uploaded by git cl split.

R=beaufort.francois@gmail.com

Bug: 1001114
Change-Id: I0d6265ccaccab772bac9e7d2ff111ff111432000
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786670
Auto-Submit: Jeremy Roman <jbroman@chromium.org>
Commit-Queue: François Beaufort <beaufort.francois@gmail.com>
Reviewed-by: François Beaufort <beaufort.francois@gmail.com>
Cr-Commit-Position: refs/heads/master@{#693843}

This reverts commit e5bdbac6.

Reason for revert: Wrong revert, test added here is not failing. The other CL introduced a flaky, which is reverted in:
https://chromium-review.googlesource.com/c/chromium/src/+/1787277

Original change's description:
> Revert "Notification scheduler: Add icon database metrics."
> 
> This reverts commit 63c7234e.
> 
> Reason for revert: Unit test failure (crbug.com/1001018)
> 
> Original change's description:
> > Notification scheduler: Add icon database metrics.
> > 
> > This CL adds icon database metrics, also fixes a typo in
> > stats::LogImpressionEvent.
> > 
> > Bug: 963298
> > Change-Id: I2fcd44d773166345760ee647c26e464c9ca0f89e
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1785161
> > Commit-Queue: Xing Liu <xingliu@chromium.org>
> > Reviewed-by: Hesen Zhang <hesen@chromium.org>
> > Reviewed-by: Ilya Sherman <isherman@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#693510}
> 
> TBR=isherman@chromium.org,dtrainor@chromium.org,xingliu@chromium.org,hesen@chromium.org
> 
> Change-Id: I94b298d9754e4651f757de2ba82a369be6523801
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 963298
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786372
> Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
> Commit-Queue: Hitoshi Yoshida <peria@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#693607}

TBR=peria@chromium.org,isherman@chromium.org,dtrainor@chromium.org,xingliu@chromium.org,hesen@chromium.org

Change-Id: I7db11edabd91c9a8265b7f8961054def38f804ab
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 963298
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786644Reviewed-by: Xing Liu <xingliu@chromium.org>
Commit-Queue: Xing Liu <xingliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#693842}