[Android] Add missing onError event to Image

[SergeyKorochansky]

This PR adds onError prop to Image on Android. Closes #7440

Motivation:
We have image feed and show refresh button when image loading had failed.

Usage of onError event is necessary to track loading failures

[ghost]

By analyzing the blame information on this pull request, we identified @andreicoman11 and @Bhullnatik to be potential reviewers.

[ghost]

@webzepter updated the pull request.

[brentvatne]

Hi there! Thanks for the PR. This was actually intentionally left out because Fresco doesn't fully support it yet, see: #3791 (comment)

[mkonicek]

@brentvatne Looks like we emit an ON_ERROR event in native:

react-native/ReactAndroid/src/main/java/com/facebook/react/views/image/ReactImageView.java

Line 214 in bcf48e7

new ImageLoadEvent(getId(), ImageLoadEvent.ON_ERROR));

This PR would make it possible to handle it in JS, correct?

[SergeyKorochansky]

@mkonicek yes

[ghost]

It's been a while since the last commit was reviewed and the labels show this pull request needs review. Based on the blame information for the files in this pull request we identified @rigdern as a potential reviewer. Could you take a look please or cc someone with more context?

[brentvatne]

@mkonicek - the problem is that the behaviour is inconsistent with iOS-- once Fresco has similar error semantics this is easy. I don't follow that project closely and I'm not sure what the current state is. if we merge this we have to keep in mind, and document, that the callbacks fire in different situations on each platform.

[lacker]

Hey, so chatting with @brentvatne and based on the discussion in #3791 (comment) and this thread it seems like this current behavior is working as intended, at least as long as Fresco's behavior makes it impossible for iOS and Android to behave consistently. I'm going to close this pull request, but if it is indeed possible to have this with consistent behavior across iOS and Android, feel free to reopen.