GitLab

Latest version of Closure compiler is smarter and stricter. The errors
need to be fixed to unblock rolling the latest Closure compiler version.

Bug: 831004
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I2e22cfd40d621263532206e2f816048c6e6a3778
Reviewed-on: https://chromium-review.googlesource.com/1006179
Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org>
Reviewed-by: Joel Hockey <joelhockey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550071}

This patch adds CSS Variable palette to Spectrum in Styles Sidebar Pane.

R=dgozman

Change-Id: I5327b875d65039e8955e804e4b1b3208cd683a33
Reviewed-on: https://chromium-review.googlesource.com/1008998Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Commit-Queue: Andrey Lushnikov <lushnikov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550070}

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

Bug: 786332
Change-Id: Idf6f5dd37a0c6d510a17a6f4c5546a86d9c07333
Reviewed-on: https://chromium-review.googlesource.com/983155Reviewed-by: Alexander Timin <altimin@chromium.org>
Reviewed-by: Dan Sanders <sandersd@chromium.org>
Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550069}

According to UX spec, labels in FeaturePodButton should show while hover
background. In order to implement this, this CL adds
FeaturePodLabelButton class. Also, this CL fixes FeaturePodIconButton
padding which was different from UX spec.

According to the spec, icon and labels in feature pod button are
different click targets, which is still not implemented in this CL. It
will be done in another CL.

Screencast: http://dr/file/d/1FPylXzoO_xCTc8ywQU2NezUug9WgTz-q/view
UX spec: http://shortn/_U7QSE9nhSk

TEST=manual
BUG=821671

Change-Id: I596abbf5226932c9ac4f342e29c83fa4cc12185d
Reviewed-on: https://chromium-review.googlesource.com/1008942Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550068}

It's unused since crrev.com/b23a486c
This CL has no behavior changes.

Change-Id: If2b93b3c5fb105e4e14ed389be2d50d1bfcdb41b
Reviewed-on: https://chromium-review.googlesource.com/1006456Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550067}

This is needed so the client can present a list of saved autofill
profiles.

Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I95438aa53d6ee77d842892e6b7bb1b88f2e29c40
Reviewed-on: https://chromium-review.googlesource.com/1006260
Commit-Queue: John Wu <jzw@chromium.org>
Reviewed-by: Hiroshi Ichikawa <ichikawa@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550066}

CORB tests can use WebPreferences::web_security_enabled=false instead.

BUG=829877

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I7720f997bc747d24a5d7c5e820fb18b1f3ba896a
Reviewed-on: https://chromium-review.googlesource.com/1004002Reviewed-by: Łukasz Anforowicz <lukasza@chromium.org>
Commit-Queue: Charlie Reis <creis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550065}

This CL fixes an issue where buttons were not visible in immersive mode
due to being painted into layers. This has been fixed by disabling the
layers when entering immersive mode, which disables animations, but keeps
the buttons visible.

Bug: 787640
Change-Id: Ie34f71a30b7a3f995d01e11fdeb66ce49ab12c0f
Reviewed-on: https://chromium-review.googlesource.com/1004155
Commit-Queue: calamity <calamity@chromium.org>
Reviewed-by: Alan Cutter <alancutter@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550064}

move_blink_source.py missed to replace them.
This CL has no behavior changes.

Bug: 768828
Change-Id: I699bcc9baf0c4dbada2e8d15c28c61702b5a15e8
Reviewed-on: https://chromium-review.googlesource.com/1009265Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550063}

Font sizes for answers in suggest need to be relative to
CONTEXT_OMNIBOX_PRIMARY, which is a larger size under touch.
Currently the non-"base" font sizes are using hard-coded sizes
from ResourceBundle.

The spec also calls for baselines to be aligned (no subscript/
superscript), and drops the "large" font size.

Move towards looking up fonts using views::style::GetFont().

Bug: 808730
Change-Id: Ic633a1cc38e3845e2186cce737feae33ced58178
Reviewed-on: https://chromium-review.googlesource.com/991696
Commit-Queue: Patti <patricialor@chromium.org>
Reviewed-by: Michael Wasserman <msw@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550062}

