Bug 145907 – Add the remaining missing override specifiers under Source/WebCore/

• Component: New Bugs
• Status: RESOLVED FIXED

[Zan Dobersek]

Add the remaining missing override specifiers under Source/WebCore/

[Zan Dobersek]

Created attachment 254779 [details]
Patch

[Darin Adler]

Comment on attachment 254779 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=254779&action=review

> Source/WebCore/ChangeLog:13
> +        Where the addition had to be performed on all virtual methods of the
> +        class, the virtual specifier was removed so now only the override
> +        specifier is in use.

Still a bit controversial, but Antti did suggest this and nobody said no!

> Source/WebCore/ChangeLog:22
> +        (WebCore::EqualPowerPanner::reset): Deleted.

I understand that the prepare change log script generated all of these, but please remove them. I don’t think it does anyone any good to list all these functions as deleted in the change log.

[Zan Dobersek]

Comment on attachment 254779 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=254779&action=review

>> Source/WebCore/ChangeLog:22
>> +        (WebCore::EqualPowerPanner::reset): Deleted.
> 
> I understand that the prepare change log script generated all of these, but please remove them. I don’t think it does anyone any good to list all these functions as deleted in the change log.

Will do.

[Zan Dobersek]

Committed r185518: <http://trac.webkit.org/changeset/185518>