GitLab

https://skia.googlesource.com/skia.git/+log/108a494984da..02497d40161f

git log 108a494984da..02497d40161f --date=short --first-parent --format='%ad %ae %s'
2020-02-24 egdaniel@google.com Update nanobench and skpbench to use flush API for gpu syncing.
2020-02-24 kjlubick@google.com [canvaskit] Expose releaseResourcesAndAbandonContext on GrContext
2020-02-24 westont@google.com Update tasks.json based on Go Deps Cl.
2020-02-24 bsalomon@google.com Prevent unbounded invalidator growth on SkPathRef from GrSWPathRenderer
2020-02-24 skia-autoroll@skia-public.iam.gserviceaccount.com Roll skia/third_party/skcms 605b00d7728f..cbf3b9df1ae1 (1 commits)
2020-02-24 csmartdalton@google.com Lift calls for setting dynamic state to Ganesh level
2020-02-24 csmartdalton@google.com Remove "setData" operations from GrMtlPipelineState::setTextures

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

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/skia-autoroll
Please CC jlavrova@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-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
Bug: None
Tbr: jlavrova@google.com
Change-Id: I164f72adcbd657fb903e0c726db4fe6b1ef0065b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2070432Reviewed-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@{#744010}

This cl adds the new fields display_globally and display_countries.

Bug: 1040145
Change-Id: Ice70d5c44537950420c9aeca822c652919d6298d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2067561Reviewed-by: Eric Orth <ericorth@chromium.org>
Commit-Queue: Katharine Daly <dalyk@google.com>
Cr-Commit-Position: refs/heads/master@{#744009}

Remove the asynchronous initialization which was not useful, and add a
method to get the output (decoded) audio config.

Merge-With: eureka-internal/367396

Bug: internal b/148685511
Change-Id: I050b97f1b55921249c0fab634a945fb9e2ac595e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2069274Reviewed-by: Yuchen Liu <yucliu@chromium.org>
Commit-Queue: Kenneth MacKay <kmackay@chromium.org>
Cr-Commit-Position: refs/heads/master@{#744008}

ReleaseAcceleratedWidget() calls RemoveCompositor() which removes an
entry from |compositor_data_map_|. Instead of iterating over this map,
first copy the compositors to a new set.

Bug: 1042259, 1055223
Change-Id: I65e7c4255b6f0a5305667c13aab404c5eddfd92d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2070102Reviewed-by: kylechar <kylechar@chromium.org>
Commit-Queue: Sean Gilhuly <sgilhuly@chromium.org>
Cr-Commit-Position: refs/heads/master@{#744007}

Bug: None
TBR=kbr@chromium.org

Change-Id: Ie0e3c511480a434fb7d0e24e6cf76d3f76a98bb9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2066932Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#744006}

Fixed: 1054864
Change-Id: I41291fb5f83ac8af038d657544f394fba4bcbbc4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2068737
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Reviewed-by: Emily Stark <estark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#744005}

It will be easier to make changes to the extraction code here without
affecting the AppIndexing feature in Clank. We will be using extraction
for data from Media Feeds as explained here:
https://github.com/beccahughes/media-feeds/blob/master/explainer.md

Bug: 1053598
Change-Id: Ibf5f05005fd301aed12a4814ac602ab06700f1e5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2067612Reviewed-by: Becca Hughes <beccahughes@chromium.org>
Commit-Queue: Sam Bowen <sgbowen@google.com>
Cr-Commit-Position: refs/heads/master@{#744004}

In cases where drawing additional smaller quads instead of one large
quad is significantly less costly, split one quad into multiple quads.

In order to minimize the cost of drawing additional quads and resizing the quad list
we limit the when quads may be split. We limit the number of quads one
quad may be split into and enforce a minimum number of fragments saved
by splitting.

Test: viz_perftests,viz_unittests
Bug: 1022544
Change-Id: Id47c0a5b33b0de456fad10635c86b880882fb549
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2057289Reviewed-by: Maggie Chen <magchen@chromium.org>
Reviewed-by: ccameron <ccameron@chromium.org>
Reviewed-by: Khushal <khushalsagar@chromium.org>
Reviewed-by: Daniele Castagna <dcastagna@chromium.org>
Commit-Queue: Sasha McIntosh <sashamcintosh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#744003}

Bug: 1012975, 1018268
Change-Id: I4802f7576907fcfbe124e2675f98e30644c9eeba
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2068928Reviewed-by: Theresa  <twellington@chromium.org>
Commit-Queue: Henrique Nakashima <hnakashima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#744002}

Otherwise this leads to crashes in VTDecompressionSessionCreate.

The CL http://crrev.com/c/2013806 changed the order but
the intention was to fix an issue on Linux/Wayland. So
restore the previous behavior on Mac. Also restore the
previous behavior on other operating systems to make the
change safer. And guard the new order with OS_LINUX.

Bug: 1047643
Change-Id: I9637a684996ee095d32a336610cb65a7d0d20370
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2068207Reviewed-by: Maggie Chen <magchen@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Julien Isorce <julien.isorce@chromium.org>
Cr-Commit-Position: refs/heads/master@{#744001}

This reverts commit fafef524.

Reason for revert: Should be enough space now after crrev.com/c/2065419

Original change's description:
> Temporarily remove net_unittests from chromeos-amd64-generic-rel.
> 
> To unblock LKGM rolls. Will still run on the betty bot in the meantime.
> 
> Bug: 1053547
> Change-Id: Ifb2ffc7f06a6183265209d5e06c63a16215b1316
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2065414
> Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
> Commit-Queue: Ben Pastene <bpastene@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#742909}

TBR=stevenjb@chromium.org,bpastene@chromium.org,malaykeshav@chromium.org

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

Bug: 1053547
Change-Id: I5892a41f300425b1dd97730bad2fee72d8279870
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2067615Reviewed-by: Ben Pastene <bpastene@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#744000}

When I initiailly created this hack in [1], the intention was to always
use fcntl() over fcntl64(), however the hack was written to invert the
_FILE_OFFSET_BITS=64, but not when it's set to a value like 32, as is
the case in some projects.

[1] https://chromium-review.googlesource.com/c/chromium/src/+/1862623

BUG=b/147463741
R=thakis

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

This will make it easier to move these into //components/permissions.

Bug: 1025609
Change-Id: Ic0745279a303c447beac0c3072dd59e494a15d1f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2065620
Commit-Queue: Clark DuVall <cduvall@chromium.org>
Reviewed-by: Andy Paicu <andypaicu@chromium.org>
Reviewed-by: Matthew Jones <mdjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743998}

Doing so is a bit surprising, and makes local development not match what
users see.

Bug: 1027123
Change-Id: I5f05de15b75c0ba0bf5d022ae9be50405415df99
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2069544
Auto-Submit: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: Peter Wen <wnwen@chromium.org>
Commit-Queue: Peter Wen <wnwen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743997}

There are four tests that are flaky on the  Win10 FYI x64 SkiaRenderer
GL (NVIDIA) bot. The failures might be related to running the test on a
real GPU so just mark them as flaky for now.

Bug: 1026375
Change-Id: I825873b70f4ad7a2cffcfa75a30da635c144b248
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2070101Reviewed-by: Jonathan Ross <jonross@chromium.org>
Commit-Queue: Jonathan Ross <jonross@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743996}

This reverts commit 52dbe8bb.

Reason for revert: that broke navigation to facebook.com, see bug for details.

BUG=1055450

Original change's description:
> Move WebInsecureRequestPolicy values to mojom
> 
> This CL is a precursor to converting
> FrameMsg_EnforceInsecureRequestPolicy to blink RemoteFrame mojom.
> It moves WebInsecureRequestPolicy values to mojom to use the values
> in content and blink and removes WebInsecureRequestPolicy.
> 
> Bug: 1047284
> Change-Id: If030952dbd0c9e98bb0c4a11479ea175ade91fc5
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2045293
> Commit-Queue: Julie Kim <jkim@igalia.com>
> Reviewed-by: Kentaro Hara <haraken@chromium.org>
> Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
> Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#743880}

TBR=kinuko@chromium.org,haraken@chromium.org,dtapuska@chromium.org,jkim@igalia.com

Change-Id: I6885d524a41b21fbdb0066b2cc2513408fe15943
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1047284
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2070109Reviewed-by: Andrey Kosyakov <caseq@chromium.org>
Commit-Queue: Andrey Kosyakov <caseq@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743995}

Right now it opens OS settings, which has the same set of sync options
as browser settings. Soon the OS sync settings page will be different,
and we want to send users to the browser settings page either way.

This also helps if a user checks both "Review sync options" and
"Review Google Play options" during OOBE. They will see two windows,
OS settings and browser settings.

Making syncSetup a browser settings, rather than OS settings, subpage
requires changing ARC++ URL handling in ChromeNewWindowClient for
mojo enum ChromePage::SYNCSETUP. I think this is OK, because it's
closer to the original ARC++ behavior before split settings, and
also I can find no usage of the SYNCSETUP code path in the ARC++
code (or anywhere else in google3).

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

Bug: 1054980, 901184
Change-Id: Ide777363aa83e9a7b0f87068c318bd1f274c7e2e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2068948Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743994}

https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/7f1834b06ad1..207c622c1ef0


Created with:
  gclient setdep -r src-internal@207c622c1ef0

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
Bug: None
Tbr: jbudorick@google.com
Change-Id: I8f6c81898d87d0d4080fb0a4b1d4cd264c69acd9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2070430Reviewed-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@{#743993}

Both UMA metrics Canvas.Repaint.Bounds.Percentage and
Canvas.Repaint.Region.Percentage will be expired in the upcoming version.
Since the partial update study is not finished, extend the expiry date
to M84.

Bug: 1014246
Change-Id: I3ea893a04702382ed7897ae7660541e5eeeef153
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2067812Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Commit-Queue: Yi Xu <yiyix@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743992}

This reverts commit 99fe56da.

Reason for revert: depends on https://chromium-review.googlesource.com/c/chromium/src/+/2045293 that broke navigation to facebook.com, see bug for details.

BUG=1055450

Original change's description:
> Convert FrameMsg_EnforceInsecureRequestPolicy to blink RemoteFrame mojom
> 
> This CL converts FrameMsg_EnforceInsecureRequestPolicy to method on
> blink RemoteFrame mojom. It also updates the unit test with
> intercepting RemoteFrame and removes checking the IPC message.
> 
> BUG=1047284
> 
> Change-Id: I9578c49cfcc08b3a682ec2899627dc6503965654
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2038513
> Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
> Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
> Reviewed-by: Kentaro Hara <haraken@chromium.org>
> Commit-Queue: Julie Kim <jkim@igalia.com>
> Cr-Commit-Position: refs/heads/master@{#743924}

TBR=kinuko@chromium.org,haraken@chromium.org,dtapuska@chromium.org,jkim@igalia.com

Change-Id: I39e9cdaa9e63df07cfdca5105a1933ef3c6a860d
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1047284
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2068683Reviewed-by: Andrey Kosyakov <caseq@chromium.org>
Commit-Queue: Andrey Kosyakov <caseq@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743991}

After this CL renderer crashes won't be allowed after
RenderProcessHostWatcher::Wait returns.

Note that even before this CL, RenderProcessHostWatcher could only be
used once (i.e. for one process exit).  In particular,
|std::move(quit_closure_).Run()| would have hit a DCHECK when run a
second time.

Fixed: 1052257
Change-Id: Id1cbd9f59de57d5fe51131a5c7ac5ee14302d0d7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2068950
Commit-Queue: Avi Drissman <avi@chromium.org>
Auto-Submit: Łukasz Anforowicz <lukasza@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743990}

