From cadac62e5c2b9f5fef59ce0326bb2cd79ffbe622 Mon Sep 17 00:00:00 2001 From: "rsleevi@chromium.org" Date: Tue, 11 Jun 2013 16:46:36 +0000 Subject: Call scoped_refptr::get() rather than relying on implicit "operator T*" This upates calls to bound temporary objects to also use get(). While it has the same semantic equivalence to the existing code, this generally represents a dangerous pattern - indeed, part of the whole motivation for this change is to make this anti-pattern very visible to authors. This change simply updates all of the call sites, to allow the "operator T*" to be removed and preventing new instances. The existing instances will then be reviewed for "suspicious" changes and updated to use/pass scoped_refptr rather than T*, as appropriate. BUG=110610 TBR=darin Review URL: https://codereview.chromium.org/15984016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@205560 0039d316-1c4b-4281-b951-d872f2087c98 --- content/browser/speech/speech_recognizer_impl_unittest.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'content/browser/speech/speech_recognizer_impl_unittest.cc') diff --git a/content/browser/speech/speech_recognizer_impl_unittest.cc b/content/browser/speech/speech_recognizer_impl_unittest.cc index 8c7c2d7..04a4724 100644 --- a/content/browser/speech/speech_recognizer_impl_unittest.cc +++ b/content/browser/speech/speech_recognizer_impl_unittest.cc @@ -57,7 +57,7 @@ class SpeechRecognizerImplTest : public SpeechRecognitionEventListener, recognizer_ = new SpeechRecognizerImpl( this, kTestingSessionId, kOneShotMode, sr_engine); audio_manager_.reset(new media::MockAudioManager( - base::MessageLoop::current()->message_loop_proxy())); + base::MessageLoop::current()->message_loop_proxy().get())); recognizer_->SetAudioManagerForTests(audio_manager_.get()); int audio_packet_length_bytes = -- cgit v1.1