-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Make FetchResponseListener::process_response_eof FnOnce #22550
Copy link
Copy link
Closed
Labels
A-content/scriptRelated to the script threadRelated to the script threadA-networkC-assignedThere is someone working on resolving the issueThere is someone working on resolving the issue
Description
Specifically, in #22478 I ran into having to work around this function taking a &mut self by cloning fields out of self to be sent to another thread. It would make things much easier if we could instead move them out of self. Since this function represents the last step of processing a response, I think it would also make semantically sense to do so, and from what I've seen, it wouldn't break existing trait implementations.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-content/scriptRelated to the script threadRelated to the script threadA-networkC-assignedThere is someone working on resolving the issueThere is someone working on resolving the issue