[rbp/omxplayer] Make m_omx_render an object rather than pointer

Seems no reason for it not to be like other components, as it makes merging with command line omxplayer more difficult
[rbp/omxplayer] Remove commented out code
[rbp/omxplayer] Move openmax pipeline creation into portsettingschanged code

OpenMAX setup becomes more straightforward when components and tunnels are initialised after the first portsettingschanged message.
You get one of these after the first frame is decoded (and also when settings change, like resolution).
Doing the setup here means you know useful information about the stream, like the resolution and you avoid having to disable/enable multiple ports.

It will also make creating a pipline with the deinterlace component possible *after* the decoder has identified if stream is interlaced
[rbp/omxplayer] Support automatic deinterlace correctly.

Previously there was no distinction between deinterlace being automatic and force on.
Both did a hack parse that only worked (sometimes) for h264. Other codecs would always force on.

This change now handles the three modes correctly. Off and force on do as expected.
Automatic now queries the openmax video codec, which can reliably determine if a stream is interlaced
Merge pull request #2801 from popcornmix/automatic_deinterlace

[rbp/omxplayer] Support automatic deinterlace correctly