summaryrefslogtreecommitdiffstats
path: root/WebKit
diff options
context:
space:
mode:
authorPatrick Scott <phanna@android.com>2011-03-17 13:43:29 -0400
committerPatrick Scott <phanna@android.com>2011-03-17 13:43:29 -0400
commit6885acdefe22e146f5443ef67be43c7ee9fdcb10 (patch)
treef0aa0605bfd555381f264a623382fcca308e0c16 /WebKit
parent9f5143f9ae49a8e5fdb7ea626c4efad66096b020 (diff)
downloadexternal_webkit-6885acdefe22e146f5443ef67be43c7ee9fdcb10.zip
external_webkit-6885acdefe22e146f5443ef67be43c7ee9fdcb10.tar.gz
external_webkit-6885acdefe22e146f5443ef67be43c7ee9fdcb10.tar.bz2
Turns out args should not be decoded twice.
This should help out in some cases with redirect/login loops. Bug: 4110115 Change-Id: I42fff7e9227423b9b5ce94234ad6d606234fe252
Diffstat (limited to 'WebKit')
-rw-r--r--WebKit/android/jni/WebCoreFrameBridge.cpp3
1 files changed, 0 insertions, 3 deletions
diff --git a/WebKit/android/jni/WebCoreFrameBridge.cpp b/WebKit/android/jni/WebCoreFrameBridge.cpp
index 49eac3c..7d1adb0 100644
--- a/WebKit/android/jni/WebCoreFrameBridge.cpp
+++ b/WebKit/android/jni/WebCoreFrameBridge.cpp
@@ -1023,9 +1023,6 @@ void WebFrame::autoLogin(const std::string& loginHeader)
if (realm.isEmpty() || args.isEmpty())
return;
- // Args is double-encoded as it contains urls.
- args = WebCore::decodeURLEscapeSequences(args);
-
JNIEnv* env = getJNIEnv();
jstring jRealm = wtfStringToJstring(env, realm, true);
jstring jAccount = wtfStringToJstring(env, account);