GitLab

This needs to be updated due to bt709 interpretation change in
crbug.com/763260

Bug: 768608
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: I9321869000d10f79f82f9dd4ca778e6517ed7ef7
Reviewed-on: https://chromium-review.googlesource.com/683078Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: ccameron chromium <ccameron@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504264}

This generalizes EAP support for Ethernet and WiMAX.

Bug: 380937
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I39e5c92f648e35822a85e95d8cbd3c128d649258
Reviewed-on: https://chromium-review.googlesource.com/677131
Commit-Queue: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Toni Barzic <tbarzic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504263}

Bug: 698737
Change-Id: I954b7c6e2aa1b56a742a4a2787093fa413b5f47c
Reviewed-on: https://chromium-review.googlesource.com/676202Reviewed-by: Becca Hughes <beccahughes@chromium.org>
Reviewed-by: Steve Kobes <skobes@chromium.org>
Commit-Queue: Tommy Steimel <steimel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504262}

This will ensure that the notification is only shown if there actually
is fresh content in the system.  If not, we will not show the
notification and stop scheduling the background task until there is
fresh content.

Bug: 701939
Change-Id: Ief7f659254e80faf08e94e752eb34d0c5bb8e6ab
Reviewed-on: https://chromium-review.googlesource.com/682769
Commit-Queue: Justin DeWitt <dewittj@chromium.org>
Reviewed-by: Carlos Knippschild <carlosk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504261}

Only event_factory.cc and event_modules_factory.cc need to depend on
each of event_*headers.h. So we don't need to generate header files,
and can fold them into *.cc.

Change-Id: I38b1c4a646045d66f8c4fbf49f12de93724c9ec0
Reviewed-on: https://chromium-review.googlesource.com/681094Reviewed-by: Kenichi Ishibashi <bashi@chromium.org>
Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504260}

- Add support for elements inside android notification via
  chrome.automation.

      ChromeVox.

Bug: 763318
Test: Confirm that user can navigate elements on android notification by
Change-Id: Iee057986596202f2e95c3db6b313cc21939e98bc
Reviewed-on: https://chromium-review.googlesource.com/656751
Commit-Queue: Yuki Awano <yawano@chromium.org>
Reviewed-by: Mattias Nissler <mnissler@chromium.org>
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Reviewed-by: David Tseng <dtseng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504259}

Per spec protocol should forward the response status code as an int instead
of numbers.

See: https://github.com/ChromeDevTools/devtools-protocol/issues/58

R=pfeldman
BUG=767274

Change-Id: I450278a2889352d077373e7c1d2ea0d80367b7cd
Reviewed-on: https://chromium-review.googlesource.com/676504
Commit-Queue: Blaise Bruer <allada@chromium.org>
Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504258}

The rename will help to call out that multiple build targets exist on
Android.

Bug: 731186
Change-Id: I1dbe2ffe932b25ab8739f53c2cd05c2ee3d8b2b6
Reviewed-on: https://chromium-review.googlesource.com/682770
Commit-Queue: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: Tommy Nyquist <nyquist@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504257}

Switches to new bots for the linux perf builder.

Bug: 732463
Change-Id: Ied8d923448f40b8295ddfb2fde1adde481666075
Reviewed-on: https://chromium-review.googlesource.com/676725
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#504256}

Bug: 756607
Change-Id: I78e1fdf2411c067efd5423987ab9812f14b93736
Reviewed-on: https://chromium-review.googlesource.com/680041Reviewed-by: Scott Graham <scottmg@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504255}

ServiceWorkerScriptURLLoader should check the MIME type of fetched scripts and
abort the subsequent sequence if it's not valid. This CL adds the check and unit
tests.

Bug: 748415
Change-Id: I59a3013377f216f249ac0f3155fe9146b13a66e5
Reviewed-on: https://chromium-review.googlesource.com/672177Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504254}

After https://crrev.com/c/676963, in rare case, invalid ripple effect is
shown when the header is clicked. http://screen/29NRt5CTEEm

This CL fixes this by always returning stub InkDrop object.

TEST=manual
BUG=767246

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

Fixed some remaining issues after removing prerenderer related code in
test harness.

Bug: NONE
Change-Id: I301f1c9309b30bd031e75e019ed2c08865d7f877
Reviewed-on: https://chromium-review.googlesource.com/683310Reviewed-by: Peter Williamson <petewil@chromium.org>
Commit-Queue: Yafei Duan <romax@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504252}

This extends the LevelDBWrapperImpl API to enable users of that class to
fixup broken data in the database. That API is then used to fix
incorrectly encoded keys, which should fix all weirdness around
removeItem that was being observed.

BUG=765524

