GitLab

This is fix for
https://chromium-review.googlesource.com/c/chromium/src/+/1654757

Bug: 956776
Change-Id: I700924f1962ce20110beb5e37ecb02ea3fa616ab
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1659661Reviewed-by: Vadim Shtayura <vadimsh@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669130}

This CL moves platform/wtf/allocator.{h.cc} to platform/wtf/allocator/
and ports other dependent files to refer to a new include path.

Bug: 950130
Change-Id: If4f4941083786606fe3312dd71c317453019bddc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1558550
Commit-Queue: Abhijeet Kandalkar <abhijeet@igalia.com>
Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669129}

Previously we would first check that as area is not a viewport defining
element (VDE) before potentially removing it from its snap container.


The problem is that VDE can change over the lifetime of a document for
example it can initially be HTML and later BODY. This means that it is
possible for the existing logic to consider BODY as a snap area first
and add it to a container but don't consider it a valid snap area at
a later point and thus miss removing it from the map.

The fix simply removes the VDE check for snap areas. We don't really
need to limit VDE being a snap area. We still keep limitation for VDE
becoming a snap container but there removal logic operates regardless so
it is safe.


Bug: 972434
Change-Id: I6d87f8df53f59c58b8302ed7b536b3f6783d98f0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1657754Reviewed-by: David Bokan <bokan@chromium.org>
Commit-Queue: Majid Valipour <majidvp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669128}

https://skia.googlesource.com/skia.git/+log/4c4945a25248..87e885038893


git log 4c4945a25248..87e885038893 --date=short --no-merges --format='%ad %ae %s'
2019-06-14 fmalita@chromium.org Reland "[skottie] Use metrics for Shaper vertical alignment"


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

The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

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
TBR=herb@google.com

Change-Id: If47f1bd3d1d386802f4e40527e9c33c3169817d7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1660108Reviewed-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@{#669127}

Currently all exo tests based on ExoTestBase will use a MockInputMethod.
We switch it to use the real InputMethodChromeOS, since there's really
no good reason for using a mock here.

This change is needed to ensure some arc IME code we're adding in
InputMethodChromeOS is tested properly.

Bug: 845780
Change-Id: I3ad8a8b628fe72ccc49aba588977847c9f80a4d7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1659648Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Commit-Queue: Darren Shen <shend@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669126}

In order to be able to revert to extension after autogenerated theme
is installed the extensions should be disabled.

Change-Id: I63e2bcd4b82c804039e9868e107a936c155dfd64
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1623502Reviewed-by: Gayane Petrosyan <gayane@chromium.org>
Reviewed-by: Evan Stade <estade@chromium.org>
Commit-Queue: Gayane Petrosyan <gayane@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669125}

As new options are added (and to be added), the argument list is growing
unreasonably long. Optional arguments are split into a MountOptions
struct. Passing a default constructed MountOptions is sufficient to
mount a share anonymously.

BUG=891462

Change-Id: Ia8dac5dc4a35fada622f78a25caf975a4a1a0771
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1659649
Commit-Queue: Anand Mistry <amistry@chromium.org>
Commit-Queue: François Degros <fdegros@chromium.org>
Reviewed-by: François Degros <fdegros@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669124}

This patch adds the flag that overrides default threshold value
that the renderer calls AddReceiveFlowControlQuota() to the browser per
receiving this amount of data so that the browser can continue sending
remaining data to the renderer.
Default value is 32k Bytes.

The more the value is, browser tends the less stop sending packets
and then consumes more memory.

This flag will be used to investigate the performance issue.
You can use this flag both on chrome and content_shell:
$ ./chrome.exe --websocket-renderer-receive-quota-max=3000000
This flag sets the threshold 3M Bytes.

Implementation detail:
This patch moves the default value constant from blink to
net/websockets/websocket_channel.h so that we can pass the command line
flag to the renderer.
This change also enables us to modify the value while considering sent buffer
from a server later on for further optimization.

Bug: 865001
Change-Id: Ib0f9638ece24fbc0523b94345d09a4382e9cf6a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1642850
Commit-Queue: Yoichi Osato <yoichio@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Karan Bhatia <karandeepb@chromium.org>
Reviewed-by: Adam Rice <ricea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669123}

Changes the runner script to block script execution and generate log
spam until it has positive confirmation the "pm serve" is serving
HTTP traffic.

"pm serve" is now started immediately after the repo is created, allowing
it to bind its socket in parallel with "pm publish" operations.


Bug: 973754
Change-Id: I92730010524a646204ead8fb8b113968b4179b5a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1659594
Auto-Submit: Kevin Marshall <kmarshall@chromium.org>
Reviewed-by: Wez <wez@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669122}

Fixed typo from https://crrev.com/c/1659029

TBR=kschmi@microsoft.com

Bug: 973946
Change-Id: Id317fc7ef52c5352c88bc64e48a9740ddf4fb3c1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1659555Reviewed-by: Alice Boxhall <aboxhall@chromium.org>
Commit-Queue: Alice Boxhall <aboxhall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669121}

Bug: 973599
Change-Id: I6fab755385d27e5203ba216f9185e94a16c123ab
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1657168Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Robert Ogden <robertogden@chromium.org>
Commit-Queue: Sophie Chang <sophiechang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669120}

Extracted from https://chromium-review.googlesource.com/c/chromium/src/+/1655093
since we should merge the missing lock to M76.

R=etiennep@chromium.org

Bug: 698140, 839091
Change-Id: I46e102eeb7b12743393c066b0bb2e184f982124b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1655988Reviewed-by: Etienne Pierre-Doray <etiennep@chromium.org>
Commit-Queue: Gabriel Charette <gab@chromium.org>
Auto-Submit: Gabriel Charette <gab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669119}

TBR=lushnikov@chromium.org

Bug: none
Change-Id: I946597a3360a008e12073392152c73f24e9fdfeb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1660089Reviewed-by: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669118}

https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/839329243a13..152de7c37fc5


Created with:
  gclient setdep -r src-internal@152de7c37fc5

The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome
TBR=mmentovai@google.com,aboxhall@google.com,mstensho@google.com,oksamyt@google.com

Change-Id: Ie777306314feebe1ca05aee4c2a790d4b3022464
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1660048Reviewed-by: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com>
Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#669117}

Using input and output buffer maps was a practice of the old
VideoDecodeAccelerators and should not be needed with the new
VideoDecoder interface.

Add a few members and a new constructor to V4L2DecodeSurface so it can
keep track of the input and output buffers as well as the VideoFrame we
will decode into. That way we can get the buffers and frame from the
decode request directly, without the need to look things up and track
their state.

BUG=941330
TEST=VDAtest passes on Minnie.

Change-Id: I21b8c58fb0ee86a670dafa89ba6925968dec77fe
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1647309
Commit-Queue: Alexandre Courbot <acourbot@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: David Staessens <dstaessens@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669116}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/5db6ccdb..cdd2c3ad

Please follow these instructions for assigning/CC'ing issues:
https://v8.dev/docs/triage-issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel
CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel
CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel
CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel
CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel

TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com

Change-Id: I35b046f5c9b6c28e4d8951f999db1e93799a4e25
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1659839Reviewed-by: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com>
Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#669115}

This CL teaches the cc scheduler about the asynchronous painting of
PaintWorklets. While PaintWorklets are being painted, the pending tree
cannot be activated (as it is not fully painted). As such, we must
delay activation (even via aborting the frame) until the painting has
finished.

Note that at this time the asynchronous painting has not been landed (as
it depends on this scheduling behavior). Support will be added in
upcoming patches; see the prototype:
https://chromium-review.googlesource.com/c/chromium/src/+/1577635

Bug: 907897
Change-Id: I7d9ec466c57fbe3f8088c5ff6bf6faa140bd4b83
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1649656
Commit-Queue: Stephen McGruer <smcgruer@chromium.org>
Reviewed-by: Sunny Sachanandani <sunnyps@chromium.org>
Reviewed-by: vmpstr <vmpstr@chromium.org>
Reviewed-by: Khushal <khushalsagar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669114}

# This is an automated release commit.
# Do not revert without consulting chrome-pmo@google.com.
NOAUTOREVERT=true
TBR=kariah@chromium.org

Change-Id: Id8bbc0a0c70187ee49641f9e3dde01521a339b1e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1660148Reviewed-by: chrome-release-bot@chromium.org <chrome-release-bot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669113}

This CL is necessary to fix the gcc 4.8 build:
https://chromium.googlesource.com/chromium/llvm-project/libcxx/+/ad464887b93526235ce06bbf97cd396ed6232b9a

Wanted for this CL, which improves the debugging experience:
https://chromium.googlesource.com/chromium/llvm-project/libcxx.git/+/78822a68537b5941eb86e11a5066aa549c30998f

https://chromium.googlesource.com/chromium/llvm-project/libcxx.git/+log/5938e0582bac..ad464887b935

$ git log 5938e0582..ad464887b --date=short --no-merges --format='%ad %ae %s'
2019-06-13 thomasanderson [libc++] Fix build with gcc 4.8
2019-06-13 ldionne [libc++] Add missing #include in <cwchar> tests
2019-06-13 eric Make GCC in C++03 Unsupported
2019-06-12 ldionne [libcxx] XFAIL set/multiset CTAD tests on Apple Clang 10
2019-06-12 ldionne [libcxx] XFAIL some CTAD tests on AppleClang 10
2019-06-12 eric Apply [[nodebug]] to typedefs throughout the STL.
2019-06-11 eric Move libc++ specific tests for std::function out of the std directory
2019-06-11 eric Add documentation and tests for Clangs C++11 extensions in C++03.
2019-06-11 ldionne [libcxx] Mark CTAD tests for set and multiset as unsupported on older Apple Clangs
2019-06-11 mclow.lists Mark CTAD fail tests for set/multiset as XFAIL for older compilers that give different error messages
2019-06-11 mclow.lists XFAIL a couple of tests on apple-clang-9.1, which is a compiler that I didn't know existed
2019-06-11 eric Check in test that demonstrates ABI break for std::function.
2019-06-11 ldionne [libc++] Implement deduction guides for <set>
2019-06-11 ldionne [libcxx] Make std::tuple<> trivially constructible
2019-06-11 ldionne [libcxx] Slightly improved policy for handling experimental features
2019-06-11 mclow.lists Add a test for is_base_of and incomplete types. Because this trait uses a compiler intrinsic which was broken in many clangs, have lots of XFAILs.
2019-06-11 mclow.lists XFAIL a couple of CTAD tests on clang-6; it gives different error messages than clang 7/8/9
2019-06-10 mclow.lists Recommit r362649, which was reverted in r262680 because of bugs in std::to_chars (which have now been fixed). Speedup to_string and to_wstring for integers using stack buffer and SSO
2019-06-10 mclow.lists Implement deduction guides for map/multimap. Reviewed as https://reviews.llvm.org/D58587. Thanks to Quuxplusone for the submission.
2019-06-10 zy [libc++] Fix leading zeros in std::to_chars
2019-06-08 eric update debugging docs to be less out of date
2019-06-08 eric Substantially reduce instantiations and debug size of std::function
2019-06-08 eric Fix some incorrect std::function tests
2019-06-06 vlad Revert "Speedup to_string and to_wstring for integers using stack buffer and SSO."
2019-06-05 ldionne [libcxx][test] Include test_workarounds.h where needed
2019-06-05 mclow.lists Update issue statuses. Reviewed as https://reviews.llvm.org/D62932
2019-06-05 mclow.lists Speedup to_string and to_wstring for integers using stack buffer and SSO. Reviewed as https://reviews.llvm.org/D59178  Thanks to ivafanas for all his work on this patch.
2019-06-04 ldionne [libcxx] Add test to check min/max requirement to regular expression
2019-06-04 mclow.lists No longer reject inputs when using a locale that has grouping information _and_ the input has no grouping characters at all. We continue to reject cases when the input has grouping characters in the wrong place. Fixes PR#28704
2019-06-04 mclow.lists We had a _LIBCPP_ASSERT commented out because gcc 4.9 didn't like it. We (LLVM) now require GCC 5.1, so that's not a problem any more. Re-enable the assertion. Fixes PR#36863
2019-06-02 phosek [libcxx] Use libtool when merging archives on Apple platforms
2019-05-31 mclow.lists Add include for 'test_macros.h' to all the tests that were missing them. Thanks to Zoe for the (big, but simple) patch. NFC intended.
2019-05-30 ldionne [libcxx] Add regex test cases from PR40904
2019-05-30 jryans [Docs] Modernize references to macOS
2019-05-30 phosek [runtimes] Use _LIBCPP_HAS_COMMENT_LIB_PRAGMA in all relevant files
2019-05-30 phosek [libcxx][libcxxabi] Remove the unused CMake checks
2019-05-30 phosek [runtimes] Use -Wunknown-pragmas for the pragma check
2019-05-30 phosek [runtimes] Check if pragma comment(lib, ...) is supported first
2019-05-30 phosek [runtimes] Support ELF dependent libraries feature
2019-05-29 ldionne [NFC][libcxx] Remove trailing whitespace
2019-05-29 mclow.lists Add additional constraints on midpoint(pointer, pointer). Fixes PR#42037.
2019-05-29 eric Fix build breakage on 32-bit machines
2019-05-29 eric Rework std::type_info definition to support systems without fully merged type info names.
2019-05-28 mclow.lists Fix test that used raw string literals. Doesn't work in C++03
2019-05-28 mclow.lists Fix an incorrect 'Throws' in the regex code. Add a test for the new behavior. Reviewed as https://reviews.llvm.org/D61828. Thanks to Mark for the catch and the fix.
2019-05-24 eric fix test for older clang versions
2019-05-24 eric fix destroying delete test with older apple compilers
2019-05-24 eric Update C++2a status for destroying delete
2019-05-23 eric P0722R3:  Implement library support for destroying delete
2019-05-23 chris.bieneman [CMake] Copy C++ headers before configuring runtimes build
2019-05-23 ldionne [libcxx][tests] Fix order checking in unordered_multiset tests.
2019-05-22 phosek [runtimes] Move libunwind, libc++abi and libc++ to lib/$target/c++ and include/c++
2019-05-22 ldionne [libcxx][test] Fix order checking in some more unordered_multimap tests

