WebKit-https.git
14 months agoAdd _drawsBackground to WKWebViewConfiguration.
timothy@apple.com [Sat, 14 Jul 2018 00:03:49 +0000 (00:03 +0000)]
Add _drawsBackground to WKWebViewConfiguration.
https://bugs.webkit.org/show_bug.cgi?id=187665
rdar://problem/42182268

Reviewed by Tim Horton.

Source/WebKit:

* UIProcess/API/APIPageConfiguration.cpp:
(API::PageConfiguration::copy const): Copy m_drawsBackground, and some missing values.
* UIProcess/API/APIPageConfiguration.h:
(API::PageConfiguration::drawsBackground const): Added.
(API::PageConfiguration::setDrawsBackground): Added.
* UIProcess/API/Cocoa/WKWebView.mm:
(-[WKWebView _initializeWithConfiguration:]): Transfer _drawsBackground to page config.
* UIProcess/API/Cocoa/WKWebViewConfiguration.mm:
(-[WKWebViewConfiguration init]): Set _drawsBackground to YES.
(-[WKWebViewConfiguration encodeWithCoder:]): Encode _drawsBackground.
(-[WKWebViewConfiguration initWithCoder:]): Decode _drawsBackground.
(-[WKWebViewConfiguration copyWithZone:]): Copy _drawsBackground.
(-[WKWebViewConfiguration _drawsBackground]): Added.
(-[WKWebViewConfiguration _setDrawsBackground:]): Added.
* UIProcess/API/Cocoa/WKWebViewConfigurationPrivate.h:
* UIProcess/WebPageProxy.cpp: Set m_drawsBackground based on configuration.

Tools:

* TestWebKitAPI/Tests/WebKitCocoa/Configuration.mm:
(TestWebKitAPI.WebKit.ConfigurationDrawsBackground): Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233828 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[WinCairo] Enable WebGL when Accelerated Compositing is disabled
chris.reid@sony.com [Fri, 13 Jul 2018 23:20:53 +0000 (23:20 +0000)]
[WinCairo] Enable WebGL when Accelerated Compositing is disabled
https://bugs.webkit.org/show_bug.cgi?id=187664

Reviewed by Fujii Hironori.

AC was disabled for WinCairo in r233725 but it can still run WebGL without AC.

* html/HTMLCanvasElement.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233827 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoWeb Inspector: REGRESSION (r195723): Improve support for resources with '\r' and...
mattbaker@apple.com [Fri, 13 Jul 2018 23:11:08 +0000 (23:11 +0000)]
Web Inspector: REGRESSION (r195723): Improve support for resources with '\r' and '\r\n' line endings
https://bugs.webkit.org/show_bug.cgi?id=186453
<rdar://problem/39689180>

Reviewed by Joseph Pecoraro.

Now that the frontend no longer uses offsets from the original source
file to calculate positions within CodeMirror, it is possible to support
resources with '\r' and '\r\n' line endings in the editor.

* UserInterface/Views/CodeMirrorEditor.js:
(WI.CodeMirrorEditor.create):
(WI.CodeMirrorEditor):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233824 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoCrash under ApplicationCacheGroup::didFailLoadingEntry()
cdumez@apple.com [Fri, 13 Jul 2018 22:54:16 +0000 (22:54 +0000)]
Crash under ApplicationCacheGroup::didFailLoadingEntry()
https://bugs.webkit.org/show_bug.cgi?id=187661
<rdar://problem/42179755>

Reviewed by Youenn Fablet.

If ApplicationCacheResourceLoader::create() fails synchronously with
ApplicationCacheResourceLoader::Error::CannotCreateResource error, then
m_entryLoader will be null when didFailLoadingEntry() is called. However,
didFailLoadingEntry() fails to null check m_entryLoader before using it.

* loader/appcache/ApplicationCacheGroup.cpp:
(WebCore::ApplicationCacheGroup::didFailLoadingEntry):
(WebCore::ApplicationCacheGroup::startLoadingEntry):
* loader/appcache/ApplicationCacheGroup.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233823 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoWebKit sometimes holds WiFi/BT assertions while the Networking process is suspended
timothy_horton@apple.com [Fri, 13 Jul 2018 22:35:48 +0000 (22:35 +0000)]
WebKit sometimes holds WiFi/BT assertions while the Networking process is suspended
https://bugs.webkit.org/show_bug.cgi?id=187662
<rdar://problem/41999448>

Reviewed by Chris Dumez.

Inform WiFiAssertions when the Networking process is first going into the background,
so it has a chance of dropping its assertions even in cases where the system
suspends the process before we receive prepareToSuspend.

* NetworkProcess/NetworkProcess.cpp:
(WebKit::NetworkProcess::processDidTransitionToForeground):
(WebKit::NetworkProcess::processDidTransitionToBackground):
* NetworkProcess/NetworkProcess.h:
* NetworkProcess/NetworkProcess.messages.in:
* NetworkProcess/curl/NetworkProcessCurl.cpp:
(WebKit::NetworkProcess::platformProcessDidTransitionToForeground):
(WebKit::NetworkProcess::platformProcessDidTransitionToBackground):
* NetworkProcess/soup/NetworkProcessSoup.cpp:
(WebKit::NetworkProcess::platformProcessDidTransitionToForeground):
(WebKit::NetworkProcess::platformProcessDidTransitionToBackground):
* UIProcess/Network/NetworkProcessProxy.cpp:
(WebKit::NetworkProcessProxy::sendProcessDidTransitionToForeground):
(WebKit::NetworkProcessProxy::sendProcessDidTransitionToBackground):
* UIProcess/Network/NetworkProcessProxy.h:
* UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::updateProcessAssertions):
Plumb the foreground/background transition to NetworkProcess.

* NetworkProcess/cocoa/NetworkProcessCocoa.mm:
(WebKit::NetworkProcess::platformPrepareToSuspend):
(WebKit::NetworkProcess::platformProcessDidTransitionToBackground):
(WebKit::NetworkProcess::platformProcessDidTransitionToForeground):
Make use of SuspensionReason to explain to WiFiAssertions the
difference between prepareToSuspend and didTransitionToBackground,
so that it can adjust the timing of dropping the assertion.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233822 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoAdd release assertion to check thread in TimerBase::setNextFireTime
achristensen@apple.com [Fri, 13 Jul 2018 22:33:35 +0000 (22:33 +0000)]
Add release assertion to check thread in TimerBase::setNextFireTime
https://bugs.webkit.org/show_bug.cgi?id=187666

Reviewed by Ryosuke Niwa.

This should give us insight into what is causing <rdar://problem/33352721>

* platform/Timer.cpp:
(WebCore::TimerBase::setNextFireTime):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233821 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoWeb Inspector: Basic blocks highlighting should use line/column locations instead...
mattbaker@apple.com [Fri, 13 Jul 2018 22:27:30 +0000 (22:27 +0000)]
Web Inspector: Basic blocks highlighting should use line/column locations instead of offsets
https://bugs.webkit.org/show_bug.cgi?id=187613
<rdar://problem/42131808>

Reviewed by Joseph Pecoraro.

* UserInterface/Controllers/BasicBlockAnnotator.js:
Basic blocks sent from the backend include offsets into the original
file, rather than line/column locations. In order to translate to positions
within CodeMirror, we need to calculate the original line and column
for each block.

(WI.BasicBlockAnnotator.prototype.insertAnnotations):
(WI.BasicBlockAnnotator.prototype._calculateBasicBlockPositions.offsetToPosition):
(WI.BasicBlockAnnotator.prototype._calculateBasicBlockPositions):
(WI.BasicBlockAnnotator.prototype._annotateBasicBlockExecutionRanges.):
(WI.BasicBlockAnnotator.prototype._annotateBasicBlockExecutionRanges):
(WI.BasicBlockAnnotator.prototype._highlightTextForBasicBlock):

* UserInterface/Models/SourceCodePosition.js:
(WI.SourceCodePosition.prototype.offsetColumn):

* UserInterface/Views/TextEditor.js:
(WI.TextEditor.prototype.getTextInRange):
(WI.TextEditor.prototype.addStyleToTextRange):
Better encapsulation for CodeMirror positions.

* UserInterface/Workers/Formatter/FormatterUtilities.js:
(get if):
Update String.prototype.lineEndings to support additional line separators.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233820 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[ MacOS WK1 ] Layout Tests in media/media-fragments/ are flaky
commit-queue@webkit.org [Fri, 13 Jul 2018 22:26:10 +0000 (22:26 +0000)]
[ MacOS WK1 ] Layout Tests in media/media-fragments/ are flaky
https://bugs.webkit.org/show_bug.cgi?id=187557

Unreviewed test gardening.

Patch by Truitt Savell <tsavell@apple.com> on 2018-07-13

* platform/mac-wk1/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233819 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoREGRESSION (r231676): watchOS WebKit usually doesn't load in the background
timothy_horton@apple.com [Fri, 13 Jul 2018 22:22:42 +0000 (22:22 +0000)]
REGRESSION (r231676): watchOS WebKit usually doesn't load in the background
https://bugs.webkit.org/show_bug.cgi?id=187663
<rdar://problem/42181185>

Reviewed by Chris Dumez.

* UIProcess/ios/WebPageProxyIOS.mm:
(WebKit::WebPageProxy::applicationDidEnterBackground):
"Screen lock" is very aggressive on watchOS; we want to do our usual
30 seconds of loading in the background when you drop your wrist,
so disable this power optimization on that platform.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233818 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoWeb Inspector: Execution highlighting in the frontend should be line/column-based
mattbaker@apple.com [Fri, 13 Jul 2018 21:53:14 +0000 (21:53 +0000)]
Web Inspector: Execution highlighting in the frontend should be line/column-based
https://bugs.webkit.org/show_bug.cgi?id=187532
<rdar://problem/42035580>

Reviewed by Joseph Pecoraro.

Source code offsets from Esprima should not be used to calculate ranges
in CodeMirror for expression highlighting.

