Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(896)

Unified Diff: content/browser/service_worker/service_worker_url_request_job_unittest.cc

Issue 877623002: [ServiceWorker] Fills SSLInfo of the response from a SW with the SSLInfo of the SW script. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/service_worker/service_worker_url_request_job_unittest.cc
diff --git a/content/browser/service_worker/service_worker_url_request_job_unittest.cc b/content/browser/service_worker/service_worker_url_request_job_unittest.cc
index 28986dea607564b6864f13bfc6349eff63b75249..8103b8f4faeb55c884ec9295dc65560637a9c5f3 100644
--- a/content/browser/service_worker/service_worker_url_request_job_unittest.cc
+++ b/content/browser/service_worker/service_worker_url_request_job_unittest.cc
@@ -31,8 +31,11 @@
#include "content/public/test/test_browser_context.h"
#include "content/public/test/test_browser_thread_bundle.h"
#include "net/base/io_buffer.h"
+#include "net/base/test_data_directory.h"
#include "net/http/http_request_headers.h"
#include "net/http/http_response_headers.h"
+#include "net/ssl/ssl_info.h"
+#include "net/test/cert_test_util.h"
#include "net/url_request/url_request.h"
#include "net/url_request/url_request_context.h"
#include "net/url_request/url_request_job_factory_impl.h"
@@ -125,6 +128,15 @@ class ServiceWorkerURLRequestJobTest : public testing::Test {
GURL("http://example.com/service_worker.js"),
1L,
helper_->context()->AsWeakPtr());
+ net::HttpResponseInfo http_info;
+ http_info.ssl_info.cert =
+ net::ImportCertFromFile(net::GetTestCertsDirectory(),
+ "ok_cert.pem");
+ EXPECT_TRUE(http_info.ssl_info.is_valid());
+ http_info.ssl_info.security_bits = 0x100;
+ // SSL3 TLS_DHE_RSA_WITH_AES_256_CBC_SHA
+ http_info.ssl_info.connection_status = 0x300039;
+ version_->SetMainScriptHttpResponseInfo(http_info);
scoped_ptr<ServiceWorkerProviderHost> provider_host(
new ServiceWorkerProviderHost(
@@ -180,6 +192,10 @@ class ServiceWorkerURLRequestJobTest : public testing::Test {
EXPECT_EQ(expected_status_text,
request_->response_headers()->GetStatusText());
EXPECT_EQ(expected_response, url_request_delegate_.response_data());
+ const net::SSLInfo& ssl_info = request_->response_info().ssl_info;
+ EXPECT_TRUE(ssl_info.is_valid());
+ EXPECT_EQ(ssl_info.security_bits, 0x100);
+ EXPECT_EQ(ssl_info.connection_status, 0x300039);
}
bool HasInflightRequests() {

Powered by Google App Engine
This is Rietveld 408576698