Change-Id: I4049445d46d30925774c2fec9dfd4dcdaba12e04
Reviewed-on: https://chromium-review.googlesource.com/678250Reviewed-by: Mark Pearson <mpearson@chromium.org>
Reviewed-by: Michael Nordman <michaeln@chromium.org>
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504251}

This also includes a WebGL conformance roll.

BUG=765953
TEST=webgl conformance
R=kbr@chromium.org

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: I166bada31aefac271e145a545f82975bc3a0dfed
Reviewed-on: https://chromium-review.googlesource.com/671843
Commit-Queue: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504250}

When catching up the animation, if the StartAnimation call for
advancing the next frame was delayed, if n is the desired current frame,
we advance uptill frame n-1 and a post a task to further advance to
frame n. This task also calls StartAnimation to schedule a task for
the next frame of the animation. In this call, kDoNotCatchUp is used to
avoid going through a catch up loop and schedule this task immediately
if necessary.

Clean up the logic to advance to frame n during catch up and schedule
the next task synchronously. And add a test.

R=chrishtr@chromium.org

Change-Id: I5fdb1d978001b0bf5f04cdaf4dada110db065801
Reviewed-on: https://chromium-review.googlesource.com/674056
Commit-Queue: Khushal <khushalsagar@chromium.org>
Reviewed-by: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504249}

Adds a new UMA metric to track whether or not translation conditions are
met for each search term.

Bug: 
Change-Id: Ib240408c8df1862f55f592f5413599a730209fd1
Reviewed-on: https://chromium-review.googlesource.com/679447
Commit-Queue: Mehran Mahmoudi <mahmoudi@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Reviewed-by: Donn Denman <donnd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504248}

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/f7cc2170e1ab..852ba7672ce0

$ git log f7cc2170e..852ba7672 --date=short --no-merges --format='%ad %ae %s'
2017-09-25 dtu [pinpoint] Docstrings for models/change/repository.py.
2017-09-25 dtu [pinpoint] Don't include list of differences in bug post on error.
2017-09-25 dtu [pinpoint] Move mann_whitney_u.py to models/
2017-09-25 dtu [pinpoint] Add favicon.

Created with:
  roll-dep src/third_party/catapult


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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=sullivan@chromium.org

Change-Id: I86b73b1dd43f4de091f258bced6f974171b276a6
Reviewed-on: https://chromium-review.googlesource.com/683232
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504247}

This change introduces a new service (SharedWorkerFactory) exported
from renderer processes to support the instantiation of shared worker
instances.

The SharedWorkerFactory is instantiated from within the
SharedWorkerServiceImpl::TryReserve function, which runs on the UI
thread. This way we defer connecting to that service until we want
to create a shared worker. It also allows the unit tests to swap in
a mock implementation of SharedWorkerFactory.

In a future CL, it'll be possible to remove the
SharedWorkerMessageFilter. Even though that class now no longer
filters any Chrome IPC messages, its other functionality is still
needed. Similarly, we still need to allocate a routing ID for the
shared worker even though that is not needed by the shared worker
infrastructure. Dev Tools uses the routing ID to help lookup the
shared worker, so that'll need to change before we clean this up.

Also, in a future CL, it'll be possible to move the shared worker
service code to run entirely on the UI thread. This will allow a lot
of code simplification as we'll no longer have to deal w/ the race
conditions during process reservation. A bunch of complexity will
disappear. To try to keep this CL manageable, I chose to postpone
the threading changes to a follow-up CL.

The lifetime of the SharedWorker instance in the renderer process is
a bit squirrely thanks to how WebSharedWorker manages its lifetime.
This CL does not attempt any changes there.

Bug: 612308
Change-Id: Ic59dc1074b479e018d515d7ec715a38c4dd22791
Reviewed-on: https://chromium-review.googlesource.com/659139Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Kinuko Yasuda (slow) <kinuko@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Commit-Queue: Darin Fisher <darin@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#502910}
Reviewed-on: https://chromium-review.googlesource.com/680254Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504246}

After this we can get rid of a lot of the code to keep track of
reverse relations in AXPlatformNodeWin and just get it from
AXTree instead.

Bug: 757707
Change-Id: I79fd5c276b50d4bf1409f8e20af7efe8abf928ca
Reviewed-on: https://chromium-review.googlesource.com/676098Reviewed-by: Doug Turner <dougt@chromium.org>
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504245}

Using this new enum code the correct error will be displayed to the user
when then try to manually add a PPD which is too large. Before the
reported error was that the PPD was unable to be retrieved by the
server.

R=skau@chromium.org

Bug: 766767,765481
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I281d92f89c7baff5ce5530924d06c76ea1480791
Reviewed-on: https://chromium-review.googlesource.com/673551
Commit-Queue: David Valleau <valleau@chromium.org>
Reviewed-by: Sean Kau <skau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504244}