In the previous CL:
https://chromium-review.googlesource.com/c/chromium/src/+/2037836
We deprecated throughput UKM and created a new UKM called
PercentDroppedFrames. But we forgot to update the UKM reporting for
the new UKM. This CL fixes it. With the fix, I can see UKM being
reported locally.

Bug: 1048780
Change-Id: Iece951a7be3b5731413fd1aa86569691ba8128a5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2062752Reviewed-by: Bryan McQuade <bmcquade@chromium.org>
Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743989}

This CL also changes the SharedWorkerService::Observer interface so
that SharedWorkerId replaces SharedWorkerInstance to uniquely
identify shared workers outside of content/.

The motivation behind this change is to make it cheaper to use
the shared worker identifier as a key into a map, since copying
a SharedWorkerInstance is quite expensive.

Bug: 1054596
Change-Id: I7270d9e5fabac2d2e00aecd407a745a1cc779791
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2067389
Commit-Queue: Patrick Monette <pmonette@chromium.org>
Reviewed-by: François Doray <fdoray@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743988}

https://android.googlesource.com/platform/external/perfetto.git/+log/db63601344ca..3bde4517dec7

git log db63601344ca..3bde4517dec7 --date=short --first-parent --format='%ad %ae %s'
2020-02-24 ilkos@google.com Substring matcher for focus mode

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

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/perfetto-chromium-autoroll
Please CC perfetto-bugs@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

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

