Lots of files touched for a very simple change. Everywhere we used a const M...

media/filters/video_renderer_base.cc

 // Copyright (c) 2009 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 "media/base/buffers.h"
 #include "media/base/filter_host.h"
 #include "media/base/pipeline.h"
 #include "media/filters/video_renderer_base.h"
 
 namespace media {
@@ skipping 27 matching lines @@
       submit_reads_task_(NULL),
       preroll_complete_(false) {
 }
 
 VideoRendererBase::~VideoRendererBase() {
   Stop();
 }
 
 // static
 bool VideoRendererBase::IsMediaFormatSupported(
-    const MediaFormat* media_format) {
+    const MediaFormat& media_format) {
   int width;
   int height;
   return ParseMediaFormat(media_format, &width, &height);
 }
 
 // static
-bool VideoRendererBase::ParseMediaFormat(const MediaFormat* media_format,
+bool VideoRendererBase::ParseMediaFormat(const MediaFormat& media_format,
                                          int* width_out,
                                          int* height_out) {
-  DCHECK(media_format && width_out && height_out);
+  DCHECK(width_out && height_out);
   std::string mime_type;
-  return (media_format->GetAsString(MediaFormat::kMimeType, &mime_type) &&
+  return (media_format.GetAsString(MediaFormat::kMimeType, &mime_type) &&
           mime_type.compare(mime_type::kUncompressedVideo) == 0 &&
-          media_format->GetAsInteger(MediaFormat::kWidth, width_out) &&
-          media_format->GetAsInteger(MediaFormat::kHeight, height_out));
+          media_format.GetAsInteger(MediaFormat::kWidth, width_out) &&
+          media_format.GetAsInteger(MediaFormat::kHeight, height_out));
 }
 
 void VideoRendererBase::Stop() {
   OnStop();
   AutoLock auto_lock(lock_);
   DiscardAllFrames();
   if (submit_reads_task_) {
     // The task is owned by the message loop, so we don't delete it here.  We
     // know the task won't call us because we canceled it, and we know we are
     // on the pipeline thread, since we're in the filter's Stop method, so there
     // is no threading problem.  Just let the task be run by the message loop
     // and then be killed.
     submit_reads_task_->Cancel();
     submit_reads_task_ = NULL;
   }
   decoder_ = NULL;
 }
 
 bool VideoRendererBase::Initialize(VideoDecoder* decoder) {
   int width, height;
   decoder_ = decoder;
-  if (ParseMediaFormat(decoder_->GetMediaFormat(), &width, &height) &&
+  if (ParseMediaFormat(decoder_->media_format(), &width, &height) &&
       OnInitialize(width, height)) {
     host_->SetVideoSize(width, height);
     host_->SetTimeUpdateCallback(
         NewCallback(this, &VideoRendererBase::TimeUpdateCallback));
     SubmitReads();
     return true;
   }
   decoder_ = NULL;
   return false;
 }
@@ skipping 123 matching lines @@
                           (*frame_out)->GetDuration();
       } else {
         time_next_frame = queue_[1]->GetTimestamp();
       }
     }
     host_->ScheduleTimeUpdateCallback(time_next_frame);
   }
 }
 
 }  // namespace