Skip the ChartCanvas redraw on the Viewport's internal default state usage

[arai-a]

Fixed #5159

The issue happen in the following situation:

1. ChartViewportImpl.UNSAFE_componentWillReceiveProps performs setState with the default state, which is containerWidth === 0 and containerHeight === 0
2. ChartViewportImpl.UNSAFE_componentWillReceiveProps schedules ChartViewportImpl._setSize
3. ChartCanvas.render is called for the containerWidth === 0/containerHeight === 0 state, but it doesn't use the containerWidth at that point.
4. ChartCanvas.componentDidUpdate schedules ChartCanvas._doDrawCanvas for the state update with containerWidth === 0 and containerHeight === 0
5. in the requestAnimationFrame callback, ChartViewportImpl_setSize calculates the container size, and peforms setState with the updated containerWidth/containerHeight
6. in the requestAnimationFrame callback, ChartCanvas._doDrawCanvas resizes the canvas to 0x0
7. ChartCanvas.render is called for the updated containerWidth/containerHeight state
8. ChartCanvas.componentDidUpdate schedules ChartCanvas._doDrawCanvas for the state update with the updated containerWidth/containerHeight
9. in the requestAnimationFrame callback, ChartCanvas._doDrawCanvas resizes the canvas to the correct size

Here, the steps 4 and 6 are the problem.

Given that we know the state update happens twice in a sequence, and the first one is only for the Viewport's internal use, ChartCanvas.componentDidUpdate should ignore it.

This is one of the possible solutions, with a minimal change to avoid the problematic resize.

Deploy preview: before / after

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.69%. Comparing base (0dd124f) to head (bf7d35d).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5744      +/-   ##
==========================================
+ Coverage   85.67%   85.69%   +0.02%     
==========================================
  Files         315      315              
  Lines       31110    31122      +12     
  Branches     8472     8564      +92     
==========================================
+ Hits        26654    26671      +17     
+ Misses       4026     4021       -5     
  Partials      430      430              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[canova]

Sorry for the delay on the reviews, it's been quite a busy few weeks. I'll get to the other PRs too.

Thanks for fixing it and for the tests! Although this works, I'm a bit afraid that it's like a band-aid solution instead of fixing the underlying issue. Let's land this, and we can think about a more fundamental solution.