Bug: chromium:149833691
Tbr: perfetto-bugs@google.com
Change-Id: Iee9017c048ccc40d25544a5c2c1dae4ec7b754d6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2070429Reviewed-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@{#743987}

The team working on Assistant/Quick Answers will now also watch:
- Assistant files in ambient/ui.
- Quick answers directories.

Bug: None
Change-Id: I732e4e9f2bfdf1fea5d23d48b604c46099ac09ea
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2068932Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org>
Commit-Queue: David Black <dmblack@google.com>
Cr-Commit-Position: refs/heads/master@{#743986}

Several attempts at de-flaking this test on the L bots did not help. The
flake cannot be reproduced locally using the same device and OS.

L coverage is not super important and the bot is not on CQ. So just
disable it on L.

This also reverts the last de-flake attempt:
https://chromium-review.googlesource.com/c/chromium/src/+/2063539

Bug: 1035894
Change-Id: I850b96360c08a0d7029729a4a08f7c51efa22b73
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2068783Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Bo <boliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743985}

This reverts commit 22b25dd3.

Reason for revert: NFC settings crash on Android L and M. (crbug/1055216)

Original change's description:
> [WebNFC] Enable feature flag by default for Origin Trial
>
> For the WebNFC Origin Trial the feature flag needs to be enabled by
> default and configured as "disable only" in runtime_features.cc so that
> it can still be used as a kill switch but will not enable the feature
> on sites which don't have a valid Origin Trial token.
>
> Bug: 1035048
> Change-Id: I139a79dd0f93013bc59d5023017b49c820532fb1
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2052198
> Commit-Queue: Reilly Grant <reillyg@chromium.org>
> Reviewed-by: Charlie Reis <creis@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#743309}

TBR=creis@chromium.org,reillyg@chromium.org

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

Bug: 1035048, 1055216
Change-Id: I6617ee1b2bc6b3d1fb719d0454d4f8f4735bd44c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2070106Reviewed-by: Reilly Grant <reillyg@chromium.org>
Reviewed-by: Charlie Reis <creis@chromium.org>
Commit-Queue: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743984}

