RESOLVED FIXED 102420
[TexMap][Cairo][Qt] Refactor BitmapTextureGL::updateContents() . 
https://bugs.webkit.org/show_bug.cgi?id=102420
Summary [TexMap][Cairo][Qt] Refactor BitmapTextureGL::updateContents() .
Viatcheslav Ostapenko
Reported 2012-11-15 12:56:38 PST
Simplify a BitmapTextureGL::updateContents().
Attachments
patch (8.40 KB, patch)
2012-11-15 13:16 PST, Viatcheslav Ostapenko 		gyuyoung.kim: review+
DetailsFormatted DiffDiff
Patch for landing with extra line removed. (8.40 KB, patch)
2012-11-15 19:46 PST, Viatcheslav Ostapenko 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Viatcheslav Ostapenko
Comment 1 2012-11-15 13:16:30 PST
Created attachment 174505 [details] patch
Gyuyoung Kim
Comment 2 2012-11-15 18:37:37 PST
Comment on attachment 174505 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=174505&action=review Looks make sense. > Source/WebCore/platform/graphics/texmap/TextureMapperGL.cpp:694 > + Unneeded line.
Viatcheslav Ostapenko
Comment 3 2012-11-15 19:46:58 PST
Created attachment 174595 [details] Patch for landing with extra line removed.
WebKit Review Bot
Comment 4 2012-11-15 20:32:07 PST
Comment on attachment 174595 [details] Patch for landing with extra line removed. Clearing flags on attachment: 174595 Committed r134887: <http://trac.webkit.org/changeset/134887>
WebKit Review Bot
Comment 5 2012-11-15 20:32:11 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.