GitLab

images/color-profile-image-filter-all.html,
images/yuv-decode-eligible/color-profile-filter.html, and
images/yuv-decode-eligible/color-profile-layer-filter.html have minor
visual differences due to blur, so add them to crbug.com/972621.

Some of the tests have minor visual differences due to sharpness;
rebaseline and enable them as per the discussion in crbug.com/972637.

Bug: 954328
Change-Id: I31b597899190269ab09130aaf5bbd2cdb7d9caf5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1757264Reviewed-by: Jonathan Ross <jonross@chromium.org>
Commit-Queue: Sean Gilhuly <sgilhuly@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689698}

Bug: 994501
Change-Id: Ic41a4f1f4a314e6343dc3bb21bd929e96a83e5ac
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1763302Reviewed-by: Peter Boström <pbos@chromium.org>
Commit-Queue: Charlene Yan <cyan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689697}

No functional change, just an API tweak for shorter code.

Bug: 709130
Change-Id: I042e2b3c2bb295d4d045cdc21bc907b25d17f318
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764257Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Auto-Submit: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689696}

Bug: 992711
Change-Id: I1ffd1948ce606d8c37630a38659db4bf8eedc1d3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764469
Commit-Queue: Yann Dago <ydago@chromium.org>
Reviewed-by: Owen Min <zmin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689695}

- LayerTreeHostCommonTestWithLayerLists: test class for future tests
  in layer list mode in layer_tree_host_common_unittest.cc

- GetXXXNode() for easy getting existing property nodes from layers

- Bug fixes and improvements of property_tree_test_utils

Bug: 994361
Change-Id: Icfca2ffae847ce97ba4b2dfaa831d6241783e867
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1766100Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689694}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/7f90416c08be..c13c5f67a13f

git log 7f90416c08be..c13c5f67a13f --date=short --no-merges --format='%ad %ae %s'
2019-08-22 ehmaldonado@chromium.org depot_tools: Ignore python3 files generated by windows bootstrap.

Created with:
  gclient setdep -r src/third_party/depot_tools@c13c5f67a13f

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

Bug: None
Change-Id: I0e150cfb4cc2eaa7d08ceea7e20bd1c391f8b517
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764907
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#689693}

This keeps track of changes to the local network stack while waiting
for Shill to save them.  These pending changes are persisted to disk
in a pref so they can be retried on the next login if needed.

Bug: 966270
Change-Id: If3cf00873a4fc142b755e005da561a3cbb78a73f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761434Reviewed-by: Marc Treib <treib@chromium.org>
Reviewed-by: Gabriel Charette <gab@chromium.org>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Commit-Queue: Jon Mann <jonmann@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689692}

This is needed for the OptGuideKeyedService implementation to come where
it needs to sort of check the existence of the hint before proceeding
via CanApplyOptimization which takes in a nav handle

Bug: 969558
Change-Id: Idd4abf9e6b2d8b9ddb5ccb2e304aa9e5defe5011
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1766094
Commit-Queue: Sophie Chang <sophiechang@chromium.org>
Reviewed-by: Robert Ogden <robertogden@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689691}

This reverts commit 1e76b45f.

Reason for revert:

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

Sample Failed Build: https://ci.chromium.org/buildbot/chromium.memory/Linux%20TSan%20Tests/43203

Sample Failed Step: content_unittests

Original change's description:
> [scheduler] Registering the executor in TLS.
> 
> Bug: 968047
> Change-Id: Idaf9c72f13419ac29804a91e54be86d71ff9b7f7
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1751251
> Commit-Queue: Karolina Soltys <ksolt@chromium.org>
> Reviewed-by: Karolina Soltys <ksolt@chromium.org>
> Reviewed-by: Ted Choc <tedchoc@chromium.org>
> Reviewed-by: Alex Clarke <alexclarke@chromium.org>
> Auto-Submit: Karolina Soltys <ksolt@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#689646}


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

ConnectivityProbingManager::OnConnectivityProbingReceived is actually
called on every single packet received, even if the packet is not a
connectivity probe. Renamed to OnPacketReceived to reflect the fact
correctly.

Bug: 996419
Change-Id: Icb01b5d96864bc0dd331509964f1527b52b48db2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764263
Commit-Queue: Zhongyi Shi <zhongyi@chromium.org>
Reviewed-by: Ryan Hamilton <rch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689689}

Issue:
The confirm infob bar will contain an invisible link when there is no actual
link. NVDA focus is going to invisible links in the info bars.
This change includes a fix to set the accessibility role of link to
kIgnored when it's empty and invisible.

