GitLab

This CL fixes SpellingMenuObserverTest.SuggestionsForceTopSeparator by
implementing TestingProfile::CreateMainNetworkContext() for network
service.

Bug: 779217
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: Ib5c134c401a66feed97faf41d9e0b17b6ed2badf
Reviewed-on: https://chromium-review.googlesource.com/963402
Commit-Queue: Jun Cai <juncai@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545523}

This reverts commit 8bc6affd.

Reason for revert: seems to break assistant at runtime

Original change's description:
> Add platform file provier to assistant
> 
> Bug: 820255
> Test: Manual
> Change-Id: I465d79c3b9cc2d8c0e1b0d6ea33a6b7db4833c37
> Reviewed-on: https://chromium-review.googlesource.com/961081
> Commit-Queue: Yue Li <updowndota@chromium.org>
> Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org>
> Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#544930}

TBR=xiyuan@chromium.org,xiaohuic@chromium.org,updowndota@chromium.org

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

Bug: 820255
Change-Id: I4b73a56219dd9dd58bc5072235fc7f8ce0b1f9e9
Reviewed-on: https://chromium-review.googlesource.com/978521Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org>
Commit-Queue: Xiaohui Chen <xiaohuic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545522}

Design doc (public): https://goo.gl/HmBYy6

This patch proposes the new API for the Shared Memory based on two
concepts, a shared memory region and a shared memory mapping. The
region controls the platform-specific handle and mediates sharing and
access control. The mapping is just a reference to the memory bytes
which knows how to unmap itself. Ownership is per-instance: neither a
region nor a mapping may be copied, and resources are freed when then
instance dies. This is accomplished through move-only semantics.

The goal of the new API is to provide a correct cross-platform
implementation that makes the sharing mode explicit via the C++ type
system.

Thus, this CL introduces two classes for shared memory mappings:
- WritableSharedMemoryMapping owns and provides an access to writable
shared memory
- ReadOnlySharedMemoryMapping for read-only shared memory

and three classes representing shared memory regions:
- UnsafeSharedMemoryRegion allows to create
WritableSharedMemoryMapping instances and cannot be used for making
read-only mappings
- ReadOnlySharedMemoryRegion, on the contrary, can produce only
ReadOnlySharedMemoryMapping instances and it also creates one writable
mapping in the caller process address space
- WritableSharedMemoryRegion allows to create writable
mappings and can be eventually converted to ReadOnlySharedMemoryRegion.
The important limitation in comparison with UnsafeSharedMemoryRegion
is that the WritableSharedMemoryRegion cannot be duplicated so in
order to pass the writable region to another process, the ownership of
the region must be moved/transferred to this process.

Bug: 795291
Change-Id: I7d1d0a24932145cc0f449b01ceb8cafef9b7e407
Reviewed-on: https://chromium-review.googlesource.com/893458
Commit-Queue: Alexandr Ilin <alexilin@chromium.org>
Reviewed-by: Matthew Cary <mattcary@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Erik Chen <erikchen@chromium.org>
Reviewed-by: Robert Sesek <rsesek@chromium.org>
Reviewed-by: Ken Rockot <rockot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545521}

In cases where there is a source tab, we are currently missing to set
the source ID, because at the time DidOpenRequestedUrl is received in
SyncSessionRouterTabHelper, the Java side Tab is not initializes and
TabHelpers are not attached yet. This plumbs the parent WebContents
during initialization, so that we can get another try at setting
the source tab ID.

Change-Id: Ic4194916518650e5c0e70b2f4d43ffaa183999d0
Reviewed-on: https://chromium-review.googlesource.com/972330Reviewed-by: Ted Choc <tedchoc@chromium.org>
Reviewed-by: Nicolas Zea (slow) <zea@chromium.org>
Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Commit-Queue: Yusuf Ozuysal <yusufo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545520}

- The suggestions should include bullets according to the mock.
- The suggested values should always include all the passwords regardless of what is typed into the username field.

Bug: 819626
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ied795b335e00052f3629c2a0f30e26b9e51458b3
Reviewed-on: https://chromium-review.googlesource.com/978218
Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org>
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545519}

Colors the toolbar white under Material refresh among others. These
colors are already used by the touch UI. This checks IsNewerMaterialUi()
which is true for both touch-optimized and material refresh.