Created with:
  roll-dep src/buildtools/third_party/libc++/trunk

BUG=None
R=thakis

Change-Id: Iaa0f8a7d6d8bec28f6026fbb71b620eac6c0188c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1658438
Auto-Submit: Thomas Anderson <thomasanderson@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669112}

Because PWAs (and anything using RemoteCocoa) need an app shim to be
running to get anything on-screen, we added code to re-create an app
shim if one cannot be launched.

But this re-creation wasn't guarded on being a PWA, and as a consequence
platform apps would go and create app shims that nobody expected to
exist. So let's not do that.

Some tests were inappropriately relying on this behavior. Update
WebAppShortcutCreator to be more explicit about where it creates and
updates its shortcuts.

Also, re-enable tests in component build (that was fixed a while ago
in crrev.com/619648).

Bug: 941160
Change-Id: I5fb233f346095116d28ddbbfa9e23aca1bb1eab3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1657522Reviewed-by: Dominick Ng <dominickn@chromium.org>
Commit-Queue: ccameron <ccameron@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669111}

https://chromium.googlesource.com/chromiumos/chromite.git/+log/1e83e141f541..17937cb2333c


git log 1e83e141f541..17937cb2333c --date=short --no-merges --format='%ad %ae %s'
2019-06-14 jflat@chromium.org chromeos_config: Make samus-kernelnext paladin important


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

The AutoRoll server is located here: https://autoroll.skia.org/r/chromite-chromium-autoroll

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

CQ_INCLUDE_TRYBOTS=luci.chromium.try:chromeos-kevin-rel
TBR=chrome-os-gardeners@google.com

Change-Id: Id886a70d413206317c43fccbac734b3c340ac314
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1659536Reviewed-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@{#669110}

These were primarily intended to be run as pixel tests. The trace_test
suite seems more stressful and we don't have the resources to
investigate the context loss issues here.

Bug: 974077
Change-Id: Ia6bb6a028e73459bae1ac8cd8de2188235b84a28
Tbr: ynovikov@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1659605Reviewed-by: Kenneth Russell <kbr@chromium.org>
Reviewed-by: Yuly Novikov <ynovikov@chromium.org>
Commit-Queue: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669109}

