GitLab

Bug: 888429
Change-Id: I6b71736b7455ed2c4ca2a235c34df024c013ffa5
Reviewed-on: https://chromium-review.googlesource.com/c/1254202Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: Patrik Höglund <phoglund@chromium.org>
Commit-Queue: Oleh Prypin <oprypin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625251}

Adding a call back to the view controller when being popped.

Change-Id: Ib2b3800b3204ae52fd437681b4362a25682d15ec
Reviewed-on: https://chromium-review.googlesource.com/c/1425608
Commit-Queue: Jérôme Lebel <jlebel@chromium.org>
Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625250}

For now, the hookup is static; changing the OS level settings will not
cause the settings to be recalculated.

Test: Manually verified functional on Mac and Windows.
Bug: 722548
Change-Id: Iccb601a28d26a7ac659677041ce637bdd19e7db3
Reviewed-on: https://chromium-review.googlesource.com/c/1417754
Commit-Queue: Stephen McGruer <smcgruer@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Ian Vollick <vollick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625249}

Bug: 922839
Change-Id: Ifd5e64178533aead481322038164bd4229c40102
Reviewed-on: https://chromium-review.googlesource.com/c/1423899
Commit-Queue: Wez <wez@chromium.org>
Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625248}

Modifies the swarming dimensions used for AR instrumentation tests to
match against Playstore version. The AR tests fail if the installed
Playstore version is too old, so we want to ensure that we're running
on a known-good version.

Bug: 914927
Change-Id: I3cc3b1c74aa75b9d9040e26e9eb8a969b465eaa7
Reviewed-on: https://chromium-review.googlesource.com/c/1429179
Auto-Submit: Brian Sheedy <bsheedy@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Reviewed-by: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625247}

Before this change, a supposedly non-resizeable window could participate
in a multi-resize. Non-resizeable should mean the user has no way to
resize the window.

Test: ash_unittests MultiWindowResizeControllerTest.*NonResizeable*
Bug: 921810
Change-Id: I1e3a24b1267b6afd6b68d666e6ab033ddbd7543c
Reviewed-on: https://chromium-review.googlesource.com/c/1423746
Commit-Queue: Avery Musbach <amusbach@chromium.org>
Reviewed-by: Robert Flack <flackr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625246}

...even when the device is in tablet mode position.

Bug: 903526
Change-Id: Ibc6e2cfe547a431ef546ade1b5842d89001a6980
Reviewed-on: https://chromium-review.googlesource.com/c/1422443
Commit-Queue: Xiaoqian Dai <xdai@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625245}

The normal flow:
1.A form is loaded on a page, NewPasswordFormManager is created.
2.The form is submitted (by the user or by JavaScript)
3.The submitted data are saved provisionally in the corresponding
NewPasswordFormManager
4.The form is disappeared from the page (that's considered as submission
success criteria), the proviosionally saved data are used for offering
saving.

But in principle, step 1 might be missing, for example when JavaScript
creates a dummy form, copies user typed credentials to the form and
submits the form.

This CL ensures, that we will save even in that case, by creating
NewPasswordFormManager in step 2, if it is not created yet.

Bug: 831123
Change-Id: I17b9d42d6ed4bb2daf2f052e84b376490f556bc8
Reviewed-on: https://chromium-review.googlesource.com/c/1392088
Commit-Queue: Vadym Doroshenko <dvadym@chromium.org>
Reviewed-by: Maxim Kolosovskiy <kolos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625244}

1) Pass the converted |dwTransportUsed| value into the response.
2) Require the attestation statement be a map, not just any CBOR value.

Also add unit tests.

Change-Id: Ifcf2c4936b08a45eed0be648be6307d71b48a9f1
Reviewed-on: https://chromium-review.googlesource.com/c/1429620
Commit-Queue: Adam Langley <agl@chromium.org>
Reviewed-by: Adam Langley <agl@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625243}

Change-Id: I804c3314accef5590c02aba545f6f0ea84c654e1
Reviewed-on: https://chromium-review.googlesource.com/c/1430000
Commit-Queue: Jordan Demeulenaere <jdemeulenaere@chromium.org>
Auto-Submit: Jordan Demeulenaere <jdemeulenaere@chromium.org>
Reviewed-by: Stephane Zermatten <szermatt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625242}

When the validation popup is hidden, this unexpected code path occurs:
- HideValidationMessage() is called continuously, and keeps restarting
the timer used to call Reset()
- As a result, Reset() is never reached

The fix: do not restart the timer if a hide is in progress.

This fix is blocking an accessibility fix, which needs an accurate signal
when the validation message is no longer visible.

Bug: None
Change-Id: I0d69abf006a881dd7cc147dfe49ada50217b8c4a
Reviewed-on: https://chromium-review.googlesource.com/c/1417890Reviewed-by: Kent Tamura <tkent@chromium.org>
Commit-Queue: Aaron Leventhal <aleventhal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625241}

AppCacheNamespace is only used in AppCache's browser-side
implementation. Having it in content/common/appcache_interfaces.h is
misleading, as it suggests that the class is shared with the renderer.

Bug: 611938
Change-Id: Iffcb77f27408f133c97417663d48d244f2a672bd
Reviewed-on: https://chromium-review.googlesource.com/c/1429246
Auto-Submit: Victor Costan <pwnall@chromium.org>
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625240}

This CL uses a workaraound to enable storing duplicate URLs in the Cache
Storage, until it is fully supported.
The way it works is that it appends the unique_id+request_index to the
URL query, and removes it again when extracted from the cache.

When a request is matched, we search the cache with ignored query mode,
then we recheck that the queries in fact match. This also allows to
store duplicate URLs with different methods.

This CL should be submitted with a second part, that performs a cache
migration from the old format to the new one.

Bug: 871174
Change-Id: Id801361424fbb9464016daba153155c3259b6671
Reviewed-on: https://chromium-review.googlesource.com/c/1414937
Commit-Queue: Rayan Kanso <rayankans@chromium.org>
Reviewed-by: Mugdha Lakhani <nator@google.com>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625239}

The CryptAuthKeyRegistry stores key bundles that are enrolled with
CryptAuth. This storage is persisted as a pref, and retrieved on
construction.

Bug: 899080
Change-Id: Ifad5bb2d4ce38de43420061fb375f4fe43416ed4
Reviewed-on: https://chromium-review.googlesource.com/c/1407257
Commit-Queue: Josh Nohle <nohle@chromium.org>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625238}

The specification (https://w3c.github.io/reporting/#queue-report)
states that the username, password, and fragment of the report URL
should be cleared before queueing a report. We were neglecting to do
this. This CL strips those fields so as to conform to the specification.

Bug: None
Change-Id: I9a04592ee14b3ab953de721b53208cde0cb9be0f
Reviewed-on: https://chromium-review.googlesource.com/c/1427249
Commit-Queue: Lily Chen <chlily@chromium.org>
Reviewed-by: Misha Efimov <mef@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625237}

Interactive doodles don't use the image (data_uri field) included
in the ddljson response. Don't require that responses (on Desktop)
contain an image. Allow logo_cache to cache a doodle that doesn't
include an image.

Bug: 800170
Change-Id: I2161b4b2d33ffdaaa463dd9e59ed2c2f77a9cc1a
Reviewed-on: https://chromium-review.googlesource.com/c/1427564Reviewed-by: Marc Treib <treib@chromium.org>
Commit-Queue: Kyle Milka <kmilka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625236}

TBR=jochen@chromium.org

Bug: 924183
Change-Id: I149ca188f0aa18d7d8ab351460402987808fe8e1
Reviewed-on: https://chromium-review.googlesource.com/c/1430103
Commit-Queue: Thomas Tangl <tangltom@chromium.org>
Reviewed-by: Thomas Tangl <tangltom@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625235}

Remove overflow shelf clip so that inkdrop ripples can propagate
outside the overflow bubble.

The mask isn't really useful in the first place anyway.

See before/after screenshots here:

    https://bugs.chromium.org/p/chromium/issues/detail?id=631240#c14

Bug: 631240
Change-Id: Iaa1d2f4e2e74bff34a708fa9e2cd7be5ed9a52fa
Reviewed-on: https://chromium-review.googlesource.com/c/1429639
Auto-Submit: Manu Cornet <manucornet@chromium.org>
Commit-Queue: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625234}

Added the following browser tests for the Autocomplete feature:
Retention Policy Flag ON:
- SubmitSimpleValue_Saves
- SubmitSimpleValue_OTR_DoesNotSave
- SubmitSimpleValue_Disabled_DoesNotSave
- RetentionPolicy_Init_SavesVersionPref
- RetentionPolicy_RemovesExpiredEntry
- RetentionPolicy_DoesNot_RemoveValidEntry

Retention Policy Flag OFF:
- SubmitSimpleValue_Saves
- SubmitSimpleValue_OTR_DoesNotSave
- SubmitSimpleValue_Disabled_DoesNotSave
- RetentionPolicy_DoesNot_RemoveExpiredEntry

Bug: 920620
Change-Id: Id422c68b06938a78e6c12a9011b520e59cc0e021
Reviewed-on: https://chromium-review.googlesource.com/c/1412503
Commit-Queue: Sebastien Lalancette <seblalancette@chromium.org>
Reviewed-by: Fabio Tirelo <ftirelo@chromium.org>
Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625233}

Currently, navigation predictor code defines two features:
kRecordAnchorMetricsClicked and kRecordAnchorMetricsVisible.
The original plan was to have two features to be used to
independently control two different behaviors of Chrome:
(i) UMA recording of metrics of the clicked anchor element
(ii) UMA recording of metrics of the visible anchor element.
However, we never got around to doing so, and actually used the
same feature to control all the behavior.

This CL merges the two features into a single one to make
code cleaner and simpler to understand. There is no functional
change.

Change-Id: Icedd78c9af50c0f2f76350d20d5af4bb13a7ca0d
Bug: 908725
Reviewed-on: https://chromium-review.googlesource.com/c/1416153
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Doug Arnett <dougarnett@chromium.org>
Reviewed-by: Mark Pearson <mpearson@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625232}

This creates a HttpNegotiateAuthSystem interface which the network
service can implement which calls out to the browser process for token
generation. A NegotiateAuthSystemFactory can be passed through to the
HttpAuthHandlerFactory, which will be used in the Android network
service case to override the default android AuthSystem.

This has been manually tested on Android with the instructions here:
https://chromium.googlesource.com/chromium/src.git/+/lkgr/tools/android/kerberos/README.md
If there is a more robust way of testing, please let me know.

Bug: 922127
Change-Id: Ia858c157d823cafe13bcfcbd97dd19e8a85c6c68
Reviewed-on: https://chromium-review.googlesource.com/c/1413140Reviewed-by: Mike West <mkwst@chromium.org>
Reviewed-by: Asanka Herath <asanka@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Clark DuVall <cduvall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625231}

With the shadowing warning enabled and a sufficiently new clang, there
will be warnings about gfx::Font::ITALIC (and STRIKE and UNDERSCORE)
shadowing gfx::ITALIC, coming from the enum gfx::TextStyle. To prevent
that shadowing, this renames the gfx enum values TEXT_STYLE_*,
including a new name, TEXT_STYLE_COUNT, for what would otherwise be
TEXT_STYLE_NUM_TEXT_STYLES.

This shadowing problem currently prevents content (where the shadowing
warning is already enabled) from using these headers together.

Fixes to other shadowing problems, including enabling the shadow
warning, will come in
https://chromium-review.googlesource.com/c/chromium/src/+/1421104
after this has landed.

Bug: 923078,794619
Change-Id: I3018054f06a2fd2dc04cc4e0e72e5a0d641e99c5
Reviewed-on: https://chromium-review.googlesource.com/c/1424799
Auto-Submit: Daniel Bratell <bratell@opera.com>
Reviewed-by: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Michael Wasserman <msw@chromium.org>
Commit-Queue: Daniel Bratell <bratell@opera.com>
Cr-Commit-Position: refs/heads/master@{#625230}

This CL makes enum CanvasResourceType an enum class, and
extracts the ResourceType fallback list selection logic
out of CanvasResourceProvider::Create() -- so I can add a
DCHECK() to enforce similarity between vectors.

Bug: 920626
Change-Id: Icac251b0c1e7676b4eb0b1c6876a953858920eff
Reviewed-on: https://chromium-review.googlesource.com/c/1423348
Commit-Queue: Miguel Casas <mcasas@chromium.org>
Reviewed-by: Andres Calderon Jaramillo <andrescj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625229}

Add test to ensure that handling the fetch doesn't trigger stale
while revalidate loading.

Add test to ensure that scripts loaded trigger a stale while revalidate
cache hit and resource timing entries are correct.

The PR for the spec changes is here:
https://github.com/whatwg/fetch/pull/853

BUG=348877

Change-Id: Ib07b98d0d2595b6b99857161f830343bf7516518
Reviewed-on: https://chromium-review.googlesource.com/c/1383297
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: Ben Kelly <wanderview@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625228}

This reverts commit 34d69265.

Reason for revert: breaks msan
https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Linux%20MSan%20Tests/14500

