Summary: | Web Inspector: [WebGL] Add a test to catch new WebGL context states parameters | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Andrey Adaikin <aandrey> | ||||
Component: | Web Inspector (Deprecated) | Assignee: | Andrey Adaikin <aandrey> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | apavlov, bweinstein, caseq, eric, joepeck, kbr, keishi, loislo, pfeldman, pmuellr, rik, timothy, webkit.review.bot, yurys | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Bug Depends on: | 94933, 95183 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Andrey Adaikin
2012-08-24 08:45:42 PDT
Created attachment 160716 [details]
Patch
Comment on attachment 160716 [details] Patch Looks fine -- but I don't think this test will catch any new additions to the spec. It's very unlikely that any of these state variables will be removed from future versions of the OpenGL ES (and therefore WebGL) spec. All that is likely to happen is that new state variables will be added -- and this test won't catch those. Probably the only way of doing so would be to enumerate all properties on the WebGLRenderingContext, remove those you know about, and see if there are any left. I think there is already such a test in the WebGL conformance suite; see http://www.khronos.org/webgl/wiki/Testing/Conformance . Comment on attachment 160716 [details]
Patch
Ah, I see what the test is doing now. OK, looks fine.
Comment on attachment 160716 [details] Patch Rejecting attachment 160716 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 1 ERROR: /mnt/git/webkit-commit-queue/LayoutTests/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive). Full output: http://queues.webkit.org/results/13643018 Committed r126856: <http://trac.webkit.org/changeset/126856> |