This also fixes a long standing bug when adjusting for the starting
position of an inline script. Previously the start offset from the script
TextRange was used for this purpose, but the value is often incorrect (see
https://bugs.webkit.org/show_bug.cgi?id=187532#c5). By using the starting
line/column instead, we avoid the problem.

* UserInterface/Models/ScriptSyntaxTree.js:
(WI.ScriptSyntaxTree.prototype.containersOfPosition):
(WI.ScriptSyntaxTree.prototype.containersOfOffset): Deleted.

* UserInterface/Models/SourceCodePosition.js:
(WI.SourceCodePosition.prototype.offsetColumn):

* UserInterface/Views/SourceCodeTextEditor.js:
(WI.SourceCodeTextEditor.prototype.textEditorExecutionHighlightRange.toInlineScriptPosition):
(WI.SourceCodeTextEditor.prototype.textEditorExecutionHighlightRange.fromInlineScriptPosition):
(WI.SourceCodeTextEditor.prototype.textEditorExecutionHighlightRange):
(WI.SourceCodeTextEditor.prototype.textEditorExecutionHighlightRange.convertRangeOffsetsToSourceCodeOffsets): Deleted.

* UserInterface/Views/TextEditor.js:
(WI.TextEditor.prototype._updateExecutionRangeHighlight):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233817 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[iOS] [WK1] Crash inside IOSurfacePool::platformGarbageCollectNow() in WebThread
rniwa@webkit.org [Fri, 13 Jul 2018 20:56:36 +0000 (20:56 +0000)]
[iOS] [WK1] Crash inside IOSurfacePool::platformGarbageCollectNow() in WebThread
https://bugs.webkit.org/show_bug.cgi?id=187635
<rdar://problem/34297065>

Reviewed by Simon Fraser.

r167717 added code to trigger a CA commit in the web process via platformGarbageCollectNow() in order to free IOSurface-related memory.
However, that code is also running in the web thread in apps using WebKit1, causing unwanted UIView layout on the web thread.

Fix by not triggering this CA commit if it's called on the web thread.

* platform/graphics/cocoa/IOSurfacePoolCocoa.mm:
(WebCore::IOSurfacePool::platformGarbageCollectNow):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233816 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoWebResourceLoader may try to send a IPC with a destination ID that is 0
cdumez@apple.com [Fri, 13 Jul 2018 20:40:45 +0000 (20:40 +0000)]
WebResourceLoader may try to send a IPC with a destination ID that is 0
https://bugs.webkit.org/show_bug.cgi?id=187654
<rdar://problem/39265927>

Reviewed by Brady Eidson.

WebResourceLoader may try to send a IPC with a destination ID that is 0 according to release
assertion hits we see in the wild. This can lead to HashMap corruption on recipient side when
trying to lookup a key that is 0.

As per the crash traces, we see it starts from WebLoaderStrategy::internallyFailedLoadTimerFired()
which is likely due to the Network process crashing. WebLoaderStrategy::networkProcessCrashed()
calls scheduleInternallyFailedLoad() on each ResourceLoader and clears the WebResourceLoaders in
m_webResourceLoaders. When a ResourceLoader is cancelled (marked as failed), we clear its identifier
and we normally null out the WebLoaderStrategy's coreLoader. However, if we cannot find the
WebResourceLoader in m_webResourceLoaders (because that map was already cleared) then the
WebResourceLoader's coreLoader may not get nulled out, even if the coreLoader's identifier has
been reset. We have 2 lambdas in WebResourceLoader which keep the WebResourceLoader alive and
try to send IPC and are merely doing a null check on the coreLoader before trying to send the
IPC.

To address the issue, we now clear the WebResourceLoader's coreLoader in
WebLoaderStrategy::networkProcessCrashed(). For robustness, we also check that a coreLoader's
identifier is not 0 before trying to send IPC.

* WebProcess/Network/WebLoaderStrategy.cpp:
(WebKit::WebLoaderStrategy::networkProcessCrashed):
* WebProcess/Network/WebResourceLoader.cpp:
(WebKit::WebResourceLoader::willSendRequest):
(WebKit::WebResourceLoader::didReceiveResponse):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233815 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoDark Mode: document markers are difficult to see
graouts@webkit.org [Fri, 13 Jul 2018 20:34:24 +0000 (20:34 +0000)]
Dark Mode: document markers are difficult to see
https://bugs.webkit.org/show_bug.cgi?id=187632
<rdar://problem/41099719>

Reviewed by Simon Fraser.

We update the way we draw the document markers for macOS and use more constrasting colors in dark mode.
Paving the way for future improvements, we move the drawLineForDocumentMarker() method from GraphicsContext
to RenderTheme and implement a first version in RenderThemeMac. The circles used for the underline are now
drawn directly with Core Graphics and we no longer use an image resource. To allow both GraphicsContext
and RenderTheme to have different versions of the drawLineForDocumentMarker() method, the DocumentMarkerLineStyle
enum is now an "enum class".

No new test due to webkit.org/b/105616, webkit.org/b/187655 was raised to track the creation of new tests
when it becomes possible again.

* platform/graphics/GraphicsContext.h:
* platform/graphics/GraphicsContextImpl.h:
* platform/graphics/cairo/CairoOperations.cpp:
(WebCore::Cairo::drawLineForDocumentMarker):
* platform/graphics/cairo/CairoOperations.h:
* platform/graphics/cairo/GraphicsContextImplCairo.cpp:
(WebCore::GraphicsContextImplCairo::drawLineForDocumentMarker):
* platform/graphics/cairo/GraphicsContextImplCairo.h:
* platform/graphics/cocoa/GraphicsContextCocoa.mm:
(WebCore::GraphicsContext::drawLineForDocumentMarker):
* platform/graphics/displaylists/DisplayListItems.h:
(WebCore::DisplayList::DrawLineForDocumentMarker::create):
(WebCore::DisplayList::DrawLineForDocumentMarker::DrawLineForDocumentMarker):
* platform/graphics/displaylists/DisplayListRecorder.cpp:
(WebCore::DisplayList::Recorder::drawLineForDocumentMarker):
* platform/graphics/displaylists/DisplayListRecorder.h:
* platform/graphics/nicosia/cairo/NicosiaCairoOperationRecorder.cpp:
(Nicosia::CairoOperationRecorder::drawLineForDocumentMarker):
* platform/graphics/nicosia/cairo/NicosiaCairoOperationRecorder.h:
* platform/graphics/win/GraphicsContextCGWin.cpp:
(WebCore::GraphicsContext::drawLineForDocumentMarker):
* rendering/InlineTextBox.cpp:
(WebCore::InlineTextBox::paintPlatformDocumentMarker): Call drawLineForDocumentMarker() on the RenderTheme on
macOS and on GraphicsContext in all other cases.
* rendering/RenderTheme.cpp:
(WebCore::RenderTheme::drawLineForDocumentMarker):
* rendering/RenderTheme.h:
* rendering/RenderThemeMac.h:
* rendering/RenderThemeMac.mm:
(WebCore::colorForStyle): Provide different colors for light and dark modes.
(WebCore::RenderThemeMac::drawLineForDocumentMarker): A new macOS-specific version of drawLineForDocumentMarker()
where we paint circles using Core Graphics directly.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233814 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[ WK2 ] Layout Test editing/selection/update-selection-by-style-change.html is flaky
ryanhaddad@apple.com [Fri, 13 Jul 2018 20:17:04 +0000 (20:17 +0000)]
[ WK2 ] Layout Test editing/selection/update-selection-by-style-change.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=187649

Unreviewed test gardening.

Patch by Truitt Savell <tsavell@apple.com> on 2018-07-13

* platform/mac-wk2/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233813 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agolibwebrtc.dylib Objective-C classes conflict with third-party frameworks
ddkilzer@apple.com [Fri, 13 Jul 2018 20:08:37 +0000 (20:08 +0000)]
libwebrtc.dylib Objective-C classes conflict with third-party frameworks
<https://webkit.org/b/187653>

Reviewed by Alex Christensen.

* Source/webrtc/sdk/WebKit/WebKitUtilities.mm:
- Manually add an attribute to change the class name.

* Source/webrtc/sdk/objc/Framework/Classes/Common/RTCUIApplicationStatusObserver.h:
* Source/webrtc/sdk/objc/Framework/Classes/Metal/RTCMTLI420Renderer.h:
* Source/webrtc/sdk/objc/Framework/Classes/Metal/RTCMTLNV12Renderer.h:
* Source/webrtc/sdk/objc/Framework/Classes/Metal/RTCMTLRenderer.h:
* Source/webrtc/sdk/objc/Framework/Classes/PeerConnection/RTCDtmfSender+Private.h:
* Source/webrtc/sdk/objc/Framework/Classes/PeerConnection/RTCVideoRendererAdapter.h:
* Source/webrtc/sdk/objc/Framework/Classes/PeerConnection/RTCWrappedNativeVideoDecoder.h:
* Source/webrtc/sdk/objc/Framework/Classes/PeerConnection/RTCWrappedNativeVideoEncoder.h:
* Source/webrtc/sdk/objc/Framework/Classes/UI/RTCEAGLVideoView.m:
* Source/webrtc/sdk/objc/Framework/Classes/Video/RTCAVFoundationVideoCapturerInternal.h:
* Source/webrtc/sdk/objc/Framework/Classes/Video/RTCDefaultShader.h:
* Source/webrtc/sdk/objc/Framework/Classes/Video/RTCI420TextureCache.h:
* Source/webrtc/sdk/objc/Framework/Classes/Video/RTCNV12TextureCache.h:
* Source/webrtc/sdk/objc/Framework/Classes/Video/objc_frame_buffer.h:
* Source/webrtc/sdk/objc/Framework/Classes/VideoToolbox/objc_video_decoder_factory.h:
* Source/webrtc/sdk/objc/Framework/Classes/VideoToolbox/objc_video_encoder_factory.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCAVFoundationVideoSource.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCAudioSession.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCAudioSessionConfiguration.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCAudioSource.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCAudioTrack.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCCameraPreviewView.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCCameraVideoCapturer.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCConfiguration.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCDataChannel.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCDataChannelConfiguration.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCDispatcher.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCDtmfSender.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCEAGLVideoView.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCFileLogger.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCFileVideoCapturer.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCIceCandidate.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCIceServer.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCIntervalRange.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCLegacyStatsReport.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCMTLNSVideoView.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCMTLVideoView.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCMediaConstraints.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCMediaSource.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCMediaStream.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCMediaStreamTrack.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCMetricsSampleInfo.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCNSGLVideoView.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCPeerConnection.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCPeerConnectionFactory.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCPeerConnectionFactoryOptions.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCRtpCodecParameters.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCRtpEncodingParameters.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCRtpParameters.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCRtpReceiver.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCRtpSender.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCSessionDescription.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCVideoCapturer.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCVideoCodec.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCVideoCodecFactory.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCVideoCodecH264.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCVideoDecoderVP8.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCVideoDecoderVP9.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCVideoEncoderVP8.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCVideoEncoderVP9.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCVideoFrame.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCVideoFrameBuffer.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCVideoRenderer.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCVideoSource.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCVideoTrack.h:
* Source/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCVideoViewShading.h:
- Apply two shell scripts (see bug) to add an attribute to
  change the name of all classes and protocols.

* WebKit/0012-Add-WK-prefix-to-Objective-C-classes-and-protocols.patch: Add.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233812 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoAllow BOCOM and ABC plug-ins to run unsandboxed
commit-queue@webkit.org [Fri, 13 Jul 2018 19:41:08 +0000 (19:41 +0000)]
Allow BOCOM and ABC plug-ins to run unsandboxed
https://bugs.webkit.org/show_bug.cgi?id=187652
rdar://problem/42149182

Patch by Zach Li <zachli@apple.com> on 2018-07-13
Reviewed by Youenn Fablet.

* UIProcess/Plugins/mac/PluginInfoStoreMac.mm:
(WebKit::PluginInfoStore::shouldAllowPluginToRunUnsandboxed):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233811 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoREGRESSION (r233155): Remove last references to click_annotate.cc and rtpcat.cc
ddkilzer@apple.com [Fri, 13 Jul 2018 19:17:42 +0000 (19:17 +0000)]
REGRESSION (r233155): Remove last references to click_annotate.cc and rtpcat.cc

* libwebrtc.xcodeproj/project.pbxproj: Let Xcode have its way
with the project file by removing orphaned entries.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233810 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoREGRESSION (r222476): Add missing semi-colons to EXPORTED_SYMBOLS_FILE variables
ddkilzer@apple.com [Fri, 13 Jul 2018 19:17:39 +0000 (19:17 +0000)]
REGRESSION (r222476): Add missing semi-colons to EXPORTED_SYMBOLS_FILE variables

* Configurations/libwebrtc.xcconfig:
(EXPORTED_SYMBOLS_FILE): Add missing semi-colons.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233809 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoAdd more threading release assertions
cdumez@apple.com [Fri, 13 Jul 2018 18:47:18 +0000 (18:47 +0000)]
Add more threading release assertions
https://bugs.webkit.org/show_bug.cgi?id=187647

Reviewed by Alex Christensen.

Add more threading release assertions to help debug <rdar://problem/39265927>.

* NetworkProcess/NetworkConnectionToWebProcess.cpp:
(WebKit::NetworkConnectionToWebProcess::NetworkConnectionToWebProcess):
(WebKit::NetworkConnectionToWebProcess::~NetworkConnectionToWebProcess):
* UIProcess/WebProcessProxy.cpp:
(WebKit::m_isInPrewarmedPool):
(WebKit::WebProcessProxy::~WebProcessProxy):
(WebKit::WebProcessProxy::shutDown):
(WebKit::WebProcessProxy::didFinishLaunching):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233808 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[WinCairo] Add windows storage process connection implementation
chris.reid@sony.com [Fri, 13 Jul 2018 18:28:09 +0000 (18:28 +0000)]
[WinCairo] Add windows storage process connection implementation
https://bugs.webkit.org/show_bug.cgi?id=187531

Reviewed by Fujii Hironori.

* NetworkProcess/NetworkProcess.cpp:
* Platform/IPC/Attachment.h:
* StorageProcess/StorageProcess.cpp:
* UIProcess/Storage/StorageProcessProxy.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233807 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[32bit JSC tests] stress/cow-convert-double-to-contiguous.js and stress/cow-convert...
utatane.tea@gmail.com [Fri, 13 Jul 2018 18:20:21 +0000 (18:20 +0000)]
[32bit JSC tests]  stress/cow-convert-double-to-contiguous.js and stress/cow-convert-int32-to-contiguous.js are failing
https://bugs.webkit.org/show_bug.cgi?id=187561

Reviewed by Darin Adler.

This patch fixes the issue that CoW array handling is not introduced in 32bit put_by_val code.
We clean up 32bit put_by_val code.

1. We remove inline out-of-bounds recording code since it is done in C operation code. This change
aligns 32bit implementation to 64bit implementation.

2. We add CoW array checking, which is done in 64bit implementation.

* jit/JITPropertyAccess.cpp:
(JSC::JIT::emit_op_put_by_val):
* jit/JITPropertyAccess32_64.cpp:
(JSC::JIT::emit_op_put_by_val):
(JSC::JIT::emitSlow_op_put_by_val):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233806 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[GStreamer] Use smart pointers for GstByteReader
cturner@igalia.com [Fri, 13 Jul 2018 18:20:03 +0000 (18:20 +0000)]
[GStreamer] Use smart pointers for GstByteReader
https://bugs.webkit.org/show_bug.cgi?id=187638

Reviewed by Xabier Rodriguez-Calvar.

* platform/graphics/gstreamer/GUniquePtrGStreamer.h: Add
specialisation for GstByteReader.
* platform/graphics/gstreamer/eme/WebKitClearKeyDecryptorGStreamer.cpp:
Use the new smart pointer class to avoid needing to remember where
to call gst_byte_reader_free.
(webKitMediaClearKeyDecryptorDecrypt):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233805 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[macOS] REGRESSION (r233536): Development WebContent service got opted back into...
mitz@apple.com [Fri, 13 Jul 2018 17:46:30 +0000 (17:46 +0000)]
[macOS] REGRESSION (r233536): Development WebContent service got opted back into Library Validation
https://bugs.webkit.org/show_bug.cgi?id=187640

Reviewed by Daniel Bates.

* Scripts/process-webcontent-entitlements.sh: Add the
  com.apple.security.cs.disable-library-validation to the Development service regardless of
  whether restricted entitlements are to be used, because that entitlement is not restricted.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233804 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[Curl] Move HTTP Setup logic from CurlRequest to CurlHandle for reuse.
Basuke.Suzuki@sony.com [Fri, 13 Jul 2018 17:28:29 +0000 (17:28 +0000)]
[Curl] Move HTTP Setup logic from CurlRequest to CurlHandle for reuse.
https://bugs.webkit.org/show_bug.cgi?id=187427

Reviewed by Fujii Hironori.

CurlContext will be used by Secure WebSocket client, but HTTP setup code is
in CurlRequest, which is only for regular HTTP/HTTPS transaction. This patch
allows wss client to setup CurlHandle for HTTPS communication, such as TLS,
proxy or authentication.

No new tests because there's no behavior change.

* platform/network/curl/CurlContext.cpp:
(WebCore::CurlHandle::CurlHandle):
(WebCore::CurlHandle::enableSSLForHost):
(WebCore::CurlHandle::willSetupSslCtx):
(WebCore::CurlHandle::willSetupSslCtxCallback):
(WebCore::CurlHandle::sslErrors const):
(WebCore::CurlHandle::setUrl):
(WebCore::CurlHandle::enableHttp):
(WebCore::CurlHandle::enableConnectionOnly):
(WebCore::CurlHandle::certificateInfo const):
(WebCore::CurlHandle::enableStdErrIfUsed):
(WebCore::CurlHandle::initialize): Deleted.
* platform/network/curl/CurlContext.h:
(WebCore::CurlHandle::url const):
* platform/network/curl/CurlRequest.cpp:
(WebCore::CurlRequest::setupTransfer):
(WebCore::CurlRequest::didReceiveHeader):
(WebCore::CurlRequest::didCompleteTransfer):
(WebCore::CurlRequest::finalizeTransfer):
(WebCore::CurlRequest::willSetupSslCtx): Deleted.
(WebCore::CurlRequest::willSetupSslCtxCallback): Deleted.
* platform/network/curl/CurlRequest.h:
* platform/network/curl/CurlSSLVerifier.cpp:
(WebCore::CurlSSLVerifier::CurlSSLVerifier):
(WebCore::CurlSSLVerifier::verify):
* platform/network/curl/CurlSSLVerifier.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233803 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[ MacOS Debug ] Layout Test inspector/view/asynchronous-layout.html is a Flaky Timeout
commit-queue@webkit.org [Fri, 13 Jul 2018 17:15:05 +0000 (17:15 +0000)]
[ MacOS Debug ] Layout Test inspector/view/asynchronous-layout.html is a Flaky Timeout
https://bugs.webkit.org/show_bug.cgi?id=187622

Unreviewed test gardening.

Patch by Truitt Savell <tsavell@apple.com> on 2018-07-13

* platform/mac/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233802 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoNetworkConnectionToWebProcess::m_networkResourceLoaders should use Ref<> for its...
cdumez@apple.com [Fri, 13 Jul 2018 17:14:22 +0000 (17:14 +0000)]
NetworkConnectionToWebProcess::m_networkResourceLoaders should use Ref<> for its values
https://bugs.webkit.org/show_bug.cgi?id=187629

Reviewed by Youenn Fablet.

NetworkConnectionToWebProcess::m_networkResourceLoaders should use Ref<> for its values
since they cannot be null.

* NetworkProcess/NetworkConnectionToWebProcess.cpp:
(WebKit::NetworkConnectionToWebProcess::didClose):
(WebKit::NetworkConnectionToWebProcess::scheduleResourceLoad):
(WebKit::NetworkConnectionToWebProcess::performSynchronousLoad):
* NetworkProcess/NetworkConnectionToWebProcess.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233801 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoWeb Inspector: SourceCodePosition.js missing from Test.html
mattbaker@apple.com [Fri, 13 Jul 2018 17:10:28 +0000 (17:10 +0000)]
Web Inspector: SourceCodePosition.js missing from Test.html
https://bugs.webkit.org/show_bug.cgi?id=187644

Reviewed by Brian Burg.

* UserInterface/Test.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233800 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoAdd a FrameLoaderClient willInjectUserScriptForFrame callback
cdumez@apple.com [Fri, 13 Jul 2018 16:19:31 +0000 (16:19 +0000)]
Add a FrameLoaderClient willInjectUserScriptForFrame callback
https://bugs.webkit.org/show_bug.cgi?id=187565
<rdar://problem/42095701>

Unreviewed WKTR fix after r233782.

* WebKitTestRunner/TestInvocation.cpp:
(WTR::TestInvocation::didReceiveSynchronousMessageFromInjectedBundle):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233799 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoReduce size of WebCore::URL
cdumez@apple.com [Fri, 13 Jul 2018 16:13:53 +0000 (16:13 +0000)]
Reduce size of WebCore::URL
https://bugs.webkit.org/show_bug.cgi?id=186820
<rdar://problem/42087508>

Unreviewed, re-generates Service worker registrations database used by API tests after r233789.

* TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
* TestWebKitAPI/Tests/WebKitCocoa/SimpleServiceWorkerRegistrations-2.sqlite3: Removed.
* TestWebKitAPI/Tests/WebKitCocoa/SimpleServiceWorkerRegistrations-2.sqlite3-shm: Removed.
* TestWebKitAPI/Tests/WebKitCocoa/SimpleServiceWorkerRegistrations-3.sqlite3: Renamed from Tools/TestWebKitAPI/Tests/WebKitCocoa/SimpleServiceWorkerRegistrations-2.sqlite3-wal.
* TestWebKitAPI/Tests/WebKitCocoa/WebsiteDataStoreCustomPaths.mm:
(TEST):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233798 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[GStreamer][MSE] Add GstFlowCombiner to handle non-linked inactive branches
calvaris@igalia.com [Fri, 13 Jul 2018 10:47:10 +0000 (10:47 +0000)]
[GStreamer][MSE] Add GstFlowCombiner to handle non-linked inactive branches
https://bugs.webkit.org/show_bug.cgi?id=187636

Reviewed by Carlos Garcia Campos.

When we have more than one source buffer, only one will be
rendered and the inactive branch will report linking errors that
we have to deal with.

* platform/graphics/gstreamer/GUniquePtrGStreamer.h: Added GstFlowCombiner.
* platform/graphics/gstreamer/mse/WebKitMediaSourceGStreamer.cpp:
(webkitMediaSrcChain): Combine the flow in the flow combiner.
(webkit_media_src_init): Initialize the flow combiner.
(webKitMediaSrcLinkStreamToSrcPad): Add the proxypad to the
combiner and set the chain function.
* platform/graphics/gstreamer/mse/WebKitMediaSourceGStreamerPrivate.h:
Declare the flow combiner.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233797 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[GStreamer] Add GstBufferMapped abstraction
cturner@igalia.com [Fri, 13 Jul 2018 08:10:51 +0000 (08:10 +0000)]
[GStreamer] Add GstBufferMapped abstraction
https://bugs.webkit.org/show_bug.cgi?id=187600

Reviewed by Xabier Rodriguez-Calvar.

There is a similar abstraction called `mapGstBuffer` and friends,
which have a slightly different use-case: wanting a buffer that is
mapped for a longer lifetime without have to keep track of the map
infos separately. They could be subsumed by this abstraction, but
everytime they need to write to the buffer, they'd have to remap
the memory blocks.

This abstraction is more for one-short reads and writes saving the user
from remembering to unmap the buffer and having to manage to
auxiliary GstMapInfo structures.

* platform/graphics/gstreamer/GStreamerCommon.h:
(WebCore::GstMappedBuffer::GstMappedBuffer):
(WebCore::GstMappedBuffer::~GstMappedBuffer):
(WebCore::GstMappedBuffer::data):
(WebCore::GstMappedBuffer::size const):
(WebCore::GstMappedBuffer::operator bool const):
* platform/graphics/gstreamer/InbandTextTrackPrivateGStreamer.cpp:
(WebCore::InbandTextTrackPrivateGStreamer::notifyTrackOfSample):
* platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:
(WebCore::MediaPlayerPrivateGStreamerBase::handleSyncMessage):
(WebCore::MediaPlayerPrivateGStreamerBase::initializationDataEncountered):
* platform/graphics/gstreamer/eme/WebKitClearKeyDecryptorGStreamer.cpp:
(webKitMediaClearKeyDecryptorSetupCipher):
(webKitMediaClearKeyDecryptorDecrypt):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233796 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoSignal.cpp's activeThreads() should only create its ThreadGroup once.
mark.lam@apple.com [Fri, 13 Jul 2018 07:04:11 +0000 (07:04 +0000)]
Signal.cpp's activeThreads() should only create its ThreadGroup once.
https://bugs.webkit.org/show_bug.cgi?id=187634
<rdar://problem/40662311>

Reviewed by Yusuke Suzuki.

registerThreadForMachExceptionHandling() is relying on the activeThreads()
ThreadGroup being a singleton.

* wtf/threads/Signals.cpp:
(WTF::activeThreads):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233795 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoMake it easier to hit the significant rendered text layout milestone on pages with...
wenson_hsieh@apple.com [Fri, 13 Jul 2018 06:22:33 +0000 (06:22 +0000)]
Make it easier to hit the significant rendered text layout milestone on pages with main article elements
https://bugs.webkit.org/show_bug.cgi?id=187578
<rdar://problem/42104637>

Reviewed by Ryosuke Niwa.

Source/WebCore:

Our current heuristics for triggering the significant rendered text layout milestone are very conservative, with
the intention of avoiding false positives. In practice, we can relax some of these constraints when we've
detected the presence of a main article element on the page. (e.g. in New York Times articles). See per-method
changes below for more detail.

Test: RenderingProgressTests.DidRenderSignificantAmountOfText

* dom/Document.cpp:
(WebCore::Document::registerArticleElement):
(WebCore::Document::unregisterArticleElement):
(WebCore::Document::updateMainArticleElementAfterLayout):

As a post layout task, update the main article element by looping through the articles (up to a maximum limit of
10) in search of an article element that is several times larger than the second largest article element.

* dom/Document.h:

Store a set containing the article elements in the document, as well as the current main article on the page.

(WebCore::Document::hasMainArticleElement const):
* html/Element.cpp:
(WebCore::Element::insertedIntoAncestor):
(WebCore::Element::removedFromAncestor):

Keep track of the article elements that exist in the document whenever elements with the article tag are added
to or removed from the document.

* page/FrameView.cpp:

Add new minimum thresholds for firing the significant rendered text milestone when there exists a main article.

(WebCore::FrameView::performPostLayoutTasks):
(WebCore::FrameView::updateSignificantRenderedTextMilestoneIfNeeded):

Take the main article element into consideration when determining whether to fire the significant text
layout milestone.

Tools:

Tweak an existing layout test to additionally check that the significant text layout milestone is fired on a
page with an article element that is clearly the "main" content on the page (i.e. much taller than all other
articles).

* TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
* TestWebKitAPI/Tests/WebKitCocoa/significant-text-milestone-article.html: Copied from Tools/TestWebKitAPI/Tests/WebKitCocoa/significant-text-milestone.html.
* TestWebKitAPI/Tests/WebKitCocoa/significant-text-milestone.html:
* TestWebKitAPI/Tests/ios/RenderingProgressTests.mm:
(TEST):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233794 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoJavaScript URL gives incorrect result when frame is navigated
dbates@webkit.org [Fri, 13 Jul 2018 03:13:19 +0000 (03:13 +0000)]
JavaScript URL gives incorrect result when frame is navigated
https://bugs.webkit.org/show_bug.cgi?id=187203
<rdar://problem/41438443>

Reviewed by David Kilzer.

* loader/SubframeLoader.cpp:
(WebCore::SubframeLoader::requestFrame):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233793 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoWeb Inspector: Type token positioning should use line/column locations from Esprima...
mattbaker@apple.com [Fri, 13 Jul 2018 02:06:48 +0000 (02:06 +0000)]
Web Inspector: Type token positioning should use line/column locations from Esprima instead of offsets
https://bugs.webkit.org/show_bug.cgi?id=187612
<rdar://problem/42131910>

Reviewed by Joseph Pecoraro.

* UserInterface/Controllers/TypeTokenAnnotator.js:
(WI.TypeTokenAnnotator.prototype.insertAnnotations):
(WI.TypeTokenAnnotator.prototype._insertTypeToken):
(WI.TypeTokenAnnotator.prototype._insertToken):
Use line/column locations, instead of offsets, from the AST when calculating
token positions for CodeMirror. Once in CodeMirror's string space, we
can safely convert to/from offsets.

* UserInterface/Models/ScriptSyntaxTree.js:
Retrieve line/column locations for AST nodes, in addition to offsets.
Offsets into the original file are still needed for getting type information
from the profiler in the backend.

(WI.ScriptSyntaxTree):
(WI.ScriptSyntaxTree.prototype.filterByRange):
Filter by positions, which can be safely used from CodeMirror, instead of offsets.

(WI.ScriptSyntaxTree.prototype._createInternalSyntaxTree):
(WI.ScriptSyntaxTree.prototype.filterByRange.filterForNodesInRange): Deleted.

* UserInterface/Models/SourceCodePosition.js:
Add convenience methods for comparing line/column positions, and for
converting to the format expected by CodeMirror. SourceCodePosition could
be made to interoperate with CodeMirror by exposing properties `line`
and `ch`, but making the conversion explicit improves code readability.

(WI.SourceCodePosition.prototype.equals):
(WI.SourceCodePosition.prototype.isBefore):
(WI.SourceCodePosition.prototype.isAfter):
(WI.SourceCodePosition.prototype.isWithin):
(WI.SourceCodePosition.prototype.toCodeMirror):
(WI.SourceCodePosition):

* UserInterface/Views/TextEditor.js:
(WI.TextEditor.prototype.visibleRangePositions):
(WI.TextEditor.prototype.originalPositionToCurrentPosition):
(WI.TextEditor.prototype.currentOffsetToCurrentPosition):
(WI.TextEditor.prototype.currentPositionToCurrentOffset):
(WI.TextEditor.prototype.setInlineWidget):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233792 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoChange my status to be a WebKit reviewer.
Hironori.Fujii@sony.com [Fri, 13 Jul 2018 01:20:40 +0000 (01:20 +0000)]
Change my status to be a WebKit reviewer.

Unreviewed status update.

* Scripts/webkitpy/common/config/contributors.json:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233791 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoUnreviewed contributors.json update
Hironori.Fujii@sony.com [Fri, 13 Jul 2018 01:16:26 +0000 (01:16 +0000)]
Unreviewed contributors.json update

Just run "validate-committer-lists -c" to canonicalize the style.

* Scripts/webkitpy/common/config/contributors.json:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233790 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoReduce size of WebCore::URL
achristensen@apple.com [Fri, 13 Jul 2018 00:28:36 +0000 (00:28 +0000)]
Reduce size of WebCore::URL
https://bugs.webkit.org/show_bug.cgi?id=186820

Reviewed by Yusuke Suzuki and Youenn Fablet.

Source/WebCore:

We were using 32 bits for the length of the port, which is always between 0 and 5 inclusive
because port numbers are missing or between 0 and 65535.  Let's just use 3 bits here.
We were using 32 bits for the length of the scheme, which is usually 3-5 characters and can be
longer for some custom schemes, but I've never seen one more than 20 characters.  If we assume
schemes are always less than 64MB, we can save 8 bytes per URL!

No change in behavior, just less memory use!

To restore the IPC encoding to how it was before r221165, I just encode the string and reparse it.

* platform/URL.cpp:
(WebCore::URL::invalidate):
(WebCore::URL::lastPathComponent const):
(WebCore::URL::port const):
(WebCore::URL::protocolHostAndPort const):
(WebCore::URL::path const):
(WebCore::URL::removePort):
(WebCore::URL::setPort):
(WebCore::URL::setHostAndPort):
(WebCore::URL::setPath):
* platform/URL.h:
(WebCore::URL::encode const):
(WebCore::URL::decode):
(WebCore::URL::hasPath const):
(WebCore::URL::pathStart const):
* platform/URLParser.cpp:
(WebCore::URLParser::copyBaseWindowsDriveLetter):
(WebCore::URLParser::urlLengthUntilPart):
(WebCore::URLParser::copyURLPartsUntil):
(WebCore::URLParser::shouldPopPath):
(WebCore::URLParser::popPath):
(WebCore::URLParser::parse):
(WebCore::URLParser::parsePort):
(WebCore::URLParser::parseHostAndPort):
(WebCore::URLParser::allValuesEqual):
(WebCore::URLParser::internalValuesConsistent):
* workers/service/server/RegistrationDatabase.cpp:
Increment the service worker registration schema version because of the URL encoding change.

Source/WebKit:

* NetworkProcess/cache/NetworkCacheStorage.h:
Increment cache version because of URL encoding change.

Tools:

* TestWebKitAPI/Tests/WebCore/URLParser.cpp:
(TestWebKitAPI::TEST_F):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233789 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[Cocoa] Make sure NetworkProcess::createNetworkConnectionToWebProcess() returns a...
cdumez@apple.com [Fri, 13 Jul 2018 00:03:56 +0000 (00:03 +0000)]
[Cocoa] Make sure NetworkProcess::createNetworkConnectionToWebProcess() returns a valid mach port
https://bugs.webkit.org/show_bug.cgi?id=187625

Reviewed by Youenn Fablet.

Make sure NetworkProcess::createNetworkConnectionToWebProcess() returns a valid mach port to help
debug <rdar://problem/41995022>.

* NetworkProcess/NetworkProcess.cpp:
(WebKit::NetworkProcess::createNetworkConnectionToWebProcess):
* UIProcess/Network/NetworkProcessProxy.cpp:
(WebKit::NetworkProcessProxy::didCreateNetworkConnectionToWebProcess):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233788 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoPingLoad does not need to ref the NetworkConnectionToWebProcess
cdumez@apple.com [Thu, 12 Jul 2018 23:46:17 +0000 (23:46 +0000)]
PingLoad does not need to ref the NetworkConnectionToWebProcess
https://bugs.webkit.org/show_bug.cgi?id=187624

Reviewed by Youenn Fablet.

PingLoad does not need to ref the NetworkConnectionToWebProcess and keep it alive longer than it needs to be.
Instead, ref the IPC::Connection which is lighter weight and gets invalidated when the NetworkConnectionToWebProcess
is destroyed.

* NetworkProcess/NetworkConnectionToWebProcess.cpp:
(WebKit::NetworkConnectionToWebProcess::loadPing):
(WebKit::NetworkConnectionToWebProcess::didFinishPingLoad): Deleted.
* NetworkProcess/NetworkConnectionToWebProcess.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233787 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoMake process-swap-on-navigation an experimental feature.
beidson@apple.com [Thu, 12 Jul 2018 23:37:10 +0000 (23:37 +0000)]
Make process-swap-on-navigation an experimental feature.
<rdar://problem/41995053> and https://bugs.webkit.org/show_bug.cgi?id=187558

Reviewed by Chris Dumez.

Source/WebKit:

* Shared/WebPreferences.yaml:

* UIProcess/API/C/WKPreferences.cpp:
(WKPreferencesSetProcessSwapOnNavigationEnabled):
(WKPreferencesGetProcessSwapOnNavigationEnabled):
* UIProcess/API/C/WKPreferencesRef.h:

* UIProcess/WebPreferences.cpp:
(WebKit::WebPreferences::updateBoolValueForExperimentalFeatureKey):

* UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::createWebPage): If the new web page has PSON enabled via WebPreferences,
  absorb that setting into this process pool's configuration.

Tools:

Make sure WKTR doesn't enable ProcessSwapOnNavigation by default simply because it's an experimental feature.

* WebKitTestRunner/TestController.cpp:
(WTR::TestController::createWebViewWithOptions):
(WTR::TestController::resetPreferencesToConsistentValues):

* WebKitTestRunner/TestOptions.h:
(WTR::TestOptions::shouldEnableProcessSwapOnNavigation const):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233786 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoSimplify code in NetworkConnectionToWebProcess::didReceiveMessage()
cdumez@apple.com [Thu, 12 Jul 2018 23:29:44 +0000 (23:29 +0000)]
Simplify code in NetworkConnectionToWebProcess::didReceiveMessage()
https://bugs.webkit.org/show_bug.cgi?id=187621

Reviewed by Youenn Fablet.

Simplify code in NetworkConnectionToWebProcess::didReceiveMessage() by not using HashMap iterators and
calling get() directly.

* NetworkProcess/NetworkConnectionToWebProcess.cpp:
(WebKit::NetworkConnectionToWebProcess::didReceiveMessage):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233785 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoMake sure WebProcess::ensureNetworkProcessConnection() is always called on the main...
cdumez@apple.com [Thu, 12 Jul 2018 22:24:34 +0000 (22:24 +0000)]
Make sure WebProcess::ensureNetworkProcessConnection() is always called on the main thread
https://bugs.webkit.org/show_bug.cgi?id=187607