Original change's description:
> Don't record first filling result for sign-in forms
> 
> The new form parser may decide that a form has no current-password element, but
> only new-password / confirmation elements. In this case, the parser sets the
> unique_renderer_id to 'undefined'. The credentials will be sent to the renderer
> for filling via manual fallback but the renderer won't ever fill (as there is
> no password element defined for filling.
> 
> This CL ensures that this case does not get recorded as a failure to fill. This
> is important because we record two metrics about the first fill attempt:
> PasswordManager.FirstWaitForUsernameReason and
> PasswordManager.FirstRendererFillingResult For websites that have a sign-up
> form followed by a sign-in form, we don't want to record the failure to fill on
> the sign-up form.
> 
> Bug: 918846
> Change-Id: I79e9a73ae3573a81e121eaa483e0ecef0889184f
> Reviewed-on: https://chromium-review.googlesource.com/c/1422018
> Commit-Queue: Dominic Battré <battre@chromium.org>
> Reviewed-by: Vadym Doroshenko <dvadym@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#625178}

TBR=battre@chromium.org,dvadym@chromium.org

Change-Id: I3c68004b67b5fcc65511c611c2d205297596e782
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 918846
Reviewed-on: https://chromium-review.googlesource.com/c/1430581Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625227}

UX requests menus prefer right, not above.

In reference to the menu's anchor rect:
Above prefers in order:
 - Above and right.
 - Below and left
 - Below and right.

Right prefers in order:
 - Above and right
 - Below and right
 - Below and left.


Bug: 924367
Change-Id: Ida7a24703c97d9a13bc13f387c1ef70416ff8fa7
Reviewed-on: https://chromium-review.googlesource.com/c/1428560
Auto-Submit: Alex Newcomer <newcomer@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Commit-Queue: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625226}

PasswordSyncableService code used two methods in PasswordSyncStore
to retrieve all passwords FillAutofillableLogins() and
FillBlacklistLogins() and then concatenated the results.
This has three issues:
1- This unnecessarily hit the DB twice.
2- Each DB hit involved sorting over some string field that isn't
required.
3-Those calls don't expose the DB primary key of each form which
is required by the new sync architecture.

This CL is replace those APIs in the PasswordSyncStore with a single
API that returns all logins together with the corresponding DB primary
key.

This is in preparation for a later CL that would actually use this API
in the PasswordSyncBridge that belongs to the new sync architecture.

Since the methods FillAutofillableLogins() and FillBlacklistLogins()
are used in the password manager codebase, they have been moved to be
part of the PasswordStore interface rather than the PasswordStoreSync

Bug: 902349
Change-Id: Ic94b83fcc24dfc1833e07653426c064292be5ba7
Reviewed-on: https://chromium-review.googlesource.com/c/1424941Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org>
Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625225}

Bug: 294171
Change-Id: I5d2b506816f7be51ecac2d2b761e516ba8dc350e
Reviewed-on: https://chromium-review.googlesource.com/c/1427020Reviewed-by: Olivier Robin <olivierrobin@chromium.org>
Reviewed-by: Roger McFarlane <rogerm@chromium.org>
Commit-Queue: Jeffrey Cohen <jeffreycohen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625224}

Bug: 923541
Change-Id: Ic5e2dbc0c6053e4bb5f2df792c894f45c8619580
Reviewed-on: https://chromium-review.googlesource.com/c/1426481
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: agrieve <agrieve@chromium.org>
Auto-Submit: Tibor Goldschwendt <tiborg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625223}

https://skia.googlesource.com/skia.git/+log/d562545e8a23..4f81bb73311d


git log d562545e8a23..4f81bb73311d --date=short --no-merges --format='%ad %ae %s'
2019-01-23 reed@google.com remove all support for drawText


Created with:
  gclient setdep -r src/third_party/skia@4f81bb73311d

The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll

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-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;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=djsollen@chromium.org

Change-Id: I66fb6cf8104ba5070db1f2a69d5dac1be14d5087
Reviewed-on: https://chromium-review.googlesource.com/c/1430501Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#625222}

Sync dynamicmodule repository.

Bug: 912127
Change-Id: I738fb1a7f042f72975a4b28e5cafda9c06b23584
Reviewed-on: https://chromium-review.googlesource.com/c/1388155
Commit-Queue: Anna Malova <amalova@chromium.org>
Reviewed-by: Michael van Ouwerkerk <mvanouwerkerk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625221}

