Timeline



Mar 18, 2017:

6:06 PM WebKitGTK/2.16.x edited by Michael Catanzaro
(diff)
5:58 PM Changeset in webkit [214142] by Michael Catanzaro
  • 2 edits in trunk/Source/WebCore/platform/gtk/po

Update Ukrainian translation
​https://bugs.webkit.org/show_bug.cgi?id=169812

Patch by Yuri Chornoivan <​yurchor@ukr.net> on 2017-03-18
Rubber-stamped by Michael Catanzaro.

  • uk.po:
5:58 PM Changeset in webkit [214141] by webkit@devinrousso.com
  • 2 edits in trunk/Source/WebInspectorUI

Web Inspector: Empty attribute added to a DOM tree outline element should not add whitespace within the tag
​https://bugs.webkit.org/show_bug.cgi?id=139623

Reviewed by Brian Burg.

  • UserInterface/Views/DOMTreeElement.js:

(WebInspector.DOMTreeElement.prototype._attributeEditingCommitted):
If the newText after editing has no content, remove the element for that attribute.

5:41 PM Changeset in webkit [214140] by webkit@devinrousso.com
  • 3 edits in trunk/Source/WebInspectorUI

Web Inspector: add keyboard shortcut to clear timeline records
​https://bugs.webkit.org/show_bug.cgi?id=132485

Reviewed by Brian Burg.

  • Localizations/en.lproj/localizedStrings.js:
  • UserInterface/Views/TimelineRecordingContentView.js:

(WebInspector.TimelineRecordingContentView):
(WebInspector.TimelineRecordingContentView.prototype.handleClearShortcut):

1:22 PM Changeset in webkit [214139] by jonlee@apple.com
  • 13 edits in trunk

Add support for ImplementedAs, Clamp, EnforceRange, TreatNullAs for dictionary members
​https://bugs.webkit.org/show_bug.cgi?id=169731

Reviewed by Alex Christensen.

LayoutTests/imported/w3c:

Rebaseline tests. The results seem to show a problem in the original tests.

  • web-platform-tests/WebCryptoAPI/encrypt_decrypt/aes_gcm.worker-expected.txt:
  • web-platform-tests/WebCryptoAPI/encrypt_decrypt/test_aes_gcm-expected.txt:

Source/WebCore:

  • bindings/scripts/CodeGeneratorJS.pm:

(GenerateDictionaryImplementationContent): Create a new $implementationAsKey. Look for
ImplementedAs, and if it exists, override the value with the alias. Set the parameters of
convert<>() using JSValueToNative.
(IsValidContextForJSValueToNative): Update to include IDLDictionaryMembers.
(JSValueToNative): Bypass parseEnumeration serialization for enums if the context is an
IDLDictionaryMember. Dictionary members need convert<IDLEnumeration> which throw a TypeError
or return the enum (and utilizes parseEnumeration). parseEnumeration, in contrast, returns
an optional.

  • bindings/scripts/test/TestObj.idl: Add test cases in TestDictionary.

Update test results.

  • bindings/scripts/test/JS/JSTestEventConstructor.cpp:
  • bindings/scripts/test/JS/JSTestObj.cpp:
  • bindings/scripts/test/JS/JSTestStandaloneDictionary.cpp:

LayoutTests:

Rebaseline. The enhancement progressed the results of these tests.

  • crypto/subtle/aes-gcm-encrypt-malformed-parameters-expected.txt:
  • crypto/subtle/aes-generate-key-malformed-parameters-expected.txt:
  • crypto/subtle/hmac-generate-key-malformed-parameters-expected.txt:
1:21 PM Changeset in webkit [214138] by gskachkov@gmail.com
  • 2 edits in trunk/Source/JavaScriptCore

[JSC] Remove unnecessary condition from needsDerivedConstructorInArrowFunctionLexicalEnvironment in BytecodeGenerator.cpp
​https://bugs.webkit.org/show_bug.cgi?id=169832

Reviewed by Mark Lam.

Remove already covered condition in needsDerivedConstructorInArrowFunctionLexicalEnvironment
function. Condition isConstructor() && constructorKind() == ConstructorKind::Extends is already
isClassContext.

  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::needsDerivedConstructorInArrowFunctionLexicalEnvironment):

1:03 PM Changeset in webkit [214137] by Simon Fraser
  • 3 edits in trunk/Source/WebCore

Use initializers for Document member variables
​https://bugs.webkit.org/show_bug.cgi?id=169848

Reviewed by Chris Dumez.

Move Document to use C++11 brace initializers.

No behavior change.

  • dom/Document.cpp:

(WebCore::Document::Document):

  • dom/Document.h:

(WebCore::Document::xmlStandaloneStatus):
(WebCore::Document::setHasXMLDeclaration):

12:51 PM Changeset in webkit [214136] by Chris Dumez
  • 8 edits in trunk

innerText setter inserts empty text node if value starts with newline
​https://bugs.webkit.org/show_bug.cgi?id=169600

Reviewed by Ryosuke Niwa.

LayoutTests/imported/w3c:

Re-sync tests with upstream to gain test coverage.

  • web-platform-tests/innerText/getter-expected.txt:
  • web-platform-tests/innerText/getter-tests.js:
  • web-platform-tests/innerText/setter-expected.txt:
  • web-platform-tests/innerText/setter-tests.js:
  • web-platform-tests/innerText/setter.html:

Source/WebCore:

innerText setter should not create empty text nodes as per:

No new tests, updated existing test.

  • html/HTMLElement.cpp:

(WebCore::textToFragment):

10:52 AM Changeset in webkit [214135] by Chris Dumez
  • 19 edits
    21 adds in trunk

Allow setting the prototype of cross-origin objects, as long as they don't change
​https://bugs.webkit.org/show_bug.cgi?id=169787

Reviewed by Mark Lam.

LayoutTests/imported/w3c:

Import test coverage from W3C web-platform-tests.

  • web-platform-tests/common/domain-setter.sub.html: Added.
  • web-platform-tests/common/test-setting-immutable-prototype.js: Added.

(self.testSettingImmutablePrototypeToNewValueOnly.test):
(self.testSettingImmutablePrototypeToNewValueOnly):
(self.testSettingImmutablePrototype.else.test):
(self.testSettingImmutablePrototype):

  • web-platform-tests/common/w3c-import.log:
  • web-platform-tests/html/browsers/history/the-location-interface/location-prototype-setting-cross-origin-domain.sub-expected.txt: Added.
  • web-platform-tests/html/browsers/history/the-location-interface/location-prototype-setting-cross-origin-domain.sub.html: Added.
  • web-platform-tests/html/browsers/history/the-location-interface/location-prototype-setting-goes-cross-origin-domain.sub-expected.txt: Added.
  • web-platform-tests/html/browsers/history/the-location-interface/location-prototype-setting-goes-cross-origin-domain.sub.html: Added.
  • web-platform-tests/html/browsers/history/the-location-interface/location-prototype-setting-same-origin-domain.sub-expected.txt: Added.
  • web-platform-tests/html/browsers/history/the-location-interface/location-prototype-setting-same-origin-domain.sub.html: Added.
  • web-platform-tests/html/browsers/history/the-location-interface/location-prototype-setting-same-origin-expected.txt: Added.
  • web-platform-tests/html/browsers/history/the-location-interface/location-prototype-setting-same-origin.html: Added.
  • web-platform-tests/html/browsers/history/the-location-interface/w3c-import.log:
  • web-platform-tests/html/browsers/the-windowproxy-exotic-object/w3c-import.log: Added.
  • web-platform-tests/html/browsers/the-windowproxy-exotic-object/windowproxy-prototype-setting-cross-origin-domain.sub-expected.txt: Added.
  • web-platform-tests/html/browsers/the-windowproxy-exotic-object/windowproxy-prototype-setting-cross-origin-domain.sub.html: Added.
  • web-platform-tests/html/browsers/the-windowproxy-exotic-object/windowproxy-prototype-setting-goes-cross-origin-domain.sub-expected.txt: Added.
  • web-platform-tests/html/browsers/the-windowproxy-exotic-object/windowproxy-prototype-setting-goes-cross-origin-domain.sub.html: Added.
  • web-platform-tests/html/browsers/the-windowproxy-exotic-object/windowproxy-prototype-setting-same-origin-domain.sub-expected.txt: Added.
  • web-platform-tests/html/browsers/the-windowproxy-exotic-object/windowproxy-prototype-setting-same-origin-domain.sub.html: Added.
  • web-platform-tests/html/browsers/the-windowproxy-exotic-object/windowproxy-prototype-setting-same-origin-expected.txt: Added.
  • web-platform-tests/html/browsers/the-windowproxy-exotic-object/windowproxy-prototype-setting-same-origin.html: Added.

Source/JavaScriptCore:

  • runtime/JSGlobalObject.h:

Mark JS global object as an immutable prototype exotic object to match Window.

  • runtime/JSObject.cpp:

(JSC::JSObject::setPrototypeWithCycleCheck):
Update setPrototypeWithCycleCheck() for immutable prototype exotic objects in order
to align with:

In particular, we need to call GetPrototypeOf? and return true if it returns the same
value as the new prototype. We really need to call GetPrototypeOf? and not merely
getting the prototype slot via getPrototypeDirect() since Location and Window override
GetPrototypeOf? to return null in the cross-origin case.

  • runtime/JSProxy.cpp:

(JSC::JSProxy::setPrototype):
Update JSProxy::setPrototype() to forward such calls to its target. This is needed so
we end up calling JSObject::setPrototypeWithCycleCheck() for the Window object.
Handling immutable prototype exotic objects in that method does the right thing for
Window.

Source/WebCore:

Allow setting the prototype of cross-origin objects, as long as they don't change:

Tests: imported/w3c/web-platform-tests/common/domain-setter.sub.html

imported/w3c/web-platform-tests/html/browsers/history/the-location-interface/location-prototype-setting-cross-origin-domain.sub.html
imported/w3c/web-platform-tests/html/browsers/history/the-location-interface/location-prototype-setting-goes-cross-origin-domain.sub.html
imported/w3c/web-platform-tests/html/browsers/history/the-location-interface/location-prototype-setting-same-origin-domain.sub.html
imported/w3c/web-platform-tests/html/browsers/history/the-location-interface/location-prototype-setting-same-origin.html
imported/w3c/web-platform-tests/html/browsers/the-windowproxy-exotic-object/windowproxy-prototype-setting-cross-origin-domain.sub.html
imported/w3c/web-platform-tests/html/browsers/the-windowproxy-exotic-object/windowproxy-prototype-setting-goes-cross-origin-domain.sub.html
imported/w3c/web-platform-tests/html/browsers/the-windowproxy-exotic-object/windowproxy-prototype-setting-same-origin-domain.sub.html
imported/w3c/web-platform-tests/html/browsers/the-windowproxy-exotic-object/windowproxy-prototype-setting-same-origin.html

  • bindings/js/JSLocationCustom.cpp:
  • page/Location.idl:

Drop custom setPrototype() implementation for Location and mark Location as an immutable
prototype exotic object now that JSC does the right thing for those.

LayoutTests:

  • http/tests/security/cross-frame-access-object-setPrototypeOf-expected.txt:
  • http/tests/security/cross-frame-access-object-setPrototypeOf.html:
  • js/dom/setPrototypeOf-location-window-expected.txt:
  • js/dom/setPrototypeOf-location-window.html:

Add checks for setting prototype of window and location objects to the same value
to make sure it does not throw, both in the same origin and the cross origin
cases.

  • fast/dom/Window/window-custom-prototype-crash-expected.txt:
  • http/tests/security/xss-DENIED-regular-propterty-with-iframe-proto-expected.txt:
  • js/object-literal-shorthand-construction-expected.txt:

Rebaseline tests now that the exception message is a bit different.

  • js/script-tests/prototype-assignment.js:

Remove special casing for Window, it now behaves like a regular immutable prototype
exotic object.

Mar 17, 2017:

9:32 PM Changeset in webkit [214134] by commit-queue@webkit.org
  • 6 edits in trunk/Source/WebCore

Stop sending media data as soon as peer connection is closed
​https://bugs.webkit.org/show_bug.cgi?id=169813

Patch by Youenn Fablet <​youenn@apple.com> on 2017-03-17
Reviewed by Eric Carlson.

Stopping sending of media data as soon as the peer connection is closed.
By stopping sending at source level, this may stop by a few frames/chunks earlier than before.
Making sure RealtimeOutgoingVideoSource is stopping observing the media source when being stopped and deleted.

  • Modules/mediastream/libwebrtc/LibWebRTCPeerConnectionBackend.cpp:

(WebCore::LibWebRTCPeerConnectionBackend::doStop):

  • platform/mediastream/mac/RealtimeOutgoingAudioSource.cpp:

(WebCore::RealtimeOutgoingAudioSource::stop):

  • platform/mediastream/mac/RealtimeOutgoingAudioSource.h:
  • platform/mediastream/mac/RealtimeOutgoingVideoSource.cpp:

(WebCore::RealtimeOutgoingVideoSource::stop):

  • platform/mediastream/mac/RealtimeOutgoingVideoSource.h:
8:49 PM Changeset in webkit [214133] by webkit@devinrousso.com
  • 4 edits in trunk/Source/WebInspectorUI

Web Inspector: wrong filename when saving resource from ResourceImageContentView
​https://bugs.webkit.org/show_bug.cgi?id=54454

Reviewed by Joseph Pecoraro.

  • UserInterface/Views/ResourceContentView.js:

(WebInspector.ResourceContentView.prototype.get supportsSave):
(WebInspector.ResourceContentView.prototype.get saveData):

  • UserInterface/Views/TextResourceContentView.js:

(WebInspector.TextResourceContentView.prototype.get supportsSave): Deleted.
Add save data getter to the parent class, so that both Images and Fonts can be saved.

  • UserInterface/Views/Resource.js:

(WebInspector.Resource.prototype.canRequestContent):
Drive-by: remove unused function.

7:36 PM Changeset in webkit [214132] by commit-queue@webkit.org
  • 13 edits
    1 copy
    1 add in trunk

Implement incoming webrtc data based on tracksCurr
​https://bugs.webkit.org/show_bug.cgi?id=169836

Patch by Youenn Fablet <​youenn@apple.com> on 2017-03-17
Reviewed by Eric Carlson.

Source/WebCore:

Test: webrtc/video-with-receiver.html

Constructing incoming tracks based on libwebrtc OnAddTrack.
Constructing incoming media streams based on libwebrtc OnAddStream.
Firing only addstream if legacy API flag is on.

Ensuring that media stream and media stream tracks relationship is still correctly implemented.
For that, we keep a map that relates libwebrtc media streams with WebCore media streams.
Adding the ability to get a receiver related to the track on the track event.

Implementing the possibility to create a transceiver ahead of track arrival time.
Created transceivers that are not related to any real source are kept in the peer connection back end.
When a libwebrtc track is appearing, it is associated with the track source of the corresponding transceiver based on track type.

Added the ability to create empty real time sources and set their data libwebrtc track when being available.

  • Modules/mediastream/MediaStream.cpp:

(WebCore::MediaStream::addTrackFromPlatform):

  • Modules/mediastream/MediaStream.h:
  • Modules/mediastream/libwebrtc/LibWebRTCMediaEndpoint.cpp:

(WebCore::LibWebRTCMediaEndpoint::mediaStreamFromRTCStream):
(WebCore::LibWebRTCMediaEndpoint::addRemoteStream):
(WebCore::LibWebRTCMediaEndpoint::addRemoteTrack):
(WebCore::LibWebRTCMediaEndpoint::removeRemoteStream):
(WebCore::LibWebRTCMediaEndpoint::OnAddStream):
(WebCore::LibWebRTCMediaEndpoint::OnRemoveStream):
(WebCore::LibWebRTCMediaEndpoint::OnAddTrack):
(WebCore::LibWebRTCMediaEndpoint::stop):
(WebCore::createMediaStreamTrack): Deleted.
(WebCore::LibWebRTCMediaEndpoint::addStream): Deleted.

  • Modules/mediastream/libwebrtc/LibWebRTCMediaEndpoint.h:
  • Modules/mediastream/libwebrtc/LibWebRTCPeerConnectionBackend.cpp:

(WebCore::LibWebRTCPeerConnectionBackend::doStop):
(WebCore::createReceiverForSource):
(WebCore::createEmptySource):
(WebCore::LibWebRTCPeerConnectionBackend::createReceiver):
(WebCore::LibWebRTCPeerConnectionBackend::videoReceiver):
(WebCore::LibWebRTCPeerConnectionBackend::audioReceiver):
(WebCore::LibWebRTCPeerConnectionBackend::removeRemoteStream):
(WebCore::LibWebRTCPeerConnectionBackend::addRemoteStream):

  • Modules/mediastream/libwebrtc/LibWebRTCPeerConnectionBackend.h:
  • platform/mediastream/mac/RealtimeIncomingAudioSource.cpp:

(WebCore::RealtimeIncomingAudioSource::setSourceTrack):

  • platform/mediastream/mac/RealtimeIncomingAudioSource.h:
  • platform/mediastream/mac/RealtimeIncomingVideoSource.cpp:

(WebCore::RealtimeIncomingVideoSource::setSourceTrack):

  • platform/mediastream/mac/RealtimeIncomingVideoSource.h:

LayoutTests:

  • webrtc/video-with-receiver-expected.txt: Added.
  • webrtc/video-with-receiver.html: Copied from LayoutTests/webrtc/video.html.
  • webrtc/video.html:
5:39 PM Changeset in webkit [214131] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Remove TestExpectation for a test that is no longer in the tree.

Unreviewed test gardening.

  • platform/mac/TestExpectations:
5:34 PM Changeset in webkit [214130] by eric.carlson@apple.com
  • 2 edits in trunk/Source/WebCore

[MediaStream] Compensate for video capture orientation
​https://bugs.webkit.org/show_bug.cgi?id=169313
<rdar://problem/30994785>

Unreviewed, fix a crash in r214120 found by the bots.

  • platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaStreamAVFObjC.mm:

(-[WebAVSampleBufferStatusChangeListener invalidate]): Drive-by removal of unnecessary code.
(-[WebAVSampleBufferStatusChangeListener observeValueForKeyPath:ofObject:change:context:]):
Actually retain self before callOnMainThread.
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::~MediaPlayerPrivateMediaStreamAVFObjC):
Invalidate the change listener first.

4:40 PM Changeset in webkit [214129] by dino@apple.com
  • 3 edits
    2 adds in trunk

MediaQueryList instances do not update for prefers-reduced-motion
​https://bugs.webkit.org/show_bug.cgi?id=169835
<rdar://problem/30339378>

Reviewed by Anders Carlsson.

Source/WebCore:

Tell the document it needs to evaluate the MediaQueryLists
if accessibility settings change.

Test: fast/media/mq-prefers-reduced-motion-matchMedia.html

  • page/Page.cpp:

(WebCore::Page::accessibilitySettingsDidChange):

LayoutTests:

  • fast/media/mq-prefers-reduced-motion-matchMedia-expected.txt: Added.
  • fast/media/mq-prefers-reduced-motion-matchMedia.html: Added.
4:33 PM Changeset in webkit [214128] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Mark media/modern-media-controls/airplay-placard/airplay-placard-text-section.html as flaky.
​https://bugs.webkit.org/show_bug.cgi?id=169654

Unreviewed test gardening.

  • platform/mac-wk1/TestExpectations:
4:23 PM Changeset in webkit [214127] by beidson@apple.com
  • 11 edits in trunk/Source

Make HTTPCookieStorage operate in the UIProcess in absence of a WebProcessPool.
<rdar://problem/31102330> and ​https://bugs.webkit.org/show_bug.cgi?id=169797

Reviewed by Alex Christensen.

Source/WebCore:

No new tests (No behavior change).

  • platform/network/NetworkStorageSession.h:
  • platform/network/cocoa/NetworkStorageSessionCocoa.mm:

(WebCore::NetworkStorageSession::flushCookieStore):

  • platform/network/soup/NetworkStorageSessionSoup.cpp:

(WebCore::NetworkStorageSession::flushCookieStore):

Source/WebKit2:

  • UIProcess/API/APIHTTPCookieStore.cpp:

(API::HTTPCookieStore::~HTTPCookieStore):
(API::HTTPCookieStore::cookies):
(API::HTTPCookieStore::setCookie):
(API::HTTPCookieStore::deleteCookie):
(API::HTTPCookieStore::registerObserver):
(API::HTTPCookieStore::unregisterObserver):
(API::HTTPCookieStore::cookieManagerDestroyed):
(API::HTTPCookieStore::registerForNewProcessPoolNotifications):

  • UIProcess/API/APIHTTPCookieStore.h:
  • UIProcess/WebProcessPool.cpp:

(WebKit::WebProcessPool::create):
(WebKit::WebProcessPool::notifyThisWebProcessPoolWasCreated):
(WebKit::m_hiddenPageThrottlingTimer):

  • UIProcess/WebProcessPool.h:
  • UIProcess/WebsiteData/WebsiteDataStore.cpp:

(WebKit::WebsiteDataStore::processPoolForCookieStorageOperations):
(WebKit::WebsiteDataStore::processPoolForCookieStorageNotifications): Deleted.

  • UIProcess/WebsiteData/WebsiteDataStore.h:
2:59 PM Changeset in webkit [214126] by hyatt@apple.com
  • 4 edits
    2 adds in trunk

Disable per-region boxes for multicolumn
​https://bugs.webkit.org/show_bug.cgi?id=169830

Reviewed by Zalan Bujtas.

Source/WebCore:

Test: fast/multicol/float-adjacent-to-overflow-block.html

  • rendering/RenderBlockFlow.cpp:

(WebCore::RenderBlockFlow::determineLogicalLeftPositionForChild):

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::borderBoxRectInRegion):
(WebCore::RenderBox::renderBoxRegionInfo):
Limit all of the per-region box code to RenderNamedFlowThreads.
This code should never be used by multicolumn layout.

LayoutTests:

  • fast/multicol/float-adjacent-to-overflow-block-expected.html: Added.
  • fast/multicol/float-adjacent-to-overflow-block.html: Added.
2:51 PM Changeset in webkit [214125] by Said Abou-Hallawa
  • 7 edits in trunk/Source

Time channel attack on SVG Filters
​https://bugs.webkit.org/show_bug.cgi?id=118689

Reviewed by Simon Fraser.

Source/WebCore:

The time channel attack can happen if the attacker applies FEColorMatrix
or FEConvolveMatrix and provides a matrix which is filled with subnormal
floating point values. Performing floating-point operations on subnormals
is very expensive unless the pixel in the source graphics is black (or
zero). By measuring the time a filter takes to be applied, the attacker
can know whether the pixel he wants to steal from an iframe is black or
white. By repeating the same process on all the pixels in the iframe, the
attacker can reconstruct the whole page of the iframe.

To fix this issue, the values in the matrices of these filters will clamped
to FLT_MIN. We do not want to consume too much time calculating filtered
pixels because of such tiny values. The difference between applying FLT_MIN
and applying a subnormal should not be even noticeable. Normalizing the
floating-point matrices should happen only at the beginning of the filter
platformApplySoftware().

  • platform/graphics/filters/FEColorMatrix.cpp:

(WebCore::FEColorMatrix::platformApplySoftware):

  • platform/graphics/filters/FEConvolveMatrix.cpp:

(WebCore::FEConvolveMatrix::fastSetInteriorPixels):
(WebCore::FEConvolveMatrix::fastSetOuterPixels):
(WebCore::FEConvolveMatrix::platformApplySoftware):

  • platform/graphics/filters/FEConvolveMatrix.h:
  • platform/graphics/filters/FilterEffect.h:

(WebCore::FilterEffect::normalizedFloats):

Source/WTF:

Performing arithmetic operations on subnormal floating-point numbers is
very expensive. Normalizing the floating-point number to the minimum normal
value should accelerate the calculations and there won't be a noticeable
difference in the result since all the subnormal values and the minimum
normal value are all very close to zero.

  • wtf/MathExtras.h:

(normalizedFloat):

2:31 PM Changeset in webkit [214124] by jer.noble@apple.com
  • 2 edits in trunk/Source/WebCore

Explicitly resize the audio buffer in RealtimeOutgoingAudioSource.
​https://bugs.webkit.org/show_bug.cgi?id=169814
<rdar://problem/30975496>

Reviewed by Eric Carlson.

Rather than just reserving sufficient capacity, actually change the buffer's underlying size to match.

  • platform/mediastream/mac/RealtimeOutgoingAudioSource.cpp:

(WebCore::RealtimeOutgoingAudioSource::pullAudioData):

2:28 PM Changeset in webkit [214123] by Chris Dumez
  • 2 edits in trunk/Source/WebKit2

[WK2] Disable background tab responsiveness detection on iOS
​https://bugs.webkit.org/show_bug.cgi?id=169824
<rdar://problem/30913776>

Reviewed by Dan Bernstein.

Disable background tab responsiveness detection on iOS where background tabs
often get suspended. This feature was not tested or meant to be enabled on
iOS, this was an oversight on my part.

  • UIProcess/BackgroundProcessResponsivenessTimer.cpp:

(WebKit::BackgroundProcessResponsivenessTimer::shouldBeActive):

2:18 PM Changeset in webkit [214122] by jiewen_tan@apple.com
  • 12 edits
    1 copy
    5 adds in trunk

[WebCrypto] Make sure all CryptoKey classes are structured clonable
​https://bugs.webkit.org/show_bug.cgi?id=169232
<rdar://problem/31106660>

Reviewed by Brent Fulgham.

Source/WebCore:

This patch does the following few things: 1) It makes CryptoKeyEC and CryptoKeyRaw
structured clonable; 2) It makes CryptoKeyAES and CryptoKeyHMAC to use a move importer
during deserialization; 3) It adds some tests to make sure CryptoKeyAES, CryptoKeyHMAC
and CryptoKeyRSA won't lose their internal data during serialization/deserialization.

Tests: crypto/workers/subtle/ec-postMessage-worker.html

crypto/workers/subtle/raw-postMessage-worker.html

  • bindings/js/SerializedScriptValue.cpp:

(WebCore::CloneSerializer::write):
(WebCore::CloneDeserializer::readHMACKey):
(WebCore::CloneDeserializer::readAESKey):
(WebCore::CloneDeserializer::readECKey):
(WebCore::CloneDeserializer::readRawKey):
(WebCore::CloneDeserializer::readCryptoKey):

  • crypto/keys/CryptoKeyEC.cpp:

(WebCore::CryptoKeyEC::namedCurveString):
(WebCore::CryptoKeyEC::isValidECAlgorithm):

  • crypto/keys/CryptoKeyEC.h:

LayoutTests:

  • crypto/workers/subtle/aes-postMessage-worker-expected.txt:
  • crypto/workers/subtle/aes-postMessage-worker.html:
  • crypto/workers/subtle/ec-postMessage-worker-expected.txt: Added.
  • crypto/workers/subtle/ec-postMessage-worker.html: Added.
  • crypto/workers/subtle/hmac-postMessage-worker-expected.txt:
  • crypto/workers/subtle/hmac-postMessage-worker.html:
  • crypto/workers/subtle/raw-postMessage-worker-expected.txt: Added.
  • crypto/workers/subtle/raw-postMessage-worker.html: Added.
  • crypto/workers/subtle/resources/ec-postMessage-worker.js: Added.
  • crypto/workers/subtle/resources/raw-postMessage-worker.js: Added.
  • crypto/workers/subtle/resources/rsa-postMessage-worker.js:
  • crypto/workers/subtle/rsa-postMessage-worker-expected.txt:
  • crypto/workers/subtle/rsa-postMessage-worker.html:
2:16 PM Changeset in webkit [214121] by msaboff@apple.com
  • 12 edits in trunk

Use USE_INTERNAL_SDK to compute ENABLE_FAST_JIT_PERMISSIONS instead of HAVE_INTERNAL_SDK
​https://bugs.webkit.org/show_bug.cgi?id=169817

Reviewed by Filip Pizlo.

Source/JavaScriptCore:

  • Configurations/FeatureDefines.xcconfig:

Source/WebCore:

  • Configurations/FeatureDefines.xcconfig:

Source/WebCore/PAL:

  • Configurations/FeatureDefines.xcconfig:

Source/WebKit/mac:

  • Configurations/FeatureDefines.xcconfig:

Source/WebKit2:

  • Configurations/FeatureDefines.xcconfig:

Tools:

  • TestWebKitAPI/Configurations/FeatureDefines.xcconfig:
2:14 PM Changeset in webkit [214120] by eric.carlson@apple.com
  • 9 edits in trunk/Source/WebCore

[MediaStream] Compensate for video capture orientation
​https://bugs.webkit.org/show_bug.cgi?id=169313
<rdar://problem/30994785>

Reviewed by Jer Noble.

No new tests, the mock video source doesn't support rotation. Test will be added when this
is fixed in ​https://bugs.webkit.org/show_bug.cgi?id=169822.

Add 'orientation' and 'mirrored' attributes to MediaSample

  • platform/MediaSample.h:

(WebCore::MediaSample::videoOrientation):
(WebCore::MediaSample::videoMirrored):

  • platform/graphics/avfoundation/MediaSampleAVFObjC.h:

A video sample can be rotated and/or mirrored, so the video layer may need to be rotated
and resized for display. We don't want expose this information to the renderer, so allocate
return a generic CALayer as the player's platforLayer, and add the video layer as a sublayer
so we can adjust it to display correctly. Add an enum for playback state as well as display
mode so we correctly display a black frame when video frames are available but playback has
not yet started.

  • platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaStreamAVFObjC.h:
  • platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaStreamAVFObjC.mm:

(-[WebAVSampleBufferStatusChangeListener initWithParent:]):
(-[WebAVSampleBufferStatusChangeListener invalidate]):
(-[WebAVSampleBufferStatusChangeListener beginObservingLayers]):
(-[WebAVSampleBufferStatusChangeListener stopObservingLayers]): Ditto.
(-[WebAVSampleBufferStatusChangeListener observeValueForKeyPath:ofObject:change:context:]):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::videoTransformationMatrix):
(WebCore::runWithoutAnimations):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::enqueueVideoSample):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::layerErrorDidChange):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::ensureLayers):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::destroyLayers):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::cancelLoad):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::platformLayer):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::displayLayer):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::backgroundLayer):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::currentDisplayMode):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::updateDisplayMode):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::play):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::pause):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::currentMediaTime):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::activeStatusChanged):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::updateRenderingMode):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::checkSelectedVideoTrack):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::paintCurrentFrameInContext):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::acceleratedRenderingStateChanged):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::setShouldBufferData):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::backgroundLayerBoundsChanged):
(-[WebAVSampleBufferStatusChangeListener beginObservingLayer:]): Deleted.
(-[WebAVSampleBufferStatusChangeListener stopObservingLayer:]): Deleted.
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::ensureLayer): Deleted.
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::destroyLayer): Deleted.
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::paused): Deleted.
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::shouldBePlaying): Deleted.

  • platform/mediastream/mac/AVVideoCaptureSource.h:
  • platform/mediastream/mac/AVVideoCaptureSource.mm:

(WebCore::AVVideoCaptureSource::processNewFrame): Add connection parameter so we can get the
video orientation.
(WebCore::AVVideoCaptureSource::captureOutputDidOutputSampleBufferFromConnection):

Pass sample orientation to libwebrtc.

  • platform/mediastream/mac/RealtimeOutgoingVideoSource.cpp:

(WebCore::RealtimeOutgoingVideoSource::sendFrame):
(WebCore::RealtimeOutgoingVideoSource::videoSampleAvailable):

  • platform/mediastream/mac/RealtimeOutgoingVideoSource.h:
2:12 PM Changeset in webkit [214119] by Alan Bujtas
  • 3 edits
    2 adds in trunk

Fix the flow thread state on the descendants of out of flow positioned replaced elements.
​https://bugs.webkit.org/show_bug.cgi?id=169821
<rdar://problem/30964017>

Reviewed by Simon Fraser.

Source/WebCore:

Descendants of a replaced out of flow elmement should inherit the flowthread state
from the replaced element and not from the replaced element's parent.

Test: fast/multicol/fix-inherit-when-container-is-replaced.html

  • rendering/RenderObject.cpp:

(WebCore::RenderObject::computedFlowThreadState):

LayoutTests:

  • fast/multicol/fix-inherit-when-container-is-replaced-expected.txt: Added.
  • fast/multicol/fix-inherit-when-container-is-replaced.html: Added.
1:59 PM Changeset in webkit [214118] by Jon Davis
  • 2 edits in trunk/Source/WebCore

Update feature status for outdated entries: CSS Shapes, Force Click Events
​https://bugs.webkit.org/show_bug.cgi?id=169829

Reviewed by Matt Baker.

  • features.json:
1:45 PM Changeset in webkit [214117] by achristensen@apple.com
  • 2 edits in trunk/Source/WebCore

Make SocketStreamError a private header.
​https://bugs.webkit.org/show_bug.cgi?id=169831

  • WebCore.xcodeproj/project.pbxproj:

I'll need this from WebKit2.

1:45 PM Changeset in webkit [214116] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

Update webrtc tests expectations
​https://bugs.webkit.org/show_bug.cgi?id=169828

Unreviewed.

Patch by Youenn Fablet <​youenn@apple.com> on 2017-03-17

1:42 PM Changeset in webkit [214115] by achristensen@apple.com
  • 2 edits in trunk/Source/WebCore

Add encoder for SocketStreamError
​https://bugs.webkit.org/show_bug.cgi?id=169831

Reviewed by Andy Estes.

This isn't used yet, but we'll soon need to send these across IPC.

  • platform/network/SocketStreamError.h:

(WebCore::SocketStreamError::encode):
(WebCore::SocketStreamError::decode):

1:30 PM Changeset in webkit [214114] by achristensen@apple.com
  • 11 edits
    1 add in trunk/Source/WebCore

Make SocketStreamHandle virtual functions asynchronous
​https://bugs.webkit.org/show_bug.cgi?id=169818

Reviewed by Andy Estes.

No change in behavior. Instead of returning immediately, call a completion handler.
This is in preparation for making them truly asynchronous.

I still need to be able to synchronously get the number of buffered bytes, but the
buffer itself will soon be in the NetworkProcess with a new subclass of SocketStreamHandle
that messages across IPC. The number of buffered bytes will still be stored in the WebProcess,
and when it updates, the message from SocketStreamHandleClient::didUpdateBufferedAmount will update it.

  • CMakeLists.txt:
  • WebCore.xcodeproj/project.pbxproj:
  • platform/network/SocketStreamHandle.cpp:

(WebCore::SocketStreamHandle::send):
(WebCore::SocketStreamHandle::close):
(WebCore::SocketStreamHandle::sendPendingData): Deleted.

  • platform/network/SocketStreamHandle.h:

(WebCore::SocketStreamHandle::bufferedAmount): Deleted.

  • platform/network/SocketStreamHandleImpl.cpp: Added.

(WebCore::SocketStreamHandleImpl::platformSend):
(WebCore::SocketStreamHandleImpl::sendPendingData):
(WebCore::SocketStreamHandleImpl::bufferedAmount):
m_buffer was moved from SocketStreamHandle to SocketStreamHandleImpl, so some of the logic must move with it.
I moved as much logic that was shared in the superclass to a new shared location for code shared among the subclass implementations.

  • platform/network/cf/SocketStreamHandleImpl.h:
  • platform/network/cf/SocketStreamHandleImplCFNet.cpp:

(WebCore::SocketStreamHandleImpl::platformSendInternal):
(WebCore::SocketStreamHandleImpl::platformSend): Deleted.

  • platform/network/curl/SocketStreamHandleImpl.h:
  • platform/network/curl/SocketStreamHandleImplCurl.cpp:

(WebCore::SocketStreamHandleImpl::platformSendInternal):
(WebCore::SocketStreamHandleImpl::platformSend): Deleted.

  • platform/network/soup/SocketStreamHandleImpl.h:
  • platform/network/soup/SocketStreamHandleImplSoup.cpp:

(WebCore::SocketStreamHandleImpl::platformSendInternal):
(WebCore::SocketStreamHandleImpl::platformSend): Deleted.