Bug: 993608
Change-Id: I14d6149ec494353cb26fbebfaffa0dd9d51d1f2f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1752116
Commit-Queue: Drake Lei <jilei1@microsoft.com>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689688}

This also avoids collapsing whitespace between words when pasting into the find
box.  We still trim leading/trailing whitespace since it's often unintentional.

This fails to find newlines due to block boundaries, e.g. foo<br>bar; it only
finds newlines in the actual text strings, e.g. foo\nbar.  Finding the former
will require more substantial changes to the find code.

Bug: 973253
Change-Id: Ife0388658cd23950ba958d74d38e0d8036f99e41
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764889
Auto-Submit: Peter Kasting <pkasting@chromium.org>
Commit-Queue: Michael Wasserman <msw@chromium.org>
Reviewed-by: Michael Wasserman <msw@chromium.org>
Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689687}

Fixed clobber bots failure by specifying all domains in the protocol config file.
Without this change, builds that build from scratch will fail to generate the domain output
files and will fail (clobber builds).

This is a reland of 591bf020

Original change's description:
> UI_DevTools: Backend for ui_devtools performance panel.
>
> The goal is to enable and create the performance tool for ui_devtools.
> The performance panel will record and show metrics such as FPS, CPU time utilization
> for Aura/Views UI. This CL is for the backend only. The TracingAgent is very similar
> to the TracingHandler from
> https://cs.chromium.org/chromium/src/content/browser/devtools/protocol/tracing_handler.cc.
>
> Bug: 989052
> Change-Id: Ic696e09f700e93bcd0842fdbda575ef1f54af63a
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1726651
> Commit-Queue: MyungJae Lee <myungjaelee@chromium.org>
> Reviewed-by: Ken Rockot <rockot@google.com>
> Reviewed-by: Dana Fried <dfried@chromium.org>
> Reviewed-by: oysteine <oysteine@chromium.org>
> Reviewed-by: Leonard Grey <lgrey@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#689296}

Bug: 989052
Change-Id: I6392c40d5babcd74558186c80ebc618c22aa4d7b

This is a reland CL and the only file difference is
components/ui_devtools/inspector_protocol_config.json, which has been reviewed by ui_devtools
OWNER lgrey@.

TBR=rockot@google.com,oysteine@chromium.org,dfried@chromium.org

Change-Id: I6392c40d5babcd74558186c80ebc618c22aa4d7b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765005
Commit-Queue: MyungJae Lee <myungjaelee@chromium.org>
Reviewed-by: Leonard Grey <lgrey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689686}

"Speed tracks" was confusing to me. I think these are really
just domain areas of expertise.

Also, add Bruce as primary contact for power domain.
Also, clarify difference between end user metrics and
lab metrics.
Also, rename waterfall from chromium.perf to chrome.perf.

Bug: 949246
Change-Id: I0739912ef38a6c9381d19d265cb02675932e636c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1763097
Commit-Queue: Caleb Rouleau <crouleau@chromium.org>
Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689685}

The dimensions are still not finalized, but these are at least closer
to the mocks and look more polished.

Bug: 989131
Change-Id: I0b347f57bc654011af3b0d1087c908b0d911da4a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1766350
Commit-Queue: John Lee <johntlee@chromium.org>
Reviewed-by: Peter Boström <pbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689684}

https://skia.googlesource.com/skia.git/+log/2b39cbe8619a..2a5954140b49

git log 2b39cbe8619a..2a5954140b49 --date=short --no-merges --format='%ad %ae %s'
2019-08-22 egdaniel@google.com Merge GrOpList and GrRTOpList and rename to GrOpsTask.
2019-08-22 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial).
2019-08-22 fmalita@chromium.org [svgcanvas] Named colors support
2019-08-22 egdaniel@google.com Delete unused GrTextureOpList.
2019-08-22 fmalita@chromium.org [canvaskit] Optimize CK builds for size
2019-08-22 csmartdalton@google.com Don't perform dst reads on MSAA targets that require manual resolve

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

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

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

Previously, we had compiled a set of regexes to handle doc URL patterns
we had encountered. In order to reduce the likelihood of error, this CL
instead uses the same regex used in the google3 codebase.

Change-Id: I00bde49c67d99592bc085d6d0d1a90324cb7f84c
Bug: 888093
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1700333
Commit-Queue: manuk hovanesian <manukh@chromium.org>
Reviewed-by: Justin Donnelly <jdonnelly@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689682}

No change to product, only OWNERS changes.

