fix(canvas.renderAll): setDimensions edge case

[ShaMan123]

Motivation:
#6112

The flag hasLostContext is turned on here:

fabric.js/src/static_canvas.class.js

Line 602 in a709d3b

this.hasLostContext = true;

And is never turned off.
And that seems to be the problem.

closes #6112

[ShaMan123]

Thanks @yassilah for the excellent repro that made it easy to triage this bug

Originally posted by @yassilah in #6112 (comment)

[asturur]

What the hell was hasLostContext

[asturur]

So basically after a first resize change, we have been doing non useful redrawing each render cycle.

[asturur]

@ShaMan123 you should make branch on the main repo, would be easier to add tests for me!

[asturur]

Thank you this is a great fix

[ShaMan123]

@ShaMan123 you should make branch on the main repo, would be easier to add tests for me!

I have tried in the past. Git goes crazy. Puts branches in headless state and blocks me from working on upstream.

[asturur]

clone everything in a new folder, maybe that fixes it.

[yassilah]

Thanks @yassilah for the excellent repro that made it easy to triage this bug

Originally posted by @yassilah in #6112 (comment)

Happy to have helped! 🙂

[asturur]

Thanks @yassilah for the excellent repro that made it easy to triage this bug
Originally posted by @yassilah in #6112 (comment)

Happy to have helped! 🙂

Yep a good reproducible case is often the best help to expedite things