12:52 PM Changeset in webkit [214113] by Antti Koivisto
  • 33 edits
    3 adds in trunk

Add a reload policy where only expired subresources are revalidated
​https://bugs.webkit.org/show_bug.cgi?id=169756

Reviewed by Andreas Kling.

Source/WebCore:

Test: http/tests/cache/reload-expired-only.html

The default reload behavior revalidates all resources on the page. This patch adds
a new policy that revalidates expired (and uncacheable) resources only. Using this
policy can speed up reloads significantly and also reduce network traffic and server
load.

  • history/PageCache.cpp:

(WebCore::canCachePage):

  • inspector/InspectorPageAgent.cpp:

(WebCore::InspectorPageAgent::reload):

  • loader/FrameLoader.cpp:

(WebCore::isBackForwardLoadType):
(WebCore::isReload):

Add a helper function.

(WebCore::FrameLoader::loadURL):
(WebCore::logNavigation):
(WebCore::FrameLoader::reload):

Replace bool arguments with OptionSet<ReloadOption>. Most of the patch is about adapting to this.

(WebCore::FrameLoader::transitionToCommitted):
(WebCore::FrameLoader::subresourceCachePolicy):

Return CachePolicyVerify for 'ReloadExpiredOnly' frame load type. This is the substantive
change that causes the behavior difference.

(WebCore::FrameLoader::checkLoadCompleteForThisFrame):
(WebCore::FrameLoader::defaultRequestCachingPolicy):

Flip the isMainResource condition for better readability.

(WebCore::FrameLoader::shouldPerformFragmentNavigation):
(WebCore::FrameLoader::loadDifferentDocumentItem):

  • loader/FrameLoader.h:

(WebCore::FrameLoader::reload):

  • loader/FrameLoaderTypes.h:
  • loader/HistoryController.cpp:

(WebCore::HistoryController::restoreDocumentState):

  • page/DiagnosticLoggingKeys.cpp:

(WebCore::DiagnosticLoggingKeys::reloadRevalidatingExpiredKey):

  • page/DiagnosticLoggingKeys.h:
  • replay/UserInputBridge.cpp:

(WebCore::UserInputBridge::reloadFrame):

  • replay/UserInputBridge.h:
  • testing/Internals.cpp:

(WebCore::Internals::forceReload):
(WebCore::Internals::reloadExpiredOnly):

Testing support.

  • testing/Internals.h:
  • testing/Internals.idl:

Source/WebKit/mac:

  • WebView/WebFrame.mm:

(toWebFrameLoadType):
(-[WebFrame reload]):
(-[WebFrame reloadFromOrigin]):

Source/WebKit2:

  • UIProcess/API/C/WKPage.cpp:

(WKPageReload):
(WKPageReloadWithoutContentBlockers):
(WKPageReloadFromOrigin):
(WKPageReloadExpiredOnly):

  • UIProcess/API/C/WKPage.h:
  • UIProcess/API/Cocoa/WKBrowsingContextController.mm:

(-[WKBrowsingContextController reload]):
(-[WKBrowsingContextController reloadFromOrigin]):

  • UIProcess/API/Cocoa/WKWebView.mm:

(-[WKWebView reload]):
(-[WKWebView reloadFromOrigin]):
(-[WKWebView _reloadWithoutContentBlockers]):
(-[WKWebView _reloadExpiredOnly]):

  • UIProcess/API/Cocoa/WKWebViewPrivate.h:
  • UIProcess/Automation/WebAutomationSession.cpp:

(WebKit::WebAutomationSession::reloadBrowsingContext):

  • UIProcess/WebFrameProxy.cpp:

(WebKit::WebFrameProxy::didHandleContentFilterUnblockNavigation):

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::reload):

  • UIProcess/WebPageProxy.h:
  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::reload):
(WebKit::shouldReuseCommittedSandboxExtension):

  • WebProcess/WebPage/WebPage.h:
  • WebProcess/WebPage/WebPage.messages.in:

LayoutTests:

  • http/tests/cache/reload-expired-only-expected.txt: Added.
  • http/tests/cache/reload-expired-only.html: Added.
  • http/tests/cache/resources/document-with-cached-unique-script.html: Added.
  • http/tests/cache/resources/random-cached.cgi:
12:48 PM Changeset in webkit [214112] by n_wang@apple.com
  • 4 edits
    4 adds in trunk

AX: VoiceOver no longer works corectly with editable text in the web
​https://bugs.webkit.org/show_bug.cgi?id=169801

Reviewed by Chris Fleizach.

Source/WebCore:

The issue is that since we are entering text controls when getting the text marker,
the end text marker of the input tag might associate to the placeholder div instead of
the input tag itself.
Fixed the issue by checking the shadow elements using shadowHost() instead of isShadowRoot().
Also, only enter text controls when navigating by characters/indexes, so that both Mac and
iOS will get the correct text marker ranges for shadow host elements.

Tests: accessibility/ios-simulator/element-text-range-for-text-control.html

accessibility/mac/text-markers-for-input-with-placeholder.html

  • accessibility/AXObjectCache.cpp:

(WebCore::AXObjectCache::traverseToOffsetInRange):
(WebCore::AXObjectCache::startOrEndCharacterOffsetForRange):
(WebCore::AXObjectCache::characterOffsetForIndex):

  • accessibility/AXObjectCache.h:

LayoutTests:

  • accessibility/ios-simulator/element-text-range-for-text-control-expected.txt: Added.
  • accessibility/ios-simulator/element-text-range-for-text-control.html: Added.
  • accessibility/mac/text-markers-for-input-with-placeholder-expected.txt: Added.
  • accessibility/mac/text-markers-for-input-with-placeholder.html: Added.
12:30 PM Changeset in webkit [214111] by Wenson Hsieh
  • 11 edits in trunk/Source

[WK1] Support animated transitions when performing a data interaction operation
​https://bugs.webkit.org/show_bug.cgi?id=169805
<rdar://problem/31045767>

Reviewed by Beth Dakin and Megan Gardner.

Source/WebCore:

When performing a snapshot excluding the current selection, use ScrollableArea::visibleContentRect to compute
the visible content rect rather than FrameView::visualViewportRect. In WebKit1, the latter will return the
entire content view's bounds.

  • page/TextIndicator.cpp:

(WebCore::takeSnapshots):

Source/WebKit/mac:

Adds WebKit1 support for animated transitions when performing a data interaction operation. See per-method changes
for more details.

  • DOM/DOMElement.h:
  • DOM/DOMElement.mm:

(-[DOMElement boundsInRootViewSpace]):

Introduce -[DOMElement boundsInRootViewSpace], which returns the bounds of the element in root view coordinates.

  • WebCoreSupport/WebDragClient.h:

(): Deleted.

  • WebCoreSupport/WebDragClient.mm:

(WebDragClient::didConcludeEditDrag):

Add boilerplate plumbing.

  • WebView/WebView.mm:

(-[WebUITextIndicatorData initWithImage:textIndicatorData:scale:]):

Add contentImageWithoutSelection and contentImageWithoutSelectionRectInRootViewCoordinates to WebUITextIndicatorData.

(floatRectsForCGRectArray):
(-[WebView _createImageWithPlatterForImage:boundingRect:contentScaleFactor:clippingRects:]):

Introduces a SPI helper method in WebView that clients can use to generate an image of a text platter given an
image of some text, as well as clipping rects to apply to the image.

(-[WebView _dataOperationTextIndicator]):

Add SPI to fetch the text indicator used to drive animations for the current data operation.

(-[WebView _didConcludeEditDataInteraction]):

  • WebView/WebViewData.h:
  • WebView/WebViewInternal.h:
  • WebView/WebViewPrivate.h:
11:54 AM Changeset in webkit [214110] by hyatt@apple.com
  • 12 edits
    8 adds in trunk

Initial letter does not paginate properly.
​https://bugs.webkit.org/show_bug.cgi?id=169795
<rdar://problem/23715770>

Reviewed by Zalan Bujtas.

Source/WebCore:

Tests: fast/css-generated-content/initial-letter-pagination-raised.html

fast/css-generated-content/initial-letter-pagination-raised-rl.html
fast/css-generated-content/initial-letter-pagination-sunken.html
fast/css-generated-content/initial-letter-pagination-sunken-rl.html

  • rendering/FloatingObjects.cpp:

(WebCore::FloatingObject::FloatingObject):
(WebCore::FloatingObject::copyToNewContainer):
(WebCore::FloatingObject::cloneForNewParent):
(WebCore::FloatingObject::translationOffsetToAncestor):

  • rendering/FloatingObjects.h:

(WebCore::FloatingObject::setMarginOffset):
(WebCore::FloatingObject::locationOffsetOfBorderBox):
(WebCore::FloatingObject::marginOffset):
Add the margin offset to the FloatingObject so that we can control the
placement of the border box relative to the margin box without relying
on only the CSS-specified margins. This allows for sunken initial letter
FloatingObjects to be created and propagated out to ancestor blocks for painting
without altering the CSS margins of the sunken letter.

  • rendering/RenderBlockFlow.cpp:

(WebCore::RenderBlockFlow::adjustForUnsplittableChild):
Update the logical height for initial letter containers after the initial
letter shifts to the next page, since the lines that follow need to lay out
relative to the initial letter, i.e., they shift down if it shifts down.

(WebCore::RenderBlockFlow::addOverflowFromFloats):
(WebCore::RenderBlockFlow::paintFloats):
(WebCore::RenderBlockFlow::clipOutFloatingObjects):
Patched to use the new helpers on FloatingObject rather than
xPositionForFloatIncludingMargins and yPositionForFloatIncludingMargins.

(WebCore::RenderBlockFlow::computeLogicalLocationForFloat):
Updated to set the placement and margins of the floating object before
returning. The initial letter adjustments have been factored out into their
own function.

(WebCore::RenderBlockFlow::adjustInitialLetterPosition):
A helper function called by computeLogicalLocationForFloat that handles
the placement and margin adjustments for initial letters. Note that sunken
letters no longer "hack" the top margin of the child but instead rely on
the fact that the FloatingObject has its own independent margin offset now
that can be set to a value that is distinct from the CSS supplied margin.

(WebCore::RenderBlockFlow::positionNewFloats):
Revised to no longer do the placement, but to let computeLogicalLocationForFloat
do it. The height is still set here however, since we don't know that until after
the child has received a layout.

(WebCore::RenderBlockFlow::addOverhangingFloats):
(WebCore::RenderBlockFlow::flipFloatForWritingModeForChild):
(WebCore::RenderBlockFlow::hitTestFloats):
(WebCore::RenderBlockFlow::adjustForBorderFit):
Patched to use the new FloatingObject helpers rather than
xPositionForFloatIncludingMargins and yPositionForFloatIncludingMargins.

  • rendering/RenderBlockFlow.h:

(WebCore::RenderBlockFlow::setLogicalMarginsForFloat):
New helper function for setting the margin offsets on a FloatingObject.

(WebCore::RenderBlockFlow::xPositionForFloatIncludingMargin): Deleted.
(WebCore::RenderBlockFlow::yPositionForFloatIncludingMargin): Deleted.
Deleted in favor of new FloatingObject helpers.

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::isUnsplittableForPagination):
Make initial letter unsplittable as far as pagination is concerned.

  • rendering/RootInlineBox.cpp:

(WebCore::RootInlineBox::ascentAndDescentForBox):
(WebCore::RootInlineBox::includeFontForBox):
(WebCore::RootInlineBox::includeGlyphsForBox):
(WebCore::RootInlineBox::fitsToGlyphs):
Turn off the restriction that you can't fit to vertical glyphs. The numbers returned
are fine now. This allows vertical initial letters to work.

LayoutTests:

  • fast/css-generated-content/initial-letter-pagination-raised-expected.html: Added.
  • fast/css-generated-content/initial-letter-pagination-raised.html: Added.
  • fast/css-generated-content/initial-letter-pagination-sunken-expected.html: Added.
  • fast/css-generated-content/initial-letter-pagination-sunken.html: Added.
  • platform/ios-simulator/TestExpectations:
10:58 AM Changeset in webkit [214109] by fpizlo@apple.com
  • 12 edits
    2 adds in trunk/Source

Air should be powerful enough to support Tmp-splitting
​https://bugs.webkit.org/show_bug.cgi?id=169515

Reviewed by Saam Barati.

Source/JavaScriptCore:

In the process of implementing the Tmp-splitting optimization, I made some small
clean-ups. They don't affect anything - it's basically moving code around and adding
utility functions.

  • CMakeLists.txt:
  • JavaScriptCore.xcodeproj/project.pbxproj:
  • assembler/LinkBuffer.cpp:

(JSC::LinkBuffer::allocate): testb3 was sometimes failing its checkDoesNotUseInstruction check because of uninitialized memory. This initializes the internal fragmentation slop of every JIT allocation.

  • b3/air/AirAllocateRegistersByGraphColoring.cpp:
  • b3/air/AirAllocateRegistersByGraphColoring.h:

(JSC::B3::Air::useIRC): It's useful to be able to query which register allocator we're using.

  • b3/air/AirArg.cpp:

(WTF::printInternal):

  • b3/air/AirArg.h:

(JSC::B3::Air::Arg::temperature): The temperature of a role is a useful concept to have factored out.

  • b3/air/AirBreakCriticalEdges.cpp: Added.

(JSC::B3::Air::breakCriticalEdges): I was surprised that we didn't have this already. It's a pretty fundamental CFG utility.

  • b3/air/AirBreakCriticalEdges.h: Added.
  • b3/air/AirGenerate.cpp:
  • b3/air/AirInsertionSet.h: You can't use & if you want copy-constructibility, which seems to be a prerequisite to IndexMap<BasicBlock, InsertionSet>.

(JSC::B3::Air::InsertionSet::InsertionSet):
(JSC::B3::Air::InsertionSet::code):

  • b3/air/AirLiveness.h: Teach Liveness to track only warm liveness.

(JSC::B3::Air::TmpLivenessAdapter::acceptsRole):
(JSC::B3::Air::StackSlotLivenessAdapter::acceptsRole):
(JSC::B3::Air::RegLivenessAdapter::acceptsRole):
(JSC::B3::Air::AbstractLiveness::AbstractLiveness):
(JSC::B3::Air::AbstractLiveness::LocalCalc::execute):

Source/WTF:

Teach IndexMap how to handle types that don't have a default constructor. This makes it
easy to create a IndexMap<BasicBlock, InsertionSet>, which is a super powerful
construct.

  • wtf/IndexMap.h:

(WTF::IndexMap::IndexMap):
(WTF::IndexMap::resize):
(WTF::IndexMap::clear):

10:16 AM Changeset in webkit [214108] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

RealtimeIncomingAudioSource is not stopping properly
​https://bugs.webkit.org/show_bug.cgi?id=169807

Patch by Youenn Fablet <​youenn@apple.com> on 2017-03-17
Reviewed by Eric Carlson.

Source/WebCore:

Test: webrtc/release-after-getting-track.html

  • platform/mediastream/mac/RealtimeIncomingAudioSource.cpp:

(WebCore::RealtimeIncomingAudioSource::~RealtimeIncomingAudioSource):
(WebCore::RealtimeIncomingAudioSource::stopProducingData):

LayoutTests:

  • webrtc/release-after-getting-track-expected.txt: Added.
  • webrtc/release-after-getting-track.html: Added.
10:09 AM Changeset in webkit [214107] by bshafiei@apple.com
  • 5 edits in branches/safari-603-branch/Source

Versioning.

8:57 AM Changeset in webkit [214106] by achristensen@apple.com
  • 8 edits in trunk/Source

Use completion handlers instead of return values for sending websocket data
​https://bugs.webkit.org/show_bug.cgi?id=169802

Reviewed by Carlos Garcia Campos.

Source/WebCore:

No change in behavior, but this is a baby step towards making things more asynchronous.

  • Modules/websockets/WebSocketChannel.cpp:

(WebCore::WebSocketChannel::didOpenSocketStream):
(WebCore::WebSocketChannel::processOutgoingFrameQueue):
(WebCore::WebSocketChannel::sendFrame):

  • Modules/websockets/WebSocketChannel.h:
  • platform/network/SocketStreamHandle.cpp:

(WebCore::SocketStreamHandle::send):

  • platform/network/SocketStreamHandle.h:
  • platform/network/cf/SocketStreamHandleImpl.h:

Source/WebKit2:

  • UIProcess/InspectorServer/WebSocketServerConnection.cpp:

(WebKit::WebSocketServerConnection::sendWebSocketMessage):
(WebKit::WebSocketServerConnection::sendHTTPResponseHeader):
(WebKit::WebSocketServerConnection::sendRawData):

7:11 AM Changeset in webkit [214105] by zandobersek@gmail.com
  • 7 edits in trunk/Source/WebCore

WebGPU: build fixes for non-Cocoa platforms
​https://bugs.webkit.org/show_bug.cgi?id=169781

Reviewed by Alex Christensen.

Enable building WebGPU code for other ports and fix various build
failures. This still leaves the GPUDevice::platformLayer() method
missing for other ports, as well as the general implementation of
the WebGPU platform abstractions under platform/graphics/gpu/.

  • CMakeLists.txt: Move the WebGPU build targets here.
  • PlatformMac.cmake:
  • html/canvas/WebGPULibrary.h: Explicitly include the WTFString.h

header to suppress missing WTF::String definition errors.

  • platform/graphics/gpu/GPUBuffer.cpp:

(WebCore::GPUBuffer::contents): Avoid ambiguous calls to
ArrayBuffer::create() by passing nullptr as the first argument.

  • platform/graphics/gpu/GPUDevice.h: Include the Forward.h header

to forward-declare the WTF::String class. Also remove the unused
PlatformGPUDevice typedef for non-Cocoa platforms.
(WebCore::GPUDevice::layer): Move this and the related m_layer
member variable under the PLATFORM(COCOA) guard.

  • platform/graphics/gpu/GPUFunction.cpp:

(WebCore::GPUFunction::GPUFunction): Remove the GPULibrary
parameter checks in this no-op constructor that's used on
non-Cocoa platforms.

6:13 AM Changeset in webkit [214104] by tpopela@redhat.com
  • 5 edits in trunk/Source/WebKit2

Fix uninitialized public members in WebProcess
​https://bugs.webkit.org/show_bug.cgi?id=169604

Reviewed by Michael Catanzaro.

Found by Coverity scan.

  • WebProcess/Databases/IndexedDB/WebIDBConnectionToServer.h:
  • WebProcess/InjectedBundle/InjectedBundleNavigationAction.h:
  • WebProcess/Plugins/Netscape/NetscapePlugin.h:
  • WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp:

(WebKit::NetscapePluginX11::NetscapePluginX11):

5:22 AM Changeset in webkit [214103] by magomez@igalia.com
  • 7 edits
    2 adds in trunk

Follow-up (r213833): write a layout test for 169199
​https://bugs.webkit.org/show_bug.cgi?id=169576
<rdar://problem/31024766>

Reviewed by Said Abou-Hallawa.

Source/WebCore:

Add a function to Internals to force the deletion of the ImageDecoder just after
requesting the async decoding of a frame, and implement that behaviour in BitmapImage.

Test: fast/images/clear-animation-decoder.html

  • platform/graphics/BitmapImage.cpp:

(WebCore::BitmapImage::internalStartAnimation):

  • platform/graphics/BitmapImage.h:
  • testing/Internals.cpp:

(WebCore::Internals::setClearDecoderAfterAsyncFrameRequestForTesting):

  • testing/Internals.h:
  • testing/Internals.idl:

LayoutTests:

Add a test that ensures there's no crash when deleting the ImageDecoder after requesting
an async decoding operation.

  • fast/images/clear-animation-decoder-expected.txt: Added.
  • fast/images/clear-animation-decoder.html: Added.
2:46 AM Changeset in webkit [214102] by Carlos Garcia Campos
  • 2 edits in trunk/LayoutTests

Unreviewed GTK+ gardening. Mark mediastream and webrtc new failures.

  • platform/gtk/TestExpectations:
2:45 AM Changeset in webkit [214101] by tpopela@redhat.com
  • 3 edits in trunk/Source/WebKit2

Fix uninitialized public members in NetworkProcess
​https://bugs.webkit.org/show_bug.cgi?id=169598

Reviewed by Alex Christensen.

Found by Coverity scan.

  • NetworkProcess/NetworkProcessCreationParameters.h:
  • NetworkProcess/cache/NetworkCacheStorage.cpp:
1:22 AM Changeset in webkit [214100] by Carlos Garcia Campos
  • 4 edits in trunk/Source/WebCore

[Cairo] Handle the blend mode in GraphicsContext::drawPattern
​https://bugs.webkit.org/show_bug.cgi?id=169746

Reviewed by Žan Doberšek.

We are not taking into account the blend mode when passing the cairo operator to drawPatternToCairoContext().
This is based on patch by Žan Doberšek, just adding the toCairoOperator changes to make it easier to handle
it. Instead of checking everywhere if blend mode is Normal to decide whether to use toCairoOperator with
CompositeOperator or BlendMode, there's no a single toCairoOperator that receives both parameters, but BlendMode
is optional and defaults to Normal.

  • platform/graphics/cairo/CairoUtilities.cpp:

(WebCore::toCairoCompositeOperator):
(WebCore::toCairoOperator):

  • platform/graphics/cairo/CairoUtilities.h:
  • platform/graphics/cairo/GraphicsContextCairo.cpp:

(WebCore::GraphicsContext::setPlatformCompositeOperation):
(WebCore::GraphicsContext::drawPattern):

Mar 16, 2017:

10:29 PM Changeset in webkit [214099] by webkit@devinrousso.com
  • 2 edits in trunk/Source/WebInspectorUI

Web Inspector: memory category timelines would look better with ruler tick-aligned guides
​https://bugs.webkit.org/show_bug.cgi?id=158100

Reviewed by Joseph Pecoraro.

  • UserInterface/Views/MemoryTimelineView.css:

(.timeline-view.memory > .content > .details):
(.timeline-view.memory > .content > .details > .timeline-ruler):
(body[dir=ltr] .timeline-view.memory > .content > .details > .timeline-ruler):
(body[dir=rtl] .timeline-view.memory > .content > .details > .timeline-ruler):
Make the TimelineRuler absolutely positioned so it can use the full height of the view,
providing enough space for the markers to be visible.

10:14 PM Changeset in webkit [214098] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Mark imported/w3c/web-platform-tests/html/browsers/history/the-location-interface/location_hash.html as flaky.
​https://bugs.webkit.org/show_bug.cgi?id=169572

Unreviewed test gardening.

  • platform/mac/TestExpectations:
9:49 PM Changeset in webkit [214097] by Ryan Haddad
  • 4 edits in trunk/LayoutTests

Rebaseline js/dom/global-constructors-attributes.html.

Unreviewed test gardening.

  • platform/mac-elcapitan/js/dom/global-constructors-attributes-expected.txt:
  • platform/mac-wk1/js/dom/global-constructors-attributes-expected.txt:
  • platform/mac/js/dom/global-constructors-attributes-expected.txt:
9:33 PM Changeset in webkit [214096] by Ryan Haddad
  • 3 edits in branches/safari-603.1.30.0-branch/LayoutTests

Merge r213174.

9:33 PM Changeset in webkit [214095] by Ryan Haddad
  • 2 edits in branches/safari-603.1.30.0-branch/LayoutTests

Merge r212132.

9:33 PM Changeset in webkit [214094] by Ryan Haddad
  • 1 edit
    3 adds in branches/safari-603.1.30.0-branch/LayoutTests

Add baseline for fast/canvas/webgl/context-creation-attributes.html.
Unreviewed test gardening.

  • platform/mac-wk1/fast/canvas/webgl/context-creation-attributes-expected.txt: Added.
7:14 PM Changeset in webkit [214093] by webkit@devinrousso.com
  • 4 edits in trunk/Source/WebInspectorUI

Web Inspector: Console Evaluation links in the Console should not show normally
​https://bugs.webkit.org/show_bug.cgi?id=157483

Reviewed by Matt Baker.

  • UserInterface/Models/CallFrame.js:

(WebInspector.CallFrame):
(WebInspector.CallFrame.prototype.get isConsoleEvaluation):
(WebInspector.CallFrame.fromDebuggerPayload):
(WebInspector.CallFrame.fromPayload):
Move console evaluation check to the constructor for consistency.

  • UserInterface/Views/ConsoleMessageView.js:

(WebInspector.ConsoleMessageView.prototype._appendLocationLink):

  • UserInterface/Views/StackTraceView.js:

(WebInspector.StackTraceView):
Only add CallFrameView elements if the corresponding CallFame is not a console evaluation.

7:07 PM Changeset in webkit [214092] by achristensen@apple.com
  • 2 edits in trunk/Source/WebKit2

Fix GTK build.

  • UIProcess/InspectorServer/WebSocketServerConnection.h:
6:56 PM Changeset in webkit [214091] by achristensen@apple.com
  • 3 edits in trunk/Source/WebKit2

Fix GTK build.

  • UIProcess/InspectorServer/WebSocketServerConnection.cpp:

(WebKit::WebSocketServerConnection::didReceiveSocketStreamData):

  • UIProcess/InspectorServer/WebSocketServerConnection.h:
6:43 PM Changeset in webkit [214090] by achristensen@apple.com
  • 2 edits in trunk/Source/WebCore

Fix GTK build.

  • platform/network/soup/SocketStreamHandleImplSoup.cpp:

(WebCore::SocketStreamHandleImpl::readBytes):
Changing RefPtr to Ref broke something.
For the record, EWS was orange so I couldn't see these build failures.

6:37 PM Changeset in webkit [214089] by msaboff@apple.com
  • 2 edits in trunk/Source/WebKit2

Turn off JavaScript Configuration File processing for Debug builds
​https://bugs.webkit.org/show_bug.cgi?id=169799

Reviewed by Dean Jackson.

Change the default value for m_javaScriptConfigurationFileEnabled for Debug
builds to be the same as a Release build, that is false (off).

  • UIProcess/WebProcessPool.h:
6:23 PM Changeset in webkit [214088] by achristensen@apple.com
  • 2 edits in trunk/Source/WebCore

Fix GTK build.

  • platform/network/soup/SocketStreamHandleImplSoup.cpp:

(WebCore::SocketStreamHandleImpl::readBytes):
didFailToReceiveSocketStreamData needs a parameter.

5:51 PM Changeset in webkit [214087] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

activating webrtc tests on Release bots
​https://bugs.webkit.org/show_bug.cgi?id=169775

Unreviewed.

Patch by Youenn Fablet <​youenn@apple.com> on 2017-03-16

5:51 PM Changeset in webkit [214086] by dino@apple.com
  • 7 edits
    2 adds in trunk

WebGL: Improve index validation when using uint index values
​https://bugs.webkit.org/show_bug.cgi?id=169798

Reviewed by Simon Fraser.

Source/WebCore:

Make sure that we test index validation with the correct type.
Also stop using -1 in WebGLBuffer to indicate non-existant values.

Test: fast/canvas/webgl/draw-elements-out-of-bounds-uint-index.html

  • html/canvas/WebGL2RenderingContext.cpp:

(WebCore::WebGL2RenderingContext::validateIndexArrayConservative): Use optional<> and
unsigned values.

  • html/canvas/WebGLBuffer.cpp: Use unsigned for maxIndex (they can't be negative)

and optional<> to indicate unknown value.
(WebCore::WebGLBuffer::getCachedMaxIndex):
(WebCore::WebGLBuffer::setCachedMaxIndex):

  • html/canvas/WebGLBuffer.h:
  • html/canvas/WebGLRenderingContext.cpp:

(WebCore::WebGLRenderingContext::validateIndexArrayConservative): Use optional<> and
unsigned values.

  • html/canvas/WebGLRenderingContextBase.cpp:

(WebCore::WebGLRenderingContextBase::validateVertexAttributes): No need to check if
an unsigned value is less than zero.

LayoutTests:

  • fast/canvas/webgl/draw-elements-out-of-bounds-uint-index-expected.txt: Added.
  • fast/canvas/webgl/draw-elements-out-of-bounds-uint-index.html: Added.
5:51 PM Changeset in webkit [214085] by mark.lam@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Fix exception scope verification failures in GenericArgumentsInlines.h.
​https://bugs.webkit.org/show_bug.cgi?id=165012

Reviewed by Saam Barati.

  • runtime/GenericArgumentsInlines.h:

(JSC::GenericArguments<Type>::defineOwnProperty):

5:47 PM Changeset in webkit [214084] by Simon Fraser
  • 23 edits in trunk

Improve the system tracing points
​https://bugs.webkit.org/show_bug.cgi?id=169790

Reviewed by Zalan Bujtas.

Use a more cohesive set of system trace points that give a good overview of what
WebKit is doing. Added points for resource loading, render tree building, sync messages
to the web process, async image decode, WASM and fetching cookies.

Source/JavaScriptCore:

  • wasm/WasmPlan.cpp:

(JSC::Wasm::Plan::run):

  • wasm/js/WebAssemblyFunction.cpp:

(JSC::callWebAssemblyFunction):

Source/WebCore:

  • loader/CookieJar.cpp:

(WebCore::cookies):

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::provisionalLoadStarted):
(WebCore::FrameLoader::checkLoadCompleteForThisFrame):

  • loader/SubresourceLoader.cpp:

(WebCore::SubresourceLoader::willSendRequestInternal):
(WebCore::logResourceLoaded):
(WebCore::SubresourceLoader::didFinishLoading):
(WebCore::SubresourceLoader::didFail):
(WebCore::SubresourceLoader::didCancel):

  • page/FrameView.cpp:

(WebCore::FrameView::paintContents):

  • platform/graphics/ImageFrameCache.cpp:

(WebCore::ImageFrameCache::startAsyncDecodingQueue):

  • platform/graphics/ios/DisplayRefreshMonitorIOS.mm:

(WebCore::DisplayRefreshMonitorIOS::requestRefreshCallback):
(WebCore::DisplayRefreshMonitorIOS::displayLinkFired):

  • style/RenderTreeUpdater.cpp:

(WebCore::RenderTreeUpdater::commit):

Source/WebKit/mac:

  • WebView/WebHTMLView.mm:

(-[WebHTMLView drawRect:]):

Source/WebKit2:

  • UIProcess/ChildProcessProxy.h:

(WebKit::ChildProcessProxy::sendSync):

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::handleTouchEventSynchronously):

  • UIProcess/mac/RemoteLayerTreeDrawingAreaProxy.mm:

(WebKit::RemoteLayerTreeDrawingAreaProxy::commitLayerTree):
(WebKit::RemoteLayerTreeDrawingAreaProxy::didRefreshDisplay):

  • WebProcess/WebPage/mac/RemoteLayerTreeContext.mm:

(WebKit::RemoteLayerTreeContext::buildTransaction):

  • WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm:

(WebKit::RemoteLayerTreeDrawingArea::didUpdate):
(WebKit::RemoteLayerTreeDrawingArea::BackingStoreFlusher::flush):

Source/WTF:

  • wtf/SystemTracing.h:

Tools:

  • Tracing/SystemTracePoints.plist:
5:23 PM Changeset in webkit [214083] by Matt Baker
  • 2 edits in trunk/Source/WebInspectorUI

Web Inspector: REGRESSION (r213691): Saving DOM breakpoints broken after refactoring
​https://bugs.webkit.org/show_bug.cgi?id=169779

Reviewed by Joseph Pecoraro.

Use correct method: _saveBreakpoints -> _saveDOMBreakpoints.

  • UserInterface/Controllers/DOMDebuggerManager.js:

(WebInspector.DOMDebuggerManager.prototype.addDOMBreakpoint):
Remove check before saving to be consistent with similar code in
DebuggerManager and DOMDebuggerManager.

5:20 PM Changeset in webkit [214082] by Simon Fraser
  • 29 edits in trunk/Source/WebCore

RenderView::documentBeingDestroyed() needs a new name.
​https://bugs.webkit.org/show_bug.cgi?id=166727

Reviewed by Andreas Kling.

Now that we destroy the render tree for documents going into the page cache, RenderView::documentBeingDestroyed()
is misleadingly named. Rename it to renderTreeBeingDestroyed() and fix all callers.

  • page/FrameView.cpp:

(WebCore::FrameView::scheduleRelayoutOfSubtree):

  • page/scrolling/ScrollingCoordinator.cpp:

(WebCore::ScrollingCoordinator::absoluteEventTrackingRegionsForFrame):

  • rendering/CounterNode.cpp:

(WebCore::CounterNode::resetRenderers):

  • rendering/ImageQualityController.cpp:

(WebCore::ImageQualityController::highQualityRepaintTimerFired):

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::willBeDestroyed):
(WebCore::canMergeContiguousAnonymousBlocks):
(WebCore::RenderBlock::removeChild):

  • rendering/RenderBlockFlow.cpp:

(WebCore::RenderBlockFlow::willBeDestroyed):
(WebCore::RenderBlockFlow::removeChild):

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::removeFloatingOrPositionedChildFromBlockLists):
(WebCore::RenderBox::deleteLineBoxWrapper):

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::willBeDestroyed):

  • rendering/RenderElement.cpp:

(WebCore::RenderElement::insertChildInternal):
(WebCore::RenderElement::removeChildInternal):
(WebCore::RenderElement::clearLayoutRootIfNeeded):
(WebCore::RenderElement::willBeDestroyed):

  • rendering/RenderImage.cpp:

(WebCore::RenderImage::imageChanged):
(WebCore::RenderImage::notifyFinished):

  • rendering/RenderInline.cpp:

(WebCore::RenderInline::willBeDestroyed):

  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::removeChild):
(WebCore::RenderLayer::calculateClipRects):

  • rendering/RenderLayerBacking.cpp:

(WebCore::RenderLayerBacking::notifyFlushRequired):

  • rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::layerWillBeRemoved):
(WebCore::RenderLayerCompositor::fixedRootBackgroundLayerChanged):

  • rendering/RenderLineBreak.cpp:

(WebCore::RenderLineBreak::deleteInlineBoxWrapper):

  • rendering/RenderNamedFlowFragment.cpp:

(WebCore::RenderNamedFlowFragment::attachRegion):

  • rendering/RenderObject.cpp:

(WebCore::RenderObject::resetFlowThreadStateOnRemoval):
(WebCore::RenderObject::willBeDestroyed):
(WebCore::RenderObject::destroyAndCleanupAnonymousWrappers):

  • rendering/RenderObject.h:

(WebCore::RenderObject::renderTreeBeingDestroyed):
(WebCore::RenderObject::documentBeingDestroyed): Deleted.

  • rendering/RenderQuote.cpp:

(WebCore::RenderQuote::detachQuote):

  • rendering/RenderRegion.cpp:

(WebCore::RenderRegion::attachRegion):

  • rendering/RenderReplaced.cpp:

(WebCore::RenderReplaced::willBeDestroyed):

  • rendering/RenderRubyRun.cpp:

(WebCore::RenderRubyRun::removeChild):

  • rendering/RenderTable.h:

(WebCore::RenderTable::setNeedsSectionRecalc):

  • rendering/RenderText.cpp:

(WebCore::RenderText::removeAndDestroyTextBoxes):

  • rendering/RenderVideo.cpp:

(WebCore::RenderVideo::updatePlayer):

  • rendering/svg/RenderSVGResource.cpp:

(WebCore::RenderSVGResource::markForLayoutAndParentResourceInvalidation):

  • rendering/svg/RenderSVGResourceContainer.cpp:

(WebCore::RenderSVGResourceContainer::markClientForInvalidation):

  • rendering/svg/RenderSVGText.cpp:

(WebCore::RenderSVGText::subtreeChildWasAdded):
(WebCore::RenderSVGText::subtreeChildWillBeRemoved):
(WebCore::RenderSVGText::subtreeChildWasRemoved):
(WebCore::RenderSVGText::subtreeStyleDidChange):

5:11 PM Changeset in webkit [214081] by Ryan Haddad
  • 7 edits
    2 deletes in branches/safari-603.1.30.0-branch/LayoutTests

Unreviewed test gardening.

LayoutTests/imported/w3c:

  • web-platform-tests/html/browsers/origin/cross-origin-objects/cross-origin-objects-expected.txt:

