GitLab

https://skia.googlesource.com/skia.git/+log/dcfb2338586c..be72c44ec3d8

git log dcfb2338586c..be72c44ec3d8 --date=short --first-parent --format='%ad %ae %s'
2020-05-13 mtklein@google.com S20 vk?
2020-05-13 mtklein@google.com Revert "Add an implementation and log2 variants for Wang's formula"
2020-05-13 egdaniel@google.com Reland "Reland "Add api on GrContext to update the data of GrBackendTextures.""
2020-05-13 senorblanco@chromium.org Make staging manager lists private to GrGpu.
2020-05-13 egdaniel@google.com Pass externally synchronizaed flag to vulkan memory allocator.
2020-05-13 csmartdalton@google.com Add an implementation and log2 variants for Wang's formula

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

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

Adds a button to the internals page which sends all pending reports
immediately.

This button simply sends all reports for now. A more
complicated UI was considered, which would allow individual rows to
be selected and sent. However, this requires changes to the
ConversionStorage interface, and the existing interface can be
extended if this functionality is desirable.

This change also restructures communication between
ConversionManagerImpl and ConversionReporterImpl to be callback based.
This is done to allow binding the HandleSentReport callback to a
BarrierClosure which holds the WebUI callback. This allows the WebUI
to update once all reports have finished sending.

Bug: 1057240
Change-Id: If82773b01b9df98d6d1f52cf35aee8fd6bdcf751
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2192436
Commit-Queue: John Delaney <johnidel@chromium.org>
Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Reviewed-by: Charlie Harrison <csharrison@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768561}

Checks for UserPluginVmAllowed user policy stored in PluginVmAllowed
pref in order to allow Plugin VM only if corresponding user policy is set to
true.

Test: unit_tests --gtest_filter="*PluginVm*"
Bug: 1081744
Change-Id: Id4e6eadc651a9dc2d30b7f137759bcdacae5ab5b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2198782
Commit-Queue: Olya Kalitova <okalitova@chromium.org>
Reviewed-by: Roman Sorokin [CET] <rsorokin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768560}

Follow adapted version of: crbug/1051776
as migration plan:
Step 1: (This CL)
- add .gitignore entry for new path
- add entry to src/DEPS for HelpApp
- point references to app to the new entry

Step 2: (CL internal: https://chrome-internal-review.googlesource.com/c/chrome/src-internal/+/2967280)
- remove entry for app in src-internal/DEPS

Cq-Include-Trybots: chrome/try:linux-chromeos-chrome
Change-Id: Iea0ccd46670f91824fc4c95ceb630c4bc7af791f
Bug: 1080474
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2182702
Commit-Queue: Jose Magana <jomag@chromium.org>
Reviewed-by: Trent Apted <tapted@chromium.org>
Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Reviewed-by: Rachel Carpenter <carpenterr@chromium.org>
Reviewed-by: Michael Moss <mmoss@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768559}

The dialog is shown when a zero state suggestion app list search
result is removed, and asks the user to confirm the suggestion removal.

It's positioned relative to the SearchBoxView, but was not properly
getting updated on the app list/search box bounds changes.
It was using ContentsView::SearchBoxUpdateObserver to detect search box
bounds updates, but that did not work reliably (and did not handle app
list contents animations at all, nor bounds updates due to app list view
state changes).

This CL removes this observer interface and lets search results page
view handle the dialog positioning, and hides the dialog when the app
list page changes away from search results page.

Introduces SearchResultPageAnchoredDialog to encapsulate
*   the dialog widget positioning relative to the search box bounds (in
    app list contents view coordinates)
*   coordinate conversions between the app list contents view (in app
    list window coordinates) and the dialog widget (screen coordinates)
*   detecting the dialog widget closure
*   detecting app list window bounds changes

Note that the helper observes the app list window bounds changes to
detect when the app list view bounds changes that do not require
contents view layout (and do not cause app list contents view bounds
change) - the removal dialog has to be re-positioned in this case as
well (its bounds are in the screen coordinates). An example of this
update is when the virtual keyboard gets hidden/shown and app list view
is shown.

BUG=942653

Change-Id: I1b12f6b518d952a6a0f492ab73c4d6afb24f1bea
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2198357
Commit-Queue: Toni Baržić <tbarzic@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768558}

This patch adds the missing API in WEBGL_video_texture spec, renames
VideoElementTargetVideoTexture to shareVideoImageWEBGL, and corrects
the success logic in shareVideoImageWEBGL.

