NQE: Use ResponseHeaders to get the HTTP status code

net/nqe/network_quality_estimator_unittest.cc

 // Copyright 2015 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.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <limits>
 #include <map>
 #include <memory>
 #include <string>
 #include <utility>
 #include <vector>
 
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/metrics/histogram_samples.h"
 #include "base/run_loop.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/test/histogram_tester.h"
 #include "base/test/simple_test_tick_clock.h"
 #include "base/threading/platform_thread.h"
 #include "base/time/time.h"
 #include "build/build_config.h"
 #include "net/base/load_flags.h"
 #include "net/base/network_change_notifier.h"
+#include "net/http/http_response_headers.h"
+#include "net/http/http_response_info.h"
 #include "net/http/http_status_code.h"
 #include "net/nqe/effective_connection_type.h"
 #include "net/nqe/external_estimate_provider.h"
 #include "net/nqe/network_quality_estimator_test_util.h"
 #include "net/nqe/network_quality_observation.h"
 #include "net/nqe/network_quality_observation_source.h"
 #include "net/nqe/observation_buffer.h"
 #include "net/socket/socket_performance_watcher.h"
 #include "net/socket/socket_performance_watcher_factory.h"
 #include "net/url_request/url_request.h"
@@ skipping 2139 matching lines @@
 
     // Get the bits at index 11-17 which contain the downstream throughput.
     EXPECT_EQ(test.expected_downstream_throughput_kbps,
               (buckets.at(0).min >> kBitsPerMetric >> kBitsPerMetric) % 128);
 
     // Get the bits at index 18-24 which contain the resource fetch time.
     EXPECT_LE(0, (buckets.at(0).min >> kBitsPerMetric) % 128);
 
     // Get the bits at index 25-31 which contain the resource load size.
     EXPECT_LE(0, (buckets.at(0).min) % 128);
+
+    // Start another main-frame request which is redirected to an HTTPS URL.
+    // Redirection should not cause any crashes.
+    std::unique_ptr<URLRequest> request_3(context.CreateRequest(
+        estimator.GetRedirectURL(), DEFAULT_PRIORITY, &test_delegate));
+    request_3->Start();
+    base::RunLoop().Run();
+    EXPECT_FALSE(request_3->original_url().SchemeIsCryptographic());
+    EXPECT_TRUE(request_3->url().SchemeIsCryptographic());
+    EXPECT_TRUE(!request_3->response_info().headers.get() ||
+                request_3->response_info().headers->response_code() != HTTP_OK);
+    // Correlation metric should not be logged for redirected requests.
+    histogram_tester.ExpectTotalCount(
+        "NQE.Correlation.ResourceLoadTime.0Kb_128Kb", 1);
   }
 }
 
 class TestNetworkQualitiesCacheObserver
     : public nqe::internal::NetworkQualityStore::NetworkQualitiesCacheObserver {
  public:
   TestNetworkQualitiesCacheObserver()
       : network_id_(net::NetworkChangeNotifier::CONNECTION_UNKNOWN,
                     std::string()),
         notification_received_(0) {}
@@ skipping 106 matching lines @@
     if (expected_count == 1) {
       EffectiveConnectionType last_notified_type =
           observer.effective_connection_types().at(
               observer.effective_connection_types().size() - 1);
       EXPECT_EQ(i, last_notified_type);
     }
   }
 }
 
 }  // namespace net