Closed Bug 1405165 Opened 7 years ago Closed 7 years ago

Intermittent leakcheck | default process: 14673 bytes leaked (BackstagePass, CondVar, ConsoleReportCollector, HttpBaseChannel, LoadInfo, ...)

Categories

(Toolkit :: Safe Browsing, defect, P5)

defect

Tracking

()

RESOLVED INCOMPLETE

People

(Reporter: intermittent-bug-filer, Unassigned)

References

Details

(Keywords: intermittent-failure, memory-leak, Whiteboard: [stockwell unknown])

Keywords: mlk
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INCOMPLETE
Status: RESOLVED → REOPENED
Resolution: INCOMPLETE → ---
There are 31 failures in the last 7 days. Platforms: Linux x64, OS X 10.10, Windows 7 and windows10-64 all debug. Recent log failure: https://treeherder.mozilla.org/logviewer.html#?repo=autoland&job_id=157301486 Relevant part of the log: ERROR - TEST-UNEXPECTED-FAIL | leakcheck | default process: 16380 bytes leaked (BackstagePass, CondVar, ConsoleReportCollector, HttpBaseChannel, LoadInfo, ...) 2531 15:10:59 INFO - 2532 15:10:59 INFO - == BloatView: ALL (cumulative) LEAK AND BLOAT STATISTICS, tab process 765 2533 15:10:59 INFO - 2534 15:10:59 INFO - |<----------------Class--------------->|<-----Bytes------>|<----Objects---->| 2535 15:10:59 INFO - | | Per-Inst Leaked| Total Rem| 2536 15:10:59 INFO - 0 |TOTAL | 49 0| 44671 0| 2537 15:10:59 INFO - 2538 15:10:59 INFO - nsTraceRefcnt::DumpStatistics: 781 entries 2539 15:10:59 INFO - 2540 15:10:59 INFO - TEST-PASS | leakcheck | tab process: no leaks detected! 2541 15:10:59 INFO - 2542 15:10:59 INFO - == BloatView: ALL (cumulative) LEAK AND BLOAT STATISTICS, tab process 766 2543 15:10:59 INFO - 2544 15:10:59 INFO - |<----------------Class--------------->|<-----Bytes------>|<----Objects---->| 2545 15:10:59 INFO - | | Per-Inst Leaked| Total Rem| 2546 15:10:59 INFO - 0 |TOTAL | 46 0| 67108 0| 2547 15:10:59 INFO - 2548 15:10:59 INFO - nsTraceRefcnt::DumpStatistics: 865 entries 2549 15:10:59 INFO - 2550 15:10:59 INFO - TEST-PASS | leakcheck | tab process: no leaks detected! 2551 15:10:59 INFO - REFTEST INFO | Removed pending crash reports at '/Users/cltbld/Library/Application Support/firefox/Crash Reports' 2552 15:10:59 INFO - REFTEST INFO | Running with e10s: True 2553 15:10:59 INFO - REFTEST INFO | Application command: /Users/cltbld/tasks/task_1516316654/build/application/Firefox NightlyDebug.app/Contents/MacOS/firefox -marionette -foreground -profile /var/folders/vy/_zxsfnfj6ydc3fdzn9_h6dww00000w/T/tmphEMlhe.mozrunner 2554 15:10:59 INFO - ### XPCOM_MEM_BLOAT_LOG defined -- logging bloat/leaks to /var/folders/vy/_zxsfnfj6ydc3fdzn9_h6dww00000w/T/tmphEMlhe.mozrunner/runreftest_leaks.log 2555 15:11:01 INFO - Unable to read VR Path Registry from /Users/cltbld/Library/Application Support/OpenVR/.openvr/openvrpaths.vrpath 2556 15:11:01 INFO - 1516317061091 Marionette INFO Enabled via --marionette 2557 :selenamarie Could you please take a look?
Flags: needinfo?(sdeckelmann)
Whiteboard: [stockwell needswork]
Component: General → XPConnect
Flags: needinfo?(sdeckelmann)
Whiteboard: [stockwell unknown] → [stockwell unknown] [memShrink]
The failures in the last week are all in debug-test-verify-e10s. I think this is another instance of bug 1432246, due to the presence of URL classifier stuff in the leak. This is either an instance of safe browsing leaking when we shut down the browser too quickly, or maybe that's not expected to work and the test harness is failing to disable safe browsing.
Component: XPConnect → Safe Browsing
Depends on: 1432246
Product: Core → Toolkit
Geoff, we think this is more of an issue with the TV suite (see comment 14 and bug 1432246), do you know who can take a look? Our guess is that we should be disabling safe browsing during testing.
Flags: needinfo?(gbrown)
Whiteboard: [stockwell unknown] [memShrink] → [stockwell unknown]
Thanks Eric. I'll have a preliminary look but may well try to hand this off to someone else. At a glance, I don't see a pref for disabling safe browsing and I don't know the implications of doing that.
Francois - can you help out gbrown?
Flags: needinfo?(francois)
(In reply to Geoff Brown [:gbrown] from comment #17) > Thanks Eric. I'll have a preliminary look but may well try to hand this off > to someone else. At a glance, I don't see a pref for disabling safe browsing > and I don't know the implications of doing that. This is how to disable Safe Browsing entirely: browser.safebrowsing.blockedURIs.enabled = false browser.safebrowsing.malware.enabled = false browser.safebrowsing.phishing.enabled = false plugins.flashBlock.enabled = false plugins.show_infobar = false privacy.trackingprotection.annotate_channels = false privacy.trackingprotection.enabled = false privacy.trackingprotection.pbmode.enabled = false
Flags: needinfo?(francois)
(In reply to François Marier [:francois] from comment #19) Thanks! Most of these are already set for reftests (which seem to be causing all the trouble), but one pref is missing. > This is how to disable Safe Browsing entirely: > > browser.safebrowsing.blockedURIs.enabled = false > browser.safebrowsing.malware.enabled = false > browser.safebrowsing.phishing.enabled = false https://dxr.mozilla.org/mozilla-central/rev/35dfa0882568c06f2d81b6749179815cd4f82886/layout/tools/reftest/reftest-preferences.js#92-94 > plugins.flashBlock.enabled = false https://dxr.mozilla.org/mozilla-central/rev/35dfa0882568c06f2d81b6749179815cd4f82886/layout/tools/reftest/reftest-preferences.js#102 > privacy.trackingprotection.annotate_channels = false > privacy.trackingprotection.enabled = false > privacy.trackingprotection.pbmode.enabled = false https://dxr.mozilla.org/mozilla-central/source/layout/tools/reftest/reftest-preferences.js?q=file%3Areftest-preferences.js&redirect_type=single#103-105 > plugins.show_infobar = false Does not appear to be set. I'll see if setting this helps.
(In reply to François Marier [:francois] from comment #19) > This is how to disable Safe Browsing entirely: > > browser.safebrowsing.blockedURIs.enabled = false > browser.safebrowsing.malware.enabled = false > browser.safebrowsing.phishing.enabled = false > plugins.flashBlock.enabled = false > plugins.show_infobar = false > privacy.trackingprotection.annotate_channels = false > privacy.trackingprotection.enabled = false > privacy.trackingprotection.pbmode.enabled = false I forgot one: browser.safebrowsing.passwords.enabled = false This ones is not necessary, but might be good to add too: browser.safebrowsing.downloads.enabled = false This is where the above prefs are used to enable the various Safe Browsing lists: https://searchfox.org/mozilla-central/rev/7ed2a637dc305e90fed6ffb041c6b8fa162b66bd/toolkit/components/url-classifier/SafeBrowsing.jsm#223-231
(In reply to Geoff Brown [:gbrown] from comment #20) > > plugins.show_infobar = false > > Does not appear to be set. I'll see if setting this helps. Oh, but plugins.show_infobar = false by default anyway: https://dxr.mozilla.org/mozilla-central/rev/35dfa0882568c06f2d81b6749179815cd4f82886/browser/app/profile/firefox.js#708
(In reply to François Marier [:francois] from comment #21) > I forgot one: > > browser.safebrowsing.passwords.enabled = false Got that one already: https://dxr.mozilla.org/mozilla-central/source/layout/tools/reftest/reftest-preferences.js?q=file%3Areftest-preferences.js&redirect_type=single#95 > This ones is not necessary, but might be good to add too: > > browser.safebrowsing.downloads.enabled = false That one is not modified in test prefs. I'll try setting it = false.
I'm making some minor pref changes in bug 1432246; I'm not too optimistic about those being effective, but worth a try. I'll be watching for new failures.
Flags: needinfo?(gbrown)
Status: REOPENED → RESOLVED
Closed: 7 years ago7 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.