GitLab

And send the HEART capability when it's set.

Bug: 1158468
Change-Id: Ib7b64ecd4dff4aa23701de0eeceac0b8d1635ee1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594121
Commit-Queue: Dan H <harringtond@chromium.org>
Reviewed-by: Carlos Knippschild <carlosk@chromium.org>
Reviewed-by: Justin DeWitt <dewittj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844147}

The test is flaky on Linux, ChromeOS and Android.

TBR=dominickn@chromium.org

Bug: 1167226
Change-Id: I98fb83da5c47fd0aed1e05045fbd2ecbf2886b3b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2633090
Commit-Queue: Fabrice de Gans-Riberi <fdegans@chromium.org>
Auto-Submit: Fabrice de Gans-Riberi <fdegans@chromium.org>
Reviewed-by: Yi Gu <yigu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844146}

Not all devices report this information, so do not test for it.

Bug: 1167123, 1164549
Change-Id: Ia2e0262e7a6692cccdc114bbf2decea76c4655ad
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2633668Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Robert Sesek <rsesek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844145}

This CL changes the subresource filter to handle the action of the user
initiating a reload of a page with blocked ads entirely within the
component, rather than delegating to the embedder. This change
concretely means that WebLayer now obtains this behavior for free. To
test proper integration in WebLayer, this CL also adds browsertests of
the expected behavior in //weblayer. We disable these browsertests on
Windows due to https://crbug.com/1152429.

Bug: 1116095
Change-Id: I270aff6251d0a498af77b9748bf129036e4d3541
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2606345
Commit-Queue: Colin Blundell <blundell@chromium.org>
Reviewed-by: Charlie Harrison <csharrison@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844144}

Lock screen and lock screen app profiles are not regular user profiles.

They disable sync and should be skipped when reading sync settings.

Bug: b:175067939
Change-Id: I6634ce77efbb39a493df0161a16796650cac73c9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2628052Reviewed-by: Gabriel Marin <gmx@chromium.org>
Commit-Queue: Bin Du <dubin@google.com>
Cr-Commit-Position: refs/heads/master@{#844143}

* Extract creation of policy_request into a standalone function

Bug: 1150887
Test: CloudPolicyClientTest*
Change-Id: Ie5f67e1272ef9b2692b71ae0c7effade3016999b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2550565
Commit-Queue: Michael Ershov <miersh@google.com>
Reviewed-by: Pavol Marko <pmarko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844142}

A previous security fix prevented text documents from creating
preloader_ objects. Calling ScanAndPreload in these cases is
incorrect and normally DCHECKed (but it seems like this case is
sufficiently rare that it made it into the wild).

Bug: 1166786, 901056
Change-Id: Ia1530536d644396601e5d7ab88a878ae378ad832
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2630742
Auto-Submit: Richard Townsend <richard.townsend@arm.com>
Commit-Queue: Mason Freed <masonfreed@chromium.org>
Reviewed-by: Mason Freed <masonfreed@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844141}

Currently the load time we have includes the time taken on the sequenced
task runner. Since CriticalPersistedTabData and TabState operations are
both performed on the sequenced task runner, we block
CriticalPersistedTabData read over the TabState read. This unfairly
biases the CriticalPersistedTabData load time metric making it larger
than it should be. It is better to compare the two independent of the
sequenced task runner.

Bug: 1166787
Change-Id: Ibc716f66cf9ec745d741f471bbd372048ef9edb9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2627730Reviewed-by: Brian White <bcwhite@chromium.org>
Reviewed-by: Yusuf Ozuysal <yusufo@chromium.org>
Commit-Queue: David Maunder <davidjm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844140}

Nearby visibility was being pushed to the second page of the system
tray. This made it hard to discover. This cl moves Nearby and screen
capture higher in the ordering to increase the likelihood that they
end up on the first page.

This new ordering was discussed here:
https://groups.google.com/a/google.com/g/chromeos-ui-review-listeners/c/pp87GLQT3a0

Screenshot: https://screenshot.googleplex.com/9hzZ4VsRw38Mnb5.png

Fixed: 1164456
Change-Id: I5d700757d4893ffea829a816758b1a3ffc372955
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2622518
Commit-Queue: Curt Clemens <cclem@google.com>
Reviewed-by: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844139}

GetAppUserDisplayMode.

Change-Id: I42a81b31adb4b3156f722c55385b862d06213503
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2629464
Auto-Submit: Jarryd Goodman <jarrydg@chromium.org>
Commit-Queue: Daniel Murphy <dmurph@chromium.org>
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844138}

Migrated all the usages of base::Callback &
base::Bind in StatusCollector and its derived
classes.

Bug: 714018
Change-Id: Iba62d767ef2adb9cc136262b6edeeff8e0001f11
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2630027Reviewed-by: Maksim Ivanov <emaxx@chromium.org>
Commit-Queue: Aya Elsayed <ayaelattar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844137}

Change-Id: Iaeaa84546ce5e3085ffcb93366714163b556e62a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2631209
Auto-Submit: David Dorwin <ddorwin@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Reviewed-by: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844136}

All other observable impacts of origin-keying, namely the impact on
postMessage and document.domain, are already in place for cross-origin
isolated agent clusters. So we only need to update the
window.originAgentCluster getter.

Fixed: 1163687
Change-Id: Ie10f26def8de2046d2b0ce69b37d56b3c89db57b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2630231Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Domenic Denicola <domenic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844135}

Move processing the OpenChannelToPepperPlugin to mojo and handling on
the Pepper specific PepperRendererConnection class. This will allow
the removal of the RenderFrameMessageFilter in a following CL.

The IPC::ChannelHandle that was passed via legacy IPC is now passed
via handle<message_pipe>. There are conversions from the ChannelHandle
to a ScopedMessagePipeHandle, this is safe because Legacy IPC param
traits took ownership of the ChannelHandle during its serialization.
Now it is just more explicit.

BUG=1157519

Change-Id: I6d8a979f98fd91848ae12fd5a6a7234e3ce5e217
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2630226
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: Bill Budge <bbudge@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Arthur Sonzogni <arthursonzogni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844134}

They are unused.

BUG=None
TEST=try

Change-Id: I969698f44dadb424b89897ec52ff7488c65d3e83
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2632352Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: Yusuke Sato <yusukes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844133}

BUG=None
TEST=try

Change-Id: Ia714c1dce72a9ac13aacc12900cbca11efeac890
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2632565Reviewed-by: Xiaoqian Dai <xdai@chromium.org>
Commit-Queue: Yusuke Sato <yusukes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844132}

Sample build:
https://ci.chromium.org/ui/p/chromium/builders/ci/Mac10.15%20Tests/7525/test-results?expected=false&exonerated=true&flaky=true

Bug: 1167210
Change-Id: If80065473d3cac08bce82562cc730944b1cd0c4b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2633349
Auto-Submit: Yi Gu <yigu@chromium.org>
Reviewed-by: Alison Maher <almaher@microsoft.com>
Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Yi Gu <yigu@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844131}

This test was flaking on P so I've just re-enabled it on everything
except P. This should get us a bit more coverage which is nice.

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

uploading v2.0

The previous CL (http://crrev.com/c/2527060) broke autofill captured
sites tests and was reverted. The issue was fixed in
http://crrev.com/c/2552281. So, this CL is just relanding + rebasing
(AutofillDownloadManager now created in AutofillHandler, not
AutofillManager). See diff in https://chromium-review.googlesource.com/c/chromium/src/+/2629310/1..7

Bug: 1111809

Change-Id: Iab4669843d58661cc6b9af6508f51411932770a4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2629310Reviewed-by: Christoph Schwering <schwering@google.com>
Reviewed-by: Christos Froussios <cfroussios@chromium.org>
Commit-Queue: Maxim Kolosovskiy  <kolos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844129}

See linked bug for details.

TBR=alancutter@chromium.org

Bug: 1167176
Change-Id: I499e3886178d41c9f0036b381e2ee5f57c1aa16c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2633582
Commit-Queue: Yi Gu <yigu@chromium.org>
Reviewed-by: Anatoliy Potapchuk <apotapchuk@chromium.org>
Reviewed-by: Fabrice de Gans-Riberi <fdegans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844128}

This CL creates gn targets for lacros tast tests, and the main
motivation is to run lacros tast tests on Chromium CI.

Bug: 1158590
Change-Id: I783779c3386a39743b8227367189b0b87832b9dc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2606188
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Reviewed-by: Ben Pastene <bpastene@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@google.com>
Cr-Commit-Position: refs/heads/master@{#844127}

Puts dispatcher to a special mode where it pretends to recognize all
selectors and silently fail on shutdown.

Bug: 1137686, 1163129
Change-Id: Ib065d7f564f22c5be2285201d5dcd9dea39f60d1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2472039
Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org>
Reviewed-by: Mark Cogan <marq@chromium.org>
Auto-Submit: Stepan Khapugin <stkhapugin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844126}

https://quiche.googlesource.com/quiche.git/+log/e84d83393be4..9aef115356ce

