Closed Bug 1540289 Opened 5 years ago Closed 5 years ago

62,500 instances of "'!parent'" emitted from netwerk/ipc/NeckoParent.cpp during linux64 debug testing

Categories

(Core :: Networking, defect, P1)

Product:
Core
Component:
Networking
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla68
Tracking Flags:
Tracking Status
firefox68 --- fixed

People

(Reporter: erahm, Assigned: kershaw)

References

(Blocks 1 open bug)

Details

(Whiteboard: [necko-triaged])

Attachments

(1 file)

Bug 1540289 - Don't init socket process bridge if the pref is disabled
47 bytes, text/x-phabricator-request
Details | Review

62516 WARNING: '!parent', file netwerk/ipc/NeckoParent.cpp, line 955

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

  1703 - test-linux64/debug-web-platform-tests-sw-e10s-10 wpt10
  1702 - test-linux64/debug-web-platform-tests-e10s-10 wpt10
  1469 - test-linux64/debug-web-platform-tests-sw-e10s-4 wpt4
  1466 - test-linux64/debug-web-platform-tests-e10s-4 wpt4
  1455 - test-linux64/debug-web-platform-tests-sw-e10s-18 wpt18
  1455 - test-linux64/debug-web-platform-tests-e10s-18 wpt18
  1425 - test-linux64/debug-web-platform-tests-sw-e10s-14 wpt14
  1418 - test-linux64/debug-web-platform-tests-e10s-14 wpt14
  1402 - test-linux64/debug-web-platform-tests-e10s-15 wpt15
  1400 - test-linux64/debug-web-platform-tests-sw-e10s-15 wpt15
  1395 - test-linux64/debug-web-platform-tests-sw-e10s-11 wpt11
  1394 - test-linux64/debug-web-platform-tests-e10s-11 wpt11
  1300 - test-linux64/debug-web-platform-tests-sw-e10s-9 wpt9
  1300 - test-linux64/debug-web-platform-tests-e10s-9 wpt9
  1192 - test-linux64/debug-web-platform-tests-sw-e10s-5 wpt5
  1191 - test-linux64/debug-web-platform-tests-e10s-5 wpt5
  1138 - test-linux64/debug-web-platform-tests-e10s-3 wpt3
  1136 - test-linux64/debug-web-platform-tests-sw-e10s-3 wpt3
  1136 - test-linux64/debug-web-platform-tests-sw-e10s-8 wpt8
  1135 - test-linux64/debug-web-platform-tests-e10s-8 wpt8
  1123 - test-linux64/debug-web-platform-tests-sw-e10s-7 wpt7
  1123 - test-linux64/debug-web-platform-tests-e10s-7 wpt7
  1048 - test-linux64/debug-web-platform-tests-sw-e10s-13 wpt13
  1048 - test-linux64/debug-web-platform-tests-e10s-13 wpt13
  1041 - test-linux64/debug-web-platform-tests-e10s-16 wpt16
  1039 - test-linux64/debug-web-platform-tests-sw-e10s-16 wpt16
   963 - test-linux64/debug-web-platform-tests-e10s-17 wpt17
   962 - test-linux64/debug-web-platform-tests-sw-e10s-17 wpt17
   955 - test-linux64/debug-web-platform-tests-e10s-2 wpt2
   955 - test-linux64/debug-web-platform-tests-sw-e10s-2 wpt2
   944 - test-linux64/debug-web-platform-tests-e10s-1 wpt1
   944 - test-linux64/debug-web-platform-tests-sw-e10s-1 wpt1
   913 - test-linux64/debug-web-platform-tests-sw-e10s-12 wpt12
   912 - test-linux64/debug-web-platform-tests-e10s-12 wpt12
   718 - test-linux64/debug-web-platform-tests-e10s-6 wpt6
   718 - test-linux64/debug-web-platform-tests-sw-e10s-6 wpt6
   527 - test-linux64/debug-web-platform-tests-wdspec-e10s-1 Wd1
   527 - test-linux64/debug-web-platform-tests-wdspec-sw-e10s-1 Wd1
   438 - test-linux64/debug-web-platform-tests-wdspec-sw-e10s-2 Wd2
   438 - test-linux64/debug-web-platform-tests-wdspec-e10s-2 Wd2
   304 - test-linux64/debug-marionette-e10s Mn
   300 - test-linux64/debug-marionette-headless-e10s MnH
   263 - test-linux64/debug-reftest-e10s-1 R1
   261 - test-linux64/debug-reftest-no-accel-e10s-1 Ru1
   261 - test-linux64/debug-reftest-e10s-3 R3
   260 - test-linux64/debug-reftest-e10s-6 R6
   258 - test-linux64/debug-reftest-no-accel-sw-e10s-1 Ru1
   258 - test-linux64/debug-reftest-sw-e10s-6 R6
   257 - test-linux64/debug-reftest-no-accel-e10s-6 Ru6
   255 - test-linux64/debug-reftest-no-accel-sw-e10s-3 Ru3
   254 - test-linux64/debug-web-platform-tests-reftests-e10s-1 Wr1
   253 - test-linux64/debug-reftest-no-accel-sw-e10s-2 Ru2
   253 - test-linux64/debug-reftest-sw-e10s-3 R3
   253 - test-linux64/debug-web-platform-tests-reftests-sw-e10s-1 Wr1
   253 - test-linux64/debug-reftest-no-accel-sw-e10s-6 Ru6
   252 - test-linux64/debug-reftest-sw-e10s-1 R1
   252 - test-linux64/debug-reftest-sw-e10s-4 R4
   252 - test-linux64/debug-reftest-no-accel-e10s-3 Ru3
   252 - test-linux64/debug-reftest-sw-e10s-2 R2
   250 - test-linux64/debug-reftest-no-accel-sw-e10s-4 Ru4
   249 - test-linux64/debug-reftest-no-accel-e10s-2 Ru2
   249 - test-linux64/debug-reftest-e10s-4 R4
   248 - test-linux64/debug-reftest-no-accel-e10s-4 Ru4
   247 - test-linux64/debug-reftest-e10s-2 R2
   247 - test-linux64/debug-reftest-sw-e10s-7 R7
   242 - test-linux64/debug-reftest-no-accel-e10s-7 Ru7
   239 - test-linux64/debug-reftest-e10s-7 R7
   238 - test-linux64/debug-reftest-no-accel-sw-e10s-7 Ru7
   225 - test-linux64/debug-reftest-e10s-5 R5
   222 - test-linux64/debug-reftest-no-accel-e10s-5 Ru5
   219 - test-linux64/debug-reftest-no-accel-sw-e10s-5 Ru5
   218 - test-linux64/debug-web-platform-tests-reftests-sw-e10s-5 Wr5
   218 - test-linux64/debug-reftest-sw-e10s-5 R5
   217 - test-linux64/debug-mochitest-clipboard-e10s cl
   216 - test-linux64/debug-web-platform-tests-reftests-e10s-5 Wr5
   215 - test-linux64/debug-mochitest-clipboard-sw-e10s cl
   213 - test-linux64/debug-web-platform-tests-reftests-e10s-6 Wr6
   211 - test-linux64/debug-web-platform-tests-reftests-sw-e10s-6 Wr6
   209 - test-linux64/debug-web-platform-tests-reftests-e10s-4 Wr4
   209 - test-linux64/debug-web-platform-tests-reftests-sw-e10s-4 Wr4
   204 - test-linux64/debug-web-platform-tests-reftests-e10s-3 Wr3
   204 - test-linux64/debug-web-platform-tests-reftests-sw-e10s-3 Wr3
   195 - test-linux64/debug-web-platform-tests-reftests-e10s-2 Wr2
   191 - test-linux64/debug-web-platform-tests-reftests-sw-e10s-2 Wr2
   182 - test-linux64/debug-mochitest-browser-chrome-e10s-10 bc10
   168 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-6 bc6
   167 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-5 bc5
   166 - test-linux64/debug-mochitest-browser-chrome-e10s-3 bc3
   164 - test-linux64/debug-mochitest-browser-chrome-e10s-9 bc9
   162 - test-linux64/debug-mochitest-browser-chrome-e10s-5 bc5
   157 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-3 bc3
   151 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-11 bc11
   150 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-9 bc9
   147 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-1 bc1
   147 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-7 bc7
   143 - test-linux64/debug-mochitest-browser-chrome-e10s-4 bc4
   141 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-10 bc10
   139 - test-linux64/debug-mochitest-browser-chrome-e10s-6 bc6
   134 - test-linux64/debug-mochitest-browser-chrome-e10s-7 bc7
   133 - test-linux64/debug-mochitest-browser-chrome-e10s-8 bc8
   130 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-4 bc4
   130 - test-linux64/debug-mochitest-browser-chrome-e10s-1 bc1
   129 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-8 bc8
   127 - test-linux64/debug-mochitest-browser-chrome-e10s-2 bc2
   113 - test-linux64/debug-firefox-ui-functional-local-e10s en-US
    92 - test-linux64/debug-mochitest-browser-chrome-e10s-11 bc11
    88 - test-linux64/debug-mochitest-plain-headless-sw-e10s-15 h15
    88 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-2 bc2
    86 - test-linux64/debug-mochitest-plain-headless-e10s-15 h15
    83 - test-linux64/debug-mochitest-sw-e10s-15 15
    78 - test-linux64/debug-mochitest-e10s-15 15
    48 - test-linux64/debug-mochitest-devtools-chrome-e10s-3 dt3
    48 - test-linux64/debug-mochitest-e10s-8 8
    48 - test-linux64/debug-mochitest-sw-e10s-8 8
    47 - test-linux64/debug-mochitest-plain-headless-sw-e10s-5 h5
    47 - test-linux64/debug-mochitest-sw-e10s-5 5
    47 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-2 dt2
    47 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-4 dt4
    47 - test-linux64/debug-mochitest-e10s-5 5
    47 - test-linux64/debug-mochitest-plain-headless-e10s-5 h5
    45 - test-linux64/debug-mochitest-sw-e10s-7 7
    45 - test-linux64/debug-mochitest-devtools-chrome-e10s-2 dt2
    44 - test-linux64/debug-mochitest-sw-e10s-1 1
    44 - test-linux64/debug-mochitest-plain-headless-e10s-11 h11
    44 - test-linux64/debug-mochitest-plain-headless-sw-e10s-11 h11
    44 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-6 dt6
    44 - test-linux64/debug-mochitest-plain-headless-e10s-1 h1
    44 - test-linux64/debug-mochitest-plain-headless-sw-e10s-1 h1
    44 - test-linux64/debug-mochitest-plain-headless-sw-e10s-8 h8
    44 - test-linux64/debug-mochitest-plain-headless-e10s-9 h9
    44 - test-linux64/debug-mochitest-plain-headless-e10s-8 h8
    44 - test-linux64/debug-mochitest-e10s-1 1
    44 - test-linux64/debug-mochitest-plain-headless-sw-e10s-9 h9
    44 - test-linux64/debug-mochitest-devtools-chrome-e10s-6 dt6
    43 - test-linux64/debug-mochitest-devtools-chrome-e10s-4 dt4
    43 - test-linux64/debug-mochitest-plain-headless-sw-e10s-7 h7
    40 - test-linux64/debug-mochitest-e10s-6 6
    40 - test-linux64/debug-mochitest-sw-e10s-10 10
    40 - test-linux64/debug-mochitest-sw-e10s-6 6
    40 - test-linux64/debug-mochitest-e10s-11 11
    40 - test-linux64/debug-mochitest-sw-e10s-14 14
    40 - test-linux64/debug-mochitest-e10s-10 10
    40 - test-linux64/debug-mochitest-plain-headless-e10s-3 h3
    40 - test-linux64/debug-mochitest-e10s-9 9
    40 - test-linux64/debug-mochitest-plain-headless-sw-e10s-6 h6
    40 - test-linux64/debug-mochitest-e10s-3 3
    40 - test-linux64/debug-mochitest-sw-e10s-11 11
    40 - test-linux64/debug-mochitest-e10s-14 14
    40 - test-linux64/debug-mochitest-plain-headless-sw-e10s-3 h3
    40 - test-linux64/debug-mochitest-sw-e10s-9 9
    40 - test-linux64/debug-mochitest-sw-e10s-3 3
    40 - test-linux64/debug-mochitest-plain-headless-e10s-6 h6
    39 - test-linux64/debug-mochitest-devtools-chrome-e10s-5 dt5
    38 - test-linux64/debug-mochitest-e10s-7 7
    38 - test-linux64/debug-mochitest-devtools-chrome-e10s-1 dt1
    38 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-5 dt5
    36 - test-linux64/debug-mochitest-plain-headless-sw-e10s-13 h13
    36 - test-linux64/debug-mochitest-plain-headless-sw-e10s-14 h14
    36 - test-linux64/debug-mochitest-plain-headless-e10s-4 h4
    36 - test-linux64/debug-mochitest-e10s-12 12
    36 - test-linux64/debug-mochitest-plain-headless-e10s-7 h7
    36 - test-linux64/debug-mochitest-plain-headless-sw-e10s-4 h4
    36 - test-linux64/debug-mochitest-sw-e10s-4 4
    36 - test-linux64/debug-mochitest-e10s-16 16
    36 - test-linux64/debug-mochitest-e10s-4 4
    36 - test-linux64/debug-mochitest-plain-headless-sw-e10s-10 h10
    36 - test-linux64/debug-mochitest-plain-headless-e10s-13 h13
    36 - test-linux64/debug-mochitest-sw-e10s-16 16
    36 - test-linux64/debug-mochitest-plain-headless-e10s-14 h14
    36 - test-linux64/debug-mochitest-plain-headless-e10s-10 h10
    36 - test-linux64/debug-mochitest-sw-e10s-12 12
    33 - test-linux64/debug-mochitest-devtools-chrome-e10s-7 dt7
    33 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-7 dt7
    32 - test-linux64/debug-mochitest-plain-headless-sw-e10s-12 h12
    32 - test-linux64/debug-mochitest-sw-e10s-13 13
    32 - test-linux64/debug-mochitest-plain-headless-e10s-16 h16
    32 - test-linux64/debug-mochitest-plain-headless-sw-e10s-16 h16
    32 - test-linux64/debug-mochitest-e10s-13 13
    32 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-8 dt8
    32 - test-linux64/debug-mochitest-plain-headless-e10s-12 h12
    30 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-1 dt1
    28 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-14 bc14
    27 - test-linux64/debug-mochitest-browser-chrome-e10s-14 bc14
    27 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-3 dt3
    25 - test-linux64/debug-mochitest-plain-headless-e10s-2 h2
    25 - test-linux64/debug-mochitest-e10s-2 2
    25 - test-linux64/debug-mochitest-plain-headless-sw-e10s-2 h2
    25 - test-linux64/debug-mochitest-sw-e10s-2 2
    24 - test-linux64/debug-mochitest-media-sw-e10s-3 mda3
    24 - test-linux64/debug-mochitest-browser-chrome-e10s-12 bc12
    24 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-12 bc12
    24 - test-linux64/debug-mochitest-media-e10s-3 mda3
    21 - test-linux64/debug-firefox-ui-functional-remote-e10s en-US
    20 - test-linux64/debug-mochitest-webgl1-core-sw-e10s gl1c
    20 - test-linux64/debug-mochitest-webgl1-core-e10s gl1c
    20 - test-linux64/debug-mochitest-gpu-sw-e10s gpu
    20 - test-linux64/debug-mochitest-devtools-chrome-e10s-8 dt8
    20 - test-linux64/debug-mochitest-gpu-e10s gpu
    19 - test-linux64/debug-telemetry-tests-client-e10s c
    17 - test-linux64/debug-mochitest-chrome-sw-1 c1
    17 - test-linux64/debug-mochitest-chrome-1 c1
    12 - test-linux64/debug-mochitest-browser-chrome-e10s-15 bc15
    12 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-16 bc16
    12 - test-linux64/debug-mochitest-chrome-sw-3 c3
    12 - test-linux64/debug-mochitest-media-sw-e10s-1 mda1
    12 - test-linux64/debug-mochitest-chrome-3 c3
    12 - test-linux64/debug-mochitest-browser-chrome-e10s-16 bc16
    12 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-15 bc15
    12 - test-linux64/debug-mochitest-media-e10s-2 mda2
    12 - test-linux64/debug-mochitest-browser-chrome-e10s-13 bc13
    12 - test-linux64/debug-mochitest-browser-chrome-sw-e10s-13 bc13
    12 - test-linux64/debug-mochitest-devtools-chrome-e10s-11 dt11
    12 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-11 dt11
    12 - test-linux64/debug-mochitest-media-sw-e10s-2 mda2
    12 - test-linux64/debug-mochitest-media-e10s-1 mda1
    11 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-12 dt12
    10 - test-linux64/debug-reftest-e10s-8 R8
    10 - test-linux64/debug-mochitest-devtools-chrome-e10s-13 dt13
    10 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-9 dt9
    10 - test-linux64/debug-reftest-no-accel-sw-e10s-8 Ru8
    10 - test-linux64/debug-mochitest-devtools-chrome-e10s-15 dt15
    10 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-14 dt14
    10 - test-linux64/debug-mochitest-devtools-chrome-e10s-14 dt14
    10 - test-linux64/debug-mochitest-devtools-chrome-e10s-9 dt9
    10 - test-linux64/debug-reftest-sw-e10s-8 R8
    10 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-15 dt15
    10 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-10 dt10
    10 - test-linux64/debug-mochitest-devtools-chrome-e10s-12 dt12
    10 - test-linux64/debug-mochitest-devtools-chrome-e10s-10 dt10
    10 - test-linux64/debug-mochitest-devtools-chrome-e10s-16 dt16
    10 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-13 dt13
    10 - test-linux64/debug-mochitest-devtools-chrome-sw-e10s-16 dt16
     9 - test-linux64/debug-reftest-no-accel-e10s-8 Ru8
     5 - test-linux64/debug-crashtest-sw-e10s C
     5 - test-linux64/debug-crashtest-e10s C
     4 - test-linux64/debug-mochitest-webgl1-ext-sw-e10s gl1e
     4 - test-linux64/debug-mochitest-webgl1-ext-e10s gl1e
     3 - test-linux64/debug-mochitest-chrome-2 c2
     3 - test-linux64/debug-mochitest-chrome-sw-2 c2

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

  4164 - [e10s] Shutdown
    41 - [e10s] js/xpconnect/tests/marionette/test_loader_global_sharing.py TestLoaderGlobalSharing.test_global_sharing_settings
    40 - [e10s] /webdriver/tests/new_window/user_prompts.py
    36 - [e10s] /webdriver/tests/get_element_css_value/user_prompts.py
    36 - [e10s] /webdriver/tests/maximize_window/user_prompts.py
    36 - [e10s] file:///builds/worker/workspace/build/tests/reftest/tests/layout/reftests/reftest-sanity/font-default.html == file:///builds/worker/workspace/build/tests/reftest/tests/layout/reftests/reftest-sanity/font-default.html
    36 - [e10s] /webdriver/tests/get_current_url/user_prompts.py
    36 - [e10s] /webdriver/tests/get_window_handle/user_prompts.py
    36 - [e10s] /webdriver/tests/find_elements/user_prompts.py
    36 - [e10s] /webdriver/tests/get_element_text/user_prompts.py

