1011086 - CSP in C++: Multi Process builds fail in SendReports when there is no LoadGroup

Status: RESOLVED WORKSFORME

(Core :: DOM: Security, defect)

Description

[Christoph Kerschbaumer [:ckerschb]]

If SetReqeustContext is not providing aChannel as an argument, or if the loadGroup of that channel is null, then multi process builds will fail to AsyncOpen, because they cannot query the iTabChild in AsyncOpen. Currently we silently ignore that problem, but we should find out the correct #defines of multiprocess gecko and return with an error code in such cases.

Comment 1

[Sid Stamm [:geekboy or :sstamm]]

Well, we should probably not just "fail early" when we know it'll be a problem, but find a way to get the right loadGroup or other relevant information so the reports actually go out.

Comment 2

[Christoph Kerschbaumer [:ckerschb]]

That's not a problem anymore after fixing Bug 1188028.