1223563 - Simplify promise rejection in `FetchBody<Derived>::ContinueConsumeBody`

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Lina Butler [:lina]]

https://dxr.mozilla.org/mozilla-central/rev/cc473fe5dc512c450634506f68cbacfb40a06a23/dom/fetch/Fetch.cpp#1072-1081 can be simplified using the variant `Promise::MaybeReject(ErrorResult&)` (https://dxr.mozilla.org/mozilla-central/rev/cc473fe5dc512c450634506f68cbacfb40a06a23/dom/promise/Promise.h#129-132).

Comment 1

[Boris Zbarsky [:bzbarsky]]

This was blocking my patches for bug 1224007, so I ended up just fixing it....

Comment 2

[Sunny Sidhu]

Attachment: rev1 - Applies simplification in Fetch.cpp

Hi Kit 

Can you please provide feedback on my patch, also can you tell me how I can test these?

Comment 3

[Lina Butler [:lina]]

Comment on attachment 8688528 [details] [diff] [review]
rev1 - Applies simplification in Fetch.cpp

Review of attachment 8688528 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks, looks good!

You'll need access to the Try server to test out your change. Please file a bug for Level 1 commit access (https://www.mozilla.org/en-US/about/governance/policies/commit/) and needinfo? me; I'll vouch for you. If you installed the Mozilla VCS tools with `mach mercurial-setup`, you can then trigger a Try run with `hg push-to-try -m "try: trychooser-syntax"`. Here's a cheat sheet for the syntax: http://trychooser.pub.build.mozilla.org/

Comment 4

[Sunny Sidhu]

(In reply to Kit Cambridge [:kitcambridge] from comment #3)
> Comment on attachment 8688528 [details] [diff] [review]
> rev1 - Applies simplification in Fetch.cpp
> 
> Review of attachment 8688528 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Thanks, looks good!
> 
> You'll need access to the Try server to test out your change. Please file a
> bug for Level 1 commit access
> (https://www.mozilla.org/en-US/about/governance/policies/commit/) and
> needinfo? me; I'll vouch for you. If you installed the Mozilla VCS tools
> with `mach mercurial-setup`, you can then trigger a Try run with `hg
> push-to-try -m "try: trychooser-syntax"`. Here's a cheat sheet for the
> syntax: http://trychooser.pub.build.mozilla.org/

All done! 

Thank you for your help. :)

Comment 5

[Sunny Sidhu]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b64baa99c97d

Comment 6

[Boris Zbarsky [:bzbarsky]]

Fixed by the patches in bug 1224007.

Comment 7

[Lina Butler [:lina]]

Sorry I missed this. :-( Looks like :bz's patch series landed first. Thanks for contributing, though!