GitLab

GAIA ID is now provided as part of PolicyData for user policies.
It could be exposed together with user email if it's OK from privacy
perspective.
It will be useful because username field doesn't change on user rename
and GAIA ID is now used for PolicyData user identification.

BUG=831655
TEST=Open chrome://policy

Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ia4cfea54c80a2a5cfa7f9e8803f7ca50f9596c70
Reviewed-on: https://chromium-review.googlesource.com/1016903Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Drew Wilson <atwilson@chromium.org>
Commit-Queue: Sergey Poromov <poromov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551706}

//ios/web_view needs this to manage a list of credit cards.

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

Current implementation does not match spec.

Bug: 834081
Change-Id: Ida585a10e38d002e1fc9333f1e16eeeb9dfc93d6
Reviewed-on: https://chromium-review.googlesource.com/1016000Reviewed-by: Theresa <twellington@chromium.org>
Commit-Queue: Doug Arnett <dougarnett@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551704}

This acts as a "global" context for resource coordination signals that aren't appropriately aimed at any individual coordination unit. This will be required as part of the CPU and memory usage plumbing and aggregation.

BUG=829575

Change-Id: I5424351924a77c378c49c2d7aca892464a2d5064
Reviewed-on: https://chromium-review.googlesource.com/999017Reviewed-by: Sigurður Ásgeirsson <siggi@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Fadi Meawad <fmeawad@chromium.org>
Commit-Queue: Chris Hamilton <chrisha@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551703}

These themes are Aura themes, but do not depend on USE_AURA or any Aura code,
so it is fine to expose them on other platforms. This allows the Mac "legacy"
omnibox to use the same logic around NativeThemeAuraDark as other platforms do.

Bug: 826754
Change-Id: I69a6ffd5ebbb52cd2765edc99ec4b8c567e71e41
Reviewed-on: https://chromium-review.googlesource.com/1014738
Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551702}

Bug: b/76228145
Test: manual, unit test
Change-Id: I5661da5cd92f37661a030bbc2d3b1cac2b8bdd39
Reviewed-on: https://chromium-review.googlesource.com/1016692Reviewed-by: Alex Sakhartchouk <alexst@chromium.org>
Reviewed-by: Michael Spang <spang@chromium.org>
Commit-Queue: Ryan Daum <rdaum@chromium.org>
Commit-Queue: Alex Sakhartchouk <alexst@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551701}

These can just call base and net directly, leaving only the sandbox-specific
WebFileUtilities::GetFileInfo.

Change-Id: Iebfbede46212575ae27ab21f38ff3d5adcc58227
Reviewed-on: https://chromium-review.googlesource.com/1016135Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Jeremy Roman <jbroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551700}

Bug: 833416
Change-Id: I9cbe9fdf66a4e8e75c10043b62fe18a8dc0039af
Reviewed-on: https://chromium-review.googlesource.com/1014502
Commit-Queue: Ganggui Tang <gogerald@chromium.org>
Reviewed-by: Mathieu Perreault <mathp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551699}

This reverts commit 23e0b326.

Reason for revert: Suspected cause of net_unittests failures on Mac-10.10

Original change's description:
> Add cookie security metrics
> 
> Log the degree of protection against cookie theft, split by 1st/3rd
> party.
> 
> Includes a bit of TransportSecurityState refactoring (no functional
> change) to provide easy access to HSTS lifetime.
> 
> Bug: 749059
> Change-Id: Ib9ab7cff6a7fb4c65e84d7085e70ef13d5c00ba9
> Reviewed-on: https://chromium-review.googlesource.com/978142
> Reviewed-by: Nick Harper <nharper@chromium.org>
> Reviewed-by: Ilya Sherman <isherman@chromium.org>
> Reviewed-by: Mike West <mkwst@chromium.org>
> Commit-Queue: Thiemo Nagel <tnagel@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#551644}

TBR=isherman@chromium.org,tnagel@chromium.org,nharper@chromium.org,mkwst@chromium.org

Change-Id: I9e7411956eb5d748b547edcf6aec7f40dde88d8d
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 749059
Reviewed-on: https://chromium-review.googlesource.com/1017141Reviewed-by: Mohsen Izadi <mohsen@chromium.org>
Commit-Queue: Mohsen Izadi <mohsen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551698}

This should catch future regressions as we make changes to EventHandlers
and EventRewriters for Accessibility.

Change-Id: Ifd3b2e4554b9a0f8673c0bb0ebb5ddbd994501d5
Reviewed-on: https://chromium-review.googlesource.com/1015840Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: Katie Dektar <katie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551697}

When the activity is recreated, e.g. due to a change in font size,
the ClearBrowsingDataFetcher instance that is shared between
ClearBrowsingDataTabs and ClearBrowsingDataPreferencesBasic/-Advanced
is not wired up correctly again. As the fetcher just acts as a cache for
important sites and history data, we can store it in savedInstanceState
when getting suspended.

