GitLab

This CL removes ARGUMENT_ACCOUNT from SyncCustomizationFragment and
AccountManagementFragment. This was a fragment argument before
https://crrev.com/1239083007, but now it's not used and can be safely
removed.

Bug: 856182
Change-Id: Ib6ce0050b6222ef39fbf25fd280218af2c4051ba
Reviewed-on: https://chromium-review.googlesource.com/1114854Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: Boris Sazonov <bsazonov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570837}

For this interface method and implementation, adds comments, renames
|type| parameter to |content_type|, and removes unnecessary .data()
following Utf8(); all per CR comments on the recently added similar
WebSourceBuffer::{Can}ChangeType() methods.

BUG=605134

Change-Id: I0cd259a1a1508edff91287eb60b49721d61e9187
Reviewed-on: https://chromium-review.googlesource.com/1115578Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Matthew Wolenetz <wolenetz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570836}

Change-Id: I7d632fe07f24f006c594da04b0648794d4293434
Reviewed-on: https://chromium-review.googlesource.com/1116283
Commit-Queue: Muyuan Li <muyuanli@chromium.org>
Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570835}

Set text color to a custom gray color and do not set the background color provided by the fallback style.

Bug: 851390
Change-Id: I185b609577652f8b23a92d0dcc76a485fb967f28
Reviewed-on: https://chromium-review.googlesource.com/1112632
Commit-Queue: Chris Lu <thegreenfrog@chromium.org>
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Reviewed-by: edchin <edchin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570834}

The flag made sense when developing initially to allow development on
linux, but as we hit windows only bits, the flag no longer makes sense.

Bug: 832938
Change-Id: I0a1da618114714bc82e0b6e20a2e31ed781a5e10
Reviewed-on: https://chromium-review.googlesource.com/1112710Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Commit-Queue: Hector Carmona <hcarmona@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570833}

https://chromium.googlesource.com/catapult.git/+log/484f9f764dc5..e82ab88bd0dd


git log 484f9f764dc5..e82ab88bd0dd --date=short --no-merges --format='%ad %ae %s'
2018-06-27 eyaich@google.com Removing benchmark_duration and benchmark_total_duration


Created with:
  gclient setdep -r src/third_party/catapult@e82ab88bd0dd

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.

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

BUG=chromium:856293
TBR=sullivan@chromium.org

Change-Id: I4e6f6bf4cff7ae3585b8975415d6a2352667d371
Reviewed-on: https://chromium-review.googlesource.com/1117140Reviewed-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@{#570832}

Bug: 855251
Change-Id: Ic751c558b5f8ecb5a8334e2c5499ad9e1bbcf80e
Reviewed-on: https://chromium-review.googlesource.com/1112759
Commit-Queue: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570831}

This will abort all ongoing fetches, and mark their data for deletion.

Bug: 841385
Change-Id: I19eba7a294dd348c0c0a5324f3b23b438b571c03

Change-Id: I19eba7a294dd348c0c0a5324f3b23b438b571c03
Reviewed-on: https://chromium-review.googlesource.com/1103559
Commit-Queue: Mugdha Lakhani <nator@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570830}

There was an issue when we handled multiple touch pointer actions.
SyntheticPointerAction should handle different actions from multiple
touch pointers.

Bug: 856638
Change-Id: I771181bcfe452631074c95874fd5c1a7807d3e74
Reviewed-on: https://chromium-review.googlesource.com/1113898
Commit-Queue: Lan Wei <lanwei@chromium.org>
Reviewed-by: Ella Ge <eirage@chromium.org>
Reviewed-by: Timothy Dresser <tdresser@chromium.org>
Reviewed-by: Navid Zolghadr <nzolghadr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570829}

The case where the first field was rationalized would crash.

If the first field is rationalized, and we want to apply the
rationalization to the fields before that, we would end up at
the biggest unsigned int (since it's a size_t), which would
cause a crash.
Solution: If the first field is rationalized, we don't need
to apply the rationalization to the fields before that obviously.
Just return.
Test added.

Bug: 856962,857011
Change-Id: I0f4471e57fa0fa8bae9b7345ce774122f25e3410
Reviewed-on: https://chromium-review.googlesource.com/1117131
Commit-Queue: Parastoo Geranmayeh <parastoog@google.com>
Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570828}