As a step to make to use IdentityManager,this CL replaces all uses of ATS
with IdentityManager's ones in chrome_signin_view_controller_unittest.mm.

Bug: 922800
Change-Id: I4d64e98af4ba8d4ca334d4e6d805b309758a9e08
Reviewed-on: https://chromium-review.googlesource.com/c/1429721Reviewed-by: Jérôme Lebel <jlebel@chromium.org>
Commit-Queue: Gyuyoung Kim <gyuyoung@igalia.com>
Cr-Commit-Position: refs/heads/master@{#625220}

Sync visual properties is only called in response to an IPC. The close
IPC drops the IPC route before removing the WebWidget so IPCs can't
arrive that would SynchronizeVisualProperties() after the WebWidget is
gone.

The one exception is the screen metrics emulator which can call it from
its destructor, but it is also destroyed in CloseWebWidget() before the
WebWidget is destroyed precisely to avoid these sorts of things (with
a TODO about how to make that more clear..).

R=avi@chromium.org

Change-Id: I0621f4e3f4a708dd209b996129b9e17a765d6296
Bug: 912193
Reviewed-on: https://chromium-review.googlesource.com/c/1427881Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: danakj <danakj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625219}

Bug: 922713
Change-Id: Ia1cbb340b050878734a5e33d52b7f6adfd6b5f7e
Reviewed-on: https://chromium-review.googlesource.com/c/1428082
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Tommy Nyquist <nyquist@chromium.org>
Auto-Submit: Tibor Goldschwendt <tiborg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625218}

Bug: 916991
Change-Id: Id978bded0e9906a3b6077ff7b00efa612593a70f
Reviewed-on: https://chromium-review.googlesource.com/c/1425726
Commit-Queue: Denis Kuznetsov <antrim@chromium.org>
Reviewed-by: Julian Pastarmov <pastarmovj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625217}

It is a precondition to enable warmer welcome notification.

Bug: b:123077811
Test: Manual
Change-Id: Iad04ba8f231e31aa969aecd4121735c94196dcfa
Reviewed-on: https://chromium-review.googlesource.com/c/1422758Reviewed-by: Tao Wu <wutao@chromium.org>
Commit-Queue: Yilei Li <yileili@google.com>
Cr-Commit-Position: refs/heads/master@{#625216}

Race conditions can lead to abort/pause events after the fetch is
complete, in which case they should be ignored.

This CL also adds an unrelated test for aborting from the UI.

Change-Id: I00a290581653c0243815a39b03340aea8899fb85
Reviewed-on: https://chromium-review.googlesource.com/c/1429987Reviewed-by: Mugdha Lakhani <nator@google.com>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Commit-Queue: Rayan Kanso <rayankans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625215}

height: <percent> sizes of flexbox could be incorrect after flexbox
resize.

Bug: 907911
Change-Id: I19616f7f9cda7cd55a7206b3bb379f4957fccd4c
Reviewed-on: https://chromium-review.googlesource.com/c/1428579
Commit-Queue: Aleks Totic <atotic@chromium.org>
Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625214}

Bug: 910218
Change-Id: Ic27b7153552e8ac6cd9a8ceb67207a4004607b4c
Reviewed-on: https://chromium-review.googlesource.com/c/1424847
Commit-Queue: Zakhar Voit <voit@google.com>
Reviewed-by: Jacob Dufault <jdufault@chromium.org>
Reviewed-by: Sergey Poromov <poromov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625213}

This updates the minimum SDK version to 19 (KitKat), which in practice
deprecates supports for Android Jelly Bean (SDK 16, 17, 18).

//components/cronet still needs to support Jelly Bean, so those
manifests are upgraded from SDK level 14 to 16, referring to the meta
bug for Jelly Bean support in cronet.

The WebAPK minSdkVersion will be updated in a different CL.

Bug: 923477, 922656
Change-Id: Ib1b8b4269d930a8cecb3e45d6ca3ff57f7006d00
Reviewed-on: https://chromium-review.googlesource.com/c/1423117Reviewed-by: Peter Beverloo <peter@chromium.org>
Reviewed-by: Ken Rockot <rockot@google.com>
Reviewed-by: Misha Efimov <mef@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Reviewed-by: Bo <boliu@chromium.org>
Reviewed-by: Joe Downing <joedow@chromium.org>
Reviewed-by: agrieve <agrieve@chromium.org>
Commit-Queue: Tommy Nyquist <nyquist@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625212}