GitLab

Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Id605c1ab97e273ae9e783e3925457e5ca87daf5c
Reviewed-on: https://chromium-review.googlesource.com/809126
Commit-Queue: Chris Pickel <sfiera@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522057}

This reverts commit c47b4d11.

BUG=chromium:791131
TEST=video_ChromeRTCHWDecodeUsed on peach_pit
Reason for revert: This CL breaks video_ChromeRTCHWDecodeUsed

Original change's description:
> WebRTC: Migrate completely to new video codec factories API
> 
> This CL makes the Chromium HW video factories implement the new
> interfaces directly. It also adds SW codecs, simulcast, and SW fallback
> codecs manually, instead of relying on WebRTC to do this. This will
> give Chromium more flexibility in the future, and will allow us to
> simplify the WebRTC code, since it's only Chromium that uses some of
> these features.
> 
> Bug: webrtc:7925
> Change-Id: I2bbd8b9a8c52bfe963d601f34188faf47d59e67e
> Reviewed-on: https://chromium-review.googlesource.com/788410
> Reviewed-by: Kentaro Hara <haraken@chromium.org>
> Reviewed-by: Emircan Uysaler <emircan@chromium.org>
> Commit-Queue: Magnus Jedvert <magjed@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#520062}

TBR=haraken@chromium.org,emircan@chromium.org,magjed@chromium.org

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

Bug: webrtc:7925
Change-Id: I081f9f5d961707924fe6eed4a97ead9c9f5ed12c
Reviewed-on: https://chromium-review.googlesource.com/810429Reviewed-by: Pawel Osciak <posciak@chromium.org>
Commit-Queue: Pawel Osciak <posciak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522056}

This is a second reland attempt for
07edac9e after a fix for the
underlying issue has landed:
https://chromium-review.googlesource.com/c/chromium/src/+/807664

Original change's description:
> Migrate away from deprecated RunLoop API
>
> Instead, we keep around an instance of base::RunLoop and pass it
> explicitly for Quit(), which makes sure the code quits the intended loop
> and not a nested one.
>
> Bug: 783774
> Change-Id: I3ea5092699bbae127f581e5c955cb27904d98f0a
> Reviewed-on: https://chromium-review.googlesource.com/784790
> Reviewed-by: Pavel Yatsuk <pavely@chromium.org>
> Commit-Queue: Mikel Astiz <mastiz@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#519664}

TBR=pavely@chromium.org

Bug: 783774
Change-Id: I615b4a92ff59a8f4cf871d63327bb8ba063d1cfc
Reviewed-on: https://chromium-review.googlesource.com/803314Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522055}

Bug: NONE
Change-Id: I5e79a366942948f8361f25db0d2cae68a412c016
Reviewed-on: https://chromium-review.googlesource.com/809107
Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522054}

TBR=chrome-os-gardeners@google.com

Change-Id: I95aacbb926d1b3923e51160be0a40a004bc93a67
Reviewed-on: https://chromium-review.googlesource.com/809823Reviewed-by: ChromeOS Commit Bot <chromeos-commit-bot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522053}

First create an EGLImage from a given gl texture.
Then export the EGLImage as dmabuf.

In addition to the new unit test included in this CL,
this will be useful for the subsequent CL,
https://chromium-review.googlesource.com/c/chromium/src/+/766787
which will enable existing VaapiDrmPicture when passing --use-gl=egl.
While existing VaapiTFPPicture(X11) will be left for --use-gl=desktop.

Added unit test ui/gl/gl_image_native_pixmap_unittest.cc
The test is enabled on non-ozone but should work on ozone too.
If any of the pre-conditions fail then the test is skipped.
Specifically it requires the extension EGL_MESA_image_dma_buf_export.
So it works on Intel, AMDGPU and nouveau drivers but skipped on NVIDIA.

Bug: 785201
Test: gl_unittests --gtest_filter=*NativePixmap*

Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Id9b4e56aaaaf5fd9994f4b144ce74a234d848253
Reviewed-on: https://chromium-review.googlesource.com/766687
Commit-Queue: Julien Isorce <julien.isorce@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522052}

We have not supported valgrind for some time (it was never ported to
the GN build IIUC, and has been obsoleted by ASan/MSan/TSan).

BUG=791518

Change-Id: I9d6f5125de817b9334b295a8a78589004bb08294
Reviewed-on: https://chromium-review.googlesource.com/809132
Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com>
Reviewed-by: Mark Seaborn <mseaborn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522051}

Service Factory does not own the service. BrowserStates owns it. Service
Factory only creates that service.

This CL updates the comments where appropriate.

Bug: None
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Icaf33df99b1c27f7c9be4a098e1ca25d86bd0687
Reviewed-on: https://chromium-review.googlesource.com/809816
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522050}

We are adding network traffic annotation to all data send requests in
Chrome to provide transparency and audibility.
This CL is the first step in adding this argument to sockets data send.
In this CL the argument is added to "Write" function of
"net/socket/socket.h", and all files affected by this change are
minimally updated to compile. The added argument to all functions
has a default value to minimize code change. Once all callers have been
updated to pass a traffic annotation, the default will be removed.

Bug: 656607
Change-Id: Ib3bef20ffa8d58d9b2fedc8b6970ff0cf66f844b
Reviewed-on: https://chromium-review.googlesource.com/806219
Commit-Queue: Ramin Halavati <rhalavati@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Eric Roman <eroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522049}

After we remove LayoutBoxModelObject::SetSelectionState() implementation,
LayoutObject::SetSelectionState() is no longer needed to be virtual
since there are no other calls.
Thus, we can remove |virtual| from LayoutObject::SetSelectionState().

Bug: 739062
Change-Id: I49b1426f9fc89586d1e818b10385c18c5223399e
Reviewed-on: https://chromium-review.googlesource.com/802603
Commit-Queue: Yoichi Osato <yoichio@chromium.org>
Reviewed-by: yosin (OOO Dec 11 to Jan 8) <yosin@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522048}

Directory components/signin/core/common was removed but still mentioned
in several DEPS files.

Bug: 
Change-Id: Ie1ab30ed38463e84953a8f98108e7ac7e19041cb
Reviewed-on: https://chromium-review.googlesource.com/807904Reviewed-by: Vaclav Brozek <vabr@chromium.org>
Commit-Queue: Alexander Yashkin <a-v-y@yandex-team.ru>
Cr-Commit-Position: refs/heads/master@{#522047}

::slotted rules are treated as a separate set of style rules
for Shadow DOM V1 fast path, but once V0 shadow root is created
anywhere in the document, the set has to be merged into
tree boundary crossing rule sets.

This is a regression when V1 fast path for ::slotted rules was
introduced at https://crrev.com/492282.

To trigger this bug, the following conditions have to be met:
1. V1 shadow root is added to the document and
   stylesheet including ::slotted() rules is added and resolved
   (the style is applied to some part)
2. V0 shadow root is added to the same document
3. style for the element which ::slotted() rule would apply is
   recalculated
   (Expected: ::slotted() rule still applies, Actual: it does not)

If V0 shadow root is created at the same time or before the
style sheet including ::slotted() rules is added to the document,
this bug doesn't occur.

Bug: 791400
Change-Id: I11bd028cf8020f11a5ab42cb680a4c8c18e47c2d
Reviewed-on: https://chromium-review.googlesource.com/807968
Commit-Queue: Takayoshi Kochi <kochi@chromium.org>
Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522046}

We have not supported valgrind for some time (it was never ported to
the GN build IIUC, and has been obsoleted by ASan/MSan/TSan).

BUG=791518

Change-Id: Id24289347d0811ed51a0b9ac036c746ef18bf75f
Reviewed-on: https://chromium-review.googlesource.com/809426Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com>
Cr-Commit-Position: refs/heads/master@{#522045}

This patch asserts that we clear the NonAttachedStyle at the end of
attaching LayoutTree. This ensures correctness of
https://chromium-review.googlesource.com/c/chromium/src/+/765097

Bug: 792080
Change-Id: Ia3a1d376bddb84fb4c8751739144ef07e3bd18ae
Reviewed-on: https://chromium-review.googlesource.com/809989Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Commit-Queue: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522044}

This is part of efforts to replace base::ThreadTaskRunnerHandle::Get()
with other appropriate task runners in the renderer.

Bug: 786332
Change-Id: Ie55366522fb749fb895806b0ec423a22e9842330
Reviewed-on: https://chromium-review.googlesource.com/808093
Commit-Queue: Tommi <tommi@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Reviewed-by: Tommi <tommi@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522043}

