GitLab

Whenever card suggestions are shown, and the user has server cards and is
eligible for FIDO authentication, a preflight call is made to retrieve
authentication details. Logging the number of times this call is made.

Also includes a bug fix, something that was caught by the metrics test.

Bug: 949269
Change-Id: Ic1067de5a137598c0758f787a41d7a311f7d0a0e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1783858Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Tommy Martino <tmartino@chromium.org>
Reviewed-by: Jared Saul <jsaul@google.com>
Commit-Queue: Manas Verma <manasverma@google.com>
Cr-Commit-Position: refs/heads/master@{#704488}

https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/9514a2a7642d..d65a3ffb5ea1


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

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll
Please CC jbudorick@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.chrome.try:linux-chromeos-chrome
TBR=jbudorick@google.com

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

Previous to this CL, hitting CTRL-A inside a TextArea
containing text that ends with a newline would cause the
selectionEnd property to be incorrect. selectionEnd returns
a number that's 1 pass the end of text.

This is because TextControl automatically inserts an BRElement
at the end when the last child is a carriage return. And the
inserted node is taken into consideration when calculating the
selection range.

The fix is to ignore this last child if it's a BRElement.

Tested with carriage return inserted into the beginning, middle and end
of text to make sure there's no regression.

Bug: 882614
Change-Id: Idc7a1f098556fc74b89f3041e3f5f35e127eed78
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1838549Reviewed-by: Mason Freed <masonfreed@chromium.org>
Reviewed-by: Kent Tamura <tkent@chromium.org>
Commit-Queue: Yu Han <yuzhehan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704486}

This is a reland of bc1f6367

Original change's description:
> [fuchsia] Add wildcard support for hosts filtering.
>
> Add wildcard-based filtering for hosts in UrlRequestRewriteRules.
>
> Bug: 1011156
>
> Change-Id: I982b2241e622194dbf39ccd8519af8f92ee593c9
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1838706
> Commit-Queue: Fabrice de Gans-Riberi <fdegans@chromium.org>
> Reviewed-by: Wez <wez@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#704406}

TBR=wez@chromium.org

Bug: 1011156
Change-Id: Iee15c86f44a6f6a1d2496f79483add4ea12d2799
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1851324Reviewed-by: Fabrice de Gans-Riberi <fdegans@chromium.org>
Reviewed-by: Wez <wez@chromium.org>
Commit-Queue: Fabrice de Gans-Riberi <fdegans@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704485}

This CL is just the boilerplate of creating new classes and wiring them
up to WebAppProvider.

Eventually, the functionality from web_app_extension_shortcut should be
migrated into this class. A WIP CL for beginning this is available here:
https://crrev.com/c/1846422.

The motivation for beginning this now is file handling on Linux, as
registering and unregistering file handlers in the OS is tightly
coupled to shortcut management.

Bug: 860581
Change-Id: I3e28018d6fd6656fccd3c22ed62b9576db43f51c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1847708
Commit-Queue: Jay Harris <harrisjay@chromium.org>
Reviewed-by: Alexey Baskakov <loyso@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704484}

Navigation Sheet should be closed when its Tab is closed. In most cases
the sheet is close prior to Tab, but the opposite can happen if incognito
tabs are closed via Android notification. This CL ensures the sheet gets
closed upon tab closure.

Bug: 1011073
Change-Id: Id9729c2d8987bfe37ff81f47a8070d40a86e00e7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1842134Reviewed-by: Matthew Jones <mdjones@chromium.org>
Reviewed-by: Theresa  <twellington@chromium.org>
Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704483}

This code is disabled by default behind kDesktopPWAsWithoutExtensions and
kDesktopPWAsUSS base features.

- Implement WebAppSyncBridge::MergeSyncData and ApplySyncChanges.
- Implement "Local changes" propagation to change_processor.

Unit tests and browser tests will be added later in a follow up CL.
An MVP implementation of the bridge needed first.

This CL follows Chrome Sync's Model API doc
https://chromium.googlesource.com/chromium/src/+/HEAD/docs/sync/model_api.md

In next CLs:
Implement sync-initiated installs and uninstalls.

TBR=alancutter@chromium.org

Bug: 860583
Change-Id: I65d3cae6f47dd8fc8fdbe2c7cfb2ed13012f90f3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1830494Reviewed-by: Alexey Baskakov <loyso@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Commit-Queue: Alexey Baskakov <loyso@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704482}

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

