Timeline



Jun 5, 2017:

11:30 PM Changeset in webkit [217821] by Ryan Haddad
  • 3 edits in trunk/LayoutTests

Rebaseline fast/mediacapturefromelement/CanvasCaptureMediaStream-2d-events.html and update TestExpectations.

Unreviewed test gardening.

  • fast/mediacapturefromelement/CanvasCaptureMediaStream-2d-events-expected.txt:
10:44 PM Changeset in webkit [217820] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Skip storage/domstorage/sessionstorage/set-item-synchronous-keydown.html on iOS.
https://bugs.webkit.org/show_bug.cgi?id=172943

Unreviewed test gardening.

  • platform/ios/TestExpectations:
9:53 PM Changeset in webkit [217819] by sbarati@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Try to fix features.json by adding an ESNext section.

Unreviewed.

  • features.json:
9:47 PM Changeset in webkit [217818] by Wenson Hsieh
  • 6 edits in trunk

Refactor -[WebItemProviderPasteboard valuesForPasteboardType:inItemSet:] to check readable types
https://bugs.webkit.org/show_bug.cgi?id=172891
<rdar://problem/32204540>

Reviewed by Darin Adler.

Source/WebCore:

Tweaks some logic in PlatformPasteboardIOS and WebItemProviderPasteboard to correctly construct objects in
-valuesForPasteboardType:inItemSet:. Previously, we have a hard-coded map of UTI type to Class in the form of
static helpers titled is{RichText, String, URL, Image, Color}Type in WebItemProviderPasteboard. We would use
these functions to determine whether an NSAttributedString, NSString, NSURL, UIImage or UIColor should be
constructed using the loaded item provider data. This is incorrect for some UTIs, such as public.html, which
cannot actually be used to construct an NSAttributedString -- this caused -valuesForPasteboardType:inItemSet: to
always return nil when attempting to create an object corresponding to public.html.

To fix this, we refactor -valuesForPasteboardType:inItemSet: to instead iterate through UIItemProviderReading-
conformant classes in search for a class that can be created for the given UTI type. If no such class exists, we
then fall back to custom WebKit handling of the dropped UTI type, which so far only includes reading an NSString
with public.plain-text if the UTI is public.html (i.e. reading the HTML source from loaded item provider data).

Covered by 2 new API tests:
DataInteractionTests.ExternalSourceHTMLToContentEditable
DataInteractionTests.ExternalSourceAttributedStringToContentEditable

  • platform/ios/PlatformPasteboardIOS.mm:

(WebCore::PlatformPasteboard::stringForType):
(WebCore::PlatformPasteboard::readString):

In some cases, -[WebItemProviderPasteboard valuesForPasteboardType:inItemSet:] may now return an
NSAttributedString when an NSString was previously created. This adjusts for that possibility by returning just
the plain text, -[NSAttributedString string], if the read object was an NSAttributedString.

  • platform/ios/WebItemProviderPasteboard.mm:

(allLoadableClasses):
(classForTypeIdentifier):
(-[WebItemProviderPasteboard valuesForPasteboardType:inItemSet:]):

See description above for more details.

(isRichTextType): Deleted.
(isStringType): Deleted.
(isURLType): Deleted.
(isColorType): Deleted.
(isImageType): Deleted.

Removes these heuristics that attempt to "guess" the best UIItemProviderReading class to try and load for a
given UTI type.

  • rendering/RenderText.cpp:

(WebCore::RenderText::draggedContentRangesBetweenOffsets):

Adds a nil check to fix a debug assertion hit when running DataInteractionTests.

Tools:

Adds 2 new API tests to exercise data interaction of HTML data and an attributed string into a rich
contenteditable. See WebCore ChangeLog for more details.

  • TestWebKitAPI/Tests/ios/DataInteractionTests.mm:

(TestWebKitAPI::TEST):

9:00 PM Changeset in webkit [217817] by Michael Catanzaro
  • 2 edits in trunk/LayoutTests

Unreviewed GTK expectations gardening

Mark a few tests as crashing.

  • platform/gtk/TestExpectations:
8:51 PM Changeset in webkit [217816] by ddkilzer@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Follow-up: Update JSC's features.json
https://bugs.webkit.org/show_bug.cgi?id=172942

Rubber-stamped by Jon Davis.

  • features.json: Change "Supported in preview" to

"Supported" to try to fix <https://webkit.org/status/>.

8:22 PM Changeset in webkit [217815] by ggaren@apple.com
  • 2 edits in trunk/Source/bmalloc

Try to fix the GTK build.

Unreviewed.

  • bmalloc/List.h:
7:59 PM Changeset in webkit [217814] by sbarati@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

We don't properly parse init_expr when the opcode is an unexpected opcode
https://bugs.webkit.org/show_bug.cgi?id=172945

Reviewed by JF Bastien.

The bug is a simple typo. It should use the constant
true instead of false when invoking the WASM_PARSER_FAIL_IF
macro. This failure is already caught by spec tests that fail
on arm64 devices.

  • wasm/WasmModuleParser.cpp:
7:59 PM Changeset in webkit [217813] by Chris Dumez
  • 2 edits in trunk/LayoutTests

ASSERTION FAILED: RunLoop::isMain() in com.apple.WebKit: IPC::Connection::sendSyncMessage + 128
https://bugs.webkit.org/show_bug.cgi?id=172943
<rdar://problem/31288058>

Reviewed by Alexey Proskuryakov.

Make test clean a little bit more robust.

  • storage/domstorage/sessionstorage/set-item-synchronous-keydown.html:
7:34 PM Changeset in webkit [217812] by keith_miller@apple.com
  • 4 edits in trunk/Source/JavaScriptCore

OMG tier up checks should be a patchpoint
https://bugs.webkit.org/show_bug.cgi?id=172944

Reviewed by Saam Barati.

Tier up checks in BBQ should be done as a patchpoint rather than individual B3 opcodes.
In order to reduce code generated out of line in each function. We generate a single stub
that pushes all the callee-saves. This looks like a 5-10% compile time speedup.

  • wasm/WasmB3IRGenerator.cpp:

(JSC::Wasm::B3IRGenerator::B3IRGenerator):
(JSC::Wasm::B3IRGenerator::emitTierUpCheck):
(JSC::Wasm::B3IRGenerator::addLoop):

  • wasm/WasmThunks.cpp:

(JSC::Wasm::triggerOMGTierUpThunkGenerator):

  • wasm/WasmThunks.h:
7:21 PM Changeset in webkit [217811] by ggaren@apple.com
  • 12 edits in trunk/Source/bmalloc

bmalloc: Small and large objects should share memory
https://bugs.webkit.org/show_bug.cgi?id=172880
<rdar://problem/31494732>

Reviewed by Sam Weinig.

This reduces our high water mark memory usage on JetStream on macOS
by 10%-20%. It also has the nice side effect that we can free small
object metadata after returning from a high water mark.

No change in throughput.

Our old algorithm allocated small object chunks and large objects in
segregated virtual memory and never recycled addresses between them.
This provided a slight security benefit because we could apply guard
pages between the segregated ranges and we would never reuse the same
virtual address for object and metadata memory.

Our new algorithm allocates small object chunks from the large object
allocator. This naturally recycles memory between small chunks and large
objects, and between small chunks of different page classes. This allows
us to shift memory between allocation types as a program moves between
different phases of allocation, and to delete small object chunk metadata
when a program shrinks back from a high water mark.

Two intuitions I had about memory use turned out to be backwards in
this context:

(1) I thought that this optimization would work because it allowed you to
allocate and free a 4MB object and then reuse that large allocation to
service small allocations. In practice, the common benefit seems to be
the opposite: After you allocate and free many small objects, you can
stitch them together to allocate a large object without growing the heap.

(2) I thought that it would be more memory-efficient to allocate
fine-grained pages from the large object allocator. In practice, giving
the large object allocator too many arbitrarily-sized ranges to manage
leads to fragmentation. Meanwhile, segregated fit is a powerful memory
optimization. So, it's best to return small object memory to the large
allocator only when a whole small object chunk is free.

  • bmalloc/Chunk.h:

(bmalloc::Chunk::ref):
(bmalloc::Chunk::deref):
(bmalloc::Chunk::refCount):
(bmalloc::Chunk::freePages): We keep a free list per chunk and refcount
each chunk so we can notice when a chunk becomes empty, and return it
to the large allocator.

(bmalloc::forEachPage): A new helper function for iterating the pages
in a Chunk.

(bmalloc::Chunk::Chunk): Use forEachPage instead of manual iteration.
Use { } initialization because we don't get zero-initialized by the OS
anymore.

  • bmalloc/Heap.cpp:

(bmalloc::Heap::Heap):
(bmalloc::Heap::concurrentScavenge):
(bmalloc::Heap::scavenge): Don't bother unlocking while scavenging. I
wasn't able to show it to be a consistent speedup. A more promising
approach, if we find a motivating example, is for the scavenger to give
up and return early if any other client is waiting on the lock.

(bmalloc::Heap::allocateSmallChunk): New helper function for allocating
a small chunk. It allocates through the large allocator to facilitate
sharing. We still allocate a chunk at a time instead of a page at a time.
Surprisingly, more precise page-at-a-time allocation is worse for memory
use because of fragmentation. Segregated fit is a powerful optimization.

(bmalloc::Heap::deallocateSmallChunk): New helper function for deallocating
a small chunk.

(bmalloc::Heap::allocateSmallPage): Updated for new APIs.

(bmalloc::Heap::deallocateSmallLine): Updated for new APIs. Note that
we cache one free chunk per page class. This avoids churn in the large
allocator when you free(malloc(X)).

(bmalloc::Heap::allocateSmallBumpRangesByMetadata):
(bmalloc::Heap::allocateSmallBumpRangesByObject):
(bmalloc::Heap::tryAllocateLarge):
(bmalloc::Heap::scavengeSmallPages): Deleted.
(bmalloc::Heap::scavengeLargeObjects): Deleted.

  • bmalloc/Heap.h:
  • bmalloc/LargeMap.h:

(bmalloc::LargeMap::begin):
(bmalloc::LargeMap::end): Added iteration helpers for scavenging.

  • bmalloc/LargeRange.h:

(bmalloc::LargeRange::physicalSize): Added a comment about something
that I confused myself about in this patch.

  • bmalloc/List.h:

(bmalloc::List::iterator::operator*):
(bmalloc::List::iterator::operator->):
(bmalloc::List::iterator::operator!=):
(bmalloc::List::iterator::operator++):
(bmalloc::List::begin):
(bmalloc::List::end):
(bmalloc::List::pushFront):
(bmalloc::List::remove):
(bmalloc::ListNode::ListNode): Deleted. Added iteration helpers for
scavenging. Changed the default state of a Node to null pointers instead
of self pointers to distinguish the null node from the empty node for
easier debugging.

  • bmalloc/Sizes.h: Changed the chunk size to 1MB to increase the chances

of a chunk becoming free and recyclable.

  • bmalloc/SmallPage.h:

(bmalloc::SmallPage::hasPhysicalPages):
(bmalloc::SmallPage::setHasPhysicalPages): Track physical state by page
instead of implicitly by which list a page is in. It's simpler not
to have to move chunks and pages between physical vs virtual lists.

(bmalloc::SmallPage::SmallPage): Deleted.

  • bmalloc/VMHeap.cpp:

(bmalloc::VMHeap::tryAllocateLargeChunk):
(bmalloc::VMHeap::allocateSmallChunk): Deleted.

  • bmalloc/VMHeap.h:

(bmalloc::VMHeap::allocateSmallPage): Deleted.
(bmalloc::VMHeap::deallocateSmallPage): Deleted. Small chunk allocation
just forwards to the large allocator now.

  • bmalloc/bmalloc.h:

(bmalloc::api::scavenge):

7:03 PM Changeset in webkit [217810] by Chris Dumez
  • 3 edits
    2 adds in trunk

ASSERTION FAILED: RunLoop::isMain() in com.apple.WebKit: IPC::Connection::sendSyncMessage + 128
https://bugs.webkit.org/show_bug.cgi?id=172943
<rdar://problem/31288058>

Reviewed by Alexey Proskuryakov.

Source/WebKit2:

In Connection::sendMessage(), make sure we only ever transform asynchronous messages into synchronous
ones if sendMessage() is called on the main thread. This is necessary because we no longer support
sending synchronous messages from a background thread since r205125.

  • Platform/IPC/Connection.cpp:

(IPC::Connection::sendMessage):
(IPC::Connection::sendSyncMessage):

LayoutTests:

Add better test coverage.

  • storage/domstorage/sessionstorage/set-item-synchronous-keydown-expected.txt: Added.
  • storage/domstorage/sessionstorage/set-item-synchronous-keydown.html: Added.
5:03 PM Changeset in webkit [217809] by Joseph Pecoraro
  • 2 edits in trunk/Source/WebKit2

Revert r217405, this headerpad increase is no longer needed.

Rubber-stamped by Dan Bernstein.

  • Configurations/BaseXPCService.xcconfig:
5:03 PM Changeset in webkit [217808] by Joseph Pecoraro
  • 4 edits in trunk/Source/JavaScriptCore

Remove unused VM members
https://bugs.webkit.org/show_bug.cgi?id=172941

Reviewed by Mark Lam.

  • runtime/HashMapImpl.h:

(JSC::HashMapImpl::selectStructure): Deleted.

  • runtime/VM.cpp:

(JSC::VM::VM):

  • runtime/VM.h:
4:52 PM Changeset in webkit [217807] by commit-queue@webkit.org
  • 17 edits
    1 add in trunk

Web Inspector: Improve ES6 Class instances in Heap Snapshot instances view
https://bugs.webkit.org/show_bug.cgi?id=172848
<rdar://problem/25709212>

Patch by Joseph Pecoraro <Joseph Pecoraro> on 2017-06-05
Reviewed by Saam Barati.

JSTests:

  • typeProfiler/inheritance.js:

Rewrite the test slightly for clarity. The hoisting was confusing.

  • heapProfiler/class-names.js: Added.

(MyES5Class):
(MyES6Class):
(MyES6Subclass):
Test object types and improved class names.

  • heapProfiler/driver/driver.js:

(CheapHeapSnapshotNode):
(CheapHeapSnapshot):
(createCheapHeapSnapshot):
(HeapSnapshot):
(createHeapSnapshot):
Update snapshot parsing from version 1 to version 2.

Source/JavaScriptCore:

  • heap/HeapSnapshotBuilder.h:
  • heap/HeapSnapshotBuilder.cpp:

Update the snapshot version. Change the node's 0 | 1 internal value
to be a 32bit bit flag. This is nice in that it is both compatible
with the previous snapshot version and the same size. We can use more
flags in the future.

(JSC::HeapSnapshotBuilder::json):
In cases where the classInfo gives us "Object" check for a better
class name by checking (o).proto.constructor.name. We avoid this
check in cases where (o).hasOwnProperty("constructor") which is the
case for most Foo.prototype objects. Otherwise this would get the
name of the Foo superclass for the Foo.prototype object.

  • runtime/JSObject.cpp:

(JSC::JSObject::calculatedClassName):
Handle some possible edge cases that were not handled before. Such
as a JSObject without a GlobalObject, and an object which doesn't
have a default getPrototype. Try to make the code a little clearer.

Source/WebInspectorUI:

  • UserInterface/Workers/HeapSnapshot/HeapSnapshot.js:

(HeapSnapshot):
Support the new snapshot version. The only thing that changes are the
node flags, and its actually completely compatible with version 1.

(HeapSnapshot.updateCategoriesAndMetadata):
List the count of object type instances in each class category.

(HeapSnapshot.prototype.serializeNode):
Include whether or not the node is an object type.

  • UserInterface/Proxies/HeapSnapshotNodeProxy.js:

(WebInspector.HeapSnapshotNodeProxy):
(WebInspector.HeapSnapshotNodeProxy.deserialize):
Add a new Node isObjectType property based on the new data.

  • UserInterface/Views/HeapSnapshotClassDataGridNode.js:

(WebInspector.HeapSnapshotClassDataGridNode.prototype.createCellContent):

  • UserInterface/Views/HeapSnapshotClusterContentView.js:

(WebInspector.HeapSnapshotClusterContentView.iconStyleClassNameForClassName):
If a class contains 50% or more object type instances then it as such
instead of defaulting to native.

  • UserInterface/Views/HeapSnapshotDataGridTree.js:

(WebInspector.HeapSnapshotInstancesDataGridTree.prototype.populateTopLevel):

  • UserInterface/Views/HeapSnapshotInstanceDataGridNode.js:

(WebInspector.HeapSnapshotInstanceDataGridNode.prototype.createCellContent):
We can be more specific than the default if the individual instance is
known to be an object type.

LayoutTests:

  • inspector/unit-tests/heap-snapshot-expected.txt:
  • inspector/unit-tests/heap-snapshot.html:

Update for the new node flag.

4:51 PM Changeset in webkit [217806] by mitz@apple.com
  • 2 edits in trunk/Source/WebCore

Build fix for macOS 10.12

  • platform/spi/cocoa/AVKitSPI.h:
4:32 PM Changeset in webkit [217805] by sbarati@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Update JSC's features.json
https://bugs.webkit.org/show_bug.cgi?id=172942

Rubber stamped by Mark Lam.

  • features.json:
4:22 PM Changeset in webkit [217804] by jmarcell@apple.com
  • 5 edits in branches/safari-603-branch/Source

Versioning.

4:15 PM Changeset in webkit [217803] by mitz@apple.com
  • 6 edits in trunk/Source

Tried to fix the build when targrting macOS 10.12 using the macOS 10.13 developer beta SDK.

Source/WebCore:

  • platform/mac/WebPlaybackControlsManager.h:
  • platform/mac/WebPlaybackControlsManager.mm:

(-[WebPlaybackControlsManager togglePictureInPicture]):

Source/WebKit2:

  • UIProcess/Cocoa/WebViewImpl.h:
  • UIProcess/Cocoa/WebViewImpl.mm:

(WebKit::WebViewImpl::mediaPlaybackControlsView):

4:04 PM Changeset in webkit [217802] by mitz@apple.com
  • 5 edits in trunk/Source

Tried to fix the build when targrting macOS 10.12 using the macOS 10.13 developer beta SDK.

Source/WebCore:

  • platform/spi/cocoa/AVKitSPI.h:

Source/WebKit/mac:

  • WebView/WebView.mm:

(-[WebView updateMediaTouchBar]):

  • WebView/WebViewData.h:
4:01 PM Changeset in webkit [217801] by dbates@webkit.org
  • 2 edits
    1 add in trunk/Tools

webkitpy: Abstract Executive class
https://bugs.webkit.org/show_bug.cgi?id=172928
<rdar://problem/32569531>

Reviewed by Ryosuke Niwa.

Adds an AbstractExecutive class and makes Executive extend AbstractExecutive.
This will make it straightforward to create additional derived classes that
adhere to the same interface, say for iOS on-device testing.

  • Scripts/webkitpy/common/system/abstractexecutive.py: Added.

(AbstractExecutive):
(AbstractExecutive.run_and_throw_if_fail):
(AbstractExecutive.cpu_count):
(AbstractExecutive.interpreter_for_script):
(AbstractExecutive.shell_command_for_script):
(AbstractExecutive.kill_process):
(AbstractExecutive.check_running_pid):
(AbstractExecutive.running_pids):
(AbstractExecutive.wait_newest):
(AbstractExecutive.wait_limited):
(AbstractExecutive.interrupt):
(AbstractExecutive.default_error_handler):
(AbstractExecutive.ignore_error):
(AbstractExecutive._stringify_args):
(AbstractExecutive.command_for_printing):
(AbstractExecutive.run_command):
(AbstractExecutive.popen):
(AbstractExecutive.run_in_parallel):

  • Scripts/webkitpy/common/system/executive.py:

(Executive):
(Executive.cpu_count):
(Executive.running_pids):
(Executive.kill_all):
(Executive._compute_stdin):
(Executive.interpreter_for_script): Deleted.
(Executive.shell_command_for_script): Deleted.
(Executive.wait_newest): Deleted.
(Executive.wait_limited): Deleted.
(Executive.default_error_handler): Deleted.
(Executive.ignore_error): Deleted.
(Executive.command_for_printing): Deleted.

3:41 PM Changeset in webkit [217800] by Konstantin Tokarev
  • 2 edits in trunk/Source/WebCore

Unreviewed, fix missing semicolon in r217795

  • platform/graphics/win/MediaPlayerPrivateMediaFoundation.cpp:

(WebCore::calculateNaturalSize):

3:40 PM Changeset in webkit [217799] by jer.noble@apple.com
  • 28 edits
    4 adds in trunk/Source

Allow clients to specify a list of codecs which should require hardware decode support.
https://bugs.webkit.org/show_bug.cgi?id=172787

Reviewed by Alex Christensen.

Source/WebCore:

Add a new setting, checked by HTMLMediaElement, which allows MediaPlayerPrivate implementation
to require that the specified codecs have hardware support. This requirement will be supported
in the normal media loading path and the MSE path on Cocoa ports.

  • WebCore.xcodeproj/project.pbxproj:
  • html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::mediaCodecsRequiringHardwareSupport):

  • html/HTMLMediaElement.h:
  • page/Settings.cpp:

(WebCore::Settings::setMediaCodecsRequiringHardwareSupport):

  • page/Settings.h:

(WebCore::Settings::mediaCodecsRequiringHardwareSupport):

  • platform/cocoa/VideoToolboxSoftLink.cpp:
  • platform/cocoa/VideoToolboxSoftLink.h:
  • platform/graphics/MediaPlayer.h:

(WebCore::MediaPlayerClient::mediaCodecsRequiringHardwareSupport):

  • platform/graphics/avfoundation/objc/AVAssetTrackUtilities.h: Added.
  • platform/graphics/avfoundation/objc/AVAssetTrackUtilities.mm: Added.

(WebCore::assetTrackMeetsHardwareDecodeRequirements):

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

(WebCore::MediaPlayerPrivateAVFoundationObjC::checkPlayability):
(WebCore::MediaPlayerPrivateAVFoundationObjC::assetStatus):

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

(WebCore::SourceBufferPrivateAVFObjC::didParseStreamDataAsAsset):

Source/WebKit2:

Add a WKWebPageConfiguration (and associated WebPreferences and WKPreferencesRef) properties
allowing clients to require specified codecs have hardware decode support. Also open up the
sandbox on Cocoa ports to services required to check for hardware decode availability.

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

(WKPreferencesSetMediaCodecsRequiringHardwareSupport):
(WKPreferencesCopyMediaCodecsRequiringHardwareSupport):

  • UIProcess/API/C/WKPreferencesRefPrivate.h:
  • UIProcess/API/Cocoa/WKWebView.mm:

(-[WKWebView _initializeWithConfiguration:]):

  • UIProcess/API/Cocoa/WKWebViewConfiguration.mm:

(-[WKWebViewConfiguration copyWithZone:]):
(-[WKWebViewConfiguration _mediaCodecsRequiringHardwareSupport]):
(-[WKWebViewConfiguration _setMediaCodecsRequiringHardwareSupport:]):

  • UIProcess/API/Cocoa/WKWebViewConfigurationPrivate.h:
  • WebProcess/com.apple.WebProcess.sb.in:

Source/WTF:

Add a couple of convenience methods:

  • a String::split() that returns a vector (rather than taking an out-reference to a vector).
  • A Vector::map() template which takes a Callable and returns a Vector of a different type.
  • wtf/Vector.h:

(WTF::Vector::map):

  • wtf/text/WTFString.h:

(WTF::String::split):

3:28 PM Changeset in webkit [217798] by mitz@apple.com
  • 2 edits in trunk/Source/WebCore

Tried to fix the build when targrting macOS 10.12 using the macOS 10.13 developer beta SDK.

  • platform/mac/WebPlaybackControlsManager.mm:

(-[WebPlaybackControlsManager togglePictureInPicture]):

3:12 PM Changeset in webkit [217797] by mitz@apple.com
  • 2 edits in trunk/Source/WebCore

Tried to fix the build when targrting macOS 10.12 using the macOS 10.13 developer beta SDK.

  • platform/spi/mac/AVFoundationSPI.h:
2:54 PM Changeset in webkit [217796] by mitz@apple.com
  • 2 edits in trunk/Source/WebCore

Tried to fix the build when targrting macOS 10.12 using the macOS 10.13 developer beta SDK.

  • platform/spi/cocoa/IOSurfaceSPI.h:
2:45 PM Changeset in webkit [217795] by Konstantin Tokarev
  • 3 edits in trunk/Source/WebCore

Fix MediaPlayerPrivateMediaFoundation::naturalSize
https://bugs.webkit.org/show_bug.cgi?id=172908

Reviewed by Alex Christensen.

MediaPlayerPrivateMediaFoundation::naturalSize should return size of
video frame corrected by PAR, not size of video element.

No new tests needed.

  • platform/graphics/win/MediaPlayerPrivateMediaFoundation.cpp:

(WebCore::MediaPlayerPrivateMediaFoundation::load):
(WebCore::MediaPlayerPrivateMediaFoundation::naturalSize):
(WebCore::MediaPlayerPrivateMediaFoundation::setNaturalSize):
(WebCore::calculateNaturalSize):
(WebCore::MediaPlayerPrivateMediaFoundation::CustomVideoPresenter::setMediaType):

  • platform/graphics/win/MediaPlayerPrivateMediaFoundation.h:
2:04 PM Changeset in webkit [217794] by Alan Bujtas
  • 4 edits in trunk/Source/WebCore

Destroy the associated renderer subtree when display: contents node is deleted.
https://bugs.webkit.org/show_bug.cgi?id=172920
<rdar://problem/32446045>

Reviewed by Antti Koivisto.

Since display: contents node does not create a renderer, we need to explicitly check
and distinguish it from the display: none case.

Covered by existing tests.

  • dom/ContainerNode.cpp:

(WebCore::destroyRenderTreeIfNeeded):

  • dom/Node.cpp:

(WebCore::Node::~Node): Promote ASSERT(!renderer()) to ASSERT_WITH_SECURITY_IMPLICATION.

  • dom/Text.cpp:

(WebCore::Text::~Text): Redundant assert. Text is a Node.

2:00 PM Changeset in webkit [217793] by Jonathan Bedard
  • 3 edits in trunk/Tools

webkitpy: Add iOS to platform
https://bugs.webkit.org/show_bug.cgi?id=172931

Reviewed by Daniel Bates.

  • Scripts/webkitpy/common/system/platforminfo.py:

PlatformInfo.init): Use platform_module.release() as the os_version for iOS device.
(PlatformInfo.is_ios): Added.
(PlatformInfo._determine_os_name): A platform name of 'ios' means that the os name is 'ios'.

  • Scripts/webkitpy/common/system/platforminfo_mock.py:

(MockPlatformInfo.is_ios): Added.

1:54 PM Changeset in webkit [217792] by clopez@igalia.com
  • 2 edits in trunk/Tools

[WPE][JHBuild] Switch github repository from git:// to https://
https://bugs.webkit.org/show_bug.cgi?id=172921

Reviewed by Alex Christensen.

  • wpe/jhbuild.modules:
1:29 PM Changeset in webkit [217791] by Beth Dakin
  • 16 edits in trunk/Source

Modify Netflix controlsManager quirk to prevent only scrubbing
https://bugs.webkit.org/show_bug.cgi?id=172881
-and corresponding-
rdar://problem/32228660

Reviewed by Andy Estes.

Source/WebCore:

Instead of preventing Netflix from getting a controlsManager at all, this patch
ONLY prevents touch bar scrubbing from working. This means that Netflix will now
get controls in the TouchBar, it means that the play/pause and PiP buttons will be
functional, and it means that the timeline will accurately represent the current
time, BUT users will not be able to use the timeline to scrub the video. This also
allows Netflix to work with other MediaRemote clients such as AirPods.

