Android media: VideoFrame should not store so many sync points.

content/browser/media/capture/desktop_capture_device_aura.cc

Index: content/browser/media/capture/desktop_capture_device_aura.cc
diff --git a/content/browser/media/capture/desktop_capture_device_aura.cc b/content/browser/media/capture/desktop_capture_device_aura.cc
index 1880e3a7b035642e9512d08cc0979fe753f9d598..1032f7eef162b7355eb6431f88c401b1907142a1 100644
--- a/content/browser/media/capture/desktop_capture_device_aura.cc
+++ b/content/browser/media/capture/desktop_capture_device_aura.cc
@@ -296,14 +296,9 @@ void CopyOutputFinishedForVideo(
 }
 
 void RunSingleReleaseCallback(scoped_ptr<cc::SingleReleaseCallback> cb,
-                              const std::vector<uint32>& sync_points) {
+                              uint32 sync_point) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-  GLHelper* gl_helper = ImageTransportFactory::GetInstance()->GetGLHelper();
-  DCHECK(gl_helper);
-  for (unsigned i = 0; i < sync_points.size(); i++)
-    gl_helper->WaitSyncPoint(sync_points[i]);
-  uint32 new_sync_point = gl_helper->InsertSyncPoint();
-  cb->Run(new_sync_point, false);
+  cb->Run(sync_point, false);

[no sievers, 2014/07/10 18:52:29]

maybe remove this function since it doesn't do any extra work anymore and use
|cb| directly?

[dshwang, 2014/07/10 19:23:11]

you're right. I'll update.

 }
 
 void DesktopVideoCaptureMachine::DidCopyOutput(