GitLab

Restoring InspectorSession may send protocol notifications
synchronously, so we should setup all bindings beforehand.

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

The element is no longer used and is deprecated in Polymer 2.

Bug=802912

Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I7e84de8cd64be8f838f8e2ae52d8dc4ccfd251c7
Reviewed-on: https://chromium-review.googlesource.com/877392Reviewed-by: Dave Schuyler <dschuyler@chromium.org>
Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531348}

Just need to rename a variabled named "interface" which is also used in
Windows headers.

Change-Id: Ibc1bebf11414130924d46053fc9017a9d62586f5
Reviewed-on: https://chromium-review.googlesource.com/880603Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531347}

This cl replaces the bitmap-based tab progress spinner with the GPU-
accelerated Material Design version.

BUG=602558,542427

patch from issue 1997893002 at patchset 60001 (http://crrev.com/1997893002#ps60001)

Change-Id: Id926579bffc0a7c11c35bea6452dd1b0662d6aa2
Reviewed-on: https://chromium-review.googlesource.com/580131Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Sidney San Martín <sdy@chromium.org>
Commit-Queue: Jayson Adams <shrike@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531346}

Use of the common name for matching the hostname is fundamentally
insecure, as it allows for the bypass of security controls within
certificates. Remove the remaining aspects of supporting logic that
allow common name fallback, which were enabled by Enterprise Policy,
now that the Enterprise Policy has been removed.

BUG: 804349

Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I6e595f106b0d989935c9edc0d4c02e9a0f0f4584
Reviewed-on: https://chromium-review.googlesource.com/878901
Commit-Queue: Ryan Sleevi <rsleevi@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Misha Efimov <mef@chromium.org>
Reviewed-by: Eric Roman <eroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531345}

Change-Id: I45fd9150165dd2c55b9e5c3cdb639853b7e84532
Reviewed-on: https://chromium-review.googlesource.com/879725Reviewed-by: Ryan Hamilton <rch@chromium.org>
Commit-Queue: Yixin Wang <wangyix@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531344}

This reverts commit a3cb5eee.

Reason for revert: <INSERT REASONING HERE>

Original change's description:
> media: Simplify CdmService
> 
> - Restrict CdmService to only host library CDM. Previously CdmService
>   is generic, but it's never used anywhere else, and that assumption
>   has caused some unnecessary code complexity.
> - As such, also remove ENABLE_STANDALONE_CDM_SERVICE and replace it
>   with ENABLE_LIBRARY_CDMS where applicable.
> - Remove the use of media::mojom::InterfaceFactory for the CdmService,
>   and replace it with a dedicated media::mojom::CdmFactory.
> - Remove the use of MojoMediaClient in CdmService, and replace it with
>   CdmService::Client.
> 
> BUG=771791
> 
> Change-Id: I49e0fc4958b18e1b619e20ab4772245b256ab454
> Reviewed-on: https://chromium-review.googlesource.com/852556
> Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
> Reviewed-by: John Rummell <jrummell@chromium.org>
> Reviewed-by: Nasko Oskov <nasko@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#531329}

TBR=nasko@chromium.org,xhwang@chromium.org,jrummell@chromium.org

Change-Id: I330da3e6cd95e801d1def4abe4f9b8ec5c8fff1b
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 771791
Reviewed-on: https://chromium-review.googlesource.com/881362Reviewed-by: Joe Downing <joedow@chromium.org>
Commit-Queue: Joe Downing <joedow@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531343}

We should not check for resource tree model when inspecting Node.js
target.

R=dgozman@chromium.org

Bug: chromium:787754
Change-Id: I04503a249acc05182524da40f83765e6ad7c37a5
Reviewed-on: https://chromium-review.googlesource.com/881468
Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531342}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/0ac874e7..260f54b2

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_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;master.tryserver.chromium.android:android_optional_gpu_tests_rel

TBR=hablich@chromium.org,machenbach@chromium.org,kozyatinskiy@chromium.org,sergiyb@chromium.org

Change-Id: I28f653355b4f22cc2f3a636eb6c09a3edbbb79e8
Reviewed-on: https://chromium-review.googlesource.com/881422Reviewed-by: v8 autoroll <v8-autoroll@chromium.org>
Commit-Queue: v8 autoroll <v8-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531341}

When glXGetSyncValuesOML is true, we expected this counter to be non-0
and added this error to find bugs, but it has become standard behaviour
so this log is only generating noise.

R=kbr@chromium.org

Bug: 772844
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: I38377c74d45197cf33b2caf3cf81239075f8cae8
Reviewed-on: https://chromium-review.googlesource.com/881285Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: danakj <danakj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531340}