We have not supported valgrind for some time (it was never ported to
the GN build IIUC, and has been obsoleted by ASan/MSan/TSan).

BUG=791518

Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I0948b46f523be2a28894306b9377646d111c22a8
Reviewed-on: https://chromium-review.googlesource.com/809131Reviewed-by: Antoine Labour <piman@chromium.org>
Reviewed-by: Abhishek Arya <inferno@chromium.org>
Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com>
Cr-Commit-Position: refs/heads/master@{#522042}

These tests were added/changed in r522016.
-SiteIsolationStatsGathererBrowserTest/SiteIsolationStatsGathererBrowserTest.CrossSiteDocumentBlockingForMimeType/0
-SiteIsolationStatsGathererBrowserTest/SiteIsolationStatsGathererBrowserTest.CrossSiteDocumentBlockingForMimeType/1
-CrossSiteDocumentBlockingTest.BlockDocuments
-CrossSiteDocumentBlockingIsolatedOriginTest.BlockDocumentsFromIsolatedOrigin
-CrossSiteDocumentBlockingTest.RangeRequest

Most failures (even the non-StatsGatherer ones) seem to be about histograms.

Bug: 786505, 729848
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I2302d55d5de41c904c38c675fc57122316ceec15
NOTRY: true
TBR: jam
Reviewed-on: https://chromium-review.googlesource.com/810247Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522041}

This patch implements serialization of CSSUnparsedValue correctly.
There was a previous implementation, but stuff like 'var(--A)'
serialized to '--A', not 'var(--A)'.

Spec: https://drafts.css-houdini.org/css-typed-om-1/#unparsedvalue-serialization

Bug: 782103
Change-Id: Ieaecddb2f1a271b023e4fb7a720077e5ffcfc01f
Reviewed-on: https://chromium-review.googlesource.com/805176
Commit-Queue: Darren Shen <shend@chromium.org>
Reviewed-by: nainar <nainar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522040}

The tests changed names in r521963:
DeclarativeNetRequestResourceTypeBrowserTest.Test1/0
DeclarativeNetRequestResourceTypeBrowserTest.Test1/1
DeclarativeNetRequestResourceTypeBrowserTest.Test2/1
DeclarativeNetRequestResourceTypeBrowserTest.Test2/0

Bug: 721414
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I9979c953d34305968f6a2eacc3873eadc2faae32
NOTRY: true
TBR: jam
Reviewed-on: https://chromium-review.googlesource.com/810445Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522039}

InstallableManager caches the data it fetches as part of the
installability check. This means that sites which change their manifest,
or inject a manifest after the check is run are incorrectly classified.

This CL makes InstallableManager listen to the
WebContentsObserver::DidUpdateWebManifestURL() method, and invalidates
its internal state when the method fires. This addresses the issue by
making InstallableManager refetch data when we know that the manifest
URL is different.

Tests are added to ensure the correct behaviour.

BUG=792299

Change-Id: If73128b7b86bd741c16235c8a8e7ec5f0caeb165
Reviewed-on: https://chromium-review.googlesource.com/809994
Commit-Queue: Dominick Ng <dominickn@chromium.org>
Reviewed-by: Ben Wells <benwells@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522038}

This CL adds a new method of the profiling_service mojom interface which
asynchronously returns the list of profiled processes. This is used in two places:
  * By the chrome://memory-internals to determine whether a process is being
  profiled. This also allows chrome://memory-internals to expose a "start
  profiling" button for non-profiled processes.
  * By the background profiling triggers, since traces should only be triggered
  if a profiled process exceeds the thresholds.

Bug: 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I44da48eb1203b82f9cef8907fc7c2ab0fdcfdfce
Reviewed-on: https://chromium-review.googlesource.com/809505
Commit-Queue: Erik Chen <erikchen@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Albert J. Wong <ajwong@chromium.org>
Reviewed-by: Brett Wilson <brettw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522037}

This Cl initializes new User object with user_type from UserContext.
This way we no longer create users (with profiles) of type "Regular",
and then change it to some other type.

- ChangeUserChildStatus notification is removed.
- User object is no longer allowed to change type after initialization
  (except for Supervised users due to legacy reasons)
- New User object user type is initialized from UserContext user type.

TBR=sky@chromium.org,elijahtaylor@chromium.org

Bug: 789068
Change-Id: Ia8ce5e75f3baa342af87fe5d5b860e444284c783
Reviewed-on: https://chromium-review.googlesource.com/804968
Commit-Queue: Alexander Alekseev <alemate@chromium.org>
Reviewed-by: Achuith Bhandarkar <achuith@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Mihai Sardarescu <msarda@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522036}

Modernize some code and fix nits.

Change-Id: I5dfed71d27cc7d8a9638a88dc8e575ffe5a47d7d
Reviewed-on: https://chromium-review.googlesource.com/804700
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522035}

Bug: 640621
Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I33516d4a22a9f2da017749f1cf9eb8cd04314725
Reviewed-on: https://chromium-review.googlesource.com/809192Reviewed-by: smut <smut@google.com>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Reviewed-by: Andrei Kapishnikov <kapishnikov@chromium.org>
Reviewed-by: Miriam Gershenson <mgersh@chromium.org>
Commit-Queue: Andrei Kapishnikov <kapishnikov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522034}

This is a clean-up and doesn't change behavior. Worklets care about the lifetime
of Document not LocalFrame. To make it clearer, this CL removes dependency on
LocalFrame from Worklet.

Bug: 792307, 792108
Change-Id: I24268c2f59efa70f26a0024bb02ee8a35d01ba7f
Reviewed-on: https://chromium-review.googlesource.com/810405
Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522033}

Migrate ConnectionTypeObserver to NetworkChangeObserver in job_scheduler.h

Bug: 780475
Change-Id: I4b5699a703550bd61b11bdbed66cfee712090ff4
Reviewed-on: https://chromium-review.googlesource.com/760242Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org>
Reviewed-by: Paul Jensen <pauljensen@chromium.org>
Commit-Queue: Ryo Hashimoto <hashimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522032}

This patch removes the styles-history.js test: the test is testing the
UISourceCode.history that was recently removed.

BUG=733495
TBR=dgozman

Change-Id: Idddd379ed4b9ae9415232400979003c6130b27a9
Reviewed-on: https://chromium-review.googlesource.com/809715Reviewed-by: Andrey Lushnikov <lushnikov@chromium.org>
Commit-Queue: Andrey Lushnikov <lushnikov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522031}

After no streams have been playing out audio for a few seconds, trigger
power-save mode. Note that paused streams do not count as playing out
for this purpose.

Bug: internal b/64462358
Change-Id: I6ed4910490705d39e499b5f5e63432f8534993ff
Reviewed-on: https://chromium-review.googlesource.com/807749
Commit-Queue: Kenneth MacKay <kmackay@chromium.org>
Reviewed-by: Luke Halliwell <halliwell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522030}

This script is intended to be run on buildbots, after they've collected
the results of a swarming execution. The script takes a single test results
json file, and uploads any artifacts in the file to cloud storage, to a
public bucket (support for private buckets will be added later).

Bug: 772208
Change-Id: Ideca5912d2c8f64a62fafe21f6537f59250c62f3
Reviewed-on: https://chromium-review.googlesource.com/749624
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522029}

Change-Id: I856dbeb0da04d6242dc5eb02722aed95a56b5d03
Reviewed-on: https://chromium-review.googlesource.com/809948Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Commit-Queue: Pavel Feldman <pfeldman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522028}

Send the AcceleratedWidget to the client on init, so that the
compositor can be correctly initialized.

BUG=786453

Change-Id: Iad74feb8d669abf2470702fad890f33c2fe11f91
Reviewed-on: https://chromium-review.googlesource.com/810210
Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522027}

'content-image-texture-target' command line parameter was added
to communicate from the browser to the renderer the texture targets
to use when binding a texture given a pair usage/format.