I'm proposing myself for ownership of:

 * Markdown files: I've been owning aw/docs/ for a while, and feel
   comfortable owning markdown files outside of that folder as well
 * Build files: I'm fairly comfortable with GN, and would like to help
   facilitate the project in crbug.com/934152.

Bug: 918221, 934152
Test: N/A
Change-Id: Ib42d76763764ab4ef37ce93ca2eebbf94eeb9954
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764828Reviewed-by: Changwan Ryu <changwan@chromium.org>
Commit-Queue: Nate Fischer <ntfschr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689681}

This is a revert of https://chromium-review.googlesource.com/c/chromium/src/+/1340875.

This is possible now that BGPT is launched.

Bug: 900602,909036

Change-Id: I750deb202dd9fd2478e955ea243f85b14c59dde6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1763335
Commit-Queue: Chris Harrelson <chrishtr@chromium.org>
Reviewed-by: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689680}

This was a (rather childish) attempt at resetting the fragment builder
sufficiently if the column fragments we had created had to be discarded,
because of column balancing.

This approach won't work once we add support for column-span:all,
because we'll then perform individual column balancing for each column
row (between or around the spanners).

Change-Id: Idaa1f702c4bba688cec222d76dabc3e12214f5ed
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1766069Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Reviewed-by: Christian Biesinger <cbiesinger@chromium.org>
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689679}

Bug: 996469
Change-Id: I002033132524ed862df8b99c11cf147de0d1a2cb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764754Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Danan S <danan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689678}

Navigation with the VO cursor should not visit hidden elements.
This had regressed because we started including hidden elements
in the accessible object hierarchy on purpose, so that when
a page accidentally overused aria-hidden, that users would
still hear something useful when tabbing to focusable items.
With this CL, users will not generally be able to accidentally
navigate to aria-hidden items, but if they manage to actually
focus one with the keyboard, it will still announce what it is.

Bug: 982408
Change-Id: I19d5fe4947e5fb1c7493f0e67d7fe8e13d9a255e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761717Reviewed-by: Alice Boxhall <aboxhall@chromium.org>
Commit-Queue: Aaron Leventhal <aleventhal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689677}

For scans that return synchronously, the verdicts will be present in
the initial upload response. This CL checks that response for verdicts
and adds a test for this behavior.

Bug: 980784
Change-Id: I3f7c17f293d66f6aa410819f453b4a24f337e3c0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764408
Commit-Queue: Daniel Rubery <drubery@chromium.org>
Auto-Submit: Daniel Rubery <drubery@chromium.org>
Reviewed-by: Roger Tawa <rogerta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689676}

The VizDisplayCompositor is default enabled and can't be disabled on
Android. Remove the entry in fieldtrial_testing_config.json and remove
from chrome://flags.

Bug: 936425
Change-Id: Iaf8539c36e3d30789cbcf2031c07c25f3bb8bb07
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761558
Auto-Submit: kylechar <kylechar@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Commit-Queue: kylechar <kylechar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689675}

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

https://quiche.googlesource.com/quiche.git/+log/0b20fdfd4bea..c8c02a5f788c

$ git log 0b20fdfd4..c8c02a5f7 --date=short --no-merges --format='%ad %ae %s'
2019-08-22 renjietang Make sure QPACK stream types are sent along with SETTINGS in one packet.
2019-08-22 renjietang Optimize code in handling wrong HTTP/3 frames.
2019-08-22 bnc Remove QpackProgressiveDecoderTests.
2019-08-22 bnc Signal error on duplicate SETTINGS identifiers.
2019-08-22 bnc Make QpackEncoder::EncodeHeaderList() and FirstPassEncode() take header_list by const ref.

Created with:
  roll-dep src/net/third_party/quiche/src src/third_party/quic_trace/src

Change-Id: Ie9e7cef88064b1bef449835089555449f61ef1ad
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765999
Commit-Queue: Ryan Hamilton <rch@chromium.org>
Commit-Queue: Nick Harper <nharper@chromium.org>
Auto-Submit: Ryan Hamilton <rch@chromium.org>
Reviewed-by: Nick Harper <nharper@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689673}

Enter does not automatically trigger an onclick event and needs to be
explicitly handled.

Also move enter/space handling to keyup for shortcut options to match
the default behavior of space on the hide toggle (which triggers on
keyup).

Bug: 996649
Change-Id: Ib8886aa7f05907108b316b512464b429e820ba71
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764884Reviewed-by: Kyle Milka <kmilka@chromium.org>
Commit-Queue: Kristi Park <kristipark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689672}

The cc layer compositor is setting |is_fast_rounded_corner| and the
value is being read by SurfaceAggregator. However, SharedQuadState is
typically sent over IPC. Since |is_fast_rounded_corner| isn't
serialized it's always false by the time it arrives in SurfaceAggregator
if originating in another process.