Bug: chromium:810165, chromium:822069
Change-Id: I293537b0b03806ef1205a33a895cffa4d9552e9d
Reviewed-on: https://chromium-review.googlesource.com/976807Reviewed-by: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Evan Stade <estade@chromium.org>
Commit-Queue: Peter Boström <pbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545518}

In an effort to commit navigation faster, we are facing some failing
test: https://chromium-review.googlesource.com/c/chromium/src/+/951732

This CL update some of them. In these tests, there are a client-side
redirect. A new navigation is initiated by the document while
it is not fully parsed. These tests are waiting for 2 load stops:
  1) one for the initial navigation,
  2) one for the redirect.
It turns out, the second navigation may be fast enough, so that it
commits before the first document has been loaded.
DocumentLoader::LoadFailed() is called for the first document.
The browser receives DidStopLoading() only once instead of two.

Bug: 705744
Change-Id: I4201124c0e70d64204e3c9070aae588a684e0913
Reviewed-on: https://chromium-review.googlesource.com/960664
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: Camille Lamy <clamy@chromium.org>
Reviewed-by: Emily Stark <estark@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545517}

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

Found from auditing suspicious classes in main dex.
https://maps.google.com shows location just fine without it.

Bug: 820570
Change-Id: Idb0d05c3598d966cbf3cfab667a5d4ddd876c541
Reviewed-on: https://chromium-review.googlesource.com/971022Reviewed-by: Miguel Casas <mcasas@chromium.org>
Commit-Queue: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545515}

Buffer estimation paths were annotated to give info about which
estimation method was used. The decisions about how to act on estimated
buffers remain unchanged (for now).

Hopefully UMA data will show that splicing on top of buffers with
estimated duration is quite rare. This would give us confidence to
dissable this sort of splicing.

Once disabled, we could transition WebM audio buffer duration estimation
to use max-of-buffers-so-far (no longer in fear of over-triggering
splicing), which resolves the linked bug.

For buffers with duration type kConstantEstimate (all buffers had same
duration), the idea (down the road) is to treat this as if its not an
estimate at all and continue to allow splicing.

Bug: 396634
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;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: Icbe930cf013c24b05e033fd2aa5d5b7f83bf9ac6
Reviewed-on: https://chromium-review.googlesource.com/967514Reviewed-by: Steven Holte <holte@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Chrome Cunningham <chcunningham@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545514}

This reverts commit 65ae686a.

Reason for revert: breaks legacy page flips, since HardwareDisplayPlaneManager::IsCompatible tests with RGBA and fails.

Original change's description:
> viz: Make primary plane transparent
> 
> This CL enables alpha blending at scanout for the primary plane.
> In this way chromecast will be able to punch a hole in the primary
> plane and show content underneath.
> 
> Note that on platform using legacy pageflip we'll still be
> scanning out without alpha blending (crrev.com/c/964984).
> 
> Bug: 821944, b/74997524
> Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel
> Change-Id: I654935dc630f7a9f6cb4717cd574a21a2749c442
> Reviewed-on: https://chromium-review.googlesource.com/976961
> Reviewed-by: Daniel Nicoara <dnicoara@chromium.org>
> Commit-Queue: Daniele Castagna <dcastagna@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#545478}

TBR=dnicoara@chromium.org,dcastagna@chromium.org,hoegsberg@chromium.org

Change-Id: If2d49fb3d9b1042653f7d4359b699d784c757d14
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 821944, b/74997524
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel
Reviewed-on: https://chromium-review.googlesource.com/978581Reviewed-by: Daniele Castagna <dcastagna@chromium.org>
Commit-Queue: Daniele Castagna <dcastagna@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545513}

This reverts commit b5d6aaf9.

Reason for revert: This change cancels the download if the task did not complete within 60 seconds timeout.

Original change's description:
> Set 60 seconds as timeoutIntervalForResource for Downloads.
> 
> Background NSURLSession never fails due to connectivity errors. Instead
> that session waits for connectivity up to timeoutIntervalForResource
> timeout which defaults to 1 week.
> 
> This CL changes the timeout to 60 seconds.
> 
> Bug: 824018
> Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
> Change-Id: I841fc4580fa6bc6724198d825bb1736b91027fc4
> Reviewed-on: https://chromium-review.googlesource.com/976623
> Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
> Commit-Queue: Eugene But <eugenebut@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#545482}

