Bug 130597

Summary: Windows doesn't build with video disabled
Product: WebKit Reporter: Alex Christensen <alex.christensen>
Component: MediaAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Minor CC: commit-queue, eric.carlson, glenn, jer.noble, philipj, sergio
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Windows 8   
Attachments:
Description Flags
Patch none

Alex Christensen
Reported 2014-03-21 09:08:47 PDT
Here are some compile fixes from WinCairo without video enabled.
Attachments
Patch (5.15 KB, patch)
2014-03-21 09:12 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alex Christensen
Comment 1 2014-03-21 09:12:32 PDT
Created attachment 227442 [details] Patch
Eric Carlson
Comment 2 2014-03-21 11:27:18 PDT
Those missing #if ENABLE(VIDEO)s were my fault, sorry!
WebKit Commit Bot
Comment 3 2014-03-21 12:04:50 PDT
Comment on attachment 227442 [details] Patch Clearing flags on attachment: 227442 Committed r166085: <http://trac.webkit.org/changeset/166085>
WebKit Commit Bot
Comment 4 2014-03-21 12:04:55 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.