Reviewed by Alex Christensen.

Add release assertion to make sure that ensureNetworkProcessConnection() is always called on the main
thread. Calling it on a background thread would not be safe. It would not be safe because:
1. We check if we have a network process connection and then create one if we don't without any locking.
2. It is not safe to construct or use a NetworkProcessConnection object from a non-main thread

* WebProcess/WebProcess.cpp:
(WebKit::WebProcess::ensureNetworkProcessConnection):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233784 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoAssert that the IPC::Connection is valid in Connection::dispatchMessage(Decoder&)
cdumez@apple.com [Thu, 12 Jul 2018 22:21:39 +0000 (22:21 +0000)]
Assert that the IPC::Connection is valid in Connection::dispatchMessage(Decoder&)
https://bugs.webkit.org/show_bug.cgi?id=187617

Reviewed by Youenn Fablet.

Assert that the IPC::Connection is valid in Connection::dispatchMessage(Decoder&) as
m_client would be stale otherwise.

* Platform/IPC/Connection.cpp:
(IPC::Connection::dispatchMessage):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233783 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoAdd a FrameLoaderClient willInjectUserScriptForFrame callback
youenn@apple.com [Thu, 12 Jul 2018 21:34:35 +0000 (21:34 +0000)]
Add a FrameLoaderClient willInjectUserScriptForFrame callback
https://bugs.webkit.org/show_bug.cgi?id=187565

Reviewed by Alex Christensen.

Source/WebCore:

Test: http/tests/contentextensions/injected-script-callback.html.

* loader/FrameLoaderClient.h:
* page/Frame.cpp:
(WebCore::Frame::injectUserScriptImmediately):
Calling the new callback whenever being about to inject a new script.

Source/WebKit:

Introduce a new WKBundlePageLoaderClient callback that is called everytime a user script is injected.
Implement WebFrameLoaderClient::willInjectUserScript by calling this new callback.

* WebProcess/InjectedBundle/API/APIInjectedBundlePageLoaderClient.h:
(API::InjectedBundle::PageLoaderClient::willInjectUserScriptForFrame):
* WebProcess/InjectedBundle/API/c/WKBundlePageLoaderClient.h:
* WebProcess/InjectedBundle/API/mac/WKWebProcessPlugInBrowserContextController.mm:
(setUpPageLoaderClient):
* WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.cpp:
(WebKit::InjectedBundlePageLoaderClient::globalObjectIsAvailableForFrame):
(WebKit::InjectedBundlePageLoaderClient::willInjectUserScriptForFrame):
* WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.h:
* WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::willInjectUserScript):
* WebProcess/WebCoreSupport/WebFrameLoaderClient.h:

Tools:

Added new test runner API to check for willInjectUserScriptForFrame callback.

* WebKitTestRunner/InjectedBundle/Bindings/TestRunner.idl:
* WebKitTestRunner/InjectedBundle/InjectedBundle.h:
(WTR::InjectedBundle::resetUserScriptInjectedCount):
(WTR::InjectedBundle::increaseUserScriptInjectedCount):
(WTR::InjectedBundle::userScriptInjectedCount const):
* WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp:
(WTR::InjectedBundlePage::InjectedBundlePage):
(WTR::InjectedBundlePage::resetAfterTest):
(WTR::InjectedBundlePage::willInjectUserScriptForFrame):
* WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:
* WebKitTestRunner/InjectedBundle/TestRunner.cpp:
(WTR::TestRunner::userScriptInjectedCount const):
(WTR::TestRunner::injectUserScript):
* WebKitTestRunner/InjectedBundle/TestRunner.h:
* WebKitTestRunner/TestController.cpp:
(WTR::TestController::injectUserScript):
* WebKitTestRunner/TestController.h:
* WebKitTestRunner/TestInvocation.cpp:
(WTR::TestInvocation::didReceiveSynchronousMessageFromInjectedBundle):
* WebKitTestRunner/cocoa/TestControllerCocoa.mm:
(WTR::TestController::injectUserScript):

LayoutTests:

* http/tests/contentextensions/injected-script-callback-expected.txt: Added.
* http/tests/contentextensions/injected-script-callback.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233782 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[iOS] When bringing MobileSafari to the foreground, images are drawn asynchronously...
commit-queue@webkit.org [Thu, 12 Jul 2018 21:24:38 +0000 (21:24 +0000)]
[iOS] When bringing MobileSafari to the foreground, images are drawn asynchronously after removing a snapshot that included them
https://bugs.webkit.org/show_bug.cgi?id=187374
<rdar://problem/41249545>

Patch by Said Abou-Hallawa <sabouhallawa@apple.com> on 2018-07-12
Reviewed by Tim Horton.

UIProcess should block committing all the layer tree transactions for
immediate update until it receives a one whose activityStateChangeID is
greater than or equal to the one it sends with SetActivityState message.

* Scripts/webkit/messages.py:
Fix the messages code generator to include DrawingAreaInfo.h for the WebKit
typedef ActivityStateChangeID.

* Shared/DrawingAreaInfo.h:
Define ActivityStateChangeID to be uint64_t. Add a definition for the case
when the UIProcess won't be blocked for a reply back from the WebProcess.

* Shared/RemoteLayerTree/RemoteLayerTreeTransaction.h:
(WebKit::RemoteLayerTreeTransaction::activityStateChangeID const):
(WebKit::RemoteLayerTreeTransaction::setActivityStateChangeID):
Add a member for activityStateChangeID in the LayerTreeTransaction.

* Shared/RemoteLayerTree/RemoteLayerTreeTransaction.mm:
(WebKit::RemoteLayerTreeTransaction::encode const):
(WebKit::RemoteLayerTreeTransaction::decode):
Make LayerTreeTransaction know how to encode and decode activityStateChangeID.

* UIProcess/DrawingAreaProxy.h:
(WebKit::DrawingAreaProxy::waitForDidUpdateActivityState):
* UIProcess/RemoteLayerTree/RemoteLayerTreeDrawingAreaProxy.h:
* UIProcess/RemoteLayerTree/RemoteLayerTreeDrawingAreaProxy.mm:
(WebKit::RemoteLayerTreeDrawingAreaProxy::commitLayerTree):
(WebKit::RemoteLayerTreeDrawingAreaProxy::waitForDidUpdateActivityState):
Make the DrawingArea in the UIProcess wait the layer tree with a certain
activityStateChangeID.

* UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::dispatchActivityStateChange):
(WebKit::WebPageProxy::waitForDidUpdateActivityState):
If m_activityStateChangeWantsSynchronousReply is true, generate a new
activityStateChangeID and send it in the SetActivityState message.

* UIProcess/WebPageProxy.h:
(WebKit::WebPageProxy::takeNextActivityStateChangeID):
A simple way to generate a new activityStateChangeID.

* UIProcess/mac/TiledCoreAnimationDrawingAreaProxy.h:
* UIProcess/mac/TiledCoreAnimationDrawingAreaProxy.mm:
(WebKit::TiledCoreAnimationDrawingAreaProxy::waitForDidUpdateActivityState):
Fix the UIProcess functions' prototype for Mac.

* WebProcess/WebPage/AcceleratedDrawingArea.cpp:
(WebKit::AcceleratedDrawingArea::activityStateDidChange):
* WebProcess/WebPage/AcceleratedDrawingArea.h:
Fix the UIProcess functions' prototype for GTK.

* WebProcess/WebPage/DrawingArea.h:
(WebKit::DrawingArea::activityStateDidChange):
* WebProcess/WebPage/RemoteLayerTree/RemoteLayerTreeDrawingArea.h:
Replace wantsDidUpdateActivityState with activityStateChangeID. Treat
activityStateChangeID == ActivityStateChangeAsynchronous as if
wantsDidUpdateActivityState == false.

* WebProcess/WebPage/RemoteLayerTree/RemoteLayerTreeDrawingArea.mm:
(WebKit::RemoteLayerTreeDrawingArea::flushLayers):
Make the WebPrcess pass the activityStateChangeID which it receives from
the SetActivityState message to the LayerTreeTransaction.

(WebKit::RemoteLayerTreeDrawingArea::activityStateDidChange):
When receiving the SetActivityState, treat activityStateChangeID !=
ActivityStateChangeAsynchronous as if wantsDidUpdateActivityState == true.

* WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::reinitializeWebPage):
(WebKit::WebPage::setActivityState):
* WebProcess/WebPage/WebPage.h:
* WebProcess/WebPage/WebPage.messages.in:
Replace the boolean wantsDidUpdateActivityState with activityStateChangeID
in the SetActivityState message.

* WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.h:
* WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.mm:
(WebKit::TiledCoreAnimationDrawingArea::TiledCoreAnimationDrawingArea):
(WebKit::TiledCoreAnimationDrawingArea::activityStateDidChange):
(WebKit::TiledCoreAnimationDrawingArea::didUpdateActivityStateTimerFired):
Fix the WebProcess functions' prototype for Mac.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233781 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoREGRESSION (r232356): After zooming a page in and out, it's possible to temporarily...
antti@apple.com [Thu, 12 Jul 2018 20:11:30 +0000 (20:11 +0000)]
REGRESSION (r232356): After zooming a page in and out, it's possible to temporarily have missing tiles (google.com, apple.com)
https://bugs.webkit.org/show_bug.cgi?id=187553
<rdar://problem/41863803>