Change-Id: I1f891f5d90d03a40c851f68067d9801305131e3e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1851444Reviewed-by: Chrome Release Bot (LUCI) <chrome-official-brancher@chops-service-accounts.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#704481}

The Arabic string for 'Refresh' changed, causing de-alignment issues.
Space on the right side is reduced to account for a longer string.

Initial -
screenshot: https://screenshot.googleplex.com/YTso9OyFquU

2nd patch -
screenshots:
  Normal Text:
  - https://screenshot.googleplex.com/auLs71d3Ops
  - https://screenshot.googleplex.com/YeMT1v0Wdgy
  Exaggerated long text:
  - https://screenshot.googleplex.com/BSJaTPBnGmb
  - https://screenshot.googleplex.com/Om5iuYQUfT1

Bug: 987195
Change-Id: Ib1de302b0eaab700270a5a1fd46abf9609fd7870
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1850412
Commit-Queue: Regan Hsu <hsuregan@chromium.org>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Reviewed-by: Xiaoqian Dai <xdai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704480}

This CL migrates ImeServiceTest with new Mojo types
using Remote. It replaces old Mojo types for InputChannel
with new Mojo types.

Bug: 955171
Change-Id: Ic9fb98c6342bd4d5338d731a7f08da099a5309a4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1849614Reviewed-by: Ken Rockot <rockot@google.com>
Reviewed-by: Shu Chen <shuchen@chromium.org>
Commit-Queue: Julie Kim <jkim@igalia.com>
Cr-Commit-Position: refs/heads/master@{#704479}

https://chromium.googlesource.com/angle/angle.git/+log/2fa9d73469ff..d7323cf7f12d

git log 2fa9d73469ff..d7323cf7f12d --date=short --no-merges --format='%ad %ae %s'
2019-10-10 jmadill@chromium.org Fix a warning with VS2019.
2019-10-09 jmadill@chromium.org Roll Chromium DEPS (October 2019).
2019-10-09 jmadill@chromium.org Vulkan: Get rid of "TextureVkViews".

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

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

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

FWIW: I tried using capturing (true as third arg to addEventListener)
and stopPropagation()/e.defaultPrevented. Because keypress seems to
be handled before click, it didn't make a difference. So I landed at
target filtering. This has the benefit of never navigating for
elements we don't add to this list, but with the detriment of needing
to explicitly add any new UI elements.

R=mahmadi@chromium.org

Fixed: 1012996
Change-Id: Ie36a90210eee8238d05d026bf370a8bc95160a2c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1851228Reviewed-by: Moe Ahmadi <mahmadi@chromium.org>
Commit-Queue: Moe Ahmadi <mahmadi@chromium.org>
Commit-Queue: Dan Beam <dbeam@chromium.org>
Auto-Submit: Dan Beam <dbeam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704477}

This reverts commit 43c06dab.

Reason for revert:

Findit (https://goo.gl/kROfz5) identified CL at revision 704455 as the
culprit for failures in the build cycles as shown on:
https://analysis.chromium.org/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtLzQzYzA2ZGFiNGU2YWYwOWY4MGQ0YWVmMjMwOTI2YzJiNzdiMGY4ZGUM

Sample Failed Build: https://ci.chromium.org/buildbot/chromium.linux/Cast%20Audio%20Linux/55207

Sample Failed Step: compile

Original change's description:
> Replace .mojom-blink.h with .mojom-blink-forward.h in blink - 11
> 
> To reduce pre-processed size of header files have been including
> foo.mojom-blink.h, this CL replaces .mojom-blink.h with
> .mojom-blink-forward.h in blink header files.
> 
> This CL has no behavior changes and following CLs will
> continue replacing them more.
> 
> Bug: 1001360
> Change-Id: I467c6ee86c44e581a6fee3cdcb8a5fdf6ed29aa9
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1847599
> Reviewed-by: Kent Tamura <tkent@chromium.org>
> Commit-Queue: Gyuyoung Kim <gyuyoung@igalia.com>
> Cr-Commit-Position: refs/heads/master@{#704455}


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

This CL converts MdnsResponder in mdns_responder.mojom using
SharedRemote and PendingRemote.

Bug: 955171,978694
Change-Id: Idd336a690d55e65817ec2a42bd173dc2b3dd59fc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1846217
Commit-Queue: Julie Kim <jkim@igalia.com>
Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Reviewed-by: Ken Rockot <rockot@google.com>
Cr-Commit-Position: refs/heads/master@{#704475}

The idea is to move autofill bubble showing logic out of browser window
to keep it short and clean. Also this extracted class can be a observer
for autofill data saving and notify avatar so that avatar itself does
not need to be such observers.

Also add avatar button accessor in the ToolbarButtonProvider.

Bug: 964127
Change-Id: I46f85f01b1da9d8d85d9049a1acd95ebe9bb2c21
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1845827
Commit-Queue: Siyu An <siyua@chromium.org>
Reviewed-by: Peter Boström <pbos@chromium.org>
Reviewed-by: Jared Saul <jsaul@google.com>
Cr-Commit-Position: refs/heads/master@{#704474}

This allows us to inject failures to the StartJob and StopJob
functions for better unit test coverage.

BUG=b:142144019
TEST=try

Change-Id: I17ebd3ddd74cd78bb7b123a7fda0a377d53963ab
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1850572Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org>
Commit-Queue: Yusuke Sato <yusukes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704473}

This reverts commit 3a54e6fa.

Reason for revert: Possibly no longer needed after
#703693 disabled g2gtest()s in cross builds in a different way.

Original change's description:
> Don't build js2gtest()s in win/cross builds.
> 
> Bug: 1010561
> Change-Id: Ic6b0d656af1944d9dbb16f5fb96d70fe1ad90f14
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1835157
> Auto-Submit: Nico Weber <thakis@chromium.org>
> Commit-Queue: Dan Beam <dbeam@chromium.org>
> Reviewed-by: Dan Beam <dbeam@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#702156}

TBR=thakis@chromium.org,dbeam@chromium.org

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

Bug: 1010561
Change-Id: I7b8fd3399786da5d51b827b846d2b03c72efb337
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1851085
Commit-Queue: Nico Weber <thakis@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704472}

As of cl/258693469, our code no longer uses ProcessText API, which has
been deprecated.

We delete the unused code.

Bug: 1009903
Change-Id: I4d920d6b147df086c01fb4d5e3c18f31b5ed2f5e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1833018Reviewed-by: Keith Lee <keithlee@chromium.org>
Reviewed-by: Shu Chen <shuchen@chromium.org>
Commit-Queue: Darren Shen <shend@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704471}

Currently, the value of ScrollLeft / ScrollTop / ScrollTo for a box in
Element is the offset to the origin of ScrollableArea(left-top corner).
This behavior isn't consistent with Document-scroll or the behavior of
other vendors either whose origin is ScrollOrigin. There're compatibility
problems when the box has leftward or upwards scroll overflow direction.
According to the Specification, the scroll x-coordinate of a leftward box
is nonpositive, and the scroll y-coordinate of an upwards box is also
nonpositive. With using the origin of ScrollableArea, the coordinate
is always nonnegative.

In order to fix it, this patch transforms the scroll coordinate of a
box in Element interface to use ScrollOrigin as its origin.

There are a few cases needed to recalculate the scroll coordinate to
meet this change. Since the origin of scroll coordinate transforms
from the ScrollableArea origin to ScrollOrigin(), current_coordinate
is equal to old_coordinate - ScrollOrigin. E.g.
current_scrollLeft = old_scrollLeft - ScrollOrigin().X().

This behavior is guarded by a feature flag.

See intent to ship blink-dev thread:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/7X2CKPGeEa0

Bug: 721759
Change-Id: I0ceed62e6845c6e5cd976e59b36f292d60bb669c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1700001Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: David Bokan <bokan@chromium.org>
Reviewed-by: Frédéric Wang <fwang@igalia.com>
Commit-Queue: cathie chen <cathiechen@igalia.com>
Cr-Commit-Position: refs/heads/master@{#704470}

Updating naming in response to https://crrev.com/c/1848912 and the
discussion on chromium-dev list.

Bug: 842296
Change-Id: I35c512a9139c993225ea95e9e345c9f5ca46f673
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1849208
Auto-Submit: Aaron Colwell <acolwell@chromium.org>
Commit-Queue: Alex Moshchuk <alexmos@chromium.org>
Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704469}

This CL deletes
AppBannerUiDelegateAndroid::OnNativeAppInstallStarted()
AppBannerUiDelegateAndroid::OnNativeAppInstallFinished()
which are unused as a result of http://crrev.com/c/1338337

BUG=1013004

Change-Id: I8d53baa1c8b8d324b18d89e5f0f84cc5c1c87f98
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1847673
Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org>
Reviewed-by: Dominick Ng <dominickn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704468}

