GitLab

ConnectWithInProcessFrameSinkManager() isn't used anymore.

Bug: none
Change-Id: Ie8177debbfae168be7297fa925a574b2298a523a
Reviewed-on: https://chromium-review.googlesource.com/879487
Commit-Queue: kylechar <kylechar@chromium.org>
Commit-Queue: ccameron <ccameron@chromium.org>
Reviewed-by: ccameron <ccameron@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531051}

Embedded frame invalidation/painting goes through PlatformChromeClient.

- Invalidate PlatformChromeClient when any content in the embedded frame
  is invalidated;

- Apply InnerBorderRadiusClip when painting embedded contents.

Bug: 771643
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I1b2796a45f4e48f6912dddd820f33e18ec62e9f1
Reviewed-on: https://chromium-review.googlesource.com/879127
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531050}

This new interface serves two purposes:

1. By holding this class, we can make sure the underlying CDM or
CdmProxy are kept alive. This provides a unified way to support both CDM
and CdmProxy.

2. This helps avoid the need to hold a reference to the CDM directly,
further separating the media stack and the EME stack. After this change,
we should be able to move content_decryption_module.h from media/base to
media/cdm (in a later CL).

This CL also adds CdmContextRefImpl, a default CdmContextRef impl that
holds a reference to a media::ContentDecryptionModule.

BUG=799310,786736,803710
TEST=No functionality change. All existing tests pass.

Change-Id: I64bae76d5a5a0912728ebd8af7ea1ef57a638305
Reviewed-on: https://chromium-review.googlesource.com/876767
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Reviewed-by: John Rummell <jrummell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531049}

KEY_STATUS_MAX is the last key status, the total count should + 1.

BUG=internal b/72168172
TEST=cast_cdm_widevine_test

Change-Id: I6348e2ace9353655a3c31fc009d462a6ce6ebff1
Reviewed-on: https://chromium-review.googlesource.com/879561
Commit-Queue: Yuchen Liu <yucliu@chromium.org>
Reviewed-by: Luke Halliwell <halliwell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531048}

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

Fixed a bug which would start a clearing of cached pages on Chrome
startup that might have impact on startup related perf metrics.

Bug: NONE
Change-Id: Ide9143e381c9aa85c44729f1a12312ce710c5a85
Reviewed-on: https://chromium-review.googlesource.com/872112
Commit-Queue: Yafei Duan <romax@chromium.org>
Reviewed-by: Carlos Knippschild <carlosk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531046}

This reverts commit 45582a71.

Reason for revert: Reneable framebusting on trunk now that this is merged to M64 branch.

Original change's description:
> Disable framebusting intervention for M64.
> 
> Bug: 624061
> Change-Id: I0b863938467b81dae3fbecd17b23aaf6faffe2a7
> Reviewed-on: https://chromium-review.googlesource.com/876508
> Reviewed-by: Ojan Vafai <ojan@chromium.org>
> Reviewed-by: Nasko Oskov <nasko@chromium.org>
> Commit-Queue: Nate Chapin <japhet@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#530608}

TBR=nasko@chromium.org,ojan@chromium.org,japhet@chromium.org

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

Bug: 624061
Change-Id: I0d3963d7746944cd37123f9d4d8b88549c12e6c1
Reviewed-on: https://chromium-review.googlesource.com/878924Reviewed-by: Nate Chapin <japhet@chromium.org>
Commit-Queue: Nate Chapin <japhet@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531045}

Dynamically sets clip bounds while animating

Bug: 769810
Change-Id: Iabde490af4d134aeee7d0a0c10bf13bbf98f70cc
Reviewed-on: https://chromium-review.googlesource.com/871412Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: Daniel Park <danielpark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531044}

This and subsequent tests necessitate some slightly complex logic for ensuring
that the resposne callback is invoked appropriately. Rather than reinvent an
ad-hoc mocking framework, make reasonable use of the one already in use in
other content_junit_tests.

Bug: 794320
Change-Id: I4f39f5c3ca531b6c802da49e0bb2067d0b6e3045
Reviewed-on: https://chromium-review.googlesource.com/879342Reviewed-by: Bo <boliu@chromium.org>
Commit-Queue: Jeremy Roman <jbroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531043}

Bug: 796247
Change-Id: I5614ab12b396f86f6c385d80ecc72c8f59fc0778
Reviewed-on: https://chromium-review.googlesource.com/834450Reviewed-by: Peter Williamson <petewil@chromium.org>
Commit-Queue: Cathy Li <chili@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531042}

https://webrtc.googlesource.com/src.git/+log/9c1848952ede..700ef33edcb5

$ git log 9c1848952..700ef33ed --date=short --no-merges --format='%ad %ae %s'

Created with:
  roll-dep src/third_party/webrtc


The AutoRoll server is located here: https://webrtc-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=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng;master.tryserver.chromium.win:win-msvc-dbg
TBR=webrtc-chromium-sheriffs-robots@google.com

Change-Id: I5affe04ad7bf41797611934b0c09d5b0b9037df2
Reviewed-on: https://chromium-review.googlesource.com/879485Reviewed-by: WebRTC Roll Bot <webrtc-autoroll@chromium.org>
Commit-Queue: WebRTC Roll Bot <webrtc-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531041}

Fixing request job unit tests. The test was using an empty file path
which will make the pages deleted due to quota-based temporary page
clearing.
Also rearranged the order of model initialization to inject clock and
set testing switches before tasks gets executed.

Bug: NONE
Change-Id: I639d6e90bbf29d23313c11ba54374edeba120da0
Reviewed-on: https://chromium-review.googlesource.com/877045Reviewed-by: Jian Li <jianli@chromium.org>
Commit-Queue: Yafei Duan <romax@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531040}

Jumbo build experiments noticed that no code ever read engine_
in XmppIqEntry in libjingle_xmpp. Normal build didn't because
"friend class XmppEngineImpl" required the compiler to know
what XmppEngineImpl does as well.

Bug: 777434
Change-Id: I661a5346617a92ee912060bebf35801dbe1a8797
Reviewed-on: https://chromium-review.googlesource.com/873371Reviewed-by: Tommi <tommi@chromium.org>
Commit-Queue: Harald Alvestrand <hta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531039}

Change |input_data| parameter of CBORReader::Read() from const
std::vector<uint8_t>& to base::span<uint8_t const> to prevent
unnecessary intermediate copies.

Bug: 803605
Change-Id: Id96df6a9c806bb39549f5696f73853faf71c3a94
Reviewed-on: https://chromium-review.googlesource.com/875006Reviewed-by: Kim Paulhamus <kpaulhamus@chromium.org>
Reviewed-by: Balazs Engedy <engedy@chromium.org>
Commit-Queue: Jun Choi <hongjunchoi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531038}

Now correctly sanitizing effective-make-and-models at all ppd_provider
entrypoints.

BUG=chromium:804014
TEST=checked manually that fixed worked; updated unitests to follow

Change-Id: Ib437dadb3d669f670a0d2142445ac49644c5705d
Reviewed-on: https://chromium-review.googlesource.com/877323Reviewed-by: Sean Kau <skau@chromium.org>
Commit-Queue: Luum Habtemariam <luum@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531037}

Currently a handful of low level places depend upon
//services/ui. This was done at a time when the only way to launch a
service was via the service. Now that services may be packaged in
other ways (such as baked into a binary), these dependencies on the
service may result in unnecessarily building the target.

This patch pushes the dependencies into the actual exe targets as they
are the best place to make this decision as to whether the ui service
is needed.

BUG=783326
TEST=none, build only changes

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

Bug: 637098
Change-Id: I18ddab35db1ff3155b847576880de4df4e1290ba
Reviewed-on: https://chromium-review.googlesource.com/875522Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Trent Apted <tapted@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531035}

If a <progress> element has its appearance removed with -webkit-appearance:none,
it should still retain its role for accessibility. This CL also adds a test
to check that various other control types are not affected by this issue.

R=aboxhall@chromium.org

Bug: 687828
Change-Id: I1a30e581dd109b34cae69ec9737cb233ceb5f1fb
Reviewed-on: https://chromium-review.googlesource.com/861054Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: Rob Dodson <robdodson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531034}

One of the consequence of https://crrev.com/c/867177 is that
you can't get ImeAdapter any longer with WebContents from CVC
once |CVC.destroy()| is called. #onCreateInputConnection can
occur afterwards, which needs to be treated as handled with
the default action returning null.

This helps the failing test pass again. The test is for tablet only,
therefore evaded the trybot step.

Bug: 803995
Change-Id: Ia4de2199737aba7ec49d820fc5237a20f96690e7
Reviewed-on: https://chromium-review.googlesource.com/878079
Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org>
Reviewed-by: Bo <boliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531033}

We were incorrectly purging unlocked discardable to "make room" in the
cache, although unlocked entries don't take up room. This resulted in
non-optimal cache purging when we hit at-raster scenarios.

Now that all cached items are discardable, this change removes all
purging other than that needed to handle our max items in cache limit.

Bug: 802819, 804328
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: I5cece605bbd40283073c075e78b36867c8c7efbf
Reviewed-on: https://chromium-review.googlesource.com/879064
Commit-Queue: Eric Karl <ericrk@chromium.org>
Reviewed-by: Khushal <khushalsagar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531032}

Bug: 803727
Change-Id: I2fc28069cd33106b3c9019b402e9f549d956e54c
Reviewed-on: https://chromium-review.googlesource.com/879466Reviewed-by: Dan Erat <derat@chromium.org>
Commit-Queue: Jacob Dufault <jdufault@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531031}

