Take render target dimensions into account when drawing

[adroitwhiz]

Depends on #555

Resolves

Resolves #545
Resolves #558

Proposed Changes

This PR changes _drawThese to accept optional framebufferWidth and framebufferHeight parameters, and default to the "native size" (480x360) if they're not given.

These dimensions are used to calculate drawables' "framebuffer-space" scales.

Reason for Changes

See #545.

[adroitwhiz]

@cwillisf This resolves #558, which is priority 2, so I want to make sure this hasn't fallen off your radar (given it has no "needs review" tag and is sandwiched in between several Dependabot PRs).

[cwillisf]

It looks to me like the extractDrawable methods should provide buffer dimensions when calling _drawThese since it might use the query buffer OR a new, potentially very large buffer.

[adroitwhiz]

Good catch! I've done this for extractDrawableScreenSpace--does anything still use extractDrawable other than the playground? I can't see anything in any public repos. Regardless, it looks like extractDrawable does everything at the "Scratch-space" resolution anyway, so I think it should be left as is.

[fsih]

LG, thanks!

We're planning to not deploy changes during December, due to the annual hour of code and holidays, so will be holding off on merging until January