GitLab

DirectSharedImageVideoProvider was in video_frame_factory_impl.cc,
mostly so that review of the refactor would be simpler.  This CL
moves it into separate files.  There is very little functional
change, except that TextureOwner creation is now moved out of the
SharedImageVideoProvider, and into VideoFrameFactoryImpl.  It's
still somewhat hacky, but at least it's hacky closer to where it
should actually be done.

This CL also adds more unittests for VideoFrameFactoryImpl, since
it's now much easier to inject mocks for everything.  It doesn't add
any tests for DirectSharedImageVideoProvider, since that depends on
a refactor to move codec output buffer management out of CodecImage.

Change-Id: I521fec5c9c03d00878bd4ff4844304f5f893a3f5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1646901
Commit-Queue: Frank Liberato <liberato@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#676083}