2016-09-14

NECKO WEEKLY MEETING NOTES

Thurs Sept 7, 2016

STICKY

  • Triage duty:
  • new untriaged bugs: https://tinyurl.com/zhk9yjf
  • NECKO-ACTIVE bugs: https://tinyurl./j8pj35u
  • NECKO NEXT bugs: https://tinyurl.com/zjwjfee
  • NECKO-BACKLOG bugs: https://tinyurl.com/hpkcxx3

Round Table:

Nick

  • the continuing saga of the predictor leak - PLZ HALP!
  • hpack bug (but apparently no one ever sends a table size change?)
  • telemetry & prefability of hpack table size
  • reviews, reviews, reviews

Honza

Michal

  • 1277826 Crash in shutdownhang | mozilla::net::CacheIndex::WriteLogToDisk
  • 1286209 Intermittent dom/tests/mochitest/ajax/scriptaculous/test_Scriptaculous
  • 1249304 Optimize sorting of CacheIndex::mFrecencyArray
  • 1272587 Add cache API to delete recent entries

Patrick

last week: mpendingq, 256fd telemetry, 0.9 telemetry, namespaces, neat tls, alt-svc, immutable bug, some non-necko stuff this week: neat tls, alt-svc, immutable bug, some non-necko stuff. more reviews.

Daniel

beers with Vallentin in Stockholm NEAT - working on libuv and checking things under the hood triaging network id "thinking"

Valentin

last week:

reviews 1264566 - [e10s] can't delete file after uploading it r?jduell 1231565 - Allow storing alternate data (ex: JS Bytecode) in the HTTP cache next week:

tests for alt-data 1260527 - open IPDL in HttpChannelChild earlier

Dragana

SDT

Bug 1188435 - Support TCP fastopen

Bug 1242597 - reference loop in nsSocketTransport AsyncCopy

NEAT

Gary

[Question]

  • Why |PR_POLL_WRITE| is set when it's readable?
    • https://dxr.mozilla.org/mozilla-central/rev/f5d043ce6d36a3c461cbd829d4a4a38394b7c436/security/nss/lib/ssl/sslsock.c#3018
  • How to implement |PRPollFN| correctly? Return value versus output parameter.

Junior

Bug 1294361 - Firefox translates '%' into "%2525", making displaying of certain files impossible on Gerrit Code Review 2.5.1

*- morphed to Tech Evangelism Bug 268200 - speed up nsStandardURL creation

*- bail-out from canonicalizing ipv4, 1.5% improvement *- cache url objects

Amy

Bug 976073- prevent HTTP responses from setting cookies with secure flag

r-, already modified a new path for the spec of new version. https://tools.ietf.org/html/draft-ietf-httpbis-cookie-alone-01 Bug 1013635 - http-on-response-set-cookie notification should be sync

I have modified dispatch to sync, but I found it still caused the error as below: StorageUI.prototype.onUpdate@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/storage/ui.js:348 Bug 1293067 - (feature request): Implement a cookie limit warning

I have added a notify function when cookie size too big on nsCookieService.cpp, and I will create a warning message on web console.