Title: [238936] trunk/Source/WebKit
Revision
238936
Author
[email protected]
Date
2018-12-06 11:35:35 -0800 (Thu, 06 Dec 2018)

Log Message

Unreviewed, a quick fix after r238919

Add an early return such that the error message is actually meaningful.

* UIProcess/WebAuthentication/Mock/MockHidService.cpp:
(WebKit::MockHidService::platformStartDiscovery):

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (238935 => 238936)


--- trunk/Source/WebKit/ChangeLog	2018-12-06 19:32:38 UTC (rev 238935)
+++ trunk/Source/WebKit/ChangeLog	2018-12-06 19:35:35 UTC (rev 238936)
@@ -1,3 +1,12 @@
+2018-12-06  Jiewen Tan  <[email protected]>
+
+        Unreviewed, a quick fix after r238919
+
+        Add an early return such that the error message is actually meaningful.
+
+        * UIProcess/WebAuthentication/Mock/MockHidService.cpp:
+        (WebKit::MockHidService::platformStartDiscovery):
+
 2018-12-06  Alex Christensen  <[email protected]>
 
         Remove unused LoaderStrategy::storeDerivedDataToCache and associated dead code

Modified: trunk/Source/WebKit/UIProcess/WebAuthentication/Mock/MockHidService.cpp (238935 => 238936)


--- trunk/Source/WebKit/UIProcess/WebAuthentication/Mock/MockHidService.cpp	2018-12-06 19:32:38 UTC (rev 238935)
+++ trunk/Source/WebKit/UIProcess/WebAuthentication/Mock/MockHidService.cpp	2018-12-06 19:35:35 UTC (rev 238936)
@@ -41,8 +41,10 @@
 
 void MockHidService::platformStartDiscovery()
 {
-    if (!!m_configuration.hid)
+    if (!!m_configuration.hid) {
         deviceAdded(nullptr);
+        return;
+    }
     LOG_ERROR("No hid authenticators is available.");
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to