[1] https://hg.mozilla.org/mozilla-central/annotate/69e9ee0ef3dd/netwerk/ipc/NeckoParent.cpp#l955

This seems to usually be associated with the following warning sequence:

WARNING: '!parent', file netwerk/ipc/NeckoParent.cpp, line 955
Couldn't convert chrome URL: chrome://branding/locale/brand.properties
WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file netwerk/base/nsIOService.cpp, line 942

which seems to imply we're trying to access the socket process for a chrome:// URL, but that might be misleading.

Kershaw, you added this code in bug 1513057, would you mind taking a look?

Blocks: 1513057
Flags: needinfo?(kershaw)

Hi. I would like to work on this. Could you guide me a little?

Flags: needinfo?(erahm)

(In reply to Eric Rahm [:erahm] from comment #1)

This seems to usually be associated with the following warning sequence:

WARNING: '!parent', file netwerk/ipc/NeckoParent.cpp, line 955
Couldn't convert chrome URL: chrome://branding/locale/brand.properties
WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file netwerk/base/nsIOService.cpp, line 942

which seems to imply we're trying to access the socket process for a chrome:// URL, but that might be misleading.

Kershaw, you added this code in bug 1513057, would you mind taking a look?

Since socket process is default disabled, trying to initialize socket process bridge causes this warning message.
I think we should add a pref check in SocketProcessBridgeChild::GetSocketProcessBridge.

Assignee: nobody → kershaw
Flags: needinfo?(kershaw)
Priority: -- → P1
Whiteboard: [necko-triaged]

(In reply to Khyati Agarwal from comment #2)

Hi. I would like to work on this. Could you guide me a little?

Hi Khyati,

Thanks for offering your help. But I am afraid of this bug might need some background knowledge first.
Maybe you can start with some good first bugs [1].

[1] https://wiki.mozilla.org/Good_first_bug

Flags: needinfo?(erahm)
Pushed by kjang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c126bc298380
Don't init socket process bridge if the pref is disabled r=dragana,bwc

https://hg.mozilla.org/mozilla-central/rev/c126bc298380

Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox68: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: