GitLab

Currently we compare the whole PipelineDecoderInfo to decide where
there's a change to report. But we only store the |decoder_name| for the
next comparison, so info like |is_decrypting_demuxer_stream| is not
updated. Then we'll keep reporting PipelineDecoderInfo even if there's
no new change.

This CL fixes this issue by storeing the whole PipelineDecoderInfo
instead of just the |decoder_name|. Additional logging is also added for
the convenience in debugging.

Bug: 1018372
Test: Adds new unit test to cover this
Change-Id: I6d309075b694f6a7cde6a4662c2200e2606652f3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1917600Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Ted Meyer <tmathmeyer@chromium.org>
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#715571}