From 6885acdefe22e146f5443ef67be43c7ee9fdcb10 Mon Sep 17 00:00:00 2001
From: Patrick Scott <phanna@android.com>
Date: Thu, 17 Mar 2011 13:43:29 -0400
Subject: Turns out args should not be decoded twice.

This should help out in some cases with redirect/login loops.

Bug: 4110115
Change-Id: I42fff7e9227423b9b5ce94234ad6d606234fe252
---
 WebKit/android/jni/WebCoreFrameBridge.cpp | 3 ---
 1 file changed, 3 deletions(-)

(limited to 'WebKit/android')

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);
-- 
cgit v1.1