This CL fixes some thread-safety issues in our video decoder tests. These might
be responsible for some rare crashes we've been seeing when running the tests on
our test infrastructure, but are not reproducible locally.

Most callbacks will be triggered on the same thread that scheduled these, but
this is not always the case. However WeakPtrs are not thread-safe, so this CL
introduces functors that guarantee that WeakPtrs are only dereferenced on the
thread that created them.

This CL also adds an additional sequence check to make sure events are sent on
the correct thread, and a comment is added about the use of base::Unretained for
the video player event callback.

TEST=./video_decode_accelerator_tests on samus

BUG=981718

Change-Id: If04268953f9be5dabc4ec11759fb3223f45055cd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1843998
Commit-Queue: David Staessens <dstaessens@chromium.org>
Reviewed-by: Chih-Yu Huang <akahuang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704467}

This CL converts IsolatedXRRuntimeProviderClientPtr and
mojo::Binding<T> in chrome to the new Mojo type, and use
pending_remote<T> in isolated_xr_service.mojom.

Bug: 955171
Change-Id: I2c186d27cb3c3c346504a61ec70839346a21b033
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1847706
Commit-Queue: Miyoung Shin <myid.shin@igalia.com>
Reviewed-by: Sam McNally <sammc@chromium.org>
Reviewed-by: Ken Rockot <rockot@google.com>
Reviewed-by: Klaus Weidner <klausw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704466}

This CL is a follow-up for r694827 (Set |request_initiator| to the
website, not to the content script's origin) which started selecting
Chrome-Extension-specific factories based on a (newly-introduced)
network::ResourceRequest::isolated_world_origin field (rather than the
old network::ResourceRequest::request_initiator field).  After this CL,
the |initiator_specific_factories| and similar names are no longer
accurate - this CL fixes this by changing such names to refer to
isolated worlds.

Ideally this CL would be part of r694827 - it is done as a separate CL
to 1) keep r694827 smaller / more focused and 2) make sure that r694827
"sticks" first.

Bug: 940068
Tbr: nasko@chromium.org, kinuko@chromium.org
Change-Id: I02c75e97dc456a3961043af2d783bfb3a8e54e24
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1835972
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Reviewed-by: Ben Wells <benwells@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704465}

We have to pause BLE scan before setting scan parameters. This CL
will make the API easier to use.

Bug: internal b/140961065
Test: Build and run, check HCI logs
Change-Id: Ia63b392ab09505d3d284d150d71b07d134d8a4e6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1850801Reviewed-by: Yuchen Liu <yucliu@chromium.org>
Commit-Queue: Tiansong Cui <tiansong@google.com>
Auto-Submit: Tiansong Cui <tiansong@google.com>
Cr-Commit-Position: refs/heads/master@{#704464}

win: Assert goma is not used with cl.exe, and stop setting symbol_level to 1 in non-clang goma builds.

We're removing support for MSVC on goma, so this combination of toggles
will be impossible soon. Assert it's unused, and remove a bit of code
I found that checked for this combination.

Bug: 1006238
Change-Id: Id00f887d37139262970d9d6a1a9c78fc18bb0bb5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1846674Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704463}

Bug: 850328
Change-Id: I7bc5fcd828f7f47230c4c967a28fb24525fd59b6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1838382
Commit-Queue: Yilkal Abe <yilkal@chromium.org>
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Reviewed-by: Michael Giuffrida <michaelpg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704462}

A page may not be submitting frames because blink isn't allowing it to
submit frames based on various states. In these cases, the browser
should not be showing UI indicating that the page is unresponsive.
This change adds plumbing for blink to inform the browser process that
it is the one throttling frame requests so that the browser can suspend
any timeout logic, and potentially (in the future) take over rendering
a more customized message.

Bug: 1009813
Change-Id: I2d1a3bb609eb305225493b67c630b32999735a6b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1841585Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Klaus Weidner <klausw@chromium.org>
Commit-Queue: Alexander Cooper <alcooper@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704461}

Replaces usage of DataPipe, which is deprecated, with CreateDataPipe.
There is no change in behavior, if the pipe fails to be created we will
crash.

Bug: 1012150
Change-Id: Ib077284f14c7232a0a78afcf0d1d3e1cc4f07380
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1846621Reviewed-by: Ken Rockot <rockot@google.com>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704460}

BUG=1005640

Change-Id: I441c389203be3c7eb1f06ddbdce09d3441677958
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1830483
Commit-Queue: Nancy Wang <nancylingwang@chromium.org>
Reviewed-by: Nigel Tao <nigeltao@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704459}

