GitLab

Change-Id: Ic745c4a3136f8d0783e169fccfb53cf4d0c412e5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600642Reviewed-by: Bence Béky <bnc@chromium.org>
Commit-Queue: Nick Harper <nharper@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657780}

This reverts commit ec2ce628.

Reason for revert: Pursuing alternate strategy of preserving base::MD5 interface, but replacing its implementation with boringssl.

Original change's description:
> Replace base::MD5Context with boringssl in //media
> 
> Testing with media_unittests
> 
> Bug: 755368
> Change-Id: I9b45735a5d3054207d3bc340bce45fb25014a447
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1568847
> Auto-Submit: Daniel McArdle <dmcardle@chromium.org>
> Commit-Queue: Daniel McArdle <dmcardle@chromium.org>
> Reviewed-by: Dan Sanders <sandersd@chromium.org>
> Reviewed-by: David Benjamin <davidben@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#653350}

TBR=davidben@chromium.org,sandersd@chromium.org,dmcardle@chromium.org

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

Bug: 755368
Change-Id: I79c40ea2ee09637b78abc7ad50b049980c934e0f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600777Reviewed-by: Dan Sanders <sandersd@chromium.org>
Commit-Queue: Daniel McArdle <dmcardle@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657779}

These were caught by presubmit hook, so this CL prepares for it
in advance, so the main diff looks easier to review.

In summary,

- base::Closure -> base::RepeatingClosure.
- base::Bind -> base::BindRepeating.
- remove ununsed header inclusions (eg base/threading/thread_task_runner_handle.h).
- stop null-checking by comparing against nullptr.
- base::ThreadChecker -> THREAD_CHECKER.
- removed unused content/ includes.

BUG=787261

Change-Id: I15d7d7e9d9a31bfbf2683c70f955033d2b11f228
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600518
Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
Commit-Queue: Guido Urdaneta <guidou@chromium.org>
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Auto-Submit: Antonio Gomes <tonikitoo@igalia.com>
Cr-Commit-Position: refs/heads/master@{#657778}

Bug: 950091
Change-Id: Ia71b231c84e65ed223a125054993992917227e25
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600397
Commit-Queue: Jonathan Metzman <metzman@chromium.org>
Auto-Submit: Jonathan Metzman <metzman@chromium.org>
Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657777}

Bug: 937995
Change-Id: I64f090e6096200cf404d2f2e4c33ce813ff90eed
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594178Reviewed-by: Luciano Pacheco <lucmult@chromium.org>
Reviewed-by: Michael Giuffrida <michaelpg@chromium.org>
Commit-Queue: Danan S <danan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657776}

Migrate the following from mojom::SessionController:
- SetSessionInfo
- UpdateUserSession
- SetUserSessionOrder

Bug: 958214
Change-Id: I33d0210753e61801ac16d16f08360e0121d0f160
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600432Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Daniele Castagna <dcastagna@chromium.org>
Commit-Queue: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657775}

1. Prevent Chrome reaching directly into ash::AcceleratorController by
creating a pure virtual interface. The implementation is renamed
AcceleratorControllerImpl.

2. Move volume accelerator handling to ash. Convert VolumeController
interface into a single callback that Chrome provides. This callback
is run whenever the volume is changed via the keyboard.

Bug: 958225, 958131
Change-Id: Icc59608ad092eb9f53cda0637d0585ffdfa7f925
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1595507
Commit-Queue: Evan Stade <estade@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Auto-Submit: Evan Stade <estade@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657774}

Bug: 959929
Change-Id: I5f0784568f0d6147fc32de5c54b1f8799f472ef9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600636Reviewed-by: David Benjamin <davidben@chromium.org>
Commit-Queue: Eric Roman <eroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657773}

Change-Id: Ie05c8115b56eb54b3c520618a97b0db9243c6df8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600633
Auto-Submit: Caleb Rouleau <crouleau@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657772}

UI surfaces like the settings window or file manager are "native OS UI"
and should not follow the browser's font size setting.

The implementation is based on checking URLs because the Webkit prefs
must be configured early in WebContents initializion, before the
Browser initialization is complete (and hence registered with things
like SettingsWindowManager).

Bug: 955071
Test: added to browser_tests and unit_tests

Change-Id: I982b778498945e159d4a33c39f5e84ac6bc45f41
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594085Reviewed-by: Ken Rockot <rockot@google.com>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657771}

https://android.googlesource.com/platform/external/perfetto.git/+log/d2baff227833..3cd2c388997d