Separate PDF plugin specific code from other PDF functions.

Change-Id: If61f2316edc99b469ff1c7fb97d4a09fa98d8b5e
Reviewed-on: https://chromium-review.googlesource.com/879731
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531339}

Follow-up to chromium-review.googlesource.com/814595
target "peformance_test_suite" not found in
//testing/buildbot/gn_isolate_map.pyl

Bug: 757933
Change-Id: I070598678f586c703664619c5da917fe63747e50
Reviewed-on: https://chromium-review.googlesource.com/878724Reviewed-by: Emily Hanley <eyaich@chromium.org>
Commit-Queue: Ashley Enstad <ashleymarie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531338}

Accessibility events can be difficult to debug when it is unclear what
object the events occur on. This improves our event log format so that
instead of just the role/name/value of events, we include something like
<fieldset> or <div#foo> in the output, so that the tag and id are logged.

This will help with debugging using both our event tests and with
the ax_dump_event tool.

Bug: None
Change-Id: I79744395dd61f792358bc3ceb44ccadbedce7494
Reviewed-on: https://chromium-review.googlesource.com/876310Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: Aaron Leventhal <aleventhal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531337}

BUG=753691
R=ellyjones@chromium.org

Change-Id: I474706ba795c9a1951a898b09a54161439372116
Reviewed-on: https://chromium-review.googlesource.com/881173Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Jochen Eisinger <jochen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531336}

Daydream expects its icons to be in png format, and play store
validation will fail if it doesn't find correctly formatted icons.

This excludes '.dd.png' files from the automatic conversion to webp
when building ChromeModern

See b/72294258.

TBR=dtrainor@chromium.org

Bug: 802080
Change-Id: Icd55d5a08536697ec97c08701782c925165631c0
Reviewed-on: https://chromium-review.googlesource.com/881461Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Reviewed-by: agrieve <agrieve@chromium.org>
Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531335}

With this change "gn check" no longer assumes all #include paths are
relative to the source root. Instead it searches for the includes by
checking each of the directories listed in include_dirs for the target.

A number of new issues have been discovered. The trivial ones have been
fixed, the larger and more complex ones have been reported and the related
parts of the project have been excluded from the default gn check run.

BUG=794926

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: I39c17a7ac47dc024dd374c8891ce4911809195cb
Reviewed-on: https://chromium-review.googlesource.com/827014
Commit-Queue: Brett Wilson <brettw@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: David Reveman <reveman@chromium.org>
Reviewed-by: Yuzhu Shen <yzshen@chromium.org>
Reviewed-by: Greg Thompson <grt@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Brett Wilson <brettw@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531334}

This makes it easy to see that we don't delete things again once an
entry is conceptially unlinked by a Doom. This also fixes the
recent SimpleCacheQueuedOpenOnDoomedEntry testcase to behave more causally
consistently.

Also adds a testcase that covers new path --- but doesn't demonstrate any
fix.

Bug: 
Change-Id: I88d797eb23e0206ce3164bc1642ff464f6ba69e8
Reviewed-on: https://chromium-review.googlesource.com/862229Reviewed-by: Josh Karlin <jkarlin@chromium.org>
Commit-Queue: Maks Orlovich <morlovich@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531333}

In some cases the UI changes related to Fullscreen being disabled need to animate.
AnimatedScopedFullscreenDisabler works just like ScopedFullscreenDisabler but increments the
disabled counter inside an animation block.

Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I79710443b8fa9d8137a03ce71035ccc9b077e1c6
Reviewed-on: https://chromium-review.googlesource.com/879568
Commit-Queue: Sergio Collazos <sczs@chromium.org>
Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org>
Reviewed-by: Kurt Horimoto <kkhorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531332}

This was regressed in https://chromium-review.googlesource.com/c/chromium/src/+/858257, which
assumed that ImageResourceContents::Create() would never return nullptr. That would have been true
if https://chromium-review.googlesource.com/c/chromium/src/+/860995 hadn't been reverted.

Bug: 804829
Test: http/tests/navigation/image-css-load-in-subframe-unload-handler.html

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: If53103d79907d5a01111aead2fdc93ce3491b3d7
Reviewed-on: https://chromium-review.googlesource.com/881642Reviewed-by: Stephen Chenney <schenney@chromium.org>
Reviewed-by: Hiroshige Hayashizaki <hiroshige@chromium.org>
Commit-Queue: Nate Chapin <japhet@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531331}

When an IME tries to set a composition immediately after a non-editable element
inside an editable region, it's possible for the selection base to get stuck
inside the non-editable region, in which case the attempt to replace the
selection fails. The fix is to pass the kCanSkipOverEditingBoundary flag when we
call MostForwardCaretPosition() on the selection base so we can find the actual
editable region.

