Context Navigation

← Previous Period
Next Period →

Timeline

May 4, 2014:

11:46 PM WebKitGTK/2.4.x edited by Carlos Garcia Campos

(diff)

11:45 PM Changeset in webkit [168258] by Carlos Garcia Campos

2 edits in releases/WebKitGTK/webkit-2.4/Source/WebKit2

Merge r167884 - [GTK][WK2] Missing return statement in webkit_plugin_get_description()
https://bugs.webkit.org/show_bug.cgi?id=132263

Reviewed by Carlos Garcia Campos.

UIProcess/API/gtk/WebKitPlugin.cpp:

(webkit_plugin_get_description): Actually return the data of the cached
plugin description CString.

11:42 PM WebKitGTK/2.4.x edited by Carlos Garcia Campos

(diff)

11:42 PM Changeset in webkit [168257] by Carlos Garcia Campos

2 edits in releases/WebKitGTK/webkit-2.4/Source/WebKit2

Merge r167883 - [GTK] Crash in debug build with removing windowed plugin child widgets from the view
https://bugs.webkit.org/show_bug.cgi?id=132252

Reviewed by Philippe Normand.

It crashes due to an assert in HashTable that checks the iterators
validity. The problem is that we are iterating the children map
and the callback called on every iteration might modify the map,
making the iterators invalid. This happens when the WebView is
destroyed, GtkContainer calls gtk_container_foreach() with
gtk_widget_destroy as callback. When a widget inside a container
is destroyed, it's removed from the container, and in our case,
the child widget is removed from the map. This fixes several
crashes when running layout tests in debug bot.

UIProcess/API/gtk/WebKitWebViewBase.cpp:

(webkitWebViewBaseContainerForall): Use copyKeysToVector() instead
of using a range iterator for the map keys and check in every
iteration that the child widget from the keys vector is still
present in the map before calling the callback.

11:24 PM Changeset in webkit [168256] by akling@apple.com

10 edits in trunk/Source

Optimize JSRopeString for resolving directly to AtomicString.
<https://webkit.org/b/132548>

Source/JavaScriptCore:
If we know that the JSRopeString we are resolving is going to be used
as an AtomicString, we can try to avoid creating a new string.

We do this by first resolving the rope into a stack buffer, and using
that buffer as a key into the AtomicString table. If there is already
an AtomicString with the same characters, we reuse that instead of
constructing a new StringImpl.

JSString gains these two public functions:

AtomicString toAtomicString()

Returns an AtomicString, tries to avoid allocating a new string
if possible.

AtomicStringImpl* toExistingAtomicString()

Returns a non-null AtomicStringImpl* if one already exists in the
AtomicString table. If none is found, the rope is left unresolved.

Reviewed by Filip Pizlo.

runtime/JSString.cpp:

(JSC::JSRopeString::resolveRopeInternal8):
(JSC::JSRopeString::resolveRopeInternal16):
(JSC::JSRopeString::resolveRopeToAtomicString):
(JSC::JSRopeString::clearFibers):
(JSC::JSRopeString::resolveRopeToExistingAtomicString):
(JSC::JSRopeString::resolveRope):
(JSC::JSRopeString::outOfMemory):

runtime/JSString.h:

(JSC::JSString::toAtomicString):
(JSC::JSString::toExistingAtomicString):

Source/WebCore:
Add two bindings generator attributes for parameters to influence
the way that JS rope strings are resolved:

AtomicString

Generates code that avoids allocating a new StringImpl if there
is already an existing AtomicString we can reuse.

RequiresExistingAtomicString

Generates code that fails immediately if the provided string
is not found in the AtomicString table. This is now used for
document.getElementById(), and works because any existing ID
is guaranteed to be in the table.

Reviewed by Filip Pizlo.

bindings/scripts/CodeGeneratorJS.pm:

(GenerateParametersCheck):
(JSValueToNative):

bindings/scripts/IDLAttributes.txt:
dom/Document.idl:

Source/WTF:
Add AtomicString::find([LU]Char*, unsigned length) helpers for finding
an existing AtomicString without a StringImpl on hand.

Reviewed by Filip Pizlo.

wtf/text/AtomicString.h:
wtf/text/AtomicString.cpp:

(WTF::AtomicString::find):

11:10 PM Changeset in webkit [168255] by akling@apple.com

7 edits
9 deletes in trunk

Unreviewed, rolling out r168254.

Very crashy on debug JSC tests.

Reverted changeset:

"jsSubstring() should be lazy"
https://bugs.webkit.org/show_bug.cgi?id=132556
http://trac.webkit.org/changeset/168254

9:54 PM Changeset in webkit [168254] by fpizlo@apple.com

7 edits
9 adds in trunk

jsSubstring() should be lazy
https://bugs.webkit.org/show_bug.cgi?id=132556

Reviewed by Andreas Kling.

Source/JavaScriptCore:
jsSubstring() is now lazy by using a special rope that is a substring instead of a
concatenation. To make this patch super simple, we require that a substring's base is
never a rope. Hence, when resolving a rope, we either go down a non-recursive substring
path, or we go down a concatenation path which may see exactly one level of substrings in
its fibers.

This is up to a 50% speed-up on microbenchmarks and a 10% speed-up on Octane/regexp.

heap/MarkedBlock.cpp:

(JSC::MarkedBlock::specializedSweep):

runtime/JSString.cpp:

(JSC::JSRopeString::visitFibers):
(JSC::JSRopeString::resolveRope):
(JSC::JSRopeString::resolveRopeSlowCase8):
(JSC::JSRopeString::resolveRopeSlowCase):
(JSC::JSRopeString::outOfMemory):

runtime/JSString.h:

(JSC::JSRopeString::finishCreation):
(JSC::JSRopeString::append):
(JSC::JSRopeString::create):
(JSC::JSRopeString::offsetOfFibers):
(JSC::JSRopeString::fiber):
(JSC::JSRopeString::substringBase):
(JSC::JSRopeString::substringOffset):
(JSC::JSRopeString::substringSentinel):
(JSC::JSRopeString::isSubstring):
(JSC::jsSubstring):

runtime/RegExpMatchesArray.cpp:

(JSC::RegExpMatchesArray::reifyAllProperties):

runtime/StringPrototype.cpp:

(JSC::stringProtoFuncSubstring):

LayoutTests:
These tests get 35-50% faster.

js/regress/script-tests/substring-concat-weird.js: Added.

(foo):

js/regress/script-tests/substring-concat.js: Added.

(foo):

js/regress/script-tests/substring.js: Added.

(foo):

js/regress/substring-concat-expected.txt: Added.
js/regress/substring-concat-weird-expected.txt: Added.
js/regress/substring-concat-weird.html: Added.
js/regress/substring-concat.html: Added.
js/regress/substring-expected.txt: Added.
js/regress/substring.html: Added.

8:49 PM Changeset in webkit [168253] by Simon Fraser

2 edits in trunk/Source/WebCore

[iOS WK2] Compositing layers in iframes are misplaced
https://bugs.webkit.org/show_bug.cgi?id=132554
<rdar://problem/16203027>

Reviewed by Benjamin Poulain.

Have requiresScrollLayer() only consider frameView.delegatesScrolling()
for the main frame, so that iframes get scroll layers (even though they
will never scroll), so that the rest of geometry code works as on other
platforms.

rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::requiresScrollLayer):

6:16 PM Changeset in webkit [168252] by gyuyoung.kim@samsung.com

3 edits in trunk/Source/WebCore

Use std::unique_ptr in m_taskTimer of Notification class
https://bugs.webkit.org/show_bug.cgi?id=132544

Reviewed by Andreas Kling.

No new tests, no behavior change.

Modules/notifications/Notification.cpp:

(WebCore::Notification::Notification):

Modules/notifications/Notification.h: Use std::unique_ptr instead of OwnPtr.

5:43 PM Changeset in webkit [168251] by akling@apple.com

2 edits in trunk/LayoutTests

Mark compositing/visibility/visibility-image-layers-dynamic.html as failing after r168244.

TestExpectations:

5:42 PM Changeset in webkit [168250] by psolanki@apple.com

2 edits in trunk/Source/WebKit2

Reduce calls to CFURLCacheCopySharedURLCache
https://bugs.webkit.org/show_bug.cgi?id=132464
<rdar://problem/16806694>

Address review comments by collapsing multi-line code into a single ASSERT.

NetworkProcess/mac/NetworkResourceLoaderMac.mm:

(WebKit::NetworkResourceLoader::tryGetShareableHandleFromSharedBuffer):

5:21 PM Changeset in webkit [168249] by timothy_horton@apple.com

2 edits in trunk/Source/WebCore

Long hangs under IOSurfacePool::evict
https://bugs.webkit.org/show_bug.cgi?id=132549
<rdar://problem/16769469>

Reviewed by Simon Fraser.

platform/graphics/cg/IOSurfacePool.cpp:

(WebCore::IOSurfacePool::willAddSurface):
Run eviction before adding the new surface to m_bytesCached/m_inUseBytesCached.
We use the additionalSize parameter to make space for the new surface.

(WebCore::IOSurfacePool::evict):
If we want to free up the entire pool, we can do so by throwing away everything.
This also avoids an underflow if additionalSize is larger than the maximum pool size.

3:53 PM Changeset in webkit [168248] by psolanki@apple.com

19 edits in trunk/Source

Shortcircuit shouldUseCredentialStorage callback
https://bugs.webkit.org/show_bug.cgi?id=132308
<rdar://problem/16806708>

Reviewed by Alexey Proskuryakov.

If we are going to return true from the shouldUseCredentialStorage callback then we don't
really need to have CFNetwork/Foundation call us. We can just disable the callback and
CFNetwork will assume true. Add a separate subclass that implements this callback when we
need to return false. We can also eliminate the corresponding async callbacks. This avoids
pingponging between dispatch queue and main thread in the common case.

No new tests because no change in functionality.

Source/WebCore:

WebCore.exp.in:
platform/network/ResourceHandle.cpp:
platform/network/ResourceHandle.h:
platform/network/ResourceHandleClient.cpp:
platform/network/ResourceHandleClient.h:
platform/network/cf/ResourceHandleCFNet.cpp:

(WebCore::ResourceHandle::createCFURLConnection):
(WebCore::ResourceHandle::shouldUseCredentialStorage):

platform/network/cf/ResourceHandleCFURLConnectionDelegate.h:
platform/network/cf/ResourceHandleCFURLConnectionDelegateWithOperationQueue.cpp:

(WebCore::ResourceHandleCFURLConnectionDelegateWithOperationQueue::shouldUseCredentialStorage):

platform/network/cf/ResourceHandleCFURLConnectionDelegateWithOperationQueue.h:
platform/network/cf/SynchronousResourceHandleCFURLConnectionDelegate.cpp:
platform/network/cf/SynchronousResourceHandleCFURLConnectionDelegate.h:
platform/network/mac/ResourceHandleMac.mm:

(WebCore::ResourceHandle::start):
(WebCore::ResourceHandle::makeDelegate):
(WebCore::ResourceHandle::delegate):
(WebCore::ResourceHandle::platformLoadResourceSynchronously):
(WebCore::ResourceHandle::shouldUseCredentialStorage):

platform/network/mac/WebCoreResourceHandleAsOperationQueueDelegate.h:
platform/network/mac/WebCoreResourceHandleAsOperationQueueDelegate.mm:

(-[WebCoreResourceHandleWithCredentialStorageAsOperationQueueDelegate connectionShouldUseCredentialStorage:]):

platform/network/soup/ResourceHandleSoup.cpp:

Source/WebKit2:

NetworkProcess/NetworkResourceLoader.cpp: Remove shouldUseCredentialStorageAsync() callbacks.
NetworkProcess/NetworkResourceLoader.h:

3:39 PM Changeset in webkit [168247] by Beth Dakin

3 edits in trunk/LayoutTests

Even more re-baselining for anchor point after
http://trac.webkit.org/changeset/168244

platform/mac-mountainlion/compositing/contents-opaque/control-layer-expected.txt:
platform/mac/compositing/contents-opaque/control-layer-expected.txt:

2:16 PM Changeset in webkit [168246] by Beth Dakin

9 edits in trunk/LayoutTests

Additional re-baselining for anchor point after
http://trac.webkit.org/changeset/168244

compositing/contents-opaque/control-layer-expected.txt:
css3/compositing/blend-mode-accelerated-with-multiple-stacking-contexts-expected.txt:
css3/compositing/blend-mode-ancestor-clipping-layer-expected.txt:
css3/compositing/blend-mode-blended-element-overlapping-composited-sibling-should-have-compositing-layer-expected.txt:
css3/compositing/blend-mode-parent-of-composited-blended-has-layer-expected.txt:
css3/compositing/blend-mode-with-accelerated-sibling-expected.txt:
css3/compositing/blend-mode-with-composited-descendant-should-have-layer-expected.txt:
css3/filters/filtered-compositing-descendant-expected.txt:

2:01 PM Changeset in webkit [168245] by Brent Fulgham

9 edits in trunk/Source/WebCore

[iOS] deviceScaleFactor is being double-applied when rendering captions in full screen mode
https://bugs.webkit.org/show_bug.cgi?id=132481
<rdar://problem/16507482>

Reviewed by Jer Noble.

Add a new 'syncTextTrackBounds' method (and relaying functions) to keep the text track container in sync
with changes to the video player's display layer.

html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::syncTextTrackBounds: Added.

html/HTMLMediaElement.h:
html/shadow/MediaControlElements.cpp:

(WebCore::MediaControlTextTrackContainerElement::updateDisplay): Don't set the platform scale factor here. It is already
being accounted for in the createTextTrackRepresentationImage method.
(WebCore::MediaControlTextTrackContainerElement::updateSizes): Synchronize the text track representation
with any new video layer changes.

platform/graphics/MediaPlayer.cpp:

(WebCore::MediaPlayer::syncTextTrackBounds): Added.

platform/graphics/MediaPlayer.h:
platform/graphics/MediaPlayerPrivate.h:

(WebCore::MediaPlayerPrivateInterface::syncTextTrackBounds): Added.

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

(WebCore::MediaPlayerPrivateAVFoundationObjC::setVideoFullscreenLayer): Use new sync function.
(WebCore::MediaPlayerPrivateAVFoundationObjC::setVideoFullscreenFrame): Ditto.
(WebCore::MediaPlayerPrivateAVFoundationObjC::syncTextTrackBounds): Added. Keep the text track layer size in sync with
the current video layer size. This may change during animations, rotations, etc.
(WebCore::MediaPlayerPrivateAVFoundationObjC::setTextTrackRepresentation): Use new sync function.

1:40 PM Changeset in webkit [168244] by Beth Dakin

245 edits in trunk

Top content inset: Margin tiles should not display in the inset area when pinned
to the top of the page
https://bugs.webkit.org/show_bug.cgi?id=132535
-and corresponding-
<rdar://problem/16613039>

Reviewed by Simon Fraser.

Source/WebCore:
Prior to this change, topContentInset was implemented by positioning
RenderLayerCompositor’s m_rootContentLayer based on the inset value. In order to
ensure that no content is displayed in the inset area when we are pinned to the
top of the page, we’ll have to take a different approach. In this patch, when you
are pinned to the top of the page, the m_rootContentLayer is positioned at (0,0),
much like it would be without an inset, but the m_clip layer IS positioned at (0,
topContentInset). Then for all y-scroll values between 0 and topContentInset, the
positions of the clip layer and the contents layer are adjusted so that the clip
layer approaches a position of (0,0), and the root layer approaches a position of
(0, topContentInset). This makes sure that any content above the top the document
is aways clipped out of the inset area.

In order to achieve this, the scrolling thread needs to know about the
topContentInset, the clip layer, and the root contents layer.

AsyncScrollingCoordinator::updateScrollingNode() now takes an additional parameter
for the clip layer. Also export the topContentInset symbol for UI-side
compositing.

WebCore.exp.in:

Here is the new computation. Implemented in one spot that can be called from the
scrolling thread, AsyncScrollingCoordinator and RenderLayerCompositor.

page/FrameView.cpp:

(WebCore::FrameView::yPositionForInsetClipLayer):
(WebCore::FrameView::yPositionForRootContentLayer):

page/FrameView.h:

Set the topContentInset, the insetClipLayer, and the scrolledContentsLayer when
appropriate.

page/scrolling/AsyncScrollingCoordinator.cpp:

(WebCore::AsyncScrollingCoordinator::frameViewLayoutUpdated):
(WebCore::AsyncScrollingCoordinator::frameViewRootLayerDidChange):

Set or sync the positions for the inset layer and the contents layer.
(WebCore::AsyncScrollingCoordinator::updateScrollPositionAfterAsyncScroll):
(WebCore::AsyncScrollingCoordinator::updateScrollingNode):

page/scrolling/AsyncScrollingCoordinator.h:

Convenience functions for getting the clipLayer() and the rootContentLayer() from
the RenderLayerCompositor.

page/scrolling/ScrollingCoordinator.cpp:

(WebCore::ScrollingCoordinator::insetClipLayerForFrameView):
(WebCore::ScrollingCoordinator::rootContentLayerForFrameView):

page/scrolling/ScrollingCoordinator.h:

(WebCore::ScrollingCoordinator::updateScrollingNode):

ScrollingStateScrolling nodes have two new members now. m_insetClipLayer and
m_topContentInset. We can use m_scrolledContentsLayer for the rootContentsLayer
since previously that member was not used for FrameViews; it was only used for
accelerated overflow:scroll nodes.

page/scrolling/ScrollingStateScrollingNode.cpp:

(WebCore::ScrollingStateScrollingNode::ScrollingStateScrollingNode):
(WebCore::ScrollingStateScrollingNode::setTopContentInset):
(WebCore::ScrollingStateScrollingNode::setInsetClipLayer):

page/scrolling/ScrollingStateScrollingNode.h:

Similarly, ScrollingTreeScrollingNode has two new members for m_topContentInset
and m_clipLayer.

page/scrolling/ScrollingTreeScrollingNode.cpp:

(WebCore::ScrollingTreeScrollingNode::ScrollingTreeScrollingNode):
(WebCore::ScrollingTreeScrollingNode::updateBeforeChildren):

page/scrolling/ScrollingTreeScrollingNode.h:

(WebCore::ScrollingTreeScrollingNode::topContentInset):

page/scrolling/mac/ScrollingTreeScrollingNodeMac.h:
page/scrolling/mac/ScrollingTreeScrollingNodeMac.mm:

(WebCore::ScrollingTreeScrollingNodeMac::updateBeforeChildren):

Compute positions for the two new layers.
(WebCore::ScrollingTreeScrollingNodeMac::setScrollLayerPosition):

Now use the static FrameView functions to compute the positions for these layers.

rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::frameViewDidChangeSize):
(WebCore::RenderLayerCompositor::positionForClipLayer):
(WebCore::RenderLayerCompositor::clipLayer):
(WebCore::RenderLayerCompositor::rootContentLayer):
(WebCore::RenderLayerCompositor::updateRootLayerPosition):
(WebCore::RenderLayerCompositor::updateOverflowControlsLayers):
(WebCore::RenderLayerCompositor::ensureRootLayer):
(WebCore::RenderLayerCompositor::updateScrollCoordinatedLayer):

rendering/RenderLayerCompositor.h:

Source/WebKit2:
Encode and decode the ScrollingStateScrollingNode’s m_topContentInset even though
we don’t use it for anything yet. Since the headerLayer and footerLayer are not
encoded or decoded yet, I did not encode/decode the insetClipLayer yet, which,
like the header and footer layers, would not yet be used.

Shared/Scrolling/RemoteScrollingCoordinatorTransaction.cpp:

(ArgumentCoder<ScrollingStateScrollingNode>::encode):
(ArgumentCoder<ScrollingStateScrollingNode>::decode):

LayoutTests:
Anchor point!

compositing/absolute-inside-out-of-view-fixed-expected.txt:
compositing/animation/filling-animation-overlap-at-end-expected.txt:
compositing/animation/filling-animation-overlap-expected.txt:
compositing/animation/layer-for-filling-animation-expected.txt:
compositing/backing/backface-visibility-in-3dtransformed-expected.txt:
compositing/backing/no-backing-for-clip-expected.txt:
compositing/backing/no-backing-for-clip-overhang-expected.txt:
compositing/backing/no-backing-for-clip-overlap-expected.txt:
compositing/backing/no-backing-for-perspective-expected.txt:
compositing/bounds-in-flipped-writing-mode-expected.txt:
compositing/clip-child-by-non-stacking-ancestor-expected.txt:
compositing/columns/ancestor-clipped-in-paginated-expected.txt:
compositing/columns/clipped-in-paginated-expected.txt:
compositing/columns/composited-columns-expected.txt:
compositing/columns/composited-columns-vertical-rl-expected.txt:
compositing/columns/composited-in-paginated-expected.txt:
compositing/columns/composited-in-paginated-rl-expected.txt:
compositing/columns/composited-in-paginated-writing-mode-rl-expected.txt:
compositing/columns/composited-lr-paginated-repaint-expected.txt:
compositing/columns/composited-nested-columns-expected.txt:
compositing/columns/composited-rl-paginated-repaint-expected.txt:
compositing/columns/hittest-composited-in-paginated-expected.txt:
compositing/columns/rotated-in-paginated-expected.txt:
compositing/columns/untransformed-composited-in-paginated-expected.txt:
compositing/contents-opaque/background-clip-expected.txt:
compositing/contents-opaque/background-color-expected.txt:
compositing/contents-opaque/body-background-painted-expected.txt:
compositing/contents-opaque/body-background-skipped-expected.txt:
compositing/contents-opaque/filter-expected.txt:
compositing/contents-opaque/hidden-with-visible-child-expected.txt:
compositing/contents-opaque/hidden-with-visible-text-expected.txt:
compositing/contents-opaque/layer-opacity-expected.txt:
compositing/contents-opaque/layer-transform-expected.txt:
compositing/contents-opaque/overflow-hidden-child-layers-expected.txt:
compositing/contents-opaque/visibility-hidden-expected.txt:
compositing/contents-scale/animating-expected.txt:
compositing/contents-scale/rounded-contents-scale-expected.txt:
compositing/contents-scale/scaled-ancestor-expected.txt:
compositing/contents-scale/simple-scale-expected.txt:
compositing/contents-scale/z-translate-expected.txt:
compositing/filters/sw-layer-overlaps-hw-shadow-expected.txt:
compositing/filters/sw-nested-shadow-overlaps-hw-nested-shadow-expected.txt:
compositing/filters/sw-shadow-overlaps-hw-layer-expected.txt:
compositing/filters/sw-shadow-overlaps-hw-shadow-expected.txt:
compositing/geometry/ancestor-overflow-change-expected.txt:
compositing/geometry/bounds-clipped-composited-child-expected.txt:
compositing/geometry/bounds-ignores-hidden-composited-descendant-expected.txt:
compositing/geometry/bounds-ignores-hidden-dynamic-expected.txt:
compositing/geometry/bounds-ignores-hidden-dynamic-negzindex-expected.txt:
compositing/geometry/bounds-ignores-hidden-expected.txt:
compositing/geometry/clip-expected.txt:
compositing/geometry/clip-inside-expected.txt:
compositing/geometry/composited-in-columns-expected.txt:
compositing/geometry/fixed-position-flipped-writing-mode-expected.txt:
compositing/geometry/flipped-writing-mode-expected.txt:
compositing/geometry/foreground-layer-expected.txt:
compositing/geometry/layer-due-to-layer-children-deep-switch-expected.txt:
compositing/geometry/layer-due-to-layer-children-switch-expected.txt:
compositing/geometry/limit-layer-bounds-clipping-ancestor-expected.txt:
compositing/geometry/limit-layer-bounds-fixed-expected.txt:
compositing/geometry/limit-layer-bounds-fixed-positioned-expected.txt:
compositing/geometry/limit-layer-bounds-opacity-transition-expected.txt:
compositing/geometry/limit-layer-bounds-overflow-root-expected.txt:
compositing/geometry/limit-layer-bounds-positioned-expected.txt:
compositing/geometry/limit-layer-bounds-positioned-transition-expected.txt:
compositing/geometry/limit-layer-bounds-transformed-expected.txt:
compositing/geometry/limit-layer-bounds-transformed-overflow-expected.txt:
compositing/geometry/negative-text-indent-with-overflow-hidden-layer-expected.txt:
compositing/geometry/preserve-3d-switching-expected.txt:
compositing/iframes/become-composited-nested-iframes-expected.txt:
compositing/iframes/become-overlapped-iframe-expected.txt:
compositing/iframes/composited-parent-iframe-expected.txt:
compositing/iframes/connect-compositing-iframe-delayed-expected.txt:
compositing/iframes/connect-compositing-iframe-expected.txt:
compositing/iframes/connect-compositing-iframe2-expected.txt:
compositing/iframes/connect-compositing-iframe3-expected.txt:
compositing/iframes/enter-compositing-iframe-expected.txt:
compositing/iframes/iframe-resize-expected.txt:
compositing/iframes/iframe-size-from-zero-expected.txt:
compositing/iframes/invisible-nested-iframe-hide-expected.txt:
compositing/iframes/overlapped-iframe-expected.txt:
compositing/iframes/overlapped-iframe-iframe-expected.txt:
compositing/iframes/overlapped-nested-iframes-expected.txt:
compositing/iframes/page-cache-layer-tree-expected.txt:
compositing/iframes/scrolling-iframe-expected.txt:
compositing/images/clip-on-directly-composited-image-expected.txt:
compositing/layer-creation/animation-overlap-with-children-expected.txt:
compositing/layer-creation/fixed-position-and-transform-expected.txt:
compositing/layer-creation/fixed-position-change-out-of-view-in-view-expected.txt:
compositing/layer-creation/fixed-position-out-of-view-expected.txt:
compositing/layer-creation/fixed-position-out-of-view-scaled-expected.txt:
compositing/layer-creation/fixed-position-out-of-view-scaled-scroll-expected.txt:
compositing/layer-creation/fixed-position-transformed-into-view-expected.txt:
compositing/layer-creation/fixed-position-under-transform-expected.txt:
compositing/layer-creation/no-compositing-for-fixed-position-under-transform-expected.txt:
compositing/layer-creation/no-compositing-for-preserve-3d-expected.txt:
compositing/layer-creation/overflow-scroll-overlap-expected.txt:
compositing/layer-creation/overlap-animation-clipping-expected.txt:
compositing/layer-creation/overlap-animation-expected.txt:
compositing/layer-creation/overlap-child-layer-expected.txt:
compositing/layer-creation/overlap-clipping-expected.txt:
compositing/layer-creation/overlap-transformed-3d-expected.txt:
compositing/layer-creation/overlap-transformed-and-clipped-expected.txt:
compositing/layer-creation/overlap-transformed-layer-expected.txt:
compositing/layer-creation/overlap-transformed-preserved-3d-expected.txt:
compositing/layer-creation/overlap-transforms-expected.txt:
compositing/layer-creation/rotate3d-overlap-expected.txt:
compositing/layer-creation/scroll-partial-update-expected.txt:
compositing/layer-creation/spanOverlapsCanvas-expected.txt:
compositing/layer-creation/stacking-context-overlap-expected.txt:
compositing/layer-creation/stacking-context-overlap-nested-expected.txt:
compositing/layer-creation/translatez-overlap-expected.txt:
compositing/masks/mask-layer-size-expected.txt:
compositing/overflow-trumps-transform-style-expected.txt:
compositing/overflow/clip-descendents-expected.txt:
compositing/overflow/content-gains-scrollbars-expected.txt:
compositing/overflow/content-loses-scrollbars-expected.txt:
compositing/overflow/overflow-scrollbar-layer-positions-expected.txt:
compositing/overflow/overflow-scrollbar-layers-expected.txt:
compositing/overflow/resize-painting-expected.txt:
compositing/plugins/no-backing-store-expected.txt:
compositing/plugins/small-to-large-composited-plugin-expected.txt:
compositing/regions/fixed-in-named-flow-clip-descendant-expected.txt:
compositing/regions/fixed-in-named-flow-expected.txt:
compositing/regions/fixed-in-named-flow-from-abs-in-named-flow-expected.txt:
compositing/regions/fixed-in-named-flow-from-outflow-expected.txt:
compositing/regions/fixed-in-named-flow-got-transformed-parent-expected.txt:
compositing/regions/fixed-in-named-flow-lost-transformed-parent-expected.txt:
compositing/regions/fixed-in-named-flow-overlap-composited-expected.txt:
compositing/regions/fixed-in-named-flow-transformed-parent-expected.txt:
compositing/regions/fixed-transformed-in-named-flow-expected.txt:
compositing/repaint/absolute-painted-into-composited-ancestor-expected.txt:
compositing/repaint/fixed-background-scroll-expected.txt:
compositing/repaint/positioned-movement-expected.txt:
compositing/repaint/repaint-on-layer-grouping-change-expected.txt:
compositing/repaint/resize-repaint-expected.txt:
compositing/rtl/rtl-absolute-expected.txt:
compositing/rtl/rtl-absolute-overflow-expected.txt:
compositing/rtl/rtl-absolute-overflow-scrolled-expected.txt:
compositing/rtl/rtl-fixed-expected.txt:
compositing/rtl/rtl-fixed-overflow-expected.txt:
compositing/rtl/rtl-fixed-overflow-scrolled-expected.txt:
compositing/rtl/rtl-relative-expected.txt:
compositing/tiled-layers-hidpi-expected.txt:
compositing/visibility/layer-visible-content-expected.txt:
platform/mac-wk2/compositing/contents-opaque/body-background-painted-expected.txt:
platform/mac-wk2/compositing/contents-opaque/body-background-skipped-expected.txt:
platform/mac-wk2/compositing/repaint/fixed-background-scroll-expected.txt:
platform/mac-wk2/compositing/rtl/rtl-absolute-expected.txt:
platform/mac-wk2/compositing/rtl/rtl-absolute-overflow-expected.txt:
platform/mac-wk2/compositing/rtl/rtl-absolute-overflow-scrolled-expected.txt:
platform/mac-wk2/compositing/rtl/rtl-fixed-expected.txt:
platform/mac-wk2/compositing/rtl/rtl-fixed-overflow-expected.txt:
platform/mac-wk2/compositing/tiling/rotated-tiled-clamped-expected.txt:
platform/mac-wk2/compositing/tiling/rotated-tiled-preserve3d-clamped-expected.txt:
platform/mac-wk2/compositing/tiling/tile-cache-zoomed-expected.txt:
platform/mac-wk2/compositing/tiling/tiled-layer-resize-expected.txt:
platform/mac-wk2/compositing/visible-rect/iframe-no-layers-expected.txt:
platform/mac-wk2/tiled-drawing/fast-scroll-div-latched-div-expected.txt:
platform/mac-wk2/tiled-drawing/fast-scroll-div-latched-div-with-handler-expected.txt:
platform/mac-wk2/tiled-drawing/fast-scroll-div-latched-mainframe-expected.txt:
platform/mac-wk2/tiled-drawing/fast-scroll-div-latched-mainframe-with-handler-expected.txt:
platform/mac-wk2/tiled-drawing/fast-scroll-iframe-latched-iframe-expected.txt:
platform/mac-wk2/tiled-drawing/fast-scroll-iframe-latched-iframe-with-handler-expected.txt:
platform/mac-wk2/tiled-drawing/fast-scroll-iframe-latched-mainframe-expected.txt:
platform/mac-wk2/tiled-drawing/fast-scroll-iframe-latched-mainframe-with-handler-expected.txt:
platform/mac-wk2/tiled-drawing/fast-scroll-select-latched-mainframe-expected.txt:
platform/mac-wk2/tiled-drawing/fast-scroll-select-latched-mainframe-with-handler-expected.txt:
platform/mac-wk2/tiled-drawing/fast-scroll-select-latched-select-expected.txt:
platform/mac-wk2/tiled-drawing/fast-scroll-select-latched-select-with-handler-expected.txt:
platform/mac-wk2/tiled-drawing/fixed-background/fixed-background-no-image-expected.txt:
platform/mac-wk2/tiled-drawing/fixed-background/fixed-body-background-body-layer-expected.txt:
platform/mac-wk2/tiled-drawing/fixed-background/fixed-body-background-expected.txt:
platform/mac-wk2/tiled-drawing/fixed-background/fixed-body-background-opacity-html-expected.txt:
platform/mac-wk2/tiled-drawing/fixed-background/fixed-body-background-positioned-expected.txt:
platform/mac-wk2/tiled-drawing/fixed-background/fixed-body-background-transformed-html-expected.txt:
platform/mac-wk2/tiled-drawing/fixed-background/fixed-body-background-zoomed-expected.txt:
platform/mac-wk2/tiled-drawing/fixed-background/fixed-html-background-expected.txt:
platform/mac-wk2/tiled-drawing/fixed-background/fixed-non-propagated-body-background-expected.txt:
platform/mac-wk2/tiled-drawing/simple-document-with-margin-tiles-expected.txt:
platform/mac-wk2/tiled-drawing/sticky/sticky-layers-expected.txt:
platform/mac-wk2/tiled-drawing/tile-coverage-after-scroll-expected.txt:
platform/mac-wk2/tiled-drawing/tile-coverage-after-scroll-speculative-expected.txt:
platform/mac-wk2/tiled-drawing/tile-coverage-scroll-to-bottom-expected.txt:
platform/mac-wk2/tiled-drawing/tile-coverage-slow-scrolling-expected.txt:
platform/mac-wk2/tiled-drawing/tile-coverage-speculative-expected.txt:
platform/mac-wk2/tiled-drawing/tile-size-slow-zoomed-expected.txt:
platform/mac-wk2/tiled-drawing/tiled-drawing-scroll-position-page-cache-restoration-expected.txt:
platform/mac-wk2/tiled-drawing/tiled-drawing-zoom-expected.txt:
platform/mac-wk2/tiled-drawing/tiled-drawing-zoom-scrolled-expected.txt:
platform/mac-wk2/tiled-drawing/use-tiled-drawing-expected.txt:
platform/mac/compositing/canvas/accelerated-canvas-compositing-expected.txt:
platform/mac/compositing/geometry/fixed-position-composited-switch-expected.txt:
platform/mac/compositing/iframes/invisible-nested-iframe-show-expected.txt:
platform/mac/compositing/iframes/resizer-expected.txt:
platform/mac/compositing/images/direct-image-object-fit-expected.txt:
platform/mac/compositing/layer-creation/overlap-animation-container-expected.txt:
platform/mac/compositing/overflow/clipping-behaviour-change-is-not-propagated-to-descendants-expected.txt:
platform/mac/compositing/overflow/clipping-behaviour-change-is-not-propagated-to-descendants2-expected.txt:
platform/mac/compositing/overflow/composited-scrolling-creates-a-stacking-container-expected.txt:
platform/mac/compositing/overflow/composited-scrolling-paint-phases-expected.txt:
platform/mac/compositing/reflections/direct-image-object-fit-reflected-expected.txt:
platform/mac/compositing/repaint/invalidations-on-composited-layers-expected.txt:
platform/mac/compositing/tiling/backface-preserve-3d-tiled-expected.txt:
platform/mac/compositing/tiling/crash-reparent-tiled-layer-expected.txt:
platform/mac/compositing/tiling/huge-layer-add-remove-child-expected.txt:
platform/mac/compositing/tiling/huge-layer-img-expected.txt:
platform/mac/compositing/tiling/huge-layer-with-layer-children-expected.txt:
platform/mac/compositing/tiling/huge-layer-with-layer-children-resize-expected.txt:
platform/mac/compositing/tiling/rotated-tiled-clamped-expected.txt:
platform/mac/compositing/tiling/rotated-tiled-preserve3d-clamped-expected.txt:
platform/mac/compositing/tiling/tile-cache-zoomed-expected.txt:
platform/mac/compositing/tiling/tiled-layer-resize-expected.txt:
platform/mac/compositing/video/video-object-fit-expected.txt:
platform/mac/compositing/visible-rect/2d-transformed-expected.txt:
platform/mac/compositing/visible-rect/3d-transform-style-expected.txt:
platform/mac/compositing/visible-rect/3d-transformed-expected.txt:
platform/mac/compositing/visible-rect/animated-expected.txt:
platform/mac/compositing/visible-rect/animated-from-none-expected.txt:
platform/mac/compositing/visible-rect/clipped-by-viewport-expected.txt:
platform/mac/compositing/visible-rect/clipped-visible-rect-expected.txt:
platform/mac/compositing/visible-rect/flipped-preserve-3d-expected.txt:
platform/mac/compositing/visible-rect/iframe-and-layers-expected.txt:
platform/mac/compositing/visible-rect/iframe-no-layers-expected.txt:
platform/mac/compositing/visible-rect/nested-transform-expected.txt:
platform/mac/compositing/visible-rect/scrolled-expected.txt:

12:44 PM Changeset in webkit [168243] by benjamin@webkit.org

6 edits in trunk/Source/WebCore

Clear the Selector Query caches on memory pressure
https://bugs.webkit.org/show_bug.cgi?id=132545

Reviewed by Andreas Kling.

The Selector Query Cache can use quite a bit of memory if many
complex selectors are compiled. This patch makes sure the cache gets
cleared on memory pressure.

dom/Document.cpp:

(WebCore::Document::clearSelectorQueryCache):
(WebCore::Document::setCompatibilityMode):
(WebCore::Document::updateBaseURL):

dom/Document.h:
dom/SelectorQuery.cpp:

(WebCore::SelectorQueryCache::invalidate): Deleted.

dom/SelectorQuery.h:
platform/MemoryPressureHandler.cpp:

(WebCore::MemoryPressureHandler::releaseMemory):

2:04 AM Changeset in webkit [168242] by gyuyoung.kim@samsung.com

8 edits in trunk/Source/WebCore

Convert OwnPtr to std::unique_ptr in CDM
https://bugs.webkit.org/show_bug.cgi?id=132467

Reviewed by Andreas Kling.

Use std::unique_ptr in CDM class.

No new tests, no behavior change.

Modules/encryptedmedia/CDM.cpp:

(WebCore::CDM::create):
(WebCore::CDM::CDM):

Modules/encryptedmedia/CDM.h:
Modules/encryptedmedia/CDMPrivateMediaPlayer.h:

(WebCore::CDMPrivateMediaPlayer::CDMPrivateMediaPlayer):
(WebCore::CDMPrivateMediaPlayer::create):

Modules/encryptedmedia/MediaKeys.cpp:

(WebCore::MediaKeys::create):
(WebCore::MediaKeys::MediaKeys):

Modules/encryptedmedia/MediaKeys.h:
WebCore.exp.in:

May 3, 2014:

11:26 PM Changeset in webkit [168241] by commit-queue@webkit.org

19 edits in trunk/Source

Unreviewed, rolling out r168232.
https://bugs.webkit.org/show_bug.cgi?id=132543

broke ~15 tests on WK2 debug (Requested by kling on #webkit).

Reverted changeset:

"Shortcircuit shouldUseCredentialStorage callback"
https://bugs.webkit.org/show_bug.cgi?id=132308
http://trac.webkit.org/changeset/168232

9:22 PM Changeset in webkit [168240] by bshafiei@apple.com

5 edits in branches/safari-537.76-branch/Source

Versioning.

9:19 PM Changeset in webkit [168239] by bshafiei@apple.com

1 copy in tags/Safari-537.76.1

New tag.

8:49 PM Changeset in webkit [168238] by bshafiei@apple.com

2 edits in branches/safari-537.76-branch/Source/WebCore

Follow-up fix for the merge of r167480.

Rubber stamped by Tim Horton.

platform/KURL.cpp:

(WebCore::KURL::host): Return empty string instead of null string.

8:43 PM Changeset in webkit [168237] by akling@apple.com

6 edits in trunk/Source/WebCore

RenderSVGResourcePattern should deal in RenderElement&.
<https://webkit.org/b/132536>

Tweak buildPattern() and buildTileImageTransform() to take RenderElement&
instead of RenderObject* since we know that these functions will always
be called with non-null RenderElement subclasses.

Reviewed by Sam Weinig.

rendering/svg/RenderSVGResourceContainer.cpp:

(WebCore::RenderSVGResourceContainer::shouldTransformOnTextPainting):

rendering/svg/RenderSVGResourceContainer.h:
rendering/svg/RenderSVGResourceGradient.cpp:

(WebCore::RenderSVGResourceGradient::applyResource):

rendering/svg/RenderSVGResourcePattern.cpp:

(WebCore::RenderSVGResourcePattern::buildPattern):
(WebCore::RenderSVGResourcePattern::applyResource):
(WebCore::RenderSVGResourcePattern::buildTileImageTransform):

rendering/svg/RenderSVGResourcePattern.h:

8:25 PM Changeset in webkit [168236] by benjamin@webkit.org

3 edits
2 adds in trunk

CSS JIT: optimize direct / indirect adjacent's traversal backtracking
https://bugs.webkit.org/show_bug.cgi?id=132319

Patch by Yusuke Suzuki <Yusuke Suzuki> on 2014-05-03
Reviewed by Benjamin Poulain.

Source/WebCore:
Since adjacent backtracking stack reference is pre-allocated
in prologue in http://trac.webkit.org/changeset/166834,
clearing stack phase is not needed. So we can drop
JumpToClearAdjacentTail from backtracking action and simplify
backtracking handling.
And optimize direct / indirect adjacent's traversal backtracking by
using appropriate backtracking height.

When solving adjacent traversal backtracking action,
1) When there's no descendant relation on the right, traversal
failure becomes global failure.
2) When tagNameMatchedBacktrackingStartHeightFromDescendant ==
heightFromDescendant + 1, the descendant backtracking starts with
the parent of the current element. So we can use the current element
and the backtracking action is JumpToDescendantTreeWalkerEntryPoint.
3) Otherwise, currently we take the conservative approach,
JumpToDescendantTail.

NOTE:
And if hasDescendantRelationOnTheRight is true and there's no child
fragment on the right, the backtracking element register is not
effective. So we should ensure that fragment doesn't use the
backtracking element register. Such a fragment fulfills the following
conditions. 1. tagNameMatchedBacktrackingStartHeightFromDescendant is
always 1 (tagNames.size(), that contains only descendant fragment) 2.
heightFromDescendant is always 0 (-- See
computeBacktrackingHeightFromDescendant implementation) Therefore such
a fragment's action always becomes
JumpToDescendantTreeWalkerEntryPoint. So we can ensure that the
backtracking element register is not used.

Test: fast/selectors/backtracking-adjacent.html

cssjit/SelectorCompiler.cpp:

(WebCore::SelectorCompiler::solveDescendantBacktrackingActionForChild):
(WebCore::SelectorCompiler::solveAdjacentTraversalBacktrackingAction):
(WebCore::SelectorCompiler::solveBacktrackingAction):
(WebCore::SelectorCompiler::SelectorCodeGenerator::computeBacktrackingInformation):
(WebCore::SelectorCompiler::SelectorCodeGenerator::linkFailures):
(WebCore::SelectorCompiler::SelectorCodeGenerator::generateAdjacentBacktrackingTail):
(WebCore::SelectorCompiler::isAfterChildRelation): Deleted.

LayoutTests:

fast/selectors/backtracking-adjacent-expected.txt: Added.
fast/selectors/backtracking-adjacent.html: Added.

8:22 PM Changeset in webkit [168235] by akling@apple.com

2 edits in trunk/Source/WebCore

Clear the JSString cache when under memory pressure.
<https://webkit.org/b/132539>

The WeakGCMap used for fast mapping from StringImpl* to JSString*
can actually get pretty big, and if we find ourselves under memory
pressure, it's entirely inessential.

1.1 MB progression on Membuster3.

Reviewed by Sam Weinig.

platform/MemoryPressureHandler.cpp:

(WebCore::MemoryPressureHandler::releaseMemory):

6:55 PM Changeset in webkit [168234] by timothy_horton@apple.com

3 edits in trunk/Source/WebKit2

[iOS] REGRESSION (WebKit2): Page isn't clipped properly during back/forward swipe
https://bugs.webkit.org/show_bug.cgi?id=132538
<rdar://problem/16266027>

Reviewed by Simon Fraser.

UIProcess/ios/ViewGestureControllerIOS.mm:

(WebKit::ViewGestureController::beginSwipeGesture):
(WebKit::ViewGestureController::endSwipeGesture):

UIProcess/mac/ViewGestureController.h:

Add a clipping layer above the live swipe layer, which explicitly clips to bounds.

6:48 PM Changeset in webkit [168233] by benjamin@webkit.org

6 edits in trunk/Source/WebCore

[iOS][WK2] Support disabling speculative tiling
https://bugs.webkit.org/show_bug.cgi?id=132512

Reviewed by Tim Horton.

Move ScrollView::setScrollVelocity() and ScrollView::computeCoverageRect() to FrameView.
When speculative tiling is disabled, return an unmodified exposed rect.

Time/velocity adjusments are completely unnecessary at the moment since speculative tiling
is enabled as soon as the view scrolls.

WebCore.exp.in:
page/FrameView.cpp:

(WebCore::FrameView::setScrollVelocity):
(WebCore::FrameView::computeCoverageRect):

page/FrameView.h:
platform/ScrollView.h:
platform/ios/ScrollViewIOS.mm:

(WebCore::ScrollView::setScrollVelocity): Deleted.
(WebCore::ScrollView::computeCoverageRect): Deleted.

5:17 PM Changeset in webkit [168232] by psolanki@apple.com

19 edits in trunk/Source

Shortcircuit shouldUseCredentialStorage callback
https://bugs.webkit.org/show_bug.cgi?id=132308
<rdar://problem/16806708>

Reviewed by Alexey Proskuryakov.

No new tests because no change in functionality.

Source/WebCore:

WebCore.exp.in:
platform/network/ResourceHandle.cpp:
platform/network/ResourceHandle.h:
platform/network/ResourceHandleClient.cpp:
platform/network/ResourceHandleClient.h:
platform/network/cf/ResourceHandleCFNet.cpp:

(WebCore::ResourceHandle::createCFURLConnection):
(WebCore::ResourceHandle::shouldUseCredentialStorage):

platform/network/cf/ResourceHandleCFURLConnectionDelegate.h:
platform/network/cf/ResourceHandleCFURLConnectionDelegateWithOperationQueue.cpp:

(WebCore::ResourceHandleCFURLConnectionDelegateWithOperationQueue::shouldUseCredentialStorage):

platform/network/cf/ResourceHandleCFURLConnectionDelegateWithOperationQueue.h:
platform/network/cf/SynchronousResourceHandleCFURLConnectionDelegate.cpp:
platform/network/cf/SynchronousResourceHandleCFURLConnectionDelegate.h:
platform/network/mac/ResourceHandleMac.mm:

platform/network/mac/WebCoreResourceHandleAsOperationQueueDelegate.h:
platform/network/mac/WebCoreResourceHandleAsOperationQueueDelegate.mm:

(-[WebCoreResourceHandleWithCredentialStorageAsOperationQueueDelegate connectionShouldUseCredentialStorage:]):

platform/network/soup/ResourceHandleSoup.cpp:

Source/WebKit2:

NetworkProcess/NetworkResourceLoader.cpp: Remove shouldUseCredentialStorageAsync() callbacks.
NetworkProcess/NetworkResourceLoader.h:

5:13 PM Changeset in webkit [168231] by psolanki@apple.com

3 edits in trunk/Source/WebKit2

Reduce calls to CFURLCacheCopySharedURLCache
https://bugs.webkit.org/show_bug.cgi?id=132464
<rdar://problem/16806694>

Reviewed by Alexey Proskuryakov.

CFURLCacheCopySharedURLCache grabs a mutex and can sometimes block. Avoid that by stashing
the cache reference in a static.

NetworkProcess/NetworkResourceLoader.h: Coalesce ifdef'd code.
NetworkProcess/mac/NetworkResourceLoaderMac.mm:

(WebKit::NetworkResourceLoader::tryGetShareableHandleFromSharedBuffer):
(WebKit::NetworkResourceLoader::willCacheResponseAsync): Use more correct ifdef for
Foundation based callback.

3:16 PM Changeset in webkit [168230] by akling@apple.com

5 edits
2 adds in trunk

Invalidate scrollbars when custom scrollbar style changes dynamically.
<https://webkit.org/b/132529>

Source/WebCore:
Add a ScrollView::styleDidChange() and call that from RenderView::styleDidChange()
so that the scrollbars are sure to get repainted with potentially different style.

Reviewed by Antti Koivisto.

Test: fast/css/scrollbar-dynamic-style-change.html

platform/ScrollView.cpp:

(WebCore::ScrollView::styleDidChange):

platform/ScrollView.h:
rendering/RenderView.cpp:

(WebCore::RenderView::styleDidChange):

LayoutTests:
Reviewed by Antti Koivisto.

fast/css/scrollbar-dynamic-style-change-expected.html: Added.
fast/css/scrollbar-dynamic-style-change.html: Added.

1:46 PM Changeset in webkit [168229] by weinig@apple.com

9 edits
3 copies in trunk/Source/WebKit2

[Cocoa WebKit2] Add basic _WKWebsiteDataStore implementation
https://bugs.webkit.org/show_bug.cgi?id=132526

Reviewed by Anders Carlsson.

Renames WKSession to _WKWebsiteDataStore to better reflect its intended use (renaming the implementation object will come later).
Makes _WKWebsiteDataStore work as a wrapped Objective-C object.
Adds the ability to set a _WKWebsiteDataStore on the WKWebViewConfiguration.

Shared/Cocoa/APIObject.mm:

(API::Object::newObject):

UIProcess/API/Cocoa/WKSession.h:
UIProcess/API/Cocoa/WKSession.mm:

(-[WKSession dealloc]): Deleted.
(-[WKSession ephemeral]): Deleted.
(-[WKSession API::]): Deleted.

UIProcess/API/Cocoa/WKSessionInternal.h:

(WebKit::wrapper): Deleted.

UIProcess/API/Cocoa/WKWebView.mm:

(-[WKWebView initWithFrame:configuration:]):

UIProcess/API/Cocoa/WKWebViewConfiguration.mm:

(-[WKWebViewConfiguration copyWithZone:]):
(-[WKWebViewConfiguration _websiteDataStore]):
(-[WKWebViewConfiguration _setWebsiteDataStore:]):

UIProcess/API/Cocoa/WKWebViewConfigurationPrivate.h:
UIProcess/API/Cocoa/_WKWebsiteDataStore.h: Copied from Source/WebKit2/UIProcess/API/Cocoa/WKSession.h.
UIProcess/API/Cocoa/_WKWebsiteDataStore.mm: Copied from Source/WebKit2/UIProcess/API/Cocoa/WKSession.mm.

(+[_WKWebsiteDataStore nonPersistentDataStore]):
(-[_WKWebsiteDataStore isNonPersistentDataStore]):
(-[WKSession ephemeral]): Deleted.

UIProcess/API/Cocoa/_WKWebsiteDataStoreInternal.h: Copied from Source/WebKit2/UIProcess/API/Cocoa/WKSessionInternal.h.
WebKit2.xcodeproj/project.pbxproj:

1:32 PM Changeset in webkit [168228] by Simon Fraser

5 edits in trunk

[UI-side compositing] Assertion in PlatformCAFilters::setFiltersOnLayer with animated reference filter
https://bugs.webkit.org/show_bug.cgi?id=132528
<rdar://problem/16671660>

Reviewed by Tim Horton.

Source/WebKit2:
Allow PASSTHROUGH filters to be encoded and sent to the UI process; they can be set
on layers as the result of a filter animation using a reference filter, and just get
ignored anyway, but encoding them maintains consistency of the filters list.

Shared/WebCoreArgumentCoders.cpp:

(IPC::ArgumentCoder<FilterOperation>::encode): Assert during encoding if
we try to encode a NONE or REFERENCE filter (to match the decoding assertions).
(IPC::decodeFilterOperation): Allow decoding of PASSTHROUGH filters. Have
trying to decode a NONE or REFERENCE filter mark the message as invalid.
(IPC::ArgumentCoder<IDBKeyData>::decode): Mark the message invalid when receiving
unexpected key types.

Shared/mac/RemoteLayerTreeTransaction.mm:

(WebKit::RemoteLayerTreeTextStream::operator<<): Have the logging not crash if
a filter is null (should never happen).

LayoutTests:
Make the animation duration a little longer to cause bug 132528 to reproduce more
reliably. The test does notifyDone() from an animation start event, so this doesn't
increase test duration.

css3/filters/crash-filter-animation-invalid-url.html:

1:32 PM Changeset in webkit [168227] by Simon Fraser

5 edits
2 adds in trunk

Very fuzzy layers under non-decompasable matrices
https://bugs.webkit.org/show_bug.cgi?id=132516
<rdar://problem/16717478>

Reviewed by Sam Weinig.

Source/WebCore:
r155977 added code to modify layer contentsScale based on a root-relative
scale, so that scaled-up layers remained sharp. It does this by decomposing
an accumulated matrix, but failed to test whether the decomposition
succeeded. This would result in contentsScale of 0, which is clamped to 0.1,
resulting in very fuzzy layers.

Fix by testing for success of decomposition.

Test: compositing/contents-scale/non-decomposable-matrix.html

platform/graphics/ca/GraphicsLayerCA.cpp:

(WebCore::maxScaleFromTransform):

platform/graphics/transforms/TransformationMatrix.cpp:

(WebCore::TransformationMatrix::decompose2): Return early for identity matrices,
with fix for m11 and m22.
(WebCore::TransformationMatrix::decompose4): Return early for identity matrices.

platform/graphics/transforms/TransformationMatrix.h:

Make Decomposed2Type and Decomposed4Type into C++ structs.
(WebCore::TransformationMatrix::Decomposed2Type::operator==): Added to make it easier
to write code that asserts that decomposition is correct.
(WebCore::TransformationMatrix::Decomposed4Type::operator==): Ditto.

LayoutTests:
Compare scaling under non-decomposable and decomposable matrices.

compositing/contents-scale/non-decomposable-matrix-expected.html: Added.
compositing/contents-scale/non-decomposable-matrix.html: Added.

12:13 PM Changeset in webkit [168226] by Simon Fraser

2 edits in trunk/Source/WebCore

Fix crash in WebKit client app when zooming
https://bugs.webkit.org/show_bug.cgi?id=132475
<rdar://problem/16703405>

Reviewed by Tim Horton.

It's possible for a WebTiledBackingLayer CALayer to remain in the CALayer
hierarchy after we've called -invalidate on it, which clears the _tileController.
Project the getters against null derefs to handle this.

platform/graphics/ca/mac/WebTiledBackingLayer.mm:

(-[WebTiledBackingLayer isOpaque]):
(-[WebTiledBackingLayer acceleratesDrawing]):
(-[WebTiledBackingLayer contentsScale]):

11:52 AM Changeset in webkit [168225] by rakuco@webkit.org

2 edits in trunk

[CMake] Define SHOULD_INSTALL_JS_SHELL before including ports Options files.
https://bugs.webkit.org/show_bug.cgi?id=132525

Reviewed by Martin Robinson.

CMakeLists.txt: If OPTION() is called after Options${PORT}.cmake is included, it will

override whatever value a port may have set for it. The GTK+ port, for example, tries to set
it to SHOULD_INSTALL_JS_SHELL to ON by default, even though it did not happen before.

10:40 AM Changeset in webkit [168224] by bshafiei@apple.com

2 edits in tags/Safari-538.33.1/Source/bmalloc

Merged r168152.

10:36 AM Changeset in webkit [168223] by Alan Bujtas

1 edit
2 adds in trunk/LayoutTests

Subpixel rendering: Add hidpi fieldset/legend test case to check fieldset's cliprect when legend text is present.
https://bugs.webkit.org/show_bug.cgi?id=132524

Reviewed by Simon Fraser.

This is the hidpi test for r168221. (Fieldset legend has a horizontal line
through, when the fieldset is painted on odd device pixel position.)

fast/forms/hidpi-fieldset-on-subpixel-position-when-legend-is-present-expected.html: Added.
fast/forms/hidpi-fieldset-on-subpixel-position-when-legend-is-present.html: Added.

3:37 AM Changeset in webkit [168222] by Carlos Garcia Campos

2 edits in trunk/Source/WebCore

Unreviewed. Fix GTK+ build after r168209.

platform/leveldb/LevelDBDatabase.cpp:

(WebCore::LevelDBDatabase::write):

12:47 AM Changeset in webkit [168221] by Alan Bujtas

2 edits in trunk/Source/WebCore

Subpixel rendering: Fieldset legend has a horizontal line through, when the fieldset is painted on odd device pixel position.
https://bugs.webkit.org/show_bug.cgi?id=132521
<rdar://problem/16803305>

Reviewed by Simon Fraser.

Use device pixel snapping when the fieldset's border gets clipped out for the legend's text. It ensures that
the device pixel snapped border gets properly clipped out.

Existing fieldset tests cover it.

rendering/RenderFieldset.cpp:

(WebCore::RenderFieldset::paintBoxDecorations):

May 2, 2014:

9:09 PM Changeset in webkit [168220] by Alan Bujtas

2 edits in trunk/LayoutTests

[Mac] Unreviewed gardening.

Cliprect does not cover textarea properly on certain subpixel positions.

platform/mac/TestExpectations:

6:16 PM Changeset in webkit [168219] by Alan Bujtas

9 edits in trunk/Source/WebCore

CodeCleanup: Remove *MaintainsPixelAlignment from GraphicsLayer*.
https://bugs.webkit.org/show_bug.cgi?id=132501

Reviewed by Simon Fraser.

WebCore.exp.in:
platform/graphics/GraphicsLayer.cpp:

(WebCore::GraphicsLayer::GraphicsLayer):

platform/graphics/GraphicsLayer.h:

(WebCore::GraphicsLayer::setMaintainsPixelAlignment): Deleted.
(WebCore::GraphicsLayer::maintainsPixelAlignment): Deleted.
(WebCore::GraphicsLayer::pixelAlignmentOffset): Deleted.

platform/graphics/ca/GraphicsLayerCA.cpp:

(WebCore::GraphicsLayerCA::updateGeometry):
(WebCore::GraphicsLayerCA::computePixelAlignment):
(WebCore::GraphicsLayerCA::setMaintainsPixelAlignment): Deleted.

platform/graphics/ca/GraphicsLayerCA.h:
rendering/RenderLayerBacking.cpp:

(WebCore::RenderLayerBacking::createGraphicsLayer):

rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::keepLayersPixelAligned): Deleted.

rendering/RenderLayerCompositor.h:

6:15 PM Changeset in webkit [168218] by mrowe@apple.com

2 edits in trunk/Tools

<https://webkit.org/b/132505> Make it possible to tell copy-webkitlibraries-to-product-directory which OS X version to copy for

Reviewed by Dan Bernstein.

Scripts/copy-webkitlibraries-to-product-directory: Add an --osx-version argument and use the passed value

when determining which LLVM archive to extract.

6:15 PM Changeset in webkit [168217] by Simon Fraser

5 edits in trunk/Source/WebCore

[iOS WK2] Tiled layer content missing on pages with animated tiled layers
https://bugs.webkit.org/show_bug.cgi?id=132507
<rdar://problem/16765740>

Reviewed by Tim Horton.

Updating the tiling area of content TileControllers while
CSS animations are running depends on GraphicsLayerUpdater
triggering repeated layer flushes. With UI-side compositing, those
flushes were happening, but nothing triggered RemoteLayerTreeDrawingArea
to flush changes to the UI process.

Fix by having RenderLayerCompositor schedule a flush, rather
than just doing a flush, in response to GraphicsLayerUpdater.

Also change the name of the GraphicsLayerUpdaterClient function
to indicate that it suggests that a flush is required soon, rather than
that the flushing has to be synchronous.

platform/graphics/GraphicsLayerUpdater.cpp:

(WebCore::GraphicsLayerUpdater::displayRefreshFired):

platform/graphics/GraphicsLayerUpdater.h:
rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::flushLayersSoon):
(WebCore::RenderLayerCompositor::flushLayers): Deleted.

rendering/RenderLayerCompositor.h:

6:13 PM Changeset in webkit [168216] by commit-queue@webkit.org

3 edits in trunk/Source/WebCore

Use displayNameForTrack instead of textTrack->label() for captions.
https://bugs.webkit.org/show_bug.cgi?id=131311

Patch by Jeremy Jones <jeremyj@apple.com> on 2014-05-02
Reviewed by Darin Adler.

Use the same mechanism as the desktop to build the captions list so it has the correct
names in the correct order including none and automatic.

platform/ios/WebVideoFullscreenModelMediaElement.h:
platform/ios/WebVideoFullscreenModelMediaElement.mm:

(WebVideoFullscreenModelMediaElement::setMediaElement):
move legible track code into updateLegibleOptions()

(WebVideoFullscreenModelMediaElement::handleEvent):
updateLegibleOptions on addTrack and removeTrack

(WebVideoFullscreenModelMediaElement::selectLegibleMediaOption):
select the corresponding TextTrack on HTMLMediaElement.

(WebVideoFullscreenModelMediaElement::updateLegibleOptions):
use the same mechanism as desktop to build the captions menu.

6:12 PM Changeset in webkit [168215] by commit-queue@webkit.org

5 edits in trunk/Source/WebCore

CSS-based Media Controls Show Different times content longer than 1 hour.
https://bugs.webkit.org/show_bug.cgi?id=132443

Patch by Jeremy Jones <jeremyj@apple.com> on 2014-05-02
Reviewed by Jer Noble.

Modules/mediacontrols/mediaControlsApple.css:

(audio::-webkit-media-controls-timeline-container .hour-long-time):
This class has a wider width for longer duration times.

Modules/mediacontrols/mediaControlsApple.js:

(Controller.prototype.updateDuration):
Apply .hour-long-time class as appropriate.

(Controller.prototype.formatTime):
More robust formatting to handle hours.

Modules/mediacontrols/mediaControlsiOS.css:

(audio::-webkit-media-controls-timeline-container .hour-long-time):
This class has a wider width for longer duration times.

Modules/mediacontrols/mediaControlsiOS.js:

(ControllerIOS.prototype.formatTime):
More robust formatting to handle hours.

6:09 PM Changeset in webkit [168214] by jer.noble@apple.com

3 edits in trunk/Source/WebCore

[MSE][Mac] AVAssetTrack returns incorrect track size
https://bugs.webkit.org/show_bug.cgi?id=132469

Reviewed by Brent Fulgham.

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

(WebCore::SourceBufferPrivateAVFObjC::didParseStreamDataAsAsset): Remove the sizeChanged() notification.
(WebCore::SourceBufferPrivateAVFObjC::processCodedFrame): Cache the last parsed video frame size.
(WebCore::SourceBufferPrivateAVFObjC::naturalSize): Return the cached value.

5:19 PM Changeset in webkit [168213] by mitz@apple.com

4 edits
1 delete in trunk/Source/WebKit2

[Cocoa] Remove unused WKErrorRecoveryAttempting
https://bugs.webkit.org/show_bug.cgi?id=132503

Reviewed by Anders Carlsson.

UIProcess/API/Cocoa/WKBrowsingContextController.mm:

(didFailProvisionalLoadWithErrorForFrame):
(didFailLoadWithErrorForFrame):
(createErrorWithRecoveryAttempter): Deleted.
(-[WKBrowsingContextController attemptRecoveryFromError:]): Deleted.

UIProcess/API/Cocoa/WKErrorRecoveryAttempting.m: Removed.
UIProcess/API/Cocoa/_WKFormDelegate.h:
WebKit2.xcodeproj/project.pbxproj:

5:05 PM Changeset in webkit [168212] by matthew_hanson@apple.com

6 edits
2 copies in branches/safari-537.76-branch

Merge r166420.

5:02 PM Changeset in webkit [168211] by Alan Bujtas

2 edits in trunk/Source/WebCore

Subpixel rendering[iOS]: Use pixelSnappedRoundedRectForPainting() to clip text area rect.
https://bugs.webkit.org/show_bug.cgi?id=132499
<rdar://problem/16631050>

Reviewed by Simon Fraser.

Snap to device pixels properly instead of relying on float arithmetics while converting from RoundedRect
to FloatRoundedRect. This is the second, cleanup part of the text-area decoration is off-by-one painting issue.

Currently not testable.

rendering/RenderThemeIOS.mm:

(WebCore::RenderThemeIOS::paintTextFieldDecorations):

5:00 PM Changeset in webkit [168210] by enrica@apple.com

2 edits in trunk/Source/WebKit2

REGRESSION (WebKit2) Need to support reanalyze button for Chinese Traditional.
https://bugs.webkit.org/show_bug.cgi?id=132504
<rdar://problem/16778862>

Reviewed by Benjamin Poulain.

For traditional Chinese we support reanalyzing of the text to perform
transformations on the text based on the selected language and keyboard.
This is done by pressing the Reanalyze button on the system
menu. This patch adds the necessary code to canPerformAction to enable
the button when appropriate as well as the implementation of the action
itself.

UIProcess/ios/WKContentViewInteraction.mm:

(-[WKContentView _reanalyze:]):
(-[WKContentView canPerformAction:withSender:]):

4:57 PM Changeset in webkit [168209] by akling@apple.com

18 edits
3 deletes in trunk/Source/WebCore

Remove HistogramSupport.
<https://webkit.org/b/132354>

Prune some leftover Chromium gunk that no other ports ever used.

Reviewed by Simon Fraser.

CMakeLists.txt:
Modules/indexeddb/IDBDatabase.cpp:

(WebCore::IDBDatabase::createObjectStore):
(WebCore::IDBDatabase::deleteObjectStore):
(WebCore::IDBDatabase::transaction):

Modules/indexeddb/IDBFactory.cpp:

(WebCore::IDBFactory::openInternal):
(WebCore::IDBFactory::deleteDatabase):

Modules/indexeddb/IDBHistograms.h: Removed.
Modules/indexeddb/leveldb/IDBBackingStoreLevelDB.cpp:

(WebCore::IDBBackingStoreLevelDB::open):
(WebCore::IDBBackingStoreLevelDB::openInMemory):
(WebCore::recordInternalError): Deleted.

Modules/indexeddb/leveldb/IDBBackingStoreTransactionLevelDB.cpp:
WebCore.vcxproj/WebCore.vcxproj:
WebCore.vcxproj/WebCore.vcxproj.filters:
WebCore.xcodeproj/project.pbxproj:
css/CSSParser.cpp:

(WebCore::cssPropertyID):

dom/Document.cpp:

(WebCore::Document::~Document):
(WebCore::histogramMutationEventUsage): Deleted.

dom/ShadowRoot.cpp:
fileapi/Blob.cpp:
fileapi/WebKitBlobBuilder.cpp:

(WebCore::BlobBuilder::append):

history/PageCache.cpp:

(WebCore::logCanCacheFrameDecision):
(WebCore::logCanCachePageDecision):

platform/HistogramSupport.cpp: Removed.
platform/HistogramSupport.h: Removed.
platform/leveldb/LevelDBDatabase.cpp:

(WebCore::LevelDBDatabase::open):
(WebCore::histogramLevelDBError): Deleted.

rendering/RenderLayer.cpp:

(WebCore::RenderLayer::updateNeedsCompositedScrolling):

xml/XMLHttpRequest.cpp:

(WebCore::XMLHttpRequest::send):

4:48 PM Changeset in webkit [168208] by matthew_hanson@apple.com

5 edits in branches/safari-537.76-branch/Source

Merge r167548.

4:38 PM Changeset in webkit [168207] by beidson@apple.com

2 edits in trunk/Source/WebCore

Stop flipping the ImageControlsButton
<rdar://problem/16773238> and https://bugs.webkit.org/show_bug.cgi?id=132502

Reviewed by Tim Horton.

rendering/RenderThemeMac.mm:

(WebCore::RenderThemeMac::paintImageControlsButton):

4:36 PM Changeset in webkit [168206] by enrica@apple.com

2 edits in trunk/Source/WebKit2

REGRESSION (WebKit2) Need to support transliterate chinese button (简⇄繁) for Traditional Chinese.
https://bugs.webkit.org/show_bug.cgi?id=132500
<rdar://problem/16778870>

Reviewed by Benjamin Poulain.

For traditional Chinese we support the transliterate to simplified Chinese.
This is done by pressing the 简⇄繁 button on the system
menu. This patch adds the necessary code to canPerformAction to enable
the button when appropriate as well as the implementation of the action
itself.

UIProcess/ios/WKContentViewInteraction.mm:

(-[WKContentView _transliterateChinese:]):
(-[WKContentView canPerformAction:withSender:]):

4:11 PM Changeset in webkit [168205] by commit-queue@webkit.org

8 edits in trunk/Source

Fullscreen UI does not appear after WebProcess has crashed
https://bugs.webkit.org/show_bug.cgi?id=132442

Patch by Jeremy Jones <jeremyj@apple.com> on 2014-05-02
Reviewed by Darin Adler.

Source/WebCore:
Clean up immediately when there is a WebProcess crash.

WebCore.exp.in:
platform/ios/WebVideoFullscreenInterfaceAVKit.h:
platform/ios/WebVideoFullscreenInterfaceAVKit.mm:

(WebVideoFullscreenInterfaceAVKit::invalidate):
Clean-up resources immediately.

Source/WebKit2:
Cleanup WebVideoFullscreenManagerProxy after a WebProcess crash.

UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::reattachToWebProcess):
recreate WebVideoFullscreenManagerProxy after a WebProcess crash.

(WebKit::WebPageProxy::resetState):
invalidate and release WebVideoFullscreenManagerProxy on crash.

UIProcess/ios/WebVideoFullscreenManagerProxy.h:
UIProcess/ios/WebVideoFullscreenManagerProxy.mm:

(WebKit::WebVideoFullscreenManagerProxy::~WebVideoFullscreenManagerProxy):
don't assume m_page is valid.

(WebKit::WebVideoFullscreenManagerProxy::invalidate):
do cleanup invalidation in reponse to a WebProcess crash.

3:56 PM Changeset in webkit [168204] by commit-queue@webkit.org

2 edits in trunk/Source/WebCore

Unreviewed, rolling out r168192.
https://bugs.webkit.org/show_bug.cgi?id=132498

