GitLab

Allow control of the deprecation via runtime enabled features, with a first
step for logging a warning to the console, and the second step to block it
outright.

Intent:  https://groups.google.com/a/chromium.org/d/msg/blink-dev/35t5cJQ3J_Q/FH45dl0vAwAJ

Change-Id: I1be1001cbbef152458119b1516750bb4f1d1e4de
Reviewed-on: https://chromium-review.googlesource.com/975611
Commit-Queue: Daniel Vogelheim <vogelheim@chromium.org>
Reviewed-by: Mike West <mkwst@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549495}

As a follow-up to crrev/999712, this CL updates the GridCommands protocol to,
where possible, express actions on tabs in the tab grid in terms of item IDs instead
of indexes.

Bug: 986379
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ib92ba83fe37ac789b693d006e0ca60cb665b22b5
Reviewed-on: https://chromium-review.googlesource.com/1000860
Commit-Queue: Mark Cogan <marq@chromium.org>
Reviewed-by: Stepan Khapugin <stkhapugin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549494}

This CL adds an API to OAuth2TokenServiceDelegate::Observer to
listen to changes in auth errors.

AccountReconcilor uses this API to trigger a ListAccount call
when a token becomes invalid.
This is necessary because in that case, the account can be invalid
in the cookie without the cookie changing. Chrome needs to make a
call to ListAccount to have an accurate view of the Gaia accounts in
the cookie.

Bug: 823714
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ie22226adfe6c425acaffe875a314c4a6b8de5e88
Reviewed-on: https://chromium-review.googlesource.com/986268
Commit-Queue: David Roger <droger@chromium.org>
Reviewed-by: Mihai Sardarescu <msarda@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549493}

Bug: 826664, 830624
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.win:win_optional_gpu_tests_rel
Change-Id: I08e1ee4237f4e221baca4a0adec7a8bff19c1fa6
Reviewed-on: https://chromium-review.googlesource.com/997740
Commit-Queue: Henrik Grunell <grunell@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Reviewed-by: Max Morin <maxmorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549492}

As a feature, the value could only be off/on.
As a switch, the value can be force-off, force-on, or default.
Default is at the moment OFF, but will later be made ON for
non-mobile builds where the user has given appropriate consent.

Bug: 775415
Change-Id: I75b46074434505d65a3a60f2858068d1ad6513c2
Reviewed-on: https://chromium-review.googlesource.com/1002849Reviewed-by: Henrik Grunell <grunell@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Commit-Queue: Elad Alon <eladalon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549491}

TBR=huangml@chromium.org

Bug: 814990
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I6db1b62ec0b0d007fdcde56b866ce6500ec71234
Reviewed-on: https://chromium-review.googlesource.com/1005078
Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org>
Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549490}

ChannelState was invented in order to address synchronous IPC failures.
As mojo doesn't have such synchronous failures, we can remove it to
simplify the code.

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: Ia4e5c57c65b0f431a6770fa29bd71b706c15479d
Reviewed-on: https://chromium-review.googlesource.com/1004072
Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Adam Rice <ricea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549489}

This prevents a reload scheduled after TabWebContentsObserver.renderProcessGone
from racing with a subsequent page load.

Bug: 828400
Change-Id: I5743736aded76a227a1cda0a4fa1824751df9a3f
Reviewed-on: https://chromium-review.googlesource.com/992612Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Reviewed-by: Camille Lamy <clamy@chromium.org>
Reviewed-by: Benoit L <lizeb@chromium.org>
Commit-Queue: Matthew Cary <mattcary@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549488}

Makes the location bar transform to a new defocused state where the
ToolbarModel's display URL and security icon are displayed.
Transitions to normal textfield on touch.

Introduces new class: LocationView, a simple view displaying
the location icon and text.
Introduces new class: LocationBarViewController, which displays the
two location bar views: LocationView and LocationBarView.
Removes unused focus/defocus animation code from location bar view.

Bug: 821801, 821799
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: If058ec14fdc466b13f7917ba668be80d4781a4f8
Reviewed-on: https://chromium-review.googlesource.com/943861
Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org>
Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Reviewed-by: Justin Cohen <justincohen@chromium.org>
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549487}

This reverts commit 21f2ef4c.

Reason for revert: Will rewrite this.

Original change's description:
> Bind StreamFactory in audio service.
> 
> Bug: 803102
> Change-Id: I4d8cb112445095045c492488f7c3b8d9853b6c23
> Reviewed-on: https://chromium-review.googlesource.com/1004634
> Commit-Queue: Max Morin <maxmorin@chromium.org>
> Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
> Reviewed-by: Olga Sharonova <olka@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#549481}

TBR=kinuko@chromium.org,olka@chromium.org,maxmorin@chromium.org