The map was fully populated (every combination of usage/format), and with time
the command line parameter grew to a string that is now more than 1k:

--content-image-texture-target=0,0,3553;0,1,3553;0,2,3553;0,3,3553;0,4,3553;0,5,3553;0,6,3553;0,7,3553;0,8,3553;0,9,3553;0,10,3553;0,11,3553;0,12,3553;0,13,3553;0,14,3553;0,15,3553;0,16,3553;0,17,3553;1,0,3553;1,1,3553;1,2,3553;1,3,3553;1,4,3553;1,5,3553;1,6,3553;1,7,3553;1,8,3553;1,9,3553;1,10,3553;1,11,3553;1,12,3553;1,13,3553;1,14,3553;1,15,3553;1,16,3553;1,17,3553;2,0,3553;2,1,3553;2,2,3553;2,3,3553;2,4,3553;2,5,3553;2,6,3553;2,7,3553;2,8,3553;2,9,3553;2,10,3553;2,11,3553;2,12,3553;2,13,3553;2,14,3553;2,15,3553;2,16,3553;2,17,3553;3,0,3553;3,1,3553;3,2,3553;3,3,3553;3,4,3553;3,5,3553;3,6,3553;3,7,3553;3,8,3553;3,9,3553;3,10,3553;3,11,3553;3,12,3553;3,13,3553;3,14,3553;3,15,3553;3,16,3553;3,17,3553;4,0,3553;4,1,3553;4,2,3553;4,3,3553;4,4,3553;4,5,3553;4,6,3553;4,7,3553;4,8,3553;4,9,3553;4,10,3553;4,11,3553;4,12,3553;4,13,3553;4,14,3553;4,15,3553;4,16,3553;4,17,3553;5,0,3553;5,1,3553;5,2,3553;5,3,3553;5,4,3553;5,5,3553;5,6,3553;5,7,3553;5,8,3553;5,9,3553;5,10,3553;5,11,3553;5,12,3553;5,13,3553;5,14,3553;5,15,3553;5,16,3553;5,17,3553;6,0,3553;6,1,3553;6,2,3553;6,3,3553;6,4,3553;6,5,3553;6,6,3553;6,7,3553;6,8,3553;6,9,3553;6,10,3553;6,11,3553;6,12,3553;6,13,3553;6,14,3553;6,15,3553;6,16,3553;6,17,3553

Most entries are just GL_TEXTURE_2D (3553).
Since we have only one texture target per platform that is not the standard TEXTURE_2D,
this CL changes the full map, to a list containing only usage/format that should be
used with the platform specific texture target, omitting the pairs that should use
TEXTURE_2D, significantly reducing the command line size.

Bug: 
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Id6c289465a24e0d76c50db9d2befd4053ef440c3
Reviewed-on: https://chromium-review.googlesource.com/783255Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: David Reveman <reveman@chromium.org>
Reviewed-by: Robert Sesek <rsesek@chromium.org>
Commit-Queue: Daniele Castagna <dcastagna@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522026}

This new histogram allows us to know why a navigation ended up being
intercept or why it didn't. It'll help us understand how often users get
into apps as well as how often users get into websites for which they
have an app installed.

Bug: 785661
Change-Id: I2668f34421958b88885f53113b82a391777d9f36
Reviewed-on: https://chromium-review.googlesource.com/798250
Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Reviewed-by: Ben Wells <benwells@chromium.org>
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Reviewed-by: Matt Giuca <mgiuca@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522025}

This is currently passed GPU to browser via IPC. Add support for mojo.

In practice, none of the paramters are read on non-macOS platforms.
Remove the ifdef guards from everything but the mach port, because
of preprocessor guards aren't present in mojo.

Bug: 772576
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Icae234c7969ff26a337b025b6ae325e3abc92314
Reviewed-on: https://chromium-review.googlesource.com/809541Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: ccameron <ccameron@chromium.org>
Commit-Queue: ccameron <ccameron@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522024}

InsertParagraphSeperator gets DCHECK(IsEditablePosition)
when checking for leading white speace position.
This happens on a unusual HTML having,
webkit-appearance style applied.