Was not the correct fix (blurry!) (Requested by bfulgham_ on
#webkit).

Reverted changeset:

"[iOS] deviceScaleFactor is being double-applied when
rendering captions in full screen mode"
https://bugs.webkit.org/show_bug.cgi?id=132481
http://trac.webkit.org/changeset/168192

3:54 PM Changeset in webkit [168203] by benjamin@webkit.org

2 edits in trunk/Source/WebKit2

[iOS][WK2] More animation madness for when the extendedBackground is modified in an animation block
https://bugs.webkit.org/show_bug.cgi?id=132497

Patch by Benjamin Poulain <bpoulain@apple.com> on 2014-05-02
Reviewed by Beth Dakin.

UIProcess/API/Cocoa/WKWebView.mm:

(updateTopAndBottomExtendedBackgroundExclusionIfNecessary):
Do not early return when _extendedBackgroundExclusionInsets.left is empty. That way, if executed in an animation block,
the top of bottom view will animate from their current size to an empty width.

When creating the layer, set up their height without animation. Otherwise it is possible to see the height animating
while the left inset is animating.

(-[WKWebView _setExtendedBackgroundExclusionInsets:]):
When replacing the scrollview, make sure the frame and color are not animated. Otherwise the transition between
scrollView and _mainExtendedBackgroundView can be visible if _setExtendedBackgroundExclusionInsets: is invoked
in an animation block.

(-[WKWebView _endAnimatedResize]):
Nuke the top and bottom insets when possible. [WKWebView _endAnimatedResize] is unfrequent, so it is a good opportunity
to free the memory.

3:47 PM Changeset in webkit [168202] by jeremyj-wk@apple.com

1 edit in trunk/Tools/ChangeLog

Add Jeremy Jones as a committer. https://bugs.webkit.org/show_bug.cgi?id=132492

3:41 PM Changeset in webkit [168201] by ap@apple.com

7 edits in trunk/Source

Remove Blob contentDisposition handling
https://bugs.webkit.org/show_bug.cgi?id=132490

Reviewed by Sam Weinig.

Source/WebCore:
Dead code.

platform/network/BlobData.h:

(WebCore::BlobData::contentDisposition): Deleted.
(WebCore::BlobData::setContentDisposition): Deleted.

platform/network/BlobRegistryImpl.cpp:

(WebCore::BlobRegistryImpl::registerBlobURL):
(WebCore::BlobRegistryImpl::registerBlobURLForSlice):

platform/network/BlobResourceHandle.cpp:

(WebCore::BlobResourceHandle::notifyResponseOnSuccess):

platform/network/BlobStorageData.h:

(WebCore::BlobStorageData::create):
(WebCore::BlobStorageData::contentType):
(WebCore::BlobStorageData::BlobStorageData):
(WebCore::BlobStorageData::contentDisposition): Deleted.

Source/WebKit2:

Shared/FileAPI/BlobRegistrationData.cpp:

(WebKit::BlobRegistrationData::encode):
(WebKit::BlobRegistrationData::decode):

3:39 PM Changeset in webkit [168200] by matthew_hanson@apple.com

3 edits in branches/safari-537.76-branch/Source/JavaScriptCore

Merge r167544.

3:38 PM Changeset in webkit [168199] by andersca@apple.com

4 edits in trunk/Source/WebCore

Implement FormData decoding using KeyedDecoder
https://bugs.webkit.org/show_bug.cgi?id=132494

Reviewed by Tim Horton.

platform/KeyedCoding.h:

(WebCore::KeyedDecoder::decodeEnum):

platform/network/FormData.cpp:

(WebCore::decodeElement):
(WebCore::FormData::decode):

platform/network/FormData.h:

3:37 PM Changeset in webkit [168198] by commit-queue@webkit.org

2 edits in trunk/Tools

Add Jeremy as a committer.
https://bugs.webkit.org/show_bug.cgi?id=132492

Patch by Jeremy Jones <jeremyj@apple.com> on 2014-05-02
Reviewed by Jer Noble.

Add Jeremy Jones to the committers file.

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

3:24 PM Changeset in webkit [168197] by msaboff@apple.com

2 edits in trunk/Source/JavaScriptCore

"arm64 function not 4-byte aligned" warnings when building JSC
https://bugs.webkit.org/show_bug.cgi?id=132495

Reviewed by Geoffrey Garen.

Added ".align 4" for both ARM Thumb2 and ARM 64 to silence the linker.

llint/LowLevelInterpreter.cpp:

3:05 PM Changeset in webkit [168196] by benjamin@webkit.org

2 edits in trunk/Source/WebKit2

[iOS][WK2] Adapt the rubberband constraints when the page get smaller than the scrollview - insets
https://bugs.webkit.org/show_bug.cgi?id=132457

Reviewed by Enrica Casucci.

It is quite common for us to have a WKContentView that is scalled smaller than the WKWebView, content
insets included.

In those cases, update the constraints to fit the content properly in the view.

UIProcess/ios/WKScrollView.mm:

(valuesAreWithinOnePixel):
(-[WKScrollView _rubberBandOffsetForOffset:maxOffset:minOffset:range:outside:]):

3:04 PM Changeset in webkit [168195] by mrowe@apple.com

2 edits in trunk/Source/WebKit2

-[_WKThumbnailView _requestSnapshotIfNeeded] assumes that taking a snapshot will always succeed
<https://webkit.org/b/132489> / <rdar://problem/16704660>

Reviewed by Tim Horton.

UIProcess/API/Cocoa/_WKThumbnailView.mm:

(-[_WKThumbnailView _requestSnapshotIfNeeded]): Don't attempt to create a CGImageRef if we failed
to create a ShareableBitmap. This handles both the callback receiving a null Handle and a failure
within ShareableBitmap::create.

2:58 PM Changeset in webkit [168194] by matthew_hanson@apple.com

2 edits in branches/safari-537.76-branch/Source/JavaScriptCore

Merge r167354.

2:43 PM Changeset in webkit [168193] by beidson@apple.com

4 edits in trunk/Source/WebKit/mac

Implement new delegate method -sharingService:sourceFrameOnScreenForShareItem:.
<rdar://problem/16797425> and https://bugs.webkit.org/show_bug.cgi?id=132484

Reviewed by Tim Horton.

Misc/WebSharingServicePickerController.mm:

(-[WebSharingServicePickerController sharingService:sourceFrameOnScreenForShareItem:]):

WebCoreSupport/WebContextMenuClient.h:
WebCoreSupport/WebContextMenuClient.mm:

(WebContextMenuClient::screenRectForHitTestNode):

2:39 PM Changeset in webkit [168192] by Brent Fulgham

2 edits in trunk/Source/WebCore

[iOS] deviceScaleFactor is being double-applied when rendering captions in full screen mode
https://bugs.webkit.org/show_bug.cgi?id=132481

Reviewed by Jer Noble.

html/shadow/MediaControlElements.cpp:

(WebCore::MediaControlTextTrackContainerElement::updateDisplay): Don't set the platform scale factor here. It is already
being accounted for in the createTextTrackRepresentationImage method.

2:28 PM Changeset in webkit [168191] by matthew_hanson@apple.com

7 edits in branches/safari-537.76-branch/Source/JavaScriptCore

Merge r167336.

2:24 PM Changeset in webkit [168190] by Simon Fraser

12 edits in trunk

[iOS WK2] Don't create backing store for -webkit-overflow-scrolling:touch that can't scroll
https://bugs.webkit.org/show_bug.cgi?id=132487
<rdar://problem/16758041>

Reviewed by Sam Weinig.

Source/WebCore:

Previously, -webkit-overflow-scrolling:touch would cause us to make compositing
layers for any element that had overflow: auto or scroll on either axis. This
created lots of backing store when not required.

Improve this to only create compositing for scrolling when there is actually
scrollable overflow. This makes things slightly more complex, because we can
only know when layout is up to date.

rendering/RenderBox.cpp:

(WebCore::RenderBox::computeRectForRepaint): usesCompositedScrolling() tells
us if we're actually doing composited overflow.

rendering/RenderLayer.cpp:

(WebCore::RenderLayer::hasTouchScrollableOverflow):
(WebCore::RenderLayer::handleTouchEvent):

rendering/RenderLayer.h:
rendering/RenderLayerBacking.cpp:

(WebCore::layerOrAncestorIsTransformedOrUsingCompositedScrolling):
(WebCore::RenderLayerBacking::updateGraphicsLayerConfiguration): Only update
scrolling and clipping layers if layout is not pending.
(WebCore::RenderLayerBacking::updateGraphicsLayerGeometry):
(WebCore::RenderLayerBacking::updateScrollingLayers): The caller calls
updateInternalHierarchy(), so no need to do it here.

rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::requiresCompositingForScrolling): We
can only determine that we're scrollable after layout.
(WebCore::isStickyInAcceleratedScrollingLayerOrViewport):
(WebCore::isMainFrameScrollingOrOverflowScrolling):

LayoutTests:

These are all progressions, and show that we make layers in fewer cases.

platform/ios-sim/compositing/overflow/iframe-inside-overflow-clipping-expected.txt:
platform/ios-sim/compositing/overflow/overflow-auto-with-touch-no-overflow-expected.txt:
platform/ios-sim/compositing/overflow/overflow-overlay-with-touch-no-overflow-expected.txt:
platform/ios-sim/compositing/overflow/overflow-scroll-with-touch-no-overflow-expected.txt:
platform/ios-sim/compositing/overflow/subpixel-overflow-expected.txt:

2:21 PM Changeset in webkit [168189] by mhahnenberg@apple.com

2 edits in trunk/Source/JavaScriptCore

Fix cloop build after r168178

bytecode/CodeBlock.cpp:

2:03 PM Changeset in webkit [168188] by andersca@apple.com

10 edits in trunk/Source

Clean up FormDataElement
https://bugs.webkit.org/show_bug.cgi?id=132483

Reviewed by Sam Weinig.

Source/WebCore:

platform/network/FormData.cpp:

(WebCore::FormData::FormData):
(WebCore::FormData::deepCopy):
(WebCore::FormData::expandDataStore):
(WebCore::FormData::flatten):
(WebCore::FormData::resolveBlobReferences):
(WebCore::FormData::generateFiles):
(WebCore::FormData::hasGeneratedFiles):
(WebCore::FormData::hasOwnedGeneratedFiles):
(WebCore::FormData::removeGeneratedFilesIfNeeded):
(WebCore::encodeElement):
(WebCore::decodeElement):

platform/network/FormData.h:

(WebCore::FormDataElement::FormDataElement):
(WebCore::operator==):

platform/network/cf/FormDataStreamCFNet.cpp:

(WebCore::advanceCurrentStream):
(WebCore::setHTTPBody):

platform/network/curl/FormDataStreamCurl.cpp:

(WebCore::FormDataStream::read):

platform/network/curl/ResourceHandleManager.cpp:

(WebCore::setupFormData):

platform/network/soup/ResourceHandleSoup.cpp:

(WebCore::addFormElementsToSoupMessage):

Source/WebKit2:

NetworkProcess/NetworkResourceLoader.cpp:

(WebKit::NetworkResourceLoader::NetworkResourceLoader):

Shared/Network/NetworkResourceLoadParameters.cpp:

(WebKit::NetworkResourceLoadParameters::encode):

1:53 PM Changeset in webkit [168187] by commit-queue@webkit.org

13 edits
1 add
2 deletes in trunk/Source/ThirdParty/ANGLE

Update ANGLE Windows build.
https://bugs.webkit.org/show_bug.cgi?id=132456

Patch by Alex Christensen <achristensen@webkit.org> on 2014-05-02
Reviewed by Brent Fulgham.

ANGLE.vcxproj/libEGLCommon.props:
ANGLE.vcxproj/libGLESv2.vcxproj:
ANGLE.vcxproj/libGLESv2.vcxproj.filters:
ANGLE.vcxproj/libGLESv2Common.props:
ANGLE.vcxproj/translator_common.vcxproj:
ANGLE.vcxproj/translator_common.vcxproj.filters:
ANGLE.vcxproj/translator_glsl.vcxproj:
ANGLE.vcxproj/translator_glsl.vcxproj.filters:
ANGLE.vcxproj/translator_hlsl.vcxproj:
ANGLE.vcxproj/translator_hlsl.vcxproj.filters:

Updated ANGLE build.

src/ANGLE.sln: Removed.
src/build_angle.gyp: Removed.
src/commit.h: Added.
changes.diff:
src/libGLESv2/Program.cpp:

(gl::InfoLog::append):
Fixed typo.

1:03 PM Changeset in webkit [168186] by jer.noble@apple.com

2 edits in trunk/Source/WebCore

[iOS] Wireless playback button not blue when active
https://bugs.webkit.org/show_bug.cgi?id=132473

Reviewed by Simon Fraser.

The playback button's class was being set to 'undefined', due to Controller.ClassNames.active being undefined.

Modules/mediacontrols/mediaControlsApple.js:

1:00 PM Changeset in webkit [168185] by beidson@apple.com

5 edits in trunk/Source/WebKit/mac

Crash inside [WebSharingServicePickerController clear]
<rdar://problem/16791944> and https://bugs.webkit.org/show_bug.cgi?id=132477

Reviewed by Tim Horton.

Misc/WebSharingServicePickerController.h:
Misc/WebSharingServicePickerController.mm:

(-[WebSharingServicePickerController clear]): clear can be called twice, so null check _menuClient.

WebCoreSupport/WebContextMenuClient.h:
WebCoreSupport/WebContextMenuClient.mm:

(WebContextMenuClient::~WebContextMenuClient): For a sanity check, call clear on the picker here.

12:54 PM Changeset in webkit [168184] by Brian Burg

2 edits in trunk/Tools

Hard to figure out how to run a single test with run-api-tests
https://bugs.webkit.org/show_bug.cgi?id=116332

Reviewed by Alexey Proskuryakov.

Scripts/run-api-tests: add two examples to the help message.

12:43 PM Changeset in webkit [168183] by ap@apple.com

10 edits in trunk/Source/WebCore

Don't abuse Blob deserialization constructor in WebSocket
https://bugs.webkit.org/show_bug.cgi?id=132478

Reviewed by Sam Weinig.

Modules/websockets/WebSocketChannel.cpp:

(WebCore::WebSocketChannel::send):
(WebCore::WebSocketChannel::enqueueBlobFrame): This is the important change -
Blob::create was called for no reason. If the blob came from a worker, it was
already cloned for cross-thread messaging, otherwise there is no reason to make
a new one.

fileapi/Blob.h:

(WebCore::Blob::deserialize):
(WebCore::Blob::create): Deleted.

fileapi/File.h:

(WebCore::File::deserialize):
(WebCore::File::create): Deleted.
Renamed a special case of "create" function to avoid explaining what it is for.

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

(WebCore::WebSocket::send):

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

(WebCore::WorkerThreadableWebSocketChannel::send): Print a full URL in LOG(),
not one shortened to 1024 characters.
(WebCore::WorkerThreadableWebSocketChannel::Peer::send):
(WebCore::WorkerThreadableWebSocketChannel::mainThreadSendBlob):
(WebCore::WorkerThreadableWebSocketChannel::Bridge::send):

Modules/websockets/WorkerThreadableWebSocketChannel.h:
bindings/js/SerializedScriptValue.cpp:

(WebCore::CloneDeserializer::readFile):
(WebCore::CloneDeserializer::readTerminal):

12:42 PM Changeset in webkit [168182] by matthew_hanson@apple.com

2 edits in branches/safari-537.76-branch/Source/WebCore

Merge r165053.

12:34 PM Changeset in webkit [168181] by andersca@apple.com

5 edits in trunk/Source

Add and implement KeyedDecoder::decodeBytes
https://bugs.webkit.org/show_bug.cgi?id=132479

Reviewed by Tim Horton.

Source/WebCore:

platform/KeyedCoding.h:

(WebCore::KeyedDecoder::decodeBytes):

Source/WebKit2:

Shared/cf/KeyedDecoder.cpp:

(WebKit::KeyedDecoder::decodeBytes):

Shared/cf/KeyedDecoder.h:

12:28 PM Changeset in webkit [168180] by Joseph Pecoraro

20 edits
1 copy
1 add in trunk/Source

[iOS] WebKit2 File Upload Support
https://bugs.webkit.org/show_bug.cgi?id=132024

Reviewed by Enrica Casucci.

Source/WebCore:

English.lproj/Localizable.strings:

New localized strings for <input type="file"> on iOS.

Source/WebKit2:

Configurations/WebKit2.xcconfig:

Include MobileCoreServices on iOS for kUTTypeImage/kUTTypeMovie.

WebKit2.xcodeproj/project.pbxproj:

Add new files.

UIProcess/WebOpenPanelResultListenerProxy.h:
UIProcess/WebOpenPanelResultListenerProxy.cpp:

(WebKit::filePathsFromFileURLs):
(WebKit::WebOpenPanelResultListenerProxy::chooseFiles):

UIProcess/WebPageProxy.h:
UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::runOpenPanel):
(WebKit::WebPageProxy::didChooseFilesForOpenPanelWithDisplayStringAndIcon):

WebProcess/WebPage/WebOpenPanelResultListener.h:
WebProcess/WebPage/WebOpenPanelResultListener.cpp:

(WebKit::WebOpenPanelResultListener::didChooseFilesWithDisplayStringAndIcon):

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

(WebKit::WebPage::didChooseFilesForOpenPanelWithDisplayStringAndIcon):
Message forwarding for choosing files and providing a display string and icon,
leading down to the existing WebCore FileChooser method.

UIProcess/PageClient.h:
UIProcess/ios/PageClientImplIOS.h:
UIProcess/ios/PageClientImplIOS.mm:

(WebKit::PageClientImpl::handleRunOpenPanel):
Add a default handler for file open panel on iOS.
Forwards to the content view.

UIProcess/ios/WKContentViewInteraction.h:
UIProcess/ios/WKContentViewInteraction.mm:

(-[WKContentView lastInteractionLocation]):
(-[WKContentView _webTouchEventsRecognized:]):
(-[WKContentView _highlightLongPressRecognized:]):
(-[WKContentView _longPressRecognized:]):
(-[WKContentView _singleTapRecognized:]):
(-[WKContentView _doubleTapRecognized:]):
(-[WKContentView _twoFingerDoubleTapRecognized:]):
Keep track of the last interaction location. This matches previous behavior
of showing the file upload popover where the user tapped, to handle
cases where the <input> is hidden.

(-[WKContentView _showRunOpenPanel:resultListener:]):
(-[WKContentView fileUploadPanelDidDismiss:]):
Handle showing the cleaning up after the file upload panel.

UIProcess/ios/forms/WKFileUploadPanel.h:
UIProcess/ios/forms/WKFileUploadPanel.mm: Added.

(squareCropRectForSize):
(squareImage):
(thumbnailSizedImageForImage):
(-[_WKFileUploadItem isVideo]):
(-[_WKFileUploadItem fileURL]):
(-[_WKFileUploadItem displayImage]):
(-[_WKImageFileUploadItem initWithFilePath:originalImage:]):
(-[_WKImageFileUploadItem isVideo]):
(-[_WKImageFileUploadItem fileURL]):
(-[_WKImageFileUploadItem displayImage]):
(-[_WKVideoFileUploadItem initWithFilePath:mediaURL:]):
(-[_WKVideoFileUploadItem isVideo]):
(-[_WKVideoFileUploadItem fileURL]):
(-[_WKVideoFileUploadItem displayImage]):
Helper class for each image picker selection. Knows how to get
a file URL and thumbnail display image for the item.

(-[WKFileUploadPanel initWithView:]):
(-[WKFileUploadPanel dealloc]):
(-[WKFileUploadPanel _dispatchDidDismiss]):
(-[WKFileUploadPanel _cancel]):
(-[WKFileUploadPanel _chooseFiles:displayString:iconImage:]):
Lifetime of the upload panel requires that either cancel or choose
must happen as we go through the file picking process.

(-[WKFileUploadPanel presentWithParameters:WebKit::resultListener:WebKit::]):
(-[WKFileUploadPanel dismiss]):
API to show or dismiss the panel.

(-[WKFileUploadPanel _dismissDisplayAnimated:]):
Helper to clean up the UI as it progresses or completes no matter the device idiom.

(-[WKFileUploadPanel _presentPopoverWithContentViewController:animated:]):
(-[WKFileUploadPanel _presentFullscreenViewController:animated:]):
UI presentation for the appropriate idiom.

(-[WKFileUploadPanel _mediaTypesForPickerSourceType:]):
(-[WKFileUploadPanel _showMediaSourceSelectionSheet]):
(-[WKFileUploadPanel _showPhotoPickerWithSourceType:]):
Showing the action sheet or image picker.

(-[WKFileUploadPanel popoverControllerDidDismissPopover:]):
(-[WKFileUploadPanel _willMultipleSelectionDelegateBeCalled]):
(-[WKFileUploadPanel imagePickerController:didFinishPickingMediaWithInfo:]):
(-[WKFileUploadPanel imagePickerController:didFinishPickingMultipleMediaWithInfo:]):
(-[WKFileUploadPanel imagePickerControllerDidCancel:]):
Action sheet or image picker handlers.

(-[WKFileUploadPanel _processMediaInfoDictionaries:successBlock:failureBlock:]):
(-[WKFileUploadPanel _processMediaInfoDictionaries:atIndex:processedResults:processedImageCount:processedVideoCount:successBlock:failureBlock:]):
(-[WKFileUploadPanel _uploadItemFromMediaInfo:successBlock:failureBlock:]):
(-[WKFileUploadPanel _displayStringForPhotos:videos:]):
Processing selections from the image picker to FileUploadItems.

12:01 PM Changeset in webkit [168179] by andersca@apple.com

2 edits in trunk/Source/WebKit

Remove unsuccessful build fix attempts
https://bugs.webkit.org/show_bug.cgi?id=132476

Reviewed by Dan Bernstein.

WebKit.xcodeproj/project.pbxproj:

11:52 AM Changeset in webkit [168178] by mhahnenberg@apple.com

11 edits
2 adds in trunk/Source/JavaScriptCore

Add a DFG function whitelist
https://bugs.webkit.org/show_bug.cgi?id=132437

Reviewed by Geoffrey Garen.

Often times when debugging, using bytecode ranges isn't enough to narrow down to the
particular DFG block that's causing issues. This patch adds the ability to whitelist
specific functions specified in a file to enable further filtering without having to recompile.

CMakeLists.txt:
JavaScriptCore.vcxproj/JavaScriptCore.vcxproj:
JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters:
JavaScriptCore.xcodeproj/project.pbxproj:
dfg/DFGCapabilities.cpp:

(JSC::DFG::isSupported):
(JSC::DFG::mightInlineFunctionForCall):
(JSC::DFG::mightInlineFunctionForClosureCall):
(JSC::DFG::mightInlineFunctionForConstruct):

dfg/DFGFunctionWhitelist.cpp: Added.

(JSC::DFG::FunctionWhitelist::ensureGlobalWhitelist):
(JSC::DFG::FunctionWhitelist::FunctionWhitelist):
(JSC::DFG::FunctionWhitelist::parseFunctionNamesInFile):
(JSC::DFG::FunctionWhitelist::contains):

dfg/DFGFunctionWhitelist.h: Added.
runtime/Options.cpp:

(JSC::parse):
(JSC::Options::dumpOption):

runtime/Options.h:

11:46 AM Changeset in webkit [168177] by Simon Fraser

4 edits in trunk/Tools

Fix several memory leaks found by code inspection
https://bugs.webkit.org/show_bug.cgi?id=132472

Reviewed by Geoffrey Garen.

Fix memory leaks.

TestWebKitAPI/Tests/mac/WillSendSubmitEvent.mm:

(TestWebKitAPI::TEST):

WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp:

(WTR::InjectedBundlePage::willSendRequestForFrame):

WebKitTestRunner/mac/PlatformWebViewMac.mm:

(WTR::PlatformWebView::changeWindowScaleIfNeeded):

11:12 AM Changeset in webkit [168176] by matthew_hanson@apple.com

8 edits
3 copies in branches/safari-537.76-branch

Merge r164917.

10:29 AM Changeset in webkit [168175] by Simon Fraser

4 edits in trunk/Source/WebKit2

[iOS WK2] Animations on vox.com look wrong
https://bugs.webkit.org/show_bug.cgi?id=132462
<rdar://problem/16731884>

Reviewed by Sam Weinig.

PlatformCALayerRemote was managing animations incorrectly; aninations
would stick around in m_properties.addedAnimations and get added a second
time by mistake.

Animations have to be managed a little differently to other properties,
since they are not steady-state things. A given commit has to send over
the added and removed animations, and then clear the layer properties.

Do this by adding PlatformCALayerRemote::didCommit(), which is called
after the layer properties have been encoded, and have it clear the lists
of added and removed animations.

removeAnimationForKey() also has to remove the animation from addedAnimations
so that an add/remove in the same commit doesn't send the animation to the
UI process.

WebProcess/WebPage/mac/PlatformCALayerRemote.cpp:

(WebKit::PlatformCALayerRemote::recursiveBuildTransaction):
(WebKit::PlatformCALayerRemote::didCommit):
(WebKit::PlatformCALayerRemote::removeAnimationForKey):

WebProcess/WebPage/mac/PlatformCALayerRemote.h:
WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm:

(WebKit::RemoteLayerTreeDrawingArea::flushLayers):

10:29 AM Changeset in webkit [168174] by Simon Fraser

2 edits in trunk/Source/WebKit2

[iOS WK2] Can't scroll on gatesnotes.com
https://bugs.webkit.org/show_bug.cgi?id=132459
<rdar://problem/16770909>

Reviewed by Benjamin Poulain.

The custom UIView hit-testing code was finding views that were created by
the compositing code for clipping, above the UIScrollViews. We only ever
need to find UIScrollViews here for touch overflow-scrolling, so constrain
the hit-testing code to only return UIScrollViews.

UIProcess/ios/RemoteLayerTreeHostIOS.mm:

(-[UIView _recursiveFindDescendantScrollViewAtPoint:withEvent:]):
(-[UIView _findDescendantViewAtPoint:withEvent:]):
(-[UIView _recursiveFindDescendantViewAtPoint:withEvent:]): Deleted.

10:27 AM Changeset in webkit [168173] by commit-queue@webkit.org

3 edits in trunk/LayoutTests

Unreviewed, rolling out r168150.
https://bugs.webkit.org/show_bug.cgi?id=132471

This test is still broken (Requested by ap on #webkit).

Reverted changeset:

"REGRESSION(r125251): It made svg/custom/use-instanceRoot-as-
event-target.xhtml assert and flakey"
https://bugs.webkit.org/show_bug.cgi?id=93812
http://trac.webkit.org/changeset/168150

10:23 AM Changeset in webkit [168172] by fpizlo@apple.com

5 edits
6 adds in trunk

DFGAbstractInterpreter should not claim Int52 arithmetic creates Int52s
https://bugs.webkit.org/show_bug.cgi?id=132446

Reviewed by Mark Hahnenberg.

Source/JavaScriptCore:
Basically any arithmetic operation can turn an Int52 into an Int32 or vice-versa, and
our modeling of Int52Rep nodes is such that they can have either Int32 or Int52 type
to indicate a bound on the value. This is useful for knowing, for example, that
Int52Rep(Int32:) returns a value that cannot be outside the Int32 range. Also,
ValueRep(Int52Rep:) uses this to determine whether it may return a double or an int.
But this means that all arithmetic operations must be careful to note that they may
turn Int32 inputs into an Int52 output or vice-versa, as these new tests show.

dfg/DFGAbstractInterpreterInlines.h:

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

dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::makeSafe):

tests/stress/int52-ai-add-then-filter-int32.js: Added.

(foo):

tests/stress/int52-ai-mul-and-clean-neg-zero-then-filter-int32.js: Added.

(foo):

tests/stress/int52-ai-mul-then-filter-int32-directly.js: Added.

(foo):

tests/stress/int52-ai-mul-then-filter-int32.js: Added.

(foo):

tests/stress/int52-ai-neg-then-filter-int32.js: Added.

(foo):

tests/stress/int52-ai-sub-then-filter-int32.js: Added.

(foo):

Tools:
Test the FTL by default now that it's enabled by default.

Scripts/run-javascriptcore-tests:

9:19 AM Changeset in webkit [168171] by Chris Fleizach

5 edits in trunk/Source/WebCore

AX: WK2: iOS web page scrolling doesn't work with VoiceOver
https://bugs.webkit.org/show_bug.cgi?id=132028

Reviewed by Mario Sanchez Prada.

With the AX tree residing in the WebProcess, scrolling needs to be implemented in
WebCore using accessibilityScroll: in order for accessibility clients to scroll through the AX API.

accessibility/AccessibilityObject.cpp:

(WebCore::AccessibilityObject::scrollViewAncestor):
(WebCore::AccessibilityObject::scrollToMakeVisibleWithSubFocus):

accessibility/AccessibilityObject.h:
accessibility/ios/WebAccessibilityObjectWrapperIOS.h:
accessibility/ios/WebAccessibilityObjectWrapperIOS.mm:

(-[WebAccessibilityObjectWrapper _accessibilityConvertPointToViewSpace:]):
(-[WebAccessibilityObjectWrapper _accessibilityScrollToVisible]):
(-[WebAccessibilityObjectWrapper accessibilityScroll:]):
(-[WebAccessibilityObjectWrapper postScrollStatusChangeNotification]):
(-[WebAccessibilityObjectWrapper _accessibilityScrollPosition]):
(-[WebAccessibilityObjectWrapper _accessibilityScrollSize]):
(-[WebAccessibilityObjectWrapper _accessibilityScrollVisibleRect]):

9:04 AM Changeset in webkit [168170] by commit-queue@webkit.org

9 edits in trunk/Source/WebInspectorUI

Web Inspector: CodeMirror 4 CSS mode new state data structure breaks helpers.
https://bugs.webkit.org/show_bug.cgi?id=132149

Patch by Jono Wells <jonowells@apple.com> on 2014-05-02
Reviewed by Joseph Pecoraro.

The update to CodeMirror 4 included dramatic changes to the CSS mode,
particularly the way it handles tokens. state.stack is gone, replaced
by state.context.

Tools/PrettyPrinting/CodeMirrorFormatters.js:
Tools/PrettyPrinting/codemirror.js:
UserInterface/External/CodeMirror/codemirror.js:
UserInterface/External/CodeMirror/livescript.js:
UserInterface/External/CodeMirror/runmode.js:

Updates from ToT CodeMirror.

UserInterface/Controllers/CodeMirrorCompletionController.js:

(WebInspector.CodeMirrorCompletionController.prototype._generateCSSCompletions):

UserInterface/Views/CodeMirrorAdditions.js:
UserInterface/Views/CodeMirrorFormatters.js: lastToken is null now for ":" characters.

Changes to match structural changes to the state object and changes to expected values of lastToken.

6:52 AM Changeset in webkit [168169] by jeremyj@apple.com

3 edits in trunk/Source/WebCore

Pause playback on exit fullscreen when inline playback not allowed.
https://bugs.webkit.org/show_bug.cgi?id=132450

Reviewed by Jer Noble.

Use correct method of determining if inline playback is allowed.

html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::parseAttribute):
Use m_mediaSession->requiresFullscreenForVideoPlayback to detect if inline playback is allowed.

platform/ios/WebVideoFullscreenModelMediaElement.mm:

(WebVideoFullscreenModelMediaElement::requestExitFullscreen):
Prevent duplicate exit requests.

5:03 AM Changeset in webkit [168168] by commit-queue@webkit.org

2 edits in trunk/Source/WebKit2

REGRESSION(r168118): [GTK] build broken due to shouldTrackVisitedLinks
https://bugs.webkit.org/show_bug.cgi?id=132447

Unreviewed GTK build fix.

shouldTrackVisitedLinks was removed on r168118.
Visited link coloring works as expected after this change.

Patch by Carlos Alberto Lopez Perez <clopez@igalia.com> on 2014-05-02

UIProcess/gtk/WebContextGtk.cpp:

(WebKit::WebContext::platformInitializeWebProcess): Remove
shouldTrackVisitedLinks parameter.

2:35 AM Changeset in webkit [168167] by commit-queue@webkit.org

33 edits in trunk

[CSS Blending] Remove support for non-separable blend modes from background-blend-mode
https://bugs.webkit.org/show_bug.cgi?id=132327

Patch by Ion Rosca <Ion Rosca> on 2014-05-02
Reviewed by Dean Jackson.

Source/WebCore:
Removed support for non-separable background blend modes from the CSS parser.
Covered by existing tests.

css/CSSParser.cpp:

(WebCore::CSSParser::parseFillProperty):

LayoutTests:

css3/compositing/background-blend-mode-gif-color-2.html:
css3/compositing/background-blend-mode-gif-color.html:
css3/compositing/background-blend-mode-gradient-color.html:
css3/compositing/background-blend-mode-gradient-gradient.html:
css3/compositing/background-blend-mode-gradient-image.html:
css3/compositing/background-blend-mode-image-color.html:
css3/compositing/background-blend-mode-image-image.html:
css3/compositing/background-blend-mode-image-svg.html:
css3/compositing/background-blend-mode-multiple-background-layers.html:
css3/compositing/background-blend-mode-property-expected.txt:
css3/compositing/background-blend-mode-property-parsing-expected.txt:
css3/compositing/background-blend-mode-svg-color.html:
css3/compositing/effect-background-blend-mode-tiled.html:
css3/compositing/effect-background-blend-mode.html:
css3/compositing/script-tests/background-blend-mode-property-parsing.js:
css3/compositing/script-tests/background-blend-mode-property.js:
platform/mac/css3/compositing/background-blend-mode-gif-color-2-expected.png:
platform/mac/css3/compositing/background-blend-mode-gif-color-2-expected.txt:
platform/mac/css3/compositing/background-blend-mode-gif-color-expected.png:
platform/mac/css3/compositing/background-blend-mode-gif-color-expected.txt:
platform/mac/css3/compositing/background-blend-mode-gradient-color-expected.png:
platform/mac/css3/compositing/background-blend-mode-gradient-gradient-expected.png:
platform/mac/css3/compositing/background-blend-mode-gradient-image-expected.png:
platform/mac/css3/compositing/background-blend-mode-image-color-expected.png:
platform/mac/css3/compositing/background-blend-mode-image-color-expected.txt:
platform/mac/css3/compositing/background-blend-mode-image-image-expected.png:
platform/mac/css3/compositing/background-blend-mode-image-image-expected.txt:
platform/mac/css3/compositing/background-blend-mode-image-svg-expected.png:
platform/mac/css3/compositing/background-blend-mode-multiple-background-layers-expected.png:
platform/mac/css3/compositing/background-blend-mode-svg-color-expected.png:

1:59 AM Changeset in webkit [168166] by gyuyoung.kim@samsung.com

15 edits in trunk/Source/WebCore

Clean up #include <OwnPtr.h>|<PassOwnPtr.h> in Supplementable classes
https://bugs.webkit.org/show_bug.cgi?id=132466

Reviewed by Tim Horton.

Since r168144, Supplementable classes don't need to include OwnPtr.h or PassOwnPtr.h.
Clean up those inclusions.

No new tests, just clean up patch.

Modules/encryptedmedia/CDMPrivate.h:
Modules/gamepad/NavigatorGamepad.cpp:
Modules/geolocation/GeolocationController.cpp:

(WebCore::GeolocationController::create): Deleted. Don't need to have a factory function.

