Make Callback::Reset() return a copy to support use-cases where Run() ends up modifying |*this|.

media/filters/gpu_video_decoder.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/filters/gpu_video_decoder.h"
 
 #include "base/bind.h"
 #include "base/message_loop.h"
 #include "base/stl_util.h"
 #include "media/base/demuxer_stream.h"
@@ skipping 501 matching lines @@
   if (!vda_)
     return;
 
   DCHECK(ready_video_frames_.empty());
 
   // This needs to happen after the Reset() on vda_ is done to ensure pictures
   // delivered during the reset can find their time data.
   input_buffer_time_data_.clear();
 
   if (!pending_reset_cb_.is_null())
-    ResetAndRunCB(&pending_reset_cb_);
+    pending_reset_cb_.ResetAndRun();
 
   if (!pending_read_cb_.is_null())
     EnqueueFrameAndTriggerFrameDelivery(VideoFrame::CreateEmptyFrame());
 }
 
 void GpuVideoDecoder::NotifyError(media::VideoDecodeAccelerator::Error error) {
   if (!gvd_loop_proxy_->BelongsToCurrentThread()) {
     gvd_loop_proxy_->PostTask(FROM_HERE, base::Bind(
         &GpuVideoDecoder::NotifyError, this, error));
     return;
   }
   if (!vda_)
     return;
   vda_ = NULL;
   DLOG(ERROR) << "VDA Error: " << error;
   if (host())
     host()->SetError(PIPELINE_ERROR_DECODE);
 }
 
 }  // namespace media