Closed Bug 570779 Opened 14 years ago Closed 14 years ago

Validate enums in Blend* functions

Categories

(Core :: Graphics: CanvasWebGL, defect)

Product:
Core
Component:
Graphics: CanvasWebGL
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: bjacob, Assigned: bjacob)

Details

Attachments

(1 file)

Validate BlendXxx enums
5.07 KB, patch
vlad
: review+
Details | Diff | Splinter Review 
The attached patch fixes:
  BlendEquation
  BlendEquationSeparate
  BlendFunc
  BlendFuncSeparate
Attachment #449907 - Attachment is patch: true
Attachment #449907 - Attachment mime type: application/octet-stream → text/plain
Attachment #449907 - Flags: review?(vladimir)
There's some additional checking that needs to happen here that isn't in the spec yet; will find the details for you shortly.
Comment on attachment 449907 [details] [diff] [review]
Validate BlendXxx enums

I can't find that extra info; it should get added to the spec soon, but let's take this and just do a followup.
Attachment #449907 - Flags: review?(vladimir) → review+
http://hg.mozilla.org/mozilla-central/rev/d152b06227e5
Assignee: nobody → bjacob
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: