Summary: | Added override specifier to PluginViewBase::isPluginViewBase() | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Konstantin Tokarev <annulen> | ||||||
Component: | WebCore Misc. | Assignee: | Konstantin Tokarev <annulen> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | achristensen, commit-queue, mcatanzaro, simon.fraser | ||||||
Priority: | P2 | ||||||||
Version: | WebKit Local Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Konstantin Tokarev
2016-10-10 09:38:51 PDT
Created attachment 291114 [details]
Patch
Comment on attachment 291114 [details]
Patch
Sounds good, but please don't move the declaration to the top.
Move it to bottom or keep in place? I think it's better to separate new virtual APIs from inherited from base class Created attachment 291180 [details]
Patch
Comment on attachment 291180 [details] Patch Clearing flags on attachment: 291180 Committed r207034: <http://trac.webkit.org/changeset/207034> All reviewed patches have been landed. Closing bug. |