Use more C++17
[WebKit-https.git] / Source / WebKit / ChangeLog
index 35dc9aa..2005a22 100644 (file)
@@ -1,3 +1,173 @@
+2018-05-21  Yusuke Suzuki  <utatane.tea@gmail.com>
+
+        Use more C++17
+        https://bugs.webkit.org/show_bug.cgi?id=185176
+
+        Reviewed by JF Bastien.
+
+        * Configurations/Base.xcconfig:
+        * DerivedSources.make:
+
+2018-05-21  Carlos Garcia Campos  <cgarcia@igalia.com>
+
+        [GTK][Wayland] UI process crash when closing the window
+        https://bugs.webkit.org/show_bug.cgi?id=185818
+
+        Reviewed by Michael Catanzaro.
+
+        This happens when a page containing a text field is loaded but the focus remains in the url bar when the window
+        is closed. This is because we are sending a notify-in to the IM context, but the focus is still in the URL
+        bar. That confuses the wayland input method manager that tries to free the text of the web view IM context that has
+        already been deleted.
+
+        * UIProcess/gtk/InputMethodFilter.cpp:
+        (WebKit::InputMethodFilter::setEnabled): Only send notify-in if the view is actually focused.
+
+2018-05-21  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Remove unused and no-op WKContextSetCookieStorageDirectory
+        https://bugs.webkit.org/show_bug.cgi?id=185857
+
+        Reviewed by Youenn Fablet.
+
+        Deleted C API which didn't do anything useful, and consequently not used by anyone.
+
+        * UIProcess/API/C/WKContext.cpp:
+        (WKContextSetCookieStorageDirectory): Deleted.
+        * UIProcess/API/C/WKContextPrivate.h:
+        * UIProcess/WebProcessPool.h:
+
+2018-05-21  Chris Nardi  <cnardi@chromium.org>
+
+        Remove dead exception in MediaList.appendMedium
+        https://bugs.webkit.org/show_bug.cgi?id=185278
+
+        Reviewed by Chris Dumez.
+
+        Remove code pertaining to an exception being thrown by appendMedium().
+
+        * WebProcess/InjectedBundle/API/gtk/DOM/WebKitDOMMediaList.cpp:
+        (webkit_dom_media_list_append_medium):
+
+2018-05-21  Aditya Keerthi  <akeerthi@apple.com>
+
+        [iOS] Click events only fire once when editing
+        https://bugs.webkit.org/show_bug.cgi?id=185777
+
+        Reviewed by Tim Horton.
+
+        gestureRecognizerShouldBegin: was returning false for the single tap gesture when a node was being
+        edited. This is an artifact of how the gesture was previously handled with the text selection assistant.
+        This condition is now removed, allowing the single tap gesture to go through and correctly propagate the
+        click event.
+
+        Also added an early return to _didGetTapHighlightForRequest: in order to prevent the tap highlight from
+        being shown when the node is already being assisted.
+
+        * UIProcess/ios/WKContentViewInteraction.mm:
+        (-[WKContentView _didGetTapHighlightForRequest:color:quads:topLeftRadius:topRightRadius:bottomLeftRadius:bottomRightRadius:]):
+        (-[WKContentView gestureRecognizerShouldBegin:]):
+
+2018-05-21  Daniel Bates  <dabates@apple.com>
+
+        REGRESSION (r231107): CSP report-only policies are ignored for beacon, importScripts, fetch(), EventSource, and XHR
+        https://bugs.webkit.org/show_bug.cgi?id=185789
+        <rdar://problem/40380175>
+
+        Reviewed by Andy Estes.
+
+        Have NetworkLoadChecker implement the ContentSecurityPolicyClient interface and support logging
+        console messages, sending CSP reports, and dispatching SecurityPolicyViolation events.
+
+        * NetworkProcess/NetworkConnectionToWebProcess.cpp:
+        (WebKit::NetworkConnectionToWebProcess::loadPing):
+        * NetworkProcess/NetworkLoadChecker.cpp:
+        (WebKit::NetworkLoadChecker::NetworkLoadChecker): Modified to take a reference to the NetworkConnectionToWebProcess,
+        the web page ID, the web frame ID, and the resource load identifier. These details are necessary
+        in order to implement the ContentSecurityPolicyClient interface.
+        (WebKit::NetworkLoadChecker::isAllowedByContentSecurityPolicy): Added.
+        (WebKit::NetworkLoadChecker::continueCheckingRequest): Write in terms of isAllowedByContentSecurityPolicy().
+        (WebKit::NetworkLoadChecker::contentSecurityPolicy): Pass ourself as the client so that we receive
+        delegate callbacks.
+        (WebKit::NetworkLoadChecker::addConsoleMessage): Added.
+        (WebKit::NetworkLoadChecker::sendCSPViolationReport): Added.
+        (WebKit::NetworkLoadChecker::enqueueSecurityPolicyViolationEvent): Added.
+        * NetworkProcess/NetworkLoadChecker.h:
+        * NetworkProcess/NetworkResourceLoader.cpp:
+        (NetworkResourceLoader::enqueueSecurityPolicyViolationEvent): Added.
+        * NetworkProcess/NetworkResourceLoader.h:
+        * NetworkProcess/PingLoad.cpp:
+        (WebKit::PingLoad::PingLoad): Modified to take a reference to the NetworkConnectionToWebProcess and pass
+        this through to the NetworkLoadChecker along with the web page ID, web frame ID and resource load identifier.
+        * NetworkProcess/PingLoad.h:
+        * WebProcess/WebPage/WebPage.cpp:
+        (WebKit::WebPage::enqueueSecurityPolicyViolationEvent): Added.
+        * WebProcess/WebPage/WebPage.h:
+        * WebProcess/WebPage/WebPage.messages.in: Add message EnqueueSecurityPolicyViolationEvent.
+
+2018-05-21  Brian Burg  <bburg@apple.com>
+
+        Web Automation: always return an empty cookie list if document.cookieURL() is empty
+        https://bugs.webkit.org/show_bug.cgi?id=185838
+        <rdar://problem/37737526>
+
+        Reviewed by Tim Horton.
+
+        * WebProcess/Automation/WebAutomationSessionProxy.cpp:
+        (WebKit::WebAutomationSessionProxy::getCookiesForFrame):
+        This crashes in CFNetwork code because an empty cookie URL is not a valid input.
+        Just return an empty list since there couldn't be any cookies returned.
+
+2018-05-21  Brian Burg  <bburg@apple.com>
+
+        Web Automation: terminate the automation session if the network or storage process crashes
+        https://bugs.webkit.org/show_bug.cgi?id=185827
+        <rdar://problem/40424020>
+
+        Reviewed by Tim Horton.
+
+        If one of the processes crashes, the page may be in an undefined state and
+        automation will fail in unpredictable ways. It's better to just give up immediately.
+
+        * UIProcess/WebProcessPool.cpp:
+        (WebKit::WebProcessPool::networkProcessFailedToLaunch):
+        (WebKit::WebProcessPool::storageProcessCrashed):
+
+2018-05-21  Sihui Liu  <sihui_liu@apple.com>
+
+        Add a diskCacheSizeOverride accessor function on WKContextConfigurationRef
+        https://bugs.webkit.org/show_bug.cgi?id=185826
+        <rdar://problem/39732113>
+
+        Reviewed by Alex Christensen.
+
+        * UIProcess/API/C/WKContextConfigurationRef.cpp:
+        (WKContextConfigurationDiskCacheSizeOverride):
+        (WKContextConfigurationSetDiskCacheSizeOverride):
+        * UIProcess/API/C/WKContextConfigurationRef.h:
+
+2018-05-21  Jer Noble  <jer.noble@apple.com>
+
+        Complete fix for enabling modern EME by default
+        https://bugs.webkit.org/show_bug.cgi?id=185770
+        <rdar://problem/40368220>
+
+        Reviewed by Eric Carlson.
+
+        * Configurations/FeatureDefines.xcconfig:
+
+2018-05-21  Sam Weinig  <sam@webkit.org>
+
+        Modernize RenderStyleConstants.h - Part 1
+        https://bugs.webkit.org/show_bug.cgi?id=185809
+
+        Reviewed by Yusuke Suzuki.
+
+        * WebProcess/WebPage/ios/WebPageIOS.mm:
+        (WebKit::WebPage::rangeForWebSelectionAtPosition):
+        (WebKit::WebPage::getPositionInformation):
+        Update for new enum names.
+
 2018-05-21  Carlos Garcia Campos  <cgarcia@igalia.com>
 
         Unreviewed. Update OptionsGTK.cmake and NEWS for 2.21.2 release.