[cronet] Add mechanism for restarting CronetEnvironment

components/grpc_support/bidirectional_stream_unittest.cc

 // Copyright 2016 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 <stdint.h>
 #include <list>
 #include <map>
 #include <string>
 
 #include "base/logging.h"
@@ skipping 18 matching lines @@
 const bidirectional_stream_header_array kTestHeadersArray = {2, 2,
                                                              kTestHeaders};
 }  // namespace
 
 namespace grpc_support {
 
 class BidirectionalStreamTest : public ::testing::TestWithParam<bool> {
  protected:
   void SetUp() override {
     StartQuicTestServer();
+    StartTestStreamEngine(GetQuicTestServerPort());
   }
 
   void TearDown() override {
     ShutdownQuicTestServer();
+    ShutdownTestStreamEngine();

[mef, 2017/04/26 22:19:41]

I'd move it above ShutdownQuicTestServer() to make it easier to reason about
their relative lifetimes.

[lilyhoughton, 2017/04/27 13:53:39]

Done.

   }
 
   BidirectionalStreamTest() {}
   ~BidirectionalStreamTest() override {}
 
   stream_engine* engine() {
     return GetTestStreamEngine(GetQuicTestServerPort());
   }
 
  private:
@@ skipping 595 matching lines @@
   ASSERT_EQ(TestBidirectionalStreamCallback::ON_FAILED, test.response_step);
   ASSERT_EQ(net::ERR_NAME_NOT_RESOLVED, test.net_error);
   bidirectional_stream_destroy(test.stream);
 }
 
 INSTANTIATE_TEST_CASE_P(BidirectionalStreamDelayRequestHeadersUntilFlush,
                         BidirectionalStreamTest,
                         ::testing::Values(true, false));
 
 }  // namespace grpc_support