Closed Bug 1582385 Opened 5 years ago Closed 6 months ago

4,100 instances of "'NS_FAILED(rv)'" emitted from dom/quota/ActorsParent.cpp during linux64 debug testing

Categories

(Core :: Storage: Quota Manager, defect, P2)

Product:
Core
Component:
Storage: Quota Manager
Type:
defect

Tracking

()

Status:
RESOLVED WORKSFORME

People

(Reporter: erahm, Unassigned)

References

(Blocks 1 open bug)

Details

4139 WARNING: 'NS_FAILED(rv)', file dom/quota/ActorsParent.cpp, line 8008

This warning [1] shows up in the following test suites:

  1133 - test-linux64/debug-mochitest-e10s-2 2
   263 - test-linux64/debug-mochitest-fis-e10s-2 2
   263 - test-linux64/debug-mochitest-sw-e10s-2 2
   197 - test-linux64/debug-mochitest-e10s-12 12
   197 - test-linux64/debug-mochitest-fis-e10s-12 12
   197 - test-linux64/debug-mochitest-sw-e10s-12 12
   189 - test-linux64/debug-mochitest-e10s-5 5
   185 - test-linux64/debug-mochitest-fis-e10s-5 5
   182 - test-linux64/debug-mochitest-sw-e10s-5 5
   110 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-10 bc10
    97 - test-linux64/debug-mochitest-devtools-chrome-fis-e10s-10 dt10
    97 - test-linux64/debug-web-platform-tests-sw-e10s-17 wpt17
    92 - test-linux64/debug-mochitest-browser-chrome-e10s-9 bc9
    81 - test-linux64/debug-mochitest-devtools-chrome-e10s-10 dt10
    81 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-10 dt10
    71 - test-linux64/debug-web-platform-tests-e10s-17 wpt17
    58 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-1 bc1
    54 - test-linux64/debug-mochitest-browser-chrome-e10s-8 bc8
    53 - test-linux64/debug-mochitest-browser-chrome-fis-e10s-2 bc2
    40 - test-linux64/debug-mochitest-browser-chrome-e10s-16 bc16
    40 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-16 bc16
    39 - test-linux64/debug-web-platform-tests-sw-e10s-12 wpt12
    39 - test-linux64/debug-web-platform-tests-e10s-12 wpt12
    36 - test-linux64/debug-mochitest-browser-chrome-fis-e10s-7 bc7
    36 - test-linux64/debug-mochitest-browser-chrome-fis-e10s-16 bc16
    34 - test-linux64/debug-mochitest-browser-chrome-e10s-4 bc4
    28 - test-linux64/debug-mochitest-e10s-3 3
    28 - test-linux64/debug-mochitest-sw-e10s-3 3
    27 - test-linux64/debug-mochitest-fis-e10s-3 3
    23 - test-linux64/debug-web-platform-tests-e10s-10 wpt10
    23 - test-linux64/debug-web-platform-tests-sw-e10s-10 wpt10
    19 - test-linux64/debug-mochitest-browser-chrome-e10s-6 bc6
    17 - test-linux64/debug-mochitest-devtools-chrome-fis-e10s-2 dt2
    14 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-2 bc2
    13 - test-linux64/debug-mochitest-fis-e10s-10 10
    10 - test-linux64/debug-mochitest-fis-e10s-15 15
    10 - test-linux64/debug-mochitest-sw-e10s-15 15
    10 - test-linux64/debug-mochitest-e10s-15 15
     8 - test-linux64/debug-mochitest-fis-e10s-9 9
     8 - test-linux64/debug-mochitest-sw-e10s-10 10
     8 - test-linux64/debug-mochitest-e10s-10 10
     6 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-6 bc6
     3 - test-linux64/debug-web-platform-tests-sw-e10s-7 wpt7
     3 - test-linux64/debug-web-platform-tests-e10s-7 wpt7
     2 - test-linux64/debug-crashtest-sw-e10s C
     2 - test-linux64/debug-mochitest-browser-chrome-e10s-13 bc13
     2 - test-linux64/debug-crashtest-e10s C
     2 - test-linux64/debug-mochitest-browser-chrome-fis-e10s-9 bc9
     2 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-13 bc13
     1 - test-linux64/debug-mochitest-e10s-14 14
     1 - test-linux64/debug-mochitest-browser-chrome-fis-e10s-1 bc1
     1 - test-linux64/debug-mochitest-browser-chrome-e10s-10 bc10
     1 - test-linux64/debug-mochitest-browser-chrome-fis-e10s-4 bc4
     1 - test-linux64/debug-mochitest-fis-e10s-14 14
     1 - test-linux64/debug-mochitest-sw-e10s-14 14
     1 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-11 bc11

It shows up in 534 tests. A few of the most prevalent:

   168 - [e10s] dom/base/test/test_x-frame-options.html
    67 - [e10s] browser/base/content/test/sanitize/browser_cookiePermission.js
    66 - [e10s] browser/base/content/test/sanitize/browser_cookiePermission_containers.js
    50 - [e10s] browser/components/preferences/in-content/tests/siteData/browser_siteData2.js
    40 - [e10s] toolkit/components/antitracking/test/browser/browser_siteSpecificWorkArounds.js
    36 - [e10s] dom/base/test/test_setting_opener.html
    30 - [e10s] browser/components/extensions/test/browser/browser_ext_tabs_saveAsPDF.js
    29 - [e10s] toolkit/components/antitracking/test/browser/browser_blockingDOMCache.js
    28 - [e10s] dom/base/test/test_img_referrer.html
    26 - [e10s] toolkit/components/antitracking/test/browser/browser_partitionedLocalStorage.js

[1] https://hg.mozilla.org/mozilla-central/annotate/a3a081ae714f1123bdc23c9d9ef53dfaa783a8de/dom/quota/ActorsParent.cpp#l8008

Yeah, we should fix this.

Priority: -- → P2
Depends on: 1686910
Severity: normal → S3

I don't see this specific warning any more in the top 100.

Status: NEW → RESOLVED
Closed: 6 months ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.