Bug: 834169
Change-Id: I49f3896ec669477736ca39e07fce5126f83e1f49
Reviewed-on: https://chromium-review.googlesource.com/1016621Reviewed-by: Theresa <twellington@chromium.org>
Commit-Queue: Christian Dullweber <dullweber@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551696}

This CL updates the asset for the "Add bookmark" action and adds a new
asset for the "Edit bookmark" action.

Bug: 829718
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I57afdc8fc963ff6620842c4ee3c105b2b8c97fb4
Reviewed-on: https://chromium-review.googlesource.com/1016604
Commit-Queue: Gauthier Ambard <gambard@chromium.org>
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551695}

Checking a window name is OK for now because
NotificationSurfaces are not used outside ARC++.
We will set an application id for a notification surface and use it in a
follow up CL.

Bug: 829383, 833259
Test: Manual. Inline reply on ARC notifications works.
Change-Id: Icf3d43df527988f92ca88f7b89b673fdd02b00bc
Reviewed-on: https://chromium-review.googlesource.com/1015440Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Reviewed-by: David Reveman <reveman@chromium.org>
Reviewed-by: Yusuke Sato <yusukes@chromium.org>
Commit-Queue: Yuichiro Hanada <yhanada@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551694}

This reverts commit f1291fb3.

Reason for revert: broke test bot: crbug.com/834307

Original change's description:
> [Nav Experiment] Propagate virtual URL in ReloadWithUserAgentType.
> 
> This hides the internal URL used by WKBasedNavigationManager to force
> create an entry in WKBackForwardList when reloading with a different
> user agent type (see crrev.com/c/919175).
> 
> Also fixed a bug in NavigationManagerTest which wasn't testing the
> redirect case when GetParam() == TEST_WK_BASED_NAVIGATION_MANAGER.
> 
> Bug: 833958
> Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
> Change-Id: I095178302bfc4d209c2d058aab736777cc8b2de4
> Reviewed-on: https://chromium-review.googlesource.com/1016002
> Reviewed-by: Eugene But <eugenebut@chromium.org>
> Commit-Queue: Danyao Wang <danyao@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#551548}

TBR=eugenebut@chromium.org,danyao@chromium.org

Change-Id: I82169134c6efc6e1574840eeaae311497d91b02f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 833958
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Reviewed-on: https://chromium-review.googlesource.com/1017140Reviewed-by: Danyao Wang <danyao@chromium.org>
Commit-Queue: Danyao Wang <danyao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551693}

If the title in the user consent view doesn't fit on line, it should be
wrapped.

Screenshot:
https://drive.google.com/open?id=1ucGkSOu0_YjoFaZMO_U2LNAqhrvUHyhE

Related to:
crrev.com/c/1009906

Bug: 827072
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Id8ddc57e8f311146fadb4558427d082caa37bfac
Reviewed-on: https://chromium-review.googlesource.com/1016607Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Commit-Queue: Jérôme Lebel <jlebel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551692}

This is *not* a complete implementation, but enough to pass some of
the existing tests (copied from existing window service).

BUG=none
TEST=covered by tests
TBR=tsepez@chromium.org

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

NOPRESUBMIT=true

Change-Id: I622971eb17a86b299aeb9c088e3339e1112ea755
Reviewed-on: https://chromium-review.googlesource.com/1015600
Commit-Queue: Ryan Hamilton <rch@chromium.org>
Reviewed-by: Bence Béky <bnc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551690}

The operator is already used in production, even though it was
originally intended for tests only. This CL makes the operator available
for production code, and fixes it so that it compares all the fields.

Bug: None
Change-Id: I87185529cd61dadf15408b65d09288f967a8fc1b
Reviewed-on: https://chromium-review.googlesource.com/1016644Reviewed-by: Mihai Sardarescu <msarda@chromium.org>
Commit-Queue: David Roger <droger@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551689}

Bug: 827604
Change-Id: I67b0478de52e7f5bf4bd659afc5b305407072ee3
Reviewed-on: https://chromium-review.googlesource.com/1015514Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Leonard Grey <lgrey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551688}

This CL force a layout of the toolbar when the loading state change, to
prevent the button changing from "stop" to "reload" with an animation
when there is another animation happening at the same time.

Bug: 831763
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I1bb5265c308067626ea00e154f8f18fd791701e5
Reviewed-on: https://chromium-review.googlesource.com/1016761Reviewed-by: Justin Cohen <justincohen@chromium.org>
Commit-Queue: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551687}

This change changes BluetoothAdapter::DiscoverySessionErrorCallback from
base::Callback to base::OnceCallback. Furthermore, it updates relevant
methods to pass it by value and to std::move instances of the callback.

Bug: 821766, 834319

