CompositionNotifySink needs a virtual destructor

RESOLVED FIXED in mozilla16

Status

()

Core
Graphics: Layers
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

unspecified
mozilla16
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [build_warning])

Attachments

(1 attachment)

PluginStreamParent.cpp
In file included from /Users/ehsanakhgari/moz/mozilla-central/dom/plugins/ipc/PluginInstanceParent.cpp:7:
In file included from /Users/ehsanakhgari/moz/mozilla-central/dom/plugins/ipc/PluginInstanceParent.h:10:
In file included from ../../../ipc/ipdl/_ipdlheaders/mozilla/plugins/PPluginInstanceParent.h:9:
In file included from ../../../ipc/ipdl/_ipdlheaders/mozilla/plugins/PPluginInstance.h:14:
In file included from ../../../dist/include/IPC/IPCMessageUtils.h:23:
In file included from ../../../dist/include/gfxPattern.h:14:
../../../dist/include/nsAutoPtr.h:71:11: warning: delete called on 'mozilla::layers::CompositionNotifySink' that is abstract but has non-virtual destructor [-Wdelete-non-virtual-dtor]
          delete mRawPtr;
          ^
/Users/ehsanakhgari/moz/mozilla-central/dom/plugins/ipc/PluginInstanceParent.cpp:55:23: note: in instantiation of member function
      'nsAutoPtr<mozilla::layers::CompositionNotifySink>::~nsAutoPtr' requested here
PluginInstanceParent::PluginInstanceParent(PluginModuleParent* parent,
                      ^
Created attachment 634261 [details] [diff] [review]
Patch (v1)
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #634261 - Flags: review?(joe)
Comment on attachment 634261 [details] [diff] [review]
Patch (v1)

r++++ would review again
Attachment #634261 - Flags: review?(joe) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/cc5fee7d85c4
Target Milestone: --- → mozilla16

Comment 4

5 years ago
https://hg.mozilla.org/mozilla-central/rev/cc5fee7d85c4
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.