GitLab

This reverts commit e2d97e7a.

Reason for revert: Reland with the null pointer issue fixed.

LayoutSVGEllipse and LayoutSVGRect override LayoutSVGShape methods not
to create paths in case that optimized algorithm can be used.
However, the original condition in their ShapeDependentStrokeContains()
might call LayoutSVGShape::ShapeDependentStrokeContains() without
use_path_fallback_ in some cases (e.g. when the shape is invalid).

Now ensure ShapeDependentStrokeContains() is called only if
use_path_fallback_ is set. Also ensure that use_path_fallback_ is set
whenever we need it.

Original change's description:
> Revert "[PE] Ensure update of LayoutSVGShape::NoScalingStrokeTransform"
>
> This reverts commit 85b96ea9.
>
> Reason for revert: crbug.com/849968
>
> Original change's description:
> > [PE] Ensure update of LayoutSVGShape::NoScalingStrokeTransform
> >
> > In Pre-SPv175 we forced subtree paint invalidation on non-composited
> > transform change. SPv175 no longer does that, causing
> > NonScalingStrokeTransform not updated on ancestor transform change.
> >
> > We also had a non-obvious bug that LayoutSVGShape::StrokeBoundingBox
> > didn't get updated on ancestor transform change.
> >
> > Now always explicitly update non-scaling-stroke data during layout.
> >
> > Bug: 849080
> > Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> > Change-Id: Ia61eb94f43e53a71a80e1102e4d605e4331f44b1
> > Reviewed-on: https://chromium-review.googlesource.com/1086715
> > Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
> > Reviewed-by: Fredrik Söderquist <fs@opera.com>
> > Cr-Commit-Position: refs/heads/master@{#564584}
>
> Change-Id: Ifbdd23b74dc45b5c8bc66c3d64bff580d5306f78
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 849080, 849968
> Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> Reviewed-on: https://chromium-review.googlesource.com/1087332
> Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#564793}

Bug: 849080, 849968
Change-Id: I15c2f80a2f80d11ccf356328ad41e8ab9d8de72f
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Reviewed-on: https://chromium-review.googlesource.com/1089090
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Reviewed-by: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#565050}

Move tough_image_decode_cases into the rendering folder and refactor
classes to inherit from RenderingStory

Bug: 849392
Change-Id: I888b1226bc8663c28d79e5871215547af1052b4a
Reviewed-on: https://chromium-review.googlesource.com/1089203
Commit-Queue: Amy Qiu <amyqiu@google.com>
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#565049}

This is a step in unifying canvas and OffscreenCanvas code paths

BUG=788439

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I6f3a3f7bed8b36f502bc7ce138b4bf21920629f7
Reviewed-on: https://chromium-review.googlesource.com/1089582Reviewed-by: Fernando Serboncini <fserb@chromium.org>
Commit-Queue: Justin Novosad <junov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565048}

Change-Id: I4121159c9326842a2e868a58e34a00359b1756c9
Reviewed-on: https://chromium-review.googlesource.com/1088312Reviewed-by: Mike Wittman <wittman@chromium.org>
Commit-Queue: Xi Cheng <chengx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565047}

How I generated this: https://gist.github.com/danbeam/d5fe7b1bcfebc9a94d60c9975b842e97

R=dpapad@chromium.org
BUG=none

Change-Id: Ic8d85539990d9ccc4a2e0faa8a3928ee2c878adf
Reviewed-on: https://chromium-review.googlesource.com/1089716
Commit-Queue: Dan Beam (no longer on Chrome) <dbeam@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565046}

This reverts commit cd9fa081.

This CL fixes the reason that the DCHECK failed causing
the original revert.

Original change's description:
> Revert "[ios] Hook up remote tabs in tab grid"
>
> This reverts commit 5316b1cb.
>
> Reason for revert: breaking ios_chrome_settings_egtests
>
> Original change's description:
> > [ios] Hook up remote tabs in tab grid
> >
> > This CL hooks up the remote tabs in the tab grid.
> >
> > A user can sign in from the remote tabs page in the tab grid. The
> > transition animation that displays the signin interaction controller
> > looks bad (and we are investigating the cause). If the user is signed
> > in but does not have sync enabled, the user can enable sync from here.
> >
> > Currently, tapping on a tab does nothing. This will be enabled in a
> > future CL.
> >
> > Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
> > Change-Id: Ifba06919e1fb496917254a0da7e5462a4b854d79
> > Reviewed-on: https://chromium-review.googlesource.com/1084080
> > Commit-Queue: edchin <edchin@chromium.org>
> > Reviewed-by: Eugene But <eugenebut@chromium.org>
> > Reviewed-by: Sergio Collazos <sczs@chromium.org>
> > Reviewed-by: edchin <edchin@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#564684}
>
> TBR=rohitrao@chromium.org,eugenebut@chromium.org,edchin@chromium.org,sczs@chromium.org,javierrobles@chromium.org
>
> Change-Id: Idc0b0b2def356a173949298bc7f560ee4dad3744
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
> Reviewed-on: https://chromium-review.googlesource.com/1088566
> Reviewed-by: Olivier Robin <olivierrobin@chromium.org>
> Commit-Queue: Olivier Robin <olivierrobin@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#564828}

TBR=rohitrao@chromium.org,eugenebut@chromium.org,olivierrobin@chromium.org,edchin@chromium.org,sczs@chromium.org

Change-Id: I34dca3dadbe0d356aff881559353876736722224
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Reviewed-on: https://chromium-review.googlesource.com/1089251Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Reviewed-by: Sergio Collazos <sczs@chromium.org>
Reviewed-by: edchin <edchin@chromium.org>
Commit-Queue: edchin <edchin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565045}

Change-Id: I81c5655e357b6211ccb32f1b82b829bb8d402f0b
Reviewed-on: https://chromium-review.googlesource.com/1088230
Commit-Queue: Jordy Greenblatt <jordynass@chromium.org>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565044}

The closure in case is ContentHash::DidFetchVerifiedContents
which takes a |const FetchParams&| parameter, but does not use it.
This CL removes it.

This is a driven by simplication as part of the preparation for
making FetchParams non-copyable.

Next, FetchParams will store a scoped_refptr<network::SharedURLLoaderFactory>
rather than a raw network::mojom::URLLoaderFactory pointer.

BUG=844926

Change-Id: I237c93be558f4cc7c65c5de2f73e8722a246863d
Reviewed-on: https://chromium-review.googlesource.com/1089272Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
Cr-Commit-Position: refs/heads/master@{#565043}

This reverts commit f58a0ada.

Reason for revert: Suspecting this CL for the failures of CrSettingsSiteListTest.AddExceptionDialog and CrSettingsSiteListTest.EditExceptionDialog. https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=CrSettingsSiteListTest.*ExceptionDialog 

Original change's description:
> WebUI[MD-refresh]: switches remaining paper-input in Settings to cr-input.
> 
> Bug: 832177
> Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
> Change-Id: Iae1036e0a8975f65a37ff9ccc7b6084b4a6d45ef
> Reviewed-on: https://chromium-review.googlesource.com/1081243
> Commit-Queue: Scott Chen <scottchen@chromium.org>
> Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#564968}

TBR=dpapad@chromium.org,scottchen@chromium.org

Change-Id: I4a0a973a5d54324389e2164e041bfb085dffd631
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 832177
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Reviewed-on: https://chromium-review.googlesource.com/1089870Reviewed-by: Kevin McNee <mcnee@chromium.org>
Commit-Queue: Kevin McNee <mcnee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565042}

WebAPK shell currently sends us activity creation timestamp in the
intent extras. This CL wires it through to the splashscreen UMA handler
and records time to splashscreen shown/hidden from the launch of the
WebAPK.

Bug: 842023
Change-Id: Ie8c7289de4ce0f31cddec3bfec8e538a73469ae9
Reviewed-on: https://chromium-review.googlesource.com/1073875
Commit-Queue: Mohamed Heikal <mheikal@chromium.org>
Reviewed-by: Brian White <bcwhite@chromium.org>
Reviewed-by: Peter Kotwicz <pkotwicz@chromium.org>
Reviewed-by: Samuel Huang <huangs@chromium.org>
Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565041}

Two changes are required:

1) Explicitly upcast the result of dispatch_get_global_queue() to a
dispatch_queue_t*, since it changed to return dispatch_queue_global_t*;
2) Add empty implementations of the new methods on ICCameraDeviceDelegate for
ImageCaptureDevice

TBR=maxmorin@chromium.org

Bug: 849689
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_optional_gpu_tests_rel
Change-Id: I9448d1b07f8d4116ecdb0c75a7796472e51c6dd8
Reviewed-on: https://chromium-review.googlesource.com/1087169Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Reviewed-by: Robert Sesek <rsesek@chromium.org>
Reviewed-by: Tommy Li <tommycli@chromium.org>
Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565040}

As part of Mojoification of VRDevice, this change removes ExitPresent,
and moves exclusive session management out of the VRDevice interface.

BUG=795510

Change-Id: I60c7dca54b62eb518e1895d7a2d0df628fbd84d8
Reviewed-on: https://chromium-review.googlesource.com/1087718
Commit-Queue: Bill Orr <billorr@chromium.org>
Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565039}

This CL adds CSS to use ellipsis when overflow menu text is too long.

Bug: 839107
Change-Id: I367549c2e669d8e501f70157217544af96e74f0d
Reviewed-on: https://chromium-review.googlesource.com/1089382Reviewed-by: Becca Hughes <beccahughes@chromium.org>
Commit-Queue: Tommy Steimel <steimel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565038}

Separate out the different concepts:
- Suggestions are available or not (call ViewAccessibility::OnInputSuggestionsAvailable or
OnInputSuggestionsUnavailable). This is now a static so no view is required in case the menu
for the suggestions has not yet been created.
- Menu is open -- the AX code now just uses the menustart/end event to track this
- Menu item is selected -- the AX code now just uses the selection event on a menuitem when
menus are open

By separating out these concepts, a11y can expose the fake menu item focus only when a menu
item is actually selected, and indicate when a textfield has suggestions available even
if the menu popup is not visible.

Bug: 848427
Change-Id: I4a75c58402d1dd1e5da0368b59f6c9ba7fc8775c
Reviewed-on: https://chromium-review.googlesource.com/1081022
Commit-Queue: Aaron Leventhal <aleventhal@chromium.org>
Reviewed-by: Evan Stade <estade@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: Fabio Tirelo <ftirelo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565037}

Before using the UsageTimeLimit on the TimeLimitProcessor we convert this to higher level structs, that allow the processor to work better. I was going to submit all together, but it was too big, so I split into two cls.

These structs here didn't need to have their interfaces public, since its going to be used solely by the TimeLimitProcessor. However, I exposed its interface on the internal namespace in order to make it testable. Usually, we don't want to test private code, but these converters have a complex logic, and I thought it was better to have this explicitly tested.

Bug: 823536
Change-Id: Ia90c5e83b5d0f149ae93024a1671b93850584f91
Reviewed-on: https://chromium-review.googlesource.com/1071687
Commit-Queue: Rahul Chaturvedi <rkc@chromium.org>
Reviewed-by: Rahul Chaturvedi <rkc@chromium.org>
Reviewed-by: Jacob Dufault <jdufault@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565036}

This CL adds LSQ predictor to input prediction.
LSQ predictor uses a quadratic least square regression model:
y = b0 + b1 * x + b2 * x ^ 2.

The resampling is still behind flag: kResampleInputEvents.

design doc:
https://docs.google.com/document/d/1DLfibi2NkV85p7AfEtNkvy24d283VRBSd3hz1Qh24Jw/edit#

Bug: 836352
Change-Id: Ie1959c6b95152a80076debfb762c49d88a758af3
Reviewed-on: https://chromium-review.googlesource.com/1003220
Commit-Queue: Ella Ge <eirage@chromium.org>
Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: danakj <danakj@chromium.org>
Reviewed-by: Navid Zolghadr <nzolghadr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565035}

