2017-02-16
NECKO WEEKLY MEETING NOTES
Thurs Feb 16, 2017
STICKY
- Triage duty: dragana
- 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:
- Necko Taipei meeting will be April 17-21 (17th is travel/jetlag day)
- Brown bag talk to Taipei office?
---PROJECT UPDATES--- - rust-url meeting notes: https://public.etherpad-mozilla.org/p/mozloha-rust-url - rcwn https://public.etherpad-mozilla.org/p/mozaloha-rcwn - service workers https://public.etherpad-mozilla.org/p/mozaloha-sw-necko
QUIC (Pat, Dragana, Daniel)
Race-Cache-with-Network (Michal, Valentin, Junior)
- Landed telemetry for time load difference
- Racing code is reviewed, and ready to land.
- Next: racing logic + improvement telemetry
Context-Driven Priority (Honza, Kershaw, Amy, Gary, Nick)
- maybe we should give this a read? http://web.mit.edu/ravinet/www/polaris_nsdi16.pdf
PBackgroundHttp (SC, NIck)
Nick
- channel throttling landed
- h2 early-data close (i hope!)
- h2 ORIGIN
- PTO next week
Honza
Michal
- RCWN
- 1308683 Intermittent /websockets/constructor/014.html
- 1338973 Crash in nsCOMArray_base::InsertObjectsAt
- 1335440 A second Secure WebSocket connection to the same localhost server fails
- 1300672 Websocket connection fails
- 1330277 Crash in mozilla::net::CacheFileChunkBuffer::SetDataSize
- 1325336 Generate statistics for cache performance on the given HW
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
NEAT
Valentin
[landed]
Bug 1325081 - Make nsHttpChannel able to do network and cache requests at the same time
Bug 1320088 - [Captive Portal] Detection fails when FF is open prior to engaging Wifi
Bug 1275746 - Don't allow empty hostname for URLTYPE_AUTHORITY urls
Bug 1339809 - Update rust-url to version 1.4.0
Bug 1337629 - Add more restrictions to the host parser - *[]<> not allowed in hostname anymore
[working on]
Bug 945240 - Consider making Location.href punycode non-ASCII hostnames
several resource-timing + web-patform-test bugs
Dragana
SDT
Some security bugs
Bug 1334776 - Header name interning leaks across origins
Bug 1337826 - V52 beta 4 under Windows 7 crashes once per hour, was once a week before
Bug 1340164 - Fix socket transport status events
Gary
- Bug 1321841 - Firefox network loss after connection to Cisco VPN.
- Unified Presto
- Provide this service to Mozillians
- Cowork with Hasal team
Junior
(land)Bug 1325325 - Check HTTP_NET_VS_CACHE_*_REVALIDATED probes for correctness
(land)Bug 1331324 - Prevent network load time of custom conditional request from being reported
(land)Bug 1325090 - Reduce number of HTTP_NET_VS_CACHE_ probes
-
drop the non-significant probes and merge the similar probes (f-)Bug 1325322 - Change HTTP_NET_VS_CACHE_ probes to cover bigger time span
-
resolution should be better. hope I can cook another patch for feedback today (Investigation)Bug 1325088 - Add net-response-time-onstart/onstop to cache index
-
study cache index
-
michal sends a note of cache index (r?)Bug 482934 - Response to a non-necko-initiated conditional request should be cached
-
leave previous implementation as a reference, and use solution provided by honza (open the cache entry as connect and stop writing for 304 case)
Amy
Bug 1325909 - Add a new bucket and modify the bucket name on telemetry COOKIE_LEAVE_SECURE_ALONE.
-
Modify by Daniel’s suggestions. Bug 1328163 - support protocol-agnostic cookie whitelisting
-
Finished:implement part.
-
Not finish:test cases. Bug 1293067 - (feature request): Implement a cookie limit warning
-
r? part one: implement.
-
r? part two: test cases. Bug 1312774 - Allow channels marked as "urgentStart" to bypass connection limits
-
f? Honda.
Kershaw
Bug 1326339 - Let nsHttpConnectionMgr be aware of the active tab
- f+
- Approach approved by bz
Bug 1309653 - Give Necko the right notifications on active tab change and navigation start
- WIP
Bug 1319359 - Not to block the throbber if the active requests in load group are all tracking requests
- Wait for the final decision from Honza
Bug 469228 - Support keep-alive connections in httpd.js
- Study Honza's patch
SC
[HttpChannel]
- PBackground-ify HttpChannel IPC
- Bug 1325915 - remove invocation order dependency on OnProgress and OnDataAvailable in HttpChannelParent (waiting for review)
- update patch according to review comment, waiting for honza's review for my v2 patch
- Bug 1320744 - HttpChannelChild should implement nsIThreadRetargetableRequest
- update patch according to review comment
- fixing thread-safety issue found on try server, not all attributes in nsHttpChannel/HttpBaseChannel are thread-safe for Release()