NQE: Use ResponseHeaders to get the HTTP status code

net/nqe/network_quality_estimator_test_util.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 "net/nqe/network_quality_estimator_test_util.h"
 
 #include "base/files/file_path.h"
 #include "base/run_loop.h"
 #include "net/base/load_flags.h"
 #include "net/test/embedded_test_server/http_response.h"
@@ skipping 66 matching lines @@
   http_response->set_code(HTTP_OK);
   http_response->set_content("hello");
   http_response->set_content_type("text/plain");
   return std::move(http_response);
 }
 
 const GURL TestNetworkQualityEstimator::GetEchoURL() const {
   return embedded_test_server_.GetURL("/echo.html");
 }
 
+const GURL TestNetworkQualityEstimator::GetRedirectURL() const {
+  return embedded_test_server_.GetURL("/redirect302-to-https");

[mmenke, 2016/11/08 21:52:56]

Doesn't look like the embedded test server is correctly configured to use the
directory with this file in it, so it's just returning the default response.

You need to use "PathService::Get(base::DIR_SOURCE_ROOT, &path);", and then
append net/data/url_request_unittest to that in your test_util code.

[tbansal1, 2016/11/09 04:59:18]

Fixed but in a different way.

+}
+
 EffectiveConnectionType
 TestNetworkQualityEstimator::GetEffectiveConnectionType() const {
   if (effective_connection_type_)
     return effective_connection_type_.value();
   return NetworkQualityEstimator::GetEffectiveConnectionType();
 }
 
 EffectiveConnectionType
 TestNetworkQualityEstimator::GetRecentEffectiveConnectionType(
     const base::TimeTicks& start_time) const {
@@ skipping 91 matching lines @@
 double TestNetworkQualityEstimator::RandDouble() const {
   return rand_double_;
 }
 
 nqe::internal::NetworkID TestNetworkQualityEstimator::GetCurrentNetworkID()
     const {
   return nqe::internal::NetworkID(current_network_type_, current_network_id_);
 }
 
 }  // namespace net