test_env.py isn't compatible with python3 due to dict.iteritem() calls,
and perhaps more. Consequently, when running the script via the shebang
on an environment that defaults to python3, the script will fail.

This fixes that by explicitly setting invocations of `./test_env.py` to
use the system's python2 (if it has one).

Bug: 942720
Change-Id: I6dc18b5fedace510c61628e88701e89410715ecb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2068896Reviewed-by: John Budorick <jbudorick@chromium.org>
Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743983}

Keep the token for the whole lifetime. So it could be reused in case
user returns to the lock screen page.

Also if the token expires on the fingerprint setup dialog - get back to the
lock settings.

Fixed: 899195, b/144105775
Change-Id: I918bb7186f37c8f5c41add1e14556ecc01de30f7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2035968
Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743982}

As preparation to revert commit

Bug: chromium:1035309
Change-Id: I91417547e188f2ef4028cd556b4b3026302dbd96
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2069142Reviewed-by: Mathias Bynens <mathias@chromium.org>
Commit-Queue: Wolfgang Beyer <wolfi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743981}

The server will be reporting failure as a separate verdict, rather
than using the status field (right now the status field is always
unset). The corresponding server-side change is cl/296334194

Change-Id: Iea78604e6d07871f7e5c798dab5b933969d21d25
Bug: 1045986
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2068038
Commit-Queue: Daniel Rubery <drubery@chromium.org>
Reviewed-by: Xinghui Lu <xinghuilu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743980}

