[Cronet] Copy certificates before using in tests

components/cronet/android/test/src/org/chromium/net/QuicTestServer.java

Index: components/cronet/android/test/src/org/chromium/net/QuicTestServer.java
diff --git a/components/cronet/android/test/src/org/chromium/net/QuicTestServer.java b/components/cronet/android/test/src/org/chromium/net/QuicTestServer.java
index 0e9c58788cbafb9b473606c2dd2da07fd6b01011..4745e7ff195ceaf6f76af8d6da77692755db2799 100644
--- a/components/cronet/android/test/src/org/chromium/net/QuicTestServer.java
+++ b/components/cronet/android/test/src/org/chromium/net/QuicTestServer.java
@@ -7,6 +7,7 @@ package org.chromium.net;
 import android.content.Context;
 import android.os.ConditionVariable;
 
+import org.chromium.base.ContextUtils;
 import org.chromium.base.Log;
 import org.chromium.base.annotations.CalledByNative;
 import org.chromium.base.annotations.JNINamespace;
@@ -73,6 +74,7 @@ public final class QuicTestServer {
     }
 
     public static long createMockCertVerifier() {
+        TestFilesInstaller.installIfNeeded(ContextUtils.getApplicationContext());

[xunjieli, 2016/12/12 19:37:21]

startQuicTestServer() already calls installIfNeeded(). Is it enough? 

[pauljensen, 2016/12/12 19:52:42]

No, this a public API with no documented or enforced ordering requirements.
Tests call QuicTestServer.createMockCertVerifier() without starting a
QuicTestServer, for example all BidirectionalStreamTest's use the setUp()
function which does not start a QuicServer but calls
QuicTestServer.createMockCertVerifier().

         return MockCertVerifier.createMockCertVerifier(CERTS_USED, true);
     }