Open Bug 1842111 Opened 1 year ago Updated 1 year ago

Rename CPOW things in ProcessHangMonitor

Categories

(Core :: DOM: Content Processes, task)

task

Tracking

()

People

(Reporter: mccr8, Unassigned)

References

(Blocks 1 open bug)

Details

ProcessHangMonitor has InitiateCPOWTimeout, ShouldTimeOutCPOWs and mCPOWTimeout, which is confusing because we don't have CPOWs any more. It looks like these are still relevant, but maybe a new name would be better? ShouldTimeOutCPOWs is called from ContentParent::ShouldContinueFromReplyTimeout, so maybe they could be renamed to reply timeouts instead of CPOW timeouts.

The comment near the top of ProcessHangMonitor.cpp includes this sentence which presumably should be updated: "The parent cancels any ongoing CPOW requests and then posts a runnable to the main thread that notifies Firefox frontend code of the hang."

You need to log in before you can comment on or make changes to this bug.