GitLab

NOTRY=true
TBR=avi@chromium.org

Bug: 872474, 872258
Change-Id: I891cb4297476ab14d15878bcebafb109a5c3c60c
Reviewed-on: https://chromium-review.googlesource.com/1168472Reviewed-by: David Tseng <dtseng@chromium.org>
Commit-Queue: David Tseng <dtseng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581725}

And disable http/tests/media/video-load-metadata-decode-error.html,
which appears to have a dependency on GPUs.

Fixes to original change:
- Instead of targeting bots w/o GPUs, this instead targets any 10.12 bot (GPUs or not).
- Doesn't change the suite on the Mac10.12 builder since all 10.12 bots have GPUs:
https://chromium.googlesource.com/chromium/src/+/2021963d6d4b6204fb4c935f487631e577ffc6ae/testing/buildbot/waterfalls.pyl#2892

Bug: 854874, 871445
Change-Id: Idf3603785500fe3aeee3f978e4e4f7ec18c76ba4
Reviewed-on: https://chromium-review.googlesource.com/1166338Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581724}

The BattOr code is soon going to be delete and these builders
aren't needed anymore.

R=dpranke@chromium.org
CC=​nednguyen@chromium.org

Bug: 859514
Change-Id: I30c0f8ada35a0ff2070228e667b29258de5aa9a5
Reviewed-on: https://chromium-review.googlesource.com/1167975Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Charlie Andrews <charliea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581723}

BUG=865292

Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: Ia654723c1fa870921658fa2d53bc77114627fcd2
Reviewed-on: https://chromium-review.googlesource.com/1167112Reviewed-by: Sergio Collazos <sczs@chromium.org>
Commit-Queue: Rohit Rao <rohitrao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581722}

This is a reland of 30d8a3b9

This fixes an issue where FlushForTesting would timeout if the InterfacePtr
was destroyed in the middle of a flush. Also fixes a webview test that was
flaky on component builds.

Original change's description:
> Fix flaky webrequest related network service tests in component builds
>
> These tests needed some extra synchronization due to the changes made in
> http://crrev.com/c/1139048. These were flaking in component builds.
>
> Bug: 871224
> Change-Id: Ibe370f9b9c8ebb94ee1dd21ba204c06e91518057
> Reviewed-on: https://chromium-review.googlesource.com/1163987
> Commit-Queue: Clark DuVall <cduvall@chromium.org>
> Reviewed-by: John Abd-El-Malek <jam@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#580980}

Bug: 871224, 871720
Change-Id: Ib66c759ef422124f166583c57f89f0a56d92f841
Reviewed-on: https://chromium-review.googlesource.com/1165504Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Ken Rockot <rockot@chromium.org>
Commit-Queue: Clark DuVall <cduvall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581721}

Will be used by clients to associated their installation request to the
installed app.

Bug: 864904
Change-Id: Ib439b669340373e8db3822a2c149f6ea2041d631
Reviewed-on: https://chromium-review.googlesource.com/1156191
Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Reviewed-by: Dominick Ng <dominickn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581720}

Missed during the last change to this code. Unsuppress the associated
WebGL conformance test.

Bug: 857303
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;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: Iefb2f90e6c440ae5eabc1bedd4171fbfe9a3dd0a
Reviewed-on: https://chromium-review.googlesource.com/1166454Reviewed-by: Kai Ninomiya <kainino@chromium.org>
Commit-Queue: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581719}

This reverts commit 99df3b26.

Reason for revert: Caused failures on msan bot.

Original change's description:
> aura: makes each Window be associated with an Env
> 
> This is to avoid using Env::GetInstance(), so that there can be multiple
> Envs created at once.
> 
> BUG=847992
> TEST=covered by tests
> 
> Change-Id: I2e2604c8f8a356ca9cc6771ed050ef8431e7b840
> Reviewed-on: https://chromium-review.googlesource.com/1165861
> Commit-Queue: Scott Violet <sky@chromium.org>
> Reviewed-by: Michael Wasserman <msw@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#581568}

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

Change-Id: I50fb5f0d27011219ed2569b1035a78e01b9b3de9
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 847992
Reviewed-on: https://chromium-review.googlesource.com/1168323Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581718}

Moves focues with the selected item.
Marks the menu and items with the proper roles.
Exposes whether or not an item is checked via its accessible name.

Change-Id: Id8a9955df9c05b21291052f2460321618caf90d8
Reviewed-on: https://chromium-review.googlesource.com/1164026
Commit-Queue: Joel Einbinder <einbinder@chromium.org>
Reviewed-by: Andrey Lushnikov <lushnikov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581717}

This reverts commit 9feb99aa.

Reason for revert: a patch earlier in the chain caused msan failures.

