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