S13nSW: Update wpt/service-workers/service-worker/navigation-preload/broken-chunked-encoding.https.html

Before this CL, the test expected that preloadResponse is
rejected when decoding a chunked body is failed. However preloadResponse
is supposed to be resolved once the headers are arrived. It should be
resolved instead of rejected. Update the test to expect that
preloadResponse is resolved even for broken chunked body.

With S13nSW enabled, this test will start passing but it wouldn't
pass w/o S13nSW.

This CL also adds a test case where broken chunked body is transfered
with delays.

Bug: 830472
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: Ibc2909dfc4537f209417a13cff63898ad39229a7
Reviewed-on: https://chromium-review.googlesource.com/1004562
Commit-Queue: Kenichi Ishibashi <bashi@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550061}

This CL forces the hosted app browser actions to be 32x32, even in
touch optimized mode. This fixes an issue where the hosted app title bar
was too tall.

Bug: 830285
Change-Id: I1bc3539f8e43ca332783511dbb937389f09eb20b
Reviewed-on: https://chromium-review.googlesource.com/1006135
Commit-Queue: calamity <calamity@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Alan Cutter <alancutter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550060}

https://skia.googlesource.com/skia.git/+log/ff7fa50f0c95..c6c5eade823a

$ git log ff7fa50f0..c6c5eade8 --date=short --no-merges --format='%ad %ae %s'
2018-04-11 fmalita Reland "Fix handling of MaskFilter matrices"
2018-04-11 angle-skia-autoroll Roll third_party/externals/angle2/ 785b20b7a..2dadd1d08 (11 commits)
2018-04-11 benjaminwagner Upgrade Windows Skolo bots.
2018-04-11 reed remove unused/dumb methods from SkRect
2018-04-11 reed Clean-up paints for drawImage (etc.) to enforce that we always ignore stroking and patheffects.
2018-04-11 caryclark more path is rect bugs
2018-04-11 mtklein skcms→9334796 reject non-XYZ/Lab PCS profiles
2018-04-11 liyuqian Revert "Add private fIsBadForDAA bit to SkPath"
2018-04-11 mtklein skcms→56f2a09 add sRGB profile and ApproximatelyEqualProfiles()
2018-04-11 liyuqian Add private fIsBadForDAA bit to SkPath
2018-04-11 robertphillips Add Vulkan to the DDL bots
2018-04-11 mtklein roll to particular targets
2018-04-11 benjaminwagner Switch to latest Windows Intel Skylake GPU driver.
2018-04-11 fmalita Fix SkTCopyOnFirstWrite copy semantics
2018-04-11 reed no need to tile draws that don't involve paths
2018-04-11 mtklein skcms→91a8958 Detect and canonicalize identity tables
2018-04-11 mtklein skcms→307ec0f remove a layer of nesting in skcms_Curve
2018-04-10 robertphillips Fix alpha threshold image filter GPU implementation
2018-04-10 egdaniel Remove guard around getBackendTexture/RenderTarget calls
2018-04-11 robertphillips Revert "Fix handling of MaskFilter matrices"
2018-04-10 angle-skia-autoroll Roll third_party/externals/angle2/ 0ce016c1b..785b20b7a (7 commits)
2018-04-10 fmalita Fix handling of MaskFilter matrices
2018-04-10 robertphillips Get ViaDDL working with Vulkan
2018-04-10 brianosman skcms→e753c6d disable -Wdouble-promotion
2018-04-10 egdaniel Move GrBackendRenderTarget over to new system of getting backed infos
2018-04-10 liyuqian Remove unused field fBitmapBytes

Created with:
  roll-dep src/third_party/skia
BUG=chromium:824145,chromium:821353,chromium:821353


The AutoRoll server is located here: https://autoroll.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=master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
TBR=borenet@chromium.org

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

To support column-count which takes <integer>s, we had implement integer
handling as resolved by:
https://github.com/w3c/css-houdini-drafts/issues/574#issuecomment-366845604

