Closed Bug 1321052 Opened 5 years ago Closed 5 years ago

Convert ProcessHangMonitor to use endpoints

Categories

(Core :: DOM: Content Processes, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: mccr8, Assigned: mccr8)

References

Details

Attachments

(1 file)

No description provided.
Comment on attachment 8824252 [details]
Bug 1321052 - Convert ProcessHangMonitor to use endpoints.

https://reviewboard.mozilla.org/r/102768/#review104316

::: dom/ipc/ProcessHangMonitorIPC.h:24
(Diff revision 1)
>  namespace dom {
>  class ContentParent;
>  } // namespace dom
>  
>  PProcessHangMonitorParent*
> -CreateHangMonitorParent(mozilla::dom::ContentParent* aContentParent,
> +CreateHangMonitorParent(dom::ContentParent* aContentParent,

This no longer needs to be exposed in the header file.
Attachment #8824252 - Flags: review?(wmccloskey) → review+
(In reply to Bill McCloskey (:billm) from comment #2)
> This no longer needs to be exposed in the header file.

Good catch. Thanks for the review.
Pushed by amccreight@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6555c8f64155
Convert ProcessHangMonitor to use endpoints. r=billm
https://hg.mozilla.org/mozilla-central/rev/6555c8f64155
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.