Open Bug 1627997 Opened 5 years ago Updated 5 years ago

Typing page-icon urls in the url bar shouldn't load places in content processes.

Categories

(Toolkit :: Places, defect, P3)

defect

Tracking

()

People

(Reporter: emilio, Unassigned)

References

Details

Right now if you type a page-icon: url in the urlbar, in a debug build, you get a content process crash hitting this assertion, when getting the favicon service in PageIconProtocolHandler.cpp.

We should either route the call through the parent process somehow (which may not be too hard), or somehow show the unknown-protocol error, but returning an error in PageIconProtocolHandler::NewChannel just leaves the tab spinning, which is not great.

Priority: -- → P3

Because this bug's Severity has not been changed from the default since it was filed, and it's Priority is P3 (Backlog,) indicating it has been triaged, the bug's Severity is being updated to S3 (normal.)

Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.