Continuation of work in bug 646094.

This CL was uploaded by git cl split.

R=dtrainor@chromium.org

Bug: 846456
Change-Id: Ied7851bf95858ac7c9580c028cb11cf2b8290374
Reviewed-on: https://chromium-review.googlesource.com/1095984Reviewed-by: Simeon Anfinrud <sanfin@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Commit-Queue: Tiger Oakes <tigero@google.com>
Cr-Commit-Position: refs/heads/master@{#570827}

Fix bad rebase. This is a reland of
https://chromium-review.googlesource.com/c/chromium/src/+/1093882.

TBR=xiyuan@chromium.org

Bug: 839583
Change-Id: I70d994b15131404f479e3d0c1e585549fecbf6af
Reviewed-on: https://chromium-review.googlesource.com/1115724
Commit-Queue: Jacob Dufault <jdufault@chromium.org>
Reviewed-by: Xiaoyin Hu <xiaoyinh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570826}

Invalidated cookies were not handled correctly. This CL adds proper
support for them.

TBR=treib

Bug: 854799
Change-Id: Iad3ee729172a7e44d737b25fb665f0009d78de84
Reviewed-on: https://chromium-review.googlesource.com/1110218
Commit-Queue: David Roger <droger@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Reviewed-by: Mihai Sardarescu <msarda@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570825}

This is one more step towards making the bluetooth service fully
async.

Bug: 76155468
Test: chromecast/device/bluetooth::cast_bluetooth_unittests
Change-Id: Icc9cc151fbf54c5686b56d40008f573fb85ea736
Reviewed-on: https://chromium-review.googlesource.com/1114221Reviewed-by: Stephen Lanham <slan@chromium.org>
Commit-Queue: Eli Ribble <eliribble@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570824}

This CL adds a dialog that will pop up when the Media Remoting feature
is about to kick in, to ask the user whether the feature should be
enabled. This CL just adds strings/layout, so the dialog is not
functional yet, and the code path to open it is missing.

Screenshot:
https://drive.google.com/open?id=12CVIusU7nNR-uALCrRThnJMynZpPqU27

Bug: 849020
Change-Id: I3c85b62df495fc699e5b600a0dbb54808a198daf
Reviewed-on: https://chromium-review.googlesource.com/1095571
Commit-Queue: Takumi Fujimoto <takumif@chromium.org>
Reviewed-by: Bret Sepulveda <bsep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570823}

End the test when the CSS transform is reflected in the dump, rather
than by using a separate label.

Bug: 856367
Change-Id: I7ba49d483705d2ad04d1bc0a10a2ce3546e2c626
Reviewed-on: https://chromium-review.googlesource.com/1114353Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: James Wallace-Lee <jamwalla@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570822}

Also removed arraysize in this file, since it has been
deprecated in favor of base::size().

Bug: 831394
Change-Id: I8b279a188d786f3ef234565de8e4647970b0f0ae
Reviewed-on: https://chromium-review.googlesource.com/1114223Reviewed-by: Ken Rockot <rockot@chromium.org>
Reviewed-by: Varun Khaneja <vakh@chromium.org>
Commit-Queue: Daniel Rubery <drubery@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570821}

Extracts RTCRtpSendParameters from RTCRtpParameters.
Extracts RTCRtpEncodingParameters from RTCRtpCodingParameters.
Adds required to many fields and update tests accordindly.

Bug: 803494
Change-Id: I416c0884abf15ab8d55ce5ae481f15ac8012c73d
Reviewed-on: https://chromium-review.googlesource.com/1114844Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Henrik Boström <hbos@chromium.org>
Commit-Queue: Florent Castelli <orphis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570820}

This change adds kpaulhamus@ and hongjunchoi@ as OWNERs for
//device/fido. Both of them have numerous contributions to this folder
and are very familiar with the codebase. Furthermore, both of them
have obtained the required committer status.

Bug: NONE
Change-Id: Idfa3343e865f862288a214e06f0b065f205b6546
Reviewed-on: https://chromium-review.googlesource.com/1116921Reviewed-by: Reilly Grant <reillyg@chromium.org>
Reviewed-by: Balazs Engedy <engedy@chromium.org>
Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570819}