Remove the quirk code from here, since this is no longer the right place to test
if the MediaElement is hosted at Netflix.

  • html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::bestMediaElementForShowingPlaybackControlsManager):
(WebCore::needsPlaybackControlsManagerQuirk): Deleted.

Pass around a new bool allowsTouchBarScrubbing.

  • platform/cocoa/WebPlaybackSessionModel.h:

(WebCore::WebPlaybackSessionModelClient::allowsTouchBarScrubbingChanged):

  • platform/cocoa/WebPlaybackSessionModelMediaElement.h:

Here’s where we check if we need quirky behavior.

  • platform/cocoa/WebPlaybackSessionModelMediaElement.mm:

(WebCore::WebPlaybackSessionModelMediaElement::updateForEventName):
(WebCore::WebPlaybackSessionModelMediaElement::allowsTouchBarScrubbing):

Return the bool we have plumbed through tracking whether the current site
allowsTouchBarScrubbing from the existing delegate method
canBeginTouchBarScrubbing. We will always return NO for video elements on Netflix,
and this will prevent users from be able to scrub with the TouchBar on Netflix.

  • platform/mac/WebPlaybackControlsManager.h:
  • platform/mac/WebPlaybackControlsManager.mm:

(-[WebPlaybackControlsManager canBeginTouchBarScrubbing]):

More plumbing.

  • platform/mac/WebPlaybackSessionInterfaceMac.h:
  • platform/mac/WebPlaybackSessionInterfaceMac.mm:

(WebCore::WebPlaybackSessionInterfaceMac::allowsTouchBarScrubbingChanged):
(WebCore::WebPlaybackSessionInterfaceMac::setPlayBackControlsManager):

Source/WebKit2:

Plumbing for allowsTouchBarScrubbing. See WebCore ChangeLog for more details.

  • UIProcess/Cocoa/WebPlaybackSessionManagerProxy.h:
  • UIProcess/Cocoa/WebPlaybackSessionManagerProxy.messages.in:
  • UIProcess/Cocoa/WebPlaybackSessionManagerProxy.mm:

(WebKit::WebPlaybackSessionModelContext::setAllowsTouchBarScrubbing):
(WebKit::WebPlaybackSessionManagerProxy::setAllowsTouchBarScrubbing):

  • WebProcess/cocoa/WebPlaybackSessionManager.h:
  • WebProcess/cocoa/WebPlaybackSessionManager.mm:

(WebKit::WebPlaybackSessionInterfaceContext::allowsTouchBarScrubbingChanged):
(WebKit::WebPlaybackSessionManager::allowsTouchBarScrubbingChanged):

11:19 AM Changeset in webkit [217790] by jmarcell@apple.com
  • 1 copy in tags/Safari-603.3.4

Tag Safari-603.3.4.

10:49 AM Changeset in webkit [217789] by sbarati@apple.com
  • 2 edits in trunk/Websites/browserbench.org

Update index.html to include ARES-6.

Rubber stamped by Ryosuke Niwa.

  • index.html:
9:50 AM Changeset in webkit [217788] by Ryan Haddad
  • 6 edits
    8 adds in branches/safari-603-branch/LayoutTests

Merge r216126.

9:35 AM Changeset in webkit [217787] by Ryan Haddad
  • 41 edits
    2 adds in trunk/LayoutTests

Rebaseline tests for Windows after r217772.
https://bugs.webkit.org/show_bug.cgi?id=172866

Unreviewed test gardening.

  • platform/win/svg/W3C-SVG-1.1-SE/paths-dom-02-f-expected.txt:
  • platform/win/svg/W3C-SVG-1.1/animate-elem-06-t-expected.txt:
  • platform/win/svg/W3C-SVG-1.1/animate-elem-07-t-expected.txt:
  • platform/win/svg/W3C-SVG-1.1/animate-elem-08-t-expected.txt:
  • platform/win/svg/W3C-SVG-1.1/animate-elem-83-t-expected.txt:
  • platform/win/svg/W3C-SVG-1.1/filters-example-01-b-expected.txt:
  • platform/win/svg/W3C-SVG-1.1/paths-data-01-t-expected.txt:
  • platform/win/svg/W3C-SVG-1.1/paths-data-02-t-expected.txt:
  • platform/win/svg/W3C-SVG-1.1/paths-data-12-t-expected.txt:
  • platform/win/svg/W3C-SVG-1.1/text-path-01-b-expected.txt:
  • platform/win/svg/batik/filters/feTile-expected.txt:
  • platform/win/svg/batik/filters/filterRegions-expected.txt:
  • platform/win/svg/batik/masking/maskRegions-expected.txt:
  • platform/win/svg/batik/paints/gradientLimit-expected.txt:
  • platform/win/svg/batik/paints/patternPreserveAspectRatioA-expected.txt:
  • platform/win/svg/batik/paints/patternRegionA-expected.txt:
  • platform/win/svg/batik/paints/patternRegions-expected.txt:
  • platform/win/svg/batik/paints/patternRegions-positioned-objects-expected.txt: Added.
  • platform/win/svg/batik/text/longTextOnPath-expected.txt: Added.
  • platform/win/svg/batik/text/smallFonts-expected.txt:
  • platform/win/svg/batik/text/textAnchor-expected.txt:
  • platform/win/svg/batik/text/textDecoration-expected.txt:
  • platform/win/svg/batik/text/textEffect-expected.txt:
  • platform/win/svg/batik/text/textEffect2-expected.txt:
  • platform/win/svg/batik/text/textEffect3-expected.txt:
  • platform/win/svg/batik/text/textFeatures-expected.txt:
  • platform/win/svg/batik/text/textGlyphOrientationHorizontal-expected.txt:
  • platform/win/svg/batik/text/textLayout-expected.txt:
  • platform/win/svg/batik/text/textLayout2-expected.txt:
  • platform/win/svg/batik/text/textLength-expected.txt:
  • platform/win/svg/batik/text/textOnPathSpaces-expected.txt:
  • platform/win/svg/batik/text/textPosition-expected.txt:
  • platform/win/svg/batik/text/textProperties-expected.txt:
  • platform/win/svg/batik/text/textProperties2-expected.txt:
  • platform/win/svg/batik/text/textStyles-expected.txt:
  • platform/win/svg/custom/broken-internal-references-expected.txt:
  • platform/win/svg/custom/path-textPath-simulation-expected.txt:
  • platform/win/svg/dom/SVGPathSegList-segment-modification-expected.txt:
  • platform/win/svg/hixie/perf/001-expected.txt:
  • platform/win/svg/hixie/perf/002-expected.txt:
  • platform/win/svg/text/text-path-01-b-expected.txt:
  • platform/win/svg/zoom/page/zoom-svg-through-object-with-override-size-expected.txt:
9:27 AM Changeset in webkit [217786] by Carlos Garcia Campos
  • 4 edits in trunk/Source/WebCore

[GStreamer] Deadlock in MediaPlayerPrivateGStreamer::changePipelineState, web process often locks up on seeking in a youtube video that has already fully buffered
https://bugs.webkit.org/show_bug.cgi?id=170003

Reviewed by Michael Catanzaro.

When video sink is requested to render a frame, the GstBaseSink preroll mutex is taken. Then WebKit media player
schedules a repaint in the main thread, taking the draw mutex and waiting on draw condition. It can happen that
before the repaint is done in the main thread, a pause is requested in the main thread, causing a change state
from PLAYING to PAUSE. When the change state reaches the video sink gst_base_sink_change_state() tries to get
the preroll mutex. This causes a deadlock because the main thread is waiting to get the preroll mutex, but the
other thread is waiting for the main thread to do the repaint. GStreamer handles this case by calling unlock()
on the video sink before trying to get the preroll mutex, but the media player doesn't cancel the pending
repaint when using coordinated graphics. This patch adds a new signal to WebKitVideoSink "repaint-cancelled" to
notify the media player to cancel the pending prepaint.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:

(WebCore::MediaPlayerPrivateGStreamerBase::cancelRepaint): Release the draw mutex and notify the condition.
(WebCore::MediaPlayerPrivateGStreamerBase::repaintCancelledCallback): Call cancelRepaint().
(WebCore::MediaPlayerPrivateGStreamerBase::createVideoSink): Connect to WebKitVideoSink::repaint-cancelled.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.h:
  • platform/graphics/gstreamer/VideoSinkGStreamer.cpp:

(webkitVideoSinkRepaintCancelled): Emit WebKitVideoSink::repaint-cancelled.
(webkitVideoSinkUnlock): Call webkitVideoSinkRepaintCancelled().
(webkitVideoSinkStop): Ditto.
(webkit_video_sink_class_init): Add WebKitVideoSink::repaint-cancelled signal.

9:25 AM Changeset in webkit [217785] by Carlos Garcia Campos
  • 4 edits in trunk/Source/WebCore

[GStreamer] Cleanup ifdefs in MediaPlayerPrivateGStreamerBase
https://bugs.webkit.org/show_bug.cgi?id=172918

Reviewed by Michael Catanzaro.

The code in MediaPlayerPrivateGStreamerBase is quite difficult to follow with all the ifdefs for the
possibilities when rendering. Some of them are even dead code now that all ports using GStreamer are using also
coordinated graphics.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:

(WebCore::MediaPlayerPrivateGStreamerBase::MediaPlayerPrivateGStreamerBase):
(WebCore::MediaPlayerPrivateGStreamerBase::~MediaPlayerPrivateGStreamerBase):
(WebCore::MediaPlayerPrivateGStreamerBase::pushTextureToCompositor):
(WebCore::MediaPlayerPrivateGStreamerBase::repaint):
(WebCore::MediaPlayerPrivateGStreamerBase::triggerRepaint):

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.h:
  • platform/graphics/gstreamer/VideoSinkGStreamer.cpp:

(VideoRenderRequestScheduler::VideoRenderRequestScheduler):
(VideoRenderRequestScheduler::stop):
(VideoRenderRequestScheduler::requestRender):
(webkitVideoSinkRequestRender):

9:05 AM Changeset in webkit [217784] by Carlos Garcia Campos
  • 2 edits in trunk/Source/WebCore

Unreviewed. Fix compile warning when building GTK+ port without OpenGL.

  • platform/graphics/texmap/BitmapTexturePool.cpp:

(WebCore::BitmapTexturePool::createTexture):

9:03 AM Changeset in webkit [217783] by Carlos Garcia Campos
  • 2 edits in trunk/Source/WebKit2

Unreviewed. Fix GTK+ build with OpenGL disabled after r217779.

  • WebProcess/WebPage/AcceleratedDrawingArea.cpp:

(WebKit::AcceleratedDrawingArea::mainFrameContentSizeChanged):

8:59 AM Changeset in webkit [217782] by Ryan Haddad
  • 3 edits in trunk/LayoutTests

Remove debug-only flag for flaky test http/tests/navigation/statistics.html.
https://bugs.webkit.org/show_bug.cgi?id=172454

Unreviewed test gardening.

  • platform/ios-wk2/TestExpectations:
  • platform/mac-wk2/TestExpectations:
8:42 AM Changeset in webkit [217781] by timothy_horton@apple.com
  • 2 edits in trunk/Tools

ASSERTION FAILED: m_currentScriptCallbackID in UIScriptContext::requestUIScriptCompletion(JSStringRef) running fast/events/ios/autocorrect-with-range-selection.html
https://bugs.webkit.org/show_bug.cgi?id=172887
<rdar://problem/32546061>

Reviewed by Sam Weinig.

  • WebKitTestRunner/ios/UIScriptControllerIOS.mm:

(WTR::UIScriptController::applyAutocorrection):
applyAutocorrection can call its completion handler synchronously,
which makes UIScriptController unhappy (see bug 172884).

5:33 AM Changeset in webkit [217780] by magomez@igalia.com
  • 2 edits in trunk/Source/WebCore

[GTK][WPE] Do not force video sample's buffer release when the platformLayerProxy is inactive
https://bugs.webkit.org/show_bug.cgi?id=172916

Reviewed by Carlos Garcia Campos.

When MediaPlayerPrivateGStreamerBase's plarformLayerProxy was inactive, a copy of the video sample
was being done (without the buffer) and set, causing the sample's buffer to get freed. This was done
to avoid stalling the gstreamer pipeline in situations where the video was set to display:none, as the
platformLayerProxy would keep all the available buffers. But this can't happen nowadays as setting
the video to display:none invalidates the proxy, causing it to release the references to the
gstreamer buffers it's using. Also, the current code is causing a crash when using gstreamer-gl and
the video is hidden but its contents are being painted through webgl or an accelerated canvas. So,
remove this sample copy as it's not necessary anymore.

Covered by existent tests.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:

(WebCore::MediaPlayerPrivateGStreamerBase::pushTextureToCompositor):

5:01 AM Changeset in webkit [217779] by Carlos Garcia Campos
  • 9 edits
    4 deletes in trunk/Source/WebKit2

[WPE] Use AcceleratedDrawingArea instead of its fork
https://bugs.webkit.org/show_bug.cgi?id=172496

Reviewed by Žan Doberšek.

WPE uses its own drawing area implementation, which is actually a fork of AcceleratedDrawingArea, but simplified
for the case of compositing being always forced. AcceleratedDrawingArea already handles the case of compositing
being forced, so now that WPE is upstream we could simply use AcceleratedDrawingArea instead.

  • PlatformWPE.cmake:
  • Shared/DrawingAreaInfo.h: Remove DrawingAreaTypeWPE type.
  • UIProcess/API/wpe/DrawingAreaProxyWPE.cpp: Removed.
  • UIProcess/API/wpe/DrawingAreaProxyWPE.h: Removed.
  • UIProcess/API/wpe/PageClientImpl.cpp:

(WebKit::PageClientImpl::createDrawingAreaProxy): Create an AcceleratedDrawingAreaProxy.

  • WebProcess/WebPage/AcceleratedDrawingArea.cpp:

(WebKit::AcceleratedDrawingArea::mainFrameContentSizeChanged): Moved from DrawingAreaImpl since it actually
belongs here.

  • WebProcess/WebPage/DrawingArea.cpp:

(WebKit::DrawingArea::create): Create an AcceleratedDrawingArea for WPE port.

  • WebProcess/WebPage/DrawingAreaImpl.cpp: Remove mainFrameContentSizeChanged() that doesn't belong here.
  • WebProcess/WebPage/DrawingAreaImpl.h:
  • WebProcess/WebPage/wpe/DrawingAreaWPE.cpp: Removed.
  • WebProcess/WebPage/wpe/DrawingAreaWPE.h: Removed.
2:11 AM Changeset in webkit [217778] by commit-queue@webkit.org
  • 3 edits in trunk/LayoutTests

Rebaseline media/modern-media-controls/audio/audio-controls-metrics.html
https://bugs.webkit.org/show_bug.cgi?id=172915

Patch by Antoine Quint <Antoine Quint> on 2017-06-05
Reviewed by Antoine Quint.

The default height of <audio> elements is now 31pt.

  • media/modern-media-controls/audio/audio-controls-metrics-expected.txt:
  • media/modern-media-controls/audio/audio-controls-metrics.html:
12:45 AM Changeset in webkit [217777] by magomez@igalia.com
  • 2 edits in trunk/Source/WebCore

[GTK][WPE] Fix the condition to decide whether the MediaPlayer can be rendered accelerated
https://bugs.webkit.org/show_bug.cgi?id=172852

Reviewed by Michael Catanzaro.

Before this change, in order to determine whether the MediaPlayer rendering could be accelerated we
needed the player to have a renderer or it would return false. The renderer was needed in order to
access the RenderLayerCompositor and check whether the chrome would be overwriting the accelerated
compositing setting. But we don't have that option in WebKitGTK+ or WPE, so we can just check the
accelerated compositing setting directly.

Another problems with the approach before this change is that accelerated rendering was being disabled
when the video element was not visible, as there's no video renderer. Due to this, cases where the video
is painted through an accelerated canvas or webgl would not be using hardware accelerated copies.

Covered by existent tests.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:

(WebCore::MediaPlayerPrivateGStreamerBase::acceleratedRenderingStateChanged):

Jun 4, 2017:

11:09 PM Changeset in webkit [217776] by Simon Fraser
  • 5 edits
    2 adds in trunk

Percentages are calculated wrong in SVG transform CSS property
https://bugs.webkit.org/show_bug.cgi?id=172901

Reviewed by Zalan Bujtas.
Source/WebCore:

The code added in r217236 was just fetching viewBox(), but that can be empty.
SVGLengthContext::determineViewport() does the correct thing to get
the appropriate viewport.

Test: svg/transforms/percent-transform-values-viewbox.html

  • svg/SVGGraphicsElement.cpp:

(WebCore::SVGGraphicsElement::animatedLocalTransform):

LayoutTests:

  • svg/transforms/percent-transform-values-viewbox-expected.html: Added.
  • svg/transforms/percent-transform-values-viewbox.html: Added.
  • svg/transforms/transform-origin-css-property-expected.xhtml:
  • svg/transforms/transform-origin-css-property.xhtml: Make this test a bit less

annoying to maintain by applying the transforms relative to the fill-box, which means all
the transform origins are no longer offset by the box position. Other tests exercise
view-box relative transform origins.

1:56 PM Changeset in webkit [217775] by eric.carlson@apple.com
  • 8 edits
    2 adds in trunk

[MediaStream] Page capture state not reported correctly
https://bugs.webkit.org/show_bug.cgi?id=172897
<rdar://problem/32493318>

Reviewed by Youenn Fablet.

Source/WebCore:

Test: fast/mediastream/media-stream-track-muted.html

  • Modules/mediastream/MediaStream.cpp:

(WebCore::MediaStream::mediaState): Test for audio and video mute on the tracks.
(WebCore::MediaStream::characteristicsChanged): Call statusDidChange when m_mediaState
changes instead of m_muted, so the page is informed when just audio or video mute state
changes.

  • Modules/mediastream/MediaStream.h:
  • testing/Internals.cpp:

(WebCore::Internals::setMediaStreamTrackMuted): New.

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

LayoutTests:

  • fast/mediastream/media-stream-track-muted-expected.txt: Added.
  • fast/mediastream/media-stream-track-muted.html: Added.
11:19 AM Changeset in webkit [217774] by Darin Adler
  • 26 edits in trunk/Source

Streamline handling of attributes, using references as much as possible
https://bugs.webkit.org/show_bug.cgi?id=172899

Reviewed by Chris Dumez.

Source/WebCore:

  • accessibility/AccessibilityNodeObject.cpp:

(WebCore::AccessibilityNodeObject::valueForRange): Use reference to value of
attribute instead of copy to avoid reference count churn.
(WebCore::AccessibilityNodeObject::maxValueForRange): Ditto.
(WebCore::AccessibilityNodeObject::minValueForRange): Ditto.

  • accessibility/AccessibilitySVGElement.cpp:

(WebCore::AccessibilitySVGElement::childElementWithMatchingLanguage): Ditto.
(WebCore::AccessibilitySVGElement::accessibilityDescription): Ditto.

  • css/PropertySetCSSStyleDeclaration.cpp:

(WebCore::StyleAttributeMutationScope::StyleAttributeMutationScope): Ditto.

  • editing/cocoa/DataDetection.mm:

(WebCore::DataDetection::shouldCancelDefaultAction): Use the
equalLettersIgnoringASCIICase function instead of using
both equalIgnoringASCIICase and convertToASCIILowercase.

  • html/HTMLAudioElement.cpp:

(WebCore::HTMLAudioElement::HTMLAudioElement): Marked this inline since we
want it inlined the one place it's used.
(WebCore::HTMLAudioElement::create): Use auto and named the local variable
just element.
(WebCore::HTMLAudioElement::createForJSConstructor): Call create rather than
repeating the code from create. Use setAttributeWithoutSynchronization directly
rather than calling helper functions. Removed unneeded null check;
setAttributeWithoutSynchronization handles null by removing the attribute.

  • html/HTMLAudioElement.h: Changed src argument of createForJSConstructor to

take AtomicString since that is what we need for an attribute value.

  • html/HTMLAudioElement.idl: Use [AtomicString] for src argument.
  • html/HTMLElement.cpp:

(WebCore::HTMLElement::directionality): Use reference to value of
attribute instead of copy to avoid reference count churn.
(WebCore::HTMLElement::shouldAutocorrect): Ditto.

  • html/HTMLImageElement.cpp:

(WebCore::HTMLImageElement::bestFitSourceFromPictureElement): Call the new
parsedMediaAttribute instead of the old mediaQuerySet function. Other than the
name, the only change is that the result is now a const pointer.

  • html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::updateType): Removed unnecessary local variable,
resulting in code that is slightly cleaner and possibly eliminating reference
count chrun.

  • html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::setSrc): Deleted.
(WebCore::HTMLMediaElement::selectNextSourceChild): Use parsedMediaAttribute and
removed rendudant direct check of whether mediaAttr is present. Changed logging to
get the media attribute directly rather than with a helper function. Changed type
to get the type attribute directly rather than with a helper function.
(WebCore::HTMLMediaElement::sourceWasAdded): Take a reference rather than a pointer.
(WebCore::HTMLMediaElement::sourceWasRemoved): Ditto.
(WebCore::HTMLMediaElement::doesHaveAttribute): Use reference to value of
attribute instead of copy to avoid reference count churn.

  • html/HTMLMediaElement.h: Updated for the above.
  • html/HTMLSourceElement.cpp:

(WebCore::HTMLSourceElement::insertedInto): Pass reference instead of pointer.
(WebCore::HTMLSourceElement::removedFrom): Ditto.
(WebCore::HTMLSourceElement::setSrc): Deleted.
(WebCore::HTMLSourceElement::media): Deleted.
(WebCore::HTMLSourceElement::setMedia): Deleted.
(WebCore::HTMLSourceElement::type): Deleted.
(WebCore::HTMLSourceElement::setType): Deleted.
(WebCore::HTMLSourceElement::parseAttribute): Clear out m_cachedParsedMediaAttribute.
This makes the parsing be lazy. The old code would parse the attribute value to make
a MediaQuerySet here, but we do it in parsedMediaAttribute now.
(WebCore::HTMLSourceElement::parsedMediaAttribute): Added. Creates a MediaQuerySet
if needed, and returns it or null.

  • html/HTMLSourceElement.h: Removed include of MediaList.h. Removed unneeded media,

type, setSrc, setMedia, and setType functions. Replaced the mediaQuerySet function
with the parsedMediaAttribute, which returns a const pointer rather than non-const,
and is also non-inline because it lazily creates the MediaQuerySet as needed. Replaced
m_mediaQuerySet with m_cachedParsedMediaAttribute.

  • html/HTMLSourceElement.idl: Use [Reflect] for both type and media.
  • html/MediaDocument.cpp: Modernize the MediaDocumentParser constructor.

(WebCore::MediaDocumentParser::createDocumentStructure): Instead of setSrc, use
setAttributeWithoutSynchronization for the src attribute just as we do for all the
other attributes of the newly created video element.

  • html/MediaElementSession.cpp:

(WebCore::MediaElementSession::wirelessVideoPlaybackDisabled): Use reference to
value of attribute instead of copy to avoid reference count churn.

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::defaultSubstituteDataForURL): Ditto.

  • rendering/RenderThemeIOS.mm:

(WebCore::getAttachmentProgress): Ditto.

  • rendering/RenderThemeMac.mm:

(WebCore::AttachmentLayout::layOutSubtitle): Ditto.
(WebCore::RenderThemeMac::paintAttachment): Ditto.

  • svg/SVGHKernElement.cpp:

(WebCore::SVGHKernElement::buildHorizontalKerningPair): Ditto.

  • svg/SVGVKernElement.cpp:

(WebCore::SVGVKernElement::buildVerticalKerningPair): Ditto.

  • svg/animation/SVGSMILElement.cpp:

(WebCore::SVGSMILElement::buildPendingResource): Ditto. Also use parentElement
instead of writing out code that does exactly what it does.

Source/WebKit/mac:

  • WebView/WebHTMLRepresentation.mm:

(matchLabelsAgainstElement): Added handling of nullptr since nothing guarantees the
passed in element can't be null. Removed unnecessary conversion from NSString to String
and then back to NSString.

11:00 AM Changeset in webkit [217773] by weinig@apple.com
  • 41 edits
    38 adds
    4 deletes in trunk

Can't use Object.defineProperty() to add an item to a DOMStringMap or Storage
https://bugs.webkit.org/show_bug.cgi?id=172687

Reviewed by Darin Adler.

Source/WebCore:

  • Adds support for generating the defineOwnProperty ClassInfo method table hook to add support for Object.defineProperty(). The implementation follows WebIDL section 3.9.3 DefineOwnProperty?. (https://heycam.github.io/webidl/#legacy-platform-object-defineownproperty)
  • Adds support for generating named setters in addition to the already supported indexed setters as much of the required work was needed to make defineOwnProperty work for those properties. This patch does not aim to follow WebIDL faithfully, but rather to generate as close to the custom code as possible. A follow up change will attempt to match WebIDL more closely (which will also require changes to GetOwnPropertySlot?).
  • Removes the need for custom bindings in DOMStringMap, HTMLOptionsCollection and HTMLSelectElement. Gets us one function away from supporting Storage.
  • Tidies up generated headers a bit by grouping all the ClassInfo method table hooks together.
  • Removes support for CustomIndexedSetter, which is no longer used.

Test: js/dom/legacy-platform-object-defineOwnProperty.html

  • CMakeLists.txt:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSBindingsAllInOne.cpp:

Removed now unneeded custom bindings.

  • bindings/js/JSDOMStringMapCustom.cpp: Removed.
  • bindings/js/JSHTMLOptionsCollectionCustom.cpp: Removed.
  • bindings/js/JSHTMLSelectElementCustom.cpp: Removed.
  • bindings/js/JSHTMLSelectElementCustom.h: Removed.
  • bindings/js/JSStorageCustom.cpp:

(WebCore::JSStorage::putDelegate): Deleted.
Remove more code that is now generated.

  • bindings/scripts/CodeGeneratorJS.pm:

(GenerateIndexedGetter):

  • Add missing check for indexed setter operation when determining if the property should be ReadOnly.
  • Add support for custom implementation name, matching GenerateNamedGetter.
  • Convert to taking an outputArray and indent to allow more flexibility going forward.

(GenerateNamedGetter):

  • Add missing check for named setter operation when determining if the property should be ReadOnly.
  • Convert to taking an outputArray and indent to allow more flexibility going forward.

(GenerateGetOwnPropertySlotBody):

  • Convert to taking an outputArray and indent to allow more flexibility going forward.
  • Take over responsibility for checking and bailing if CustomGetOwnPropertySlot is set.

(GenerateGetOwnPropertySlotBodyByIndex):

  • Convert to taking an outputArray and indent to allow more flexibility going forward.
  • Take over responsibility for checking and bailing if CustomGetOwnPropertySlotByIndex is set.

(GenerateGetOwnPropertyNames):

  • Convert to taking an outputArray and indent to allow more flexibility going forward.
  • Take over responsibility for checking and bailing if CustomEnumerateProperty is set.

(GenerateInvokeIndexedPropertySetter):
Added. Implements the 'invoke an indexed property setter' algorithm (https://heycam.github.io/webidl/#invoke-indexed-setter)

(GenerateInvokeNamedPropertySetter):
Added. Implements the 'invoke a named property setter' algorithm (https://heycam.github.io/webidl/#invoke-named-setter)

(GeneratePut):
(GeneratePutByIndex):

  • Convert to taking an outputArray and indent to allow more flexibility going forward.
  • Take over responsibility for checking and bailing if CustomPut is set.
  • Adds support for CEReactions.
  • Adopts GenerateInvokeIndexedPropertySetter and GenerateInvokeNamedPropertySetter to simplify code.
  • Removes support for CustomIndexedSetter, which is no longer used.
  • Replaces use direct checking of OverrideBuiltins with check if any interface it inherits has it, which is how it is specified to work.

(GenerateIsUnforgeablePropertyName):
Added. Helper subroutine to generate a function that compares a property name
agains all the property names defined as Unforgeable on the interface. Used by
GenerateDefineOwnProperty.

(GenerateDefineOwnProperty):
Added. Implements section 3.9.3 DefineOwnProperty? (https://heycam.github.io/webidl/#legacy-platform-object-defineownproperty)
Adds support for CEReactions.

(GenerateDeletePropertyCommon):
Switch from getting OverrideBuiltins directly off the interface to using InheritsExtendedAttribute.

(GenerateNamedDeleterDefinition):
Switch signature to match peer generators.

(InstanceOverridesGetOwnPropertySlot):
Update for rename of JSCustomGetOwnPropertySlotAndDescriptor to CustomGetOwnPropertySlotAndDescriptor.

(InstanceOverridesGetOwnPropertySlotByIndex):
Added. Like InstanceOverridesGetOwnPropertySlot, but checks for the new CustomGetOwnPropertySlotByIndex
rather than CustomGetOwnPropertySlot.

(InstanceOverridesGetOwnPropertyNames):
Added. Moves complex predicate out of line.

(InstanceOverridesPut):
Removes now unused CustomIndexedSetter, and adds CustomPutFunction, which used to be checked separately.

(InstanceOverridesDefineOwnProperty):
Added.

(GenerateHeader):

  • Moves more structure flags together.
  • Simplifies predicates and moves the ClassInfo method table hooks together.

(GenerateImplementation):

  • Moves the ClassInfo method table hooks together
  • Adds call to GenerateDefineOwnProperty.

(GenerateLegacyCallerDefinitions):
Simplify bail condition to match other generators.

(GeneratePrototypeDeclaration):
Update for new extended attribute names.

(InstanceOverridesCall):
Renamed from IsCallable for consistency.

(HasComplexGetOwnProperty):
Deleted. Unused.

(InstanceOverridesPutImplementation):
Deleted. Unused.

(InstanceOverridesPutDeclaration):
Deleted. Unused.

(IsCallable):
Renamed, for consistency, to InstanceOverridesCall .

  • bindings/scripts/IDLAttributes.json:

Renames a few of the extended attributes to remove the JS prefix. This time
around I only did the ones in the area I was working, but we should probably
remove most of the rest.

  • css/CSSStyleDeclaration.idl:

Update for JSCustomGetOwnPropertySlotAndDescriptor -> CustomGetOwnPropertySlotAndDescriptor rename.

  • dom/DOMStringMap.idl:

Remove CustomNamedSetter and add uncomment out the setter.

  • dom/DatasetDOMStringMap.cpp:

(WebCore::DatasetDOMStringMap::setNamedItem):
(WebCore::DatasetDOMStringMap::setItem): Deleted.

  • dom/DatasetDOMStringMap.h:

Rename setItem to setNamedItem, which is what the generator expects.

  • dom/Node.idl:

Update for JSCustomPushEventHandlerScope -> CustomPushEventHandlerScope rename.

  • html/HTMLAppletElement.idl:
  • html/HTMLEmbedElement.idl:
  • html/HTMLObjectElement.idl:

Update for JSCustomGetOwnPropertySlotAndDescriptor -> CustomGetOwnPropertySlotAndDescriptor rename.

  • html/HTMLCollection.cpp:

(WebCore::HTMLCollection::isSupportedPropertyName):

  • html/HTMLCollection.h:

Add isSupportedPropertyName function which is used by the bindings and is now needed.

  • html/HTMLElement.idl:

Update for JSCustomPushEventHandlerScope -> CustomPushEventHandlerScope rename.

  • html/HTMLOptionsCollection.h:

(WebCore::HTMLOptionsCollection::setItem):
Add setItem() implementation which just forwards to the select element. Also, add
a type alias to make the implementations of item and namedItem less verbose.

  • html/HTMLOptionsCollection.idl:

Remove CustomIndexedSetter and uncomment the setter.

  • html/HTMLSelectElement.idl:

Remove CustomIndexedSetter and uncomment the setter. Also, reformat
to match the WHATWG spec.

  • page/DOMWindow.idl:

Update for JSCustomDefineOwnProperty -> CustomDefineOwnProperty rename. Add
CustomGetOwnPropertySlotByIndex which is now needed to remove some unsound
assumptions the generator was making.

  • page/Location.idl:

Update for removal of JS prefix from a bunch of extended attributes.

  • page/UserMessageHandlersNamespace.idl:

Update for JSCustomGetOwnPropertySlotAndDescriptor -> CustomGetOwnPropertySlotAndDescriptor rename.

  • storage/Storage.idl:

Remove CustomNamedSetter and uncomment the setter.

  • bindings/scripts/test/JS/JSInterfaceName.h:
  • bindings/scripts/test/JS/JSTestEventTarget.h:
  • bindings/scripts/test/JS/JSTestIndexedSetterNoIdentifier.cpp: Added.
  • bindings/scripts/test/JS/JSTestIndexedSetterNoIdentifier.h: Added.
  • bindings/scripts/test/JS/JSTestIndexedSetterThrowingException.cpp: Added.
  • bindings/scripts/test/JS/JSTestIndexedSetterThrowingException.h: Added.
  • bindings/scripts/test/JS/JSTestIndexedSetterWithIdentifier.cpp: Added.
  • bindings/scripts/test/JS/JSTestIndexedSetterWithIdentifier.h: Added.
  • bindings/scripts/test/JS/JSTestInterface.cpp:
  • bindings/scripts/test/JS/JSTestNamedAndIndexedSetterNoIdentifier.cpp: Added.
  • bindings/scripts/test/JS/JSTestNamedAndIndexedSetterNoIdentifier.h: Added.
  • bindings/scripts/test/JS/JSTestNamedAndIndexedSetterThrowingException.cpp: Added.
  • bindings/scripts/test/JS/JSTestNamedAndIndexedSetterThrowingException.h: Added.
  • bindings/scripts/test/JS/JSTestNamedAndIndexedSetterWithIdentifier.cpp: Added.
  • bindings/scripts/test/JS/JSTestNamedAndIndexedSetterWithIdentifier.h: Added.
  • bindings/scripts/test/JS/JSTestNamedDeleterNoIdentifier.h:
  • bindings/scripts/test/JS/JSTestNamedDeleterThrowingException.h:
  • bindings/scripts/test/JS/JSTestNamedDeleterWithIdentifier.cpp:
  • bindings/scripts/test/JS/JSTestNamedDeleterWithIdentifier.h:
  • bindings/scripts/test/JS/JSTestNamedDeleterWithIndexedGetter.h:
  • bindings/scripts/test/JS/JSTestNamedSetterNoIdentifier.cpp: Added.
  • bindings/scripts/test/JS/JSTestNamedSetterNoIdentifier.h: Added.
  • bindings/scripts/test/JS/JSTestNamedSetterThrowingException.cpp: Added.
  • bindings/scripts/test/JS/JSTestNamedSetterThrowingException.h: Added.
  • bindings/scripts/test/JS/JSTestNamedSetterWithIdentifier.cpp: Added.
  • bindings/scripts/test/JS/JSTestNamedSetterWithIdentifier.h: Added.
  • bindings/scripts/test/JS/JSTestNamedSetterWithOverrideBuiltins.cpp: Added.
  • bindings/scripts/test/JS/JSTestNamedSetterWithOverrideBuiltins.h: Added.
  • bindings/scripts/test/JS/JSTestNamedSetterWithUnforgableProperties.cpp: Added.
  • bindings/scripts/test/JS/JSTestNamedSetterWithUnforgableProperties.h: Added.
  • bindings/scripts/test/JS/JSTestNamedSetterWithUnforgablePropertiesAndOverrideBuiltins.cpp: Added.
  • bindings/scripts/test/JS/JSTestNamedSetterWithUnforgablePropertiesAndOverrideBuiltins.h: Added.
  • bindings/scripts/test/JS/JSTestObj.cpp:
  • bindings/scripts/test/JS/JSTestObj.h:
  • bindings/scripts/test/JS/JSTestOverrideBuiltins.h:
  • bindings/scripts/test/TestIndexedSetterNoIdentifier.idl: Added.
  • bindings/scripts/test/TestIndexedSetterThrowingException.idl: Added.
  • bindings/scripts/test/TestIndexedSetterWithIdentifier.idl: Added.
  • bindings/scripts/test/TestNamedAndIndexedSetterNoIdentifier.idl: Added.
  • bindings/scripts/test/TestNamedAndIndexedSetterThrowingException.idl: Added.
  • bindings/scripts/test/TestNamedAndIndexedSetterWithIdentifier.idl: Added.
  • bindings/scripts/test/TestNamedDeleterWithIdentifier.idl:
  • bindings/scripts/test/TestNamedSetterNoIdentifier.idl: Added.
  • bindings/scripts/test/TestNamedSetterThrowingException.idl: Added.
  • bindings/scripts/test/TestNamedSetterWithIdentifier.idl: Added.
  • bindings/scripts/test/TestNamedSetterWithOverrideBuiltins.idl: Added.
  • bindings/scripts/test/TestNamedSetterWithUnforgableProperties.idl: Added.
  • bindings/scripts/test/TestNamedSetterWithUnforgablePropertiesAndOverrideBuiltins.idl: Added.

Added new tests and updated some results.

LayoutTests:

  • js/dom/legacy-platform-object-defineOwnProperty-expected.txt: Added.
  • js/dom/legacy-platform-object-defineOwnProperty.html: Added.

New tests for Object.defineProperty on Storage and DOMStringMap.

  • js/dom/named-property-deleter.html:

Fix test which was deleting the wrong property. This doesn't change the result,
but is now actually testing the thing it meant to.

  • storage/domstorage/localstorage/delete-defineproperty-removal-expected.txt:
  • storage/domstorage/localstorage/delete-defineproperty-removal.html:

Convert to testharness.js and show that the results are now correct.

10:57 AM Changeset in webkit [217772] by Simon Fraser
  • 83 edits in trunk

Object bounding box wrong for some paths
https://bugs.webkit.org/show_bug.cgi?id=172866

Reviewed by Tim Horton.
Source/WebCore:

RenderSVGShape::calculateObjectBoundingBox() should use boundingRect()
rather than fastBoundingRect(), because the latter includes control points.

Covered by existing tests.

  • rendering/svg/RenderSVGShape.cpp:

(WebCore::RenderSVGShape::calculateObjectBoundingBox):

LayoutTests:

So many new baselines.

  • platform/ios/svg/W3C-SVG-1.1-SE/paths-dom-02-f-expected.txt:
  • platform/ios/svg/W3C-SVG-1.1/animate-elem-06-t-expected.txt:
  • platform/ios/svg/W3C-SVG-1.1/animate-elem-08-t-expected.txt:
  • platform/ios/svg/W3C-SVG-1.1/animate-elem-83-t-expected.txt:
  • platform/ios/svg/W3C-SVG-1.1/filters-example-01-b-expected.txt:
  • platform/ios/svg/W3C-SVG-1.1/metadata-example-01-b-expected.txt:
  • platform/ios/svg/W3C-SVG-1.1/paths-data-01-t-expected.txt:
  • platform/ios/svg/W3C-SVG-1.1/paths-data-02-t-expected.txt:
  • platform/ios/svg/W3C-SVG-1.1/paths-data-03-f-expected.txt:
  • platform/ios/svg/W3C-SVG-1.1/paths-data-12-t-expected.txt:
  • platform/ios/svg/W3C-SVG-1.1/text-path-01-b-expected.txt:
  • platform/ios/svg/batik/text/longTextOnPath-expected.txt:
  • platform/ios/svg/batik/text/smallFonts-expected.txt:
  • platform/ios/svg/batik/text/textAnchor-expected.txt:
  • platform/ios/svg/batik/text/textDecoration-expected.txt:
  • platform/ios/svg/batik/text/textEffect-expected.txt:
  • platform/ios/svg/batik/text/textEffect2-expected.txt:
  • platform/ios/svg/batik/text/textEffect3-expected.txt:
  • platform/ios/svg/batik/text/textFeatures-expected.txt:
  • platform/ios/svg/batik/text/textGlyphOrientationHorizontal-expected.txt:
  • platform/ios/svg/batik/text/textLayout-expected.txt:
  • platform/ios/svg/batik/text/textLayout2-expected.txt:
  • platform/ios/svg/batik/text/textLength-expected.txt:
  • platform/ios/svg/batik/text/textOnPath-expected.txt:
  • platform/ios/svg/batik/text/textOnPathSpaces-expected.txt:
  • platform/ios/svg/batik/text/textPosition-expected.txt:
  • platform/ios/svg/batik/text/textPosition2-expected.txt:
  • platform/ios/svg/batik/text/textProperties-expected.txt:
  • platform/ios/svg/batik/text/textProperties2-expected.txt:
  • platform/ios/svg/batik/text/textStyles-expected.txt:
  • platform/ios/svg/batik/text/verticalText-expected.txt:
  • platform/ios/svg/batik/text/verticalTextOnPath-expected.txt:
  • platform/ios/svg/custom/broken-internal-references-expected.txt:
  • platform/ios/svg/custom/path-textPath-simulation-expected.txt:
  • platform/ios/svg/custom/relative-sized-use-on-symbol-expected.txt:
  • platform/ios/svg/custom/relative-sized-use-without-attributes-on-symbol-expected.txt:
  • platform/ios/svg/custom/second-inline-text-expected.txt:
  • platform/ios/svg/text/text-path-01-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1-SE/paths-dom-02-f-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/animate-elem-06-t-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/animate-elem-07-t-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/animate-elem-08-t-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/animate-elem-83-t-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-example-01-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/metadata-example-01-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/paths-data-01-t-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/paths-data-02-t-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/paths-data-03-f-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/paths-data-12-t-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/text-path-01-b-expected.txt:
  • platform/mac/svg/batik/filters/filterRegions-expected.txt:
  • platform/mac/svg/batik/paints/patternPreserveAspectRatioA-expected.txt:
  • platform/mac/svg/batik/paints/patternRegionA-expected.txt:
  • platform/mac/svg/batik/paints/patternRegions-expected.txt:
  • platform/mac/svg/batik/text/smallFonts-expected.txt:
  • platform/mac/svg/batik/text/textAnchor-expected.txt:
  • platform/mac/svg/batik/text/textEffect2-expected.txt:
  • platform/mac/svg/batik/text/textEffect3-expected.txt:
  • platform/mac/svg/batik/text/textFeatures-expected.txt:
  • platform/mac/svg/batik/text/textGlyphOrientationHorizontal-expected.txt:
  • platform/mac/svg/batik/text/textLayout-expected.txt:
  • platform/mac/svg/batik/text/textLayout2-expected.txt:
  • platform/mac/svg/batik/text/textOnPathSpaces-expected.txt:
  • platform/mac/svg/batik/text/textPosition2-expected.txt:
  • platform/mac/svg/batik/text/textProperties-expected.txt:
  • platform/mac/svg/batik/text/textProperties2-expected.txt:
  • platform/mac/svg/batik/text/textStyles-expected.txt:
  • platform/mac/svg/custom/animate-path-morphing-expected.txt:
  • platform/mac/svg/custom/broken-internal-references-expected.txt:
  • platform/mac/svg/custom/path-textPath-simulation-expected.txt:
  • platform/mac/svg/custom/relative-sized-use-on-symbol-expected.txt:
  • platform/mac/svg/custom/relative-sized-use-without-attributes-on-symbol-expected.txt:
  • platform/mac/svg/custom/second-inline-text-expected.txt:
  • platform/mac/svg/dom/SVGPathSegList-segment-modification-expected.txt:
  • platform/mac/svg/hixie/perf/001-expected.txt:
  • platform/mac/svg/hixie/perf/002-expected.txt:
  • platform/mac/svg/text/text-path-01-b-expected.txt:
  • platform/mac/svg/zoom/page/zoom-svg-through-object-with-override-size-expected.txt:
  • svg/custom/control-points-for-S-and-T-expected.txt:
  • svg/custom/use-nested-transform-expected.txt:
9:38 AM Changeset in webkit [217771] by Konstantin Tokarev
  • 10 edits
    1 add in trunk/Source

Fix build of Windows-specific code with ICU 59.1
https://bugs.webkit.org/show_bug.cgi?id=172729

Reviewed by Darin Adler.

Source/JavaScriptCore:

Fix conversions from WTF::String to wchar_t* and vice versa.

  • jsc.cpp:

(currentWorkingDirectory):
(fetchModuleFromLocalFileSystem):

  • runtime/DateConversion.cpp:

(JSC::formatDateTime):

Source/WebCore:

Fix conversions from WTF::String to wchar_t* and vice versa.
No new tests needed.

  • platform/graphics/win/MediaPlayerPrivateMediaFoundation.cpp:

(WebCore::mimeTypeCache):
(WebCore::MediaPlayerPrivateMediaFoundation::startCreateMediaSource):

Source/WebKit/win:

Fix conversions from WTF::String to wchar_t* and vice versa.

  • Plugins/PluginDatabaseWin.cpp:

(WebCore::addPluginPathsFromRegistry):
(WebCore::PluginDatabase::getPluginPathsInDirectories):
(WebCore::addMozillaPluginDirectories):
(WebCore::addWindowsMediaPlayerPluginDirectory):
(WebCore::addAdobeAcrobatPluginDirectory):
(WebCore::addJavaPluginDirectory):
(WebCore::safariPluginsDirectory):
(WebCore::addMacromediaPluginDirectories):

  • Plugins/PluginPackageWin.cpp:

(WebCore::getVersionInfo):
(WebCore::PluginPackage::fetchInfo):
(WebCore::PluginPackage::load):

  • Plugins/PluginViewWin.cpp:

(WebCore::PluginView::handlePostReadFile):

Source/WTF:

  • wtf/text/win/WCharStringExtras.h: Added new header file wuth helper

functions for conversion between WTF::String and wchart_t*.
(WTF::stringToNullTerminatedWChar):
(WTF::wcharToString):
(WTF::nullTerminatedWCharToString):

9:26 AM Changeset in webkit [217770] by Yusuke Suzuki
  • 2 edits in trunk/Source/JavaScriptCore

[JSC] Drop unnecessary USE(CF) guard for getenv
https://bugs.webkit.org/show_bug.cgi?id=172903

Reviewed by Sam Weinig.

getenv is not related to USE(CF) and OS(UNIX). It seems that this
ifdef only hits in WinCairo, but WinCairo can use getenv.
Moreover, in VM::VM, we already use getenv without any ifdef guard.

This patch just drops it.

  • runtime/VM.cpp:

(JSC::enableAssembler):

9:21 AM Changeset in webkit [217769] by Yusuke Suzuki
  • 2 edits in trunk/Source/JavaScriptCore

[JSC] Drop OS(DARWIN) for uintptr_t type conflict
https://bugs.webkit.org/show_bug.cgi?id=172904

Reviewed by Sam Weinig.

In non-Darwin environment, uintptr_t may have the same type
to uint64_t. We avoided the compile error by using OS(DARWIN).
But, since it depends on cstdint implementaion rather than OS, it is flaky.
Instead, we just use template parameter IntegralType.
And we describe the type constraint in a SFINAE manner.

  • dfg/DFGOpInfo.h:

(JSC::DFG::OpInfo::OpInfo):

8:51 AM Changeset in webkit [217768] by Jonathan Bedard
  • 7 edits in trunk/LayoutTests

Unreviewed test gardening

Gardening test expectations for layout tests on iOS device. Moving
expectations for tests which fail on Simulator but pass on device.

  • platform/ios-simulator-wk1/TestExpectations:
  • platform/ios-simulator-wk2/TestExpectations:
  • platform/ios-simulator/TestExpectations:
  • platform/ios-wk1/TestExpectations:
  • platform/ios-wk2/TestExpectations:
  • platform/ios/TestExpectations:
8:21 AM Changeset in webkit [217767] by Chris Dumez
  • 7 edits
    3 adds in trunk

Implement DOMMatrixReadOnly.transformPoint()
https://bugs.webkit.org/show_bug.cgi?id=172900

Reviewed by Simon Fraser.

LayoutTests/imported/w3c:

Rebaseline W3C test now that more checks are passing.

  • web-platform-tests/css/geometry-1/DOMMatrix-newobject-expected.txt:

Source/WebCore:

Implement DOMMatrixReadOnly.transformPoint():

Test: imported/blink/fast/dom/geometry-interfaces-dom-matrix-transformPoint.html

  • css/DOMMatrixReadOnly.cpp:

(WebCore::DOMMatrixReadOnly::transformPoint):

  • css/DOMMatrixReadOnly.h:
  • css/DOMMatrixReadOnly.idl:

LayoutTests:

  • imported/blink/fast/dom/geometry-interfaces-dom-matrix-transformPoint-expected.txt: Added.
  • imported/blink/fast/dom/geometry-interfaces-dom-matrix-transformPoint.html: Added.
  • imported/blink/fast/dom/resources/geometry-interfaces-test-helpers.js: Added.

Import test coverage from Blink.

4:26 AM Changeset in webkit [217766] by zandobersek@gmail.com
  • 3 edits
    2 adds in trunk/LayoutTests

Unreviewed GTK+ gardening. Add a few test failure expectations, linking them to
existing bugs. Update or add GTK+-specific baselines for three tests.

  • platform/gtk/TestExpectations:
  • platform/gtk/accessibility/disabled-controls-not-focusable-expected.txt: Added.
  • platform/gtk/compositing/backing-store-attachment-1-expected.txt: Added.
  • platform/gtk/css1/box_properties/acid_test-expected.txt:
3:24 AM Changeset in webkit [217765] by zandobersek@gmail.com
  • 6 edits in trunk/Source/WebCore

[GCrypt] Improve comments in AES, PBKDF2, RSA-SSA algorithm implementations
https://bugs.webkit.org/show_bug.cgi?id=172894

Reviewed by Michael Catanzaro.

Add or improve code comments in libgcrypt implementations for the AES_CBC,
AES_GCM, AES_KW, PBKDF2 and RSA-SSA algorithms.

  • crypto/gcrypt/CryptoAlgorithmAES_CBCGCrypt.cpp: Add generic comments that

describe each operation that's performed.
(WebCore::gcryptEncrypt):
(WebCore::gcryptDecrypt):

  • crypto/gcrypt/CryptoAlgorithmAES_GCMGCrypt.cpp: Ditto.

(WebCore::gcryptEncrypt):
(WebCore::gcryptDecrypt):

  • crypto/gcrypt/CryptoAlgorithmAES_KWGCrypt.cpp: Ditto.

(WebCore::gcryptWrapKey):
(WebCore::gcryptUnwrapKey):

  • crypto/gcrypt/CryptoAlgorithmPBKDF2GCrypt.cpp: Ditto.

(WebCore::gcryptDeriveBits):

  • crypto/gcrypt/CryptoAlgorithmRSASSA_PKCS1_v1_5GCrypt.cpp:

(WebCore::gcryptVerify): Align the verification results comment with the
one that's used in the ECDSA implementation.

Jun 3, 2017:

6:54 PM Changeset in webkit [217764] by Chris Dumez
  • 10 edits
    2 adds in trunk

Implement DOMMatrix's fromFloat32Array / fromFloat64Array & toFloat32Array / toFloat64Array
https://bugs.webkit.org/show_bug.cgi?id=172898

Reviewed by Sam Weinig.

LayoutTests/imported/w3c:

Rebaseline web-platform-test now that more checks are passing.

  • web-platform-tests/css/geometry-1/DOMMatrix-newobject-expected.txt:

Source/WebCore:

Implement DOMMatrix's fromFloat32Array / fromFloat64Array & toFloat32Array / toFloat64Array
as per:

Test: http/wpt/geometry/DOMMatrix-from-to-typed-arrays.html

  • css/DOMMatrix.cpp:

(WebCore::DOMMatrix::fromFloat32Array):
(WebCore::DOMMatrix::fromFloat64Array):

  • css/DOMMatrix.h:
  • css/DOMMatrix.idl:
  • css/DOMMatrixReadOnly.cpp:

(WebCore::DOMMatrixReadOnly::fromFloat32Array):
(WebCore::DOMMatrixReadOnly::fromFloat64Array):
(WebCore::DOMMatrixReadOnly::toFloat32Array):
(WebCore::DOMMatrixReadOnly::toFloat64Array):

  • css/DOMMatrixReadOnly.h:
  • css/DOMMatrixReadOnly.idl:

LayoutTests:

Add layout test coverage.

  • http/wpt/geometry/DOMMatrix-from-to-typed-arrays-expected.txt: Added.
  • http/wpt/geometry/DOMMatrix-from-to-typed-arrays.html: Added.
2:09 PM Changeset in webkit [217763] by Simon Fraser
  • 12 edits
    1 copy
    3 adds in trunk

Implement DOMPointReadOnly.matrixTransform()
https://bugs.webkit.org/show_bug.cgi?id=172896

Reviewed by Chris Dumez.

LayoutTests/imported/w3c:

New baselines.

  • web-platform-tests/css/geometry-1/DOMPoint-002-expected.txt:
  • web-platform-tests/css/geometry-1/historical-expected.txt:

Source/WebCore:

DOMPointReadOnly.matrixTransform() creates the matrix or throws, then uses
a new function in TransformationMatrix to map x,y,z,w through the matrix.

Test: http/wpt/geometry/DOMPoint-003.html

  • CMakeLists.txt:
  • WebCore.xcodeproj/project.pbxproj:
  • css/DOMMatrixReadOnly.h:

(WebCore::DOMMatrixReadOnly::transformationMatrix):

  • dom/DOMPointReadOnly.cpp: Copied from Source/WebCore/dom/DOMPointReadOnly.idl.

(WebCore::DOMPointReadOnly::matrixTransform):

  • dom/DOMPointReadOnly.h:
  • dom/DOMPointReadOnly.idl:
  • platform/graphics/transforms/TransformationMatrix.cpp:

(WebCore::TransformationMatrix::map4ComponentPoint):

  • platform/graphics/transforms/TransformationMatrix.h:

LayoutTests:

DOMPoint-003.html should be upstreamed at some point.

  • http/wpt/geometry/DOMPoint-003-expected.txt: Added.
  • http/wpt/geometry/DOMPoint-003.html: Added.
11:07 AM Changeset in webkit [217762] by mmaxfield@apple.com
  • 4 edits in trunk/Source/WebCore

Cleanup arguments to preparePlatformFont() and fontWithFamily() in FontCacheCoreText.cpp
https://bugs.webkit.org/show_bug.cgi?id=172886

Reviewed by Simon Fraser.

The number of arguments is getting out of control.

No new tests because there is no behavior change.

  • platform/graphics/FontCache.h:
  • platform/graphics/cocoa/FontCacheCoreText.cpp:

(WebCore::preparePlatformFont):
(WebCore::fontWithFamily):
(WebCore::FontCache::createFontPlatformData):
(WebCore::FontCache::systemFallbackForCharacters):

  • platform/graphics/mac/FontCacheMac.mm:

(WebCore::platformFontWithFamilySpecialCase):

  • platform/graphics/mac/FontCustomPlatformData.cpp:

(WebCore::FontCustomPlatformData::fontPlatformData):

  • platform/spi/cocoa/CoreTextSPI.h:
10:58 AM Changeset in webkit [217761] by Jonathan Bedard
  • 2 edits in trunk/LayoutTests

Unreviewed test gardening

  • platform/ios/TestExpectations: Update expectation for tests passing

after https://bugs.webkit.org/show_bug.cgi?id=139968.

9:31 AM Changeset in webkit [217760] by Yusuke Suzuki
  • 9 edits
    4 adds in trunk

Script modules should be able to import data urls
https://bugs.webkit.org/show_bug.cgi?id=171594

Reviewed by Youenn Fablet.

LayoutTests/imported/w3c:

  • web-platform-tests/html/semantics/scripting-1/the-script-element/data-url-expected.txt:
  • web-platform-tests/html/webappapis/scripting/processing-model-2/compile-error-data-url-expected.txt:
  • web-platform-tests/html/webappapis/scripting/processing-model-2/runtime-error-data-url-expected.txt:

Source/WebCore:

Tests: js/dom/classic-script-with-data-url.html

js/dom/modules/module-script-with-data-url.html

Script tag resources should have SameOriginDataURLFlag::Set to load data URLs
since script tags should treat data URLs as same origin.

  • bindings/js/CachedScriptFetcher.cpp:

(WebCore::CachedScriptFetcher::requestScriptWithCache):

LayoutTests:

  • http/tests/security/script-with-dataurl-expected.txt:
  • http/tests/security/script-with-dataurl.html:

Now, the spec is changed and errors from data: scripts should not be muted.
https://github.com/whatwg/html/issues/1778

  • js/dom/classic-script-with-data-url-expected.txt: Added.
  • js/dom/classic-script-with-data-url.html: Added.
  • js/dom/modules/module-script-with-data-url-expected.txt: Added.
  • js/dom/modules/module-script-with-data-url.html: Added.
9:21 AM Changeset in webkit [217759] by Simon Fraser
  • 4 edits in trunk/Source

Sort the project file.

Source/WebCore:

  • WebCore.xcodeproj/project.pbxproj:

Source/WebKit2:

  • WebKit2.xcodeproj/project.pbxproj:
9:11 AM Changeset in webkit [217758] by Jonathan Bedard
  • 3 edits in trunk/Tools

webkitpy: Add contextmanager to disable logging for a block
https://bugs.webkit.org/show_bug.cgi?id=172876

Reviewed by Daniel Bates.

Add a context manager which will use an OutputCapture object to capture logging
in a block and hold it in a variable.

  • Scripts/webkitpy/common/system/outputcapture.py:

(OutputCaptureScope):
(OutputCaptureScope.init): Construct with OutputCapture object.
(OutputCaptureScope.enter): Begin capturing output.
(OutputCaptureScope.exit): Restore output and save captured output to a variable.
output and retain the resulting log.

  • Scripts/webkitpy/common/system/outputcapture_unittest.py:

(OutputCaptureTest.test_output_capture_scope): Added.

8:26 AM Changeset in webkit [217757] by Jonathan Bedard
  • 3 edits in trunk/LayoutTests

Move css3/flexbox/image-percent-max-height.html failure expectation.
https://bugs.webkit.org/show_bug.cgi?id=126898

Unreviewed test gardening

css3/flexbox/image-percent-max-height.html fails on iOS-simulator,
but passes on device.

  • platform/ios-simulator/TestExpectations:
  • platform/ios/TestExpectations:
6:36 AM Changeset in webkit [217756] by Csaba Osztrogonác
  • 2 edits in trunk/Source/JavaScriptCore

[ARM] Unreviewed buildfix after r217711.

  • assembler/MacroAssemblerARM.h:

(JSC::MacroAssemblerARM::xor32):

4:24 AM Changeset in webkit [217755] by Yusuke Suzuki
  • 4 edits
    1 add in trunk

ASSERTION FAILED: "We should only declare a function as a lexically scoped variable in scopes where var declarations aren't allowed. ..." for function redeclaration with async function module export
https://bugs.webkit.org/show_bug.cgi?id=168844

Reviewed by Saam Barati.

JSTests:

  • modules/async-function-export.js: Added.

(f):
(export.async.f):

Source/JavaScriptCore:

As the same to the exported function declaration, we should set statementDepth = 1 for exported async function declaration.

  • parser/Parser.cpp:

(JSC::DepthManager::DepthManager):
(JSC::Parser<LexerType>::parseExportDeclaration):

  • parser/Parser.h:

(JSC::Parser::DepthManager::DepthManager): Deleted.
(JSC::Parser::DepthManager::~DepthManager): Deleted.

3:29 AM Changeset in webkit [217754] by zandobersek@gmail.com
  • 7 edits
    1 add in trunk/Source/WebCore

[GCrypt] Gather SUBTLE_CRYPTO utility functions in a single header
https://bugs.webkit.org/show_bug.cgi?id=172870

Reviewed by Jiewen Tan.

Gather the helper functions used across different source files
for libgcrypt-backed SUBTLE_CRYPTO implementations in a single
header file.

  • crypto/gcrypt/CryptoAlgorithmECDSAGCrypt.cpp:

(WebCore::hashCryptoDigestAlgorithm): Deleted.
(WebCore::hashAlgorithmName): Deleted.
(WebCore::mpiData): Deleted.

  • crypto/gcrypt/CryptoAlgorithmHKDFGCrypt.cpp:

(WebCore::gcryptDeriveBits):
(WebCore::macAlgorithmForHashFunction): Deleted.

  • crypto/gcrypt/CryptoAlgorithmPBKDF2GCrypt.cpp:

(WebCore::gcryptDeriveBits):

  • crypto/gcrypt/CryptoAlgorithmRSAES_PKCS1_v1_5GCrypt.cpp:

(WebCore::mpiData): Deleted.

  • crypto/gcrypt/CryptoAlgorithmRSASSA_PKCS1_v1_5GCrypt.cpp:

(WebCore::hashCryptoDigestAlgorithm): Deleted.
(WebCore::hashAlgorithmName): Deleted.
(WebCore::mpiData): Deleted.

  • crypto/gcrypt/CryptoAlgorithmRSA_OAEPGCrypt.cpp:

(WebCore::hashAlgorithmName): Deleted.
(WebCore::mpiData): Deleted.

  • crypto/gcrypt/GCryptUtilities.h: Added.

(WebCore::hashAlgorithmName):
(WebCore::hmacAlgorithm):
(WebCore::digestAlgorithm):
(WebCore::hashCryptoDigestAlgorithm):
(WebCore::mpiData):

3:27 AM Changeset in webkit [217753] by zandobersek@gmail.com
  • 2 edits in trunk/Source/WebCore

[GCrypt] Fix PK verification for ECDSA
https://bugs.webkit.org/show_bug.cgi?id=172857

Reviewed by Michael Catanzaro.

No new tests -- covered by existing tests.

  • crypto/gcrypt/CryptoAlgorithmECDSAGCrypt.cpp:

(WebCore::gcryptVerify): Return true if the verification succeeded or false
in any other case, avoiding spilling information about the exact cause of
verification failure.

1:07 AM Changeset in webkit [217752] by Chris Dumez
  • 3 edits in trunk/LayoutTests

http/tests/cache/disk-cache/shattered-deduplication.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=172868
<rdar://problem/31249379>

Reviewed by Antti Koivisto.

http/tests/cache/disk-cache/shattered-deduplication.html was flaky due to PDF banner
fade-in / fade-out animation.

<rdar://problem/31249379> is fixed for me with the proposed changes.

  • http/tests/cache/disk-cache/shattered-deduplication-expected.html:
  • Wait 1 second before calling notifyDone() to give the PDF as much time to render as

the test page.

  • Reduce frame height to 30 to not have to worry about the PDF tool banner being visible or not (as it fades in and out). 30 is still enough to see the color and confirm the test is passing.
  • http/tests/cache/disk-cache/shattered-deduplication.html:
  • Reduce frame height to 30 to not have to worry about the PDF tool banner being visible or not (as it fades in and out). 30 is still enough to see the color and confirm the test is passing.

Jun 2, 2017:

9:40 PM Changeset in webkit [217751] by Alan Bujtas
  • 4 edits in trunk

Cleanup FrameView::autoSizeIfEnabled.
https://bugs.webkit.org/show_bug.cgi?id=172889
<rdar://problem/32550783>

Reviewed by Tim Horton.

Source/WebCore:

This patch ensures that we always have a clean tree (and a valid and current document size) when
setting m_autoSizeContentSize.

  • page/FrameView.cpp:

(WebCore::FrameView::autoSizeIfEnabled):

Tools:

  • TestWebKitAPI/Tests/WebKit2Cocoa/AutoLayoutIntegration.mm:

(-[AutoLayoutWKWebView load:withWidth:expectingContentSize:resettingWidth:]): progression.
(TEST):

8:44 PM Changeset in webkit [217750] by Devin Rousso
  • 6 edits in trunk/Source/WebInspectorUI

Web Inspector: Don't create NavigationSidebarPanel classes until they are needed by a Tab
https://bugs.webkit.org/show_bug.cgi?id=172621

Reviewed by Timothy Hatcher.

  • UserInterface/Views/ContentBrowserTabContentView.js:

(WebInspector.ContentBrowserTabContentView):
(WebInspector.ContentBrowserTabContentView.prototype.shown):

  • UserInterface/Views/TabContentView.js:

(WebInspector.TabContentView):
(WebInspector.TabContentView.prototype.get navigationSidebarPanel):

  • UserInterface/Views/NetworkTabContentView.js:

(WebInspector.NetworkTabContentView.prototype.canShowRepresentedObject):

  • UserInterface/Views/SearchTabContentView.js:

(WebInspector.SearchTabContentView.prototype.canShowRepresentedObject):
Use public getter for navigationSidebarPanel.

  • UserInterface/Views/ResourceSidebarPanel.js:

(WebInspector.ResourceSidebarPanel):
(WebInspector.ResourceSidebarPanel.prototype.initialLayout): Added.
Load information about the current frame once the sidebar panel is displayed.

8:37 PM Changeset in webkit [217749] by Devin Rousso
  • 7 edits in trunk/Source/WebInspectorUI

Web Inspector: Use initialLayout for DetailsSidebarPanel classes
https://bugs.webkit.org/show_bug.cgi?id=172381

Reviewed by Timothy Hatcher.

  • UserInterface/Views/ApplicationCacheDetailsSidebarPanel.js:

(WebInspector.ApplicationCacheDetailsSidebarPanel):
(WebInspector.ApplicationCacheDetailsSidebarPanel.prototype.initialLayout): Added.
(WebInspector.ApplicationCacheDetailsSidebarPanel.prototype.layout):

  • UserInterface/Views/DOMNodeDetailsSidebarPanel.js:

(WebInspector.DOMNodeDetailsSidebarPanel):
(WebInspector.DOMNodeDetailsSidebarPanel.prototype.initialLayout): Added.
(WebInspector.DOMNodeDetailsSidebarPanel.prototype.layout):

  • UserInterface/Views/IndexedDatabaseDetailsSidebarPanel.js:

(WebInspector.IndexedDatabaseDetailsSidebarPanel):
(WebInspector.IndexedDatabaseDetailsSidebarPanel.prototype.initialLayout): Added.
(WebInspector.IndexedDatabaseDetailsSidebarPanel.prototype.layout):

  • UserInterface/Views/LayerTreeDetailsSidebarPanel.js:

(WebInspector.LayerTreeDetailsSidebarPanel):
(WebInspector.LayerTreeDetailsSidebarPanel.prototype.initialLayout): Added.
(WebInspector.LayerTreeDetailsSidebarPanel.prototype.layout):

  • UserInterface/Views/ProbeDetailsSidebarPanel.js:

(WebInspector.ProbeDetailsSidebarPanel):
(WebInspector.ProbeDetailsSidebarPanel.prototype.initialLayout): Added.

  • UserInterface/Views/ResourceDetailsSidebarPanel.js:

(WebInspector.ResourceDetailsSidebarPanel):
(WebInspector.ResourceDetailsSidebarPanel.prototype.set resource):
(WebInspector.ResourceDetailsSidebarPanel.prototype.initialLayout): Added.
(WebInspector.ResourceDetailsSidebarPanel.prototype.layout):
(WebInspector.ResourceDetailsSidebarPanel.prototype._applyResourceEventListeners):

7:27 PM Changeset in webkit [217748] by Wenson Hsieh
  • 2 edits in trunk/Source/WebCore

REGRESSION(r216212): RenderReplaced::paint() should not save and restore the context unless it has to
https://bugs.webkit.org/show_bug.cgi?id=172883
<rdar://problem/32548614>

Reviewed by Tim Horton.

After implementing dragged content fading, RenderReplace::paint is now always guarded by unnecessary calls to
GraphicsContext::save and GraphicsContext::restore, even when there is no dragged content being rendered. To
address this, we initialize our GraphicsContextStateSaver with saveAndRestore = false, indicating that we don't
want to immediately try and save the context.

If we are in a dragged content range, we will then call GraphicsContextStateSaver::save, which saves the
graphics context and also causes the GraphicsContextStateSaver to eventually try and restore() when it is
destroyed. Otherwise, in the common codepath where the renderer is not in a dragged content range, the
constructor and destructor of GraphicsContextStateSaver will be no-ops with respect to saving and restoring the
graphics context.

  • rendering/RenderReplaced.cpp:

(WebCore::RenderReplaced::paint):

6:35 PM Changeset in webkit [217747] by Yusuke Suzuki
  • 1 edit
    4 adds in trunk/JSTests

ES2015 modules - Export default function hoisting
https://bugs.webkit.org/show_bug.cgi?id=170174

Reviewed by Saam Barati.

This issue itself is solved by fixing function declaration hoisting rules.
This patch just adds the specific test to modules tests to ensure the bug
is fixed.

  • modules/export-default-function-hoisting.js: Added.
  • modules/export-default-function-hoisting/cappuccino.js: Added.

(import.drinkCocoa.from.string_appeared_here.export.default.drinkCappuccino):

  • modules/export-default-function-hoisting/cocoa.js: Added.

(export.default.drinkCocoa):

6:18 PM Changeset in webkit [217746] by Devin Rousso
  • 4 edits in trunk/Source/WebInspectorUI

Web Inspector: The CodeMirror instance in the ConsolePrompt does't need to be refreshed each time it is shown
https://bugs.webkit.org/show_bug.cgi?id=172608

Reviewed by Timothy Hatcher.

  • UserInterface/Views/ConsolePrompt.js:

(WebInspector.ConsolePrompt.prototype.shown): Deleted.

  • UserInterface/Views/DatabaseContentView.js:

(WebInspector.DatabaseContentView.prototype.shown): Deleted.

  • UserInterface/Views/QuickConsole.js:

(WebInspector.QuickConsole):

6:09 PM Changeset in webkit [217745] by keith_miller@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

Defer installing mach breakpoint handler until watchdog is actually called
https://bugs.webkit.org/show_bug.cgi?id=172885

Reviewed by Saam Barati.

Eagerly installing the mach breakpoint handler causes issues with Xcode GUI debugging.
This hides the issue, so it won't occur as often.

  • runtime/VMTraps.cpp:

(JSC::VMTraps::SignalSender::send):
(JSC::VMTraps::VMTraps): Deleted.

  • runtime/VMTraps.h:
4:58 PM Changeset in webkit [217744] by Devin Rousso
  • 5 edits
    1 add in trunk/Source/WebInspectorUI

Web Inspector: Attributes table in Node Details Sidebar should have editable keys and values
https://bugs.webkit.org/show_bug.cgi?id=167076
<rdar://problem/30033629>

Reviewed by Timothy Hatcher.

  • UserInterface/Views/DOMNodeDetailsSidebarPanel.js:

(WebInspector.DOMNodeDetailsSidebarPanel.prototype._refreshAttributes):
(WebInspector.DOMNodeDetailsSidebarPanel.prototype._attributeNodeValueChanged):
(WebInspector.DOMNodeDetailsSidebarPanel.prototype._createAttributesDataGrid): Deleted.
Rework creation of the attributes DataGrid so that it is only created once and cleared for
new content. Nodes in the DataGrid are now EditableDataGridNode instances so that attribute
names and values can be changed from within the sidebar. The DataGrid is also sorted based
on attribute name, to make finding attributes easier.

  • UserInterface/Views/DataGrid.css:

(.data-grid tr.editable .cell-content > input):
(body:not(.window-inactive, .window-docked-inactive) .data-grid:matches(:focus, .force-focus) tr.editable.selected .cell-content > input):

  • UserInterface/Views/DetailsSection.css:

(.details-section > .content .data-grid tr:not(.editable) td.value-column):
(.details-section > .content .data-grid tr:not(.editable) td.value-column > div):
(.details-section > .content .data-grid td.value-column): Deleted.
(.details-section > .content .data-grid td.value-column > div): Deleted.

  • UserInterface/Main.html:
  • UserInterface/Views/EditableDataGridNode.js: Added.

(WebInspector.EditableDataGridNode):
(WebInspector.EditableDataGridNode.prototype.get element):
(WebInspector.EditableDataGridNode.prototype.createCellContent):
(WebInspector.EditableDataGridNode.prototype._handleKeyPress):
(WebInspector.EditableDataGridNode.prototype._handleBlur):
(WebInspector.EditableDataGridNode.prototype._notifyInputElementValueChanged):
Special type of DataGridNode that wraps the content of each cell in an <input>.

4:48 PM Changeset in webkit [217743] by Chris Dumez
  • 8 edits in trunk

ResourceLoadStatistics are not using unique paths during test runs
https://bugs.webkit.org/show_bug.cgi?id=172861
<rdar://problem/32442251>

Reviewed by Darin Adler.

Source/WebKit2:

Add WKContextConfigurationSetResourceLoadStatisticsDirectory() C API to allow
WKTR to use a custom directory.

  • UIProcess/API/APIProcessPoolConfiguration.cpp:

(API::ProcessPoolConfiguration::createWithWebsiteDataStoreConfiguration):
(API::ProcessPoolConfiguration::ProcessPoolConfiguration):
(API::ProcessPoolConfiguration::copy):

  • UIProcess/API/APIProcessPoolConfiguration.h:
  • UIProcess/API/C/WKContextConfigurationRef.cpp:

(WKContextConfigurationCopyResourceLoadStatisticsDirectory):
(WKContextConfigurationSetResourceLoadStatisticsDirectory):

  • UIProcess/API/C/WKContextConfigurationRef.h:
  • UIProcess/WebProcessPool.cpp:

(WebKit::legacyWebsiteDataStoreConfiguration):

Tools:

Make sure WebKitTestRunner sets a temporary path for saving resource
load statistics.

  • WebKitTestRunner/TestController.cpp:

(WTR::TestController::generateContextConfiguration):

4:40 PM Changeset in webkit [217742] by mmaxfield@apple.com
  • 11 edits
    4 adds in trunk

REGRESSION(r213464): [iOS] Fonts get too bold when the "Bold Text" accessibility setting is enabled
https://bugs.webkit.org/show_bug.cgi?id=172737
Source/WebCore:

Reviewed by Simon Fraser.

Revert the part of r213464 which changed how we calculate the weights of system fonts.

Add an InternalSetting to trigger our mock of the accessbility "Bold Text" setting. We can't
use the real setting, so instead we just ask for bold fonts instead of regular fonts when this
setting is true.

Tests: fast/text/accessibility-bold-system-font.html

fast/text/accessibility-bold-system-font-2.html

  • platform/graphics/FontCache.h:

(WebCore::FontCache::shouldMockBoldSystemFontForAccessibility):
(WebCore::FontCache::setShouldMockBoldSystemFontForAccessibility):

  • platform/graphics/ios/FontCacheIOS.mm:

(WebCore::platformFontWithFamilySpecialCase):

  • rendering/RenderTheme.h:

(WebCore::RenderTheme::shouldMockBoldSystemFontForAccessibility):
(WebCore::RenderTheme::setShouldMockBoldSystemFontForAccessibility):

  • rendering/RenderThemeIOS.h:
  • rendering/RenderThemeIOS.mm:

(WebCore::cssWeightOfSystemFont):
(WebCore::RenderThemeIOS::updateCachedSystemFontDescription):

  • testing/InternalSettings.cpp:

(WebCore::InternalSettings::Backup::Backup):
(WebCore::InternalSettings::Backup::restoreTo):
(WebCore::InternalSettings::setShouldMockBoldSystemFontForAccessibility):

  • testing/InternalSettings.h:
  • testing/InternalSettings.idl:

LayoutTests:

<rdar://problem/31608236>

Reviewed by Simon Fraser.

  • fast/text/accessibility-bold-system-font-2-expected.html: Added.
  • fast/text/accessibility-bold-system-font-2.html: Added.
  • fast/text/accessibility-bold-system-font-expected.txt: Added.
  • fast/text/accessibility-bold-system-font.html: Added.
  • platform/mac/TestExpectations:
4:11 PM Changeset in webkit [217741] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Mark webrtc/captureCanvas-webrtc.html as flaky on iOS.
https://bugs.webkit.org/show_bug.cgi?id=170870

Unreviewed test gardening.

  • platform/ios-wk2/TestExpectations:
4:10 PM Changeset in webkit [217740] by Jonathan Bedard
  • 3 edits in trunk/LayoutTests

Move compositing/overflow/dynamic-composited-scrolling-status.html failure expectation.
https://bugs.webkit.org/show_bug.cgi?id=152439

Unreviewed test gardening

compositing/overflow/dynamic-composited-scrolling-status.html fails on iOS-simulator WK2,
but passes on device.

  • platform/ios-simulator-wk2/TestExpectations:
  • platform/ios-wk2/TestExpectations:
4:07 PM Changeset in webkit [217739] by Ryan Haddad
  • 3 edits in trunk/LayoutTests

Mark webrtc/audio-replace-track.html as flaky on iOS and remove Timeout expectation for macOS.
https://bugs.webkit.org/show_bug.cgi?id=171895

Unreviewed test gardening.

  • platform/ios-wk2/TestExpectations:
  • platform/mac-wk2/TestExpectations:
4:07 PM Changeset in webkit [217738] by beidson@apple.com
  • 7 edits in trunk

Prevent scheme handlers from handling all built-in URL schemes.
<rdar://problem/32404790> and https://bugs.webkit.org/show_bug.cgi?id=172869

Reviewed by Andy Estes.

Source/WebCore:

Covered by API test.

This patch refactors SchemeRegistry to keep a base, constant set of each of the special
URL schemes that WebKit knows about by default.

It then exposes that list through a new method to support WK2 API.

  • platform/SchemeRegistry.cpp:

(WebCore::allBuiltinSchemes):
(WebCore::builtinLocalURLSchemes):
(WebCore::localURLSchemes):
(WebCore::builtinSecureSchemes):
(WebCore::secureSchemes):
(WebCore::builtinSchemesWithUniqueOrigins):
(WebCore::schemesWithUniqueOrigins):
(WebCore::builtinEmptyDocumentSchemes):
(WebCore::emptyDocumentSchemes):
(WebCore::builtinCanDisplayOnlyIfCanRequestSchemes):
(WebCore::canDisplayOnlyIfCanRequestSchemes):
(WebCore::SchemeRegistry::removeURLSchemeRegisteredAsLocal):
(WebCore::builtinCORSEnabledSchemes):
(WebCore::CORSEnabledSchemes):
(WebCore::SchemeRegistry::isBuiltinScheme):

  • platform/SchemeRegistry.h:

Source/WebKit2:

  • UIProcess/API/Cocoa/WKWebView.mm:

(+[WKWebView handlesURLScheme:]): Check against WebCore's new master list of URL schemes.

Tools:

  • TestWebKitAPI/Tests/WebKit2Cocoa/WKURLSchemeHandler-1.mm:
4:06 PM Changeset in webkit [217737] by Simon Fraser
  • 31 edits
    2 copies
    2 adds in trunk/Source

All scroll peformance logging should happen in the UI process
https://bugs.webkit.org/show_bug.cgi?id=172874
rdar://problem/24474830

Reviewed by Tim Horton.

Printing to stdout from the WebProcess doesn't show up anywhere in non-development builds,
so we have to log from the UI process.

Achieve this by adding a PerformanceLoggingClient to Page which forwards messages to
WebPageProxy, which prints them with the same format they had before. Some plumbing through
PlatformCALayerClient and GraphicsLayerClient is required to get messages from TileControllers
through to something that can get to Page.

Source/WebCore:

  • WebCore.xcodeproj/project.pbxproj:
  • page/Page.cpp:

(WebCore::Page::Page):
(WebCore::Page::~Page):

  • page/Page.h:

(WebCore::Page::performanceLoggingClient):

  • page/PageConfiguration.cpp:
  • page/PageConfiguration.h:
  • page/PerformanceLoggingClient.cpp: Added.

(WebCore::PerformanceLoggingClient::synchronousScrollingReasonsAsString):

  • page/PerformanceLoggingClient.h: Added.

(WebCore::PerformanceLoggingClient::~PerformanceLoggingClient):

  • page/scrolling/AsyncScrollingCoordinator.cpp:

(WebCore::AsyncScrollingCoordinator::reportExposedUnfilledArea):
(WebCore::AsyncScrollingCoordinator::reportSynchronousScrollingReasonsChanged):

  • page/scrolling/AsyncScrollingCoordinator.h:
  • page/scrolling/ScrollingTree.h:

(WebCore::ScrollingTree::reportSynchronousScrollingReasonsChanged):
(WebCore::ScrollingTree::reportExposedUnfilledArea):

  • page/scrolling/ThreadedScrollingTree.cpp:

(WebCore::ThreadedScrollingTree::reportSynchronousScrollingReasonsChanged):
(WebCore::ThreadedScrollingTree::reportExposedUnfilledArea):

  • page/scrolling/ThreadedScrollingTree.h:
  • page/scrolling/mac/ScrollingTreeFrameScrollingNodeMac.h:
  • page/scrolling/mac/ScrollingTreeFrameScrollingNodeMac.mm:

(WebCore::ScrollingTreeFrameScrollingNodeMac::commitStateBeforeChildren):
(WebCore::ScrollingTreeFrameScrollingNodeMac::setScrollPosition):
(WebCore::ScrollingTreeFrameScrollingNodeMac::exposedUnfilledArea):
(WebCore::ScrollingTreeFrameScrollingNodeMac::logExposedUnfilledArea): Deleted.
(WebCore::logThreadedScrollingMode): Deleted.

  • platform/graphics/GraphicsLayerClient.h:

(WebCore::GraphicsLayerClient::logFilledVisibleFreshTile):

  • platform/graphics/ca/GraphicsLayerCA.cpp:

(WebCore::GraphicsLayerCA::platformCALayerLogFilledVisibleFreshTile):

  • platform/graphics/ca/GraphicsLayerCA.h:
  • platform/graphics/ca/PlatformCALayerClient.h:

(WebCore::PlatformCALayerClient::platformCALayerLogFilledVisibleFreshTile):

  • platform/graphics/ca/TileController.cpp:

(WebCore::TileController::logFilledVisibleFreshTile):

  • platform/graphics/ca/TileController.h:
  • platform/graphics/ca/TileGrid.cpp:

(WebCore::TileGrid::platformCALayerPaintContents):

  • platform/graphics/ca/TileGrid.h:
  • rendering/RenderLayerBacking.cpp:

(WebCore::RenderLayerBacking::logFilledVisibleFreshTile):

  • rendering/RenderLayerBacking.h:

Source/WebKit2:

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::logScrollingEvent):

  • UIProcess/WebPageProxy.h:
  • UIProcess/WebPageProxy.messages.in:
  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/WebCoreSupport/WebPerformanceLoggingClient.cpp: Added.

(WebKit::WebPerformanceLoggingClient::WebPerformanceLoggingClient):
(WebKit::WebPerformanceLoggingClient::logScrollingEvent):

  • WebProcess/WebCoreSupport/WebPerformanceLoggingClient.h: Added.
  • WebProcess/WebPage/WebPage.cpp:

(WebKit::m_cpuLimit):

3:20 PM Changeset in webkit [217736] by commit-queue@webkit.org
  • 8 edits in trunk

[JSCOnly] Build static jsc.exe on Windows
https://bugs.webkit.org/show_bug.cgi?id=172833

Patch by Stephan Szabo <stephan.szabo@am.sony.com> on 2017-06-02
Reviewed by Konstantin Tokarev.

.:

  • Source/cmake/OptionsJSCOnly.cmake:

Source/WTF:

  • wtf/PlatformJSCOnly.cmake:

Tools:

  • CMakeLists.txt:
  • Scripts/build-jsc:
3:12 PM Changeset in webkit [217735] by Alan Bujtas
  • 2 edits in trunk/Tools

Disable ViewportSizeForViewportUnits tests on iOS.
https://bugs.webkit.org/show_bug.cgi?id=172872

Reviewed by Tim Horton.

  • TestWebKitAPI/Tests/WebKit2/ViewportSizeForViewportUnits.mm:
2:16 PM Changeset in webkit [217734] by eric.carlson@apple.com
  • 9 edits
    2 adds in trunk

[MediaStream iOS] Cleanup video muting/unmuting when tab visibility changes
https://bugs.webkit.org/show_bug.cgi?id=172858

Reviewed by Youenn Fablet.

Source/WebCore:

Test: platform/ios/mediastream/video-muted-in-background-tab.html

  • dom/Document.cpp:

(WebCore::Document::visibilityStateChanged): Call notifyMediaCaptureOfVisibilityChanged.
(WebCore::Document::notifyMediaCaptureOfVisibilityChanged): Renamed from notifyVisibilityChangedToMediaCapture.
Set m_videoCaptureMutedForVisibilityChange when capture is muted because the document is hidden,
and clear it when visibility changes when capture is disabled. Don't unmute when the document
becomes visible unless this m_videoCaptureMutedForVisibilityChange is still true.
(WebCore::Document::notifyVisibilityChangedToMediaCapture): Deleted.

  • dom/Document.h:
  • platform/mediastream/RealtimeMediaSource.h:
  • platform/mediastream/RealtimeMediaSourceCenter.cpp:

(WebCore::RealtimeMediaSourceCenter::setVideoCaptureMutedForPageVisibility): Renamed from
setVisibility.
(WebCore::RealtimeMediaSourceCenter::setVisibility): Deleted.

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

(WebCore::AVVideoCaptureSourceFactory::setVideoCaptureMutedForPageVisibility): Ditto.
(WebCore::AVVideoCaptureSourceFactory::setVisibility): Deleted.

  • platform/mock/MockRealtimeVideoSource.cpp:

(WebCore::MockRealtimeVideoSourceFactory::setVideoCaptureMutedForPageVisibility): Ditto.
(WebCore::MockRealtimeVideoSourceFactory::setVisibility): Deleted.

LayoutTests:

  • platform/ios/mediastream/video-muted-in-background-tab-expected.txt: Added.
  • platform/ios/mediastream/video-muted-in-background-tab.html: Added.
1:57 PM Changeset in webkit [217733] by Devin Rousso
  • 4 edits in trunk/Source/WebInspectorUI

Web Inspector: Existing query in Search tab doesn't perform search on reload
https://bugs.webkit.org/show_bug.cgi?id=172663

Reviewed by Timothy Hatcher.

  • Localizations/en.lproj/localizedStrings.js:
  • UserInterface/Views/SearchSidebarPanel.css:

(.sidebar > .panel.navigation.search.changed > .banner):
(.sidebar > .panel.navigation.search.changed > .banner > a):
(.sidebar > .panel.navigation.search.changed > :matches(.content, .empty-content-placeholder)):

  • UserInterface/Views/SearchSidebarPanel.js:

(WebInspector.SearchSidebarPanel.prototype.performSearch.resourcesCallback):
(WebInspector.SearchSidebarPanel.prototype._mainResourceDidChange):
(WebInspector.SearchSidebarPanel.prototype._contentChanged):
Listen for any "resource added" event, and show the content changed banner when one occurs.

1:48 PM Changeset in webkit [217732] by Matt Lewis
  • 3 edits in trunk/LayoutTests

Moved test expectation for http/tests/preload/viewport/meta-viewport-link-headers.php to correct file.
https://bugs.webkit.org/show_bug.cgi?id=172518

Unreviewed test gardening

  • platform/ios-simulator/TestExpectations:
  • platform/ios/TestExpectations:
12:57 PM Changeset in webkit [217731] by Chris Dumez
  • 3 edits in trunk/LayoutTests

REGRESSION: LayoutTest fast/scrolling/arrow-key-scroll-in-rtl-document.html is a flaky failure
https://bugs.webkit.org/show_bug.cgi?id=161549
<rdar://problem/28213429>

Reviewed by Simon Fraser.

  • fast/scrolling/arrow-key-scroll-in-rtl-document.html:

Call eventSender.callAfterScrollingCompletes() in the next event loop iteration to
address the flakiness.

  • platform/mac-wk2/TestExpectations:

Unskip test.

12:40 PM Changeset in webkit [217730] by fred.wang@free.fr
  • 6 edits
    5 adds in trunk

[Mac] Include frames in the scrolling tree when ScrollingTreeIncludesFrames=true
https://bugs.webkit.org/show_bug.cgi?id=172825

Patch by Frederic Wang <fwang@igalia.com> on 2017-06-02
Reviewed by Simon Fraser.

Source/WebCore:

This commit generalizes RenderLayerCompositor/ScrollingCoordinator so that it handles the
case of non-main frames. When ScrollingTreeIncludesFrames=true, all the frames will appear
in the scrolling tree on Mac WK2.

Test: fast/scrolling/scrolling-tree-includes-frame.html

  • page/scrolling/ScrollingCoordinator.cpp:

(WebCore::ScrollingCoordinator::coordinatesScrollingForFrameView): Do not to restrict to the
main frame.

  • rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::updateBacking): Do not to restrict to the main frame.
(WebCore::RenderLayerCompositor::useCoordinatedScrollingForLayer): Make this function a class
member so it can call hasCoordinatedScrolling and no longer needs a RenderView parameter.
(WebCore::RenderLayerCompositor::updateScrollCoordinatedStatus): Do not to restrict to the
main frame.

  • rendering/RenderLayerCompositor.h: Introduce useCoordinatedScrollingForLayer.

LayoutTests:

Add a new test to check whether a non-main frame appears in the scrolling tree when
ScrollingTreeIncludesFrames=true. For now this is only happens for Mac WK2. Also update
the iOS WK2 reference for scrolling-iframe-expected.html

  • fast/scrolling/scrolling-tree-includes-frame-expected.txt: Added.
  • fast/scrolling/scrolling-tree-includes-frame.html: Added.
  • platform/ios-simulator/fast/scrolling/scrolling-tree-includes-frame-expected.txt: Added.
  • platform/ios-wk2/compositing/iframes/scrolling-iframe-expected.txt: Update reference.
  • platform/mac-wk1/fast/scrolling/scrolling-tree-includes-frame-expected.txt: Added.
12:15 PM Changeset in webkit [217729] by Chris Dumez
  • 2 edits in trunk/Source/WebKit2

REGRESSION (r206386): Xactimate Website Crashes @ com.apple.WebKit: WebKit::NPRuntimeObjectMap::convertJSValueToNPVariant + 255
https://bugs.webkit.org/show_bug.cgi?id=172846
<rdar://problem/31093005>

Reviewed by Mark Lam.

Follow-up to r217695 to deal with exceptions potentially thrown by
NPRuntimeObjectMap::convertJSValueToNPVariant() as well.

  • WebProcess/Plugins/Netscape/NPJSObject.cpp:

(WebKit::NPJSObject::invoke):

12:04 PM Changeset in webkit [217728] by commit-queue@webkit.org
  • 6 edits in trunk/Source/WebKit2

Add abilility to specify presentingApplicationPID for a WebProcessPool.
https://bugs.webkit.org/show_bug.cgi?id=172839
rdar://problem/32247170

Patch by Jeremy Jones <jeremyj@apple.com> on 2017-06-02
Reviewed by Tim Horton.

Expose presentingApplicationPID through WebProcessPoolConfiguration.
This allows rehosted WebViews to specify the hosting application.

  • UIProcess/API/APIProcessPoolConfiguration.cpp:

(API::ProcessPoolConfiguration::copy):

  • UIProcess/API/APIProcessPoolConfiguration.h:
  • UIProcess/API/Cocoa/_WKProcessPoolConfiguration.h:
  • UIProcess/API/Cocoa/_WKProcessPoolConfiguration.mm:

(-[_WKProcessPoolConfiguration setPresentingApplicationPID:]):
(-[_WKProcessPoolConfiguration presentingApplicationPID]):

  • UIProcess/WebProcessPool.cpp:

(WebKit::WebProcessPool::ensureNetworkProcess):
(WebKit::WebProcessPool::createNewWebProcess):

12:04 PM Changeset in webkit [217727] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

[WebCore] DragImageRef should correspond to USE(CAIRO)
https://bugs.webkit.org/show_bug.cgi?id=172863

Patch by Don Olmstead <don.olmstead@am.sony.com> on 2017-06-02
Reviewed by Carlos Alberto Lopez Perez.

No new tests. No change in behavior.

  • platform/DragImage.h:
11:42 AM Changeset in webkit [217726] by fred.wang@free.fr
  • 6 edits
    7 adds in trunk

[Mac] Use compositing for frames when ScrollingTreeIncludesFrames=true
https://bugs.webkit.org/show_bug.cgi?id=172851

Patch by Frederic Wang <fwang@igalia.com> on 2017-06-02
Reviewed by Simon Fraser.

Source/WebCore:

When fast frame scrolling is supported on mac (bug 171667), scrollable frames will require
special composited layer. This commit enables that on mac but only when
ScrollingTreeIncludesFrames=true, so that the default behavior is unchanged.

Test: compositing/iframes/compositing-for-scrollable-iframe.html

  • rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::requiresCompositingForScrollableFrame): Do not enable frame
compositing when ScrollingTreeIncludesFrames=false.
Rename ScrollableInnerFrameTrigger to ScrollableNonMainFrameTrigger.

  • page/ChromeClient.h: Rename ScrollableInnerFrameTrigger.

Source/WebKit2:

When fast frame scrolling is supported on mac (bug 171667), scrollable frames will require
special composited layer. This commit enables that on mac but only when
ScrollingTreeIncludesFrames=true, so that the default behavior is unchanged.

  • WebProcess/WebCoreSupport/WebChromeClient.h: Make scrollable inner frame trigger

compositing on Mac by default. This is ignored on mac when ScrollingTreeIncludesFrames=false.
Rename ScrollableInnerFrameTrigger to ScrollableNonMainFrameTrigger.

LayoutTests:

This commit adds a test to check whether scrollable iframes require a special composited
layer when ScrollingTreeIncludesFrames=true. This is currently only done on Mac WK2.

  • compositing/iframes/compositing-for-scrollable-iframe.html: Added. This is a basic test

printing the layer tree of a page with an iframe.

  • platform/ios-simulator/compositing/iframes/compositing-for-scrollable-iframe-expected.txt: Added.
  • platform/mac/compositing/iframes/compositing-for-scrollable-iframe-expected.txt: Added.
  • platform/mac-wk1/compositing/iframes/compositing-for-scrollable-iframe-expected.txt: Added.
11:38 AM Changeset in webkit [217725] by Alan Bujtas
  • 13 edits
    1 add in trunk

Allow clients to override viewport size for CSS viewport units.
https://bugs.webkit.org/show_bug.cgi?id=172838
<rdar://problem/32523441>

Reviewed by Tim Horton.

Source/WebKit2:

This enables auto-sizing clients to ensure that styles relative to the viewport are
resolved to a set value (presumably the size of the container) as opposed to the preferred
document size (which is set as the FrameView size).

  • Shared/WebPageCreationParameters.cpp:

(WebKit::WebPageCreationParameters::encode):
(WebKit::WebPageCreationParameters::decode):

  • Shared/WebPageCreationParameters.h:
  • UIProcess/API/Cocoa/WKWebView.mm:

(-[WKWebView _viewportSizeForCSSViewportUnits]):
(-[WKWebView _setViewportSizeForCSSViewportUnits:]):

  • UIProcess/API/Cocoa/WKWebViewPrivate.h:
  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::creationParameters):
