GitLab

Add instructions on how to allow running APKs from unknown sources.

BUG=none
TEST=manual

Change-Id: I61539cff081555d9255454fa3f30b986c2c5c6f6
Reviewed-on: https://chromium-review.googlesource.com/1154543Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: Thiemo Nagel <tnagel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579345}

This is a reland of a3ec8289

Original change's description:
> Re-enable webrtc audio quality browsertests on windows.
> 
> Tests were previously disabled when they started failing (more context in linked bug). Re-enabling because tests pass locally on win7 machine.
> 
> Bug: 850936
> Change-Id: Ie1ab4e82daf2c0265e932237d3edf6b3399ee803
> Reviewed-on: https://chromium-review.googlesource.com/1139066
> Reviewed-by: Henrik Boström <hbos@chromium.org>
> Commit-Queue: Marina Ciocea <marinaciocea@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#575638}

Bug: 850936
Change-Id: I44be2227ffed90d25549669de4dd3f354a98d92a
Reviewed-on: https://chromium-review.googlesource.com/1156224Reviewed-by: Henrik Boström <hbos@chromium.org>
Commit-Queue: Marina Ciocea <marinaciocea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579344}

Tooltip is not needed for the "sign in another user..." button because
it's same as the label itself. The default behavior of the label can add
tooltip with the label text only when the label is truncated.

Test: manual
Bug: 866439
Change-Id: I4dee9ea44b751d64f07b0458311a653bffb0a283
Reviewed-on: https://chromium-review.googlesource.com/1152644Reviewed-by: Tetsui Ohkubo <tetsui@chromium.org>
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Commit-Queue: Tatsuhisa Yamaguchi <yamaguchi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579343}

https://chromium.googlesource.com/chromium/src/third_party/freetype2.git/+log/b532d7ce708c..578bcf103a12