Bug: 65337
Change-Id: I60eccbb32f782f98f147c95f8bbbebf7f92a7c34
Reviewed-on: https://chromium-review.googlesource.com/1082009
Commit-Queue: David Bertoni <dbertoni@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565034}

Matching the recents changes in libwebrtc[0] enable MediaTek just for H264

[0] https://chromium.googlesource.com/external/webrtc/+/5b6891afddcef18ea699efcd457dfab80c22ac70

BUG=848511
R=braveyao@chromium.org

Change-Id: I0431b48730eb14149530a1265d173cca7e8944f3
Reviewed-on: https://chromium-review.googlesource.com/1086408Reviewed-by: Weiyong Yao <braveyao@chromium.org>
Reviewed-by: Chrome Cunningham <chcunningham@chromium.org>
Commit-Queue: Weiyong Yao <braveyao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565033}

The comment refers to a field that does not exist.

Bug: 746182
Change-Id: I76586b938980f95e93a01386ea7bde7844d59f4d
Reviewed-on: https://chromium-review.googlesource.com/1087324Reviewed-by: Frank Liberato <liberato@chromium.org>
Commit-Queue: CJ DiMeglio <lethalantidote@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565032}

Freezing can only work on hidden pages, currently there is no easy way
in webdriver that allows us to hide the page.

A new webdriver API is in the works to enable that, but meanwhile, for
all testing API, we have to couple hiding with freezing.

This CL does that and reenables the test with Network Service.

Bug: 837709, 848604
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: Ib5c513aefb219cef38a563be81fe8abf4b354051
Reviewed-on: https://chromium-review.googlesource.com/1089354
Commit-Queue: Fadi Meawad <fmeawad@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565031}

The tests are passing on CI now, so make it block the CQ.

NOTRY=true

Bug: 831847
Change-Id: I9767bb792b99cf3a47975062f6820c1fc82f8f6a
Reviewed-on: https://chromium-review.googlesource.com/1035794
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565030}

Leave the flag in place so we can revert the state in case we have any blockers.

R=escordeiro@chromium.org, yusukes@chromium.org

Bug: 828983
Change-Id: I9886bde062fc076acc34fe14cf42203784ef18f5
Reviewed-on: https://chromium-review.googlesource.com/1089150
Commit-Queue: B. Menegola <menegola@chromium.org>
Reviewed-by: Yusuke Sato <yusukes@chromium.org>
Reviewed-by: Yury Khmel <khmel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565029}

- Make test constant names more specific.
- Use auto where applicable.
- using Requirement =
  blink::WebMediaKeySystemConfiguration::Requirement;
- Move ASSERT_TRUE into SelectConfig*() functions.
- clang format

Bug: 848532
Test: Test cleanup only
Change-Id: I8c7613dad4c582adfcfb3d9d97c5456f86177257
Reviewed-on: https://chromium-review.googlesource.com/1089465Reviewed-by: Dan Sanders <sandersd@chromium.org>
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565028}

This changes sign_app.sh to take a provisioning profile path as an
argument, which it will copy into the app bundle Content/ directory. It
then signs the app with --entitlements pointing at a newly created
entitlements.plist file, in order to grant the keychain-access-group
entitlement.

The entitlement is needed to create and access cryptographic key
material in the Secure Enclave Processor (SEP) on compatible macOS
devices.

Bug: 848052
Change-Id: Ib002fc73d250409e555f4caee8848a2c3e95e591
Reviewed-on: https://chromium-review.googlesource.com/1079833
Commit-Queue: Martin Kreichgauer <martinkr@google.com>
Reviewed-by: Robert Sesek <rsesek@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565027}

It was reporting a preferred size inconsistent with its Layout(),
causing potential clipping.

To fix, anticipate the required alignment using constants.

BUG=847904
R=bsep@chromium.org
CC=tapted@chromium.org

Change-Id: I775dbe8a346839914b6dda4965a9c0e97c852661
Reviewed-on: https://chromium-review.googlesource.com/1087965Reviewed-by: Bret Sepulveda <bsep@chromium.org>
Commit-Queue: Greg Thompson <grt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565026}

