Changeset 236486 in webkit

Timestamp:

Sep 25, 2018 4:21:25 PM (6 years ago)

Author:

jiewen_tan@apple.com

Message:

Unreviewed, a quick fix after r236481.

• UIProcess/WebsiteData/WebsiteDataStore.cpp:

(WebKit::WebsiteDataStore::setMockWebAuthenticationConfiguration):
Add an early return, so we don't end up with setting MockWebAuthenticationConfiguration twice.

Location:

trunk/Source/WebKit

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/WebsiteData/WebsiteDataStore.cpp (modified) (1 diff)

trunk/Source/WebKit/ChangeLog

@@ +1 @@
+2018-09-25  Jiewen Tan  <jiewen_tan@apple.com>
+
+        Unreviewed, a quick fix after r236481.
+
+        * UIProcess/WebsiteData/WebsiteDataStore.cpp:
+        (WebKit::WebsiteDataStore::setMockWebAuthenticationConfiguration):
+        Add an early return, so we don't end up with setting MockWebAuthenticationConfiguration twice.
+
 2018-09-25  John Wilander  <wilander@apple.com>
 

trunk/Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.cpp

@@ -1663 +1663 @@
 void WebsiteDataStore::setMockWebAuthenticationConfiguration(MockWebAuthenticationConfiguration&& configuration)
 {
-    if (!m_authenticatorManager->isMock())
+    if (!m_authenticatorManager->isMock()) {
         m_authenticatorManager = makeUniqueRef<MockAuthenticatorManager>(WTFMove(configuration));
+        return;
+    }
     static_cast<MockAuthenticatorManager*>(&m_authenticatorManager)->setTestConfiguration(WTFMove(configuration));
 }