The ReadVideo method of OMXPlugin uses an infinite 'for' loop with conditionals to break or continue. This makes the control flow hard to follow and should be refactored so the infinite loop is not needed.
Created attachment 631217 [details] [diff] [review] Refactored method
Needs bug 761762 landed first, waiting for that to be ready before checkin-needed on this.
https://hg.mozilla.org/mozilla-central/rev/2fd06ff2ed50 (Merged by Ed Morley)