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

Unified Diff: components/cronet/android/test/native_test_server.cc

Issue 1085903002: Enable Sdch in Cronet (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: constants Created 5 years, 8 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: components/cronet/android/test/native_test_server.cc
diff --git a/components/cronet/android/test/native_test_server.cc b/components/cronet/android/test/native_test_server.cc
index 97261645d361c60480eb00597df1b5e8b7c562b7..ce8c95153845286f36953e25c212413d66503cca 100644
--- a/components/cronet/android/test/native_test_server.cc
+++ b/components/cronet/android/test/native_test_server.cc
@@ -4,14 +4,25 @@
#include "native_test_server.h"
+#include <string>
+
#include "base/android/jni_android.h"
#include "base/android/jni_string.h"
+#include "base/android/path_utils.h"
#include "base/bind.h"
#include "base/files/file_path.h"
+#include "base/files/file_util.h"
+#include "base/macros.h"
#include "base/memory/scoped_ptr.h"
#include "base/path_service.h"
#include "base/strings/string_util.h"
+#include "base/strings/stringprintf.h"
+#include "components/cronet/android/cronet_url_request_context_adapter.h"
+#include "components/cronet/android/url_request_context_adapter.h"
#include "jni/NativeTestServer_jni.h"
+#include "net/base/url_util.h"
+#include "net/dns/host_resolver_impl.h"
+#include "net/dns/mock_host_resolver.h"
#include "net/http/http_status_code.h"
#include "net/test/embedded_test_server/embedded_test_server.h"
#include "net/test/embedded_test_server/http_request.h"
@@ -22,22 +33,66 @@ namespace cronet {
namespace {
-const char echo_body_path[] = "/echo_body";
-const char echo_header_path[] = "/echo_header";
-const char echo_all_headers_path[] = "/echo_all_headers";
-const char echo_method_path[] = "/echo_method";
-const char redirect_to_echo_body_path[] = "/redirect_to_echo_body";
+const char kEchoBodyPath[] = "/echo_body";
+const char kEchoHeaderPath[] = "/echo_header";
+const char kEchoAllHeadersPath[] = "/echo_all_headers";
+const char kEchoMethodPath[] = "/echo_method";
+const char kRedirectToEchoBodyPath[] = "/redirect_to_echo_body";
+const char kFakeSdchDomain[] = "fake.sdch.domain";
+// Path that advertises the dictionary passed in query params if client
+// supports Sdch encoding. E.g. /sdch/index?q=LeQxM80O will make the server
+// responds with "Get-Dictionary: /sdch/dict/LeQxM80O".
+const char kSdchPath[] = "/sdch/index";
+// Path that returns encoded response if client has the right dictionary.
+const char kSdchTestPath[] = "/sdch/test";
+// Path where dictionaries are stored.
+const char kSdchDictPath[] = "/sdch/dict/";
net::test_server::EmbeddedTestServer* g_test_server = nullptr;
-scoped_ptr<net::test_server::HttpResponse> UploadServerRequestHandler(
+class CustomHttpResponse : public net::test_server::HttpResponse {
+ public:
+ CustomHttpResponse(const std::string& headers, const std::string& contents)
+ : headers_(headers), contents_(contents) {}
+
+ std::string ToResponseString() const override {
+ return headers_ + "\r\n" + contents_;
+ }
+
+ void AddHeader(const std::string& key_value_pair) {
+ headers_.append(base::StringPrintf("%s\r\n", key_value_pair.c_str()));
+ }
+
+ private:
+ std::string headers_;
+ std::string contents_;
+
+ DISALLOW_COPY_AND_ASSIGN(CustomHttpResponse);
+};
+
+scoped_ptr<CustomHttpResponse> ConstructResponseBasedOnFile(
+ const base::FilePath& file_path) {
+ std::string file_contents;
+ bool read_file = base::ReadFileToString(file_path, &file_contents);
+ DCHECK(read_file);
+ base::FilePath headers_path(
+ file_path.AddExtension(FILE_PATH_LITERAL("mock-http-headers")));
+ std::string headers_contents;
+ bool read_headers = base::ReadFileToString(headers_path, &headers_contents);
+ DCHECK(read_headers);
+ scoped_ptr<CustomHttpResponse> http_response(
+ new CustomHttpResponse(headers_contents, file_contents));
+ return http_response.Pass();
+}
+
+scoped_ptr<net::test_server::HttpResponse> NativeTestServerRequestHandler(
const net::test_server::HttpRequest& request) {
DCHECK(g_test_server);
scoped_ptr<net::test_server::BasicHttpResponse> response(
new net::test_server::BasicHttpResponse());
response->set_content_type("text/plain");
- if (request.relative_url == echo_body_path) {
+ if (request.relative_url == kEchoBodyPath) {
if (request.has_content) {
response->set_content(request.content);
} else {
@@ -46,7 +101,7 @@ scoped_ptr<net::test_server::HttpResponse> UploadServerRequestHandler(
return response.Pass();
}
- if (StartsWithASCII(request.relative_url, echo_header_path, true)) {
+ if (StartsWithASCII(request.relative_url, kEchoHeaderPath, true)) {
GURL url = g_test_server->GetURL(request.relative_url);
auto it = request.headers.find(url.query());
if (it != request.headers.end()) {
@@ -57,19 +112,66 @@ scoped_ptr<net::test_server::HttpResponse> UploadServerRequestHandler(
return response.Pass();
}
- if (request.relative_url == echo_all_headers_path) {
+ if (request.relative_url == kEchoAllHeadersPath) {
response->set_content(request.all_headers);
return response.Pass();
}
- if (request.relative_url == echo_method_path) {
+ if (request.relative_url == kEchoMethodPath) {
response->set_content(request.method_string);
return response.Pass();
}
- if (request.relative_url == redirect_to_echo_body_path) {
+ if (request.relative_url == kRedirectToEchoBodyPath) {
response->set_code(net::HTTP_TEMPORARY_REDIRECT);
- response->AddCustomHeader("Location", echo_body_path);
+ response->AddCustomHeader("Location", kEchoBodyPath);
+ return response.Pass();
+ }
+
+ // Unhandled requests result in the Embedded test server sending a 404.
+ return scoped_ptr<net::test_server::BasicHttpResponse>();
+}
+
+scoped_ptr<net::test_server::HttpResponse> SdchRequestHandler(
+ const net::test_server::HttpRequest& request) {
+ DCHECK(g_test_server);
+ base::FilePath dir_path;
+ bool get_data_dir = base::android::GetDataDirectory(&dir_path);
+ DCHECK(get_data_dir);
+ dir_path = dir_path.Append(FILE_PATH_LITERAL("test"));
+
+ if (StartsWithASCII(request.relative_url, kSdchPath, true)) {
+ base::FilePath file_path = dir_path.Append("sdch/index");
+ scoped_ptr<CustomHttpResponse> response =
+ ConstructResponseBasedOnFile(file_path).Pass();
+ // Check for query params to see which dictionary to advertise.
+ // For instance, ?q=dictionaryA will make the server advertise dictionaryA.
+ GURL url = g_test_server->GetURL(request.relative_url);
+ std::string dictionary;
+ if (!net::GetValueForKeyInQuery(url, "q", &dictionary)) {
+ CHECK(false) << "dictionary is not found in query params of "
+ << request.relative_url;
+ }
+ auto accept_encoding_header = request.headers.find("Accept-Encoding");
+ if (accept_encoding_header != request.headers.end()) {
+ if (accept_encoding_header->second.find("sdch") != std::string::npos)
+ response->AddHeader(base::StringPrintf(
+ "Get-Dictionary: %s%s", kSdchDictPath, dictionary.c_str()));
+ }
+ return response.Pass();
+ }
+
+ if (StartsWithASCII(request.relative_url, kSdchTestPath, true)) {
+ auto avail_dictionary_header = request.headers.find("Avail-Dictionary");
+ if (avail_dictionary_header != request.headers.end()) {
+ base::FilePath file_path = dir_path.Append(
+ "sdch/" + avail_dictionary_header->second + "_encoded");
+ return ConstructResponseBasedOnFile(file_path).Pass();
+ }
+ scoped_ptr<net::test_server::BasicHttpResponse> response(
+ new net::test_server::BasicHttpResponse());
+ response->set_content_type("text/plain");
+ response->set_content("Sdch is not used.\n");
return response.Pass();
}
@@ -77,6 +179,30 @@ scoped_ptr<net::test_server::HttpResponse> UploadServerRequestHandler(
return scoped_ptr<net::test_server::BasicHttpResponse>();
}
+void RegisterHostResolverProcHelper(
+ net::URLRequestContext* url_request_context) {
+ net::HostResolverImpl* resolver =
+ static_cast<net::HostResolverImpl*>(url_request_context->host_resolver());
+ scoped_refptr<net::RuleBasedHostResolverProc> proc =
+ new net::RuleBasedHostResolverProc(NULL);
+ proc->AddRule(kFakeSdchDomain, "127.0.0.1");
+ resolver->set_proc_params_for_test(
+ net::HostResolverImpl::ProcTaskParams(proc.get(), 1u));
+ JNIEnv* env = base::android::AttachCurrentThread();
+ Java_NativeTestServer_onHostResolverProcRegistered(env);
+}
+
+void RegisterHostResolverProcOnNetworkThread(
+ CronetURLRequestContextAdapter* context_adapter) {
+ RegisterHostResolverProcHelper(context_adapter->GetURLRequestContext());
+}
+
+// TODO(xunjieli): Delete this once legacy API is removed.
+void RegisterHostResolverProcOnNetworkThreadLegacyAPI(
+ URLRequestContextAdapter* context_adapter) {
+ RegisterHostResolverProcHelper(context_adapter->GetURLRequestContext());
+}
+
} // namespace
jboolean StartNativeTestServer(JNIEnv* env,
@@ -87,15 +213,36 @@ jboolean StartNativeTestServer(JNIEnv* env,
return false;
g_test_server = new net::test_server::EmbeddedTestServer();
g_test_server->RegisterRequestHandler(
- base::Bind(&UploadServerRequestHandler));
- // Add a second handler for paths that UploadServerRequestHandler does not
- // handle.
+ base::Bind(&NativeTestServerRequestHandler));
+ g_test_server->RegisterRequestHandler(base::Bind(&SdchRequestHandler));
base::FilePath test_files_root(
base::android::ConvertJavaStringToUTF8(env, jtest_files_root));
+
+ // Add a third handler for paths that NativeTestServerRequestHandler does not
+ // handle.
g_test_server->ServeFilesFromDirectory(test_files_root);
return g_test_server->InitializeAndWaitUntilReady();
}
+void RegisterHostResolverProc(JNIEnv* env,
+ jclass jcaller,
+ jlong jadapter,
+ jboolean jlegacy_api) {
+ if (jlegacy_api == JNI_TRUE) {
+ URLRequestContextAdapter* context_adapter =
+ reinterpret_cast<URLRequestContextAdapter*>(jadapter);
+ context_adapter->PostTaskToNetworkThread(
+ FROM_HERE, base::Bind(&RegisterHostResolverProcOnNetworkThreadLegacyAPI,
+ base::Unretained(context_adapter)));
+ } else {
+ CronetURLRequestContextAdapter* context_adapter =
+ reinterpret_cast<CronetURLRequestContextAdapter*>(jadapter);
+ context_adapter->PostTaskToNetworkThread(
+ FROM_HERE, base::Bind(&RegisterHostResolverProcOnNetworkThread,
+ base::Unretained(context_adapter)));
+ }
+}
+
void ShutdownNativeTestServer(JNIEnv* env, jclass jcaller) {
if (!g_test_server)
return;
@@ -105,13 +252,13 @@ void ShutdownNativeTestServer(JNIEnv* env, jclass jcaller) {
jstring GetEchoBodyURL(JNIEnv* env, jclass jcaller) {
DCHECK(g_test_server);
- GURL url = g_test_server->GetURL(echo_body_path);
+ GURL url = g_test_server->GetURL(kEchoBodyPath);
return base::android::ConvertUTF8ToJavaString(env, url.spec()).Release();
}
jstring GetEchoHeaderURL(JNIEnv* env, jclass jcaller, jstring jheader) {
DCHECK(g_test_server);
- GURL url = g_test_server->GetURL(echo_header_path);
+ GURL url = g_test_server->GetURL(kEchoHeaderPath);
GURL::Replacements replacements;
std::string header = base::android::ConvertJavaStringToUTF8(env, jheader);
replacements.SetQueryStr(header.c_str());
@@ -121,19 +268,19 @@ jstring GetEchoHeaderURL(JNIEnv* env, jclass jcaller, jstring jheader) {
jstring GetEchoAllHeadersURL(JNIEnv* env, jclass jcaller) {
DCHECK(g_test_server);
- GURL url = g_test_server->GetURL(echo_all_headers_path);
+ GURL url = g_test_server->GetURL(kEchoAllHeadersPath);
return base::android::ConvertUTF8ToJavaString(env, url.spec()).Release();
}
jstring GetEchoMethodURL(JNIEnv* env, jclass jcaller) {
DCHECK(g_test_server);
- GURL url = g_test_server->GetURL(echo_method_path);
+ GURL url = g_test_server->GetURL(kEchoMethodPath);
return base::android::ConvertUTF8ToJavaString(env, url.spec()).Release();
}
jstring GetRedirectToEchoBody(JNIEnv* env, jclass jcaller) {
DCHECK(g_test_server);
- GURL url = g_test_server->GetURL(redirect_to_echo_body_path);
+ GURL url = g_test_server->GetURL(kRedirectToEchoBodyPath);
return base::android::ConvertUTF8ToJavaString(env, url.spec()).Release();
}
@@ -144,6 +291,13 @@ jstring GetFileURL(JNIEnv* env, jclass jcaller, jstring jfile_path) {
return base::android::ConvertUTF8ToJavaString(env, url.spec()).Release();
}
+jstring GetSdchURL(JNIEnv* env, jclass jcaller) {
+ DCHECK(g_test_server);
+ std::string url(base::StringPrintf("http://%s:%d", kFakeSdchDomain,
+ g_test_server->port()));
+ return base::android::ConvertUTF8ToJavaString(env, url).Release();
+}
+
bool RegisterNativeTestServer(JNIEnv* env) {
return RegisterNativesImpl(env);
}

Powered by Google App Engine
This is Rietveld 408576698