[Webkit-unassigned] [Bug 75779] [Texmap] Move surface management from TextureMapperNode to TextureMapper

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=75779





--- Comment #3 from Noam Rosenthal <noam.rosenthal at nokia.com>  2012-01-07 19:30:12 PST ---
(In reply to comment #2)
> (From update of attachment 121562 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=121562&action=review
> 
> > Source/WebCore/platform/graphics/texmap/TextureMapper.cpp:45
> > +
> 
> if i understood correctly, if (!newTexture) will be true just in the first loop iteration, so we could move this if out of for loop.
> Doing something like:
> 
> RefPtr<BitmapTexture> newTexture = m_texturePool[0];
> int index = 0;
> for (size_t i = 1 ....)
> 
> what do you think?

Yes, sounds less yucky. Will revise :)

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.