Yeah, these aren't too complicated so I don't mind the large series. However:
* 1/9 could use tests; should be easy enough to plug into test_default_presenter_allocate().
* 2/9... wow, that's surprising; it allows BI_BITFIELDS but not 16- or 24-bit BI_RGB. Okay then.
* 3/9 is fine, just want to write that for reference I did check with Helen and the NV11 card does report NV12.
* Why do we need 4/9? Shouldn't it fail CreateSurface()?
* 5/9 is probably fine, although as someone less familiar with how this area works I'd appreciate an explanation of why the current code is wrong. It's not exactly easy to look up, anyway.
* 6/9 should presumably only be applied if the application has called SetAspectRatioMode(VMR_ARMODE_LETTER_BOX). Has it? And if it has, we presumably need to also clear the rest of the surface to black.
* 8/9 would probably make more sense if you just fit it into the table, with depth=16 and depth=24 entries.