GitLab

https://chromium.googlesource.com/external/leveldb.git/+log/6fa45666703a..d177a0263cce

$ git log 6fa456667..d177a0263 --date=short --no-merges --format='%ad %ae %s'
2018-04-17 costan Replace port_posix with port_stdcxx.
2018-04-16 falke.marco Fix sign mismatch warnings in GCC.
2018-04-13 costan Add forgotten <limits> header to util/logging.cc.
2018-04-13 costan Reimplement ConsumeDecimalNumber.
2018-04-13 costan Add tests for ConsumeDecimalNumber.
2018-04-11 costan Document the building process.
2018-04-10 costan Replace NULL with nullptr in C++ files.
2018-04-10 costan Remove PLATFORM_IS_LITTLE_ENDIAN from port/posix.h.
2018-04-09 costan Define LEVELDB_HAS_PORT_CONFIG_H for old compilers.

Created with:
  roll-dep src/third_party/leveldatabase/src

Change-Id: Icabf330020abf70e91b00c835d34af070bcea4e2
Reviewed-on: https://chromium-review.googlesource.com/1015774Reviewed-by: Chris Mumford <cmumford@chromium.org>
Commit-Queue: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552445}

Finish removing fuzzing to fix CHECK failure.

Bug: 835177
Change-Id: Ia65d58861a4842cba214eb2dc7bb5e01b8740949
Reviewed-on: https://chromium-review.googlesource.com/1022336Reviewed-by: Abhishek Arya <inferno@chromium.org>
Commit-Queue: Jonathan Metzman <metzman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552444}

To check whether an InlineBox is an InlineTextBox, the current function
to call is IsInlineTextBox() instead of IsText().

This patch changes one caller using IsText(), to fix a bad cast.

Bug: 834850
Change-Id: I37acb6234ae60384138074d430a31dac9e6e11a7
Reviewed-on: https://chromium-review.googlesource.com/1019922Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: Yoichi Osato <yoichio@chromium.org>
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552443}

Bug: 833590
Change-Id: Idde436685e6f5400271c2b850c2740c9ec5c9fb0
TBR: thakis@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/1022394Reviewed-by: Erik Chen <erikchen@chromium.org>
Commit-Queue: Erik Chen <erikchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552442}

Like for inline-installation, the Chrome Web Store should be able to get
the full referrer chain of how the user got to the CWS item detail page.
This change adds a new API that the CWS (and only the CWS) will be able
to call in order to report & detect misleading user acquisition patterns.

Note that the SafeBrowsing "download" ping which is sent if the user
chooses to install the extension already contains the referrer data. This
change effectively allows the CWS to collect the data before the user
chooses to install the extension.

Bug: 807090
Change-Id: I6b7f85fca75fc134a3d6207cb47a1f3674d15c96
Reviewed-on: https://chromium-review.googlesource.com/1015792
Commit-Queue: proberge <proberge@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Reviewed-by: Jialiu Lin <jialiul@chromium.org>
Reviewed-by: Gayane Petrosyan <gayane@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552441}

TBR=jbudorick@chromium.org

Bug: 835056
Change-Id: I27d45a5b0e80ec9412deaf8574d950924f1e9813
Reviewed-on: https://chromium-review.googlesource.com/1022514Reviewed-by: Ben Pastene <bpastene@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552440}

This CL implements a workaround for the race condition between the mojo
tap notification and the selection-changed notification.  Now we never enter
the SELECTION_CLEARED internal state unless there was a previous
selection that got cleared.

Before this CL every tap caused a transition to the SELECTION_CLEARED
internal state even when there was no previous selection.  This would usually
happen before the Tap notification, but now that's faster due to mojo so it
sometimes arrives beforehand.  With this change we ignore selection changes
from empty to empty so there's only one notification for most taps. Now we only
enter the SELECTION_CLEARED state when it's really needed (in the tap
near previous selection use-case).

Also does some minor test cleanup and re-enabling disabled tests that
were flaky.

BUG=818897

Change-Id: I269815ad087aabf93332e99a6c7d1208addabe3e
Reviewed-on: https://chromium-review.googlesource.com/1018300
Commit-Queue: Donn Denman <donnd@chromium.org>
Reviewed-by: Theresa <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552439}

BUG=818736

Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ib36d98ce7d4a89b1384f8ea5169705d7b64a87c9
Reviewed-on: https://chromium-review.googlesource.com/1022370
Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org>
Reviewed-by: Kurt Horimoto <kkhorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552438}