Change-Id: Ibac2304c7540f2e9d80069ff19abb36ccbbd5527
Reviewed-on: https://chromium-review.googlesource.com/1013944
Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org>
Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551686}

LocationBarLayout#getDisplayText used to exit early if we were viewing
an NTP and avoid adding the LRM character on Android 4.2 and below.
Since it no longer did, the location bar string wasn't technically
empty but contained just the LRM character, causing the cursor and hint
to disappear.

This just adds a check to make sure we're not adding the LRM character
to an empty string.

Bug: 833437
Change-Id: Ia24b5f18d496ad77e9b4b437f977734c56dd519e
Reviewed-on: https://chromium-review.googlesource.com/1015964Reviewed-by: Maria Khomenko <mariakhomenko@chromium.org>
Commit-Queue: Troy Hildebrandt <thildebr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551685}

It's possible to URLLoader to have a posted ReadMore task when it
learns its producer data pipe has been closed. If this happens, when
ReadMore is executed, it would DCHECK, since reading from a pipe a
consumer knows is closed it a possible symptom of a bug.

This CL fixes that behavior, by deleting the URLLoader when either the
data pipe or the client pipe is closed, or when the request is
complete, rather than waiting for both consumer pipes to be closed.

Bug: 832600
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I0334773c8e701eb7dd106fa1c0fe0890e80861e7
Reviewed-on: https://chromium-review.googlesource.com/1012455Reviewed-by: Andrey Kosyakov <caseq@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Matt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551684}

New file name is wpt_update_expectations.py for consistency.

Bug: 829697
Change-Id: I7a2a01c58ac277729af201819bb60a7004aa6141
Reviewed-on: https://chromium-review.googlesource.com/1014920
Commit-Queue: Kent Tamura <tkent@chromium.org>
Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551683}

As described in the design doc ([1], accessible for Google employees
only), the proposed bridge integrates all relevant classes for syncing
sessions and exposes a USS API for a processor to use. This includes
propagating local changes to the server, as well as receiving remote
updates for foreign sessions.

The class is analogous to SessionsSyncManager and implements similar
logic, although many bits are handled in SessionStore (preceding
patch) to make the bridge simpler.

Some functionality is still pending implementation, including the
actual wiring to use the bridge instead of SessionsSyncManager.

[1] https://docs.google.com/document/d/1UGM1yhHznmXvfeGvckdzBcrE6GLpgk7LsLmgPhFcIdk/edit#

Bug: 681921
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I40ccfad85061076079ab3159f09fcea4a4e6c096
Reviewed-on: https://chromium-review.googlesource.com/911091
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551682}

All uses have been replaced with the well-maintained base::Timer.

TBR=kinuko@chromium.org

Bug: 760326
Change-Id: I1542f0f03de11961e67ef4447966dd145327f886
Reviewed-on: https://chromium-review.googlesource.com/1012741
Commit-Queue: François Doray <fdoray@chromium.org>
Reviewed-by: Taiju Tsuiki <tzik@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551681}

Previously, HostNameHandler::OnDeviceHostnamePropertyChanged() could
synchronously wait for machine statistics to be loaded by a
TaskPriority::BACKGROUND task while being called on the main thread:

chromeos::system::StatisticsProviderImpl::WaitForStatisticsLoaded()
chromeos::system::StatisticsProviderImpl::GetMachineStatistic()
chromeos::system::StatisticsProvider::GetEnterpriseMachineID()
policy::HostnameHandler::OnDeviceHostnamePropertyChanged()
chromeos::NetworkStateHandler::NotifyDefaultNetworkChanged()
chromeos::internal::ShillPropertyHandler::GetIPConfigCallback()

This was problematic since a TaskPriority::BACKGROUND task can take
"an arbitrarily long time to complete", and the main thread is
supposed to be responsibe at all times.

This CL fixes the issue by continuing the work asynchronously when
machine statistics are loaded rather than waiting synchronously.

Bug: 831835
Change-Id: Idde6fb85f584b0110e673025f83fe53b86b657ea
Reviewed-on: https://chromium-review.googlesource.com/1012182
Commit-Queue: François Doray <fdoray@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551680}

https://webrtc.googlesource.com/src.git/+log/4736d4e52456..31ef5f0d1bea

$ git log 4736d4e52..31ef5f0d1 --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
TBR=webrtc-chromium-sheriffs-robots@google.com

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

We can't make any assumptions about the contents of |reportingInfo|
in our net-internals page.  For one, the JSON content might be fully
arbitrary if the user decides to edit a net-log file.  And even if
the content actually comes from the //net stack, if the Reporting
stack is disabled at compile time, the |reportingInfo| delivered to
the net-internals page will not contain |reports| or |clients|
elements.

NOTRY=true

Bug: 829353,833628
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I69f4ab43416924b71569bfe80a121521eada716e
Reviewed-on: https://chromium-review.googlesource.com/1015549
Commit-Queue: Douglas Creager <dcreager@google.com>
Reviewed-by: Eric Roman <eroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551678}

Bug: 806071
Change-Id: Ideeda5b542760559575bad211c0950a7a9122078
Reviewed-on: https://chromium-review.googlesource.com/1014254Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551677}

This CL adjusts the width constraints of the popup menus, allowing the
menu to grow bigger on phone having bigger screens, but make sure there
is enough space to dismiss the menu on smaller screens.

Bug: 828433
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I38fd6f7a1842ef50aa5df2ee50fbfb6b54fd9d86
Reviewed-on: https://chromium-review.googlesource.com/1013712Reviewed-by: Olivier Robin <olivierrobin@chromium.org>
Commit-Queue: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551676}

This CL lands server change 192811627 by dahollings.

BUG=488484

Change-Id: I88f23288b91d51c4736e89f1c6ab1fb7a837e49f
Reviewed-on: https://chromium-review.googlesource.com/1016701
Commit-Queue: Bence Béky <bnc@chromium.org>
Commit-Queue: Dianna Hu <diannahu@chromium.org>
Reviewed-by: Dianna Hu <diannahu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551675}

Using wpt-import in Chromium 194ca2e6.
With Chromium commits locally applied on WPT:
303956b6 "Enable WPT tests for the Generic Sensor classes"
afb472c0 "Try to fix crash when terminating a worker while it is XHR-ing to a blob."


Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/15754

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:
dgrogan@chromium.org:
  external/wpt/css/css-tables
mek@chromium.org:
  external/wpt/webmessaging
yhirano@chromium.org, mkwst@chromium.org, japhet@chromium.org:
  external/wpt/fetch

TBR=robertma

No-Export: true
Change-Id: Ic67ac97617532e281b0103c43d3489e053ddb51c
Reviewed-on: https://chromium-review.googlesource.com/1016840
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@{#551674}

SigninManager is an old API that's being deprecated.
Instead of querying the authenticated account from there, just get it
from ProfileSyncService directly.

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

https://chromium.googlesource.com/angle/angle.git/+log/c90d4d38c895..eb79e337c9f7

$ git log c90d4d38c..eb79e337c --date=short --no-merges --format='%ad %ae %s'
2018-04-18 jmadill Vulkan: Do not build mock ICD on Android.

Created with:
  roll-dep src/third_party/angle


The AutoRoll server is located here: https://angle-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=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
TBR=jmadill@chromium.org

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

Bug: 830936
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;master.tryserver.chromium.linux:linux_vr
Change-Id: I44792a3b2d9b4bfa971c28407f0dd0c1831f5ca6
Reviewed-on: https://chromium-review.googlesource.com/1015886Reviewed-by: Ian Vollick <vollick@chromium.org>
Commit-Queue: Yash Malik <ymalik@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551671}

v8_context_snapshot is not necessary when compiling.
This allows to compile chrome/renderer and chrome/child before snapshot is made.

Change-Id: I41fee7c51982dcb34bc7b66d31e6d701d6a24aeb
Reviewed-on: https://chromium-review.googlesource.com/1013886Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551670}

This CL:
* Hooks up heap_profiling::Supervisor to AwBrowserMainParts. This removes boiler
  plate logic and adds support for heap profiling of non-browser processes.
* Adds HeapProfilingTest. This end to end test reuses shared logic with
  desktop/Android Chrome, see heap_profiling::TestDriver for test details.
* Adds the decorator SkipSingleProcessTests. Applies the decorator to
  HeapProfilingTest. By default, Android Webview launches all instrumentation
  tests twice - once in single process and once in multi process mode. In single
  process mode, the test uses the flag --single-process, which is incompatible
  with the current implementation of the heap profiling service.

Bug: 827545
Change-Id: I7fb67585cd132e8326c55c1b0a6e6987c6b3905d
Reviewed-on: https://chromium-review.googlesource.com/1012739
Commit-Queue: Erik Chen <erikchen@chromium.org>
Reviewed-by: Richard Coles <torne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551669}

This flag will be used for developing new Password Manager design and
form parsing (details go/new-cpm-design-refactoring).

Bug: 831123
Change-Id: I4b1becd5d926b6d8e3d841094f793d901c7998e6
Reviewed-on: https://chromium-review.googlesource.com/1016648
Commit-Queue: Vadym Doroshenko <dvadym@chromium.org>
Reviewed-by: Vaclav Brozek <vabr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551668}

This CL adds the asset used in the TabGrid popup menu action "Close
Tab".

Bug: 831152
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I8479a982ac8c280a33a036217b0688ac45b3ee0f
Reviewed-on: https://chromium-review.googlesource.com/1016908Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Commit-Queue: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551667}