Summary: | Possible CFArray leak in MediaPlayerPrivateAVFoundationCF.cpp | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Joseph Pecoraro <joepeck> | ||||||
Component: | WebCore Misc. | Assignee: | Joseph Pecoraro <joepeck> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bfulgham, commit-queue, eric.carlson, joepeck | ||||||
Priority: | P2 | ||||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Joseph Pecoraro
2015-08-31 21:09:08 PDT
Interesting, it seems like two function nearly right next to each other are nearly identical: static const HashSet<String>& avfMIMETypes() static const HashSet<String>& mimeTypeCache() Maybe they can be merged. The only different seems to be, one has lowercase strings, and the other doesn't. However, MediaPlayerPrivateAVFoundationObjC only has one implementation, and it does the lowercase strings, so I'll combine to just that one. Created attachment 260349 [details]
[PATCH] Proposed Fix
Attachment 260349 [details] did not pass style-queue:
ERROR: Source/WebCore/ChangeLog:8: You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible. [changelog/nonewtests] [5]
Total errors found: 1 in 2 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 260350 [details]
[PATCH] Proposed Fix
Comment on attachment 260350 [details] [PATCH] Proposed Fix Clearing flags on attachment: 260350 Committed r189209: <http://trac.webkit.org/changeset/189209> All reviewed patches have been landed. Closing bug. |