GitLab

Fixed: 1131119
Change-Id: I9ce8308f5c5832c7156cc60ff55a6b5c561161d8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424650
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Auto-Submit: dpapad <dpapad@chromium.org>
Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809558}

Fix the behavior of the "Remove fingerprint" search tag so that it
dynamically changes based on the number of registered fingerprints.

Bug: 1116553
Change-Id: Ifd109dbf6757bede7b079eb7a31d7db3b45b27b0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424507
Commit-Queue: Daniel Classon <dclasson@google.com>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809557}

Design Doc: https://docs.google.com/document/d/1yf_88IrTCNbrfMaZnU4gwUu_Fi9f6q7ngacWc7-d4zg/edit?usp=sharing

Bug: 1130078
Change-Id: I4a2c51f80f8d484e64c67312cdae2a57a3e0faa9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2268338Reviewed-by: Robert Liao <robliao@chromium.org>
Commit-Queue: Allen Bauer <kylixrd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809556}

Bug: 813093
Change-Id: I384979ae9a027b4961adefe2e906a2f0cb2176e7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424551Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Commit-Queue: Alexei Svitkine <asvitkine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809555}

It's only called for client versions <=83, which are no longer
supported. Hence the calling code can be deleted, and the
method removed from the aidl.

The public-facing NewTabCallback method is only made concrete as
removing it would break client compilation.

Also change MAX_SKEW to 3, restoring it to the value that was used
before we missed shipping version 82.

Bug: 1110811
Change-Id: I8c3eaaedd59f546c60a41b37e4e2bc070b7da0c8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2414991
Commit-Queue: Evan Stade <estade@chromium.org>
Auto-Submit: Evan Stade <estade@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809554}

The strings in this change are from UX.

Bug: 1126056
Change-Id: I72393fbfec457d80f07f5d44dc457e29bcc42d9f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2422479Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809553}

TBR=bbudge@chromium.org

Bug: 1129846
Change-Id: Ia00bb034ea0f4f0071c11b0c3d4dd43edf73d699
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2425166Reviewed-by: Darwin Huang <huangdarwin@chromium.org>
Commit-Queue: Darwin Huang <huangdarwin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809552}

Also make some UI adjustments to match specs.
Screenshots: http://dr/corp/drive/folders/1ga0FKZujvJKfSy3SG7zQ5ZwYxOSYZBNx

Bug: 1120093
Change-Id: Ic464538f1e99654ae5018bd0d51c8025fb1fd69a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2417463
Commit-Queue: Sophey Dong <sophey@chromium.org>
Reviewed-by: Tanya Gupta <tgupta@chromium.org>
Reviewed-by: Kyle Milka <kmilka@chromium.org>
Reviewed-by: Matthew Jones <mdjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809551}

The default thread size for Chrome on 32-bit Windows is 1.5 MiB which is
overkill for the BrokerServicesBase thread. Address-space exhaustion can
be an issue for 32-bit Chrome, especially on 32-bit Windows, so this
change reduces the size to 0.5 MiB.

Bug: 981238, 1023804
Change-Id: I8cc90b506f3a14895f8e7a84fc0138c0e1d0775e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2422685
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: Will Harris <wfh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809550}

The legacy fragmentation engine may fragment line boxes in some cases,
but this is wrong, since lines are monolithic.

Bug: 829028
Change-Id: I58c70487c16f9f14fdd3ae7698b5f8e0c255758e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2425044Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809549}

- Mocks changed and this functionality is now all combined
  in the battery status card
- This reverts commit a407643d.

Bug: 1125150
Test: browser_tests --gtest_filter=DiagnosticsApp*
Change-Id: I0a8f9356039885bba84a2df3a3eb3771893995c2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2422818Reviewed-by: Bailey Berro <baileyberro@chromium.org>
Commit-Queue: Zentaro Kavanagh <zentaro@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809548}

It's 12px in figma but the icon has some space by itself, so reduce
it to 8px. UX approved (http://screen/wRVCzrD4gDoQvmo) and seems
consistent with http://go/cros-ux-text-button-inner-layout

View of Add Input Methods: http://screen/3ESJMTZJkJuf5zy
View of Add Languages: http://screen/7FrqMDeEtWG6pds
Spec: http://go/cros-lang-settings-figma

Bug: 1113439
Change-Id: Id381b58ac38ecffac6896b34bc1572f7d95e483b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2423751Reviewed-by: Regan Hsu <hsuregan@chromium.org>
Commit-Queue: My Nguyen <myy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809547}