LayoutTests:

  • TestExpectations:
  • compositing/canvas/accelerated-canvas-compositing-size-limit-expected.txt:
  • pageoverlay/overlay-remove-reinsert-view-expected.txt:
  • platform/mac-wk2/TestExpectations:
  • workers/sab/sent-from-worker-transfer-expected.txt: Removed.
  • workers/sab/sent-from-worker-transfer.html: Removed.
5:05 PM Changeset in webkit [214080] by commit-queue@webkit.org
  • 24 edits
    2 adds in trunk

Wrap legacy WebRTC API in runtime flag
​https://bugs.webkit.org/show_bug.cgi?id=169646

Patch by Youenn Fablet <​youenn@apple.com> on 2017-03-16
Reviewed by Alex Christensen.

Source/WebCore:

Test: webrtc/legacy-api.html

Adding a runtime flag for legacy API.
Making some WebIDL peer connection members runtime-enabled.
Adding a JS private getter on this flag so that overloaded operations behave as promise-based only if flag is turned off.

  • Modules/mediastream/RTCPeerConnection.idl:
  • Modules/mediastream/RTCPeerConnectionInternals.js:
  • bindings/js/JSDOMGlobalObject.cpp:

(WebCore::isWebRTCLegacyAPIEnabled):
(WebCore::JSDOMGlobalObject::addBuiltinGlobals):

  • bindings/js/WebCoreBuiltinNames.h:
  • page/RuntimeEnabledFeatures.h:

(WebCore::RuntimeEnabledFeatures::webRTCLegacyAPIEnabled):
(WebCore::RuntimeEnabledFeatures::setWebRTCLegacyAPIEnabled):

Source/WebKit/mac:

  • WebView/WebPreferenceKeysPrivate.h:
  • WebView/WebPreferences.mm:

(+[WebPreferences initialize]):
(-[WebPreferences webRTCLegacyAPIEnabled]):
(-[WebPreferences setWebRTCLegacyAPIEnabled:]):

  • WebView/WebPreferencesPrivate.h:
  • WebView/WebView.mm:

(-[WebView _preferencesChanged:]):

Source/WebKit2:

Adding preference to legacy API.

  • Shared/WebPreferencesDefinitions.h:
  • UIProcess/API/C/WKPreferences.cpp:

(WKPreferencesSetWebRTCLegacyAPIEnabled):
(WKPreferencesGetWebRTCLegacyAPIEnabled):

  • UIProcess/API/C/WKPreferencesRef.h:
  • WebProcess/InjectedBundle/InjectedBundle.cpp:

(WebKit::InjectedBundle::overrideBoolPreferenceForTestRunner):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::updatePreferences):

Tools:

  • DumpRenderTree/mac/DumpRenderTree.mm:

(enableExperimentalFeatures):

  • WebKitTestRunner/InjectedBundle/Bindings/TestRunner.idl:
  • WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:

(WTR::InjectedBundle::beginTesting):

  • WebKitTestRunner/InjectedBundle/TestRunner.cpp:

(WTR::TestRunner::setWebRTCLegacyAPIEnabled):

  • WebKitTestRunner/InjectedBundle/TestRunner.h:

LayoutTests:

  • webrtc/legacy-api-expected.txt: Added.
  • webrtc/legacy-api.html: Added.
4:51 PM Changeset in webkit [214079] by mark.lam@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Array concat operation should check for length overflows.
​https://bugs.webkit.org/show_bug.cgi?id=169796
<rdar://problem/31095276>

Reviewed by Keith Miller.

  • runtime/ArrayPrototype.cpp:

(JSC::concatAppendOne):
(JSC::arrayProtoPrivateFuncConcatMemcpy):

4:30 PM Changeset in webkit [214078] by beidson@apple.com
  • 14 edits
    1 move in trunk

Update the WKHTTPCookieStore API to be simpler and add observers.
<rdar://problem/31096000> and ​https://bugs.webkit.org/show_bug.cgi?id=169776

Reviewed by Alex Christensen.
Source/WebKit2:

When reviewing the API internally we realized the NSHTTPCookieStorage model doesn't
necessarily fit perfectly, and that to handle partitioned cookies going forward we
need to be much simpler right now.

We also realized that we should implement the "cookie store changed" observer for
any clients that might be providing a cookie viewing API.

  • UIProcess/API/APIHTTPCookieStore.cpp:

(API::HTTPCookieStore::~HTTPCookieStore):
(API::HTTPCookieStore::deleteCookie):
(API::APIWebCookieManagerProxyObserver::APIWebCookieManagerProxyObserver):
(API::HTTPCookieStore::registerObserver):
(API::HTTPCookieStore::unregisterObserver):
(API::HTTPCookieStore::cookiesDidChange):
(API::HTTPCookieStore::cookieManagerDestroyed):
(API::HTTPCookieStore::registerForNewProcessPoolNotifications):
(API::HTTPCookieStore::unregisterForNewProcessPoolNotifications):
(API::HTTPCookieStore::setCookies): Deleted.
(API::HTTPCookieStore::removeCookiesSinceDate): Deleted.
(API::HTTPCookieStore::setHTTPCookieAcceptPolicy): Deleted.
(API::HTTPCookieStore::getHTTPCookieAcceptPolicy): Deleted.

  • UIProcess/API/APIHTTPCookieStore.h:
  • UIProcess/API/Cocoa/WKHTTPCookieStore.h:
  • UIProcess/API/Cocoa/WKHTTPCookieStore.mm:

(WKHTTPCookieStoreObserver::WKHTTPCookieStoreObserver):
(-[WKHTTPCookieStore dealloc]):
(-[WKHTTPCookieStore allCookies:]):
(-[WKHTTPCookieStore addObserver:]):
(-[WKHTTPCookieStore removeObserver:]):
(-[WKHTTPCookieStore fetchCookies:]): Deleted.
(-[WKHTTPCookieStore fetchCookiesForURL:completionHandler:]): Deleted.
(-[WKHTTPCookieStore setCookies:forURL:mainDocumentURL:completionHandler:]): Deleted.
(-[WKHTTPCookieStore removeCookiesSinceDate:completionHandler:]): Deleted.
(-[WKHTTPCookieStore setCookieAcceptPolicy:completionHandler:]): Deleted.
(kitCookiePolicyToNSCookiePolicy): Deleted.
(-[WKHTTPCookieStore fetchCookieAcceptPolicy:]): Deleted.

  • UIProcess/Cocoa/WebProcessPoolCocoa.mm:

(WebKit::WebProcessPool::platformInitializeNetworkProcess):

  • UIProcess/WebCookieManagerProxy.cpp:

(WebKit::WebCookieManagerProxy::~WebCookieManagerProxy):
(WebKit::WebCookieManagerProxy::processPoolDestroyed):
(WebKit::WebCookieManagerProxy::registerObserver):
(WebKit::WebCookieManagerProxy::unregisterObserver):
(WebKit::WebCookieManagerProxy::cookiesDidChange):
(WebKit::WebCookieManagerProxy::setCookieObserverCallback): Deleted.

  • UIProcess/WebCookieManagerProxy.h:

(WebKit::WebCookieManagerProxy::Observer::~Observer):

Add the ability for interested clients to listen for the creation of new WebProcessPools,
which is needed by APIHTTPCookieStore:

  • UIProcess/WebProcessPool.cpp:

(WebKit::generateListenerIdentifier):
(WebKit::processPoolCreationListenerFunctionMap):
(WebKit::WebProcessPool::registerProcessPoolCreationListener):
(WebKit::WebProcessPool::unregisterProcessPoolCreationListener):
(WebKit::WebProcessPool::create):

  • UIProcess/WebProcessPool.h:
  • UIProcess/WebsiteData/WebsiteDataStore.cpp:

(WebKit::WebsiteDataStore::processPoolForCookieStorageNotifications):
(WebKit::WebsiteDataStore::isAssociatedProcessPool):
(WebKit::WebsiteDataStore::processPools):

  • UIProcess/WebsiteData/WebsiteDataStore.h:

Tools:

  • TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
  • TestWebKitAPI/Tests/WebKit2Cocoa/WKHTTPCookieStore.mm: Renamed from Tools/TestWebKitAPI/Tests/WebKit2Cocoa/WKHTTPCookieStorage.mm.

(-[CookieObserver cookiesDidChangeInCookieStore:]):
(TEST): Also test delete and observers.

4:19 PM Changeset in webkit [214077] by webkit@devinrousso.com
  • 4 edits in trunk/Source/WebInspectorUI

Web Inspector: add "Save Selected" context menu item to Console
​https://bugs.webkit.org/show_bug.cgi?id=151837

Reviewed by Joseph Pecoraro.

  • Localizations/en.lproj/localizedStrings.js:
  • UserInterface/Base/Main.js:

(WebInspector.saveDataToFile):
Allow data to be saved without a provided URL. It will default to localized "Untitled".

  • UserInterface/Views/LogContentView.js:

(WebInspector.LogContentView.prototype._handleContextMenuEvent):

4:11 PM Changeset in webkit [214076] by webkit@devinrousso.com
  • 3 edits in trunk/Source/WebInspectorUI

Web Inspector: RTL: selected frame indicator is not positioned correctly
​https://bugs.webkit.org/show_bug.cgi?id=169763

Reviewed by Matt Baker.

  • UserInterface/Views/RenderingFrameTimelineOverviewGraph.js:

(WebInspector.RenderingFrameTimelineOverviewGraph.prototype._updateFrameMarker):
(WebInspector.RenderingFrameTimelineOverviewGraph.prototype._mouseClicked):
Treat the current mouse position (event.pageX) as a negative value when in RTL, meaning that
the delta movement from the starting position is flipped.

  • UserInterface/Views/TimelineRuler.css:

(body[dir=ltr] .timeline-ruler > .header > .divider,):
(body[dir=rtl] .timeline-ruler > .header > .divider,):
(.timeline-ruler > .header > .divider):
(.timeline-ruler > .header > .divider > .label):
(body[dir=ltr] .timeline-ruler > .header > .divider > .label):
(body[dir=rtl] .timeline-ruler > .header > .divider > .label):
(.timeline-ruler > .markers > .divider):
(.timeline-ruler > .markers > .marker):
(body[dir=ltr] .timeline-ruler > .markers > .marker):
(body[dir=rtl] .timeline-ruler > .markers > .marker):
(.timeline-ruler > .markers > .marker::before):
(body[dir=ltr] .timeline-ruler > .markers > .marker::before):
(body[dir=rtl] .timeline-ruler > .markers > .marker::before):
(body[dir=ltr] .timeline-ruler > .markers > .marker.current-time):
(body[dir=rtl] .timeline-ruler > .markers > .marker.current-time):
(.timeline-ruler > .markers > .marker.current-time::before):
(.timeline-ruler > .markers > .marker.current-time): Deleted.

(.timeline-ruler > .selection-handle):
(body[dir=ltr] .timeline-ruler > .selection-handle.left):
(body[dir=rtl] .timeline-ruler > .selection-handle.left):
(body[dir=ltr] .timeline-ruler > .selection-handle.right):
(body[dir=rtl] .timeline-ruler > .selection-handle.right):
Drive-by: add CSS variable for clarity.

3:53 PM Changeset in webkit [214075] by commit-queue@webkit.org
  • 14 edits in trunk/LayoutTests

Rebase webrtc tests
​https://bugs.webkit.org/show_bug.cgi?id=169791

Unreviewed.

Patch by Youenn Fablet <​youenn@apple.com> on 2017-03-16

  • TestExpectations:
  • fast/mediastream/MediaStream-add-remove-tracks-expected.txt:
  • fast/mediastream/MediaStreamTrack-expected.txt:
  • fast/mediastream/MediaStreamTrack-onended-expected.txt:
  • fast/mediastream/RTCPeerConnection-addIceCandidate-expected.txt:
  • fast/mediastream/RTCPeerConnection-events-expected.txt:
  • fast/mediastream/RTCPeerConnection-media-setup-single-dialog-expected.txt:
  • fast/mediastream/RTCPeerConnection-media-setup-two-dialogs-expected.txt:
  • fast/mediastream/RTCPeerConnection-remotely-assigned-transceiver-mid-expected.txt:
  • fast/mediastream/RTCPeerConnection-setLocalDescription-offer-expected.txt:
  • fast/mediastream/RTCPeerConnection-setRemoteDescription-offer-expected.txt:
  • fast/mediastream/RTCPeerConnection-statsSelector-expected.txt:
  • platform/mac/TestExpectations:
3:32 PM Changeset in webkit [214074] by jiewen_tan@apple.com
  • 18 edits
    6 copies
    43 adds in trunk

[WebCrypto] Support SPKI/PKCS8 for Elliptic Curve
​https://bugs.webkit.org/show_bug.cgi?id=169318
<rdar://problem/31081956>

Reviewed by Brent Fulgham.

LayoutTests/imported/w3c:

  • web-platform-tests/WebCryptoAPI/import_export/ec_importKey.worker-expected.txt:
  • web-platform-tests/WebCryptoAPI/import_export/test_ec_importKey-expected.txt:

Source/WebCore:

This patch adds SPKI/PKCS8 support for Elliptic Curve cryptos. We can now import/export
SPKI/PKCS8 Elliptic Curve keys after this change. Few things to note: 1) This patch
implements a loose DER encoder/decoder for hacking the underlying CommonCrypto library.
2) It only permits id-ecPublicKey as the AlgorithmIdentifier following OpenSSL/Chrome(BoringSSL).
3) It follows OpenSSL/Chrome(BoringSSL) to replace ECParameters in ECPrivateKey with custom
tags. Hence, we should fully comply with OpenSSL/Chrome(BoringSSL).

Tests: crypto/subtle/ec-import-jwk-key-export-pkcs8-key.html

crypto/subtle/ec-import-jwk-key-export-spki-key.html
crypto/subtle/ec-import-pkcs8-key-export-jwk-key.html
crypto/subtle/ec-import-pkcs8-key-export-pkcs8-key-p256.html
crypto/subtle/ec-import-pkcs8-key-export-pkcs8-key-p384.html
crypto/subtle/ec-import-raw-key-export-spki-key.html
crypto/subtle/ec-import-spki-key-export-jwk-key.html
crypto/subtle/ec-import-spki-key-export-raw-key.html
crypto/subtle/ec-import-spki-key-export-spki-key-p256.html
crypto/subtle/ec-import-spki-key-export-spki-key-p384.html
crypto/subtle/ecdh-generate-export-key-spki-p256.html
crypto/subtle/ecdh-generate-export-key-spki-p384.html
crypto/subtle/ecdh-generate-export-pkcs8-p256.html
crypto/subtle/ecdh-generate-export-pkcs8-p384.html
crypto/subtle/ecdh-import-pkcs8-key-p256.html
crypto/subtle/ecdh-import-pkcs8-key-p384.html
crypto/subtle/ecdh-import-spki-key-p256.html
crypto/subtle/ecdh-import-spki-key-p384.html
crypto/workers/subtle/ec-generate-export-pkcs8-key.html
crypto/workers/subtle/ec-generate-export-spki-key.html
crypto/workers/subtle/ec-import-pkcs8-key.html
crypto/workers/subtle/ec-import-spki-key.html

  • WebCore.xcodeproj/project.pbxproj:
  • crypto/algorithms/CryptoAlgorithmECDH.cpp:

(WebCore::CryptoAlgorithmECDH::importKey):
(WebCore::CryptoAlgorithmECDH::exportKey):

  • crypto/gnutls/CryptoKeyECGnuTLS.cpp:

(WebCore::CryptoKeyEC::platformExportRaw):
(WebCore::CryptoKeyEC::platformImportSpki):
(WebCore::CryptoKeyEC::platformExportSpki):
(WebCore::CryptoKeyEC::platformImportPkcs8):
(WebCore::CryptoKeyEC::platformExportPkcs8):
(WebCore::CryptoKeyEC::exportRaw): Deleted.

  • crypto/keys/CryptoKeyEC.cpp:

(WebCore::CryptoKeyEC::importSpki):
(WebCore::CryptoKeyEC::importPkcs8):
(WebCore::CryptoKeyEC::exportRaw):
(WebCore::CryptoKeyEC::exportSpki):
(WebCore::CryptoKeyEC::exportPkcs8):

  • crypto/keys/CryptoKeyEC.h:
  • crypto/mac/CommonCryptoDERUtilities.h: Added.

(WebCore::bytesUsedToEncodedLength):
(WebCore::extraBytesNeededForEncodedLength):
(WebCore::addEncodedASN1Length):
(WebCore::bytesNeededForEncodedLength):

  • crypto/mac/CryptoKeyECMac.cpp:

(WebCore::compareBytes):
(WebCore::CryptoKeyEC::platformExportRaw):
(WebCore::CryptoKeyEC::platformImportJWKPrivate):
(WebCore::getOID):
(WebCore::CryptoKeyEC::platformImportSpki):
(WebCore::CryptoKeyEC::platformExportSpki):
(WebCore::CryptoKeyEC::platformImportPkcs8):
(WebCore::CryptoKeyEC::platformExportPkcs8):
(WebCore::CryptoKeyEC::exportRaw): Deleted.
Enlarge the robust of exportRaw.

  • crypto/mac/CryptoKeyRSAMac.cpp:

(WebCore::CryptoKeyRSA::exportSpki):
(WebCore::CryptoKeyRSA::exportPkcs8):
Enhance the implementation.
(WebCore::bytesUsedToEncodedLength): Deleted.
(WebCore::bytesNeededForEncodedLength): Deleted.
(WebCore::addEncodedASN1Length): Deleted.
Moved to CommonCryptoDERUtilities.h.

LayoutTests:

  • crypto/subtle/ec-export-key-malformed-parameters-expected.txt:
  • crypto/subtle/ec-export-key-malformed-parameters.html:
  • crypto/subtle/ec-import-jwk-key-export-jwk-key-private-expected.txt:
  • crypto/subtle/ec-import-jwk-key-export-jwk-key-private.html:
  • crypto/subtle/ec-import-jwk-key-export-pkcs8-key-expected.txt: Added.
  • crypto/subtle/ec-import-jwk-key-export-pkcs8-key.html: Added.
  • crypto/subtle/ec-import-jwk-key-export-spki-key-expected.txt: Added.
  • crypto/subtle/ec-import-jwk-key-export-spki-key.html: Added.
  • crypto/subtle/ec-import-key-malformed-parameters-expected.txt:
  • crypto/subtle/ec-import-key-malformed-parameters.html:
  • crypto/subtle/ec-import-pkcs8-key-export-jwk-key-expected.txt: Added.
  • crypto/subtle/ec-import-pkcs8-key-export-jwk-key.html: Added.
  • crypto/subtle/ec-import-pkcs8-key-export-pkcs8-key-p256-expected.txt: Added.
  • crypto/subtle/ec-import-pkcs8-key-export-pkcs8-key-p256.html: Added.
  • crypto/subtle/ec-import-pkcs8-key-export-pkcs8-key-p384-expected.txt: Added.
  • crypto/subtle/ec-import-pkcs8-key-export-pkcs8-key-p384.html: Added.
  • crypto/subtle/ec-import-raw-key-export-spki-key-expected.txt: Added.
  • crypto/subtle/ec-import-raw-key-export-spki-key.html: Added.
  • crypto/subtle/ec-import-spki-key-export-jwk-key-expected.txt: Added.
  • crypto/subtle/ec-import-spki-key-export-jwk-key.html: Added.
  • crypto/subtle/ec-import-spki-key-export-raw-key-expected.txt: Added.
  • crypto/subtle/ec-import-spki-key-export-raw-key.html: Added.
  • crypto/subtle/ec-import-spki-key-export-spki-key-p256-expected.txt: Added.
  • crypto/subtle/ec-import-spki-key-export-spki-key-p256.html: Added.
  • crypto/subtle/ec-import-spki-key-export-spki-key-p384-expected.txt: Added.
  • crypto/subtle/ec-import-spki-key-export-spki-key-p384.html: Added.
  • crypto/subtle/ecdh-generate-export-key-spki-p256-expected.txt: Added.
  • crypto/subtle/ecdh-generate-export-key-spki-p256.html: Added.
  • crypto/subtle/ecdh-generate-export-key-spki-p384-expected.txt: Added.
  • crypto/subtle/ecdh-generate-export-key-spki-p384.html: Added.
  • crypto/subtle/ecdh-generate-export-pkcs8-p256-expected.txt: Added.
  • crypto/subtle/ecdh-generate-export-pkcs8-p256.html: Added.
  • crypto/subtle/ecdh-generate-export-pkcs8-p384-expected.txt: Added.
  • crypto/subtle/ecdh-generate-export-pkcs8-p384.html: Added.
  • crypto/subtle/ecdh-import-pkcs8-key-p256-expected.txt: Added.
  • crypto/subtle/ecdh-import-pkcs8-key-p256.html: Added.
  • crypto/subtle/ecdh-import-pkcs8-key-p384-expected.txt: Added.
  • crypto/subtle/ecdh-import-pkcs8-key-p384.html: Added.
  • crypto/subtle/ecdh-import-spki-key-p256-expected.txt: Added.
  • crypto/subtle/ecdh-import-spki-key-p256.html: Added.
  • crypto/subtle/ecdh-import-spki-key-p384-expected.txt: Added.
  • crypto/subtle/ecdh-import-spki-key-p384.html: Added.
  • crypto/workers/subtle/ec-generate-export-pkcs8-key-expected.txt: Added.
  • crypto/workers/subtle/ec-generate-export-pkcs8-key.html: Added.
  • crypto/workers/subtle/ec-generate-export-spki-key-expected.txt: Added.
  • crypto/workers/subtle/ec-generate-export-spki-key.html: Added.
  • crypto/workers/subtle/ec-import-pkcs8-key-expected.txt: Added.
  • crypto/workers/subtle/ec-import-pkcs8-key.html: Added.
  • crypto/workers/subtle/ec-import-spki-key-expected.txt: Added.
  • crypto/workers/subtle/ec-import-spki-key.html: Added.
  • crypto/workers/subtle/resources/ec-generate-export-pkcs8-key.js: Added.
  • crypto/workers/subtle/resources/ec-generate-export-spki-key.js: Added.
  • crypto/workers/subtle/resources/ec-import-pkcs8-key.js: Added.
  • crypto/workers/subtle/resources/ec-import-spki-key.js: Added.
3:15 PM Changeset in webkit [214073] by achristensen@apple.com
  • 6 edits in trunk/Source/WebCore

Don't use an optional for didReceiveSocketStreamData's length
​https://bugs.webkit.org/show_bug.cgi?id=169699

Reviewed by Brady Eidson.

Rather than sending a message with an optional length, send a message with a length
and send a different message if the receiving of bytes failed.

No new tests (no behavior change).

  • Modules/websockets/WebSocketChannel.cpp:

(WebCore::WebSocketChannel::didReceiveSocketStreamData):

  • Modules/websockets/WebSocketChannel.h:
  • platform/network/SocketStreamHandleClient.h:
  • platform/network/cf/SocketStreamHandleImplCFNet.cpp:

(WebCore::SocketStreamHandleImpl::readStreamCallback):

  • platform/network/soup/SocketStreamHandleImplSoup.cpp:

(WebCore::SocketStreamHandleImpl::readBytes):

3:14 PM Changeset in webkit [214072] by Alan Bujtas
  • 5 edits
    2 adds in trunk

Simple line layout: Extend webkit-hyphenate-limit-lines to cover subsequent words.
​https://bugs.webkit.org/show_bug.cgi?id=169631

Reviewed by Antti Koivisto.

Source/WebCore:

This patch extends webkit-hyphenate-limit-lines to cover separate words across multiple lines.
Example:

webkit-hyphenate-limit-lines: 1;
"if this line wraps at foo-
bar, the next line should
not hyphenate at all. How-
ever the line after that
could."

Test: fast/text/simple-line-layout-hyphenation-limit-lines-accross-words.html

  • rendering/SimpleLineLayout.cpp:

(WebCore::SimpleLineLayout::LineState::setHyphenationDisabled):
(WebCore::SimpleLineLayout::LineState::isHyphenationDisabled):
(WebCore::SimpleLineLayout::updateLineConstrains):
(WebCore::SimpleLineLayout::hyphenPositionForFragment):
(WebCore::SimpleLineLayout::splitFragmentToFitLine):
(WebCore::SimpleLineLayout::createLineRuns):
(WebCore::SimpleLineLayout::createTextRuns):

  • rendering/SimpleLineLayoutTextFragmentIterator.h:

(WebCore::SimpleLineLayout::TextFragmentIterator::TextFragment::hasHyphen):
(WebCore::SimpleLineLayout::TextFragmentIterator::TextFragment::splitWithHyphen):
(WebCore::SimpleLineLayout::TextFragmentIterator::TextFragment::wrappingWithHyphenCounter): Deleted.

LayoutTests:

  • fast/text/simple-line-layout-hyphenation-limit-lines-accross-words-expected.html: Added.
  • fast/text/simple-line-layout-hyphenation-limit-lines-accross-words.html: Added.
2:53 PM Changeset in webkit [214071] by mark.lam@apple.com
  • 8 edits in trunk/Source/JavaScriptCore

The new array with spread operation needs to check for length overflows.
​https://bugs.webkit.org/show_bug.cgi?id=169780
<rdar://problem/31072182>

Reviewed by Filip Pizlo.

  • dfg/DFGOperations.cpp:
  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compileNewArrayWithSpread):

  • ftl/FTLLowerDFGToB3.cpp:

(JSC::FTL::DFG::LowerDFGToB3::compileNewArrayWithSpread):

  • ftl/FTLOperations.cpp:

(JSC::FTL::operationMaterializeObjectInOSR):

  • llint/LLIntSlowPaths.cpp:
  • runtime/CommonSlowPaths.cpp:

(JSC::SLOW_PATH_DECL):

  • runtime/JSGlobalObject.cpp:
2:49 PM Changeset in webkit [214070] by BJ Burg
  • 4 edits
    2 adds in trunk/Source/WebKit2

[iOS WK2] Web Automation: implement platform methods for simulating keyboard events
​https://bugs.webkit.org/show_bug.cgi?id=169487
<rdar://problem/28360564>

Reviewed by Joseph Pecoraro.

Reimplement platformSimulateKey{Stroke, Sequence} to simulate events coming from UIKit.
These events are sent directly into WebCore without going through the keyboard system,
since we don't need to show the keyboard when simulating keystrokes. For example, the
keystrokes may not be representable given the current keyboard layout and system language.

  • UIProcess/Automation/WebAutomationSession.cpp:
  • UIProcess/Automation/WebAutomationSession.h:

Un-stub implementations that now exist for iOS.

  • UIProcess/Automation/ios/WebAutomationSessionIOS.mm: Added.

(WebKit::WebAutomationSession::sendSynthesizedEventsToPage): Added.
This is similar to the Mac version, but it sends events directly into WebPageProxy
instead of sending them to the NSWindow containing the WebView.

(WebKit::WebAutomationSession::platformSimulateKeyStroke):
Use a somewhat different strategy for determining the charCode/keyCode since we cannot
rely on the system keyboard / input manager to generate and deliver real key events.
Compared to the Mac platform method, this requires setting up fewer fields and does
not use AppKit's current modifier state to determine the effect of sticky modifiers.

(WebKit::WebAutomationSession::platformSimulateKeySequence):
This is almost verbatim from the Mac platform method but uses a different event class.

  • WebKit2.xcodeproj/project.pbxproj: Add new file.
2:19 PM Changeset in webkit [214069] by fpizlo@apple.com
  • 14 edits
    3 adds in trunk

FTL should support global and eval code
​https://bugs.webkit.org/show_bug.cgi?id=169656

Reviewed by Geoffrey Garen and Saam Barati.

JSTests:

Added basic performance tests of global and eval code. These tests will run a lot faster in with
the FTL because of the object allocation.

  • microbenchmarks/eval-code-ftl-reentry.js: Added.
  • microbenchmarks/eval-code-ftl.js: Added.
  • microbenchmarks/global-code-ftl.js: Added.
  • stress/arith-log-on-various-types.js: This was a flaky fail with concurrent JIT, so I stopped running it with concurrent JIT. The failure was its assertion about how many times something gets compiled.

Source/JavaScriptCore:

Turned off the restriction against global and eval code running in the FTL, and then fixed all of
the things that didn't work.

This is a big speed-up on microbenchmarks that I wrote for this patch. One of the reasons why we
hadn't done this earlier is that we've never seen a benchmark that needed it. Global and eval
code rarely gets FTL-hot. Still, this seems like possibly a small JetStream speed-up.

  • dfg/DFGJITCode.cpp:

(JSC::DFG::JITCode::setOSREntryBlock): I outlined this for better debugging.

  • dfg/DFGJITCode.h:

(JSC::DFG::JITCode::setOSREntryBlock): Deleted.

  • dfg/DFGNode.h:

(JSC::DFG::Node::isSemanticallySkippable): It turns out that global code often has InvalidationPoints before LoopHints. They are also skippable from the standpoint of OSR entrypoint analysis.

  • dfg/DFGOperations.cpp: Don't do any normal compiles of global code - just do OSR compiles.
  • ftl/FTLCapabilities.cpp: Enable FTL for global and eval code.

(JSC::FTL::canCompile):

  • ftl/FTLCompile.cpp: Just debugging clean-ups.

(JSC::FTL::compile):

  • ftl/FTLJITFinalizer.cpp: Implement finalize() and ensure that we only do things with the entrypoint buffer if we have one. We won't have one for eval code that we aren't OSR entering into.

(JSC::FTL::JITFinalizer::finalize):
(JSC::FTL::JITFinalizer::finalizeFunction):
(JSC::FTL::JITFinalizer::finalizeCommon):

  • ftl/FTLJITFinalizer.h:
  • ftl/FTLLink.cpp: When entering a function normally, we need the "entrypoint" to put the arity check code. Global and eval code don't need this.

(JSC::FTL::link):

  • ftl/FTLOSREntry.cpp: Fix a dataLog statement.

(JSC::FTL::prepareOSREntry):

  • ftl/FTLOSRExitCompiler.cpp: Remove dead code that happened to assert that we're exiting from a function.

(JSC::FTL::compileStub):

2:10 PM Changeset in webkit [214068] by msaboff@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

WebAssembly: function-tests/load-offset.js fails on ARM64
​https://bugs.webkit.org/show_bug.cgi?id=169724

Reviewed by Keith Miller.

We need to use the two source version of Add64 to create a Wasm address with the
other source the first child.

  • b3/B3LowerToAir.cpp:

(JSC::B3::Air::LowerToAir::lower):

1:55 PM Changeset in webkit [214067] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

Fix quotes around --pretty format git flag.
​https://bugs.webkit.org/show_bug.cgi?id=169712

Patch by Kocsen Chung <Kocsen Chung> on 2017-03-16
Reviewed by Ryosuke Niwa.

Remove double quotes from command list element '--pretty="format:%ct"' since it will cause
subprocess to not work as expected.
Additionally, take advantage of git -C flag.

  • Scripts/webkitpy/common/checkout/scm/git.py:

(Git.native_revision):
Take advantage of git -C flag.
(Git.timestamp_of_native_revision):
Take advantage of git -C flag and remove double quotes from git formatter.
We know this works from committer_email_for_revision() function.

1:53 PM Changeset in webkit [214066] by webkit@devinrousso.com
  • 2 edits in trunk/Source/WebInspectorUI

Web Inspector: RTL: filter mode icon in Debugger sidebar is misaligned
​https://bugs.webkit.org/show_bug.cgi?id=169766

Reviewed by Matt Baker.

  • UserInterface/Views/FilterBar.css:

(.filter-bar > .navigation-bar > .item):
(body[dir=ltr] .filter-bar > .navigation-bar > .item):
(body[dir=rtl] .filter-bar > .navigation-bar > .item):

1:53 PM Changeset in webkit [214065] by rniwa@webkit.org
  • 15 edits
    8 adds in trunk/Websites/perf.webkit.org

Add the file uploading capability to the perf dashboard.
​https://bugs.webkit.org/show_bug.cgi?id=169737

Reviewed by Chris Dumez.

Added /privileged-api/upload-file to upload a file, and /api/uploaded-file/ to download the file
and retrieve its meta data based on its SHA256. We treat two files with the identical SHA256 as
identical since anyone who can upload a file using this mechanism can execute arbitrary code in
our bots anyway. This is important for avoiding uploading a large darwinup roots multiple times
to the server, saving both user's time/bandwidth and server's disk space.

  • config.json: Added uploadDirectory, uploadFileLimitInMB, and uploadUserQuotaInMB as options.
  • init-database.sql: Added uploaded_files table.
  • public/api/uploaded-file.php: Added.

(main): /api/uploaded-file/N would download uploaded_file with id=N. /api/uploaded-file/?sha256=X
would return the meta data for uploaded_file with sha256=X.
(stream_file_content): Streams the file content in 64KB chunks. We support Range & If-Range HTTP
request headers so that browsers can pause and resume downloading of a large root file.
(parse_range_header): Parses Range HTTP request header.

  • public/include/json-header.php:

(remote_user_name): Use the default argument of NULL.

  • public/include/manifest-generator.php:

(ManifestGenerator::generate): Include the maximum upload size in the manifest file to let the
frontend code preemptively check the file size before attempting to submit a file.

  • public/include/uploaded-file-helpers.php: Added.

(format_uploaded_file):
(uploaded_file_path_for_row):

  • public/privileged-api/upload-file-form.html: Added. For debugging purposes.

(fetchCSRFfToken):
(upload):

  • public/privileged-api/upload-file.php: Added.

(main):
(query_total_file_size):
(create_uploaded_file_from_form_data):

  • public/shared/common-remote.js:

(CommonRemoteAPI.prototype.postFormData): Added.
(CommonRemoteAPI.prototype.postFormDataWithStatus): Added.
(CommonRemoteAPI.prototype.sendHttpRequestWithFormData): Added.
(CommonRemoteAPI.prototype._asJSON): Throw an exception instead of calling a non-existent reject.

  • public/v3/models/uploaded-file.js: Added.

(UploadedFile): Added.
(UploadedFile.uploadFile): Added.
(UploadedFile.fetchUnloadedFileWithIdenticalHash): Added. Finds the file with the same SHA256 in
the server to avoid uploading a large custom root multiple times.
(UploadedFile._computeSHA256Hash): Added.

  • public/v3/privileged-api.js:

(PrivilegedAPI.prototype.sendRequest): Added the options dictionary as a third argument. For now,
only support useFormData boolean.

  • public/v3/remote.js:

(BrowserRemoteAPI.prototype.sendHttpRequestWithFormData): Added.

  • server-tests/api-manifest.js: Updated per the inclusion of fileUploadSizeLimit in the manifest.
  • server-tests/api-uploaded-file.js: Added.
  • server-tests/privileged-api-upload-file-tests.js: Added.
  • server-tests/resources/temporary-file.js: Added.

(TemporaryFile): Added. A helper class for creating a temporary file to upload.
(TemporaryFile.makeTemporaryFileOfSizeInMB):
(TemporaryFile.makeTemporaryFile):
(TemporaryFile.inject):

  • server-tests/resources/test-server.conf: Set upload_max_filesize and post_max_size for testing.
  • server-tests/resources/test-server.js:

(TestServer.prototype.testConfig): Use uploadFileLimitInMB and uploadUserQuotaInMB of 2MB and 5MB.
(TestServer.prototype._ensureDataDirectory): Create a directory to store uploaded files inside
the data directory. In a production server, we can place it outside ServerRoot / DocumentRoot.
(TestServer.prototype.cleanDataDirectory): Delete the aforementioned directory as needed.

  • tools/js/database.js:

(tableToPrefixMap): Added uploaded_files.

  • tools/js/remote.js:

(NodeRemoteAPI.prototype.sendHttpRequest): Added a dictionary to specify request headers and
a callback to process the response as arguments. Fixed the bug that any 2xx code other than 200
was resulting in a rejected promise. Also include the response headers in the result for tests.
Finally, when content is a function, call that instead of writing the content since FormData
requires a custom logic.
(NodeRemoteAPI.prototype.sendHttpRequestWithFormData): Added.

  • tools/js/v3-models.js: Include uploaded-file.js.
  • tools/run-tests.py:

