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

content/browser/renderer_host/image_transport_factory_android.cc

Index: content/browser/renderer_host/image_transport_factory_android.cc
diff --git a/content/browser/renderer_host/image_transport_factory_android.cc b/content/browser/renderer_host/image_transport_factory_android.cc
index cf3ff8b1b7c2e8f6be68cb89a2d001935f55a6b3..7b0d37cf9ba5b0014e587149b556a05f8005cb6c 100644
--- a/content/browser/renderer_host/image_transport_factory_android.cc
+++ b/content/browser/renderer_host/image_transport_factory_android.cc
@@ -104,6 +104,13 @@ GLHelper* CmdBufferImageTransportFactory::GetGLHelper() {
 }  // anonymous namespace
 
 // static
+void ImageTransportFactoryAndroid::InitializeForUnitTests(

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

Do we need a method for teardown also?

With unit tests it can create this nasty problems when you leak static objects
depending on how they run. 

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

Ah, you're right. I mimic this method after
ImageTransportFactory::InitializeForUnitTests(scoped_ptr<ImageTransportFactory>
factory).
ImageTransportFactory class has Terminate() method for this reason!. I'll
update.

+    ImageTransportFactoryAndroid* test_factory) {
+  DCHECK(!g_factory);
+  g_factory = test_factory;
+}
+
+// static
 ImageTransportFactoryAndroid* ImageTransportFactoryAndroid::GetInstance() {
   if (!g_factory)
     g_factory = new CmdBufferImageTransportFactory();