Basically, a non-integer passed to <integer> is 'out of range', so we
wrap it in a calc(). Coincidentally, column-count also needs to be
positive, so we had to change ComputedStyle::SetColumnCount to clamp.
Note that this doesn't change CSSOM behaviour, as the parser
would've rejected non-positive values so ComputedStyle only gets
positive values. Since Typed OM skips parsing, we have to clamp in
ComputedStyle as well.

Bug: 820299
Change-Id: Ia56e4bdb55826e72d46c66666addda168dea3d2a
Reviewed-on: https://chromium-review.googlesource.com/1001084Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Commit-Queue: Darren Shen <shend@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550058}

On Android O+, avoid a pixel copy by having the XRWebGLDrawingBuffer
render directly into a mailbox texture that's supplied with the
presenting VSync. This texture is backed by a GLImage using a
GpuMemoryBuffer that's using an AndroidHardwareBuffer which is shared
with VrShellGl's native GL context.

Bug: 761436
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.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2;master.tryserver.chromium.linux:linux_vr;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I487e2719f0f19f536231b8bec583340ad3d74bd1
Reviewed-on: https://chromium-review.googlesource.com/1002550
Commit-Queue: Klaus Weidner <klausw@chromium.org>
Reviewed-by: Martin Barbella <mbarbella@chromium.org>
Reviewed-by: Ian Vollick <vollick@chromium.org>
Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Reviewed-by: Bill Orr <billorr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550057}

TBR=mlamouri@chromium.org

Bug: 831688
Change-Id: I2086288cef72e2c2cc7ba581d4b7fb4f00562969
Reviewed-on: https://chromium-review.googlesource.com/1007887Reviewed-by: Tommy Steimel <steimel@chromium.org>
Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Reviewed-by: Becca Hughes <beccahughes@chromium.org>
Commit-Queue: Mounir Lamouri <mlamouri@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550056}

Includes addition of interaction model/observer.
See bug for additional details.

Bug: b:77476776
Change-Id: Ic171f30f95aa66d74836213bd424a2d3a91d3869
Reviewed-on: https://chromium-review.googlesource.com/994535
Commit-Queue: David Black <dmblack@google.com>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Muyuan Li <muyuanli@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550055}

This patch adds a context menu for Crostini app launcher items, of which
the only entry is "Pin to shelf" (or Unpin), inherited from the default
context menu -- note that clicking pinned entries currently does nothing
and the shelf icon is missing. Eventually we'd like to support also
"App info" and "Uninstall" entries.

Bug: 824549
Change-Id: Iee3038707b67232f6d813ecca5b675df973a86e7
Reviewed-on: https://chromium-review.googlesource.com/1006564
Commit-Queue: Timothy Loh <timloh@chromium.org>
Reviewed-by: Nicholas Verne <nverne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550054}

Following bot results are included.
4732 4734 4735 4739 4741 4744 4747 4749 4750 4751

2 lines were removed and 5 lines were deflaked by consecutive
results since 4675.

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: Iee39d2d76cdf36b7f36646f7734de5f49809b740
Reviewed-on: https://chromium-review.googlesource.com/1006236
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550053}

This is patch 3 of a 3-way patch, and must be landed after
crrev.com/c/981777 and crrev.com/i/595643.

Additional cleanup occured downstream in crrev.com/i/602658

Bug: 805123
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Iae4ce1a4d9055fa6e8f7a328872be1ca51fcf49e
Reviewed-on: https://chromium-review.googlesource.com/998444Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550052}

Replaces a flaky check in xr_frame_provider with an early return since
it's a valid state to be in, e.g. when exiting presentation.

Bug: 831243
Change-Id: Ie506095b5b5a1083ad669a000857a70b40f42a60
Reviewed-on: https://chromium-review.googlesource.com/1007622Reviewed-by: Klaus Weidner <klausw@chromium.org>
Commit-Queue: Klaus Weidner <klausw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550051}