- Remove the tweak from bug 881513, a revert of https://crrev.com/c/1450396
- Enable the import of external/wpt/webdriver

Bug: 881513
Change-Id: I9c03081d52576b2057d18bd8a3efd2b11a41033f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1615704
Commit-Queue: Robert Ma <robertma@chromium.org>
Reviewed-by: Robert Ma <robertma@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669108}

In BleCharacteristicFinder, we currently override onGatt*Added
BluetoothAdapter::Observer functions which are all deprecated. When we
get all the characteristics we're looking for, we call back immediately.
Instead, this CL just waits for GattServicesDiscovered before parsing
the services and characteristics.

This does not affect latency at all because GATT service discovery is
done in batch anyway in BlueZ before all of the callbacks are just
triggered in series. This is much simpler and will also allow for an
easy discovery process when fixing work profiles (see
go/beto-work-profiles).

Bug: b/67412244
Change-Id: I22d3baf485b386ee5ba6a41e2b254a5f9919b01d
Tested: Heavily manually tested and also added unit tests.
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1658432
Commit-Queue: Jeremy Klein <jlklein@chromium.org>
Auto-Submit: Jeremy Klein <jlklein@chromium.org>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669107}

https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/f457ae77aa76..839329243a13


Created with:
  gclient setdep -r src-internal@839329243a13

The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome
TBR=mmentovai@google.com,aboxhall@google.com,mstensho@google.com,oksamyt@google.com

Change-Id: I4c831511ea21e6691da27976be2adf811bd88ca7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1659537Reviewed-by: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com>
Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#669106}

Bug: 973731, 855319
Change-Id: I759663fd82492c6dc17cb63c03557bc729d71102
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1657384
Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Commit-Queue: Simon Que <sque@chromium.org>
Reviewed-by: Simon Que <sque@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669105}

Bug: 932339
Change-Id: Ib835835ac8e58262091232be7ba285c2901f3746
Cq-Depend: chromium:1630343,chromium:1630135
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1631157
Commit-Queue: Julian Watson <juwa@google.com>
Reviewed-by: Joel Hockey <joelhockey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669104}

Bug: 898684
Change-Id: I05ae0f474c5b9978932bd2bc98033075b12bd5ff
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1656672Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Yuly Novikov <ynovikov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669103}

This CL changes ExternallyInstalledWebAppPrefs::GetInstalledAppUrls to
BuildAppIdsMap which does not check for installation, to fix a dependency
cycle when querying for apps in the constructor of WebAppProvider.

This also represents a step towards refactoring the WebApp test environment.

Bug: 973324

Change-Id: Ia0ad9f29fe6694a548acd716885a7d538d376332
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1654676
Commit-Queue: calamity <calamity@chromium.org>
Reviewed-by: Alexey Baskakov <loyso@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669102}

The test was getting the profile from a recently closed app browser.
Change the test to retrieve the profile from the harness.

