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)
Toolkit
Safe Browsing
Tracking
()
RESOLVED
INCOMPLETE
People
(Reporter: intermittent-bug-filer, Unassigned)
References
Details
(Keywords: intermittent-failure, memory-leak, Whiteboard: [stockwell unknown])
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 3•7 years ago
|
||
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INCOMPLETE
Updated•7 years ago
|
Status: RESOLVED → REOPENED
Resolution: INCOMPLETE → ---
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 11•7 years ago
|
||
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]
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Updated•7 years ago
|
Component: General → XPConnect
Flags: needinfo?(sdeckelmann)
Whiteboard: [stockwell unknown] → [stockwell unknown] [memShrink]
Comment 14•7 years ago
|
||
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.
Comment hidden (Intermittent Failures Robot) |
Comment 16•7 years ago
|
||
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]
Comment 17•7 years ago
|
||
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.
Comment 19•7 years ago
|
||
(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)
Comment 20•7 years ago
|
||
(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.
Comment 21•7 years ago
|
||
(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
Comment 22•7 years ago
|
||
(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
Comment 23•7 years ago
|
||
(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.
Comment 24•7 years ago
|
||
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)
Comment hidden (Intermittent Failures Robot) |
Comment 26•7 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → INCOMPLETE
You need to log in
before you can comment on or make changes to this bug.
Description
•