heuristic.

This CL adds a set of the top 500 keywords to the binary at compile
time, to enable usage of this information in the phishing detection
keywords heuristic. Additionally, this CL cleans up and refactors the
compile-time processing and embedding of information related to the top
500 keywords, into the binary.

Bug: 1012476
Change-Id: I9d75d962d0425b0f037c87a5d6801c7c9299c476
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1849183
Commit-Queue: Aidan Beggs <beggs@google.com>
Reviewed-by: Mustafa Emre Acer <meacer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704458}

NativeFileSystem uses RenderFrameHost::IsCurrent which is problematic
for bfcache, as it means that for some frames the permissions might be
requested while the frame is in back-forward cache (IsCurrent is true
for iframes) and for some the requests will be denied and will not be
resumed after restoring from back-forward cache.

Disable back-forward cache as soon as we make a request for now.

Some background on why we sometimes need to disable bfcache:
https://docs.google.com/document/d/1NjZeusdS1kyEkZyfLggndU1A6qVt0Y1sa-LRUxnMoK8

R=mek@chromium.org
CC=bfcache-bugs@chromium.org
BUG=1001087

Change-Id: I6a1f0153df4c13fbaba900af049e0c852d340513
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1837813Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
Commit-Queue: Alexander Timin <altimin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704457}

This change adds basic tests that capture current behavior around the
Audio Context Selector.

It is meant as a start and not a complete list of tests,
more tests should be added in the future.

This also indirectly tests the UI.ListModel, UI.SoftDropDown, and
UI.ToolbarItem.

Change-Id: I7506c06e6cf8b865ab2645c9e9e4004df162e629
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1824060Reviewed-by: Yang Guo <yangguo@chromium.org>
Reviewed-by: Hongchan Choi <hongchan@chromium.org>
Reviewed-by: Mandy Chen <mandy.chen@microsoft.com>
Commit-Queue: John Emau <John.Emau@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#704456}

To reduce pre-processed size of header files have been including
foo.mojom-blink.h, this CL replaces .mojom-blink.h with
.mojom-blink-forward.h in blink header files.

This CL has no behavior changes and following CLs will
continue replacing them more.

Bug: 1001360
Change-Id: I467c6ee86c44e581a6fee3cdcb8a5fdf6ed29aa9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1847599Reviewed-by: Kent Tamura <tkent@chromium.org>
Commit-Queue: Gyuyoung Kim <gyuyoung@igalia.com>
Cr-Commit-Position: refs/heads/master@{#704455}

Bug: 692909
Change-Id: If4c656288728f6c52440fc4b2e834e3e2b6e8c61
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1846611Reviewed-by: Jesse Doherty <jwd@chromium.org>
Commit-Queue: Kenichi Ishibashi <bashi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704454}

https://chromium.googlesource.com/chromiumos/chromite.git/+log/5ee5b8853db6..992c5b481bab

git log 5ee5b8853db6..992c5b481bab --date=short --no-merges --format='%ad %ae %s'
2019-10-09 engeg@google.com buildbucket_api: Add retry to GetBuild and extend sleep duration

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

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: Ifbc28622327883953b1353a19241e44bcb2626c6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1849635Reviewed-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@{#704453}

Bug: 1012387
Change-Id: I9627134583db5f0b0c8390ccf4792b5287a5a0b4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1847869Reviewed-by: Mason Freed <masonfreed@chromium.org>
Reviewed-by: Kent Tamura <tkent@chromium.org>
Commit-Queue: Sanket Joshi <sajos@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#704452}

When the dropdown is opened, the entire element should appear as a
single element, with no space between the input field and the opened
dropdown.

http://go/siygs

BUG=977313

Change-Id: I81912cb7f6101a8fc8a54f60a4095e42a210ca8b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1847703
Commit-Queue: Anand Mistry <amistry@chromium.org>
Reviewed-by: Michael Giuffrida <michaelpg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704451}

This reverts changes from:

CL:1808426

which caused the ripple circle around the page switcher button to get cut off.

Bug: 1011518
Change-Id: I5da49ce4122cdb5dbf083eda6a8f23ec371f5946
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1846012Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Reviewed-by: Toni Baržić <tbarzic@chromium.org>
Commit-Queue: Yulun Wu <yulunwu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704450}

Change-Id: Ie673d63862039d35a1c769db62c5d59562d4fde9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1850725
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704449}