Adds the helper function
BlinkAXTreeSource::TruncateAndAddStringAttribute, which truncates string
attributes when Blink's tree is serialized for accessibility. String
attributes are truncated at BlinkAXTreeSource::kMaxStringAttributeLength
bytes, currently set to 10000 bytes.

Bug: 644805
Change-Id: Ibd1af03a77814aa524dcb067ad1be3f3f4291c8b
Reviewed-on: https://chromium-review.googlesource.com/1113946Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: James Wallace-Lee <jamwalla@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570818}

https://chromium.googlesource.com/webm/libvpx.git/+log/8648a64c83b5..583859d7395c

$ git log 8648a64c8..583859d73 --date=short --no-merges --format='%ad %ae %s'
2018-06-24 marpan vp9: Fixes for lossless mode for real-time mode.
2018-06-25 marpan vp9-svc: Fix to frame dropping when layer is skipped.
2018-06-21 huisu Add a partition search breakout model
2018-06-20 jianj VP9 SVC: Add tests for layer sync on base layer.
2018-06-21 zoeliu Add extra altref option for hierarchical structure
2018-06-21 zoeliu Single out ref frame update functionality
2018-06-21 jianj Add capibility to configure decoder in encode tests.
2018-06-20 jingning Refactor block partition level rate distortion cost computation
2018-06-19 zoeliu Add bit allocation for hierarchical layer
2018-06-20 johannkoenig libyuv: remove problematic functions
2018-06-20 jingning Disable tpl model in high bd route
2018-06-18 huisu Add a partition search breakout model
2018-06-12 marpan vp9-svc: Add support for spatial layer sync frames.
2018-06-19 zoeliu Add hierarchical structure in GF group
2018-06-07 jianj vp9: Enable cyclic refresh for HBD in real-time.
2018-05-25 jingning Build temporal prediction dependency propagation
2018-05-30 jingning Refactor partition mode cost calculation
2018-06-12 huisu Improve the partition search breakout speed feature
2018-06-14 luc include msvc.h for snprintf support in benchmarks
2018-06-17 zoeliu Add update types for hierarchical refs
(...)

Created with:
  roll-dep src/third_party/libvpx/source/libvpx
R=johannkoenig@google.com

Change-Id: I311600eed599ec41c6e6007c155dc3dbddfbd02d
Reviewed-on: https://chromium-review.googlesource.com/1115799Reviewed-by: Johann Koenig <johannkoenig@google.com>
Commit-Queue: Jerome Jiang <jianj@google.com>
Cr-Commit-Position: refs/heads/master@{#570817}

Fixes a bug where safe areas are only sent to the main frame.
This should be the current fullscreen frame.

BUG=855721

Change-Id: I32fcf1190384bd9878711f7ae66e2fb4ea183df4
Reviewed-on: https://chromium-review.googlesource.com/1114144Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Commit-Queue: Becca Hughes <beccahughes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570816}

Bug: 856451
TBR: tandrii
Change-Id: I65a66a6a33e7a9751f2106a1f94a6e0c32120230
Reviewed-on: https://chromium-review.googlesource.com/1117208Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570815}

Mime handlers create a nested WebContents to contain the mime handler
extension. Currently, these run with a separate session storage
namespace, resulting in session storage being cleared if a tab recreates
a mime handler, such as due to a reload. Change mime handler WebContents
creation to inherit the session storage namespaces of their embedders so
the session storage behaves similarly to iframes within a single tab.

Bug: 819761
Change-Id: I3b4f653836190ed86572eb12cdbeef3fbc157683
Reviewed-on: https://chromium-review.googlesource.com/1116633
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570814}

Also remove the custom URLLoaderFactory in content_hash_fetcher_unittest. This isn't needed anymore as TestURLLoaderFactory now supports Clone.

Cq-Include-Trybots: luci.chromium.try:linux_mojo
Change-Id: Id70ec605886a6b0f2cb9bf7392fac4eff85fc1b3
Reviewed-on: https://chromium-review.googlesource.com/1115346Reviewed-by: Doug Turner <dougt@chromium.org>
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570813}