This implementation is not use yet, the start_recording_callback
constructor parameter will be passed from ContentAudioStreamFactoryClient
to enable tab UI recording indicator. The connection_erorr_callback will
notify ContentAudioStreamFactoryClient when the stream is destructed.

Design: http://drawings/1_ZIKj6lihGKRjq4Mflduitmkn_REqpHFeqVNelBGHHk

Bug: 803102
Change-Id: Ic57324f761389fa38e2618f541024e5caf49ff82
Reviewed-on: https://chromium-review.googlesource.com/980977Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Yuri Wiitala <miu@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Olga Sharonova <olka@chromium.org>
Reviewed-by: Max Morin <maxmorin@chromium.org>
Commit-Queue: Marina Ciocea <marinaciocea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550050}

BUG=b:76445849, chromium:828119,  chromium:828482
TEST=jpeg_decode_accelerator_unittest and video_encode_accelerator_unittest on StoneyRidge device

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: Ifa61026390fe31e88e0929257a521745e9a72fad
Reviewed-on: https://chromium-review.googlesource.com/983334
Commit-Queue: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Miguel Casas <mcasas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550049}

Moving observer calls a bit up in the code so we don't miss hierarchy
events. We could miss them if |parent_source| is nullptr and we return
from the function earlier than events are sent.
Sorry for not catching this earlier.

Bug: 816802
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel
Change-Id: Ic72e1b1553df6883540c97ff2cead2240fef7c13
Reviewed-on: https://chromium-review.googlesource.com/1008404Reviewed-by: kylechar <kylechar@chromium.org>
Commit-Queue: Illia Martyniuk <illiam@google.com>
Cr-Commit-Position: refs/heads/master@{#550048}

https://pdfium.googlesource.com/pdfium.git/+log/6bebd2e3cfb7..87b67f842fe5

$ git log 6bebd2e3c..87b67f842 --date=short --no-merges --format='%ad %ae %s'
2018-04-11 tsepez Remove CFX_DIBAttribute::m_strAuthor
2018-04-11 hnakashima Fix crash when ColorSpace references itself directly.
2018-04-11 tsepez Fix issues with PDFium third_party/base/span.h
2018-04-11 tsepez Remove use of GetBuffer()/ReleaseBuffer() when c_str() is sufficient.
2018-04-11 tsepez Make cxfa_fmlexer.cpp resilient to null strings
2018-04-11 tsepez Fix trailing whitespace handling in CPDF_StreamParser::ReadInlineStream()
2018-04-11 rharrison Reserve space to reduce memory operations while encoding barcode
2018-04-11 rharrison Add documentation about Valgrind LLD workaround

Created with:
  roll-dep src/third_party/pdfium
BUG=chromium:828206,chromium:831498,chromium:802242,chromium:830706


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: I7134510b37a0aab373bdff663020825afe63436f
Reviewed-on: https://chromium-review.googlesource.com/1008962Reviewed-by: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#550047}

Add utility method FilterDeviceList in device/bluetooth/chromeos
to filter the devices before updating in bluetooth UI.

Extension change is in a separate CL:
https://chromium-review.googlesource.com/c/chromium/src/+/1004414/

Design doc: http://go/cros-bluetooth-filtering

Bug: 817599
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I2a5cfb2ecdacbbe633770bb0c59b2e914b4e4b1f
Reviewed-on: https://chromium-review.googlesource.com/999014
Commit-Queue: Xiaoyin Hu <xiaoyinh@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Miao-chen Chou <mcchou@chromium.org>
Reviewed-by: Rahul Chaturvedi <rkc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550046}

This CL updates the way hosted app windows pick title bar colors
while a theme color is set on chromeOS.

This aligns the behaviour with Android, see:
https://codereview.chromium.org/1956803002.

Before:
https://bugs.chromium.org/p/chromium/issues/attachment?aid=329991&signed_aid=3q0FM93vRJrPJV7kMinOTw==&inline=1
https://bugs.chromium.org/p/chromium/issues/attachment?aid=333012&signed_aid=VNXQQhjxyzYYyg36S3nnOQ==&inline=1
https://bugs.chromium.org/p/chromium/issues/attachment?aid=330203&signed_aid=DoC1IRqCuP659D2dMoL1_Q==&inline=1

