Text can wrap between hyphens and commas
[WebKit-https.git] / Source / WebCore / ChangeLog
index 123deee..21a24ef 100644 (file)
@@ -1,3 +1,548 @@
+2018-05-22  Myles C. Maxfield  <mmaxfield@apple.com>
+
+        Text can wrap between hyphens and commas
+        https://bugs.webkit.org/show_bug.cgi?id=185899
+        <rdar://problem/40118983>
+
+        Reviewed by Zalan Bujtas.
+
+        Chrome, Firefox, and ICU all agree that text shouldn't wrap there.
+
+        Test: fast/text/line-break-hyphen-comma.html
+
+        * rendering/BreakLines.cpp:
+
+2018-05-22  Simon Fraser  <simon.fraser@apple.com>
+
+        Add notifyutil callbacks to dump the memory cache, and the list of live Pages and Documents
+        https://bugs.webkit.org/show_bug.cgi?id=185853
+
+        Reviewed by Keith Miller.
+
+        Add a notifyutil callback to dump the PageCache, which dumps the stats, and the list of live pages.
+        This can be invoked on iOS and macOS via "notifyutil -p com.apple.WebKit.showPageCache".
+        
+        Add a notifyutil callback that dumps the list of all Pages, and the list of all Documents, with
+        Document pointer address and URL.
+        This can be invoked on iOS and macOS via "notifyutil -p com.apple.WebKit.showAllDocuments".
+
+        * history/PageCache.cpp:
+        (WebCore::PageCache::PageCache):
+        (WebCore::PageCache::dump const):
+        * history/PageCache.h:
+        * loader/cache/CachedResource.h:
+        (WebCore::CachedResource::numberOfClients const):
+        (WebCore::CachedResource::count const): Deleted.
+        * loader/cache/MemoryCache.cpp:
+        (WebCore::MemoryCache::MemoryCache):
+        (WebCore::MemoryCache::pruneLiveResourcesToSize):
+        (WebCore::MemoryCache::dumpStats):
+        (WebCore::MemoryCache::dumpLRULists const):
+        * loader/cache/MemoryCache.h:
+        * page/mac/PageMac.mm:
+        (WebCore::Page::platformInitialize):
+
+2018-05-22  Dean Jackson  <dino@apple.com>
+
+        Optimized path zoom animation needs a valid UIImage and CGRect
+        https://bugs.webkit.org/show_bug.cgi?id=185883
+        <rdar://problem/40306056>
+
+        Build fix.
+
+        * loader/FrameLoadRequest.h:
+
+2018-05-22  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r232081.
+        https://bugs.webkit.org/show_bug.cgi?id=185895
+
+        Broke two API tests (Requested by bfulgham_ on #webkit).
+
+        Reverted changeset:
+
+        "Avoid keeping FormState alive longer than necessary"
+        https://bugs.webkit.org/show_bug.cgi?id=185877
+        https://trac.webkit.org/changeset/232081
+
+2018-05-22  Dean Jackson  <dino@apple.com>
+
+        Optimized path zoom animation needs a valid UIImage and CGRect
+        https://bugs.webkit.org/show_bug.cgi?id=185883
+        <rdar://problem/40306056>
+
+        Reviewed by Jon Lee.
+
+        Pass the bounding box of the element that was clicked onto
+        the UI process, so it can perform an animation from that spot.
+
+        This involved adding an IntRect to the ResourceRequest, and passing
+        that info into it from the HTMLAnchorElement, using a new struct
+        called SystemPreviewInfo.
+
+        * html/HTMLAnchorElement.cpp:
+        (WebCore::HTMLAnchorElement::handleClick):
+        * loader/FrameLoadRequest.cpp:
+        (WebCore::FrameLoadRequest::FrameLoadRequest):
+        * loader/FrameLoadRequest.h:
+        (WebCore::FrameLoadRequest::FrameLoadRequest):
+        (WebCore::FrameLoadRequest::isSystemPreview const):
+        (WebCore::FrameLoadRequest::systemPreviewRect const):
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::urlSelected):
+        (WebCore::FrameLoader::loadURL):
+        * loader/FrameLoader.h:
+        (WebCore::FrameLoader::urlSelected):
+        * loader/FrameLoaderTypes.h:
+        * platform/network/ResourceRequestBase.cpp:
+        (WebCore::ResourceRequestBase::systemPreviewRect const):
+        (WebCore::ResourceRequestBase::setSystemPreviewRect):
+        * platform/network/ResourceRequestBase.h:
+
+2018-05-22  Chris Dumez  <cdumez@apple.com>
+
+        [POSIX] Use access() instead of stat() in FileSystem::fileExists()
+        https://bugs.webkit.org/show_bug.cgi?id=185882
+
+        Reviewed by Geoffrey Garen.
+
+        Use access() instead of stat() in FileSystem::fileExists(). stat() returns a lot of information we
+        do not leverage and local benchmarking on macOS shows access() being > 80% faster than stat():
+        stat: 0.31567 (lower is better)
+        access: 0.16074 (lower is better)
+        stat: 0.303665 (lower is better)
+        access: 0.165468 (lower is better)
+
+        * platform/posix/FileSystemPOSIX.cpp:
+        (WebCore::FileSystem::fileExists):
+
+2018-05-22  Brent Fulgham  <bfulgham@apple.com>
+
+        Avoid keeping FormState alive longer than necessary
+        https://bugs.webkit.org/show_bug.cgi?id=185877
+        <rdar://problem/39329219>
+
+        Reviewed by Ryosuke Niwa.
+
+        A number of crash fixes were done to prevent FormState objects from being
+        accessed after their relevant Frames had been destroyed. Unfortunately, this
+        could cause the FormState to persist after the owning Frame had been
+        destroyed, resulting in nullptr dereferences.
+
+        This patch does the following:
+
+        1. Changes to use WeakPtr's for FormState objects passed to completion handlers,
+           rather than RefPtr, since those completion handlers might fire as part of
+           the clean-up process during Frame destruction. This allows us to use the FormState
+           if they are still valid, but gracefully handle cases where a form submission
+           is cancelled in-flight.
+        2. Removes some extraneous WTFMove() calls being made on bare FormState pointers.
+        3. Changes the trap from Bug 183704 so that it only fires if the FormState object
+           is being retained more than once.
+
+        * loader/DocumentLoader.cpp:
+        (WebCore::DocumentLoader::willSendRequest): Update for new CompletionHandler
+        signature.
+        * loader/FormState.cpp:
+        (WebCore::FormState::willDetachPage): Revise trap to check for retain counts
+        above one.
+        * loader/FormState.h:
+        (WebCore::FormState::weakPtrFactory const): Added.
+        * loader/FrameLoader.cpp:
+        (WebCore::FrameLoader::loadFrameRequest): Revise to use WeakPtr for FormState
+        passed to the completion handler.
+        (WebCore::FrameLoader::loadURL): Update for new CompletionHandler signature.
+        (WebCore::FrameLoader::load): Ditto.
+        (WebCore::FrameLoader::loadWithDocumentLoader): Ditto.
+        (WebCore::FrameLoader::loadPostRequest): Ditto.
+        * loader/PolicyChecker.cpp:
+        (WebCore::PolicyChecker::checkNavigationPolicy): Revise to use WeakPtr for
+        FormState passed to the completion handler. Remove some extraneous WTFMove()
+        calls on bare pointers.
+        (WebCore::PolicyChecker::checkNewWindowPolicy): Ditto.
+        * loader/PolicyChecker.h:
+
+2018-05-22  Sihui Liu  <sihui_liu@apple.com>
+
+        Conversion between SecurityOriginData and DatabaseIdentifier is asymmetric when port is null
+        https://bugs.webkit.org/show_bug.cgi?id=185715
+
+        Reviewed by Geoffrey Garen.
+
+        Fixed the issue of null port when converting between SecurityOriginData and DatabaseIdentifier.
+
+        Test: WKWebView.LocalStorageFetchDataRecords.
+
+        * page/SecurityOriginData.cpp:
+        (WebCore::SecurityOriginData::fromDatabaseIdentifier):
+
+2018-05-22  Keith Miller  <keith_miller@apple.com>
+
+        We should have a CoW storage for NewArrayBuffer arrays.
+        https://bugs.webkit.org/show_bug.cgi?id=185003
+
+        Reviewed by Filip Pizlo.
+
+        * bindings/js/JSDOMConvertSequences.h:
+        (WebCore::Detail::NumericSequenceConverter::convertArray):
+        (WebCore::Detail::SequenceConverter::convertArray):
+
+2018-05-22  Ryan Haddad  <ryanhaddad@apple.com>
+
+        Unreviewed, rolling out r232052.
+
+        Breaks internal builds.
+
+        Reverted changeset:
+
+        "Use more C++17"
+        https://bugs.webkit.org/show_bug.cgi?id=185176
+        https://trac.webkit.org/changeset/232052
+
+2018-05-22  Carlos Garcia Campos  <cgarcia@igalia.com>
+
+        [GTK] WebDriver: Network process crash when running imported/w3c/webdriver/tests/delete_cookie/delete.py::test_unknown_cookie
+        https://bugs.webkit.org/show_bug.cgi?id=185867
+
+        Reviewed by Michael Catanzaro.
+
+        We need to null check the value returned by URL::createSoupURI() before passing it to soup.
+
+        * platform/network/soup/CookieJarSoup.cpp:
+        (WebCore::setCookiesFromDOM):
+        (WebCore::cookiesForSession):
+        (WebCore::getRawCookies):
+        (WebCore::deleteCookie):
+        * platform/network/soup/NetworkStorageSessionSoup.cpp:
+        (WebCore::NetworkStorageSession::getCookies):
+
+2018-05-22  Zalan Bujtas  <zalan@apple.com>
+
+        [LFC] Implement positioning for non-replaced out-of-flow elements.
+        https://bugs.webkit.org/show_bug.cgi?id=185858
+
+        Reviewed by Antti Koivisto.
+
+        In certain cases, the out-of-flow element's final position depends on the element's size.
+        Call computeOutOfFlowPosition() after width/height are resolved.
+
+        * layout/FormattingContext.cpp:
+        (WebCore::Layout::FormattingContext::computeOutOfFlowPosition const):
+        (WebCore::Layout::FormattingContext::layoutOutOfFlowDescendants const):
+        (WebCore::Layout::FormattingContext::computeOutOfFlowNonReplacedPosition const):
+        (WebCore::Layout::FormattingContext::computeOutOfFlowReplacedPosition const):
+        * layout/FormattingContext.h:
+
+2018-05-22  Olivier Blin  <olivier.blin@softathome.com>
+
+        Fix build without MathML
+        https://bugs.webkit.org/show_bug.cgi?id=185865
+
+        Reviewed by Frédéric Wang.
+
+        Regression(r226654): [RenderTreeBuilder] Move MathML addChild logic to RenderTreeBuilder
+        https://bugs.webkit.org/show_bug.cgi?id=181443
+
+        Regression(r229694): Put the DOM in IsoHeaps
+        https://bugs.webkit.org/show_bug.cgi?id=183546
+
+        No new tests, build fix.
+
+        * mathml/MathMLUnknownElement.cpp:
+        * mathml/MathMLUnknownElement.h:
+        * rendering/updating/RenderTreeBuilder.cpp:
+        (WebCore::RenderTreeBuilder::RenderTreeBuilder):
+        (WebCore::RenderTreeBuilder::attach):
+        * rendering/updating/RenderTreeBuilder.h:
+        * rendering/updating/RenderTreeBuilderMathML.cpp:
+        * rendering/updating/RenderTreeBuilderMathML.h:
+
+2018-05-22  Yacine Bandou  <yacine.bandou_ext@softathome.com>
+
+        [GStreamer] Don't set the ReadyState to HaveNothing when an error occurs in playback pipeline
+        https://bugs.webkit.org/show_bug.cgi?id=185725
+
+        Reviewed by Philippe Normand.
+
+        The ReadyState should not be set to HaveNothing when an error occurs in playback pipeline, because
+        at least we should have the metadata in order to have an error in pipeline.
+
+        Here is the definition of HaveNothing state in W3C spec https://dev.w3.org/html5/spec-preview/media-elements.html#ready-states
+        "HAVE_NOTHING (numeric value 0): No information regarding the media resource is available. No data for the current
+        playback position is available. Media elements whose networkState attribute is NETWORK_EMPTY are always in the HAVE_NOTHING state."
+
+        In MSE case, this patch fixes the crashes of the followings WPT encrypted-media tests:
+        - clearkey-mp4-playback-temporary-clear-encrypted.https.html
+        - clearkey-mp4-playback-temporary-multikey-sequential.https.html
+        - clearkey-mp4-playback-temporary-multikey-sequential-readyState.https.html
+
+        Here is the cause of the crashes: When an error occurs in playback pipeline like no decipher key, in case of encrypted content,
+        the MediaPlayerPrivateGstreamer sets NetworkState to FormatError which causes the detachment of MediaElement from MediaSource,
+        then MediaPlayerPrivateGstreamer sets the ReadyState to HaveNothing which causes a trying again to play the same URI,
+        thus the crash occurs because the MediaElement is detached from MediaSource, see bugzilla for more details.
+        Note: these crashes should be fixed in 185242 but unfortunately it isn't the case. See bug 185242 for more details.
+
+        * platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:
+        (WebCore::MediaPlayerPrivateGStreamer::handleMessage):
+
+2018-05-22  Yacine Bandou  <yacine.bandou_ext@softathome.com>
+
+        [MSE][GStreamer] Fix the deadlock caused by bug 185242 (r231351)
+        https://bugs.webkit.org/show_bug.cgi?id=185723
+
+        Reviewed by Xabier Rodriguez-Calvar.
+
+        After a detailed investigation, we found that, the patch r231351 doesn't fix the crash that expected to fix,
+        it just replaces it by a deadlock. Now the crash is fixed in the bug 185725.
+
+        This Patch will remove a part of r231351 that causes the deadlock.
+
+        * platform/graphics/gstreamer/mse/WebKitMediaSourceGStreamer.cpp:
+        (webKitMediaSrcFreeStream):
+
+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:
+        * platform/ios/LegacyTileGrid.mm:
+        (WebCore::LegacyTileGrid::dropDistantTiles):
+
+2018-05-21  Zalan Bujtas  <zalan@apple.com>
+
+        [LFC] Box::isDescendantOf() should work with out-of-flow elements.
+        https://bugs.webkit.org/show_bug.cgi?id=185812
+
+        Reviewed by Antti Koivisto.
+
+        Use the containing block chain instead of the parent chain to check for isDescendantOf().
+        (containing block is not always the direct parent)
+
+        * layout/layouttree/LayoutBox.cpp:
+        (WebCore::Layout::Box::isDescendantOf const):
+
+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.
+
+        MediaList.appendMedium was able to throw an exception, but MediaQuerySet::add() always
+        returned true, making it impossible for that exception to be thrown. This matched the
+        spec, as |appendMedium| is not specified to throw an exception. Remove the dead code
+        surrounding the exception, and make MediaQuerySet::add() return false if the medium is
+        not added.
+
+        No new/modified tests as there should be no functional changes.
+
+        * css/MediaList.cpp:
+        (WebCore::MediaQuerySet::add):
+        (WebCore::MediaList::appendMedium):
+        * css/MediaList.h:
+        * css/MediaList.idl:
+
+2018-05-21  Chris Dumez  <cdumez@apple.com>
+
+        File's structured serialization should serialize lastModified attribute
+        https://bugs.webkit.org/show_bug.cgi?id=185773
+
+        Reviewed by Youenn Fablet.
+
+        Update our implementation for the stuctured serialization of a File to include
+        its lastModified attribute, as per:
+        - https://w3c.github.io/FileAPI/#file-section
+
+        No new tests, rebaselined existing test.
+
+        * bindings/js/SerializedScriptValue.cpp:
+        (WebCore::CloneSerializer::write):
+        (WebCore::CloneDeserializer::readFile):
+        * fileapi/File.cpp:
+        (WebCore::File::File):
+        * fileapi/File.h:
+
+2018-05-21  Zalan Bujtas  <zalan@apple.com>
+
+        [LFC] Add FormattingContext::validateGeometryConstraintsAfterLayout
+        https://bugs.webkit.org/show_bug.cgi?id=185811
+
+        Reviewed by Antti Koivisto.
+
+        Also implement LayoutDescendantIterator and fix LayoutIterator.
+
+        * WebCore.xcodeproj/project.pbxproj:
+        * layout/FormattingContext.cpp:
+        (WebCore::Layout::FormattingContext::validateGeometryConstraintsAfterLayout const):
+        * layout/FormattingContext.h:
+        * layout/blockformatting/BlockFormattingContext.cpp:
+        (WebCore::Layout::BlockFormattingContext::layout const):
+        * layout/layouttree/LayoutDescendantIterator.h: Added.
+        (WebCore::Layout::LayoutDescendantIterator<T>::LayoutDescendantIterator):
+        (WebCore::Layout::LayoutDescendantIterator<T>::operator):
+        (WebCore::Layout::LayoutDescendantIteratorAdapter<T>::LayoutDescendantIteratorAdapter):
+        (WebCore::Layout::LayoutDescendantIteratorAdapter<T>::begin):
+        (WebCore::Layout::LayoutDescendantIteratorAdapter<T>::end):
+        (WebCore::Layout::LayoutDescendantIteratorAdapter<T>::at):
+        (WebCore::Layout::descendantsOfType):
+        * layout/layouttree/LayoutIterator.h:
+        (WebCore::Layout::LayoutBoxTraversal::firstChild):
+        (WebCore::Layout::LayoutBoxTraversal::nextAncestorSibling):
+        (WebCore::Layout::LayoutBoxTraversal::next):
+        (WebCore::Layout::LayoutBoxTraversal::nextSkippingChildren):
+        (WebCore::Layout::Traversal::firstChild):
+        (WebCore::Layout::Traversal::nextSibling):
+        (WebCore::Layout::Traversal::previousSibling):
+        (WebCore::Layout::Traversal::findAncestorOfType):
+        (WebCore::Layout::Traversal::firstWithin):
+        (WebCore::Layout::Traversal::next):
+        (WebCore::Layout::LayoutIterator<T>::traversePreviousSibling):
+        (WebCore::Layout::LayoutIterator<T>::traverseAncestor):
+        (WebCore::Layout::Traversal::nextAncestorSibling): Deleted.
+        (WebCore::Layout::Traversal::nextWithin): Deleted.
+
+2018-05-21  Basuke Suzuki  <Basuke.Suzuki@sony.com>
+
+        [Curl] Bugfix for r231968, Suspend/Resume with WK2
+        https://bugs.webkit.org/show_bug.cgi?id=185830
+
+        The flags are needed to be updated even if it is cancelled or finished to cleanup correctly.
+        Also fixed trivial bug for very rare case (hard to produce the situation).
+
+        Reviewed by Youenn Fablet.
+
+        No new tests because WK2 is not ready for WinCairo yet in public., tested internaly.
+
+        * platform/network/curl/CurlRequest.cpp:
+        (WebCore::CurlRequest::suspend):
+        (WebCore::CurlRequest::resume):
+        (WebCore::CurlRequest::pausedStatusChanged):
+
+2018-05-21  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r232030.
+        https://bugs.webkit.org/show_bug.cgi?id=185850
+
+        "Caused
+        TestWebKitAPI.IndexedDB.StructuredCloneBackwardCompatibility
+        API test to fail" (Requested by cdumez on #webkit).
+
+        Reverted changeset:
+
+        "File's structured serialization should serialize lastModified
+        attribute"
+        https://bugs.webkit.org/show_bug.cgi?id=185773
+        https://trac.webkit.org/changeset/232030
+
+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.
+
+        Fixes an issue where CSP report-only policies were ignored for DocumentThreadableLoader and
+        PingLoad initiated loads as a result of moving CSP processing to NetworkProcess.
+
+        Have NetworkLoadChecker implement the ContentSecurityPolicyClient interface and support logging
+        console messages, sending CSP reports, and dispatching SecurityPolicyViolation events. To support
+        the latter we introduce a new WebPage message, EnqueueSecurityPolicyViolationEvent, to enqueue
+        a SecurityPolicyViolationEvent created from an event init dictionary on the document's event
+        dispatch queue.
+
+        Additionally, shorten the description for a ResourceError caused by CSP to "Blocked by Content Security Policy"
+        because the CSP code run in NetworkProcess can now log its more detailed error description to
+        Web Inspector.
+
+        Tests: http/tests/security/contentSecurityPolicy/connect-src-beacon-allowed.html
+               http/tests/security/contentSecurityPolicy/connect-src-beacon-blocked.html
+               http/tests/security/contentSecurityPolicy/report-only-connect-src-beacon-redirect-blocked.php
+               http/tests/security/contentSecurityPolicy/report-only-connect-src-xmlhttprequest-redirect-to-blocked.php
+
+        * WebCore.xcodeproj/project.pbxproj: Change SecurityPolicyViolationEvent.h from a project header to
+        a private header so that we can include it in WebKit code.
+        * dom/Document.cpp:
+        (WebCore::Document::enqueueSecurityPolicyViolationEvent): Added.
+        * dom/Document.h:
+
+        * dom/EventInit.h:
+        (WebCore::EventInit::encode const):
+        (WebCore::EventInit::decode
+        * dom/SecurityPolicyViolationEvent.h:
+        (WebCore::SecurityPolicyViolationEvent::Init::encode const):
+        (WebCore::SecurityPolicyViolationEvent::Init::decode):
+        Support encoding and decoding for the event.
+
+        * loader/DocumentLoader.cpp:
+        (WebCore::DocumentLoader::enqueueSecurityPolicyViolationEvent): Formerly named "dispatchSecurityPolicyViolationEvent".
+        (WebCore::DocumentLoader::dispatchSecurityPolicyViolationEvent): Deleted; renamed to "enqueueSecurityPolicyViolationEvent".
+        * loader/DocumentLoader.h:
+
+        * loader/DocumentThreadableLoader.cpp:
+        (WebCore::DocumentThreadableLoader::redirectReceived): While I am here, move the check for whether the loader
+        strategy took responsibility for performing security checks to be before we perform the CSP check to avoid doing
+        such CSP checks twice in the case that the loader strategy already did them.
+        (WebCore::DocumentThreadableLoader::didFail): Remove code that checked the CSP policy if the load failed. When
+        the loader strategy (NetworkProcess) is responsible for performing security checks then this code would never
+        be executed for a violation of a CSP report-only policy because the loader does not and should not fail the load
+        for a report-only violations. As the name implies, a report-only violation is only reported. That is, it is not
+        enforced such that the load is blocked; => fail the load.
+        (WebCore::DocumentThreadableLoader::reportContentSecurityPolicyError): Update the error description to more
+        accurately describe the error and be consistent with the error message used in NetworkProcess. This error
+        message is shown for a redirect blocked by CSP regardless of whether the redirect was to a same-origin or
+        cross-origin resource. I chose to make the error message more vague than necessary for simplicity because
+        the CSP code will log a more detailed message for this error than could ever be captured by error message
+        for the ResourceError. Also use ASCIILiteral to efficiently construct the String object for the error
+        message.
+
+        * page/csp/ContentSecurityPolicy.cpp:
+        (WebCore::ContentSecurityPolicy::reportViolation const): Build up a SecurityPolicyViolationEvent::Init and
+        pass that to the delegate to dispatch.
+        * page/csp/ContentSecurityPolicy.h: Export allowScriptFromSource() and allowChildContextFromSource() so that
+        we can call them from WebKit.
+        * page/csp/ContentSecurityPolicyClient.h: Update for renaming.
+        * platform/network/ResourceRequestBase.h: Define a new requester type to be able to differentiate a request
+        initiated by importScripts() from other requests. We use this to perform the appropriate CSP checks in NetworkProcess.
+        * workers/WorkerScriptLoader.cpp:
+        (WebCore::WorkerScriptLoader::loadSynchronously): Set the requester on the ResourceRequest to ResourceRequest::Requester::ImportScripts
+        so that we can differentiate this request from other requests. See remark for file ResourceRequestBase.h for
+        more details.
+
+2018-05-21  Chris Dumez  <cdumez@apple.com>
+
+        File's structured serialization should serialize lastModified attribute
+        https://bugs.webkit.org/show_bug.cgi?id=185773
+
+        Reviewed by Youenn Fablet.
+
+        Update our implementation for the stuctured serialization of a File to include
+        its lastModified attribute, as per:
+        - https://w3c.github.io/FileAPI/#file-section
+
+        No new tests, rebaselined existing test.
+
+        * bindings/js/SerializedScriptValue.cpp:
+        (WebCore::CloneSerializer::write):
+        (WebCore::CloneDeserializer::readFile):
+        * fileapi/File.cpp:
+        (WebCore::File::File):
+        * fileapi/File.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  Olivier Blin  <olivier.blin@softathome.com>
 
         [CMake][WebCore] fix sqlite include dir variable