[TexMap] Clean up TextureMapperShaderProgram
Created attachment 277983 [details] Patch
Comment on attachment 277983 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=277983&action=review > Source/WebCore/platform/graphics/texmap/TextureMapperGL.cpp:77 > HashMap<TextureMapperShaderProgram::Options, RefPtr<TextureMapperShaderProgram> >::AddResult result = m_programs.add(options, nullptr); > if (result.isNewEntry) > - result.iterator->value = TextureMapperShaderProgram::create(m_context, options); > + result.iterator->value = TextureMapperShaderProgram::create(*m_context, options); > > return result.iterator->value; This will be easier to read using auto for the AddResult and we could probably use ::ensure() instead of ::add()
Comment on attachment 277983 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=277983&action=review >> Source/WebCore/platform/graphics/texmap/TextureMapperGL.cpp:77 >> return result.iterator->value; > > This will be easier to read using auto for the AddResult and we could probably use ::ensure() instead of ::add() This was done in bug #157303.
Created attachment 278068 [details] Patch for landing
Created attachment 278073 [details] Patch for landing
Created attachment 278160 [details] Patch for landing
Comment on attachment 278160 [details] Patch for landing Clearing flags on attachment: 278160 Committed r200451: <http://trac.webkit.org/changeset/200451>
All reviewed patches have been landed. Closing bug.