Original change's description:
> wm/views: makes objects using Env take Env
> 
> This allows for multiple Envs at the same time. Additionally:
> . NativeWidgetPrivate::CreateNativeWidget: now takes InitParams
> . NativeWidgetPrivate::IsMouseButtonDown() is now an member function.
> 
> BUG=847992
> TEST=covered by tests
> 
> Change-Id: Ic35b274c0a927a7c602809a7e611383b5d3dd3e2
> Reviewed-on: https://chromium-review.googlesource.com/1166148
> Commit-Queue: Scott Violet <sky@chromium.org>
> Reviewed-by: Michael Wasserman <msw@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#581621}

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

Change-Id: I9410bf471a71d07b32f49a139b7afa9ced0f3fea
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 847992
Reviewed-on: https://chromium-review.googlesource.com/1168403Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581716}

This reverts commit 04308f3d.

Reason for revert: Patch earlier in the chain caused failures on msan bot.

Original change's description:
> window-service: makes objects using Env take Env (or get from WindowService)
> 
> This allows for multiple Envs at the same time.
> 
> BUG=847992
> TEST=covered by tests
> 
> Change-Id: I20dda0f7e3fde50027c9c867562718e49152d238
> Reviewed-on: https://chromium-review.googlesource.com/1167654
> Reviewed-by: Michael Wasserman <msw@chromium.org>
> Commit-Queue: Scott Violet <sky@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#581701}

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

Change-Id: If149ac9dc90f0ef6cec5e59f744cd0c3189d01e6
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 847992
Reviewed-on: https://chromium-review.googlesource.com/1168402Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581715}

Bug: 869526
Change-Id: I71cd2977f6dd3d0589727ca3550ff4208b987c2e
Reviewed-on: https://chromium-review.googlesource.com/1165946
Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Reviewed-by: Jan Krcal <jkrcal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581714}

Bug: 853688
Change-Id: Ifa83c61a4bf2865379e27945bd1dc7eba1e74fc8
Reviewed-on: https://chromium-review.googlesource.com/1164162
Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Reviewed-by: Jan Krcal <jkrcal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581713}

This CL is to fix the reverted change 1165855. It was missing an
import and therefore failing its browsertest. This initial patchset
consists of the changes in 1165855 to make it easier to see the fix.

Here is the original description for 1165855:

As the multidevice Settings UI subpage fills up, it will need a few
browsertests, so this CL is just to set that up so the infrastructure
is there and to add a couple basic tests for the existing contents.


Bug: 824568
Change-Id: I3ea2c48168a2bec0be29e5867a162684ac4e903c
Reviewed-on: https://chromium-review.googlesource.com/1167967Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Commit-Queue: Jordy Greenblatt <jordynass@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581712}

Make ash map VKEY_SNAPSHOT to the TAKE_SCREENSHOT action.
7ab3497a updated Chrome to map the "Print Screen" key to
VKEY_SNAPSHOT rather than VKEY_PRINT.

Bug: 683097, 846919, 872094
Change-Id: I3e8de9e8b2d110d8e06649fc0553c6d9b7cab35a
Reviewed-on: https://chromium-review.googlesource.com/1166037
Commit-Queue: Dan Erat <derat@chromium.org>
Reviewed-by: Kevin Schoedel <kpschoedel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581711}

Bug: 870270
Cq-Include-Trybots: luci.chromium.try:closure_compilation
Change-Id: Ib19dd0974d318468d602e0b220c0cc5c46a2bc31
Reviewed-on: https://chromium-review.googlesource.com/1161445
Commit-Queue: Esmael El-Moslimany <aee@chromium.org>
Reviewed-by: Scott Chen <scottchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581710}

Move the following from BridgedNativeWidget to
BridgedNativeWidgetHostImpl:
- ui::Compositor (as RecyclableCompositorMac)
- ui::LayerDelegate subclass
- ui::LayerOwner subclass
- ui::AcceleratedWidgetMacNSView subclass

Update BridgedNativeWidgetHost interface to include methods to
indicate the desired size and visibility of the ui::Compositor.

Add methods to BridgedNativeWidget which will in the future be
transferred to the abstract interface (when BridgedNativeWidget is
renamed to BridgedNativeWidgetImpl, which I'm still putting off for
merge-ability sake).
- OnCompositorCreated indicates that the compositor is created and
  should receive size and visibility updates
- SetCALayerParams sends the parameters needed to display the
  compositor contents in another process

Bug: 859152
Change-Id: I62209be44035f6b2b596afeb827cd951f30ef40c
Reviewed-on: https://chromium-review.googlesource.com/1166196
Commit-Queue: ccameron <ccameron@chromium.org>
Reviewed-by: Trent Apted <tapted@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581709}

This change updates the background icon supplied by the UX designer.

