Bug 65552 - Move WTF_USE_AVFOUNDATION from JavaScriptCore/wtf/platform.h to WebCore/config.h
Summary: Move WTF_USE_AVFOUNDATION from JavaScriptCore/wtf/platform.h to WebCore/config.h
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: 528+ (Nightly build)
Hardware: Macintosh All
: P2 Normal
Assignee: Jeff Miller
URL:
Keywords: InRadar
Depends on:
Blocks: 65400
  Show dependency treegraph
 
Reported: 2011-08-02 11:08 PDT by Jeff Miller
Modified: 2011-08-04 13:12 PDT (History)
0 users

See Also:


Attachments
Patch (2.34 KB, patch)
2011-08-02 11:18 PDT, Jeff Miller
aroben: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jeff Miller 2011-08-02 11:08:55 PDT
Move WTF_USE_AVFOUNDATION from JavaScriptCore/wtf/platform.h to WebCore/config.h
Comment 1 Jeff Miller 2011-08-02 11:18:55 PDT
Created attachment 102668 [details]
Patch
Comment 2 Adam Roben (:aroben) 2011-08-02 11:20:26 PDT
Comment on attachment 102668 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=102668&action=review

> Source/JavaScriptCore/ChangeLog:8
> +        Move WTF_USE_AVFOUNDATION from JavaScriptCore/wtf/platform.h to WebCore/config.h
> +        https://bugs.webkit.org/show_bug.cgi?id=65552
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        * wtf/Platform.h: Removed WTF_USE_AVFOUNDATION.

I think it's worth mentioning what the benefit is.
Comment 3 Jeff Miller 2011-08-02 11:23:26 PDT
Comment on attachment 102668 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=102668&action=review

>> Source/JavaScriptCore/ChangeLog:8
>> +        * wtf/Platform.h: Removed WTF_USE_AVFOUNDATION.
> 
> I think it's worth mentioning what the benefit is.

I will do so before landing, thanks!
Comment 4 Jeff Miller 2011-08-02 11:30:14 PDT
Committed r92210: <http://trac.webkit.org/changeset/92210>
Comment 5 Jeff Miller 2011-08-02 11:32:23 PDT
<rdar://problem/9882973>