Improve logging output for DecoderBuffer and VideoFrame

media/cdm/cdm_adapter.cc

Index: media/cdm/cdm_adapter.cc
diff --git a/media/cdm/cdm_adapter.cc b/media/cdm/cdm_adapter.cc
index 2a2068065501bed2ff767f5a9ddb4af534649654..96fa077dbb5ad836b920cde5d93302d414f988f0 100644
--- a/media/cdm/cdm_adapter.cc
+++ b/media/cdm/cdm_adapter.cc
@@ -661,6 +661,8 @@ void CdmAdapter::DecryptAndDecodeVideo(
   std::vector<cdm::SubsampleEntry> subsamples;
   scoped_ptr<VideoFrameImpl> video_frame(new VideoFrameImpl());
 
+  DVLOG(3) << __FUNCTION__
+           << " encrypted: " << encrypted->AsHumanReadableString();

[xhwang, 2015/12/18 23:07:43]

I like AsHumanReadableString(). But I don't feel we should enable it here by
default. It will generate too much noise that could hide real useful logs...

[jrummell, 2015/12/22 01:30:43]

Changed to DVLOG(5), so you only see it if you enable everything.

   ToCdmInputBuffer(encrypted, &subsamples, &input_buffer);
   cdm::Status status =
       cdm_->DecryptAndDecodeFrame(input_buffer, video_frame.get());
@@ -682,6 +684,8 @@ void CdmAdapter::DecryptAndDecodeVideo(
       frame_data + video_frame->PlaneOffset(VideoFrameImpl::kUPlane),
       frame_data + video_frame->PlaneOffset(VideoFrameImpl::kVPlane),
       base::TimeDelta::FromMicroseconds(video_frame->Timestamp()));
+  DVLOG(3) << __FUNCTION__
+           << " decoded_frame: " << decoded_frame->AsHumanReadableString();

[xhwang, 2015/12/18 23:07:44]

ditto

   video_decode_cb.Run(Decryptor::kSuccess, decoded_frame);
 }