Bug 102420

Summary: [TexMap][Cairo][Qt] Refactor BitmapTextureGL::updateContents() .
Product: WebKit Reporter: Viatcheslav Ostapenko <ostap73>
Component: PlatformAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: igor.oliveira, laszlo.gombos, noam, webkit.review.bot
Priority: P3    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch
gyuyoung.kim: review+
Patch for landing with extra line removed. none

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.