Since the layoutBlock in this scenario has a
logical height because of appearance property,
and layout for table elements like colgroup is
LayoutBlockFlow, therefore the visuallyEquivalentAlgorithm
considers it as a EditablePosition,
and the command for InsertParagraph is executed.

Bug: 777378
Change-Id: I2abce4d09e2526f081cd810fc663aec61cfb24bf
Reviewed-on: https://chromium-review.googlesource.com/781331Reviewed-by: yosin (OOO Dec 11 to Jan 8) <yosin@chromium.org>
Reviewed-by: Yoichi Osato <yoichio@chromium.org>
Commit-Queue: Tanvir Rizvi <tanvir.rizvi@samsung.com>
Cr-Commit-Position: refs/heads/master@{#522023}

Change-Id: I18cbf052d93c11ab857f0253f6e4ce099e3ea2bc
Reviewed-on: https://chromium-review.googlesource.com/805174
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522022}

Previously nullable values in the animations code were tracked by
storing nulls as quiet_NaN() and using std::isnan as a null-detector.
It is much more explicit to store such values using WTF::Optional, which
forces code to consider whether or not the offset exists.

Bug: 791086
Change-Id: I4db0457b593d6be61dedb5c13a9daf8b671206b0
Reviewed-on: https://chromium-review.googlesource.com/804116Reviewed-by: Eric Willigers <ericwilligers@chromium.org>
Reviewed-by: Andrey Kosyakov <caseq@chromium.org>
Commit-Queue: Stephen McGruer <smcgruer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522021}

To avoid path conflicts, we install chrome os components (only) to
dedicated path naming space (cros_components/[component name]).

Since the components currently used are small (printer drivers), the
cost to migrate it small. It's a good idea to migrate now.

BUG=chromium:789726
TEST=update (ignore old locations)/install printer driver component on device.

Change-Id: I3276f73eaab50ec562d247d06eedb5c96ba183bf
Reviewed-on: https://chromium-review.googlesource.com/798225
Commit-Queue: Xiaochu Liu <xiaochu@chromium.org>
Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522020}

This reverts commit 6ae6cf7e.

Reason for revert: Speculative revert for ash_unittests-mus failure on chromium.chromiumos/linux-chromeos-rel, started here:
https://uberchromegw.corp.google.com/i/chromium.chromiumos/builders/linux-chromeos-rel/builds/3239
This is the only mus related CL in the regression range.


Original change's description:
> mus/aura: Wait for the real AcceleratedWidget.
> 
> When mus does not host viz, aura handles the real AcceleratedWidget. So
> in such cases, WindowTreeHostMus needs to wait for the widget to be
> avaiable, which is still created by mus and sent to aura over mojom.
> 
> BUG=786453, 791619
> 
> Change-Id: I335365bfe4793c8fafb8d08eaa45c09c50e17a59
> Reviewed-on: https://chromium-review.googlesource.com/807887
> Reviewed-by: Scott Violet <sky@chromium.org>
> Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#521903}

TBR=sadrul@chromium.org,sky@chromium.org,msw@chromium.org

Change-Id: I9b15d1ad8fc36e6a4b4864e1f7bfd6df2b3745a5
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 786453, 791619
Reviewed-on: https://chromium-review.googlesource.com/810404Reviewed-by: Kunihiko Sakamoto <ksakamoto@chromium.org>
Commit-Queue: Kunihiko Sakamoto <ksakamoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522019}

https://pdfium.googlesource.com/pdfium.git/+log/6eda94d78313..4bd2ad8cf901

$ git log 6eda94d78..4bd2ad8cf --date=short --no-merges --format='%ad %ae %s'
2017-12-06 dsinclair Move V8 helper methods to their own class.

Created with:
  roll-dep src/third_party/pdfium


The AutoRoll server is located here: https://pdfium-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.


TBR=dsinclair@chromium.org

Change-Id: I824417a15e29eb8ad87ee6f19af8c9288b6a5ca1
Reviewed-on: https://chromium-review.googlesource.com/810265
Reviewed-by: <pdfium-deps-roller@chromium.org>
Commit-Queue: <pdfium-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522018}