(WebKit::WebPageProxy::setViewportSizeForCSSViewportUnits):

  • UIProcess/WebPageProxy.h:

(WebKit::WebPageProxy::viewportSizeForCSSViewportUnits):

  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::WebFrameLoaderClient::transitionToCommittedForNewPage):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::m_cpuLimit):
(WebKit::WebPage::setViewportSizeForCSSViewportUnits):

  • WebProcess/WebPage/WebPage.h:

(WebKit::WebPage::viewportSizeForCSSViewportUnits):

  • WebProcess/WebPage/WebPage.messages.in:

Tools:

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

(TEST):

11:20 AM Changeset in webkit [217724] by Simon Fraser
  • 20 edits in trunk

Get <chrono> out of StdLibExtras.h
https://bugs.webkit.org/show_bug.cgi?id=172744

Reviewed by Zalan Bujtas.

Source/WebCore:

Only bring in <chrono> and namespace std::literals::chrono_literals when necessary.

  • fileapi/FileReader.cpp:
  • inspector/InspectorOverlay.cpp:
  • loader/CrossOriginPreflightResultCache.cpp:
  • loader/cache/CachedResource.cpp:

(WebCore::CachedResource::freshnessLifetime):

  • page/ResourceUsageThread.cpp:

(WebCore::ResourceUsageThread::threadBody):

  • page/cocoa/MemoryReleaseCocoa.mm:

