Bug 43873

Summary: regression in generateMipmap
Product: WebKit Reporter: Nick <computerfreak97>
Component: WebGLAssignee: Zhenyao Mo <zmo>
Status: RESOLVED FIXED    
Severity: Normal CC: cmarrin, dglazkov, kbr, zmo
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac (Intel)   
OS: OS X 10.6   
Bug Depends on: 33805    
Bug Blocks:    
Attachments:
Description Flags
patch dglazkov: review+, zmo: commit-queue-

Nick
Reported 2010-08-11 13:27:39 PDT
There are no textures in the demo at: https://cvs.khronos.org/svn/repos/registry/trunk/public/webgl/sdk/demos/mozilla/spore/index.html. It is confirmed that there are by looking at in in chrome/firefox.
Attachments
patch (14.23 KB, patch)
2010-08-12 14:08 PDT, Zhenyao Mo 		dglazkov: review+
zmo: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Zhenyao Mo
Comment 1 2010-08-12 11:01:20 PDT
It should be a recent regression. My build from this Monday still shows the texture.
Zhenyao Mo
Comment 2 2010-08-12 13:26:27 PDT
I am looking into this.
Zhenyao Mo
Comment 3 2010-08-12 13:31:43 PDT
Ok, it's a bug in the NPOT/Texture completeness patch, so it is before this Monday's build. I was fixing it on Monday; that's why my Monday's build shows the texture correctly. Only I got distracted with other bugs, so I haven't committed the fix. Will upload a patch shortly after.
Zhenyao Mo
Comment 4 2010-08-12 14:03:14 PDT
After generateMipmap() on a POT texture, it should draw successfully with the texture.
Zhenyao Mo
Comment 5 2010-08-12 14:08:59 PDT
Created attachment 64258 [details] patch test is in sync with khronos conformance test, except one failing case. Commented out that failing case for now.
Kenneth Russell
Comment 6 2010-08-12 15:39:31 PDT
Comment on attachment 64258 [details] patch Looks good to me.
Dimitri Glazkov (Google)
Comment 7 2010-08-12 15:51:51 PDT
Comment on attachment 64258 [details] patch ok.
Zhenyao Mo
Comment 8 2010-08-12 15:59:47 PDT
Committed r65277: <http://trac.webkit.org/changeset/65277>
Note You need to log in before you can comment on or make changes to this bug.