Reviewed by Simon Fraser.

We ignore zoom scale from UI process if it doesn't match the last remote layer tree transaction. However nothing
guarantees that we receive the real scale again until user interacts with the page. This means the web process
scale and the UI process scale can be wildly out of sync.

This problem becomes more likely to be visible when
- we are under memory pressure so we don't update tiles during zooming and rely on the one final update afterwards
- the page generates lots of layer tree transactions

* WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::willCommitLayerTree):

Remember the last transaction where we actually changed the page scale.

* WebProcess/WebPage/WebPage.h:
* WebProcess/WebPage/ios/WebPageIOS.mm:
(WebKit::WebPage::scaleFromUIProcess const):

Only reject the UI process scale if there has been a scale changing transaction meanwhile.
The transaction id test was added in r218149 and the problem it fixed remains fixed. This change also matches better
what it was purpoted to implement.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233780 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoUTF-16 XHTML files need svn:mime-type to be handled correctly by Windows SVN
ross.kirsling@sony.com [Thu, 12 Jul 2018 19:52:23 +0000 (19:52 +0000)]
UTF-16 XHTML files need svn:mime-type to be handled correctly by Windows SVN
https://bugs.webkit.org/show_bug.cgi?id=187571

Reviewed by Alex Christensen.

`svn propset`s cannot be committed with `webkit-patch land`.
This commit actually does what the previous one claimed to.

* dom/xhtml/level3/core/documentgetinputencoding03.xhtml:
* dom/xhtml/level3/core/entitygetinputencoding02.xhtml:
* dom/xhtml/level3/core/entitygetxmlversion02.xhtml:
* dom/xhtml/level3/core/external_foo.ent:
Added property svn:mime-type and removed property svn:eol-style (to align with documentgetxmlencoding03.xhtml).

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233779 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoKeep Selections within Shadow DOM boundaries
megan_gardner@apple.com [Thu, 12 Jul 2018 19:46:23 +0000 (19:46 +0000)]
Keep Selections within Shadow DOM boundaries
https://bugs.webkit.org/show_bug.cgi?id=187556
<rdar://problem/41664567>

Reviewed by Ryosuke Niwa.

Source/WebCore:

Expose needed functionality to WebKit to help with determing editing and shadow dom boundries.

Only exposing functionality to WebKit.

* dom/TreeScope.h:
* editing/Editing.h:
* editing/VisibleSelection.cpp:
(WebCore::VisibleSelection::adjustPositionForEnd const):
(WebCore::VisibleSelection::adjustPositionForStart const):
(WebCore::adjustPositionForEnd): Deleted.
(WebCore::adjustPositionForStart): Deleted.
* editing/VisibleSelection.h:

Source/WebKit:

Update rangeForPosition to take into account both editing and shadow DOM boundries.

* WebProcess/WebPage/ios/WebPageIOS.mm:
(WebKit::rangeForPoint):
(WebKit::WebPage::updateSelectionWithTouches):
(WebKit::rangeForPosition): Deleted.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233778 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoIndexedDB: database file of subframe cannot be removed
sihui_liu@apple.com [Thu, 12 Jul 2018 19:14:32 +0000 (19:14 +0000)]
IndexedDB: database file of subframe cannot be removed
https://bugs.webkit.org/show_bug.cgi?id=187564

Reviewed by Youenn Fablet.

Source/WebCore:

For IndexedDB, if openingOrigin is different from mainFrameOrigin, there will be another
layer in the database file path.
IDBServer should delete database files recursively to make sure all files are removed.

Covered by modified test: WebKit.WebsiteDataStoreCustomPaths.

* Modules/indexeddb/server/IDBServer.cpp:
(WebCore::IDBServer::removeAllDatabasesForOriginPath):

Tools:

* TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
* TestWebKitAPI/Tests/WebKitCocoa/WebsiteDataStoreCustomPaths.mm:
(TEST):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233777 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoUTF-16 XHTML files need svn:mime-type to be handled correctly by Windows SVN
ross.kirsling@sony.com [Thu, 12 Jul 2018 19:07:57 +0000 (19:07 +0000)]
UTF-16 XHTML files need svn:mime-type to be handled correctly by Windows SVN
https://bugs.webkit.org/show_bug.cgi?id=187571

Reviewed by Alex Christensen.

* dom/xhtml/level3/core/documentgetinputencoding03.xhtml:
* dom/xhtml/level3/core/entitygetinputencoding02.xhtml:
* dom/xhtml/level3/core/entitygetxmlversion02.xhtml:
* dom/xhtml/level3/core/external_foo.ent:
Added property svn:mime-type and removed property svn:eol-style (to align with documentgetxmlencoding03.xhtml).

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233776 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoMake WebKit.LinkColorWithSystemAppearance work across system versions.
timothy@apple.com [Thu, 12 Jul 2018 18:37:47 +0000 (18:37 +0000)]
Make WebKit.LinkColorWithSystemAppearance work across system versions.
https://bugs.webkit.org/show_bug.cgi?id=187573

Reviewed by Wenson Hsieh.

* TestWebKitAPI/Tests/WebKitCocoa/SystemColors.mm:
(TestWebKitAPI.WebKit.LinkColorWithSystemAppearance): Dynamically generate the expected string.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233775 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoRevert: iOS port should define HAVE_RUNLOOP_TIMER
ddkilzer@apple.com [Thu, 12 Jul 2018 18:35:27 +0000 (18:35 +0000)]
Revert: iOS port should define HAVE_RUNLOOP_TIMER
<https://webkit.org/b/187370>

* wtf/Platform.h: This is only needed on macOS when using
-[WebView scheduleInRunLoop:forMode:] and
-[WebView unscheduleFromRunLoop:forMode:] to make WK1 WebViews
load synchronously using -loadHTMLString: or -loadData:.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233774 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoDisable IsoHeaps when Gigacage is off
msaboff@apple.com [Thu, 12 Jul 2018 18:27:26 +0000 (18:27 +0000)]
Disable IsoHeaps when Gigacage is off
https://bugs.webkit.org/show_bug.cgi?id=187160

Reviewed by Saam Barati.

Relanding change sets 233547 and 233550 with the added fix that Gigacage is also
enabled for DumpRenderTree.

Updated determineMallocFallbackState to base enabling of Iso Heaps on Gigacage
being enabled.  We do this because if Gigacage is disabled, it may be due to lack
of address space.

To work around a compiler issue uncovered by the change above, I added explicit
instantiation of PerThread's static variables.  Defined the same explicit
instantiated static variables with export scope in the new file PerThread.cpp
to eliminate separate variables allocations in each linked framework / library.

* CMakeLists.txt:
* bmalloc.xcodeproj/project.pbxproj:
* bmalloc/IsoTLS.cpp:
(bmalloc::IsoTLS::determineMallocFallbackState):
* bmalloc/PerThread.cpp: Added.
* bmalloc/PerThread.h:
* bmalloc/ProcessCheck.mm:
(bmalloc::gigacageEnabledForProcess):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233773 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoNeed to handle CodeBlock::replacement() being null.
mark.lam@apple.com [Thu, 12 Jul 2018 17:39:48 +0000 (17:39 +0000)]
Need to handle CodeBlock::replacement() being null.
https://bugs.webkit.org/show_bug.cgi?id=187569
<rdar://problem/41468692>

Reviewed by Saam Barati.

CodeBlock::replacement() may return a nullptr.  Some of our code already checks
for this while others do not.  We should add null checks in all the places that
need it.

* bytecode/CodeBlock.cpp:
(JSC::CodeBlock::hasOptimizedReplacement):
(JSC::CodeBlock::jettison):
(JSC::CodeBlock::numberOfDFGCompiles):
(JSC::CodeBlock::setOptimizationThresholdBasedOnCompilationResult):
* dfg/DFGOperations.cpp:
* dfg/DFGToFTLDeferredCompilationCallback.cpp:
(JSC::DFG::ToFTLDeferredCompilationCallback::compilationDidBecomeReadyAsynchronously):
(JSC::DFG::ToFTLDeferredCompilationCallback::compilationDidComplete):
* jit/JITOperations.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233772 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoREGRESSION (r230163): Videos cannot be seen full screen in Complete Anatomy app
jer.noble@apple.com [Thu, 12 Jul 2018 17:03:41 +0000 (17:03 +0000)]
REGRESSION (r230163): Videos cannot be seen full screen in Complete Anatomy app
https://bugs.webkit.org/show_bug.cgi?id=187527
<rdar://problem/40511527>

Reviewed by Ryosuke Niwa.

Do not enable element fullscreen mode unless apps specifically opt-in. The Fullscreen API is
an experimental feature on iOS, but not on Mac, but we can't simply not return the ExperimentalFeature
object from the list of experimental features, as this keeps Safari from being able to apply a
NSUserDefault value for that feature. Instead, add a property to API::ExperimentalFeature and
_WKExperimentalFeature called "hidden", which signals to clients whether to display the feature
in their UI.