Two fields were given the ordinal @21. Due to a bug in the bindings
generator, this was allowed. In practice this meant the ordinal for the
second @21 field was actually @22, and all subsequent ordinals should be
bumped by one.

This CL corrects the assigned ordinals to reflect the reality of the
generated code. The stable format of this structure is unaffected by
this change.

NOPRESUBMIT=True

Bug: None
Change-Id: I82591eb4f36fa8e1121d5c0edfbfaa1656f57e31
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2422482
Commit-Queue: Ken Rockot <rockot@google.com>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809546}

SMB shares are now implemented by a FUSE filesystem that natively
supports statvfs(). Allow the available space on an SMB share to be
displayed in Files.app's gear menu.

Test: browser_tests --gtest_filter="*showAvailableStorage*"
Bug: 1082035
Change-Id: I144f2f7ff0cb3106ec33d3a5e9bff6a87d367c6e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2423125
Commit-Queue: Josh Simmons <simmonsjosh@google.com>
Reviewed-by: Luciano Pacheco <lucmult@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809545}

Removes large libraries that were pulled in as a result of an indirect
but necessary dep on Blink.

The exclusions can be removed along with the Blink MessagePort
dependency, once a fix for crbug.com/1126571 lands and is
integrated with NamedMessagePortConnector.

Bug: 1131102
Bug: 1126571
Change-Id: I7b25d50470d667f44424b925d3982efa268c24ae
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424512
Commit-Queue: Kevin Marshall <kmarshall@chromium.org>
Reviewed-by: Fabrice de Gans-Riberi <fdegans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809544}

Roll Chrome Win32 PGO profile from chrome-win32-master-1600754349-faa1edece0f28695a43c5cb3d7e6ca8a741e4346.profdata to chrome-win32-master-1600775907-34bfad4572405dae937638ebe3b1eeb7936a17be.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 sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@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: sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@google.com
Change-Id: Id2efa8cde75209a04b77414b6c39703745487806
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424405Reviewed-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@{#809543}

Test: none
Bug: 1111445
Change-Id: I8ee9546149c0c052677f897efedd7f950cd95c70
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2417370Reviewed-by: Min Chen <minch@chromium.org>
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809542}

We weren't properly checking for backward-compatibility (but were rather
checking for strict equality) of types nested within generics. So if a
stable structure S has a field of type `array<T>` and T is changing in
a backward-compatible way, S would trigger a presubmit failure.

This change corrects the logic here to properly nest such
backward-compatibility checks.

Fixed: 1130671
Change-Id: If739bac2440927fbaf24f7feae5bbd04f9b29c09
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2422189Reviewed-by: Oksana Zhuravlova <oksamyt@chromium.org>
Commit-Queue: Ken Rockot <rockot@google.com>
Cr-Commit-Position: refs/heads/master@{#809541}

Bug: 1112539
Change-Id: I4cd0774d2728b14559d7f45cab221b0c809713d9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2425103Reviewed-by: Garrett Beaty <gbeaty@chromium.org>
Commit-Queue: Garrett Beaty <gbeaty@chromium.org>
Auto-Submit: Erik Staab <estaab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809540}

https://chrome-internal.googlesource.com/chrome/deps/barhopper.git/+log/47934f2c634c..d5422b6ec462

$ git log 47934f2c6..d5422b6ec --date=short --no-merges --format='%ad %ae %s'
2020-09-21 azeemarshad [Barhopper] Fix tezcode build issues.

Created with:
  roll-dep src/third_party/barhopper

Change-Id: Id3113e08277cce3e34f0ba9da4528c93dae7c635
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424683Reviewed-by: Reilly Grant <reillyg@chromium.org>
Commit-Queue: Azeem Arshad <azeemarshad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809539}

This cl causes the bindings generator to automatically collect UKM when
these attributes are accessed. This will be used to analyze how much
entropy leaked by this API can be used for cross-site tracking.

Bug: 973801
Change-Id: Id53fa88b0181ec746d37d00677f2eeef045e300d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424847
Commit-Queue: Chris Fredrickson <cfredric@google.com>
Reviewed-by: David Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809538}

This CL changes BrowserURLHandlerImpl::RewriteURLIfNecessary so that it
returns early(and doesn't mutate the |url| in the in-out argument) if
|url| is invalid.  This helps avoid scenarios where
RewriteUrlForNavigation (in navigation_controller_impl.cc) ends up
generating a NavigationEntry with an invalid virtual URL that
(accidentally/incorrectly) gets rewritten into a valid URL.

Bug: 1116280
Change-Id: I114cf8c8d9459b6931ae659f62a100679b994d5e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2385921Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809537}

