Call scoped_refptr<T>::get() rather than relying on implicit "operator T*"

media/filters/video_renderer_base_unittest.cc

 // Copyright (c) 2012 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 "base/bind.h"
 #include "base/callback.h"
 #include "base/callback_helpers.h"
 #include "base/debug/stack_trace.h"
 #include "base/message_loop.h"
 #include "base/stl_util.h"
@@ skipping 491 matching lines @@
   Flush();
   QueuePrerollFrames(kFrameDurationInMs * 6);
 
   Preroll(kFrameDurationInMs * 6 + 1, PIPELINE_OK);
   EXPECT_EQ(kFrameDurationInMs * 6, GetCurrentTimestampInMs());
   Shutdown();
 }
 
 TEST_F(VideoRendererBaseTest, GetCurrentFrame_Initialized) {
   Initialize();
-  EXPECT_TRUE(GetCurrentFrame());  // Due to prerolling.
+  EXPECT_TRUE(GetCurrentFrame().get());  // Due to prerolling.
   Shutdown();
 }
 
 TEST_F(VideoRendererBaseTest, GetCurrentFrame_Playing) {
   Initialize();
   Play();
-  EXPECT_TRUE(GetCurrentFrame());
+  EXPECT_TRUE(GetCurrentFrame().get());
   Shutdown();
 }
 
 TEST_F(VideoRendererBaseTest, GetCurrentFrame_Paused) {
   Initialize();
   Play();
   Pause();
-  EXPECT_TRUE(GetCurrentFrame());
+  EXPECT_TRUE(GetCurrentFrame().get());
   Shutdown();
 }
 
 TEST_F(VideoRendererBaseTest, GetCurrentFrame_Flushed) {
   Initialize();
   Play();
   Pause();
 
   // Frame shouldn't be updated.
   ResetCurrentFrame();
   Flush();
-  EXPECT_FALSE(GetCurrentFrame());
+  EXPECT_FALSE(GetCurrentFrame().get());
 
   Shutdown();
 }
 
 TEST_F(VideoRendererBaseTest, GetCurrentFrame_EndOfStream) {
   Initialize();
   Play();
   Pause();
   Flush();
 
   // Preroll only end of stream frames.
   QueueEndOfStream();
 
   // Frame shouldn't be updated.
   ResetCurrentFrame();
   Preroll(0, PIPELINE_OK);
-  EXPECT_FALSE(GetCurrentFrame());
+  EXPECT_FALSE(GetCurrentFrame().get());
 
   // Start playing, we should immediately get notified of end of stream.
   Play();
   WaitForEnded();
 
   Shutdown();
 }
 
 TEST_F(VideoRendererBaseTest, GetCurrentFrame_Shutdown) {
   Initialize();
 
   // Frame shouldn't be updated.
   ResetCurrentFrame();
   Shutdown();
-  EXPECT_FALSE(GetCurrentFrame());
+  EXPECT_FALSE(GetCurrentFrame().get());
 }
 
 // Stop() is called immediately during an error.
 TEST_F(VideoRendererBaseTest, GetCurrentFrame_Error) {
   Initialize();
 
   // Frame shouldn't be updated.
   ResetCurrentFrame();
   Stop();
-  EXPECT_FALSE(GetCurrentFrame());
+  EXPECT_FALSE(GetCurrentFrame().get());
 }
 
 // Verify that a late decoder response doesn't break invariants in the renderer.
 TEST_F(VideoRendererBaseTest, StopDuringOutstandingRead) {
   Initialize();
   Play();
 
   // Advance time a bit to trigger a Read().
   AdvanceTimeInMs(kFrameDurationInMs);
   WaitForPendingRead();
@@ skipping 50 matching lines @@
   InSequence s;
 
   EXPECT_CALL(*decoder_, Initialize(_, _, _))
       .WillOnce(RunCallback<1>(DECODER_ERROR_NOT_SUPPORTED));
   InitializeRenderer(DECODER_ERROR_NOT_SUPPORTED);
 
   Stop();
 }
 
 }  // namespace media