Change-Id: I7ecdcf316e904356ba8373d888fbec910a1b71d1
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 803102
Reviewed-on: https://chromium-review.googlesource.com/1005174Reviewed-by: Max Morin <maxmorin@chromium.org>
Commit-Queue: Max Morin <maxmorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549486}

TBR=hbos@chromium.org

Bug: 829401
Change-Id: I98b7d3149599e876d673a84718215521ddf6cfac
Reviewed-on: https://chromium-review.googlesource.com/1004995Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org>
Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549485}

Fixed typo as requested by translators.

Bug: b:74624289
Change-Id: Ib9474151f11ca655c857d411d0190ad521b6bce8
Reviewed-on: https://chromium-review.googlesource.com/1001578
Commit-Queue: Alexander Hendrich <hendrich@google.com>
Reviewed-by: Julian Pastarmov <pastarmovj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549484}

There are many declarations to create scopedFD from MojoHandle.
Create util header file, arc_video_accelerator_util.h, to collect the duplicated
declarations.

BUG=None
TEST=Play Youtube App

Change-Id: I9252757831c5c1edd34ebc4bd82bf93c991166d0
Reviewed-on: https://chromium-review.googlesource.com/1002715Reviewed-by: Kuang-che Wu <kcwu@chromium.org>
Commit-Queue: Hirokazu Honda <hiroh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549483}

Some phases require iterability of a page. This property is already
established before LazySweep during OutOfLineAllocate but was missing in
the case where we completed sweeping for *all* arenas.

Bug: chromium:757440
Change-Id: I103b55a0d9dba11574ef56fae38db482f7fdad6a
Reviewed-on: https://chromium-review.googlesource.com/1004581Reviewed-by: Keishi Hattori <keishi@chromium.org>
Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549482}

Bug: 803102
Change-Id: I4d8cb112445095045c492488f7c3b8d9853b6c23
Reviewed-on: https://chromium-review.googlesource.com/1004634
Commit-Queue: Max Morin <maxmorin@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Olga Sharonova <olka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549481}

Instead of the proper value that may have many digits and make no sense
to a human.

This is a reland of https://chromium-review.googlesource.com/c/chromium/src/+/998915
It is fixing the test to make it more reliable and rebased the CL.

Bug: 829627
Change-Id: Ic550051859367dcd92237d527285f00210098142
Reviewed-on: https://chromium-review.googlesource.com/1000864Reviewed-by: Tommy Steimel <steimel@chromium.org>
Commit-Queue: Mounir Lamouri <mlamouri@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549480}

We used to rely on this taking place lazily via
MinPreferredLogicalWidth(), but that method won't necessarily be called
if the table is a flex/grid item.

Bug: 810327
Change-Id: Ic817bd109544d4b9e961552d0a3a38f127e6e548
Reviewed-on: https://chromium-review.googlesource.com/1000781
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Reviewed-by: David Grogan <dgrogan@chromium.org>
Reviewed-by: Christian Biesinger <cbiesinger@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549479}

R=dpapad@chromium.org

Bug: None
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I5a18ac1278077545aee2c69be4d20102af4b325b
Reviewed-on: https://chromium-review.googlesource.com/997844Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Andrey Kraynov <iceman@yandex-team.ru>
Cr-Commit-Position: refs/heads/master@{#549478}

Instead of first manually updating SyncedSessionTracker and later
producing the protos (SessionTab, SessionSpecifics), we now propose the
opposite and:
1. Produce the specifics first from SyncedTabDelegate.
2. Update the tracker using the specifics.

This flow is a lot nicer because, once other calling sites are updated,
we could move the updating of SyncedSessionTracker a lot closer to where
the persistence layers & sync are updated
(LocalSessionEventHandlerImpl::WriteBatch), to guarantee that the two
stay consistent.

This patch does not address the analogous updates for placeholder tabs
and the header updates.

Bug: 681921
Change-Id: I6bf7e90f9a3b236ce9e7b1f985fc0626369d8659
Reviewed-on: https://chromium-review.googlesource.com/1002176
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Jan Krcal <jkrcal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549477}

TBR=hubbe@chromium.org

Bug: 825733
Change-Id: If99bac5fc625623fb79e8a12b92a3f070478955e
Reviewed-on: https://chromium-review.googlesource.com/1004955Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org>
Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549476}

This reverts commit 16d83bf5.

Reason for revert: Performance issues

Original change's description:
> Media Controls: Move to different layer
>
> will-change will force the media controls to use a different
> layer. According to Pinpoint this will reduce CC memory usage
> by 18% and GPU memory usage by 6%.
>
> BUG=821961,821414
>
> Change-Id: I13c62f79577dfef253acf5f3b89cecaf0c5c1ad3
> Reviewed-on: https://chromium-review.googlesource.com/996309
> Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
> Commit-Queue: Becca Hughes <beccahughes@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#548524}

