Merge pull request #1781 from wordpress-mobile/issue/17760-reader-comment-full-images

issue/1760-reader-comment-full-images

Author: maxme

WordPressUtils/src/main/java/org/wordpress/android/util/WPImageGetter.java

@@ -19,8 +19,8 @@
  * adapted from existing ImageGetter code in NoteCommentFragment
  */
 public class WPImageGetter implements Html.ImageGetter {
-    private WeakReference<TextView> mWeakView;
-    private int mMaxSize;
+    private final WeakReference<TextView> mWeakView;
+    private final int mMaxSize;
     private ImageLoader mImageLoader;
     private Drawable mLoadingDrawable;
     private Drawable mFailedDrawable;
@@ -43,18 +43,6 @@ public WPImageGetter(TextView view, int maxSize, ImageLoader imageLoader, Drawab
         mFailedDrawable = failedDrawable;
     }
 
-    public void setImageLoader(ImageLoader imageLoader) {
-        mImageLoader = imageLoader;
-    }
-
-    public void setLoadingDrawable(Drawable loadingDrawable) {
-        mLoadingDrawable = loadingDrawable;
-    }
-
-    public void setFailedDrawable(Drawable failedDrawable) {
-        mFailedDrawable = failedDrawable;
-    }
-
     private TextView getView() {
         return mWeakView.get();
     }
@@ -80,9 +68,6 @@ public Drawable getDrawable(String source) {
             source = PhotonUtils.getPhotonImageUrl(source, mMaxSize, 0);
         }
 
-        TextView view = getView();
-        // Drawable loading = view.getContext().getResources().getDrawable(R.drawable.remote_image); FIXME: here
-        // Drawable failed = view.getContext().getResources().getDrawable(R.drawable.remote_failed);
         final RemoteDrawable remote = new RemoteDrawable(mLoadingDrawable, mFailedDrawable);
 
         mImageLoader.get(source, new ImageLoader.ImageListener() {
@@ -97,43 +82,40 @@ public void onErrorResponse(VolleyError error) {
 
             @Override
             public void onResponse(ImageLoader.ImageContainer response, boolean isImmediate) {
-                if (response.getBitmap() != null) {
-                    // make sure view is still valid
-                    TextView view = getView();
-                    if (view == null) {
-                        AppLog.w(T.UTILS, "WPImageGetter view is invalid");
-                        return;
-                    }
-
-                    Drawable drawable = new BitmapDrawable(view.getContext().getResources(), response.getBitmap());
-                    final int oldHeight = remote.getBounds().height();
-                    int maxWidth = view.getWidth() - view.getPaddingLeft() - view.getPaddingRight();
-                    if (mMaxSize > 0 && (maxWidth > mMaxSize || maxWidth == 0)) {
-                        maxWidth = mMaxSize;
-                    }
-                    remote.setRemoteDrawable(drawable, maxWidth);
-
-                    // image is from cache? don't need to modify view height
-                    if (isImmediate) {
-                        return;
-                    }
-
-                    int newHeight = remote.getBounds().height();
-                    view.invalidate();
-                    // For ICS
-                    view.setHeight(view.getHeight() + newHeight - oldHeight);
-                    // Pre ICS
-                    view.setEllipsize(null);
+                if (response.getBitmap() == null) {
+                    AppLog.w(T.UTILS, "WPImageGetter null bitmap");
+                }
+
+                TextView view = getView();
+                if (view == null) {
+                    AppLog.w(T.UTILS, "WPImageGetter view is invalid");
+                    return;
+                }
+
+                int maxWidth = view.getWidth() - view.getPaddingLeft() - view.getPaddingRight();
+                if (mMaxSize > 0 && (maxWidth > mMaxSize || maxWidth == 0)) {
+                    maxWidth = mMaxSize;
+                }
+
+                Drawable drawable = new BitmapDrawable(view.getContext().getResources(), response.getBitmap());
+                remote.setRemoteDrawable(drawable, maxWidth);
+
+                // force textView to resize correctly if image isn't cached by resetting the content
+                // to itself - this way the textView will use the cached image, and resizing to
+                // accommodate the image isn't necessary
+                if (!isImmediate) {
+                    view.setText(view.getText());
                 }
             }
         });
+
         return remote;
     }
 
     private static class RemoteDrawable extends BitmapDrawable {
-        protected Drawable mRemoteDrawable;
-        protected Drawable mLoadingDrawable;
-        protected Drawable mFailedDrawable;
+        Drawable mRemoteDrawable;
+        final Drawable mLoadingDrawable;
+        final Drawable mFailedDrawable;
         private boolean mDidFail = false;
 
         public RemoteDrawable(Drawable loadingDrawable, Drawable failedDrawable) {
@@ -158,11 +140,6 @@ public void setBounds(int x, int y, int width, int height) {
             }
         }
 
-        public void setRemoteDrawable(Drawable remote) {
-            mRemoteDrawable = remote;
-            setBounds(0, 0, mRemoteDrawable.getIntrinsicWidth(), mRemoteDrawable.getIntrinsicHeight());
-        }
-
         public void setRemoteDrawable(Drawable remote, int maxWidth) {
             // null sentinel for now
             if (remote == null) {