$ git log e84d83393..9aef11535 --date=short --no-merges --format='%ad %ae %s'
2021-01-14 quiche-dev Fixes test flakiness by forcing the test client to reconnect after fuzzed requests.
2021-01-14 renjietang In QuicConnection, cancel the previous path validation request before starting a new one.
2021-01-14 mattm Send correct QUIC CRYPTO_ERROR connection error for TLS errors
2021-01-14 renjietang In http/end_to_end_test, use live client address to send STATELESS_RESET.
2021-01-14 wub Minor tweaks to TlsServerHandshaker: - Change TlsServerHandshaker::SelectCertStatus() to TlsServerHandshaker::select_cert_status(). - Set expected_ssl_error to SSL_ERROR_WANT_READ in OnSelectCertificateDone(). - Add a test for async select cert and async signature.
2021-01-14 wub In crypto_test_utils, move connection close frame between two test connections. This allows tests to check connection close errors on both ends.
2021-01-14 wub Add a flag count for --gfe2_restart_flag_quic_tls_prefer_server_cipher_and_curve_list.

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

Change-Id: I3f634a094103c70bf9db30bae3646a740f6202ec
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2633321
Auto-Submit: Bence Béky <bnc@chromium.org>
Commit-Queue: Renjie Tang <renjietang@chromium.org>
Reviewed-by: Renjie Tang <renjietang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844125}

The Finch setting might not be activated for some cases, and we want to
provide developers with thread priority information via chrome://tracing.

Bug: 1165842
Change-Id: I50cf618587acb8ce01237c894a21ad4965e22a62
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2631288Reviewed-by: Raymond Toy <rtoy@chromium.org>
Auto-Submit: Hongchan Choi <hongchan@chromium.org>
Commit-Queue: Hongchan Choi <hongchan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844124}

Previously no cookie settings were provided when NetworkContext was
created, and the default in-memory cookie store was used. Provide a
cookie path so that the on-disk cookie store is used.

Bug: internal b/177479549
Test: Launch YT on device and verified cookies survived reboot.
Change-Id: Icf6ac063ffc0fc13de36bb6dd6b62edc8e4496a5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2631358Reviewed-by: Luke Halliwell (slow) <halliwell@chromium.org>
Commit-Queue: Junbo Ke <juke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844123}

The experiment launched.

Bug: 1126640
Change-Id: I674ef2336f3f22cf3859775805ec7997a4308281
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2631929Reviewed-by: Yann Dago <ydago@chromium.org>
Commit-Queue: Caitlin Fischer <caitlinfischer@google.com>
Cr-Commit-Position: refs/heads/master@{#844122}

Convert FrameHostMsg_DidCreateOutOfProcessPepperInstance and
FrameHostMsg_DidDeleteOutOfProcessPepperInstance to mojo messages.
These messages can use the PepperBrowserConnection

The implementation for RenderFrameMessageFilter can move into the pepper
subclasses themselves.

BUG=1157519

Change-Id: Ic051729ed0edf99d5bd8165fe5d4d5e21b3cf42c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2626413
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: Bill Budge <bbudge@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844121}

* Delete default constructors in mock callback observers
because writing them explicitly doesn’t change anything.

* Make `MockDeviceManagementService service_` a strict mock
so we don’t need to have EXPECT_CALL().Times(0) in many test.

* Change some `EXPECT_CALL(..., Func(_))` into
`EXPECT_CALL(..., Func)` where checking arguments is not useful.

* Remove some arguments from Expect* functions where they were
unused.

* Extract creation of policy_response into a standalone
function to create it only when necessary and decrease implicit
dependencies.

* Reformat the file.

Bug: 1150887
Test: CloudPolicyClientTest*
Change-Id: I3c150a59161a6272d37f19234520a8ac9cb850ea
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2550032
Commit-Queue: Michael Ershov <miersh@google.com>
Reviewed-by: Pavol Marko <pmarko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844120}

Updating everything to match the latest style guide preferences, mostly
using references much more liberally and converting output params to
returns wherever reasonable.

Also switching from copies to moves in a couple places that I happened
to notice, and a couple more random cleanups to make Tricium happy.

Change-Id: I999f511835e70e61ee8c0e7c5dd7e6b7f2396aea
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2626786Reviewed-by: Matt Menke <mmenke@chromium.org>
Commit-Queue: Eric Orth <ericorth@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844119}

Currently, the ChromeVox behavior is broken. It doesn't echo typed text,
nor does it read out typed text.

This CL fixes ChromeVox behavior by calling
Textfield::GetAccessibleNodeData() in
DeskNameView::GetAccessibleNodeData().

Test: manual
Bug: 1166732, 1166693
Change-Id: I54306c8b3ea02000aef7663ace32ffd314d5c235
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2632045Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Commit-Queue: Jeremy Chinsen <chinsenj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844118}

Inflated distributions are useful for running A/B tests to understand
the impact of shipping larger binaries.

Bug: 1154844
Change-Id: Ifdfeeb200b58174e5c3e7f346ecf013780111d25
Fixed: 1154844
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2625888
Commit-Queue: Joshua Pawlicki <waffles@chromium.org>
Reviewed-by: Robert Sesek <rsesek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844117}

This generally should only happen if the renderer is compromised.

BUG=none
TEST=none

Change-Id: I88d522e1e5f3f557c7f976541b6dc50fb0cfed81
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2631296Reviewed-by: Shimi Zhang <ctzsm@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844116}

The server-side configs show that these studies have launched.

Bug: 1126640
Change-Id: I29f802ad203fecc0eab77f68df169e11d02099c0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2631433Reviewed-by: Becca Hughes <beccahughes@chromium.org>
Commit-Queue: Caitlin Fischer <caitlinfischer@google.com>
Cr-Commit-Position: refs/heads/master@{#844115}

Roll Chrome Win32 PGO profile from chrome-win32-master-1610690109-0f7956766f9641d6835b78b89edaf84f3a491ef2.profdata to chrome-win32-master-1610711423-66d1757656af1c1220147c4ed72bfc529787f050.profdata

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/pgo-win32-chromium
Please CC pgo-profile-sheriffs@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/+doc/master/autoroll/README.md

Cq-Include-Trybots: luci.chrome.try:win-chrome
Tbr: pgo-profile-sheriffs@google.com
Change-Id: I385df950dcf035cdf4d92d0aee7e6726f79f47e4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2632836Reviewed-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@{#844114}

https://chromium.googlesource.com/angle/angle.git/+log/5678b56f5e21..d2e76b73efc2

2021-01-15 angle-autoroll@skia-public.iam.gserviceaccount.com Roll SwiftShader from 5255c465bbba to aedd133cd85f (1 revision)
2021-01-15 angle-autoroll@skia-public.iam.gserviceaccount.com Roll vulkan-deps from 5869a809d551 to 7b69f3d087b8 (5 revisions)

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 timvp@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/+doc/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-asan;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:linux-swangle-try-x64;luci.chromium.try:win-swangle-try-x86
Bug: None
Tbr: timvp@google.com
Change-Id: I68ed451d3ee283335eeccf2a0ad54769072448cf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2632837Reviewed-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@{#844113}

Speedup: 2783ms -> 36ms (99% speedup)

Bug: 1166919
Change-Id: If7b9e406fb5c1ce46e4a224542fe0ab60ee34c32
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2631227
Commit-Queue: David Maunder <davidjm@chromium.org>
Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844112}

Before this CL only the icon with the '+' was clickable. The full card
should be clickable.

Fixed: 1167120
Change-Id: I4c326ad8c17fc34425ed3cb9934304eac801777a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2632668
Commit-Queue: Monica Basta <msalama@chromium.org>
Reviewed-by: David Roger <droger@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844111}

BUG=None
TEST=try

Change-Id: Ic3983403036f8e11749c7969dddd027e8e46d820
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2632391Reviewed-by: Pavol Marko <pmarko@chromium.org>
Commit-Queue: Yusuke Sato <yusukes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844110}

We're seeing jank in LookalikeUrlService::OnFetchEngagedSites from uma
sampling profiler. There's a few possible culprits:
- First call to GetDomainInfo causes icu loading
- url_formatter::GetSkeletons is expensive and called on many urls
- IsEngagementAtLeast is expensive and called on many urls

To better understand the problem, this CL adds relevant trace events and
records count of engaged sites as UMA histogram.

Bug: 1135279
Change-Id: Ie6ee4ed2dbeeed26e4bbdd72726a0ad48e9bc4bf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2615322Reviewed-by: Etienne Bergeron <etienneb@chromium.org>
Reviewed-by: Weilun Shi <sweilun@chromium.org>
Reviewed-by: Joe DeBlasio <jdeblasio@chromium.org>
Commit-Queue: Etienne Pierre-Doray <etiennep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844109}

Seems due to crrev.com/c/2500441. See linked bug for details.

Bug: 1167158
Change-Id: I266c816d846fe6b90c65b5d77c9a4f65714c01b5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2633362Reviewed-by: Anatoliy Potapchuk <apotapchuk@chromium.org>
Reviewed-by: David Bertoni <dbertoni@chromium.org>
Commit-Queue: Yi Gu <yigu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844108}