TBR=mlamouri@chromium.org,beccahughes@chromium.org


Bug: 821961, 821414
Change-Id: I5df8f67e4e779dc673ca9cf89221c734c5a2235c
Reviewed-on: https://chromium-review.googlesource.com/1003332
Commit-Queue: Mounir Lamouri <mlamouri@chromium.org>
Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Reviewed-by: Becca Hughes <beccahughes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549475}

Following bot results are included.
4683 4686 4687 4698

Deflaking was skipped due to too few results since the last update.

TBR=eae@chromium.org, mstensho@chromium.org
NOTRY=true

Bug: 591099
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: Iac1eeb7e2f3836130eb38555272f85780d68d8fd
Reviewed-on: https://chromium-review.googlesource.com/1002415
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549474}

Had to modify the test harness to allow numbers in keywords. This
coincidentally fixed an error in one of the tests (transform-style).

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

This reverts commit 2b9a46a9.

Reason for revert: <INSERT REASONING HERE>

Original change's description:
> [MacViews] Fix for incorrect extension padding
>
> Move the extensions 2pt to the right on Mac to match the native Mac
> placements.
>
> Bug: 792593
> Change-Id: I579520a37043667dfa289b110287c034447c3812
> Reviewed-on: https://chromium-review.googlesource.com/982353
> Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
> Commit-Queue: Sarah Chan <spqchan@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#546536}

TBR=ellyjones@chromium.org,spqchan@chromium.org

Bug: 792593
Change-Id: I517f941cd1b950e45d2879636e1ad0c07c2f5a49
Reviewed-on: https://chromium-review.googlesource.com/989535
Commit-Queue: Sarah Chan <spqchan@chromium.org>
Reviewed-by: Sarah Chan <spqchan@chromium.org>
Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549472}

TBR=eae@chromium.org

Bug: 591099
Change-Id: Idd34bd927a74e7411eb7f8e0067063330cb316f6
Reviewed-on: https://chromium-review.googlesource.com/1000861Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org>
Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549471}

Resolving post-merge nits in cr/980993.

Bug: 826257
Change-Id: Ifd201f8ee98f71079d5377034ff593d731061308
Reviewed-on: https://chromium-review.googlesource.com/1000862
Commit-Queue: Rayan Kanso <rayankans@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549470}

This patch changes "{paragraph,word}-granularity.html" to utilize
|selection_test()| for ease of maintenance.

Because of "{paragraph,word}-granularity.html" does similar steps, double-click
vs. triple-click, this patch integrates them into "word-granularity.html".

Following patch will move "word-granularity.html" to "selection/mouse/" for
better categorization of tests.

Bug: 679977
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: Idadf93448d31d6b72ff9dfdd6fbefeb55cb8be2d
Reviewed-on: https://chromium-review.googlesource.com/1004534
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: Yoichi Osato <yoichio@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549469}

Bug: 768828
Change-Id: Ib191dd491b06a9c465fad3f645f24e4977c924e7
Reviewed-on: https://chromium-review.googlesource.com/1004565
Commit-Queue: Kent Tamura <tkent@chromium.org>
Commit-Queue: Yuki Shiino <yukishiino@chromium.org>
Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549468}

Exposes the location bar flag to chrome://flags. Uses it to switch
between the two location bars.

Bug: None
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I1ec939de576ff14aa75b3db4b7082ecd7a801975
Reviewed-on: https://chromium-review.googlesource.com/1000778
Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org>
Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549467}

For code health we use Ephemeral range instead of
passsing position in RemoveInlineStyle.
With this we catch a bug at call sites,
as EphemeralRange construction takes care of the
current DCHECKS in RemoveInlineStyle

Bug: 826106
Change-Id: Ibaa37789f7671aa464b4c87d30f12bd023a67a3d
Reviewed-on: https://chromium-review.googlesource.com/1001437Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Commit-Queue: Tanvir Rizvi <tanvir.rizvi@samsung.com>
Cr-Commit-Position: refs/heads/master@{#549466}

Slots may be deleted after recording a callback for them. Filter out
those slots accordingly during processing.

Bug: chromium:757440
Change-Id: I23c17c2f79af41a8539b8d4734ec677fb0b99c0b
Reviewed-on: https://chromium-review.googlesource.com/1000865
Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549465}

TBR=rdevlin.cronin@chromium.org
NOTRY=true

Bug: 831078
Change-Id: Id97988d690315e7d2734a11255dcc9ba901c3d78
Reviewed-on: https://chromium-review.googlesource.com/1004560Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
Commit-Queue: Makoto Shimazu <shimazu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549464}

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

