Closed Bug 1576995 Opened 2 years ago Closed 2 years ago

[meta] Audit uses of ContentParent:: and ContentChild::mRemoteType


(Core :: DOM: Content Processes, defect, P1)




Fission Milestone M4.1


(Reporter: mccr8, Assigned: mccr8)


(Blocks 1 open bug)


(Keywords: meta)

Kris pointed out that ChromeUtils::RequestProcInfo() ends up returning an uninitialized ProcType if the remoteType is webIsolated=. I'll make that case fail better, fix it for Fission, and also audit the other C++ uses of mRemoteType which may also not be properly handling webIsolated=. Maybe it could be turned into a real data type.

Anyways, the RequestProcInfo() thing is causing widget/tests/browser/browser_test_procinfo.js to fail intermittently.

Depends on: 1577819
Depends on: 1579218

I've now looked at all of the uses of ContentChild::mRemoteType (and the places that value flows out to). Bug 1579218 is the only issue I found.

ContentParent::TryToRecycle() looks at ContentParent::mRemoteType and compares it to DEFAULT_REMOTE_TYPE, but it uses that to determine if it should use the preallocated process, which I assume is just broken completely with Fission so I won't concern myself with it.

Depends on: 1579835
Depends on: 1580212
Fission Milestone: --- → M4.1

I think this is fixed enough.

Closed: 2 years ago
Keywords: meta
Resolution: --- → FIXED
Summary: Audit uses of ContentParent:: and ContentChild::mRemoteType → [meta] Audit uses of ContentParent:: and ContentChild::mRemoteType
You need to log in before you can comment on or make changes to this bug.