After:
https://bugs.chromium.org/p/chromium/issues/attachment?aid=333244&signed_aid=J5yNTnY4-7RqGeYu32N8OA==&inline=1
https://bugs.chromium.org/p/chromium/issues/attachment?aid=333245&signed_aid=VOf6afQMQJbCApYu21wmyw==&inline=1
https://bugs.chromium.org/p/chromium/issues/attachment?aid=333257&signed_aid=OJzGQyVnXzCfXF6bZ16TCg==&inline=1
https://bugs.chromium.org/p/chromium/issues/attachment?aid=333258&signed_aid=jygsmXqxXfoPsPZXb_gqfg==&inline=1

Bug: 814121
Change-Id: I10a2e837d0c8a938072171715119a04c28e4f36a
Reviewed-on: https://chromium-review.googlesource.com/999213
Commit-Queue: Alan Cutter <alancutter@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550045}

Verdana is NOT metrically-compatible with Arimo, but it is chosen as the
replacement since they are both from the Sans family.  In all of the layout test
deltas, it appears the choice of Verdana was purely coincidental, so it should
be safe to switch to Arimo.

BUG=787020

Change-Id: Id8c82888b2e844e35200d20532b88d87066792bc
Reviewed-on: https://chromium-review.googlesource.com/1006221Reviewed-by: Michael Wasserman <msw@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Thomas Anderson <thomasanderson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550044}

Bug: 825034
Change-Id: Iba0493cdf2b0edc1c52c919f2e13ba62f2ed1487
Reviewed-on: https://chromium-review.googlesource.com/1008415Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Commit-Queue: Xiaoqian Dai <xdai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550043}

Adds the new audio::GroupMember implementation to provide muting and
snooping functionality to audio::OutputController.

Bug: 824019
Change-Id: Ib21eccf43d4f6994657731545ceb3f4218c18cbe
Reviewed-on: https://chromium-review.googlesource.com/1001512
Commit-Queue: Olga Sharonova <olka@chromium.org>
Reviewed-by: Olga Sharonova <olka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550042}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/5888d6f67672..7e9e2c82bdf1

$ git log 5888d6f67..7e9e2c82b --date=short --no-merges --format='%ad %ae %s'

Created with:
  roll-dep src/third_party/depot_tools


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


TBR=agable@chromium.org

Change-Id: Ica9d5e07ba3b181009f0c2581a682097cadba89f
Reviewed-on: https://chromium-review.googlesource.com/1008418Reviewed-by: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#550041}

After this CL, SignedHTTPExchange related errors will be shown in DevTool's console.
Screenshot: crbug.com/830505#c3

Bug: 830505
Change-Id: Icfc184d98ca2bb1f83e378c596b8b73f4da8a1b6
Reviewed-on: https://chromium-review.googlesource.com/1004880
Commit-Queue: Tsuyoshi Horo <horo@chromium.org>
Reviewed-by: Kunihiko Sakamoto <ksakamoto@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550040}

This reverts commit 30b94a8a.

Original change's description:
> Revert "VR: Clean up WebVR headset insertion."
>
> This reverts commit f2ccbfd1.
>
> Reason for revert: Speculative revert for crbug.com/831589
>
> Original change's description:
> > VR: Clean up WebVR headset insertion.
> >
> > This CL makes it explicitly clear when we care about whether the page
> > was listening for activate before pause, and when we should autopresent
> > on entering VR (for headset insertion).
> >
> > This also removes the workaround in vr_display_impl that allowed pages
> > without focus to handle displayActivate, favoring handling this case
> > more correctly in VrShellDelegate.
> >
> > Bug: 829513
> > Change-Id: Ie19c55305d0c59ec3f576010ded0f61ae906a084
> > Reviewed-on: https://chromium-review.googlesource.com/1004927
> > Reviewed-by: Yash Malik <ymalik@chromium.org>
> > Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#549736}
>
> TBR=mthiesse@chromium.org,ymalik@chromium.org
>
> Change-Id: I92a56254e933381770a56495553facfbcb2e5c59
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 829513
> Reviewed-on: https://chromium-review.googlesource.com/1007374
> Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
> Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#549886}