Modules/geolocation/GeolocationController.h:
Modules/mediasource/MediaSource.h:
Modules/mediastream/UserMediaController.h:
Modules/notifications/NotificationCenter.h:
Modules/notifications/NotificationController.cpp:
Modules/quota/StorageErrorCallback.h:
Modules/vibration/Vibration.h:
Modules/webdatabase/Database.cpp:
Modules/websockets/ThreadableWebSocketChannelClientWrapper.h:
Modules/websockets/WebSocket.cpp:
Modules/websockets/WebSocket.h:

12:39 AM Changeset in webkit [168165] by bshafiei@apple.com

4 edits in tags/Safari-538.33.1/Source/WebKit

Merged r168138.

12:35 AM Changeset in webkit [168164] by bshafiei@apple.com

1 edit in tags/Safari-538.33.1/Source/WebKit/WebKit.xcodeproj/project.pbxproj

Re-merged r168073.

12:34 AM WebKitGTK/2.4.x edited by berto@igalia.com

(diff)

12:34 AM Changeset in webkit [168163] by bshafiei@apple.com

2 edits in tags/Safari-538.33.1/Source/WebKit

Re-merged r168072.

12:33 AM Changeset in webkit [168162] by bshafiei@apple.com

2 edits in tags/Safari-538.33.1/Source/WebKit

Re-merged r168071.

12:32 AM Changeset in webkit [168161] by bshafiei@apple.com

2 edits in tags/Safari-538.33.1/Source/WebKit

Re-merged r168062.

12:30 AM Changeset in webkit [168160] by bshafiei@apple.com

2 edits in tags/Safari-538.33.1/Source/WebKit

Re-merged r168061.

12:27 AM Changeset in webkit [168159] by bshafiei@apple.com

2 edits in tags/Safari-538.33.1/Source/WebKit

Re-merged r168058.

12:24 AM Changeset in webkit [168158] by bshafiei@apple.com

132 edits
4 copies
2 deletes in tags/Safari-538.33.1

Re-merged r168047.

May 1, 2014:

11:23 PM Changeset in webkit [168157] by timothy_horton@apple.com

2 edits in trunk/Source/WebKit/mac

More 32-bit build fixes.

Misc/WebSharingServicePickerController.mm:

(-[WebSharingServicePickerController sharingService:didShareItems:]):

11:17 PM Changeset in webkit [168156] by bshafiei@apple.com

5 edits in tags/Safari-538.33.1/Source

Versioning.

11:15 PM Changeset in webkit [168155] by bshafiei@apple.com

1 copy in tags/Safari-538.33.1

New tag.

10:55 PM Changeset in webkit [168154] by mitz@apple.com

1 edit in trunk/Source/bmalloc/ChangeLog

Added Radar link to the last ChangeLog entry.

10:55 PM Changeset in webkit [168153] by timothy_horton@apple.com

2 edits in trunk/Source/WebKit/mac

Temporarily "fix" the 32-bit build.

Misc/WebSharingServicePickerController.mm:

This will fail miserably at runtime, but we shouldn't ever get here in a 32-bit process.

10:53 PM Changeset in webkit [168152] by mitz@apple.com

2 edits in trunk/Source/bmalloc

Fixed production builds for the iOS Simulator.

Configurations/bmalloc.xcconfig: Include INSTALL_PATH_PREFIX in

PRIVATE_HEADERS_FOLDER_PATH when installing.

10:49 PM Changeset in webkit [168151] by ryuan.choi@samsung.com

6 edits in trunk/Source/WebCore

Remove TiledBackingStore stuff from Frame
https://bugs.webkit.org/show_bug.cgi?id=132458

Reviewed by Andreas Kling.

Although TiledBackingStore is using for the CoordinatedGraphics,
CoordinatedGraphics does not use tiledBackingStore of Frame.

This patch removed TiledBackingStore related code of the Frame.

page/Frame.cpp:

(WebCore::Frame::Frame):
(WebCore::Frame::setView):
(WebCore::Frame::setTiledBackingStoreEnabled): Deleted.
(WebCore::Frame::tiledBackingStorePaintBegin): Deleted.
(WebCore::Frame::tiledBackingStorePaint): Deleted.
(WebCore::Frame::tiledBackingStorePaintEnd): Deleted.
(WebCore::Frame::tiledBackingStoreContentsRect): Deleted.
(WebCore::Frame::tiledBackingStoreVisibleRect): Deleted.
(WebCore::Frame::tiledBackingStoreBackgroundColor): Deleted.

page/Frame.h:

(WebCore::Frame::tiledBackingStore): Deleted.

page/FrameView.cpp:

(WebCore::FrameView::repaintContentRectangle):

page/Settings.cpp:

(WebCore::Settings::Settings):
(WebCore::Settings::setTiledBackingStoreEnabled): Deleted.

page/Settings.h:

(WebCore::Settings::tiledBackingStoreEnabled): Deleted.

10:19 PM Changeset in webkit [168150] by ap@apple.com

3 edits in trunk/LayoutTests

REGRESSION(r125251): It made svg/custom/use-instanceRoot-as-event-target.xhtml assert and flakey
https://bugs.webkit.org/show_bug.cgi?id=93812

This is most likely fixed, unmarking the test.

platform/gtk/TestExpectations:
platform/mac/TestExpectations:

9:28 PM Changeset in webkit [168149] by bshafiei@apple.com

2 edits in tags/Safari-538.30.3/WebKitLibraries

Merged r168143.

9:23 PM Changeset in webkit [168148] by mmaxfield@apple.com

5 edits in trunk/Source/WebCore

Migrate all uses of DeviceMotionController and DeviceOrientationController to std::unique_ptr
https://bugs.webkit.org/show_bug.cgi?id=132461

Unreviewed build fix.

No new tests.

dom/Document.cpp:

(WebCore::Document::Document):

dom/Document.h:
platform/ios/DeviceMotionClientIOS.h:

(WebCore::DeviceMotionClientIOS::create): Deleted.

platform/ios/DeviceOrientationClientIOS.h:

(WebCore::DeviceOrientationClientIOS::create): Deleted.

9:12 PM Changeset in webkit [168147] by bshafiei@apple.com

5 edits in tags/Safari-538.30.3/Source

Versioning.

9:09 PM Changeset in webkit [168146] by bshafiei@apple.com

1 copy in tags/Safari-538.30.3

New tag.

8:31 PM Changeset in webkit [168145] by beidson@apple.com

4 edits in trunk/Source/WebKit/mac

Update service picker API usage.
<rdar://problem/16772674> and https://bugs.webkit.org/show_bug.cgi?id=132452

Reviewed by Tim Horton.

Misc/WebSharingServicePickerController.h:
Misc/WebSharingServicePickerController.mm:

(-[WebSharingServicePickerController didShareImageData:confirmDataIsValidTIFFData:]):

Factor out a common "didShare" handler that optionally validates whether the data represents an image.

(-[WebSharingServicePickerController sharingService:didShareItems:]): Update API usage, including marshalling

on off-main thread call back to the main thread.

WebCoreSupport/WebContextMenuClient.mm:

(WebContextMenuClient::contextMenuForEvent): Update API usage.

7:28 PM Changeset in webkit [168144] by gyuyoung.kim@samsung.com

48 edits in trunk/Source/WebCore

Convert OwnPtr and PassOwnPtr uses to std::unique_ptr in Supplement
https://bugs.webkit.org/show_bug.cgi?id=132165

Reviewed by Darin Adler.

According to convert from PassOwnPtr to std::unique_ptr, provideTo() callers also begin
to use std::make_unique.

Modules/battery/BatteryController.cpp:

(WebCore::provideBatteryTo):

Modules/battery/BatteryController.h:
Modules/battery/NavigatorBattery.cpp:

(WebCore::NavigatorBattery::from):

Modules/gamepad/NavigatorGamepad.cpp:

(WebCore::NavigatorGamepad::from):

Modules/geolocation/GeolocationController.cpp:

(WebCore::provideGeolocationTo):

Modules/geolocation/GeolocationController.h:
Modules/geolocation/NavigatorGeolocation.cpp:

(WebCore::NavigatorGeolocation::from):

Modules/indexeddb/DOMWindowIndexedDatabase.cpp:

(WebCore::DOMWindowIndexedDatabase::from):

Modules/indexeddb/PageGroupIndexedDatabase.cpp:

(WebCore::PageGroupIndexedDatabase::from):

Modules/indexeddb/WorkerGlobalScopeIndexedDatabase.cpp:

(WebCore::WorkerGlobalScopeIndexedDatabase::from):

Modules/mediastream/UserMediaController.cpp:

(WebCore::provideUserMediaTo):
(WebCore::UserMediaController::create): Deleted.

Modules/mediastream/UserMediaController.h:
Modules/navigatorcontentutils/NavigatorContentUtils.cpp:

(WebCore::provideNavigatorContentUtilsTo):
(WebCore::NavigatorContentUtils::create): Deleted.

Modules/navigatorcontentutils/NavigatorContentUtils.h:
Modules/notifications/DOMWindowNotifications.cpp:

(WebCore::DOMWindowNotifications::from):

Modules/notifications/NotificationController.cpp:

(WebCore::provideNotification):
(WebCore::NotificationController::create): Deleted.

Modules/notifications/NotificationController.h:
Modules/notifications/WorkerGlobalScopeNotifications.cpp:

(WebCore::WorkerGlobalScopeNotifications::from):

Modules/proximity/DeviceProximityController.cpp:

(WebCore::provideDeviceProximityTo):
(WebCore::DeviceProximityController::create): Deleted.

Modules/proximity/DeviceProximityController.h:
Modules/quota/DOMWindowQuota.cpp:

(WebCore::DOMWindowQuota::from):

Modules/quota/NavigatorStorageQuota.cpp:

(WebCore::NavigatorStorageQuota::from):

Modules/quota/WorkerNavigatorStorageQuota.cpp:

(WebCore::WorkerNavigatorStorageQuota::from):

Modules/speech/DOMWindowSpeechSynthesis.cpp:

(WebCore::DOMWindowSpeechSynthesis::from):

Modules/speech/SpeechRecognitionController.cpp:

(WebCore::provideSpeechRecognitionTo):

Modules/vibration/Vibration.cpp:

(WebCore::provideVibrationTo):
(WebCore::Vibration::create): Deleted.

Modules/vibration/Vibration.h:
dom/DeviceMotionController.cpp:

(WebCore::provideDeviceMotionTo):
(WebCore::DeviceMotionController::create): Deleted.

dom/DeviceMotionController.h:
dom/DeviceOrientationController.cpp:

(WebCore::provideDeviceOrientationTo):
(WebCore::DeviceOrientationController::create): Deleted.

dom/DeviceOrientationController.h:
page/SpeechInput.cpp:

(WebCore::provideSpeechInputTo):
(WebCore::SpeechInput::create): Deleted.

page/SpeechInput.h:
platform/Supplementable.h:

(WebCore::Supplement::provideTo):
(WebCore::Supplementable::provideSupplement):

testing/InternalSettings.cpp:

(WebCore::InternalSettings::from):

6:45 PM Changeset in webkit [168143] by Brent Fulgham

2 edits in trunk/WebKitLibraries

Correct case of environment variables in auto-version.sh scripts
https://bugs.webkit.org/show_bug.cgi?id=132455

Reviewed by Dean Jackson.

Although we have historically used RC_PROJECTSOURCEVERSION in our scripts,
the actual environment variable is RC_ProjectSourceVersion. Old versions of
Cygwin converted this to all-caps by default, but this is no longer the case.
We need to use the proper case to avoid build failures.

win/tools/scripts/auto-version.sh: Although we have historically coded

this as 'RC_PROJECTSOURCEVERSION', it is actually 'RC_ProjectSourceVersion'.

6:40 PM Changeset in webkit [168142] by enrica@apple.com

2 edits in trunk/Source/WebKit2

REGRESSION (WebKit2) Need to support Learn button.
https://bugs.webkit.org/show_bug.cgi?id=132454
<rdar://problem/16778889>

Reviewed by Benjamin Poulain.

For traditional Chinese we support the ability to add shortcuts
for typing. This is done by pressing the Learn button on the system
menu. This patch adds the necessary code to canPerformAction to enable
the button when appropriate as well as the implementation of the action
itself.
It also adds a check for the Replace button not to be shown when the
selection only contains CJ characters.

UIProcess/ios/WKContentViewInteraction.mm:

(-[WKContentView _addShortcut:]):
(-[WKContentView canPerformAction:withSender:]):

6:28 PM Changeset in webkit [168141] by Lucas Forschler

3 edits in tags/Safari-538.33/Source/WebCore

Merged r168113.

6:26 PM Changeset in webkit [168140] by Lucas Forschler

5 edits in tags/Safari-538.33

Merged r168088.

6:21 PM Changeset in webkit [168139] by ryuan.choi@samsung.com

7 edits in trunk

[EFL][WK1] Drop ewk_view_setting_tiled_backingstore APIs
https://bugs.webkit.org/show_bug.cgi?id=132240

Reviewed by Anders Carlsson.

Source/WebKit/efl:
Since we moved to use TextureMapper, this option is meaningless now.
Removed ewk_view_setting_tiled_backing_store_enabled_{get|set} and related codes.

ewk/ewk_view.cpp:

(ewk_view_setting_tiled_backing_store_enabled_set): Deleted.
(ewk_view_setting_tiled_backing_store_enabled_get): Deleted.

ewk/ewk_view.h:
tests/test_ewk_view.cpp:

(TEST_F): Deleted.

Tools:
Removed ewk_view_setting_tiled_backingstore related code.

DumpRenderTree/efl/DumpRenderTreeChrome.cpp:

(DumpRenderTreeChrome::createView):
(shouldUseTiledBackingStore): Deleted.

EWebLauncher/main.c:

(windowCreate):
(parseUserArguments):

5:34 PM Changeset in webkit [168138] by andersca@apple.com

4 edits in trunk/Source/WebKit

Support OS-version-specific install paths for WebKit.framework
https://bugs.webkit.org/show_bug.cgi?id=132448
<rdar://problem/16784932>

Reviewed by Dan Bernstein.

Source/WebKit:

WebKit.xcodeproj/project.pbxproj:

Add a build step to put a symlink in place from PrivateFrameworks to Frameworks.

Source/WebKit/mac:

WebKitLegacy/WebKit.m:

Add version specific install names.

5:28 PM WebKitGTK/KeepingTheTreeGreen edited by clopez@igalia.com

(diff)

5:04 PM Changeset in webkit [168137] by akling@apple.com

3 edits in trunk/Source/WebCore

HTMLMediaElement: Remove two unnecessary virtual overrides.
<https://webkit.org/b/132445>

Now that ENABLE_PLUGIN_PROXY_FOR_VIDEO is gone, we can remove
these overrides:

defaultEventHandler()
willRespondToMouseClickEvents()

Reviewed by Darin Adler.

html/HTMLMediaElement.cpp:
html/HTMLMediaElement.h:

4:59 PM Changeset in webkit [168136] by benjamin@webkit.org

2 edits in trunk/Source/WebCore

ViewportConfiguration::minimumScale() uses the initial scale as initial value
https://bugs.webkit.org/show_bug.cgi?id=132451
<rdar://problem/16780111>

Patch by Benjamin Poulain <bpoulain@apple.com> on 2014-05-01
Reviewed by Enrica Casucci.

page/ViewportConfiguration.cpp:

(WebCore::ViewportConfiguration::minimumScale):
The initial minimum scale was set to the initial scale, preventing some pages from zooming
out.

4:59 PM Changeset in webkit [168135] by benjamin@webkit.org

2 edits in trunk/Source/WebKit2

[iOS][WK2] Tweak the extended background exclusion for MobileSafari
https://bugs.webkit.org/show_bug.cgi?id=132449

Patch by Benjamin Poulain <bpoulain@apple.com> on 2014-05-01
Reviewed by Beth Dakin.

Some tweaks for Mobile:
-Use UIViews instead of CALayers to have the same animation timing as the top views.
-The left extended background insets should not exclude the top and bottom insets.

Since this code is in the middle of 2 hot paths, also added some performance tweaks.

UIProcess/API/Cocoa/WKWebView.mm:

(-[WKWebView initWithFrame:configuration:]):
In the normal case, we do not have extended background exclusion. To avoid creating a background view, we use
the scrollview to render the background.
The separate background view is created lazily if needed.

(-[WKWebView _updateScrollViewBackground]):
(-[WKWebView _frameOrBoundsChanged]):
(updateTopAndBottomExtendedBackgroundExclusionIfNecessary):
This create the top and bottom extended background view as needed and update their frames for the current insets.

(-[WKWebView _setObscuredInsets:]):
(-[WKWebView _setExtendedBackgroundExclusionInsets:]):
When an exclusion inset is needed, create a view for it, transfer the color from the ScrollView, and reset the color
of the scrollview.

4:55 PM Changeset in webkit [168134] by Lucas Forschler

132 edits
2 copies
4 deletes in tags/Safari-538.33

Rollout of r168047.

4:53 PM Changeset in webkit [168133] by Lucas Forschler

2 edits in tags/Safari-538.33/Source/WebKit

Rollout of r168058.

4:52 PM Changeset in webkit [168132] by Lucas Forschler

2 edits in tags/Safari-538.33/Source/WebKit

Rollout of r168061.

4:52 PM Changeset in webkit [168131] by Lucas Forschler

2 edits in tags/Safari-538.33/Source/WebKit

Rollout of r168062.

4:49 PM Changeset in webkit [168130] by Lucas Forschler

2 edits in tags/Safari-538.33/Source/WebKit

Rollout of r168071.

4:48 PM Changeset in webkit [168129] by Lucas Forschler

2 edits in tags/Safari-538.33/Source/WebKit

Rollout of r168072.

4:48 PM Changeset in webkit [168128] by Lucas Forschler

2 edits in tags/Safari-538.33/Source/WebKit

Rollout of r168073.

4:19 PM Changeset in webkit [168127] by ap@apple.com

2 edits in trunk/Source/WebCore

Build fix for !ENABLE(BLOB) builds.

fileapi/ThreadableBlobRegistry.cpp:

(WebCore::ThreadableBlobRegistry::registerBlobURL):
(WebCore::ThreadableBlobRegistry::registerBlobURLForSlice):

4:09 PM Changeset in webkit [168126] by ap@apple.com

23 edits in trunk/Source

Move size computation for Blob constructor into BlobRegistryImpl
https://bugs.webkit.org/show_bug.cgi?id=132439

Reviewed by Sam Weinig.

Source/WebCore:

Modules/websockets/WebSocket.cpp: (WebCore::WebSocket::didReceiveBinaryData):

Don't pass the size, anyone who cares can get it from BlobData.

bindings/js/JSBlobCustom.cpp: (WebCore::JSBlobConstructor::constructJSBlob):

Updated for BlobBuilder changes.

fileapi/Blob.cpp: (WebCore::Blob::Blob):
fileapi/Blob.h: (WebCore::Blob::create):

No longer take a precomputed size with BlobData, BlobRegistry will compute it as
part of registration.

fileapi/File.h:
fileapi/File.cpp:

(WebCore::File::File): Don't pass a size (that's unknown anyway).
(WebCore::File::captureSnapshot): Deleted. Finally, only the registry is responsible
for snapshot tracking now (I doubt that either new or old code is particularly compliant).

fileapi/ThreadableBlobRegistry.cpp:

(WebCore::ThreadableBlobRegistry::registerBlobURL):

fileapi/ThreadableBlobRegistry.h:

Plumbing to make this version of registerBlobURL return a size. I may make size
calculation lazy and the function async later, but this is needed to move the behavior
to the right place first.

fileapi/WebKitBlobBuilder.h:
fileapi/WebKitBlobBuilder.cpp:

(WebCore::BlobBuilder::BlobBuilder):
(WebCore::BlobBuilder::append):
(WebCore::BlobBuilder::appendBytesData):
(WebCore::BlobBuilder::finalize):
(WebCore::BlobBuilder::getBlob): Deleted.
Don't track sizes or modification times, registry will do that with appropriate laziness.
Cleaned up the API - now that BlobBuilder is not exposed to JS as an object, it
does not need to be reusable.

platform/network/BlobRegistry.h: Made this version of registerBlobURL return a size.
platform/network/BlobRegistryImpl.h:
platform/network/BlobRegistryImpl.cpp:

(WebCore::BlobRegistryImpl::appendStorageItems): Assert that length computations are accurate.
(WebCore::BlobRegistryImpl::registerBlobURL): Compute a size to return, and record
modification time as necessary.

xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::responseBlob): Don't pass the

size, which can be computed from data. Soon, I want to add a constructor that doesn't
require wrapping a single Vector as BlobData to construct a Blob.

Source/WebKit2:
Plumbing to make this version of registerBlobURL synchronous for now.
I expect to make it async again when data structures on client side are simplified.

NetworkProcess/FileAPI/NetworkBlobRegistry.cpp:

(WebKit::NetworkBlobRegistry::registerBlobURL):

NetworkProcess/FileAPI/NetworkBlobRegistry.h:
NetworkProcess/NetworkConnectionToWebProcess.cpp:

(WebKit::NetworkConnectionToWebProcess::registerBlobURL):

NetworkProcess/NetworkConnectionToWebProcess.h:
NetworkProcess/NetworkConnectionToWebProcess.messages.in:
WebProcess/FileAPI/BlobRegistryProxy.cpp:

(WebKit::BlobRegistryProxy::registerBlobURL):

WebProcess/FileAPI/BlobRegistryProxy.h:

3:54 PM Changeset in webkit [168125] by eric.carlson@apple.com

2 edits in trunk/Source/WebCore

[iOS] audio and video should automatically play to active external device
https://bugs.webkit.org/show_bug.cgi?id=132428

Reviewed by Jer Noble.

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

(WebCore::MediaPlayerPrivateAVFoundationObjC::createAVPlayer): Set

AVPlayer.usesExternalPlaybackWhileExternalScreenIsActive.

3:53 PM Changeset in webkit [168124] by Beth Dakin

2 edits in trunk/Source/WebKit2

REGRESSION(168053): Repro crash navigating to another page after selecting phone
numbers on a page
https://bugs.webkit.org/show_bug.cgi?id=132444
-and corresponding-
<rdar://problem/16787285>

Reviewed by Darin Adler.

Missing null-check.

WebProcess/WebPage/mac/TelephoneNumberOverlayControllerMac.mm:

(WebKit::TelephoneNumberOverlayController::drawRect):

3:03 PM Changeset in webkit [168123] by andersca@apple.com

8 edits in trunk/Source/WebKit2

Remove WKBundleSetShouldTrackVisitedLinks and associate code
https://bugs.webkit.org/show_bug.cgi?id=132441

Reviewed by Sam Weinig.

WebProcess/InjectedBundle/API/c/WKBundle.cpp:

(WKBundleSetShouldTrackVisitedLinks): Deleted.

WebProcess/InjectedBundle/API/c/WKBundlePrivate.h:
WebProcess/InjectedBundle/InjectedBundle.cpp:

(WebKit::InjectedBundle::setShouldTrackVisitedLinks): Deleted.

WebProcess/InjectedBundle/InjectedBundle.h:
WebProcess/WebPage/VisitedLinkTableController.cpp:

(WebKit::VisitedLinkTableController::addVisitedLink):

WebProcess/WebProcess.cpp:

(WebKit::WebProcess::WebProcess):
(WebKit::WebProcess::setShouldTrackVisitedLinks): Deleted.

WebProcess/WebProcess.h:

(WebKit::WebProcess::shouldTrackVisitedLinks): Deleted.

2:57 PM WebKitGTK/KeepingTheTreeGreen edited by clopez@igalia.com

(diff)

2:53 PM Changeset in webkit [168122] by andersca@apple.com

8 edits in trunk

window.testRunner.keepWebHistory() should update the UI process state
https://bugs.webkit.org/show_bug.cgi?id=132440

Reviewed by Dan Bernstein.

Source/WebKit2:

UIProcess/API/C/WKPage.cpp:

(WKPageGetAddsVisitedLinks):
(WKPageSetAddsVisitedLinks):

UIProcess/API/C/WKPagePrivate.h:

Tools:

WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:

(WTR::InjectedBundle::postSetAddsVisitedLinks):

WebKitTestRunner/InjectedBundle/InjectedBundle.h:
WebKitTestRunner/InjectedBundle/TestRunner.cpp:

(WTR::TestRunner::keepWebHistory):

WebKitTestRunner/TestInvocation.cpp:

(WTR::TestInvocation::invoke):
(WTR::TestInvocation::didReceiveMessageFromInjectedBundle):

2:44 PM WebKitGTK/KeepingTheTreeGreen edited by clopez@igalia.com

(diff)

2:00 PM Changeset in webkit [168121] by hyatt@apple.com

3 edits in trunk/Source/WebCore

REGRESSION (r168046): [New Multicolumn] Selection into and out of column-span elements doesn't work
https://bugs.webkit.org/show_bug.cgi?id=132066

Reviewed by Oliver Hunt.

Make a new SelectionIterator struct that knows how to drill into and out of
column span placeholders. Also change spans to be selection roots (in the painting
sense).

rendering/RenderBlock.cpp:

(WebCore::RenderBlock::isSelectionRoot):
(WebCore::RenderBlock::selectionGaps):

rendering/RenderView.cpp:

(WebCore::SelectionIterator::SelectionIterator):
(WebCore::SelectionIterator::checkForSpanner):
(WebCore::SelectionIterator::current):
(WebCore::SelectionIterator::next):
(WebCore::RenderView::subtreeSelectionBounds):
(WebCore::RenderView::repaintSubtreeSelection):
(WebCore::RenderView::setSubtreeSelection):

1:56 PM Changeset in webkit [168120] by fpizlo@apple.com

3 edits in trunk/Source/WebKit2

Roll out r60161.

Rubber stamped by Mark Hahnenberg.

This breaks our debugging workflow.

Shared/WebProcessCreationParameters.cpp:

(WebKit::WebProcessCreationParameters::decode):

UIProcess/mac/WebContextMac.mm:

(WebKit::registerUserDefaultsIfNeeded):
(WebKit::WebContext::platformInitializeWebProcess):

1:56 PM Changeset in webkit [168119] by Simon Fraser

9 edits
6 adds in trunk

Don't always make backing store for -webkit-backface-visibility:hidden
https://bugs.webkit.org/show_bug.cgi?id=132420

Reviewed by Sam Weinig.

Source/WebCore:
Previously, -webkit-backface-visibility:hidden unconditionally created
compositing layers with backing store. This results in high memory use
on pages with this style applied to many elements (a cargo-cult "optimization").

Fix by only having -webkit-backface-visibility:hidden create compositing layers
if some ancestor has a 3D transform. That's the only scenario in which the
element can be flipped around to reveal the back side, so the only time we need
to do compositing for this property. In future, we could be smarter, and only
consider 3D transforms in the current preserve-3d context.

Tests: compositing/backing/backface-visibility-in-3dtransformed.html

compositing/backing/backface-visibility-in-transformed.html
compositing/backing/backface-visibility.html

rendering/RenderLayer.cpp:

(WebCore::RenderLayer::RenderLayer):
(WebCore::RenderLayer::updateLayerPositions):
(WebCore::RenderLayer::hitTestLayer):

rendering/RenderLayer.h:
rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::requiresCompositingLayer):
(WebCore::RenderLayerCompositor::requiresOwnBackingStore):
(WebCore::RenderLayerCompositor::requiresCompositingForBackfaceVisibility):

rendering/RenderLayerCompositor.h:

LayoutTests:
Dump layers for elements with backface-visibility: hidden with various types
of ancestors.

compositing/backing/backface-visibility-expected.txt: Added.
compositing/backing/backface-visibility-in-3dtransformed-expected.txt: Added.
compositing/backing/backface-visibility-in-3dtransformed.html: Added.
compositing/backing/backface-visibility-in-transformed-expected.txt: Added.
compositing/backing/backface-visibility-in-transformed.html: Added.
compositing/backing/backface-visibility.html: Added.
inspector-protocol/layers/layers-anonymous.html: Don't use backface-visibility

for force a layer.

1:39 PM Changeset in webkit [168118] by andersca@apple.com

21 edits in trunk/Source/WebKit2

WKWebView doesn't track visited links (for visited link coloring)
https://bugs.webkit.org/show_bug.cgi?id=132438
<rdar://problem/16704519>

Reviewed by Dan Bernstein.

Shared/WebProcessCreationParameters.cpp:

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

Shared/WebProcessCreationParameters.h:
UIProcess/API/APIHistoryClient.h:

(API::HistoryClient::addsVisitedLinks):
(API::HistoryClient::shouldTrackVisitedLinks): Deleted.

UIProcess/API/C/WKContext.cpp:

(WKContextSetHistoryClient):

UIProcess/API/Cocoa/WKWebView.mm:

(-[WKWebView _addsVisitedLinks]):
(-[WKWebView _setAddsVisitedLinks:]):

UIProcess/API/Cocoa/WKWebViewPrivate.h:
UIProcess/API/mac/WKView.mm:

(-[WKView initWithFrame:context:configuration:webView:]):

UIProcess/Cocoa/HistoryClient.h:
UIProcess/Cocoa/HistoryClient.mm:

(WebKit::HistoryClient::shouldTrackVisitedLinks): Deleted.

UIProcess/VisitedLinkProvider.cpp:

(WebKit::VisitedLinkProvider::addVisitedLinkHashFromPage):

UIProcess/VisitedLinkProvider.h:
UIProcess/VisitedLinkProvider.messages.in:
UIProcess/WebContext.cpp:

(WebKit::WebContext::setHistoryClient):
(WebKit::WebContext::createNewWebProcess):

UIProcess/WebContext.h:

(WebKit::WebContext::processes):

UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::WebPageProxy):

UIProcess/WebPageProxy.h:

(WebKit::WebPageProxy::addsVisitedLinks):
(WebKit::WebPageProxy::setAddsVisitedLinks):

WebProcess/WebPage/VisitedLinkTableController.cpp:

(WebKit::VisitedLinkTableController::addVisitedLink):

WebProcess/WebProcess.cpp:

(WebKit::WebProcess::initializeWebProcess):

WebProcess/WebProcess.h:
WebProcess/WebProcess.messages.in:

1:32 PM Changeset in webkit [168117] by benjamin@webkit.org

3 edits in trunk/Source/WebKit2

[iOS][WK2] The highlight view needs to be in WKWebView coordinates
https://bugs.webkit.org/show_bug.cgi?id=132435
<rdar://problem/16708861>

Patch by Benjamin Poulain <bpoulain@apple.com> on 2014-05-01
Reviewed by Tim Horton.

_UIHighlightView needs to be in WKWebView coordinates so that it render unscaled for any page scale factor.
The view needs to be a child of WKContentView so that it moves/scales with the page.

To fix the issue, add an inverse transform root layer for the hightlight, and scale the coordinates to their
inverse scaled counterpart.

The scale is not updated live with the scaling of WKContentView but that should be fine since the view disappear
on any scaling operation (and I would prefer not add live painting during scaling animation).

UIProcess/ios/WKContentViewInteraction.h:
UIProcess/ios/WKContentViewInteraction.mm:

(-[WKContentView cleanupInteraction]):
(-[WKContentView _didGetTapHighlightForRequest:color:WebCore::quads:WebCore::topLeftRadius:WebCore::topRightRadius:WebCore::bottomLeftRadius:WebCore::bottomRightRadius:WebCore::]):
(-[WKContentView _cancelInteraction]):

1:03 PM Changeset in webkit [168116] by ggaren@apple.com

2 edits in trunk/Source/JavaScriptCore

JavaScriptCore fails to build with some versions of clang
https://bugs.webkit.org/show_bug.cgi?id=132436

Reviewed by Anders Carlsson.

runtime/ArgumentsIteratorConstructor.cpp: Since we call

putDirectWithoutTransition, and it calls putWillGrowOutOfLineStorage,
and both are marked inline, it's valid for the compiler to decide
to inline both and emit neither in the binary. Therefore, we need
both inline definitions to be available in the translation unit at
compile time, or we'll try to link against a function that doesn't exist.

12:24 PM Changeset in webkit [168115] by achristensen@apple.com

6 edits
80 adds
24 deletes in trunk/Source

Finish updating ANGLE.
https://bugs.webkit.org/show_bug.cgi?id=132434

Reviewed by Dean Jackson.

Source/ThirdParty/ANGLE:

ANGLE.xcodeproj/project.pbxproj:

Removed Uniform.cpp which is no longer in ANGLE.

changes.diff:

Added more changes from ANGLE which are already included in WebKit.

Updated ANGLE source files to e7a453a5bd76705ccb151117fa844846d4aa90af. Long list of changes omitted.

Source/WebCore:

CMakeLists.txt:

Removed Uniform.cpp which is no longer in ANGLE.

12:23 PM Changeset in webkit [168114] by matthew_hanson@apple.com

11 edits in tags/Safari-538.33

Rollout r167964.

12:10 PM Changeset in webkit [168113] by hyatt@apple.com

3 edits in trunk/Source/WebCore

REGRESSION (new multi-column): WebKit2.ResizeReversePaginatedWebView fails on debug bots
https://bugs.webkit.org/show_bug.cgi?id=132429

Reviewed by Alexey Proskuryakov.

Make sure to get the column count directly from the column set.
I was returning the theoretical column count from the flow thread,
and that wasn't the right value.

rendering/RenderMultiColumnSet.h:
rendering/RenderView.cpp:

(WebCore::RenderView::pageCount):

12:02 PM Changeset in webkit [168112] by Brent Fulgham

6 edits
2 adds in trunk

Fix handling of attributes prior to compiling shader
https://bugs.webkit.org/show_bug.cgi?id=132430

Reviewed by Dean Jackson.

Source/WebCore:
WebGL programs that called bindAttribLocations prior to compiling shader sources
would perform the bind using the non-hashed symbol name, but would later create
the attributes as hashed names. Consequently, the program would refer to
attributes that were never actually part of any shader, resulting in some amazing
display artifacts.

This patch adds a dictionary of hashed symbol names so that we can tell the WebGL
program the proper name that will be used when the shader is eventually compiled,
allowing the WebGL program to link against the proper symbol after compiling and
linking completes.

platform/graphics/GraphicsContext3D.h:
platform/graphics/opengl/GraphicsContext3DOpenGLCommon.cpp:

(WebCore::generateHashedName): Function uses the ANGLE hashing
function to generate correct symbol.
(WebCore::GraphicsContext3D::mappedSymbolName): If we haven't compiled shaders yet, look
in our set of potentially unused attributes.
(WebCore::GraphicsContext3D::originalSymbolName): Ditto, for reverse lookup.

Source/WTF:
WebGL programs that called bindAttribLocations prior to compiling shader sources
would perform the bind using the non-hashed symbol name, but would later create
the attributes as hashed names. Consequently, the program would refer to
attributes that were never actually part of any shader, resulting in some amazing
display artifacts.

wtf/HexNumber.h:

(WTF::appendUnsigned64AsHex): Add uint64_t-compatible hex->string converter.

LayoutTests:

fast/canvas/webgl/gl-bind-attrib-location-before-compile-test-expected.txt: Added.
fast/canvas/webgl/gl-bind-attrib-location-before-compile-test.html: Added.

