Title: [247655] trunk/Source/WebCore
Revision
247655
Author
aes...@apple.com
Date
2019-07-19 15:30:47 -0700 (Fri, 19 Jul 2019)

Log Message

[Apple Pay] Log less in PaymentCoordinator::shouldEnableApplePayAPIs()
https://bugs.webkit.org/show_bug.cgi?id=199958

Reviewed by Tim Horton.

Only log when APIs are disabled due to user scripts.

* Modules/applepay/PaymentCoordinator.cpp:
(WebCore::PaymentCoordinator::shouldEnableApplePayAPIs const):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (247654 => 247655)


--- trunk/Source/WebCore/ChangeLog	2019-07-19 22:11:42 UTC (rev 247654)
+++ trunk/Source/WebCore/ChangeLog	2019-07-19 22:30:47 UTC (rev 247655)
@@ -1,3 +1,15 @@
+2019-07-19  Andy Estes  <aes...@apple.com>
+
+        [Apple Pay] Log less in PaymentCoordinator::shouldEnableApplePayAPIs()
+        https://bugs.webkit.org/show_bug.cgi?id=199958
+
+        Reviewed by Tim Horton.
+
+        Only log when APIs are disabled due to user scripts.
+
+        * Modules/applepay/PaymentCoordinator.cpp:
+        (WebCore::PaymentCoordinator::shouldEnableApplePayAPIs const):
+
 2019-07-19  Dean Jackson  <d...@apple.com>
 
         Add a menu item to toggle between showing and hiding link previews

Modified: trunk/Source/WebCore/Modules/applepay/PaymentCoordinator.cpp (247654 => 247655)


--- trunk/Source/WebCore/Modules/applepay/PaymentCoordinator.cpp	2019-07-19 22:11:42 UTC (rev 247654)
+++ trunk/Source/WebCore/Modules/applepay/PaymentCoordinator.cpp	2019-07-19 22:30:47 UTC (rev 247655)
@@ -258,10 +258,8 @@
 
 bool PaymentCoordinator::shouldEnableApplePayAPIs(Document& document) const
 {
-    if (m_client.supportsUnrestrictedApplePay()) {
-        RELEASE_LOG_IF_ALLOWED("shouldEnableApplePayAPIs() -> true (unrestricted client)");
+    if (m_client.supportsUnrestrictedApplePay())
         return true;
-    }
 
     bool shouldEnableAPIs = true;
     document.page()->userContentProvider().forEachUserScript([&](DOMWrapperWorld&, const UserScript&) {
@@ -268,7 +266,9 @@
         shouldEnableAPIs = false;
     });
 
-    RELEASE_LOG_IF_ALLOWED("shouldEnableApplePayAPIs() -> %d", shouldEnableAPIs);
+    if (!shouldEnableAPIs)
+        RELEASE_LOG_IF_ALLOWED("shouldEnableApplePayAPIs() -> false (user scripts)");
+
     return shouldEnableAPIs;
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to