Chrome OWP Storage Weekly

[Joshua Bell]

Storage team (SFO/MTV) weekly updates. The Storage team works on APIs like Indexed DB, Cache API, DOM Storage as well as the underpinnings like leveldb, sqlite, blobs, quota, and related components like FileAPI and DataTransfer.

Triage

• pwnall@ keeps the baton
• many "recharge" items that need snoozing
• DataTransfer untriaged count is creeping up

Servicification 

• AppCache vs. Network Service - michaeln@ 
• Have a CL that makes it fully functional... modulo some bugs
• Working on another CL that refactors existing code to enable it, but not very far along
• Cache Storage - cmumford@
• Getting back to it after wrapping up leveldb memory tracing; have a CL that's been kept fresh
• Local Storage - mek@
• Mojoified local storage hit stable this week - no fires reported \o/

Memory - cmumford@

• Finishing up leveldb memory infra instrumentation; only about a quarter of changes have landed, but everything is up for review; blocked on minor things, will take remainder of this week
• ssid@ looking to work on local storage memory reductions; expect reviews coming our way
  

Cache Storage - cmumford@, kristipark@

• Opaque response padding landed again and stuck \o/ - cmumford@
• jkarlin landed follow on CL to modify padding size passed on network performance experiments
• working through bugs fixes for layout tests of DevTools live updating - kristipark@

Async Cookies - bsittler@, costan@

• Tests - bsittler@
• Have tests working in WPT as a directory full of ".tentative" files
• Discussion about whether a dedicated "tentative" top level directory is the way to go
• Continuing to pursue ".wpt" support for tests in general
• Implementation - costan@
• (no progress this week due to external factors)

Indexed DB - dmurph@, kristipark@

• Identified and have fix in review for blob leak; working on browser test; fix also ensures quota is updated after blob purge - dmurph@
• Next up: enable stats gathering for sweeper (for prioritizing tombstone cleanup work) - dmurph@
• Patch up for live updating in DevTools; in review right now - kristipark@

Miscellaneous

• crc32c third_party library landed in beta; discovered it's crashing on certain ARM CPUs; need to check for presence of both special instructions in use; disabled in 62, will do both checks in 63 - pwnall@
• discussion on namespace/file location for common leveldb stuff that's all currently in the "env"  in third_party/leveldatabase

[Joshua Bell]

Storage team (SFO/MTV) weekly updates. The Storage team works on APIs like Indexed DB, Cache API, DOM Storage as well as the underpinnings like leveldb, sqlite, blobs, quota, and related components like FileAPI and DataTransfer.

Meeting was short and quiet this week due to BlinkOn and other distractions.

Triage

• pwnall@ keeps the baton
• jsbell@ did a pass on monday to drive the numbers down

Indexed DB

• DevTools live updating - kristipark@

• Mostly complete; split work into two CLs since it was getting big
• CL for back end is in review now
• CL for front end - still working on unit tests
• All going to plan per design doc; one unanticipated change was to notifications to support dirty marker for views; needed to split into two notifications, and pass db/store name along in the case a store changed
• Large value wrapping - pwnall@
• Finished up design doc (retroactive)
• IDB v2 Spec Implementation Report - bsittler@, jsbell@
• Test results in multiple browsers needed for spec "exit criteria"; did a pass on tests in Safari TP to enumerate failures

Async Cookies

• Splitting up tests, and making them runnable standalone (where possible) - bsittler@
• Getting back to implementation this week - pwnall@

Miscellaneous

• "Flags API" - jsbell@
• Got implementation back to working, handling detached frames
• Explainer on github, working through comments from Jake

[Joshua Bell]

Triage

• Hand off baton to bsittler
  
• Not a lot of new issues, other than localStorage remove issue (see below)

Async Cookies API

• Working through network service integration - may need to escalate to get unblocked - pwnall@ 
  
• Layout tests snapshot uploaded and running on try bots; mysterious flakes though - bsittler@
• next up: splitting up individual tests - bsittler@
• Optimistic schedule: partial impl in 63 behind a flag, full impl in 64 behind a flag, origin trial in 65?

Indexed DB

• wanting feedback on design doc for value unwrapping - pwnall@
  
• DevTools live updating - reviewers have been out, trying to get this wrapped up this week - kristipark@
• DevTools right click/delete - blocked on previous change - kristipark@

SQLite - pwnall@

• Rolled to new version (3.20.1)
• Worked w/ upstream maintainers to resolve some issues, incorporated patches
• Thanks to cmumford@ for code reviews
• Helping external contributor audit SQLite build config/options

LevelDB - cmumford@

• Landed all but 2 of the memory infra integration changes; one in discussion, the other (iterator memory tracking) seems infeasible (lack of LDB APIs) and may need to just be histograms counting numbers of iterators
• Converting internal use of leveldb into shared library more complex than initially thought; changes in leveldb to support this up for review now

Async Cookies API

• Working through network service integration - may need to escalate to get unblocked - pwnall@ 
  
• Layout tests snapshot uploaded and running on try bots; mysterious flakes though - bsittler@
• next up: splitting up individual tests - bsittler@
  

Local Storage Servicification - mek@

• Bug found in stable with removed items reappearing; glitch in migration code. Fix landed

Blob Mojoification - mek@

• Landing MessagePort OnionSoupification
• Next up: back to Blob Mojoification proper (which depends on above), figure out what's left.

Miscellaneous

• Gathering initial feedback on "locks api" proposal; seem to be converging on API shape - jsbell@
• There was discussion about file system API at BlinkOn. Need to keep momentum going.