11:49 AM Changeset in webkit [168111] by Alan Bujtas

3 edits
2 adds in trunk

Subpixel rendering: Make selection gaps painting subpixel aware.
https://bugs.webkit.org/show_bug.cgi?id=132169

Reviewed by Simon Fraser.

Push selection gaps painting to device pixel boundaries instead of integral CSS pixel positions.

Source/WebCore:
Test: fast/inline/hidpi-selection-gap-on-subpixel-position.html

rendering/RenderBlock.cpp:

(WebCore::RenderBlock::blockSelectionGap):
(WebCore::RenderBlock::logicalLeftSelectionGap):
(WebCore::RenderBlock::logicalRightSelectionGap):

LayoutTests:

fast/inline/hidpi-selection-gap-on-subpixel-position-expected.html: Added.
fast/inline/hidpi-selection-gap-on-subpixel-position.html: Added. : &nbsp is needed to make

this test pass on WK2. Font rendering reports differences. 0.9 transparency is added so that
text selection code does modify the color's alpha channel and I can properly match it.

11:31 AM Changeset in webkit [168110] by ggaren@apple.com

2 edits in trunk/Source/WTF

Link against bmalloc in production builds
https://bugs.webkit.org/show_bug.cgi?id=132413

Reviewed by Sam Weinig.

Production builders have been configured to handle this, so let's build
it.

Configurations/WTF.xcconfig:

11:20 AM Changeset in webkit [168109] by ddkilzer@apple.com

7 edits in trunk

Really remove ENABLE_PLUGIN_PROXY_FOR_VIDEO
<http://webkit.org/b/132432>

Reviewed by Tim Horton.

Source/cmake/WebKitFeatures.cmake:
Source/cmakeconfig.h.cmake:
Remove ENABLE_PLUGIN_PROXY_FOR_VIDEO as build option.

Source/WebCore:

platform/graphics/wince/MediaPlayerPrivateWinCE.h: Remove

methods and ivar in ENABLE(PLUGIN_PROXY_FOR_VIDEO).

Tools:

Scripts/webkitperl/FeatureList.pm: Remove support for

--plugin-proxy-for-video switch.

10:48 AM Changeset in webkit [168108] by commit-queue@webkit.org

3 edits
2 adds in trunk

[CSS Grid Layout] Clamping the number of repetitions in repeat()
https://bugs.webkit.org/show_bug.cgi?id=131023

Patch by Javier Fernandez <jfernandez@igalia.com> on 2014-05-01
Reviewed by Brent Fulgham.

Source/WebCore:
The ED suggests now to be able to clamp the number of repetitions when
using the repeat() function, taking precautions about excessive memory
usage.

The implemented max repetitions is 10K.

Test: fast/css-grid-layout/grid-element-repeat-max-repetitions.html

css/CSSParser.cpp:

(WebCore::CSSParser::parseGridTrackRepeatFunction):

LayoutTests:
Test to ensure the number of repetitions used in the repeat() function
is clamped to 10K.

fast/css-grid-layout/grid-element-repeat-max-repetitions-expected.txt: Added.
fast/css-grid-layout/grid-element-repeat-max-repetitions.html: Added.

10:26 AM Changeset in webkit [168107] by commit-queue@webkit.org

11 edits in trunk

Unreviewed, rolling out r167964.
https://bugs.webkit.org/show_bug.cgi?id=132431

Memory improvements should not regress memory usage (Requested
by olliej on #webkit).

Reverted changeset:

"Don't hold on to parameter BindingNodes forever"
https://bugs.webkit.org/show_bug.cgi?id=132360
http://trac.webkit.org/changeset/167964

9:42 AM Changeset in webkit [168106] by ap@apple.com

2 edits in trunk/LayoutTests

fast/multicol/fixed-stack.html failing since introduction.
https://bugs.webkit.org/show_bug.cgi?id=132421

TestExpectations: Changed expectation from Skip to ImageOnyFailure

9:38 AM Changeset in webkit [168105] by matthew_hanson@apple.com

7 edits
2 copies in branches/safari-537.76-branch

Merge r167295.

9:25 AM Changeset in webkit [168104] by matthew_hanson@apple.com

3 edits
2 copies in branches/safari-537.76-branch

Merge r166736.

9:06 AM Changeset in webkit [168103] by matthew_hanson@apple.com

3 edits
4 copies in branches/safari-537.76-branch

Merge r167480.

8:55 AM Changeset in webkit [168102] by matthew_hanson@apple.com

2 edits in branches/safari-537.76-branch/Source/WebCore

Merge r167524.

8:48 AM Changeset in webkit [168101] by fpizlo@apple.com

2 edits in trunk/Source/JavaScriptCore

Fix trivial debug-only race-that-crashes in CallLinkStatus and explain why the remaining races are totally awesome
https://bugs.webkit.org/show_bug.cgi?id=132427

Reviewed by Mark Hahnenberg.

bytecode/CallLinkStatus.cpp:

(JSC::CallLinkStatus::computeFor):

8:46 AM Changeset in webkit [168100] by matthew_hanson@apple.com

3 edits
2 copies in branches/safari-537.76-branch

Merge r167672.

8:43 AM Changeset in webkit [168099] by jer.noble@apple.com

2 edits in trunk/Source/WebCore

[MSE] Seeking between two buffered ranges enquues incorrect buffers.
https://bugs.webkit.org/show_bug.cgi?id=132416

Reviewed by Eric Carlson.

std::equal_range(begin, end, value) will return an empty range if equal values cannot
be found. But the range is not necessarily [end, end). It may be some other value n,
such that the empty range is [n, n). Check to see if the returned range is empty in
findSampleContainingPresentationTime() and its reverse version, and if so, explicitly
return presentationEnd() or reversePresentationEnd() respectively.

Drive-by fix: make the comparator functions take const& arguments to minimize object
creation.

Modules/mediasource/SampleMap.cpp:

(WebCore::SampleIsLessThanMediaTimeComparator::operator()):
(WebCore::SampleIsGreaterThanMediaTimeComparator::operator()):
(WebCore::SampleMap::findSampleContainingPresentationTime):
(WebCore::SampleMap::reverseFindSampleContainingPresentationTime):

8:39 AM Changeset in webkit [168098] by matthew_hanson@apple.com

4 edits in branches/safari-537.76-branch

Merge r167569 (committing on behalf of Dana Burkart.)

8:34 AM Changeset in webkit [168097] by commit-queue@webkit.org

128 edits
20 adds in trunk/LayoutTests

[GTK] Unreviewed GTK gardening.
Rebaseline affected tests by the new multi-column mode that was
enabled on r168046, and later modified on r168076 and r168088.

Patch by Carlos Alberto Lopez Perez <clopez@igalia.com> on 2014-05-01

platform/gtk/TestExpectations: Include two new flaky tests after

the new multi-colum mode and remove expectations for the ones that
now pass.

platform/gtk/css3/unicode-bidi-isolate-basic-expected.txt:
platform/gtk/fast/block/float/float-not-removed-from-next-sibling4-expected.png:
platform/gtk/fast/block/float/float-not-removed-from-next-sibling4-expected.txt:
platform/gtk/fast/borders/border-antialiasing-expected.txt:
platform/gtk/fast/line-grid/line-grid-inside-columns-expected.txt:
platform/gtk/fast/line-grid/line-grid-into-columns-expected.txt:
platform/gtk/fast/multicol/block-axis-horizontal-bt-expected.txt:
platform/gtk/fast/multicol/block-axis-horizontal-tb-expected.txt:
platform/gtk/fast/multicol/block-axis-vertical-lr-expected.txt:
platform/gtk/fast/multicol/block-axis-vertical-rl-expected.txt:
platform/gtk/fast/multicol/border-padding-pagination-expected.png:
platform/gtk/fast/multicol/border-padding-pagination-expected.txt:
platform/gtk/fast/multicol/client-rects-expected.png:
platform/gtk/fast/multicol/client-rects-expected.txt:
platform/gtk/fast/multicol/client-rects-spanners-complex-expected.txt: Added.
platform/gtk/fast/multicol/client-rects-spanners-expected.txt: Added.
platform/gtk/fast/multicol/column-break-with-balancing-expected.txt:
platform/gtk/fast/multicol/column-count-with-rules-expected.txt:
platform/gtk/fast/multicol/column-rules-expected.png:
platform/gtk/fast/multicol/column-rules-expected.txt:
platform/gtk/fast/multicol/column-rules-stacking-expected.png:
platform/gtk/fast/multicol/column-rules-stacking-expected.txt:
platform/gtk/fast/multicol/columns-shorthand-parsing-expected.txt:
platform/gtk/fast/multicol/float-avoidance-expected.txt:
platform/gtk/fast/multicol/float-multicol-expected.png:
platform/gtk/fast/multicol/float-multicol-expected.txt:
platform/gtk/fast/multicol/float-paginate-complex-expected.txt:
platform/gtk/fast/multicol/float-paginate-empty-lines-expected.txt:
platform/gtk/fast/multicol/float-paginate-expected.txt:
platform/gtk/fast/multicol/layers-in-multicol-expected.txt:
platform/gtk/fast/multicol/layers-split-across-columns-expected.txt:
platform/gtk/fast/multicol/margin-collapse-expected.txt:
platform/gtk/fast/multicol/max-height-columns-block-expected.png:
platform/gtk/fast/multicol/max-height-columns-block-expected.txt:
platform/gtk/fast/multicol/nested-columns-expected.png:
platform/gtk/fast/multicol/nested-columns-expected.txt:
platform/gtk/fast/multicol/newmulticol/client-rects-expected.png: Added.
platform/gtk/fast/multicol/newmulticol/client-rects-expected.txt:
platform/gtk/fast/multicol/overflow-across-columns-expected.txt:
platform/gtk/fast/multicol/overflow-across-columns-percent-height-expected.png:
platform/gtk/fast/multicol/overflow-across-columns-percent-height-expected.txt:
platform/gtk/fast/multicol/overflow-unsplittable-expected.png:
platform/gtk/fast/multicol/overflow-unsplittable-expected.txt:
platform/gtk/fast/multicol/paginate-block-replaced-expected.txt:
platform/gtk/fast/multicol/pagination/BottomToTop-bt-expected.png: Added.
platform/gtk/fast/multicol/pagination/BottomToTop-bt-expected.txt:
platform/gtk/fast/multicol/pagination/BottomToTop-lr-expected.png: Added.
platform/gtk/fast/multicol/pagination/BottomToTop-lr-expected.txt:
platform/gtk/fast/multicol/pagination/BottomToTop-rl-expected.png: Added.
platform/gtk/fast/multicol/pagination/BottomToTop-rl-expected.txt:
platform/gtk/fast/multicol/pagination/BottomToTop-tb-expected.png: Added.
platform/gtk/fast/multicol/pagination/BottomToTop-tb-expected.txt:
platform/gtk/fast/multicol/pagination/LeftToRight-bt-expected.png: Added.
platform/gtk/fast/multicol/pagination/LeftToRight-bt-expected.txt:
platform/gtk/fast/multicol/pagination/LeftToRight-lr-expected.png: Added.
platform/gtk/fast/multicol/pagination/LeftToRight-lr-expected.txt:
platform/gtk/fast/multicol/pagination/LeftToRight-rl-expected.png: Added.
platform/gtk/fast/multicol/pagination/LeftToRight-rl-expected.txt:
platform/gtk/fast/multicol/pagination/LeftToRight-tb-expected.png: Added.
platform/gtk/fast/multicol/pagination/LeftToRight-tb-expected.txt:
platform/gtk/fast/multicol/pagination/RightToLeft-bt-expected.png: Added.
platform/gtk/fast/multicol/pagination/RightToLeft-bt-expected.txt:
platform/gtk/fast/multicol/pagination/RightToLeft-lr-expected.png: Added.
platform/gtk/fast/multicol/pagination/RightToLeft-lr-expected.txt:
platform/gtk/fast/multicol/pagination/RightToLeft-rl-expected.png: Added.
platform/gtk/fast/multicol/pagination/RightToLeft-rl-expected.txt:
platform/gtk/fast/multicol/pagination/RightToLeft-tb-expected.png: Added.
platform/gtk/fast/multicol/pagination/RightToLeft-tb-expected.txt:
platform/gtk/fast/multicol/pagination/TopToBottom-bt-expected.png: Added.
platform/gtk/fast/multicol/pagination/TopToBottom-bt-expected.txt:
platform/gtk/fast/multicol/pagination/TopToBottom-lr-expected.png: Added.
platform/gtk/fast/multicol/pagination/TopToBottom-lr-expected.txt:
platform/gtk/fast/multicol/pagination/TopToBottom-rl-expected.png: Added.
platform/gtk/fast/multicol/pagination/TopToBottom-rl-expected.txt:
platform/gtk/fast/multicol/pagination/TopToBottom-tb-expected.png: Added.
platform/gtk/fast/multicol/pagination/TopToBottom-tb-expected.txt:
platform/gtk/fast/multicol/positioned-split-expected.txt:
platform/gtk/fast/multicol/positive-leading-expected.png:
platform/gtk/fast/multicol/positive-leading-expected.txt:
platform/gtk/fast/multicol/scrolling-overflow-expected.txt:
platform/gtk/fast/multicol/shadow-breaking-expected.png:
platform/gtk/fast/multicol/shadow-breaking-expected.txt:
platform/gtk/fast/multicol/shrink-to-column-height-for-pagination-expected.png: Added.
platform/gtk/fast/multicol/shrink-to-column-height-for-pagination-expected.txt:
platform/gtk/fast/multicol/single-line-expected.txt:
platform/gtk/fast/multicol/span/anonymous-before-child-parent-crash-expected.png:
platform/gtk/fast/multicol/span/anonymous-before-child-parent-crash-expected.txt:
platform/gtk/fast/multicol/span/anonymous-split-block-crash-expected.png:
platform/gtk/fast/multicol/span/anonymous-split-block-crash-expected.txt:
platform/gtk/fast/multicol/span/anonymous-style-inheritance-expected.txt:
platform/gtk/fast/multicol/span/clone-anonymous-block-non-inline-child-crash-expected.png:
platform/gtk/fast/multicol/span/clone-anonymous-block-non-inline-child-crash-expected.txt:
platform/gtk/fast/multicol/span/clone-flexbox-expected.txt:
platform/gtk/fast/multicol/span/clone-summary-expected.txt:
platform/gtk/fast/multicol/span/span-as-immediate-child-complex-splitting-expected.png:
platform/gtk/fast/multicol/span/span-as-immediate-child-complex-splitting-expected.txt:
platform/gtk/fast/multicol/span/span-as-immediate-child-generated-content-expected.png:
platform/gtk/fast/multicol/span/span-as-immediate-child-generated-content-expected.txt:
platform/gtk/fast/multicol/span/span-as-immediate-child-property-removal-expected.txt:
platform/gtk/fast/multicol/span/span-as-immediate-columns-child-dynamic-expected.txt:
platform/gtk/fast/multicol/span/span-as-immediate-columns-child-expected.txt:
platform/gtk/fast/multicol/span/span-as-immediate-columns-child-removal-expected.txt:
platform/gtk/fast/multicol/span/span-as-nested-columns-child-dynamic-expected.png:
platform/gtk/fast/multicol/span/span-as-nested-columns-child-dynamic-expected.txt:
platform/gtk/fast/multicol/span/span-as-nested-columns-child-expected.png:
platform/gtk/fast/multicol/span/span-as-nested-columns-child-expected.txt:
platform/gtk/fast/multicol/span/span-as-nested-inline-block-child-expected.txt:
platform/gtk/fast/multicol/span/span-margin-collapsing-expected.txt:
platform/gtk/fast/multicol/table-margin-collapse-expected.txt:
platform/gtk/fast/multicol/table-vertical-align-expected.txt:
platform/gtk/fast/multicol/unsplittable-inline-block-expected.txt:
platform/gtk/fast/multicol/vertical-lr/border-padding-pagination-expected.png:
platform/gtk/fast/multicol/vertical-lr/border-padding-pagination-expected.txt:
platform/gtk/fast/multicol/vertical-lr/column-break-with-balancing-expected.txt:
platform/gtk/fast/multicol/vertical-lr/column-count-with-rules-expected.txt:
platform/gtk/fast/multicol/vertical-lr/column-rules-expected.png:
platform/gtk/fast/multicol/vertical-lr/column-rules-expected.txt:
platform/gtk/fast/multicol/vertical-lr/float-avoidance-expected.txt:
platform/gtk/fast/multicol/vertical-lr/float-multicol-expected.txt:
platform/gtk/fast/multicol/vertical-lr/float-paginate-complex-expected.txt:
platform/gtk/fast/multicol/vertical-lr/float-paginate-expected.txt:
platform/gtk/fast/multicol/vertical-lr/nested-columns-expected.png:
platform/gtk/fast/multicol/vertical-lr/nested-columns-expected.txt:
platform/gtk/fast/multicol/vertical-lr/unsplittable-inline-block-expected.txt:
platform/gtk/fast/multicol/vertical-rl/border-padding-pagination-expected.png:
platform/gtk/fast/multicol/vertical-rl/border-padding-pagination-expected.txt:
platform/gtk/fast/multicol/vertical-rl/column-break-with-balancing-expected.txt:
platform/gtk/fast/multicol/vertical-rl/column-count-with-rules-expected.txt:
platform/gtk/fast/multicol/vertical-rl/column-rules-expected.png:
platform/gtk/fast/multicol/vertical-rl/column-rules-expected.txt:
platform/gtk/fast/multicol/vertical-rl/float-avoidance-expected.txt:
platform/gtk/fast/multicol/vertical-rl/float-multicol-expected.txt:
platform/gtk/fast/multicol/vertical-rl/float-paginate-complex-expected.txt:
platform/gtk/fast/multicol/vertical-rl/float-paginate-expected.txt:
platform/gtk/fast/multicol/vertical-rl/nested-columns-expected.png:
platform/gtk/fast/multicol/vertical-rl/nested-columns-expected.txt:
platform/gtk/fast/multicol/vertical-rl/rule-style-expected.txt:
platform/gtk/fast/multicol/vertical-rl/unsplittable-inline-block-expected.txt:
platform/gtk/fast/overflow/paged-x-div-expected.txt:
platform/gtk/fast/overflow/paged-x-div-with-column-gap-expected.txt:
platform/gtk/fast/overflow/paged-x-on-root-expected.txt:
platform/gtk/fast/overflow/paged-x-with-column-gap-expected.txt:
platform/gtk/fast/overflow/paged-y-div-expected.txt:
platform/gtk/fast/overflow/paged-y-on-root-expected.txt:
platform/gtk/fast/repaint/multicol-repaint-expected.png:
platform/gtk/fast/repaint/multicol-repaint-expected.txt:

8:27 AM Changeset in webkit [168096] by matthew_hanson@apple.com

6 edits
2 copies in branches/safari-537.76-branch

Merge r166650.

6:31 AM Changeset in webkit [168095] by Alan Bujtas

5 edits
2 adds in trunk

Subpixel rendering: Inline text selection painting should not snap to integral CSS pixel position.
https://bugs.webkit.org/show_bug.cgi?id=132164

Reviewed by Darin Adler.

Inline text selection painting now snaps to device pixels. It uses the same rounding logic as
other painting functions.

Source/WebCore:
Test: fast/inline/hidpi-select-inline-on-subpixel-position.html

rendering/EllipsisBox.cpp:

(WebCore::EllipsisBox::paintSelection):

rendering/InlineTextBox.cpp:

(WebCore::InlineTextBox::paintSelection):
(WebCore::alignSelectionRectToDevicePixels): Deleted.

rendering/InlineTextBox.h:

LayoutTests:

fast/inline/hidpi-select-inline-on-subpixel-position-expected.html: Added.
fast/inline/hidpi-select-inline-on-subpixel-position.html: Added.

5:42 AM Changeset in webkit [168094] by ryuan.choi@samsung.com

9 edits in trunk/Source

[EFL] There are many warnings with software backend
https://bugs.webkit.org/show_bug.cgi?id=132422

Reviewed by Gyuyoung Kim.

Source/WebCore:
ecore_evas_gl_x11_window_get should be called when only engine is opengl_x11.

This patch refactors not to call unnecessary API by checking engine type.
In addition, removed unnecessary isUsingEcoreX().

platform/efl/EflScreenUtilities.cpp:

(WebCore::applyFallbackCursor):
(WebCore::getEcoreXWindow):
(WebCore::isUsingEcoreX): Deleted.

platform/efl/EflScreenUtilities.h:

Source/WebKit/efl:

ewk/ewk_view.cpp:

(_ewk_view_priv_new):
(ewk_view_cursor_set):

Source/WebKit2:

UIProcess/API/efl/EwkView.cpp:

(EwkView::updateCursor):
(EwkView::transformToScreen):

Source/WTF:

wtf/efl/EflTypedefs.h: Added Ecore_X_Window typedef

5:09 AM WebKitGTK/KeepingTheTreeGreen edited by clopez@igalia.com

(diff)

2:57 AM Changeset in webkit [168093] by mitz@apple.com

2 edits in trunk/Source/WebKit

<rdar://problem/16780403> REGRESSION: Plugin tests failing on Mac/WebKit1.

WebKit.xcodeproj/project.pbxproj: Keep the WebKitPluginHost.app symlink in

WebKit.framework for now.

1:33 AM Changeset in webkit [168092] by cabanier@adobe.com

6 edits in trunk

Calling createPattern with a broken image must throw an invalidstate error
https://bugs.webkit.org/show_bug.cgi?id=132407

Reviewed by Dirk Schulze.

Source/WebCore:
Per the WebIDL spec, passing non-finite parameter to a method that
takes doubles, should generate a type error.

Tests:

canvas/philip/tests/2d.imageData.create2.nonfinite.html:
fast/canvas/canvas-2d-imageData-create-nonfinite.html:

html/canvas/CanvasRenderingContext2D.cpp:

(WebCore::CanvasRenderingContext2D::createImageData):

LayoutTests:

canvas/philip/tests/2d.imageData.create2.nonfinite.html:
fast/canvas/canvas-2d-imageData-create-nonfinite-expected.txt:
fast/canvas/resources/canvas-2d-imageData-create-nonfinite.js:

12:37 AM Changeset in webkit [168091] by matthew_hanson@apple.com

7 edits
4 deletes in tags/Safari-538.33

Rollout r167889.

12:17 AM Changeset in webkit [168090] by akling@apple.com

2 edits in trunk/LayoutTests

Skip fast/multicol/fixed-stack.html
<https://webkit.org/b/132421>

TestExpectations:

12:05 AM Changeset in webkit [168089] by matthew_hanson@apple.com

28 edits
1 copy in tags/Safari-538.33/Source

Merge r168085.

Apr 30, 2014:

11:44 PM Changeset in webkit [168088] by hyatt@apple.com

5 edits in trunk

REGRESSION (r168046): [New Multicolumn] LeftToRight-rl.html (and all the other reversed/block-axis pagination tests) fail
https://bugs.webkit.org/show_bug.cgi?id=132419

Reviewed by Andreas Kling.

Source/WebCore:

rendering/RenderMultiColumnSet.cpp:

(WebCore::RenderMultiColumnSet::initialBlockOffsetForPainting):
Don't flip here. The old code needed to do that, but the new code doesn't.

LayoutTests:

platform/mac/fast/multicol/pagination/LeftToRight-rl-expected.png:
platform/mac/fast/multicol/pagination/TopToBottom-bt-expected.png:

11:38 PM Changeset in webkit [168087] by matthew_hanson@apple.com

5 edits in trunk/Source

Versioning.

11:04 PM Changeset in webkit [168086] by matthew_hanson@apple.com

1 copy in tags/Safari-538.33

New Tag.

10:49 PM Changeset in webkit [168085] by ap@apple.com

28 edits
1 copy in trunk/Source

Roll out fix for https://bugs.webkit.org/show_bug.cgi?id=131637:
Clean up unnecessary methods in the BackForwardClient interface

It broke a regression test and an API test.

Source/WebCore:

WebCore.exp.in:
history/BackForwardClient.h:

(WebCore::BackForwardClient::backItem):
(WebCore::BackForwardClient::currentItem):
(WebCore::BackForwardClient::forwardItem):

history/BackForwardController.cpp:

(WebCore::BackForwardController::BackForwardController):

history/BackForwardController.h:

(WebCore::BackForwardController::client):

history/BackForwardList.cpp:

(WebCore::BackForwardList::BackForwardList):
(WebCore::BackForwardList::close):

history/BackForwardList.h:

(WebCore::BackForwardList::create):
(WebCore::BackForwardList::page):

page/Page.cpp:

(WebCore::Page::Page):
(WebCore::Page::PageClients::PageClients):

page/Page.h:

Source/WebKit/efl:

ewk/ewk_history.cpp:

(ewk_history_clear):
(ewk_history_new):
(ewk_history_free):

ewk/ewk_view.cpp:

(_ewk_view_priv_new):
(ewk_view_history_enable_get):
(ewk_view_history_enable_set):
(ewk_view_history_get):

Source/WebKit/mac:

History/WebBackForwardList.mm:

(-[WebBackForwardList initWithBackForwardList:]):
(-[WebBackForwardList init]):
(-[WebBackForwardList dealloc]):
(-[WebBackForwardList finalize]):
(-[WebBackForwardList setPageCacheSize:]):
(-[WebBackForwardList pageCacheSize]):

History/WebBackForwardListInternal.h:
WebCoreSupport/WebFrameLoaderClient.mm:

(WebFrameLoaderClient::canCachePage):

WebView/WebFrameView.mm:

(-[WebFrameView keyDown:keyDown:]):

WebView/WebView.mm:

(-[WebView _loadBackForwardListFromOtherView:]):
(-[WebView initWithCoder:]):
(-[WebView encodeWithCoder:]):
(-[WebView backForwardList]):
(-[WebView setMaintainsBackForwardList:]):

Source/WebKit/win:

WebBackForwardList.cpp:

(WebBackForwardList::WebBackForwardList):
(WebBackForwardList::~WebBackForwardList):
(WebBackForwardList::createInstance):

WebBackForwardList.h:
WebView.cpp:

(WebView::backForwardList):
(WebView::canGoBack):
(WebView::canGoForward):
(WebView::loadBackForwardListFromOtherView):

Source/WebKit2:

WebKit2.xcodeproj/project.pbxproj:
WebProcess/InjectedBundle/InjectedBundleBackForwardList.cpp:

(WebKit::InjectedBundleBackForwardList::clear):

WebProcess/WebPage/WebBackForwardListProxy.cpp:

(WebKit::WebBackForwardListProxy::WebBackForwardListProxy):
(WebKit::WebBackForwardListProxy::addItem):
(WebKit::WebBackForwardListProxy::goToItem):
(WebKit::WebBackForwardListProxy::itemAtIndex):
(WebKit::WebBackForwardListProxy::backListCount):
(WebKit::WebBackForwardListProxy::forwardListCount):
(WebKit::WebBackForwardListProxy::close):
(WebKit::WebBackForwardListProxy::clear):

WebProcess/WebPage/WebBackForwardListProxy.h:

(WebKit::WebBackForwardListProxy::create):

WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::WebPage):

WebProcess/WebPage/ios/WebBackForwardListProxyIOS.mm: Copied from Source/WebKit2/WebProcess/WebPage/ios/WebBackForwardListProxyIOS.mm.

9:48 PM Changeset in webkit [168084] by beidson@apple.com

4 edits in trunk/Source/WebKit2

Only reveal selection service UI after a short delay.
<rdar://problem/16777346> and https://bugs.webkit.org/show_bug.cgi?id=132418

Reviewed by Sam Weinig.

WebProcess/WebPage/SelectionOverlayController.cpp:

(WebKit::SelectionOverlayController::SelectionOverlayController):
(WebKit::SelectionOverlayController::destroyOverlay): Also stop the hover timer.
(WebKit::SelectionOverlayController::selectionRectsDidChange): Mark the highlight as dirty.
(WebKit::SelectionOverlayController::hoverTimerFired): If the mouse is still over the highlight,

set the visible flag and setNeedsDisplay().

WebProcess/WebPage/SelectionOverlayController.h:

WebProcess/WebPage/mac/SelectionOverlayControllerMac.mm:

(WebKit::SelectionOverlayController::drawRect): Recreate the highlight if it exists but

is marked as dirty. After doing that, possibly reset the hover timer.
Also, only performing the actual drawing if the visible flag is set.

(WebKit::SelectionOverlayController::mouseEvent): If the mouse moves on or off the highlight,

start or stop the hover timer accordingly.

(WebKit::SelectionOverlayController::mouseHoverStateChanged): Handle starting/stopping the

hover timer.

(WebKit::SelectionOverlayController::clearHighlight): Deleted.

9:20 PM Changeset in webkit [168083] by commit-queue@webkit.org

2 edits in trunk/Tools

Kernel sysctl interface hw.activecpu don't exists on Linux.
https://bugs.webkit.org/show_bug.cgi?id=132286

Patch by Carlos Alberto Lopez Perez <clopez@igalia.com> on 2014-04-30
Reviewed by Filip Pizlo.

Scripts/run-jsc-stress-tests: Redirect stderr to null when

calling sysctl over hw.activecpu

7:46 PM Changeset in webkit [168082] by Simon Fraser

4 edits
2 deletes in trunk/Source/WebCore

More iOS build fixing. MediaPlayerPrivateIOS is defunct and can be removed.
Fix some build errors in other media files.

WebCore.xcodeproj/project.pbxproj:
platform/graphics/MediaPlayer.cpp:
platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
platform/graphics/ios/MediaPlayerPrivateIOS.h: Removed.
platform/graphics/ios/MediaPlayerPrivateIOS.mm: Removed.

7:46 PM Changeset in webkit [168081] by Simon Fraser

2 edits in trunk/Source/WebCore

Fix the iOS build, which no longer needs these calls
to enter/exitFullscreen.

html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::parseAttribute):

7:44 PM Changeset in webkit [168080] by benjamin@webkit.org

2 edits in trunk/Source/WebKit2

[iOS][WK2] Animated resize incorrectly assumes the layout width is the same as the view width
https://bugs.webkit.org/show_bug.cgi?id=132373
<rdar://problem/16762178>

Reviewed by Tim Horton.

UIProcess/API/Cocoa/WKWebView.mm:

(-[WKWebView _beginAnimatedResizeWithUpdates:]):
Fix a couple of bugs when the left/right obscured insets change, and/or when the minimum layout size
is narrower than the view itself.

In order:
-We need to perfom and update if the insets change since the unobscuredRect will also change.
-The min/max zoom scale should be based on the minimum layout size, that's the definition of minimum layout size

in scrollview coordinate :)

-The old web view width in content coordinate could be narrower than the old view bounds if there are left or right

insets.

7:19 PM Changeset in webkit [168079] by Simon Fraser

50 edits in trunk/Source

Remove ENABLE_PLUGIN_PROXY_FOR_VIDEO
https://bugs.webkit.org/show_bug.cgi?id=132396

Reviewed by Eric Carlson.

Source/JavaScriptCore:

Remove ENABLE_PLUGIN_PROXY_FOR_VIDEO and related code.

Configurations/FeatureDefines.xcconfig:

Source/WebCore:

Remove ENABLE_PLUGIN_PROXY_FOR_VIDEO and related code.

Configurations/FeatureDefines.xcconfig:
WebCore.exp.in:
css/StyleResolver.cpp:

(WebCore::StyleResolver::canShareStyleWithElement):

dom/DOMImplementation.cpp:

(WebCore::DOMImplementation::createDocument):

editing/TextIterator.cpp:

(WebCore::isRendererReplacedElement):

html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::HTMLMediaElement):
(WebCore::HTMLMediaElement::parseAttribute):

html/HTMLMediaElement.h:

(WebCore::HTMLMediaElement::setNeedWidgetUpdate): Deleted.

html/HTMLMediaSession.cpp:

(WebCore::HTMLMediaSession::showPlaybackTargetPicker):
(WebCore::HTMLMediaSession::hasWirelessPlaybackTargets):
(WebCore::HTMLMediaSession::setHasPlaybackTargetAvailabilityListeners):

html/HTMLVideoElement.cpp:

(WebCore::HTMLVideoElement::createElementRenderer):
(WebCore::HTMLVideoElement::didAttachRenderers):
(WebCore::HTMLVideoElement::parseAttribute):
(WebCore::HTMLVideoElement::setDisplayMode):

html/shadow/MediaControlElements.cpp:

(WebCore::MediaControlTextTrackContainerElement::updateSizes):

loader/EmptyClients.cpp:

(WebCore::EmptyFrameLoaderClient::createMediaPlayerProxyPlugin): Deleted.

loader/EmptyClients.h:
loader/FrameLoaderClient.h:
loader/SubframeLoader.cpp:

(WebCore::SubframeLoader::loadPlugin):
(WebCore::SubframeLoader::loadMediaPlayerProxyPlugin): Deleted.

loader/SubframeLoader.h:
page/FrameView.cpp:

(WebCore::FrameView::updateEmbeddedObject):

page/Settings.cpp:

(WebCore::Settings::setVideoPluginProxyEnabled): Deleted.

page/Settings.h:

(WebCore::Settings::isVideoPluginProxyEnabled): Deleted.

platform/graphics/GraphicsLayerClient.h:

(WebCore::GraphicsLayerClient::mediaLayerMustBeUpdatedOnMainThread): Deleted.

platform/graphics/MediaPlayer.cpp:

(WebCore::installedMediaEngines):
(WebCore::MediaPlayer::MediaPlayer):
(WebCore::MediaPlayer::loadWithNextMediaEngine):
(WebCore::NullMediaPlayerPrivate::deliverNotification): Deleted.
(WebCore::NullMediaPlayerPrivate::setMediaPlayerProxy): Deleted.
(WebCore::NullMediaPlayerPrivate::setControls): Deleted.
(WebCore::MediaPlayer::deliverNotification): Deleted.
(WebCore::MediaPlayer::setMediaPlayerProxy): Deleted.
(WebCore::MediaPlayer::setControls): Deleted.

platform/graphics/MediaPlayer.h:
platform/graphics/MediaPlayerPrivate.h:

(WebCore::MediaPlayerPrivateInterface::deliverNotification): Deleted.
(WebCore::MediaPlayerPrivateInterface::setMediaPlayerProxy): Deleted.
(WebCore::MediaPlayerPrivateInterface::setControls): Deleted.

platform/graphics/ca/GraphicsLayerCA.cpp:

(WebCore::GraphicsLayerCA::updateGeometry):
(WebCore::GraphicsLayerCA::updateContentsRects):
(WebCore::GraphicsLayerCA::mediaLayerMustBeUpdatedOnMainThread): Deleted.

platform/graphics/ca/GraphicsLayerCA.h:
rendering/RenderEmbeddedObject.cpp:

(WebCore::RenderEmbeddedObject::canHaveChildren):

rendering/RenderLayerBacking.cpp:

(WebCore::RenderLayerBacking::mediaLayerMustBeUpdatedOnMainThread): Deleted.

rendering/RenderLayerBacking.h:
rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::requiresCompositingForVideo):

Source/WebKit/mac:

Remove ENABLE_PLUGIN_PROXY_FOR_VIDEO and related code.

