[TexMap] Operate with a reference to the TextureMapper wherever possible
Created attachment 267090 [details] Patch Will first run it through the EWS.
Comment on attachment 267090 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=267090&action=review r=me when this passes the EWS. > Source/WebCore/ChangeLog:10 > + TextureMapperLayer member variable which is null until the TextureMapper object Perhaps you could use Optional<TextureMapper&> in this case? > Source/WebCore/platform/graphics/texmap/TextureMapperLayer.cpp:81 > + TextureMapperPaintOptions options(*m_textureMapper); Probably should do ASSERT(m_textureMapper) here.
Comment on attachment 267090 [details] Patch Looks like it passed just fine through the EWS.
Comment on attachment 267090 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=267090&action=review >> Source/WebCore/ChangeLog:10 >> + TextureMapperLayer member variable which is null until the TextureMapper object > > Perhaps you could use Optional<TextureMapper&> in this case? Let's try that. >> Source/WebCore/platform/graphics/texmap/TextureMapperLayer.cpp:81 >> + TextureMapperPaintOptions options(*m_textureMapper); > > Probably should do ASSERT(m_textureMapper) here. OK.
Comment on attachment 267090 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=267090&action=review >>> Source/WebCore/ChangeLog:10 >>> + TextureMapperLayer member variable which is null until the TextureMapper object >> >> Perhaps you could use Optional<TextureMapper&> in this case? > > Let's try that. Optional<> doesn't work for references, and the ownership of the TextureMapper object should be kept in LayerTreeHost/CoordinatedGraphicsScene, so this isn't possible.
Created attachment 267353 [details] Patch for landing
Comment on attachment 267353 [details] Patch for landing Clearing flags on attachment: 267353 Committed r194103: <http://trac.webkit.org/changeset/194103>
All reviewed patches have been landed. Closing bug.