TBR=sdefresne@chromium.org,eugenebut@chromium.org

Change-Id: I750099a0e3d786930a9d57dacdf62597be8d36f7
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 824018
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Reviewed-on: https://chromium-review.googlesource.com/978471Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545512}

TBR=ashleymarie@chromium.org
NOTRY=true

Bug: 823357
Change-Id: Ia44f54405c5b6ad0dd0f3b859753de3cea7c35b4
Reviewed-on: https://chromium-review.googlesource.com/978478Reviewed-by: Emily Hanley <eyaich@chromium.org>
Commit-Queue: Emily Hanley <eyaich@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545511}

Switches to using an accessibility role of alert dialog which is more appropriate for a notification toast than a window role.
Alert dialogs are widgets that may contain interactive controls that need to be announced by a screen reader as soon as they appear.
Also, an alert event should be fired regardless to whether the notification has been updated or has been initiallly set.
R=aleventhal@chromium.org, dmazzoni@chromium.org, dewittj@chromium.org, peter@chromium.org

Tested: Manually with Jaws and NVDA screen readers on Windows
Change-Id: Ida3abe94b9dc32a5c95ce6b14c6fac11670348e3
Reviewed-on: https://chromium-review.googlesource.com/968672Reviewed-by: Justin DeWitt <dewittj@chromium.org>
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: Nektarios Paisios <nektar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545510}

Bug: 821170
Change-Id: I6ad4d15ce7fbf0f168527aaac4029654b6632dd3
Reviewed-on: https://chromium-review.googlesource.com/976186Reviewed-by: Yoav Weiss <yoav@yoav.ws>
Commit-Queue: Erik Luo <luoe@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545509}

Also implements getLastWrittenValue() and converts two tests to use
these.

FakeRemoteGattDescriptor::setNextWriteResponse allows tests to set the
next response for a write.

BUG=719825

Change-Id: I2807d23a55f79eeca35338e25916c6d5e0115e89
Reviewed-on: https://chromium-review.googlesource.com/964901
Commit-Queue: Ovidio Henriquez <odejesush@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Reviewed-by: Conley Owens <cco3@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545508}

Bug: skia:6718
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel
Change-Id: I75700d44a337f768e2c30cf92b78b721b915334e
Reviewed-on: https://chromium-review.googlesource.com/977946Reviewed-by: weiliangc <weiliangc@chromium.org>
Commit-Queue: Brian Salomon <bsalomon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545507}

Mark as flaky on specific platforms.

BUG=825262
TBR=eae@chromium.org
NO-TRY=True

Change-Id: Iea0332aa3d2e192866b47dfef8aa2034ae24b13d
Reviewed-on: https://chromium-review.googlesource.com/978452Reviewed-by: Matthew Wolenetz <wolenetz@chromium.org>
Commit-Queue: Matthew Wolenetz <wolenetz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545506}

This change refactors the URL bar into a set of buttons, layouts, and a
much smaller dedicated element to present the URL text.  In the future,
the URL text should be made into a Text subclass, or similar, to take
advantage of text layout.

This change also refactors and fixes some pieces of the vector icon
button class.

BUG=821946

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;master.tryserver.chromium.linux:linux_vr;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I186e6c32dbf8c61ddba7a5f9fe01c584067bd43e
Reviewed-on: https://chromium-review.googlesource.com/976685
Commit-Queue: Christopher Grant <cjgrant@chromium.org>
Reviewed-by: Ian Vollick <vollick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545505}

Change-Id: I612740b3a1196b7165a1555a3947753b598c2b65
Reviewed-on: https://chromium-review.googlesource.com/978378Reviewed-by: Eric Stevenson <estevenson@chromium.org>
Commit-Queue: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545504}

This is a reland of 48d1ecaf

With CL:967415 and CL:964984, we've fixed the issues where RGBA
framebuffers slipped through to KMS on devices without RGBA support.