Bug: 790777,801441
Change-Id: I1014901742671f6fbb153c7117eaa1b271a265f7
Reviewed-on: https://chromium-review.googlesource.com/879839
Commit-Queue: Ryan Landay <rlanday@chromium.org>
Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531330}

- Restrict CdmService to only host library CDM. Previously CdmService
  is generic, but it's never used anywhere else, and that assumption
  has caused some unnecessary code complexity.
- As such, also remove ENABLE_STANDALONE_CDM_SERVICE and replace it
  with ENABLE_LIBRARY_CDMS where applicable.
- Remove the use of media::mojom::InterfaceFactory for the CdmService,
  and replace it with a dedicated media::mojom::CdmFactory.
- Remove the use of MojoMediaClient in CdmService, and replace it with
  CdmService::Client.

BUG=771791

Change-Id: I49e0fc4958b18e1b619e20ab4772245b256ab454
Reviewed-on: https://chromium-review.googlesource.com/852556
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Reviewed-by: John Rummell <jrummell@chromium.org>
Reviewed-by: Nasko Oskov <nasko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531329}

Separate logic for creating users list (small and extra small
display style) into ScrollableUsersListView.
ScrollableUsersListView:
* is initialized with appropriate params in LockContentsView
* adjusts its insets according to device orientation
* provides functionality to draw gradient at the top and bottom of the
background

Bug: 792742
Change-Id: I1d6087f7332261044a2d896b328570da124b9eb2
Reviewed-on: https://chromium-review.googlesource.com/875150Reviewed-by: Jacob Dufault <jdufault@chromium.org>
Commit-Queue: Aga Wronska <agawronska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531328}

Bug: 
Change-Id: I455c0d38cf11742d1651f5986dc9be148504b45b
Reviewed-on: https://chromium-review.googlesource.com/879332Reviewed-by: Albert J. Wong <ajwong@chromium.org>
Commit-Queue: Albert J. Wong <ajwong@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531327}

make browser_main_loop.cc line 266 and 666 match each other.

Change-Id: Id2d7c21b0f58afbb585971057b77e0983b4c04ae
Reviewed-on: https://chromium-review.googlesource.com/879565Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531326}

In jumbo builds multiple cc files are compiled in the same
translation unit and then they will all share the
anonymous namespace which can cause collisions if
variables have the same name. That was the case for two
kFilteredMessageClasses variables.

This renames them to:
kExtensionFilteredMessageClasses
kRenderFilteredMessageClasses

This is a followup to an equivalent change done in //content
https://chromium-review.googlesource.com/721279

Bug: 746957
Change-Id: I959600adcc81dbd6e405c25aa0272acefc0387d9
Reviewed-on: https://chromium-review.googlesource.com/878362Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Daniel Bratell <bratell@opera.com>
Cr-Commit-Position: refs/heads/master@{#531325}

In jumbo build experiments, two kNotificationPrefix variables
ended up in the same translation unit and clashed. This patch
gives those two variables unique names.

Bug: 746957
Change-Id: I200e9cb47129a1f20b8edec91ff0ff432a3c22d9
Reviewed-on: https://chromium-review.googlesource.com/874072
Commit-Queue: Daniel Bratell <bratell@opera.com>
Reviewed-by: Drew Wilson <atwilson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531324}

This CL makes two changes to the autocapitalize IDL behavior to match the new
spec:
https://html.spec.whatwg.org/multipage/interaction.html#autocapitalization

- For elements with no autocapitalize attribute set, or which have the attribute
  set with an empty value (e.g. <input autocapitalize> or <input
  autocapitalize="">), return an empty string instead of "sentences".

- All <input> elements will now have the same behavior. Email, URL, and password
  inputs will no longer be special-cased to always return "none".

This CL also adds some test cases for this new behavior.

Bug: 776618
Change-Id: I4027e81380d5ff7ea7da254b98cfa67b1d202b58
Reviewed-on: https://chromium-review.googlesource.com/872012
Commit-Queue: Ryan Landay <rlanday@chromium.org>
Reviewed-by: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531323}

A few were skipped for no reason. Some of them turned out to pass, while
others fail. Then there were quite a few manual tests, and those we
still need to skip.

Made some (mostly minor) changes to some of the tests to make
them pass. They were making invalid assumptions about
fonts (glyphs with unexpected negative bearing, kerning, etc.)
and line heights. One test was also assuming that painting the
same glyph multiple times would produce the exact same result as
painting it just once, but that's not necessarily true for
anti-aliasing.

