[Chromecast] Make buffering logging less confusing

chromecast/media/cma/pipeline/media_pipeline_impl.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chromecast/media/cma/pipeline/media_pipeline_impl.h"
 
 #include <algorithm>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 334 matching lines @@
     metrics::CastMetricsHelper::GetInstance()->RecordApplicationEvent(
         "Cast.Platform.Ended");
 
     base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE,
                                                   pending_flush_task_->done_cb);
     pending_flush_task_.reset();
   }
 }
 
 void MediaPipelineImpl::OnBufferingNotification(bool is_buffering) {
-  CMALOG(kLogControl) << __FUNCTION__ << " is_buffering=" << is_buffering;
+  LOG(INFO) << __FUNCTION__ << " is_buffering=" << is_buffering;

[alokp, 2017/01/05 23:57:38]

We should continue to use VLOG here.

   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK((backend_state_ == BACKEND_STATE_PLAYING) ||
          (backend_state_ == BACKEND_STATE_PAUSED));
   DCHECK(buffering_controller_);
   DCHECK_EQ(is_buffering, buffering_controller_->IsBuffering());
 
   if (!client_.buffering_state_cb.is_null()) {
     if (is_buffering) {
       // TODO(alokp): WebMediaPlayerImpl currently only handles HAVE_ENOUGH.
       // See WebMediaPlayerImpl::OnPipelineBufferingStateChanged,
       // http://crbug.com/144683.
-      LOG(WARNING) << "Ignoring buffering notification.";
+      NOTIMPLEMENTED();
     } else {
       client_.buffering_state_cb.Run(::media::BUFFERING_HAVE_ENOUGH);
     }
   }
 
   if (is_buffering && (backend_state_ == BACKEND_STATE_PLAYING)) {
     media_pipeline_backend_->Pause();
     backend_state_ = BACKEND_STATE_PAUSED;
     metrics::CastMetricsHelper::GetInstance()->RecordApplicationEvent(
         "Cast.Platform.Pause");
@@ skipping 146 matching lines @@
 
 void MediaPipelineImpl::ResetBitrateState() {
   elapsed_time_delta_ = base::TimeDelta::FromSeconds(0);
   audio_bytes_for_bitrate_estimation_ = 0;
   video_bytes_for_bitrate_estimation_ = 0;
   last_sample_time_ = base::TimeTicks::Now();
 }
 
 }  // namespace media
 }  // namespace chromecast