* Scripts/GeneratePreferences.rb:
* Scripts/PreferencesTemplates/WebPreferencesExperimentalFeatures.cpp.erb:
* Shared/WebPreferences.yaml:
* Shared/WebPreferencesDefaultValues.h:
* UIProcess/API/APIExperimentalFeature.cpp:
(API::ExperimentalFeature::create):
(API::ExperimentalFeature::ExperimentalFeature):
* UIProcess/API/APIExperimentalFeature.h:
* UIProcess/API/Cocoa/_WKExperimentalFeature.h:
* UIProcess/API/Cocoa/_WKExperimentalFeature.mm:
(-[_WKExperimentalFeature isHidden]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233771 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[Mac] Run Unlock Keychain more reliably
ap@apple.com [Thu, 12 Jul 2018 17:00:58 +0000 (17:00 +0000)]
[Mac] Run Unlock Keychain more reliably
https://bugs.webkit.org/show_bug.cgi?id=187604

Reviewed by Dan Bernstein.

Move unlocking to a separate target, and make it a dependency for all targets that
produce binaries.

This is more than strictly required to fix the build, but this way, we won't need
to remember about keychain when project structure changes. Another consideration that
wasn't initially apparent to me is that is should be possible to build any target,
not just "All", so projects aren't necessarily built in the same order even now.

* WebKit.xcodeproj/project.pbxproj:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233770 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoAdd compile guard for enabling NSRunLoop in the WebContent process.
pvollan@apple.com [Thu, 12 Jul 2018 16:02:10 +0000 (16:02 +0000)]
Add compile guard for enabling NSRunLoop in the WebContent process.
https://bugs.webkit.org/show_bug.cgi?id=187563

Reviewed by Chris Dumez.

Source/WebCore:

No new tests, no change in behavior.

* platform/mac/EventLoopMac.mm:
(WebCore::EventLoop::cycle):

Source/WebKit:

Also, replace __MAC_OS_X_VERSION_MIN_REQUIRED >= 101400 with ENABLE(WEBPROCESS_WINDOWSERVER_BLOCKING) where appropriate.

* Platform/mac/LayerHostingContext.mm:
(WebKit::LayerHostingContext::createForExternalHostingProcess):
* Shared/ChildProcess.h:
* Shared/mac/ChildProcessMac.mm:
* Shared/mac/HangDetectionDisablerMac.mm:
(WebKit::setClientsMayIgnoreEvents):
* UIProcess/Cocoa/WebProcessPoolCocoa.mm:
(WebKit::WebProcessPool::registerNotificationObservers):
(WebKit::WebProcessPool::unregisterNotificationObservers):
* UIProcess/Launcher/mac/ProcessLauncherMac.mm:
(WebKit::shouldLeakBoost):
* UIProcess/WebProcessPool.h:
* WebProcess/Plugins/PDF/PDFLayerControllerSPI.h:
* WebProcess/Plugins/PDF/PDFPlugin.mm:
(WebKit::PDFPlugin::PDFPlugin):
* WebProcess/WebProcess.h:
* WebProcess/WebProcess.messages.in:
* WebProcess/cocoa/WebProcessCocoa.mm:
(WebKit::WebProcess::platformInitializeWebProcess):
(WebKit::WebProcess::stopRunLoop):

Source/WTF:

* wtf/FeatureDefines.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233769 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoUnreviewed GTK+ and WPE gardening after r233765.
magomez@igalia.com [Thu, 12 Jul 2018 14:35:38 +0000 (14:35 +0000)]
Unreviewed GTK+ and WPE gardening after r233765.

* platform/gtk/TestExpectations:
* platform/wpe/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233768 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoNewly added float should trigger full layout on the block.
zalan@apple.com [Thu, 12 Jul 2018 14:23:45 +0000 (14:23 +0000)]
Newly added float should trigger full layout on the block.
https://bugs.webkit.org/show_bug.cgi?id=187251
<rdar://problem/41726137>

Reviewed by David Kilzer.

Source/WebCore:

RenderBlockFlow::determineStartPosition() is one of the places where we decide the extent of the line layout for the current block.
In here we try to figure out the first line in the block that requires layout. In certain cases when floats are present,
(due to their intrusive behavior) we just trigger a full layout on the entire block.
One of the special cases is when a new float is added to the block. determineStartPosition() checks for such floats (floats inserted
after the "last known float") and marks the block for full layout. However it missed the case when other, unrelated mutations happened
in addition to this newly inserted float. This patch fixes this case by checking if the floats after the "last know float" actually need layout.

Test: fast/inline/new-float-needs-layout-when-line-is-dirty.html

* rendering/RenderBlockLineLayout.cpp:
(WebCore::RenderBlockFlow::determineStartPosition):

LayoutTests:

* fast/inline/new-float-needs-layout-when-line-is-dirty-expected.txt: Added.
* fast/inline/new-float-needs-layout-when-line-is-dirty.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233767 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[GStreamer] Add pads to the GstFlowCombiner in MediaStreamSrc
commit-queue@webkit.org [Thu, 12 Jul 2018 13:19:08 +0000 (13:19 +0000)]
[GStreamer] Add pads to the GstFlowCombiner in MediaStreamSrc
https://bugs.webkit.org/show_bug.cgi?id=187552

Patch by Thibault Saunier <tsaunier@igalia.com> on 2018-07-12
Reviewed by Alejandro G. Castro.

This was overlooked and it is the way the API is supposed to be used.

* platform/mediastream/gstreamer/GStreamerMediaStreamSource.cpp:
(WebCore::webkitMediaStreamSrcAddPad):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233766 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[JSC] Thread VM& to JSCell::methodTable(VM&)
utatane.tea@gmail.com [Thu, 12 Jul 2018 08:43:38 +0000 (08:43 +0000)]
[JSC] Thread VM& to JSCell::methodTable(VM&)
https://bugs.webkit.org/show_bug.cgi?id=187548

Reviewed by Saam Barati.

Source/JavaScriptCore:

This patch threads VM& to methodTable(VM&) and remove methodTable().
We add VM& parameter to estimatedSize() to thread VM& in estimatedSize implementations.

* API/APICast.h:
(toJS):
* API/JSCallbackObject.h:
* API/JSCallbackObjectFunctions.h:
(JSC::JSCallbackObject<Parent>::className):
* bytecode/CodeBlock.cpp:
(JSC::CodeBlock::estimatedSize):
* bytecode/CodeBlock.h:
* bytecode/UnlinkedCodeBlock.cpp:
(JSC::UnlinkedCodeBlock::estimatedSize):
* bytecode/UnlinkedCodeBlock.h:
* debugger/DebuggerScope.cpp:
(JSC::DebuggerScope::className):
* debugger/DebuggerScope.h:
* heap/Heap.cpp:
(JSC::GatherHeapSnapshotData::GatherHeapSnapshotData):
(JSC::GatherHeapSnapshotData::operator() const):
(JSC::Heap::gatherExtraHeapSnapshotData):
* heap/HeapSnapshotBuilder.cpp:
(JSC::HeapSnapshotBuilder::json):
* runtime/ArrayPrototype.cpp:
(JSC::arrayProtoFuncToString):
* runtime/ClassInfo.h:
* runtime/DirectArguments.cpp:
(JSC::DirectArguments::estimatedSize):
* runtime/DirectArguments.h:
* runtime/HashMapImpl.cpp:
(JSC::HashMapImpl<HashMapBucket>::estimatedSize):
* runtime/HashMapImpl.h:
* runtime/JSArrayBuffer.cpp:
(JSC::JSArrayBuffer::estimatedSize):
* runtime/JSArrayBuffer.h:
* runtime/JSBigInt.cpp:
(JSC::JSBigInt::estimatedSize):
* runtime/JSBigInt.h:
* runtime/JSCell.cpp:
(JSC::JSCell::dump const):
(JSC::JSCell::estimatedSizeInBytes const):
(JSC::JSCell::estimatedSize):
(JSC::JSCell::className):
* runtime/JSCell.h:
* runtime/JSCellInlines.h:
* runtime/JSGenericTypedArrayView.h:
* runtime/JSGenericTypedArrayViewInlines.h:
(JSC::JSGenericTypedArrayView<Adaptor>::estimatedSize):
* runtime/JSObject.cpp:
(JSC::JSObject::estimatedSize):
(JSC::JSObject::className):
(JSC::JSObject::toStringName):
(JSC::JSObject::calculatedClassName):
* runtime/JSObject.h:
* runtime/JSProxy.cpp:
(JSC::JSProxy::className):
* runtime/JSProxy.h:
* runtime/JSString.cpp:
(JSC::JSString::estimatedSize):
* runtime/JSString.h:
* runtime/RegExp.cpp:
(JSC::RegExp::estimatedSize):
* runtime/RegExp.h:
* runtime/WeakMapImpl.cpp:
(JSC::WeakMapImpl<WeakMapBucket>::estimatedSize):
* runtime/WeakMapImpl.h:

Source/WebCore:

* bindings/js/JSDOMConstructorBase.h:
(WebCore::JSDOMConstructorBase::className):
* bindings/js/JSPluginElementFunctions.cpp:
(WebCore::pluginElementCustomGetCallData):
* bindings/scripts/CodeGeneratorJS.pm:
(GenerateHeader):
(GenerateImplementation):
* bindings/scripts/test/JS/JSInterfaceName.cpp:
(WebCore::JSInterfaceName::estimatedSize):
* bindings/scripts/test/JS/JSInterfaceName.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233765 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoUnreviewed, add myself as a WebKit committer.
yousuke.kimoto@sony.com [Thu, 12 Jul 2018 08:40:18 +0000 (08:40 +0000)]
Unreviewed, add myself as a WebKit committer.

* Scripts/webkitpy/common/config/contributors.json:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233764 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[WPE] Pass the backend library name as command line parameter to the web process
carlosgc@webkit.org [Thu, 12 Jul 2018 05:56:15 +0000 (05:56 +0000)]
[WPE] Pass the backend library name as command line parameter to the web process
https://bugs.webkit.org/show_bug.cgi?id=186841

Reviewed by Žan Doberšek.

Source/WebKit:

* UIProcess/Launcher/glib/ProcessLauncherGLib.cpp:
(WebKit::ProcessLauncher::launchProcess): Add --backend-library parameter when launching the web process.
* WebProcess/wpe/WebProcessMainWPE.cpp: Call wpe_loader_init() with the library passed as --backend-library parameter.

Tools:

* MiniBrowser/wpe/main.cpp:
(main): Stop using WPE_BACKEND_LIBRARY env var.
* Scripts/run-wpe-tests:
(WPETestRunner.__init__): Ditto.
* Scripts/webkitpy/port/wpe.py:
(WPEPort.setup_environ_for_server): Ditto.
* wpe/backends/ViewBackend.cpp:
(WPEToolingBackends::ViewBackend::ViewBackend): Call wpe_loader_init() to ensure WPEBackend-fdo is used.
* wpe/jhbuild.modules: Upgrade WPEBackend to version 0.2.0.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233763 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoUnreviewed, rolling out r233714.
commit-queue@webkit.org [Thu, 12 Jul 2018 05:40:20 +0000 (05:40 +0000)]
Unreviewed, rolling out r233714.
https://bugs.webkit.org/show_bug.cgi?id=187579

it made tests time out (Requested by pizlo on #webkit).

Reverted changeset:

"Change the reoptimization backoff base to 1.3 from 2"
https://bugs.webkit.org/show_bug.cgi?id=187540
https://trac.webkit.org/changeset/233714

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233762 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoMediaDevices should derive from EventTarget in its IDL
youenn@apple.com [Thu, 12 Jul 2018 05:27:17 +0000 (05:27 +0000)]
MediaDevices should derive from EventTarget in its IDL
https://bugs.webkit.org/show_bug.cgi?id=187575

Reviewed by Chris Dumez.

Source/WebCore:

Test: fast/mediastream/MediaDevices-addEventListener.html

* Modules/mediastream/MediaDevices.idl:

LayoutTests:

* fast/mediastream/MediaDevices-addEventListener-expected.txt: Added.
* fast/mediastream/MediaDevices-addEventListener.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233761 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[GLIB] Add API to allow creating variadic functions
carlosgc@webkit.org [Thu, 12 Jul 2018 05:18:20 +0000 (05:18 +0000)]
[GLIB] Add API to allow creating variadic functions
https://bugs.webkit.org/show_bug.cgi?id=187517

Reviewed by Michael Catanzaro.

Source/JavaScriptCore:

Add a _variadic alternate method for jsc_class_add_constructor, jsc_class_add_method and
jsc_value_new_function. In that case the callback always receives a GPtrArray of JSCValue.

* API/glib/JSCCallbackFunction.cpp:
(JSC::JSCCallbackFunction::create): Make the parameters optional.
(JSC::JSCCallbackFunction::JSCCallbackFunction): Ditto.
(JSC::JSCCallbackFunction::call): Handle the case of parameters being nullopt by creating a GPtrArray of
JSCValue for the arguments.
(JSC::JSCCallbackFunction::construct): Ditto.
* API/glib/JSCCallbackFunction.h:
* API/glib/JSCClass.cpp:
(jscClassCreateConstructor): Make the parameters optional.
(jsc_class_add_constructor_variadic): Pass nullopt as parameters to jscClassCreateConstructor.
(jscClassAddMethod): Make the parameters optional.
(jsc_class_add_method_variadic): Pass nullopt as parameters to jscClassAddMethod.
* API/glib/JSCClass.h:
* API/glib/JSCValue.cpp:
(jsc_value_object_define_property_accessor): Update now that parameters are optional.
(jscValueFunctionCreate): Make the parameters optional.
(jsc_value_new_function_variadic): Pass nullopt as parameters to jscValueFunctionCreate.
* API/glib/JSCValue.h:
* API/glib/docs/jsc-glib-4.0-sections.txt:

Tools:

Add test cases.

* TestWebKitAPI/Tests/JavaScriptCore/glib/TestJSC.cpp:
(testJSCFunction):
(fooCreateWithFooV):
(multiplyFooV):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233760 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[GLIB] Add jsc_context_get_global_object() to GLib API
carlosgc@webkit.org [Thu, 12 Jul 2018 05:13:56 +0000 (05:13 +0000)]
[GLIB] Add jsc_context_get_global_object() to GLib API
https://bugs.webkit.org/show_bug.cgi?id=187515

Reviewed by Michael Catanzaro.

Source/JavaScriptCore:

This wasn't exposed because we have convenient methods in JSCContext to get and set properties on the global
object. However, getting the global object could be useful in some cases, for example to give it a well known
name like 'window' in browsers and GJS.

* API/glib/JSCContext.cpp:
(jsc_context_get_global_object):
* API/glib/JSCContext.h:
* API/glib/docs/jsc-glib-4.0-sections.txt:

Tools:

Add test cases.

* TestWebKitAPI/Tests/JavaScriptCore/glib/TestJSC.cpp:
(testJSCGlobalObject):
(main):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233759 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[GLIB] Handle G_TYPE_STRV in glib API
carlosgc@webkit.org [Thu, 12 Jul 2018 05:11:55 +0000 (05:11 +0000)]
[GLIB] Handle G_TYPE_STRV in glib API
https://bugs.webkit.org/show_bug.cgi?id=187512

Reviewed by Michael Catanzaro.

Source/JavaScriptCore:

Add jsc_value_new_array_from_strv() and handle G_TYPE_STRV types in function parameters.

* API/glib/JSCContext.cpp:
(jscContextGValueToJSValue):
(jscContextJSValueToGValue):
* API/glib/JSCValue.cpp:
(jsc_value_new_array_from_strv):
* API/glib/JSCValue.h:
* API/glib/docs/jsc-glib-4.0-sections.txt:

Tools:

Add test cases.

* TestWebKitAPI/Tests/JavaScriptCore/glib/TestJSC.cpp:
(testJSCTypes):
(joinFunction):
(testJSCFunction):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233758 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[ews-build] EWS should unapply patch and re-run jsc tests when patch fails jsc tests
aakash_jain@apple.com [Thu, 12 Jul 2018 04:08:20 +0000 (04:08 +0000)]
[ews-build] EWS should unapply patch and re-run jsc tests when patch fails jsc tests
https://bugs.webkit.org/show_bug.cgi?id=187549

Reviewed by Alexey Proskuryakov.

* BuildSlaveSupport/ews-build/factories.py:
(JSCTestsFactory.__init__):
* BuildSlaveSupport/ews-build/steps.py:
(CompileJSCOnlyToT): Build ToT without patch.
(CompileJSCOnlyToT.doStepIf): Run this step only if patch failed to build.
(RunJavaScriptCoreTests.evaluateCommand): Set patchFailedJSCTests property appropriately.
(ReRunJavaScriptCoreTests): Re-runs JSC tests.
(ReRunJavaScriptCoreTests.doStepIf): Run this step only if patch failed JSC tests.
(ReRunJavaScriptCoreTests.evaluateCommand): Set patchFailedJSCTests property appropriately.
(RunJavaScriptCoreTestsToT): Run JSC tests on ToT without patch.
(RunJavaScriptCoreTestsToT.doStepIf): Run this step only if patch failed JSC tests.
* BuildSlaveSupport/ews-build/steps_unittest.py: Added unit-tests.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233757 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoUpdate iOS fullscreen alert text
jonlee@apple.com [Thu, 12 Jul 2018 04:06:27 +0000 (04:06 +0000)]
Update iOS fullscreen alert text
https://bugs.webkit.org/show_bug.cgi?id=187576
rdar://problem/42052284

Reviewed by Ryosuke Niwa.

Source/WebCore:

* English.lproj/Localizable.strings:

Source/WebKit:

* UIProcess/ios/fullscreen/WKFullScreenViewController.mm:
(-[WKFullScreenViewController _showPhishingAlert]): Update text and string keys.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233756 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoUnreviewed, rolling out r233742.
commit-queue@webkit.org [Thu, 12 Jul 2018 03:37:51 +0000 (03:37 +0000)]
Unreviewed, rolling out r233742.
https://bugs.webkit.org/show_bug.cgi?id=187577

Binary incompatible change with respect to Service Worker
registration map (Requested by dydz on #webkit).

Reverted changeset:

"Reduce size of WebCore::URL"
https://bugs.webkit.org/show_bug.cgi?id=186820
https://trac.webkit.org/changeset/233742

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233755 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoLayout Test editing/selection/navigation-clears-editor-state.html is flaky
rniwa@webkit.org [Thu, 12 Jul 2018 03:36:18 +0000 (03:36 +0000)]
Layout Test editing/selection/navigation-clears-editor-state.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=187389

Reviewed by Wenson Hsieh.

Use 200 iterations like we did before r233701 and mark it as SLOW.

* TestExpectations:
* editing/selection/navigation-clears-editor-state.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233754 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoREGRESSION (231276): Attempting to copy an image fails
rniwa@webkit.org [Thu, 12 Jul 2018 02:57:53 +0000 (02:57 +0000)]
REGRESSION (231276): Attempting to copy an image fails
https://bugs.webkit.org/show_bug.cgi?id=187212
Source/WebCore:

<rdar://problem/41540074>

Patch by Aditya Keerthi <akeerthi@apple.com> on 2018-07-11
Reviewed by Ryosuke Niwa.

Renamed methods to make it clear that one URL and one title are being passed in.

Test: editing/mac/pasteboard/can-copy-url-without-title.html

* platform/PasteboardStrategy.h:
* platform/PlatformPasteboard.h:
* platform/ios/PlatformPasteboardIOS.mm:
(WebCore::PlatformPasteboard::setURL):
* platform/mac/PasteboardMac.mm:
(WebCore::writeURLForTypes):
(WebCore::Pasteboard::writeTrustworthyWebURLsPboardType):
* platform/mac/PlatformPasteboardMac.mm:
(WebCore::PlatformPasteboard::setURL):

Source/WebKit:

<rdar://problem/41540074>

Patch by Aditya Keerthi <akeerthi@apple.com> on 2018-07-11
Reviewed by Ryosuke Niwa.

r210683 introduced logic to prevent file URLs from being copied to the clipboard
in unexpected cases. In order to achieve this functionality,
checkURLReceivedFromWebProcess was called on all items in the pathnames array
passed into WebPasteboardProxy::setPasteboardPathnamesForType. However, this
method is a misnomer, as the pathnames array always contains exactly one URL and
one title for the URL.

Renamed methods to make it clear that one URL and one title are being passed in
and updated logic to ensure that checkURLReceivedFromWebProcess is only called
on the URL and not the title.

* Shared/WebCoreArgumentCoders.cpp:
(IPC::ArgumentCoder<PasteboardURL>::encode):
(IPC::ArgumentCoder<PasteboardURL>::decode):
* Shared/WebCoreArgumentCoders.h:
* UIProcess/Cocoa/WebPasteboardProxyCocoa.mm:
(WebKit::WebPasteboardProxy::setPasteboardURL):
* UIProcess/WebPasteboardProxy.h:
* UIProcess/WebPasteboardProxy.messages.in:
* WebProcess/WebCoreSupport/WebPlatformStrategies.cpp:
(WebKit::WebPlatformStrategies::setURL):
* WebProcess/WebCoreSupport/WebPlatformStrategies.h:

Source/WebKitLegacy/mac:

<rdar://problem/41540074>

Patch by Aditya Keerthi <akeerthi@apple.com> on 2018-07-11
Reviewed by Ryosuke Niwa.

Renamed methods to make it clear that one URL and one title are being passed in.

* WebCoreSupport/WebPlatformStrategies.h:
* WebCoreSupport/WebPlatformStrategies.mm:
(WebPlatformStrategies::setURL):

Tools:

Patch by Aditya Keerthi <akeerthi@apple.com> on 2018-07-11
Reviewed by Ryosuke Niwa.
<rdar://problem/41540074>

Augmented test to include a URL where the lastPathComponent looks similar to a
file URL.

* TestWebKitAPI/Tests/mac/ContextMenuCanCopyURL.html:
* TestWebKitAPI/Tests/mac/ContextMenuCanCopyURL.mm:
(TestWebKitAPI::TEST):

LayoutTests:

<rdar://problem/41540074>

Patch by Aditya Keerthi <akeerthi@apple.com> on 2018-07-11
Reviewed by Ryosuke Niwa.

Added test to ensure that we can copy a URL without a title, and the URL's
lastPathComponent appears like a file URL.

* editing/mac/pasteboard/can-copy-url-without-title-expected.txt: Added.
* editing/mac/pasteboard/can-copy-url-without-title.html: Added.
* platform/mac-wk1/TestExpectations: The added test is WK2-only due to webkit.org/b/187230.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233753 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoAdd SPI for immediate injection of user scripts
commit-queue@webkit.org [Thu, 12 Jul 2018 02:32:19 +0000 (02:32 +0000)]
Add SPI for immediate injection of user scripts
https://bugs.webkit.org/show_bug.cgi?id=173342
<rdar://problem/29202285>

Patch by Alex Christensen <achristensen@webkit.org> on 2018-07-11
Reviewed by Brady Eidson, Youenn Fablet, and Geoff Garen.

Source/WebCore:

The new SPI is WKUserContentController._addUserScriptImmediately.
It is covered by new API tests.
Existing functionality remains unchanged unless the new SPI is adopted.

* page/Frame.cpp:
(WebCore::Frame::injectUserScripts):
(WebCore::Frame::injectUserScriptImmediately):
Move injection functionality to allow us to call it directly from the new SPI.
* page/Frame.h:
* page/Page.cpp:
(WebCore::Page::forEachPage):
* page/Page.h:

Source/WebKit:

* UIProcess/API/C/WKPageGroup.cpp:
(WKPageGroupAddUserScript):
* UIProcess/API/C/WKUserContentControllerRef.cpp:
(WKUserContentControllerAddUserScript):
* UIProcess/API/Cocoa/WKUserContentController.mm:
(-[WKUserContentController addUserScript:]):
(-[WKUserContentController _addUserScriptImmediately:]):
* UIProcess/API/Cocoa/WKUserContentControllerPrivate.h:
* UIProcess/UserContent/WebUserContentControllerProxy.cpp:
(WebKit::WebUserContentControllerProxy::addUserScript):
* UIProcess/UserContent/WebUserContentControllerProxy.h:
* WebProcess/UserContent/WebUserContentController.cpp:
(WebKit::WebUserContentController::addUserScripts):
(WebKit::WebUserContentController::addUserScriptInternal):
If we are to inject the script internally, inject it into the appropriate pages.
If we're injecting into the top frame only, there's no need to traverse the frame tree.
(WebKit::WebUserContentController::addUserScript):
* WebProcess/UserContent/WebUserContentController.h:
* WebProcess/UserContent/WebUserContentController.messages.in:
* WebProcess/WebPage/WebPage.cpp:
(WebKit::m_cpuLimit):

Tools:

* TestWebKitAPI/Tests/WebKit2Cocoa/UserContentController.mm:
(waitForMessage):
(TEST):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233752 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[iOS WK2] Address a possible cause of missing tiles
simon.fraser@apple.com [Thu, 12 Jul 2018 02:09:12 +0000 (02:09 +0000)]
[iOS WK2] Address a possible cause of missing tiles
https://bugs.webkit.org/show_bug.cgi?id=187570
rdar://problem/40941118

Reviewed by Tim Horton.

If the web process crashes, it's possible for the user to trigger a scroll before
the process is re-launched. The pre-commit handler will bail early on the _isValid
check without clearing _hasScheduledVisibleRectUpdate, so that remains YES and we
get stuck doing no more visible content rect updates.

Fix by adding -[WKWebView _didRelaunchProcess] (WKContentView got this already), and
in that clear state that could have been set in the UI process while the web process
connection was invalid.

Also add more release logging around process termination, crash and relaunch.

* UIProcess/API/Cocoa/WKWebView.mm:
(-[WKWebView _processDidExit]):
(-[WKWebView _didRelaunchProcess]):
(-[WKWebView _scheduleVisibleContentRectUpdateAfterScrollInView:]):
(-[WKWebView _updateVisibleContentRects]):
* UIProcess/API/Cocoa/WKWebViewInternal.h:
* UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::reattachToWebProcess):
(WebKit::WebPageProxy::processDidTerminate):
* UIProcess/WebProcessProxy.cpp:
(WebKit::WebProcessProxy::didClose):
(WebKit::WebProcessProxy::didFinishLaunching):
* UIProcess/ios/PageClientImplIOS.mm:
(WebKit::PageClientImpl::didRelaunchProcess):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233751 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoMove WHLSL spec work to the W3C GPUWeb GitHub repository
mmaxfield@apple.com [Thu, 12 Jul 2018 01:35:16 +0000 (01:35 +0000)]
Move WHLSL spec work to the W3C GPUWeb GitHub repository

See https://github.com/gpuweb/WHLSL

* WebGPUShadingLanguageRI/SpecWork/Makefile: Removed.
* WebGPUShadingLanguageRI/SpecWork/WSL.g4: Removed.
* WebGPUShadingLanguageRI/SpecWork/WSL.ott: Removed.
* WebGPUShadingLanguageRI/SpecWork/source/conf.py: Removed.
* WebGPUShadingLanguageRI/SpecWork/source/index.rst: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233750 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoRelease logging dumps "Cleaning up dragging state…" every time gesture recognizers...
wenson_hsieh@apple.com [Thu, 12 Jul 2018 00:52:52 +0000 (00:52 +0000)]
Release logging dumps "Cleaning up dragging state…" every time gesture recognizers are reset in WKContentView
https://bugs.webkit.org/show_bug.cgi?id=187562

Reviewed by Tim Horton.

Tweaks up a release logging statement, such that it only logs when performing a drag or drop.

* UIProcess/ios/WKContentViewInteraction.mm:
(-[WKContentView cleanUpDragSourceSessionState]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233749 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoForce WKTR to use the light appearance
dino@apple.com [Thu, 12 Jul 2018 00:44:24 +0000 (00:44 +0000)]
Force WKTR to use the light appearance
https://bugs.webkit.org/show_bug.cgi?id=187572
<rdar://problem/42097805>

Reviewed by Simon Fraser.

Set the NSAppearance on the NSWindow to light mode.

* WebKitTestRunner/mac/PlatformWebViewMac.mm:
(WTR::PlatformWebView::PlatformWebView): Use the Aqua appearance.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233748 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[WinCairo] Unreviewed test gardening.
ross.kirsling@sony.com [Thu, 12 Jul 2018 00:18:15 +0000 (00:18 +0000)]
[WinCairo] Unreviewed test gardening.

* platform/wincairo/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233747 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoUnreviewed, rolling out r233741.
commit-queue@webkit.org [Thu, 12 Jul 2018 00:08:39 +0000 (00:08 +0000)]
Unreviewed, rolling out r233741.
https://bugs.webkit.org/show_bug.cgi?id=187568

"WKTR vs exp features, booooooo" (Requested by bradee-oh on
#webkit).

Reverted changeset:

"Make process-swap-on-navigation an experimental feature."
https://bugs.webkit.org/show_bug.cgi?id=187558
https://trac.webkit.org/changeset/233741

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233746 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoLayout Test media/modern-media-controls/media-controller/media-controller-auto-hide...
ryanhaddad@apple.com [Wed, 11 Jul 2018 23:30:28 +0000 (23:30 +0000)]
Layout Test media/modern-media-controls/media-controller/media-controller-auto-hide-rewind-with-mouse-enter.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=187420

Unreviewed test gardening.

Patch by Truitt Savell <tsavell@apple.com> on 2018-07-11

* platform/mac/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233745 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoSkip JSC test stress/keep-checks-when-converting-to-lazy-js-constant-in-strength...
ryanhaddad@apple.com [Wed, 11 Jul 2018 21:09:48 +0000 (21:09 +0000)]
Skip JSC test stress/keep-checks-when-converting-to-lazy-js-constant-in-strength-reduction.js
https://bugs.webkit.org/show_bug.cgi?id=187559

Unreviewed test gardening.

* stress/keep-checks-when-converting-to-lazy-js-constant-in-strength-reduction.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233743 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoReduce size of WebCore::URL
commit-queue@webkit.org [Wed, 11 Jul 2018 20:48:49 +0000 (20:48 +0000)]
Reduce size of WebCore::URL
https://bugs.webkit.org/show_bug.cgi?id=186820

Patch by Alex Christensen <achristensen@webkit.org> on 2018-07-11
Reviewed by Yusuke Suzuki.

Source/WebCore:

We were using 32 bits for the length of the port, which is always between 0 and 5 inclusive
because port numbers are missing or between 0 and 65535.  Let's just use 3 bits here.
We were using 32 bits for the length of the scheme, which is usually 3-5 characters and can be
longer for some custom schemes, but I've never seen one more than 20 characters.  If we assume
schemes are always less than 64MB, we can save 8 bytes per URL!

No change in behavior, just less memory use!

* platform/URL.cpp:
(WebCore::URL::invalidate):
(WebCore::URL::lastPathComponent const):
(WebCore::URL::port const):
(WebCore::URL::protocolHostAndPort const):
(WebCore::URL::path const):
(WebCore::URL::removePort):
(WebCore::URL::setPort):
(WebCore::URL::setHostAndPort):
(WebCore::URL::setPath):
* platform/URL.h:
(WebCore::URL::encode const):
(WebCore::URL::decode):
(WebCore::URL::hasPath const):
(WebCore::URL::pathStart const):
* platform/URLParser.cpp:
(WebCore::URLParser::copyBaseWindowsDriveLetter):
(WebCore::URLParser::urlLengthUntilPart):
(WebCore::URLParser::copyURLPartsUntil):
(WebCore::URLParser::shouldPopPath):
(WebCore::URLParser::popPath):
(WebCore::URLParser::parse):
(WebCore::URLParser::parsePort):
(WebCore::URLParser::parseHostAndPort):
(WebCore::URLParser::allValuesEqual):
(WebCore::URLParser::internalValuesConsistent):

Tools:

* TestWebKitAPI/Tests/WebCore/URLParser.cpp:
(TestWebKitAPI::TEST_F):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233742 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoMake process-swap-on-navigation an experimental feature.
beidson@apple.com [Wed, 11 Jul 2018 20:47:05 +0000 (20:47 +0000)]
Make process-swap-on-navigation an experimental feature.
<rdar://problem/41995053> and https://bugs.webkit.org/show_bug.cgi?id=187558

Reviewed by Geoff Garen.

* Shared/WebPreferences.yaml:

* UIProcess/WebPreferences.cpp:
(WebKit::WebPreferences::updateBoolValueForExperimentalFeatureKey):

* UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::createWebPage): If the new web page has PSON enabled via WebPreferences,
  absorb that setting into this process pool's configuration.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233741 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoIterator of Array.keys() returns object in wrong order
utatane.tea@gmail.com [Wed, 11 Jul 2018 20:29:47 +0000 (20:29 +0000)]
Iterator of Array.keys() returns object in wrong order
https://bugs.webkit.org/show_bug.cgi?id=185197

Reviewed by Keith Miller.

JSTests:

We align the shape of our iterator result to the spec: {value,done} instead of {done,value}.

* stress/iterator-field-order.js: Added.
(shouldBe):
(shouldBe.JSON.stringify.array.values.next):
(shouldBe.JSON.stringify.array.keys.next):
(shouldBe.JSON.stringify.array.entries.next):
(async.asyncIterator):
(iterator.next.then):
(generator):
(shouldBe.JSON.stringify.generator.next):
(shouldBe.JSON.stringify.map.keys.next):
(shouldBe.JSON.stringify.map.values.next):
(shouldBe.JSON.stringify.map.entries.next):
(set new):

Source/JavaScriptCore:

* builtins/ArrayIteratorPrototype.js:
(globalPrivate.arrayIteratorValueNext):
(globalPrivate.arrayIteratorKeyNext):
(globalPrivate.arrayIteratorKeyValueNext):
* builtins/AsyncFromSyncIteratorPrototype.js:
* builtins/AsyncGeneratorPrototype.js:
(globalPrivate.asyncGeneratorResolve):
* builtins/GeneratorPrototype.js:
(globalPrivate.generatorResume):
* builtins/MapIteratorPrototype.js:
(globalPrivate.mapIteratorNext):
* builtins/SetIteratorPrototype.js:
(globalPrivate.setIteratorNext):
* builtins/StringIteratorPrototype.js:
(next):
* runtime/IteratorOperations.cpp:
(JSC::createIteratorResultObjectStructure):
(JSC::createIteratorResultObject):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233740 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoFix remaining Cross-Origin-Resource-Policy failures, if any
youenn@apple.com [Wed, 11 Jul 2018 19:36:16 +0000 (19:36 +0000)]
Fix remaining Cross-Origin-Resource-Policy failures, if any
https://bugs.webkit.org/show_bug.cgi?id=186761
<rdar://problem/41209829>

Reviewed by Alex Christensen.

LayoutTests/imported/w3c:

Fixed some tests for correctness.

* web-platform-tests/fetch/cross-origin-resource-policy/fetch-in-service-worker-expected.txt:
* web-platform-tests/fetch/cross-origin-resource-policy/fetch.any-expected.txt:
* web-platform-tests/fetch/cross-origin-resource-policy/fetch.any.js:
(promise_test):
* web-platform-tests/fetch/cross-origin-resource-policy/fetch.any.worker-expected.txt:
* web-platform-tests/fetch/cross-origin-resource-policy/scheme-restriction.https.window-expected.txt: Added.
* web-platform-tests/fetch/cross-origin-resource-policy/scheme-restriction.https.window.html: Added.
* web-platform-tests/fetch/cross-origin-resource-policy/scheme-restriction.https.window.js:
* web-platform-tests/fetch/cross-origin-resource-policy/syntax.any-expected.txt:
* web-platform-tests/fetch/cross-origin-resource-policy/syntax.any.worker-expected.txt:

Source/WebCore:

Add case-sensitive check for CORP header value, as per fetch specification.
Add HTTP->HTTPS check for same-site case, as per fetch specification.
https://fetch.spec.whatwg.org/#cross-origin-resource-policy-check

Test: imported/w3c/web-platform-tests/fetch/cross-origin-resource-policy/scheme-restriction.https.window.html

* loader/CrossOriginAccessControl.cpp:
(WebCore::shouldCrossOriginResourcePolicyCancelLoad):
* platform/network/HTTPParsers.cpp:
(WebCore::parseCrossOriginResourcePolicyHeader):

Tools:

* TestWebKitAPI/Tests/WebCore/HTTPParsers.cpp:
(TestWebKitAPI::TEST):

 2018-07-11  Aakash Jain  <aakash_jain@apple.com>

[ews-build] EWS should unapply the patch and build ToT when patch fails to build
https://bugs.webkit.org/show_bug.cgi?id=187358

Reviewed by Alexey Proskuryakov.

* BuildSlaveSupport/ews-build/factories.py:
(BuildFactory.__init__):
* BuildSlaveSupport/ews-build/steps.py:
(CompileWebKit): Set haltOnFailure to false so as to run next steps (e.g.: un-apply patch, compile ToT).
(CompileWebKit.evaluateCommand): Set patchFailedToBuild property.
(CompileWebKitToT): Build ToT without patch.
(CompileWebKitToT.doStepIf): Run this step only if patch failed to build.
* BuildSlaveSupport/ews-build/steps_unittest.py: Added unit-tests. Also lower-cased Release and Debug.

LayoutTests:

Remove redundant tests with WPT.

* http/wpt/cross-origin-resource-policy/fetch-expected.txt: Removed.
* http/wpt/cross-origin-resource-policy/fetch-in-iframe-expected.txt: Removed.
* http/wpt/cross-origin-resource-policy/fetch-in-iframe.html: Removed.
* http/wpt/cross-origin-resource-policy/fetch.html: Removed.
* http/wpt/cross-origin-resource-policy/iframe-loads-expected.txt: Removed.
* http/wpt/cross-origin-resource-policy/iframe-loads.html: Removed.
* http/wpt/cross-origin-resource-policy/image-loads-expected.txt: Removed.
* http/wpt/cross-origin-resource-policy/image-loads.html: Removed.
* http/wpt/cross-origin-resource-policy/resources/hello.py: Removed.
* http/wpt/cross-origin-resource-policy/resources/iframe.py: Removed.
* http/wpt/cross-origin-resource-policy/resources/iframeFetch.html: Removed.
* http/wpt/cross-origin-resource-policy/resources/redirect.py: Removed.
* http/wpt/cross-origin-resource-policy/resources/script.py: Removed.
* http/wpt/cross-origin-resource-policy/script-loads-expected.txt: Removed.
* http/wpt/cross-origin-resource-policy/script-loads.html: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233739 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoDisable all network caching for HLS streams.
jer.noble@apple.com [Wed, 11 Jul 2018 19:29:58 +0000 (19:29 +0000)]
Disable all network caching for HLS streams.
https://bugs.webkit.org/show_bug.cgi?id=187544
<rdar://problem/41863600>

Reviewed by Chris Dumez.

Source/WebKit:

Revert the behavior added in r215263 where Media responses are cached if they are from
a resource whose expected content length is <4MB.

* NetworkProcess/cache/NetworkCache.cpp:
(WebKit::NetworkCache::makeStoreDecision):
(WebKit::NetworkCache::expectedTotalResourceSizeFromContentRange): Deleted.

LayoutTests:

* http/tests/cache/disk-cache/disk-cache-media-small-expected.txt:
* http/tests/cache/disk-cache/disk-cache-media-small.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233738 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[WinCairo] MIME type registry doesn't explicitly recognize *.xht
ross.kirsling@sony.com [Wed, 11 Jul 2018 19:22:07 +0000 (19:22 +0000)]
[WinCairo] MIME type registry doesn't explicitly recognize *.xht
https://bugs.webkit.org/show_bug.cgi?id=187555

Reviewed by Konstantin Tokarev.

Follow-up to r233715 -- WinCairo bots are failing on *.xht layout tests as they lack a registry key for this filetype.
(Hopefully this is the last one to add.)

* platform/win/MIMETypeRegistryWin.cpp:
(WebCore::MIMETypeRegistry::getMIMETypeForExtension):
Add *.xht to the list of recognized filetypes.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233737 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoVersioning.
kocsen_chung@apple.com [Wed, 11 Jul 2018 18:34:26 +0000 (18:34 +0000)]
Versioning.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233735 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago 2018-07-11 Aakash Jain <aakash_jain@apple.com>
aakash_jain@apple.com [Wed, 11 Jul 2018 18:10:10 +0000 (18:10 +0000)]
 2018-07-11  Aakash Jain  <aakash_jain@apple.com>

[ews-build] EWS should unapply the patch and build ToT when patch fails to build
https://bugs.webkit.org/show_bug.cgi?id=187358

Reviewed by Alexey Proskuryakov.

* BuildSlaveSupport/ews-build/factories.py:
(BuildFactory.__init__):
* BuildSlaveSupport/ews-build/steps.py:
(CompileWebKit): Set haltOnFailure to false so as to run next steps (e.g.: un-apply patch, compile ToT).
(CompileWebKit.evaluateCommand): Set patchFailedToBuild property.
(CompileWebKitToT): Build ToT without patch.
(CompileWebKitToT.doStepIf): Run this step only if patch failed to build.
* BuildSlaveSupport/ews-build/steps_unittest.py: Added unit-tests. Also lower-cased Release and Debug.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233734 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[WSL] WIP, some work on the spec.
rmorisset@apple.com [Wed, 11 Jul 2018 17:29:31 +0000 (17:29 +0000)]
[WSL] WIP, some work on the spec.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233733 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[Web Animations] Make WPT test at interfaces/KeyframeEffect/processing-a-keyframes...
graouts@webkit.org [Wed, 11 Jul 2018 16:40:27 +0000 (16:40 +0000)]
[Web Animations] Make WPT test at interfaces/KeyframeEffect/processing-a-keyframes-argument-001.html pass reliably
https://bugs.webkit.org/show_bug.cgi?id=186501
<rdar://problem/41000224>

Unreviewed build fix after r233729 and r233730.

* animation/KeyframeEffectReadOnly.cpp:
(WebCore::processIterableKeyframes):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233731 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[Web Animations] Make WPT test at interfaces/KeyframeEffect/processing-a-keyframes...
graouts@webkit.org [Wed, 11 Jul 2018 16:34:37 +0000 (16:34 +0000)]
[Web Animations] Make WPT test at interfaces/KeyframeEffect/processing-a-keyframes-argument-001.html pass reliably
https://bugs.webkit.org/show_bug.cgi?id=186501
<rdar://problem/41000224>

Unreviewed build fix after r233729.

* animation/KeyframeEffectReadOnly.cpp:
(WebCore::processIterableKeyframes):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233730 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[Web Animations] Make WPT test at interfaces/KeyframeEffect/processing-a-keyframes...
graouts@webkit.org [Wed, 11 Jul 2018 15:54:00 +0000 (15:54 +0000)]
[Web Animations] Make WPT test at interfaces/KeyframeEffect/processing-a-keyframes-argument-001.html pass reliably
https://bugs.webkit.org/show_bug.cgi?id=186501
<rdar://problem/41000224>

Reviewed by Dean Jackson.

LayoutTests/imported/w3c:

Mark 2 new WPT progressions.

* web-platform-tests/web-animations/interfaces/KeyframeEffect/processing-a-keyframes-argument-001-expected.txt:

Source/WebCore:

There were two remaining assertions that we were failing in this WPT test file, both related to processing iterable keyframes.
The first one was failing because didn't correctly propagate the TypeError exception in the forEachInIterable() callback. The
second one was failing because we didn't use the "process a keyframe-like object" procedure when processing iterable keyframes
and, as such, we didn't correctly sort property alphabetically before reading their values.

To fix this second issue, we make processIterableKeyframes() use processKeyframeLikeObject(). To do so, we update processKeyframeLikeObject()
to accept a new boolean flag to match the "allow lists" flag from the specification. We also ensure we sort the properties *before*
reading from them which we didn't use to do previously.

* animation/KeyframeEffectReadOnly.cpp:
(WebCore::processKeyframeLikeObject):
(WebCore::processIterableKeyframes):
(WebCore::processPropertyIndexedKeyframes):
* animation/KeyframeEffectReadOnly.h:
* animation/KeyframeEffectReadOnly.idl:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233729 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoSimpleLineLayout::FlowContents wastes 54KB of Vector capacity on nytimes.com
zalan@apple.com [Wed, 11 Jul 2018 13:30:14 +0000 (13:30 +0000)]
SimpleLineLayout::FlowContents wastes 54KB of Vector capacity on nytimes.com
https://bugs.webkit.org/show_bug.cgi?id=186709
<rdar://problem/41173793>

Reviewed by Simon Fraser.

The size of the m_segments vector in SimpleLineLayoutFlowContents is alway pre-computed and don't change after the initial append.

Not testable.

* rendering/SimpleLineLayoutFlowContents.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233728 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[GTK] Unreviewed test gardening
Ms2ger@igalia.com [Wed, 11 Jul 2018 10:02:05 +0000 (10:02 +0000)]
[GTK] Unreviewed test gardening
https://bugs.webkit.org/show_bug.cgi?id=187551

Unreviewed gardening.

WebDriverTests:

* TestExpectations.json:
  - imported/selenium/py/test/selenium/webdriver/common: fixed in r233548
  - imported/w3c/webdriver/tests/actions/key.py: fixed in r233499
  - imported/w3c/webdriver/tests/actions/special_keys.py: properly escape test names

LayoutTests:

* platform/gtk/TestExpectations: Mark new test as failing: it depends on mac-only code.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233727 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[WPE] AC for fixed-position elements is not enabled
zandobersek@gmail.com [Wed, 11 Jul 2018 08:18:28 +0000 (08:18 +0000)]
[WPE] AC for fixed-position elements is not enabled
https://bugs.webkit.org/show_bug.cgi?id=187514

Reviewed by Carlos Garcia Campos.

Enable accelerated compositing for fixed-position elements when using
AcceleratedDrawingArea and AC is force-enabled. This effectively enables
this option for the WPE port which is using the AcceleratedDrawingArea
implementation and not DrawingAreaImpl like the GTK+ port.

* WebProcess/WebPage/AcceleratedDrawingArea.cpp:
(WebKit::AcceleratedDrawingArea::updatePreferences):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233726 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months ago[Win][WebKit] Disable accelerated compositing until it is supported
chris.reid@sony.com [Wed, 11 Jul 2018 07:59:21 +0000 (07:59 +0000)]
[Win][WebKit] Disable accelerated compositing until it is supported
https://bugs.webkit.org/show_bug.cgi?id=187503

Reviewed by Žan Doberšek.

* UIProcess/win/WebView.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233725 268f45cc-cd09-0410-ab3c-d52691b4dbfc

14 months agoUnreviewed, clean up test262 expectations
utatane.tea@gmail.com [Wed, 11 Jul 2018 07:59:12 +0000 (07:59 +0000)]
Unreviewed, clean up test262 expectations

* test262/expectations.yaml:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233724 268f45cc-cd09-0410-ab3c-d52691b4dbfc