This caused a performance regression with VizDisplayCompositor enabled
on Chrome OS. Browser UI CompositorFrames started being serialized and
we were no longer using the fast rounded corner rect optimization for
windows in tablet overview mode.

Bug: 987684, 996422
Change-Id: I2b12cbfae30846ef8fe0bb3cc5394798e899a069
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1763434
Commit-Queue: kylechar <kylechar@chromium.org>
Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Reviewed-by: Malay Keshav <malaykeshav@chromium.org>
Reviewed-by: Will Harris <wfh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689671}

This change moves ArcInstanceThrottle and ArcThrottleObserver
to chrome/browser/chromeos/arc/instance_throttle
Bug:b:133417350
Test: Build and flash to eve. Monitor /var/log/chrome/chrome to see throttling de/activate according to the different conditions.

Change-Id: Ied98988b65bc6a357601c90935e513fe276b6e12
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1762780
Commit-Queue: Willie Koomson <wvk@google.com>
Reviewed-by: Yury Khmel <khmel@chromium.org>
Reviewed-by: Yusuke Sato <yusukes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689670}

Bug: internal b/135576340
Test: Build and run
Change-Id: Iae347132d9302488a832902f38b6be8a4168783e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1762286Reviewed-by: Yuchen Liu <yucliu@chromium.org>
Commit-Queue: Tiansong Cui <tiansong@google.com>
Cr-Commit-Position: refs/heads/master@{#689669}

This CL adds the target context to the event's will_dispatch_callback
method so that the TabEventRouter can decide what level of scrubbing is
needed for the Tab data it dispatches. In a target context of WebUI,
no scrubbing is needed.

Actual usage of the context param will come in a follow-up CL.

Bug: 994337
Change-Id: I55d7ff66b1353cb6df34931f209f3e68a339cace
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1763090
Commit-Queue: John Lee <johntlee@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689668}

Chrome OS Printing dialog allows users to add additional accounts for
finding more printers.

When Chrome OS Account Manager is available on Chrome OS, we will need
to display native account management UIs, instead of Gaia webpages.

Bug: 982641
Change-Id: I42238967d3e0ee2bbee499797b69bcc42b01d6c6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1763413Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Kush Sinha <sinhak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689667}

This reverts commit 1863eb4a.

Reason for revert: breaks win32-archive-rel build: https://ci.chromium.org/p/chromium/builders/ci/win32-archive-rel/4104

Original change's description:
> Enable building OpenXR by default in Windows.
> 
> Bug: 989254
> Change-Id: I8f2ad0214cc17c324774e08ee34baba87dae7f42
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1745502
> Commit-Queue: Patrick To <patrto@microsoft.com>
> Reviewed-by: Alexander Cooper <alcooper@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#689609}

TBR=ddorwin@chromium.org,rafael.cintron@microsoft.com,patrto@microsoft.com,alcooper@chromium.org

Change-Id: I897e1e6ada7e3c35fcc0377fc428a717c6db7aae
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 989254
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1766453Reviewed-by: Mike Wittman <wittman@chromium.org>
Commit-Queue: Mike Wittman <wittman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689666}

This is a reland of d7b74986

As usual, if this fails **please** paste the build failure, not just
silent revert, logs can be hard to find in retrospect.

Reason for reland:
 - joenotcharles@ fixed the windows build in
   https://chromium-review.googlesource.com/c/chromium/src/+/1764471
 - google chromeos build failure fixed in
   https://chromium-review.googlesource.com/c/chromium/src/+/1764256

Original change's description:
> Reland "Reland [base] Require task posters to specify an explicit destination"
>
> This is a reland of dd34a877
>
> Rebased on a few more fixes including a local full build of:
>  - Windows + is_chrome_branded
>  - ChromeOS + is_chrome_branded
>    (except for the libassistant portion which is giving weird errors...)
>
> Original change's description:
> > Reland [base] Require task posters to specify an explicit destination
> >
> > This patch makes it a requirement for all task traits to specify their
> > thread affinity. Previously task traits that did not name a specific
> > thread would implicitly run on the thread pool, which could be
> > surprising. To make this more obvious, thread pool tasks must now use
> > the base::ThreadPool() trait.
> >
> > This is the final reland of
> > https://chromium-review.googlesource.com/c/chromium/src/+/1635827
> > (will attempt to fix errors that caused original revert before
> >  relanding).
> >
> > Bug: 968047
> > Change-Id: Ice339db270a27815b2be68200930e5bc105e8a96
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1759056
> > Commit-Queue: Gabriel Charette <gab@chromium.org>
> > Reviewed-by: François Doray <fdoray@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#688782}
>
> Bug: 968047
> Change-Id: Iccd12e0a9f1535cb10b903777c5004a7d1acb34a
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761362
> Commit-Queue: Gabriel Charette <gab@chromium.org>
> Reviewed-by: François Doray <fdoray@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#689097}

Bug: 968047
Change-Id: Ie338781aba9b5f5990ffe74e02bb85f08fe683d1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764398
Commit-Queue: Gabriel Charette <gab@chromium.org>
Reviewed-by: François Doray <fdoray@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689665}

Bug: 996870
Change-Id: If8c35b69d94302fbc45db4e0d43a5aed4c2b1329
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1766351Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Commit-Queue: Robert Ogden <robertogden@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689664}

We should raise exceptions for expectations where the tag condition list mixes mobile and
desktop tags.

Bug: chromium:973936
Change-Id: I52395ead03afb022063bf565f6c1e742cb5f4086
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1754982
Commit-Queue: Rakib Hasan <rmhasan@google.com>
Reviewed-by: Juan Antonio Navarro Pérez <perezju@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689663}

https://android.googlesource.com/platform/external/perfetto.git/+log/2891cbc02f12..a52177c11e72

git log 2891cbc02f12..a52177c11e72 --date=short --no-merges --format='%ad %ae %s'
2019-08-22 primiano@google.com Merge "Client API: allow to configure SMB size and page size"
2019-08-22 primiano@google.com Merge "Fix chromium roll (INSTANTIATE_TEST_CASE_P/SUITE)"
2019-08-22 treehugger-gerrit@google.com Merge "Temporarily disable ConcurrentProfile test."
2019-08-22 nicomazz@google.com Fix perfetto's extension listener registration
2019-08-22 primiano@google.com Merge "Fix signed-unsigned conversions"
2019-08-22 treehugger-gerrit@google.com Merge "Add support for Dvorak keyboard layout"
2019-08-22 treehugger-gerrit@google.com Merge "client api: Introduce a bare-bones TrackEvent library"
2019-08-22 primiano@google.com Merge "Speculative fix for Windows build failure"
2019-08-22 treehugger-gerrit@google.com Merge "Removing useless permissions to the extension"
2019-08-22 oysteine@google.com Merge "Separate out the parts of HeapProfiling that CPUProfiling will reuse into StackProfiling"
2019-08-22 lalitm@google.com Merge "trace_processor: add Table and Column classes"
2019-08-21 treehugger-gerrit@google.com Merge "[GPU TRACE] Add UI control to record GPU sched trace events"
2019-08-21 lalitm@google.com trace_processor: add SparseVector implementation and tests
2019-08-21 lalitm@google.com trace_processor: add RowMap implementation and tests
2019-08-21 lalitm@google.com trace_processor: add bitvector implementation
2019-08-21 treehugger-gerrit@google.com Merge "[GPU TRACE] Add GPU scheduling trace events"
2019-08-21 primiano@google.com Merge "Remove use of -isystem for googletest"

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

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

Bug: chromium:139788095,chromium:135177627,chromium:135177627,chromium:135177627
Change-Id: I0c5963ea1b2a6e433ad3a16eab5b76fa4bc870f4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765834Reviewed-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@{#689662}

Update the case of bubble banner's accept button. Also, use the
proper test survey's site id as the default value.

BUG=996547

Change-Id: Id5e2c0ba2512eda171f82af4d4b7c159b89ac573
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1766002
Commit-Queue: Wei Li <weili@chromium.org>
Commit-Queue: Robert Liao <robliao@chromium.org>
Reviewed-by: Robert Liao <robliao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689661}

Migrate base::fuchsia::ServiceDirectory to sys::OutgoingDirectory and
sys::ServiceDirectory because functionality in base::fuchsia is now
in the sys_cpp package in the Fuchsia SDK.

Both the source code and unit tests are updated.

Bug: 974072
Change-Id: I505da402f99f8b1beeec3084b8b1c5502b9ac0a9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1758835Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Commit-Queue: Sharon Yang <yangsharon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689660}

Bug: 934152
Test: gn check out/Default //android_webview/renderer
Change-Id: I6d192ae647e048e2222d2405b5a7237052212385
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764678
Commit-Queue: Robbie McElrath <rmcelrath@chromium.org>
Reviewed-by: Nate Fischer <ntfschr@chromium.org>
Reviewed-by: Changwan Ryu <changwan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689659}