Bug: 827065
Change-Id: I6dc555a31615371f10e082a27f894a6b2fcbf981
Reviewed-on: https://chromium-review.googlesource.com/987596
Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549463}

This reverts commit 34f59cac.

Reason for revert: Speculative revert: appears to be breaking the stock Chromium Mac builder for WebRTC: https://ci.chromium.org/buildbot/chromium.webrtc/Mac%20Builder/93792
FAILED: obj/extensions/common/api/libgenerated_api.a 

Original change's description:
> ONC: Add FTEnabled property to WiFi
> 
> Support FTEnabled property in ONC.
> 
> BUG=791202
> TEST=chromeos_unittests
> TEST=chrome.networkingPrivate.createNetwork with "FTEnabled":true
> enables the device to connect to an FT network. Setting
> "FTEnabled":false disables the device from connecting to an FT
> network.
> 
> Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
> Change-Id: I2470753fd139505704987e301b18983dc51c5c47
> Reviewed-on: https://chromium-review.googlesource.com/982275
> Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
> Commit-Queue: Matthew Wang <matthewmwang@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#549366}

TBR=stevenjb@chromium.org,cernekee@chromium.org,kirtika@chromium.org,matthewmwang@chromium.org

Change-Id: Ie29fdeaabecf4344ded8e0c5f9f6de1a04d8962e
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 791202
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Reviewed-on: https://chromium-review.googlesource.com/1004794Reviewed-by: Patrik Höglund <phoglund@chromium.org>
Commit-Queue: Patrik Höglund <phoglund@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549462}

This patch removes the grid logic from
LayoutBox::ContainingBlockLogicalWidth|HeightForContent().
Instead it initializes the OverrideContainingBlock size to zero
in LayoutGrid::UpdateBlockLayout().

For grid items we don't want to use the grid container size
(which would be the regular containing block size),
so we want to use the overridden value.
And while we don't have such value, it's better we use zero
(so percentages are not resolved against the wrong thing).

This change is covered by existent tests.

Change-Id: If6a96abe7148203a2cdb9ffb2b9dd487239f7aae
Reviewed-on: https://chromium-review.googlesource.com/989734
Commit-Queue: Manuel Rego Casasnovas <rego@igalia.com>
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Reviewed-by: Sergio Villar <svillar@igalia.com>
Cr-Commit-Position: refs/heads/master@{#549461}

https://chromium.googlesource.com/angle/angle.git/+log/0ce016c1bd12..fe4bbe6cbec6

$ git log 0ce016c1b..fe4bbe6cb --date=short --no-merges --format='%ad %ae %s'
2018-04-06 brandon1.jones Create Stubs For Unimplemented Extensions

Created with:
  roll-dep src/third_party/angle


The AutoRoll server is located here: https://angle-chromium-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;master.tryserver.chromium.win:win_optional_gpu_tests_rel
TBR=ynovikov@chromium.org

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

This CL is in preparation for moving this code into its only remaining
consumer, which triggers a presubmit error that the code needs
formatting.

Bug: 729590
Change-Id: I64246c2cca6f44a4ce60573ff6283c0f0c4b6d72
Reviewed-on: https://chromium-review.googlesource.com/1004614Reviewed-by: Mihai Sardarescu <msarda@chromium.org>
Reviewed-by: Colin Blundell <blundell@chromium.org>
Commit-Queue: Colin Blundell <blundell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549459}

Previously, this only worked in browser windows. Now, it uses the button
provider to detect if the menu is open, in either browser or hosted app
windows.

Bug: 791399
Change-Id: I885a4979e729eaedbb647903209c54b63286da22
Reviewed-on: https://chromium-review.googlesource.com/991892
Commit-Queue: Matt Giuca <mgiuca@chromium.org>
Reviewed-by: Bret Sepulveda <bsep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549458}

Bug: webrtc:8982
Change-Id: I978e0a2544f6f71c250da9b0ec7aeb8e1a901bb9
Reviewed-on: https://chromium-review.googlesource.com/997336Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Commit-Queue: Jonas Olsson <jonasolsson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549457}

AtomicInline fragment use legacy_paint_offset which is the offset to its
parent, while others use paint_offset which is the offset to line_box.
The legacy_paint_offset is not correct when AtomicInline is child of an
inline element that has fixed border/margin/padding. It's not easy to
deal with nested situation by passing legacy_paint_offset.

In this solution, compute the legacy_paint_offset when paint AtomicInline:
legacy_paint_offset = paint_offset - offset from parent to linebox.

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_layout_ng;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Ie3bcfbbf3b9239c5c5e77504242aed40e553ea0e
Reviewed-on: https://chromium-review.googlesource.com/995313
Commit-Queue: cathie chen <cathiechen@tencent.com>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549456}