Original change's description:
> ozone/drm: Render primary framebuffers as RGBA
>
> In preparation for hardware plane underlay support in ChromeOS, we
> need to allow for RGBA primary framebuffers. When using an underlay,
> we cut out a transparent rectangle in the primary framebuffer to allow
> the underlay to show through, while still allowing for RGBA content on
> top of the underlay (for example, video controls or annotations).
> When not using an underlay, the primary framebuffer will be displayed
> as RGBX (opaque) to avoid unnecessary blending in the display
> controller.
>
> Bug: 789288
> Change-Id: I2fc94e524e250ec9b7e11cd6801e1c8308046a10
> Reviewed-on: https://chromium-review.googlesource.com/801974
> Reviewed-by: Alex Sakhartchouk <alexst@chromium.org>
> Reviewed-by: Stéphane Marchesin <marcheu@chromium.org>
> Reviewed-by: David Reveman <reveman@chromium.org>
> Reviewed-by: danakj <danakj@chromium.org>
> Reviewed-by: Antoine Labour <piman@chromium.org>
> Reviewed-by: Daniele Castagna <dcastagna@chromium.org>
> Commit-Queue: Kristian H. Kristensen <hoegsberg@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#542877}

TBR: piman
Bug: 789288
Change-Id: Ie1032615d8970c6a74f354ad78c30679706e40ed
Reviewed-on: https://chromium-review.googlesource.com/975782
Commit-Queue: Daniele Castagna <dcastagna@chromium.org>
Reviewed-by: Alex Sakhartchouk <alexst@chromium.org>
Reviewed-by: Daniele Castagna <dcastagna@chromium.org>
Reviewed-by: David Reveman <reveman@chromium.org>
Reviewed-by: Stéphane Marchesin <marcheu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545503}

This CL is an initial implementation of the grid-to-tab animation.

GridTransitionAnimation now supports animations in both directions.
The reverse (grid-to-tab) animation is implemented similarly to the
tab-to-grid animation, with different relative timing for the various
stages, and starting with the cells in the regular positions and
animating to the expanded position.

GridToVisibleTabAnimator likewise is structured similarly to its
opposite, and the disappearing transitions in TabGridViewController
also mirror their appearing counterparts.

This produces a lot of near-duplicate code. Follow-up CLs will
refactor this into a cleaner structure.

The animation itself also needs improvement, particularly the
interactions with the tab grid view controller and the BVC at
either end of the transition. That will also be addressed in follow-
up CLs.

Bug: 804539
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: If38c185b3c510b3665f5761ebce67ea4ad36d6dc
Reviewed-on: https://chromium-review.googlesource.com/975610
Commit-Queue: Mark Cogan <marq@chromium.org>
Reviewed-by: edchin <edchin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545502}

Bug: 790142
Change-Id: I039765f398d97a57b2d44ddd335616af22bb99b8
Reviewed-on: https://chromium-review.googlesource.com/977082Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Zhiling Huang <hzl@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545501}

Bug: 790142
Change-Id: I28f37ad920756cea9030e4daffa13d58732fcf94
Reviewed-on: https://chromium-review.googlesource.com/977054Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Zhiling Huang <hzl@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545500}

Bug: 790142
Change-Id: I75ff6b9450560165ddce782754a74d5384d66556
Reviewed-on: https://chromium-review.googlesource.com/977052Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Zhiling Huang <hzl@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545499}

When searching for https:// with query in omnibox enabled, https was
highlighted green as it would be for a standard URL. This fix disables
URL emphasis when displaying search terms.

Bug: 824971
Change-Id: Ibaf1a996931c861697263cbabbf6b65e235076f5
Reviewed-on: https://chromium-review.googlesource.com/976864Reviewed-by: Maria Khomenko <mariakhomenko@chromium.org>
Commit-Queue: Troy Hildebrandt <thildebr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545498}

Commit d9bd4752 ("Read Cinnamon settings for window button layout") causes
GCC to emit a warning:

    ../../chrome/browser/ui/libgtkui/settings_provider_gsettings.cc: In constructor ‘libgtkui::SettingsProviderGSettings::SettingsProviderGSettings(libgtkui::GtkUi*)’:
    ../../chrome/browser/ui/libgtkui/settings_provider_gsettings.cc:46:29: warning: operation on ‘settings_schema’ may be undefined [-Wsequence-point]
               ? settings_schema = kCinnamonPreferencesSchema
                 ~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~

The code can at least be simplified, as the assignments inside the ternary
operator are not necessary (or idiomatic).

Change-Id: Icdd0fca8ce12881e49ccc39dc467e21f9c96f07c
Reviewed-on: https://chromium-review.googlesource.com/978241
Commit-Queue: Thomas Anderson <thomasanderson@chromium.org>
Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545497}

