Changed order of calls to properly close MediaPicker Activity / ViewController and set the picker result

[Alex-Dobrynin]

Fixes this issue Alex-Dobrynin/Xamarin.Controls.ImageCropper#8

Shortly, at the moment of getting the result of mediapicker, the activity or viewcontroller remains opened. And if open another activity or viewcontroller right after the result of mediapicker you will not see any new activity/viewcontroller opened. So to fix this we need to change the order of calls:

1. we need to close activity/viewcontroller
2. we need to set the result of mediapicker

Actual result: we need to put some Task.Delay after mediapicker result to be able open new activity/viewcontroller
Expected result: we able to open new activity/viewcontroller right after getting the picker result.

To reproduce it, you can try open new activity or viewcontroller from the topmost one (which is still mediapicker's activity/viewcontrolelr) right after the mediapicker result.

Affects only android/ios

[dotnet-policy-service]

Hey there @Alex-Dobrynin! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

[Alex-Dobrynin]

@dotnet-policy-service agree

[jsuarezruiz]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 3 pipeline(s).

[Alex-Dobrynin]

will this be reviewed some day?

[azure-pipelines]

Commenter does not have sufficient privileges for PR 21998 in repo dotnet/maui

[Alex-Dobrynin]

Somebody will review this?

[Axemasta]

I've just run into this issue, its really painful since you have to wait out the result with a Task.Delay and have to set that delay to be high to prevent the race condition kicking in....

Would love to see this in soon!

[Alex-Dobrynin]

@Axemasta for now you may use this package MPowerKit.MediaPlugin, you will not have this issue there. It is a port of James Montemagno's MediaPlugin, but with some fixes