Bug 203302 – WKWebView: WKURLSchemeHandler request don't have Range headers

• Status: RESOLVED
• Resolution: FIXED
• Component: Media

[jcesarmobile]

The problem:
When serving an Apache Cordova or Ionic Capacitor app from a custom scheme, if the app contains a video tag, it makes multiple requests to the video file (I guess it's to return the video in chunks).
The problem is, the requests don't include Range headers, so it's impossible to return just the chunks, and also, as it's requested multiple times, the memory increases a lot.

Expected:

When using a WKURLSchemeHandler, the requests should include Range headers so the video can be returned by the requested chunks.
Alternatively, if the app is being served from a custom scheme, then a single request should be made to the WKURLSchemeHandler (when serving the app from file:// protocol only a request is made)


Sample project reproducing the issue (without using Apache Cordova nor Ionic Capacitor to simplify the code)

https://github.com/jcesarmobile/no-ranges-bug

Check the WebViewHandler class, it prints urlSchemeTask.request.allHTTPHeaderFields content and it doesn't include Range. 
https://github.com/jcesarmobile/no-ranges-bug/blob/master/no-ranges-bug/WebViewHandler.swift#L34

Also if the app is inspected in Safari remote inspector, the requests don't show Range in headers neither.

[Radar WebKit Bug Importer]

<rdar://problem/56557566>

[Alex Christensen]

WKURLSchemeHandler is behaving correctly here, and on Mac the same code has a range request.  The problem is on iOS the request we receive from AVAssetResourceLoader does not have a range request, but on Mac the request we receive in the WebCoreNSURLSession has a range request.  I'm uploading a test that verifies this is the case.

[Alex Christensen]

Created attachment 381757 [details]
Patch

[Geoffrey Garen]

Comment on attachment 381757 [details]
Patch

r=me

Can you file a bug report for AVAssetResourceLoader?

[jcesarmobile]

Are you asking me or somebody else?
In case it's me, where should I report the new issue?

[Alex Christensen]

I think he was asking me to report another bug and use this just to commit the test that verifies existing behavior.  I'm still talking to people and deciding what to do about this.

[Taehyun Kim]

We have faced the same no Range header problem using WKURLSchemeHandler for custom video scheme on iOS.

Does it still remain an unresolved bug even after 6 months later from the first report?

Nonetheless, is there any one who happens to know how to detour this bug so that we can manage to catch the Range data from AVAssetResourceLoader?

It is so desperate to finalize our project.

[jbgt]

Hi, any new on this? Is there anywhere we can follow this issue(AVAssetResourceLoader or something else)?

[Felix Wansing]

Hello, is there anything new in this Bug?  The problem persists and makes it impossible to view larger files from local storage.

[Alex Christensen]

Created attachment 427459 [details]
Patch

[Geoffrey Garen]

Comment on attachment 427459 [details]
Patch

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

r=me - plz fix test build

> Tools/TestWebKitAPI/Tests/WebKitCocoa/WKURLSchemeHandler-1.mm:1514
> +            @"Content-Range" : [NSString stringWithFormat:@"bytes %llu-%llu/%lu", rangeBegin, rangeEnd, [videoData length]],
> +            @"Content-Length" : [NSString stringWithFormat:@"%llu", rangeEnd - rangeBegin + 1]

/Volumes/Data/worker/watchOS-7-Build-EWS/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKURLSchemeHandler-1.mm:1513:105: error: values of type 'NSUInteger' should not be used as format arguments; add an explicit cast to 'unsigned long' instead [-Werror,-Wformat]

[Alex Christensen]

Created attachment 427464 [details]
Patch

[Alex Christensen]

iOS test failures look related.  Will investigate.

[Alex Christensen]

It seems like blob URLs are going through this path too.  I'm going to exclude the blob protocol.

[Alex Christensen]

Created attachment 427587 [details]
Patch

[Alex Christensen]

r276932