Bug 181554 - FormDataElement::lengthInBytes should use ThreadableBlobRegistry
Summary: FormDataElement::lengthInBytes should use ThreadableBlobRegistry
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Service Workers (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: youenn fablet
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-01-11 13:20 PST by youenn fablet
Modified: 2018-01-12 12:42 PST (History)
3 users (show)

See Also:

Attachments
Patch (1.39 KB, patch)
2018-01-11 13:23 PST, youenn fablet 		no flags Details | Formatted Diff | Diff
Adding test (10.76 KB, patch)
2018-01-12 09:45 PST, youenn fablet 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description youenn fablet 2018-01-11 13:20:16 PST 
This is used by Cache API which is exposed to workers.
Comment 1 youenn fablet 2018-01-11 13:23:20 PST 
Created attachment 331107 [details]
Patch
Comment 2 Chris Dumez 2018-01-12 09:03:24 PST 
Comment on attachment 331107 [details]
Patch

Why don't we hit the isMainThread() assertion in blobRegistry() if this is an issue? At the very least, this looks testable.
Comment 3 youenn fablet 2018-01-12 09:45:43 PST 
Created attachment 331209 [details]
Adding test
Comment 4 WebKit Commit Bot 2018-01-12 12:42:00 PST 
Comment on attachment 331209 [details]
Adding test

Clearing flags on attachment: 331209

Committed r226906: <https://trac.webkit.org/changeset/226906>
Comment 5 WebKit Commit Bot 2018-01-12 12:42:01 PST 
All reviewed patches have been landed.  Closing bug.
Comment 6 Radar WebKit Bug Importer 2018-01-12 12:42:33 PST 
<rdar://problem/36479493>