2017-11-09

NECKO WEEKLY MEETING NOTES

Thurs October 19th, 2017


TRIAGE

  • Note: use these 2 queries when you're on triage duty: all bugs should be [necko-triaged] and have a priority

    • UNTRIAGED bugs: https://mzl.la/2wAv7XZ
    • MALFORMED bugs (have necko-triaged but no priority): https://mzl.la/2xXf2jc
  • P1 bugs https://mzl.la/2he1C85

  • P2 bugs https://mzl.la/2yaD1aO

  • P3 bugs (backlog) https://mzl.la/2xcYzEL

  • P5 bugs (would take): https://mzl.la/2hdnQXy

  • Triage Notes

    • Android crashes on infra are currently not symbolicated: https://bugzilla.mozilla.org/show_bug.cgi?id=1389805 (makes triaging/diagnosing android crash bugs hard/impossible)

Nick

Gary <<<<<===== Nov 09 -> 16

Honza

Junior

Michal

Amy

Daniel

Kershaw

Valentin

SC

Dragana


Round Table:


Nick

Honza

Michal

  • profiling RCWN
  • 1395760 Page gets cached only partially (possibly generating a corrupted cache entry?)
  • 1397709 Force refresh not propogated into data or (perma caching)
  • 1395403 Receiving large binary websocket messages consumes too much CPU
  • 1399822 WebSocket onopen event is fired when unexpected Sec-WebSocket-Protocol is received
  • 1352916 Crash in mozilla::net::CacheFileMetadata::GetElement
  • 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

    • TLS 1.3-checking (to be system-) add-on (help from Kershaw)
    • TRR (Trusted Recursive Resolver) - DNS-over-HTTPS

    Valentin

    Bug 1405961 - Assertion failure: !aStamp.IsNull() PerformanceNavigationTiming

    Bug 1394906 - Implement immutable, threadsafe MozURL

    Bug 1407540 - MozURL should support base url in the CTOR

    reviews

    MozURL wrapper implementing nsIURI

    RCWN debugging

    Dragana

    103 Early Hint

    security bug

    NEAT

    TFO

    Gary

    Bug 1373708 - Convert Jar channels to stop using main thread I/O

    • blocked by Bug 1407547 - FAIL | browser/base/content/test/urlbar/browser_urlbarAboutHomeLoading.js | Uncaught exception

      • see if it has been fixed by latest m-c Bug 1399520 - Intermittent AddressSanitizer: heap-use-after-free
    • sec-approval? Bug 1391356 - Convert JAR channel clients that use Open to use AsyncOpen

    • open sub-bugs, see if they are necessary to be fixed.

    Junior

    Bug 870460 - Lazy load of cookie service blocks main thread while cookie database loads

    • backout: xperf startup io whitelist(r+), also worried by perf team, and BaseAppShell leakage(r?) Bug 1401883 - Intermittent leakcheck | default process: 176 bytes leaked (Mutex, nsBaseAppShell)

    Bug 1400655 - json view serviceworker

    • LOAD_CALL_CONTENT_SNIFFERS is gone in Child with InterceptedChannel

    Amy

    Bug 1344090 - Firefox's cookie limitations does not allow to store 150 cookies per-domain , but only for the root domain, it is confusing.

    • Conclusion:

    • Stop to implement the cookie priorities.

    • Start to implement the batch deletion.

    • Increase the limitation of cookie from per-host. Bug 1328163 - support protocol-agnostic cookie whitelisting

    • Rebased. Bug 1409102 - 57beta: Accept 3rd-party cookies from visited doesn't seem to work with embedded dailymotion videos..

    • Can't reproduce.

    Kershaw

    Bug 1404804 - Tracking black list is checked twice when TP is enabled

    • landed

    Bug 1390223 - Experiment: Don't mark dynamically added css (aka elements) as Leaders

    • WIP for determining if a CSS link element is created from parser or script
    • Need to create some fake pages for analyzing

    TLS-1.3 addon

    • Make the popup window work

    SC

    [PBg-Http]

    • video/audio loading is now OMT as well (thanks to Taipei Media team!)

    • https://mzl.la/2kQBQvz

    • https://mzl.la/2kRVxD7

    • debugging for regressions

    • Bug 1405818 - Crash in mozilla::net::HttpChannelChild::Release

    • guessing and discussing possible scenario that causes multi-thread refcount issue

    • Bug 1409553 - Assertion failure: redirectParentChannel, at netwerk/protocol/http/HttpChannelParent.cpp:1035

    • yet another ServiceWorker + HTTP redirection bug.

    • reproduce with RR with moz log, still analyzing the root cause [Socket Process]

    • study Google Chrome approach

    • preparing system design and rollout plan

    • [under construction] https://docs.google.com/presentation/d/12Jg2eXT7nne2G8MtS3E2p7yRGaQJrvXg8iiQOhYggFU/edit#slide=id.g28b523e674_0_82 [Others]

    • Bug 1407683 - Crash after failed nsIXMLHttpRequest instance

    • use XPCOM singleton factory method and StaticRefPtr to replace static raw pointer. landed on m-c

    • Bug 1390708 - .hqx files are all corrupted when downloaded with FF unless going through SAVE AS...

    • BinHex format will be decompressed by stream converter when opening the URL, and save the decoded content with .hqx filename

    • intend to remove BinHex stream converter support from m-c, under discussion