Configurations/FeatureDefines.xcconfig:
Plugins/WebPluginContainerPrivate.h:
Plugins/WebPluginController.h:
Plugins/WebPluginController.mm:

(-[WebPluginController dealloc]):
(-[WebPluginController plugInsAreRunning]):
(-[WebPluginController stopAllPlugins]):
(-[WebPluginController stopPluginsForPageCache]):
(-[WebPluginController restorePluginsFromCache]):
(-[WebPluginController addPlugin:]):
(-[WebPluginController destroyPlugin:]):
(-[WebPluginController destroyAllPlugins]):
(-[NSView isMediaPlugInProxyView]): Deleted.
(-[NSView setIsMediaPlugInProxyView:]): Deleted.
(-[WebPluginController mediaPlugInProxyViewCreated:]): Deleted.
(+[WebPluginController pluginViewHidden:]): Deleted.
(mediaProxyClient): Deleted.
(-[WebPluginController _webPluginContainerSetMediaPlayerProxy:forElement:]): Deleted.
(-[WebPluginController _webPluginContainerPostMediaPlayerNotification:forElement:]): Deleted.

WebCoreSupport/WebFrameLoaderClient.h:
WebCoreSupport/WebFrameLoaderClient.mm:

(pluginView):

WebView/WebView.mm:

(-[WebView _preferencesChanged:]):
(-[WebView _videoProxyPluginForMIMEType:]): Deleted.

WebView/WebViewInternal.h:

Source/WebKit2:

Remove ENABLE_PLUGIN_PROXY_FOR_VIDEO and related code.

Configurations/FeatureDefines.xcconfig:
WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::WebFrameLoaderClient::createMediaPlayerProxyPlugin): Deleted.
(WebKit::WebFrameLoaderClient::hideMediaPlayerProxyPlugin): Deleted.
(WebKit::WebFrameLoaderClient::showMediaPlayerProxyPlugin): Deleted.

WebProcess/WebCoreSupport/WebFrameLoaderClient.h:
WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::updatePreferences):

Source/WTF:

Remove ENABLE_PLUGIN_PROXY_FOR_VIDEO and related code.

wtf/FeatureDefines.h:

7:15 PM Changeset in webkit [168078] by Simon Fraser

4 edits
3 adds in trunk

[iOS WK2] Some accerated overflow-scroll doesn't scroll correctly
https://bugs.webkit.org/show_bug.cgi?id=132375

Reviewed by Tim Horton.

Source/WebCore:
We set the size of the scrolling layer (which becomes the bounds of
the UIScrollView) to a non-pixel-snapped padding box size, but the
size of the contents layer is an integral-snapped scroll size.
This would result in a fractional difference between the two, which
makes us thing that the element is scrollable when it really is not.

Fix by setting the size of the scroll layer to pixel snapped client size,
which is what we also use for scrollability computation.

Added some FIXMEs in code that requires pixel snapping.

Also use #if PLATFORM(IOS)/#else to bracket some code that never runs on iOS
but tries to do something similar to iOS-only code.

rendering/RenderLayerBacking.cpp:

(WebCore::RenderLayerBacking::updateGraphicsLayerGeometry):

LayoutTests:
New test that dumps compositing layers on iOS so we can see the sizes of the
scroll layers that get created.

compositing/overflow/subpixel-overflow-expected.txt: Added.
compositing/overflow/subpixel-overflow.html: Added.
platform/ios-sim/compositing/overflow/subpixel-overflow-expected.txt: Added.
platform/mac/compositing/overflow/composited-scrolling-creates-a-stacking-container-expected.txt:

This is a progression; the old code failed to take the scrollbar width into
account, and the new code does.

7:15 PM Changeset in webkit [168077] by Simon Fraser

13 edits in trunk/LayoutTests

Rebaseline compositing/overflow tests for iOS.

platform/ios-sim/compositing/overflow/composited-scrolling-creates-a-stacking-container-expected.txt:
platform/ios-sim/compositing/overflow/composited-scrolling-paint-phases-expected.txt:
platform/ios-sim/compositing/overflow/content-gains-scrollbars-expected.txt:
platform/ios-sim/compositing/overflow/fixed-position-ancestor-clip-expected.txt:
platform/ios-sim/compositing/overflow/overflow-scroll-expected.txt:
platform/ios-sim/compositing/overflow/overflow-scrollbar-layers-expected.txt:
platform/ios-sim/compositing/overflow/remove-overflow-crash2-expected.txt:
platform/ios-sim/compositing/overflow/scrollbar-painting-expected.txt:
platform/ios-sim/compositing/overflow/scrolling-content-clip-to-viewport-expected.txt:
platform/ios-sim/compositing/overflow/scrolling-without-painting-expected.txt:
platform/ios-sim/compositing/overflow/textarea-scroll-touch-expected.txt:
platform/ios-sim/compositing/overflow/updating-scrolling-content-expected.txt:

7:12 PM Changeset in webkit [168076] by hyatt@apple.com

35 edits
2 adds in trunk

REGRESSION (r168046): [New Multicolumn] Painting order is wrong for columns and fixed positioned elements
https://bugs.webkit.org/show_bug.cgi?id=132377

Reviewed by Simon Fraser.

Source/WebCore:
Added fast/multicol/fixed-stack.html

rendering/RenderLayer.cpp:

(WebCore::RenderLayer::shouldBeSelfPaintingLayer):
(WebCore::RenderLayer::shouldBeNormalFlowOnly):
Change the flow thread layer for multicolumn layout to be normal flow only and to
stay self-painting. This has the effect of keeping the flow thread layer grouped
with the enclosing multicolumn layer, and this keeps the paint order correct when
compositing kicks in (or when something would otherwise try to get between the
two layers).

LayoutTests:

compositing/columns/composited-nested-columns-expected.txt:
fast/multicol/fixed-stack-expected.html: Added.
fast/multicol/fixed-stack.html: Added.
fast/multicol/flipped-blocks-border-after-expected.txt:
fast/multicol/progression-reverse-expected.txt:
fast/multicol/single-line-expected.txt:
fast/multicol/vertical-lr/rules-with-border-before-expected.txt:
fast/multicol/vertical-rl/rule-style-expected.txt:
fast/multicol/vertical-rl/rules-with-border-before-expected.txt:
platform/mac/fast/multicol/client-rects-expected.txt:
platform/mac/fast/multicol/client-rects-spanners-complex-expected.txt:
platform/mac/fast/multicol/client-rects-spanners-expected.txt:
platform/mac/fast/multicol/column-break-with-balancing-expected.txt:
platform/mac/fast/multicol/column-count-with-rules-expected.txt:
platform/mac/fast/multicol/float-paginate-complex-expected.txt:
platform/mac/fast/multicol/layers-in-multicol-expected.txt:
platform/mac/fast/multicol/layers-split-across-columns-expected.txt:
platform/mac/fast/multicol/newmulticol/client-rects-expected.txt:
platform/mac/fast/multicol/span/span-as-immediate-child-complex-splitting-expected.txt:
platform/mac/fast/multicol/span/span-as-immediate-child-generated-content-expected.txt:
platform/mac/fast/multicol/span/span-as-immediate-child-property-removal-expected.txt:
platform/mac/fast/multicol/span/span-as-immediate-columns-child-dynamic-expected.txt:
platform/mac/fast/multicol/span/span-as-immediate-columns-child-expected.txt:
platform/mac/fast/multicol/span/span-as-immediate-columns-child-removal-expected.txt:
platform/mac/fast/multicol/span/span-as-nested-columns-child-dynamic-expected.txt:
platform/mac/fast/multicol/span/span-as-nested-columns-child-expected.txt:
platform/mac/fast/multicol/span/span-margin-collapsing-expected.txt:
platform/mac/fast/multicol/table-vertical-align-expected.txt:
platform/mac/fast/multicol/vertical-lr/column-break-with-balancing-expected.txt:
platform/mac/fast/multicol/vertical-lr/column-count-with-rules-expected.txt:
platform/mac/fast/multicol/vertical-lr/float-paginate-complex-expected.txt:
platform/mac/fast/multicol/vertical-rl/column-break-with-balancing-expected.txt:
platform/mac/fast/multicol/vertical-rl/column-count-with-rules-expected.txt:
platform/mac/fast/multicol/vertical-rl/float-paginate-complex-expected.txt:

6:47 PM Changeset in webkit [168075] by eric.carlson@apple.com

14 edits in trunk/Source

[iOS] do not pause video when entering background while playing to external device
https://bugs.webkit.org/show_bug.cgi?id=132374

Reviewed by Jer Noble.

Source/WebCore:

WebCore.exp.in: Update beginInterruption signature.

html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::canOverrideBackgroundPlaybackRestriction): New, return true when

playing to external device.

html/HTMLMediaElement.h:

platform/audio/MediaSession.cpp:

(WebCore::MediaSession::beginInterruption): Add interruption type to beginInterruption.

platform/audio/MediaSession.h:

platform/audio/MediaSessionManager.cpp:

(WebCore::MediaSessionManager::beginInterruption): Ditto.
(WebCore::MediaSessionManager::applicationWillEnterBackground): Ditto.
(WebCore::MediaSessionManager::systemWillSleep): Ditto.

platform/audio/MediaSessionManager.h:

platform/audio/ios/AudioDestinationIOS.h:
platform/audio/ios/MediaSessionManagerIOS.mm:

(-[WebMediaSessionHelper interruption:]): Ditto.

platform/audio/mac/AudioDestinationMac.h: Make most methods private. Add

canOverrideBackgroundPlaybackRestriction. Add missing overrides.

testing/Internals.cpp:

(WebCore::Internals::beginMediaSessionInterruption): Pass interruption type.

Source/webkit:

WebKit.vcxproj/WebKitExportGenerator/WebKitExports.def.in:

6:18 PM Changeset in webkit [168074] by beidson@apple.com

12 edits
2 adds in trunk/Source/WebKit2

If there are no services available, do not show the service controls UI
<rdar://problem/16735665> and https://bugs.webkit.org/show_bug.cgi?id=132410

Reviewed by Tim Horton.

Add a lightweight class that lazily polls the appropriate APIs for whether or not appropriate services
are installed and usable on the system:

UIProcess/mac/ServicesController.h: Added.

(WebKit::ServicesController::imageServicesExist):
(WebKit::ServicesController::selectionServicesExist):

UIProcess/mac/ServicesController.mm: Added.

(WebKit::ServicesController::shared):
(WebKit::ServicesController::ServicesController):
(WebKit::ServicesController::refreshExistingServices):
(WebKit::ServicesController::refreshExistingServicesTimerFired):

Add "image services exist" and "selection services exist" parameters:

Shared/WebProcessCreationParameters.cpp:

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

Shared/WebProcessCreationParameters.h:

UIProcess/WebContext.cpp:

(WebKit::WebContext::createNewWebProcess):
(WebKit::WebContext::refreshExistingServices): Called when the context menu proxy realizes that

services no longer exist.

UIProcess/WebContext.h:

Each WebProcess hangs on to its own copy of the flags for whether or not the services exist:

WebProcess/WebProcess.cpp:

(WebKit::WebProcess::WebProcess):
(WebKit::WebProcess::initializeWebProcess):
(WebKit::WebProcess::setEnabledServices):

WebProcess/WebProcess.h:

(WebKit::WebProcess::imageServicesExist):
(WebKit::WebProcess::selectionServicesExist):

UIProcess/mac/WebContextMenuProxyMac.mm:

(WebKit::WebContextMenuProxyMac::setupServicesMenu): If the menu creation failed, the set of services

on the system must have changed. So ask the WebContext to refresh them.

WebProcess/WebPage/SelectionOverlayController.cpp:

(WebKit::SelectionOverlayController::selectionRectsDidChange): If services don't exist, don't create an

overlay (and destroy any existing overlay!)

WebProcess/WebPage/mac/SelectionOverlayControllerMac.mm:

(WebKit::SelectionOverlayController::drawRect): If services don't exist, don't draw, and destroy the overlay.

WebProcess/WebProcess.messages.in:
WebKit2.xcodeproj/project.pbxproj:

6:04 PM Changeset in webkit [168073] by andersca@apple.com

2 edits in trunk/Source/WebKit

Try yet again to fix the build.

WebKit.xcodeproj/project.pbxproj:

5:53 PM Changeset in webkit [168072] by andersca@apple.com

2 edits in trunk/Source/WebKit

Another build fix attempt.

WebKit.xcodeproj/project.pbxproj:

5:44 PM Changeset in webkit [168071] by andersca@apple.com

2 edits in trunk/Source/WebKit

Try to fix the iOS build.

WebKit.xcodeproj/project.pbxproj:

5:27 PM Changeset in webkit [168070] by barraclough@apple.com

2 edits in trunk/Source/WebKit2

https://bugs.webkit.org/show_bug.cgi?id=132415
Fix snapshotting on WebKit2

Reviewed by Geoff Garen

UIProcess/API/Cocoa/WKWebView.mm:

(-[WKWebView _snapshotRect:intoImageOfWidth:completionHandler:]):

Use a VisibilityToken to keep the process runnable.

5:23 PM Changeset in webkit [168069] by barraclough@apple.com

2 edits in trunk/Source/WebKit2

Fix PageVisibility on iOS
https://bugs.webkit.org/show_bug.cgi?id=132393

Rubber stamped by Tim Horton

UIProcess/ios/PageClientImplIOS.mm:

(WebKit::PageClientImpl::isViewWindowActive):
(WebKit::PageClientImpl::isViewFocused):
(WebKit::PageClientImpl::isViewVisible):
(WebKit::PageClientImpl::isViewVisibleOrOccluded):
(WebKit::PageClientImpl::isVisuallyIdle):

m_contentView -> m_webView

5:17 PM Changeset in webkit [168068] by matthew_hanson@apple.com

1 edit
2 copies in branches/safari-537.76-branch/LayoutTests

Merge r166645.

5:12 PM Changeset in webkit [168067] by Manuel Rego Casasnovas

7 edits in trunk/Source/WebCore

Unreviewed, rolling out r167879 and r167942.
https://bugs.webkit.org/show_bug.cgi?id=132408

OrderIterator changes caused regressions in flexbox (Requested
by rego on #webkit).

We're keeping the new layout test introduced in r167942
(fast/flexbox/order-iterator-crash.html) to avoid similar
regressions in the future.

Reverted changesets:

"OrderIterator refactoring to avoid extra loops"
https://bugs.webkit.org/show_bug.cgi?id=119061
http://trac.webkit.org/changeset/167879

"REGRESSION (r167879): Heap-use-after-free in
WebCore::RenderFlexibleBox"
https://bugs.webkit.org/show_bug.cgi?id=132337
http://trac.webkit.org/changeset/167942

5:08 PM Changeset in webkit [168066] by enrica@apple.com

2 edits in trunk/Source/WebCore

Cursor gets thinner on empty lines.
https://bugs.webkit.org/show_bug.cgi?id=132411
<rdar://problem/15994556>

Reviewed by Benjamin Poulain.

RenderLineBreak::localCaretRect should not define
locally the constant caretWidth, but use the one from
RenderObject.h which knows about the differences between
iOS and the other platforms.

rendering/RenderLineBreak.cpp:

(WebCore::RenderLineBreak::localCaretRect):

4:58 PM Changeset in webkit [168065] by Simon Fraser

3 edits in trunk

Make sure the "All" targets build WebKitLegacy, rather than WebKit.

Reviewed by Dan Bernstein/Anders Carlsson.

WebKit.xcworkspace/xcshareddata/xcschemes/All Source (target WebProcess).xcscheme:
WebKit.xcworkspace/xcshareddata/xcschemes/All Source.xcscheme:

4:48 PM Changeset in webkit [168064] by Lucas Forschler

1 edit in branches/safari-537.76-branch/Source/WebCore/html/HTMLSelectElement.cpp

Merge 2nd have of <rdar://problem/16701836>.

4:45 PM Changeset in webkit [168063] by benjamin@webkit.org

3 edits in trunk/Source/WebKit2

[iOS][WK2] Add a SPI to exclude the extended background from some areas of WKWebView
https://bugs.webkit.org/show_bug.cgi?id=132406
<rdar://problem/16762197>

Reviewed by Beth Dakin.

Move the extended background to a separate layer bellow the UIScrollView.

The geometry of that layer is then changed based on ExtendedBackgroundExclusionInsets
as needed.

UIProcess/API/Cocoa/WKWebView.mm:

(-[WKWebView initWithFrame:configuration:]):
(-[WKWebView _updateScrollViewBackground]):
(-[WKWebView _frameOrBoundsChanged]):
(-[WKWebView _setExtendedBackgroundExclusionInsets:]):
(-[WKWebView _extendedBackgroundExclusionInsets]):
(-[WKWebView pageExtendedBackgroundColor]): Deleted.

UIProcess/API/Cocoa/WKWebViewPrivate.h:

4:32 PM Changeset in webkit [168062] by andersca@apple.com

2 edits in trunk/Source/WebKit

Another build fix attempt.

WebKit.xcodeproj/project.pbxproj:

Remove headers if it seems like the WKWebViewPrivate header imports itself.

4:21 PM Changeset in webkit [168061] by andersca@apple.com

2 edits in trunk/Source/WebKit

Another build fix.

WebKit.xcodeproj/project.pbxproj:

Remove WK headers from WebKit.framework as well when WebKitLegacy WK headers are encountered.

4:17 PM Changeset in webkit [168060] by vjaquez@igalia.com

6 edits in trunk/Source/WebCore

[GStreamer] Use GstMetaVideo
https://bugs.webkit.org/show_bug.cgi?id=132247

Reviewed by Philippe Normand.

In WebKitVideoSink we announce the usage of GstMetaVideo, but we do
not use it when handling the video frames. This might break
some decoders and filters that rely on buffer's meta, rather
that in the caps structures.

This patch enables the use of GstMetaVideo through the GstVideoFrame
API. And it is used everywhere the buffer mapping is required.

Also this patch changes to nullptr where zeros were used.

Also, compile conditionally the video buffer conversion when it is
ARGB/BGRA, since it is only required for the Cairo backend.

No new tests, already covered by current tests.

platform/graphics/gstreamer/GStreamerUtilities.cpp:

(WebCore::getVideoSizeAndFormatFromCaps): init the GstVideoInfo before
used and remove caps fixate check since it is done by
gst_video_info_from_caps().

platform/graphics/gstreamer/ImageGStreamer.h:
platform/graphics/gstreamer/ImageGStreamerCairo.cpp:

(ImageGStreamer::ImageGStreamer): use GstVideoFrame for buffer mapping
and unmapping.
(ImageGStreamer::~ImageGStreamer): ditto.

platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:

(WebCore::MediaPlayerPrivateGStreamerBase::updateTexture): ditto.
(WebCore::MediaPlayerPrivateGStreamerBase::currentVideoSinkCaps):
return nullptr if failed.

platform/graphics/gstreamer/VideoSinkGStreamer.cpp:

(webkitVideoSinkRender): rely on GstVideoInfo rather than on the
caps. Use GstVideoFrame for buffer mapping and unmapping. Add guards
for buffer transformation, since it's only used by Cairo.
(webkitVideoSinkDispose): remove glib version guards.
(webkitVideoSinkSetCaps): update the value of the private
GstVideoInfo.

4:16 PM Changeset in webkit [168059] by vjaquez@igalia.com

5 edits in trunk/Source

[GTK][GStreamer] Remove unnecessary GLIB_CHECK_VERSION #ifdefs
https://bugs.webkit.org/show_bug.cgi?id=132390

Reviewed by Philippe Normand.

Since EFL port use GLib 2.38 and GTK+, 2.33.2, I assume it is OK
remove, in GTK+ and GST, the existing glib version guards.

Source/WebCore:
No new tests, already covered by current tests.

platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:

(WebCore::MediaPlayerPrivateGStreamerBase::MediaPlayerPrivateGStreamerBase):
(WebCore::MediaPlayerPrivateGStreamerBase::~MediaPlayerPrivateGStreamerBase):

platform/graphics/gstreamer/VideoSinkGStreamer.cpp:

(webkitVideoSinkDispose):
(webkitVideoSinkFinalize): Deleted.

Source/WTF:
This code was rollback from r149879 because Qt MIPS used it. But since
Qt is gone, it is safe to remove now.

wtf/gobject/GRefPtr.cpp:

(WTF::refGPtr): Deleted.
(WTF::derefGPtr): Deleted.

4:07 PM Changeset in webkit [168058] by andersca@apple.com

2 edits in trunk/Source/WebKit

Try to fix the build.

WebKit.xcodeproj/project.pbxproj:

Remove old WK forwarding headers from WebKitLegacy.

4:00 PM Changeset in webkit [168057] by Simon Fraser

3 edits in trunk

Let Xcode have its way with the WebKit workspace.

WebKit.xcworkspace/xcshareddata/xcschemes/All Source (target WebProcess).xcscheme:
WebKit.xcworkspace/xcshareddata/xcschemes/All Source.xcscheme:

4:00 PM Changeset in webkit [168056] by Simon Fraser

2 edits in trunk/Source/WebKit2

Fix build error when building for iOS simulator.

UIProcess/ios/ProcessAssertion.mm:

3:57 PM Changeset in webkit [168055] by achristensen@apple.com

208 edits
44 adds
3 deletes in trunk/Source

Updated ANGLE.
https://bugs.webkit.org/show_bug.cgi?id=132367
<rdar://problem/16211451>

Reviewed by Dean Jackson.

Source/ThirdParty/ANGLE:

ANGLE.plist:

Updated and xml encoded.

ANGLE.xcodeproj/project.pbxproj:

Added needed new source files.

changes.diff:

Added to keep track of differences between WebKit's copy of ANGLE and the master repository.

Updated ANGLE source files to e7a453a5bd76705ccb151117fa844846d4aa90af. Long list of changes omitted.

Source/WebCore:

CMakeLists.txt

Fixed ANGLE compiling with the update.

platform/graphics/opengl/Extensions3DOpenGLCommon.cpp:

(WebCore::Extensions3DOpenGLCommon::getTranslatedShaderSourceANGLE):
Removed SH_MAP_LONG_VARIABLE_NAMES which is no longer defined in ANGLE.
See https://chromium.googlesource.com/angle/angle/+/3cdfcce86b38ef31a0afd71855887193a7924468

platform/graphics/opengl/Extensions3DOpenGLES.h:
platform/graphics/opengl/Extensions3DOpenGLES.cpp:

Updated type names from ANGLE.

3:52 PM Changeset in webkit [168054] by ap@apple.com

16 edits in trunk/Source

Move Blob.slice() implementation into BlobRegistryImpl
https://bugs.webkit.org/show_bug.cgi?id=132402

Reviewed by Anders Carlsson.

Source/WebCore:
Part or centralizing the responsibility for file size tracking.

fileapi/Blob.cpp:

(WebCore::Blob::Blob):
(WebCore::Blob::slice): Deleted.

fileapi/Blob.h:

(WebCore::Blob::slice):

fileapi/ThreadableBlobRegistry.cpp:

(WebCore::ThreadableBlobRegistry::registerBlobURL):
(WebCore::ThreadableBlobRegistry::registerBlobURLForSlice):
(WebCore::registerBlobURLTask): Deleted.
(WebCore::registerBlobURLFromTask): Deleted.

fileapi/ThreadableBlobRegistry.h:
platform/network/BlobRegistry.h:
platform/network/BlobRegistryImpl.cpp:

(WebCore::BlobRegistryImpl::appendStorageItems):
(WebCore::BlobRegistryImpl::registerBlobURLForSlice):
(WebCore::BlobRegistryImpl::blobSize):

platform/network/BlobRegistryImpl.h:

Source/WebKit2:

NetworkProcess/FileAPI/NetworkBlobRegistry.cpp:

(WebKit::NetworkBlobRegistry::registerBlobURLForSlice):

NetworkProcess/FileAPI/NetworkBlobRegistry.h:
NetworkProcess/NetworkConnectionToWebProcess.cpp:

(WebKit::NetworkConnectionToWebProcess::registerBlobURLForSlice):

NetworkProcess/NetworkConnectionToWebProcess.h:
NetworkProcess/NetworkConnectionToWebProcess.messages.in:
WebProcess/FileAPI/BlobRegistryProxy.cpp:

(WebKit::BlobRegistryProxy::registerBlobURLForSlice):

WebProcess/FileAPI/BlobRegistryProxy.h:

3:45 PM Changeset in webkit [168053] by Beth Dakin

3 edits in trunk/Source/WebKit2

Phone number data detection UI is offset for iframes, pages with topContentInset
https://bugs.webkit.org/show_bug.cgi?id=132372
-and corresponding-
<rdar://problem/16651235>

Reviewed by Tim Horton.

Make the overlay an OverlayType::Document, which will keep everything relative to
the main Document’s coordinates.

WebProcess/WebPage/TelephoneNumberOverlayController.cpp:

(WebKit::TelephoneNumberOverlayController::createOverlayIfNeeded):

Make frames work by converting to the main document’s coordinate space.

WebProcess/WebPage/mac/TelephoneNumberOverlayControllerMac.mm:

(WebKit::TelephoneNumberOverlayController::drawRect):

3:44 PM Changeset in webkit [168052] by Brent Fulgham

2 edits in trunk/Source/WebCore

Unreviewed build fix after r168041.

WebCore.exp.in: Add missing iOS exports.

3:22 PM Changeset in webkit [168051] by fpizlo@apple.com

7 edits
1 add in trunk/Source/JavaScriptCore

Argument flush formats should not be presumed to be JSValue since 'this' is weird
https://bugs.webkit.org/show_bug.cgi?id=132404

Reviewed by Michael Saboff.

dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compileCurrentBlock): Don't assume that arguments are flushed as JSValue. Use the logic for locals instead.

dfg/DFGSpeculativeJIT32_64.cpp:

(JSC::DFG::SpeculativeJIT::compile): SetArgument "changes" the format because before this we wouldn't know we had arguments.

dfg/DFGSpeculativeJIT64.cpp:

(JSC::DFG::SpeculativeJIT::compile): Ditto.

dfg/DFGValueSource.cpp:

(JSC::DFG::ValueSource::dumpInContext): Make this easier to dump.

dfg/DFGValueSource.h:

(JSC::DFG::ValueSource::operator!): Make this easier to dump because Operands<T> uses T::operator!().

ftl/FTLOSREntry.cpp:

(JSC::FTL::prepareOSREntry): This had a useful assertion for everything except 'this'.

tests/stress/strict-to-this-int.js: Added.

(foo):
(Number.prototype.valueOf):
(test):

3:10 PM Changeset in webkit [168050] by roger_fong@apple.com

2 edits in trunk/Source/WebKit2

Unreviewed. Unnecessary explicit initialization of LayoutUnit from r167985.

WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::determinePrimarySnapshottedPlugIn):
Darin points out that I can just use “/2” instead of “/LayoutUnit(2.0)”.

3:05 PM Changeset in webkit [168049] by Beth Dakin

3 edits in trunk/Source/WebCore

Always-visible scrollbars continuously repaint after non-momentum scrollling
https://bugs.webkit.org/show_bug.cgi?id=132403
-and corresponding-
<rdar://problem/16553878>

Reviewed by Simon Fraser.

No longer universally opt into presentation value mode whenever the scroll
position changes on the scrolling thread. We really only want it for momentum
scrolls, and this will ensure that we always set it to NO once we have set it to
YES.

page/scrolling/mac/ScrollingTreeScrollingNodeMac.mm:

(WebCore::ScrollingTreeScrollingNodeMac::handleWheelEvent):
(WebCore::ScrollingTreeScrollingNodeMac::setScrollLayerPosition):

Expose shouldUsePresentationValue.

platform/mac/NSScrollerImpDetails.h:

3:03 PM Changeset in webkit [168048] by roger_fong@apple.com

22 edits in trunk/LayoutTests

Enable snapshot tests on mac wk2.
https://bugs.webkit.org/show_bug.cgi?id=131871.
Reviewed by Darin Adler.

platform/mac-wk2/TestExpectations:
platform/mac-wk2/plugins/snapshotting/autoplay-dominant-expected.txt:
platform/mac-wk2/plugins/snapshotting/autoplay-plugin-blocked-by-image-expected.txt:
platform/mac-wk2/plugins/snapshotting/autoplay-plugin-blocked-by-image-expected.txt:
platform/mac-wk2/plugins/snapshotting/autoplay-similar-to-dominant-after-delay-expected.txt:
platform/mac-wk2/plugins/snapshotting/autoplay-similar-to-dominant-expected.txt:
platform/mac-wk2/plugins/snapshotting/quicktime-plugin-snapshotted-expected.txt:
platform/mac-wk2/plugins/snapshotting/restart-expected.txt:
platform/mac-wk2/plugins/snapshotting/set-plugin-size-to-tiny-expected.txt:
platform/mac-wk2/plugins/snapshotting/simple-expected.txt:
platform/mac-wk2/plugins/snapshotting/snapshot-plugin-not-quite-blocked-by-image-expected.txt:
plugins/snapshotting/autoplay-dominant.html:
plugins/snapshotting/autoplay-plugin-blocked-by-image.html:
plugins/snapshotting/autoplay-plugin-mostly-blocked-by-image.html:
plugins/snapshotting/autoplay-similar-to-dominant-after-delay.html:
plugins/snapshotting/autoplay-similar-to-dominant.html:
plugins/snapshotting/quicktime-plugin-snapshotted.html:
plugins/snapshotting/restart.html:
plugins/snapshotting/set-plugin-size-to-tiny.html:
plugins/snapshotting/simple.html:
plugins/snapshotting/snapshot-plugin-not-quite-blocked-by-image.html:

3:02 PM Changeset in webkit [168047] by andersca@apple.com

132 edits
2 copies
1 move
1 add
1 delete in trunk

Move the legacy WebKit API into WebKitLegacy.framework and move it inside WebKit.framework
https://bugs.webkit.org/show_bug.cgi?id=132399
<rdar://problem/15920046>

Reviewed by Dan Bernstein.

Source/WebCore:
Allow WebKitLegacy to link against WebCore.

Configurations/WebCore.xcconfig:

Source/WebKit:

WebKit.xcodeproj/project.pbxproj:

Add a new build target that builds the legacy WebKit API in WebKitLegacy.framework. This framework
is then re-exported by WebKit.framework, and its headers are copied to WebKit.framework. All the WebKitLegacy
headers are made private, and the MigrateHeadersFromWebKitLegacy.make Makefile will copy all the headers specified
in WEBKIT_PUBLIC_HEADERS to WebKit/Headers.

Source/WebKit/ios:
Import WebKitLegacy headers instead of WebKit headers.

DefaultDelegates/WebDefaultFormDelegate.h:
DefaultDelegates/WebDefaultFrameLoadDelegate.m:
DefaultDelegates/WebDefaultResourceLoadDelegate.m:
DefaultDelegates/WebDefaultUIKitDelegate.h:
Misc/WebGeolocationProviderIOS.h:
Misc/WebNSStringExtrasIPhone.h:
WebCoreSupport/WebFrameIOS.h:
WebCoreSupport/WebFrameIOS.mm:
WebCoreSupport/WebFrameIPhone.h:
WebCoreSupport/WebSelectionRect.h:
WebCoreSupport/WebVisiblePosition.h:
WebView/WebPDFViewIOS.h:
WebView/WebPDFViewIOS.mm:
WebView/WebPDFViewIPhone.h:
WebView/WebPDFViewPlaceholder.h:
WebView/WebPDFViewPlaceholder.mm:
WebView/WebPlainWhiteView.h:
WebView/WebUIKitDelegate.h:

Source/WebKit/mac:

Carbon/CarbonUtils.h:
Carbon/HIViewAdapter.h:
Carbon/HIWebView.h:

Import WebKitLegacy headers instead of WebKit headers.

Configurations/WebKit.xcconfig:

Move the definitions needed for building WebKitLegacy.framework to WebKitLegacy.xcconfig and
add the relevant definitions needed for re-exporting WebKitLegacy.framework.

Configurations/WebKitLegacy.xcconfig:

Add definitions needed for building the "old" WebKit API as WebKitLegacy.framework.

DOM/WebDOMOperations.h:
DOM/WebDOMOperations.mm:
DOM/WebDOMOperationsInternal.h:
DOM/WebDOMOperationsPrivate.h:
DefaultDelegates/WebDefaultContextMenuDelegate.h:
DefaultDelegates/WebDefaultContextMenuDelegate.mm:
DefaultDelegates/WebDefaultEditingDelegate.m:
History/WebBackForwardListInternal.h:
History/WebBackForwardListPrivate.h:
History/WebHistoryItemPrivate.h:
History/WebHistoryPrivate.h:
History/WebURLsWithTitles.m:

Import WebKitLegacy headers instead of WebKit headers.

MigrateHeaders.make:

Update header paths now that all WebKitLegacy headers are private.
Remove migration of WebKit2 headers, that is done by MigrateHeadersFromWebKitLegacy.make now.

Misc/WebCoreStatistics.h:
Misc/WebDownload.h:
Misc/WebDownload.mm:
Misc/WebDownloadInternal.h:
Misc/WebElementDictionary.mm:
Misc/WebIconDatabasePrivate.h:
Misc/WebKit.h:
Misc/WebKitErrors.m:
Misc/WebKitErrorsPrivate.h:
Misc/WebKitNSStringExtras.mm:
Misc/WebLocalizableStrings.mm:
Misc/WebNSDataExtras.m:
Misc/WebNSDictionaryExtras.m:
Misc/WebNSEventExtras.m:
Misc/WebNSFileManagerExtras.mm:
Misc/WebNSImageExtras.m:
Misc/WebNSPasteboardExtras.mm:
Misc/WebNSViewExtras.h:
Misc/WebNSViewExtras.m:
Panels/WebAuthenticationPanel.m:
Panels/WebPanelAuthenticationHandler.m:
Plugins/Hosted/HostedNetscapePluginStream.h:
Plugins/Hosted/NetscapePluginInstanceProxy.h:
Plugins/Hosted/WebKitPluginAgent.defs:
Plugins/Hosted/WebKitPluginAgentReply.defs:
Plugins/Hosted/WebKitPluginClient.defs:
Plugins/Hosted/WebKitPluginHost.defs:
Plugins/Hosted/WebKitPluginHostTypes.defs:
Plugins/WebBaseNetscapePluginView.mm:
Plugins/WebBasePluginPackage.h:
Plugins/WebBasePluginPackage.mm:
Plugins/WebNetscapeContainerCheckPrivate.h:
Plugins/WebNetscapePluginEventHandlerCocoa.h:
Plugins/WebNetscapePluginStream.h:
Plugins/WebNetscapePluginView.h:
Plugins/WebNetscapePluginView.mm:
Plugins/WebPlugin.h:
Plugins/WebPluginController.h:
Plugins/WebPluginDatabase.h:
Plugins/WebPluginPackage.h:
Plugins/WebPluginPackage.mm:
Plugins/WebPluginViewFactory.h:
Plugins/WebPluginViewFactoryPrivate.h:
Plugins/npapi.mm:
Storage/WebDatabaseManagerPrivate.h:
WebCoreSupport/WebContextMenuClient.mm:
WebCoreSupport/WebFrameLoaderClient.mm:
WebCoreSupport/WebFrameNetworkingContext.mm:
WebCoreSupport/WebGeolocationClient.mm:
WebCoreSupport/WebInspectorClient.mm:
WebCoreSupport/WebJavaScriptTextInputPanel.m:
WebCoreSupport/WebKeyGenerator.mm:
WebInspector/WebInspectorPrivate.h:
WebInspector/WebNodeHighlight.h:
WebInspector/WebNodeHighlightView.h:

Import WebKitLegacy headers instead of WebKit headers.

WebKitLegacy/MigrateHeadersFromWebKitLegacy.make: Added.

New makefile that handles copying WebKitLegacy headers to the WebKit framework, rewriting WebKitLegacy
imports to WebKit imports. (On iOS the WebKit headers just forward to the relevant WebKitLegacy headers).
On OS X, this also handles copying WebKit2 headers to the WebKit framework, rewriting WebKit2 imports to WebKit imports
and getting rid of C SPI imports.

WebKitLegacy/MigrateHeadersToLegacy.make: Removed.

This is no longer needed.

WebKitLegacy/WebKit.h: Added.

New umbrella header that imports the modern API if available, as well as the legacy API (using WebKit/WebKitLegacy.h).

WebKitLegacy/WebKit.m:

This is an empty file so we'll have something to link.

WebKitLegacy/WebKitPrivate.h:

New SPI header that imports the private headers of the modern API.

WebView/WebDataSource.h:
WebView/WebDataSource.mm:
WebView/WebDataSourcePrivate.h:
WebView/WebDelegateImplementationCaching.h:
WebView/WebDocument.h:
WebView/WebDocumentInternal.h:
WebView/WebDocumentPrivate.h:
WebView/WebEditingDelegate.h:
WebView/WebEditingDelegatePrivate.h:
WebView/WebFrameLoadDelegate.h:
WebView/WebFrameLoadDelegatePrivate.h:
WebView/WebFramePrivate.h:
WebView/WebFrameView.h:
WebView/WebFrameViewInternal.h:
WebView/WebFrameViewPrivate.h:
WebView/WebHTMLRepresentation.h:
WebView/WebHTMLRepresentation.mm:
WebView/WebHTMLRepresentationPrivate.h:
WebView/WebHTMLView.h:
WebView/WebHTMLView.mm:
WebView/WebHTMLViewPrivate.h:
WebView/WebPDFView.h:
WebView/WebPolicyDelegatePrivate.h:
WebView/WebPreferencesPrivate.h:
WebView/WebResourcePrivate.h:
WebView/WebUIDelegate.h:
WebView/WebUIDelegatePrivate.h:
WebView/WebView.h:
WebView/WebView.mm:
WebView/WebViewPrivate.h:

Import WebKitLegacy headers instead of WebKit headers.

migrate-headers.sh:

Derived sources are put in DerivedSources/WebKitLegacy now.

Tools:

Scripts/check-for-webkit-framework-include-consistency:

Allos WAK headers in WebKitLegacy as well as WebKit.

TestWebKitAPI/Tests/WebKit2Cocoa/Download.mm:
TestWebKitAPI/Tests/WebKit2Cocoa/Navigation.mm:

Update header imports.

2:53 PM Changeset in webkit [168046] by hyatt@apple.com

171 edits in trunk

[New Multicolumn] Enable new multi-column mode
https://bugs.webkit.org/show_bug.cgi?id=131825

Reviewed by Simon Fraser.

Source/WebKit/mac:

WebView/WebPreferences.mm:

(+[WebPreferences initialize]):

Source/WebKit2:

Shared/WebPreferencesStore.h:

LayoutTests:

compositing/columns/composited-columns-expected.txt:
compositing/columns/composited-in-paginated-writing-mode-rl-expected.txt:
compositing/columns/composited-nested-columns-expected.txt:
compositing/columns/hittest-composited-in-paginated-expected.txt:
fast/dom/Element/getBoundingClientRect-expected.txt:
fast/dom/Element/getBoundingClientRect.html:
fast/multicol/flipped-blocks-border-after-expected.txt:
fast/multicol/pagination-h-horizontal-bt-expected.txt:
fast/multicol/pagination-h-horizontal-tb-expected.txt:
fast/multicol/pagination-h-vertical-lr-expected.txt:
fast/multicol/pagination-h-vertical-rl-expected.txt:
fast/multicol/pagination-v-horizontal-bt-expected.txt:
fast/multicol/pagination-v-horizontal-tb-expected.txt:
fast/multicol/pagination-v-vertical-lr-expected.txt:
fast/multicol/pagination-v-vertical-rl-expected.txt:
fast/multicol/progression-reverse-expected.txt:
fast/multicol/single-line-expected.txt:
fast/multicol/span/before-child-anonymous-column-block-expected.txt:
fast/multicol/span/generated-child-split-flow-crash-expected.txt:
fast/multicol/vertical-lr/rules-with-border-before-expected.txt:
fast/multicol/vertical-rl/rule-style-expected.txt:
fast/multicol/vertical-rl/rules-with-border-before-expected.txt:
platform/mac/css3/unicode-bidi-isolate-basic-expected.png:
platform/mac/css3/unicode-bidi-isolate-basic-expected.txt:
platform/mac/fast/block/float/float-not-removed-from-next-sibling4-expected.png:
platform/mac/fast/block/float/float-not-removed-from-next-sibling4-expected.txt:
platform/mac/fast/borders/border-antialiasing-expected.png:
platform/mac/fast/borders/border-antialiasing-expected.txt:
platform/mac/fast/line-grid/line-align-left-edges-expected.png:
platform/mac/fast/line-grid/line-align-right-edges-expected.png:
platform/mac/fast/line-grid/line-grid-contains-value-expected.png:
platform/mac/fast/line-grid/line-grid-floating-expected.png:
platform/mac/fast/line-grid/line-grid-inside-columns-expected.png:
platform/mac/fast/line-grid/line-grid-inside-columns-expected.txt:
platform/mac/fast/line-grid/line-grid-into-columns-expected.png:
platform/mac/fast/line-grid/line-grid-into-columns-expected.txt:
platform/mac/fast/line-grid/line-grid-into-floats-expected.png:
platform/mac/fast/line-grid/line-grid-positioned-expected.png:
platform/mac/fast/multicol/block-axis-horizontal-bt-expected.txt:
platform/mac/fast/multicol/block-axis-horizontal-tb-expected.txt:
platform/mac/fast/multicol/block-axis-vertical-lr-expected.txt:
platform/mac/fast/multicol/block-axis-vertical-rl-expected.txt:
platform/mac/fast/multicol/border-padding-pagination-expected.png:
platform/mac/fast/multicol/border-padding-pagination-expected.txt:
platform/mac/fast/multicol/client-rects-expected.png:
platform/mac/fast/multicol/client-rects-expected.txt:
platform/mac/fast/multicol/column-break-with-balancing-expected.txt:
platform/mac/fast/multicol/column-count-with-rules-expected.txt:
platform/mac/fast/multicol/column-rules-expected.png:
platform/mac/fast/multicol/column-rules-expected.txt:
platform/mac/fast/multicol/column-rules-stacking-expected.txt:
platform/mac/fast/multicol/columns-shorthand-parsing-expected.txt:
platform/mac/fast/multicol/float-avoidance-expected.txt:
platform/mac/fast/multicol/float-multicol-expected.txt:
platform/mac/fast/multicol/float-paginate-complex-expected.txt:
platform/mac/fast/multicol/float-paginate-empty-lines-expected.txt:
platform/mac/fast/multicol/float-paginate-expected.txt:
platform/mac/fast/multicol/layers-in-multicol-expected.png:
platform/mac/fast/multicol/layers-in-multicol-expected.txt:
platform/mac/fast/multicol/layers-split-across-columns-expected.txt:
platform/mac/fast/multicol/margin-collapse-expected.txt:
platform/mac/fast/multicol/max-height-columns-block-expected.png:
platform/mac/fast/multicol/max-height-columns-block-expected.txt:
platform/mac/fast/multicol/nested-columns-expected.png:
platform/mac/fast/multicol/nested-columns-expected.txt:
platform/mac/fast/multicol/overflow-across-columns-expected.txt:
platform/mac/fast/multicol/overflow-across-columns-percent-height-expected.txt:
platform/mac/fast/multicol/overflow-unsplittable-expected.txt:
platform/mac/fast/multicol/paginate-block-replaced-expected.txt:
platform/mac/fast/multicol/pagination/BottomToTop-bt-expected.txt:
platform/mac/fast/multicol/pagination/BottomToTop-lr-expected.txt:
platform/mac/fast/multicol/pagination/BottomToTop-rl-expected.txt:
platform/mac/fast/multicol/pagination/BottomToTop-tb-expected.txt:
platform/mac/fast/multicol/pagination/LeftToRight-bt-expected.txt:
platform/mac/fast/multicol/pagination/LeftToRight-lr-expected.txt:
platform/mac/fast/multicol/pagination/LeftToRight-rl-expected.png:
platform/mac/fast/multicol/pagination/LeftToRight-rl-expected.txt:
platform/mac/fast/multicol/pagination/LeftToRight-tb-expected.txt:
platform/mac/fast/multicol/pagination/RightToLeft-bt-expected.txt:
platform/mac/fast/multicol/pagination/RightToLeft-lr-expected.txt:
platform/mac/fast/multicol/pagination/RightToLeft-rl-expected.txt:
platform/mac/fast/multicol/pagination/RightToLeft-tb-expected.txt:
platform/mac/fast/multicol/pagination/TopToBottom-bt-expected.png:
platform/mac/fast/multicol/pagination/TopToBottom-bt-expected.txt:
platform/mac/fast/multicol/pagination/TopToBottom-lr-expected.txt:
platform/mac/fast/multicol/pagination/TopToBottom-rl-expected.txt:
platform/mac/fast/multicol/pagination/TopToBottom-tb-expected.txt:
platform/mac/fast/multicol/positioned-split-expected.txt:
platform/mac/fast/multicol/positive-leading-expected.txt:
platform/mac/fast/multicol/scrolling-overflow-expected.png:
platform/mac/fast/multicol/scrolling-overflow-expected.txt:
platform/mac/fast/multicol/shadow-breaking-expected.png:
platform/mac/fast/multicol/shadow-breaking-expected.txt:
platform/mac/fast/multicol/shrink-to-column-height-for-pagination-expected.txt:
platform/mac/fast/multicol/span/anonymous-before-child-parent-crash-expected.png:
platform/mac/fast/multicol/span/anonymous-before-child-parent-crash-expected.txt:
platform/mac/fast/multicol/span/anonymous-split-block-crash-expected.png:
platform/mac/fast/multicol/span/anonymous-split-block-crash-expected.txt:
platform/mac/fast/multicol/span/anonymous-style-inheritance-expected.txt:
platform/mac/fast/multicol/span/clone-anonymous-block-non-inline-child-crash-expected.png:
platform/mac/fast/multicol/span/clone-anonymous-block-non-inline-child-crash-expected.txt:
platform/mac/fast/multicol/span/clone-flexbox-expected.txt:
platform/mac/fast/multicol/span/clone-summary-expected.txt:
platform/mac/fast/multicol/span/generated-child-split-flow-crash-expected.png:
platform/mac/fast/multicol/span/span-as-immediate-child-complex-splitting-expected.png:
platform/mac/fast/multicol/span/span-as-immediate-child-complex-splitting-expected.txt:
platform/mac/fast/multicol/span/span-as-immediate-child-generated-content-expected.png:
platform/mac/fast/multicol/span/span-as-immediate-child-generated-content-expected.txt:
platform/mac/fast/multicol/span/span-as-immediate-child-property-removal-expected.png:
platform/mac/fast/multicol/span/span-as-immediate-child-property-removal-expected.txt:
platform/mac/fast/multicol/span/span-as-immediate-columns-child-dynamic-expected.png:
platform/mac/fast/multicol/span/span-as-immediate-columns-child-dynamic-expected.txt:
platform/mac/fast/multicol/span/span-as-immediate-columns-child-expected.txt:
platform/mac/fast/multicol/span/span-as-immediate-columns-child-removal-expected.png:
platform/mac/fast/multicol/span/span-as-immediate-columns-child-removal-expected.txt:
platform/mac/fast/multicol/span/span-as-nested-columns-child-dynamic-expected.png:
platform/mac/fast/multicol/span/span-as-nested-columns-child-dynamic-expected.txt:
platform/mac/fast/multicol/span/span-as-nested-columns-child-expected.png:
platform/mac/fast/multicol/span/span-as-nested-columns-child-expected.txt:
platform/mac/fast/multicol/span/span-as-nested-inline-block-child-expected.txt:
platform/mac/fast/multicol/span/span-margin-collapsing-expected.txt:
platform/mac/fast/multicol/table-margin-collapse-expected.txt:
platform/mac/fast/multicol/table-vertical-align-expected.txt:
platform/mac/fast/multicol/unsplittable-inline-block-expected.txt:
platform/mac/fast/multicol/vertical-lr/border-padding-pagination-expected.png:
platform/mac/fast/multicol/vertical-lr/border-padding-pagination-expected.txt:
platform/mac/fast/multicol/vertical-lr/column-break-with-balancing-expected.txt:
platform/mac/fast/multicol/vertical-lr/column-count-with-rules-expected.png:
platform/mac/fast/multicol/vertical-lr/column-count-with-rules-expected.txt:
platform/mac/fast/multicol/vertical-lr/column-rules-expected.png:
platform/mac/fast/multicol/vertical-lr/column-rules-expected.txt:
platform/mac/fast/multicol/vertical-lr/float-avoidance-expected.txt:
platform/mac/fast/multicol/vertical-lr/float-multicol-expected.txt:
platform/mac/fast/multicol/vertical-lr/float-paginate-complex-expected.txt:
platform/mac/fast/multicol/vertical-lr/float-paginate-expected.txt:
platform/mac/fast/multicol/vertical-lr/nested-columns-expected.png:
platform/mac/fast/multicol/vertical-lr/nested-columns-expected.txt:
platform/mac/fast/multicol/vertical-lr/unsplittable-inline-block-expected.txt:
platform/mac/fast/multicol/vertical-rl/border-padding-pagination-expected.png:
platform/mac/fast/multicol/vertical-rl/border-padding-pagination-expected.txt:
platform/mac/fast/multicol/vertical-rl/column-break-with-balancing-expected.txt:
platform/mac/fast/multicol/vertical-rl/column-count-with-rules-expected.png:
platform/mac/fast/multicol/vertical-rl/column-count-with-rules-expected.txt:
platform/mac/fast/multicol/vertical-rl/column-rules-expected.png:
platform/mac/fast/multicol/vertical-rl/column-rules-expected.txt:
platform/mac/fast/multicol/vertical-rl/float-avoidance-expected.txt:
platform/mac/fast/multicol/vertical-rl/float-multicol-expected.txt:
platform/mac/fast/multicol/vertical-rl/float-paginate-complex-expected.txt:
platform/mac/fast/multicol/vertical-rl/float-paginate-expected.txt:
platform/mac/fast/multicol/vertical-rl/nested-columns-expected.png:
platform/mac/fast/multicol/vertical-rl/nested-columns-expected.txt:
platform/mac/fast/multicol/vertical-rl/unsplittable-inline-block-expected.txt:
platform/mac/fast/overflow/paged-x-div-expected.png:
platform/mac/fast/overflow/paged-x-div-expected.txt:
platform/mac/fast/overflow/paged-x-div-with-column-gap-expected.png:
platform/mac/fast/overflow/paged-x-div-with-column-gap-expected.txt:
platform/mac/fast/overflow/paged-x-on-root-expected.png:
platform/mac/fast/overflow/paged-x-on-root-expected.txt:
platform/mac/fast/overflow/paged-x-with-column-gap-expected.png:
platform/mac/fast/overflow/paged-x-with-column-gap-expected.txt:
platform/mac/fast/overflow/paged-y-div-expected.png:
platform/mac/fast/overflow/paged-y-div-expected.txt:
platform/mac/fast/overflow/paged-y-on-root-expected.png:
platform/mac/fast/overflow/paged-y-on-root-expected.txt:
platform/mac/fast/repaint/multicol-repaint-expected.png:
platform/mac/fast/repaint/multicol-repaint-expected.txt:

2:48 PM Changeset in webkit [168045] by Simon Fraser

10 edits
1 copy
2 moves
2 adds
3 deletes in trunk/Tools

[iOS WK2] Add test URL to crash reports for the UI process, clean up project
https://bugs.webkit.org/show_bug.cgi?id=131954

Reviewed by Darin Adler.

WebKitTestRunner was adding application-specific information to crash reports
to log the test path, but only in the web process. Fix it to also do this
for the UI process, for both iOS and OS X.

Moved InjectedBundlePageMac.mm to InjectedBundlePageCocoa.mm and compile it for
both iOS and OS X.

Factored crash reprorter-related code into CrashReporterInfo, and call it from
a new TestController::platformWillRunTest() function on Mac and iOS.

Also remove Xcode-added unit test junk from the project.

WebKitTestRunner/InjectedBundle/cocoa/InjectedBundlePageCocoa.mm: Copied from Tools/WebKitTestRunner/InjectedBundle/ios/InjectedBundlePageIOS.mm.

(WTR::InjectedBundlePage::platformDidStartProvisionalLoadForFrame):

WebKitTestRunner/TestController.cpp:

(WTR::TestController::runTest):

WebKitTestRunner/TestController.h:
WebKitTestRunner/TestInvocation.cpp:

(WTR::TestInvocation::url):

WebKitTestRunner/TestInvocation.h:
WebKitTestRunner/WebKitTestRunner.xcodeproj/project.pbxproj:
WebKitTestRunner/WebKitTestRunnerAppTests/WebKitTestRunnerAppTests-Info.plist: Removed.
WebKitTestRunner/WebKitTestRunnerAppTests/WebKitTestRunnerAppTests.m: Removed.
WebKitTestRunner/WebKitTestRunnerAppTests/en.lproj/InfoPlist.strings: Removed.
WebKitTestRunner/cocoa/CrashReporterInfo.h: Renamed from Tools/WebKitTestRunner/InjectedBundle/ios/InjectedBundlePageIOS.mm.
WebKitTestRunner/cocoa/CrashReporterInfo.mm: Renamed from Tools/WebKitTestRunner/InjectedBundle/mac/InjectedBundlePageMac.mm.

(WTR::testPathFromURL):
(WTR::setCrashReportApplicationSpecificInformationToURL):

WebKitTestRunner/efl/TestControllerEfl.cpp:

(WTR::TestController::platformWillRunTest):

WebKitTestRunner/gtk/TestControllerGtk.cpp:

(WTR::TestController::platformWillRunTest):

WebKitTestRunner/ios/TestControllerIOS.mm:

(WTR::TestController::platformWillRunTest):
(WTR::TestController::setHidden):

WebKitTestRunner/mac/TestControllerMac.mm:

(WTR::TestController::platformWillRunTest):

2:48 PM Changeset in webkit [168044] by Simon Fraser

2 edits in trunk/Tools

Fix DRT assertion about mock scrollbars, which doesn't apply to iOS.

Reviewed by Tim Horton.

DumpRenderTree/mac/DumpRenderTree.mm:

(resetWebPreferencesToConsistentValues):
(prepareConsistentTestingEnvironment):

2:27 PM Changeset in webkit [168043] by hyatt@apple.com

8 edits in trunk/Source/WebCore

[New Multicolumn] ASSERTs in fast/dynamic/continuation-detach-crash.html
https://bugs.webkit.org/show_bug.cgi?id=132392

Reviewed by Dean Jackson.

rendering/RenderFlowThread.cpp:

(WebCore::RenderFlowThread::setRegionRangeForBox):
Add ASSERTs in case we ever try to set regions from the wrong flow thread
as part of the box's region range.

rendering/RenderMultiColumnFlowThread.cpp:

(WebCore::RenderMultiColumnFlowThread::flowThreadDescendantInserted):
Add a bunch of code that handles the discovery of a span from an outer flow thread
being inserted into an inner flow thread. This forces us to delete that placeholder
and shift the outer spanning content into the inner flow thread in order to get a new
mapping/placeholder created in the inner flow thread.

(WebCore::RenderMultiColumnFlowThread::flowThreadRelativeWillBeRemoved):
Tighten this code to use the parent() just in case we change the invariant of
parent = containingBlock later.

rendering/RenderMultiColumnFlowThread.h:

Add a static guard when shifting a spanner to prevent the outer flow thread from
thinking the spanner belongs to it when it gets punted out of the inner flow thread.
A better long-term solution might be to make the spanner map global instead of
per-flow thread.

rendering/RenderMultiColumnSpannerPlaceholder.cpp:

(WebCore::RenderMultiColumnSpannerPlaceholder::RenderMultiColumnSpannerPlaceholder):

rendering/RenderMultiColumnSpannerPlaceholder.h:

Cache the flow thread so that we can get back to it in order to detect if the
placeholder belongs to us or not.

rendering/RenderObject.cpp:

(WebCore::RenderObject::insertedIntoTree):
Notifications from insertedIntoTree are problematic, since this can be called during
the layout of the outer flow thread for content inside an inner flow thread that is
not getting a layout yet. This makes the currentFlowThread in the flow thread controller
inaccurate, so we have to add code to clear it out and put it back.

1:47 PM Changeset in webkit [168042] by Chris Fleizach

7 edits
3 adds in trunk

AX: Make "contenteditable" regions into AXTextAreas
https://bugs.webkit.org/show_bug.cgi?id=132379

Reviewed by Mario Sanchez Prada.

Source/WebCore:
Make contenteditable regions into AXTextAreas. This will allow for a more standardized
interface for interaction with assistive technologies.

Test: accessibility/content-editable-as-textarea.html

accessibility/AccessibilityNodeObject.cpp:

(WebCore::AccessibilityNodeObject::hasContentEditableAttributeSet):

accessibility/AccessibilityObject.cpp:

(WebCore::AccessibilityObject::contentEditableAttributeIsEnabled):

accessibility/AccessibilityObject.h:
accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::documentBasedSelectedTextRange):
(WebCore::AccessibilityRenderObject::selectedText):
(WebCore::AccessibilityRenderObject::selectedTextRange):
(WebCore::AccessibilityRenderObject::renderObjectIsObservable):
(WebCore::AccessibilityRenderObject::determineAccessibilityRole):
(WebCore::AccessibilityRenderObject::ariaSelectedTextRange): Deleted.

accessibility/AccessibilityRenderObject.h:

LayoutTests:

accessibility/content-editable-as-textarea.html: Added.
platform/mac-mountainlion/accessibility/content-editable-as-textarea-expected.txt: Added.
platform/mac/accessibility/content-editable-as-textarea-expected.txt: Added.

1:42 PM Changeset in webkit [168041] by Brian Burg

28 edits
1 delete in trunk/Source

Clean up unnecessary methods in the BackForwardClient interface
https://bugs.webkit.org/show_bug.cgi?id=131637

Reviewed by Andreas Kling.

Source/WebCore:
Demote back/current/forwardItem and iOS-specific methods from the
BackForwardClient interface. Convert the class to not be refcounted.

No new tests, no behavior was changed.

WebCore.exp.in:
history/BackForwardClient.h:

(WebCore::BackForwardClient::~BackForwardClient): Deleted.
(WebCore::BackForwardClient::backItem): Deleted.
(WebCore::BackForwardClient::currentItem): Deleted.
(WebCore::BackForwardClient::forwardItem): Deleted.

history/BackForwardController.cpp:

(WebCore::BackForwardController::BackForwardController):

history/BackForwardController.h: Take ownership of the passed BackForwardClient.

(WebCore::BackForwardController::client): Return a reference.

history/BackForwardList.h: Remove the Page field, since it isn't used any more.

(WebCore::BackForwardList::create): Deleted.

page/Page.h: Remove RefPtr from PageClient.
page/Page.cpp:

(WebCore::PageClients::PageClients): Initialize client to nullptr.

Source/WebKit/efl:
Remove uses of reference counting for BackForwardList.
Use references to BackForwardClient instead of pointers.
Stop using BackForwardClient::page() since it was removed.

ewk/ewk_history.cpp:

(ewk_history_clear):
(ewk_history_new):
(ewk_history_free):

ewk/ewk_view.cpp:

(_ewk_view_priv_new):
(ewk_view_history_enable_get):
(ewk_view_history_enable_set):
(ewk_view_history_get):

Source/WebKit/mac:
BackForwardClient instances now have ownership lifetime semantics, so
WebBackForwardList now explicitly deletes its inner BackForwardList.

Convert uses of WebCore::BackForwardList through backForward().client() to
accept references instead of pointers.

Use BackForwardController methods rather than directly operating with the
BackForwardClient where possible.

Remove page cache-related methods that are not used anywhere and that call
BackForwardClient::page(), which is removed by this change.

History/WebBackForwardList.mm:

History/WebBackForwardListInternal.h:
WebCoreSupport/WebFrameLoaderClient.mm:

(WebFrameLoaderClient::canCachePage):

WebView/WebFrameView.mm:

(-[WebFrameView keyDown:keyDown:]):

WebView/WebView.mm:

(-[WebView _loadBackForwardListFromOtherView:]):
(-[WebView initWithCoder:]):
(-[WebView encodeWithCoder:]):
(-[WebView backForwardList]):
(-[WebView setMaintainsBackForwardList:]):

Source/WebKit/win:
Remove uses of reference counting for BackForwardList.
Use BackForwardController instead of BackForwardClient where possible.

WebBackForwardList.cpp:

(WebBackForwardList::WebBackForwardList):
(WebBackForwardList::~WebBackForwardList):
(WebBackForwardList::createInstance):

WebBackForwardList.h:
WebView.cpp:

(WebView::backForwardList):
(WebView::canGoBack):
(WebView::canGoForward):
(WebView::loadBackForwardListFromOtherView):

Source/WebKit2:
Remove stubs for iOS-specific methods in the BackForwardClient interface.

Construct a WebBackForwardListProxy directly, and make the WebCore page
own the BackForwardClient instance. Convert uses of backForward().client()
to accept references instead of raw pointers.

WebKit2.xcodeproj/project.pbxproj:
WebProcess/InjectedBundle/InjectedBundleBackForwardList.cpp:

(WebKit::InjectedBundleBackForwardList::clear):

WebProcess/WebPage/WebBackForwardListProxy.cpp: Store a reference to

WebCore::Page instead of a pointer.
(WebKit::WebBackForwardListProxy::WebBackForwardListProxy):
(WebKit::WebBackForwardListProxy::create): Deleted.
(WebKit::WebBackForwardListProxy::addItem):
(WebKit::WebBackForwardListProxy::goToItem):
(WebKit::WebBackForwardListProxy::itemAtIndex):
(WebKit::WebBackForwardListProxy::backListCount):
(WebKit::WebBackForwardListProxy::forwardListCount):
(WebKit::WebBackForwardListProxy::close):
(WebKit::WebBackForwardListProxy::clear):
(WebKit::WebBackForwardListProxy::isActive):

WebProcess/WebPage/WebBackForwardListProxy.h:
WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::WebPage):

WebProcess/WebPage/ios/WebBackForwardListProxyIOS.mm: Removed.

1:30 PM Changeset in webkit [168040] by Brian Burg

2 edits in trunk/Source/WebInspectorUI

Web Inspector: clean up and decompose InspectorBackend functionality
https://bugs.webkit.org/show_bug.cgi?id=132387

Reviewed by Joseph Pecoraro.

Aside from renaming variables and other minor cleanup, this patch
changes the following:

When calling a command, only store callback data when a callback is passed.
Use explicit model objects for the agent, event, enum, and commands.
Separate the agent models from encoding and decoding of JSON messages.

UserInterface/Protocol/InspectorBackend.js:

(InspectorBackendClass):
(InspectorBackendClass.prototype.registerCommand):
(InspectorBackendClass.prototype.registerEnum):
(InspectorBackendClass.prototype.registerEvent):
(InspectorBackendClass.prototype.registerDomainDispatcher):
(InspectorBackendClass.prototype.dispatch):
(InspectorBackendClass.prototype.runAfterPendingDispatches):
(InspectorBackendClass.prototype._agentForDomain):
(InspectorBackendClass.prototype._registerSentCommand):
(InspectorBackendClass.prototype._dispatchCallback):
(InspectorBackendClass.prototype._dispatchEvent):
(InspectorBackendClass.prototype._invokeCommand):
(InspectorBackendClass.prototype._reportProtocolError):
(InspectorBackend.Agent):
(InspectorBackend.Agent.prototype.get domainName):
(InspectorBackend.Agent.prototype.set dispatcher):
(InspectorBackend.Agent.prototype.addEnum):
(InspectorBackend.Agent.prototype.addCommand):
(InspectorBackend.Agent.prototype.addEvent):
(InspectorBackend.Agent.prototype.getEvent):
(InspectorBackend.Agent.prototype.dispatchEvent):
(InspectorBackend.Command):
(InspectorBackend.Command.create):
(InspectorBackend.Command.prototype.get qualifiedName):
(InspectorBackend.Command.prototype.get commandName):
(InspectorBackend.Command.prototype.get callSignature):
(InspectorBackend.Command.prototype.get replySignature):
(InspectorBackend.Command.prototype.invoke):
(InspectorBackend.Command.prototype.supports):
(InspectorBackend.Command.prototype._invokeWithArguments):
(InspectorBackend.Event):
(InspectorBackend.Enum):
(InspectorBackendClass.prototype.callback): Deleted.
(InspectorBackendClass.prototype._registerPendingResponse): Deleted.
(InspectorBackendClass.prototype._invokeMethod): Deleted.
(InspectorBackendClass.prototype._getAgent): Deleted.
(InspectorBackendClass.prototype.reportProtocolError): Deleted.
(InspectorBackendCommand): Deleted.
(InspectorBackendCommand.create): Deleted.
(InspectorBackendCommand.prototype.invoke): Deleted.
(InspectorBackendCommand.prototype.supports): Deleted.
(InspectorBackendCommand.prototype._invokeWithArguments): Deleted.

1:00 PM Changeset in webkit [168039] by andersca@apple.com

2 edits in trunk/Tools

check-for-inappropriate-macros-in-external-headers should get the product name, not the project name
https://bugs.webkit.org/show_bug.cgi?id=132397

Reviewed by Dan Bernstein.

Scripts/check-for-inappropriate-macros-in-external-headers:

12:52 PM Changeset in webkit [168038] by barraclough@apple.com

14 edits
2 adds in trunk/Source/WebKit2

Fix PageVisibility on iOS
https://bugs.webkit.org/show_bug.cgi?id=132393

Reviewed by Andreas Kling.

Currently page visibility API doesn't work correctly on WK2 iOS for a few reasons,
the most significant of which being that the moment a WKWebView leaves the window
we'll suspend the content process, which removes the possibility for any notification
to be delivered. This patch addresses this issue, by allowing the process to run for
long enough for the notification to be delivered.

1) Introduce a new class, ProcessThrottler, to encapsulate the process suspension logic.
2) WebPageProxy uses ProcessThrottler::VisibilityToken to communicate visibility to the throttler.
3) WebPageProxy tracks pending didUpdateViewState messages to detect when the view state update in

the web content process has completed.

4) Distiguish between 'Background' and 'Suspended' states in the ProcessAssertion.

Shared/ChildProcessProxy.h:
- moved m_assertion to NetworkProcessProxy / WebProcessProxy.
UIProcess/Network/NetworkProcessProxy.h:
- added m_assertion.
UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::WebPageProxy):

initialize VisibilityToken state.

(WebKit::WebPageProxy::reattachToWebProcess):

reinitialize VisibilityToken state.

(WebKit::WebPageProxy::viewStateDidChange):

update VisibilityToken, increment m_pendingViewStateUpdates as necessary.

(WebKit::WebPageProxy::updateVisibilityToken):

update the VisibiliyToken based on page visibility, and whether an update is still pending.

(WebKit::WebPageProxy::didUpdateViewState):

detect when a view state change has completed in the web process, and update throttle state as necessary.

UIProcess/WebPageProxy.h:

(WebKit::WebPageProxy::didUpdateViewState): Deleted.

moved to .cpp.

UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::WebProcessProxy):

initialize m_throttler.

(WebKit::WebProcessProxy::didFinishLaunching):

notify the PageThrottler of the new connection.

UIProcess/WebProcessProxy.h:

(WebKit::WebProcessProxy::throttler):

added accessor.

(WebKit::WebProcessProxy::updateProcessState): Deleted.

moved trottling login to ProcessThrottler.

UIProcess/ios/PageClientImplIOS.h:
UIProcess/ios/PageClientImplIOS.mm:

(WebKit::PageClientImpl::isViewWindowActive):
(WebKit::PageClientImpl::isViewFocused):
(WebKit::PageClientImpl::isViewVisible):
(WebKit::PageClientImpl::isViewVisibleOrOccluded):

these should only be true when the view is in a window.

(WebKit::PageClientImpl::isVisuallyIdle):

this should only be true when the view is not in a window.

UIProcess/ios/ProcessAssertion.h:
UIProcess/ios/ProcessAssertion.mm:

(WebKit::flagsForState):

map from enum -> BKSAssertion flags values.

(WebKit::ProcessAssertion::ProcessAssertion):

use flagsForState (add support for Suspended state).

(WebKit::ProcessAssertion::setState):

use flagsForState (add support for Suspended state).

UIProcess/ios/ProcessThrottler.h: Added.

(WebKit::ProcessThrottler::VisibilityToken::visibility):

accessor.

(WebKit::ProcessThrottler::VisibilityToken::setVisibility):

update Visibility value; update the token as necessary.

(WebKit::ProcessThrottler::ProcessThrottler):

constructor; does not take an assention until didConnnectToProcess is called.

(WebKit::ProcessThrottler::visibilityToken):

create a VisibilityToken.

(WebKit::ProcessThrottler::didConnnectToProcess):

take an assertion.

(WebKit::ProcessThrottler::weakPtr):

create a weak pointer, used for references from VisibilityToken to the throttler.

(WebKit::ProcessThrottler::assertionState):

determine the correct AssertionState for the process, based on current visibility.

(WebKit::ProcessThrottler::updateAssertion):

update assertion, called in response to visibility change.

UIProcess/ios/ProcessThrottler.mm: Added.

(WebKit::ProcessThrottler::VisibilityToken::VisibilityToken):

constructor.

(WebKit::ProcessThrottler::VisibilityToken::~VisibilityToken):

set visibility to hidden to reset.

(WebKit::ProcessThrottler::VisibilityToken::hideTimerFired):

automatically decay from Hiding -> Hidden on a timeout.

(WebKit::ProcessThrottler::VisibilityToken::setVisibilityInternal):

update counters tracking visibility in ProcessThrottler.

UIProcess/ios/WKContentView.mm:

(-[WKContentView didMoveToWindow]):

This affects all view state flags, not just the 'InWindow' state. If the view moves out of a window request a reply from the WebContent - we use this to detect when the page visibility event has completed.

UIProcess/ios/WebProcessProxyIOS.mm:

(WebKit::WebProcessProxy::updateProcessState): Deleted.

removed.

WebKit2.xcodeproj/project.pbxproj:
- added new files.

12:47 PM Changeset in webkit [168037] by andersca@apple.com

4 edits in trunk/Source/WebKit2