(main): Add form-data as a new dependency.

1:50 PM Changeset in webkit [214064] by Matt Baker
  • 5 edits
    1 delete in trunk/Source/WebInspectorUI

Web Inspector: Refactoring: remove DebuggerTreeElement class
​https://bugs.webkit.org/show_bug.cgi?id=169697

Reviewed by Devin Rousso.

Remove unnecessary class DebuggerTreeElement from TreeElement class hierarchy.
The sourceCodeLocation property on IssueMessage and Breakpoint (the debugger
objects) can be tested for generically using TreeElement.prototype.representedObject.

  • UserInterface/Main.html:

Remove file.

  • UserInterface/Views/BreakpointTreeElement.js:
  • UserInterface/Views/IssueTreeElement.js:

Change base class to GeneralTreeElement.

  • UserInterface/Views/DebuggerSidebarPanel.js:

(WebInspector.DebuggerSidebarPanel.prototype._addBreakpoint):
(WebInspector.DebuggerSidebarPanel.prototype._compareTreeElements):
(WebInspector.DebuggerSidebarPanel.prototype._addIssue):
(WebInspector.DebuggerSidebarPanel.prototype._compareDebuggerTreeElements): Deleted.
Renamed to just "_compareTreeElements".

  • UserInterface/Views/DebuggerTreeElement.js: Removed.
1:42 PM Changeset in webkit [214063] by commit-queue@webkit.org
  • 6 edits in trunk

[mac-wk2 debug] LayoutTest webrtc/video-mute.html is timing out
​https://bugs.webkit.org/show_bug.cgi?id=169758

Patch by Youenn Fablet <​youenn@apple.com> on 2017-03-16
Reviewed by Eric Carlson.

Source/WebKit2:

Test is timing out due to the UIProcess requesting to disable/enable ICE candidate filtering
on a page whose preference is set to not enabling ICE candidate filtering.
Current fix is to act upon what UI process asks if the page preference is to enable ICE candidate filtering.
If ICE candidate filtering preference is disable for that page (WebKitTestRunner case), no disabling/enabling is done at all.

  • UIProcess/UserMediaProcessManager.cpp:

(WebKit::UserMediaProcessManager::endedCaptureSession):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::m_backgroundCPULimit):
(WebKit::m_backgroundCPULimit):
(WebKit::WebPage::disableICECandidateFiltering):
(WebKit::WebPage::enableICECandidateFiltering):

  • WebProcess/WebPage/WebPage.h:

LayoutTests:

1:28 PM Changeset in webkit [214062] by webkit@devinrousso.com
  • 2 edits in trunk/Source/WebInspectorUI

Web Inspector: RTL: goto arrow for source locations is pointing wrong way in Timeline data grids
​https://bugs.webkit.org/show_bug.cgi?id=169764

Reviewed by Matt Baker.

  • UserInterface/Views/DataGrid.css:

(body[dir=rtl] .data-grid td .go-to-arrow):

12:06 PM Changeset in webkit [214061] by achristensen@apple.com
  • 12 edits in trunk

Clean up WKContentExtension API
​https://bugs.webkit.org/show_bug.cgi?id=169772

Reviewed by Geoffrey Garen.

Source/WebKit2:

Based on feedback I have decided to do the following:

  1. Rename lookup to lookUp because look and up are two words.
  2. Change the parameter of removeContentExtension to a WKContentExtension* to match addContentExtension.
  3. Add an identifier property accessor to WKContentExtension.
  4. Change the order of the new error types. This wasn't exactly based on any feedback, but they look more organized now.
  • UIProcess/API/Cocoa/WKContentExtension.h:
  • UIProcess/API/Cocoa/WKContentExtension.mm:

(-[WKContentExtension identifier]):

  • UIProcess/API/Cocoa/WKContentExtensionStore.h:
  • UIProcess/API/Cocoa/WKContentExtensionStore.mm:

(toWKErrorCode):
(-[WKContentExtensionStore lookUpContentExtensionForIdentifier:completionHandler:]):
(-[WKContentExtensionStore lookupContentExtensionForIdentifier:completionHandler:]): Deleted.

  • UIProcess/API/Cocoa/WKError.h:
  • UIProcess/API/Cocoa/WKError.mm:

(localizedDescriptionForErrorCode):

  • UIProcess/API/Cocoa/WKUserContentController.h:
  • UIProcess/API/Cocoa/WKUserContentController.mm:

(-[WKUserContentController removeContentExtension:]):

  • UIProcess/API/Cocoa/_WKUserContentExtensionStore.mm:

(toUserContentExtensionStoreError):
(-[_WKUserContentExtensionStore lookupContentExtensionForIdentifier:completionHandler:]):

Tools:

  • TestWebKitAPI/Tests/WebKit2Cocoa/WKUserContentExtensionStore.mm:

(TEST_F):
(-[ContentExtensionDelegate webView:runJavaScriptAlertPanelWithMessage:initiatedByFrame:completionHandler:]):

11:58 AM Changeset in webkit [214060] by zandobersek@gmail.com
  • 2 edits
    1 add
    1 delete in trunk/Source/WebCore

[GLib] Add SSLKeyGeneratorGLib.cpp
​https://bugs.webkit.org/show_bug.cgi?id=169740

Reviewed by Sergio Villar Senin.

Move the getSupportedKeySizes() and signedPublicKeyAndChallengeString()
functions from TemporaryLinkStubs.cpp to SSLKeyGeneratorGLib.cpp. The
userIdleTime() function can be removed in this process since it's not
used anywhere anymore.

  • PlatformGTK.cmake:
  • platform/glib/SSLKeyGeneratorGLib.cpp: Renamed from Source/WebCore/platform/gtk/TemporaryLinkStubs.cpp.

(WebCore::getSupportedKeySizes):
(WebCore::signedPublicKeyAndChallengeString):

11:43 AM Changeset in webkit [214059] by Alan Bujtas
  • 3 edits
    2 adds in trunk

Stay inside the continuation while searching for a candidate ancestor for insertion.
​https://bugs.webkit.org/show_bug.cgi?id=169768
<rdar://problem/30959936>

Reviewed by David Hyatt.

Source/WebCore:

Test: fast/inline/continuation-crash-with-anon-ancestors.html

  • rendering/RenderInline.cpp:

(WebCore::RenderInline::addChildToContinuation):

LayoutTests:

  • fast/inline/continuation-crash-with-anon-ancestors-expected.txt: Added.
  • fast/inline/continuation-crash-with-anon-ancestors.html: Added.
11:31 AM Changeset in webkit [214058] by matthew_hanson@apple.com
  • 1 copy in tags/Safari-603.1.30.4.6

Tag Safari-603.1.30.4.6.

11:28 AM Changeset in webkit [214057] by matthew_hanson@apple.com
  • 3 edits in branches/safari-603-branch/Source/WebKit/win

Merge r213846. rdar://problem/30983702

11:28 AM Changeset in webkit [214056] by matthew_hanson@apple.com
  • 3 edits in branches/safari-603-branch/Source/WebKit/win

Merge r213737. rdar://problem/30983702

11:26 AM Changeset in webkit [214055] by matthew_hanson@apple.com
  • 5 edits in branches/safari-603.1.30.4-branch/Source

Versioning

11:18 AM Changeset in webkit [214054] by matthew_hanson@apple.com
  • 3 edits in branches/safari-603.1.30.4-branch/Source/WebKit/win

Merge r213846. rdar://problem/30983702

11:18 AM Changeset in webkit [214053] by matthew_hanson@apple.com
  • 2 edits in branches/safari-603.1.30.4-branch/Source/WebCore

Roll out r213847 via r214013. rdar://problem/30983702

11:18 AM Changeset in webkit [214052] by matthew_hanson@apple.com
  • 5 edits in branches/safari-603.1.30.4-branch/Source

Versioning.

11:01 AM QtWebKitSecurity edited by Konstantin Tokarev
Removed info about 2.2.0 that is not the latest stable release for a … (diff)
10:31 AM Changeset in webkit [214051] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Mark fast/mediacapturefromelement/CanvasCaptureMediaStream-request-frame-events.html as flaky.
​https://bugs.webkit.org/show_bug.cgi?id=169719

Unreviewed test gardening.

  • platform/mac-wk2/TestExpectations:
10:20 AM Changeset in webkit [214050] by Ryan Haddad
  • 3 edits in trunk/LayoutTests

TestExpectations gardening for modern-media-controls tests.

Unreviewed test gardening.

  • platform/mac-wk1/TestExpectations:
  • platform/mac/TestExpectations:
10:20 AM Changeset in webkit [214049] by Ryan Haddad
  • 3 edits in trunk/LayoutTests

Mark imported/w3c/web-platform-tests/IndexedDB/fire-success-event-exception.html as flaky.
​https://bugs.webkit.org/show_bug.cgi?id=169760

Unreviewed test gardening.

  • platform/ios-simulator-wk2/TestExpectations:
  • platform/mac-wk2/TestExpectations:
10:19 AM Changeset in webkit [214048] by BJ Burg
  • 3 edits in trunk/Source/WebInspectorUI

Web Inspector: RTL: add FindBanner support for RTL layout
​https://bugs.webkit.org/show_bug.cgi?id=168723

Reviewed by Timothy Hatcher.

Due to the large number of CSS tricks used by this widget, adding RTL
support involves changing a large number of rules. The important parts:

  • Use '.segmented.{previous,next}-result' instead of '.segmented.

{left,right}', this makes the style rules way less confusing for RTL.

  • Flip icons so icons look like "< >" but actions are reversed.
  • Get rid of the weird expanding button divider. This is impossible

to mirror with the existing adjacent sibling combinator, and it does
not match the system appearance. Just replace the mini-divider with a
full divider and remove related dynamic rules that trigger on :active.

  • Always keep the button divider between next/previous on the rightmost

button, to avoid subtle differences between LTR and RTL.

  • Fix some mistakes where we weren't using --border-color and also

didn't adjust the handcoded border color for the inactive window case.

  • UserInterface/Views/FindBanner.css:

(.find-banner):
(body[dir=ltr] .find-banner > :first-child,):
(body[dir=ltr] .find-banner > :last-child,):
(.find-banner > input[type="search"]):
(body[dir=ltr] .find-banner > input[type="search"]):
(body[dir=rtl] .find-banner > input[type="search"]):
(.find-banner > button:active:not(:disabled)):
(.find-banner > button.segmented):
(.find-banner > button.segmented.previous-result):
(body[dir=ltr] .find-banner > button.segmented.previous-result):
(body[dir=rtl] .find-banner > button.segmented.previous-result):
(body[dir=ltr] .find-banner > button.segmented.previous-result > .glyph):
(body[dir=rtl] .find-banner > button.segmented.previous-result > .glyph):
(body[dir=ltr] .find-banner > button.segmented.previous-result:active:not(:disabled) > .glyph):
(body[dir=rtl] .find-banner > button.segmented.previous-result:active:not(:disabled) > .glyph):
(.find-banner > button.segmented.next-result):
(body[dir=ltr] .find-banner > button.segmented.next-result):
(body[dir=rtl] .find-banner > button.segmented.next-result):
(body[dir=ltr] .find-banner > button.segmented.next-result > .glyph):
(body[dir=rtl] .find-banner > button.segmented.next-result > .glyph):
(body[dir=ltr] .find-banner > button.segmented.next-result:active:not(:disabled) > .glyph):
(body[dir=rtl] .find-banner > button.segmented.next-result:active:not(:disabled) > .glyph):
(.find-banner.console-find-banner > input[type="search"]):
(body[dir=ltr] .find-banner.console-find-banner > input[type="search"]):
(body[dir=rtl] .find-banner.console-find-banner > input[type="search"]):
(body[dir=ltr] .find-banner.console-find-banner > input[type="search"]::-webkit-textfield-decoration-container):
(body[dir=rtl] .find-banner.console-find-banner > input[type="search"]::-webkit-textfield-decoration-container):
(.find-banner > :first-child): Deleted.
(.find-banner > :last-child): Deleted.
(.find-banner > button.segmented.left): Deleted.
(.find-banner > button:not(:active).segmented.right:before): Deleted.
(.find-banner > button.segmented.left:active + button.segmented.right,): Deleted.
(.find-banner > button.segmented.left:active + button.segmented.right:before): Deleted.
(.find-banner > button.segmented.left > .glyph): Deleted.
(.find-banner > button.segmented.left:active:not(:disabled) > .glyph): Deleted.
(.find-banner > button.segmented.right): Deleted.
(.find-banner > button.segmented.right > .glyph): Deleted.
(.find-banner > button.segmented.right:active:not(:disabled) > .glyph): Deleted.
(.find-banner.console-find-banner > input[type="search"]::-webkit-textfield-decoration-container): Deleted.

  • UserInterface/Views/FindBanner.js:

(WebInspector.FindBanner):
Change left and right to logical names for the buttons, and inline
some style classes with single uses.

10:16 AM Changeset in webkit [214047] by dbates@webkit.org
  • 3 edits
    2 adds in trunk/LayoutTests

Update test fast/events/pageshow-pagehide-on-back-cached-with-frames.html to ensure
that pageshow events are dispatched to frames in post-order traversal
<​https://bugs.webkit.org/show_bug.cgi?id=169682>

Reviewed by Brady Eidson.

  • fast/events/pageshow-pagehide-on-back-cached-with-frames-expected.txt:
  • fast/events/pageshow-pagehide-on-back-cached-with-frames.html:
  • fast/events/resources/pageshow-pagehide-subframe-cachable-2.html: Added.
  • fast/events/resources/pageshow-pagehide-subsubframe-cachable-2.html: Added.
10:12 AM Changeset in webkit [214046] by Jonathan Bedard
  • 3 edits in trunk/Tools

webktpy: Fix device regular expression for parsing simctl output
​https://bugs.webkit.org/show_bug.cgi?id=169757

Reviewed by Alex Christensen.

If a device is unavailable, the changes in r213164 mean that the
name of the device will consume the UDID and return the state as
the device UDID and the availability string as the device state.

  • Scripts/webkitpy/xcode/simulator.py:

(Simulator): Make UDID regular expression more strict.

  • Scripts/webkitpy/xcode/simulator_unittest.py: Test an unavailable device.
9:49 AM Changeset in webkit [214045] by jonlee@apple.com
  • 20 edits
    4 adds in trunk/Source

Add FIXMEs to update WebRTC
​https://bugs.webkit.org/show_bug.cgi?id=169735

Reviewed by Youenn Fablet.

Source/JavaScriptCore:

  • runtime/CommonIdentifiers.h: Add RTCIceTransport.

Source/WebCore:

Rearrange order of methods and properties based on the 13 March 2017
version of the WebRTC spec.

  • CMakeLists.txt: Add RTCIceTransport and RTCIceTransportState.
  • DerivedSources.make:
  • Modules/mediastream/RTCDTMFSender.idl:
  • Modules/mediastream/RTCDTMFToneChangeEvent.idl:
  • Modules/mediastream/RTCDataChannelEvent.idl:
  • Modules/mediastream/RTCIceCandidate.idl:
  • Modules/mediastream/RTCIceCandidateEvent.idl:
  • Modules/mediastream/RTCIceServer.idl:
  • Modules/mediastream/RTCIceTransport.cpp: Added.
  • Modules/mediastream/RTCIceTransport.idl: Added.
  • Modules/mediastream/RTCIceTransport.h: Minor update to rename state to transportState.

(WebCore::RTCIceTransport::state):
(WebCore::RTCIceTransport::setState):
(WebCore::RTCIceTransport::transportState): Deleted.
(WebCore::RTCIceTransport::setTransportState): Deleted.

  • Modules/mediastream/RTCIceTransportState.h: Added.
  • Modules/mediastream/RTCIceTransportState.idl: Added.
  • Modules/mediastream/RTCPeerConnection.h:
  • Modules/mediastream/RTCPeerConnection.idl:
  • Modules/mediastream/RTCRtpReceiver.idl:
  • Modules/mediastream/RTCRtpSender.idl:
  • Modules/mediastream/RTCRtpTransceiver.idl:
  • Modules/mediastream/RTCStatsReport.idl:
  • WebCore.xcodeproj/project.pbxproj:
  • Modules/mediastream/MediaEndpointPeerConnection.cpp:

(WebCore::MediaEndpointPeerConnection::iceTransportStateChanged): Refactor.

9:09 AM Changeset in webkit [214044] by commit-queue@webkit.org
  • 14 edits
    2 copies
    6 adds in trunk

Improve WebRTC track enabled support
​https://bugs.webkit.org/show_bug.cgi?id=169727

Patch by Youenn Fablet <​youenn@apple.com> on 2017-03-16
Reviewed by Alex Christensen.

Source/WebCore:

Tests: webrtc/peer-connection-audio-mute2.html

webrtc/peer-connection-remote-audio-mute.html
webrtc/video-remote-mute.html

Making sure muted/disabled sources produce silence/black frames.
For outgoing audio/video sources, this should be done by the actual a/v providers.
We keep this filtering here until we are sure they implement that.

  • platform/audio/mac/AudioSampleDataSource.mm:

(WebCore::AudioSampleDataSource::pullAvalaibleSamplesAsChunks): Ensuring disabled audio tracks send silence.
Used for outgoing webrtc tracks.

  • platform/mediastream/mac/MockRealtimeAudioSourceMac.mm:

(WebCore::MockRealtimeAudioSourceMac::render): Ditto.

  • platform/mediastream/mac/RealtimeIncomingAudioSource.cpp:

(WebCore::RealtimeIncomingAudioSource::OnData): Ditto.

  • platform/mediastream/mac/RealtimeIncomingVideoSource.cpp:

(WebCore::RealtimeIncomingVideoSource::pixelBufferFromVideoFrame): Generating black frames if muted.
(WebCore::RealtimeIncomingVideoSource::OnFrame):

  • platform/mediastream/mac/RealtimeIncomingVideoSource.h:
  • platform/mediastream/mac/RealtimeOutgoingVideoSource.cpp:

(WebCore::RealtimeOutgoingVideoSource::videoSampleAvailable): Ensuring we quit after sending black frame.

LayoutTests:

  • TestExpectations:
  • webrtc/audio-peer-connection-webaudio.html:
  • webrtc/peer-connection-audio-mute-expected.txt:
  • webrtc/peer-connection-audio-mute.html:
  • webrtc/peer-connection-audio-mute2-expected.txt: Added.
  • webrtc/peer-connection-audio-mute2.html: Added.
  • webrtc/peer-connection-remote-audio-mute-expected.txt: Added.
  • webrtc/peer-connection-remote-audio-mute.html: Added.
  • webrtc/video-mute-expected.txt:
  • webrtc/video-mute.html:
  • webrtc/video-remote-mute-expected.txt: Added.
  • webrtc/video-remote-mute.html: Added.
8:23 AM Changeset in webkit [214043] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

LibWebRTC outgoing source should be thread safe refcounted
​https://bugs.webkit.org/show_bug.cgi?id=169726

Patch by Youenn Fablet <​youenn@apple.com> on 2017-03-16
Reviewed by Alex Christensen.

Preventive fix.
Also fixing the size of the buffer for the audio source as its reserved size is byte count.

  • platform/mediastream/mac/RealtimeOutgoingAudioSource.h:
  • platform/mediastream/mac/RealtimeOutgoingVideoSource.h:
8:21 AM Changeset in webkit [214042] by commit-queue@webkit.org
  • 15 edits in trunk/Source/WebCore

MediaStreamTrack remote() no longer exists
​https://bugs.webkit.org/show_bug.cgi?id=169734

Patch by Youenn Fablet <​youenn@apple.com> on 2017-03-16
Reviewed by Alex Christensen.

Removing MediaStreamTrack remote.
Removing RealtimeMediaSource::remote as isCaptureSource which is closer to its use in MediaStreamPrivate to
detect whether sources are capture sources or not.

  • Modules/mediastream/MediaStream.cpp:

(WebCore::MediaStream::mediaState):

  • platform/mediastream/MediaStreamPrivate.cpp:

(WebCore::MediaStreamPrivate::hasCaptureVideoSource):
(WebCore::MediaStreamPrivate::hasCaptureAudioSource):
(WebCore::MediaStreamPrivate::hasLocalVideoSource): Deleted.
(WebCore::MediaStreamPrivate::hasLocalAudioSource): Deleted.

  • platform/mediastream/MediaStreamPrivate.h:
  • Modules/mediastream/MediaStreamTrack.cpp:

(WebCore::MediaStreamTrack::stopProducingData):
(WebCore::MediaStreamTrack::remote): Deleted.

  • Modules/mediastream/MediaStreamTrack.h:
  • Modules/mediastream/MediaStreamTrack.idl:
  • Modules/webaudio/MediaStreamAudioSource.h:
  • platform/mediastream/MediaStreamPrivate.cpp:

(WebCore::MediaStreamPrivate::hasLocalVideoSource):
(WebCore::MediaStreamPrivate::hasLocalAudioSource):

  • platform/mediastream/MediaStreamTrackPrivate.cpp:

(WebCore::MediaStreamTrackPrivate::isCaptureTrack):
(WebCore::MediaStreamTrackPrivate::remote): Deleted.

  • platform/mediastream/MediaStreamTrackPrivate.h:
  • platform/mediastream/RealtimeMediaSource.cpp:

(WebCore::RealtimeMediaSource::reset):

  • platform/mediastream/RealtimeMediaSource.h:

(WebCore::RealtimeMediaSource::isCaptureSource):
(WebCore::RealtimeMediaSource::remote): Deleted.
(WebCore::RealtimeMediaSource::setRemote): Deleted.

  • platform/mediastream/mac/AVMediaCaptureSource.h:
  • platform/mock/MockRealtimeAudioSource.h:
  • platform/mock/MockRealtimeVideoSource.h:
6:52 AM Changeset in webkit [214041] by Yusuke Suzuki
  • 2 edits in trunk/Source/JavaScriptCore

Unreviewed, copy m_numberOfArgumentsToSkip
​https://bugs.webkit.org/show_bug.cgi?id=164582

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::CodeBlock):

6:24 AM Changeset in webkit [214040] by Yusuke Suzuki
  • 4 edits in trunk/Source/JavaScriptCore

Unreviewed, fix numParameter() - 1 OSRExit materialization
​https://bugs.webkit.org/show_bug.cgi?id=164582

When materializing rest parameters, we rely on that numParameter() - 1 equals to
the numberOfArgumentsToSkip. But this assumption is broken in r214029.

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::finishCreation):

  • bytecode/CodeBlock.h:

(JSC::CodeBlock::numberOfArgumentsToSkip):

  • ftl/FTLOperations.cpp:

(JSC::FTL::operationMaterializeObjectInOSR):

6:13 AM Changeset in webkit [214039] by Manuel Rego Casasnovas
  • 3 edits
    2 adds in trunk

[css-grid] Crash on debug removing a positioned child
​https://bugs.webkit.org/show_bug.cgi?id=169739

Reviewed by Sergio Villar Senin.

Source/WebCore:

When we add or remove a positioned item we don't need to mark
the grid as dirty, because positioned items do not affect the layout
of the grid at all.

This was causing a crash when a positioned item was removed
after a layout. As after the positioned item was removed,
the method RenderGrid::layoutBlock() was not called,
so when the grid was repainted we got a crash.

Test: fast/css-grid-layout/grid-crash-remove-positioned-item.html

  • rendering/RenderGrid.cpp:

(WebCore::RenderGrid::addChild): Add early return to avoid marking
the grid as dirty for positioned grid items.
(WebCore::RenderGrid::removeChild): Ditto.

LayoutTests:

Add new test that checks that adding and removing a positioned grid item
doesn't cause any crashes.

  • fast/css-grid-layout/grid-crash-remove-positioned-item-expected.txt: Added.
  • fast/css-grid-layout/grid-crash-remove-positioned-item.html: Added.
5:58 AM Changeset in webkit [214038] by commit-queue@webkit.org
  • 19 edits
    1 add in trunk

[ESnext] Implement Object Spread
​https://bugs.webkit.org/show_bug.cgi?id=167963

Patch by Caio Lima <Caio Lima> on 2017-03-16
Reviewed by Yusuke Suzuki.

JSTests:

  • stress/object-spread.js: Added.

(let.assert):
(assert.sameValue):
(let.o.get a):
(let.obj.get c):
(cthulhu.get x):
(let.obj.set c):
(calls.o.get z):
(calls.o.get a):
(try.let.obj.get foo):
(get calls):

Source/JavaScriptCore:

This patch implements ECMA262 stage 3 Object Spread proposal [1].
It's implemented using CopyDataProperties to copy all enumerable keys
from object being spreaded.

It's also fixing CopyDataProperties that was using
Object.getOwnPropertyNames to list all keys to be copied, and now is
using Relect.ownKeys.

[1] - ​https://github.com/sebmarkbage/ecmascript-rest-spread

  • builtins/GlobalOperations.js:

(globalPrivate.copyDataProperties):

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::setConstantIdentifierSetRegisters):

  • bytecode/UnlinkedCodeBlock.h:

(JSC::UnlinkedCodeBlock::addSetConstant):

  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::emitLoad):

  • bytecompiler/BytecodeGenerator.h:
  • bytecompiler/NodesCodegen.cpp:

(JSC::PropertyListNode::emitBytecode):
(JSC::ObjectPatternNode::bindValue):
(JSC::ObjectSpreadExpressionNode::emitBytecode):

  • parser/ASTBuilder.h:

(JSC::ASTBuilder::createObjectSpreadExpression):
(JSC::ASTBuilder::createProperty):

  • parser/NodeConstructors.h:

(JSC::PropertyNode::PropertyNode):
(JSC::ObjectSpreadExpressionNode::ObjectSpreadExpressionNode):

  • parser/Nodes.h:

(JSC::ObjectSpreadExpressionNode::expression):

  • parser/Parser.cpp:

(JSC::Parser<LexerType>::parseProperty):

  • parser/SyntaxChecker.h:

(JSC::SyntaxChecker::createObjectSpreadExpression):
(JSC::SyntaxChecker::createProperty):

  • runtime/JSGlobalObject.cpp:

(JSC::JSGlobalObject::init):

  • runtime/JSGlobalObjectFunctions.cpp:

(JSC::privateToObject): Deleted.

  • runtime/JSGlobalObjectFunctions.h:

LayoutTests:

  • js/parser-syntax-check-expected.txt:
  • js/script-tests/parser-syntax-check.js:
4:59 AM Changeset in webkit [214037] by clopez@igalia.com
  • 5 edits in trunk

[WebRTC] SDP sess-id in the "o=" line should be a value between 0 and LLONG_MAX.
​https://bugs.webkit.org/show_bug.cgi?id=169681

Reviewed by Alejandro G. Castro.

Source/WebCore:

Use an int64_t type for storing sess-id and ensure that the value generated is always between 0 and LLONG_MAX (263-1).
To the JS world (sdp.js) we pass and read this value as a string type.

The JS world (sdp.js) was already using a string type for sess-id.
This caused a bug when reading the sess-id value at configurationFromJSON(): no value was obtained because a numeric type was expected.

So, after this patch, sess-id is passed and read as a string from sdp.js, and converted to an int64_t type for internal use.
This way we also avoid a rounding problem that happens with values near LLONG_MAX when converting from int64_t to double when passed to the JS world,
and that could cause that we end passing a number bigger than LLONG_MAX to sdp.js.

  • Modules/mediastream/SDPProcessor.cpp:

(WebCore::configurationFromJSON):
(WebCore::configurationToJSON):

  • platform/mediastream/MediaEndpointSessionConfiguration.h:

(WebCore::MediaEndpointSessionConfiguration::sessionId):
(WebCore::MediaEndpointSessionConfiguration::setSessionId):
(WebCore::MediaEndpointSessionConfiguration::MediaEndpointSessionConfiguration):

LayoutTests:

Check that the generated sess-id value its between the limits before validating it.
This is covered by the tests fast/mediastream/RTCPeerConnection-inspect-answer.html
and fast/mediastream/RTCPeerConnection-inspect-offer.html.
Both tests will fail if the sess-id value generated is not within the limits.

  • fast/mediastream/resources/sdp-utils.js:

(printComparableSessionDescription):

4:55 AM Changeset in webkit [214036] by Carlos Garcia Campos
  • 3 edits
    1 copy
    1 add
    2 deletes in trunk/Source/WTF

[UNIX] Implement currentSearchLocaleID() and currentTextBreakLocaleID()
​https://bugs.webkit.org/show_bug.cgi?id=169745

Reviewed by Yusuke Suzuki.

Add a common implementation for Unix based ports using setlocale.

  • wtf/PlatformGTK.cmake:
  • wtf/PlatformJSCOnly.cmake:
  • wtf/text/gtk/TextBreakIteratorInternalICUGtk.cpp: Removed.
  • wtf/text/unix/TextBreakIteratorInternalICUUnix.cpp: Renamed from Source/WTF/wtf/text/jsconly/TextBreakIteratorInternalICUJSCOnly.cpp.

(WTF::currentSearchLocaleID):
(WTF::currentTextBreakLocaleID):

2:57 AM Changeset in webkit [214035] by Carlos Garcia Campos
  • 3 edits in trunk/Source/WTF

[GLIB] MainThread implementation is not GTK+ specific
​https://bugs.webkit.org/show_bug.cgi?id=169736

Reviewed by Yusuke Suzuki.

It's implemented in MainThreadGLib.cpp for glib based ports, so ifdefs in MainThread should not be
PLATFORM(GTK) but USE(GLIB).

  • wtf/MainThread.cpp:
  • wtf/MainThread.h:
1:40 AM Changeset in webkit [214034] by tpopela@redhat.com
  • 4 edits in trunk/Source/WebKit2

Fix uninitialized public members in Webkit2/UIProcess/API/gtk
​https://bugs.webkit.org/show_bug.cgi?id=169602

Reviewed by Michael Catanzaro.

Found by Coverity scan.

  • UIProcess/API/gtk/WebKitMimeInfo.cpp:
  • UIProcess/API/gtk/WebKitSettings.cpp:
  • UIProcess/API/gtk/WebKitWebViewBase.cpp:
12:51 AM Changeset in webkit [214033] by zandobersek@gmail.com
  • 2 edits in trunk/Source/WebKit2

Unreviewed follow-up to r213787. Move the DatabaseProcess build targets
to the top of WebKit2_SOURCES list in order to maintain the alphabetical
order.

  • CMakeLists.txt:

Mar 15, 2017:

11:48 PM Changeset in webkit [214032] by achristensen@apple.com
  • 2 edits in trunk/Source/WebKit2

Fix CMake build.

  • PlatformMac.cmake:
11:29 PM Changeset in webkit [214031] by Dewei Zhu
  • 3 edits in trunk/Websites/perf.webkit.org

Fix unit test and bug fix for 'pull-os-versions.js' script.
​https://bugs.webkit.org/show_bug.cgi?id=169701

Reviewed by Ryosuke Niwa.

Fix unit tests warnings on node-6.10.0.
Fix 'pull-os-versions.js' does not fetch new builds and report.

  • server-tests/tools-os-build-fetcher-tests.js:

(then):
(beforeEach):
(afterEach):

  • tools/pull-os-versions.js:

(syncLoop):

11:10 PM Changeset in webkit [214030] by jonlee@apple.com
  • 23 edits
    6 adds in trunk/Source/WebCore

Flatten RTC enum naming
​https://bugs.webkit.org/show_bug.cgi?id=169664

Reviewed by Youenn Fablet.

Use consistent names of RTC enums throughout WebCore. This means surfacing
ICE enums out of PeerConnectionState. Keep the old names around for other
ports.

Add RTCIceConnectionState, RTCIceGatheringState, and RTCSignalingState enums.

  • CMakeLists.txt:
  • DerivedSources.make:
  • Modules/mediastream/RTCIceConnectionState.h: Added. The enum is defined in

PeerConnectionStates.h, so just include that file.

  • Modules/mediastream/RTCIceConnectionState.idl: Added.
  • Modules/mediastream/RTCIceGatheringState.h: Added.
  • Modules/mediastream/RTCIceGatheringState.idl: Added.
  • Modules/mediastream/RTCSignalingState.h: Added.
  • Modules/mediastream/RTCSignalingState.idl: Added.
  • WebCore.xcodeproj/project.pbxproj:
  • platform/mediastream/PeerConnectionStates.h: Move the existing enums into

WebCore, but keep aliases to the old names within the PeerConnectionStates
namespace.

Refactor to use the new enum names.

  • Modules/mediastream/MediaEndpointPeerConnection.cpp: Refactor.
  • Modules/mediastream/MediaEndpointPeerConnection.h:
  • Modules/mediastream/PeerConnectionBackend.cpp:
  • Modules/mediastream/PeerConnectionBackend.h:
  • Modules/mediastream/RTCConfiguration.h:
  • Modules/mediastream/RTCConfiguration.idl: Add FIXMEs for bringing this up

to spec.

  • Modules/mediastream/RTCIceTransport.h:
  • Modules/mediastream/RTCPeerConnection.cpp: Refactor the three functions

below to using the enum instead of returning strings. This allows remove of
the internal* versions of these functions.
(WebCore::RTCPeerConnection::signalingState): Deleted.
(WebCore::RTCPeerConnection::iceGatheringState): Deleted.
(WebCore::RTCPeerConnection::iceConnectionState): Deleted.

  • Modules/mediastream/RTCPeerConnection.h: Replace internalSignalingState,

internalIceGatheringState, and internalIceConnectionState.

  • Modules/mediastream/RTCPeerConnection.idl:
  • Modules/mediastream/libwebrtc/LibWebRTCMediaEndpoint.cpp:
  • Modules/mediastream/libwebrtc/LibWebRTCPeerConnectionBackend.cpp:
  • platform/mediastream/MediaEndpoint.h:
  • platform/mediastream/MediaEndpointConfiguration.cpp:
  • platform/mediastream/MediaEndpointConfiguration.h:
  • platform/mock/MockMediaEndpoint.cpp:
  • platform/mock/MockMediaEndpoint.h:
  • platform/mediastream/openwebrtc/MediaEndpointOwr.cpp:

(WebCore::MediaEndpointOwr::processIceTransportStateChange):

10:12 PM Changeset in webkit [214029] by Yusuke Suzuki
  • 16 edits
    2 adds in trunk

[JSC] Default parameter part should be retrieved by op_get_argument opcode instead of changing arity
​https://bugs.webkit.org/show_bug.cgi?id=164582

Reviewed by Saam Barati.

JSTests:

  • stress/function-with-defaults-inlining.js: Added.

(shouldBe):
(ok):
(a):

  • stress/function-with-defaults-non-inlining.js: Added.

(shouldBe):
(ok):
(a):

Source/JavaScriptCore:

Previously we implement the default parameters as follows.

  1. We count the default parameters as the usual parameters.
  2. We just get the argument register.
  3. Check it with op_is_undefined.
  4. And fill the binding with either the argument register or default value.

The above is simple. However, it has the side effect that it always increase the arity of the function.
While function.length does not increase, internally, the number of parameters of CodeBlock increases.
This effectively prevent our DFG / FTL to perform inlining: currently we only allows DFG to inline
the function with the arity less than or equal the number of passing arguments. It is OK. But when using
default parameters, we frequently do not pass the argument for the parameter with the default value.
Thus, in our current implementation, we frequently need to fixup the arity. And we frequently fail
to inline the function.

This patch fixes the above problem by not increasing the arity of the function. When we encounter the
parameter with the default value, we use op_argument to get the argument instead of using the argument
registers.

This improves six-speed defaults.es6 performance by 4.45x.

defaults.es6 968.4126+-101.2350 217.6602+-14.8831 definitely 4.4492x faster

  • bytecode/UnlinkedFunctionExecutable.cpp:

(JSC::UnlinkedFunctionExecutable::UnlinkedFunctionExecutable):

  • bytecode/UnlinkedFunctionExecutable.h:
  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::BytecodeGenerator):
(JSC::BytecodeGenerator::initializeDefaultParameterValuesAndSetupFunctionScopeStack):
(JSC::BytecodeGenerator::initializeNextParameter):
(JSC::BytecodeGenerator::initializeParameters):

  • bytecompiler/BytecodeGenerator.h:
  • bytecompiler/NodesCodegen.cpp:

(JSC::FunctionNode::emitBytecode):

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::inliningCost):

  • parser/ASTBuilder.h:

(JSC::ASTBuilder::createFunctionMetadata):

  • parser/Nodes.cpp:

(JSC::FunctionMetadataNode::FunctionMetadataNode):

  • parser/Nodes.h:

(JSC::FunctionParameters::size):
(JSC::FunctionParameters::at):
(JSC::FunctionParameters::append):
(JSC::FunctionParameters::isSimpleParameterList):

  • parser/Parser.cpp:

(JSC::Parser<LexerType>::isArrowFunctionParameters):
(JSC::Parser<LexerType>::parseGeneratorFunctionSourceElements):
(JSC::Parser<LexerType>::parseAsyncFunctionSourceElements):
(JSC::Parser<LexerType>::parseFormalParameters):
(JSC::Parser<LexerType>::parseFunctionBody):
(JSC::Parser<LexerType>::parseFunctionParameters):
(JSC::Parser<LexerType>::parseFunctionInfo):

  • parser/Parser.h:
  • parser/SyntaxChecker.h:

(JSC::SyntaxChecker::createFunctionMetadata):

  • runtime/FunctionExecutable.h:
  • runtime/JSFunction.cpp:

(JSC::JSFunction::createBuiltinFunction):
(JSC::JSFunction::reifyLength):

9:49 PM Changeset in webkit [214028] by Yusuke Suzuki
  • 10 edits
    2 adds in trunk

[DFG] ToString operation should have fixup for primitives to say this node does not have side effects
​https://bugs.webkit.org/show_bug.cgi?id=169544

Reviewed by Saam Barati.

JSTests:

  • microbenchmarks/template-string-array.js: Added.

(test):

  • stress/to-string-non-cell-use.js: Added.

(shouldBe):
(shouldThrow):

Source/JavaScriptCore:

Our DFG ToString only considers well about String operands. While ToString(non cell operand) does not have
any side effect, it is not modeled well in DFG.

This patch introduces a fixup for ToString with NonCellUse edge. If this edge is set, ToString does not
clobber things (like ToLowerCase, producing String). And ToString(NonCellUse) allows us to perform CSE!

Our microbenchmark shows 32.9% improvement due to dropped GetButterfly and CSE for ToString().

baseline patched

template-string-array 12.6284+-0.2766 9.4998+-0.2295 definitely 1.3293x faster

And SixSpeed template_string.es6 shows 16.68x performance improvement due to LICM onto this non-side-effectful ToString().

baseline patched

template_string.es6 3229.7343+-40.5705 193.6077+-36.3349 definitely 16.6818x faster

  • dfg/DFGAbstractInterpreterInlines.h:

(JSC::DFG::AbstractInterpreter<AbstractStateType>::executeEffects):

  • dfg/DFGClobberize.h:

(JSC::DFG::clobberize):

  • dfg/DFGFixupPhase.cpp:

(JSC::DFG::FixupPhase::fixupToStringOrCallStringConstructor):

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compileToStringOrCallStringConstructorOnCell):
(JSC::DFG::SpeculativeJIT::speculateNotCell):

  • dfg/DFGSpeculativeJIT.h:
  • dfg/DFGSpeculativeJIT32_64.cpp:

(JSC::DFG::SpeculativeJIT::compile):

  • dfg/DFGSpeculativeJIT64.cpp:

(JSC::DFG::SpeculativeJIT::compile):

  • ftl/FTLLowerDFGToB3.cpp:

(JSC::FTL::DFG::LowerDFGToB3::compileToStringOrCallStringConstructor):
(JSC::FTL::DFG::LowerDFGToB3::lowNotCell):
(JSC::FTL::DFG::LowerDFGToB3::speculateNotCell):

9:10 PM Changeset in webkit [214027] by jer.noble@apple.com
  • 17 edits
    1 copy
    5 adds
    1 delete in trunk/Source

Optionally capture audio in the UIProcess
​https://bugs.webkit.org/show_bug.cgi?id=169609

Reviewed by Alex Christensen.

Source/WebCore:

Export some previously un-exported symbols from WebCore for use in WebKit2.

  • WebCore.xcodeproj/project.pbxproj:
  • platform/audio/WebAudioBufferList.h:
  • platform/mediastream/RealtimeMediaSource.h:

Source/WebKit2:

Add a new class pair, UserMediaCaptureManager/Proxy, to allow a RealtimeMediaSource
requested in a WebProcess to be created in the UIProcess and push its audio data across the
process boundary to its clients in the WebProcess. Because these classes are clients of the
RealtimeMediaSourceCenter::singleton(), they must be a singleton in the WebProcess as well,
so they are attached to the WebProcess class in the web process and to WebProcessProxy in
the UIProcess.

  • UIProcess/Cocoa/UserMediaCaptureManagerProxy.cpp: Added.

(WebKit::UserMediaCaptureManagerProxy::SourceProxy::SourceProxy):
(WebKit::UserMediaCaptureManagerProxy::SourceProxy::~SourceProxy):
(WebKit::UserMediaCaptureManagerProxy::SourceProxy::source):
(WebKit::UserMediaCaptureManagerProxy::SourceProxy::description):
(WebKit::UserMediaCaptureManagerProxy::SourceProxy::numberOfFrames):
(WebKit::UserMediaCaptureManagerProxy::UserMediaCaptureManagerProxy):
(WebKit::UserMediaCaptureManagerProxy::~UserMediaCaptureManagerProxy):
(WebKit::UserMediaCaptureManagerProxy::createMediaSourceForCaptureDeviceWithConstraints):
(WebKit::UserMediaCaptureManagerProxy::startProducingData):
(WebKit::UserMediaCaptureManagerProxy::stopProducingData):

  • UIProcess/Cocoa/UserMediaCaptureManagerProxy.h:

(WebKit::UserMediaCaptureManagerProxy::process):

  • UIProcess/Cocoa/UserMediaCaptureManagerProxy.messages.in: Added.
  • WebProcess/cocoa/UserMediaCaptureManager.cpp: Added.

(WebKit::nextSessionID):
(WebKit::UserMediaCaptureManager::Source::Source):
(WebKit::UserMediaCaptureManager::Source::~Source):
(WebKit::UserMediaCaptureManager::Source::setCapabilities):
(WebKit::UserMediaCaptureManager::Source::setSettings):
(WebKit::UserMediaCaptureManager::Source::description):
(WebKit::UserMediaCaptureManager::Source::setStorage):
(WebKit::UserMediaCaptureManager::Source::setRingBufferFrameBounds):
(WebKit::UserMediaCaptureManager::Source::audioSamplesAvailable):
(WebKit::UserMediaCaptureManager::UserMediaCaptureManager):
(WebKit::UserMediaCaptureManager::~UserMediaCaptureManager):
(WebKit::UserMediaCaptureManager::supplementName):
(WebKit::UserMediaCaptureManager::initialize):
(WebKit::UserMediaCaptureManager::createMediaSourceForCaptureDeviceWithConstraints):
(WebKit::UserMediaCaptureManager::sourceStopped):
(WebKit::UserMediaCaptureManager::sourceMutedChanged):
(WebKit::UserMediaCaptureManager::sourceEnabledChanged):
(WebKit::UserMediaCaptureManager::sourceSettingsChanged):
(WebKit::UserMediaCaptureManager::storageChanged):
(WebKit::UserMediaCaptureManager::ringBufferFrameBoundsChanged):
(WebKit::UserMediaCaptureManager::audioSamplesAvailable):
(WebKit::UserMediaCaptureManager::startProducingData):
(WebKit::UserMediaCaptureManager::stopProducingData):

  • WebProcess/cocoa/UserMediaCaptureManager.h: Added.
  • WebProcess/cocoa/UserMediaCaptureManager.messages.in: Added.

Initialize UserMediaCaptureManager/Proxy:

  • UIProcess/Cocoa/WebProcessPoolCocoa.mm:

(WebKit::WebProcessPool::platformInitializeWebProcess):

  • UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::WebProcessProxy):

  • UIProcess/WebProcessProxy.h:
  • WebProcess/WebProcess.cpp:

(WebKit::WebProcess::WebProcess):

WebUserMediaClientMac is no longer necessary now that the UserMediaCaptureManager overrides
the factories:

  • WebProcess/WebCoreSupport/WebUserMediaClient.cpp:

(WebKit::WebUserMediaClient::WebUserMediaClient):
(WebKit::WebUserMediaClient::initializeFactories): Deleted.

Add a new preference and WebProcessCreationParameters member to control whether
capturing is done in the UIProcess:

  • Shared/WebPreferencesDefinitions.h:
  • Shared/WebProcessCreationParameters.cpp:

(WebKit::WebProcessCreationParameters::encode):
(WebKit::WebProcessCreationParameters::decode):

  • Shared/WebProcessCreationParameters.h:

Add new files to the project:

  • DerivedSources.make:
  • WebKit2.xcodeproj/project.pbxproj:
8:41 PM Changeset in webkit [214026] by Ryan Haddad
  • 2 edits in trunk/Source/JavaScriptCore

Revert part of r213978 to see if it resolves LayoutTest crashes.
​https://bugs.webkit.org/show_bug.cgi?id=169729

Reviewed by Alexey Proskuryakov.

8:15 PM Changeset in webkit [214025] by clopez@igalia.com
  • 3 edits in trunk/Source/WTF

[CMake][JSCOnly] Fix build with GLib event loop
​https://bugs.webkit.org/show_bug.cgi?id=169730

Reviewed by Michael Catanzaro.

  • wtf/MainThread.cpp:
  • wtf/PlatformJSCOnly.cmake: WorkQueueGLib was removed in r199713.
7:48 PM Changeset in webkit [214024] by webkit@devinrousso.com
  • 2 edits in trunk/Source/WebInspectorUI

Web Inspector: Selecting text in the console does not do what I expect
​https://bugs.webkit.org/show_bug.cgi?id=169570

Reviewed by Matt Baker.

  • UserInterface/Views/ConsoleMessageView.css:

(.console-message-text > span > :matches(.console-message-enclosed, .console-message-preview, .console-message-preview-divider)):
(.console-message .console-message-location):
(.console-message-location.call-frame):
Prevent text selection on anything other than the message text. The other information that
is not selectable can still be copied by selecting the message itself and copying.

6:56 PM Changeset in webkit [214023] by Alan Bujtas
  • 5 edits
    2 adds in trunk

Do not reparent floating object until after intruding/overhanging dependency is cleared.
​https://bugs.webkit.org/show_bug.cgi?id=169711
<rdar://problem/30959743>

Reviewed by Simon Fraser.

Source/WebCore:

This patch ensures that we cleanup the m_floatingObjects for siblings before reparenting the fresh float.

Test: fast/block/float/inline-becomes-float-and-moves-around.html

  • rendering/RenderBlockFlow.cpp:

(WebCore::RenderBlockFlow::styleDidChange):

  • rendering/RenderElement.cpp:

(WebCore::RenderElement::styleDidChange):

  • rendering/RenderElement.h:

(WebCore::RenderElement::noLongerAffectsParentBlock):

LayoutTests:

  • fast/block/float/inline-becomes-float-and-moves-around-expected.txt: Added.
  • fast/block/float/inline-becomes-float-and-moves-around.html: Added.
6:01 PM Changeset in webkit [214022] by commit-queue@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

[jsc][mips] Fix compilation error introduced in r213652
​https://bugs.webkit.org/show_bug.cgi?id=169723

Patch by Guillaume Emont <​guijemont@igalia.com> on 2017-03-15
Reviewed by Mark Lam.

The new replaceWithBkpt() contains a lapsus in it
(s/code/instructionStart) and won't compile.

  • assembler/MIPSAssembler.h:

(JSC::MIPSAssembler::replaceWithBkpt):

6:01 PM Changeset in webkit [214021] by Nikita Vasilyev
  • 3 edits in trunk/Source/WebInspectorUI

Web Inspector: WebSockets: Update Arrow Up icon to fit in with the rest of our iconography
​https://bugs.webkit.org/show_bug.cgi?id=169696
<rdar://problem/31073748>

Reviewed by Matt Baker.

  • UserInterface/Images/ArrowUp.svg:
  • UserInterface/Images/gtk/ArrowUp.svg:
4:40 PM Changeset in webkit [214020] by commit-queue@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

Switch back to ISO 4217 for Intl CurrencyDigits data
​https://bugs.webkit.org/show_bug.cgi?id=169182

Previously, a patch switched Intl.NumberFormat to use CLDR data through
ICU to get the default number of decimal digits for a currency.
However, that change actually violated the ECMA 402 specification,
which references ISO 4217 as the data source. This patch reverts to
an in-line implementation of that data.

Patch by Daniel Ehrenberg <​littledan@chromium.org> on 2017-03-15
Reviewed by Saam Barati.

  • runtime/IntlNumberFormat.cpp:

(JSC::computeCurrencySortKey):
(JSC::extractCurrencySortKey):
(JSC::computeCurrencyDigits):

4:29 PM Changeset in webkit [214019] by timothy_horton@apple.com
  • 4 edits
    1 add in trunk

Null deref under callAfterNextPresentationUpdate
​https://bugs.webkit.org/show_bug.cgi?id=169710
<rdar://problem/30987863>

Reviewed by Simon Fraser.

Source/WebKit2:

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::callAfterNextPresentationUpdate):
Call the callback with an error if we don't have a web process or drawing area.

Tools:

  • TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
  • TestWebKitAPI/Tests/WebKit2Cocoa/DoAfterNextPresentationUpdateAfterCrash.mm: Added.

(TEST):

4:17 PM Changeset in webkit [214018] by sbarati@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

WebAssembly: When we GC to try to get a fast memory, we should call collectAllGarbage(), not collectSync()
​https://bugs.webkit.org/show_bug.cgi?id=169704

Reviewed by Mark Lam.

We weren't always sweeping the memory needed to free
the WasmMemory we wanted to use. collectAllGarbage()
will do this if the JS objects wrapping WasmMemory
are dead.

This patch also moves the increment of the allocatedFastMemories
integer to be thread safe.

  • wasm/WasmMemory.cpp:

(JSC::Wasm::tryGetFastMemory):

3:56 PM Changeset in webkit [214017] by dino@apple.com
  • 10 edits
    1 add in trunk/Source/WebCore

Make a base class for WebGL and WebGPU contexts
​https://bugs.webkit.org/show_bug.cgi?id=169651
<rdar://problem/31053489>

Reviewed by Simon Fraser.

Add a new pure virtual base class, GPUBasedCanvasRenderingContext, that
will be used by WebGL and WebGPU rendering contexts. This allows us
to avoid some code duplication, since many places treat the two
as the same.

Also rename is3d() -> isWebGL() and isGPU() -> isWebGPU().

  • WebCore.xcodeproj/project.pbxproj: New file.
  • html/HTMLCanvasElement.cpp:

(WebCore::HTMLCanvasElement::getContextWebGL):
(WebCore::HTMLCanvasElement::getContextWebGPU):
(WebCore::HTMLCanvasElement::reset):
(WebCore::HTMLCanvasElement::paint):
(WebCore::HTMLCanvasElement::isGPUBased):
(WebCore::HTMLCanvasElement::getImageData):
(WebCore::HTMLCanvasElement::isGPU): Deleted.
(WebCore::HTMLCanvasElement::is3D): Deleted.

  • html/HTMLCanvasElement.h:
  • html/canvas/CanvasRenderingContext.h:

(WebCore::CanvasRenderingContext::isWebGL):
(WebCore::CanvasRenderingContext::isWebGPU):
(WebCore::CanvasRenderingContext::isGPUBased):
(WebCore::CanvasRenderingContext::is3d): Deleted.
(WebCore::CanvasRenderingContext::isGPU): Deleted.

  • html/canvas/GPUBasedCanvasRenderingContext.h: Added.

(WebCore::GPUBasedCanvasRenderingContext::GPUBasedCanvasRenderingContext):

  • html/canvas/WebGLRenderingContextBase.cpp:

(WebCore::WebGLRenderingContextBase::WebGLRenderingContextBase):

  • html/canvas/WebGLRenderingContextBase.h:
  • html/canvas/WebGPURenderingContext.cpp:

(WebCore::WebGPURenderingContext::WebGPURenderingContext):

  • html/canvas/WebGPURenderingContext.h:
  • rendering/RenderLayerBacking.cpp:

(WebCore::canvasCompositingStrategy):

3:49 PM Changeset in webkit [214016] by mark.lam@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Fix exception scope verification failures in jsc.cpp.
​https://bugs.webkit.org/show_bug.cgi?id=164968

Reviewed by Saam Barati.

  • jsc.cpp:

(WTF::CustomGetter::customGetter):

(GlobalObject::moduleLoaderResolve):
(GlobalObject::moduleLoaderFetch):

  • The only way modules would throw an exception is if we encounter an OutOfMemory error. This should be extremely rare. At this point, I don't think it's worth doing the dance to propagate the exception when this happens. Instead, we'll simply do a RELEASE_ASSERT that we don't see any exceptions here.

(functionRun):
(functionRunString):
(functionLoadModule):
(functionCheckModuleSyntax):
(box):
(dumpException):
(runWithScripts):

3:46 PM Changeset in webkit [214015] by matthew_hanson@apple.com
  • 1 copy in tags/Safari-604.1.30.4.5

Tag Safari-604.1.30.4.5.

3:44 PM Changeset in webkit [214014] by dbates@webkit.org
  • 6 edits in trunk/Source/WebCore

Iteratively dispatch DOM events after restoring a cached page
​https://bugs.webkit.org/show_bug.cgi?id=169703
<rdar://problem/31075903>

Reviewed by Brady Eidson.

Make dispatching of DOM events when restoring a page from the page cache symmetric with
dispatching of events when saving a page to the page cache.

  • history/CachedFrame.cpp:

(WebCore::CachedFrameBase::restore): Move code to dispatch events from here to FrameLoader::didRestoreFromCachedPage().

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::commitProvisionalLoad): Ensure that no DOM events are dispatched during
restoration of a cached page. Call didRestoreFromCachedPage() after restoring the page to
dispatch DOM events on the restored frames.
(WebCore::FrameLoader::willRestoreFromCachedPage): Renamed; formerly named prepareForCachedPageRestore().
(WebCore::FrameLoader::didRestoreFromCachedPage): Added.
(WebCore::FrameLoader::prepareForCachedPageRestore): Renamed to willRestoreFromCachedPage().

  • loader/FrameLoader.h:
  • page/FrameTree.cpp:

(WebCore::FrameTree::traverseNextInPostOrderWithWrap): Returns the next Frame* in a post-order
traversal of the frame tree optionally wrapping around to the deepest first child in the tree.
(WebCore::FrameTree::deepFirstChild): Added.

  • page/FrameTree.h:
3:43 PM Changeset in webkit [214013] by matthew_hanson@apple.com
  • 2 edits in branches/safari-603.1.30.4-branch/Source/WebCore

Merge r213847. rdar://problem/30983702

3:43 PM Changeset in webkit [214012] by matthew_hanson@apple.com
  • 3 edits in branches/safari-603.1.30.4-branch/Source/WebKit/win

Merge r213737. rdar://problem/30983702

3:29 PM Changeset in webkit [214011] by webkit@devinrousso.com
  • 3 edits
    4 adds in trunk/Source/WebInspectorUI

Web Inspector: Add icons for SVG Image cluster path components
​https://bugs.webkit.org/show_bug.cgi?id=169687

Reviewed by Joseph Pecoraro.

  • UserInterface/Images/Image.svg: Added.
  • UserInterface/Images/Source.svg: Added.
  • UserInterface/Images/gtk/Image.svg: Added.
  • UserInterface/Images/gtk/Source.svg: Added.
  • UserInterface/Views/PathComponentIcons.css:

(.image-icon .icon):
(.source-icon .icon):

  • UserInterface/Views/SVGImageResourceClusterContentView.js:

(WebInspector.SVGImageResourceClusterContentView):

3:09 PM Changeset in webkit [214010] by hyatt@apple.com
  • 10 edits
    2 adds in trunk

Positioned SVG not sized correctly
​https://bugs.webkit.org/show_bug.cgi?id=169693
<rdar://problem/30996893>

Reviewed by Zalan Bujtas.

Source/WebCore:

Test: svg/in-html/rect-positioned.html

Change computeReplacedLogicalHeight to take an estimatedUsedWidth. This
value is used instead of the available logical width to resolve replaced
elements without intrinsic sizes but with aspect ratios set.

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::computeReplacedLogicalHeight):

  • rendering/RenderBox.h:
  • rendering/RenderReplaced.cpp:

(WebCore::RenderReplaced::computeConstrainedLogicalWidth):
(WebCore::RenderReplaced::computeReplacedLogicalWidth):
(WebCore::RenderReplaced::computeReplacedLogicalHeight):

  • rendering/RenderReplaced.h:
  • rendering/RenderVideo.cpp:

(WebCore::RenderVideo::computeReplacedLogicalHeight): Deleted.

  • rendering/RenderVideo.h:
  • rendering/svg/RenderSVGRoot.cpp:

(WebCore::RenderSVGRoot::computeReplacedLogicalWidth):
(WebCore::RenderSVGRoot::computeReplacedLogicalHeight):

  • rendering/svg/RenderSVGRoot.h:

LayoutTests:

  • svg/in-html/rect-positioned-expected.html: Added.
  • svg/in-html/rect-positioned.html: Added.
3:04 PM Changeset in webkit [214009] by webkit@devinrousso.com
  • 2 edits in trunk/Source/WebInspectorUI

Web Inspector: RTL: flip Memory timelines
​https://bugs.webkit.org/show_bug.cgi?id=169694

Reviewed by Brian Burg.

  • UserInterface/Views/MemoryCategoryView.css:

(body[dir=rtl] .memory-category-view > .graph):

2:59 PM Changeset in webkit [214008] by rniwa@webkit.org
  • 9 edits
    1 add in trunk/Websites/perf.webkit.org

In-browser and node.js implementations of RemoteAPI should share some code
​https://bugs.webkit.org/show_bug.cgi?id=169695

Rubber-stamped by Antti Koivisto.

Extracted CommonRemoteAPI out of RemoteAPI implementations for node.js and browser.

  • public/shared/common-remote.js: Added.

(CommonRemoteAPI): Added.
(CommonRemoteAPI.prototype.postJSON): Extracted from RemoteAPI.
(CommonRemoteAPI.prototype.postJSONWithStatus): Ditto.
(CommonRemoteAPI.prototype.getJSON): Ditto.
(CommonRemoteAPI.prototype.getJSONWithStatus): Ditto.
(CommonRemoteAPI.prototype.sendHttpRequest): Added. Needs to implemented by a subclass.
(CommonRemoteAPI.prototype._asJSON): Added.
(CommonRemoteAPI.prototype._checkStatus): Added.

  • public/v3/index.html: Include common-remote.js.
  • public/v3/privileged-api.js:

(PrivilegedAPI): Use class now that we don't include data.js.
(PrivilegedAPI.sendRequest): Modernized the code.
(PrivilegedAPI.requestCSRFToken): Ditto.

  • public/v3/remote.js:

(BrowserRemoteAPI): Renamed from RemoteAPI. window.RemoteAPI is now an instance of this class.
(BrowserRemoteAPI.prototype.sendHttpRequest): Moved from RemoteAPI.sendHttpRequest.
(BrowserRemoteAPI.prototype.sendHttpRequest):

  • server-tests/privileged-api-create-analysis-task-tests.js: Updated tests since NodeJSRemoteAPI

now throws the JSON status as an error to be consistent with BrowserRemoteAPI.

  • server-tests/privileged-api-create-test-group-tests.js: Ditto.
  • server-tests/privileged-api-upate-run-status.js: Ditto.
  • tools/js/buildbot-triggerable.js:

(BuildbotTriggerable.prototype.syncOnce): Just use postJSONWithStatus instead of manually
checking the status.

  • tools/js/remote.js:

(NodeRemoteAPI): Renamed from RemoteAPI. Still exported as RemoteAPI.
(NodeRemoteAPI.prototype.constructor):
(NodeRemoteAPI.prototype.sendHttpRequest): Modernized the code.

2:47 PM Changeset in webkit [214007] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Clean up TestExpectations for some WPT LayoutTests.

Unreviewed test gardening.

2:31 PM Changeset in webkit [214006] by Chris Dumez
  • 6 edits
    1 add in trunk

Document state (e.g. form data) is lost after a tab is terminated in the background for power reasons
​https://bugs.webkit.org/show_bug.cgi?id=169635
<rdar://problem/31046729>

Reviewed by Andreas Kling.

Source/WebKit2:

Document state (e.g. form data) was lost after a tab was terminated in the background for power
reasons. To address the issue, we now save the document state whenever a page is backgrounded.
This document state automatically gets restored when reloading the page after termination.

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::visibilityDidChange):
(WebKit::WebPage::setActivityState):

  • WebProcess/WebPage/WebPage.h:

Tools:

Add API test coverage.

  • TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
  • TestWebKitAPI/Tests/WebKit2/mac/RestoreStateAfterTermination.mm: Added.

(TestWebKitAPI::runJavaScriptAlert):
(TestWebKitAPI::didFinishLoad):
(TestWebKitAPI::didCrash):
(TestWebKitAPI::TEST):

  • TestWebKitAPI/Tests/WebKit2/simple-form.html:
1:19 PM Changeset in webkit [214005] by mark.lam@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Fix missing exception checks in Interpreter.cpp.
​https://bugs.webkit.org/show_bug.cgi?id=164964

Reviewed by Saam Barati.

  • interpreter/Interpreter.cpp:

(JSC::eval):
(JSC::sizeOfVarargs):
(JSC::sizeFrameForVarargs):
(JSC::Interpreter::executeProgram):
(JSC::Interpreter::executeCall):
(JSC::Interpreter::executeConstruct):
(JSC::Interpreter::prepareForRepeatCall):
(JSC::Interpreter::execute):

1:00 PM Changeset in webkit [214004] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebInspectorUI

JSContext Inspector: NetworkAgent may be unavailable
​https://bugs.webkit.org/show_bug.cgi?id=169691

Patch by Joseph Pecoraro <Joseph Pecoraro> on 2017-03-15
Reviewed by Brian Burg.

  • UserInterface/Views/ResourceDetailsSidebarPanel.js:

(WebInspector.ResourceDetailsSidebarPanel.prototype._refreshRequestAndResponse):
Check that NetworkAgent even exists. An augmented JSContext may have Resources
without having a NetworkAgent.

12:56 PM Changeset in webkit [214003] by Wenson Hsieh
  • 2 edits in trunk/Source/WebKit2

WebContent crash due to bad variant access in WebKit: WebKit::WebPage::expandedRangeFromHandle
​https://bugs.webkit.org/show_bug.cgi?id=169657
<rdar://problem/30631070>

Reviewed by Tim Horton.

In WebPageIOS.mm, the call to unionDOMRanges from WebPage::expandedRangeFromHandle invokes
Range::compareBoundaryPoints, assuming that the return value is not an exception, and then attempts to perform
integer comparison on the result. This is one speculative cause of the web content crash in the radar.

There isn't a known way to reproduce this crash.

  • WebProcess/WebPage/ios/WebPageIOS.mm:

(WebKit::unionDOMRanges):

12:35 PM Changeset in webkit [214002] by Nikita Vasilyev
  • 2 edits in trunk/Source/WebInspectorUI

REGRESSION (r213622): Web Inspector: DataGrid headers should NOT be centered
​https://bugs.webkit.org/show_bug.cgi?id=169645
<rdar://problem/31051520>

Reviewed by Matt Baker.

  • UserInterface/Views/DataGrid.css:

(.data-grid th):
(body[dir=ltr] .data-grid th):
(body[dir=rtl] .data-grid th):

12:34 PM Changeset in webkit [214001] by timothy_horton@apple.com
  • 7 edits
    3 deletes in trunk/Source/WebKit2

Un-deprecate the original PDFPlugin
​https://bugs.webkit.org/show_bug.cgi?id=169655

Reviewed by Anders Carlsson.

  • Shared/mac/PDFKitImports.h:
  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/Plugins/PDF/DeprecatedPDFLayerControllerSPI.h: Removed.
  • WebProcess/Plugins/PDF/DeprecatedPDFPlugin.h: Removed.
  • WebProcess/Plugins/PDF/DeprecatedPDFPlugin.mm: Removed.
  • WebProcess/Plugins/PDF/PDFLayerControllerSPI.h:
  • WebProcess/Plugins/PDF/PDFPlugin.h:
  • WebProcess/Plugins/PDF/PDFPlugin.mm:
  • WebProcess/Plugins/PDF/PDFPluginAnnotation.mm:

The transition to the non-deprecated PDFPlugin never happened,
and we have a very different plan now.

12:34 PM Changeset in webkit [214000] by rniwa@webkit.org
  • 6 edits in trunk/Websites/perf.webkit.org

Fix server tests after r213998 and r213969
​https://bugs.webkit.org/show_bug.cgi?id=169690

Reviewed by Antti Koivisto.

Fixed the existing server tests.

  • public/v3/models/analysis-task.js:

(AnalysisTask.prototype._updateRemoteState): Use the relative path from the root so that it works inside tests.
(AnalysisTask.prototype.associateBug): Ditto.
(AnalysisTask.prototype.dissociateBug): Ditto.
(AnalysisTask.prototype.associateCommit): Ditto.
(AnalysisTask.prototype.dissociateCommit): Ditto.
(AnalysisTask._fetchSubset): Ditto.
(AnalysisTask.fetchAll): Ditto.

  • public/v3/models/test-group.js:

(TestGroup.prototype.updateName): Ditto.
(TestGroup.prototype.updateHiddenFlag): Ditto.
(TestGroup.createAndRefetchTestGroups): Ditto.
(TestGroup.cachedFetch): Ditto.

  • server-tests/api-manifest.js: Reverted an inadvertant change in r213969.
  • tools/js/database.js:

(tableToPrefixMap): Added analysis_strategies.

  • unit-tests/analysis-task-tests.js: Updated expectations per changes to AnalysisTask.
12:15 PM Changeset in webkit [213999] by webkit@devinrousso.com
  • 4 edits
    1 add in trunk/Source/WebInspectorUI

Web Inspector: SVG image content view should toggle between image and source
​https://bugs.webkit.org/show_bug.cgi?id=16079

Reviewed by Joseph Pecoraro.

  • Localizations/en.lproj/localizedStrings.js:
  • UserInterface/Main.html:
  • UserInterface/Views/ResourceClusterContentView.js:

(WebInspector.ResourceClusterContentView.prototype.get responseContentView):
Show the SVG cluster content view if the MIME type matches an SVG file.

  • UserInterface/Views/SVGImageResourceClusterContentView.js: Added.

(WebInspector.SVGImageResourceClusterContentView):
(WebInspector.SVGImageResourceClusterContentView.prototype.get resource):
(WebInspector.SVGImageResourceClusterContentView.prototype.get selectionPathComponents):
(WebInspector.SVGImageResourceClusterContentView.prototype.shown):
(WebInspector.SVGImageResourceClusterContentView.prototype.closed):
(WebInspector.SVGImageResourceClusterContentView.prototype.saveToCookie):
(WebInspector.SVGImageResourceClusterContentView.prototype.restoreFromCookie):
(WebInspector.SVGImageResourceClusterContentView.prototype._pathComponentForContentView):
(WebInspector.SVGImageResourceClusterContentView.prototype._identifierForContentView):
(WebInspector.SVGImageResourceClusterContentView.prototype._showContentViewForIdentifier):
(WebInspector.SVGImageResourceClusterContentView.prototype._pathComponentSelected):

12:15 PM Changeset in webkit [213998] by rniwa@webkit.org
  • 5 edits
    2 adds in trunk/Websites/perf.webkit.org

Add tests for privileged-api/create-analysis-task and privileged-api/create-test-group
​https://bugs.webkit.org/show_bug.cgi?id=169688

Rubber-stamped by Antti Koivisto.

Added tests for privileged-api/create-analysis-task and privileged-api/create-test-group, and fixed newly found bugs.

  • public/privileged-api/create-analysis-task.php:

(main): Fixed the bug that we were not explicitly checking whether start_run and end_run were integers or not.
Also return InvalidTimeRange when start and end times are identical as that makes no sense for an analysis task.

  • public/privileged-api/create-test-group.php:

(main): Fixed a bug that we were not explicitly checking task and repetitionCount to be an integer.
(ensure_commit_sets): Fixed the bug that the number of commit sets weren't checked.

  • server-tests/privileged-api-create-analysis-task-tests.js: Added.
  • server-tests/privileged-api-create-test-group-tests.js: Added.
  • server-tests/resources/common-operations.js:

(prepareServerTest): Increase the timeout from 1s to 5s.

  • server-tests/resources/mock-data.js:

(MockData.addMockData): Use a higher database ID of 20 for a mock build_slave to avoid a conflict with auto-generated IDs.

12:07 PM Changeset in webkit [213997] by webkit@devinrousso.com
  • 5 edits in trunk/Source/WebInspectorUI

Web Inspector: RTL: add support for Timeline ruler selections/movement
​https://bugs.webkit.org/show_bug.cgi?id=169588

Reviewed by Matt Baker.

  • UserInterface/Views/TimelineOverview.css:

(body[dir=ltr] .timeline-overview.frames > .timeline-ruler:not(.both-handles-clamped) > .selection-handle.right):
(body[dir=rtl] .timeline-overview.frames > .timeline-ruler:not(.both-handles-clamped) > .selection-handle.left):
(body[dir=ltr] .timeline-overview.frames > .timeline-ruler:not(.both-handles-clamped) > .shaded-area.right):
(body[dir=rtl] .timeline-overview.frames > .timeline-ruler:not(.both-handles-clamped) > .shaded-area.left):
(.timeline-overview.frames > .timeline-ruler:not(.both-handles-clamped) > .selection-handle.right): Deleted.
(.timeline-overview.frames > .timeline-ruler:not(.both-handles-clamped) > .shaded-area.right): Deleted.
Fixes alignment of selection handles when in Frames view.

  • UserInterface/Views/TimelineOverview.js:

(WebInspector.TimelineOverview.prototype._handleScrollEvent):
Support horizontal scrolling in timelines by treating the current scroll value as negative
when in RTL, flipping the scroll direction.

  • UserInterface/Views/TimelineRuler.css:

(body[dir=ltr] .timeline-ruler > .selection-handle.left):
(body[dir=rtl] .timeline-ruler > .selection-handle.left):
(body[dir=ltr] .timeline-ruler > .selection-handle.right):
(body[dir=rtl] .timeline-ruler > .selection-handle.right):
(body[dir=ltr] .timeline-ruler > .shaded-area.left):
(body[dir=rtl] .timeline-ruler > .shaded-area.left):
(body[dir=ltr] .timeline-ruler > .shaded-area.right):
(body[dir=rtl] .timeline-ruler > .shaded-area.right):
(.timeline-ruler > .selection-handle.left): Deleted.
(.timeline-ruler > .selection-handle.right): Deleted.
(.timeline-ruler > .shaded-area.left): Deleted.
(.timeline-ruler > .shaded-area.right): Deleted.
Flip the position alignment of the selection handles and shaded areas when in RTL.

  • UserInterface/Views/TimelineRuler.js:

(WebInspector.TimelineRuler.prototype._handleMouseDown):
(WebInspector.TimelineRuler.prototype._handleMouseMove):
(WebInspector.TimelineRuler.prototype._handleMouseUp):
(WebInspector.TimelineRuler.prototype._handleSelectionHandleMouseDown):
(WebInspector.TimelineRuler.prototype._handleSelectionHandleMouseMove):
Treat the current mouse position (event.pageX) as a negative value when in RTL, meaning that
the delta movement from the starting position is flipped.

11:51 AM Changeset in webkit [213996] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

[Modern Media Controls] Using the arrow keys to navigate in the tracks menu also scrolls the page
​https://bugs.webkit.org/show_bug.cgi?id=169671
<rdar://problem/31060091>

Patch by Antoine Quint <Antoine Quint> on 2017-03-15
Reviewed by Eric Carlson.

