fixup! [MM] Use ESPP for media rendering

1) Reduce log statement
Too many buffering related logs are printed. They are
removed.

2) Added missing logs in TizenRendererImpl

Reference: https://review.tizen.org/gerrit/283138/

Change-Id: Iaa1996711130a5c5a0482d94c445eaaab372baf7
Signed-off-by: Koyyani Maheswari <m.koyyani@samsung.com>

diff --git a/tizen_src/chromium_impl/content/browser/media/tizen_renderer_impl.cc b/tizen_src/chromium_impl/content/browser/media/tizen_renderer_impl.cc
index 943ec52..0b9fd58 100644 (file)
--- a/tizen_src/chromium_impl/content/browser/media/tizen_renderer_impl.cc
+++ b/tizen_src/chromium_impl/content/browser/media/tizen_renderer_impl.cc
@@ -159,6 +159,7 @@ TizenRendererImpl::TizenRendererImpl(
       web_contents_(web_contents),
       renderer_extension_receiver_(this,
                                    std::move(renderer_extension_receiver)) {
+  LOG(INFO) << "(" << static_cast<void*>(this) << ") " << __func__;
   DCHECK_EQ(WebContents::FromRenderFrameHost(
                 RenderFrameHost::FromID(process_id, routing_id)),
             web_contents);
@@ -184,6 +185,7 @@ TizenRendererImpl::TizenRendererImpl(
       task_runner_(task_runner),
       sink_(sink),
       renderer_extension_receiver_(this) {
+  LOG(INFO) << "(" << static_cast<void*>(this) << ") " << __func__;
   // TODO(dalecurtis): Remove once experiments for http://crbug.com/470940 are
   // complete.
   int threshold_ms = 0;
@@ -196,6 +198,7 @@ TizenRendererImpl::TizenRendererImpl(
 }
 
 TizenRendererImpl::~TizenRendererImpl() {
+  LOG(INFO) << "(" << static_cast<void*>(this) << ") " << __func__;
   DCHECK(task_runner_->BelongsToCurrentThread());
 
   weak_factory_.InvalidateWeakPtrs();
@@ -210,6 +213,7 @@ TizenRendererImpl::~TizenRendererImpl() {
 void TizenRendererImpl::Initialize(media::MediaResource* media_resource,
                                    media::RendererClient* client,
                                    media::PipelineStatusCallback init_cb) {
+  LOG(INFO) << "(" << static_cast<void*>(this) << ") " << __func__;
   DCHECK(task_runner_->BelongsToCurrentThread());
   DCHECK_EQ(state_, STATE_UNINITIALIZED);
   DCHECK(init_cb);

diff --git a/tizen_src/chromium_impl/media/filters/esplusplayer_buffer_observer_impl.cc b/tizen_src/chromium_impl/media/filters/esplusplayer_buffer_observer_impl.cc
index 4eb5b0d..1a8e21a 100644 (file)
--- a/tizen_src/chromium_impl/media/filters/esplusplayer_buffer_observer_impl.cc
+++ b/tizen_src/chromium_impl/media/filters/esplusplayer_buffer_observer_impl.cc
@@ -205,9 +205,10 @@ bool BufferObserverImpl::EnsureSimilarBufferedDuration() {
 
     CallbackIfNeed(DemuxerStream::AUDIO, audio_status);
     CallbackIfNeed(DemuxerStream::VIDEO, video_status);
-    LOG(INFO) << "Uneven A/V buffering, audio fill: " << audio.duration_percent
-              << "%, video fill: " << video.duration_percent
-              << "%, audio DTS - video DTS: " << av_diff << "ms";
+    DVLOG(INFO) << "Uneven A/V buffering, audio fill: "
+                << audio.duration_percent
+                << "%, video fill: " << video.duration_percent
+                << "%, audio DTS - video DTS: " << av_diff << "ms";
     return true;
   } else {
     return false;

diff --git a/tizen_src/chromium_impl/media/filters/media_player_esplusplayer.cc b/tizen_src/chromium_impl/media/filters/media_player_esplusplayer.cc
index e83e60a..d14f24c 100644 (file)
--- a/tizen_src/chromium_impl/media/filters/media_player_esplusplayer.cc
+++ b/tizen_src/chromium_impl/media/filters/media_player_esplusplayer.cc
@@ -890,8 +890,8 @@ void MediaPlayerESPlusPlayer::UpdateBufferedDtsDifference() {
 
 void MediaPlayerESPlusPlayer::OnBufferingStatusChanged(DemuxerStream::Type type,
                                                        BufferStatus status) {
-  LOG(INFO) << __func__ << " " << DemuxerStream::GetTypeName(type) << " : "
-            << GetString(status);
+  DVLOG(INFO) << __func__ << " " << DemuxerStream::GetTypeName(type) << " : "
+              << GetString(status);
   const auto RequestStateChange = [this](esplusplayer_state state) {
     LOG(INFO) << __func__ << " req. change state : " << GetString(state);
     switch (state) {