Image with ImageFillContain seen stretched before correcting for aspect.

[datboy-design]

Describe the bug:

Image first shows as stretched to fill canvas. After a short pause (probably processing time) the image is re-displayed corrected for aspect.

To Reproduce:

Run this code with local image

package main

import (
"fmt"
"imaging"
"os"

"fyne.io/fyne"
"fyne.io/fyne/app"
"fyne.io/fyne/canvas"

)

func main() {
myApp := app.New()
w := myApp.NewWindow("Image")
w.Resize(fyne.NewSize(600, 600)) // problem for all sizes tried

// Open JPEG file and get io.Reader

fileName := os.Args[1]
file, err := os.Open(fileName)
if err != nil {
	fmt.Printf("Error opening file %s\n", fileName)
	return
}

// Set the AutroOrientation option for decoding.
// We need to use imaging,Decode because it checks for and corrects for orientation data.
// If present this is held in an EXIF tag. image.Decode() follows the standard
// JPEG library and assumes file will be soley JFIF and correctly orientated.
    // Using imaging package to get round this.

opt := imaging.AutoOrientation(true)  // create option for imaging.Decode()
img, err := imaging.Decode(file, opt) // decoded file as image.Image

if err != nil {
	fmt.Printf("Error decoding image %s\nError %v\n", fileName, err)
	return
}

image := canvas.NewImageFromImage(img)
image.FillMode = canvas.ImageFillContain
w.SetContent(image)
w.ShowAndRun()

}

Screenshots:

Device (please complete the following information):

• OS: "Linux Ubuntu"
• Version: "18.04.5 LTS (Bionic Beaver)"
• Go version: "go1.15 linux/amd64"
• Fyne version: "v1.3.3-3-g6c9ed1b5"]

[andydotxyz]

On develop for testing.

[stuartmscott]

I can still reproduce. Run the below with any jpeg image as go run main.go /path/to/image.jpg

package main

import (
	"bytes"
	"fyne.io/fyne"
	"fyne.io/fyne/app"
	"fyne.io/fyne/canvas"
	"fyne.io/fyne/widget"
	"image/jpeg"
	"io/ioutil"
	"log"
	"os"
)

func main() {
	// Create Fyne App
	a := app.New()

	// Create Fyne Window
	w := a.NewWindow("Image Jump")

	// Create image to hold image
	img := &canvas.Image{
		FillMode: canvas.ImageFillOriginal,
	}
	scroller := widget.NewScrollContainer(img)

	// Create goroutine to load file contents and update image
	go func() {
		data, err := ioutil.ReadFile(os.Args[1])
		if err != nil {
			log.Println("Error:", err)
			return
		}
		i, err := jpeg.Decode(bytes.NewReader(data))
		if err != nil {
			log.Println("Error:", err)
			return
		}
		img.Image = i
		img.Refresh()
		scroller.Refresh()
	}()

	// Add scroller to window
	w.SetContent(scroller)

	w.Resize(fyne.NewSize(480, 600))
	w.ShowAndRun()
}

[stuartmscott]

Oops, I just noticed I'm using ImageFillOriginal and this issue is about ImageFillContain