Bug: 972930
Change-Id: I3eb0f907ed53b3e7dc31fd9c1932c52dbfe75545
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1659551
Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Commit-Queue: Jay Harris <harrisjay@google.com>
Auto-Submit: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Reviewed-by: Jay Harris <harrisjay@google.com>
Cr-Commit-Position: refs/heads/master@{#669101}

This reverts commit 6e65ba90.

Reason for revert:

Findit (https://goo.gl/kROfz5) identified CL at revision 669034 as the
culprit for flakes in the build cycles as shown on:
https://analysis.chromium.org/p/chromium/flake-portal/analysis/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyQwsSDEZsYWtlQ3VscHJpdCIxY2hyb21pdW0vNmU2NWJhOTA3ZjUyMGNlZmU5YjYyMDg3MWNmZmIyZTc1N2NhNmQ2Nww

Sample Failed Build: https://ci.chromium.org/buildbot/chromium.linux/Linux%20Tests%20%28dbg%29%281%29%2832%29/59319

Sample Failed Step: content_browsertests

Sample Flaky Test: AccessibilityIpcErrorBrowserTest.DISABLE_ResetBrowserAccessibilityManager

Original change's description:
> Disable flaky test AccessibilityIpcErrorBrowserTest.ResetBrowserAccessibilityManager
> 
> TBR=kschmi@microsoft.com
> 
> Bug: 973946
> 
> Change-Id: I69b54fdfe9421acacaf98c2bcf684680e7cb66fd
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1659029
> Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org>
> Reviewed-by: Kurt Catti-Schmidt <kschmi@microsoft.com>
> Cr-Commit-Position: refs/heads/master@{#669034}


Change-Id: Ic5a44c3d23a11cd0f209a6143067d51e803a617a
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 973946
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1660008
Cr-Commit-Position: refs/heads/master@{#669100}

One is the screenshot was showing weird on actual device.

The other is the screenshot is not deleted when using
SetBoundsDirect. That was because it was not animating, but
still managed by TabletModeWindowManager. The screenshot
should have been deleted if it is either not animating or
not tracked, not require both, which this CL fixes.

Test: manual
Bug: 948706
Change-Id: I87add8592a8f1e6bbec55dbe44e93eb52844c5a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1659020
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669099}

This is a quickfix for a crash when a heap profile is being reported
over UMA on Android.

BUG=973430

Change-Id: I07cf28ef18809ab82c1a447f686871f6bedb71fd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1659025
Commit-Queue: Alexei Filippov <alph@chromium.org>
Auto-Submit: Alexei Filippov <alph@chromium.org>
Reviewed-by: Mike Wittman <wittman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669098}

Change-Id: Ie778b542676ff6d2f4850d422d1984a72905dcb2
Bug: 860438
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1659356
Commit-Queue: Regan Hsu <hsuregan@chromium.org>
Reviewed-by: Esmael El-Moslimany <aee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669097}

It kicks eslint for *.mjs via tools/web_dev_style/js_checker.js.

Change-Id: I669639d438205b03b5bd04de0ccd6d9e6169604a
Bug: 972476
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1654732
Commit-Queue: Kent Tamura <tkent@chromium.org>
Reviewed-by: Dan Beam <dbeam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669096}

The set_paused() and set_connection_error_handler() methods on
extensions::SerialConnection are not trivial and are renamed to
SetPaused() and SetConnectionErrorHandler().

Bug: 893334
Change-Id: I10cc92a5f61c9907ccbf5e6975a7028f9a9b688e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1658682
Commit-Queue: Reilly Grant <reillyg@chromium.org>
Auto-Submit: Reilly Grant <reillyg@chromium.org>
Reviewed-by: Ovidio de Jesús Ruiz-Henríquez <odejesush@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669095}

Google services settings should be able to be opened from the incognito
mode.

Bug: 967674
Change-Id: I9008c0159b7729c17a7fc703c5a61171d2dd85e6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1640555Reviewed-by: Mihai Sardarescu <msarda@chromium.org>
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Auto-Submit: Jérôme Lebel <jlebel@chromium.org>
Commit-Queue: Jérôme Lebel <jlebel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669094}

This CL removes signature and sequence ID from the heartbeat request
since they are no longer required by the server. It also rewrites
HeartbeatSenderTest to mock the HeartbeatClient directly instead of
creating in process channel and mocking the server, so that the tests
can be more deterministic.

Bug: 971363
Change-Id: I3b9be0819172d06da657536c56744532df39a75e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1653602
Commit-Queue: Joe Downing <joedow@chromium.org>
Reviewed-by: Joe Downing <joedow@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669093}

This CL is a simple code move taking FindAppIdForUrl out of
WebAppTabHelperBase and putting it in web_app::AppRegistrar
for easier reuse of the functionality.

There are no behavioural changes in this CL.

Bug: 956810
Change-Id: I74858b74421763aa1e7050a5d0672321b8019718
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1638212
Commit-Queue: Alan Cutter <alancutter@chromium.org>
Reviewed-by: Alexey Baskakov <loyso@chromium.org>
Auto-Submit: Alan Cutter <alancutter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669092}

ServiceWorker subresource fallback currently has a work-around for CORS
requests.  Because CORS has been implemented in blink in the past the
code had to message back into blink in order correctly perform CORS
on the fallback request.  This create performance problems.

This CL skips this fallback workaround if the out-of-renderer CORS
(OOR-CORS) feature is enabled.

Bug: 973633
Change-Id: If8a3510291639fda359fd425c2e08173f2a87cf7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1658868
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#669091}