Log error messages in windows video capture

media/video/capture/win/video_capture_device_mf_win.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 "media/video/capture/win/video_capture_device_mf_win.h"
 
 #include <mfapi.h>
 #include <mferror.h>
 
 #include "base/memory/ref_counted.h"
@@ skipping 314 matching lines @@
       // fail with HRESULT_FROM_WINHRESULT_FROM_WIN32(ERROR_INVALID_FUNCTION).
       // It's not clear to me why this is, but it is possible that it has
       // something to do with this bug:
       // http://support.microsoft.com/kb/979567
       OnError(hr);
     }
   }
 }
 
 void VideoCaptureDeviceMFWin::OnError(HRESULT hr) {
-  std::string log_msg = base::StringPrintf("VideoCaptureDeviceMFWin: %x", hr);
-  DLOG(ERROR) << log_msg;
+  const std::string reason =

[tommi (sloooow) - chröme, 2014/08/28 08:48:46]

move this variable into the scope below.
(alternatively, you can skip using an explicit variable and pass the return
value of StringPrintf() to OnError())

+      base::StringPrintf("VideoCaptureDeviceMFWin: %x", hr);
   if (client_.get())
-    client_->OnError(log_msg);
+    client_->OnError(reason);
 }
 
 }  // namespace media