Bug: None
Change-Id: I27fe19f5d1df2d36f42d711ab3bc7e4622f7c940
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2068267Reviewed-by: Matt Menke <mmenke@chromium.org>
Commit-Queue: Ming-Chuan Lin <mingcl@google.com>
Cr-Commit-Position: refs/heads/master@{#743979}

5853fbdc did this for .jar files. This
CL addresses compile_resources.py.

Bug: 1036426
Change-Id: I991132f4b2b1bc99002ec2a153d35ab815bd84b8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2070370
Commit-Queue: Andrew Grieve <agrieve@chromium.org>
Commit-Queue: Peter Wen <wnwen@chromium.org>
Auto-Submit: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: Peter Wen <wnwen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743978}

borenet@chromium.org -> borenet@google.com

Change-Id: I190fde6ee81e3b4804b73f05d813ef165fc7c44d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2067867Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Eric Boren <borenet@google.com>
Cr-Commit-Position: refs/heads/master@{#743977}

Bug:1027046,984851

Per issue tracked in http://crbug.com/1027046, the functionality tied
to the 'turn-off-streaming-media-caching' flag will be leveraged as
part of an experiment to determine whether it should be permanently
enabled. Since the flag is set to expire in M82, this change extends
the expiration to M86 (given we do not have a clear timeframe for
the experiment at this time).

Change-Id: I8b8778bc836b5e738860dd104156d4927a931af0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2068726Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Shawn Pickett <shawnpi@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#743976}

Use onPageLoadFinished() rather than onDidFinishNavigation() for capture
experiments. This ensures the page is loaded and most of the content is
renderered before we attempt a capture. At present this is happening as
a navigation occurs which is too soon as there is often no content.

Known Issue: some sites may trigger too soon as they lazily load
resources. It would be better to trigger on navigate away, but AFAICT
such a trigger/handler doesn't exist as we normally avoid doing work
when unloading the page (requires further investigation).

Bug: 1055353
Change-Id: I33b99835f96543c476a858dd629ce1b41c130b78
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2070104Reviewed-by: Mehran Mahmoudi <mahmoudi@chromium.org>
Commit-Queue: Calder Kitagawa <ckitagawa@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743975}

This prepares for crrev.com/c/2066976 by plumbing the PerfettoService
pointer down into MockProducerHost, which will require it to obtain a
reference to the service's task runner.

Bug: 1006753
Change-Id: I947357d56b4abc3b8435ededc8165e70f5b44286
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2070199Reviewed-by: Sami Kyöstilä <skyostil@chromium.org>
Commit-Queue: Eric Seckler <eseckler@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743974}

Vulcan functions are loaded from shared library dynamically. Calling
those function pointer with cfi will cause SIGILL / ILL_ILLOPN.
So blacklist cfi sanitize for Vulcan function calls.

Bug: 1016885
Change-Id: I73989cebc9ab35d6f35fac508e8f0bcfc23cbd2e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2066937Reviewed-by: Peter Collingbourne <pcc@chromium.org>
Commit-Queue: Peng Huang <penghuang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743973}

This CL expands the per-file ownership to all the files that exclusively
contain functionality related to enterprise scanning (in contrast to APP
scanning)

Change-Id: Ia8a7ee1012fc81d12651ff79a6861bc43c538132
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2065908
Commit-Queue: Daniel Rubery <drubery@chromium.org>
Reviewed-by: Varun Khaneja <vakh@chromium.org>
Reviewed-by: Roger Tawa <rogerta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743972}

Hide Gaia screen when switches from 0 to >0 users, and show the
account picker screen otherwise.

Also remove can_close_ param from ShowGaiaSignin - relies on the user
list now.

Bug: 647451
Change-Id: Iec4e8182b6e07393a10d2336a9818d7d1b2d0b48
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2035670Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Denis Kuznetsov [CET] <antrim@chromium.org>
Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743971}