From 131b7162d14fa221abb6ce77275244687c3f4bbb Mon Sep 17 00:00:00 2001 From: Steve Block Date: Fri, 17 Dec 2010 14:14:31 +0000 Subject: Don't strip HTTP response headers with empty values This change affects the Android HTTP stack only. We now match the XHR spec - http://www.w3.org/TR/XMLHttpRequest/#the-getresponseheader-method - and provide header strings even if they are empty. Bug: 3294032 Change-Id: Ic42e5e97e0a86ae2d08680ad689a2ab249d6ecc7 --- WebKit/android/jni/WebCoreResourceLoader.cpp | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) (limited to 'WebKit/android/jni/WebCoreResourceLoader.cpp') diff --git a/WebKit/android/jni/WebCoreResourceLoader.cpp b/WebKit/android/jni/WebCoreResourceLoader.cpp index 0c7bad9..d44df4c 100644 --- a/WebKit/android/jni/WebCoreResourceLoader.cpp +++ b/WebKit/android/jni/WebCoreResourceLoader.cpp @@ -141,11 +141,8 @@ void WebCoreResourceLoader::SetResponseHeader(JNIEnv* env, jobject obj, jint nat LOG_ASSERT(response, "nativeSetResponseHeader must take a valid response pointer!"); LOG_ASSERT(key, "How did a null value become a key?"); - if (val) { - WTF::String valStr = jstringToWtfString(env, val); - if (!valStr.isEmpty()) - response->setHTTPHeaderField(jstringToWtfString(env, key), valStr); - } + if (val) + response->setHTTPHeaderField(jstringToWtfString(env, key), jstringToWtfString(env, val)); } jint WebCoreResourceLoader::CreateResponse(JNIEnv* env, jobject obj, jstring url, jint statusCode, -- cgit v1.1