$ git log b532d7ce7..578bcf103 --date=short --no-merges --format='%ad %ae %s'
2018-07-30 wl [cff] Fix typo.
2018-07-29 wl * src/pcf/pcfread.c (pcf_get_encodings): Another thinko.
2018-07-28 apodtele * builds/unix/configure.raw: Minor.
2018-07-28 apodtele [smooth] Fix Harmony memory management.
2018-07-28 wl [type1] Avoid segfaults with `FT_Get_PS_Font_Value'.
2018-07-27 wl [truetype] Make `TT_Set_MM_Blend' idempotent (#54388).
2018-07-27 wl [psaux, type1]: More tracing improvements.
2018-07-26 wl Document FT_Module_Class's `module_interface' field.
2018-07-25 wl [psaux, type1] Trace PostScript dictionaries and other things.
2018-07-25 wl Fix minor ASAN run-time warnings.
2018-07-24 apodtele [build] Fortify dllexport/dllimport attributes (#53969,#54330).
2018-07-24 wl [type1] Check relationship between number of axes and designs.
2018-07-24 htl10 [truetype] Match ttdebug's naming of instruction mnemonics.
2018-07-24 wl * src/pcf/pcfread.c (pcf_get_encodings): Thinko.
2018-07-22 wl * src/pcf/pcfread.c (pcf_get_encodings): Check index of defaultChar.
2018-07-22 wl * src/pcf/pcfread.c (pcf_load_font): Fix number of glyphs.
2018-07-22 wl [cid] Sanitize `BlueShift' and `BlueFuzz'.
2018-07-22 wl Fix links.
2018-07-21 wl Minor comment improvement.
2018-07-21 wl [pcf] Fix handling of the undefined glyph.
2018-07-21 wl Improve stream extraction macro documentation.
2018-07-20 prince.cherusker Move the legacy fuzz target to the `freetype-testing' repository.
2018-07-20 wl [cff] Avoid left-shift of negative numbers (#54322).
2018-07-17 wl Allow FT_ENCODING_NONE for `FT_Select_Charmap'.
2018-07-17 wl * src/pcf/pcfread.c (pcf_get_encodings): Trace `defaultChar'.
2018-07-16 prince.cherusker * include/freetype/internal/ftcalc.h: Add macros for handling harmless over-/underflowing `FT_Int' values.
2018-07-16 wl * src/truetype/ttgxvar.c (tt_set_mm_blend): Fix off-by-one error.
2018-07-12 wl * src/base/ftoutln.c (FT_Outline_Get_Orientation): Init `cbox'.

Created with:
  roll-dep src/third_party/freetype/src
R=bungeman@chromium.org,drott@chromium.org

CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_chromium_msan_rel_ng

PDFium-Issue: pdfium:1125
Change-Id: I09a0c66426dfdade532fe262d0ace7656e3d1b53
Reviewed-on: https://chromium-review.googlesource.com/1155652Reviewed-by: Dominik Röttsches <drott@chromium.org>
Commit-Queue: Dominik Röttsches <drott@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579342}

This PR accidentally made the "addTrack(0 streams)" test call addTrack()
with a stream, which broke the tests:
https://github.com/web-platform-tests/wpt/pull/12141

Test restored and all "addTrack/addTransceiver(X streams)" are made to
consistently use "new MediaStream()" the same way.

Bug: 869036
Change-Id: Ieaf0c73a1996aeb2e87e18f2f9065fa1e6fee1e7
Reviewed-on: https://chromium-review.googlesource.com/1155125Reviewed-by: Harald Alvestrand <hta@chromium.org>
Commit-Queue: Henrik Boström <hbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579341}

This CL switch only imported header file to unblock WebRTC refactoring
to move sigslot target to proper location. But it is required that
all usages of sigslot will be eliminated from chromium code base. Here
is a ticket to track this effort: https://crbug.com/867820

Bug: 867820, webrtc:8366
Change-Id: I431a178048886833b8bbd8508ce5a3937997f5ab
Reviewed-on: https://chromium-review.googlesource.com/1150537Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Olga Sharonova <olka@chromium.org>
Commit-Queue: Artem Titov <titovartem@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579340}

Live bytes are computed during sweeping for regular garbage collection.
The live bytes computation was missing from the logic that makes the
heap consistent after performing marking just for taking a snapshot.
The result is that ProcessHeap counters are not usable when just
performing a garbage collection for snapshots as the counters would
always be zeroed out.

Bug: chromium:865438
Change-Id: Id0d7e7a75895a6e6bb0ae62959f3e7a36b3d26fa
Reviewed-on: https://chromium-review.googlesource.com/1155121Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579339}

This change contains a small fix to perform an explicit connection check
after receiving a BluetoothLEDevice instance. This is necessary for
paired devices, where the connection handler gets registered too late to
get notified of the initial GATT connection initiated by the OS.

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

https://chromium.googlesource.com/chromiumos/chromite.git/+log/9093ff098254..2c3d9c837512


git log 9093ff098254..2c3d9c837512 --date=short --no-merges --format='%ad %ae %s'
2018-07-31 manojgupta@google.com cros_setup_toolchains: Enable llvm-libunwind in cross-* pkgs.


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

The AutoRoll server is located here: https://chromite-chromium-roll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=chrome-os-gardeners@chromium.org

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

This CL adds announcements to be read with VoiceOver for the in product
helps.

Bug: 866936
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: Ie89fa1b2b0179a8070351c2ce926ac18158ebdef
Reviewed-on: https://chromium-review.googlesource.com/1154536Reviewed-by: edchin <edchin@chromium.org>
Commit-Queue: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579336}

#enable-show-autofill-signatures flag is enabled.

<form name="change_password_form" >
  <input type="text" name="username" pm_parser_annotation="username_element" >
  <input type="password" name="current-password" pm_parser_annotation="password_element" >
  <input type="password" name="new-password" pm_parser_annotation="new_password_element" >
  <input type="password" name="password-again" pm_parser_annotation="confirmation_password_element" >
</form>

Bug: 824793
Change-Id: Ie02de13798abfc0aca0dfe9c47667ff4c52b4163
Reviewed-on: https://chromium-review.googlesource.com/1151315
Commit-Queue: Maxim Kolosovskiy <kolos@chromium.org>
Reviewed-by: Dominic Battré <battre@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579335}

@IntDef/@StringDef annotation are preferred way for declaring
set of String/int values.

1. they need less space in APK than enum, see
https://developer.android.com/topic/performance/reduce-apk-size#remove-enums
2. they give more control over allowed values than "static final" values

Main goal of patch is writing "static final" values, enum
and some classes in one common @IntDef/@StringDef form:

1. with @IntDef/@StringDef first, @Retention second
   and related @interface third
2. with values inside @interface
3. with NUM_ENTRIES declaring number of entries if necessary
4. with comment about numbering from 0 without gaps when necessary
5. with @Retention(RetentionPolicy.SOURCE)
6. without "static final" in the @interface

Change-Id: If5663352e5aaf731e8c5a166a6ac1c9c59988867
Reviewed-on: https://chromium-review.googlesource.com/1150224Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Commit-Queue: Marcin Wiącek <marcin@mwiacek.com>
Cr-Commit-Position: refs/heads/master@{#579334}

https://webrtc.googlesource.com/src.git/+log/1899c1270f0c..168b4977f3c2


git log 1899c1270f0c..168b4977f3c2 --date=short --no-merges --format='%ad %ae %s'
2018-07-31 buildbot@webrtc.org Roll chromium_revision fdf31b15..fb3a9711 (579160:579262)
2018-07-30 peah@webrtc.org AEC3: Increasing the accuracy of the detection for early reverb


Created with:
  gclient setdep -r src/third_party/webrtc@168b4977f3c2

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=luci.chromium.try:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng

BUG=chromium:None,chromium:865397
TBR=webrtc-chromium-sheriffs-robots@google.com

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

Most callers anyway need to retrieve information from ProfileSyncService
about the selected datatypes, so let's instead make sure all paths to
DataTypeManager::Configure() are centralized in ProfileSyncService.

This will simplify extending the functions signature in follow-up
patches.

Bug: 866814,851476
Change-Id: Ied3975b9ed833d34aee1802aae36c1332b3b68e0
Reviewed-on: https://chromium-review.googlesource.com/1152812
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579332}

The pref for the Safe Sites filter, kSafeSitesFilterBehavior, should be
accessible from //components/policy, where the filter itself will check
it once implemented.

Note that this policy's pref is unused until the follow-up CL.

Bug: 819405
Change-Id: If87877488067ee008f16d2dc848f5e2ab01de25f
Reviewed-on: https://chromium-review.googlesource.com/1117877Reviewed-by: Ramin Halavati <rhalavati@chromium.org>
Reviewed-by: Julian Pastarmov <pastarmovj@chromium.org>
Commit-Queue: Michael Giuffrida <michaelpg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579331}

As far as I can tell, both of these flags just mean "respect
symbol_level when sanitizers are enabled". This change makes it so
that we always do that, removing the need for both flags. Because
the default symbol level is 1 in sanitizer builds, we still get -g1
(which is equivalent to -gline-tables-only) by default. Users can
now adjust the symbol level simply by setting symbol_level.

Change-Id: Ied9f589c268e7e5c625b4cce46d76e51452e473b
Reviewed-on: https://chromium-review.googlesource.com/1156125Reviewed-by: Oliver Chang <ochang@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Peter Collingbourne <pcc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579330}

Many call sites which use this function do not
bother about the clearing the typing but they use
this function which is confusing.
Rather we will use FrameSelection::SetSelection
and pass default options now.
This CL does it from the EditorCommand.

Bug: 867563
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng
Change-Id: Ie25813f1cd76b00001a0447118c9a3e8cd42cc52
Reviewed-on: https://chromium-review.googlesource.com/1154861
Commit-Queue: Tanvir Rizvi <tanvir.rizvi@samsung.com>
Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579329}

A primary use case for the All sites page should be to view the sites
that are using the most disk space on the user’s device. Currently
grouped sites show the number of cookies. This patch adds support for
showing the amount of local storage used for each origin.

Sorting by data usage in all sites will now sort by local storage, using
the number of cookies as a tie breaker, i.e. sites with any amount of
local storage will be ordered before sites with no local storage and
cookies.

Cq-Include-Trybots: luci.chromium.try:closure_compilation
Change-Id: I3816279f9795fbaa59d67279548a1deba097f852
Bug: 835712
Reviewed-on: https://chromium-review.googlesource.com/1148169
Commit-Queue: Patti <patricialor@chromium.org>
Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579328}

https://skia.googlesource.com/skia.git/+log/946a162a5239..0bf5408286ea


git log 946a162a5239..0bf5408286ea --date=short --no-merges --format='%ad %ae %s'
2018-07-31 brucewang@google.com Implement Subpixel mode for color format image.


Created with:
  gclient setdep -r src/third_party/skia@0bf5408286ea

The AutoRoll server is located here: https://autoroll.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.blink:linux_trusty_blink_rel;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=scroggo@chromium.org

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

This allows them to be opened via chrome.fileManagerPrivate.executeTask.
Otherwise, they get filtered out in apps::LaunchPlatformAppWithFileHandler(..)

Note this doesn't change the default app for opening video files from
the file manager. It just allows the files to be passed via the
executeTask API. Adds FileTasksBrowserTest.DefaultHandlerChangeDetector
to ensure default handlers do not change unless it is intended to do so
in future.

Bug: 867705
Change-Id: I758b8599b6fa1f0b60f72bf3466d796ec0aedaa3
Reviewed-on: https://chromium-review.googlesource.com/1151039Reviewed-by: Noel Gordon <noel@chromium.org>
Reviewed-by: Naoki Fukino <fukino@chromium.org>
Commit-Queue: Trent Apted <tapted@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579326}

SelectPopup is the only object invoked from native before Java
side initialization is performed, which led to adding a call in
|WebContentsImpl.initialize| to trigger the init operation. This
CL removes the explicit init by having native do it in a lazy
manner before invoking Java side |show|.

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

The absolute path for the mount point of Android files is long and not user
friendly. We should replace the path with the "Play files" label, which we use
in Files app UI.

Bug: 863616
Test: Manually tested on ARC-enabled device with multiple languages.
Change-Id: I96548e96489c8480736f8cfc1de53f1c838eb98b
Reviewed-on: https://chromium-review.googlesource.com/1145100Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Naoki Fukino <fukino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579324}

We need to enforce exit_time_destructors compiler flag warning everywhere.

Bug: 860581
Change-Id: I7cbb2ef3ba5c71a69233bc8bd50725786b6ea0b1
Reviewed-on: https://chromium-review.googlesource.com/1155276Reviewed-by: Trent Apted <tapted@chromium.org>
Reviewed-by: Dominick Ng <dominickn@chromium.org>
Commit-Queue: Alexey Baskakov <loyso@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579323}

Most test extensions manifest's have a descripton: the Files, Gallery,
Video Player, Audio Player test extension do not (how very odd). Let's
give them a description, for consistency with all test extensions.

Bug: 836254
Change-Id: I43dce817da53f9ae7a584f4b30907c3c0649ee85
Reviewed-on: https://chromium-review.googlesource.com/1156118
Commit-Queue: Noel Gordon <noel@chromium.org>
Reviewed-by: Luciano Pacheco <lucmult@chromium.org>
Reviewed-by: Noel Gordon <noel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579322}

Previously the field constructed_tree_scopes is added
to CSSStyleSheet to track addition of constructed CSSStyleSheets
in tree scopes when adding/removing them from
TreeScope.moreStyleSheets.

moreStyleSheets already was removed at blink[1].
so constructed_tree_scopes is not needed anymore.

[1]https://chromium-review.googlesource.com/1125741

Bug: 869244
Change-Id: Icc0b68393fd100d08ee826d6dbe0859468ee1e23
Reviewed-on: https://chromium-review.googlesource.com/1155286
Commit-Queue: Hwanseung Lee <hs1217.lee@samsung.com>
Reviewed-by: Eric Willigers <ericwilligers@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579321}

A test is added to check if an extension with command key binding can
not be accessed when it is not allowed in incognito.

Bug: 862075
Change-Id: Id49fb16958a9913978f8b054740c0ef4d0977c25
Reviewed-on: https://chromium-review.googlesource.com/1141727Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Commit-Queue: Ramin Halavati <rhalavati@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579320}

This reverts commit 903b12a4.

Reason for revert: https://crbug.com/869189 .

Original change's description:
> ChromeOS OOBE: Add Welcome screen animations.
>
> This Cl replaces static images with animations.
>
> Bug: 852038
> Change-Id: Id58f69e4275231dc4b6491358d10a343f529fc2f
> Reviewed-on: https://chromium-review.googlesource.com/1153832
> Commit-Queue: Alexander Alekseev <alemate@chromium.org>
> Reviewed-by: Wenzhao (Colin) Zang <wzang@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#578882}

TBR=alemate@chromium.org,wzang@chromium.org

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

Bug: 852038,869189
Change-Id: Icb34fb1b55268e0779e14deac01245397906c567
Reviewed-on: https://chromium-review.googlesource.com/1155503
Commit-Queue: Alexander Alekseev <alemate@chromium.org>
Reviewed-by: Alexander Alekseev <alemate@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579319}

[LayoutNG] Label its own bug number for compositing/layer-creation/stacking-context-overlap-nested.html

TBR=atotic@chromium.org
NOTRY=true

Bug: 869265
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng
Change-Id: I9a87e971df42fcef062344aae8a9bc34e7171ffd
Reviewed-on: https://chromium-review.googlesource.com/1156116
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579318}

This is for more than 30 mins compile steps like below builds.

3289s https://ci.chromium.org/b/8939589875030577744
2860s https://ci.chromium.org/b/8939533321978116048
2593s https://ci.chromium.org/b/8939552683394930304
2569s https://ci.chromium.org/b/8939542011766010272

Bug: 804251
Change-Id: I57f2a63c4e1fdcbdc21eeea5ed421b41237db268
Reviewed-on: https://chromium-review.googlesource.com/1155285Reviewed-by: Shinya Kawanaka <shinyak@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579317}

Name various test extensions used in integration test consistently, in
particular, the test extensions used in the providers.js tests [1].

[1] these are the only integration testcases that explicitly depend on
test extension manifest name. Add a helper to return the name.

Bug: 869240
Change-Id: I9d5ab74af9083201ab2c0173c2e505ec6f6e05be
Reviewed-on: https://chromium-review.googlesource.com/1154784Reviewed-by: Trent Apted <tapted@chromium.org>
Reviewed-by: Luciano Pacheco <lucmult@chromium.org>
Commit-Queue: Noel Gordon <noel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579316}

A primary use case for the All Sites page should be to view the sites that are
using the most disk space on the user's device. Implement this in this patch
with support for sorting via the number of cookies (descending), with local and
website storage to come later.

Bug: 835712
Cq-Include-Trybots: luci.chromium.try:closure_compilation
Change-Id: Ia8ea3f75d1d102956bf31f9d409e642d47698aca
Reviewed-on: https://chromium-review.googlesource.com/1146534
Commit-Queue: Patti <patricialor@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579315}

The web::test::HttpServer instance returned by GetSharedInstance has
zero reference count initially, HttpServer::StartOrDie implicitly
increments the reference count through base::Bind, and its
embedded_test_server_ destruction decrements the reference count.
That is, once the server stopped, the shared instance is destroyed, and
GetSharedInstance() starts returning a stale pointer.

After this CL, the resulting instance of GetSharedInstance has non-zero
reference count, so that it keeps alive after server stop.

Bug: 866456
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I1f13dff1e86d08efb28b5805ae73426b874a4f80
Reviewed-on: https://chromium-review.googlesource.com/1146881Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: Taiju Tsuiki <tzik@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579314}

Add metrics to measure the amount of time it takes for the BeginFrame
to travel to the Client from the DisplayCompositor.

Bug:814909

related design doc:
https://docs.google.com/document/d/1XIaAPkajbML0s0neajasWzvt856dUjF_1zAWO5KykCQ/edit

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I6a13b5fcdbfc458e2d2dc84312e2e37459c11ac4
Reviewed-on: https://chromium-review.googlesource.com/1146337
Commit-Queue: Yi Xu <yiyix@chromium.org>
Reviewed-by: Mark Pearson <mpearson@chromium.org>
Reviewed-by: Fady Samuel <fsamuel@chromium.org>
Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579313}

This reverts commit 1f75f99f.

Reason for revert: This breaks compile on "Google Chrome Win" builder due to a integer sign mismatch in incompatible_applications_updater_win.cc(289,32).

Original change's description:
> Fix to remove /wd4018 (-Wno-sign-compare) for /build/config
> 
> This is a part of effort to remove /wd4018 warning suppression.
> 
> Master CL is
> https://chromium-review.googlesource.com/c/chromium/src/+/1152755
> 
> This CL was uploaded by git cl split.
> 
> R=​dpranke@chromium.org
> 
> Bug: 868248, 588506
> Change-Id: I3c31dc8f72266eda85904c4a860e156fadb2a76e
> Reviewed-on: https://chromium-review.googlesource.com/1152840
> Commit-Queue: Nico Weber <thakis@chromium.org>
> Reviewed-by: Nico Weber <thakis@chromium.org>
> Reviewed-by: Dirk Pranke <dpranke@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#579311}

TBR=thakis@chromium.org,dpranke@chromium.org,pmonette@chromium.org,tikuta@chromium.org

Change-Id: Iaed7b5e6f2ff3c53e1846162ca8d14689f7ae5e1
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 868248, 588506
Reviewed-on: https://chromium-review.googlesource.com/1156144Reviewed-by: Patti <patricialor@chromium.org>
Commit-Queue: Patti <patricialor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579312}

This is a part of effort to remove /wd4018 warning suppression.

Master CL is
https://chromium-review.googlesource.com/c/chromium/src/+/1152755

This CL was uploaded by git cl split.

R=dpranke@chromium.org

Bug: 868248, 588506
Change-Id: I3c31dc8f72266eda85904c4a860e156fadb2a76e
Reviewed-on: https://chromium-review.googlesource.com/1152840
Commit-Queue: Nico Weber <thakis@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579311}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/b0e855a33353..1aefad2a5982


git log b0e855a33353..1aefad2a5982 --date=short --no-merges --format='%ad %ae %s'
2018-07-30 recipe-roller@chromium.org Roll recipe dependencies (trivial).


Created with:
  gclient setdep -r src/third_party/depot_tools@1aefad2a5982

The AutoRoll server is located here: https://depot-tools-chromium-roll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=agable@chromium.org

Change-Id: I8a228ba7556e1f43844e70d022aa9ec25a0593bf
Reviewed-on: https://chromium-review.googlesource.com/1155884Reviewed-by: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#579310}

Hide the Google Photos option for custom backgrounds as it will
not be launched in M69.

All service requests occur after clicking on the option, so simply
hiding it is sufficient.

Bug: 869230
Change-Id: I55ca201978323cd7031e618b6a9039b93a485130
Reviewed-on: https://chromium-review.googlesource.com/1155978Reviewed-by: Kristi Park <kristipark@chromium.org>
Commit-Queue: Kyle Milka <kmilka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579309}

This relands commit 748ac91e.

Includes workaround for LTO bug exposed by orderfile generation:
add -Wl,-u,__cyg_profile_func_enter_bare to ldflags. A fix for the
bug is under review: https://reviews.llvm.org/D50017

Original change's description:
> Reland "build: Enable ThinLTO in official Android builds."
>
> This relands commit 1ba5d611.
>
> Fixes chromeos build issue by checking whether we have a secondary
> toolchain before comparing it with the current toolchain.
>
> Original change's description:
> > build: Enable ThinLTO in official Android builds.
> >
> > With this change we start building Chromium with ThinLTO when targeting
> > Android. In 64-bit Monochrome builds, the 32-bit binary is also built
> > with ThinLTO. This change is expected to reduce code size significantly
> > (in local measurements it reduced the size of MonochromePublic.apk by 2.3MB)
> > and is a prerequisite for enabling control flow integrity in official
> > builds.
> >
> > Note that although this change enables ThinLTO, it does not enable
> > the cross-TU optimizations normally associated with LTO. Enabling
> > cross-TU optimizations is a separate project.
> >
> > On my local machine (a Lenovo P920) libmonochrome.so link times increase
> > as follows:
> >
> >          before  after (first link)  after (incremental links)
> > 32-bit    18s          140s                     50s
> > 64-bit    13s          107s                     41s
> >
> > All known blockers have now been fixed. This CL is likely to uncover
> > unknown blockers.
> >
> > Bug: 469376
> > Change-Id: I8981e17abd50bc5ca00440e0d74dda878c911749
> > Reviewed-on: https://chromium-review.googlesource.com/1150870
> > Commit-Queue: Peter Collingbourne <pcc@chromium.org>
> > Reviewed-by: Nico Weber <thakis@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#578195}
>
> Bug: 469376, 868426
> Change-Id: Ie46ba0db0ac68130bc600415777de5c813b28ebd
> Reviewed-on: https://chromium-review.googlesource.com/1153612
> Reviewed-by: Dirk Pranke <dpranke@chromium.org>
> Commit-Queue: Peter Collingbourne <pcc@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#578854}

Bug: 469376, 867878
Change-Id: Iae4cc698b5955df5ae1970c71646fa05fca61109
Reviewed-on: https://chromium-review.googlesource.com/1155977Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Peter Collingbourne <pcc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579308}

This patch introduces a DCHECK checking if DocumentMarker.endoffset is
not over text length when adding each marker.

Bug: 868229
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng
Change-Id: I4f2681460d1de1aacf78b55d7bba454dfc557cf2
Reviewed-on: https://chromium-review.googlesource.com/1149763
Commit-Queue: Yoichi Osato <yoichio@chromium.org>
Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579307}

If the "No destinations" message was visible in the recent destinations
list, the iron-list containing the destinations was pushed down and
extended invisibly over the "Manage" link for the print destinations,
preventing the link from working.

Fix this by ensuring that the iron-lists are the correct height: height
of the parent - height of parent's header line - height of "No
destinations" message (if showing).

Bug: 868274
Change-Id: Idf574923d69ae1c0ff1265efe62b9cc67b70af11
Reviewed-on: https://chromium-review.googlesource.com/1153430
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579306}