Ignoring if we stay with this function or not, should `cameraCartesianPosition` and  `cameraCartographicPosition` be part of `frameState`?  In general, the trend has been to move more things to `frameState` so we don't have big breaking changes.  I can even see us moving `context` and `commandList` to `frameState` (no need to do this now).