Fix blank gif avatars (#6060)

## Issue
6010: hyperchat send display issues with GIF profiles

The FreezeFrameWrapper have no `src` to freeze due to the lazy loading.

## Fix
Delay the freezing by making the lazy-load effect return a state to indicate when then `src` has been loaded.

Since the lazy-loader will `unobserve` after loading, the state will never go back to false, so we don't need to handle the case of preventing `new FreezeFrameWrapper` from being called multiple times from it's effect.

ui/component/fileThumbnail/FreezeframeWrapper.jsx

@@ -10,11 +10,13 @@ const FreezeframeWrapper = (props) => {
   // eslint-disable-next-line
   const { src, className, children } = props;
 
-  useEffect(() => {
-    freezeframe.current = new Freezeframe(imgRef.current);
-  }, []);
+  const srcLoaded = useLazyLoading(imgRef);
 
-  useLazyLoading(imgRef);
+  useEffect(() => {
+    if (srcLoaded) {
+      freezeframe.current = new Freezeframe(imgRef.current);
+    }
+  }, [srcLoaded]);
 
   return (
     <div className={classnames(className, 'freezeframe-wrapper')}>

ui/effects/use-lazy-loading.js

@@ -1,6 +1,6 @@
 // @flow
 import type { ElementRef } from 'react';
-import { useEffect } from 'react';
+import React, { useEffect } from 'react';
 
 /**
  * Helper React hook for lazy loading images
@@ -13,6 +13,8 @@ export default function useLazyLoading(
   threshold: number = 0.25,
   deps: Array<any> = []
 ) {
+  const [srcLoaded, setSrcLoaded] = React.useState(false);
+
   useEffect(() => {
     if (!elementRef.current) {
       return;
@@ -30,6 +32,7 @@ export default function useLazyLoading(
             if (target.dataset.src) {
               // $FlowFixMe
               target.src = target.dataset.src;
+              setSrcLoaded(true);
               return;
             }
 
@@ -49,4 +52,6 @@ export default function useLazyLoading(
 
     lazyLoadingObserver.observe(elementRef.current);
   }, deps);
+
+  return srcLoaded;
 }