Bug: 834404
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I95e93dfae14271e2cd08a42685b8e3f3ad9ff169
Reviewed-on: https://chromium-review.googlesource.com/1022076Reviewed-by: Roger McFarlane <rogerm@chromium.org>
Commit-Queue: Moe Ahmadi <mahmadi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552437}

This switch supersedes the current SupportExperimentalCdmInterface
feature, which is removed in this CL.

A library CDM interface is "supported" if it's implemented by CdmAdapter
and CdmWrapper. Typically multiple CDM interfaces are supported:
- The latest stable CDM interface.
- Previous stable CDM interface(s), for supporting older CDMs.
- Experimental CDM interface(s), for development.

A library CDM interface is "enabled" if it's enabled at runtime, e.g.
being able to be registered and creating CDM instances. Typically
experimental CDM interfaces are supported, but not enabled.

The new switch will override default enabled CDM interface versions,
which provides finer control in a test environment or for local
debugging, including enabling the support of experimental CDM
interfaces.

This is in preparation of supporting CDM_10 by default in Chromium,
where we need to be able test CDM_9, CDM_10 and potentially CDM_11
with the same set of tests.

switch.

Bug: 733821
Test: No functionality change. Existing tests updated to use the new
Change-Id: I84d79f7b9e63e80434139bedb6d5533baacf8ff0
Reviewed-on: https://chromium-review.googlesource.com/1018726Reviewed-by: Sorin Jianu <sorin@chromium.org>
Reviewed-by: Nick Carter <nick@chromium.org>
Reviewed-by: John Rummell <jrummell@chromium.org>
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552436}

These histograms were added in a
previous CL (http://shortn/_1VKvI480Ld), but I forgot to
add them to histograms.xml.

Bug: 830088
Change-Id: I8496ee47d11d0e631df46fff3706b3b668bba570
Reviewed-on: https://chromium-review.googlesource.com/1022230Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552435}

This keeps webview displaying overlay help hidden until new content is
loaded. Applied for both, OOBE and in-session OptIn.

Test: Locally
Bug: 832003
Bug: 821327
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ibc6574f770bbbeb2d33dd042699f0e149256eec8
Reviewed-on: https://chromium-review.googlesource.com/1021841
Commit-Queue: Yury Khmel <khmel@google.com>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552434}

Both canvas.toBlob() and OffscreenCanvas.convertToBlob() require support of
idle tasks, which are only enabled when threaded compositing is turned on for
layout tests.

Bug: 831595
Change-Id: If47e0a1c6c6669c24a34424d20020a2969d2a3fc
Reviewed-on: https://chromium-review.googlesource.com/1020365Reviewed-by: Justin Novosad <junov@chromium.org>
Commit-Queue: Olivia Lai <xlai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552433}

This reverts commit 523d88f8.

Reason for revert: Tests should be fixed now.

Original change's description:
> Disable flaky services java tests.
> 
> TBR=sky@chromium.org
> 
> Bug: 834288, 833088
> Change-Id: Ide3dc52c89a77be635b1143dcb4c6ae28d3dcb0e
> Reviewed-on: https://chromium-review.googlesource.com/1021120
> Reviewed-by: Rune Lillesveen <futhark@chromium.org>
> Commit-Queue: Rune Lillesveen <futhark@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#552283}

TBR=futhark@chromium.org

Change-Id: I1168452281fb0331ac7ee9e4cefbd53bef1c3318
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 834288, 833088
Reviewed-on: https://chromium-review.googlesource.com/1022551Reviewed-by: Ken Rockot <rockot@chromium.org>
Commit-Queue: Ken Rockot <rockot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552432}

    The CL removes the comment in file "components/ntp_snippets/features.h".

    There are related features with names based on the ArticleSuggestions name.
    So the CL removes the TODO and leaves the whole code intact.

BUG=
R=jkrcal@chromium.org
Signed-off-by: fr.fang <fr.fang@samsung.com>
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ie6cd5b9409b1ae6e5778e8b6c467d061cf835afe
Reviewed-on: https://chromium-review.googlesource.com/989383
Commit-Queue: Filip Gorski <fgorski@chromium.org>
Reviewed-by: Filip Gorski <fgorski@chromium.org>
Reviewed-by: Jan Krcal <jkrcal@chromium.org>
Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552431}

Upcoming network servicification changes require that the cookies
to be deleted be serializable for transport over IPC. This change
converts all net::CookieStore::CookiePredicate's to a structure
(net::CookieStore::CookieDeletionInfo).