These were removed from the build in 7426194d but
accidentally left in the tree.

TBR=peter@chromium.org

Bug: 755413
Change-Id: Iba88cc7091f897c00a9d2b2173482f2fbaf86317
Reviewed-on: https://chromium-review.googlesource.com/878909Reviewed-by: Evan Stade <estade@chromium.org>
Commit-Queue: Evan Stade <estade@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531030}

In preparation for eliding parts of the displayed URL in the steady
state Omnibox, we are going to introduce a GetURLForDisplay method.

To keep things clear, we're renaming the old method from
GetFormattedURL to GetFormattedFullURL to clarify that this old method
(generally) yields a string with the equivalent meaning to the input
URL.

This is in contrast to the new GetURLForDisplay method which may
perform destructive (meaning-changing) elisions.

Bug: 797354
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I8dbcffe47dd4252b6f1dd589e58583f851965d55

TBR=rohitrao@chromium.org

Change-Id: I8dbcffe47dd4252b6f1dd589e58583f851965d55
Reviewed-on: https://chromium-review.googlesource.com/876646
Commit-Queue: Tommy Li <tommycli@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531029}

The native functions invoked are sometimes not run as a task which is
registered with TaskAnnotator. Add events on generated jni functions to
be able to trace them.
Causes 80KB regression on the binary size, so use for temporary
debugging in Dev channel and revert CL.

BUG=801260

Change-Id: I80931fc07d74ed27edebe3cf35bce689f8138099
Reviewed-on: https://chromium-review.googlesource.com/853081
Commit-Queue: Siddhartha S <ssid@chromium.org>
Reviewed-by: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531028}

This patch moves message handling logic to DevToolsSession,
removes duplicated logic from several DevToolsAgentHost implementations
and hides multiple public methods exposed by the session.

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

These files appear to be disconnected and unused. I contacted the original
author (rlp@) for their thoughts. If they recall correctly this feature was
removed. It sounds like these are just orphans.

(code health)

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

Before this change, the behavior between the having the flag
(#enable-downloads-foreground) enabled or not was different. With the
flag enabled, swiping away a paused download notification cancels it and
without the flag enabled, swiping away did not cancel. This change makes
the behavior consistent in that swiping away a paused download does not
cancel the download, rather, allows the user to go through downloads
home to cancel it themselves.

Bug: 803999
Change-Id: I1bd413575629885f7caeda11c0a5eff59f6c7375
Reviewed-on: https://chromium-review.googlesource.com/877187Reviewed-by: David Trainor <dtrainor@chromium.org>
Commit-Queue: Joy Ming <jming@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531025}

Bug: None
Change-Id: I6e4717ab767dfc7b9f65e2621faf14237c65ab54
Reviewed-on: https://chromium-review.googlesource.com/879400Reviewed-by: Matthew Jones <mdjones@chromium.org>
Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531024}

Bug: None
Change-Id: I8e007ad264c7cc0f20b7e0a6a2b7e3c266d54e25
Reviewed-on: https://chromium-review.googlesource.com/879393Reviewed-by: Matthew Jones <mdjones@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531023}

Bug: 624061
Test: http/tests/security/frameNavigation/xss-ALLOWED-same-origin-top-navigation-without-user-gesture.html
Change-Id: I78909ab0726b85881225de413302886479761a8f
Reviewed-on: https://chromium-review.googlesource.com/877407
Commit-Queue: Nate Chapin <japhet@chromium.org>
Reviewed-by: Mike West <mkwst@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531022}

In QUIC, let control frame manager send and manage control frames. Protected by --quic_reloadable_flag_quic_use_control_frame_manager.

Merge internal change: 181644997

Change-Id: Ibfc98acd42c465b9e093a4b8e339821b69d73eaf
Reviewed-on: https://chromium-review.googlesource.com/878284Reviewed-by: Ryan Hamilton <rch@chromium.org>
Commit-Queue: Fan Yang <fayang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531021}

This renames net/proxy to net/proxy_resolution, and hoists net/proxy/proxy_server* to net/base.

NOPRESUBMIT=true
TBR=tbansal@chromium.org,boliu@chromium.org,stevenjb@chromium.org,gene@chromium.org,sky@chromium.org,halliwell@chromium.org,kinuko@chromium.org,mkwst@chromium.org,reillyg@chromium.org,dimich@chromium.org,skyostil@chromium.org,eugenebut@chromium.org,garykac@chromium.org,eugenis@chromium.org,rhalavati@chromium.org

Bug: 613632
Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.linux:linux_mojo;master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Id8ceccb9b75ab1fc87e184de8661527444eedf1f
Reviewed-on: https://chromium-review.googlesource.com/868176
Commit-Queue: Lily Houghton <lilyhoughton@chromium.org>
Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Evgeniy Stepanov <eugenis@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Bo <boliu@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Gene Gutnik <gene@chromium.org>
Reviewed-by: Ramin Halavati <rhalavati@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Reviewed-by: Eric Roman <eroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531020}

Before this CL, there was a bug on L and lower devices (pre-Marshmallow)
where the notification for the completed download would disappear if
Chrome was swiped away from the recent activities drawer. This is
because on pre-Marshmallow devices, swiping away an activity means
killing the process immediately after it completes, which does not allow
us to properly deal with the notification and it dies with the service.
Even though we have backup measures for downloads in other states (ie.
in-progress or paused) with the service START_STICKY, but since we do
not persist information about completed or failed downloads, we are
unable to recreate the notifications properly. Therefore, in this CL
we fix that bug by pre-emptively launching a notification with a new
notification id for all completed or failed downloads in phones that
are L or below before calling stopForeground.

For more information, see the writeup at go/downloads-on-l-devices.

Bug: 747563
Change-Id: I079ba924cfa4cb48304e5d94813225ea3eccd8b9
Reviewed-on: https://chromium-review.googlesource.com/869138
Commit-Queue: Joy Ming <jming@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Reviewed-by: Min Qin <qinmin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531019}

Bug: 804294, 804296
Change-Id: I9c854705f6481d3b3479a0251ee810800bdcfcde
Reviewed-on: https://chromium-review.googlesource.com/879351Reviewed-by: Theresa <twellington@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: Becky Zhou <huayinz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531018}

Change-Id: Ib3306e8c136b74c2cdaa9c5e9c33f995a2f4c5e6
Reviewed-on: https://chromium-review.googlesource.com/879390Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: François Doray <fdoray@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531017}

The use of XML_PARSE_RECOVER in libxml2 is discouraged in production
code as it hides errors in invalid XML and exercises some less-tested
code paths in libxml2.

Bug: 804379
Change-Id: Ic7a93d4a7e30d250beb8f018b923baad074d4f0d
Reviewed-on: https://chromium-review.googlesource.com/879106Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Joel Hockey <joelhockey@chromium.org>
Commit-Queue: Jay Civelli <jcivelli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531016}

R=abdulsyed@chromium.org

Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I86972689478940ab2110fe1cd1b869bed5eff43f
Reviewed-on: https://chromium-review.googlesource.com/879562Reviewed-by: Krishna Govind <govind@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531015}

This reverts commit 6c4efb8d.

Reason for revert:

Findit (https://goo.gl/kROfz5) identified CL at revision 531003 as the
culprit for failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtLzZjNGVmYjhkZDY2YzYxYjVlZTU1NDUzZTAyMWY0M2Q1OGE0ZDMwOTMM

Sample Failed Build: https://ci.chromium.org/buildbot/chromium.chrome/Google%20Chrome%20Win/26183

Original change's description:
> Makes metrics_reporting_state not use g_browser_process
> 
> This is a precursor to loading local state, about flags and field
> trials earlier, before BrowserProcess has been created.
> 
> BUG=800358
> TEST=none
> 
> Change-Id: I22dc0f6422e53073c06e183f6dc396b49a524633
> Reviewed-on: https://chromium-review.googlesource.com/865154
> Commit-Queue: Scott Violet <sky@chromium.org>
> Reviewed-by: Mark Pearson <mpearson@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#531003}

No-Presubmit: true
No-Tree-Checks: true
No-Try: true
BUG=800358

Change-Id: I93683145391bc9d869a71e19e945f97033fbfc9c
Reviewed-on: https://chromium-review.googlesource.com/879531
Cr-Commit-Position: refs/heads/master@{#531014}

Replace FloatLabelLayout with CompatibilityTextInputLayout
as per suggestion from
https://chromium-review.googlesource.com/c/chromium/src/+/875270#message-605ab443ccb5ae5408f35bf74e9341830ccb4e43

Change-Id: I69bf048cc02ae4b0ae962b5dd8513b4afd8df661
Reviewed-on: https://chromium-review.googlesource.com/877240Reviewed-by: Theresa <twellington@chromium.org>
Commit-Queue: Marcin Wiącek <marcin@mwiacek.com>
Cr-Commit-Position: refs/heads/master@{#531013}

Fixes the bug where the launcher would animate off screen on small
scale factors for the HALF animation.

Also enforces EXPECT_EQ(EXPECTED, ACTUAL) within
app_list_presenter_delegate_unittest.cc

Bug: 804033
Change-Id: Iac717640dc69677aa69a09b9a7a95414b4779aec
Reviewed-on: https://chromium-review.googlesource.com/879388Reviewed-by: Jenny Zhang <jennyz@chromium.org>
Commit-Queue: Alex Newcomer <newcomer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531012}