Post GpuMemoryBufferVideoFramePool deletion on media thread

content/renderer/media/media_stream_video_renderer_sink.cc

Index: content/renderer/media/media_stream_video_renderer_sink.cc
diff --git a/content/renderer/media/media_stream_video_renderer_sink.cc b/content/renderer/media/media_stream_video_renderer_sink.cc
index 709f9d51d8f11139c35d11b2eb15235c36a52848..ade92b326b3f1822a422a3c726d65db777b23a80 100644
--- a/content/renderer/media/media_stream_video_renderer_sink.cc
+++ b/content/renderer/media/media_stream_video_renderer_sink.cc
@@ -40,6 +40,10 @@ MediaStreamVideoRendererSink::MediaStreamVideoRendererSink(
 }
 
 MediaStreamVideoRendererSink::~MediaStreamVideoRendererSink() {
+  if (gpu_memory_buffer_pool_) {
+    media_task_runner_->DeleteSoon(FROM_HERE,
+                                   gpu_memory_buffer_pool_.release());
+  }
 }
 
 void MediaStreamVideoRendererSink::Start() {
@@ -109,6 +113,9 @@ void MediaStreamVideoRendererSink::OnVideoFrame(
     return;
   }
 
+  // |gpu_memory_buffer_pool_| would be deleted on |media_task_runner_| in dtor,

[Daniele Castagna, 2016/02/10 21:33:46]

Maybe something like
|gpu_memory_buffer_pool_| deletion is going to be posted to |media_task_runner_|
...
base::Unretained() usage is fine since |gpu_memory_buffer_pool_| outlives the
task.

[emircan, 2016/02/10 23:04:55]

Done.

+  // so we know that |gpu_memory_buffer_pool_| is alive when this task is
+  // posted and base::Unretained() usage should be fine.
   media_task_runner_->PostTask(
       FROM_HERE,
       base::Bind(