| Summary: | [Mac][EME] Crash in CDMSessionMediaSourceAVFObjC::layerDidReceiveError() - NSError not KVO compliant for key NSUnderlyingError. | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Jer Noble <jer.noble> | ||||||
| Component: | New Bugs | Assignee: | Jer Noble <jer.noble> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | commit-queue, webkit-bug-importer | ||||||
| Priority: | P2 | Keywords: | InRadar | ||||||
| Version: | 528+ (Nightly build) | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| Attachments: |
|
||||||||
|
Description
Jer Noble
2015-01-15 16:44:40 PST
Created attachment 244726 [details]
Patch
Comment on attachment 244726 [details]
Patch
This has an extra change log entry.
How are we regression-testing this fix?
(In reply to comment #2) > Comment on attachment 244726 [details] > Patch > > This has an extra change log entry. Whoops, will fix. > How are we regression-testing this fix? We have no idea when or why AVFoundation is firing this type of error (with a code of AVErrorUnknown). Ironically, once we correct this piece and start returning the underlying error, the clients of WebKit's EME API can start reporting what underlying error they're seeing. And we may be able to write a regression or manual test at that point. Created attachment 245354 [details]
Patch for landing
Comment on attachment 245354 [details] Patch for landing Clearing flags on attachment: 245354 Committed r179563: <http://trac.webkit.org/changeset/179563> |