Add missing vtable override specifiers under Source/WebCore/html
Created attachment 252314 [details] Patch
Comment on attachment 252314 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=252314&action=review > Source/WebCore/ChangeLog:1 > +2015-05-04 Žan Doberšek <zdobersek@igalia.com> There seems to be some character encoding issue with respect to the characters on this line.
Comment on attachment 252314 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=252314&action=review > Source/WebCore/html/track/TrackListBase.h:54 > + virtual EventTargetInterface eventTargetInterface() const override = 0; There should not be an override specifier here.
Comment on attachment 252314 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=252314&action=review >> Source/WebCore/html/track/TrackListBase.h:54 >> + virtual EventTargetInterface eventTargetInterface() const override = 0; > > There should not be an override specifier here. Never mind. As indicated by the comment on line 53, this is a redeclaration of EventTarget::eventTargetInterface().
(In reply to comment #2) > Comment on attachment 252314 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=252314&action=review > > > Source/WebCore/ChangeLog:1 > > +2015-05-04 Žan Doberšek <zdobersek@igalia.com> > > There seems to be some character encoding issue with respect to the > characters on this line. This appears to be a bug in PrettyPatch.
Comment on attachment 252314 [details] Patch Clearing flags on attachment: 252314 Committed r183858: <http://trac.webkit.org/changeset/183858>
All reviewed patches have been landed. Closing bug.