Bug: 776222
Change-Id: I17f2839dcef9c63346348c6d155e4eb6eb877760
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2194795
Commit-Queue: Shaobo Yan <shaobo.yan@intel.com>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768557}

Screen record:
https://drive.google.com/file/d/18vdtOt5pgAOLyaDH5BKRgcYcJ3uRX6tL/view?usp=sharing

Bug: 1081538
Change-Id: I96b96524b574f8ffeb20aea93e31a5724a1f114e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2199691
Commit-Queue: Ganggui Tang <gogerald@chromium.org>
Commit-Queue: Yusuf Ozuysal <yusufo@chromium.org>
Auto-Submit: Ganggui Tang <gogerald@chromium.org>
Reviewed-by: Yusuf Ozuysal <yusufo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768556}

The flag is separated from cc::Layer::contents_opaque() for LCD text.
By default it has the same value as contents_opaque(), but when
contents_opaque() is false, blink can set contents_opaque_for_text()
to true if all texts will be painted on opaque background, e.g.
  <div style="overflow: hidden; background: white; border-radius: 10px">
    TEXT
  </div>

This reduces 30%~40% non-LCD-text pixels for contents not opaque reason
(except background is not opaque), and 6%~7% overall non-LCD-text
pixels:
https://ct.skia.org/results/cluster-telemetry/tasks/chromium_perf_runs/wangxianzhu-ChromiumPerf-4900/html/index.html
https://ct.skia.org/results/cluster-telemetry/tasks/chromium_perf_runs/wangxianzhu-ChromiumPerf-4901/html/index.html

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

Carbon specs: https://carbon.googleplex.com/cros-ux/pages/show-off/app-structure#3e13a03d-5684-488c-b935-4129aedd6a68
Demo: https://screencast.googleplex.com/cast/NTE1NzAyMzQ5Njc5ODIwOHxjY2U4OGY5Ni1iYw

Bug: b/154777087
Change-Id: Ibd79be825751334be65025155d0a1b17a807dbeb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2198058Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Reviewed-by: Rachel Carpenter <carpenterr@chromium.org>
Commit-Queue: Callistus Tan <callistus@google.com>
Cr-Commit-Position: refs/heads/master@{#768554}

Bug: 1082193
Change-Id: Ie9b3aa6b93214b720972ef9e9d46827d9cff1025
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2198653
Commit-Queue: Megan Jablonski <megjablon@chromium.org>
Reviewed-by: Doug Arnett <dougarnett@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768553}

fuchsia.camera3 API provides camera application to clients, which are
expected to rotate the image when presenting it on the screen. Updated
VideoCaptureDeviceFuchsia to watch camera orientation and apply
corresponding transformation to the video frames.

Bug: 866669
Change-Id: I05ea0a3d4a59975ae0ed111c949b434fb6482b95
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2189619
Commit-Queue: Sergey Ulanov <sergeyu@chromium.org>
Reviewed-by: Kevin Marshall <kmarshall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768552}

This introduces x509_certificate_model::ProcessRawSubjectPublicKeyInfo
which takes a DER-encoded X.509 SubjectPublicKeyInfo as input, and
returns a string intended for displaying the public key represented by
it.

It will be used in the UI code for the Chrome OS built-in certificate
provisioning feature. When a certificate provisioning process is in
progress, only a key pair is on the device and not a certificate (yet).
This function will be used to display the public key of the key pair on
the UI.

Bug: 1045895, 1081396
Change-Id: I8428dec1a03ac1fdc3edffdaeeda304be0cdb9be
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2036053
Commit-Queue: Pavol Marko <pmarko@chromium.org>
Reviewed-by: David Benjamin <davidben@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768551}

Bug: 1077348, 1080466
Change-Id: I1d27c2e041960b99345148a1668431c780f18b6a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2200070
Commit-Queue: Erik Chen <erikchen@chromium.org>
Auto-Submit: Erik Chen <erikchen@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768550}

As a followup to [1], this CL ensures that on-by-default Client Hints
don't get removed from cross-origin redirects.
It similarly makes sure that legacy hints that don't abide to
FeaturePolicy get a free pass on the removal in that case.
Finally, it makes sure that UA-Mobile's FeaturePolicy's default value
is "all".

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

Bug: 911952, 1082072
Change-Id: I1f329a3c24397a287a1cbc333cd0976bb16d640a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2199107
Commit-Queue: Yoav Weiss <yoavweiss@chromium.org>
Reviewed-by: Maksim Orlovich <morlovich@chromium.org>
Reviewed-by: Aaron Tagliaboschi <aarontag@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768549}

After we previously run as an inconclusive beta trial by itself we
grouped it with the HistoryService ThreadPool trial to try and coax a larger
impact from the ThreadPool migration. This meant groups ran with either both
services on the ThreadPool, none or only HistoryService (status quo).

28 day aggregation trial in beta showed no likely negative impact. Crashes
seemed slightly elevated in the group when running both the HistoryService
and the ProfileSyncService on the ThreadPool but closer examination suggests
this is simply noise since all top-crashers are the same.

Bug: 1014464
Change-Id: I3a1604413fcf1f4b5350c3b9161251f46a449865
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2198932
Commit-Queue: Oliver Li <olivierli@google.com>
Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768548}

In Web App kiosk mode, even though web apps are enabled, system web apps
are not. We should explicitly disable them here.

Bug: 1061068
Change-Id: Iea6f6f439d11c408ba3c9a48d4987f5cdcad0da3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2199280
Commit-Queue: Anatoliy Potapchuk <apotapchuk@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768547}

Introduce getEnabledRulesets extension function which returns the IDs
for the set of enabled static rulesets.

BUG=754526

Change-Id: Ie7647c718b96953a7ba2fbb0e2322443b9a18fec
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2189692
Commit-Queue: Karan Bhatia <karandeepb@chromium.org>
Reviewed-by: Kelvin Jiang <kelvinjiang@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768546}

1. PageInfoController uses the now componentized
PermissionParamsListBuilder.
2. A PermissionParamsListBuilderDelegate is created for WebLayer
and passed into PageInfoController.

Bug: 1052375
Change-Id: I71b1d61e60c755887b5ae767d4a945aa4eec8d3b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2185070
Commit-Queue: Mugdha Lakhani <nator@chromium.org>
Reviewed-by: Emily Stark <estark@chromium.org>
Reviewed-by: Clark DuVall <cduvall@chromium.org>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768545}

for flakiness.

The UnlimitedStorageForLocalButNotSync was disabled long ago
for flakiness. This test passed 3000 iterations on the bots
with no flakes.

Bug: 227296
Change-Id: I24da0f1718658348dafb866c2eb6e7770400f843
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2196959Reviewed-by: Archana Simha <archanasimha@chromium.org>
Commit-Queue: David Bertoni <dbertoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768544}

Rename OSExchangeDataProviderAuraX11 to OSExchangeDataProviderX11.

AuraX11 classes were named as "AuraX11" for historical reasons, but X11 no
longer requires Aura, so X11 is a more concise name.

No intended logic changes. Only names and comments should have changed.

Change-Id: I444466032621a65d4e176f3278e5b0bb6f7dad30
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2197867
Commit-Queue: Darwin Huang <huangdarwin@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Auto-Submit: Darwin Huang <huangdarwin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768543}

App list folders should expand from the folder's origin.The bounds for
the folder should be 8 dip from the edge of the screen and should not be
limited by the app list container bounds.


Bug: 1018305
Change-Id: I3597dbe96c697f5c268ddd9cc5157c62f7600f5e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2197422
Commit-Queue: Yulun Wu <yulunwu@chromium.org>
Reviewed-by: Yulun Wu <yulunwu@chromium.org>
Reviewed-by: Alex Newcomer <newcomer@chromium.org>
Reviewed-by: Toni Baržić <tbarzic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768542}

Introduce the updateEnabledRulesets extension function which allows
extensions to update the set of enabled static rulesets.

This is a partial implementation and the set of enabled rulesets still
needs to be persisted to prefs so that it is sticky across extension
loads.

BUG=754526

Change-Id: I5b8e76ef4cc5a3c1049131109057e4f20efc8d64
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2186989
Commit-Queue: Karan Bhatia <karandeepb@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768541}

See linked bug for rationale. This does not change functionality.

Bug: 1065087
Change-Id: Id6e938b64594cc029f716270cfbb0435b9efacd0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2198373Reviewed-by: Robert Sesek <rsesek@chromium.org>
Reviewed-by: Alexander Cooper <alcooper@chromium.org>
Commit-Queue: Alex Gough <ajgo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768540}

Introduces and adds UserPluginVmAllowed policy, that controls whether
Plugin VM is allowed for a user. Policy handling will be added in the
follow up CL. Will co-exist with the current PluginVmAllowed device
policy.

Test: browser_tests --gtest_filter="PolicyPref*"
Bug: 1081744
Change-Id: Ic3f794d340215a69ba0ad6814326d1ddfff0bf73
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2199057
Commit-Queue: Olya Kalitova <okalitova@chromium.org>
Reviewed-by: Roman Sorokin [CET] <rsorokin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768539}

The change allows a Presentation API to control a 1UA mode presentation.

Bug: 1069126
Change-Id: I943293d884c77319f865273105cf93f0c6727ee2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2186109
Commit-Queue: John Williams <jrw@chromium.org>
Reviewed-by: Brandon Tolsch <btolsch@chromium.org>
Reviewed-by: Takumi Fujimoto <takumif@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768538}

This reverts commit e87e4072.

Reason for revert: compilation failure. See https://ci.chromium.org/p/chrome/builders/ci/arm64-builder-rel/21384.

Original change's description:
> [Offline indicator v2] Add metrics
> 
> Bug: 1068727
> Change-Id: I5470452fcdfd5d9b7310e65802ef2867326fb27a
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2191376
> Reviewed-by: Brian White <bcwhite@chromium.org>
> Reviewed-by: Theresa  <twellington@chromium.org>
> Commit-Queue: Sinan Sahin <sinansahin@google.com>
> Cr-Commit-Position: refs/heads/master@{#768507}

TBR=bcwhite@chromium.org,twellington@chromium.org,sinansahin@google.com

Change-Id: I04a42405e1803ca808e7465589d31dc8940fa9f7
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1068727
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2199988Reviewed-by: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768537}

https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/8355052cad11..0a227c84a559

git log 8355052cad11..0a227c84a559 --date=short --first-parent --format='%ad %ae %s'
2020-05-13 paolosev@microsoft.com Make DWARFSymbolServer build on Windows

Created with:
  gclient setdep -r src/third_party/devtools-frontend/src@0a227c84a559

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/devtools-frontend-chromium
Please CC devtools-waterfall-sheriff-onduty@grotations.appspotmail.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: None
Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com
Change-Id: Id1678c6de07662c77f04eb712f6b8a99e62840d0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2199661Reviewed-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@{#768536}

We don't need to check if a URL is "HTTP" and then also check that it's
not "chrome://"

Bug: 1044855
Change-Id: Idf843951668d88de831dfe28a4f567f0b9717893
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2185832Reviewed-by: Jeffrey Cohen <jeffreycohen@chromium.org>
Commit-Queue: Dominic Schulz <dominicschulz@google.com>
Cr-Commit-Position: refs/heads/master@{#768535}

Logs
- NewTabPage.Promos.LinkClicked
- NewTabPage.Promos.RequestLatency2
- NewTabPage.Promos.RequestLatency2.Failure
- NewTabPage.Promos.RequestLatency2.SuccessWithoutPromo
- NewTabPage.Promos.RequestLatency2.SuccessWithPromo
- NewTabPage.Promos.ShownTime
on chrome://new-tab-page and extends and updates ownership of the
metrics.

Fixed: 1072874
Change-Id: Id22a10e4e376d987607a5795ffdd42631d0120cd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2197997
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Alex Gough <ajgo@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Reviewed-by: Esmael Elmoslimany <aee@chromium.org>
Auto-Submit: Tibor Goldschwendt <tiborg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768534}

external/wpt/pointerevents/pointerevent_capture_mouse.html
is failing in blink_web_tests on WebKit Linux Leak

Bug: 998948
Change-Id: I32248d6cf9183605a4fd5b5ebaf959af6179a75c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2200007
Auto-Submit: Dana Fried <dfried@chromium.org>
Reviewed-by: Oksana Zhuravlova <oksamyt@chromium.org>
Commit-Queue: Dana Fried <dfried@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768533}

Setting the switch will cause ash-chrome to launch the lacros-chrome
binary from the given path rather than from component updater.

Bug: 1077348
Change-Id: I50f5a1be84c31df434b37997007112e7c1296767
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2200078
Commit-Queue: Erik Chen <erikchen@chromium.org>
Commit-Queue: James Cook <jamescook@chromium.org>
Auto-Submit: Erik Chen <erikchen@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768532}

This should not change behaviour.

TEST=Manual browser_tests.exe --gtest_filter=EdgeImporter*

Bug: 1065087
Change-Id: Ia67cd29625ff34575682da57a45ab68bd3104d3d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2196838
Auto-Submit: Alex Gough <ajgo@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Gabriel Charette <gab@chromium.org>
Commit-Queue: Gabriel Charette <gab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768531}

appParams is encoded in the presentation URL by the sender SDK in JSON
format, and is passed to Cast MRP as a string.

Bug: b/153748066, 1074358
Change-Id: I3e62baa4ebad7b73e026e3ac73a5db38c9425e63
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2166132
Commit-Queue: Muyao Xu <muyaoxu@google.com>
Reviewed-by: mark a. foltz <mfoltz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768530}

We want the spoken feedback user to be notified about what what happens
when the user presses the "change position" button on the floating menu.

Bug: 1061068
Change-Id: Ib1c6a70f7d94825ee057e1d7fa2406b569a84564
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2199262
Commit-Queue: Anatoliy Potapchuk <apotapchuk@chromium.org>
Reviewed-by: David Tseng <dtseng@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768529}

Currently works on browser windows due to ScopedRootForNewWindows due
to opening sync (or before AfterItemSelected callback is fired). V1 and
V2 apps are launched async though.

This moves deleting the scoped object until ShelfItemStatusChanged
which can be called by window activation change, which is async. It's
a little worrisome that we might not clear the changed root window so
added a delayed task to ensure deletion (when I tested window
activation always happened though).

Tried adding a test in ChromeLauncherControllerTest but it doesn't seem
like there was support for launching a app in a separate window, even
by launching manually and not clicking on the shelf shortcut.

Test: manual
Bug: 1080390
Change-Id: I0d5f6126571f92d86c5d4530d205b6cfc3e635be
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2198550Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768528}

Change-Id: I05e3f6f6b3521e8df097f03faaeb3f41e4c3419b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2198921Reviewed-by: Mark Mentovai <mark@chromium.org>
Commit-Queue: Robert Sesek <rsesek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768527}

This change adds text which indicates the synced status of networks
to the network details page.  To do this, it  adds an is_syncable
property to network_config::mojom::ManagedWiFiProperties to determine
if a network is eligible, and the existing [Os]SyncBrowserProxy to
determine if wifi sync is enabled for the active user.

Bug: 966270
Change-Id: Iac01c901b40cff92cb0c6bf804bc397bdfd81285
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2189644Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Reviewed-by: James Vecore <vecore@google.com>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Commit-Queue: Jon Mann <jonmann@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768526}

This patch records additional metrics for TextFragmentAnchor to get
insight into how the parameters are used. It records an enum indicating
which combination of parameters were specified, as well as the length
of the text matching parameters and the total match length.

These metrics will help determine whether there should be a minimum
range match length requirement, and what that length should be, to
discourage the overuse of text ranges which may be easier to manually
create but result in a more fragile link.

Bug: 1039778
Change-Id: I71c46ee219b9718f33fb8e922b576ec939798dfb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2196951
Commit-Queue: Nick Burris <nburris@chromium.org>
Reviewed-by: Brian White <bcwhite@chromium.org>
Reviewed-by: David Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768525}

This is a reland of 02ee5e26

Original change's description:
> [Fuchsia] Add unittests for VideoCaptureDeviceFuchsia
>
> Added unittests of for VideoCaptureDeviceFuchsia with fake
> fuchsia.camera3.Device and fuchsia.camera3.Stream implementations.
>
> In the future the fakes can be used for web_engine_integration_tests.
>
> Bug: 866669
> Change-Id: I0b5b0267127f9843b12123eb9e9a61890ec95a25
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2181184
> Reviewed-by: Emircan Uysaler <emircan@chromium.org>
> Reviewed-by: Kevin Marshall <kmarshall@chromium.org>
> Commit-Queue: Sergey Ulanov <sergeyu@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#768422}

TBR=kmarshall@chromium.org

Bug: 866669
Change-Id: I6c68aee7c50b8e3dc93744446042920c79c37db9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2200000Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Commit-Queue: Sergey Ulanov <sergeyu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768524}

Third party theme background images are served via
chrome[-search]://theme/IDR_THEME_NTP_BACKGROUND?<theme-id>. The WebUI
NTP embeds the background image via a chrome-untrusted:// iframe since
most background images are fetched from the web. This means to use the
common WebUI NTP background image machinery the theme image has to be
available to a chrome-untrusted:// iframe. Therefore, this CL adds support
for serving the theme data source on the chrome-untrusted scheme.

With that, this CL sets the background image URL to
chrome-untrusted://theme/IDR_THEME_NTP_BACKGROUND?<theme-id> if a
theme with a third-party background image is set. Furthermore, this CL
adds white pills around the shortcut titles as the local NTP does it.

Bug: 1081441
Change-Id: Ia26546c395fabbcee17062ebec17956042cd85f8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2197661
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Nasko Oskov <nasko@chromium.org>
Reviewed-by: Esmael Elmoslimany <aee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768523}