The call to WebViewDelegate.getDataDirectorySuffix() was missing an API
level check and only working correctly because the upstream version of
the class was a stub that returned null - this broke non-P devices after
upstreaming.

Bug: 857066
Change-Id: I43e790e3b5ab9f8e26b35fa15c753ab0782a4342
Reviewed-on: https://chromium-review.googlesource.com/1117027Reviewed-by: Tobias Sargeant <tobiasjs@chromium.org>
Commit-Queue: Richard Coles <torne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570812}

* Most templates use a hard-coded list of build_utils.py deps.
* A few use exec_script() when compute_inputs_for_analyze = true.

Bug: 843562
Change-Id: I0bb1ca7927f7366d23c23b8beb56d39a22208a06
Reviewed-on: https://chromium-review.googlesource.com/1114122
Commit-Queue: agrieve <agrieve@chromium.org>
Reviewed-by: Eric Stevenson <estevenson@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570811}

Pre-refresh, during dragging, tabs could slide anywhere in the strip (from
tab left = 0 to tab right = tabstrip width).  In refresh, tabs should not drag
over the frame grab width or a LEADING or TRAILING new tab button.

This also prevents strange visual artifacts in TRAILING mode that currently
happen because a tab can be dragged past the point where it's supposed to be
allowed to remain visible.  With the updated code, tabs can't be dragged this
far, so the artifacts can't occur.

Along the way, I moved the Finch feature for the new tab button position into
the one function that uses it, because while trying to test locally by
commenting out parts of that function, I kept getting "unusued variable"
warnings.

Bug: 852936
Test: On Mac with refresh enabled, dragging a tab to the right edge of the strip should not result in flickering/disappearing tabs.
Change-Id: Idef63702d5c3161a38e1662feec8373239334c0d
Reviewed-on: https://chromium-review.googlesource.com/1114299Reviewed-by: Allen Bauer <kylixrd@chromium.org>
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570810}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/b0de1093..56b60a2e

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;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel

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

Change-Id: Ib5be3feec7c019250980dc87c92183b40ec13934
Reviewed-on: https://chromium-review.googlesource.com/1117119Reviewed-by: V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com>
Commit-Queue: V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#570809}

This refactors the code that sends the safe areas moving
it to WebContentsImpl which will be useful for metrics later
on.

BUG=855721

Change-Id: I9d4976ec9bdeddef3b0d8a1a824b8a65f78516ba
Reviewed-on: https://chromium-review.googlesource.com/1112071Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Commit-Queue: Becca Hughes <beccahughes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570808}

In particular, this gets rid of ProfileSyncServiceStartupCrosTest, which
was not meaningfully different from the non-Cros version.
Also adds a few extra assertions and comments.

Bug: none
Change-Id: Iabba34647d084b1518a2cea239c9f25d463aa27b
Reviewed-on: https://chromium-review.googlesource.com/1117073
Commit-Queue: Marc Treib <treib@chromium.org>
Reviewed-by: Jan Krcal <jkrcal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570807}

Dismissing SSL interstitial may change SSL status, in which case
WebStateImpl should call WebStateObserver::DidChangeVisibleSecurityState

Bug: None
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: If9737cfa13829f17c2f4d9b5d965df2197f71669
Reviewed-on: https://chromium-review.googlesource.com/1114839
Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org>
Reviewed-by: Stepan Khapugin <stkhapugin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570806}

This CL adds keyboard shortcuts related to opening tabs to the tab grid.
- Command + n = open new (regular) tab
- Command + Shift + n = open new incognito tab
- Command + t = open new tab in current window

Bug: 818451
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: Iae029bf53f5a2d71c2fe569b6481ee9eeec98ef0
Reviewed-on: https://chromium-review.googlesource.com/1116257
Commit-Queue: edchin <edchin@chromium.org>
Reviewed-by: Mark Cogan <marq@chromium.org>
Reviewed-by: edchin <edchin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570805}

Currently managed Chrome OS devices support remote commands issued by the admin
on CPanel. Add a command that wipes all the users off of the device when run
(equivalent to removing all the user PODs manually from the login screen).

