[TexMap] Clean up virtual method declarations in GraphicsLayerTextureMapper
Created attachment 240271 [details] Patch
Comment on attachment 240271 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=240271&action=review > Source/WebCore/platform/graphics/texmap/GraphicsLayerTextureMapper.h:35 > +class GraphicsLayerTextureMapper final : public GraphicsLayer, TextureMapperPlatformLayer::Client { Why is TextureMapperPlatformLayer::Client no longer public here?
Comment on attachment 240271 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=240271&action=review >> Source/WebCore/platform/graphics/texmap/GraphicsLayerTextureMapper.h:35 >> +class GraphicsLayerTextureMapper final : public GraphicsLayer, TextureMapperPlatformLayer::Client { > > Why is TextureMapperPlatformLayer::Client no longer public here? There's no need for that -- there are no member variables that we would need access to (or at all) in that class, and the overriding methods in GraphicsLayerTextureMapper are also private.
(In reply to comment #3) > Comment on attachment 240271 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=240271&action=review > > >> Source/WebCore/platform/graphics/texmap/GraphicsLayerTextureMapper.h:35 > >> +class GraphicsLayerTextureMapper final : public GraphicsLayer, TextureMapperPlatformLayer::Client { > > > > Why is TextureMapperPlatformLayer::Client no longer public here? > > There's no need for that -- there are no member variables that we would need > access to (or at all) in that class, and the overriding methods in > GraphicsLayerTextureMapper are also private. Is it common in WebKit to use private inheritance?
Comment on attachment 240271 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=240271&action=review >>>> Source/WebCore/platform/graphics/texmap/GraphicsLayerTextureMapper.h:35 >>>> +class GraphicsLayerTextureMapper final : public GraphicsLayer, TextureMapperPlatformLayer::Client { >>> >>> Why is TextureMapperPlatformLayer::Client no longer public here? >> >> There's no need for that -- there are no member variables that we would need access to (or at all) in that class, and the overriding methods in GraphicsLayerTextureMapper are also private. > > Is it common in WebKit to use private inheritance? I count 8 cases in WebCore, 14 in WebKit2. Not the most common pattern, but not shunned either.
Comment on attachment 240271 [details] Patch Clearing flags on attachment: 240271 Committed r175116: <http://trac.webkit.org/changeset/175116>
All reviewed patches have been landed. Closing bug.