Skip to content
  • Victor Costan's avatar
    sqlite: Upgrade from 3.20.1 to 3.21.0. · 8a87f7e5
    Victor Costan authored
    Release notes for the new version:
    https://sqlite.org/releaselog/3_21_0.html
    
    Patch 0003 (Modify default VFS to support WebDatabase) was updated to
    reflect conflicts introduced by the following commits:
    * https://www.sqlite.org/src/info/3075cfa07489eaf1
    * https://www.sqlite.org/src/info/1a7e0b61c8a6bdd3
    
    The patch was updated to reflect the rename of pUnused to
    pPreallocatedUnused and an extra assert added. Both changes were in
    src/os_unix.c.
    
    Patch 0004 (Virtual table supporting recovery of corrupted databases)
    was updated to reflect the addition of new compile-time flags to
    main.mk.
    
    Patch 0005 (Custom shell.c helpers to load Chromium's ICU data) was
    updated to reflect conflicts introduced by
    https://www.sqlite.org/src/info/36acc0a97fdcc6f5 which replaced
    src/shell.c with src/shell.c.in, which is used to generate src/shell.c
    via Tcl. The change in the build process also introduced conflicts in
    main.mk.
    
    Patch 0009 (Fix _CRT_RAND_S conflict in sqliteInt.h) was removed
    completely. It is no longer necessary, because
    https://www.sqlite.org/src/info/3a2793aa65727cbb removed the use of
    _CRT_RAND_S in src/sqliteInt.h.
    
    Patches 0012 and 0013 were removed, because they were backports for
    upstream changes that fixed clusterfuzz issues. Upgrading removed the
    need for backports altogether.
    
    The new SQLite release allows ATTACH and DETACH to be used inside
    transactions, so sql/connection_unittest.cc was updated to reflect the
    new behavior. The test for attaching databases was broken into two
    tests, covering the behavior with and without a transaction. The former
    test (covering the behavior with a transaction) has separate
    implementations for iOS, where we use the system SQLite, which might be
    old, and every other platform, where we ship our own SQLite version.
    
    Bug: 780626, 701522
    Change-Id: I7a8378cb317966388ebe9903d9e61162d3b3d3c9
    Reviewed-on: https://chromium-review.googlesource.com/754522
    
    
    Reviewed-by: default avatarChris Mumford <cmumford@chromium.org>
    Commit-Queue: Victor Costan <pwnall@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#515403}
    8a87f7e5