Using wpt-import in Chromium 548147b1.
With Chromium commits locally applied on WPT:
4d3c601c "Refactor wpt/2dcontext/drawing-images-to-the-canvas/ layout tests:"
f493473b "[Media Router] Update Cast Presentation URL logic - tests only"


Build: https://build.chromium.org/p/chromium.infra.cron/builders/wpt-importer/builds/2430

Note to sheriffs: This CL imports external tests and adds
expectations for those tests; if this CL is large and causes
a few new failures, please fix the failures by adding new
lines to TestExpectations rather than reverting. See:
https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md

Directory owners for changes in this CL:
tyoshino@chromium.org, yhirano@chromium.org, mkwst@chromium.org, japhet@chromium.org:
  external/wpt/fetch

TBR=qyearsley@chromium.org

No-Export: true
Change-Id: I1af298a681aa9c96c7cadff6f843e1da6b740013
Reviewed-on: https://chromium-review.googlesource.com/682937
Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Reviewed-by: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504243}

Change-Id: Ia28a6a8417720b4d20ef1633adf19ef1b80bfad9
Reviewed-on: https://chromium-review.googlesource.com/651344Reviewed-by: Brett Wilson <brettw@chromium.org>
Commit-Queue: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504242}

This is a temporary CL to try to locate the reason why some Android
installs are collecting a large number of metrics files without
uploading them.

Bug: 760317
Change-Id: I174f502d2a3a9b59c95d62e85565da81c688d7ec
Reviewed-on: https://chromium-review.googlesource.com/682836Reviewed-by: Ilya Sherman <isherman@chromium.org>
Commit-Queue: Brian White <bcwhite@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504241}

Add select to speak and switch access to services that enable all ARc++ apps for native accessibility.

Test: enable Select to Speak. Run with any app and verify it works.


Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I3b19ed8c28f252254def2462679103c559a93696
Reviewed-on: https://chromium-review.googlesource.com/675837Reviewed-by: Hidehiko Abe <hidehiko@chromium.org>
Reviewed-by: Yuki Awano <yawano@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: David Tseng <dtseng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504240}

The underlying issue has been fixed.

Bug: 762745
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: I9899e66b3ed149aeec086ee0ce41ad4f07f1cadc
Reviewed-on: https://chromium-review.googlesource.com/683244Reviewed-by: Yuly Novikov <ynovikov@chromium.org>
Commit-Queue: Antoine Labour <piman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504239}

This reverts commit d94db169.

Reason for revert: Seeing lots of layout failures. Suspecting this CL.
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linux%20Trusty%20Leak

Original change's description:
> Media Controls: Replace painter with CSS.
> 
> Replace MediaControlPainter with CSS. Add some classes to some
> elements to allow styling by state. This is a resubmit of the
> button code only from crrev.com/c/584771.
> 
> BUG=746872
> 
> TBR=pfeldman@chromium.org
> 
> Change-Id: If79ce339a10f6671ddd02a223468bb3d7bbcd1df
> Reviewed-on: https://chromium-review.googlesource.com/657024
> Commit-Queue: Becca Hughes <beccahughes@chromium.org>
> Reviewed-by: Becca Hughes <beccahughes@chromium.org>
> Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#504095}

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

Change-Id: I698bec05d24045896e664e287415cba5f19d9f60
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 746872
Reviewed-on: https://chromium-review.googlesource.com/683434Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Commit-Queue: Hector Carmona <hcarmona@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504238}

Implemented ClearStorageTask, which is responsible for clearing expired
temporary pages from metadata store, and their associated archive files.
This is replacing the functionality of the OfflinePageStorageManager.

Bug: 753595
Change-Id: I037923a1293950998446e9992e247155063d0b5a
Reviewed-on: https://chromium-review.googlesource.com/674057
Commit-Queue: Yafei Duan <romax@chromium.org>
Reviewed-by: Filip Gorski <fgorski@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504237}

The requests do file system access, which shouldn't run on the thread
Catalog runs on (IO thread).

BUG=763529
TEST=none

Change-Id: Ie5324370fde6ed8ad8206de40190b4a1c3bf5676
Reviewed-on: https://chromium-review.googlesource.com/682648
Commit-Queue: Scott Violet <sky@chromium.org>
Reviewed-by: Ken Rockot <rockot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504236}

At some point, some of the tests under fast/canvas were re-written to
use testharness.js. However, some of the tests are not correctly written.
For example, assert_throws(null, ...) will generate an -expected.txt
file saying: Test bug: need to pass exception to assert_throws()

This CL corrects those test by checking the right exception type in
the assert_throws. This change will get rid of those expected.txt files.