Preemptive header fixes for when WebKit.framework is going to provide the modern API
https://bugs.webkit.org/show_bug.cgi?id=132394

Reviewed by Dan Bernstein.

UIProcess/API/Cocoa/WKBrowsingContextPolicyDelegate.h:

(NS_ENUM):
Delete WKNavigationType for now. Ultimately the entire delegate should be removed.

UIProcess/API/Cocoa/WKPreferences.h:

Add a header guard so we can avoid importing WKPreferences.h more than once.

WebKit2.xcodeproj/project.pbxproj:

_WKScriptWorld.h should be private, not public.
WKScriptMessagePrivate.h and WKUserContentControllerPrivate.h should be private, not project.

12:42 PM Changeset in webkit [168036] by Brent Fulgham

4 edits in trunk/Source/WebCore

[Mac, iOS] Support caption activation via JS webkitHasClosedCaptions method
https://bugs.webkit.org/show_bug.cgi?id=132320

Reviewed by Eric Carlson.

Modules/mediacontrols/mediaControlsApple.css:

(video::-webkit-media-text-track-container .hidden): Added.

html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::configureTextTrackGroup): Added call to
new 'updateCaptionsContainer'.
(WebCore::HTMLMediaElement::updateCaptionContainer): Added.

html/HTMLMediaElement.h:

12:29 PM Changeset in webkit [168035] by commit-queue@webkit.org

2 edits in trunk/Source/WebKit2

Web Inspector: Ensure UIProcess checks in to webinspectord after spawning a WebProcess
https://bugs.webkit.org/show_bug.cgi?id=132389

Patch by Joseph Pecoraro <Joseph Pecoraro> on 2014-04-30
Reviewed by Timothy Hatcher.

We aggregate WebProcess WebView's under the UIProcess. If the UIProcess
didn't connect to webinspectord these WebViews would have remained
hidden. Always have the UIProcess connect to webinspectord when it
spawns a WebProcess and may have a child process holding views that
it ultimately owns and should display under the UIProcess name.

UIProcess/WebContext.cpp:

(WebKit::WebContext::createNewWebProcess):

12:29 PM Changeset in webkit [168034] by Lucas Forschler

6 edits
1 copy in branches/safari-537.76-branch

Merge r167211. <rdar://problem/16701806>

12:00 PM Changeset in webkit [168033] by Alan Bujtas

2 edits in trunk/Source/WebKit2

[iOS]Subpixel rendering: Extra line of pixels next to the YouTube loading indicator.
https://bugs.webkit.org/show_bug.cgi?id=132391

Reviewed by Simon Fraser.

CG and GraphicsContext clipping should use the same coordinates. Snapping either one
while leaving the other unsnapped results in clipping mismatch and that may produce
unpainted areas.

Not testable.

Shared/mac/RemoteLayerBackingStore.mm:

(WebKit::RemoteLayerBackingStore::drawInContext):

10:50 AM Changeset in webkit [168032] by ap@apple.com

12 edits in trunk/Source

https://bugs.webkit.org/show_bug.cgi?id=132363
Make Blob RawData immutable

Reviewed by Anders Carlsson.

Source/WebCore:

Modules/websockets/WebSocket.cpp: (WebCore::WebSocket::didReceiveBinaryData):

Create RawData in one step.

WebCore.exp.in: Don't export a constructor that we no longer have (and the new one is inline).

fileapi/Blob.cpp: Removed entirely dead code.

fileapi/WebKitBlobBuilder.cpp:
fileapi/WebKitBlobBuilder.h:

Updated to collect data in a plain Vector, so that we don't have to modify RawData.
Removed FIXMEs about renaming - there used to be a BlobBuilder exposed to JS, but
now this is just a helper to implement JS Blob constructor. We should probably
still rename it, but not how the FIXME suggested.

platform/network/BlobData.cpp:

(WebCore::BlobDataItem::detachFromCurrentThread): RawData::detachFromCurrentThread()
was a no-op.
(WebCore::BlobDataHandle::BlobDataHandle): Deleted. This was entirely dead code.
(WebCore::BlobDataHandle::~BlobDataHandle): Ditto.

platform/network/BlobData.h: Made RawData immutable.

xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::responseBlob):

Create RawData in one step.

Source/WebKit2:

Shared/FileAPI/BlobRegistrationData.cpp: (WebKit::BlobRegistrationData::decode):

Create RawData in one step.

10:41 AM Changeset in webkit [168031] by ddkilzer@apple.com

1 edit
2 copies
1 delete in trunk/LayoutTests

Move iphone-simulator test results landed in r167402 to the correct directory

platform/ios-sim/media/media-document-controls-size-expected.txt: Renamed from LayoutTests/platform/iphone-simulator/media/media-document-controls-size-expected.txt.
platform/ios-sim/media/media-document-controls-size.html: Renamed from LayoutTests/platform/iphone-simulator/media/media-document-controls-size.html.

10:30 AM Changeset in webkit [168030] by Lucas Forschler

6 edits
1 delete in branches/safari-537.76-branch

Rollout of r168020.

10:26 AM Changeset in webkit [168029] by weinig@apple.com

3 edits in trunk/Source/WebKit/mac

[iOS] -[WebHTMLView selectionImageForcingBlackText:] returns blank image on iOS
https://bugs.webkit.org/show_bug.cgi?id=132359

Reviewed by Darin Adler.

WebView/WebHTMLView.mm:

(imageFromRect):
(selectionImage):
(-[WebHTMLView selectionImageForcingBlackText:selectionImageForcingBlackText:]):
Bring back the old FrameSnapshottingMac code for drawing the selection image as drag code for this
is still unimplemented on iOS.

WebView/WebView.mm:

(-[WebView initSimpleHTMLDocumentWithStyle:frame:preferences:groupName:]):
Set the page scale for "simple HTML documents" (this is already done for the all other documents),
to ensure that the snapshot is the correct size.

9:55 AM Changeset in webkit [168028] by beidson@apple.com

2 edits in trunk/Source/WebKit2

Followup to: Handle selection services menu.
<rdar://problem/16727798> and https://bugs.webkit.org/show_bug.cgi?id=132362

UIProcess/mac/WebContextMenuProxyMac.mm:

(WebKit::WebContextMenuProxyMac::setupServicesMenu): Remove an ASSERT from previous version of the patch

that landed. I’d forgotten to add this stray change to my staging area.

9:55 AM Changeset in webkit [168027] by Martin Robinson

4 edits in trunk

[GTK] Make it easier to run CMake for downstreams
https://bugs.webkit.org/show_bug.cgi?id=132370

Reviewed by Carlos Garcia Campos.

Source/cmake/OptionsGTK.cmake: Turn PRODUCTION_MODE into DEVELOPER_MODE.

Tools:

Scripts/webkitdirs.pm:

(generateBuildSystemFromCMakeProject): Pass -DDEVELOPER_MODE when building
the GTK+ port.

9:45 AM Changeset in webkit [168026] by Carlos Garcia Campos

1 copy in releases/WebKitGTK/webkit-2.2.7

Tagging the WebKitGTK+ 2.2.7 release

9:43 AM Changeset in webkit [168025] by ap@apple.com

2 edits in trunk/LayoutTests

compositing/repaint/repaint-on-layer-grouping-change.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=132385

platform/mac/TestExpectations: Marked as such.

9:42 AM Changeset in webkit [168024] by hyatt@apple.com

2 edits in trunk/Source/WebCore

[New Multicolumn] Region offset not factored in when mapping to local coords
https://bugs.webkit.org/show_bug.cgi?id=132383

Reviewed by Anders Carlsson.

Make sure to cache the offset of the multicolumn set from its parent and then
add that in to the translation offset.

rendering/RenderMultiColumnFlowThread.cpp:

(WebCore::RenderMultiColumnFlowThread::mapAbsoluteToLocalPoint):

8:52 AM Changeset in webkit [168023] by Carlos Garcia Campos

4 edits in releases/WebKitGTK/webkit-2.2

Unreviewed. Update NEWS and Versions.m4 for 2.2.7 release.

Source/autotools/Versions.m4: Bump version numbers.

Source/WebKit/gtk:

NEWS: Add release notes.

8:03 AM Changeset in webkit [168022] by Lucas Forschler

3 edits
2 copies in branches/safari-537.76-branch

Merged r166049. <rdar://problem/16701945>

7:45 AM Changeset in webkit [168021] by Lucas Forschler

3 edits in branches/safari-537.76-branch/Source/WebCore

Merged r167278. <rdar://problem/16701788>

7:42 AM Changeset in webkit [168020] by Lucas Forschler

6 edits
1 copy in branches/safari-537.76-branch

Merged r167211. <rdar://problem/16701806>

7:29 AM Changeset in webkit [168019] by Lucas Forschler

3 edits
2 copies in branches/safari-537.76-branch

Merge r167167. <rdar://problem/16701836>

7:25 AM Changeset in webkit [168018] by Lucas Forschler

3 edits
2 copies in branches/safari-537.76-branch

Merged r167135. <rdar://problem/16701888>

7:23 AM Changeset in webkit [168017] by Lucas Forschler

4 edits
2 copies in branches/safari-537.76-branch

Merge r167093. <rdar://problem/16701798>

7:12 AM Changeset in webkit [168016] by Lucas Forschler

3 edits
2 copies in branches/safari-537.76-branch

Merge r167092. <rdar://problem/16701793>

7:08 AM Changeset in webkit [168015] by Lucas Forschler

2 edits in branches/safari-537.76-branch/Source/WebKit2

Merge r166810. <rdar://problem/16701862>

7:05 AM Changeset in webkit [168014] by Lucas Forschler

3 edits
2 copies in branches/safari-537.76-branch

Merge r166601. <rdar://problem/16701781>

7:02 AM Changeset in webkit [168013] by Lucas Forschler

3 edits
2 copies in branches/safari-537.76-branch

Merge 166457. <rdar://problem/16701904>

6:59 AM Changeset in webkit [168012] by Lucas Forschler

3 edits
2 copies in branches/safari-537.76-branch

Merge 166428. <rdar://problem/16701982>

6:51 AM Changeset in webkit [168011] by Csaba Osztrogonác

2 edits in trunk/Source/WebCore

Move removeEquivalentProperties functions to EditingStyle
https://bugs.webkit.org/show_bug.cgi?id=131093

Patch by Zsolt Borbely <zsborbely.u-szeged@partner.samsung.com> on 2014-04-30
Reviewed by Csaba Osztrogonác.

A follow-up to r167967. Use single line declaration for template methods.

editing/EditingStyle.h:

6:48 AM Changeset in webkit [168010] by Lucas Forschler

3 edits
2 copies in branches/safari-537.76-branch

Merge r166236. <rdar://problem/16701937>

6:30 AM Changeset in webkit [168009] by Carlos Garcia Campos

2 edits in releases/WebKitGTK/webkit-2.2/Source/WebKit2

Merge r167795 - [GTK] Plugin process crashes with GTK2 windowed plugins
https://bugs.webkit.org/show_bug.cgi?id=132127

Reviewed by Martin Robinson.

It happens sometimes because the socket is used before the plug
has been added. A runtime critical warnings is shown and it
sometimes ends up crashing.

WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp:

(WebKit::NetscapePlugin::platformPostInitializeWindowed): Do not
show the plug widget until the socket is connected.

3:44 AM Changeset in webkit [168008] by Carlos Garcia Campos

2 edits in releases/WebKitGTK/webkit-2.2/Source/WebKit2

Merge r167656 - [GTK] Crash after getting web view context property with g_object_get
https://bugs.webkit.org/show_bug.cgi?id=131983

Reviewed by Philippe Normand.

The problem is that the getter is using g_value_take_object() and
the default context is destroyed when the caller releases its ref.

UIProcess/API/gtk/WebKitWebView.cpp:

(webkitWebViewGetProperty): Use g_value_set_object() instead of
g_value_take_object().

3:41 AM Changeset in webkit [168007] by Carlos Garcia Campos

2 edits in releases/WebKitGTK/webkit-2.2/Source/WebCore

Merge r166304 - REGRESSION(r162679): Poster image visible under the video
https://bugs.webkit.org/show_bug.cgi?id=130783

Reviewed by Simon Fraser.

In the listed revision, we started checking for isRenderImage()
instead of isImage(). RenderMedias return 'true' for the first
but 'false' for the second. Change the if() statement to check
for isRenderMedia() in addition to !isRenderImage().

rendering/RenderLayerBacking.cpp:

(WebCore::RenderLayerBacking::isDirectlyCompositedImage):

3:36 AM WebKitGTK/KeepingTheTreeGreen edited by ltilve@igalia.com

(diff)

3:34 AM Changeset in webkit [168006] by Carlos Garcia Campos

3 edits
2 adds in releases/WebKitGTK/webkit-2.2

Merge r166090 - Source/WebCore: Fix a crash when assigning an object to document.location
https://bugs.webkit.org/show_bug.cgi?id=130213

Reviewed by Geoffrey Garen.

Convert location to string before we make use the document.
This prevents us from attempting to navigate a frame that
has already been removed.

Test: fast/dom/navigation-with-sideeffects-crash.html

bindings/js/JSDocumentCustom.cpp:

(WebCore::JSDocument::location):
(WebCore::JSDocument::setLocation):

LayoutTests: Fix semantics of JS execution when assigning an object to document.location
https://bugs.webkit.org/show_bug.cgi?id=130213

Reviewed by Geoffrey Garen.

fast/dom/navigation-with-sideeffects-expected.txt: Added.
fast/dom/navigation-with-sideeffects.html: Added.

3:21 AM Changeset in webkit [168005] by Carlos Garcia Campos

4 edits
2 adds in releases/WebKitGTK/webkit-2.2

Merge r165921 - Crash with long selector list
https://bugs.webkit.org/show_bug.cgi?id=123006

Reviewed by Andreas Kling.

Source/WebCore:

Test: fast/css/long-selector-list-crash.html

css/CSSSelectorList.cpp:

(WebCore::CSSSelectorList::CSSSelectorList):
(WebCore::CSSSelectorList::adoptSelectorVector):
(WebCore::CSSSelectorList::operator=):

css/StyleRule.cpp:

(WebCore::StyleRule::create):

Add a bunch of asserts.

(WebCore::StyleRule::splitIntoMultipleRulesWithMaximumSelectorComponentCount):

This could produce a zero-length selector list.

LayoutTests:

fast/css/long-selector-list-crash-expected.txt: Added.
fast/css/long-selector-list-crash.html: Added.

3:16 AM Changeset in webkit [168004] by Carlos Garcia Campos

2 edits in releases/WebKitGTK/webkit-2.2/Source/JavaScriptCore

Merge r165902 - Update type of local vars to match the type of String length.
<https://webkit.org/b/130077>

Reviewed by Geoffrey Garen.

runtime/JSStringJoiner.cpp:

(JSC::JSStringJoiner::join):

3:13 AM Changeset in webkit [168003] by Carlos Garcia Campos

5 edits
4 adds in releases/WebKitGTK/webkit-2.2

Merge r165821 - Mutating rules returned by getMatchedCSSRules can result in crash
https://bugs.webkit.org/show_bug.cgi?id=130209

Source/WebCore:

Reviewed by Andreas Kling.

The non-standard getMatchedCSSRules API returns CSSStyleRule objects that don't
have parent stylesheet pointer (as we don't know which sheet the rule originated from).
Mutating the rule via such wrapper can lead to crashes later as we fail to invalidate
the underlying stylesheet.

Fix by disallowing mutation of style rules that don't have parent sheet pointer. CSSStyleRule
has two mutable properties selectorText and style. The latter gives back CSSStyleDeclaration.
This patch disallows mutations in both cases for CSSStyleRules that don't have parent stylesheet
pointer.

While it is technically possible to have CSSRules that are legitimately disconnected
from stylesheet (by removing rule from sheet while holding a reference to it) it never
makes sense to mutate such rule as there is no way to do anything with it afterwards.

Tests: fast/css/getMatchedCSSProperties-rule-mutation.html

fast/css/getMatchedCSSRules-crash.html

css/CSSStyleRule.cpp:

(WebCore::CSSStyleRule::setSelectorText):

Bail out if parent stylesheet is null.

css/PropertySetCSSStyleDeclaration.cpp:

(WebCore::PropertySetCSSStyleDeclaration::setCssText):
(WebCore::PropertySetCSSStyleDeclaration::setProperty):
(WebCore::PropertySetCSSStyleDeclaration::removeProperty):
(WebCore::PropertySetCSSStyleDeclaration::setPropertyInternal):

Allow StyleRuleCSSStyleDeclaration subclass cancel the mutation via
boolean return value from willMutate.

(WebCore::StyleRuleCSSStyleDeclaration::willMutate):

Disallow mutation if the owning CSSStyleRule is null or has null stylesheet.

(WebCore::StyleRuleCSSStyleDeclaration::didMutate):

We never get here with null rule or stylesheet anymore.

css/PropertySetCSSStyleDeclaration.h:

(WebCore::PropertySetCSSStyleDeclaration::willMutate):

LayoutTests:

Reviewed by Andreas Kling.

fast/css/getMatchedCSSProperties-rule-mutation-expected.txt: Added.
fast/css/getMatchedCSSProperties-rule-mutation.html: Added.
fast/css/getMatchedCSSRules-crash-expected.txt: Added.
fast/css/getMatchedCSSRules-crash.html: Added.

2:48 AM Changeset in webkit [168002] by Carlos Garcia Campos

5 edits
3 adds in releases/WebKitGTK/webkit-2.2

Merge r165339 - SerializedScriptValue may move Identifiers between worlds
https://bugs.webkit.org/show_bug.cgi?id=129979

Reviewed by Andreas Kling.

Source/WebCore:

Test: fast/workers/worker-copy-shared-blob-url.html

Don't use Strings to store blob URLs as String's may be Identifiers
and they can only exist in one world/thread at a time.

Modules/indexeddb/IDBObjectStore.cpp:

(WebCore::IDBObjectStore::put):

bindings/js/SerializedScriptValue.cpp:

(WebCore::CloneDeserializer::deserializeString):
(WebCore::SerializedScriptValue::addBlobURL):
(WebCore::SerializedScriptValue::SerializedScriptValue):

bindings/js/SerializedScriptValue.h:

LayoutTests:

Add test cases

fast/workers/resources/worker-copy-shared-blob-url-worker.js: Added.

(count.0.onmessage):

fast/workers/worker-copy-shared-blob-url-expected.txt: Added.
fast/workers/worker-copy-shared-blob-url.html: Added.

2:36 AM Changeset in webkit [168001] by Carlos Garcia Campos

4 edits in releases/WebKitGTK/webkit-2.2/Source/WebCore

Merge r165145 - ASSERT(newestManifest) fails in WebCore::ApplicationCacheGroup::didFinishLoadingManifest()
https://bugs.webkit.org/show_bug.cgi?id=129753
<rdar://problem/12069835>

Reviewed by Alexey Proskuryakov.

Fixes an issue where an assertion failure would occur when visiting a web site whose on-disk
app cache doesn't contain a manifest resource.

For some reason an app cache for a web site may be partially written to disk. In particular, the
app cache may only contain a CacheGroups entry. That is, the manifest resource and origin records
may not be persisted to disk. From looking over the code, we're unclear how such a situation can occur
and hence have been unable to create such an app cache. We were able to reproduce this issue using
an app cache database file that was provided by a person that was affected by this issue.

No test included because it's not straightforward to write a test for this change.

loader/appcache/ApplicationCacheGroup.cpp:

(WebCore::ApplicationCacheGroup::checkIfLoadIsComplete): Assert that m_cacheBeingUpdated->manifestResource()
is non-null. Currently we only document this assumption in a code comment. Also separated a single assertion
expression into two assertion expressions to make it straightforward to identify the failing sub-expression
on failure.

loader/appcache/ApplicationCacheStorage.cpp:

(WebCore::ApplicationCacheStorage::store): Modified to call ApplicationCacheStorage::deleteCacheGroupRecord()
to remove a cache group and associated cache records (if applicable) before inserting a cache group entry.
This replacement approach will ultimately repair incomplete app cache data for people affected by this bug.
(WebCore::ApplicationCacheStorage::loadCache): Log an error and return nullptr if the cache we loaded doesn't
have a manifest resource.
(WebCore::ApplicationCacheStorage::deleteCacheGroupRecord): Added.
(WebCore::ApplicationCacheStorage::deleteCacheGroup): Extracted deletion logic for cache group record into
ApplicationCacheStorage::deleteCacheGroupRecord().

loader/appcache/ApplicationCacheStorage.h:

2:33 AM Changeset in webkit [168000] by Carlos Garcia Campos

4 edits
3 adds in releases/WebKitGTK/webkit-2.2

Merge r165138 - Fix crash in CompositeEditCommand::cloneParagraphUnderNewElement()
<http://webkit.org/b/129751>
<rdar://problem/16237965>

Reviewed by Jon Honeycutt.

Merged from Blink (patch by Yuta Kitamura):
https://src.chromium.org/viewvc/blink?revision=168160&view=revision
http://crbug.com/345005

The root cause is CompositeEditCommand::moveParagraphWithClones() passing
two positions |start| and |end| which do not follow the document order,
i.e. in some situations |start| is located after |end| because of
the difference in affinity.

This patch fixes this crash by normalizing |end| to |start| in such situations.
It also adds an ASSERT that checks the relationship between |start| and |end|.

Source/WebCore:

Test: editing/execCommand/format-block-crash.html

editing/CompositeEditCommand.cpp:

(WebCore::CompositeEditCommand::cloneParagraphUnderNewElement):
(WebCore::CompositeEditCommand::moveParagraphWithClones):

editing/CompositeEditCommand.h:

LayoutTests:

editing/execCommand/format-block-crash-expected.txt: Added.
editing/execCommand/format-block-crash.html: Added.
editing/execCommand/resources/format-block-crash-iframe.html: Added.

2:30 AM Changeset in webkit [167999] by Carlos Garcia Campos

3 edits
2 adds in releases/WebKitGTK/webkit-2.2

Merge r164933 - Ensure keySplines is valid in SMIL animations
<http://webkit.org/b/129547>
<rdar://problem/15676128>

Reviewed by Darin Adler.

Merged from Blink (patch by Philip Rogers):
https://src.chromium.org/viewvc/blink?revision=156452&view=revision
http://crbug.com/276111

This patch fixes a crash in SMIL animations when keySplines are not
specified. The SMIL spec is clear on this:
http://www.w3.org/TR/2001/REC-smil-animation-20010904/#AnimFuncCalcMode
"If there are any errors in the keyTimes specification (bad values,
too many or too few values), the animation will have no effect."

This patch simply checks that keyTimes is not empty. Previously,
splinesCount was set to be m_keySplines.size() + 1 in
SVGAnimationElement.cpp; this patch changes splinesCount to be equal
to m_keySplines.size() to make the logic easier to follow and to
match other checks in SVGAnimationElement::startedActiveInterval.

Source/WebCore:

Test: svg/animations/animate-keysplines-crash.html

svg/SVGAnimationElement.cpp:

(WebCore::SVGAnimationElement::startedActiveInterval):

LayoutTests:

svg/animations/animate-keysplines-crash-expected.txt: Added.
svg/animations/animate-keysplines-crash.html: Added.

2:26 AM Changeset in webkit [167998] by Carlos Garcia Campos

2 edits in releases/WebKitGTK/webkit-2.2/Source/WebCore

Merge r164876 - Properly clear m_logicallyLastRun to remove use-after-free possibility
https://bugs.webkit.org/show_bug.cgi?id=129489

Reviewed by David Hyatt.

A use-after-free issue was caught in Blink because m_logicallyLastRun
is not cleared when the item it points to is deleted. Clearing it
turns the use-after-free into a segfault, and prevents any future
use-after-frees from happening.

platform/text/BidiRunList.h:

(WebCore::BidiRunList<Run>::deleteRuns):

2:23 AM Changeset in webkit [167997] by Carlos Garcia Campos

2 edits in releases/WebKitGTK/webkit-2.2/Source/WTF

Merge r164408 - Crash in WTF::StringBuilder::append()

https://bugs.webkit.org/show_bug.cgi?id=125817
<rdar://problem/15671883>

Reviewed by Oliver Hunt.

wtf/text/StringBuilder.cpp:

(WTF::expandedCapacity):
Ensure that we return a new capacity of at least 'requiredLength' in
the case where requiredLength is large. Also, use unsigned rather than
size_t for the parameters and the return value, as callers pass
unsigned arguments and treat the result as an unsigned int.

2:20 AM Changeset in webkit [167996] by Carlos Garcia Campos

3 edits
2 adds in releases/WebKitGTK/webkit-2.2

Merge r164367 - Do not dispatch change event twice in single step action
https://bugs.webkit.org/show_bug.cgi?id=116936
<rdar://problem/16086828>

Reviewed by Ryosuke Niwa.

Merged from Blink (patch by Kent Tamura):
https://src.chromium.org/viewvc/blink?view=rev&revision=151175

Source/WebCore:

Test: fast/forms/number/number-type-update-by-change-event.html

html/InputType.cpp:

(WebCore::InputType::stepUpFromRenderer):

LayoutTests:

fast/forms/number/number-type-update-by-change-event-expected.txt: Added.
fast/forms/number/number-type-update-by-change-event.html: Added.

2:15 AM Changeset in webkit [167995] by Carlos Garcia Campos

3 edits
2 adds in releases/WebKitGTK/webkit-2.2

Merge r164204 - Ensure that removing an iframe from the DOM tree disconnects its Frame.
<https://webkit.org/b/128889>
<rdar://problem/15671221>

Merged from Blink (patch by Adam Klein):
https://src.chromium.org/viewvc/blink?revision=156174&view=revision

Source/WebCore:

SubframeLoadingDisabler wasn't catching the case when an <iframe> was,
in its unload handler, removed and re-added to the same parent.
Fix this by using a count of SubframeLoadingDisablers that are on the
stack for a given root, rather than a simple boolean.

Test: fast/frames/reattach-in-unload.html

html/HTMLFrameOwnerElement.h:

(WebCore::SubframeLoadingDisabler::disabledSubtreeRoots):

LayoutTests:

fast/frames/reattach-in-unload-expected.txt: Added.
fast/frames/reattach-in-unload.html: Added.

1:57 AM Changeset in webkit [167994] by Carlos Garcia Campos

3 edits
2 adds in releases/WebKitGTK/webkit-2.2

Merge r164170 - ASSERT_WITH_SECURITY_IMPLICATION in WebCore::toElement
https://bugs.webkit.org/show_bug.cgi?id=128810

Patch by Renata Hodovan <rhodovan.u-szeged@partner.samsung.com> on 2014-02-15
Reviewed by Ryosuke Niwa.

Source/WebCore:

Make CompositeEditCommand::cloneParagraphUnderNewElement() to work when |outerNode|
doesn't contain |start|.

Merged from Blink https://src.chromium.org/viewvc/blink?revision=161762&view=revision by yosin@chromium.org.

Test: editing/execCommand/indent-with-uneditable-crash.html

editing/CompositeEditCommand.cpp:

(WebCore::CompositeEditCommand::cloneParagraphUnderNewElement):

LayoutTests:

editing/execCommand/indent-with-uneditable-crash-expected.txt: Added.
editing/execCommand/indent-with-uneditable-crash.html: Added.

1:54 AM Changeset in webkit [167993] by Manuel Rego Casasnovas

7 edits in trunk/Source

[CSS Grid Layout] Enable runtime feature by default
https://bugs.webkit.org/show_bug.cgi?id=132189

Reviewed by Benjamin Poulain.

Source/WebCore:

page/Settings.in: Set cssGridLayoutEnabled to true.

Source/WebKit/mac:

WebView/WebPreferences.mm:

(+[WebPreferences initialize]): Remove unneeded changes that
enable/disable the runtime feature depending on the compilation flag.
And set it to true by default.

Source/WebKit2:

Shared/WebPreferencesStore.h: Remove unneeded changes that

enable/disable the runtime feature depending on the compilation flag.
And set it to true by default.

UIProcess/gtk/ExperimentalFeatures.cpp: Set it to true by default.

1:37 AM Changeset in webkit [167992] by Carlos Garcia Campos

2 edits in releases/WebKitGTK/webkit-2.2/Source/WebKit2

Merge r167128 - wk2-gtk does not display anything
https://bugs.webkit.org/show_bug.cgi?id=125558

Reviewed by Martin Robinson.

Remove fcntl call to set access mode flags on the duplicated files
descriptor. Those flags are ignored in Linux and make fcntl to
fail in FreeBSD. We should handle the case where the passed
protection is ReadOnly.
Thanks to Raphael Kubo da Costa who proposed the solution.

Platform/unix/SharedMemoryUnix.cpp:

(WebKit::SharedMemory::createHandle):
(WebKit::accessModeFile): Deleted.

1:35 AM Changeset in webkit [167991] by Carlos Garcia Campos

3 edits in releases/WebKitGTK/webkit-2.2/Source/WebKit2

Merge r167116 - [GTK][WK2] Move Vector objects into WebEditorClient::executePendingEditorCommands() invocations
https://bugs.webkit.org/show_bug.cgi?id=131454

Reviewed by Carlos Garcia Campos.

WebProcess/WebCoreSupport/WebEditorClient.h:
WebProcess/WebCoreSupport/gtk/WebEditorClientGtk.cpp:

(WebKit::WebEditorClient::executePendingEditorCommands): Take a const reference of the Vector
object to avoid unnecessary copies. Also deploy two range-based for loops and efficiently move
the Editor::Command objects into the other Vector.

1:29 AM Changeset in webkit [167990] by Carlos Garcia Campos

5 edits in releases/WebKitGTK/webkit-2.2/Source/WebCore

Unreviewed. [GTK] Plugin process crashes when loading totem plugin
https://bugs.webkit.org/show_bug.cgi?id=131357

Make Netscape browser functions static to avoid conflicts with
plugins defining those functions as well.

GNUmakefile.list.am: Do not build npapi.cpp.
plugins/PluginPackage.cpp:

(WebCore::pluginViewForInstance):
(WebCore::NPN_MemAlloc):
(WebCore::NPN_MemFree):
(WebCore::NPN_MemFlush):
(WebCore::NPN_ReloadPlugins):
(WebCore::NPN_RequestRead):
(WebCore::NPN_GetURLNotify):
(WebCore::NPN_GetURL):
(WebCore::NPN_PostURLNotify):
(WebCore::NPN_PostURL):
(WebCore::NPN_NewStream):
(WebCore::NPN_Write):
(WebCore::NPN_DestroyStream):
(WebCore::NPN_UserAgent):
(WebCore::NPN_Status):
(WebCore::NPN_InvalidateRect):
(WebCore::NPN_InvalidateRegion):
(WebCore::NPN_ForceRedraw):
(WebCore::NPN_GetValue):
(WebCore::NPN_SetValue):
(WebCore::NPN_GetJavaEnv):
(WebCore::NPN_GetJavaPeer):
(WebCore::NPN_PushPopupsEnabledState):
(WebCore::NPN_PopPopupsEnabledState):
(WebCore::NPN_PluginThreadAsyncCall):
(WebCore::NPN_GetValueForURL):
(WebCore::NPN_SetValueForURL):
(WebCore::NPN_GetAuthenticationInfo):
(WebCore::NPN_PopUpContextMenu):

plugins/PluginPackage.h:

(WebCore::PluginPackage::browserFuncs): Add accessor to browser
function pointers.

plugins/PluginView.cpp:

(WebCore::PluginView::stop): Use browser function pointers instead
of NPN wrappers that are now private.
(WebCore::PluginView::getValueForURL): Ditto.

1:23 AM Changeset in webkit [167989] by Carlos Garcia Campos

2 edits in releases/WebKitGTK/webkit-2.2/Source/WebKit2

Merge r167883 - [GTK] Crash in debug build with removing windowed plugin child widgets from the view
https://bugs.webkit.org/show_bug.cgi?id=132252

Reviewed by Philippe Normand.

UIProcess/API/gtk/WebKitWebViewBase.cpp:

1:23 AM Changeset in webkit [167988] by Manuel Rego Casasnovas

3 edits in trunk/Source/WebCore

[CSS Grid Layout] Wrap some specific grid code under compilation flag
https://bugs.webkit.org/show_bug.cgi?id=132341

Reviewed by Benjamin Poulain.

Some static variables in RenderBox are only used for CSS Grid Layout code. Wrap them and the related methods
under ENABLE_CSS_GRID_LAYOUT compilation flag.

rendering/RenderBox.cpp:

(WebCore::RenderBox::willBeDestroyed):
(WebCore::RenderBox::containingBlockLogicalWidthForContent):
(WebCore::RenderBox::containingBlockLogicalHeightForContent):
(WebCore::RenderBox::perpendicularContainingBlockLogicalHeight):
(WebCore::RenderBox::computePercentageLogicalHeight):

rendering/RenderBox.h:

1:16 AM Changeset in webkit [167987] by Carlos Garcia Campos

2 edits in releases/WebKitGTK/webkit-2.2/Source/WebKit2

Merge r166574 - [GTK] Don't copy the ResourceResponse object in webkitWebViewDecidePolicy
https://bugs.webkit.org/show_bug.cgi?id=131015

Reviewed by Carlos Garcia Campos.

UIProcess/API/gtk/WebKitWebView.cpp:

(webkitWebViewDecidePolicy): Avoid copying the ResourceResponce object returned by webkitURIResponseGetResourceResponse.
That function already returns a reference, so the return value should be stored accordingly.

1:07 AM Changeset in webkit [167986] by Carlos Garcia Campos

4 edits in releases/WebKitGTK/webkit-2.2/Source/WebCore

Merge r166480 - [GTK] [TextureMapper] Weird brightness with some videos with acceletared compositing
https://bugs.webkit.org/show_bug.cgi?id=130665

Reviewed by Martin Robinson.

When we uploaded a video texture to the mapper we were not
considering that some videos could be decoded into a format
without alpha component. Now we check if the video has alpha and
if it does not, we remove the alpha flag when retrieving the
texture from the pool. For this, the method to get the texture
from the pool was modified to receive the flags, that is mapped to
have alpha by default in order not to break any other existing
code.

Though we have a problem with AC in WTR and that makes it
currently not testable, no new tests are needed because once this
is fixed the current test set suffices to detect a possible
regression in this.

platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:

(WebCore::MediaPlayerPrivateGStreamerBase::updateTexture): Check
the video format and decide if the texture shall be pulled with
alpha support or not.

platform/graphics/texmap/TextureMapper.cpp:

(WebCore::TextureMapper::acquireTextureFromPool): Use the flags
when resetting the texture.

platform/graphics/texmap/TextureMapper.h:

(WebCore::BitmapTexture::Flag::None): Added with 0x00.
(WebCore::TextureMapper::acquireTextureFromPool): Added flag
parameter to set up the texture with the default for including
alpha channel.

12:08 AM Changeset in webkit [167985] by roger_fong@apple.com

2 edits in trunk/Source/WebKit2

Unreviewed. Wrong units used in offset calculation from r167961.

WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::determinePrimarySnapshottedPlugIn):
I accidentally mixed and matches LayoutUnits with Ints in offset calculation here.
It should all just be in LayoutUnits.

Note: See TracTimeline for information about the timeline view.

Download in other formats:

RSS Feed