Bug: 872097
Change-Id: I82ce3f9fd1509ecf42045bd0ebf0179f72736f6a
Reviewed-on: https://chromium-review.googlesource.com/1166445Reviewed-by: apacible <apacible@chromium.org>
Reviewed-by: Evan Stade <estade@chromium.org>
Commit-Queue: apacible <apacible@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581708}

Bug: 717812
Change-Id: Ief0d69e6b676a8fd7456a16acd221851f70bc13f
Reviewed-on: https://chromium-review.googlesource.com/1164471
Commit-Queue: Chase Phillips <cmp@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581707}

Because of a change in the way update client handles extensions that
were remove during the update process UpdateService can crash when
handling update events emitted from UpdateClient.

Bug: 868906, 869663
Change-Id: I78cf5075fab84075f8472edcb2428348794307fe
Reviewed-on: https://chromium-review.googlesource.com/1155749
Commit-Queue: Minh Nguyen <mxnguyen@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Reviewed-by: Sorin Jianu <sorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581706}

There are a number of buttons that rely on the old Button style that sets
text attributes. We need to change them to use the pre-defined text
styles instead.

Bug: 872223
Change-Id: I57c5e65892b064c2e4456e3323611a35228ce545
Reviewed-on: https://chromium-review.googlesource.com/1167747
Commit-Queue: Becky Zhou <huayinz@chromium.org>
Reviewed-by: Theresa <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581705}

I added task_started time to RemovalTask structure, this was dropped
when this class was ported from content.

Bug: 796231, 759229
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I19f44ef3a72685b1fbe398e2915b8fd842307541
Reviewed-on: https://chromium-review.googlesource.com/1164571Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: Mohammad Refaat <mrefaat@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581704}

Specify neccessary python protos instead.

This is for deterministic tarballs sent to swarming for
sync_integration_tests execution.

Bug: 869348
Change-Id: Ifc873081056b7ee19f13a5d819829a05e3179d45
Reviewed-on: https://chromium-review.googlesource.com/1167343Reviewed-by: Marc Treib <treib@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581703}

Bug: 872065
Change-Id: If0de2e5cb8470e604bc97f1c3689ae8972888e6c
Reviewed-on: https://chromium-review.googlesource.com/1167968Reviewed-by: Ken Rockot <rockot@chromium.org>
Commit-Queue: Christian Fremerey <chfremer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581702}

This allows for multiple Envs at the same time.

BUG=847992
TEST=covered by tests

Change-Id: I20dda0f7e3fde50027c9c867562718e49152d238
Reviewed-on: https://chromium-review.googlesource.com/1167654Reviewed-by: Michael Wasserman <msw@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581701}

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

Center the EoC icon in its bounding box when Duet is enabled.

BUG=859592

Change-Id: I7c1e016e2454ab5abb212c229b3b16450b553e7b
Reviewed-on: https://chromium-review.googlesource.com/1166245Reviewed-by: Matthew Jones <mdjones@chromium.org>
Commit-Queue: Theresa <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581699}

Fixes the crash in ConfirmInfoBarController and potentially translate
infobar controllers after the refactoring of infobars in
crrev.com/c/1037640.

It does so by setting |delegate| to nullptr on the InfoBarController
instance before InfoBarIOS is destroyed.

Bug: 870339
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I82cca094b9d0cc7fa01f19cae83aa1930fb71a64
Reviewed-on: https://chromium-review.googlesource.com/1161324Reviewed-by: Peter Lee <pkl@chromium.org>
Commit-Queue: Moe Ahmadi <mahmadi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581698}

The NetworkConnectionTracker instance currently lives in
g_browser_process, but we need the tracker in //content, which can't
access //chrome. This adds a static getter in
network_service_instance.h, which can be used in //content.

Bug: 821009
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Change-Id: Iaedb89cf2d27f75e0b896d7fd7389ceab3fe4682
Reviewed-on: https://chromium-review.googlesource.com/1162888Reviewed-by: Julian Pastarmov <pastarmovj@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Robbie McElrath <rmcelrath@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581697}

https://chromium.googlesource.com/chromiumos/chromite.git/+log/94ec5e5e83b9..e41df534852b


git log 94ec5e5e83b9..e41df534852b --date=short --no-merges --format='%ad %ae %s'
2018-08-08 dgarrett@google.com chromeos_config: Add eve-chrome-pfq.


Created with:
  gclient setdep -r src/third_party/chromite@e41df534852b

The AutoRoll server is located here: https://chromite-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=chrome-os-gardeners@chromium.org

Change-Id: I0dee8a6e981cbbb712efb0557e00979f8aebecb7
Reviewed-on: https://chromium-review.googlesource.com/1167822Reviewed-by: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#581696}

The set_bounds request, which supersedes set_window_geometry, allows
the client to specify a target display such that window bounds are
expressed in display rather than screen coordinates.