TBR=bajones@chromium.org

Change-Id: I84fa5bd2f187859770166c1a3c79387492b50df1
Bug: 829513
Reviewed-on: https://chromium-review.googlesource.com/1007922Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Reviewed-by: Yash Malik <ymalik@chromium.org>
Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550039}

This adds two new events to aura_output interface that makes it
easier for clients to calculate an ideal contents scale.

This also adds a new request that can be used to set a unique
application ID for a surface. The application ID identifies the
general class of applications to which the surface belongs. The
compositor can use this to group multiple surfaces together.
For example, "org.chromium.FooViewer".

Note that the application ID set here takes precedence over the
app id set through XDG shell interface. To keep it simple, we
just remove the implementation of xdg surface set_app_id. If
that turns out to be useful in the future, then we can always
re-implement that and make it available as an alternative ID.

Bug: 824449, 791672
Test: wayland_client_info
Change-Id: I3219cedbd7d9ce65e73ca33127ea976acc8c4163
Reviewed-on: https://chromium-review.googlesource.com/1008664Reviewed-by: Daniele Castagna <dcastagna@chromium.org>
Commit-Queue: David Reveman <reveman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550038}

UpdateRootWindowSizeInPixels() should always use the pixel-bounds it
already knows about. So it does not need to take any input params. Fix
extension-shell to use SetBoundsInPixels() instead, because that's the
correct way to set the size of the host (which would automatically
update the size of the root-window etc.).

BUG=821987

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

https://chromium.googlesource.com/catapult.git/+log/a22719853354..cc64c0d81aab

$ git log a22719853..cc64c0d81 --date=short --no-merges --format='%ad %ae %s'
2018-04-10 simonhatch Dashboard - Fix stats being generated in some cases
2018-04-11 eakuefner [TBMv2] Add HAD_FAILURES diagnostic and use it to avoid merging
2018-04-11 agrieve device_utils.PushChangedFiles: Fix caching logic when pushing files
2018-04-09 ulan Reduce noise of memory metric.

Created with:
  roll-dep src/third_party/catapult
BUG=chromium:831388,chromium:826384


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

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

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


TBR=sullivan@chromium.org

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

*) remove zero_top_border_height_ hack.
a) Always use fullscreen_visible_fraction_  in HeaderView::GetPreferredOnScreenHeight

Cleanup the debug code that I accidentally included in other CLs.

NonClientFrameController doesn't have use immersive in tablet behavior, os
I left it as is.

Browser frame already uses immersive condition to compute the height, so
no changes in browser frame code.

BUG=b/33693796
TEST=no functional change except for a) above. a) is covered by unittests.

Change-Id: Iae7c236426216a3c251de8e40a5ecd69831c0635
Reviewed-on: https://chromium-review.googlesource.com/999967Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Mitsuru Oshima <oshima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550035}

Bug: 831451
Bug: 831046
Change-Id: I6d48e0e245c0d8c3fd3a505b9eaa6f542192457e
Reviewed-on: https://chromium-review.googlesource.com/1006455Reviewed-by: Shinya Kawanaka <shinyak@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Fumitoshi Ukai <ukai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550034}

Add GDPR text to the local discovery and Print Preview flows for
registering and accepting cloud printers, respectively.

Bug: 826591
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I4ed895c9b5db992dcd64cdcfc86206860d896739
Reviewed-on: https://chromium-review.googlesource.com/1005571
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550033}

Cleanup double StopMonitor and unnecessary condition, leftover from mouse listener removal in https://crrev.com/2577573002.

Bug: 828864
Change-Id: Ia19c5bf4f81897f49b1c0eaf0f6d85a9cfff4984
Reviewed-on: https://chromium-review.googlesource.com/1007194Reviewed-by: Max Morin <maxmorin@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Marina Ciocea <marinaciocea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550032}