Bug 160472

Summary: Temporary redirected m3u8 streaming stopped working.
Product: WebKit Reporter: Jeremy Jones <jeremyj-wk>
Component: MediaAssignee: Jeremy Jones <jeremyj-wk>
Status: RESOLVED INVALID    
Severity: Normal CC: achristensen, commit-queue, eric.carlson, jer.noble, jonlee, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Mac   
OS: Unspecified   
Bug Depends on: 160607    
Bug Blocks:    
Attachments:
Description Flags
Patch none

Jeremy Jones
Reported 2016-08-02 15:40:22 PDT
Temporary redirected m3u8 streaming stopped working.
Attachments
Patch (9.75 KB, patch)
2016-08-03 15:10 PDT, Jeremy Jones 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Jeremy Jones
Comment 1 2016-08-02 15:41:24 PDT
rdar://problem/27592694
Jeremy Jones
Comment 2 2016-08-03 15:10:38 PDT
Created attachment 285282 [details] Patch
Alex Christensen
Comment 3 2016-08-03 17:03:46 PDT
Comment on attachment 285282 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=285282&action=review This whole thing is unfortunate. r=me. This covers up bugs even more! > Source/WebCore/platform/network/cocoa/WebCoreNSURLSession.mm:590 > + This space should probably be removed so we can see that the above comment applies to the below code. > Source/WebCore/platform/network/cocoa/WebCoreNSURLSession.mm:594 > + if (mimeType.isEmpty() || mimeType == "application/octet-stream" || mimeType == "text/plain") { could you add a test with a text/plain mime type? > Source/WebCore/platform/network/cocoa/WebCoreNSURLSession.mm:673 > + ditto.
WebKit Commit Bot
Comment 4 2016-08-04 11:49:38 PDT
Comment on attachment 285282 [details] Patch Clearing flags on attachment: 285282 Committed r204128: <http://trac.webkit.org/changeset/204128>
WebKit Commit Bot
Comment 5 2016-08-04 11:49:41 PDT
All reviewed patches have been landed. Closing bug.
WebKit Commit Bot
Comment 6 2016-08-05 13:56:12 PDT
Re-opened since this is blocked by bug 160607
Jon Lee
Comment 7 2016-08-05 14:21:46 PDT
We're going to solve this problem in a different way.
Note You need to log in before you can comment on or make changes to this bug.