Revert #52647 (Don't update CSG Shape when not inside tree)

[hoontee]

Reverts #52647.
Fixes #54445.
#52642 stays fixed, presumably due to #40814.

[timothyqiu]

Looks good 👍

My PR created a situation where it's dirty but no _update_shape() is processed. So later _make_dirty() will never update anything. Sorry for that :(

#52642 is indeed fixed by #40814.

[akien-mga]

Thanks!

[akien-mga]

@hoontee @timothyqiu I need some advice to decide what to do for 3.4. #40814 was backported to 3.x and thus fixes the original issue, but I think it needs some testing in a few betas before it can be considered to cherry-pick in 3.4 (if at all).

So we're left with a regression in 3.4 which can be fixed by cherry-picking this revert, but that would reintroduce #52642 IIUC. What would you suggest?

[hoontee]

@akien-mga Ideally we would want to cherry-pick #58938 for its numerous fixes, however if that can't happen, I don't believe this one should be cherry-picked. The error in #52642 slows down the editor when there are several Paths.

Of course, not seeing any CSGShapes in other scenes sucks too, so if it's decided that CSGPolygon paths are too rare then maybe you should cherry-pick this. Hard to make a call.

[timothyqiu]

@akien-mga #52642 is mostly annoyance I think. Cherry-picking this revert brings correctness.

[akien-mga]

Cherry-picked for 3.4.4.