git log d2baff227833..3cd2c388997d --date=short --no-merges --format='%ad %ae %s'
2019-05-08 lalitm@google.com Merge "trace_processor: fix gcc compile"
2019-05-08 rsavitski@google.com cpp_headers_from_protos: hardcode relative path to buildtools clang-format


Created with:
  gclient setdep -r src/third_party/perfetto@3cd2c388997d

The AutoRoll server is located here: https://autoroll.skia.org/r/perfetto-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.


TBR=perfetto-bugs@google.com

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

*messages.* file names are generally reserved for IPC logic, and a
presumbit error is triggered if there isn't an OWNERS file with
security reviewers.

TBR=antrim@chromium.org

Change-Id: I5172a35cb2166d37aa6c994b1ad2083f8af1532b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1601258
Commit-Queue: Rayan Kanso <rayankans@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Reviewed-by: Pavel Yatsuk <pavely@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657769}

Though the user-facing control for this pref was removed, because the
pref had been synced, its value persisted, potentially locking users
into a less-than-ideal RSSI threshold. This change rips all of this
unused logic out, so that all users use the same preferred threshold.

Bug: 943875
Change-Id: Id2bd9adc3090f835816fa638685d6ca3fedea01f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1592863Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Commit-Queue: Ryan Hansberry <hansberry@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657768}

https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/1ddf2968fb2a..b6e71184c68c


Created with:
  gclient setdep -r src-internal@b6e71184c68c

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=dvadym@chromium.org,noel@chromium.org,jbroman@chromium.org,xialinyan@chromium.org

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

https://chromium.googlesource.com/angle/angle.git/+log/de991d4fe26a..f5115ed4b0f6


git log de991d4fe26a..f5115ed4b0f6 --date=short --no-merges --format='%ad %ae %s'
2019-05-08 timvp@google.com Vulkan: Enable standard derivatives extension
2019-05-08 staphany@chromium.org Roll ./third_party/googletest/src/ 9997a830e..3f5b5b8f8 (3 commits)


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

The AutoRoll server is located here: https://autoroll.skia.org/r/angle-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: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=jonahr@chromium.org

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

chromium.fyi/ios12-beta-simulator has incorrect destination `iPad Pro`
https://ci.chromium.org/p/chromium/builders/ci/ios12-beta-simulator

Need to update destination to one of valid
iPad Pro (9.7-inch)
iPad Pro (12.9-inch)
iPad Pro (12.9-inch) (2nd generation)
iPad Pro (10.5-inch)

Bug: 960431
Change-Id: Iff977a2f1d7b6ae5940f2f177552b1ad1b0562fc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1599629
Auto-Submit: Maksym Onufriienko <monufriienko@chromium.org>
Reviewed-by: Justin Cohen <justincohen@chromium.org>
Commit-Queue: Justin Cohen <justincohen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657765}

Further decreases the rate at which clicks are sent to content nodes
during instrumentation tests for the VR browser to ensure that all
clicks are reliably picked up.

Bug: 953990
Change-Id: I6f903f14f2a6bb02fef7e7a76629fa5e038e2e55
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1601414
Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
Commit-Queue: Klaus Weidner <klausw@chromium.org>
Auto-Submit: Brian Sheedy <bsheedy@chromium.org>
Reviewed-by: Klaus Weidner <klausw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657764}

When Google is the DSP we now use the local NTP built into Chrome
instead of the remote NTP specified in prepopulated_engines.json.
Remove the entry, as the loading of the local NTP is handled by
logic in chrome/browser/search/search.cc.

Bug: 583289
Change-Id: Id28db805bc53eebab6d492be666f909e0ff0b800
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1591796Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org>
Reviewed-by: Justin Donnelly <jdonnelly@chromium.org>
Commit-Queue: Kyle Milka <kmilka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657763}

XRView's were reused instead of creating new ones.  Some of the exposed
data was arrays that can be detached, which could lead to crashes. Even
after fixing it so we don't use the detached arrays, there was the
potential for debug tools to try to read them, so we now return null.

Bug: 957521, 958014
Change-Id: Id777f598abad39d9cbbbdec539f85d7915eca880
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600482Reviewed-by: Brandon Jones <bajones@chromium.org>
Commit-Queue: Bill Orr <billorr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657762}

Refactored existing NavigatorVR class into two. The new NavigatorXR implements
just the XR attribute leaving just getVRDisplays() to be implemented in the former class.

Test:
Passes the following Web Tests:
- xr/navigator_xr_blocks_getVRDisplays.html
- xr/navigator_xr_blocked_by_getVRDisplays.html

Bug: 948360
Change-Id: I24d593cd6bab5fc732af287e8b0b102a684c43be
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1597531
Commit-Queue: Suman Kancherla <sumankancherla@chromium.org>
Reviewed-by: David Dorwin <ddorwin@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657761}

