Open Bug 1861255 Opened 2 years ago Updated 2 years ago

GetHyphDictFromParent() and related IPC have no code coverage

Categories

(Core :: Layout: Text and Fonts, task)

Product:
Core
Component:
Layout: Text and Fonts
Type:
task

Tracking

()

People

(Reporter: mccr8, Unassigned, NeedInfo)

References

(Blocks 1 open bug)

Details

There's some kind of mechanism starting with GetHyphDictFromParent() and involving ContentParent::RecvGetHyphDict() that has no code coverage. Is this mechanism still needed? Can we add tests for it? I looked over the code and didn't see anything obviously dangerous but maybe I missed something.

IIRC, this may be dependent on how things are packaged for the particular platform; perhaps Android-only, but I'll need to refresh my memory of how things work there. Setting ni? to put it on my radar to have a look....

Flags: needinfo?(jfkthame)
Keywords: sec-audit

Yeah, I do see Android crashes when I look for crashes where the proto signature contains RecvGetHyphDict.

It is expected that there's no desktop coverage because this is for Android, probably, so I'll unhide this. It might be nice to block these messages on Desktop if we really don't need it there, just to be defensive.

Group: layout-core-security
Keywords: sec-audit
You need to log in before you can comment on or make changes to this bug.