Bug: 823297
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo;master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I4176e1a0acacd95b165223cebab64c61e2129903
Reviewed-on: https://chromium-review.googlesource.com/996353Reviewed-by: Christian Dullweber <dullweber@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Reviewed-by: Maks Orlovich <morlovich@chromium.org>
Commit-Queue: Chris Mumford <cmumford@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552430}

The message right now assumes mojom::CredentialManager failed,
but it could have been mojom::AuthenticatorImpl.

Bug: 832910
Change-Id: Ibaca9d938ffe854ab6ee81a3aa56721991e505e6
Reviewed-on: https://chromium-review.googlesource.com/1012967Reviewed-by: Balazs Engedy <engedy@chromium.org>
Commit-Queue: Kim Paulhamus <kpaulhamus@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552429}

Since the bottom sheet is now largely per-tab, there is logic that
closes the sheet when a new tab is selected or when a page navigates.
This handles the cases for intents, making the code that was removed
by this patch obsolete.

Bug: 833574
Change-Id: Ibe09ce81f96946eeb8980e2a79dbd9db7f56cb6f
Reviewed-on: https://chromium-review.googlesource.com/1020061Reviewed-by: Theresa <twellington@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Commit-Queue: Matthew Jones <mdjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552428}

This CL changes the test to navigate to a non-404 URL. This change
make the 404 interstitial no longer show. I suspect that the interstitial was
causing the test to be flaky.

Navigating to a non-404 URL was accomplished by:
- specifying the target port via the --host-resolver-rules command line flag
- navigating to non-https URLs (navigating to an https URL causes an
  interstitial about certificate validation to show)

This CL introduces a new test WebAPK - "http_webapk" - which has a http
scope

BUG=834466

Change-Id: I9e9dda7079e3bf2fb62f57d2ecf3ccaad5dbd10d
Reviewed-on: https://chromium-review.googlesource.com/1020229Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552427}

Adds --zero-tests-executed-ok, which makes the layout test runner exit
with an ok status if zero tests are executed. This is important for
re-running layout tests without patch, which we do on tryservers to
check if the test was being flaky.

Bug: 533481
Change-Id: I7d6bcd805b99818fdf82ecb88e71863ff72b72e7
Reviewed-on: https://chromium-review.googlesource.com/1018590Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Reviewed-by: Jeff Carpenter <jeffcarp@chromium.org>
Reviewed-by: Robert Ma <robertma@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552426}

Bug: 835342
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I1c5ae21731c6b5b339591f44926ad0a5769578ae
Reviewed-on: https://chromium-review.googlesource.com/1022182
Commit-Queue: Eugene But <eugenebut@chromium.org>
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552425}

This allows us to get rid of an ugly priority promotion hack used for
download notifications on Chrome OS.

On platforms with a message center bubble (Chrome OS), web
notifications that don't specify renotify won't pop
up again if the original popup was dismissed.

On platforms without a message center bubble (Linux, Windows when
using non-native notifications), all notifications will pop up again
on update.

Bug: none
Change-Id: I6a00b21348335b29a97a78a603c03a31ae8e5452
Reviewed-on: https://chromium-review.googlesource.com/976545Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Commit-Queue: Evan Stade <estade@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552424}

The ExtensionFunction::GetAssociatedWebContentsDeprecated() method is
unpredictable and can return surprising values in certain circumstances.
It is deprecated and should be removed to ensure that callers are using
the WebContents they expect.

Update the easyUnlockPrivate API to use GetSenderWebContents() instead.

Bug: 461394
Change-Id: I1ba338c4c551b94ab8c21d5f642cf203383aea8a
Reviewed-on: https://chromium-review.googlesource.com/1022122Reviewed-by: Toni Barzic <tbarzic@chromium.org>
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552423}

The tests have not been very useful, and has remain disabled on a
number of bots for a while. So stop running the tests on the perf
bots altogether. The tests can still be used locally to watch for
improvements/regressions for specific changes as desired/needed.

BUG=784454, 721757, 736150
BUG=698831, 792995, 723821

Change-Id: Ifede89a3f7a87d96bbb726e0911e1df26e6f9a6b
Reviewed-on: https://chromium-review.googlesource.com/1021653Reviewed-by: enne <enne@chromium.org>
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#552422}

This reverts commit bd410037.

Reason for revert: This test failure was fixed in r552392.

