[chromium] Make WebFrameImpl destructor virtual
Created attachment 95984 [details] Patch
Comment on attachment 95984 [details] Patch WebFrame.h declares ~WebFrame() { }, so it should declare virtual there as well (of course you also need to declare virtual here).
(In reply to comment #2) > (From update of attachment 95984 [details]) > WebFrame.h declares ~WebFrame() { }, so it should declare virtual there as well (of course you also need to declare virtual here). Again, why? Nobody deletes a WebFrame directly, and its destructor is protected, so it won't happen accidentally.
Comment on attachment 95984 [details] Patch Clearing flags on attachment: 95984 Committed r88105: <http://trac.webkit.org/changeset/88105>
All reviewed patches have been landed. Closing bug.