Turns out args should not be decoded twice.

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

Bug: 4110115
Change-Id: I42fff7e9227423b9b5ce94234ad6d606234fe252

diff --git a/WebKit/android/jni/WebCoreFrameBridge.cpp b/WebKit/android/jni/WebCoreFrameBridge.cpp
index 49eac3c..7d1adb0 100644 (file)
--- 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);