GitLab

This CL follows up on a comment [1] on the CL landing SetEncryptedStream(),
("We could do SetStream(..., const DecryptConfig* decrypt_config) as nullptr
is used in other places to indicate that there is no decrypt config."), by
removing SetEncryptedStream() and adding instead a default-null argument
|decrypt_config| SetStream instead(). SetStream and SetEncryptedStream
are so similar from an APi perspective that it doesn't seem to merit a
method on its own.

Note: SetStream() is fully synchronous and DecrypConfig is a plain vanilla
class, so nacked pointer is good enough (we could also consider
std::unique_ptr<> instead, down the line).


[1] https://chromium-review.googlesource.com/c/chromium/src/+/1024961/8/media/gpu/accelerated_video_decoder.h#35

Bug: 820192
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: Iec056f33a792965f8bf854d9564e3bff520f4f8d
Reviewed-on: https://chromium-review.googlesource.com/1061621Reviewed-by: Dan Sanders <sandersd@chromium.org>
Commit-Queue: Miguel Casas <mcasas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#560778}