Original change's description:
> Disable testDetectSucceedsOnValidBitmap everywhere
> 
> Disables testDetectSucceedsOnValidBitmap on all platforms due to
> consistent bot failures.
> 
> TBR=reillyg@chromium.org
> 
> BUG=835331
> 
> Cq-Include-Trybots: luci.chromium.try:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win10_chromium_x64_rel_ng
> Change-Id: I0a77529e7054673d4dcab0ce8099679f596ddde6
> Reviewed-on: https://chromium-review.googlesource.com/1022085
> Reviewed-by: Donn Denman <donnd@chromium.org>
> Reviewed-by: Reilly Grant <reillyg@chromium.org>
> Commit-Queue: Donn Denman <donnd@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#552390}

TBR=donnd@chromium.org,reillyg@chromium.org

Bug: 835331
Change-Id: Ibb6398faa46ade69464506b393a987698bf88437
Cq-Include-Trybots: luci.chromium.try:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win10_chromium_x64_rel_ng
Reviewed-on: https://chromium-review.googlesource.com/1022252Reviewed-by: Reilly Grant <reillyg@chromium.org>
Commit-Queue: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552421}

 o Basic shape is implemented.
 o Strokes are not drawn around and under the tabs.
 o No color and hover behaviors are changed. Next CL.
 o All changes are behind the refresh flag.

Bug: 822061
Change-Id: I8852e4d7da3bef0fff4e6163543eee395914b2a5
Reviewed-on: https://chromium-review.googlesource.com/1020065
Commit-Queue: Allen Bauer <kylixrd@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552420}

The feature is used by //components/webdata. This test will help make
sure the feature keeps working as we upgrade SQLite.

Bug: 829893
Change-Id: Ie94439e5abd2cad4e0e15bad92ec3af462850b4c
Reviewed-on: https://chromium-review.googlesource.com/1015206Reviewed-by: Chris Mumford <cmumford@chromium.org>
Commit-Queue: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552419}

As an alternative to the fuzzer from https://crrev.com/c/992496, which
takes an arbitrary string and generates a FormData structure to be
parsed into a PasswordForm, this CL introduces a fuzzer which creates
the FormData structure based on a protobuf decription.

The fuzzer's code is therefore easier to read. The FormData creation
cannot be controlled so tightly as in the case of the string-based
fuzzer, where, e.g., an on-purpose bias is introduced towards empty
field values.

Further, this CL slims down the 'fuzzer_support' target to only
include the unit-tested data_accessor.*. The generator helpers for
both the current and the new, proto-based, fuzzer are bundled inside
the fuzzer_test targets. This allows both only compiling the necessary
files for each fuzzer, and also the need to include the proto_library
as an indirect dependency of components_unittests, which conflicts
with the absence of optimisation for LITE.

The CL also fixes a snippet in the libprotobuf-mutator docs to follow
the style guide, which bans passing non-const references.

Bug: 828705
Change-Id: I89d7753783cc09703a5112bf03bbfdceee81394e
Reviewed-on: https://chromium-review.googlesource.com/1015800
Commit-Queue: Vaclav Brozek <vabr@chromium.org>
Reviewed-by: Jonathan Metzman <metzman@chromium.org>
Reviewed-by: Max Moroz <mmoroz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552418}

Bug: 831712
Change-Id: Ib42639e8c78ca905c395f9b4adce08848477d745
Reviewed-on: https://chromium-review.googlesource.com/1020197Reviewed-by: Jan Wilken Dörrie <jdoerrie@chromium.org>
Commit-Queue: Kim Paulhamus <kpaulhamus@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552417}

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I0e9a46a9ec873c68ef8fd0d6fc4218a5a0c3eca2
Reviewed-on: https://chromium-review.googlesource.com/1017317Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552416}

This change replaces some usages of container.size() with the more
generic base::size(container) and introduces a overload for
base::make_span taking another span. This can be useful for template
code where make_span is invoked on an arbitrary container. Passing a
base::span to code like this should not result in a compilation failure.

Bug: 828324
Change-Id: Icd6e8269689cf9d9b663b6a916e73e232350785a
Reviewed-on: https://chromium-review.googlesource.com/1021381Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552415}

Revert "When GPU process falls back to SwiftShader, about:gpu displays an extra copy of data for real GPU."

This reverts commit ecc130cf.

Reason for revert: Causes *-informational failures. See crbug.com/835343.

Original change's description:
> When GPU process falls back to SwiftShader, about:gpu displays an extra copy of data for real GPU.
> 
> BUG=828462
> TEST=manual in about:gpu
> R=​piman@chromium.org
> 
> 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
> Change-Id: I1a23827e504b2378762a59c46e23e730e9b2284e
> Reviewed-on: https://chromium-review.googlesource.com/1017300
> Reviewed-by: Kenneth Russell <kbr@chromium.org>
> Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
> Reviewed-by: Antoine Labour <piman@chromium.org>
> Reviewed-by: Daniel Cheng <dcheng@chromium.org>
> Commit-Queue: Zhenyao Mo <zmo@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#552251}

TBR=sadrul@chromium.org,dcheng@chromium.org,zmo@chromium.org,kbr@chromium.org,piman@chromium.org

Change-Id: I41eea435995cff4ff1c7cb85d00a3bcbc919df3f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 828462
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
Reviewed-on: https://chromium-review.googlesource.com/1022430Reviewed-by: Jacob Dufault <jdufault@chromium.org>
Commit-Queue: Jacob Dufault <jdufault@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552414}

R=reveman@chromium.org
BUG=834960
TEST=manual

Change-Id: I960ff91ca7c30341b11904157edd598b1caa611d
Reviewed-on: https://chromium-review.googlesource.com/1020198Reviewed-by: David Reveman <reveman@chromium.org>
Commit-Queue: Mitsuru Oshima <oshima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552413}

Currently the Ash unscaled resources includes two icons for Settings and
KSV (size: 192x192). The KSV icon needs to be accessed both at
ash/components/shortcut_viewer and chrome/browser/ui/app_list/.

Move them to ash/public/cpp/resources, so that we can access them at both
Ash and Chrome.

Bug: 834900
Test: manual
Change-Id: I1caf7f52a3105016697664965bd0dce42b83529c
Reviewed-on: https://chromium-review.googlesource.com/1020416Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Commit-Queue: Tao Wu <wutao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552412}

This makes notification_helper_process_unittest.cc reuse the code in
install_worker.*

Bug: 734095
Change-Id: I058971502e3681244e034de413acb891758045ba
Reviewed-on: https://chromium-review.googlesource.com/1014328
Commit-Queue: Xi Cheng <chengx@chromium.org>
Reviewed-by: Greg Thompson <grt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552411}

When the user clicked a link in view-source, the full URL of the markup
was sent to the server, ignoring Referrer Policy. This CL changes the
links created in view-source to use rel=noreferrer to avoid this leak.
It also sets rel=noopener to prevent the target tab from
manipulating the view-source view.

Bug: 834023, 813037
Test: browser_tests ViewSourceTest.*
Change-Id: Ifcb1dff09aefeee54fd455dcc52a8e2ccec79081
Reviewed-on: https://chromium-review.googlesource.com/1017315
Commit-Queue: Eric Lawrence <elawrence@chromium.org>
Reviewed-by: Mike West <mkwst@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552410}

In this CL, last_visibility_change_time is set to NowTicks() in the constructor. In addition, it is updated prior to notifying observers. This is done so that in proactive tab discarding, when a LifecycleUnit becomes HIDDEN, TabManager knows when it became HIDDEN (now), not the last time it became VISIBLE.

Bug: 77564
Change-Id: Ie909b4a884a7d70b5f542be7588e9fa03eb2fec7
Reviewed-on: https://chromium-review.googlesource.com/1020444
Commit-Queue: Varun Mohan <varunmohan@google.com>
Reviewed-by: François Doray <fdoray@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552409}

We use parameterized test to decide whether the feature (enable USS
implementation of sessions) is enabled or not. This reduces the risk
of regressions during development of the remaining features and the
later experimentation in dev channel.

Bug: 681921
Change-Id: I8519aab1b6c24affb18391c7a63e7f224a06a710
Reviewed-on: https://chromium-review.googlesource.com/1019320
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Reviewed-by: Mohamed Amir Yosef <mamir@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552408}

Replace usage of SharedBitmapManager with instead registering the
SharedMemory and SharedBitmapId through the TextureLayer that they
will be presented on. This gets the registration to go through the
CompositorFrameSink which is compatable with the VizDisplayCompositor.

R=piman@chromium.org

Bug: 730660
Change-Id: I55f915eb00082ba527a440956e93cbf46acf7ce6
Reviewed-on: https://chromium-review.googlesource.com/1021838
Commit-Queue: danakj <danakj@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552407}

This CL moves ShouldBlockBasedOnHeaders logic for Cross-Origin Read
Blocking (CORB) into //services/network.

Bug: 792546
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I37ddaa72af7c2f9e80370f5d8dd49bd907c3d831
Reviewed-on: https://chromium-review.googlesource.com/1013119Reviewed-by: Nick Carter <nick@chromium.org>
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552406}