[native] Consume useNativeUpdateUserImageAvatar in `useSelectFromGa…

…lleryAndUpdateUserAvatar`

Summary:
There is now only one usage of `updateImageUserAvatar` from `*EditUserAvatarProvider` and it's in `useNativeUpdateUserImageAvatar` where the call is wrapped in a `try/catch` with `native`-specific "error surfacing" handled in the `catch` block.

---

Depends on D8339

Test Plan:
`selectFromGalleryAndUpdateUserAvatar` is used in one place: `EditUserAvatar`. Specifically the select image from gallery flow.

Similar to D8339, will break `update_user_avatar` endpoint and ensure that alert surfaces as expected:

{F609751}

Will also test "happy case" where the `update_user_avatar` endpoint is working as expected.

Reviewers: ashoat, ginsu, rohan

Reviewed By: ashoat

Subscribers: tomek

Differential Revision: https://phab.comm.dev/D8340

native/avatars/avatar-hooks.js

@@ -214,18 +214,16 @@ function useNativeUpdateUserImageAvatar(): (
 }
 
 function useSelectFromGalleryAndUpdateUserAvatar(): () => Promise<void> {
-  const editUserAvatarContext = React.useContext(EditUserAvatarContext);
-  invariant(editUserAvatarContext, 'updateImageUserAvatar must be defined');
-  const { updateImageUserAvatar } = editUserAvatarContext;
+  const nativeUpdateUserImageAvatar = useNativeUpdateUserImageAvatar();
 
   const selectFromGalleryAndUpdateUserAvatar =
     React.useCallback(async (): Promise<void> => {
       const selection = await selectFromGallery();
       if (!selection) {
         return;
       }
-      await updateImageUserAvatar(selection);
-    }, [updateImageUserAvatar]);
+      await nativeUpdateUserImageAvatar(selection);
+    }, [nativeUpdateUserImageAvatar]);
 
   return selectFromGalleryAndUpdateUserAvatar;
 }