Android appears to use a different color encoding scheme than the other
platforms (ABGR vs ARGB). Retrieving the colors via *getAddr32()
returns the "raw" color with the platform-specific encoding. getColor()
reorders the bytes to be in ARGB no matter what platform it's on.

I also used existing named constants for the colors to make the tests a
bit easier to read.

Bug: 850782
Change-Id: Ic756fee15fa685cafecfc21f57b2565bf8df08a2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1594824
Commit-Queue: Aran Gilman <gilmanmh@google.com>
Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Reviewed-by: Stephen Chenney <schenney@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657760}

If we are playing a local file then we see the
"file:///" scheme as the source. We should replace
it with placeholder text.

BUG=960500

Change-Id: Ie3680ddb6927ec65c4e8d823d1016a8be9f8ee0a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600496Reviewed-by: Ken Rockot <rockot@google.com>
Commit-Queue: Becca Hughes <beccahughes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657759}

Bug: 936104
Change-Id: I07733cb7197ef5992d418a38d967544a7ee2b9d2
Tbr: avi@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1491796
Commit-Queue: Paul Meyer <paulmeyer@chromium.org>
Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657758}

Uses portal postMessage APIs instead of broadcast channel in the
following tests:

- portals-activate-inside-portal.html
- portals-no-referrer.html

Change-Id: Iaa4d8a79fed99f7b39181eec59f26d89b7e38864
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1591804Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Reviewed-by: Lucas Gadani <lfg@chromium.org>
Commit-Queue: Adithya Srinivasan <adithyas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657757}

BUG=840435

Change-Id: I9269e6ff0a063fcad38e74b3b04861b8a2a9694c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600537
Commit-Queue: Avi Drissman <avi@chromium.org>
Auto-Submit: Avi Drissman <avi@chromium.org>
Reviewed-by: Mark Mentovai <mark@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657756}

summary of discussions: go/rendering-memory

TBR=erikchen@chromium.org,asvitkine@chromium.org

BUG=957978

Change-Id: I00bd2c8e0cd8684eee78afe50af87c8552b2821a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1590587Reviewed-by: Ehsan Chiniforooshan <chiniforooshan@chromium.org>
Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Reviewed-by: Hector Dearman <hjd@chromium.org>
Commit-Queue: Ehsan Chiniforooshan <chiniforooshan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657755}

SideSwipeController dealloc access WebStateList while it may be deleted
As webStateList is owned by the TabModel and it's a weak member of this
object

Bug: 959753
Change-Id: Ic667bc9711030bc7419f6b6629ffcce0a7a4a922
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598176Reviewed-by: Justin Cohen <justincohen@chromium.org>
Commit-Queue: Mohammad Refaat <mrefaat@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657754}

https://skia.googlesource.com/skia.git/+log/5785e4a16530..ca449d589927


git log 5785e4a16530..ca449d589927 --date=short --no-merges --format='%ad %ae %s'
2019-05-08 fmalita@chromium.org [skottie] Cleanup: relocate text sources
2019-05-08 bsalomon@google.com Separate GrCaps for semaphores and fence syncs.
2019-05-08 brianosman@google.com Add support for float remainder to interpreter


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

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=robertphillips@chromium.org

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

This patch changes the type of layout needed from child to self when
laying out line items and the content size of the item changes.

R=cbiesinger@chromium.org

Change-Id: Ica76a59698500d8f2c9c6ec66a53839f010d1ac7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598195Reviewed-by: Christian Biesinger <cbiesinger@chromium.org>
Commit-Queue: vmpstr <vmpstr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657752}

This logic was previously implemented for LitePageRedirect but we've
decided to share the logic for all previews. It has been factored out
into a separate feature which is enabled by default and supports a
comma separated list which can be changed when needed on the server.

Bug: 958515
Change-Id: Ifa5070ccc824001e0dbb30ea5b7acee6da206388
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1599351Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Doug Arnett <dougarnett@chromium.org>
Commit-Queue: Robert Ogden <robertogden@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657751}

Bug: 960780,932260
Change-Id: I71e8b5ea6b6d8d81e27521da812ba8beae8f50f3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600959Reviewed-by: Antoine Labour <piman@chromium.org>
Commit-Queue: Peng Huang <penghuang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657750}

The failure of OpenThread to return a valid handle is the suspected
cause of the crash in the associated bug. This check is intended to
validate that suspicion.

Bug: 947459
Change-Id: I55e59bf903e5383d9def56c3c57f3b8b4de5182f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600315
Auto-Submit: Mike Wittman <wittman@chromium.org>
Commit-Queue: Charlie Andrews <charliea@chromium.org>
Reviewed-by: Charlie Andrews <charliea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657749}

DownloadServiceFactory will still release them and return them it as a
raw pointer because BrowserContextKeyedServiceFactories do that, but
when it becomes a SimpleKeyedServiceFactory this will be more natural.

Bug: 943724
Change-Id: I104e832288746d1ae27b49e16a9beffe5d813051
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1599744Reviewed-by: Xing Liu <xingliu@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Commit-Queue: Henrique Nakashima <hnakashima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657748}

Adds the ability to skip XR browser tests at runtime if requirements for
a test are not met.

As a result, also makes the browser test launcher properly handle gtest
reporting that a test was skipped, as previously it was treated as
an unknown error.

Currently, the only requirement that can be checked is whether a DirectX
11.1-capable GPU is present on the machine, but additional requirements
can be easily added.

Bug: 951469
Change-Id: I81c4cccf83bed3c13ec1d82ad99a16c29fba7cb5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1575030Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Bill Orr <billorr@chromium.org>
Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657747}

This is a reland of 18526eed

Diff PS3->PS4 is fix for breakage.

Original change's description:
> [service-manager] Simplify connection logic
>
> This reduces the complexity of Service Manager's logic for handling
> interconnection between service instances. Much of the complexity was
> due to connection operations once upon a time being highly
> asynchronous within SM, but that hasn't been the case for a while.
>
> Rather than having ServiceManager::Connect do way too much stuff, it's
> replaced with a simpler and more straightforward
> FindOrCreateMatchingTargetInstance, which does only what the name says.
> Previous work done by Connect() in various edge cases has been moved out
> to its prior call sites, now replaced with calls to FOCMTI.
>
> Bug: 904240
> Change-Id: Ia276f6fa61477ddc141724d30484237498eef070
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1593722
> Reviewed-by: Oksana Zhuravlova <oksamyt@chromium.org>
> Commit-Queue: Ken Rockot <rockot@google.com>
> Cr-Commit-Position: refs/heads/master@{#657401}

Bug: 904240
Change-Id: If07ff20dd8886901013f214f8b51a9e447224189
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600453Reviewed-by: Oksana Zhuravlova <oksamyt@chromium.org>
Commit-Queue: Ken Rockot <rockot@google.com>
Cr-Commit-Position: refs/heads/master@{#657746}

Add tracking for pages which can be scripted by other pages (via
window.open) to the back-forward cache metrics.

BUG=933147,957632
R=creis@chromium.org
CC=​arthursonzogni@chromium.org

Change-Id: I33712203b3c996f09a70ebb4fe221d712c7a919d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1569006
Commit-Queue: Alexander Timin <altimin@chromium.org>
Reviewed-by: Charlie Reis <creis@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657745}

This patch changes the location of downloaded pages on Android Q,
because the old location is no longer accessible.

Bug: 956679
Change-Id: I55e320d771bc16f8d36d8477a9990f73cccd02ee
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1597532Reviewed-by: Min Qin <qinmin@chromium.org>
Reviewed-by: Carlos Knippschild <carlosk@chromium.org>
Commit-Queue: Dan H <harringtond@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657744}

The via header can have multiple space separated entries, if it hits
the staging server and the prod server. So relaxes the exact equality
check.

Bug: 960105
Change-Id: Ib50ee751f40b0892376a4ae622f0ac061efb401d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600632
Commit-Queue: rajendrant <rajendrant@chromium.org>
Reviewed-by: Robert Ogden <robertogden@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657743}

This fixes the failing quic test by checking for quic or https scheme
for the DRP proxy.

Bug: 958038
Change-Id: I00610cce5de62ad028ea0a41b0a9f2723dd87fed
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600080Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Robert Ogden <robertogden@chromium.org>
Commit-Queue: rajendrant <rajendrant@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657742}

As a manager-wide property, removed enitrely from ContextHostResolver.
Because HostResolverManager::Set... was only ever set at manager-
creation time, replaced it with a new option in ManagerOptions. Because
HostResolverManager::Get... was only ever used in tests, replaced it
with a for_testing getter.

Took the opportunity to replace "no_ipv6_on_wifi" with
"check_ipv6_on_wifi" as I believe it's slightly more descriptive and
makes slightly more sense as a positive setting instead of a negative.

Bug: 934402
Change-Id: I20bfd822db886fbddded9319e1d291c300fe1489
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1582612Reviewed-by: Paul Jensen <pauljensen@chromium.org>
Reviewed-by: Asanka Herath <asanka@chromium.org>
Commit-Queue: Eric Orth <ericorth@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657741}