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)
Tracking
()
RESOLVED
INVALID
People
(Reporter: baku, Assigned: baku)
Details
Attachments
(1 file)
|
2.23 KB,
patch
|
Details | Diff | Splinter Review |
No description provided.
Attachment #8713168 -
Flags: review?(bugs)
| Assignee | ||
Updated•10 years ago
|
Attachment #8713168 -
Flags: review?(bugs)
| Assignee | ||
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → INVALID
Comment 1•10 years ago
|
||
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)
Comment 2•10 years ago
|
||
aha, invalid it is
Updated•7 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•