Closed Bug 1243730 Opened 10 years ago Closed 10 years ago

Remove non needed methods in dom/ipc/ContentParent*

Categories

(Core :: DOM: Core & HTML, defect)

39 Branch
defect
Not set
normal

Tracking

()

RESOLVED INVALID

People

(Reporter: baku, Assigned: baku)

Details

Attachments

(1 file)

Attached patch aa.patchSplinter Review
No description provided.
Attachment #8713168 - Flags: review?(bugs)
Attachment #8713168 - Flags: review?(bugs)
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → INVALID
Comment on attachment 8713168 [details] [diff] [review] aa.patch ># HG changeset patch ># User Andrea Marchesini <amarchesini@mozilla.com> ># Parent 211a4c710fb6af2cad10102c4cabc7cb525998b8 > >diff --git a/dom/ipc/ContentParent.cpp b/dom/ipc/ContentParent.cpp >--- a/dom/ipc/ContentParent.cpp >+++ b/dom/ipc/ContentParent.cpp >@@ -4782,23 +4782,16 @@ ContentParent::CheckAppHasStatus(unsigne > > bool > ContentParent::KillChild() > { > KillHard("KillChild"); > return true; > } > >-PBlobParent* >-ContentParent::SendPBlobConstructor(PBlobParent* aActor, >- const BlobConstructorParams& aParams) >-{ >- return PContentParent::SendPBlobConstructor(aActor, aParams); >-} >- So this compiles everywhere and all? SendPBlobConstructor is something which PContentParent and nsIContentParent both have, but nsIContentParent has it as pure virtual; (we have SendPBlobConstructor also in ContentBridgeParent)
aha, invalid it is
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: