26902 Initial code review

Created a branch from the very first commit to be able to review the whole code written so far.
This does not mean that a merge should be done after the review is done, this is a "dummy" pull request.
Why not just doing something like :
```
return {
    "FrameWidth" : "1920",
    "FrameHeight" : "1080",
    ....................
   "ProxyAbove8bits" : "false",
}
```
which would make the code more readable ?
And, it looks like values must be strings ? A comment saying that any value must be a string would be helpful I think.