Source/WebCore:

We need to call preventDefault() when handling a "keydown" event that we recognize.

Test: media/modern-media-controls/tracks-panel/tracks-panel-prevent-default-on-keydown.html

  • Modules/modern-media-controls/controls/tracks-panel.js:

(TracksPanel.prototype._handleKeydown):

LayoutTests:

Add a test that checks that preventDefault() is called on "keydown" events that we know to handle.

  • media/modern-media-controls/tracks-panel/tracks-panel-prevent-default-on-keydown-expected.txt: Added.
  • media/modern-media-controls/tracks-panel/tracks-panel-prevent-default-on-keydown.html: Added.
11:26 AM Changeset in webkit [213995] by Jonathan Bedard
  • 1 edit
    1 delete in trunk/LayoutTests

Remove TestExpectations.orig after r213882

Unreviewed, removing file committed by mistake.

11:21 AM Changeset in webkit [213994] by commit-queue@webkit.org
  • 6 edits in trunk

[Modern Media Controls] Tracks menu overlaps controls bar in fullscreen
​https://bugs.webkit.org/show_bug.cgi?id=169670
<rdar://problem/31060086>
Source/WebCore:

Patch by Antoine Quint <Antoine Quint> on 2017-03-15
Reviewed by Eric Carlson.

It used to be that the top of the tracks button was the same as the top of the controls
bar, but that changed when we fixed ​https://bugs.webkit.org/show_bug.cgi?id=169412. We
now use the top of the controls bar to computed the y-position for the tracks panel.

We are not adding a new test, instead we're unflaking a test that started failing
reliably once we fixed ​https://bugs.webkit.org/show_bug.cgi?id=169412.

  • Modules/modern-media-controls/controls/macos-media-controls.js:

(MacOSMediaControls.prototype.showTracksPanel):

LayoutTests:

Patch by Antoine Quint <Antoine Quint> on 2017-03-15
Reviewed by Eric Carlson.

Unflake a test that checks the tracks panel position in fullscreen.

  • media/modern-media-controls/tracks-support/tracks-support-show-panel-fullscreen-expected.txt:
  • media/modern-media-controls/tracks-support/tracks-support-show-panel-fullscreen.html:
  • platform/mac/TestExpectations:
11:15 AM Changeset in webkit [213993] by rniwa@webkit.org
  • 5 edits in trunk/Websites/perf.webkit.org

Make unit tests return a promise instead of manually calling done
​https://bugs.webkit.org/show_bug.cgi?id=169663

Reviewed by Antti Koivisto.

Make the existing unit tests always reutrn a promise instead of manually calling "done" callback as done
in r213969. The promise tests are a lot more stable and less error prone.

Also use MockRemoteAPI.waitForRequest() instead of chaining two resolved promises where appropriate.

  • unit-tests/analysis-task-tests.js:
  • unit-tests/buildbot-syncer-tests.js:
  • unit-tests/checkconfig.js:
  • unit-tests/privileged-api-tests.js:
11:15 AM Changeset in webkit [213992] by jonlee@apple.com
  • 11 edits
    8 adds in trunk/Source/WebCore

Clean up RTCPeerConnection IDL
​https://bugs.webkit.org/show_bug.cgi?id=169660

Reviewed by Youenn Fablet.

  • Modules/mediastream/RTCPeerConnection.cpp:

(WebCore::RTCPeerConnection::addTransceiver): Refactor to use RTCRtpTransceiverInit.
(WebCore::RTCPeerConnection::completeAddTransceiver):

  • Modules/mediastream/RTCPeerConnection.h: Remove redundant definitions.
  • Modules/mediastream/RTCPeerConnection.idl: Using 13 March 2017 Editor's Draft of

WebRTC spec. Move RTCOfferAnswerOptions out to separate IDLs. Keep RTCDataChannelInit and
RTCRtpTransceiverInit since they appear to be used only in RTCPeerConnection.

Reorder the properties, functions, and events based on their appearance in the spec.

Legacy MediaStream calls are placed at the end. I tried to use "partial interface" in the
same file, but in the end nothing was generated, so everything is contained in one interface
block.

  • Modules/mediastream/RTCEnums.h: Added. This will be an all-in-one header to hold the

enums.

Move RTCAnswerOptions, RTCOfferAnswerOptions, RTCOfferOptions, RTCRtpTransceiverDirection
out to their own IDL's.

  • CMakeLists.txt:
  • DerivedSources.make:
  • Modules/mediastream/RTCAnswerOptions.h: Added.
  • Modules/mediastream/RTCAnswerOptions.idl: Added.
  • Modules/mediastream/RTCOfferAnswerOptions.h:
  • Modules/mediastream/RTCOfferAnswerOptions.idl: Added.
  • Modules/mediastream/RTCOfferOptions.h: Added. Remove |offerToReceiveVideo| and

|offerToReceiveAudio|, which are not used.

  • Modules/mediastream/RTCOfferOptions.idl: Added.
  • Modules/mediastream/RTCRtpTransceiverDirection.h: Added.
  • Modules/mediastream/RTCRtpTransceiverDirection.idl: Added. Use a typedef for

RTCRtpTransceiverDirection to prevent the code generator from prefixing RTCRtpTransceiver.

  • Modules/mediastream/RTCRtpTransceiver.idl: Move RTCRtpTransceiverDirection.
  • WebCore.xcodeproj/project.pbxproj: Add IDLs and derived sources. Remove unused

HTMLMediaElementMediaStream.h. Reorder.

Refactor.

  • Modules/mediastream/RTCRtpTransceiver.h: Use RTCRtpTransceiverDirection.
  • Modules/mediastream/MediaEndpointPeerConnection.cpp:
11:15 AM Changeset in webkit [213991] by Joseph Pecoraro
  • 4 edits in trunk/Source/WebInspectorUI

Web Inspector: REGRESSION: Elements Tab > Node Details Sidebar > Properties Section is spammed with TypeErrors
​https://bugs.webkit.org/show_bug.cgi?id=153911
<rdar://problem/24520098>

Reviewed by Devin Rousso.

  • UserInterface/Views/DOMNodeDetailsSidebarPanel.js:

(WebInspector.DOMNodeDetailsSidebarPanel.prototype._refreshProperties.nodeResolved.inspectedPage_node_collectPrototypes):
(WebInspector.DOMNodeDetailsSidebarPanel.prototype._refreshProperties.nodeResolved):
Update due to naming conventions for code that evalutes in the inspected page.

(WebInspector.DOMNodeDetailsSidebarPanel.prototype._refreshProperties.fillSection):
(WebInspector.DOMNodeDetailsSidebarPanel.prototype._refreshProperties):
Create a more complete ObjectTreeView for the different sections.

  • UserInterface/Views/ObjectTreePropertyTreeElement.js:

(WebInspector.ObjectTreePropertyTreeElement.prototype._updateChildren):
PureAPI behaves the same as ClassAPI and just shows own properties.

  • UserInterface/Views/ObjectTreeView.js:

(WebInspector.ObjectTreeView.prototype.showOnlyProperties):
In only properties mode don't show the Prototype expander at the end.

(WebInspector.ObjectTreeView.prototype.setPrototypeNameOverride):
Allow a prototype name override at the top level. This will allow clients
to specify that the object at the top level is a Prototype object, so that
ObjectTreePropertyTreeElements can infer the right NativeFunctionParameter
information knowing that those properties are on a particular prototype.

(WebInspector.ObjectTreeView.prototype.update):
For the PureAPI use getOwnProperties instead of getDisplayableProperties.
This avoids the special handling we have for DOM native properties which
we bubble up to the top as value descriptors even though they are actually
accessor descriptors on prototypes.

(WebInspector.ObjectTreeView.prototype._updateProperties):
Pass the prototype name override onward for object properties.

11:03 AM Changeset in webkit [213990] by zandobersek@gmail.com
  • 2 edits in trunk/Source/WebCore

[GTK] Initialize m_button, m_clickCount members in PlatformMouseEvent constructors
​https://bugs.webkit.org/show_bug.cgi?id=169666

Reviewed by Michael Catanzaro.

Initialize the m_button and m_clickCount class members in the GTK+-specific
implementation of PlatformMouseEvent constructors to NoButton and 0,
respectively. The constructors expect to operate on passed-in GTK+ events
that will be able to initialize those two members to some valid values, but
this is not guaranteed.

  • platform/gtk/PlatformMouseEventGtk.cpp:

(WebCore::PlatformMouseEvent::PlatformMouseEvent):

11:02 AM Changeset in webkit [213989] by zandobersek@gmail.com
  • 5 edits in trunk/Source/WebCore

[TexMap] Add missing class member initializations
​https://bugs.webkit.org/show_bug.cgi?id=169665

Reviewed by Michael Catanzaro.

Zero-initialize the members in various TextureMapper classes
that are missing the proper initialization, as reported by
the Coverity tool.

  • platform/graphics/texmap/BitmapTexturePool.h:
  • platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h:
  • platform/graphics/texmap/coordinated/CoordinatedGraphicsState.h:

(WebCore::CoordinatedGraphicsLayerState::CoordinatedGraphicsLayerState):

  • platform/graphics/texmap/coordinated/SurfaceUpdateInfo.h:
10:59 AM Changeset in webkit [213988] by achristensen@apple.com
  • 14 edits in trunk

Compiled content extensions should include the JSON source
​https://bugs.webkit.org/show_bug.cgi?id=169643

Reviewed by Geoffrey Garen.

Source/WebCore:

Serializing the JSON string from which a content extension was compiled
to disk with the compiled content extension will allow us to validate content
extensions and automatically migrate older content extensions to new versions.
It less than doubles the size of the compiled content extension on disk, and when
interpreting the bytecode that memory is never read, so it doesn't increase our
dirty memory usage.

Covered by new API tests.

  • contentextensions/ContentExtensionCompiler.cpp:

(WebCore::ContentExtensions::compileRuleList):

  • contentextensions/ContentExtensionCompiler.h:

Source/WebKit2:

  • UIProcess/API/APIContentExtensionStore.cpp:

(API::ContentExtensionStore::ContentExtensionStore):
(API::ContentExtensionMetaData::fileSize):
(API::encodeContentExtensionMetaData):
(API::decodeContentExtensionMetaData):
(API::compiledToFile):
(API::createExtension):
(API::ContentExtensionStore::getContentExtensionSource):

  • UIProcess/API/APIContentExtensionStore.h:
  • UIProcess/API/Cocoa/WKContentExtensionStore.mm:

(toWKErrorCode):
(-[WKContentExtensionStore lookupContentExtensionForIdentifier:completionHandler:]):
(-[WKContentExtensionStore removeContentExtensionForIdentifier:completionHandler:]):
(-[WKContentExtensionStore _getContentExtensionSourceForIdentifier:completionHandler:]):

  • UIProcess/API/Cocoa/WKContentExtensionStorePrivate.h:
  • UIProcess/API/Cocoa/WKError.h:
  • UIProcess/API/Cocoa/_WKUserContentExtensionStore.h:
  • UIProcess/API/Cocoa/_WKUserContentExtensionStore.mm:

(toUserContentExtensionStoreError):
(-[_WKUserContentExtensionStore compileContentExtensionForIdentifier:encodedContentExtension:completionHandler:]):
(-[_WKUserContentExtensionStore lookupContentExtensionForIdentifier:completionHandler:]):
(-[_WKUserContentExtensionStore removeContentExtensionForIdentifier:completionHandler:]):

Tools:

  • TestWebKitAPI/Tests/WebCore/ContentExtensions.cpp:
  • TestWebKitAPI/Tests/WebKit2Cocoa/WKUserContentExtensionStore.mm:

(TEST_F):

10:40 AM Changeset in webkit [213987] by commit-queue@webkit.org
  • 3 edits
    4 adds in trunk

[Modern Media Controls] Captions do not default to Auto when language is changed
​https://bugs.webkit.org/show_bug.cgi?id=169675
<rdar://problem/30423369>

Patch by Antoine Quint <Antoine Quint> on 2017-03-15
Reviewed by Eric Carlson.

Source/WebCore:

Ensure we correctly mark the Off and Auto items as selected when we are using the
Off and Auto tracks.

Tests: media/modern-media-controls/tracks-support/tracks-support-auto-text-track.html

media/modern-media-controls/tracks-support/tracks-support-off-text-track.html

  • Modules/modern-media-controls/media/tracks-support.js:

(TracksSupport.prototype.tracksPanelIsTrackInSectionSelected):
(TracksSupport.prototype.tracksPanelSelectionDidChange):

LayoutTests:

Add new tests that check the cases where the expected selected track should be "Off" or "Auto".

  • media/modern-media-controls/tracks-support/tracks-support-auto-text-track-expected.txt: Added.
  • media/modern-media-controls/tracks-support/tracks-support-auto-text-track.html: Added.
  • media/modern-media-controls/tracks-support/tracks-support-off-text-track-expected.txt: Added.
  • media/modern-media-controls/tracks-support/tracks-support-off-text-track.html: Added.
10:21 AM Changeset in webkit [213986] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

Use git's -C flag when possible in VCSUtils.pm
​https://bugs.webkit.org/show_bug.cgi?id=169003

Patch by Kocsen Chung <Kocsen Chung> on 2017-03-15
Reviewed by Sam Weinig.

Use the git -C flag where appropriate to perform the
operation on a target directory and avoid unnecessary logic
to cd in and out of the target directory.

  • Scripts/VCSUtils.pm:

(isGitDirectory):
(isGitSVNDirectory):
(svnRevisionForDirectory):
(svnInfoForPath):
For all four subroutines, use git's -C flag and strip cd logic.

10:06 AM Changeset in webkit [213985] by Ryan Haddad
  • 10 edits
    1 delete in trunk/Source/WebCore

Unreviewed, rolling out r213977.

This change broke the Windows build.

Reverted changeset:

"Make a base class for WebGL and WebGPU contexts"
​https://bugs.webkit.org/show_bug.cgi?id=169651
​http://trac.webkit.org/changeset/213977

10:04 AM Changeset in webkit [213984] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Mark http/tests/media/modern-media-controls/time-labels-support/long-time.html as flaky on mac-wk1.
​https://bugs.webkit.org/show_bug.cgi?id=169677

Unreviewed test gardening.

  • platform/mac-wk1/TestExpectations:
9:38 AM Changeset in webkit [213983] by commit-queue@webkit.org
  • 10 edits
    4 adds in trunk

run-webkit-tests is always creating mock libwebrtc tracks
​https://bugs.webkit.org/show_bug.cgi?id=169658

Patch by Youenn Fablet <​youenn@apple.com> on 2017-03-15
Reviewed by Alex Christensen.

Source/WebCore:

Tests: webrtc/peer-connection-audio-mute.html

webrtc/video-mute.html

Creating real libwebrtc av tracks in case of RealTwoPeerConnections mock factory.

  • testing/MockLibWebRTCPeerConnection.cpp:

(WebCore::MockLibWebRTCPeerConnectionFactory::CreateVideoTrack):
(WebCore::MockLibWebRTCPeerConnectionFactory::CreateAudioTrack):

  • testing/MockLibWebRTCPeerConnection.h:

LayoutTests:

  • TestExpectations:
  • webrtc/audio-peer-connection-webaudio.html:
  • webrtc/peer-connection-audio-mute-expected.txt: Added.
  • webrtc/peer-connection-audio-mute.html: Added.
  • webrtc/routines.js:

(analyseAudio):

  • webrtc/video-expected.txt:
  • webrtc/video-mute-expected.txt: Added.
  • webrtc/video-mute.html: Added.
  • webrtc/video.html:
9:35 AM Changeset in webkit [213982] by commit-queue@webkit.org
  • 5 edits in trunk

Preventive clean-up: ensure RTCPeerConnection stays valid when calling postTask
​https://bugs.webkit.org/show_bug.cgi?id=169661

Patch by Youenn Fablet <​youenn@apple.com> on 2017-03-15
Reviewed by Alex Christensen.

Source/WebCore:

Protecting the RTCPeerConnection object when calling postTask since it might get collected between the task post
and task run. Also do not send negotiationNeeded event if RTCPeerConnection is closed (covered by added test).

  • Modules/mediastream/RTCPeerConnection.cpp:

(WebCore::RTCPeerConnection::updateIceGatheringState):
(WebCore::RTCPeerConnection::updateIceConnectionState):
(WebCore::RTCPeerConnection::scheduleNegotiationNeededEvent):

LayoutTests:

  • webrtc/negotiatedneeded-event-addStream-expected.txt:
  • webrtc/negotiatedneeded-event-addStream.html:
8:58 AM Changeset in webkit [213981] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Mark media/modern-media-controls/volume-down-support/volume-down-support.html as flaky.
​https://bugs.webkit.org/show_bug.cgi?id=169568

Unreviewed test gardening.

  • platform/mac/TestExpectations:
5:42 AM Changeset in webkit [213980] by commit-queue@webkit.org
  • 7 edits
    2 moves in trunk

[Modern Media Controls] Always use six digits to display time when overall media duration is an hour or more
​https://bugs.webkit.org/show_bug.cgi?id=169668
<rdar://problem/31059699>

Patch by Antoine Quint <Antoine Quint> on 2017-03-15
Reviewed by Dean Jackson.

Source/WebCore:

Always use six digits to display times when the overall media duration is an hour or more. This
ensures that we don't display too much white space around labels when we know we will eventually
need six digits to display the full time, but the current time to display is under an hour.

Test: media/modern-media-controls/time-control/time-control-use-six-digits-for-time-labels.html

  • Modules/modern-media-controls/controls/time-control.js:

(TimeControl.prototype.get useSixDigitsForTimeLabels):
(TimeControl.prototype.set useSixDigitsForTimeLabels):
(TimeControl.prototype.set width):
(TimeControl.prototype.get isSufficientlyWide):
(TimeControl.prototype._availableWidthHasChanged):
(TimeControl.prototype.get labelsMayDisplayTimesOverAnHour): Deleted.
(TimeControl.prototype.set labelsMayDisplayTimesOverAnHour): Deleted.

  • Modules/modern-media-controls/controls/time-label.js:

(TimeLabel.prototype._formattedTime):

  • Modules/modern-media-controls/media/time-labels-support.js:

(TimeLabelsSupport.prototype.syncControl):
(TimeLabelsSupport):

LayoutTests:

Rebase some tests due to the rename of the labelsMayDisplayTimesOverAnHour property to
useSixDigitsForTimeLabels. We also add an assertion in long-time.html to check that
we currently use six digits to display a time that is under an hour but where the
media duration is over an hour.

  • http/tests/media/modern-media-controls/time-labels-support/long-time-expected.txt:
  • http/tests/media/modern-media-controls/time-labels-support/long-time.html:
  • media/modern-media-controls/time-control/time-control-use-six-digits-for-time-labels-expected.txt: Renamed from LayoutTests/media/modern-media-controls/time-control/time-control-labels-may-display-times-over-an-hour-expected.txt.
  • media/modern-media-controls/time-control/time-control-use-six-digits-for-time-labels.html: Renamed from LayoutTests/media/modern-media-controls/time-control/time-control-labels-may-display-times-over-an-hour.html.
3:19 AM Changeset in webkit [213979] by commit-queue@webkit.org
  • 3 edits in trunk/LayoutTests

[mac-wk1 debug] LayoutTest media/modern-media-controls/airplay-placard/airplay-placard-text-section.html is a flaky timeout
​https://bugs.webkit.org/show_bug.cgi?id=169654
<rdar://problem/31059092>

Patch by Antoine Quint <Antoine Quint> on 2017-03-15
Reviewed by Dean Jackson.

Using an asynchronous assertion to improve reliability.

  • media/modern-media-controls/airplay-placard/airplay-placard-text-section-expected.txt:
  • media/modern-media-controls/airplay-placard/airplay-placard-text-section.html:
2:51 AM Changeset in webkit [213978] by dino@apple.com
  • 12 edits in trunk/Source

Sort Xcode project files
​https://bugs.webkit.org/show_bug.cgi?id=169669

Reviewed by Antoine Quint.

Source/JavaScriptCore:

Source/WebCore:

  • WebCore.xcodeproj/project.pbxproj:

Source/WebCore/PAL:

  • PAL.xcodeproj/project.pbxproj:

Source/WebKit:

  • WebKit.xcodeproj/project.pbxproj:

Source/WebKit2:

  • WebKit2.xcodeproj/project.pbxproj:

Source/WTF:

  • WTF.xcodeproj/project.pbxproj:
2:43 AM Changeset in webkit [213977] by dino@apple.com
  • 10 edits
    1 add in trunk/Source/WebCore

Make a base class for WebGL and WebGPU contexts
​https://bugs.webkit.org/show_bug.cgi?id=169651
<rdar://problem/31053489>

Reviewed by Simon Fraser.

Add a new pure virtual base class, GPUBasedCanvasRenderingContext, that
will be used by WebGL and WebGPU rendering contexts. This allows us
to avoid some code duplication, since many places treat the two
as the same.

Also rename is3d() -> isWebGL() and isGPU() -> isWebGPU().

  • WebCore.xcodeproj/project.pbxproj: New file.
  • html/HTMLCanvasElement.cpp:

(WebCore::HTMLCanvasElement::getContextWebGL):
(WebCore::HTMLCanvasElement::getContextWebGPU):
(WebCore::HTMLCanvasElement::reset):
(WebCore::HTMLCanvasElement::paint):
(WebCore::HTMLCanvasElement::isGPUBased):
(WebCore::HTMLCanvasElement::getImageData):
(WebCore::HTMLCanvasElement::isGPU): Deleted.
(WebCore::HTMLCanvasElement::is3D): Deleted.

  • html/HTMLCanvasElement.h:
  • html/canvas/CanvasRenderingContext.h:

(WebCore::CanvasRenderingContext::isWebGL):
(WebCore::CanvasRenderingContext::isWebGPU):
(WebCore::CanvasRenderingContext::isGPUBased):
(WebCore::CanvasRenderingContext::is3d): Deleted.
(WebCore::CanvasRenderingContext::isGPU): Deleted.

  • html/canvas/GPUBasedCanvasRenderingContext.h: Added.

(WebCore::GPUBasedCanvasRenderingContext::GPUBasedCanvasRenderingContext):

  • html/canvas/WebGLRenderingContextBase.cpp:

(WebCore::WebGLRenderingContextBase::WebGLRenderingContextBase):

  • html/canvas/WebGLRenderingContextBase.h:
  • html/canvas/WebGPURenderingContext.cpp:

(WebCore::WebGPURenderingContext::WebGPURenderingContext):

  • html/canvas/WebGPURenderingContext.h:
  • rendering/RenderLayerBacking.cpp:

(WebCore::canvasCompositingStrategy):

1:35 AM Changeset in webkit [213976] by Dewei Zhu
  • 10 edits
    7 adds in trunk/Websites/perf.webkit.org

Rewrite 'pull-os-versions' script in Javascript to add support for reporting os revisions with sub commits.
​https://bugs.webkit.org/show_bug.cgi?id=169542

Reviewed by Ryosuke Niwa.

Extend '/api/commits/<repository>/last-reported' to accept a range and return last reported commits in given range.
Rewrite 'pull-os-versions' in JavaScript and add unit tests for it.
Instead of writing query manually while searching criteria contains null columns, use the methods provided in 'db.php'.
Add '.gitignore' file to ommit files generated by while running tests/instances locally.

  • .gitignore: Added.
  • public/api/commits.php:
  • public/api/report-commits.php:
  • public/include/commit-log-fetcher.php:
  • public/include/db.php: 'null_columns' of prepare_params should be a reference.
  • public/include/report-processor.php:
  • server-tests/api-commits.js:

(then):

  • server-tests/api-report-commits-tests.js:
  • server-tests/resources/mock-logger.js: Added.

(MockLogger):
(MockLogger.prototype.log):
(MockLogger.prototype.error):

  • server-tests/resources/mock-subprocess.js: Added.

(MockSubprocess.call):
(MockSubprocess.waitingForInvocation):
(MockSubprocess.inject):
(MockSubprocess.reset):

  • server-tests/tools-buildbot-triggerable-tests.js:

(MockLogger): Deleted.
(MockLogger.prototype.log): Deleted.
(MockLogger.prototype.error): Deleted.

  • server-tests/tools-os-build-fetcher-tests.js: Added.

(beforeEach):
(return.waitingForInvocationPromise.then):
(then):
(string_appeared_here.return.waitingForInvocationPromise.then):
(return.addSlaveForReport.emptyReport.then):

  • tools/js/os-build-fetcher.js: Added.

(OSBuildFetcher):
(OSBuildFetcher.prototype._fetchAvailableBuilds):
(OSBuildFetcher.prototype._computeOrder):
(OSBuildFetcher.prototype._commitsForAvailableBuilds.return.this._subprocess.call.then.):
(OSBuildFetcher.prototype._commitsForAvailableBuilds):
(OSBuildFetcher.prototype._addSubCommitsForBuild):
(OSBuildFetcher.prototype._submitCommits):
(OSBuildFetcher.prototype.fetchAndReportNewBuilds):

  • tools/js/subprocess.js: Added.

(const.childProcess.require.string_appeared_here.Subprocess.prototype.call):
(const.childProcess.require.string_appeared_here.Subprocess):

  • tools/pull-os-versions.js: Added.

(main):
(syncLoop):

  • tools/sync-commits.py:

(Repository.fetch_commits_and_submit):

12:47 AM Changeset in webkit [213975] by zandobersek@gmail.com
  • 4 edits in trunk/Source/WebCore

Unreviewed GTK+ build fix. Sprinkle ENABLE(MEDIA_STREAM) build guards
in the Internals class to avoid compilation failures when building
with this feature disabled.

  • testing/Internals.cpp:

(WebCore::Internals::~Internals):

  • testing/Internals.h:
  • testing/Internals.idl:

Mar 14, 2017:

11:18 PM Changeset in webkit [213974] by aakash_jain@apple.com
  • 2 edits in trunk/Tools

start-queue-mac.sh should create logs directory if it doesn't exist
​https://bugs.webkit.org/show_bug.cgi?id=169634

Reviewed by Alexey Proskuryakov.

  • EWSTools/start-queue-mac.sh: Creating logs directory if it doesn't exist.
10:52 PM Changeset in webkit [213973] by tpopela@redhat.com
  • 2 edits in trunk/Source/JavaScriptCore

Wrong condition in offlineasm/risc.rb
​https://bugs.webkit.org/show_bug.cgi?id=169597

Reviewed by Mark Lam.

It's missing the 'and' operator between the conditions.

  • offlineasm/risc.rb:
10:21 PM Changeset in webkit [213972] by commit-queue@webkit.org
  • 9 edits
    2 adds in trunk

CanvasCapture should not generate a frame per each canvas draw command
​https://bugs.webkit.org/show_bug.cgi?id=169498

Patch by Youenn Fablet <​youenn@apple.com> on 2017-03-14
Reviewed by Simon Fraser & Eric Carlson.

Source/WebCore:

Test: fast/mediastream/captureStream/canvas2d-heavy-drawing.html

Making Canvas capture be asynchronous.
This ensures that one frame will be created for a set of synchronous draw commands.
In the future, we should find a better approach, like aligning with requestAnimationFrame.
Adding internals observer API for media stream tracks.

  • Modules/mediastream/CanvasCaptureMediaStreamTrack.cpp:

(WebCore::CanvasCaptureMediaStreamTrack::Source::Source):
(WebCore::CanvasCaptureMediaStreamTrack::Source::canvasChanged):

  • Modules/mediastream/CanvasCaptureMediaStreamTrack.h:
  • Modules/mediastream/MediaStreamTrack.idl:
  • platform/mediastream/RealtimeMediaSource.h:
  • testing/Internals.cpp:

(WebCore::Internals::~Internals):
(WebCore::Internals::observeMediaStreamTrack):

  • testing/Internals.h:
  • testing/Internals.idl:

LayoutTests:

  • fast/mediastream/captureStream/canvas2d-heavy-drawing-expected.txt: Added.
  • fast/mediastream/captureStream/canvas2d-heavy-drawing.html: Added.
8:28 PM Changeset in webkit [213971] by Wenson Hsieh
  • 2 edits in trunk/LayoutTests

Unreviewed, skip fast/media/video-element-in-details-collapse.html on iOS

  • platform/ios-simulator/TestExpectations:
8:27 PM Changeset in webkit [213970] by aestes@apple.com
  • 4 edits
    2 adds in trunk

[iOS] -[WKWebView _dataForDisplayedPDF] returns nil when called before an encrypted PDF has been unlocked
​https://bugs.webkit.org/show_bug.cgi?id=169653
<rdar://problem/24137675>

Reviewed by Tim Horton.

Source/WebKit2:

  • UIProcess/ios/WKPDFView.mm:

(-[WKPDFView pdfDocument]): Changed to return _cgPDFDocument directly instead of via the
UIPDFDocument, which is only created after the PDF has been unlocked.

Tools:

  • TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
  • TestWebKitAPI/Tests/WebKit2/encrypted.pdf: Added.
  • TestWebKitAPI/Tests/WebKit2Cocoa/WKPDFView.mm: Added.

(runTest):
(TEST):

8:19 PM Changeset in webkit [213969] by rniwa@webkit.org
  • 13 edits in trunk/Websites/perf.webkit.org

Make server tests return a promise instead of manually calling done
​https://bugs.webkit.org/show_bug.cgi?id=169648

Rubber-stamped by Chris Dumez.

Make the existing server tests always reutrn a promise instead of manually calling "done" callback.
The promise tests are a lot more stable and less error prone.

Also use arrow functions everywhere and use prepareServerTest, renamed from connectToDatabaseInEveryTest,
in more tests instead of manually connecting to database in every test, and reset v3 models.

  • server-tests/admin-platforms-tests.js:
  • server-tests/admin-reprocess-report-tests.js:
  • server-tests/api-build-requests-tests.js:
  • server-tests/api-manifest.js:
  • server-tests/api-measurement-set-tests.js:

(.postReports): Deleted. Not used in any test.

  • server-tests/api-report-commits-tests.js:
  • server-tests/api-report-tests.js:
  • server-tests/api-update-triggerable.js:
  • server-tests/privileged-api-upate-run-status.js:
  • server-tests/resources/common-operations.js:

(prepareServerTest): Renamed from connectToDatabaseInEveryTest. Increase the timeout and reset v3 models.

  • server-tests/tools-buildbot-triggerable-tests.js:
7:08 PM Changeset in webkit [213968] by commit-queue@webkit.org
  • 3 edits in trunk/LayoutTests

REGRESSION (r213882): 12 new/updated web-platform-tests failing
​https://bugs.webkit.org/show_bug.cgi?id=169615

Unreviewed.

Patch by Youenn Fablet <​youenn@apple.com> on 2017-03-14

6:20 PM Changeset in webkit [213967] by Wenson Hsieh
  • 3 edits
    2 adds in trunk

RenderElements should unregister for viewport visibility callbacks when they are destroyed
​https://bugs.webkit.org/show_bug.cgi?id=169521
<rdar://problem/30959545>

Reviewed by Simon Fraser.

Source/WebCore:

When registering a RenderElement for viewport visibility callbacks, we always need to make sure that it is unregistered
before it is destroyed. While we account for this in the destructor of RenderElement, we only unregister in the destructor
if we are already registered for visibility callbacks. In the call to RenderObject::willBeDestroyed(), we clear out rare
data, which holds RenderElement's viewport callback registration state, so upon entering the destructor of RenderElement,
we skip unregistration because RenderElement thinks that it is not registered.

We can mitigate this by unregistering the RenderElement earlier, in RenderElement::willBeDestroyed, prior to clearing out
the rare data. However, we'd ideally want to move the cleanup logic out of the destructor altogether and into willBeDestroyed
(see ​https://bugs.webkit.org/show_bug.cgi?id=169650).

Test: fast/media/video-element-in-details-collapse.html

  • rendering/RenderElement.cpp:

(WebCore::RenderElement::willBeDestroyed):

LayoutTests:

Adds a new layout test covering this regression. See WebCore ChangeLog for more details.

  • fast/media/video-element-in-details-collapse-expected.txt: Added.
  • fast/media/video-element-in-details-collapse.html: Added.
5:53 PM Changeset in webkit [213966] by mark.lam@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

BytecodeGenerator should use the same function to determine if it needs to store the DerivedConstructor in an ArrowFunction lexical environment.
​https://bugs.webkit.org/show_bug.cgi?id=169647
<rdar://problem/31051832>

Reviewed by Michael Saboff.

  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::usesDerivedConstructorInArrowFunctionLexicalEnvironment):
(JSC::BytecodeGenerator::initializeArrowFunctionContextScopeIfNeeded):
(JSC::BytecodeGenerator::emitPutDerivedConstructorToArrowFunctionContextScope):

  • bytecompiler/BytecodeGenerator.h:
5:18 PM Changeset in webkit [213965] by matthew_hanson@apple.com
  • 1 copy in tags/Safari-603.1.30.0.34

Tag Safari-603.1.30.0.34.

5:18 PM Changeset in webkit [213964] by matthew_hanson@apple.com
  • 1 copy in tags/Safari-603.1.30.1.33

Tag Safari-603.1.30.1.33.

5:17 PM Changeset in webkit [213963] by dino@apple.com
  • 2 edits in trunk/Source/WebCore

Rename LayerTypeWebGLLayer and use it for both WebGL and WebGPU
​https://bugs.webkit.org/show_bug.cgi?id=169628
<rdar://problems/31047025>

Fix Windows build.

  • platform/graphics/ca/win/PlatformCALayerWin.cpp:

(printLayer):

5:09 PM Changeset in webkit [213962] by matthew_hanson@apple.com
  • 1 delete in tags/Safari-603.1.30.0.34

Delete tag.

5:05 PM Changeset in webkit [213961] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebInspectorUI

Web Inspector: Exception when fetching computed styles can break future updates of section
​https://bugs.webkit.org/show_bug.cgi?id=169638
<rdar://problem/30588688>

Patch by Joseph Pecoraro <Joseph Pecoraro> on 2017-03-14
Reviewed by Devin Rousso.

  • UserInterface/Models/DOMNodeStyles.js:

(WebInspector.DOMNodeStyles.prototype.refresh.wrap):
(WebInspector.DOMNodeStyles.prototype.refresh):
Gracefully handle exceptions. If an exception did happen we
would be unable to update these in the future.

4:44 PM Changeset in webkit [213960] by BJ Burg
  • 6 edits in trunk/Source/JavaScriptCore

[Cocoa] Web Inspector: generated code for parsing an array of primitive-type enums from payload does not work
​https://bugs.webkit.org/show_bug.cgi?id=169629

Reviewed by Joseph Pecoraro.

This was encountered while trying to compile new protocol definitions that support the Actions API.

  • inspector/scripts/codegen/models.py:

(EnumType.repr): Improve debug logging so fields match the class member names.

  • inspector/scripts/codegen/objc_generator.py:

(ObjCGenerator.payload_to_objc_expression_for_member):
If the array elements are actually a primitive type, then there's no need to do any
conversion from a payload. This happens for free since the payload is a tree of
NSDictionary, NSString, NSNumber, etc.

  • inspector/scripts/tests/generic/expected/shadowed-optional-type-setters.json-result:
  • inspector/scripts/tests/generic/expected/type-declaration-object-type.json-result:

Rebaseline.

  • inspector/scripts/tests/generic/type-declaration-object-type.json:

Add new cases for properties that contain an array with enum type references and an array of anonymous enums.

4:41 PM Changeset in webkit [213959] by matthew_hanson@apple.com
  • 3 edits in branches/safari-603.1.30.1-branch/LayoutTests

Merge r213953. rdar://problem/31049877

4:41 PM Changeset in webkit [213958] by matthew_hanson@apple.com
  • 2 edits in branches/safari-603.1.30.1-branch/Source/WebCore

Merge r213949. rdar://problem/31049877

4:41 PM Changeset in webkit [213957] by matthew_hanson@apple.com
  • 5 edits in branches/safari-603.1.30.1-branch/Source

Versioning.

4:35 PM Changeset in webkit [213956] by matthew_hanson@apple.com
  • 3 edits in branches/safari-603.1.30.0-branch/LayoutTests