Bug: 725651
Change-Id: If72032e4f89fd2ae9432900456cb610abe7664b9
Reviewed-on: https://chromium-review.googlesource.com/677648Reviewed-by: Mohammad Reza Zakerinasab <zakerinasab@chromium.org>
Reviewed-by: Fernando Serboncini <fserb@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504235}

Change-Id: I559c15c3c19b6d72032b4949c9a5489ad4065bbd
Reviewed-on: https://chromium-review.googlesource.com/682766
Commit-Queue: Xi Cheng <chengx@chromium.org>
Reviewed-by: Charlie Reis (catching up) <creis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504234}

BUG=768020
R=thestig@chromium.org

Change-Id: I8ec5c4b4b4be5910f3571d7770e910f5e74ae0f4
Reviewed-on: https://chromium-review.googlesource.com/683155
Commit-Queue: Thomas Anderson <thomasanderson@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504233}

The Request class in deterministic_dispatcher.h was made moveable to support
its inclusion in a circular_deque. There was some minor cleanup of this
class.

Typedefs around changed code were converted to using statements.

Some unnecessary <deque> includes were removed. An unnecessary <list>
include is removed from remoting/protocol/message_decoder.h, and a
<list> include was addedto remoting/protocol/message_decoder_unittest.cc
which was depending on this.

TBR=stevenjb (chromeos/network owners)

Bug: 757232
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;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: I654daa3f575b21fbfd6db225d0c321bc5781f96f
Reviewed-on: https://chromium-review.googlesource.com/676201
Commit-Queue: Brett Wilson <brettw@chromium.org>
Reviewed-by: enne <enne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504232}

This also includes some layout fixes.

Bug: 380937
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I9cffb4e7e1f1a736ea2a65a74c1be5923245b4c4
Reviewed-on: https://chromium-review.googlesource.com/677486
Commit-Queue: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Toni Barzic <tbarzic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504231}

This CL makes it so that hiding or showing the URL bar continually fires
visualViewport resize events. Previously, the resize event would be
fired only once the user lifts their finger.

Bug: 762518
Change-Id: I09470b980d09b16d5cd7b60e98e7ec3653fdaa40
Reviewed-on: https://chromium-review.googlesource.com/666600
Commit-Queue: David Bokan <bokan@chromium.org>
Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504230}

It now saves them in its own pref file, rather than using a shared one.
Since these aren't meant to be directly modified by the embedder, but
rather just used as a key-value store, there's minimal benefit from
sharing a pref store with Chrome. This approach is a little simpler,
and also lets us flush to it on clean shutdown (not currently
implemented).

Also moves ClearNetworkingHistorySince over to NetworkContext, as it
relies on the HttpServerPropertiesManager API (rather than just the
HttpServerPropertiesAPI).

Bug: 715695
Change-Id: Icca19030ac95d9c9d5bc7b78b24357458ec50adc
Reviewed-on: https://chromium-review.googlesource.com/679396
Commit-Queue: Matt Menke <mmenke@chromium.org>
Reviewed-by: Dominic Battré <battre@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: Chris Palmer <palmer@chromium.org>
Reviewed-by: Ryan Hamilton <rch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504229}

Bug: 649000
Change-Id: Iad772c39ccd86c9bcc8bf9c9f87b2f1ef12bde57
Reviewed-on: https://chromium-review.googlesource.com/682757
Commit-Queue: Eric Roman <eroman@chromium.org>
Reviewed-by: Steven Valdez <svaldez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504228}

Bug: 646711
Change-Id: I407f425a8acb2eb55108403b83c93000e9edceea
Reviewed-on: https://chromium-review.googlesource.com/679598Reviewed-by: Jesse Doherty <jwd@chromium.org>
Commit-Queue: Philippe Hamel <hamelphi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504227}

* Place app list widget in the ash app list window container, instead of
  in the container for browser windows.
* Perform widget bounds computations in local coordinates. This avoids
  problems with screen-to-local computations under mash where the native
  window's parent is the desktop widget, not the window container as in
  classic ash.
* Use side shelf mode in mash, which skips some problematic opening
  animations. See issue 768437.
* Introduce AppListView::InitParams so we don't have 2 ints and 2 bools
  as adjacent parameters.
* Eliminate some unused Init() method params.

Bug: 763961, 768437
Test: app_list_unittests, manual testing of fullscreen app list with multiple monitors (stacked vertically) and/or side shelf mode.

Change-Id: I933f5130feab3d8b4edfc2b573f52748e9c82539
Reviewed-on: https://chromium-review.googlesource.com/682343
Commit-Queue: James Cook <jamescook@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504226}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/89e89852..67e0f781

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

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