It was pretty hard to figure out what
floats-clear/floats-028-ref.xht was about (old IE bug fixed?),
but I made the ref more similar to the test, so that we don't
make assumptions about line height.

Also updated expectations for LayoutNG, because it turned out that more
more of these tests pass there.

Bug: 711704, 711709, 711805, 711807
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: I6e34e037852ef31cfe8d85688865823d6f9d2d0a
Reviewed-on: https://chromium-review.googlesource.com/873310
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531322}

TBR=zmo@chromium.org
NOTRY=true

BUG=602688

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: Ie6cb900f6b89a063323564fcbfb39b4458b3e025
Reviewed-on: https://chromium-review.googlesource.com/881822
Commit-Queue: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531321}

Everything about these URLs is defined in the fileapi spec so it seems
to make sense to have the implementation in the fileapi directory rather
than in core/html.

Bug: none
Change-Id: Ic50ea770ef24e786f888a3a70cbea39b37e18736
Reviewed-on: https://chromium-review.googlesource.com/881305Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531320}

Vertical text (writing-mode: vertical-*) has different selection bounds.
Roughly speaking, selection bounds will be horizontal with different
orientation between start bound and end bound with this CSS.

We didn't check this case before, selection handles didn't get marked
correctly for dragging.

This CL checked this case to fix this.

Bug: 803320
Change-Id: Ibdf451331b652e40d716416321f01d87c951c678
Reviewed-on: https://chromium-review.googlesource.com/875370Reviewed-by: Mohsen Izadi <mohsen@chromium.org>
Commit-Queue: Shimi Zhang <ctzsm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531319}

This CL also typemaps content.mojom.URLLoaderFactoryBundle to
std::unique_ptr<content::URLLoaderFactoryBundleInfo>.

This is part of the effort to avoid using raw mojom::URLLoaderFactory pointers
in renderers and also unify interface of URLLoaderFactory getters.

Bug: 796425
Change-Id: Ia30cd253ec0c82624fbf466a70da489e4464b82f
Reviewed-on: https://chromium-review.googlesource.com/877390
Commit-Queue: Yuzhu Shen <yzshen@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531318}

Change-Id: Ic08885239a5a5c2a1da7475c033ab80aa6f49ed4
Reviewed-on: https://chromium-review.googlesource.com/879404Reviewed-by: Steven Valdez <svaldez@chromium.org>
Reviewed-by: David Benjamin <davidben@chromium.org>
Commit-Queue: Nick Harper <nharper@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531317}

R=dgozman@chromium.org

Bug: chromium:804845
Change-Id: I37ae5ec7e32954bd32cb8e58683e864cd76938c8
Reviewed-on: https://chromium-review.googlesource.com/881462Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531316}

This is a speculative fix for FYI bot crashes.

Change-Id: I8cea6aa917eb32c93f8a0f1967eb630d8c2fca88
Reviewed-on: https://chromium-review.googlesource.com/881663Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531315}

The code in pdf/ can be a GN component instead of a static_library.
Add a PDF_EXPORT macro and set boundaries on what can be used from
external callers.

Change-Id: Ib855736f145c5708b8c62c7fd0e3622827bcc623
Reviewed-on: https://chromium-review.googlesource.com/879853Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531314}

With these exclusions in place, the 64 bit sbox_integration_tests are
at parity with the 32 bit tests.

Bug: 783296
Change-Id: I75661decf5a74c4df96b9ba35097f020f09d9ffe
Reviewed-on: https://chromium-review.googlesource.com/881544Reviewed-by: Will Harris <wfh@chromium.org>
Reviewed-by: Reid Kleckner <rnk@chromium.org>
Commit-Queue: Sigurður Ásgeirsson <siggi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531313}

It was used for the Fedex cloud printing service.

Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ic96c9ecd984a95eec5c0b3c3831c1f43cc1a3254
Reviewed-on: https://chromium-review.googlesource.com/880761Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531312}

As per UX guidance, make the remaining close buttons face the user, such
that when hovered, they appear to grow, rather than slide horizontally.

BUG=
R=vollick

Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_vr;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Id1c8b1794a274f63936d1485befb18fa9718129d
Reviewed-on: https://chromium-review.googlesource.com/881524Reviewed-by: Ian Vollick <vollick@chromium.org>
Commit-Queue: Christopher Grant <cjgrant@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531311}

Change-Id: I2e2787f099f7d101b66cd523b6ae9356e046fba9
Reviewed-on: https://chromium-review.googlesource.com/881204Reviewed-by: Chris Palmer <palmer@chromium.org>
Commit-Queue: Robert Sesek <rsesek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531310}