This CL has no functional change. Some previously nested functions need
to be moved in order to be shared by findElementAtPoint.

Bug: 228355
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I478b0b962e4831a056daa6066505b2176f932ed1
Reviewed-on: https://chromium-review.googlesource.com/976541
Commit-Queue: Mike Dougherty <michaeldo@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545496}

It is essentially the same as the same as mojo_parse_message_fuzzer except that
it uses LPM to mutate and send multiple messages per run.

Bug: 769578
Change-Id: I57d0ceca2501ca4ab5479c0a3259702dd8614a82
Reviewed-on: https://chromium-review.googlesource.com/973685
Commit-Queue: Abhishek Arya <inferno@chromium.org>
Reviewed-by: Ken Rockot <rockot@chromium.org>
Reviewed-by: Oliver Chang <ochang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545495}

This change includes some cleanup, removing unused member variables and
removing a reference to a bug that has been fixed.

Bug: chromium:757625, chromium:796334
Change-Id: I6a5901f355050a47aeb59850e34ed463b79b6899
Reviewed-on: https://chromium-review.googlesource.com/978379Reviewed-by: Zentaro Kavanagh <zentaro@chromium.org>
Commit-Queue: Zentaro Kavanagh <zentaro@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545494}

Note that this is for the ImageFactory interface.
This is not for native gmbs and it is not related to
--enable-native-gpu-memory-buffers flag.

Bug: 584248

Cq-Include-Trybots: luci.chromium.try:linux_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I67c9f95bbc2a5ee987cb78d7b90c279d7a1556c7
Reviewed-on: https://chromium-review.googlesource.com/970423Reviewed-by: David Reveman <reveman@chromium.org>
Commit-Queue: Julien Isorce <julien.isorce@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545493}

Commit 115c9c57 ("Decouple fling booster from input_handler_proxy") added a
conditional block to HandleInputEvent() with a seemingly bogus block that
GCC warns about:

    ../../ui/events/blink/input_handler_proxy.cc: In member function ‘ui::InputHandlerProxy::EventDisposition ui::InputHandlerProxy::HandleInputEvent(const blink::WebInputEvent&)’:
    ../../ui/events/blink/input_handler_proxy.cc:384:33: warning: enum constant in boolean context [-Wint-in-bool-context]
                      WebInputEvent::kGestureScrollUpdate) {
                                     ^~~~~~~~~~~~~~~~~~~~
    ../../ui/events/blink/input_handler_proxy.cc:384:33: warning: enum constant in boolean context [-Wint-in-bool-context]

Compare the enum values to event.GetType() like in the other blocks.

Change-Id: I3f56888e40a0f8d6e10a6577ae7b4f36bab942e0
Reviewed-on: https://chromium-review.googlesource.com/978202Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Commit-Queue: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com>
Cr-Commit-Position: refs/heads/master@{#545492}

NoState Prefetch is a version of prerendering that should only run the
preload scanner on a document. It appears that it's possible for parsing
to occurr, however, when the document parser is torn down. This change
prevents that from happening and adds a a DCHECK to make sure it
doesn't.

Bug: 823306
Change-Id: I541515e3f75d726b56f132c1c010367a4aa829d8
Reviewed-on: https://chromium-review.googlesource.com/973613
Commit-Queue: Matthew Cary <mattcary@chromium.org>
Reviewed-by: Charlie Harrison <csharrison@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545491}

Using wpt-import in Chromium 2fc4f379.

Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/14367

Note to sheriffs: This CL imports external tests and adds
expectations for those tests; if this CL is large and causes
a few new failures, please fix the failures by adding new
lines to TestExpectations rather than reverting. See:
https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md

Directory owners for changes in this CL:
jsbell@chromium.org:
  external/wpt/resources

TBR=danyao

No-Export: true
Change-Id: I15668772a2240f0162a72f59845092e53a4921c5
Reviewed-on: https://chromium-review.googlesource.com/978043
Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Reviewed-by: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545490}

Add PlatformDisplayMirror to represent mirroring destination displays.
Have ws::DisplayManager own and manage the mirrors as directed by Ash.
Add DisplayManager::OnFirstSurfaceActivation helper to notify mirrors.
Add FrameGenerator::scale_and_center_ behavior for mirror displays.

Minor cleanup and invert a broken conditional in ws::Display.