Bug: 1068326
Change-Id: I53d9b967f9304dcc02e1ee650bc68831f3b5d1df
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332845Reviewed-by: Erik Staab <estaab@chromium.org>
Commit-Queue: Haiyang Pan <hypan@google.com>
Cr-Commit-Position: refs/heads/master@{#809536}

This change enables the v3 PpdProvider.

Bug: chromium:888189
Test: simple-chrome deployment; set up printers
Change-Id: I01ec409ad8e53b9b733f46f5d7eeff45946e5bd9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424855Reviewed-by: Sean Kau <skau@chromium.org>
Commit-Queue: Kalvin Lee <kdlee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809535}

This CL implements a TODO in PendingConnectionManagerImpl to handle
incoming requests for connections via the Nearby Connections library.
When a request is received, we create a NearbyInitiatorConnectionAttempt
(if one does not already exist), then add a new
PendingNearbyInitiatorConnectionRequest to it.

Additionally, this CL updates the initialization flow for the
SecureChannel service so that it now creates an instance of
NearbyConnectionManager.

Bug: 1106937
Change-Id: Ibe56476babfb201b748beed1fefc80efda7e304e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419374
Commit-Queue: Kyle Horimoto <khorimoto@chromium.org>
Reviewed-by: James Vecore <vecore@google.com>
Cr-Commit-Position: refs/heads/master@{#809534}

This will allow lacros to synchronously determine ash's version. This
allows Lacros to avoid using unsupported APIs.

Bug: 1130810
Change-Id: I57fb355861b70b082499e123606f97366e0c3670
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2423203Reviewed-by: Greg Kerr <kerrnel@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Commit-Queue: Erik Chen <erikchen@chromium.org>
Auto-Submit: Erik Chen <erikchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809533}

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

Change-Id: I52691a6d8cb82341a08e0331081ec8a00b3c3d50
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424831Reviewed-by: Zhongyi Shi <zhongyi@chromium.org>
Commit-Queue: Zhongyi Shi <zhongyi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809531}

TBR=bling-team@google.com

Change-Id: If0bbf09a5520b9c86c816050f920f7fcd64a58c7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424664Reviewed-by: Internal Frameworks Autoroller <bling-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com>
Commit-Queue: Internal Frameworks Autoroller <bling-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#809530}

Roll Chrome Win64 PGO profile from chrome-win64-master-1600754349-e20db2f8d281c7f1a24165725105bb03ea6ece36.profdata to chrome-win64-master-1600775907-3f1dcf02f5358f2c935175c2f1e643612d0cf1c1.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-win64-chromium
Please CC sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@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:win64-chrome
Tbr: sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@google.com
Change-Id: I715833bd9b7c42dc64eb1bfa48e69d7f196d3024
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424407Reviewed-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@{#809529}

This CL adds media controls detailed view. When media controls is
clicked outside of action buttons, we will navigate user to a
detailed view showing all playing media.

Bug: 1129784
Change-Id: I9b84089caacc36229664096af0fbf4f8a31b3f74
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2416682
Commit-Queue: Jazz Xu <jazzhsu@chromium.org>
Reviewed-by: Tommy Steimel <steimel@chromium.org>
Reviewed-by: Jenny Zhang <jennyz@chromium.org>
Reviewed-by: Tim Song <tengs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809528}

This CL moves the Speech On-Device API (SODA) internal repo to the
Chromium repo in preparation for dogfooding in accordance with
LGPL.

Change-Id: I0d073f4e98a2423b86e43b069320dd61546f39d0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2404156
Commit-Queue: Evan Liu <evliu@google.com>
Reviewed-by: Colin Blundell <blundell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809527}

Unpinning an item from its view may result in that view being deleted.
If the view is deleted, we shouldn't attempt to update its pin.

Bug: 1130086
Change-Id: Ib5c8dbe086b4446161569924b3c81a9bd37d272c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2422247
Commit-Queue: David Black <dmblack@google.com>
Reviewed-by: Ahmed Mehfooz <amehfooz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809526}

Fixes a bug where the "Remove Account" search tag showed up in search
when there was no second account to remove.

Bug: 1116553
Change-Id: I800ebc0d667d0417303f0d11458ea152e1849589
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2422457
Commit-Queue: Daniel Classon <dclasson@google.com>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809525}

This CL also adds focus ring for indication of focus.

Bug: 1129694
Change-Id: I0140b01dd6d6ef758176b942622d222005a5710e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424824Reviewed-by: Ahmed Mehfooz <amehfooz@chromium.org>
Commit-Queue: David Black <dmblack@google.com>
Cr-Commit-Position: refs/heads/master@{#809524}

Add support for PinUvAuthToken on large blobs.

https://w3c.github.io/webauthn/#sctn-large-blob-extension

Bug: 1114875
Change-Id: I57a094eed4cb0cd3d8b20722804826a46ccf0e27
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2422707
Commit-Queue: Nina Satragno <nsatragno@chromium.org>
Auto-Submit: Nina Satragno <nsatragno@chromium.org>
Reviewed-by: Martin Kreichgauer <martinkr@google.com>
Cr-Commit-Position: refs/heads/master@{#809523}

This reverts commit f2f4de88.

Reason for revert: Crashing on Win7 bots
Example: https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8868424013558569488/+/steps/capture_unittests/0/logs/Deterministic_failure:_VideoCaptureDeviceMFWinTestWithDXGI.SimpleInit__status_CRASH_/0

Original change's description:
> [Win Camera Capture] Use D3D11 with Media Foundation Video Capture
> 
> This CL updates the VideoCaptureDeviceFactoryWin/VideoCaptureDeviceMFWin
> implementations to use D3D11 with the MF device source and capture
> engine. The VideoCaptureDeviceFactoryWin creates and manages an
> IMFDXGIDeviceManager and associated D3D11 device instance. It provides
> the IMFDXGIDeviceManager to the MF device source and also plumbs it to
> the VideoCaptureDeviceMFWin implementation. The VideoCaptureDeviceMFWin
> implementation subsequently provides the IMFDXGIDeviceManager to the MF
> capture engine on initialization.
> 
> Providing a MF DXGI device manager to the MF device source and capture
> engine is required for zero-copy capture (it allows the Windows frame
> server service and MF capture engine to keep captured video frames in
> GPU memory and expose MF samples backed by D3D11 textures).
> 
> Bug: 1120900
> Change-Id: Ic5367d03f337daab2220f8d87d1bfe51f6ca8b72
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2370092
> Reviewed-by: Guido Urdaneta <guidou@chromium.org>
> Reviewed-by: Markus Handell <handellm@google.com>
> Reviewed-by: Rafael Cintron <rafael.cintron@microsoft.com>
> Reviewed-by: Ilya Nikolaevskiy <ilnik@chromium.org>
> Reviewed-by: Sunny Sachanandani <sunnyps@chromium.org>
> Commit-Queue: Isuru Pathirana <Isuru.Pathirana@microsoft.com>
> Cr-Commit-Position: refs/heads/master@{#809472}

TBR=mcasas@chromium.org,stefan@webrtc.org,sunnyps@chromium.org,rafael.cintron@microsoft.com,guidou@chromium.org,chfremer@chromium.org,ilnik@chromium.org,handellm@google.com,Isuru.Pathirana@microsoft.com,handellm@chromium.org

Change-Id: I8d3559eb85d195d76263f905ce3884c879254e7f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1120900
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424898Reviewed-by: Leonard Grey <lgrey@chromium.org>
Commit-Queue: Leonard Grey <lgrey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809522}

All search results will be displayed using primary text coloring.

Bug: 1130822
Change-Id: Id55060e52e62d263da50ec8d507b742e812902c6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2421441Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Yulun Wu <yulunwu@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Commit-Queue: Yulun Wu <yulunwu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809521}

For some LayoutObjects that are "SVG children", we will not create and
associate a SVGResourceClient. Check for the presence of such an object
FragmentPaintPropertyTreeBuilder::UpdateFilter().

Bug: 109224, 1130986
Change-Id: Iab6e73b26a302282ecbe5235d671fbfe90753d62
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424124Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#809520}

It seems that the kDefaultDocumentClass should work just the same,
as nothing currently checks for kViewSourceDocumentClass.

Bug: 920571
Change-Id: I63088c00c0fa773f8c597f0f88768a984a6fc994
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424628
Commit-Queue: Joey Arhar <jarhar@chromium.org>
Auto-Submit: Mason Freed <masonfreed@chromium.org>
Reviewed-by: Joey Arhar <jarhar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809519}