Most changes are glue or wiring things through, main files are:
- device_command_wipe_users_job
- user_removal_manager

More details at the design doc @ go/remove-users-command

Bug: chromium:827467
Change-Id: I77a130bf4dda0757a60a141713baae27d2b5b5a2
Reviewed-on: https://chromium-review.googlesource.com/1107803Reviewed-by: James Cook <jamescook@chromium.org>
Reviewed-by: Julian Pastarmov <pastarmovj@chromium.org>
Commit-Queue: Ivan Šandrk <isandrk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570804}

NOTRY=true
TBR=nednguyen@chromium.org
BUG=857108

Change-Id: I6618a9fd72d4b618d6228d6ab1bc58b98c96664e
Reviewed-on: https://chromium-review.googlesource.com/1117209Reviewed-by: oysteine <oysteine@chromium.org>
Commit-Queue: oysteine <oysteine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570803}

Bug: 856704
Change-Id: I48b28d389aa602718c5dae2afed98dc19322dd24
Reviewed-on: https://chromium-review.googlesource.com/1115356Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Jason Roberts <jasonroberts@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570802}

The implementation of this debug URL was changed in
https://chromium-review.googlesource.com/c/chromium/src/+/907675 to one
which _exit()s the process under POSIX, rather than self-terminating via
kill().  Since we detect process kills (versus self-termination, crashes
etc) by checking for SIGTERM, this broke chrome://kill.

Fix chrome://kill by explicitly kill()ing the process, and structure the
implementation to make clear that we need to simulate termination by a
peer process, specifically.

Bug: 806451, 854926
Change-Id: I0408dd1d248e04b3045f37d46051c220b33ab081
Reviewed-on: https://chromium-review.googlesource.com/1116076Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570801}

Bug: 852705
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I611e9625e988e077f61e71e58a7f44b741a20eab
Reviewed-on: https://chromium-review.googlesource.com/1116702Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Reviewed-by: Mark Cogan <marq@chromium.org>
Commit-Queue: Justin Cohen <justincohen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570800}

This reverts commit 0895c797.

Reason for revert: Missed a hardcoded gn location: https://crbug.com/857107.

Original change's description:
> Pull GN via CIPD package
> 
> The gn binary will be downloaded into third_party/gn. This CL rolls
> buildtools forward to a revision where gn is no longer in buildtools.
> 
> Because buildtools has no way to cleanup after itself, add
> build/util/gn_cleanup.py to delete the old binaries so that we don't
> accidentally depend on them.
> 
> Update mb.py to use the new location.
> 
> Update explicit location in ios/web_view/BUILD.gn for the new location.
> 
> Update explicit location in
> tools/traffic_annotation/auditor/traffic_annotation_auditor.cc for the new
> location.
> 
> Update explicit location in tools/licenses.py with for the new location.
> 
> Bug: 855791
> Bug: 794764
> Bug: 856883
> Bug: 856884
> Bug: 856878
> Bug: 856899
> Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
> Change-Id: I92c908faf4f868850eafa1b4adf6e7c33c365116
> Reviewed-on: https://chromium-review.googlesource.com/1112840
> Commit-Queue: Scott Graham <scottmg@chromium.org>
> Reviewed-by: Dirk Pranke <dpranke@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#570792}

TBR=dpranke@chromium.org,scottmg@chromium.org

Change-Id: I89e180710b5ce483106fa3b4ff6e8dec06780c07
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 855791, 794764, 856883, 856884, 856878, 856899
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Reviewed-on: https://chromium-review.googlesource.com/1117219Reviewed-by: Scott Graham <scottmg@chromium.org>
Commit-Queue: Scott Graham <scottmg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570799}

If the gpu process crashes, the viz service is gone and bitmaps need
to be re-registered, but they are still valid bitmaps. So we can keep
the ResourceId alive and not returned it as lost to the client in that
case.

R=kylechar@chromium.org

Bug: 826886
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I1ef6af3a90bf90310793fcdbbb70ea00a0d85d25
Reviewed-on: https://chromium-review.googlesource.com/1115591Reviewed-by: kylechar <kylechar@chromium.org>
Commit-Queue: danakj <danakj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570798}