Fix a PlatformDisplay test, add a CreatePlatformWindow helper.
Remove an OzonePlatform accessor, I'll fix the test separately.

TODO: Get Mash cursor mirroring enabled (via cursor compositing?)

Bug: 806318,732987
Test: Ctrl+Shift+M toggles display mirroring in --enable-features=Mash
Change-Id: I43453a2b76c397a31ebdd8fae7c8fda6399fa4bd
Reviewed-on: https://chromium-review.googlesource.com/775653
Commit-Queue: Michael Wasserman <msw@chromium.org>
Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: kylechar <kylechar@chromium.org>
Reviewed-by: Weidong Guo <weidongg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545489}

This change prepares us to run components_perftests on the perf waterfall.
It does a few things:
 - Moves the tests to run as an isolated script
 - Moves the target out of chromium_gtests and into suites run by
   isolated_scripts
 - Updates BUILD files to support the perf test runner

Bug: 648992
Change-Id: I898203dacc2b89754d2681952819b3104d9b95e9
Reviewed-on: https://chromium-review.googlesource.com/971163Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Emily Hanley <eyaich@chromium.org>
Commit-Queue: Charlie Harrison <csharrison@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545488}

The code is currently failing to build with:

    ../../chrome/browser/media/router/mojo/media_router_mojo_metrics_unittest.cc:29:40: error: parameter ‘histogram_provider1’ includes reference to array of unknown bound ‘const char []’
         const char (&histogram_provider2)[]) {
                                            ^
    ../../chrome/browser/media/router/mojo/media_router_mojo_metrics_unittest.cc:29:40: error: parameter ‘histogram_provider2’ includes reference to array of unknown bound ‘const char []’

Fix it by using a less exotic syntax for the parameters: we just need two
char arrays here.

Change-Id: I3e17f233d6ad6f4c7c24df3e5dad64b32637ab25
Reviewed-on: https://chromium-review.googlesource.com/977967Reviewed-by: mark a. foltz <mfoltz@chromium.org>
Commit-Queue: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com>
Cr-Commit-Position: refs/heads/master@{#545487}

This works around a bug in the multidex support library where if
multiple apks are searched for secondary dexes, the caching logic breaks
and files are re-extracted every time.

Bug: 824523
Change-Id: I29c23fa33077428e00e2752d16a36949e8bc517c
Reviewed-on: https://chromium-review.googlesource.com/977060
Commit-Queue: agrieve <agrieve@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545486}

Now that we are filtering inadequately batched attestation certificates,
the consent really is about make, model, and batch number of the device
being attested.

TBR=srahim
BUG=793985

Change-Id: Ib912bb850215960160a754ce2ae4db32706ae7ca
Reviewed-on: https://chromium-review.googlesource.com/972374Reviewed-by: Adam Langley <agl@chromium.org>
Commit-Queue: Adam Langley <agl@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545485}

This reverts commit 904a5dd4.

Reason for revert: <INSERT REASONING HERE>

Original change's description:
> [chrome.dial] Remove chrome.dial.
> 
> Media Router has launched in-browser discovery of DIAL devices in M63, and
> removed usage of chrome.dial from the Media Router component extension.
> 
> This removes the API from Chrome as there are no more users.
> 
> Bug: 769348
> Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
> Change-Id: I9bde034fb42ffb05aef1a8d3666775d83cbd1337
> Reviewed-on: https://chromium-review.googlesource.com/964800
> Commit-Queue: mark a. foltz <mfoltz@chromium.org>
> Reviewed-by: mark a. foltz <mfoltz@chromium.org>
> Reviewed-by: Ilya Sherman <isherman@chromium.org>
> Reviewed-by: Istiaque Ahmed <lazyboy@chromium.org>
> Reviewed-by: Adam Parker <amp@chromium.org>
> Reviewed-by: Ken Rockot <rockot@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#545170}

TBR=lazyboy@chromium.org,mfoltz@chromium.org,isherman@chromium.org,rockot@chromium.org,amp@chromium.org

Change-Id: I036449eae45fae20e3f60f43a71d9dc4e709a3b4
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 769348
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Reviewed-on: https://chromium-review.googlesource.com/978481Reviewed-by: mark a. foltz <mfoltz@chromium.org>
Commit-Queue: mark a. foltz <mfoltz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545484}