2016-09-15

NECKO WEEKLY MEETING NOTES

Thurs Sept 15, 2016

STICKY

  • Triage duty:
  • new untriaged bugs: https://tinyurl.com/zhk9yjf
  • NECKO-ACTIVE bugs: https://tinyurl.com/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 - cache race of some sort? (honza - thank you!)
  • telemetry & prefability of hpack table size (is it my bug in firefox? is it my bug in node-http2? is it both? WHO KNOWS?!)
  • errors loading recaptcha styles on super-slow connections (may not be our problem)

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

strip path from URLs passed to PAC scripts NEAT - working on libuv and checking things under the hood triaging network id "thinking"

Valentin

last week:

UI to trigger WPT tests automatically: https://presto-wpt.herokuapp.com/

  • Sign up with @mozilla.com email address and confirm account by email 1260527 - open IPDL in HttpChannelChild earlier reviews, triage figuring out DNS leak/use-after-free - bug 1185120

Dragana

Bug 1188435 - Support TCP fastopen

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.