(WebCore::jettisonExpensiveObjectsOnTopLevelNavigation):

  • platform/network/CacheValidation.cpp:

(WebCore::computeFreshnessLifetimeForHTTPFamily):

  • platform/network/CacheValidation.h:

Source/WebKit2:

Only bring in <chrono> and namespace std::literals::chrono_literals when necessary.

  • NetworkProcess/cache/NetworkCache.cpp:
  • NetworkProcess/cache/NetworkCacheFileSystem.h:
  • NetworkProcess/cache/NetworkCacheSpeculativeLoadManager.cpp:
  • NetworkProcess/cache/NetworkCacheStorage.cpp:

(WebKit::NetworkCache::computeRecordWorth):

  • Shared/mac/RemoteLayerBackingStoreCollection.mm:
  • WebProcess/Cookies/WebCookieManager.h:

Source/WTF:

We hates std::chrono. Don't include it everywhere to discourage use.

  • wtf/StdLibExtras.h:

Tools:

  • TestWebKitAPI/Tests/WTF/WorkQueue.cpp:
11:14 AM Changeset in webkit [217723] by aestes@apple.com
  • 12 edits
    2 adds in trunk

[Cocoa] Add a "supportedCountries" property to ApplePayPaymentRequest
https://bugs.webkit.org/show_bug.cgi?id=172832
<rdar://problem/30735544>

Reviewed by Alex Christensen.

Source/WebCore:

Test: http/tests/ssl/applepay/ApplePaySessionV3.html

  • Modules/applepay/ApplePayPaymentRequest.h:
  • Modules/applepay/ApplePayPaymentRequest.idl:
  • Modules/applepay/ApplePaySession.cpp:

(WebCore::convertAndValidate):

  • Modules/applepay/PaymentRequest.h:

(WebCore::PaymentRequest::supportedCountries):
(WebCore::PaymentRequest::setSupportedCountries):

  • Modules/applepay/PaymentRequestValidator.cpp:

(WebCore::PaymentRequestValidator::validate):

  • platform/spi/cocoa/PassKitSPI.h:

Source/WebKit2:

  • Shared/Cocoa/WebCoreArgumentCodersCocoa.mm:

(IPC::ArgumentCoder<PaymentRequest>::encode):
(IPC::ArgumentCoder<PaymentRequest>::decode):

  • UIProcess/ApplePay/cocoa/WebPaymentCoordinatorProxyCocoa.mm:

(WebKit::toNSSet):
(WebKit::toPKPaymentRequest):

LayoutTests:

  • http/tests/ssl/applepay/ApplePaySessionV3-expected.txt: Added.
  • http/tests/ssl/applepay/ApplePaySessionV3.html: Added.
  • platform/mac-wk2/TestExpectations:
10:58 AM Changeset in webkit [217722] by fpizlo@apple.com
  • 7 edits in trunk

Atomics.load and Atomics.store need to be fully fenced
https://bugs.webkit.org/show_bug.cgi?id=172844

Reviewed by Keith Miller.

Source/JavaScriptCore:

Implement fully fenced loads and stores in FTL using AtomicXchgAdd(0, ptr) for the load and
AtomicXchg(value, ptr) for the store.

DFG needed no changes because it implements all atomics using a CAS loop.

AtomicsObject.cpp now uses new Atomic<> API for fully fences loads and stores.

Prior to this change, we used half fences (acquire/release) for atomic loads and stores. This
is not correct according to my current understanding of the SAB memory model, which requires
that atomic operations are SC with respect to everything not just other atomics.

  • ftl/FTLLowerDFGToB3.cpp:

(JSC::FTL::DFG::LowerDFGToB3::compileAtomicsReadModifyWrite):

  • ftl/FTLOutput.cpp:

(JSC::FTL::Output::atomicWeakCAS):

  • ftl/FTLOutput.h:
  • runtime/AtomicsObject.cpp:

Source/WTF:

Add loadFullyFenced and storeFullyFenced to Atomic<>.

  • wtf/Atomics.h:

(WTF::Atomic::loadFullyFenced):
(WTF::Atomic::storeRelaxed):
(WTF::Atomic::storeFullyFenced):
(WTF::atomicLoadFullyFenced):
(WTF::atomicStoreFullyFenced):

Websites/webkit.org:

Update documentation to say that the canonical way to do fully fenced loads and stores is
AtomicXchgAdd(0, ptr) and AtomicXchg(value, ptr), respectively.

  • docs/b3/intermediate-representation.html:
10:54 AM Changeset in webkit [217721] by Devin Rousso
  • 11 edits
    3 adds in trunk

Web Inspector: Should see active Web Sockets when opening Web Inspector
https://bugs.webkit.org/show_bug.cgi?id=172312

Reviewed by Joseph Pecoraro.

Source/WebCore:

Test: http/tests/websocket/tests/hybi/inspector/before-load.html

  • Modules/websockets/WebSocket.h:
  • Modules/websockets/WebSocket.cpp:

(WebCore::WebSocket::WebSocket):
(WebCore::WebSocket::~WebSocket):
(WebCore::WebSocket::allActiveWebSockets):
(WebCore::WebSocket::allActiveWebSocketsMutex):
(WebCore::WebSocket::channel):
(WebCore::WebSocket::eventTargetInterface):
Add a static set of all WebSocket objects and provide getters to access web socket metadata.

  • Modules/websockets/ThreadableWebSocketChannel.h:

(WebCore::ThreadableWebSocketChannel::isWebSocketChannel):

  • Modules/websockets/WebSocketChannel.h:
  • Modules/websockets/WebSocketChannel.cpp:

(WebCore::WebSocketChannel::isWebSocketChannel):
(WebCore::WebSocketChannel::identifier):
(WebCore::WebSocketChannel::clientHandshakeRequest):
(WebCore::WebSocketChannel::serverHandshakeResponse):
(WebCore::WebSocketChannel::handshakeMode):
(WebCore::WebSocketChannel::isType):
Provide getters to access web socket metadata.

  • inspector/InspectorNetworkAgent.cpp:

(WebCore::InspectorNetworkAgent::enable):
Loops over the static set of all WebSocket objects and sends events for each to the frontend.

Source/WebInspectorUI:

  • UserInterface/Controllers/FrameResourceManager.js:

(WebInspector.FrameResourceManager.prototype.webSocketHandshakeResponseReceived):
(WebInspector.FrameResourceManager.prototype.initialize): Removed.
Rework order of agent enable calls to ensure that the main frame is initialized before any
websocket events are dispatched.

  • UserInterface/Models/WebSocketResource.js:

(WebInspector.WebSocketResource.ReadyState):
Change symbol strings in enum for better printing in tests.

LayoutTests:

Test that WebSockets created before the inspector loads are also tracked.

  • http/tests/websocket/tests/hybi/inspector/before-load-expected.txt: Added.
  • http/tests/websocket/tests/hybi/inspector/before-load.html: Added.
  • http/tests/websocket/tests/hybi/inspector/before-load_wsh.py: Added.

(web_socket_do_extra_handshake):
(web_socket_transfer_data):

10:01 AM Changeset in webkit [217720] by mitz@apple.com
  • 2 edits in trunk/Source/WebKit2

[Xcode] Header postprocessing should be based on the target OS version rather than the builder OS version
https://bugs.webkit.org/show_bug.cgi?id=172860

Reviewed by Anders Carlsson.

  • Configurations/WebKit.xcconfig: Use TARGET_MAC_OS_X_VERSION_MAJOR
9:47 AM Changeset in webkit [217719] by Chris Dumez
  • 2 edits in trunk/Source/WebCore

Unreviewed, fix linking errors on iOS after r217712.

  • platform/OrientationNotifer.h:

(WebCore::OrientationNotifier::Observer::setNotifier):

9:26 AM Changeset in webkit [217718] by bshafiei@apple.com
  • 2 edits in tags/Safari-604.1.23.0.5/Source/WebCore

Build fix. rdar://problem/32530825

9:25 AM Changeset in webkit [217717] by Ryan Haddad
  • 2 edits in trunk/Source/JavaScriptCore

Unreviewed, attempt to fix the iOS build after r217711.

  • assembler/MacroAssemblerARM64.h:

(JSC::MacroAssemblerARM64::xor32):
(JSC::MacroAssemblerARM64::xor64):

9:20 AM Changeset in webkit [217716] by bshafiei@apple.com
  • 7 edits in tags/Safari-604.1.23.0.5/Source

Versioning.

9:19 AM Changeset in webkit [217715] by Ryan Haddad
  • 2 edits in branches/safari-603-branch/LayoutTests

Merge r211465.

9:17 AM Changeset in webkit [217714] by bshafiei@apple.com
  • 1 copy in tags/Safari-604.1.23.0.5

New tag.

9:17 AM Changeset in webkit [217713] by Matt Lewis
  • 2 edits in trunk/LayoutTests

Marked fast/images/slower-decoding-than-animation-image.html as flaky.
https://bugs.webkit.org/show_bug.cgi?id=172666

Unreviewed test gardening.

  • platform/mac-wk2/TestExpectations:
9:08 AM Changeset in webkit [217712] by Chris Dumez
  • 2 edits in trunk/Source/WebCore

Crash under OrientationNotifier::Observer::setNotifier()
https://bugs.webkit.org/show_bug.cgi?id=172847
<rdar://problem/32519127>

Reviewed by Youenn Fablet.

Update OrientationNotifier::Observer::setNotifier() to unregister itself from
its current notifier if it already has one. This is needed because
Internals::setCameraMediaStreamTrackOrientation() may be used to override the
notifier of an observer. If we override the notifier without unregistering
the observer from its previous notifier, then the previous notifier will not
get notified when the observer gets destroyed.

No new tests, already covered by webrtc/video-rotation.html that is currently
failing on some bots.

  • platform/OrientationNotifer.h:

(WebCore::OrientationNotifier::Observer::setNotifier):

8:59 AM Changeset in webkit [217711] by fpizlo@apple.com
  • 27 edits
    1 add in trunk/Source/JavaScriptCore

GC should use scrambled free-lists
https://bugs.webkit.org/show_bug.cgi?id=172793

Reviewed by Mark Lam.

Previously, our bump'n'pop allocator would use a conventional linked-list for the free-list.
The linked-list would be threaded through free memory, as is the usual convention.

This scrambles the next pointers of that free-list. It also scrambles the head pointer, because
this leads to a more natural fast-path structure and saves one register on ARM64.

The secret with which pointers are scrambled is per-allocator. Allocators choose a new secret
every time they do a sweep-to-pop.

This doesn't change the behavior of the bump part of bump'n'pop, but it does refactor the code
quite a bit. Previously, there were four copies of the allocator fast path: two in
MarkedAllocatorInlines.h, one in MarkedAllocator.cpp, and one in AssemblyHelpers.h. The JIT one
was obviously different-looking, but the other three were almost identical. This moves all of
that logic into FreeList. There are now just two copies of the allocator: FreeListInlines.h and
AssemblyHelpers.h.

This appears to be just as fast as our previously allocator.

(JSC::FreeList::FreeList):
(JSC::FreeList::~FreeList):
(JSC::FreeList::clear):
(JSC::FreeList::initializeList):
(JSC::FreeList::initializeBump):
(JSC::FreeList::contains):
(JSC::FreeList::dump):

  • heap/FreeList.h:

(JSC::FreeList::allocationWillFail):
(JSC::FreeList::originalSize):
(JSC::FreeList::addressOfList):
(JSC::FreeList::offsetOfBlock):
(JSC::FreeList::offsetOfList):
(JSC::FreeList::offsetOfIndex):
(JSC::FreeList::offsetOfPayloadEnd):
(JSC::FreeList::offsetOfRemaining):
(JSC::FreeList::offsetOfOriginalSize):
(JSC::FreeList::FreeList): Deleted.
(JSC::FreeList::list): Deleted.
(JSC::FreeList::bump): Deleted.
(JSC::FreeList::operator==): Deleted.
(JSC::FreeList::operator!=): Deleted.
(JSC::FreeList::operator bool): Deleted.

  • heap/FreeListInlines.h: Added.

(JSC::FreeList::addFreeCell):
(JSC::FreeList::allocate):
(JSC::FreeList::forEach):
(JSC::FreeList::toOffset):
(JSC::FreeList::fromOffset):

  • heap/IncrementalSweeper.cpp:

(JSC::IncrementalSweeper::sweepNextBlock):

  • heap/MarkedAllocator.cpp:

(JSC::MarkedAllocator::MarkedAllocator):
(JSC::MarkedAllocator::didConsumeFreeList):
(JSC::MarkedAllocator::tryAllocateWithoutCollecting):
(JSC::MarkedAllocator::tryAllocateIn):
(JSC::MarkedAllocator::allocateSlowCaseImpl):
(JSC::MarkedAllocator::stopAllocating):
(JSC::MarkedAllocator::prepareForAllocation):
(JSC::MarkedAllocator::resumeAllocating):
(JSC::MarkedAllocator::sweep):
(JSC::MarkedAllocator::setFreeList): Deleted.

  • heap/MarkedAllocator.h:

(JSC::MarkedAllocator::freeList):
(JSC::MarkedAllocator::isFreeListedCell): Deleted.

  • heap/MarkedAllocatorInlines.h:

(JSC::MarkedAllocator::isFreeListedCell):
(JSC::MarkedAllocator::tryAllocate):
(JSC::MarkedAllocator::allocate):

  • heap/MarkedBlock.cpp:

(JSC::MarkedBlock::Handle::stopAllocating):
(JSC::MarkedBlock::Handle::lastChanceToFinalize):
(JSC::MarkedBlock::Handle::resumeAllocating):
(JSC::MarkedBlock::Handle::zap):
(JSC::MarkedBlock::Handle::sweep):
(JSC::MarkedBlock::Handle::isFreeListedCell):
(JSC::MarkedBlock::Handle::forEachFreeCell): Deleted.

  • heap/MarkedBlock.h:
  • heap/MarkedBlockInlines.h:

(JSC::MarkedBlock::Handle::specializedSweep):
(JSC::MarkedBlock::Handle::finishSweepKnowingSubspace):
(JSC::MarkedBlock::Handle::isFreeListedCell): Deleted.

  • heap/Subspace.cpp:

(JSC::Subspace::finishSweep):

  • heap/Subspace.h:
  • jit/AssemblyHelpers.h:

(JSC::AssemblyHelpers::emitAllocateWithNonNullAllocator):

  • runtime/JSDestructibleObjectSubspace.cpp:

(JSC::JSDestructibleObjectSubspace::finishSweep):

  • runtime/JSDestructibleObjectSubspace.h:
  • runtime/JSSegmentedVariableObjectSubspace.cpp:

(JSC::JSSegmentedVariableObjectSubspace::finishSweep):

  • runtime/JSSegmentedVariableObjectSubspace.h:
  • runtime/JSStringSubspace.cpp:

(JSC::JSStringSubspace::finishSweep):

  • runtime/JSStringSubspace.h:
  • wasm/js/JSWebAssemblyCodeBlockSubspace.cpp:

(JSC::JSWebAssemblyCodeBlockSubspace::finishSweep):

  • wasm/js/JSWebAssemblyCodeBlockSubspace.h:
8:18 AM Changeset in webkit [217710] by zandobersek@gmail.com
  • 4 edits in trunk/LayoutTests

Unreviewed WPE gardening.

  • platform/wpe/TestExpectations: Add a few test failure expectations.
  • platform/wpe/js/dom/global-constructors-attributes-dedicated-worker-expected.txt: Rebaseline.
  • platform/wpe/js/dom/global-constructors-attributes-expected.txt: Ditto.
4:35 AM Changeset in webkit [217709] by jfernandez@igalia.com
  • 7 edits
    2 adds in trunk

[css-grid] Logical margin incorrectly applied during the tracks sizing algorithm of auto tracks
https://bugs.webkit.org/show_bug.cgi?id=172836

Reviewed by Manuel Rego Casasnovas.

Source/WebCore:

When computing min-content and max-content of the content-sized tracks
we are using the marginIntrinsicLogicalWidthForChild function, which
uses the grid's writing-mode to determine wether to use the child's
margin width or height. This is not correct when the grid item is
orthogonal.

This patch changes how we compute the tracks width so we use always
the item's marginLogicalWidth, which depends only on its own writing
mode.

Test: fast/css-grid-layout/grid-track-sizing-with-margins-and-orthogonal-flows.html

  • rendering/GridTrackSizingAlgorithm.cpp:

(WebCore::GridTrackSizingAlgorithmStrategy::minContentForChild):
(WebCore::GridTrackSizingAlgorithmStrategy::maxContentForChild):

LayoutTests:

Test to verify different cases of auto-sized tracks and orthogonal items with margins.

  • fast/css-grid-layout/grid-track-sizing-with-margins-and-orthogonal-flows-expected.txt: Added.
  • fast/css-grid-layout/grid-track-sizing-with-margins-and-orthogonal-flows.html: Added.
3:18 AM Changeset in webkit [217708] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

Invalidate the shadow subtree style when slotted pseudo rules are present.
https://bugs.webkit.org/show_bug.cgi?id=172822

Patch by Emilio Cobos Álvarez <ecobos@igalia.com> on 2017-06-02
Reviewed by Antti Koivisto.

No new tests, this is tested by
fast/shadow-dom/css-scoping-slot-with-id.html, once we don't force an
Inherit style change for slots. I could add some more tests for stuff
with descendant combinators and similar I guess, though.

  • style/AttributeChangeInvalidation.cpp:

(WebCore::Style::AttributeChangeInvalidation::invalidateStyle):

  • style/ClassChangeInvalidation.cpp:

(WebCore::Style::ClassChangeInvalidation::invalidateStyle):

  • style/IdChangeInvalidation.cpp:

(WebCore::Style::IdChangeInvalidation::invalidateStyle):

2:57 AM Changeset in webkit [217707] by yoon@igalia.com
  • 5 edits in trunk/Source/WebKit2

REGRESSION(r217214): [Coordinated Graphics] UpdateAtlas didn't released while resetting testrunner
https://bugs.webkit.org/show_bug.cgi?id=172746

Reviewed by Žan Doberšek.

After clearing update atlases in CompositingCoordinator, we should
release update atlases at the compositing thread, too.

We cannot remove update atlases from the CoordinatedGraphicsScene
without regard to update scheduling, since previously committed scene
states can try to use already released update atlases.
To prevent this timing problem, releasing update atlases should be done
with a update scheduler.

  • Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp:

(WebKit::CoordinatedGraphicsScene::syncRemoteContent):
(WebKit::CoordinatedGraphicsScene::appendUpdate):

  • Shared/CoordinatedGraphics/CoordinatedGraphicsScene.h:

Use WTF::Function instead of std::function

  • Shared/CoordinatedGraphics/threadedcompositor/ThreadedCompositor.cpp:

(WebKit::ThreadedCompositor::updateSceneState):
(WebKit::ThreadedCompositor::releaseUpdateAtlases):

  • WebProcess/WebPage/CoordinatedGraphics/CompositingCoordinator.cpp:

(WebKit::ThreadedCompositor::flushPendingLayerChanges):
Release UpdateAtlases at the end of compositing states update, too.
(WebKit::CompositingCoordinator::clearUpdateAtlases):

2:18 AM Changeset in webkit [217706] by magomez@igalia.com
  • 6 edits in trunk/Source/WebCore

[GTK+][WPE] Allow TextureMapperShaderProgram to perform colorspace conversions with the source texture pixels
https://bugs.webkit.org/show_bug.cgi?id=172749

Reviewed by Žan Doberšek.

Currently TextureMapperGL is only able to paint properly textures that are in RGBA format. There are
situations where we will want it to be able to handle other formats (for example BGRA for gstreamer-gl).
To make this possible without having to perform the color conversion in the CPU, we add a new color
conversion matrix to TextureMapperShaderProgram. This matrix will perform the color space conversion
in the GPU, just after sampling the source texture.

Also, add a new flag to TextureMapperGL to indicate that a color conversion is required to handle a
texture. This flag will be set to the TextureMapperPlatformLayerBuffers as required, and TextureMapperGL
will set the appropriate color space matrix to the shader. Initially only the flag for BGRA->RGBA conversion
is defined, but support for new transformations can be added as needed.

VideoTextureCopierGStreamer also uses the TextureMapperShaderProgram to perform texture sampling, but for
the moment it just sets an identity matrix because when using gstreamer-gl, the format of the video frames
is RGBA. This will be changed soon to use BGRA.

No new tests.

  • platform/graphics/gstreamer/VideoTextureCopierGStreamer.cpp:

(WebCore::VideoTextureCopierGStreamer::copyVideoTextureToPlatformTexture):

  • platform/graphics/texmap/TextureMapperGL.cpp:

(WebCore::colorSpaceMatrixForFlags):
(WebCore::TextureMapperGL::drawTexturedQuadWithProgram):

  • platform/graphics/texmap/TextureMapperGL.h:
  • platform/graphics/texmap/TextureMapperShaderProgram.cpp:
  • platform/graphics/texmap/TextureMapperShaderProgram.h:
2:08 AM Changeset in webkit [217705] by jfernandez@igalia.com
  • 9 edits
    2 adds in trunk

[css-grid] Margin wrong applied when stretching an orthogonal item in fixed size track
https://bugs.webkit.org/show_bug.cgi?id=172590

Reviewed by Sergio Villar Senin.

LayoutTests/imported/w3c:

All the test cases of these tests pass with this change, so updating their expectations accordingly.

  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-006-expected.txt:
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-006-expected.txt:
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-006-expected.txt:

Source/WebCore:

We need to consider orthogonality when using the item's logical margin to
compute the available space for stretching.

The issue this patch fixes is only reproducible when the grid layout logic
is executed several times, since probably the item doesn't need to be
laid out again. In such cases, we just get the cached logical margins
but we were not taking orthogonality into account.

Test: fast/css-grid-layout/grid-item-stretching-must-not-depend-on-previous-layouts.html

  • rendering/RenderGrid.cpp:

(WebCore::RenderGrid::layoutBlock):
(WebCore::RenderGrid::marginLogicalSizeForChild):
(WebCore::RenderGrid::availableAlignmentSpaceForChildBeforeStretching):

  • rendering/RenderGrid.h:

LayoutTests:

  • TestExpectations: 2 tests pass now but 3 more fail because of bug #172836
  • fast/css-grid-layout/grid-item-stretching-must-not-depend-on-previous-layouts-expected.html: Added.
  • fast/css-grid-layout/grid-item-stretching-must-not-depend-on-previous-layouts.html: Added.
12:52 AM Changeset in webkit [217704] by zandobersek@gmail.com
  • 5 edits
    1 add in trunk

[WPE] Enable SUBTLE_CRYPTO
https://bugs.webkit.org/show_bug.cgi?id=172756

Reviewed by Michael Catanzaro.

Tools:

  • Scripts/webkitperl/FeatureList.pm: Set ENABLE_SUBTLE_CRYPTO to 1 for WPE.
  • wpe/jhbuild.modules: Add libgpg-error and libgcrypt dependencies.
  • wpe/patches/libgcrypt-use-only-dev-urandom-for-testing.patch: Added. Prevents

exhausting entropy when running the crypto layout tests.

LayoutTests:

  • platform/wpe/TestExpectations: Enable tests under crypto/ and

imported/w3c/web-platform-tests/WebCryptoAPI/ in general, but add
the test expectations for failing tests under those directories.

12:47 AM Changeset in webkit [217703] by Yusuke Suzuki
  • 3 edits in trunk/Source/JavaScriptCore

[JSC] Use @globalPrivate for concatSlowPath
https://bugs.webkit.org/show_bug.cgi?id=172802

Reviewed by Darin Adler.

Use @globalPrivate instead of manually putting it to JSGlobalObject.

  • builtins/ArrayPrototype.js:

(concatSlowPath): Deleted.

  • runtime/JSGlobalObject.cpp:

(JSC::JSGlobalObject::init):

Jun 1, 2017:

11:40 PM Changeset in webkit [217702] by Carlos Garcia Campos
  • 6 edits
    2 adds in trunk/Source/WebCore

[GTK] Cache RenderThemeGadget hierarchies for rendering themed elements with GTK+ 3.20+
https://bugs.webkit.org/show_bug.cgi?id=162673

Reviewed by Michael Catanzaro.

Because of the way the new theming system works in GTK+ >= 3.20 we are currently creating a gadget hierarchy
every time we need to render a styled element or get layout information about it. That's happening on every
repaint, and it's specially problematic for overlay scrollbar indicators that fade in/out when shown/hidden. We
need to cache the gadgets and simply apply the state before every paint or layout query. When using GtkWidgetPath,
calling gtk_style_context_save() breaks the gadget hierarchy, and style classes need to be set when building the
GtkWidgetPath. That means we can't cache RenderThemeGadgets, call save, apply style classes and state, and then
call restore. We need to cache gadget hierarchies with fixed style classes. Fortunately, setting the state does
work, so we don't need to also cache a different hierarchy for every possible state. For example, for the
particular case of scrollbars we would cache VerticalScrollbarRight, VerticalScrollbarLeft, HorizontalScrollbar,
VerticalScrollIndicatorRight, VerticalScrollIndicatorLeft and HorizontalScrollIndicator. In practice, we will
only have 4 of those at the same time in the cache.
This patch adds RenderThemeWidget to represent a hierarchy of gadgets with fixed style classes that can be
cached and reused to render or query style of those "widgets". It also simplifies the RenderThemeGtk and
ScrollbarThemeGtk code by removing a lot of duplicated code to build the gadget hierarchies.

  • PlatformGTK.cmake:
  • platform/gtk/RenderThemeGadget.cpp:

(WebCore::createStyleContext):
(WebCore::appendElementToPath):
(WebCore::RenderThemeGadget::state):
(WebCore::RenderThemeGadget::setState):

  • platform/gtk/RenderThemeGadget.h:
  • platform/gtk/RenderThemeWidget.cpp: Added.

(WebCore::widgetMap):
(WebCore::RenderThemeWidget::getOrCreate):
(WebCore::RenderThemeWidget::clearCache):
(WebCore::RenderThemeWidget::~RenderThemeWidget):
(WebCore::RenderThemeScrollbar::RenderThemeScrollbar):
(WebCore::RenderThemeScrollbar::stepper):
(WebCore::RenderThemeToggleButton::RenderThemeToggleButton):
(WebCore::RenderThemeButton::RenderThemeButton):
(WebCore::RenderThemeComboBox::RenderThemeComboBox):
(WebCore::RenderThemeEntry::RenderThemeEntry):
(WebCore::RenderThemeSearchEntry::RenderThemeSearchEntry):
(WebCore::RenderThemeSpinButton::RenderThemeSpinButton):
(WebCore::RenderThemeSlider::RenderThemeSlider):
(WebCore::RenderThemeProgressBar::RenderThemeProgressBar):
(WebCore::RenderThemeListView::RenderThemeListView):
(WebCore::RenderThemeIcon::RenderThemeIcon):

  • platform/gtk/RenderThemeWidget.h: Added.

(WebCore::RenderThemeEntry::entry):
(WebCore::RenderThemeEntry::selection):

  • platform/gtk/ScrollbarThemeGtk.cpp:

(WebCore::ScrollbarThemeGtk::themeChanged):
(WebCore::ScrollbarThemeGtk::updateThemeProperties):
(WebCore::widgetTypeForScrollbar):
(WebCore::contentsRectangle):
(WebCore::ScrollbarThemeGtk::trackRect):
(WebCore::ScrollbarThemeGtk::backButtonRect):
(WebCore::ScrollbarThemeGtk::forwardButtonRect):
(WebCore::ScrollbarThemeGtk::paint):
(WebCore::ScrollbarThemeGtk::scrollbarThickness):
(WebCore::ScrollbarThemeGtk::minimumThumbLength):

  • rendering/RenderThemeGtk.cpp:

(WebCore::createStyleContext):
(WebCore::setToggleSize):
(WebCore::paintToggle):
(WebCore::RenderThemeGtk::paintButton):
(WebCore::menuListColor):
(WebCore::RenderThemeGtk::popupInternalPaddingBox):
(WebCore::RenderThemeGtk::paintMenuList):
(WebCore::RenderThemeGtk::adjustTextFieldStyle):
(WebCore::RenderThemeGtk::paintTextField):
(WebCore::adjustSearchFieldIconStyle):
(WebCore::paintSearchFieldIcon):
(WebCore::RenderThemeGtk::paintSliderTrack):
(WebCore::RenderThemeGtk::adjustSliderThumbSize):
(WebCore::RenderThemeGtk::paintSliderThumb):
(WebCore::RenderThemeGtk::progressBarRectForBounds):
(WebCore::RenderThemeGtk::paintProgressBar):
(WebCore::RenderThemeGtk::adjustInnerSpinButtonStyle):
(WebCore::RenderThemeGtk::paintInnerSpinButton):
(WebCore::styleColor):
(WebCore::RenderThemeGtk::platformActiveSelectionBackgroundColor):
(WebCore::RenderThemeGtk::platformInactiveSelectionBackgroundColor):
(WebCore::RenderThemeGtk::platformActiveSelectionForegroundColor):
(WebCore::RenderThemeGtk::platformInactiveSelectionForegroundColor):
(WebCore::RenderThemeGtk::paintMediaButton):

10:55 PM Changeset in webkit [217701] by sbarati@apple.com
  • 8 edits in trunk

Ensure a good experience for ARES-6 error reporting
https://bugs.webkit.org/show_bug.cgi?id=171699

Reviewed by Filip Pizlo and Jon Davis.

This patch fixes a bug where we would silently fail running ARES-6. The bug
was that we were calling reportError with the wrong |this| value.
I also cleaned up a bit of the code around error reporting. We
now indicate which test failed, and update the status to reflect
that a failure happened.

This patch also modifies the CSS a bit to work better on smaller
screened devices. The CSS prevents the status from having a line
break both when an error is reported and when we're running the
benchmark.

  • ARES-6/driver.js:

(Driver):
(Driver.prototype.reportError):

  • ARES-6/results.js:

(Results.prototype.reportError):
(Results):

  • ARES-6/styles.css:

(.start):
(#status):
(.failed):
(#status.failed):
(.test .failed:before):
(#magic):
(@media only screen and (max-width: 784px)):
(.test):
(p):
(@media only screen and (max-width: 320px)):

10:32 PM Changeset in webkit [217700] by bshafiei@apple.com
  • 2 edits in tags/Safari-604.1.23.0.4/Source/WebCore

Merged r217692. rdar://problem/32514813

10:30 PM Changeset in webkit [217699] by bshafiei@apple.com
  • 2 edits in tags/Safari-604.1.23.0.4/Source/WebKit2

Merged r217681. rdar://problem/32518102

10:28 PM Changeset in webkit [217698] by bshafiei@apple.com
  • 7 edits in tags/Safari-604.1.23.0.4/Source

Versioning.

10:24 PM Changeset in webkit [217697] by bshafiei@apple.com
  • 1 copy in tags/Safari-604.1.23.0.4

New tag.

10:04 PM Changeset in webkit [217696] by akling@apple.com
  • 18 edits
    3 adds in trunk

[Mac] Remove backing store for layers that are outside the viewport
https://bugs.webkit.org/show_bug.cgi?id=170082
<rdar://problem/31245009>

Reviewed by Simon Fraser.

Source/WebCore:

Implement the backingStoreAttached flag in PlatformCALayerCocoa. This means that
compositing layers outside the tiling coverage rect will no longer have backing
stores, saving large amounts of memory.

Also added a canDetachBackingStore flag that is set to false for scroll control
layers, to avoid complicating coverage rect computations.

Test: compositing/backing-store-attachment-1.html

  • page/Frame.h:
  • platform/graphics/GraphicsLayer.cpp:

(WebCore::GraphicsLayer::GraphicsLayer):
(WebCore::GraphicsLayer::dumpProperties):

  • platform/graphics/GraphicsLayer.h:

(WebCore::GraphicsLayer::backingStoreAttached):
(WebCore::GraphicsLayer::setCanDetachBackingStore):
(WebCore::GraphicsLayer::canDetachBackingStore):

  • platform/graphics/GraphicsLayerClient.h:
  • platform/graphics/ca/GraphicsLayerCA.cpp:

(WebCore::GraphicsLayerCA::backingStoreAttached):
(WebCore::GraphicsLayerCA::setNeedsDisplay):
(WebCore::GraphicsLayerCA::updateCoverage):

  • platform/graphics/ca/GraphicsLayerCA.h:
  • platform/graphics/ca/PlatformCALayer.h:
  • platform/graphics/ca/cocoa/PlatformCALayerCocoa.h:
  • platform/graphics/ca/cocoa/PlatformCALayerCocoa.mm:

(PlatformCALayerCocoa::setBackingStoreAttached):
(PlatformCALayerCocoa::backingStoreAttached):

  • rendering/RenderLayerBacking.cpp:

(WebCore::RenderLayerBacking::updateOverflowControlsLayers):

  • rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::layerTreeAsText):
(WebCore::RenderLayerCompositor::updateOverflowControlsLayers):

  • testing/Internals.cpp:

(WebCore::toLayerTreeFlags):

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

Source/WebKit2:

  • WebProcess/WebPage/mac/PlatformCALayerRemote.h:

LayoutTests:

Add a simple test with two compositing layers far apart vertically. Only one of them
should have its backing store attached.

  • compositing/backing-store-attachment-1-expected.txt: Added.
  • compositing/backing-store-attachment-1.html: Added.
9:35 PM Changeset in webkit [217695] by Chris Dumez
  • 2 edits in trunk/Source/WebKit2

REGRESSION (r206386): Xactimate Website Crashes @ com.apple.WebKit: WebKit::NPRuntimeObjectMap::convertJSValueToNPVariant + 255
https://bugs.webkit.org/show_bug.cgi?id=172846
<rdar://problem/31093005>

Reviewed by Andreas Kling.

In NPJSObject::invoke(), return early if there was an exception when calling JSC::call().
Using the value returned by JSC::call() when an exception occurred is unsafe.

  • WebProcess/Plugins/Netscape/NPJSObject.cpp:

(WebKit::NPJSObject::invoke):

9:17 PM Changeset in webkit [217694] by Ryan Haddad
  • 11 edits
    3 deletes in trunk

Unreviewed, rolling out r217691.

This change broke the Windows build.

Reverted changeset:

"Web Inspector: Should see active Web Sockets when opening Web
Inspector"
https://bugs.webkit.org/show_bug.cgi?id=172312
http://trac.webkit.org/changeset/217691

9:14 PM Changeset in webkit [217693] by Joseph Pecoraro
  • 3 edits in trunk/Source/WebCore

Unreviewed Windows build fix after r217691.

  • Modules/websockets/WebSocketChannel.cpp:

(WebCore::WebSocketChannel::sendFrame):

  • Modules/websockets/WebSocketChannel.h:

Disambiguate the Function being used.

7:50 PM Changeset in webkit [217692] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

LibWebRTC might crash with frames having a null width or height
https://bugs.webkit.org/show_bug.cgi?id=172842
<rdar://problem/32514813>

Patch by Youenn Fablet <youenn@apple.com> on 2017-06-01
Reviewed by Jon Lee.

Do not send black frames in case the video width or height is zero.
Also ensure that even if a null buffer is returned by the pool, no crash might actually happen.

  • platform/mediastream/mac/RealtimeOutgoingVideoSource.cpp:

(WebCore::RealtimeOutgoingVideoSource::sendBlackFramesIfNeeded):
(WebCore::RealtimeOutgoingVideoSource::videoSampleAvailable):

7:44 PM Changeset in webkit [217691] by Devin Rousso
  • 11 edits
    3 adds in trunk

Web Inspector: Should see active Web Sockets when opening Web Inspector
https://bugs.webkit.org/show_bug.cgi?id=172312

Reviewed by Joseph Pecoraro.

Source/WebCore:

Test: http/tests/websocket/tests/hybi/inspector/before-load.html

  • Modules/websockets/WebSocket.h:
  • Modules/websockets/WebSocket.cpp:

(WebCore::WebSocket::WebSocket):
(WebCore::WebSocket::~WebSocket):
(WebCore::WebSocket::allActiveWebSockets):
(WebCore::WebSocket::allActiveWebSocketsMutex):
(WebCore::WebSocket::channel):
(WebCore::WebSocket::eventTargetInterface):
Add a static set of all WebSocket objects and provide getters to access web socket metadata.

  • Modules/websockets/ThreadableWebSocketChannel.h:

(WebCore::ThreadableWebSocketChannel::isWebSocketChannel):

  • Modules/websockets/WebSocketChannel.h:
  • Modules/websockets/WebSocketChannel.cpp:

(WebCore::WebSocketChannel::isWebSocketChannel):
(WebCore::WebSocketChannel::identifier):
(WebCore::WebSocketChannel::clientHandshakeRequest):
(WebCore::WebSocketChannel::serverHandshakeResponse):
(WebCore::WebSocketChannel::handshakeMode):
(WebCore::WebSocketChannel::isType):
Provide getters to access web socket metadata.

  • inspector/InspectorNetworkAgent.cpp:

(WebCore::InspectorNetworkAgent::enable):
Loops over the static set of all WebSocket objects and sends events for each to the frontend.

Source/WebInspectorUI:

  • UserInterface/Controllers/FrameResourceManager.js:

(WebInspector.FrameResourceManager.prototype.webSocketHandshakeResponseReceived):
(WebInspector.FrameResourceManager.prototype.initialize): Removed.
Rework order of agent enable calls to ensure that the main frame is initialized before any
websocket events are dispatched.

  • UserInterface/Models/WebSocketResource.js:

(WebInspector.WebSocketResource.ReadyState):
Change symbol strings in enum for better printing in tests.

LayoutTests:

Test that WebSockets created before the inspector loads are also tracked.

  • http/tests/websocket/tests/hybi/inspector/before-load-expected.txt: Added.
  • http/tests/websocket/tests/hybi/inspector/before-load.html: Added.
  • http/tests/websocket/tests/hybi/inspector/before-load_wsh.py: Added.

(web_socket_do_extra_handshake):
(web_socket_transfer_data):

5:50 PM Changeset in webkit [217690] by Nikita Vasilyev
  • 5 edits in trunk/Source/WebInspectorUI

Web Inspector: Web Sockets: Unable to inspect a WebSocket that receives >50 messages per second
https://bugs.webkit.org/show_bug.cgi?id=171904
<rdar://problem/32095863>

Reviewed by Matt Baker.

Every size increase of a resource causes a sidebar item to be re-attached to the DOM. Sending or
receiving a WebSocket message causes a size increase. When size increase happens too frequently,
the go-to arrow starts to flicker and becomes unclickable. This patch makes sure a sidebar item gets
re-attached only when its position (index) changes.

  • UserInterface/Views/NetworkSidebarPanel.js:

Add the go-to arrow and close buttons when treeElement.status is a spinner element.

(WebInspector.NetworkSidebarPanel.prototype.treeElementAddedOrChanged):

  • UserInterface/Views/ResourceTreeElement.js:

(WebInspector.ResourceTreeElement.prototype._updateStatus):
Setting status to "" removes the go-to arrow and close buttons in the Network tab.
Only set status to "" to remove the spinner.

  • UserInterface/Views/TimelineDataGrid.js:

(WebInspector.TimelineDataGrid.prototype._refreshDirtyDataGridNodes):

  • UserInterface/Views/TreeOutline.js:

(WebInspector.TreeOutline.prototype.reattachIfIndexChanged): Added.

4:50 PM Changeset in webkit [217689] by Ryan Haddad
  • 3 edits in trunk/LayoutTests

Mark http/tests/navigation/statistics.html as flaky.
https://bugs.webkit.org/show_bug.cgi?id=172454

Unreviewed test gardening.

  • platform/ios-wk2/TestExpectations:
  • platform/mac-wk2/TestExpectations:
4:16 PM Changeset in webkit [217688] by Simon Fraser
  • 2 edits in trunk/Source/WebCore

REGRESSION (r217296): Dragging a video only shows controls during drag event
https://bugs.webkit.org/show_bug.cgi?id=172837
rdar://problem/32434661

Reviewed by Zalan Bujtas.

r217296 changed RenderVideo::paintReplaced() from checking paintBehavior() flags on the FrameView
to paintInfo.paintBehavior flags. However, RenderLayer was stripping the PaintBehaviorFlattenCompositingLayers
flag from the paintInfo.paintBehavior flags, so fix that.

Not easily testable.

  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::paintLayerContents):

3:40 PM Changeset in webkit [217687] by Simon Fraser
  • 3 edits in trunk/Source/WebCore

Remove the redundant selectionOnly argument from RenderLayer::paintForegroundForFragments()
https://bugs.webkit.org/show_bug.cgi?id=172835

Reviewed by Zalan Bujtas.

The 'selectionOnly' state is already encoded in localPaintingInfo.paintBehavior so there's
no need to pass it separately.

  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::paintLayerContents):
(WebCore::RenderLayer::paintForegroundForFragments):

  • rendering/RenderLayer.h:
3:32 PM Changeset in webkit [217686] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Fix compilation errors in LocalizedStrings.cpp on platform not USE(CF) nor USE(GLIB)
https://bugs.webkit.org/show_bug.cgi?id=172796

Patch by Yoshiaki Jitsukawa <Yoshiaki.Jitsukawa@sony.com> on 2017-06-01
Reviewed by Alex Christensen.

No new tests needed. Fixing compilation errors.

  • platform/LocalizedStrings.cpp:

(WebCore::contextMenuItemTagLookUpInDictionary):
(WebCore::imageTitle):
(WebCore::validationMessageRangeUnderflowText):
(WebCore::validationMessageRangeOverflowText):

2:53 PM Changeset in webkit [217685] by Matt Lewis
  • 2 edits in trunk/LayoutTests

Fixed typo in test expectations.
https://bugs.webkit.org/show_bug.cgi?id=172054

Unreviewed test gardening.

  • platform/mac-wk2/TestExpectations:
2:39 PM Changeset in webkit [217684] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Rebaseline webarchive/loading/mainresource-null-mimetype-crash.html after r217680.

Unreviewed test gardening.

  • webarchive/loading/mainresource-null-mimetype-crash-expected.txt:
2:17 PM Changeset in webkit [217683] by Chris Dumez
  • 2 edits in trunk/Source/WebCore

Unreviewed, rolling out r217647.

No longer needed after r217682

Reverted changeset:

"Avoid iterator invalidation bug in
WebCore::defaultPortForProtocol"
https://bugs.webkit.org/show_bug.cgi?id=172786
http://trac.webkit.org/changeset/217647

2:02 PM Changeset in webkit [217682] by Chris Dumez
  • 2 edits in trunk/Source/WebCore

Make WebCore::defaultPortForProtocol() thread-safe
https://bugs.webkit.org/show_bug.cgi?id=172797

Reviewed by Brent Fulgham.

Make WebCore::defaultPortForProtocol() thread-safe since it is called from the SecurityOrigin
constructor and SecurityOrigin objects are constructed from various threads.

This should not regress the non-testing code paths since we only pay locking costs if
a default port override has been set by the tests.

  • platform/URL.cpp:

(WebCore::defaultPortForProtocolMapLock):
(WebCore::defaultPortForProtocolMapForTesting):
(WebCore::ensureDefaultPortForProtocolMapForTesting):
(WebCore::registerDefaultPortForProtocolForTesting):
(WebCore::clearDefaultPortForProtocolMapForTesting):
(WebCore::defaultPortForProtocol):

1:33 PM Changeset in webkit [217681] by jonlee@apple.com
  • 2 edits in trunk/Source/WebKit2

Rename an experimental feature item
https://bugs.webkit.org/show_bug.cgi?id=172830

Reviewed by Joseph Pecoraro.

  • Shared/WebPreferencesDefinitions.h:
1:09 PM Changeset in webkit [217680] by Chris Dumez
  • 5 edits in trunk/LayoutTests

webarchive/loading/mainresource-null-mimetype-crash.html / webarchive/loading/missing-data.html are flaky
https://bugs.webkit.org/show_bug.cgi?id=172823

Reviewed by Alexey Proskuryakov.

Rewrite webarchive/loading/mainresource-null-mimetype-crash.html / webarchive/loading/missing-data.html
to address flakiness. We now navigate a subframe instead of the main frame. We also mark the test as
asynchronous and wait for the subframe to finish loading before calling testRunner.notifyDone().

  • platform/mac/webarchive/loading/mainresource-null-mimetype-crash-expected.txt:
  • webarchive/loading/mainresource-null-mimetype-crash.html:
  • webarchive/loading/missing-data-expected.txt:
  • webarchive/loading/missing-data.html:
12:52 PM Changeset in webkit [217679] by matthew_hanson@apple.com
  • 2 edits in branches/safari-603-branch/Source/WebCore

Cherry-pick r212174. rdar://problem/32080671

12:52 PM Changeset in webkit [217678] by matthew_hanson@apple.com
  • 39 edits
    1 add in branches/safari-603-branch

Cherry-pick r212173. rdar://problem/32080671

12:52 PM Changeset in webkit [217677] by matthew_hanson@apple.com
  • 7 edits in branches/safari-603-branch

Cherry-pick r211501. rdar://problem/32080671

12:51 PM Changeset in webkit [217676] by matthew_hanson@apple.com
  • 8 edits in branches/safari-603-branch/Source/WebCore

Cherry-pick r210226. rdar://problem/32080671

12:51 PM Changeset in webkit [217675] by matthew_hanson@apple.com
  • 5 edits
    1 add in branches/safari-603-branch

Cherry-pick r216459. rdar://problem/32119857

12:46 PM Changeset in webkit [217674] by commit-queue@webkit.org
  • 399 edits
    9 adds
    2 deletes in trunk

[GTK] Bump GTK+ and its dependencies in jhbuild
https://bugs.webkit.org/show_bug.cgi?id=171918

Patch by Carlos Garcia Campos <cgarcia@igalia.com> on 2017-06-01
Reviewed by Carlos Alberto Lopez Perez.

Tools:

  • WebKitTestRunner/InjectedBundle/gtk/ActivateFontsGtk.cpp:

(WTR::initializeGtkSettings):

  • gtk/jhbuild.modules:
  • gtk/patches/gtk+-configure-fix-detecting-CUPS-2.x.patch: Removed.
  • gtk/patches/librsvg-2.36.1-bump-up-config.guess-to-support-aarch64.patch: Removed.

LayoutTests:

Rebaseline tests for new GTK+ version.

  • platform/gtk/css1/box_properties/acid_test-expected.txt:
  • platform/gtk/css2.1/20110323/replaced-elements-001-expected.png:
  • platform/gtk/css2.1/20110323/replaced-elements-001-expected.txt:
  • platform/gtk/css2.1/t09-c5526c-display-00-e-expected.png:
  • platform/gtk/css2.1/t09-c5526c-display-00-e-expected.txt:
  • platform/gtk/css3/flexbox/button-expected.png:
  • platform/gtk/css3/flexbox/button-expected.txt:
  • platform/gtk/css3/selectors3/html/css3-modsel-161-expected.png:
  • platform/gtk/css3/selectors3/html/css3-modsel-161-expected.txt:
  • platform/gtk/css3/selectors3/html/css3-modsel-19b-expected.png:
  • platform/gtk/css3/selectors3/html/css3-modsel-19b-expected.txt:
  • platform/gtk/css3/selectors3/html/css3-modsel-23-expected.png:
  • platform/gtk/css3/selectors3/html/css3-modsel-23-expected.txt:
  • platform/gtk/css3/selectors3/html/css3-modsel-25-expected.png:
  • platform/gtk/css3/selectors3/html/css3-modsel-25-expected.txt:
  • platform/gtk/css3/selectors3/html/css3-modsel-69-expected.png:
  • platform/gtk/css3/selectors3/html/css3-modsel-69-expected.txt:
  • platform/gtk/css3/selectors3/html/css3-modsel-70-expected.png:
  • platform/gtk/css3/selectors3/html/css3-modsel-70-expected.txt:
  • platform/gtk/css3/selectors3/xhtml/css3-modsel-161-expected.png:
  • platform/gtk/css3/selectors3/xhtml/css3-modsel-161-expected.txt:
  • platform/gtk/css3/selectors3/xhtml/css3-modsel-19b-expected.png:
  • platform/gtk/css3/selectors3/xhtml/css3-modsel-19b-expected.txt:
  • platform/gtk/css3/selectors3/xhtml/css3-modsel-23-expected.png:
  • platform/gtk/css3/selectors3/xhtml/css3-modsel-23-expected.txt:
  • platform/gtk/css3/selectors3/xhtml/css3-modsel-25-expected.png:
  • platform/gtk/css3/selectors3/xhtml/css3-modsel-25-expected.txt:
  • platform/gtk/css3/selectors3/xhtml/css3-modsel-69-expected.png:
  • platform/gtk/css3/selectors3/xhtml/css3-modsel-69-expected.txt:
  • platform/gtk/css3/selectors3/xhtml/css3-modsel-70-expected.png:
  • platform/gtk/css3/selectors3/xhtml/css3-modsel-70-expected.txt:
  • platform/gtk/css3/selectors3/xml/css3-modsel-161-expected.png:
  • platform/gtk/css3/selectors3/xml/css3-modsel-161-expected.txt:
  • platform/gtk/css3/selectors3/xml/css3-modsel-19b-expected.png:
  • platform/gtk/css3/selectors3/xml/css3-modsel-19b-expected.txt:
  • platform/gtk/css3/selectors3/xml/css3-modsel-23-expected.png:
  • platform/gtk/css3/selectors3/xml/css3-modsel-23-expected.txt:
  • platform/gtk/css3/selectors3/xml/css3-modsel-25-expected.png:
  • platform/gtk/css3/selectors3/xml/css3-modsel-25-expected.txt:
  • platform/gtk/css3/selectors3/xml/css3-modsel-69-expected.png:
  • platform/gtk/css3/selectors3/xml/css3-modsel-69-expected.txt:
  • platform/gtk/css3/selectors3/xml/css3-modsel-70-expected.png:
  • platform/gtk/css3/selectors3/xml/css3-modsel-70-expected.txt:
  • platform/gtk/editing/pasteboard/4641033-expected.png:
  • platform/gtk/editing/pasteboard/4641033-expected.txt:
  • platform/gtk/editing/pasteboard/4944770-1-expected.png:
  • platform/gtk/editing/pasteboard/4944770-1-expected.txt:
  • platform/gtk/editing/pasteboard/4944770-2-expected.png:
  • platform/gtk/editing/pasteboard/4944770-2-expected.txt:
  • platform/gtk/editing/selection/3690703-2-expected.png:
  • platform/gtk/editing/selection/3690703-2-expected.txt:
  • platform/gtk/editing/selection/3690703-expected.png:
  • platform/gtk/editing/selection/3690703-expected.txt:
  • platform/gtk/editing/selection/3690719-expected.png:
  • platform/gtk/editing/selection/3690719-expected.txt:
  • platform/gtk/editing/selection/4397952-expected.png:
  • platform/gtk/editing/selection/4397952-expected.txt:
  • platform/gtk/editing/selection/5240265-expected.png:
  • platform/gtk/editing/selection/5240265-expected.txt:
  • platform/gtk/editing/selection/caret-before-select-expected.png:
  • platform/gtk/editing/selection/caret-before-select-expected.txt:
  • platform/gtk/editing/selection/replaced-boundaries-3-expected.png:
  • platform/gtk/editing/selection/replaced-boundaries-3-expected.txt:
  • platform/gtk/editing/selection/select-box-expected.png:
  • platform/gtk/editing/selection/select-box-expected.txt:
  • platform/gtk/editing/selection/select-element-paragraph-boundary-expected.png:
  • platform/gtk/editing/selection/select-element-paragraph-boundary-expected.txt:
  • platform/gtk/editing/selection/selection-button-text-expected.png:
  • platform/gtk/editing/selection/selection-button-text-expected.txt:
  • platform/gtk/fast/block/basic/011-expected.png:
  • platform/gtk/fast/block/basic/011-expected.txt:
  • platform/gtk/fast/block/float/float-avoidance-expected.png:
  • platform/gtk/fast/block/float/float-avoidance-expected.txt:
  • platform/gtk/fast/block/margin-collapse/103-expected.png:
  • platform/gtk/fast/block/margin-collapse/103-expected.txt:
  • platform/gtk/fast/block/positioning/inline-block-relposition-expected.png:
  • platform/gtk/fast/block/positioning/inline-block-relposition-expected.txt:
  • platform/gtk/fast/box-sizing/percentage-height-expected.png:
  • platform/gtk/fast/box-sizing/percentage-height-expected.txt:
  • platform/gtk/fast/css/continuationCrash-expected.png:
  • platform/gtk/fast/css/continuationCrash-expected.txt:
  • platform/gtk/fast/css/focus-ring-exists-for-search-field-expected.png:
  • platform/gtk/fast/css/focus-ring-exists-for-search-field-expected.txt:
  • platform/gtk/fast/css/input-search-padding-expected.png:
  • platform/gtk/fast/css/input-search-padding-expected.txt:
  • platform/gtk/fast/css/margin-top-bottom-dynamic-expected.png:
  • platform/gtk/fast/css/margin-top-bottom-dynamic-expected.txt:
  • platform/gtk/fast/css/non-standard-checkbox-size-expected.png:
  • platform/gtk/fast/css/non-standard-checkbox-size-expected.txt:
  • platform/gtk/fast/css/rtl-ordering-expected.png:
  • platform/gtk/fast/css/rtl-ordering-expected.txt:
  • platform/gtk/fast/css/text-input-with-webkit-border-radius-expected.png:
  • platform/gtk/fast/css/text-input-with-webkit-border-radius-expected.txt:
  • platform/gtk/fast/css/text-overflow-input-expected.png:
  • platform/gtk/fast/css/text-overflow-input-expected.txt:
  • platform/gtk/fast/css/text-transform-select-expected.png:
  • platform/gtk/fast/css/text-transform-select-expected.txt:
  • platform/gtk/fast/dom/HTMLInputElement/input-slider-update-expected.png:
  • platform/gtk/fast/dom/HTMLInputElement/input-slider-update-expected.txt:
  • platform/gtk/fast/dom/HTMLProgressElement/indeterminate-progress-001-expected.txt:
  • platform/gtk/fast/dom/HTMLProgressElement/native-progress-bar-expected.png:
  • platform/gtk/fast/dom/HTMLProgressElement/native-progress-bar-expected.txt:
  • platform/gtk/fast/dom/HTMLProgressElement/progress-bar-value-pseudo-element-expected.png:
  • platform/gtk/fast/dom/HTMLProgressElement/progress-bar-value-pseudo-element-expected.txt:
  • platform/gtk/fast/dom/HTMLProgressElement/progress-element-expected.png:
  • platform/gtk/fast/dom/HTMLProgressElement/progress-element-expected.txt:
  • platform/gtk/fast/dom/HTMLTableColElement/resize-table-using-col-width-expected.png:
  • platform/gtk/fast/dom/HTMLTableColElement/resize-table-using-col-width-expected.txt:
  • platform/gtk/fast/dom/HTMLTextAreaElement/reset-textarea-expected.png:
  • platform/gtk/fast/dom/HTMLTextAreaElement/reset-textarea-expected.txt:
  • platform/gtk/fast/dynamic/positioned-movement-with-positioned-children-expected.txt:
  • platform/gtk/fast/forms/001-expected.png:
  • platform/gtk/fast/forms/001-expected.txt:
  • platform/gtk/fast/forms/003-expected.png:
  • platform/gtk/fast/forms/003-expected.txt:
  • platform/gtk/fast/forms/004-expected.png:
  • platform/gtk/fast/forms/004-expected.txt:
  • platform/gtk/fast/forms/basic-buttons-expected.png:
  • platform/gtk/fast/forms/basic-buttons-expected.txt:
  • platform/gtk/fast/forms/basic-inputs-expected.png:
  • platform/gtk/fast/forms/basic-inputs-expected.txt:
  • platform/gtk/fast/forms/basic-selects-expected.png:
  • platform/gtk/fast/forms/basic-selects-expected.txt:
  • platform/gtk/fast/forms/blankbuttons-expected.png:
  • platform/gtk/fast/forms/blankbuttons-expected.txt:
  • platform/gtk/fast/forms/box-shadow-override-expected.png:
  • platform/gtk/fast/forms/box-shadow-override-expected.txt:
  • platform/gtk/fast/forms/button-align-expected.png:
  • platform/gtk/fast/forms/button-align-expected.txt:
  • platform/gtk/fast/forms/button-cannot-be-nested-expected.png:
  • platform/gtk/fast/forms/button-cannot-be-nested-expected.txt:
  • platform/gtk/fast/forms/button-default-title-expected.png:
  • platform/gtk/fast/forms/button-default-title-expected.txt:
  • platform/gtk/fast/forms/button-generated-content-expected.png:
  • platform/gtk/fast/forms/button-generated-content-expected.txt:
  • platform/gtk/fast/forms/button-inner-block-reuse-expected.png:
  • platform/gtk/fast/forms/button-inner-block-reuse-expected.txt:
  • platform/gtk/fast/forms/button-positioned-expected.png:
  • platform/gtk/fast/forms/button-positioned-expected.txt:
  • platform/gtk/fast/forms/button-sizes-expected.png:
  • platform/gtk/fast/forms/button-sizes-expected.txt:
  • platform/gtk/fast/forms/button-style-color-expected.png:
  • platform/gtk/fast/forms/button-style-color-expected.txt:
  • platform/gtk/fast/forms/button-table-styles-expected.png:
  • platform/gtk/fast/forms/button-table-styles-expected.txt:
  • platform/gtk/fast/forms/button-text-transform-expected.png:
  • platform/gtk/fast/forms/button-text-transform-expected.txt:
  • platform/gtk/fast/forms/button-white-space-expected.png:
  • platform/gtk/fast/forms/button-white-space-expected.txt:
  • platform/gtk/fast/forms/control-clip-expected.png:
  • platform/gtk/fast/forms/control-clip-expected.txt:
  • platform/gtk/fast/forms/control-clip-overflow-expected.png:
  • platform/gtk/fast/forms/control-clip-overflow-expected.txt:
  • platform/gtk/fast/forms/control-restrict-line-height-expected.png:
  • platform/gtk/fast/forms/control-restrict-line-height-expected.txt:
  • platform/gtk/fast/forms/disabled-select-change-index-expected.png:
  • platform/gtk/fast/forms/disabled-select-change-index-expected.txt:
  • platform/gtk/fast/forms/file/file-input-direction-expected.png:
  • platform/gtk/fast/forms/file/file-input-direction-expected.txt:
  • platform/gtk/fast/forms/file/file-input-disabled-expected.png:
  • platform/gtk/fast/forms/file/file-input-disabled-expected.txt:
  • platform/gtk/fast/forms/form-element-geometry-expected.png:
  • platform/gtk/fast/forms/form-element-geometry-expected.txt:
  • platform/gtk/fast/forms/formmove-expected.png:
  • platform/gtk/fast/forms/formmove-expected.txt:
  • platform/gtk/fast/forms/formmove2-expected.png:
  • platform/gtk/fast/forms/formmove2-expected.txt:
  • platform/gtk/fast/forms/formmove3-expected.png:
  • platform/gtk/fast/forms/formmove3-expected.txt:
  • platform/gtk/fast/forms/indeterminate-expected.png:
  • platform/gtk/fast/forms/indeterminate-expected.txt:
  • platform/gtk/fast/forms/input-appearance-height-expected.png:
  • platform/gtk/fast/forms/input-appearance-height-expected.txt:
  • platform/gtk/fast/forms/input-button-sizes-expected.png:
  • platform/gtk/fast/forms/input-button-sizes-expected.txt:
  • platform/gtk/fast/forms/input-first-letter-expected.png:
  • platform/gtk/fast/forms/input-first-letter-expected.txt:
  • platform/gtk/fast/forms/input-number-click-expected.txt: Added.
  • platform/gtk/fast/forms/input-value-expected.png:
  • platform/gtk/fast/forms/input-value-expected.txt:
  • platform/gtk/fast/forms/menulist-deselect-update-expected.png:
  • platform/gtk/fast/forms/menulist-deselect-update-expected.txt:
  • platform/gtk/fast/forms/menulist-narrow-width-expected.png:
  • platform/gtk/fast/forms/menulist-narrow-width-expected.txt:
  • platform/gtk/fast/forms/menulist-no-overflow-expected.png:
  • platform/gtk/fast/forms/menulist-no-overflow-expected.txt:
  • platform/gtk/fast/forms/menulist-restrict-line-height-expected.png:
  • platform/gtk/fast/forms/menulist-restrict-line-height-expected.txt:
  • platform/gtk/fast/forms/menulist-separator-painting-expected.png:
  • platform/gtk/fast/forms/menulist-separator-painting-expected.txt:
  • platform/gtk/fast/forms/menulist-style-color-expected.png:
  • platform/gtk/fast/forms/menulist-style-color-expected.txt:
  • platform/gtk/fast/forms/menulist-width-change-expected.png:
  • platform/gtk/fast/forms/menulist-width-change-expected.txt:
  • platform/gtk/fast/forms/number/number-appearance-rtl-expected.png:
  • platform/gtk/fast/forms/number/number-appearance-rtl-expected.txt:
  • platform/gtk/fast/forms/number/number-appearance-spinbutton-disabled-readonly-expected.png:
  • platform/gtk/fast/forms/number/number-appearance-spinbutton-disabled-readonly-expected.txt:
  • platform/gtk/fast/forms/number/number-appearance-spinbutton-layer-expected.png:
  • platform/gtk/fast/forms/number/number-appearance-spinbutton-layer-expected.txt:
  • platform/gtk/fast/forms/number/number-spinbutton-change-and-input-events-expected.txt: Added.
  • platform/gtk/fast/forms/number/number-spinbutton-click-in-iframe-expected.txt: Added.
  • platform/gtk/fast/forms/number/number-spinbutton-gets-disabled-or-readonly-expected.txt: Added.
  • platform/gtk/fast/forms/option-script-expected.png:
  • platform/gtk/fast/forms/option-script-expected.txt:
  • platform/gtk/fast/forms/option-strip-whitespace-expected.png:
  • platform/gtk/fast/forms/option-strip-whitespace-expected.txt:
  • platform/gtk/fast/forms/option-text-clip-expected.png:
  • platform/gtk/fast/forms/option-text-clip-expected.txt:
  • platform/gtk/fast/forms/placeholder-position-expected.png:
  • platform/gtk/fast/forms/placeholder-position-expected.txt:
  • platform/gtk/fast/forms/placeholder-pseudo-style-expected.png:
  • platform/gtk/fast/forms/placeholder-pseudo-style-expected.txt:
  • platform/gtk/fast/forms/range/input-appearance-range-expected.png:
  • platform/gtk/fast/forms/range/input-appearance-range-expected.txt:
  • platform/gtk/fast/forms/range/slider-padding-expected.png:
  • platform/gtk/fast/forms/range/slider-padding-expected.txt:
  • platform/gtk/fast/forms/range/slider-thumb-shared-style-expected.png:
  • platform/gtk/fast/forms/range/slider-thumb-shared-style-expected.txt:
  • platform/gtk/fast/forms/range/slider-transformed-expected.txt:
  • platform/gtk/fast/forms/range/thumbslider-no-parent-slider-expected.png:
  • platform/gtk/fast/forms/range/thumbslider-no-parent-slider-expected.txt:
  • platform/gtk/fast/forms/search-cancel-button-style-sharing-expected.png:
  • platform/gtk/fast/forms/search-cancel-button-style-sharing-expected.txt:
  • platform/gtk/fast/forms/search-display-none-cancel-button-expected.png:
  • platform/gtk/fast/forms/search-display-none-cancel-button-expected.txt:
  • platform/gtk/fast/forms/search-input-rtl-expected.png:
  • platform/gtk/fast/forms/search-input-rtl-expected.txt: Added.
  • platform/gtk/fast/forms/search-rtl-expected.png:
  • platform/gtk/fast/forms/search-rtl-expected.txt:
  • platform/gtk/fast/forms/search-styled-expected.png:
  • platform/gtk/fast/forms/search-styled-expected.txt:
  • platform/gtk/fast/forms/search/search-size-with-decorations-expected.png:
  • platform/gtk/fast/forms/search/search-size-with-decorations-expected.txt:
  • platform/gtk/fast/forms/searchfield-heights-expected.png:
  • platform/gtk/fast/forms/searchfield-heights-expected.txt:
  • platform/gtk/fast/forms/select-align-expected.png:
  • platform/gtk/fast/forms/select-align-expected.txt:
  • platform/gtk/fast/forms/select-background-none-expected.png:
  • platform/gtk/fast/forms/select-background-none-expected.txt:
  • platform/gtk/fast/forms/select-baseline-expected.png:
  • platform/gtk/fast/forms/select-baseline-expected.txt:
  • platform/gtk/fast/forms/select-change-listbox-to-popup-expected.png:
  • platform/gtk/fast/forms/select-change-listbox-to-popup-expected.txt:
  • platform/gtk/fast/forms/select-dirty-parent-pref-widths-expected.png:
  • platform/gtk/fast/forms/select-dirty-parent-pref-widths-expected.txt:
  • platform/gtk/fast/forms/select-disabled-appearance-expected.png:
  • platform/gtk/fast/forms/select-disabled-appearance-expected.txt:
  • platform/gtk/fast/forms/select-element-focus-ring-expected.png: Added.
  • platform/gtk/fast/forms/select-element-focus-ring-expected.txt:
  • platform/gtk/fast/forms/select-initial-position-expected.png:
  • platform/gtk/fast/forms/select-initial-position-expected.txt:
  • platform/gtk/fast/forms/select-non-native-rendering-direction-expected.png: Added.
  • platform/gtk/fast/forms/select-non-native-rendering-direction-expected.txt: Added.
  • platform/gtk/fast/forms/select-selected-expected.png:
  • platform/gtk/fast/forms/select-selected-expected.txt:
  • platform/gtk/fast/forms/select-style-expected.png:
  • platform/gtk/fast/forms/select-style-expected.txt:
  • platform/gtk/fast/forms/select-visual-hebrew-expected.png:
  • platform/gtk/fast/forms/select-visual-hebrew-expected.txt:
  • platform/gtk/fast/forms/select-writing-direction-natural-expected.png:
  • platform/gtk/fast/forms/select-writing-direction-natural-expected.txt:
  • platform/gtk/fast/forms/select/optgroup-rendering-expected.png:
  • platform/gtk/fast/forms/select/optgroup-rendering-expected.txt:
  • platform/gtk/fast/forms/selectlist-minsize-expected.png:
  • platform/gtk/fast/forms/selectlist-minsize-expected.txt:
  • platform/gtk/fast/forms/stuff-on-my-optgroup-expected.png:
  • platform/gtk/fast/forms/stuff-on-my-optgroup-expected.txt:
  • platform/gtk/fast/forms/targeted-frame-submission-expected.png:
  • platform/gtk/fast/forms/targeted-frame-submission-expected.txt:
  • platform/gtk/fast/forms/validation-message-appearance-expected.png:
  • platform/gtk/fast/forms/validation-message-appearance-expected.txt:
  • platform/gtk/fast/html/details-replace-summary-child-expected.png:
  • platform/gtk/fast/html/details-replace-summary-child-expected.txt:
  • platform/gtk/fast/html/details-replace-text-expected.png:
  • platform/gtk/fast/html/details-replace-text-expected.txt:
  • platform/gtk/fast/html/keygen-expected.png:
  • platform/gtk/fast/html/keygen-expected.txt:
  • platform/gtk/fast/inline/positionedLifetime-expected.png:
  • platform/gtk/fast/inline/positionedLifetime-expected.txt:
  • platform/gtk/fast/invalid/014-expected.png:
  • platform/gtk/fast/invalid/014-expected.txt:
  • platform/gtk/fast/overflow/scroll-nested-positioned-layer-in-overflow-expected.png:
  • platform/gtk/fast/overflow/scroll-nested-positioned-layer-in-overflow-expected.txt:
  • platform/gtk/fast/overflow/scrollRevealButton-expected.png:
  • platform/gtk/fast/overflow/scrollRevealButton-expected.txt:
  • platform/gtk/fast/parser/bad-xml-slash-expected.png:
  • platform/gtk/fast/parser/bad-xml-slash-expected.txt:
  • platform/gtk/fast/parser/document-write-option-expected.png:
  • platform/gtk/fast/parser/document-write-option-expected.txt:
  • platform/gtk/fast/repaint/search-field-cancel-expected.png:
  • platform/gtk/fast/repaint/search-field-cancel-expected.txt:
  • platform/gtk/fast/repaint/slider-thumb-drag-release-expected.png:
  • platform/gtk/fast/repaint/slider-thumb-drag-release-expected.txt:
  • platform/gtk/fast/replaced/replaced-breaking-expected.png:
  • platform/gtk/fast/replaced/replaced-breaking-expected.txt:
  • platform/gtk/fast/replaced/replaced-breaking-mixture-expected.png:
  • platform/gtk/fast/replaced/replaced-breaking-mixture-expected.txt:
  • platform/gtk/fast/replaced/three-selects-break-expected.png:
  • platform/gtk/fast/replaced/three-selects-break-expected.txt:
  • platform/gtk/fast/replaced/width100percent-button-expected.png:
  • platform/gtk/fast/replaced/width100percent-button-expected.txt:
  • platform/gtk/fast/replaced/width100percent-menulist-expected.png:
  • platform/gtk/fast/replaced/width100percent-menulist-expected.txt:
  • platform/gtk/fast/replaced/width100percent-searchfield-expected.png:
  • platform/gtk/fast/replaced/width100percent-searchfield-expected.txt:
  • platform/gtk/fast/scrolling/overflow-scrollable-after-back-expected.txt: Added.
  • platform/gtk/fast/table/append-cells2-expected.png:
  • platform/gtk/fast/table/append-cells2-expected.txt:
  • platform/gtk/fast/table/remove-td-display-none-expected.png:
  • platform/gtk/fast/table/remove-td-display-none-expected.txt:
  • platform/gtk/fast/text/international/bidi-menulist-expected.png:
  • platform/gtk/fast/text/international/bidi-menulist-expected.txt:
  • platform/gtk/fast/text/international/hindi-spacing-expected.png:
  • platform/gtk/fast/text/international/hindi-spacing-expected.txt:
  • platform/gtk/fast/text/international/pop-up-button-text-alignment-and-direction-expected.png:
  • platform/gtk/fast/text/international/pop-up-button-text-alignment-and-direction-expected.txt:
  • platform/gtk/fast/text/textIteratorNilRenderer-expected.png:
  • platform/gtk/fast/text/textIteratorNilRenderer-expected.txt:
  • platform/gtk/fast/text/whitespace/normal-after-nowrap-breaking-expected.png:
  • platform/gtk/fast/text/whitespace/normal-after-nowrap-breaking-expected.txt:
  • platform/gtk/http/tests/navigation/javascriptlink-frames-expected.png:
  • platform/gtk/http/tests/navigation/javascriptlink-frames-expected.txt:
  • platform/gtk/svg/custom/foreign-object-skew-expected.png:
  • platform/gtk/svg/custom/foreign-object-skew-expected.txt:
  • platform/gtk/svg/custom/inline-svg-in-xhtml-expected.png:
  • platform/gtk/svg/custom/inline-svg-in-xhtml-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug1188-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug1188-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug1318-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug1318-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug138725-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug138725-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug18359-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug18359-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug2479-2-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug2479-2-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug2479-3-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug2479-3-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug2479-4-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug26178-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug26178-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug28928-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug28928-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug29326-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug29326-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug33855-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug33855-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug39209-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug39209-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug4382-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug4382-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug4429-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug4429-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug44505-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug44505-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug4527-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug4527-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug46368-1-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug46368-1-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug46368-2-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug46368-2-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug51037-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug51037-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug51727-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug51727-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug52505-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug52505-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug52506-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug52506-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug60749-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug60749-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug68912-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug68912-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug7342-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug7342-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug92647-2-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug92647-2-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug96334-expected.png:
  • platform/gtk/tables/mozilla/bugs/bug96334-expected.txt:
  • platform/gtk/tables/mozilla/collapsing_borders/bug41262-4-expected.png:
  • platform/gtk/tables/mozilla/collapsing_borders/bug41262-4-expected.txt:
  • platform/gtk/tables/mozilla/core/margins-expected.png:
  • platform/gtk/tables/mozilla/core/margins-expected.txt:
  • platform/gtk/tables/mozilla/dom/tableDom-expected.png:
  • platform/gtk/tables/mozilla/dom/tableDom-expected.txt:
  • platform/gtk/tables/mozilla/other/move_row-expected.png:
  • platform/gtk/tables/mozilla/other/move_row-expected.txt:
  • platform/gtk/tables/mozilla_expected_failures/bugs/bug1725-expected.png:
  • platform/gtk/tables/mozilla_expected_failures/bugs/bug1725-expected.txt:
  • platform/gtk/tables/mozilla_expected_failures/bugs/bug2479-5-expected.png:
  • platform/gtk/tables/mozilla_expected_failures/bugs/bug2479-5-expected.txt:
  • platform/gtk/tables/mozilla_expected_failures/bugs/bug58402-2-expected.png:
  • platform/gtk/tables/mozilla_expected_failures/bugs/bug58402-2-expected.txt:
  • platform/gtk/tables/mozilla_expected_failures/collapsing_borders/bug41262-5-expected.png:
  • platform/gtk/tables/mozilla_expected_failures/collapsing_borders/bug41262-5-expected.txt:
  • platform/gtk/tables/mozilla_expected_failures/collapsing_borders/bug41262-6-expected.png:
  • platform/gtk/tables/mozilla_expected_failures/collapsing_borders/bug41262-6-expected.txt:
  • platform/gtk/tables/mozilla_expected_failures/core/captions1-expected.png:
  • platform/gtk/tables/mozilla_expected_failures/core/captions1-expected.txt:
  • platform/gtk/tables/mozilla_expected_failures/core/captions2-expected.png:
  • platform/gtk/tables/mozilla_expected_failures/core/captions2-expected.txt:
  • platform/gtk/transforms/2d/zoom-menulist-expected.png:
  • platform/gtk/transforms/2d/zoom-menulist-expected.txt:
12:34 PM Changeset in webkit [217673] by commit-queue@webkit.org
  • 2 edits
    1 add in trunk/Tools

[GTK] Bump wayland and weston in jhbuild
https://bugs.webkit.org/show_bug.cgi?id=171921

Patch by Carlos Garcia Campos <cgarcia@igalia.com> on 2017-06-01
Reviewed by Carlos Alberto Lopez Perez.

  • gtk/jhbuild.modules:
  • gtk/patches/weston-libweston-include-weston-egl-ext.h-in-drm-x11-and-wa.patch: Added.
12:14 PM Changeset in webkit [217672] by aestes@apple.com
  • 12 edits in trunk

REGRESSION (r217626): ENABLE_APPLE_PAY_SESSION_V3 was disabled by mistake
https://bugs.webkit.org/show_bug.cgi?id=172828

Reviewed by Beth Dakin.

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:
12:06 PM Changeset in webkit [217671] by Jonathan Bedard
  • 2 edits in trunk/Tools

webkitpy: Remove unneeded simulator teardown
https://bugs.webkit.org/show_bug.cgi?id=171891

Reviewed by Aakash Jain.

  • Scripts/webkitpy/port/ios_simulator.py:

(IOSSimulatorPort.clean_up_test_run): Remove FIFO deletion since TCP is used
for communication between Python and WebKitTestRunner/DumpRenderTree.

11:43 AM Changeset in webkit [217670] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Mark two webrtc tests as flaky.

Unreviewed test gardening.

11:28 AM Changeset in webkit [217669] by keith_miller@apple.com
  • 24 edits
    1 copy in trunk

Undo rollout in r217638 with bug fix
https://bugs.webkit.org/show_bug.cgi?id=172824

Unreviewed, reland patch with unused set_state code removed.

Source/JavaScriptCore:

  • API/tests/ExecutionTimeLimitTest.cpp:

(dispatchTermitateCallback):
(testExecutionTimeLimit):

  • runtime/JSLock.cpp:

(JSC::JSLock::didAcquireLock):

  • runtime/Options.cpp:

(JSC::overrideDefaults):
(JSC::Options::initialize):

  • runtime/Options.h:
  • runtime/VMTraps.cpp:

(JSC::SignalContext::SignalContext):
(JSC::SignalContext::adjustPCToPointToTrappingInstruction):
(JSC::installSignalHandler):
(JSC::VMTraps::SignalSender::send):

  • tools/SigillCrashAnalyzer.cpp:

(JSC::SignalContext::SignalContext):
(JSC::SignalContext::dump):
(JSC::installCrashHandler):

  • wasm/WasmBBQPlan.cpp:

(JSC::Wasm::BBQPlan::compileFunctions):

  • wasm/WasmFaultSignalHandler.cpp:

(JSC::Wasm::trapHandler):
(JSC::Wasm::enableFastMemory):

  • wasm/WasmMachineThreads.cpp:

(JSC::Wasm::resetInstructionCacheOnAllThreads):

Source/WTF:

  • Configurations/WTF.xcconfig:
  • WTF.xcodeproj/project.pbxproj:
  • wtf/Platform.h:
  • wtf/PlatformRegisters.h:

(WTF::registersFromUContext):

  • wtf/StackBounds.h:

(WTF::StackBounds::StackBounds):

  • wtf/ThreadHolder.cpp:

(WTF::ThreadHolder::~ThreadHolder):

  • wtf/ThreadMessage.cpp:

(WTF::sendMessageUsingSignal):
(WTF::sendMessageUsingMach):
(WTF::deliverMessagesUsingMach):
(WTF::sendMessageScoped):

  • wtf/ThreadMessage.h:

(WTF::sendMessage):

  • wtf/Threading.h:

(WTF::Thread::machThread):

  • wtf/mac/MachExceptions.defs: Copied from Source/WTF/wtf/ThreadMessage.h.
  • wtf/threads/Signals.cpp:

(WTF::startMachExceptionHandlerThread):
(WTF::fromMachException):
(WTF::toMachMask):
(WTF::handleSignalsWithMach):
(WTF::setExceptionPorts):
(WTF::activeThreads):
(WTF::registerThreadForMachExceptionHandling):
(WTF::unregisterThreadForMachExceptionHandling):
(WTF::installSignalHandler):
(WTF::jscSignalHandler):

  • wtf/threads/Signals.h:

Tools:

  • TestWebKitAPI/Tests/WTF/ThreadMessages.cpp:

(runThreadMessageTest):
(TEST):

11:21 AM Changeset in webkit [217668] by Matt Lewis
  • 2 edits in trunk/LayoutTests

Adjusted loader/stateobjects/replacestate-size-iframe.html Test Expectations.
https://bugs.webkit.org/show_bug.cgi?id=172054

Unreviewed test gardening.

  • platform/mac-wk2/TestExpectations:
11:17 AM Changeset in webkit [217667] by Matt Lewis
  • 2 edits in trunk/LayoutTests

Marked loader/stateobjects/replacestate-size-iframe.html as flaky.
https://bugs.webkit.org/show_bug.cgi?id=172054

Unreviewed test gardening.

  • platform/mac-wk2/TestExpectations:
11:04 AM Changeset in webkit [217666] by fred.wang@free.fr
  • 3 edits in trunk/Source/WebCore

ScrollingCoordinator::updateSynchronousScrollingReasons should consider non-main frames
https://bugs.webkit.org/show_bug.cgi?id=172806

Patch by Frederic Wang <fwang@igalia.com> on 2017-06-01
Reviewed by Simon Fraser.

Currently, the scrolling coordinator does not manage scrolling of frames other than the main
one (bugs 171667 and 149264) and so ScrollingCoordinator::updateSynchronousScrollingReasons
is only relevant for the main frame. This commit refactors a bit ScrollingCoordinator so that
updateSynchronousScrollingReasons is considered for all the frames whose scrolling is
coordinated by the ScrollingCoordinator so non-main frames will be considered in the future.

No new tests, behavior is unchanged for now.

  • page/scrolling/ScrollingCoordinator.cpp:

(WebCore::ScrollingCoordinator::updateSynchronousScrollingReasons): Add an ASSERT to ensure
that this function is only called for frame whose scrolling is coordinated by this class.
(WebCore::ScrollingCoordinator::updateSynchronousScrollingReasonsForAllFrames): Introduce a
new function to update synchronous scrolling for subframes whose scrolling is coordinated by
this class.
(WebCore::ScrollingCoordinator::setForceSynchronousScrollLayerPositionUpdates): Update
synchronous scrolling reason for non-main frames too.
(WebCore::ScrollingCoordinator::replaySessionStateDidChange): Update synchronous scrolling
reason for non-main frames too.

  • page/scrolling/ScrollingCoordinator.h: Define updateSynchronousScrollingReasonsForAllFrames
11:01 AM Changeset in webkit [217665] by Chris Dumez
  • 2 edits in trunk/Source/WebKit2

[WK2] Update plugin process sandbox profile for flash plugin on HBONow
https://bugs.webkit.org/show_bug.cgi?id=172820
<rdar://problem/32513297>

Reviewed by Brent Fulgham.

Relax the IOKit whitelist to silence sandbox violations by Flash plugin on
HBONow.

  • PluginProcess/mac/com.apple.WebKit.plugin-common.sb.in:
10:59 AM Changeset in webkit [217664] by aestes@apple.com
  • 4 edits in trunk/Source

[Cocoa] Upstream support for JCB as a supported payment network
https://bugs.webkit.org/show_bug.cgi?id=172819

Reviewed by Tim Horton.

Source/WebCore:

  • Modules/applepay/PaymentRequest.cpp:

(WebCore::PaymentRequest::isValidSupportedNetwork):
(WebCore::isAdditionalValidSupportedNetwork): Deleted.

Source/WebKit2:

  • UIProcess/ApplePay/cocoa/WebPaymentCoordinatorProxyCocoa.mm:

(WebKit::toSupportedNetwork):
(WebKit::toAdditionalSupportedNetwork): Deleted.

10:42 AM Changeset in webkit [217663] by commit-queue@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

[JSC][MIPS] SamplingProfiler::timerLoop() sleeps for 4000+ seconds
https://bugs.webkit.org/show_bug.cgi?id=172800

Patch by Guillaume Emont <guijemont@igalia.com> on 2017-06-01
Reviewed by Saam Barati.

This fixes a static_cast<uint64_t> by making it a cast to int64_t
instead, which looks like the original intent. This fixes the
sampling-profiler tests in JSTests/stress.

  • runtime/SamplingProfiler.cpp:

(JSC::SamplingProfiler::timerLoop):

10:04 AM Changeset in webkit [217662] by Ryan Haddad
  • 3 edits in branches/safari-604.1.21-branch/Tools

Merge r217658.

10:04 AM Changeset in webkit [217661] by Chris Dumez
  • 2 edits in trunk/Source/WebCore

Unreviewed, rolling out r217660.

This should not have landed given that the new assertion is
hit on debug EWS bots

Reverted changeset:

"Make sure we don't access WebCore::defaultPortForProtocol
from non-main thread"
https://bugs.webkit.org/show_bug.cgi?id=172797
http://trac.webkit.org/changeset/217660

9:54 AM Changeset in webkit [217660] by Brent Fulgham
  • 2 edits in trunk/Source/WebCore

Make sure we don't access WebCore::defaultPortForProtocol from non-main thread
https://bugs.webkit.org/show_bug.cgi?id=172797

Reviewed by Alex Christensen.

  • platform/URL.cpp:

(WebCore::defaultPortForProtocol): Add an assertion to make sure we don't access the
DefaultPortForProtocolMapForTesting from multiple threads.

9:38 AM Changeset in webkit [217659] by commit-queue@webkit.org
  • 5 edits in trunk/LayoutTests

Rebaseline two media/modern-media-controls/macos-fullscreen-media-controls tests
https://bugs.webkit.org/show_bug.cgi?id=172810

Patch by Antoine Quint <Antoine Quint> on 2017-06-01
Reviewed by Antoine Quint.

We need to rebaseline a couple of tests to account for the width change for the left button container in fullscreen.

  • media/modern-media-controls/macos-fullscreen-media-controls/macos-fullscreen-media-controls-drag-is-prevented-over-button-expected.txt:
  • media/modern-media-controls/macos-fullscreen-media-controls/macos-fullscreen-media-controls-drag-is-prevented-over-button.html:
  • media/modern-media-controls/macos-fullscreen-media-controls/macos-fullscreen-media-controls-drag-expected.txt:
  • media/modern-media-controls/macos-fullscreen-media-controls/macos-fullscreen-media-controls-drag.html:
9:35 AM Changeset in webkit [217658] by Jonathan Bedard
  • 3 edits in trunk/Tools

webkitpy: Do not send 0 or -1 as a pid to kill_process
https://bugs.webkit.org/show_bug.cgi?id=172818

Reviewed by Darin Adler.

Sending a signal to process 0 will result in all processes in the group receiving
the signal. Sending a signal to process -1 will result in all non-system processes
receiving the signal. Both Executive.kill_process and Executive.check_running_pid
should consider these cases

  • Scripts/webkitpy/common/system/executive.py:

(Executive.kill_process): Throw exception if pid is undefined, 0 or negative.
(Executive.check_running_pid): An undefined pid, pid 0 or a negative pid will
never be running, although os.kill may succeed.

  • Scripts/webkitpy/port/simulator_process.py:

(SimulatorProcess.stop): Do not kill process if it is undefined.

9:22 AM Changeset in webkit [217657] by magomez@igalia.com
  • 2 edits in trunk/LayoutTests

Unreviewed GTK+ gardening. Update expectations of several tests failing after r217651.

  • platform/gtk/TestExpectations:
8:39 AM Changeset in webkit [217656] by commit-queue@webkit.org
  • 1 edit
    2 deletes in trunk/LayoutTests

Remove media/modern-media-controls/ios-inline-media-controls/ios-inline-media-controls-audio-background.html
https://bugs.webkit.org/show_bug.cgi?id=172808

Patch by Antoine Quint <Antoine Quint> on 2017-06-01
Reviewed by Antoine Quint.

This test no longer applies, there is no iOS-specific styling for <audio> elements anymore.

  • media/modern-media-controls/ios-inline-media-controls/ios-inline-media-controls-audio-background.html: Removed.
  • media/modern-media-controls/ios-inline-media-controls/ios-inline-media-controls-audio-background-expected.txt: Removed.
8:04 AM Changeset in webkit [217655] by gskachkov@gmail.com
  • 1 edit
    1 add in trunk/JSTests

Class Proxy can't be extended
https://bugs.webkit.org/show_bug.cgi?id=169040

Reviewed by Saam Barati.

Adding tests that check correctness of Proxy to ES6 class
Bug is already fixed in https://bugs.webkit.org/show_bug.cgi?id=164849

  • stress/proxy-class.js: Added.

(assert):
(throw.new.Error.SuperClass):
(throw.new.Error.A):
(throw.new.Error):

8:00 AM Changeset in webkit [217654] by Ryan Haddad
  • 2 edits in trunk/LayoutTests

Mark imported/w3c/web-platform-tests/css/selectors4/focus-display-none-001.html and focus-within-display-none-001.html as flaky failures
https://bugs.webkit.org/show_bug.cgi?id=172807

Unreviewed test gardening.

  • platform/mac-wk1/TestExpectations:
7:43 AM Changeset in webkit [217653] by commit-queue@webkit.org
  • 3 edits in trunk/LayoutTests

Rebaseline media/modern-media-controls/audio/audio-controls-metrics.html
https://bugs.webkit.org/show_bug.cgi?id=172715

Patch by Antoine Quint <Antoine Quint> on 2017-06-01
Reviewed by Antoine Quint.

We update the expectations for <audio> metrics to match the latest implementation.

  • media/modern-media-controls/audio/audio-controls-metrics-expected.txt:
  • media/modern-media-controls/audio/audio-controls-metrics.html:
7:14 AM Changeset in webkit [217652] by jfernandez@igalia.com
  • 10 edits
    1 copy
    234 adds
    133 deletes in trunk/LayoutTests/imported/w3c

[css-grid] Update W3C web platform tests for the CSS Grid Layout feature
https://bugs.webkit.org/show_bug.cgi?id=172783

Reviewed by Manuel Rego Casasnovas.

Imported the last CSS Grid Layout web platform tests (ed50cf49ed).

  • resources/import-expectations.json:
  • web-platform-tests/css/css-grid-1/OWNERS:
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-001-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-001.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-002-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-002.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-003-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-003.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-004-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-004.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-005-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-005.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-006-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-006.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-007-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-007.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-008-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-008.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-009-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-009.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-010-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-010.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-011-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-011.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-012-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-012.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-013-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-013.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-014-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-014.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-015-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-015.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-016-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-016.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-001-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-001.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-002-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-002.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-003-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-003.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-004-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-004.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-005-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-005.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-006-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-006.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-007-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-007.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-008-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-008.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-009-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-009.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-010-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-010.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-011-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-011.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-012-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-012.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-013-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-013.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-014-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-014.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-015-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-015.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-016-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-lr-016.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-001-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-001.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-002-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-002.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-003-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-003.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-004-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-004.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-005-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-005.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-006-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-006.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-007-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-007.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-008-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-008.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-009-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-009.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-010-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-010.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-011-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-011.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-012-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-012.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-013-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-013.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-014-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-014.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-015-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-015.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-016-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/alignment/grid-self-alignment-stretch-vertical-rl-016.html: Added.
  • web-platform-tests/css/css-grid-1/alignment/w3c-import.log:
  • web-platform-tests/css/css-grid-1/grid-definition/grid-inline-support-flexible-lengths-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-inline-support-flexible-lengths-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-inline-support-grid-template-areas-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-inline-support-grid-template-areas-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-inline-support-grid-template-columns-rows-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-inline-support-grid-template-columns-rows-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-inline-support-named-grid-lines-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-inline-support-named-grid-lines-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-inline-support-repeat-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-inline-support-repeat-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-inline-template-columns-rows-resolved-values-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-inline-template-columns-rows-resolved-values-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-support-flexible-lengths-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-support-flexible-lengths-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-support-grid-template-areas-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-support-grid-template-areas-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-support-grid-template-columns-rows-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-support-grid-template-columns-rows-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-support-named-grid-lines-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-support-named-grid-lines-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-support-repeat-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-support-repeat-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-template-columns-rows-resolved-values-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-definition/grid-template-columns-rows-resolved-values-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-definition/w3c-import.log:
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-items-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-items-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-items-002.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-items-002.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-items-003.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-items-003.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-items-inline-blocks-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-items-inline-blocks-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-001-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-001-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-002-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-002-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-002.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-002.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-003-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-003-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-003.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-003.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-004-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-004-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-004.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-004.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-005-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-005-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-005.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-auto-placement-005.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-painting-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-painting-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-painting-002.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-painting-002.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-painting-003.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-painting-003.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-painting-004.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-painting-004.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-painting-005.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-order-property-painting-005.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-002.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-002.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-003.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-003.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-004.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-004.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-005.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-005.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-001-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-001-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-002-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-002-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-002.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-002.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-003-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-003-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-003.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-003.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-004-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-004-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-004.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-004.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-005-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-005-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-005.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-005.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-006-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-006-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-006.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-inline-z-axis-ordering-overlapped-items-006.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-items-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-items-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-items-002.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-items-002.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-items-003.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-items-003.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-items-inline-blocks-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-items-inline-blocks-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-002.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-002.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-003.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-003.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-004.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-004.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-005.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-005.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-006.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-006.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-007.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-007.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-008.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-008.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-009.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-009.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-021-expected.txt: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-minimum-size-grid-items-021.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-001-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-001-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-002-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-002-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-002.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-002.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-003-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-003-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-003.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-003.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-004-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-004-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-004.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-004.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-005-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-005-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-005.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-auto-placement-005.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-painting-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-painting-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-painting-002.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-painting-002.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-painting-003.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-painting-003.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-painting-004.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-painting-004.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-painting-005.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-order-property-painting-005.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-002.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-002.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-003.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-003.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-004.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-004.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-005.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-005.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-001-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-001-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-002-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-002-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-002.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-002.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-003-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-003-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-003.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-003.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-004-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-004-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-004.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-004.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-005-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-005-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-005.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-005.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-006-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-006-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-006.html: Added.
  • web-platform-tests/css/css-grid-1/grid-items/grid-z-axis-ordering-overlapped-items-006.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-items/support/500x500-green.png: Added.
  • web-platform-tests/css/css-grid-1/grid-items/support/w3c-import.log:
  • web-platform-tests/css/css-grid-1/grid-items/w3c-import.log:
  • web-platform-tests/css/css-grid-1/grid-model/grid-computed-value-display-floated-items-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-computed-value-display-floated-items-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-display-grid-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-display-grid-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-display-inline-grid-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-display-inline-grid-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-letter-001-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-letter-001-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-letter-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-letter-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-letter-002-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-letter-002-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-letter-002.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-letter-002.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-letter-003-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-letter-003-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-letter-003.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-letter-003.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-line-001-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-line-001-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-line-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-line-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-line-002-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-line-002-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-line-002.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-line-002.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-line-003-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-line-003-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-line-003.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-first-line-003.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-float-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-float-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-floats-no-intrude-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-floats-no-intrude-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-letter-001-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-letter-001-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-letter-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-letter-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-letter-002-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-letter-002-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-letter-002.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-letter-002.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-letter-003-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-letter-003-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-letter-003.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-letter-003.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-line-001-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-line-001-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-line-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-line-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-line-002-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-line-002-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-line-002.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-line-002.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-line-003-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-line-003-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-line-003.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-first-line-003.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-float-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-float-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-floats-no-intrude-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-floats-no-intrude-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-margins-no-collapse-001-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-margins-no-collapse-001-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-margins-no-collapse-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-margins-no-collapse-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-multicol-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-multicol-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-vertical-align-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-inline-vertical-align-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-margins-no-collapse-001-expected.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-margins-no-collapse-001-expected.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-margins-no-collapse-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-margins-no-collapse-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-multicol-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-multicol-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-support-display-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-support-display-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/grid-vertical-align-001.html: Added.
  • web-platform-tests/css/css-grid-1/grid-model/grid-vertical-align-001.xht: Removed.
  • web-platform-tests/css/css-grid-1/grid-model/w3c-import.log:
  • web-platform-tests/css/css-grid-1/support/check-layout-th.js: Added.

(checkSubtreeExpectedValues):
(checkAttribute):
(assert_tolerance):
(checkExpectedValues):
(window.checkLayout):

  • web-platform-tests/css/css-grid-1/support/w3c-import.log: Copied from LayoutTests/imported/w3c/web-platform-tests/css/css-grid-1/grid-items/support/w3c-import.log.
2:02 AM Changeset in webkit [217651] by Konstantin Tokarev
  • 3 edits in trunk/Source/WebCore

Get rid of more HIGH_DPI_CANVAS leftovers
https://bugs.webkit.org/show_bug.cgi?id=172751

Reviewed by Simon Fraser.

Get rid of HTMLCanvasElement::convertLogicalToDevice() and
HTMLCanvasElement::convertDeviceToLogical() that are no longer needed,
as canvas doesn't use device scale factor anymore.

No new tests needed.

  • html/HTMLCanvasElement.cpp:

(WebCore::HTMLCanvasElement::createImageBuffer): Use integer width()
and height() instead of floating-point temporary deviceSize variable.
(WebCore::HTMLCanvasElement::baseTransform): Return unmodified
baseTransform of underlying ImageBuffer.

  • html/HTMLCanvasElement.h: Removed declarations of unused methods.
1:22 AM Changeset in webkit [217650] by tpopela@redhat.com
  • 2 edits in trunk/Source/JavaScriptCore

RELEASE_ASSERT_NOT_REACHED() in InferredType::kindForFlags() on Big-Endians
https://bugs.webkit.org/show_bug.cgi?id=170945

Patch by Tomas Popela <tpopela@redhat.com>, Mark Lam <mark.lam@apple.com> on 2017-06-01
Reviewed by Mark Lam.

Re-define PutByIdFlags as a int32_t enum explicitly because it is
stored as an int32_t value in UnlinkedInstruction. This prevents
a bug on 64-bit big endian architectures where the word order is
inverted (when we convert the UnlinkedInstruction into a CodeBlock
Instruction), resulting in the PutByIdFlags value not being stored in
the 32-bit word that the rest of the code expects it to be in.

  • bytecode/PutByIdFlags.h:
12:27 AM Changeset in webkit [217649] by Yusuke Suzuki
  • 3 edits in trunk/LayoutTests

Unreviewed, rebaseline test results according to the error message change
https://bugs.webkit.org/show_bug.cgi?id=172798

  • js/dom/string-prototype-properties-expected.txt:
  • sputnik/Conformance/15_Native_Objects/15.5_String/15.5.4/15.5.4.6_String.prototype.concat/S15.5.4.6_A1_T3-expected.txt:
Note: See TracTimeline for information about the timeline view.