Bug: 850078
Test: browser_tests
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I026e1098fedd85f6cf413d74fe596934794ae001
Reviewed-on: https://chromium-review.googlesource.com/1088873
Commit-Queue: Mathieu Perreault <mathp@chromium.org>
Reviewed-by: anthonyvd <anthonyvd@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565025}

BUG=849294

Change-Id: I8bf82b86b517d3c269af2a8580df86aa546a88ed
Reviewed-on: https://chromium-review.googlesource.com/1089351Reviewed-by: Yusuf Ozuysal <yusufo@chromium.org>
Reviewed-by: Theresa <twellington@chromium.org>
Commit-Queue: Ted Choc <tedchoc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565024}

Re-submission with fixes of cl 1081234 (reverted by 1087371.)

  * Move the cast AccessibilityManager into
    chromecast/browser/accessibility.
  * Only compile and create AccessibilityManager if extensions
    enabled.

Bug: none
Test: manual
Change-Id: Id81fdb1711af84d8eecefd2061cca061826df39a
Reviewed-on: https://chromium-review.googlesource.com/1087650Reviewed-by: Alex Sakhartchouk <alexst@chromium.org>
Reviewed-by: Wez <wez@chromium.org>
Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Commit-Queue: Ryan Daum <rdaum@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565023}

This reverts commit 1ff37758.

Reason for revert: external/wpt/webusb/idlharness.https.any.* tests are failing

Original change's description:
> Multi-global WebUSB WPT for window and worker
> 
> This change updates the existing web platform tests for WebUSB to use
> multi-global tests to test the API in the supported contexts. The
> change updates the following tests:
> * idlharness
> * usb
> * usbConnectionEvent
> 
> Bug: 841510
> Change-Id: I6b118aa4121109cd6a48054c2ebc69b831660afc
> Reviewed-on: https://chromium-review.googlesource.com/1087209
> Reviewed-by: Reilly Grant <reillyg@chromium.org>
> Commit-Queue: Ovidio Henriquez <odejesush@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#564981}

TBR=reillyg@chromium.org,chongz@chromium.org,odejesush@chromium.org

Change-Id: Ifde97a1736134477422b6fd81392b11bbd775a63
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 841510
Reviewed-on: https://chromium-review.googlesource.com/1089690Reviewed-by: Kevin McNee <mcnee@chromium.org>
Commit-Queue: Kevin McNee <mcnee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565022}

All the Aimages associated with AImageReader must be closed/deleted
before we close/delete the AImageReader.

Bug: 849218
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_optional_gpu_tests_rel
Change-Id: I9b013a7da53f0da716b869c48fe2ee024bee21a3
Reviewed-on: https://chromium-review.googlesource.com/1087827
Commit-Queue: vikas soni <vikassoni@chromium.org>
Reviewed-by: Frank Liberato <liberato@chromium.org>
Reviewed-by: Eric Karl <ericrk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565021}

Noticed this presubmit warning when working on another CL which modifies
this file. This CL adds explicit for single-parameter constructors in
this file.

Bug: None
Change-Id: I03dc030cc981751bf754f0b5f391afa747f019d9
Reviewed-on: https://chromium-review.googlesource.com/1089357Reviewed-by: Jian Li <jianli@chromium.org>
Commit-Queue: Jian Li <jianli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565020}

https://chromium.googlesource.com/catapult.git/+log/01f608b..1b40071


git log 01f608b..1b40071 --date=short --no-merges --format='%ad %ae %s'
2018-06-06 dtu@chromium.org [pinpoint] Plumb Swarming test exceptions to the job level.


Created with:
  gclient setdep -r src/third_party/catapult@1b40071

The AutoRoll server is located here: https://catapult-roll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
TBR=sullivan@chromium.org

Change-Id: I7ac3c50b2389dbf02afbaf151d2a106ac1cf9ca9
Reviewed-on: https://chromium-review.googlesource.com/1089490Reviewed-by: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#565019}

Turn on using pdf compositor service for printing by default. Still keep
the feature flag as a kill switch for now.

BUG=824867

