GitLab

This CL makes |sequence_checker_| a protected member var in
vaapi_picture.h and uses it in every method in the derived
classes in vaapi_{drm,tfp}_picture.cc. Opportunistically it
also moves the service function BufferFormatToInternalFormat()
to an anonymous namespace in vaapi_drm_picture.cc.

TEST=**No new code intended**, these classes are supposed
to be single threaded, so this is just a help for future
and present devs by enforcing assumptions. (But just in case
I tried the usual playback in Soraka-simplechrome, ensuring
  dcheck_always_on = true
in the appropriate args.gn, otherwise Release builds don't
have DCHECKs on).


Bug: 717265
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I9e96440ec7ade318f5ac4c9289fcee1e77c6d90a
Reviewed-on: https://chromium-review.googlesource.com/744903
Commit-Queue: Miguel Casas <mcasas@chromium.org>
Reviewed-by: Pawel Osciak <posciak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#512819}