Bug 80843

Summary: [TexMap] Reuse textures following the same rules as they do internally.
Product: WebKit Reporter: Jocelyn Turcotte <jturcotte>
Component: New BugsAssignee: Jocelyn Turcotte <jturcotte>
Status: RESOLVED FIXED    
Severity: Normal CC: noam
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch noam: review+

Jocelyn Turcotte
Reported 2012-03-12 08:34:28 PDT
[TexMap] Remove the texture pool.
Attachments
Patch (5.88 KB, patch)
2012-03-12 08:35 PDT, Jocelyn Turcotte 		no flags
DetailsFormatted DiffDiff
Patch (5.65 KB, patch)
2012-03-15 13:25 PDT, Jocelyn Turcotte 		noam: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Jocelyn Turcotte
Comment 1 2012-03-12 08:35:55 PDT
Created attachment 131328 [details] Patch
Noam Rosenthal
Comment 2 2012-03-15 09:17:24 PDT
Comment on attachment 131328 [details] Patch When drawing the same scene over and over again, you'd have intermediate surfaces with the same size. Thus, the texture-pool can be really helpful. Not sure about the benefit of this patch.
Jocelyn Turcotte
Comment 3 2012-03-15 13:21:11 PDT
I assumed that creating textures or is negligible but it's different for surfaces as they can get recreated for every rendered frames, along with their FBO. I'm uploading a patch that fixes the logic for texture reuse instead and keep the behavior.
Jocelyn Turcotte
Comment 4 2012-03-15 13:25:06 PDT
Created attachment 132109 [details] Patch
Jocelyn Turcotte
Comment 5 2012-03-16 03:22:13 PDT
Committed r110979: <http://trac.webkit.org/changeset/110979>
Note You need to log in before you can comment on or make changes to this bug.