Change-Id: I5b7751c5ce453a7ffcda1bf38b5f141135f850f6
Reviewed-on: https://chromium-review.googlesource.com/1087872Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: Charlie Reis <creis@chromium.org>
Commit-Queue: Wei Li <weili@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565018}

Bug: 807651, 848909
Change-Id: If109b4fed7cdbfcf3774335bdebc1db4b9182b4c
Reviewed-on: https://chromium-review.googlesource.com/1083494
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Reviewed-by: Charlie Reis <creis@chromium.org>
Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565017}

A few entry points were missing the correct validation.

Tested with improved conformance tests in
https://github.com/KhronosGroup/WebGL/pull/2654 .

Bug: 848914
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_optional_gpu_tests_rel
Change-Id: Ib98a61cc5bf378d1b3338b04acd7e1bc4c2fe008
Reviewed-on: https://chromium-review.googlesource.com/1086718Reviewed-by: Kai Ninomiya <kainino@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Commit-Queue: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565016}

R=khorimoto@chromium.org

Bug: 824568, 752273
Change-Id: I686f20e3933d35d58d94a5d83f89d5e171882b70
Reviewed-on: https://chromium-review.googlesource.com/1089373Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Commit-Queue: Ryan Hansberry <hansberry@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565015}

Bug: 849541
Change-Id: Idbd3c3a0339c933c70a5a2335a1faca5f507eb72
Reviewed-on: https://chromium-review.googlesource.com/1089118
Commit-Queue: Emily Hanley <eyaich@chromium.org>
Commit-Queue: Ned Nguyen <nednguyen@google.com>
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#565014}

When a connection is made, we remove the node from the list of tail
processing nodes.  But we didn't remove the node from the list of
finished handlers.  This is needed too because if we don't, when the
finished handlers are processed, they'll disable their outputs
incorrectly.

Bug: 849189
Test: 
Change-Id: I11a5467d8ed42d9819b19a63c27e7838225f39e1
Reviewed-on: https://chromium-review.googlesource.com/1087725Reviewed-by: Hongchan Choi <hongchan@chromium.org>
Commit-Queue: Raymond Toy <rtoy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565013}

Other versions of media/controls/modern/scrubbing.html are marked slow,
but not virtual/new-remote-playback-pipeline/, causing this to flakily
time out.

Bug: 793771, 850116
Change-Id: I3a45716ebd797f781573b2d5d86dfa58b213b271
Reviewed-on: https://chromium-review.googlesource.com/1089384Reviewed-by: Tommy Steimel <steimel@chromium.org>
Commit-Queue: Kevin McNee <mcnee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565012}

This reverts commit 6d48958d.

Reason for revert: https://bugs.chromium.org/p/chromium/issues/detail?id=850084

Original change's description:
> Ignore Intent filters with wildcard hosts when handling links.
> 
> If the Android device has a package installed that declares
> an Intent filter with `android:host="*"`, Chrome will launch
> the Intent picker UX whenever a person taps a link.
> This is a poor user experience. (Using `android:host="*"`
> arguably doesn't make much sense; normally, Intent filters
> would use `*` to match subdomains such as *.google.com.) Thus,
> skip Intent filters with such wildcard hosts when handling
> acceptable scheme links.
> 
> R=​dtrainor@chromium.org
> 
> Bug: 841133
> Change-Id: I3cb14309ed53e0bd623ed0da59af6635391a8718
> Reviewed-on: https://chromium-review.googlesource.com/1053847
> Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
> Reviewed-by: Chris Palmer <palmer@chromium.org>
> Reviewed-by: Maria Khomenko <mariakhomenko@chromium.org>
> Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#564487}

TBR=palmer@chromium.org,mthiesse@chromium.org,dtrainor@chromium.org,mariakhomenko@chromium.org,huangxueqing@xiaomi.com

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

Bug: 841133
Change-Id: I204d582d45dd27d25f47db45a6dac07d02f351b7
Reviewed-on: https://chromium-review.googlesource.com/1089470Reviewed-by: Maria Khomenko <mariakhomenko@chromium.org>
Commit-Queue: Maria Khomenko <mariakhomenko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565011}