fix ReactiveX#5468: eager hooks onError call (ReactiveX#5470)

* fix ReactiveX#5468: eager hooks onError call

Move the call to RxJavaHooks into the catch block, so we only report it if the underlying subscriber throws in its onError method.
This brings the behaviour in line with that of SafeSubscriber, which only call Hooks.onError inside the catch

*  ReactiveX#5468 move hooks OnError into correct if block

Author: DeepSpawn

src/main/java/rx/observers/SafeCompletableSubscriber.java

@@ -54,8 +54,8 @@ public void onCompleted() {
 
     @Override
     public void onError(Throwable e) {
-        RxJavaHooks.onError(e);
         if (done) {
+            RxJavaHooks.onError(e);
             return;
         }
         done = true;