diff options
| author | Steve Block <steveblock@google.com> | 2010-12-17 14:14:31 +0000 |
|---|---|---|
| committer | Steve Block <steveblock@google.com> | 2010-12-17 14:15:50 +0000 |
| commit | 131b7162d14fa221abb6ce77275244687c3f4bbb (patch) | |
| tree | d1ad5b9fd85ea0b4ae953bf1c9c742b320caba47 /WebKit/android/jni/WebCoreResourceLoader.cpp | |
| parent | 74427af10b1794f0e99b7e7a451c4a40902c98f4 (diff) | |
| download | external_webkit-131b7162d14fa221abb6ce77275244687c3f4bbb.zip external_webkit-131b7162d14fa221abb6ce77275244687c3f4bbb.tar.gz external_webkit-131b7162d14fa221abb6ce77275244687c3f4bbb.tar.bz2 | |
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
Diffstat (limited to 'WebKit/android/jni/WebCoreResourceLoader.cpp')
| -rw-r--r-- | WebKit/android/jni/WebCoreResourceLoader.cpp | 7 |
1 files changed, 2 insertions, 5 deletions
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, |