Bug: b:77972398
Test: None
Change-Id: I6c229e72bf2160b86067e768b31f1ac5148d4aff
Reviewed-on: https://chromium-review.googlesource.com/1164553
Commit-Queue: Dominik Laskowski <domlaskowski@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581695}

This change creates a service for hosting desktop VR devices so
they can be run at the appropriate level of sandboxing required to ship.

The browser process will consume this service to enable WebVR/WebXR
support to ship on Windows.

Currently the service runs in a utility process with no sandbox. A
subsequent change will enable an appropriate sandbox behind a flag to
enable testing.

Bug: 795510
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:linux_vr;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: I5d602f4db91bcd1ddb5a3edf535f90e126004d4a
Reviewed-on: https://chromium-review.googlesource.com/1155754Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Commit-Queue: Bill Orr <billorr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581694}

Bug: 367595
Change-Id: I940257b49fce15310dc680c0b7c283f48ae681cf
Reviewed-on: https://chromium-review.googlesource.com/1164580Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Reviewed-by: agrieve <agrieve@chromium.org>
Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581693}

This reverts commit f86a758d.

Reason for revert: Breaks x86 bots
https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Android%20x64%20Builder%20%28dbg%29/28950

Original change's description:
> [vr] Make VR Java android_library and separate VR JNI registration
> 
> This CL does two things:
> 
> - In order to move VR Java code into its own module move the code into a
>   separate android_library target.
> 
> - As a result of the separate library target we may miss making the VR
>   JNI registrations (in case VR moves out of chrome_public_apk). Thus,
>   separate the JNI registration generation for VR and register VR JNI
>   calls in VR code.
> 
> Bug: 862694
> Change-Id: I2b78cea7b30725d5f2b51afbcac1a6add555f355
> Reviewed-on: https://chromium-review.googlesource.com/1158715
> Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
> Reviewed-by: agrieve <agrieve@chromium.org>
> Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#581576}

TBR=mthiesse@chromium.org,agrieve@chromium.org,tiborg@chromium.org

Change-Id: Iccac3c8981c21fe105799e045f1629dbd99fb7c6
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 862694
Reviewed-on: https://chromium-review.googlesource.com/1168043Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: Ted Choc <tedchoc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581692}

BUG=800033

Change-Id: I0007ae432782e4222f1eed4aad03259a9e25a784
Reviewed-on: https://chromium-review.googlesource.com/1167556Reviewed-by: Theresa <twellington@chromium.org>
Commit-Queue: Ted Choc <tedchoc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581691}

Bug: 864705
Change-Id: I8ebce1fd5911f90b8313a27f9d9e11e229ec0b6b
Reviewed-on: https://chromium-review.googlesource.com/1162757
Commit-Queue: Vlad Tsyrklevich <vtsyrklevich@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581690}

This change drops GeoTrust from the pinning set and adds GIA G3 and
another GTS intermediate, 1O1. GIA G2 remains in the pinning set because
it's still valid until the end of 2018 and some Google properties
may be using it. But the vast majority of Google properties are now
using GlobalSign R2 and the GIA G3 and 1O1 intermediates.

Change-Id: Ia4765525f419e654295199aa654e25c955affb47
Reviewed-on: https://chromium-review.googlesource.com/1166224Reviewed-by: Ryan Sleevi <rsleevi@chromium.org>
Commit-Queue: Adam Langley <agl@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581689}

Background:
There are several corner cases that home launcher is not shown:
1. Drag app list from shelf, meanwhile switch to tablet mode.
2. Drag app list to shelf after being opened, meanwhile switch to tablet
   mode.
3. Close app list and, during the close animation, switch to tablet
   mode.

Changes:
1. Ignore drag event from both shelf or app list and reset dragging
   state when tablet mode is on.
2. Immediately stop close animation before reshow app list when tablet
   mode is on.

Bug: 872072,871923
Change-Id: I5bd7dae242d84e2f0a82839693fbd36930f8e613
Reviewed-on: https://chromium-review.googlesource.com/1167658Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Weidong Guo <weidongg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581688}

Previously, MediaCodecVideoDecoder would only release codec buffers
when starting a drain if it wasn't going to elide the drain itself.
However, since the drain (elided or not) is going to be followed
by a codec flush or destruction, we still invalidate any unrendered
output buffers.

While this isn't techincally required, AVDA did it and seems to
have much fewer codec hangs.

Bug: 869365
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;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: I903301950e190b64d8618b06689cbf9d184a96c9
Reviewed-on: https://chromium-review.googlesource.com/1167797Reviewed-by: Dan Sanders <sandersd@chromium.org>
Commit-Queue: Frank Liberato <liberato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581687}

Bug: 729848
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Change-Id: I47030b560c2187c6cb48217338f071400539cd08
Reviewed-on: https://chromium-review.googlesource.com/1167801Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Clark DuVall <cduvall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581686}