Merge r213953. rdar://problem/31049771

4:34 PM Changeset in webkit [213955] by matthew_hanson@apple.com
  • 2 edits in branches/safari-603.1.30.0-branch/Source/WebCore

Merge r213949. rdar://problem/31049771

4:18 PM Changeset in webkit [213954] by timothy_horton@apple.com
  • 4 edits in trunk/Source/WebKit2

Let PDFLayerController drive cursor updates so that it's correct more often
​https://bugs.webkit.org/show_bug.cgi?id=169626
<rdar://problem/30762943>

Reviewed by Simon Fraser.

  • WebProcess/Plugins/PDF/DeprecatedPDFLayerControllerSPI.h:
  • WebProcess/Plugins/PDF/DeprecatedPDFPlugin.h:
  • WebProcess/Plugins/PDF/DeprecatedPDFPlugin.mm:

(-[WKPDFLayerControllerDelegate setMouseCursor:]):
(WebKit::PDFPlugin::handleMouseEvent):
(WebKit::PDFPlugin::handleMouseEnterEvent):
(WebKit::pdfLayerControllerCursorTypeToCursor):
(WebKit::PDFPlugin::notifyCursorChanged):
Disable updateCursor and use the setMouseCursor delegate method when possible.

4:17 PM Changeset in webkit [213953] by aestes@apple.com
  • 3 edits in trunk/LayoutTests

Update ApplePaySession.html after r213949

Rubber-stamped by Beth Dakin.

  • http/tests/ssl/applepay/ApplePaySession-expected.txt:
  • http/tests/ssl/applepay/ApplePaySession.html:
4:06 PM Changeset in webkit [213952] by rniwa@webkit.org
  • 31 edits
    1 move
    1 delete in trunk/Websites/perf.webkit.org

Rename RootSet to CommitSet
​https://bugs.webkit.org/show_bug.cgi?id=169580

Rubber-stamped by Chris Dumez.

Renamed root_sets to commit_sets and roots to commit_set_relationships in the database schema, and renamed
related classes in public/v3/ and tools accordingly.

RootSet, MeasurementRootSet, and CustomRootSet are respectively renamed to CommitSet, MeasurementCommitSet,
and CustomCommitSet.

In order to migrate the database, run:
`
BEGIN;
ALTER TABLE root_sets RENAME TO commit_sets;
ALTER TABLE commit_sets RENAME COLUMN rootset_id TO commitset_id;
ALTER TABLE roots RENAME TO commit_set_relationships;
ALTER TABLE commit_set_relationships RENAME COLUMN root_set TO commitset_set;
ALTER TABLE commit_set_relationships RENAME COLUMN root_commit TO commitset_commit;
ALTER TABLE build_requests RENAME COLUMN request_root_set TO request_commit_set;
END;
`

  • browser-tests/index.html:
  • init-database.sql:
  • public/api/build-requests.php:

(main):

  • public/api/test-groups.php:

(main):
(format_test_group):

  • public/include/build-requests-fetcher.php:

(BuildRequestsFetcher::construct):
(BuildRequestsFetcher::results_internal):
(BuildRequestsFetcher::commit_sets): Renamed from root_sets.
(BuildRequestsFetcher::commits): Renamed from roots.
(BuildRequestsFetcher::fetch_commits_for_set_if_needed): Renamed from fetch_roots_for_set_if_needed.

  • public/privileged-api/create-test-group.php:

(main):
(ensure_commit_sets): Renamed from commit_sets_from_root_sets.

  • public/v3/components/analysis-results-viewer.js:

(AnalysisResultsViewer.prototype.buildRowGroups):
(AnalysisResultsViewer.prototype._collectCommitSetsInTestGroups): Renamed from _collectRootSetsInTestGroups.
(AnalysisResultsViewer.prototype._buildRowsForPointsAndTestGroups):
(AnalysisResultsViewer.prototype._buildRowsForPointsAndTestGroups):
(AnalysisResultsViewer.CommitSetInTestGroup): Renamed from RootSetInTestGroup.
(AnalysisResultsViewer.CommitSetInTestGroup.prototype.constructor):
(AnalysisResultsViewer.CommitSetInTestGroup.prototype.commitSet): Renamed from rootSet.
(AnalysisResultsViewer.CommitSetInTestGroup.prototype.succeedingCommitSet): Renamed from succeedingRootSet.
(AnalysisResultsViewer.TestGroupStackingBlock.prototype.constructor):
(AnalysisResultsViewer.TestGroupStackingBlock.prototype.addRowIndex):
(AnalysisResultsViewer.TestGroupStackingBlock.prototype.isComplete):
(AnalysisResultsViewer.TestGroupStackingBlock.prototype.startRowIndex):
(AnalysisResultsViewer.TestGroupStackingBlock.prototype.endRowIndex):
(AnalysisResultsViewer.TestGroupStackingBlock.prototype._computeTestGroupStatus):

  • public/v3/components/chart-revision-range.js:

(ChartRevisionRange.prototype._revisionForPoint):
(ChartRevisionRange.prototype._computeRevisionList):

  • public/v3/components/customizable-test-group-form.js:

(CustomizableTestGroupForm.prototype.constructor):
(CustomizableTestGroupForm.prototype.setCommitSetMap): Renamed from setRootSetMap.
(CustomizableTestGroupForm.prototype._submitted):
(CustomizableTestGroupForm.prototype._computeCommitSetMap): Renamed from _computeRootSetMap.
(CustomizableTestGroupForm.prototype.render): Renamed from render.
(CustomizableTestGroupForm.prototype._constructRevisionRadioButtons):

  • public/v3/components/results-table.js:

(ResultsTable.prototype.render):
(ResultsTable.prototype._createRevisionListCells):
(ResultsTable.prototype._computeRepositoryList):
(ResultsTableRow.prototype.constructor):
(ResultsTableRow.prototype.commitSet): Renamed from rootSet.

  • public/v3/components/test-group-results-table.js:

(TestGroupResultsTable.prototype.buildRowGroups):

  • public/v3/index.html:
  • public/v3/models/build-request.js:

(BuildRequest.prototype.constructor):
(BuildRequest.prototype.updateSingleton):
(BuildRequest.prototype.commitSet): Renamed from rootSet.
(BuildRequest.constructBuildRequestsFromData):

  • public/v3/models/commit-set.js: Renamed from public/v3/models/root-set.js.

(CommitSet): Renamed from RootSet.
(CommitSet.containsMultipleCommitsForRepository):
(MeasurementCommitSet): Renamed from MeasurementRootSet.
(MeasurementCommitSet.prototype.namedStaticMap):
(MeasurementCommitSet.prototype.ensureNamedStaticMap):
(MeasurementCommitSet.namedStaticMap):
(MeasurementCommitSet.ensureNamedStaticMap):
(MeasurementCommitSet.ensureSingleton):
(CustomCommitSet): Renamed from CustomRootSet.

  • public/v3/models/measurement-adaptor.js:

(MeasurementAdaptor.prototype.applyTo):

  • public/v3/models/test-group.js:

(TestGroup.prototype.constructor):
(TestGroup.prototype.addBuildRequest):
(TestGroup.prototype.repetitionCount):
(TestGroup.prototype.requestedCommitSets): Renamed from requestedRootSets.
(TestGroup.prototype.requestsForCommitSet): Renamed from requestsForRootSet.
(TestGroup.prototype.labelForCommitSet): Renamed from labelForRootSet.
(TestGroup.prototype.didSetResult):
(TestGroup.prototype.compareTestResults):
(TestGroup.prototype._valuesForCommitSet): Renamed from _valuesForRootSet.
(TestGroup.prototype.createAndRefetchTestGroups):

  • public/v3/pages/analysis-task-page.js:

(AnalysisTaskPage.prototype.render):
(AnalysisTaskPage.prototype._retryCurrentTestGroup):
(AnalysisTaskPage.prototype._createNewTestGroupFromChart):
(AnalysisTaskPage.prototype._createNewTestGroupFromViewer):
(AnalysisTaskPage.prototype._createTestGroupAfterVerifyingCommitSetList):

  • server-tests/api-build-requests-tests.js:
  • server-tests/resources/mock-data.js:

(MockData.resetV3Models):
(MockData.addMockData):
(MockData.addAnotherMockTestGroup):

  • tools/detect-changes.js:

(createAnalysisTaskAndNotify):

  • tools/js/buildbot-syncer.js:

(BuildbotSyncer.prototype._propertiesForBuildRequest):
(BuildbotSyncer.prototype._revisionSetFromCommitSetWithExclusionList):

  • tools/js/database.js:

(tableToPrefixMap):

  • tools/js/v3-models.js:
  • tools/sync-buildbot.js:

(syncLoop):

  • tools/sync-with-buildbot.py: Deleted. No longer used.
  • unit-tests/analysis-task-tests.js:
  • unit-tests/build-request-tests.js:

(sampleBuildRequestData):

  • unit-tests/buildbot-syncer-tests.js:

(sampleCommitSetData):

  • unit-tests/measurement-adaptor-tests.js:
  • unit-tests/measurement-set-tests.js:
  • unit-tests/resources/mock-v3-models.js:

(MockModels.inject):

  • unit-tests/test-groups-tests.js:

(sampleTestGroup):

4:03 PM Changeset in webkit [213951] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Mark imported/w3c/web-platform-tests/html/webappapis/scripting/events/event-handler-javascript.html as failing on ios-simulator-wk2.
​https://bugs.webkit.org/show_bug.cgi?id=169640

Unreviewed test gardening.

  • platform/ios-simulator-wk2/TestExpectations:
3:59 PM Changeset in webkit [213950] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Mark imported/w3c/web-platform-tests/html/webappapis/scripting/events/event-handler-javascript.html as failing on mac-wk2.
​https://bugs.webkit.org/show_bug.cgi?id=169640

Unreviewed test gardening.

  • platform/mac-wk2/TestExpectations:
3:53 PM Changeset in webkit [213949] by aestes@apple.com
  • 2 edits in trunk/Source/WebCore

REGRESSION (r209760): Apple Pay doesn't work on sites that specify empty contact fields
​https://bugs.webkit.org/show_bug.cgi?id=169639
<rdar://problem/30957789>

Reviewed by Anders Carlsson.

Shipping and billing contact fields are allowed to be empty.

  • Modules/applepay/ApplePaySession.cpp:

(WebCore::convertAndValidate):

3:38 PM Changeset in webkit [213948] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebKit2

Extend WKPreferences to include preferences for testing MediaCapture.
​https://bugs.webkit.org/show_bug.cgi?id=169560

Patch by Andrew Gold <​agold@apple.com> on 2017-03-14
Reviewed by Youenn Fablet.

  • UIProcess/API/Cocoa/WKPreferences.mm:

(-[WKPreferences _mockCaptureDevicesEnabled]):
(-[WKPreferences _setMockCaptureDevicesEnabled:]):
(-[WKPreferences _mediaCaptureRequiresSecureConnection]):
(-[WKPreferences _setMediaCaptureRequiresSecureConnection:]):
(-[WKPreferences _enumeratingAllNetworkInterfacesEnabled]):
(-[WKPreferences _setEnumeratingAllNetworkInterfacesEnabled:]):
(-[WKPreferences _iceCandidateFiltertingEnabled]):
(-[WKPreferences _setICECandidateFilteringEnabled:]):

  • UIProcess/API/Cocoa/WKPreferencesPrivate.h:
3:36 PM Changeset in webkit [213947] by beidson@apple.com
  • 14 edits
    5 moves in trunk

Rename WKHTTPCookieStorage to WKHTTPCookieStore.
​https://bugs.webkit.org/show_bug.cgi?id=169630

Reviewed by Tim Horton.

Source/WebKit2:

  • CMakeLists.txt:
  • Shared/API/APIObject.h:
  • Shared/Cocoa/APIObject.mm:

(API::Object::newObject):

  • UIProcess/API/APIHTTPCookieStore.cpp: Renamed from Source/WebKit2/UIProcess/API/APIHTTPCookieStorage.cpp.

(API::HTTPCookieStore::HTTPCookieStore):
(API::HTTPCookieStore::~HTTPCookieStore):
(API::HTTPCookieStore::cookies):
(API::HTTPCookieStore::setCookie):
(API::HTTPCookieStore::setCookies):
(API::HTTPCookieStore::deleteCookie):
(API::HTTPCookieStore::removeCookiesSinceDate):
(API::HTTPCookieStore::setHTTPCookieAcceptPolicy):
(API::HTTPCookieStore::getHTTPCookieAcceptPolicy):

  • UIProcess/API/APIHTTPCookieStore.h: Renamed from Source/WebKit2/UIProcess/API/APIHTTPCookieStorage.h.
  • UIProcess/API/APIWebsiteDataStore.cpp:

(API::WebsiteDataStore::httpCookieStore):
(API::WebsiteDataStore::httpCookieStorage): Deleted.

  • UIProcess/API/APIWebsiteDataStore.h:
  • UIProcess/API/Cocoa/WKHTTPCookieStore.h: Renamed from Source/WebKit2/UIProcess/API/Cocoa/WKHTTPCookieStorage.h.
  • UIProcess/API/Cocoa/WKHTTPCookieStore.mm: Renamed from Source/WebKit2/UIProcess/API/Cocoa/WKHTTPCookieStorage.mm.

(coreCookiesToNSCookies):
(-[WKHTTPCookieStore dealloc]):
(-[WKHTTPCookieStore fetchCookies:]):
(-[WKHTTPCookieStore fetchCookiesForURL:completionHandler:]):
(-[WKHTTPCookieStore setCookie:completionHandler:]):
(-[WKHTTPCookieStore deleteCookie:completionHandler:]):
(-[WKHTTPCookieStore setCookies:forURL:mainDocumentURL:completionHandler:]):
(-[WKHTTPCookieStore removeCookiesSinceDate:completionHandler:]):
(-[WKHTTPCookieStore setCookieAcceptPolicy:completionHandler:]):
(kitCookiePolicyToNSCookiePolicy):
(-[WKHTTPCookieStore fetchCookieAcceptPolicy:]):
(-[WKHTTPCookieStore _apiObject]):

  • UIProcess/API/Cocoa/WKHTTPCookieStoreInternal.h: Renamed from Source/WebKit2/UIProcess/API/Cocoa/WKHTTPCookieStorageInternal.h.

(WebKit::wrapper):

  • UIProcess/API/Cocoa/WKWebsiteDataStore.mm:

(-[WKWebsiteDataStore _httpCookieStore]):
(-[WKWebsiteDataStore _httpCookieStorage]): Deleted.

  • UIProcess/API/Cocoa/WKWebsiteDataStorePrivate.h:
  • UIProcess/WebProcessPool.cpp:
  • UIProcess/WebProcessPool.h:
  • UIProcess/WebsiteData/WebsiteDataStore.cpp:

(WebKit::WebsiteDataStore::processPoolForCookieStorageOperations):

  • WebKit2.xcodeproj/project.pbxproj:

Tools:

  • TestWebKitAPI/Tests/WebKit2Cocoa/WKHTTPCookieStorage.mm:

(TEST):

3:32 PM Changeset in webkit [213946] by andersca@apple.com
  • 6 edits in trunk/Source

Propagate PassKit errors
​https://bugs.webkit.org/show_bug.cgi?id=169633
rdar://problem/31043392

Reviewed by Dean Jackson.

Source/WebCore:

  • Modules/applepay/ApplePayError.idl:
  • Modules/applepay/ApplePaySession.cpp:

(WebCore::convert):
(WebCore::convertAndValidate):

  • Modules/applepay/PaymentRequest.h:

Source/WebKit2:

  • UIProcess/ApplePay/cocoa/WebPaymentCoordinatorProxyCocoa.mm:

(WebKit::toNSError):

3:23 PM Changeset in webkit [213945] by fpizlo@apple.com
  • 2 edits in trunk/Source/WTF

Fix some typos in this benchmark.

Rubber stamped by Saam Barati.

  • benchmarks/HashSetDFGReplay.cpp:

(main):

2:58 PM Changeset in webkit [213944] by Alan Bujtas
  • 4 edits
    2 adds in trunk

Simple line layout: Adjust hyphenation constrains based on the normal line layout line-breaking logic.
​https://bugs.webkit.org/show_bug.cgi?id=169617

Source/WebCore:

Reviewed by Antti Koivisto.

This patch ensures that simple line layout ends up with the same hyphenation context as normal line layout.

Test: fast/text/simple-line-layout-hyphenation-constrains.html

  • rendering/SimpleLineLayout.cpp:

(WebCore::SimpleLineLayout::hyphenPositionForFragment): see webkit.org/b/169613
(WebCore::SimpleLineLayout::splitFragmentToFitLine):

  • rendering/line/BreakingContext.h: Integral -> fractional.

(WebCore::tryHyphenating):

LayoutTests:

Reviewed by Antti Koivisto.

  • fast/text/simple-line-layout-hyphenation-constrains-expected.html: Added.
  • fast/text/simple-line-layout-hyphenation-constrains.html: Added.
2:48 PM Changeset in webkit [213943] by matthew_hanson@apple.com
  • 1 copy in tags/Safari-603.1.30.0.34

Tag Safari-603.1.30.0.34.

2:41 PM Changeset in webkit [213942] by webkit@devinrousso.com
  • 5 edits in trunk/Source/WebInspectorUI

Web Inspector: RTL: add support for Memory timeline
​https://bugs.webkit.org/show_bug.cgi?id=169584

Reviewed by Brian Burg.

  • UserInterface/Views/MemoryCategoryView.css:

(.memory-category-view > .details):
(body[dir=ltr] .memory-category-view > .details):
(body[dir=rtl] .memory-category-view > .details):

  • UserInterface/Views/MemoryTimelineOverviewGraph.css:

(.timeline-overview-graph.memory > .legend):
(body[dir=ltr] .timeline-overview-graph.memory > .legend):
(body[dir=rtl] .timeline-overview-graph.memory > .legend):
(.timeline-overview-graph.memory .memory-pressure-event):
(body[dir=ltr] .timeline-overview-graph.memory .memory-pressure-event):
(body[dir=rtl] .timeline-overview-graph.memory .memory-pressure-event):

  • UserInterface/Views/MemoryTimelineOverviewGraph.js:

(WebInspector.MemoryTimelineOverviewGraph.prototype.layout):
Rework the calculation for the marker offset to use "right" in RTL.

  • UserInterface/Views/MemoryTimelineView.css:

(.timeline-view.memory > .content > .details > .timeline-ruler):
(body[dir=ltr] .timeline-view.memory > .content > .details > .timeline-ruler):
(body[dir=rtl] .timeline-view.memory > .content > .details > .timeline-ruler):
(.timeline-view.memory > .content > .overview > .divider):
(body[dir=ltr] .timeline-view.memory > .content > .overview > .divider):
(body[dir=rtl] .timeline-view.memory > .content > .overview > .divider):
(.timeline-view.memory .legend):
(body[dir=ltr] .timeline-view.memory .legend):
(body[dir=rtl] .timeline-view.memory .legend):
(.timeline-view.memory .legend > .row):
(.timeline-view.memory .legend > .row > .swatch):
(body[dir=ltr] .timeline-view.memory .legend > .row > .swatch):
(body[dir=rtl] .timeline-view.memory .legend > .row > .swatch):
(body[dir=ltr] .timeline-view.memory .legend > .row > :matches(.label, .size)):
(body[dir=rtl] .timeline-view.memory .legend > .row > :matches(.label, .size)):
(.timeline-view.memory .legend > .row > .label):
(.timeline-view.memory .legend > .row > .size):

2:41 PM Changeset in webkit [213941] by jer.noble@apple.com
  • 25 edits
    2 adds in trunk/Source

Refactor: Allow WebKit2 to override the creation of RealtimeMediaSources
​https://bugs.webkit.org/show_bug.cgi?id=169227

Reviewed by Eric Carlson.

Source/WebCore:

Allow clients of RealtimeMediaSourceCenter to specify a factory for creating
RealtimeMediaSources, to be used by subclasess of RealtimeMediaSourceCenter. Add virtual
methods to retrieve the "default" factories for the RealtimeMediaSourceCenter subclass. The
requires moving the creation of sources up from CaptureDeviceManager into
RealtimeMediaSourceCenterMac, and the addition of factory methods to AVAudioCaptureSource
and AVVideoCaptureSource.

  • platform/mediastream/CaptureDeviceManager.cpp:

(CaptureDeviceManager::deviceWithUID):
(CaptureDeviceManager::bestSourcesForTypeAndConstraints): Deleted.
(CaptureDeviceManager::sourceWithUID): Deleted.

  • platform/mediastream/CaptureDeviceManager.h:
  • platform/mediastream/RealtimeMediaSource.h:
  • platform/mediastream/RealtimeMediaSourceCenter.cpp:

(WebCore::RealtimeMediaSourceCenter::setAudioFactory):
(WebCore::RealtimeMediaSourceCenter::unsetAudioFactory):
(WebCore::RealtimeMediaSourceCenter::setVideoFactory):
(WebCore::RealtimeMediaSourceCenter::unsetVideoFactory):

  • platform/mediastream/RealtimeMediaSourceCenter.h:

(WebCore::RealtimeMediaSourceCenter::audioFactory):
(WebCore::RealtimeMediaSourceCenter::videoFactory):

  • platform/mediastream/mac/AVAudioCaptureSource.h:
  • platform/mediastream/mac/AVAudioCaptureSource.mm:

(WebCore::AVAudioCaptureSource::factory):

  • platform/mediastream/mac/AVCaptureDeviceManager.h:
  • platform/mediastream/mac/AVCaptureDeviceManager.mm:

(WebCore::AVCaptureDeviceManager::createMediaSourceForCaptureDeviceWithConstraints): Deleted.

  • platform/mediastream/mac/AVMediaCaptureSource.h:
  • platform/mediastream/mac/AVVideoCaptureSource.h:
  • platform/mediastream/mac/AVVideoCaptureSource.mm:

(WebCore::AVVideoCaptureSource::factory):

  • platform/mediastream/mac/RealtimeMediaSourceCenterMac.cpp:

(WebCore::RealtimeMediaSourceCenterMac::RealtimeMediaSourceCenterMac):
(WebCore::RealtimeMediaSourceCenterMac::validateRequestConstraints):
(WebCore::RealtimeMediaSourceCenterMac::createMediaStream):
(WebCore::RealtimeMediaSourceCenterMac::bestSourcesForTypeAndConstraints):
(WebCore::RealtimeMediaSourceCenterMac::defaultAudioFactory):
(WebCore::RealtimeMediaSourceCenterMac::defaultVideoFactory):

  • platform/mediastream/mac/RealtimeMediaSourceCenterMac.h:
  • platform/mock/MockRealtimeAudioSource.cpp:

(WebCore::MockRealtimeAudioSource::factory):

  • platform/mock/MockRealtimeAudioSource.h:
  • platform/mock/MockRealtimeMediaSourceCenter.cpp:

(WebCore::MockRealtimeMediaSourceCenter::defaultAudioFactory):
(WebCore::MockRealtimeMediaSourceCenter::defaultVideoFactory):

  • platform/mock/MockRealtimeMediaSourceCenter.h:
  • platform/mock/MockRealtimeVideoSource.cpp:

(WebCore::MockRealtimeVideoSource::factory):

  • platform/mock/MockRealtimeVideoSource.h:

Source/WebKit2:

  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/WebCoreSupport/WebUserMediaClient.cpp:

(WebKit::WebUserMediaClient::WebUserMediaClient):
(WebKit::WebUserMediaClient::initializeFactories): Add empty non-Cocoa implementation.

  • WebProcess/WebCoreSupport/WebUserMediaClient.h:
  • WebProcess/WebCoreSupport/cocoa/WebUserMediaClientMac.mm: Added.

(WebKit::WebUserMediaClient::initializeFactories): Initialize a (for now) pass-through factory.

2:39 PM Changeset in webkit [213940] by dino@apple.com
  • 10 edits in trunk/Source

Rename LayerTypeWebGLLayer and use it for both WebGL and WebGPU
​https://bugs.webkit.org/show_bug.cgi?id=169628
<rdar://problems/31047025>

Reviewed by Simon Fraser.

Rename LayerTypeWebGLLayer to LayerTypeContentsProvidedLayer
and use it for both WebGLLayer and WebGPULayer, to avoid
code duplication.

Source/WebCore:

  • platform/graphics/ca/PlatformCALayer.cpp:

(WebCore::operator<<):

  • platform/graphics/ca/PlatformCALayer.h:
  • platform/graphics/ca/cocoa/PlatformCALayerCocoa.mm:

(PlatformCALayerCocoa::layerTypeForPlatformLayer):
(PlatformCALayerCocoa::PlatformCALayerCocoa):
(PlatformCALayerCocoa::commonInit):

Source/WebKit2:

  • Shared/mac/RemoteLayerBackingStore.mm:

(WebKit::RemoteLayerBackingStore::drawInContext):

  • Shared/mac/RemoteLayerTreeTransaction.mm:

(WebKit::RemoteLayerTreeTransaction::description):

  • UIProcess/ios/RemoteLayerTreeHostIOS.mm:

(WebKit::RemoteLayerTreeHost::createLayer):

  • UIProcess/mac/RemoteLayerTreeHost.mm:

(WebKit::RemoteLayerTreeHost::createLayer):

  • WebProcess/WebPage/mac/PlatformCALayerRemoteCustom.mm:

(WebKit::PlatformCALayerRemoteCustom::PlatformCALayerRemoteCustom):
(WebKit::PlatformCALayerRemoteCustom::clone):

2:37 PM Changeset in webkit [213939] by fpizlo@apple.com
  • 13 edits
    7 adds in trunk/Source

Record the HashSet/HashMap operations in DFG/FTL/B3 and replay them in a benchmark
​https://bugs.webkit.org/show_bug.cgi?id=169590

Reviewed by Saam Barati.

Source/JavaScriptCore:

Adds code to support logging some hashtable stuff in the DFG.

  • dfg/DFGAvailabilityMap.cpp:

(JSC::DFG::AvailabilityMap::pruneHeap):

  • dfg/DFGCombinedLiveness.cpp:

(JSC::DFG::liveNodesAtHead):
(JSC::DFG::CombinedLiveness::CombinedLiveness):

  • dfg/DFGCombinedLiveness.h:
  • dfg/DFGLivenessAnalysisPhase.cpp:

(JSC::DFG::LivenessAnalysisPhase::run):
(JSC::DFG::LivenessAnalysisPhase::processBlock):

  • dfg/DFGNode.cpp:
  • dfg/DFGNode.h:
  • dfg/DFGObjectAllocationSinkingPhase.cpp:

Source/WTF:

This adds LoggingHashSet and LoggingHashMap, which are drop-in replacements for HashSet and
HashMap that log everything that they do, so that you can replay it later.

This also adds a benchmark (HashSetDFGReplay) based on doing a recording of some of the HashSets
in the DFG compiler.

  • WTF.xcodeproj/project.pbxproj:
  • benchmarks/HashSetDFGReplay.cpp: Added.

(benchmark):
(main):

  • wtf/CMakeLists.txt:
  • wtf/GlobalVersion.cpp: Added.

(WTF::newGlobalVersion):

  • wtf/GlobalVersion.h: Added.
  • wtf/HashMap.h:

(WTF::X>::swap):

  • wtf/HashSet.h:

(WTF::V>::addVoid):

  • wtf/LoggingHashID.h: Added.

(WTF::LoggingHashID::LoggingHashID):
(WTF::LoggingHashID::dump):

  • wtf/LoggingHashMap.h: Added.
  • wtf/LoggingHashSet.h: Added.
  • wtf/LoggingHashTraits.h: Added.

(WTF::LoggingHashKeyTraits::print):
(WTF::LoggingHashValueTraits::print):

2:16 PM Changeset in webkit [213938] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Mark imported/w3c/web-platform-tests/cors/status.htm as flaky.
​https://bugs.webkit.org/show_bug.cgi?id=169625

Unreviewed test gardening.

1:55 PM Changeset in webkit [213937] by matthew_hanson@apple.com
  • 5 edits in branches/safari-603.1.30.0-branch/Source

‘Versioning.’

1:50 PM Changeset in webkit [213936] by jer.noble@apple.com
  • 9 edits in trunk/Source

Make classes used by Media Stream encode/decode friendly
​https://bugs.webkit.org/show_bug.cgi?id=169567

Reviewed by Eric Carlson.

Source/WebCore:

Add encode() and decode() methods and implementations to a variety of media stream related classes.

  • platform/audio/mac/CAAudioStreamDescription.h:

(WebCore::CAAudioStreamDescription::encode):
(WebCore::CAAudioStreamDescription::decode):

  • platform/mediastream/RealtimeMediaSourceSettings.h:

(WebCore::RealtimeMediaSourceSettings::width):
(WebCore::RealtimeMediaSourceSettings::setWidth):
(WebCore::RealtimeMediaSourceSettings::height):
(WebCore::RealtimeMediaSourceSettings::setHeight):
(WebCore::RealtimeMediaSourceSettings::sampleRate):
(WebCore::RealtimeMediaSourceSettings::setSampleRate):
(WebCore::RealtimeMediaSourceSettings::sampleSize):
(WebCore::RealtimeMediaSourceSettings::setSampleSize):
(WebCore::RealtimeMediaSourceSettings::encode):
(WebCore::RealtimeMediaSourceSettings::decode):

  • platform/mediastream/RealtimeMediaSourceSupportedConstraints.h:

(WebCore::RealtimeMediaSourceSupportedConstraints::encode):
(WebCore::RealtimeMediaSourceSupportedConstraints::decode):

Source/WebKit2:

Make the encoder and decoder for MediaConstraintsData a little less wordy.

  • Shared/WebCoreArgumentCoders.cpp:

(IPC::ArgumentCoder<MediaConstraintsData>::encode):
(IPC::ArgumentCoder<MediaConstraintsData>::decode):

Source/WTF:

  • wtf/MediaTime.h:

(WTF::MediaTime::encode):
(WTF::MediaTime::decode):

1:45 PM Changeset in webkit [213935] by jer.noble@apple.com
  • 5 edits
    2 adds in trunk/Source

Adapt CARingBuffer to be usable across processes
​https://bugs.webkit.org/show_bug.cgi?id=169591

Reviewed by Alex Christensen.

Source/WebCore:

When used with a SharedMemory backing store, storing the pointers to channel data at the beginning
of the channel data itself is problematic: when the SharedMemory is mapped on the far side of the
process boundary, it will not exist at the same memory location as it did on the near side. Instead
of storing these pointers inside the channel data, store them in a small (usually 1 or 2 entry) vector
recreated when the backing store is (re-)allocated.

  • platform/audio/mac/CARingBuffer.cpp:

(WebCore::CARingBuffer::CARingBuffer):
(WebCore::CARingBuffer::allocate):
(WebCore::CARingBuffer::deallocate):
(WebCore::ZeroRange):
(WebCore::StoreABL):
(WebCore::FetchABL):
(WebCore::CARingBuffer::store):
(WebCore::CARingBuffer::getCurrentFrameBounds):
(WebCore::CARingBuffer::fetch):

  • platform/audio/mac/CARingBuffer.h:

Source/WebKit2:

Add a new class which wraps a SharedMemory object and uses that shared memory as the
backing store of a CARingBuffer. This backing store can be set to "read only", which
prevents the backing from being de- or re-allocated.

  • WebKit2.xcodeproj/project.pbxproj:
  • Shared/Cocoa/SharedRingBufferStorage.cpp: Added.

(WebKit::SharedRingBufferStorage::setStorage):
(WebKit::SharedRingBufferStorage::setReadOnly):
(WebKit::SharedRingBufferStorage::allocate):
(WebKit::SharedRingBufferStorage::deallocate):
(WebKit::SharedRingBufferStorage::data):

  • Shared/Cocoa/SharedRingBufferStorage.h: Added.

(WebKit::SharedRingBufferStorage::SharedRingBufferStorage):
(WebKit::SharedRingBufferStorage::invalidate):
(WebKit::SharedRingBufferStorage::storage):
(WebKit::SharedRingBufferStorage::readOnly):

1:07 PM Changeset in webkit [213934] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Mark imported/w3c/web-platform-tests/IndexedDB/fire-error-event-exception.html as flaky on mac-wk2.
​https://bugs.webkit.org/show_bug.cgi?id=169621

Unreviewed test gardening.

  • platform/mac-wk2/TestExpectations:
1:03 PM Changeset in webkit [213933] by eric.carlson@apple.com
  • 3 edits in trunk/Source/WebKit2

[iOS] The web process should inherit application state from UI process
​https://bugs.webkit.org/show_bug.cgi?id=169156
<rdar://problem/30845473>

Reviewed by Brady Eidson.

Move PID proxy setup to platformInitializeWebProcess as suggested in
post-landing feedback.

  • WebProcess/WebPage/ios/WebPageIOS.mm:

(WebKit::WebPage::platformInitialize):

  • WebProcess/cocoa/WebProcessCocoa.mm:

(WebKit::WebProcess::platformInitializeWebProcess):

1:01 PM Changeset in webkit [213932] by commit-queue@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

Web Inspector: Remove unused Network protocol event
​https://bugs.webkit.org/show_bug.cgi?id=169619

Patch by Joseph Pecoraro <Joseph Pecoraro> on 2017-03-14
Reviewed by Mark Lam.

  • inspector/protocol/Network.json:

This became unused in r213621 and should have been removed
from the protocol file then.

12:57 PM Changeset in webkit [213931] by jer.noble@apple.com
  • 3 edits in trunk/Source/WebCore

Pulling more frames from AudioSampleDataSource than the last push added will always fail.
​https://bugs.webkit.org/show_bug.cgi?id=168644

Reviewed by Eric Carlson.

Rather than use the delta between the ring buffer's end time and the last pushed timestamp
(or what is effectively the number of samples in the last push operation) to determine if
there is enough buffered data to satisfy a pull operation, use the ring buffer's actual
buffered duration.

Then, instead of saving the last pushed timestamp, explicitly save the last push count, and
use that data to inform how much to offset the output timestamps (or what is effectively how
much to allow the source to pre-buffer).

  • platform/audio/mac/AudioSampleDataSource.cpp:

(WebCore::AudioSampleDataSource::pushSamplesInternal):
(WebCore::AudioSampleDataSource::pullSamplesInternal):

  • platform/audio/mac/AudioSampleDataSource.h:
12:29 PM Changeset in webkit [213930] by mark.lam@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

Add a null check in VMTraps::willDestroyVM() to handle a race condition.
​https://bugs.webkit.org/show_bug.cgi?id=169620

Reviewed by Filip Pizlo.

There exists a race between VMTraps::willDestroyVM() (which removed SignalSenders
from its m_signalSenders list) and SignalSender::send() (which removes itself
from the list). In the event that SignalSender::send() removes itself between
the time that VMTraps::willDestroyVM() checks if m_signalSenders is empty and the
time it takes a sender from m_signalSenders, VMTraps::willDestroyVM() may end up
with a NULL sender pointer. The fix is to add the missing null check before using
the sender pointer.

  • runtime/VMTraps.cpp:

(JSC::VMTraps::willDestroyVM):
(JSC::VMTraps::fireTrap):

  • runtime/VMTraps.h:
12:22 PM Changeset in webkit [213929] by Megan Gardner
  • 4 edits in trunk/Source

Correctly export WebItemProviderPasteboard
​https://bugs.webkit.org/show_bug.cgi?id=169578

Reviewed by Tim Horton.

Source/WebCore:

  • platform/ios/WebItemProviderPasteboard.h:

Source/WebKit/mac:

  • MigrateHeaders.make:
12:20 PM Changeset in webkit [213928] by webkit@devinrousso.com
  • 7 edits in trunk/Source/WebInspectorUI

Web Inspector: RTL: add support for Timeline graphs
​https://bugs.webkit.org/show_bug.cgi?id=169585

Reviewed by Brian Burg.

  • UserInterface/Views/HeapAllocationsTimelineOverviewGraph.js:

(WebInspector.HeapAllocationsTimelineOverviewGraph.prototype.layout):
Rework the calculation for the icon offset to use "right" in RTL.

  • UserInterface/Views/MemoryTimelineOverviewGraph.css:

(body[dir=rtl] .timeline-overview-graph.memory > .stacked-line-chart):
Flip the chart when in RTL.

  • UserInterface/Views/TimelineRecordBar.css:

(.timeline-record-bar):
(body[dir=ltr] .timeline-record-bar > .segment.inactive):
(body[dir=rtl] .timeline-record-bar > .segment.inactive):
(body[dir=ltr] .timeline-record-bar.unfinished > .segment):
(body[dir=rtl] .timeline-record-bar.unfinished > .segment):
(body[dir=ltr] .timeline-record-bar.has-inactive-segment > .segment:not(.inactive)):
(body[dir=rtl] .timeline-record-bar.has-inactive-segment > .segment:not(.inactive)):
(body[dir=ltr] :matches(:focus, .force-focus) .selected .timeline-record-bar.has-inactive-segment > .segment:not(.inactive)):
(body[dir=rtl] :matches(:focus, .force-focus) .selected .timeline-record-bar.has-inactive-segment > .segment:not(.inactive)):
(.timeline-record-bar > .segment.inactive,): Deleted.
(.timeline-record-bar.has-inactive-segment > .segment:not(.inactive)): Deleted.
(:matches(:focus, .force-focus) .selected .timeline-record-bar.has-inactive-segment > .segment:not(.inactive)): Deleted.

  • UserInterface/Views/TimelineRecordBar.js:

(WebInspector.TimelineRecordBar.prototype.refresh):
Apply the position updates to "right" in RTL.

  • UserInterface/Views/TimelineRecordFrame.js:

(WebInspector.TimelineRecordFrame.prototype.refresh):
Apply the position updates to "right" in RTL.

  • UserInterface/Views/TimelineRuler.js:

(WebInspector.TimelineRuler.prototype.layout):
(WebInspector.TimelineRuler.prototype._updatePositionOfElement):
(WebInspector.TimelineRuler.prototype._updateMarkers):
(WebInspector.TimelineRuler.prototype._updateSelection):
Apply the position updates to "right" in RTL.

12:16 PM Changeset in webkit [213927] by Ryan Haddad
  • 6 edits in trunk/Source/WebKit2

Unreviewed, rolling out r213915.

Caused WK2 LayoutTests to exit early with timeouts.

Reverted changeset:

"Fix uninitialized public members in NetworkProcess"
​https://bugs.webkit.org/show_bug.cgi?id=169598
​http://trac.webkit.org/changeset/213915

12:16 PM Changeset in webkit [213926] by Jonathan Bedard
  • 3 edits in trunk/Tools

webkitpy: Efficient app installation for device testing
​https://bugs.webkit.org/show_bug.cgi?id=169054
<rdar://problem/30790207>

Reviewed by Daniel Bates.

We should only install an app on a device once, not every time the app is run.
Move app installation to setup.

  • Scripts/webkitpy/port/ios.py:

(IOSPort.setup_test_run): Install app to device.

  • Scripts/webkitpy/port/simulator_process.py:

(SimulatorProcess.init): Remove app installation.

12:12 PM Changeset in webkit [213925] by webkit@devinrousso.com
  • 4 edits in trunk/Source/WebInspectorUI

Web Inspector: RTL: add support for DataGrid disclosure arrows and indentation
​https://bugs.webkit.org/show_bug.cgi?id=169582

Reviewed by Brian Burg.

  • UserInterface/Views/DataGrid.css:

(body[dir=rtl] .data-grid tr.parent td.disclosure::before):
Flip the image in RTL.

  • UserInterface/Views/DataGrid.js:

(WebInspector.DataGrid.prototype._updateScrollbarPadding):
Rework the padding calculation for the scrollbar offset to use "right" in RTL.

  • UserInterface/Views/DataGridNode.js:

(WebInspector.DataGridNode.prototype.get indentPadding):
(WebInspector.DataGridNode.prototype.createCell):
Rework the padding calculation for indenting (as a child) to use "right" in RTL.

(WebInspector.DataGridNode.prototype.isEventWithinDisclosureTriangle):
Calculate the position of the ::before triangle based on the layout direction.

(WebInspector.DataGridNode.prototype.get leftPadding): Deleted.

12:07 PM Changeset in webkit [213924] by webkit@devinrousso.com
  • 2 edits in trunk/Source/WebInspectorUI

Web Inspector: RTL: add support for TimelineOverview sidebar and container layout
​https://bugs.webkit.org/show_bug.cgi?id=169583

Reviewed by Brian Burg.

  • UserInterface/Views/TimelineOverview.css:

(body[dir=ltr] .timeline-overview > :matches(.navigation-bar.timelines, .tree-outline.timelines)):
(body[dir=rtl] .timeline-overview > :matches(.navigation-bar.timelines, .tree-outline.timelines)):
(body[dir=ltr] .timeline-overview:not(.frames) > :matches(.scroll-container, .timeline-ruler, .graphs-container)):
(body[dir=rtl] .timeline-overview:not(.frames) > :matches(.scroll-container, .timeline-ruler, .graphs-container)):
(.timeline-overview > .navigation-bar.timelines):
(.timeline-overview > .tree-outline.timelines):
(.timeline-overview > .scroll-container):
(.timeline-overview > .timeline-ruler):
(.timeline-overview > .graphs-container):
(.timeline-overview > .scroll-container > .scroll-width-sizer):
(body[dir=ltr] .timeline-overview > .scroll-container > .scroll-width-sizer):
(body[dir=rtl] .timeline-overview > .scroll-container > .scroll-width-sizer):
(.timeline-overview.frames > .graphs-container): Deleted.

11:53 AM Changeset in webkit [213923] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Remove redundant check for "firstLine" in RenderBlock::lineHeight()
​https://bugs.webkit.org/show_bug.cgi?id=169610

Patch by Adrian Perez de Castro <Adrian Perez de Castro> on 2017-03-14
Reviewed by Michael Catanzaro.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::lineHeight): Remove test of "firstLine" that
was already checked in the condition for the enclosing if-clause.

11:36 AM Changeset in webkit [213922] by jmarcell@apple.com
  • 1 copy in tags/Safari-604.1.12

Tag Safari-604.1.12.

11:35 AM Changeset in webkit [213921] by jmarcell@apple.com
  • 7 edits in trunk/Source

Versioning.

11:29 AM Changeset in webkit [213920] by commit-queue@webkit.org
  • 10 edits
    3 adds in trunk

[Modern Media Controls] Fullscreen controls during Live Broadcast is completely broken
​https://bugs.webkit.org/show_bug.cgi?id=169354
<rdar://problem/30636370>

Patch by Antoine Quint <Antoine Quint> on 2017-03-14
Reviewed by Dean Jackson.

Source/WebCore:

When playing a Live Broadcast video in fullscreen, we should not show any scrubbing-related
piece of UI and ensure that we show the status label.

Test: http/tests/media/modern-media-controls/macos-fullscreen-media-controls/macos-fullscreen-media-controls-live-broadcast.html

  • Modules/modern-media-controls/controls/macos-fullscreen-media-controls.css:

(.media-controls.mac.fullscreen > .controls-bar .status-label):

  • Modules/modern-media-controls/controls/macos-fullscreen-media-controls.js:

(MacOSFullscreenMediaControls.prototype.layout):

  • Modules/modern-media-controls/media/seek-support.js:

(SeekSupport.prototype.get mediaEvents):
(SeekSupport.prototype.syncControl):

LayoutTests:

Add a new test, skipped on iOS, to check scrubbing controls are disabled in fullscreen
for a Live Broadcast video and that the status label is visible. We also rebaseline a
couple of tests now that the time control is added on first layout instead of inside
the constructor.

  • http/tests/media/modern-media-controls/macos-fullscreen-media-controls/macos-fullscreen-media-controls-live-broadcast-expected.txt: Added.
  • http/tests/media/modern-media-controls/macos-fullscreen-media-controls/macos-fullscreen-media-controls-live-broadcast.html: Added.
  • media/modern-media-controls/macos-fullscreen-media-controls/macos-fullscreen-media-controls-constructor-expected.txt:
  • media/modern-media-controls/macos-fullscreen-media-controls/macos-fullscreen-media-controls-constructor.html:
  • media/modern-media-controls/macos-fullscreen-media-controls/macos-fullscreen-media-controls-time-control-styles-expected.txt:
  • media/modern-media-controls/macos-fullscreen-media-controls/macos-fullscreen-media-controls-time-control-styles.html:
  • platform/ios-simulator/TestExpectations:
11:24 AM Changeset in webkit [213919] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Skip 12 web-platform-tests until wptserver is upgraded.
​https://bugs.webkit.org/show_bug.cgi?id=169615

Unreviewed test gardening.

11:22 AM Changeset in webkit [213918] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

Nwtr ignores ImageDiff's errors for ref tests
​https://bugs.webkit.org/show_bug.cgi?id=168033

Patch by Fujii Hironori <Fujii Hironori> on 2017-03-14
Reviewed by Alexey Proskuryakov.

Nwtr checks ImageDiff's errors only for pixel tests, but for ref
tests. Those errors of ref tests also should be checked.

In the current implementation of expected mismatch ref tests,
diff_image was called if the image hashes match. This is useless
because two images are ensured identical in that case. Calling
image_hash is considered unnecessary for expected mismatch ref
tests. Do not call diff_image for them.

As the result, check the error only for expected match ref tests.

  • Scripts/webkitpy/layout_tests/controllers/single_test_runner.py:

(SingleTestRunner._compare_image): Rename a variable 'err_str' to 'error_string'.
(SingleTestRunner._compare_output_with_reference): Do not call
diff_image for expected mismatch ref tests. Check the error and
marked the test failed for expected match ref tests.

11:19 AM Changeset in webkit [213917] by commit-queue@webkit.org
  • 13 edits in trunk/Source

Web Inspector: More accurate Resource Timing data in Web Inspector
​https://bugs.webkit.org/show_bug.cgi?id=169577

Patch by Joseph Pecoraro <Joseph Pecoraro> on 2017-03-14
Reviewed by Youenn Fablet.

Source/WebCore:

  • inspector/InspectorNetworkAgent.h:
  • inspector/InspectorNetworkAgent.cpp:

(WebCore::InspectorNetworkAgent::didFinishLoading):
Use the ResourceLoader to get the start time that responseEnd is relative to
so we can send the more accurate responseEnd when available.

  • inspector/InspectorInstrumentation.cpp:

(WebCore::InspectorInstrumentation::didFinishLoadingImpl):

  • inspector/InspectorInstrumentation.h:

(WebCore::InspectorInstrumentation::didFinishLoading):

  • loader/CrossOriginPreflightChecker.cpp:

(WebCore::CrossOriginPreflightChecker::validatePreflightResponse):

  • loader/DocumentLoader.cpp:

(WebCore::DocumentLoader::finishedLoading):

  • loader/ResourceLoadNotifier.cpp:

(WebCore::ResourceLoadNotifier::didFinishLoad):
(WebCore::ResourceLoadNotifier::dispatchDidFinishLoading):
(WebCore::ResourceLoadNotifier::sendRemainingDelegateMessages):

  • loader/ResourceLoadNotifier.h:
  • loader/appcache/ApplicationCacheGroup.cpp:

(WebCore::ApplicationCacheGroup::didFinishLoading):
Pass ResourceLoader through to Web Inspector in didFinishLoading.

  • platform/network/cocoa/NetworkLoadMetrics.mm:

(WebCore::copyTimingData):
The differences from the reference start are in Seconds, not milliseconds.

Source/WebInspectorUI:

  • UserInterface/Models/ResourceTimingData.js:

(WebInspector.ResourceTimingData.prototype.get requestStart):
(WebInspector.ResourceTimingData.prototype.get responseStart):
Fallback to the startTime if available instead of using the
inaccurate WebContentProcess gathered timestamps.

(WebInspector.ResourceTimingData.prototype.markResponseEndTime):
Verify responseEnd compared to other times we may have.

10:58 AM Changeset in webkit [213916] by beidson@apple.com
  • 4 edits in trunk

REGRESSION (r213877): WebKit2.CookieManager fails.
​https://bugs.webkit.org/show_bug.cgi?id=169581

Reviewed by Tim Horton.

Source/WebKit2:

  • UIProcess/WebCookieManagerProxy.cpp:

(WebKit::WebCookieManagerProxy::processPoolDestroyed): Invalidate the new sets of callbacks.
(WebKit::WebCookieManagerProxy::processDidClose): Ditto.

Tools:

  • TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj: Reenable the test.
10:39 AM Changeset in webkit [213915] by tpopela@redhat.com
  • 6 edits in trunk/Source/WebKit2

Fix uninitialized public members in NetworkProcess
​https://bugs.webkit.org/show_bug.cgi?id=169598

Reviewed by Alex Christensen.

  • NetworkProcess/NetworkProcess.cpp:

(WebKit::NetworkProcess::NetworkProcess):

  • NetworkProcess/NetworkProcessCreationParameters.h:
  • NetworkProcess/cache/NetworkCacheStorage.cpp:
  • UIProcess/WebProcessPool.cpp:

(WebKit::WebProcessPool::WebProcessPool):

  • UIProcess/soup/WebProcessPoolSoup.cpp:

(WebKit::WebProcessPool::platformInitializeNetworkProcess):

10:30 AM Changeset in webkit [213914] by commit-queue@webkit.org
  • 6 edits
    53 adds in trunk/LayoutTests

Import web-platform-tests/cors
​https://bugs.webkit.org/show_bug.cgi?id=169565

Patch by Youenn Fablet <​youenn@apple.com> on 2017-03-14
Reviewed by Sam Weinig.

LayoutTests/imported/w3c:

  • resources/import-expectations.json:
  • web-platform-tests/cors/304-expected.txt: Added.
  • web-platform-tests/cors/304.htm: Added.
  • web-platform-tests/cors/OWNERS: Added.
  • web-platform-tests/cors/README.md: Added.
  • web-platform-tests/cors/allow-headers-expected.txt: Added.
  • web-platform-tests/cors/allow-headers.htm: Added.
  • web-platform-tests/cors/basic-expected.txt: Added.
  • web-platform-tests/cors/basic.htm: Added.
  • web-platform-tests/cors/credentials-flag-expected.txt: Added.
  • web-platform-tests/cors/credentials-flag.htm: Added.
  • web-platform-tests/cors/late-upload-events-expected.txt: Added.
  • web-platform-tests/cors/late-upload-events.htm: Added.
  • web-platform-tests/cors/origin-expected.txt: Added.
  • web-platform-tests/cors/origin.htm: Added.
  • web-platform-tests/cors/preflight-cache-expected.txt: Added.
  • web-platform-tests/cors/preflight-cache.htm: Added.
  • web-platform-tests/cors/preflight-failure-expected.txt: Added.
  • web-platform-tests/cors/preflight-failure.htm: Added.
  • web-platform-tests/cors/redirect-origin-expected.txt: Added.
  • web-platform-tests/cors/redirect-origin.htm: Added.
  • web-platform-tests/cors/redirect-preflight-2-expected.txt: Added.
  • web-platform-tests/cors/redirect-preflight-2.htm: Added.
  • web-platform-tests/cors/redirect-preflight-expected.txt: Added.
  • web-platform-tests/cors/redirect-preflight.htm: Added.
  • web-platform-tests/cors/redirect-userinfo-expected.txt: Added.
  • web-platform-tests/cors/redirect-userinfo.htm: Added.
  • web-platform-tests/cors/remote-origin-expected.txt: Added.
  • web-platform-tests/cors/remote-origin.htm: Added.
  • web-platform-tests/cors/request-headers-expected.txt: Added.
  • web-platform-tests/cors/request-headers.htm: Added.
  • web-platform-tests/cors/resources/304.py: Added.

(error):
(main):

  • web-platform-tests/cors/resources/checkandremove.py: Added.

(main):

  • web-platform-tests/cors/resources/cors-cookie.py: Added.

(main):

  • web-platform-tests/cors/resources/cors-headers.asis: Added.
  • web-platform-tests/cors/resources/cors-makeheader.py: Added.

(main):

  • web-platform-tests/cors/resources/preflight.py: Added.

(main):

  • web-platform-tests/cors/resources/remote-xhrer.html: Added.
  • web-platform-tests/cors/resources/status.py: Added.

(main):

  • web-platform-tests/cors/resources/w3c-import.log: Added.
  • web-platform-tests/cors/response-headers-expected.txt: Added.
  • web-platform-tests/cors/response-headers.htm: Added.
  • web-platform-tests/cors/simple-requests-expected.txt: Added.
  • web-platform-tests/cors/simple-requests.htm: Added.
  • web-platform-tests/cors/status-async-expected.txt: Added.
  • web-platform-tests/cors/status-async.htm: Added.
  • web-platform-tests/cors/status-expected.txt: Added.
  • web-platform-tests/cors/status-preflight-expected.txt: Added.
  • web-platform-tests/cors/status-preflight.htm: Added.
  • web-platform-tests/cors/status.htm: Added.
  • web-platform-tests/cors/support.js: Added.

(dirname):

  • web-platform-tests/cors/w3c-import.log: Added.

LayoutTests:

  • tests-options.json:
10:21 AM Changeset in webkit [213913] by adachan@apple.com
  • 2 edits
    2 adds in trunk/Source/WebKit2

[Mac] Add API to get the NSURLProtectionSpace from WKProtectionSpaceRef
​https://bugs.webkit.org/show_bug.cgi?id=169494
<rdar://problem/11872163>

Reviewed by Dan Bernstein.

  • UIProcess/API/C/mac/WKProtectionSpaceNS.h: Added.
  • UIProcess/API/C/mac/WKProtectionSpaceNS.mm: Added.

(WKProtectionSpaceCopyNSURLProtectionSpace):

  • WebKit2.xcodeproj/project.pbxproj:
10:13 AM Changeset in webkit [213912] by matthew_hanson@apple.com
  • 4 edits in branches/safari-603-branch/Source/WebCore

Roll out r212554 via r212893. rdar://problem/30812551

10:13 AM Changeset in webkit [213911] by matthew_hanson@apple.com
  • 2 edits in branches/safari-603-branch/Source/JavaScriptCore

rdar://problem/30675867

10:13 AM Changeset in webkit [213910] by matthew_hanson@apple.com
  • 2 edits in branches/safari-603-branch/Source/WebCore

rdar://problem/30657889

10:13 AM Changeset in webkit [213909] by matthew_hanson@apple.com
  • 3 edits in branches/safari-603-branch/Source/WebCore

Merge r213211. rdar://problem/30742143

10:13 AM Changeset in webkit [213908] by matthew_hanson@apple.com
  • 2 edits in branches/safari-603-branch/Source/WebCore

Merge r212822. rdar://problem/30682429

10:13 AM Changeset in webkit [213907] by matthew_hanson@apple.com
  • 19 edits in branches/safari-603-branch

Roll out r212459 via r212537. rdar://problem/30699480

10:13 AM Changeset in webkit [213906] by matthew_hanson@apple.com
  • 1 edit in branches/safari-603-branch/Source/WebCore/ChangeLog

Merge r212828. rdar://problem/30636288

10:13 AM Changeset in webkit [213905] by matthew_hanson@apple.com
  • 2 edits in branches/safari-603-branch/Source

Merge r212692. rdar://problem/30635854

10:02 AM Changeset in webkit [213904] by mark.lam@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Gardening: Speculative build fix for CLoop after r213886.
​https://bugs.webkit.org/show_bug.cgi?id=169436

Not reviewed.

  • runtime/MachineContext.h:
10:01 AM Changeset in webkit [213903] by Wenson Hsieh
  • 6 edits in trunk/Source/WebKit2

[WK2] Adopt updated data operation interfaces for data interaction
​https://bugs.webkit.org/show_bug.cgi?id=169414
<rdar://problem/30948186>

Reviewed by Tim Horton.

Plumb additional information about the data interaction caret over to the UI process after handling a data
interaction action in the web process.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::didPerformDragControllerAction):
(WebKit::WebPageProxy::resetCurrentDragInformation):

  • UIProcess/WebPageProxy.h:

(WebKit::WebPageProxy::currentDragCaretRect):
(WebKit::WebPageProxy::resetCurrentDragInformation): Deleted.

  • UIProcess/WebPageProxy.messages.in:
  • UIProcess/ios/WKContentViewInteraction.h:
  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::performDragControllerAction):

10:01 AM Changeset in webkit [213902] by Wenson Hsieh
  • 21 edits in trunk

[WK2] Data interaction tests occasionally hit assertions in debug builds
​https://bugs.webkit.org/show_bug.cgi?id=169002
<rdar://problem/30994806>

Reviewed by Tim Horton.

Source/WebCore:

Data interaction unit tests occasionally fail due to the UI process expecting the latest received EditorState to
contain post layout data, but finding that it does not in -[WKContentView selectedTextRange]. The incomplete
EditorStates in question are sent while performing a data interaction operation, due to transient changes in the
frame selection. The UI process does not need to (and should not) be informed of these selection changes at all.

We can fix this by preventing the editor client from responding to selection changes during data interaction
operation. This patch also renames setIgnoreCompositionSelectionChange to setIgnoreSelectionChanges to better
reflect the fact that it is used outside of the context of holding selection change updates during IME. We
already use this affordance in various places, such as TextIndicator (while taking a snapshot on iOS), in
FindController on iOS, and when replacing selected or dictated text. Additionally, there is no logic in
setIgnoreCompositionSelectionChange that limits its use to composition.

  • editing/Editor.cpp:

(WebCore::Editor::cancelCompositionIfSelectionIsInvalid):
(WebCore::Editor::setComposition):
(WebCore::Editor::revealSelectionAfterEditingOperation):
(WebCore::Editor::setIgnoreSelectionChanges):
(WebCore::Editor::changeSelectionAfterCommand):
(WebCore::Editor::respondToChangedSelection):
(WebCore::Editor::setIgnoreCompositionSelectionChange): Deleted.

  • editing/Editor.h:

(WebCore::Editor::ignoreSelectionChanges):
(WebCore::Editor::ignoreCompositionSelectionChange): Deleted.

  • editing/mac/EditorMac.mm:

(WebCore::Editor::selectionWillChange):

  • page/TextIndicator.cpp:

(WebCore::TextIndicator::createWithRange):

Source/WebKit/mac:

Renames setIgnoreCompositionSelectionChange to setIgnoreSelectionChanges. See WebCore ChangeLog for more details.

  • WebView/WebHTMLView.mm:

(-[WebHTMLView _updateSelectionForInputManager]):

  • WebView/WebView.mm:

(-[WebView updateTextTouchBar]):

Source/WebKit2:

Renames setIgnoreCompositionSelectionChange to setIgnoreSelectionChanges. See WebCore ChangeLog for more details.

  • Shared/EditorState.cpp:

(WebKit::EditorState::encode):
(WebKit::EditorState::decode):

  • Shared/EditorState.h:
  • UIProcess/gtk/WebPageProxyGtk.cpp:
  • UIProcess/ios/WebPageProxyIOS.mm:

(WebKit::WebPageProxy::editorStateChanged):

  • UIProcess/mac/WebPageProxyMac.mm:

(WebKit::WebPageProxy::editorStateChanged):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::editorState):
(WebKit::WebPage::performDragControllerAction):
(WebKit::WebPage::setComposition):
(WebKit::WebPage::didChangeSelection):

  • WebProcess/WebPage/ios/FindControllerIOS.mm:

(WebKit::setSelectionChangeUpdatesEnabledInAllFrames):
(WebKit::FindController::willFindString):
(WebKit::FindController::didFailToFindString):
(WebKit::FindController::didHideFindIndicator):
(WebKit::setCompositionSelectionChangeEnabledInAllFrames): Deleted.

  • WebProcess/WebPage/ios/WebPageIOS.mm:

(WebKit::WebPage::updateSelectionAppearance):
(WebKit::WebPage::replaceSelectedText):
(WebKit::WebPage::replaceDictatedText):

Tools:

Reenables and refactors data interaction tests.

  • TestWebKitAPI/Tests/ios/DataInteractionTests.mm:
  • TestWebKitAPI/ios/DataInteractionSimulator.h:
  • TestWebKitAPI/ios/DataInteractionSimulator.mm:

(-[DataInteractionSimulator _resetSimulatedState]):
(-[DataInteractionSimulator runFrom:to:]):
(-[DataInteractionSimulator _advanceProgress]):

9:59 AM Changeset in webkit [213901] by Yusuke Suzuki
  • 3 edits in trunk/Source/JavaScriptCore

[JSC] Drop unnecessary pthread_attr_t for JIT enabled Linux / FreeBSD environment
​https://bugs.webkit.org/show_bug.cgi?id=169592

Reviewed by Carlos Garcia Campos.

Since suspended mcontext_t has all the necessary information, we can drop
pthread_attr_t allocation and destroy for JIT enabled Linux / FreeBSD environment.

  • heap/MachineStackMarker.cpp:

(JSC::MachineThreads::Thread::getRegisters):
(JSC::MachineThreads::Thread::Registers::stackPointer):
(JSC::MachineThreads::Thread::Registers::framePointer):
(JSC::MachineThreads::Thread::Registers::instructionPointer):
(JSC::MachineThreads::Thread::Registers::llintPC):
(JSC::MachineThreads::Thread::freeRegisters):

  • heap/MachineStackMarker.h:
9:16 AM Changeset in webkit [213900] by commit-queue@webkit.org
  • 5 edits in trunk

[Modern Media Controls] iOS may attempt to load fullscreen icon variants
​https://bugs.webkit.org/show_bug.cgi?id=169608
<rdar://problem/31037369>

Patch by Antoine Quint <Antoine Quint> on 2017-03-14
Reviewed by Eric Carlson.

Source/WebCore:

Only return fullscreen or compact variants for macOS.

  • Modules/modern-media-controls/controls/icon-service.js:

(const.iconService.new.IconService.prototype._fileNameAndPlatformForIconNameAndLayoutTraits):
(const.iconService.new.IconService):

LayoutTests:

Amend an existing test to check that we disregard fullscreen and compact variants on iOS.

  • media/modern-media-controls/icon-service/icon-service-expected.txt:
  • media/modern-media-controls/icon-service/icon-service.html:
9:13 AM Changeset in webkit [213899] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

[Modern Media Controls] Controls are laid out incorrectly with RTL languages
​https://bugs.webkit.org/show_bug.cgi?id=169605
<rdar://problem/30975709>

Patch by Antoine Quint <Antoine Quint> on 2017-03-14
Reviewed by Eric Carlson.

Source/WebCore:

Encorce "direction: ltr" on the controls since the controls layout should not be changed
by the host page's direction.

Test: media/modern-media-controls/media-controls/media-controls-controls-bar-always-ltr.html

  • Modules/modern-media-controls/controls/controls-bar.css:

(.controls-bar):

LayoutTests:

Add a test that enforces "direction: rtl" on a parent element to check that the controls
bar use "direction: ltr" anyway.

  • media/modern-media-controls/media-controls/media-controls-controls-bar-always-ltr-expected.txt: Added.
  • media/modern-media-controls/media-controls/media-controls-controls-bar-always-ltr.html: Added.
8:52 AM Changeset in webkit [213898] by beidson@apple.com
  • 2 edits in trunk/Tools

REGRESSION (r213877): WebKit2.CookieManager fails.
​https://bugs.webkit.org/show_bug.cgi?id=169581

Unreviewed gardening

  • TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj: Skip for now until I can fix.
8:11 AM Changeset in webkit [213897] by Wenson Hsieh
  • 7 edits
    3 adds in trunk

Make RepaintRegionAccumulator hold a WeakPtr to its root RenderView
​https://bugs.webkit.org/show_bug.cgi?id=168480
<rdar://problem/30566976>

Reviewed by Antti Koivisto.

Source/WebCore:

Implements two mitigations to prevent the symptoms of the bug from occurring (see the bugzilla for more details).

Test: editing/execCommand/show-modal-dialog-during-execCommand.html

  • editing/EditorCommand.cpp:

(WebCore::Editor::Command::execute):

Do not allow edit commands to execute if the frame's document before and after layout differ (that is, edit commands
triggered by a certain document should not run on a different document).

  • rendering/RenderView.cpp:

(WebCore::RenderView::RenderView):
(WebCore::RenderView::RepaintRegionAccumulator::RepaintRegionAccumulator):

Turns RepaintRegionAccumulator's reference to its root RenderView into a WeakPtr to gracefully handle the case
where its RenderView is destroyed before RepaintRegionAccumulator's destructor gets a chance to flush the
RenderView's repaint regions.

  • rendering/RenderView.h:

LayoutTests:

Introduces a new layout test. See WebCore ChangeLog for more details.

  • TestExpectations:
  • editing/execCommand/show-modal-dialog-during-execCommand-expected.txt: Added.
  • editing/execCommand/show-modal-dialog-during-execCommand.html: Added.
  • editing/execCommand/resources/self-closing-modal-dialog.html: Added.
  • platform/mac-wk1/TestExpectations:
6:20 AM Changeset in webkit [213896] by Yusuke Suzuki
  • 7 edits in trunk

Source/WTF:
Add secondsAs<T> methods to Seconds to convert it to integers with clamp
​https://bugs.webkit.org/show_bug.cgi?id=169537

Reviewed by Carlos Garcia Campos.

When using the usual static_cast, infinity becomes 0 accidentally.
It is not intended value when using Seconds for timeout value.
Instead, we use clampToAccepting64 to convert Seconds to
integer values to pass them to the system functions.

  • wtf/MathExtras.h:

(clampToAccepting64):

  • wtf/Seconds.h:

(WTF::Seconds::minutesAs):
(WTF::Seconds::secondsAs):
(WTF::Seconds::millisecondsAs):
(WTF::Seconds::microsecondsAs):
(WTF::Seconds::nanosecondsAs):

  • wtf/cocoa/WorkQueueCocoa.cpp:

(WTF::WorkQueue::dispatchAfter):

  • wtf/glib/RunLoopGLib.cpp:

(WTF::RunLoop::dispatchAfter):
(WTF::RunLoop::TimerBase::updateReadyTime):

Tools:
[WTF] Clean up RunLoop and WorkQueue with Seconds and Function
​https://bugs.webkit.org/show_bug.cgi?id=169537

Reviewed by Carlos Garcia Campos.

  • TestWebKitAPI/Tests/WTF/Time.cpp:

(TestWebKitAPI::TEST):

2:01 AM Changeset in webkit [213895] by Carlos Garcia Campos
  • 1 copy in releases/WebKitGTK/webkit-2.15.92

WebKitGTK+ 2.15.92

2:01 AM Changeset in webkit [213894] by Carlos Garcia Campos
  • 4 edits in releases/WebKitGTK/webkit-2.16

Unreviewed. Update OptionsGTK.cmake and NEWS for 2.15.92 release.

.:

  • Source/cmake/OptionsGTK.cmake: Bump version numbers.

Source/WebKit2:

  • gtk/NEWS: Add release notes for 2.15.92.
1:30 AM Changeset in webkit [213893] by zandobersek@gmail.com
  • 3 edits in trunk/Source/WebCore

[GLib] Use USE(GLIB) guards in WebCore/workers/
​https://bugs.webkit.org/show_bug.cgi?id=169595

Reviewed by Carlos Garcia Campos.

Utilize the USE(GLIB) build guards in the WorkerRunLoop and WorkerThread
class implementations to guard GLib-specific header inclusions and GLib
API invocations, instead of the more specific PLATFORM(GTK) guards.

  • workers/WorkerRunLoop.cpp:

(WebCore::WorkerRunLoop::runInMode):

  • workers/WorkerThread.cpp:

(WebCore::WorkerThread::workerThread):

1:29 AM Changeset in webkit [213892] by zandobersek@gmail.com
  • 2 edits in trunk/Source/JavaScriptCore

[GLib] Use USE(GLIB) guards in JavaScriptCore/inspector/EventLoop.cpp
​https://bugs.webkit.org/show_bug.cgi?id=169594

Reviewed by Carlos Garcia Campos.

Instead of PLATFORM(GTK) guards, utilize the USE(GLIB) build guards
to guard the GLib-specific includes and invocations in the JSC
inspector's EventLoop class implementation.

  • inspector/EventLoop.cpp:

(Inspector::EventLoop::cycle):

1:28 AM Changeset in webkit [213891] by zandobersek@gmail.com
  • 2 edits in trunk/Source/WebCore

[Soup] Suppress compiler warnings in NetworkStorageSession
​https://bugs.webkit.org/show_bug.cgi?id=169593

Reviewed by Carlos Garcia Campos.

Return default-constructed Vector objects in the NetworkStorageSession's
getAllCookies() and getCookies() methods, avoiding compiler warnings.

  • platform/network/soup/NetworkStorageSessionSoup.cpp:

(WebCore::NetworkStorageSession::getAllCookies):
(WebCore::NetworkStorageSession::getCookies):

1:28 AM Changeset in webkit [213890] by webkit@devinrousso.com
  • 2 edits in trunk/Source/WebInspectorUI

Web Inspector: Remove unnecessary assert for Number.percentageString
​https://bugs.webkit.org/show_bug.cgi?id=169589

Reviewed by Matt Baker.

  • UserInterface/Base/Utilities.js:

(Number.percentageString):

1:22 AM Changeset in webkit [213889] by Carlos Garcia Campos
  • 2 edits in releases/WebKitGTK/webkit-2.16/Source/WebKit2

Merge r213888 - Unreviewed. Fix syntax error in GTK+ API docs.

  • UIProcess/API/gtk/WebKitWebView.cpp:

(webkit_web_view_class_init):

1:21 AM Changeset in webkit [213888] by Carlos Garcia Campos
  • 2 edits in trunk/Source/WebKit2

Unreviewed. Fix syntax error in GTK+ API docs.

  • UIProcess/API/gtk/WebKitWebView.cpp:

(webkit_web_view_class_init):

12:57 AM Changeset in webkit [213887] by webkit@devinrousso.com
  • 2 edits in trunk/Source/WebInspectorUI

Web Inspector: allow the user to copy locked CSS selectors in Style - Rules
​https://bugs.webkit.org/show_bug.cgi?id=169587

Reviewed by Matt Baker.

  • UserInterface/Views/CSSStyleDeclarationSection.css:

(.style-declaration-section:matches(.locked, .selector-locked) > .header > .selector):

12:33 AM Changeset in webkit [213886] by Yusuke Suzuki
  • 12 edits
    1 add in trunk/Source

[JSC][Linux] Implement VMTrap in Linux ports
​https://bugs.webkit.org/show_bug.cgi?id=169436

Reviewed by Mark Lam.

Source/JavaScriptCore:

This patch port VMTrap to Linux ports.
We extract MachineContext accessors from various places (wasm/, heap/ and tools/)
and use them in all the JSC code.

  • JavaScriptCore.xcodeproj/project.pbxproj:
  • heap/MachineStackMarker.cpp:

(JSC::MachineThreads::Thread::Registers::stackPointer):
(JSC::MachineThreads::Thread::Registers::framePointer):
(JSC::MachineThreads::Thread::Registers::instructionPointer):
(JSC::MachineThreads::Thread::Registers::llintPC):

  • heap/MachineStackMarker.h:
  • runtime/MachineContext.h: Added.

(JSC::MachineContext::stackPointer):
(JSC::MachineContext::framePointer):
(JSC::MachineContext::instructionPointer):
(JSC::MachineContext::argumentPointer<1>):
(JSC::MachineContext::argumentPointer):
(JSC::MachineContext::llintInstructionPointer):

  • runtime/PlatformThread.h:

(JSC::platformThreadSignal):

  • runtime/VMTraps.cpp:

(JSC::SignalContext::SignalContext):
(JSC::SignalContext::adjustPCToPointToTrappingInstruction):

  • tools/CodeProfiling.cpp:

(JSC::profilingTimer):

  • tools/SigillCrashAnalyzer.cpp:

(JSC::SignalContext::SignalContext):
(JSC::SignalContext::dump):

  • tools/VMInspector.cpp:
  • wasm/WasmFaultSignalHandler.cpp:

(JSC::Wasm::trapHandler):

Source/WTF:

Enable VMTrap mechanism for Linux and FreeBSD.

  • wtf/Platform.h:
Note: See TracTimeline for information about the timeline view.