Context Navigation

(WebCore::MediaPlayerPrivateAVFoundationCF::seekToTime):
(WebCore::MediaPlayerPrivateAVFoundationCF::platformMinTimeSeekable):
(WebCore):
(WebCore::MediaPlayerPrivateAVFoundationCF::platformMaxTimeSeekable):

platform/graphics/avfoundation/cf/MediaPlayerPrivateAVFoundationCF.h:

(MediaPlayerPrivateAVFoundationCF):

5:15 PM Changeset in webkit [148322] by barraclough@apple.com

2 edits in trunk/Source/WebKit2

Build fix - too soon!

UIProcess/API/mac/WKView.mm:

(-[WKView setFrameSize:]):

5:15 PM Changeset in webkit [148321] by rniwa@webkit.org

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

Merged r136619. <rdar://problem/13335066>

5:09 PM Changeset in webkit [148320] by rniwa@webkit.org

5 edits in branches/safari-536.30-branch

Merged r117463. <rdar://problem/13335066>

5:03 PM Changeset in webkit [148319] by Lucas Forschler

2 edits
2 deletes in branches/safari-536.30-branch/LayoutTests

Merged r138692. <rdar://problem/13334855>

4:58 PM Changeset in webkit [148318] by Lucas Forschler

1 edit
4 copies
3 adds
1 delete in branches/safari-536.30-branch/LayoutTests

Merged r138686. <rdar://problem/13334855>

4:55 PM Changeset in webkit [148317] by jpfau@apple.com

2 edits in trunk/Tools

Unreviewed, remove an old email address for myself.

Scripts/webkitpy/common/config/committers.py:

4:47 PM Changeset in webkit [148316] by Simon Fraser

2 edits in trunk/Source/WebCore

Demystify some code in RenderLayer::updateScrollableAreaSet()
https://bugs.webkit.org/show_bug.cgi?id=114543

Reviewed by Tim Horton.

Clean up some code that called functions with important side effects
inside a ternary operator, and clean up the #ifdefs.

rendering/RenderLayer.cpp:

(WebCore::RenderLayer::updateScrollableAreaSet):

4:46 PM Changeset in webkit [148315] by Lucas Forschler

7 edits
15 copies
2 deletes in branches/safari-536.30-branch

Merged r138654. <rdar://problem/13334855>

4:43 PM Changeset in webkit [148314] by mhahnenberg@apple.com

2 edits in trunk/Source/JavaScriptCore

Block freeing thread should call Region::destroy instead of delete
https://bugs.webkit.org/show_bug.cgi?id=114544

Reviewed by Oliver Hunt.

Since Region doesn't have a virtual destructor, calling delete will not properly clean up all of
the state of the Region. We should call destroy() instead.

heap/BlockAllocator.cpp:

(JSC::BlockAllocator::releaseFreeRegions):
(JSC::BlockAllocator::blockFreeingThreadMain):

4:42 PM Changeset in webkit [148313] by fpizlo@apple.com

3 edits in branches/dfgFourthTier/Source/JavaScriptCore

fourthTier: switch to using MCJIT and disable frame pointer elimination
https://bugs.webkit.org/show_bug.cgi?id=114542

Reviewed by Oliver Hunt and Michael Saboff.

ftl/FTLCompile.cpp:

(JSC::FTL::compile):

runtime/InitializeThreading.cpp:

(JSC::initializeThreadingOnce):

4:29 PM Changeset in webkit [148312] by benjamin@webkit.org

11 edits
1 add in trunk

[WK2] WebPageProxy loadURL() won't work when called just after terminateProcess()
https://bugs.webkit.org/show_bug.cgi?id=110743

Patch by Adenilson Cavalcanti <cavalcantii@gmail.com> on 2013-04-12
Reviewed by Benjamin Poulain.

Source/WebKit2:

A call to loadURL() just after terminating WebProcess will fail thanks to
WebPageProxy being in an undefined state since it is in the middle of its own
cleanup after process termination.

To properly fix this, not only WebPageProxy cleanup should be made
at WebProcess termination request, but also WebProcessProxy needs
to only return to its caller after terminating the process and
closing connections. Otherwise, WebPageProxy can even be able to
detect that WebProcess is no longer running, but a call to respawn
the process will fail.

To fix these issues, this patch moves the cleanup code to a shared private function
that is used for both the cases i.e. user termination and real crash. WebProcess
shutdown is done using a new method that ensures that all cleanup was done before
returning.

A last change introduced in this patch is that for user requested termination,
clients are no longer notified of a crash (since it is not a crash).

UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::terminateProcess):
(WebKit::WebPageProxy::processDidCrash):
(WebKit):
(WebKit::WebPageProxy::resetStateAfterProcessExited):

UIProcess/WebPageProxy.h:

(WebPageProxy):

UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::userRequestedTerminate):
(WebKit):

UIProcess/WebProcessProxy.h:

(WebProcessProxy):

Tools:

Adding a new test file to check if loading a page just after WebProcess
has crashed (or was terminated) works. The test executes the
following steps (Load, Crash, Load), thus stressing WebProcess
reattach and process termination code path.

TestWebKitAPI/GNUmakefile.am:
TestWebKitAPI/PlatformEfl.cmake:
TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
TestWebKitAPI/Tests/WebKit2/MouseMoveAfterCrash.cpp:

(TestWebKitAPI::setPageLoaderClient):
(TestWebKitAPI::TEST):

TestWebKitAPI/Tests/WebKit2/LoadPageOnCrash.cpp: Added.

(TestWebKitAPI):
(WebKit2CrashLoader):
(TestWebKitAPI::WebKit2CrashLoader::WebKit2CrashLoader):
(TestWebKitAPI::WebKit2CrashLoader::loadUrl):
(TestWebKitAPI::WebKit2CrashLoader::crashWebProcess):
(TestWebKitAPI::didFinishLoad):
(TestWebKitAPI::TEST):

TestWebKitAPI/Tests/WebKit2/WebKit2.pro:

4:18 PM Changeset in webkit [148311] by barraclough@apple.com

15 edits in trunk/Source/WebKit2

Add contentAnchor to WKView
http://bugs.webkit.org/show_bug.cgi?id=114469

Currently if you call setFrameSize: while frame size updates are disabled, or if the geometry
update times out, then the previously rendered page content will be rendered anchored to the
top left corner of the frame.

This is appropriate if the frame is being resized from the bottom right corner. In order to
achieve a more desirable appearance if the frame is being resized from another corner we
should allow the corner at which the content anchors to be specified.

Reviewed by Simon Fraser.

UIProcess/API/mac/WKView.mm:

(-[WKView setFrameSize:]):

If the content anchor has been set, then updating the frame size moves the origin within the layer.

(-[WKView _setDrawingAreaSize:]):

Instruct the drawing area to paint at the correct position within the layer.

(-[WKView _setAcceleratedCompositingModeRootLayer:]):

If the root layer is cleared this implicitly resets any origin position we had set within it.

(-[WKView initWithFrame:contextRef:pageGroupRef:relatedToPage:]):

initialize new data members.

(-[WKView setContentAnchor:]):
(-[WKView contentAnchor]):

Accessor implementation for @property contentAnchor.

UIProcess/API/mac/WKViewPrivate.h:

(NS_ENUM):

Added WKContentAnchor enum type, and @property contentAnchor.

UIProcess/DrawingAreaProxy.cpp:

(WebKit::DrawingAreaProxy::setSize):

Don't suppress geometry updates where the layer poistion has changed.

UIProcess/DrawingAreaProxy.h:

(DrawingAreaProxy):

Added m_layerPosition, layerPosition argument to setSize

UIProcess/mac/TiledCoreAnimationDrawingAreaProxy.h:

(TiledCoreAnimationDrawingAreaProxy):

Added m_lastSentLayerPosition

UIProcess/mac/TiledCoreAnimationDrawingAreaProxy.mm:

(WebKit::TiledCoreAnimationDrawingAreaProxy::didUpdateGeometry):

Don't suppress geometry updates where the layer poistion has changed.

(WebKit::TiledCoreAnimationDrawingAreaProxy::sendUpdateGeometry):

Pass layer position in UpdateGeometry message.

WebProcess/WebPage/DrawingArea.h:

(WebKit::DrawingArea::updateGeometry):

Added layerPosition argument to updateGeometry

WebProcess/WebPage/DrawingArea.messages.in:
- Added layerPosition argument to updateGeometry

WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.h:

(TiledCoreAnimationDrawingArea):

Added layerPosition argument to updateGeometry

WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.mm:

(WebKit::TiledCoreAnimationDrawingArea::updateGeometry):

Added layerPosition argument to updateGeometry

4:11 PM Changeset in webkit [148310] by ap@apple.com

3 edits
3 adds in trunk

<rdar://problem/13277362> REGRESSION (r141981): Crash when closing a Google Docs document
https://bugs.webkit.org/show_bug.cgi?id=114541

Reviewed by Brady Eidson.

Test: http/tests/ssl/ping-with-unsafe-redirect.html

loader/FrameNetworkingContext.h: (WebCore::FrameNetworkingContext::shouldClearReferrerOnHTTPSToHTTPRedirect): Added a null check. All NetworkingContext functions except for this one had it already.

4:07 PM Changeset in webkit [148309] by Lucas Forschler

1 edit in branches/safari-536.30-branch/LayoutTests/http/tests/security/xssAuditor/resources/echo-intertag.pl

Update from testRunner to layoutTestController

4:07 PM Changeset in webkit [148308] by Lucas Forschler

2 edits in branches/safari-536.30-branch/LayoutTests

Update from testRunner to layoutTestController

3:50 PM Changeset in webkit [148307] by rniwa@webkit.org

2 edits in trunk/LayoutTests

Unreviewed. Don't forget to skip the test in WebKit2 for now.

platform/mac-wk2/TestExpectations:

3:46 PM Changeset in webkit [148306] by rniwa@webkit.org

10 edits
2 adds in trunk

[Mac] REGRESSION: Auto substitution strips new lines
https://bugs.webkit.org/show_bug.cgi?id=114537

Reviewed by Enrica Casucci.

Source/WebCore:

The bug was caused by SpellingCorrectionCommand's use of InsertTextCommand. This command can't insert
new lines and there's even an assertion for it. Use TypingCommand::insertText instead.

Since TypingCommand::insertText calls appliedEditing on its own, we need to avoid calling that again in
CompositeEditCommand::apply after SpellingCorrectionCommand::doApply. Replaced the check in apply to use
callsAppliedEditingInDoApply instead of isTypingCommand, and added callsAppliedEditingInDoApply to both
TypingCommand and SpellingCorrectionCommand to return true (it returns false by default).

Test: platform/mac/editing/spelling/autocorrection-with-multi-line-text.html

editing/CompositeEditCommand.cpp:

(WebCore::CompositeEditCommand::apply): Use TypingCommand::insertText instead of InsertTextCommand
(WebCore::CompositeEditCommand::callsAppliedEditingInDoApply): Added. Returns false.

editing/CompositeEditCommand.h:

(CompositeEditCommand):

editing/SpellingCorrectionCommand.cpp:

(WebCore::SpellingCorrectionCommand::doApply):
(WebCore::SpellingCorrectionCommand::callsAppliedEditingInDoApply): Added. Returns true.

editing/SpellingCorrectionCommand.h:

(SpellingCorrectionCommand):

editing/TypingCommand.cpp:

(WebCore::TypingCommand::callsAppliedEditingInDoApply): Added. Returns true.

editing/TypingCommand.h:

(TypingCommand):

Tools:

Add a rule to replace "helloleworld" by "hello\nworld" for testing purpose.

DumpRenderTree/mac/DumpRenderTree.mm:

(resetDefaultsToConsistentValues):

LayoutTests:

Added a regression test. Unfortunately, we can't setup text substituion in DumpRenderTree
so use NSSpellChecker to exercise the relevant code path.

platform/mac/editing/spelling/autocorrection-with-multi-line-text-expected.txt: Added.
platform/mac/editing/spelling/autocorrection-with-multi-line-text.html: Added.

3:04 PM Changeset in webkit [148305] by commit-queue@webkit.org

16 edits
1 copy
1 add in trunk/Source/WebCore

Refactor TextTrack and TextTrackList to make it easier to add audio and video tracks
https://bugs.webkit.org/show_bug.cgi?id=114330

Patch by Brendan Long <b.long@cablelabs.com> on 2013-04-12
Reviewed by Jer Noble.

No new tests because this doesn't add new functionality.

CMakeLists.txt: Only compile text track files if ENABLE(VIDEO_TRACK), and add TrackListBase
Target.pri: Add TrackListBase
WebCore.vcproj/WebCore.vcproj: Correct InbandTextTrackPrivate.h's location and add TrackListBase
WebCore.xcodeproj/project.pbxproj: Add TrackListBase
html/HTMLMediaElement.cpp: Renamed "track" functions to "textTrack"

(WebCore::HTMLMediaElement::mediaPlayerDidAddTextTrack): Renamed.
(WebCore::HTMLMediaElement::mediaPlayerDidRemoveTextTrack): Renamed.
(WebCore::HTMLMediaElement::addTextTrack): Renamed.
(WebCore::HTMLMediaElement::removeTextTrack): Renamed.
(WebCore::HTMLMediaElement::removeAllInbandTracks): Use renamed functions.
(WebCore::HTMLMediaElement::didAddTextTrack): Renamed.
(WebCore::HTMLMediaElement::didRemoveTextTrack): Renamed.

html/HTMLMediaElement.h:

Renamed "track" functions to "textTrack" and only forward-declare TextTrackList if ENABLE(VIDEO_TRACK)

html/HTMLTrackElement.cpp: Renamed "track" functions to "textTrack"

(WebCore::HTMLTrackElement::insertedInto):
(WebCore::HTMLTrackElement::removedFrom):

html/track/TextTrack.cpp:

Move media element, label, language and kind into TrackBase and move EventTarget functions into TextTrack.
(WebCore::TextTrack::TextTrack): Added.
(WebCore::TextTrack::interfaceName): Added.
(WebCore::TextTrack::scriptExecutionContext): Added.
(WebCore::TextTrack::eventTargetData): Added.
(WebCore::TextTrack::ensureEventTargetData): Added.
(WebCore::TextTrack::isValidKind): Added to support TrackBase::setKind()
(WebCore::TextTrack::setKind): Override to support textTrackKindChanged()
(WebCore::TextTrack::isRendered): Use kind(), label() and language() instead of m_kind, etc.
(WebCore::TextTrack::platformTextTrack): Ditto.
(WebCore::TextTrack::isMainProgramContent): Ditto.

html/track/TextTrack.h:

Same as TextTrack.cpp, plus added defaultKindKeyword() so TrackBase::setKind() will work and added
toTextTrack().

html/track/TextTrackList.cpp:

Move functionality into TrackListBase, add toTextTrack() casts, use
TrackBase where necesssary and use owner() instead of m_owner.

Moved EventTarget functions into TrackListBase.
(TextTrackList::TextTrackList):
(TextTrackList::getTrackIndexRelativeToRenderedTracks):
(TextTrackList::item):
(TextTrackList::invalidateTrackIndexesAfterTrack):
(TextTrackList::append):
(TextTrackList::remove):
(TextTrackList::contains):

html/track/TextTrackList.h:

Move EventTarget, RefCounted and m_inbandTracks into TrackListBase.
Override length(), contains() and remove().

html/track/TrackBase.cpp: Remove EventTarget functions.

(WebCore::TrackBase::TrackBase):
Remove EventTarget, add label and language
(WebCore::TrackBase::setKind): Added

html/track/TrackBase.h:

Don't extend EventTarget (AudioTrack and VideoTrack aren't). Handle
kind, label, language and media element.
(WebCore::TrackBase::setMediaElement): Added.
(WebCore::TrackBase::mediaElement): Added.
(WebCore::TrackBase::kind): Added.
(WebCore::TrackBase::label): Added.
(WebCore::TrackBase::setLabel): Added.
(WebCore::TrackBase::language): Added.
(WebCore::TrackBase::setLanguage): Added.

html/track/TrackListBase.cpp:

Added. Handles functionality common to TextTrackList, AudioTrackList and
VideoTrackList.
(TrackListBase::TrackListBase): Added.
(TrackListBase::~TrackListBase): Added.
(TrackListBase::length): Added.
(TrackListBase::remove): Added.
(TrackListBase::contains): Added.
(TrackListBase::scheduleAddTrackEvent): Added.
(TrackListBase::scheduleRemoveTrackEvent): Added.
(TrackListBase::scheduleChangeEvent): Added.
(TrackListBase::asyncEventTimerFired): Added.

html/track/TrackListBase.h:

Copied from Source/WebCore/html/track/TextTrackList.h. Handles
functionality common to TextTrackList, AudioTrackList and VideoTrackList.
(WebCore::TrackListBase::scriptExecutionContext): Added.
(WebCore::TrackListBase::clearOwner): Added.
(WebCore::TrackListBase::owner): Added.
(WebCore::TrackListBase::isFiringEventListeners): Added.

platform/graphics/MediaPlayer.cpp:

(WebCore::MediaPlayer::addTextTrack): Renamed (was addTrack)
(WebCore::MediaPlayer::removeTextTrack): Renamed (was removeTrack)

platform/graphics/MediaPlayer.h:

(WebCore::MediaPlayerClient::mediaPlayerDidAddTextTrack): Renamed (was mediaPlayerDidAddTrack)
(WebCore::MediaPlayerClient::mediaPlayerDidRemoveTextTrack): Renamed (was mediaPlayerDidRemoveTrack)

2:54 PM Changeset in webkit [148304] by rniwa@webkit.org

2 edits in trunk/Source/WebKit2

Unreviewed, rolling out r148296 and r148297.
http://trac.webkit.org/changeset/148296
http://trac.webkit.org/changeset/148297
https://bugs.webkit.org/show_bug.cgi?id=114536

Needs more thought (Requested by ap on #webkit).

NetworkProcess/mac/com.apple.WebKit.NetworkProcess.sb.in:

2:39 PM Changeset in webkit [148303] by oliver@apple.com

18 edits in trunk/Source/WebCore

make the codegenerator actually emit polymorphism check in the "no vtable" path
https://bugs.webkit.org/show_bug.cgi?id=114533

Reviewed by David Kilzer.

Make the "no vtable" check correct so that we actually plant a polymorphism
check. Update a few interfaces to have the correct validation attributes.

Modules/webdatabase/SQLTransaction.idl:
bindings/scripts/CodeGeneratorJS.pm:

(GenerateImplementation):

bindings/scripts/test/JS/JSTestInterface.cpp:

(WebCore::toJS):

bindings/scripts/test/TestInterface.idl:
html/track/TextTrackCue.idl:
svg/SVGAnimatedAngle.idl:
svg/SVGAnimatedBoolean.idl:
svg/SVGAnimatedEnumeration.idl:
svg/SVGAnimatedInteger.idl:
svg/SVGAnimatedLength.idl:
svg/SVGAnimatedLengthList.idl:
svg/SVGAnimatedNumber.idl:
svg/SVGAnimatedNumberList.idl:
svg/SVGAnimatedPreserveAspectRatio.idl:
svg/SVGAnimatedRect.idl:
svg/SVGAnimatedString.idl:
svg/SVGAnimatedTransformList.idl:

1:44 PM Changeset in webkit [148302] by commit-queue@webkit.org

3 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Enable selecting text in single line input field without selection point being actually on the targeted text vertically
https://bugs.webkit.org/show_bug.cgi?id=114515

Patch by Yongxin Dai <yodai@rim.com> on 2013-04-12
Reviewed by Rob Buis.

PR #317924.
Internally reviewed by Mike Fenton.

It was still hard to select text in URL bar with an inverted selection handle.
We improve the issue by replacing the Y coordinate of selection point with Y
coordinate of start caret for single line input filed. Thus, the Y coordinate
is always valid regardless of the actual location of the selection handle.

WebKitSupport/SelectionHandler.cpp:

(BlackBerry::WebKit::SelectionHandler::setSelection):
(BlackBerry::WebKit::SelectionHandler::startCaretViewportRect):
(WebKit):
(BlackBerry::WebKit::SelectionHandler::caretPositionChanged):

WebKitSupport/SelectionHandler.h:

(SelectionHandler):

1:28 PM Changeset in webkit [148301] by oliver@apple.com

14 edits in trunk/Source/WebCore

Make CodeGeneratorJS plant comments to explain failures in the binding validation
https://bugs.webkit.org/show_bug.cgi?id=114528

Reviewed by Sam Weinig.

No change in behaviour, just make the generated bindings include comments
to aid diagnosing issues that may be non-obvious.

bindings/scripts/CodeGeneratorJS.pm:

(GenerateImplementation):

bindings/scripts/test/JS/JSTestActiveDOMObject.cpp:

(WebCore::toJS):

bindings/scripts/test/JS/JSTestCustomNamedGetter.cpp:

(WebCore::toJS):

bindings/scripts/test/JS/JSTestEventConstructor.cpp:

(WebCore::toJS):

bindings/scripts/test/JS/JSTestEventTarget.cpp:

(WebCore::toJS):

bindings/scripts/test/JS/JSTestException.cpp:

(WebCore::toJS):

bindings/scripts/test/JS/JSTestInterface.cpp:

(WebCore::toJS):

bindings/scripts/test/JS/JSTestMediaQueryListListener.cpp:

(WebCore::toJS):

bindings/scripts/test/JS/JSTestNamedConstructor.cpp:

(WebCore::toJS):

bindings/scripts/test/JS/JSTestObj.cpp:

(WebCore::toJS):

bindings/scripts/test/JS/JSTestOverloadedConstructors.cpp:

(WebCore::toJS):

bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.cpp:

(WebCore::toJS):

bindings/scripts/test/JS/JSTestTypedefs.cpp:

(WebCore::toJS):

1:17 PM Changeset in webkit [148300] by timothy_horton@apple.com

8 edits in trunk/Source

REGRESSION (r138858): GIFs don't start playing when they come out of background tabs
https://bugs.webkit.org/show_bug.cgi?id=108864
<rdar://problem/13140489>

Reviewed by Antti Koivisto.

Don't repaint the world when animations resume; instead, FrameView
will cause all animated images to repaint. This line also had no effect for
TiledCoreAnimationDrawingArea, which does not implement setNeedsDisplay.

WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::resumeActiveDOMObjectsAndAnimations):

loader/cache/CachedImage.cpp:

(WebCore::CachedImage::resumeAnimatingImagesForLoader): Added.
Iterate through all of the loader's cached resources. If they are bitmap images
which claim to be animatable, notify the image's observers that the animation advanced
by one frame. In reality, the image is still on the same frame, but will now be repainted,
restarting the animation loop.

loader/cache/CachedImage.h:
page/FrameView.cpp:

(WebCore::FrameView::setIsInWindow):
When the FrameView is brought into a window, repaint all animated images.

platform/graphics/BitmapImage.cpp:

(WebCore::BitmapImage::canAnimate): Added.
Return whether or not the image should animate (claims to animate, has not already
finished its animation, and has an ImageObserver) *and* has more than one frame.

platform/graphics/BitmapImage.h:

(BitmapImage): Add canAnimate.

1:10 PM Changeset in webkit [148299] by roger_fong@apple.com

2 edits in trunk/Tools

Re-enable WinEWS tests.

Scripts/webkitpy/tool/commands/earlywarningsystem.py:

(WinEWS):

12:35 PM Changeset in webkit [148298] by Martin Robinson

13 edits in trunk/Source/WebCore

r148197 broke the GTK+ build
https://bugs.webkit.org/show_bug.cgi?id=114525

Reviewed by Anders Carlsson.

Re-add the ENABLE(SCROLLING_THREAD) guards to scrolling thread files
and extend them with USE(COORDINATED_GRAPHICS). This code is unused
by other configurations currently.

page/scrolling/ScrollingStateFixedNode.cpp: Re-add guards.
page/scrolling/ScrollingStateFixedNode.h: Ditto.
page/scrolling/ScrollingStateNode.cpp: Ditto.
page/scrolling/ScrollingStateNode.h: Ditto.
page/scrolling/ScrollingStateScrollingNode.cpp: Ditto.
page/scrolling/ScrollingStateScrollingNode.h: Ditto.
page/scrolling/ScrollingStateStickyNode.cpp: Ditto.
page/scrolling/ScrollingStateStickyNode.h: Ditto.
page/scrolling/ScrollingStateTree.cpp: Ditto.
page/scrolling/ScrollingStateTree.h: Ditto.
page/scrolling/mac/ScrollingStateNodeMac.mm: Ditto.
page/scrolling/mac/ScrollingStateScrollingNodeMac.mm: Ditto.

12:28 PM Changeset in webkit [148297] by ap@apple.com

2 edits in trunk/Source/WebKit2

<rdar://problem/13334446> [Mac] Tweak sandbox profile.

Apply a post-landing review comment.

NetworkProcess/mac/com.apple.WebKit.NetworkProcess.sb.in:

12:14 PM Changeset in webkit [148296] by ap@apple.com

2 edits in trunk/Source/WebKit2

<rdar://problem/13334446> [Mac] Tweak sandbox profile.

Reviewed by Jessie Berlin.

NetworkProcess/mac/com.apple.WebKit.NetworkProcess.sb.in:

12:09 PM Changeset in webkit [148295] by jer.noble@apple.com

2 edits in trunk/Source/WebCore

(AVFoundation) WebKit fails to detect live stream.
https://bugs.webkit.org/show_bug.cgi?id=114474

Reviewed by Eric Carlson.

Remove a workaround (added by r88944) for an error in an Apple.com javascript
media controller. The error has been fixed, and the workaround is causing problems
with other media streams.

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

(WebCore::MediaPlayerPrivateAVFoundationObjC::platformDuration):

11:57 AM Changeset in webkit [148294] by Carlos Garcia Campos

15 edits
6 adds
4 deletes in trunk

[GTK] Split GtkAuthenticationDialog in two widgets
https://bugs.webkit.org/show_bug.cgi?id=103644

Reviewed by Xan Lopez.

Source/WebCore:

The current GtkAuthenticationDialog implements both the common
logic and widgets to implement a real GtkDialog and a widget to be
embedded in any container. WebKit1 uses a GtkDialog while WebKit2
embeds the dialog in the WebView. This patch splits the code to
leave in Platform only the code that is actually common, leaving
the widget implementation to the WebKit layer, using a dialog in
WebKit1 and embedding auth widget in the WebView in WebKit2.

GNUmakefile.list.am: Add new files to compilation.
platform/gtk/GtkAuthenticationDialog.cpp: Removed.
platform/gtk/GtkAuthenticationDialog.h: Removed.
platform/gtk/WebKitAuthenticationWidget.cpp: Added.

(_WebKitAuthenticationWidgetPrivate):
(packTwoColumnLayoutInBox):
(createLabel):
(createEntry):
(webkitAuthenticationWidgetInitialize):
(webkitAuthenticationWidgetFinalize):
(webkit_authentication_widget_init):
(webkit_authentication_widget_class_init):
(webkitAuthenticationWidgetNew):
(webkitAuthenticationWidgetCreateCredential):
(webkitAuthenticationWidgetGetChallenge):

platform/gtk/WebKitAuthenticationWidget.h: Added.

(_WebKitAuthenticationWidget):
(_WebKitAuthenticationWidgetClass):

Source/WebKit/gtk:

GNUmakefile.am: Add new files to compilation.
WebCoreSupport/FrameLoaderClientGtk.cpp:

(WebKit::FrameLoaderClient::dispatchDidReceiveAuthenticationChallenge):
Use helper createAuthenticationDialog() to create and show the
auth dialog.

webkit/webkitauthenticationdialog.cpp: Added.

(authenticationDialogResponseCallback):
(createAuthenticationDialog):

webkit/webkitauthenticationdialog.h: Added.
webkit/webkitsoupauthdialog.cpp:

(sessionAuthenticate): Ditto.

Source/WebKit2:

GNUmakefile.list.am: Add new files to compilation.
UIProcess/API/gtk/WebKit2GtkAuthenticationDialog.cpp: Removed.
UIProcess/API/gtk/WebKit2GtkAuthenticationDialog.h: Removed.
UIProcess/API/gtk/WebKitAuthenticationDialog.cpp: Added.

(webkitAuthenticationDialogAuthenticate): Use the
AuthenticationChallengeProxy to authenticate and destroy the dialog.
(okButtonClicked): Call webkitAuthenticationDialogAuthenticate()
with the credential.
(cancelButtonClicked): Call
webkitAuthenticationDialogAuthenticate() with a NULL credential to
continue without credential.
(webkitAuthenticationDialogInitialize): Build the UI.
(webkitAuthenticationDialogDraw): Draw a background.
(webkitAuthenticationDialogMap): Grab default button.
(webkitAuthenticationDialogConstructed): Create a GtkStyleContext
with the GtkWindow path to draw the widget background like a real
dialog.
(webkit_authentication_dialog_class_init):
(webkitAuthenticationDialogNew): Create a new
WebKitAuthenticationDialog widget.

UIProcess/API/gtk/WebKitAuthenticationDialog.h: Added.
UIProcess/API/gtk/WebKitLoaderClient.cpp:
UIProcess/API/gtk/WebKitWebView.cpp:

(webkitWebViewHandleAuthenticationChallenge): Create a
WebKitAuthenticationDialog widget and pass add it to the
container.

UIProcess/API/gtk/WebKitWebViewBase.cpp:

(_WebKitWebViewBasePrivate): Updated to use the widget directly.
(webkitWebViewChildIsInternalWidget): Ditto.
(webkitWebViewBaseAddAuthenticationDialog): Ditto.
(webkitWebViewBaseCancelAuthenticationDialog): Ditto.
(webkitWebViewBaseContainerRemove): Ditto.
(webkitWebViewBaseContainerForall): Ditto.
(resizeWebKitWebViewBaseFromAllocation): Ditto.
(webkitWebViewBaseFocus): Ditto.
(webkitWebViewBaseDestroy): Destroy the auth widget is present.
(webkit_web_view_base_class_init): Add implementation for
GtkWidgetClass::destroy.

UIProcess/API/gtk/WebKitWebViewBasePrivate.h:

Tools:

gtk/generate-gtkdoc:

(get_webkit2_options): Do not generate api docs for
WebKitAuthenticationDialog which is private in WebKit2.
(get_webkit1_options): Do not generate api docs for
webkitauthenticationdialog which is private in WebKit1.

11:45 AM Changeset in webkit [148293] by Martin Robinson

2 edits in trunk

[GTK] Lower the pango dependency
https://bugs.webkit.org/show_bug.cgi?id=114520

Reviewed by Carlos Garcia Campos.

Source/autotools/Versions.m4: Only depend on Pango 1.30.0 instead of

Pango 1.32.0. 1.32.0 isn't strictly necessary and the version of GTK+ that
we depend on (3.6.0) depends on 1.30.0.

11:43 AM Changeset in webkit [148292] by rniwa@webkit.org

2 edits in trunk/Tools

WebKit Queues should capitalize K in Wk2
https://bugs.webkit.org/show_bug.cgi?id=114524

Reviewed by Philippe Normand.

QueueStatusServer/model/queues.py:

(Queue.display_name):

11:39 AM Changeset in webkit [148291] by jer.noble@apple.com

9 edits in trunk/Source/WebCore

Add support for MediaPlayer::minTimeSeekable()
https://bugs.webkit.org/show_bug.cgi?id=114484

Reviewed by Eric Carlson.

Plumb a new minTimeSeekable() method through MediaPlayer and
MediaPlayerPrivate and use this new method in the existing
HTMLMediaElement::minTimeSeekable() method.

html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::minTimeSeekable): Pass to MediaPlayer.

platform/graphics/MediaPlayer.cpp:

(WebCore::NullMediaPlayerPrivate::minTimeSeekable): Return 0.
(WebCore::MediaPlayer::minTimeSeekable): Pass to MediaPlayerPrivate.

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

(WebCore::MediaPlayerPrivateInterface::seekable): Define in terms of

minTimeSeekable()

(WebCore::MediaPlayerPrivateInterface::minTimeSeekable): Default to 0.

platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp:

(WebCore::MediaPlayerPrivateAVFoundation::MediaPlayerPrivateAVFoundation):

Initialize new m_cachedMinTimeSeekable ivar.

(WebCore::MediaPlayerPrivateAVFoundation::maxTimeSeekableDouble):

Renamed from maxTimeSeekable().

(WebCore::MediaPlayerPrivateAVFoundation::minTimeSeekable): Added.

Cache value of platformMinTimeSeekable().

(WebCore::MediaPlayerPrivateAVFoundation::seekableTimeRangesChanged):

Reset m_cachedMinTimeSeekable.

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

(WebCore::MediaPlayerPrivateAVFoundationObjC::seekToTime): float -> double.
(WebCore::MediaPlayerPrivateAVFoundationObjC::platformMinTimeSeekable):

Added. Retrieve the lowest value from -[m_playerItem seekableTimeRanges].

(WebCore::MediaPlayerPrivateAVFoundationObjC::platformMaxTimeSeekable):

float -> double.

11:36 AM Changeset in webkit [148290] by andersca@apple.com

2 edits in trunk/Source/WebKit/win

All tests crash on Windows
https://bugs.webkit.org/show_bug.cgi?id=114522

Reviewed by Ryosuke Niwa.

In the MSVC++ ABI, a member function pointer will have a different size depending on whether the
class it belongs to is defined or not. Because of this, when passing member function pointers as parameters
it's important to ensure that the class is known by both the caller and the callee.

WebFrame.cpp:

Include PolicyChecker.h

11:06 AM Changeset in webkit [148289] by rniwa@webkit.org

20 edits
2 adds
60 deletes in trunk

Unreviewed, rolling out r147942, r148026, and r148092.
http://trac.webkit.org/changeset/147942
http://trac.webkit.org/changeset/148026
http://trac.webkit.org/changeset/148092
https://bugs.webkit.org/show_bug.cgi?id=114521

"r147942 is buggy. Rollback all the patches depending on it."
(Requested by abucur on #webkit).

Source/WebCore:

dom/ContainerNode.cpp:

(WebCore::willRemoveChildren):
(WebCore):
(WebCore::ContainerNode::removeChildren):

dom/Document.cpp:

(WebCore::Document::nodeChildrenWillBeRemoved):
(WebCore):

dom/Document.h:

(Document):

dom/Node.cpp:
dom/Node.h:

(Node):

dom/NodeTraversal.cpp:
dom/NodeTraversal.h:

(ElementTraversal):
(NodeTraversal):

dom/Range.cpp:

(WebCore::boundaryNodeChildrenWillBeRemoved):
(WebCore):
(WebCore::Range::nodeChildrenWillBeRemoved):

dom/Range.h:
html/HTMLLIElement.cpp:

(WebCore::HTMLLIElement::attach):

html/HTMLOListElement.cpp:

(WebCore::HTMLOListElement::updateItemValues):
(WebCore::HTMLOListElement::recalculateItemCount):

rendering/RenderCounter.cpp:

(WebCore::previousInPreOrder):
(WebCore::previousSiblingOrParent):
(WebCore::parentElement):
(WebCore::nextInPreOrder):

rendering/RenderListItem.cpp:

(WebCore::enclosingList):
(WebCore::RenderListItem::nextListItem):
(WebCore::previousListItem):
(WebCore::RenderListItem::calcValue):
(WebCore::RenderListItem::explicitValueChanged):
(WebCore::previousOrNextItem):
(WebCore::RenderListItem::updateListMarkerNumbers):

rendering/RenderListItem.h:

(RenderListItem):

LayoutTests:

fast/dom/MutationObserver/added-out-of-order-expected.txt:
fast/dom/MutationObserver/added-out-of-order.html:
fast/dom/Range/range-remove-children-event-expected.txt: Removed.
fast/dom/Range/range-remove-children-event.html: Removed.
fast/dom/containerNode-expected.txt: Added.
fast/dom/containerNode.html: Added.
fast/dom/shadow/shadow-and-list-elements-expected.html:
fast/events/mutation-during-innerHTML-expected.txt: Removed.
fast/events/mutation-during-innerHTML.html: Removed.
fast/lists/positioned-count-crash-expected.txt:
fast/regions/counters/extract-list-items-001-expected.html: Removed.
fast/regions/counters/extract-list-items-001.html: Removed.
fast/regions/counters/extract-list-items-002-expected.html: Removed.
fast/regions/counters/extract-list-items-002.html: Removed.
fast/regions/counters/extract-list-items-003-expected.html: Removed.
fast/regions/counters/extract-list-items-003.html: Removed.
fast/regions/counters/extract-list-items-004-expected.html: Removed.
fast/regions/counters/extract-list-items-004.html: Removed.
fast/regions/counters/extract-list-items-005-expected.html: Removed.
fast/regions/counters/extract-list-items-005.html: Removed.
fast/regions/counters/extract-list-items-006-expected.html: Removed.
fast/regions/counters/extract-list-items-006.html: Removed.
fast/regions/counters/extract-list-items-007-expected.html: Removed.
fast/regions/counters/extract-list-items-007.html: Removed.
fast/regions/counters/extract-list-items-008-expected.html: Removed.
fast/regions/counters/extract-list-items-008.html: Removed.
fast/regions/counters/extract-list-items-009-expected.html: Removed.
fast/regions/counters/extract-list-items-009.html: Removed.
fast/regions/counters/extract-list-items-010-expected.html: Removed.
fast/regions/counters/extract-list-items-010.html: Removed.
fast/regions/counters/extract-list-items-011-expected.html: Removed.
fast/regions/counters/extract-list-items-011.html: Removed.
fast/regions/counters/extract-list-items-012-expected.html: Removed.
fast/regions/counters/extract-list-items-012.html: Removed.
fast/regions/counters/extract-list-items-013-expected.html: Removed.
fast/regions/counters/extract-list-items-013.html: Removed.
fast/regions/counters/extract-list-items-014-expected.html: Removed.
fast/regions/counters/extract-list-items-014.html: Removed.
fast/regions/counters/extract-list-items-015-expected.html: Removed.
fast/regions/counters/extract-list-items-015.html: Removed.
fast/regions/counters/extract-numbered-paragraphs-divs-001-expected.html: Removed.
fast/regions/counters/extract-numbered-paragraphs-divs-001.html: Removed.
fast/regions/counters/extract-numbered-paragraphs-divs-002-expected.html: Removed.
fast/regions/counters/extract-numbered-paragraphs-divs-002.html: Removed.
fast/regions/counters/extract-numbered-paragraphs-expected.html: Removed.
fast/regions/counters/extract-numbered-paragraphs.html: Removed.
fast/regions/counters/extract-numbered-spans-display-only-some-expected.html: Removed.
fast/regions/counters/extract-numbered-spans-display-only-some.html: Removed.
fast/regions/counters/extract-ordered-lists-in-regions-001-expected.html: Removed.
fast/regions/counters/extract-ordered-lists-in-regions-001.html: Removed.
fast/regions/counters/extract-ordered-lists-in-regions-002-expected.html: Removed.
fast/regions/counters/extract-ordered-lists-in-regions-002.html: Removed.
fast/regions/counters/extract-ordered-lists-in-regions-003-expected.html: Removed.
fast/regions/counters/extract-ordered-lists-in-regions-003.html: Removed.
fast/regions/counters/extract-ordered-lists-in-regions-explicit-counters-001-expected.html: Removed.
fast/regions/counters/extract-ordered-lists-in-regions-explicit-counters-001.html: Removed.
fast/regions/counters/extract-ordered-lists-in-regions-explicit-counters-002-expected.html: Removed.
fast/regions/counters/extract-ordered-lists-in-regions-explicit-counters-002.html: Removed.
fast/regions/counters/extract-ordered-lists-in-regions-explicit-counters-003-expected.html: Removed.
fast/regions/counters/extract-ordered-lists-in-regions-explicit-counters-003.html: Removed.
fast/regions/counters/extract-ordered-lists-in-regions-explicit-counters-004-expected.html: Removed.
fast/regions/counters/extract-ordered-lists-in-regions-explicit-counters-004.html: Removed.
fast/regions/counters/extract-ordered-lists-in-regions-explicit-counters-005-expected.html: Removed.
fast/regions/counters/extract-ordered-lists-in-regions-explicit-counters-005.html: Removed.
fast/regions/counters/extract-unordered-lists-in-regions-expected.html: Removed.
fast/regions/counters/extract-unordered-lists-in-regions.html: Removed.

10:58 AM Changeset in webkit [148288] by jer.noble@apple.com

13 edits
2 adds in trunk

TimeRanges::nearest() returns incorrect results.
https://bugs.webkit.org/show_bug.cgi?id=114483

Reviewed by Eric Carlson.

Add symbols needed by WebCoreTestSupport to exports list.

Source/autotools/symbols.filter:

Source/WebCore:

Test: media/timeranges-nearest.html

TimeRanges::nearest() has had an incorrect algorithm since its
addition, which has gone unnoticed because no media engine supports
seekable ranges with greater than one entry, and no media engine
seekable ranges with startTime values > 0.

Fix the algorithm used to walk a TimeRanges object; return the closest
time, not the smallest delta between range and target time.

html/TimeRanges.cpp:

(TimeRanges::nearest):

Add some internal functions to allow us to write LayoutTests for
TimeRanges objects:

WebCore.exp.in:
testing/Internals.cpp:

(WebCore::Internals::createTimeRanges):
(WebCore::Internals::closestTimeToTimeRanges):

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

Source/WebKit:

Add symbols needed by WebCoreTestSupport to the exports list.

WebKit.vcxproj/WebKitExportGenerator/WebKitExports.def.in:

Source/WebKit/win:

Add symbols needed by WebCoreTestSupport to the exports list.

WebKit.vcproj/WebKitExports.def.in:

LayoutTests:

media/timeranges-nearest-expected.txt: Added.
media/timeranges-nearest.html: Added.

10:40 AM Changeset in webkit [148287] by oliver@apple.com

2 edits in trunk/Source/WTF

Remove bogus change

10:24 AM Changeset in webkit [148286] by Carlos Garcia Campos

4 edits in trunk/Source/WebKit2

[GTK] Web Process crash when the UI process finishes too early
https://bugs.webkit.org/show_bug.cgi?id=112729

Reviewed by Anders Carlsson.

The problem is when creating the GSocket in the WorkQeue for the
socket descriptor. GLib considers a programmer error to create a
GSocket providing an invalid socket and finishes the process with
g_error(). We are actually providing a valid socket when creating
the GSocket, but it can be invalidated by the worker thread while
the GSocket is being created. This is because
registerEventSourceHandler is called from the main thread and
unregisterEventSourceHandler can be called from the worker
thread. We are currently registering two event handlers, one to
read data from the socket and another one to close the CoreIPC
connection when the socket connection is broken. Every event
source registered uses its own GSocket (even if the file
descriptor is actually the same), so that when the UI process
finishes too early, the first event handler can be executed in the
worker thread, closing the socket descriptor, while the main
thread is creating the GSocket for the second one.
We don't really need to use a separate event handler to monitor
the connection, because GSocket always notifies when condition is
G_IO_HUP and G_IO_ERR even if they haven't been explicitly set in
g_socket_create_source(). We can register socket event sources
differently, providing also a function to be called when the
connection is closed, using a single socket and the same even source.

Platform/CoreIPC/unix/ConnectionUnix.cpp:

(CoreIPC::Connection::platformInvalidate):
(CoreIPC::Connection::open): Register a single socket event
handler providing also a function to be called when the connection
is closed.

Platform/WorkQueue.h:

(WorkQueue):

Platform/gtk/WorkQueueGtk.cpp: The EventSource class has been

split, moving everyting specific to socket event source to a
derived class SocketEventSource.
(WorkQueue::EventSource::EventSource):
(WorkQueue::EventSource::performWork):
(WorkQueue::EventSource::performWorkOnce):
(WorkQueue::EventSource::performWorkOnTermination):
(WorkQueue::EventSource::deleteEventSource):
(WorkQueue::EventSource):
(WorkQueue::SocketEventSource):
(WorkQueue::SocketEventSource::SocketEventSource):
(WorkQueue::SocketEventSource::cancel):
(WorkQueue::SocketEventSource::didClose):
(WorkQueue::SocketEventSource::checkCondition):
(WorkQueue::SocketEventSource::eventCallback):
(WorkQueue::registerSocketEventHandler):
(WorkQueue::unregisterSocketEventHandler):
(WorkQueue::dispatchOnSource):

10:23 AM Changeset in webkit [148285] by eric.carlson@apple.com

23 edits
2 adds in trunk

Support "forced" subtitles
https://bugs.webkit.org/show_bug.cgi?id=114460

Reviewed by Jer Noble.

Source/WebCore:

Test: media/track/track-forced-subtitles.html

html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::configureTextTrackGroup): Enable a forced track if nothing else is enabled.
(WebCore::HTMLMediaElement::configureTextTracks): Include forced tracks with captions+subtitles.

html/track/InbandTextTrack.cpp:

(WebCore::InbandTextTrack::InbandTextTrack): Deal with kind=forced.

html/track/TextTrack.cpp:

(WebCore::TextTrack::forcedKeyword): New keyword.
(WebCore::TextTrack::isValidKindKeyword): Include forced.
(WebCore::TextTrack::platformTextTrack): Ditto.

html/track/TextTrack.h:

page/CaptionUserPreferences.cpp:

(WebCore::CaptionUserPreferences::primaryAudioTrackLanguageOverride): New override to support testing.

page/CaptionUserPreferences.h:

page/CaptionUserPreferencesMac.mm:

(WebCore::CaptionUserPreferencesMac::textTrackSelectionScore): Support forced tracks.

platform/graphics/InbandTextTrackPrivate.h:

platform/graphics/PlatformTextTrack.h: Add Forced, minor cleanup.

platform/graphics/avfoundation/InbandTextTrackPrivateAVF.cpp:

(WebCore::InbandTextTrackPrivateAVF::processCue): Drive-by enhancement: log cue position.

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

(WebCore::InbandTextTrackPrivateAVFObjC::kind): Support forced.
(WebCore::InbandTextTrackPrivateAVFObjC::label): Drive-by cleanup.

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

(WebCore::MediaPlayerPrivateAVFoundationObjC::processTextTracks): Don't filter out forced tracks.
(WebCore::MediaPlayerPrivateAVFoundationObjC::languageOfPrimaryAudioTrack): Log the language returned.

testing/Internals.cpp:

(WebCore::Internals::setPrimaryAudioTrackLanguageOverride): New.
(WebCore::Internals::setCaptionDisplayMode): New.

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

LayoutTests:

media/content/CC+Subtitles.m4v: Added.
media/track/track-forced-subtitles-in-band-expected.txt: Added.
media/track/track-forced-subtitles-in-band.html: Added.
media/trackmenu-test.js:

platform/efl/TestExpectations: Skip new test.
platform/gtk/TestExpectations: Ditto.
platform/mac/TestExpectations: Ditto.
platform/qt/TestExpectations: Ditto.
platform/win/TestExpectations: Ditto.

10:06 AM Changeset in webkit [148284] by ap@apple.com

3 edits in trunk/Source/WebKit2

<rdar://problem/13126204> [Mac] Tweak sandbox profile.

Reviewed by Anders Carlsson.

NetworkProcess/mac/com.apple.WebKit.NetworkProcess.sb.in:
WebProcess/com.apple.WebProcess.sb.in:

9:54 AM Changeset in webkit [148283] by rniwa@webkit.org

2 edits in trunk/Source/WebCore

[Mac] Some ligatures are applied across different fronts
https://bugs.webkit.org/show_bug.cgi?id=113604

Reviewed by Darin Adler.

The bug was caused by applyFontTransforms applying ligatures without updating the font data in the glyph buffer.

Suppose we have characters AB and A uses font X and B uses font Y. Further suppose that we have ligatures for AB
in Y. The problem was that WithIterator::advanceInternal calls applyFontTransforms on AB to apply this ligature
even though A and B use different fonts. Since X doesn't contain the same ligature for A to be paired with that
of B in X, we get a blank A.

Fixed the bug by resetting lastGlyphCount after calling applyFontTransforms. This ensures that we don't call
applyFontTransforms across different fonts.

No new tests since this requires a variant of Osaka font that doesn't come with OS X.

The bug 114482 tracks the effort to add a test for this bug fix. I've already asked someone to create a custom
font for us so that we can test this bug fix but that's going to a while.

platform/graphics/WidthIterator.cpp:

(WebCore::WidthIterator::advanceInternal):

9:52 AM Changeset in webkit [148282] by rniwa@webkit.org

26 edits
6 deletes in trunk

[Mac] Enable spellchecking tests added in r141471
https://bugs.webkit.org/show_bug.cgi?id=108525

Reviewed by Darin Adler.

Source/WebCore:

Made automatic quote substitution, dash substitution, text replacement, and spelling correction
togglable via internals object. These functions are no-op when text replacement is not used.

This allows us to disable automatic spelling correction in spellcheck tests.

Enabled several tests added in r141471.

WebCore.exp.in:
testing/Internals.cpp:

(WebCore::Internals::setContinuousSpellCheckingEnabled): Don't null check editor() since it's not
null whenever frame() is not null.
(WebCore::Internals::setAutomaticQuoteSubstitutionEnabled): Added.
(WebCore::Internals::setAutomaticLinkDetectionEnabled): Added.
(WebCore::Internals::setAutomaticDashSubstitutionEnabled): Added.
(WebCore::Internals::setAutomaticTextReplacementEnabled): Added.
(WebCore::Internals::setAutomaticSpellingCorrectionEnabled): Added.

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

Tools:

DumpRenderTree/mac/DumpRenderTree.mm:

(createWebViewAndOffscreenWindow):
(resetDefaultsToConsistentValues):

WebKitTestRunner/TestController.cpp:

(WTR::TestController::resetStateToConsistentValues):

LayoutTests:

Updated the tests not to assume spelling correction suggestions were listed in a context menu since
only Chromium port did this.

editing/spelling/resources/util.js:

(initSpellTest): Disable automatic text replacements.

editing/spelling/spelling-with-underscore-selection-expected.txt: Removed.
editing/spelling/spelling-with-underscore-selection.html: Removed.

This test erroneously assumed a specific word breaking behavior that doesn't include the underscore
when selecting a word especially since Chromium was the only port that was running these tests.

editing/spelling/spelling-double-clicked-word-with-underscores-expected.txt: Removed.
editing/spelling/spelling-double-clicked-word-with-underscores.html: Removed.

Ditto.

editing/spelling/spelling-should-select-single-word-expected.txt: Removed.
editing/spelling/spelling-should-select-single-word.html: Removed.

Removed since this test doesn't do anything useful without the assertion for the context menu.

editing/spelling/spelling-double-clicked-word-expected.txt:
editing/spelling/spelling-double-clicked-word.html:
editing/spelling/spelling-exactly-selected-word-expected.txt:
editing/spelling/spelling-exactly-selected-word.html:
editing/spelling/spelling-with-punctuation-selection-expected.txt:
editing/spelling/spelling-with-punctuation-selection.html:
editing/spelling/spelling-with-whitespace-selection-expected.txt:
editing/spelling/spelling-with-whitespace-selection.html:

platform/efl/TestExpectations:
platform/gtk/TestExpectations:
platform/mac/TestExpectations:
platform/qt/TestExpectations:
platform/win/TestExpectations:
platform/wincairo/TestExpectations:
platform/wk2/TestExpectations:

9:33 AM Changeset in webkit [148281] by commit-queue@webkit.org

4 edits in trunk/Source/WebKit2

[GTK][WK2] Add document-loaded signal to WebKitWebPage
https://bugs.webkit.org/show_bug.cgi?id=110614

Patch by Manuel Rego Casasnovas <Manuel Rego Casasnovas> on 2013-04-12
Reviewed by Carlos Garcia Campos.

Add a new signal document-loaded to WebKitWebPage that will be emitted
when the DOM document has been loaded for the main frame.

UIProcess/API/gtk/tests/TestWebExtensions.cpp:

(documentLoadedCallback):
(testDocumentLoadedSignal):
(beforeAll): Add test for document-loaded signal.

UIProcess/API/gtk/tests/WebExtensionTest.cpp:

(documentLoadedCallback):
(pageCreatedCallback):
(methodCallCallback): Add new D-Bus signal DocumentLoaded in order to
test document-loaded signal.

WebProcess/InjectedBundle/API/gtk/WebKitWebPage.cpp:

(webkit_web_page_class_init):
(webkitWebPageCreate): Add document-loaded signal and emit it when
didFinishDocumentLoadForFrame is called.

9:19 AM Changeset in webkit [148280] by oliver@apple.com

2 edits in trunk/Source/WTF

[Qt][Win] r148257 broke the build
https://bugs.webkit.org/show_bug.cgi?id=114512

Build fix.

wtf/Platform.h:

9:07 AM Changeset in webkit [148279] by commit-queue@webkit.org

2 edits in trunk/Source/WebKit2

[GTK] Webkit fails to build webkit2gtk-tests-resources.gresource.
https://bugs.webkit.org/show_bug.cgi?id=114485

Patch by Hanyee Kim <choco@company100.net> on 2013-04-12
Reviewed by Martin Robinson.

Generating webkit2gtk-tests-resources.gresource can be failed due to
the non-existing target directory.
We need to make target directory before generating it.

UIProcess/API/gtk/tests/GNUmakefile.am:

8:13 AM Changeset in webkit [148278] by akling@apple.com

8 edits in trunk/Source/WebKit2

REGRESSION(r145869): Right-click on SWF contents displays the context menu at the wrong place.
<http://webkit.org/b/113836>
<rdar://problem/13587624>

Reviewed by Anders "Wesley Crusher" Carlsson.

Mac plugins expect flipped-Y screen coordinates, while DOM APIs speak in unflipped coordinates,
and we were mixing them up in the window frame caching optimization.

Solve this by having the UIProcess send both flipped and unflipped window frames to the WebProcess.
The flipped frame is passed on to plugins, and the unflipped frame is used for window.screenX/Y etc.

UIProcess/API/mac/WKView.mm:

(-[WKView _updateWindowAndViewFrames]):

No need to retrieve the window frame here as WebPageProxy::windowAndViewFramesChanged() will
always override it anyway.

UIProcess/mac/WebPageProxyMac.mm:

(WebKit::WebPageProxy::windowAndViewFramesChanged):

WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::windowRect):

WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::windowAndViewFramesChanged):

WebProcess/WebPage/WebPage.h:

(WebKit::WebPage::windowFrameInUnflippedScreenCoordinates):

WebProcess/WebPage/WebPage.messages.in:

7:05 AM Changeset in webkit [148277] by commit-queue@webkit.org

2 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Crash running layout tests
https://bugs.webkit.org/show_bug.cgi?id=114503

Patch by Carlos Garcia Campos <cgarcia@igalia.com> on 2013-04-12
Reviewed by Xan Lopez.

Hit an assertion in ~GuardedPointerBase when DRT is
deleted. DumpRenderTree class derives from
BlackBerry::Platform::LayoutTestClient that derives from
BlackBerry::Platform::GuardedPointerBase, so the DRT pointer
should no be released with delete but with
BlackBerry::Platform::deleteGuardedObject().

Api/WebPage.cpp:

(BlackBerry::WebKit::WebPagePrivate::~WebPagePrivate):

6:50 AM Changeset in webkit [148276] by commit-queue@webkit.org

5 edits in trunk/Tools

[GTK] Enable WebKit2 building on GTK EWSs
https://bugs.webkit.org/show_bug.cgi?id=114500

Patch by Manuel Rego Casasnovas <Manuel Rego Casasnovas> on 2013-04-12
Reviewed by Philippe Normand.

Add configuration in order to be able to setup GTK EWSs building
WebKit2. A new queue gtk-wk2-ews is added.

QueueStatusServer/config/queues.py:
Scripts/webkitpy/common/config/ports.py:

(DeprecatedPort.port):
(GtkWK2Port):
(GtkWK2Port.build_webkit_command):
(GtkWK2Port.run_webkit_tests_command):

Scripts/webkitpy/common/config/ports_unittest.py:

(DeprecatedPortTest.test_gtk_port):
(DeprecatedPortTest):
(DeprecatedPortTest.test_gtk_wk2_port):

Scripts/webkitpy/tool/commands/earlywarningsystem.py:

(GtkWK2EWS):

6:22 AM Changeset in webkit [148275] by commit-queue@webkit.org

3 edits in trunk/Source/WebCore

[BlackBerry] Upstream the filesystem code
https://bugs.webkit.org/show_bug.cgi?id=114437

Patch by Alberto Garcia <agarcia@igalia.com> on 2013-04-12
Reviewed by Rob Buis.

These are the changes for AsyncFileSystemBlackBerry.

This patch contains contributions from many members of the
BlackBerry WebKit team.

platform/blackberry/AsyncFileSystemBlackBerry.cpp:

(WebCore::AsyncFileSystem::isAvailable):
(WebCore):
(WebCore::AsyncFileSystem::create):
(WebCore::AsyncFileSystem::openFileSystem):
(WebCore::AsyncFileSystemBlackBerry::openFileSystem):
(WebCore::AsyncFileSystem::deleteFileSystem):
(WebCore::AsyncFileSystemBlackBerry::AsyncFileSystemBlackBerry):
(WebCore::AsyncFileSystemBlackBerry::~AsyncFileSystemBlackBerry):
(WebCore::AsyncFileSystemBlackBerry::move):
(WebCore::AsyncFileSystemBlackBerry::copy):
(WebCore::AsyncFileSystemBlackBerry::remove):
(WebCore::AsyncFileSystemBlackBerry::removeRecursively):
(WebCore::AsyncFileSystemBlackBerry::readMetadata):
(WebCore::AsyncFileSystemBlackBerry::createFile):
(WebCore::AsyncFileSystemBlackBerry::createDirectory):
(WebCore::AsyncFileSystemBlackBerry::fileExists):
(WebCore::AsyncFileSystemBlackBerry::directoryExists):
(WebCore::AsyncFileSystemBlackBerry::readDirectory):
(WebCore::AsyncFileSystemBlackBerry::createWriter):
(WebCore::AsyncFileSystemBlackBerry::createSnapshotFileAndReadMetadata):
(WebCore::AsyncFileSystemBlackBerry::fileSystemURLToPath):

platform/blackberry/AsyncFileSystemBlackBerry.h:

(WTF):
(WTF::BlackBerry::Platform::WebFileSystem):
(WebCore::AsyncFileSystemBlackBerry::create):
(AsyncFileSystemBlackBerry):

6:10 AM Changeset in webkit [148274] by mikhail.pozdnyakov@intel.com

11 edits
4 copies in trunk/Source/WebKit2

[WK2][EFL] WebView: Add callbacks to the WKViewClient to handle page viewport update
https://bugs.webkit.org/show_bug.cgi?id=110463

Reviewed by Anders Carlsson.

UIProcess/efl/ViewClientEfl.h:

(ViewClientEfl):

UIProcess/efl/WebView.cpp:

(WebKit::WebView::didChangeViewportProperties):

UIProcess/efl/WebViewClient.cpp:

(WebKit::WebViewClient::didChangeViewportAttributes):
(WebKit):

UIProcess/efl/WebViewClient.h:

Added didChangeViewportAttributes callback to the WKViewClient to
handle page viewport updates.

CMakeLists.txt:
Shared/API/c/WKBase.h:
Shared/APIObject.h:
UIProcess/API/C/WKAPICast.h:

(WebKit):

UIProcess/API/C/WKViewportAttributes.cpp: Added.

(WKViewportAttributesGetTypeID):

UIProcess/API/C/WKViewportAttributes.h: Added.
UIProcess/API/C/efl/WKView.h:
UIProcess/WebViewportAttributes.cpp: Added.

(WebKit):
(WebKit::WebViewportAttributes::WebViewportAttributes):
(WebKit::WebViewportAttributes::~WebViewportAttributes):

UIProcess/WebViewportAttributes.h: Added.

(WebKit):
(WebViewportAttributes):
(WebKit::WebViewportAttributes::create):
(WebKit::WebViewportAttributes::originalAttributes):
(WebKit::WebViewportAttributes::type):

Added WKViewportAttributesRef - WK2 C API object that is wrapping
WebCore::ViewportAttributes structure and is passed in didChangeViewportAttributes
WKViewClient callback.

UIProcess/efl/ViewClientEfl.cpp:

(WebKit::ViewClientEfl::didChangeViewportAttributes):
(WebKit):

Added implementaion of didChangeViewportAttributes WKViewClient
callback so that existing EFL WK2 funtionality is kept.

3:54 AM Changeset in webkit [148273] by jocelyn.turcotte@digia.com

2 edits in trunk/Source/WebCore

Apply our MSVC 2012 optimizer bug workaround to all minor releases
https://bugs.webkit.org/show_bug.cgi?id=114436

Reviewed by Alexey Proskuryakov.

The bug has only been partially fixed in Update 2 but our Update 1 workaround
still avoids the crash.

platform/text/TextEncodingRegistry.cpp:

(WebCore::TextEncodingNameHash::equal):

3:28 AM Changeset in webkit [148272] by mrowe@apple.com

2 edits in trunk/Source/WebCore

And again, with the correct case this time.

platform/graphics/avfoundation/objc/WebCoreAVFResourceLoader.h:

3:22 AM Changeset in webkit [148271] by mrowe@apple.com

2 edits in trunk/Source/WebCore

Build fix.

platform/graphics/avfoundation/objc/WebCoreAVFResourceLoader.h:

Add missing #include and forward declaration.

2:03 AM Changeset in webkit [148270] by commit-queue@webkit.org

3 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Crash running layout tests when WebPage is destroyed
https://bugs.webkit.org/show_bug.cgi?id=114497

Patch by Carlos Garcia Campos <cgarcia@igalia.com> on 2013-04-12
Reviewed by Xan Lopez.

PR 324649
Internally reviewed by Konrad Piascik and Rob Buis.

Use a bool member to keep track of whether the inspector is
enabled or not instead of relaying on the enable developer extras
setting, that can be set in other places than enableWebInspector().

Api/WebPage.cpp:

(BlackBerry::WebKit::WebPagePrivate::WebPagePrivate): Initialize
m_inspectorEnabled.
(BlackBerry::WebKit::WebPage::enableWebInspector): Return early if
inspector is already enabled and set m_inspectorEnabled to true.
(BlackBerry::WebKit::WebPage::disableWebInspector): Return early
if inspector is not enabled and set m_inspectorEnabled to false.
(BlackBerry::WebKit::WebPage::isWebInspectorEnabled): Check the
current value of m_inspectorEnabled instead of the developer
extras setting.

Api/WebPage_p.h:

(WebPagePrivate): Add m_inspectorEnabled.

1:55 AM Changeset in webkit [148269] by zandobersek@gmail.com

2 edits in trunk/Source/WebCore

Unreviewed functionality fix for ports enabling the MicroData feature.

html/DOMSettableTokenList.idl: Add the SkipVTableValidation IDL attribute to the DOMSettableTokenList interface.

This removes crashes in the MicroData layout tests that are occurring on EFL and GTK builders, the two ports being
the only ports that enable this feature.

1:17 AM Changeset in webkit [148268] by zarvai@inf.u-szeged.hu

3 edits in trunk/LayoutTests

[Qt] Unreviewed gardening.

platform/qt/TestExpectations:
platform/qt/fast/js/global-constructors-expected.txt: Rebaselining after r148236.

1:12 AM Changeset in webkit [148267] by Carlos Garcia Campos

2 edits in trunk/Source/WebKit2

[GTK] The style of visited links doesn't change in WebKit2
https://bugs.webkit.org/show_bug.cgi?id=112175

Reviewed by Benjamin Poulain.

The problem is that visited links were not tracked by the web
process. There's a web process initial parameter to set whether
web process should track visited links or not and it's disabled by
default.

UIProcess/gtk/WebContextGtk.cpp:

(WebKit::WebContext::platformInitializeWebProcess): Always set
shouldTrackVisitedLinks to true.

12:55 AM WebKitGTK/2.0.x edited by Manuel Rego Casasnovas

(diff)

12:38 AM Changeset in webkit [148266] by rniwa@webkit.org

1 edit
2 deletes in trunk/Tools

Remove Chromium scripts
https://bugs.webkit.org/show_bug.cgi?id=114492

Reviewed by Benjamin Poulain.

Scripts/run-chromium-webkit-unit-tests: Removed.
Scripts/update-webkit-chromium: Removed.

12:15 AM Changeset in webkit [148265] by ap@apple.com

15 edits
1 add in trunk/Source/WebCore

Move CachedRawResourceClient into a separate file
https://bugs.webkit.org/show_bug.cgi?id=114491

Reviewed by Mark Rowe.

GNUmakefile.list.am:
Target.pri:
WebCore.vcproj/WebCore.vcproj:
WebCore.vcxproj/WebCore.vcxproj:
WebCore.vcxproj/WebCore.vcxproj.filters:
WebCore.xcodeproj/project.pbxproj:
loader/DocumentLoader.cpp:
loader/DocumentLoader.h:
loader/DocumentThreadableLoader.h:
loader/archive/cf/LegacyWebArchive.cpp:
loader/cache/CachedRawResource.cpp:
loader/cache/CachedRawResource.h:
loader/cache/CachedRawResourceClient.h: Added.
loader/icon/IconLoader.h:
platform/graphics/avfoundation/objc/WebCoreAVFResourceLoader.h:

12:14 AM Changeset in webkit [148264] by zandobersek@gmail.com

18 edits in trunk

Unreviewed, rolling out r148262.
http://trac.webkit.org/changeset/148262
https://bugs.webkit.org/show_bug.cgi?id=114493

Cairo dep should now build, rolling r148247 back in (Requested
by zdobersek on #webkit).

Patch by Commit Queue <rniwa@webkit.org> on 2013-04-12

Source/autotools/FindDependencies.m4:
Source/autotools/PrintBuildConfiguration.m4:
Source/autotools/SetupWebKitFeatures.m4:

Source/WebCore:

platform/graphics/GraphicsContext.cpp:

(WebCore):

platform/graphics/ImageBuffer.cpp:

(WebCore):

platform/graphics/cairo/GLContext.h:

(GLContext):

platform/graphics/cairo/GraphicsContextCairo.cpp:

(WebCore::GraphicsContext::isAcceleratedContext):
(WebCore):

platform/graphics/cairo/ImageBufferCairo.cpp:

(WebCore::ImageBufferData::ImageBufferData):
(WebCore):
(WebCore::createCairoGLSurface):
(WebCore::ImageBuffer::ImageBuffer):
(WebCore::ImageBuffer::platformTransformColorSpace):
(WebCore::mapSurfaceToImage):
(WebCore::unmapSurfaceFromImage):
(WebCore::getImageData):
(WebCore::ImageBuffer::putByteArray):
(WebCore::ImageBufferData::paintToTextureMapper):
(WebCore::ImageBuffer::platformLayer):

platform/graphics/cairo/ImageBufferDataCairo.h:

(ImageBufferData):

platform/graphics/egl/GLContextEGL.cpp:

(WebCore::GLContextEGL::GLContextEGL):
(WebCore::GLContextEGL::~GLContextEGL):
(WebCore::GLContextEGL::cairoDevice):
(WebCore):

platform/graphics/egl/GLContextEGL.h:
platform/graphics/glx/GLContextGLX.cpp:

(WebCore::GLContextGLX::GLContextGLX):
(WebCore::GLContextGLX::~GLContextGLX):
(WebCore::GLContextGLX::cairoDevice):
(WebCore):

platform/graphics/glx/GLContextGLX.h:

(GLContextGLX):

Tools:

Scripts/webkitdirs.pm:

(buildAutotoolsProject):

gtk/jhbuild.modules:

Apr 11, 2013:

11:11 PM Changeset in webkit [148263] by ap@apple.com

10 edits
1 add in trunk/Source/WebCore

Reduce includes of CachedSVGDocument.h
https://bugs.webkit.org/show_bug.cgi?id=114489

Reviewed by Jon Honeycutt.

loader/cache/CachedSVGDocumentReference.cpp: Added.
loader/cache/CachedSVGDocumentReference.h: Moved function implementations to newly added .cpp file, and removed an include of CachedSVGDocument.h.

css/WebKitCSSSVGDocumentValue.h: Just removed an unnecessary include.

CMakeLists.txt:
GNUmakefile.list.am:
Target.pri:
WebCore.vcproj/WebCore.vcproj:
WebCore.vcxproj/WebCore.vcxproj:
WebCore.vcxproj/WebCore.vcxproj.filters:
WebCore.xcodeproj/project.pbxproj:

11:04 PM Changeset in webkit [148262] by zandobersek@gmail.com

18 edits in trunk

Unreviewed, rolling out r148247.
http://trac.webkit.org/changeset/148247
https://bugs.webkit.org/show_bug.cgi?id=114490

Cairo dep fails to build on builders due to missing EGL
headers (Requested by zdobersek on #webkit).

Patch by Commit Queue <rniwa@webkit.org> on 2013-04-11

Source/autotools/FindDependencies.m4:
Source/autotools/PrintBuildConfiguration.m4:
Source/autotools/SetupWebKitFeatures.m4:

Source/WebCore:

platform/graphics/GraphicsContext.cpp:

(WebCore):

platform/graphics/ImageBuffer.cpp:

(WebCore):

platform/graphics/cairo/GLContext.h:

(GLContext):

platform/graphics/cairo/GraphicsContextCairo.cpp:
platform/graphics/cairo/ImageBufferCairo.cpp:

(WebCore::ImageBufferData::ImageBufferData):
(WebCore::ImageBuffer::ImageBuffer):
(WebCore::ImageBuffer::platformTransformColorSpace):
(WebCore::getImageData):
(WebCore::ImageBuffer::putByteArray):
(WebCore):

platform/graphics/cairo/ImageBufferDataCairo.h:

(ImageBufferData):

platform/graphics/egl/GLContextEGL.cpp:

(WebCore::GLContextEGL::GLContextEGL):
(WebCore::GLContextEGL::~GLContextEGL):

platform/graphics/egl/GLContextEGL.h:
platform/graphics/glx/GLContextGLX.cpp:

(WebCore::GLContextGLX::GLContextGLX):
(WebCore::GLContextGLX::~GLContextGLX):

platform/graphics/glx/GLContextGLX.h:

(GLContextGLX):

Tools:

Scripts/webkitdirs.pm:

(buildAutotoolsProject):

gtk/jhbuild.modules:

10:03 PM Changeset in webkit [148261] by thakis@chromium.org

2 edits in trunk/Source/WebCore

Remove debugging code that is no longer used with current mac sdks.
https://bugs.webkit.org/show_bug.cgi?id=114486

Reviewed by Dan Bernstein.

This code compared MAC_OS_X_VERSION_MAX_ALLOWED and friends to
107 and 106 instead of the correct 1070 and 1060. Instead of fixing
this, rip the code out completely: Since WebKit now only targets
current systems, the ifdef is always false anyways.

No tests, as this only removes debugging code.

platform/graphics/mac/SimpleFontDataMac.mm:

(WebCore::SimpleFontData::platformInit):

9:21 PM Changeset in webkit [148260] by oliver@apple.com

14 edits in trunk/Source/WebCore

Fix windows build and land new binding test results

8:42 PM Changeset in webkit [148259] by benjamin@webkit.org

5 edits in trunk/Source/JavaScriptCore

Merge CharacterClassTable into CharacterClass
https://bugs.webkit.org/show_bug.cgi?id=114409

Patch by Benjamin Poulain <bpoulain@apple.com> on 2013-04-11
Reviewed by Darin Adler.

CharacterClassTable is only a pointer and a boolean.
It is a little overkill to make a separate allocation
for that.

create_regex_tables:
yarr/YarrJIT.cpp:

(JSC::Yarr::YarrGenerator::matchCharacterClass):

yarr/YarrPattern.cpp:

(JSC::Yarr::CharacterClassConstructor::charClass):

yarr/YarrPattern.h:

(CharacterClass):
(JSC::Yarr::CharacterClass::CharacterClass):

7:38 PM Changeset in webkit [148258] by commit-queue@webkit.org

3 edits
12 adds in trunk

Selection gaps don't repaint correctly with transforms
https://bugs.webkit.org/show_bug.cgi?id=111000

Patch by Shezan Baig <Shezan Baig> on 2013-04-11
Reviewed by Simon Fraser.

Source/WebCore:

When computing offsetFromRepaintContainer, we need to add UseTransforms
to the MapCoordinateFlags argument. Note that ApplyContainerFlip is
the default value for the flag, so this change effectively only adds
the UseTransforms flag.

Tests: fast/repaint/selection-gap-absolute-child.html

fast/repaint/selection-gap-fixed-child.html
fast/repaint/selection-gap-flipped-absolute-child.html
fast/repaint/selection-gap-flipped-fixed-child.html
fast/repaint/selection-gap-transformed-absolute-child.html
fast/repaint/selection-gap-transformed-fixed-child.html

rendering/RenderBlock.cpp:

(WebCore::RenderBlock::selectionGapRectsForRepaint):

LayoutTests:

fast/repaint/selection-gap-absolute-child-expected.txt: Added.
fast/repaint/selection-gap-absolute-child.html: Added.
fast/repaint/selection-gap-fixed-child-expected.txt: Added.
fast/repaint/selection-gap-fixed-child.html: Added.
fast/repaint/selection-gap-flipped-absolute-child-expected.txt: Added.
fast/repaint/selection-gap-flipped-absolute-child.html: Added.
fast/repaint/selection-gap-flipped-fixed-child-expected.txt: Added.
fast/repaint/selection-gap-flipped-fixed-child.html: Added.
fast/repaint/selection-gap-transformed-absolute-child-expected.txt: Added.
fast/repaint/selection-gap-transformed-absolute-child.html: Added.
fast/repaint/selection-gap-transformed-fixed-child-expected.txt: Added.
fast/repaint/selection-gap-transformed-fixed-child.html: Added.

6:43 PM Changeset in webkit [148257] by oliver@apple.com

21 edits in trunk/Source

Add more type validation to debug builds
https://bugs.webkit.org/show_bug.cgi?id=114478

Reviewed by Mark Hahnenberg.

Source/WebCore:

Add a bunch more type checks to the JS DOM bindings.

Modules/mediastream/MediaStream.idl:
Modules/webaudio/AudioDestinationNode.idl:
WebCore.xcodeproj/project.pbxproj:
bindings/js/JSDOMBinding.h:

(WebCore::getExistingWrapper):
(WebCore):
(WebCore::createNewWrapper):

bindings/scripts/CodeGeneratorJS.pm:

(GetNativeTypeForConversions):
(GetGnuVTableRefForInterface):
(GetGnuVTableNameForInterface):
(GetGnuMangledNameForInterface):
(GetGnuVTableOffsetForType):
(GetWinVTableRefForInterface):
(GetWinVTableNameForInterface):
(GetWinMangledNameForInterface):
(GetNamespaceForInterface):
(GetImplementationLacksVTableForInterface):
(GetSkipVTableValidationForInterface):
(GenerateImplementation):

bindings/scripts/IDLAttributes.txt:
css/CSSRuleList.idl:
css/CSSStyleDeclaration.idl:
dom/Clipboard.idl:
dom/DOMStringMap.idl:
dom/MutationRecord.idl:
dom/NodeList.idl:
html/DOMTokenList.idl:
html/track/TextTrack.idl:
inspector/ScriptProfileNode.idl:
storage/Storage.idl:
xml/XPathNSResolver.idl:

Source/WTF:

Add BINDING_VALIDATION flag and make RELEASE_ASSERT use UNLIKELY.

wtf/Assertions.h:
wtf/Platform.h:

5:50 PM Changeset in webkit [148256] by rniwa@webkit.org

12 edits
1 delete in trunk/Source/WebCore

Remove ResourceLoadInfo
https://bugs.webkit.org/show_bug.cgi?id=113302

Reviewed by Benjamin Poulain.

Co-authored by Christophe Dumez.

Removed ResourceLoadInfo since it had been only used by Chromium port.

GNUmakefile.list.am:
Target.pri:
WebCore.order:
WebCore.vcproj/WebCore.vcproj:
WebCore.vcxproj/WebCore.vcxproj:
WebCore.vcxproj/WebCore.vcxproj.filters:
WebCore.xcodeproj/project.pbxproj:
inspector/InspectorResourceAgent.cpp:

(WebCore::buildObjectForResourceResponse):

loader/FrameLoader.cpp:

(WebCore::FrameLoader::loadResourceSynchronously):

platform/network/ResourceLoadInfo.h: Removed.
platform/network/ResourceResponseBase.cpp:

(WebCore):
(WebCore::ResourceResponseBase::reportMemoryUsage):

platform/network/ResourceResponseBase.h:

(ResourceResponseBase):

5:50 PM Changeset in webkit [148255] by rniwa@webkit.org

2 edits in trunk/LayoutTests

Remove passing test expectations for ietestcenter/css3/grid/grid-items-003.htm since it started failing on
Mountain Lion WebKit2 bots after a slave reboot as well.

platform/mac-wk2/TestExpectations:

5:47 PM Changeset in webkit [148254] by Lucas Forschler

4 edits in trunk/Source

Submission Notes.

5:39 PM Changeset in webkit [148253] by timothy_horton@apple.com

2 edits in trunk/Source/WebKit2

REGRESSION (r146956): ASSERTion failure: WebKit::WebProcess::pageWillLeaveWindow
https://bugs.webkit.org/show_bug.cgi?id=114481
<rdar://problem/13534784>

Reviewed by Simon Fraser.

Inform the WebProcess of WebPages which are already in a window when created.

WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::WebPage):

5:20 PM Changeset in webkit [148252] by rniwa@webkit.org

2 edits in trunk/Source/WebCore

Another Mac Lion build fix attempt after r148197.

page/scrolling/mac/ScrollingStateScrollingNodeMac.mm:

5:08 PM Changeset in webkit [148251] by commit-queue@webkit.org

3 edits
2 adds in trunk

Unable to paste twice in input field when specific CSS present
https://bugs.webkit.org/show_bug.cgi?id=108675

Patch by Sukolsak Sakshuwong <Sukolsak Sakshuwong> on 2013-04-11
Reviewed by Ryosuke Niwa.

Source/WebCore:

When we are pre-rendering text in ReplaceSelectionCommand, we shouldn't care
whether the content is clipped by its ancestors or not. This bug was caused
by the fact the plainText() method returned an empty string because the text
was fully clipped by its ancestors. Make plainText ignore style visibility.

Test: editing/inserting/insert-text-into-text-field.html

editing/ReplaceSelectionCommand.cpp:

(WebCore::ReplacementFragment::ReplacementFragment):

LayoutTests:

editing/inserting/insert-text-into-text-field-expected.txt: Added.
editing/inserting/insert-text-into-text-field.html: Added.

4:55 PM Changeset in webkit [148250] by andersca@apple.com

8 edits in trunk/Source/WebKit2

Add support for clearing storage areas
https://bugs.webkit.org/show_bug.cgi?id=114479

Reviewed by Beth Dakin.

UIProcess/Storage/StorageManager.cpp:

(WebKit::StorageManager::StorageArea::StorageArea):
Store the quota size so we can recreate the underlying StorageMap when clearing.

(WebKit::StorageManager::StorageArea::clear):
Create a new storage map and dispatch events.

(WebKit::StorageManager::clear):
Find the right storage area and call clear.

UIProcess/Storage/StorageManager.messages.in:

Add Clear message.

WebProcess/Storage/StorageAreaImpl.cpp:

(WebKit::StorageAreaImpl::clear):
Call the storage map.

WebProcess/Storage/StorageAreaMap.cpp:

(WebKit::StorageAreaMap::clear):
Reset the cached values and send a clear message.

(WebKit::StorageAreaMap::resetValues):
New helper function.

(WebKit::StorageAreaMap::didClear):
New stub.

WebProcess/Storage/StorageAreaMap.messages.in:

Add DidClear message.

4:51 PM Changeset in webkit [148249] by Beth Dakin

4 edits in trunk/Source/WebKit2

WebKit should set the header and footer layers' contentsScale when the device
scale factor changes
https://bugs.webkit.org/show_bug.cgi?id=114471
-and corresponding-
<rdar://problem/13621288>

Reviewed by Simon Fraser.

Update the header and footer contentsScale when the device scale factor has
changed.

WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::setDeviceScaleFactor):

WebProcess/WebPage/WebPage.h:
WebProcess/WebPage/mac/WebPageMac.mm:

(WebKit::WebPage::updateHeaderAndFooterLayersForDeviceScaleChange):

4:46 PM Changeset in webkit [148248] by commit-queue@webkit.org

2 edits in trunk

[GTK] Webkit fails to build with MinGW compiler after changeset 146468
https://bugs.webkit.org/show_bug.cgi?id=114473

Patch by Paweł Forysiuk <tuxator@o2.pl> on 2013-04-11
Reviewed by Martin Robinson.

Source/autotools/CheckSystemAndBasicDependencies.m4: Fix typo.

4:39 PM Changeset in webkit [148247] by Martin Robinson

18 edits in trunk

[GTK] Add accelerated 2D canvas support using cairo-gl
https://bugs.webkit.org/show_bug.cgi?id=104672

Reviewed by Alejandro G. Castro.

Detect that we can activate accelerated canvas when CairoGL is present and
TextureMapperGL is enabled.

Source/autotools/FindDependencies.m4: Look for CairoGL.
Source/autotools/PrintBuildConfiguration.m4: Print the status of accelerated canvas activation.
Source/autotools/SetupWebKitFeatures.m4: Set the feature.

Source/WebCore:

No new tests. We do not yet have the ability to run tests against
accelerated content.

platform/graphics/GraphicsContext.cpp:

(WebCore): We don't use the stub implementation of isAcceleratedContext any longer.

platform/graphics/ImageBuffer.cpp:

(WebCore): Ditto for ImageBuffer::platformLayer.

platform/graphics/cairo/GLContext.h:

(GLContext): Added method to get a cairo_device_t from the context.

platform/graphics/cairo/GraphicsContextCairo.cpp:

(WebCore::GraphicsContext::isAcceleratedContext): Return true when we are a CairoGL surface.

platform/graphics/cairo/ImageBufferCairo.cpp:

(WebCore::ImageBufferData::ImageBufferData): Initialize the size and the texture to 0.
(WebCore::createCairoGLSurface): Added this helper.
(WebCore::ImageBuffer::ImageBuffer): When we are in accelerated rendering mode, create
a CairoGL surface.
(WebCore::ImageBuffer::platformTransformColorSpace): Do not implement this for accelerated
contexts yet.
(WebCore::mapSurfaceToImage): Added this helper, since we don't require Cairo 1.12 yet,
which provides an builtin implementation.
(WebCore::unmapSurfaceFromImage): Ditto.
(WebCore::getImageData): Map the surface to an image surface first.
(WebCore::ImageBuffer::putByteArray): Ditto.
(WebCore::ImageBufferData::paintToTextureMapper): Connect the accelerated canvas into the
TextureMapper infrastructure.
(WebCore::ImageBuffer::platformLayer): This lets the TextureMapper at the ImageBufferData.

platform/graphics/cairo/ImageBufferDataCairo.h:

(ImageBufferData): Subclass TexturMapperPlatformLayer where appropriate.

platform/graphics/egl/GLContextEGL.cpp: Added implementation of cairoDevice.
platform/graphics/egl/GLContextEGL.h: Ditto.
platform/graphics/glx/GLContextGLX.cpp: Ditto
platform/graphics/glx/GLContextGLX.h: Ditto.

Tools:

Since the value of ACCELERATED_2D_CANVAS relies on some somewhat unusual
dependencies, don't override value of the setting determined by configure.
This will make it easier for people using build-webkit, but not the
WebKit JHBuild.

Scripts/webkitdirs.pm:

(buildAutotoolsProject): Add a blacklist for settings to never override.

gtk/jhbuild.modules: Build CairoGL.

4:31 PM Changeset in webkit [148246] by rniwa@webkit.org

2 edits in trunk/Tools

build.webkit.org should link to webkit-queues instead of webkit-commit-queue
https://bugs.webkit.org/show_bug.cgi?id=114477

Reviewed by Benjamin Poulain.

BuildSlaveSupport/build.webkit.org-config/templates/root.html:

4:26 PM Changeset in webkit [148245] by andersca@apple.com

10 edits in trunk/Source

Implement removing storage area items
https://bugs.webkit.org/show_bug.cgi?id=114472

Reviewed by Beth Dakin.

Source/WebCore:

Export StorageMap::removeItem.

WebCore.exp.in:

Source/WebKit2:

UIProcess/Storage/StorageManager.cpp:

(StorageManager::StorageArea):
(WebKit::StorageManager::StorageArea::setItem):
Rename connection to sourceConnection.

(WebKit::StorageManager::StorageArea::removeItem):
Remove the item from the map and dispatch events if needed.

(WebKit::StorageManager::removeItem):
Find the right storage area, remove the item and send back a DidRemoveItem message.

UIProcess/Storage/StorageManager.messages.in:

Add RemoveItem message.

WebProcess/Storage/StorageAreaImpl.cpp:

(WebKit::StorageAreaImpl::removeItem):
Call StorageAreaMap::removeItem.

WebProcess/Storage/StorageAreaMap.cpp:

(WebKit::StorageAreaMap::removeItem):
Send a RemoveItem message to the storage manager.

(WebKit::StorageAreaMap::didRemoveItem):
Add empty stub.

WebProcess/Storage/StorageAreaMap.messages.in:

Add DidRemoveItem message.

4:25 PM Changeset in webkit [148244] by commit-queue@webkit.org

5 edits
1 add in trunk

TextTrackList's .onremovetrack is missing from IDL
https://bugs.webkit.org/show_bug.cgi?id=103421

Patch by Brendan Long <b.long@cablelabs.com> on 2013-04-11
Reviewed by Eric Carlson.

Source/WebCore:

Fixed test media/track/opera/interfaces/TextTrackList/onremovetrack.html

html/track/TextTrackList.cpp:

(TextTrackList::remove):
(TextTrackList::scheduleRemoveTrackEvent):

html/track/TextTrackList.h:

(TextTrackList):

html/track/TextTrackList.idl:

LayoutTests:

media/track/opera/interfaces/TextTrackList/onremovetrack-expected.txt: Added.

4:24 PM WebKit Team edited by silviapf@chromium.org

added my irc handles and new employer (diff)

4:20 PM Changeset in webkit [148243] by rniwa@webkit.org

2 edits in trunk/Tools

Leaks builder should not be triggered upon new commits
https://bugs.webkit.org/show_bug.cgi?id=114475

Reviewed by Jessie Berlin.

Don't trigger them on new checkins wait for Mountaion Lion (Build), which already triggers
the leak builder when builds are done.

BuildSlaveSupport/build.webkit.org-config/config.json:

4:18 PM WebKitGTK/2.0.x edited by Martin Robinson

(diff)

4:09 PM Changeset in webkit [148242] by commit-queue@webkit.org

1 edit
18 adds in trunk/Source/WebCore

[BlackBerry] Upstream the filesystem code
https://bugs.webkit.org/show_bug.cgi?id=114437

Patch by Alberto Garcia <agarcia@igalia.com> on 2013-04-11
Reviewed by Rob Buis.

This patch contains contributions from many members of the
BlackBerry WebKit team.

platform/blackberry/AsyncFileWriterBlackBerry.cpp: Added.

(WebCore):
(WebCore::AsyncFileWriterBlackBerry::write):

platform/blackberry/AsyncFileWriterBlackBerry.h: Added.

(WTF):
(WTF::BlackBerry::Platform::WebFileWriter):
(WebCore):

platform/blackberry/DOMFileSystemBlackBerry.cpp: Added.

(WebCore):
(WebCore::DOMFileSystemBase::createFileSystemURL):
(WebCore::DOMFileSystemBase::crackFileSystemURL):
(WebCore::DOMFileSystemBase::supportsToURL):
(WebCore::DOMFileSystemBase::isValidType):

platform/blackberry/LocalFileSystemBlackBerry.cpp: Added.

(WebCore):
(WebCore::fileSystemTypeString):
(WebCore::openFileSystem):
(WebCore::LocalFileSystem::deleteFileSystem):
(WebCore::LocalFileSystem::readFileSystem):
(WebCore::LocalFileSystem::requestFileSystem):

platform/blackberry/PlatformAsyncFileSystemCallbacks.cpp: Added.

(WebCore):
(WebCore::PlatformAsyncFileSystemCallbacks::notifyOpenFileSystem):
(WebCore::PlatformAsyncFileSystemCallbacks::notifySucceed):
(WebCore::PlatformAsyncFileSystemCallbacks::notifyFail):
(WebCore::getFileMetadata):
(WebCore::PlatformAsyncFileSystemCallbacks::notifyReadMetadata):
(WebCore::PlatformAsyncFileSystemCallbacks::notifyCreateSnapshotFileAndReadMetadata):
(WebCore::PlatformAsyncFileSystemCallbacks::notifyReadDirectory):
(WebCore::PlatformAsyncFileSystemCallbacks::notifyCreateFileWriter):
(WebCore::PlatformAsyncFileSystemCallbacks::createAsyncFileSystem):
(WebCore::PlatformAsyncFileSystemCallbacks::createAsyncFileWriter):
(WebCore::PlatformAsyncFileSystemCallbacks::deleteMe):

platform/blackberry/PlatformAsyncFileSystemCallbacks.h: Added.

(WebCore):

platform/blackberry/PlatformBlob.cpp: Added.

(WebCore):
(WebCore::PlatformBlob::nextDataItem):

platform/blackberry/PlatformBlob.h: Added.

(WebCore):

platform/blackberry/PlatformFileWriterClient.cpp: Added.

(WebCore):
(WebCore::PlatformFileWriterClient::notifyWrite):
(WebCore::PlatformFileWriterClient::notifyTruncate):
(WebCore::PlatformFileWriterClient::notifyFail):

platform/blackberry/PlatformFileWriterClient.h: Added.

(WebCore):
(WTF):
(WTF::WebCore::PlatformFileWriterClient):

platform/blackberry/WorkerAsyncFileSystemBlackBerry.cpp: Added.

(WebCore):
(WebCore::WorkerAsyncFileSystemBlackBerry::WorkerAsyncFileSystemBlackBerry):
(WebCore::WorkerAsyncFileSystemBlackBerry::~WorkerAsyncFileSystemBlackBerry):
(WebCore::WorkerAsyncFileSystemBlackBerry::waitForOperationToComplete):
(WebCore::WorkerAsyncFileSystemBlackBerry::openFileSystemOnMainThread):
(WebCore::WorkerAsyncFileSystemBlackBerry::deleteFileSystemOnMainThread):
(WebCore::WorkerAsyncFileSystemBlackBerry::moveOnMainThread):
(WebCore::WorkerAsyncFileSystemBlackBerry::copyOnMainThread):
(WebCore::WorkerAsyncFileSystemBlackBerry::removeOnMainThread):
(WebCore::WorkerAsyncFileSystemBlackBerry::removeRecursivelyOnMainThread):
(WebCore::WorkerAsyncFileSystemBlackBerry::readMetadataOnMainThread):
(WebCore::WorkerAsyncFileSystemBlackBerry::createFileOnMainThread):
(WebCore::WorkerAsyncFileSystemBlackBerry::createDirectoryOnMainThread):
(WebCore::WorkerAsyncFileSystemBlackBerry::fileExistsOnMainThread):
(WebCore::WorkerAsyncFileSystemBlackBerry::directoryExistsOnMainThread):
(WebCore::WorkerAsyncFileSystemBlackBerry::readDirectoryOnMainThread):
(WebCore::WorkerAsyncFileSystemBlackBerry::createWriterOnMainThread):
(WebCore::WorkerAsyncFileSystemBlackBerry::createSnapshotFileAndReadMetadataOnMainThread):
(WebCore::WorkerAsyncFileSystemBlackBerry::openFileSystem):
(WebCore::WorkerAsyncFileSystemBlackBerry::deleteFileSystem):
(WebCore::WorkerAsyncFileSystemBlackBerry::move):
(WebCore::WorkerAsyncFileSystemBlackBerry::copy):
(WebCore::WorkerAsyncFileSystemBlackBerry::remove):
(WebCore::WorkerAsyncFileSystemBlackBerry::removeRecursively):
(WebCore::WorkerAsyncFileSystemBlackBerry::readMetadata):
(WebCore::WorkerAsyncFileSystemBlackBerry::createFile):
(WebCore::WorkerAsyncFileSystemBlackBerry::createDirectory):
(WebCore::WorkerAsyncFileSystemBlackBerry::fileExists):
(WebCore::WorkerAsyncFileSystemBlackBerry::directoryExists):
(WebCore::WorkerAsyncFileSystemBlackBerry::readDirectory):
(WebCore::WorkerAsyncFileSystemBlackBerry::createWriter):
(WebCore::WorkerAsyncFileSystemBlackBerry::createSnapshotFileAndReadMetadata):

platform/blackberry/WorkerAsyncFileSystemBlackBerry.h: Added.

(WebCore):
(WorkerAsyncFileSystemBlackBerry):
(WebCore::WorkerAsyncFileSystemBlackBerry::create):

platform/blackberry/WorkerAsyncFileWriterBlackBerry.cpp: Added.

(WebCore):
(WebCore::WorkerAsyncFileWriterBlackBerry::writeOnMainThread):
(WebCore::WorkerAsyncFileWriterBlackBerry::truncateOnMainThread):
(WebCore::WorkerAsyncFileWriterBlackBerry::abortOnMainThread):
(WebCore::WorkerAsyncFileWriterBlackBerry::write):
(WebCore::WorkerAsyncFileWriterBlackBerry::truncate):
(WebCore::WorkerAsyncFileWriterBlackBerry::abort):
(WebCore::WorkerAsyncFileWriterBlackBerry::waitForOperationToComplete):
(WebCore::WorkerAsyncFileWriterBlackBerry::platformWriterClient):

platform/blackberry/WorkerAsyncFileWriterBlackBerry.h: Added.

(WebCore):

platform/blackberry/WorkerPlatformAsyncFileSystemCallbacks.cpp: Added.

(WebCore):
(WebCore::performTaskOnMainThread):
(WebCore::postTaskToMainThread):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::notifyStop):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::notifyOpenFileSystem):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::notifySucceed):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::notifyFail):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::notifyReadMetadata):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::notifyReadDirectory):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::notifyCreateFileWriter):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::createAsyncFileSystem):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::createAsyncFileWriter):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::deleteMe):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::postTaskToWorkerThread):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::notifyOpenFileSystemOnWorkerThread):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::notifySucceedOnWorkerThread):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::notifyFailOnWorkerThread):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::notifyReadMetadataOnWorkerThread):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::notifyReadDirectoryEntryOnWorkerThread):
(WebCore::WorkerPlatformAsyncFileSystemCallbacks::notifyCreateFileWriterOnWorkerThread):

platform/blackberry/WorkerPlatformAsyncFileSystemCallbacks.h: Added.

(WebCore):

platform/blackberry/WorkerPlatformFileWriterClient.cpp: Added.

(WebCore):
(WebCore::WorkerPlatformFileWriterClient::notifyWriteOnWorkerThread):
(WebCore::WorkerPlatformFileWriterClient::notifyFailOnWorkerThread):
(WebCore::WorkerPlatformFileWriterClient::notifyTruncateOnWorkerThread):
(WebCore::WorkerPlatformFileWriterClient::notifyWrite):
(WebCore::WorkerPlatformFileWriterClient::notifyTruncate):
(WebCore::WorkerPlatformFileWriterClient::notifyFail):
(WebCore::WorkerPlatformFileWriterClient::postTaskToWorkerThreadIfNeeded):

platform/blackberry/WorkerPlatformFileWriterClient.h: Added.

(WebCore):
(WTF):
(WTF::WebCore::WorkerPlatformFileWriterClient):

4:04 PM Changeset in webkit [148241] by roger_fong@apple.com

3 edits in trunk/WebKitLibraries

Build fix.

win/include/WebKitSystemInterface/WebKitSystemInterface.h:
win/lib/WebKitSystemInterface.lib:

3:56 PM Changeset in webkit [148240] by ap@apple.com

4 edits in trunk/WebKitLibraries

<rdar://problem/10416316> [Mac] WebSocket doesn't work with authenticating proxies
https://bugs.webkit.org/show_bug.cgi?id=114464

Update WebKitSystemInterface.

WebKitSystemInterface.h:
libWebKitSystemInterfaceLion.a:
libWebKitSystemInterfaceMountainLion.a:

3:17 PM Changeset in webkit [148239] by rniwa@webkit.org

2 edits in trunk/Source/WebCore

Mac Lion build fix attempt after r148197.

page/scrolling/mac/ScrollingStateNodeMac.mm:

2:50 PM Changeset in webkit [148238] by timothy_horton@apple.com

2 edits in trunk/Source/WebKit2

InjectedBundleNodeHandle::imageForRect doesn't respect device scale factor or highlighting option
https://bugs.webkit.org/show_bug.cgi?id=114466
<rdar://problem/13508513>

Reviewed by Simon Fraser.

Respect the device scale factor when creating the snapshot image.
Clear the snapshot image before drawing into it.
Respect SnapshotOptionsExcludeSelectionHighlighting.

WebProcess/InjectedBundle/DOM/InjectedBundleNodeHandle.cpp:

(WebKit::imageForRect):

2:42 PM Changeset in webkit [148237] by ap@apple.com

4 edits in trunk/Source/WebCore

<rdar://problem/10416316> [Mac] WebSocket doesn't work with authenticating proxies
https://bugs.webkit.org/show_bug.cgi?id=114464

Reviewed by Brady Eidson.

platform/mac/WebCoreSystemInterface.h:
platform/mac/WebCoreSystemInterface.mm: Updated for new wkCopyCONNECTProxyResponse signature.

platform/network/cf/SocketStreamHandleCFNet.cpp: (WebCore::SocketStreamHandle::executePACFileURL): Fixed a typo in a comment. (WebCore::SocketStreamHandle::addCONNECTCredentials): Don't crash even if the rest of the fix didn't work (which would be the case on some OS versions). (WebCore::SocketStreamHandle::readStreamCallback): Replaced most breaks with returns, because breaks were confusing in such a huge switch. Changed null proxyResponse to not be treated as authentication success, because it's not. Merged two parts of WaitingForConnect state handling for clarity. (WebCore::SocketStreamHandle::writeStreamCallback): Don't blindly assume that we can start WebSocket handshake after kCFStreamEventCanAcceptBytes. Perhaps it's nothing but a failed CONNECT, and a read callback still needs to send authentication. Without this, establishing connections was flaky. Added a check for Closed state, matching read callback.

2:41 PM Changeset in webkit [148236] by commit-queue@webkit.org

3 edits
2 adds in trunk

MutationRecord is not exposed
https://bugs.webkit.org/show_bug.cgi?id=114288

Patch by Sukolsak Sakshuwong <Sukolsak Sakshuwong> on 2013-04-11
Reviewed by Darin Adler.

Source/WebCore:

Expose MutationRecord on DOMWindow. This patch is copied from a patch
by Adam Klein. https://codereview.chromium.org/13861028

Test: fast/dom/MutationObserver/mutation-record-constructor.html

page/DOMWindow.idl:

LayoutTests:

This test is copied from a patch by Adam Klein.
https://codereview.chromium.org/13861028
The test expectation of "typeof MutationRecord" is changed from
"function" to "object". (See bug 114457)

fast/dom/MutationObserver/mutation-record-constructor-expected.txt: Added.
fast/dom/MutationObserver/mutation-record-constructor.html: Added.

2:28 PM Changeset in webkit [148235] by Lucas Forschler

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

Merged r136554. <rdar://problem/13334986>

2:01 PM Changeset in webkit [148234] by Beth Dakin

2 edits in trunk/Source/WebCore

Cannot scroll to footer on a page with overflow:hidden on the body
https://bugs.webkit.org/show_bug.cgi?id=114462
-and corresponding-
<rdar://problem/13530042>

Reviewed by Simon Fraser.

Add a new scenario in which we want to overrideHidden -- whenever we are the
mainFrame and there is a header or footer.

page/FrameView.cpp:

(WebCore::FrameView::applyOverflowToViewport):

2:01 PM WebKitGTK/2.0.x edited by kalevlember@gmail.com

Added r148230 as proposed merge (diff)

1:57 PM WebKitGTK/2.0.x edited by kalevlember@gmail.com

Added r147963 as proposed merge (diff)

1:53 PM Changeset in webkit [148233] by rniwa@webkit.org

12 edits in trunk/Source

Unreviewed, rolling out r148034, r148052, r148097, and
r148194.
http://trac.webkit.org/changeset/148034
http://trac.webkit.org/changeset/148052
http://trac.webkit.org/changeset/148097
http://trac.webkit.org/changeset/148194
https://bugs.webkit.org/show_bug.cgi?id=114463

broke mutiresolution favicons, among other things (Requested
by thorton on #webkit).

Source/WebCore:

loader/icon/IconDatabase.cpp:

(WebCore::IconDatabase::setIconDataForIconURL):

loader/icon/IconDatabase.h:

(IconDatabase):

loader/icon/IconDatabaseBase.h:
loader/icon/IconRecord.cpp:
loader/icon/IconRecord.h:

(IconRecord):

Source/WebKit2:

UIProcess/WebIconDatabase.cpp:

(WebKit::WebIconDatabase::setIconDataForIconURL):

UIProcess/WebIconDatabase.h:

(WebIconDatabase):

UIProcess/WebIconDatabase.messages.in:
WebProcess/IconDatabase/WebIconDatabaseProxy.cpp:

(WebKit::WebIconDatabaseProxy::setIconDataForIconURL):

WebProcess/Plugins/PDF/PDFPlugin.mm:

(WebKit::PDFPlugin::destroy):

1:51 PM Changeset in webkit [148232] by commit-queue@webkit.org

3 edits in trunk/Source/WebCore

[GTK][AC] Implement GraphicsLayerClutter::moveOrCopyAnimations
https://bugs.webkit.org/show_bug.cgi?id=114019

Patch by ChangSeok Oh <ChangSeok Oh> on 2013-04-11
Reviewed by Gustavo Noronha Silva.

Fill moveOrCopyAnimations that is based on Mac port implementation.

Tests: animations/3d/change-transform-in-end-event.html

animations/3d/transform-origin-vs-functions.html
compositing/animation/state-at-end-event-transform-layer.html

platform/graphics/clutter/GraphicsLayerClutter.cpp:

(WebCore::GraphicsLayerClutter::moveOrCopyLayerAnimation):
(WebCore):
(WebCore::GraphicsLayerClutter::moveOrCopyAnimations):

platform/graphics/clutter/GraphicsLayerClutter.h:

1:34 PM Changeset in webkit [148231] by andersca@apple.com

4 edits in trunk/Source

Implement StorageManager::getValues
https://bugs.webkit.org/show_bug.cgi?id=114461

Reviewed by Beth Dakin.

Source/WebCore:

Expose the item hash map.

storage/StorageMap.h:

(WebCore::StorageMap::items):

Source/WebKit2:

UIProcess/Storage/StorageManager.cpp:

(WebKit::StorageManager::StorageArea::items):
(WebKit::StorageManager::getValues):
Return the items from the right storage area.

1:22 PM Changeset in webkit [148230] by commit-queue@webkit.org

2 edits in trunk/Tools

[GTK] Windows build failure after r136056
https://bugs.webkit.org/show_bug.cgi?id=114121

Patch by Kalev Lember <kalevlember@gmail.com> on 2013-04-11
Reviewed by Martin Robinson.

TestWebKitAPI/config.h: Make sure defines are correct for other

Windows ports than apple-win.

1:14 PM Changeset in webkit [148229] by andersca@apple.com

3 edits in trunk/Source/WebKit2

Clean up related storage areas when a connection is destroyed
https://bugs.webkit.org/show_bug.cgi?id=114459

Reviewed by Beth Dakin.

UIProcess/Storage/StorageManager.cpp:

(WebKit::StorageManager::processWillCloseConnection):
Dispatch invalidateConnectionInternal on the work queue.

(WebKit::StorageManager::createStorageMap):
(WebKit::StorageManager::destroyStorageMap):
Rename m_storageAreas to m_storageAreasByConnection.

(WebKit::StorageManager::invalidateConnectionInternal):
Go over all known storage areas and remove this connection as a listener.

(WebKit::StorageManager::findStorageArea):
Rename m_storageAreas to m_storageAreasByConnection.

UIProcess/Storage/StorageManager.h:

Rename m_storageAreas to m_storageAreasByConnection.

1:08 PM Changeset in webkit [148228] by commit-queue@webkit.org

2 edits in trunk/Source/WebCore

[BlackBerry] Use the platform list of languages with fonts instead of hardcoding
https://bugs.webkit.org/show_bug.cgi?id=114452

Patch by Eli Fidler <efidler@blackberry.com> on 2013-04-11
Reviewed by Rob Buis.

BlackBerry PR 325244
Internally reviewed by Jeff Rogers

No functional change, but platform font changes won't need a WebKit change
going forward.

page/blackberry/SettingsBlackBerry.cpp:

(WebCore::Settings::initializeDefaultFontFamilies):

1:05 PM Changeset in webkit [148227] by krit@webkit.org

8 edits
6 adds in trunk

[CSS Shaders] Parse parameters descriptor
https://bugs.webkit.org/show_bug.cgi?id=114455

Reviewed by Dean Jackson.

Source/WebCore:

Add support for the 'parameters' descriptor in the filter at-rule to support
parameterization of CSS Shaders.

https://dvcs.w3.org/hg/FXTF/raw-file/tip/filters/index.html#custom-filter-parameters

Tests: css3/filters/custom-with-at-rule-syntax/parsing-parameters-property-invalid.html

css3/filters/custom-with-at-rule-syntax/parsing-parameters-property-valid.html

css/CSSComputedStyleDeclaration.cpp:

(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):

css/CSSParser.cpp: Added 'parameters' property and fail parsing if not inside

of an @filter rule.

(WebCore::CSSParser::parseValue):
(WebCore::CSSParser::parseFilterRuleParameters): New helper function to verify

values.

(WebCore):

css/CSSParser.h:
css/CSSProperty.cpp:

(WebCore::CSSProperty::isInheritedProperty):

css/CSSPropertyNames.in: Added new property name 'parameters'.
css/StyleResolver.cpp:

(WebCore::StyleResolver::applyProperty):

LayoutTests:

Tests for the 'parameters' descriptor on @filter rule.

css3/filters/custom-with-at-rule-syntax/parsing-parameters-property-invalid-expected.txt: Added.
css3/filters/custom-with-at-rule-syntax/parsing-parameters-property-invalid.html: Added.
css3/filters/custom-with-at-rule-syntax/parsing-parameters-property-valid-expected.txt: Added.
css3/filters/custom-with-at-rule-syntax/parsing-parameters-property-valid.html: Added.
css3/filters/custom-with-at-rule-syntax/script-tests/parsing-parameters-property-invalid.js: Added.

(testInvalidParametersProperty):

css3/filters/custom-with-at-rule-syntax/script-tests/parsing-parameters-property-valid.js: Added.

(testParametersProperty):

1:01 PM Changeset in webkit [148226] by fpizlo@apple.com

2 edits in trunk/PerformanceTests/SunSpider

Get rid of the 10ms delay between test executions
https://bugs.webkit.org/show_bug.cgi?id=114458
<rdar://problem/13631112>

Reviewed by Darin Adler and Geoffrey Garen.

More details on this in the bug. Long story short, I don't know why the 10ms delay is
there, and it appears to be the root of some noisyness and unreliability in the results.
Removing it makes results more consistent, which is a good thing.

resources/driver-TEMPLATE.html:

12:56 PM Changeset in webkit [148225] by zhajiang@rim.com

2 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Initial scale changes depending on the order of page navigations
https://bugs.webkit.org/show_bug.cgi?id=114448

Patch by Jacky Jiang <zhajiang@rim.com>.
Reviewed by Rob Buis.

PR: 323941
During page transitions, absoluteVisibleOverflowSize() and contentsSize()
in fixedLayoutSize() were based on the old page when we just started or
committed provisinal load. In that case, fixedLayoutSize() is very old
page dependent and can cause inconsistent behaviors. For example, if
the old page contents width is 4000, we will get DEFAULT_MAX_LAYOUT_WIDTH 1024
as the new page's fixedLayoutSize; however, we will get 768 if we
navigate to the new page directly.
To fix that, return default layout size if the page doesn't have
virtual viewport and m_loadState is Provisional.

Api/WebPage.cpp:

(BlackBerry::WebKit::WebPagePrivate::fixedLayoutSize):

12:37 PM Changeset in webkit [148224] by commit-queue@webkit.org

2 edits in trunk/Source/WebCore

Default Implementation of toString for NPObject shouldn't return NPClass & NPObject address as String
https://bugs.webkit.org/show_bug.cgi?id=114450

Patch by Arunprasad Rajkumar <arunprasadr@nds.com> on 2013-04-11
Reviewed by Anders Carlsson.

bridge/c/c_instance.cpp:

(JSC::Bindings::CInstance::stringValue):

12:17 PM Changeset in webkit [148223] by robert@webkit.org

3 edits
2 adds in trunk

REGRESSION (142152): ensure we skip past out-of-flow objects when detecting whitespace to ignore after leading empty inlines
https://bugs.webkit.org/show_bug.cgi?id=114311

Reviewed by Emil A Eklund.

Source/WebCore:

When we hit an empty inline in line layout and try to detect whether it has any whitespace after it that we ought to ignore, then
we need to skip past out-of-flow and floating objects when looking for that whitespace. Failure to do so will result in us
adding the width from the first space we encounter to our line width and over-estimating the real width of the line.

We're hitting this bug now because we have broadened the category of empty inlines that get lineboxes so we go into line layout
and have to deal with them and their subsequent whitespace there. Previously this sort of whitespace would have been simply
consumed by |skipLeadingWhitespace|.

Test: fast/inline/out-of-flow-objects-and-whitespace-after-empty-inline.html

rendering/RenderBlockLineLayout.cpp:

(WebCore::shouldSkipWhitespaceAfterStartObject):

LayoutTests:

fast/inline/out-of-flow-objects-and-whitespace-after-empty-inline-expected.txt: Added.
fast/inline/out-of-flow-objects-and-whitespace-after-empty-inline.html: Added.

12:14 PM Changeset in webkit [148222] by ojan@chromium.org

2 edits in trunk/Tools

Remove myself from watchlists.

Scripts/webkitpy/common/config/watchlist:

11:56 AM Changeset in webkit [148221] by rniwa@webkit.org

18 edits in trunk

Autocorrected text doesn't have a marker of type autocorrected
https://bugs.webkit.org/show_bug.cgi?id=114337

Reviewed by Darin Adler.

Source/WebCore:

The bug was caused by markAndReplaceFor's text replacement errornously assuming that there are
no other text checking result starting at the same offset. That resulted in us adjusting location
offsets of the subsequent results pointing at the replaced text, hit an assertion, and blow up.

Fixed the bug by skipping all remaining results for the replaced text since they're no longer
applicable.

Tests: Existing autocorection tests have been updated to assert new behavior.

editing/Editor.cpp:

(WebCore::Editor::markAndReplaceFor):

LayoutTests:

Updated tests to assert the existence of "autocorrected" markers.

platform/mac/editing/spelling/click-autocorrected-word-expected.png:
platform/mac/editing/spelling/click-autocorrected-word-expected.txt:
platform/mac/editing/spelling/click-autocorrected-word.html:
platform/mac/editing/spelling/delete-into-autocorrected-word-expected.png:
platform/mac/editing/spelling/delete-into-autocorrected-word-expected.txt:
platform/mac/editing/spelling/delete-into-autocorrected-word.html:
platform/mac/editing/spelling/forward-delete-into-autocorrected-word-expected.png:
platform/mac/editing/spelling/forward-delete-into-autocorrected-word-expected.txt:
platform/mac/editing/spelling/forward-delete-into-autocorrected-word.html:
platform/mac/editing/spelling/move-cursor-to-autocorrected-word-expected.png:
platform/mac/editing/spelling/move-cursor-to-autocorrected-word-expected.txt:
platform/mac/editing/spelling/move-cursor-to-autocorrected-word.html:
platform/mac/editing/spelling/move-cursor-to-beginning-of-autocorrected-word-expected.png:
platform/mac/editing/spelling/move-cursor-to-beginning-of-autocorrected-word-expected.txt:
platform/mac/editing/spelling/move-cursor-to-beginning-of-autocorrected-word.html:

11:54 AM Changeset in webkit [148220] by Michelangelo De Simone

8 edits
6 adds in trunk

[CSS Shaders] Parse the geometry descriptor
https://bugs.webkit.org/show_bug.cgi?id=110815

Source/WebCore:

Added initial parsing for the "geometry" descriptor, as per specification:
https://dvcs.w3.org/hg/FXTF/raw-file/tip/filters/index.html#geometry
This descriptor allows to specify the mesh geometry for custom filters.

Reviewed by Dirk Schulze.

Tests: css3/filters/custom-with-at-rule-syntax/parsing-geometry-property-invalid.html

css3/filters/custom-with-at-rule-syntax/parsing-geometry-property-valid.html

css/CSSComputedStyleDeclaration.cpp:

(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):

css/CSSParser.cpp:

(WebCore::CSSParser::parseValue):
(WebCore::CSSParser::parseGeometry): New method that parses the geometry
descriptor (grid()).
(WebCore):

css/CSSParser.h:
css/CSSProperty.cpp:

(WebCore::CSSProperty::isInheritedProperty):

css/CSSPropertyNames.in: Added conditional "geometry" property.
css/CSSValueKeywords.in: Added conditional "attached" value keyword; detached

was already there.

LayoutTests:

Positive and negative parsing tests for the "geometry" descriptor as
per specification:
https://dvcs.w3.org/hg/FXTF/raw-file/tip/filters/index.html#geometry

Reviewed by Dirk Schulze.

css3/filters/custom-with-at-rule-syntax/parsing-geometry-property-invalid-expected.txt: Added.
css3/filters/custom-with-at-rule-syntax/parsing-geometry-property-invalid.html: Added.
css3/filters/custom-with-at-rule-syntax/parsing-geometry-property-valid-expected.txt: Added.
css3/filters/custom-with-at-rule-syntax/parsing-geometry-property-valid.html: Added.
css3/filters/custom-with-at-rule-syntax/script-tests/parsing-geometry-property-invalid.js: Added.

(testInvalidGeometryProperty):

css3/filters/custom-with-at-rule-syntax/script-tests/parsing-geometry-property-valid.js: Added.

(testGeometryProperty):

11:43 AM Changeset in webkit [148219] by andersca@apple.com

6 edits in trunk/Source

Make StorageAreaMap dispatch session storage events
https://bugs.webkit.org/show_bug.cgi?id=114454

Reviewed by Beth Dakin.

Source/WebCore:

Export storage symbols and headers needed by WebKit2.

WebCore.exp.in:
WebCore.xcodeproj/project.pbxproj:

Source/WebKit2:

WebProcess/Storage/StorageAreaMap.cpp:

(WebKit::StorageAreaMap::dispatchStorageEvent):
Depending on the storage area type, call dispatchSessionStorageEvent or dispatchLocalStorageEvent.

(WebKit::StorageAreaMap::dispatchSessionStorageEvent):
Enumerate all the frames in our page and dispatch the event to the relevant ones.

(WebKit::StorageAreaMap::dispatchLocalStorageEvent):
Add empty stub for now.

11:39 AM Changeset in webkit [148218] by zandobersek@gmail.com

2 edits in trunk

Unreviewed.

Source/autotools/symbols.filter: Stop exporting redundant symbols.

11:27 AM Changeset in webkit [148217] by zandobersek@gmail.com

2 edits in trunk/LayoutTests

Unreviewed GTK gardening.

platform/gtk/TestExpectations: Removing expectations for a bunch of tests that are passing.

11:19 AM Changeset in webkit [148216] by rniwa@webkit.org

2 edits in trunk/Tools

Build fix. Don't hard-code queues.webkit.org at least during the transition.

QueueStatusServer/templates/statusbubble.html:

10:44 AM Changeset in webkit [148215] by andersca@apple.com

9 edits in trunk/Source/WebKit2

Implement more logic in StorageAreaMap
https://bugs.webkit.org/show_bug.cgi?id=114451

Reviewed by Alexey Proskuryakov.

Platform/CoreIPC/HandleMessage.h:

(CoreIPC::callMemberFunction):
Add new overload.

UIProcess/Storage/StorageManager.cpp:

(WebKit::StorageManager::StorageArea::addListener):
(WebKit::StorageManager::StorageArea::removeListener):
These take storage map IDs.

(WebKit::StorageManager::StorageArea::setItem):
Rename storageAreaID to sourceStorageAreaID.

(WebKit::StorageManager::StorageArea::dispatchEvents):
Remove the code that would not dispatch to the source connection, there may still be storage areas interested in those events.

(WebKit::StorageManager::setItem):
This now takes the source storage area ID.

(WebKit::StorageManager::findStorageArea):
This takes a storage map ID.

UIProcess/Storage/StorageManager.messages.in:

SetItem now takes the source storage area ID as well.

WebProcess/Storage/StorageAreaImpl.cpp:

(WebKit::StorageAreaImpl::setItem):
Pass the source frame to StorageAreaMap::setItem.

WebProcess/Storage/StorageAreaMap.cpp:

(WebKit::StorageAreaMap::StorageAreaMap):
Send CreateStorageMap and add the object as a message receiver.

(WebKit::StorageAreaMap::~StorageAreaMap):
Send DestroyStorageMap and remove the object as a message receiver.

(WebKit::StorageAreaMap::length):
(WebKit::StorageAreaMap::key):
(WebKit::StorageAreaMap::item):
(WebKit::StorageAreaMap::contains):
Load values and forward calls to the storage map.

(WebKit::StorageAreaMap::setItem):
This now takes the source frame as well. Update the local storage map and send a SetItem message.

(WebKit::StorageAreaMap::loadValuesIfNeeded):
Get the values from the storage manager.

(WebKit::StorageAreaMap::dispatchStorageEvent):
This now takes the source storage area ID.

WebProcess/Storage/StorageAreaMap.messages.in:

DispatchStorageEvent now takes the source storage area ID.

10:18 AM Changeset in webkit [148214] by beidson@apple.com

5 edits in trunk

Web/Plugin process deadlock initializing async plugins.
<rdar://problem/13525232> and https://bugs.webkit.org/show_bug.cgi?id=114217

Rubberstamped by Anders Carlsson.

Rollout r147953 and r147968 as we identified other causes for this.

Source/WebKit2:

PluginProcess/WebProcessConnection.cpp:

(WebKit::WebProcessConnection::createPluginAsynchronously):

WebProcess/Plugins/PluginProxy.messages.in:

LayoutTests:

platform/mac-wk2/TestExpectations:

10:02 AM Changeset in webkit [148213] by commit-queue@webkit.org

2 edits in trunk/LayoutTests

[WK2] Run toString & valueOf LayoutTests for WebKit2 also
https://bugs.webkit.org/show_bug.cgi?id=114433

Patch by Arunprasad Rajkumar <arunprasadr@nds.com> on 2013-04-11
Reviewed by Anders Carlsson.

platform/wk2/TestExpectations:

9:55 AM Changeset in webkit [148212] by rniwa@webkit.org

2 edits in trunk/Tools

Use webkit-queues.appspot.com instead of webkit-commit-queue.appspot.com
https://bugs.webkit.org/show_bug.cgi?id=114397

Reviewed by Benjamin Poulain.

Reland the patch since Alan appears to have finished the migration.

Scripts/webkitpy/common/net/statusserver.py:

(StatusServer):

9:52 AM Changeset in webkit [148211] by allan.jensen@digia.com

2 edits in trunk/Source/WebCore

handleMouseMoveEvent should let hit-testing hit-test scrollbars
https://bugs.webkit.org/show_bug.cgi?id=114430

Reviewed by Antonio Gomes.

Allow hit-testing to hit-test frame scrollbars so handleMouseMoveEvent
does not have to do it itself.

page/EventHandler.cpp:

(WebCore::EventHandler::handleMouseMoveEvent):

9:38 AM Changeset in webkit [148210] by commit-queue@webkit.org

7 edits
2 adds in trunk

[Gtk] Combo boxes should be arrow-out-of-able similar to list boxes when caret-browsing is enabled.
https://bugs.webkit.org/show_bug.cgi?id=76796

Patch by Brian Holt <brian.holt@samsung.com> on 2013-04-11
Reviewed by Chris Fleizach.

Source/WebCore:

When using caret-browsing with the right and left arrows, the
behaviour of menuLists should be the same as listBoxes. This means
not default-handling the event and passing it up the stack instead.

html/HTMLSelectElement.cpp:

(WebCore::HTMLSelectElement::menuListDefaultEventHandler):

Tools:

Make the Gtk window for the DRT a toplevel so that it gets the
focus from the window manager when the test starts.

DumpRenderTree/gtk/DumpRenderTree.cpp:

(main):

LayoutTests:

Adding a new test to verify that it is possible to arrow out of
combo boxes when caret browsing is enabled. Updated failing tests
to reflect changed accessibility events in the DRT.

platform/gtk/accessibility/aria-slider-required-attributes-expected.txt: Updated.
platform/gtk/accessibility/caret-browsing-select-focus-expected.txt: Added.
platform/gtk/accessibility/caret-browsing-select-focus.html: Added.
platform/gtk/accessibility/combo-box-collapsed-selection-changed-expected.txt: Updated.

9:32 AM Changeset in webkit [148209] by ap@apple.com

17 edits in trunk/Source

Remove some ResourceHandle.h includes
https://bugs.webkit.org/show_bug.cgi?id=114416

Reviewed by Ryosuke Niwa.

Source/WebCore:

loader/CrossOriginAccessControl.h:
loader/CrossOriginPreflightResultCache.h:

These files don't need ResourceHandle.h, just ResourceHandleTypes.h.

loader/TextTrackLoader.cpp:
loader/ThreadableLoader.h:
loader/appcache/ApplicationCacheGroup.h:
loader/cache/MemoryCache.h:
loader/mac/ResourceLoaderMac.mm:
page/Settings.cpp:
platform/network/AuthenticationChallengeBase.cpp:
platform/network/cf/CookieJarCFNet.cpp:
platform/network/cf/ResourceRequestCFNet.cpp:
xml/XSLTProcessorLibxslt.cpp:
xml/parser/XMLDocumentParser.cpp:
xml/parser/XMLDocumentParserLibxml2.cpp:

Just straightforward removal of includes.

Source/WebKit2:

WebProcess/Network/NetworkProcessConnection.cpp: Added an include that is now

necessary.

9:19 AM Changeset in webkit [148208] by allan.jensen@digia.com

9 edits in trunk

Use CSS4 image-rendering to determing image scaling quality
https://bugs.webkit.org/show_bug.cgi?id=113405

Reviewed by Simon Fraser.

Source/WebCore:

Implement the new CSS4 image-rendering property values.

Image scaling now uses low-quality (pixelated) scaling for
crisp-edges, pixelated and optimizeSpeed values, and always
use high quality (smooth) scaling for -webkit-smooth and
optimizeQuality. The latter can be used to disable low quality
scaling during CSS animations or live resizing.

The new values are protected with a new CSS4_IMAGES feature, but
the crisp-edge value has been introduced before to displace the
deprecated -webkit-optimize-contrast value, and optimizeSpeed
and optimizeQuality values are supported for SVG compatibility.

css/CSSParser.cpp:

(WebCore::isValidKeywordPropertyAndValue):

css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
(WebCore::CSSPrimitiveValue::operator EImageRendering):

css/CSSValueKeywords.in:
rendering/RenderBoxModelObject.cpp:

(WebCore::ImageQualityController::shouldPaintAtLowQuality):

rendering/RenderHTMLCanvas.cpp:

(WebCore::RenderHTMLCanvas::paintReplaced):

rendering/style/RenderStyleConstants.h:

Tools:

Introduce new flag to enable experimental support for CSS4 Images.

Scripts/webkitperl/FeatureList.pm:

9:19 AM Changeset in webkit [148207] by msaboff@apple.com

2 edits in trunk/Source/JavaScriptCore

Added UNLIKELY() suggested in https://bugs.webkit.org/show_bug.cgi?id=114366
after checking in the original change.

Rubber-stamped by Jessie Berlin.

dfg/DFGOperations.cpp:

9:02 AM Changeset in webkit [148206] by commit-queue@webkit.org

2 edits in trunk/Source/WebCore

Build fix: use of uninitialized variable in ScrollingStateTree
https://bugs.webkit.org/show_bug.cgi?id=114443

Patch by Adenilson Cavalcanti <cavalcantii@gmail.com> on 2013-04-11
Reviewed by Anders Carlsson.

Initialize pointer to zero before use, fixes build.
No new tests, no changes on behavior.

page/scrolling/ScrollingStateTree.cpp:

(WebCore::ScrollingStateTree::attachNode):

8:55 AM Changeset in webkit [148205] by krit@webkit.org

8 edits
6 adds in trunk

[CSS Shaders] Parse mix descriptor
https://bugs.webkit.org/show_bug.cgi?id=114414

Reviewed by Antti Koivisto.

Source/WebCore:

Add support for the 'mix' descriptor in the filter at-rule to support blend modes
and composite operators in CSS Shaders.
Each pixel in the fragment shader will be blended and composited with the
backdrop by taking these values into account.

https://dvcs.w3.org/hg/FXTF/raw-file/tip/filters/index.html#custom-filter-mix

Tests: css3/filters/custom-with-at-rule-syntax/parsing-mix-property-invalid.html

css3/filters/custom-with-at-rule-syntax/parsing-mix-property-valid.html

css/CSSComputedStyleDeclaration.cpp:

(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):

css/CSSParser.cpp: Added 'mix' property and fail parsing if not inside

of an @filter rule.

(WebCore::CSSParser::parseValue):
(WebCore::CSSParser::parseFilterRuleMix): New helper function to verify

values.

(WebCore):

css/CSSParser.h:
css/CSSProperty.cpp:

(WebCore::CSSProperty::isInheritedProperty):

css/CSSPropertyNames.in: Added new property name 'mix'.
css/StyleResolver.cpp:

(WebCore::StyleResolver::applyProperty):

LayoutTests:

Tests for the 'mix' descriptor on @filter rule.

css3/filters/custom-with-at-rule-syntax/parsing-mix-property-invalid-expected.txt: Added.
css3/filters/custom-with-at-rule-syntax/parsing-mix-property-invalid.html: Added.
css3/filters/custom-with-at-rule-syntax/parsing-mix-property-valid-expected.txt: Added.
css3/filters/custom-with-at-rule-syntax/parsing-mix-property-valid.html: Added.
css3/filters/custom-with-at-rule-syntax/script-tests/parsing-mix-property-invalid.js: Added.

(testInvalidMixProperty):

css3/filters/custom-with-at-rule-syntax/script-tests/parsing-mix-property-valid.js: Added.

(testMixProperty):

8:09 AM Changeset in webkit [148204] by abecsi@webkit.org

2 edits in trunk/Source/ThirdParty/ANGLE

[Qt][Mac] Linking fails with Qt 5.1
https://bugs.webkit.org/show_bug.cgi?id=114435

Reviewed by Tor Arne Vestbø.

Qt 5.1 enables C++11 by default when using clang on Mac
and links against the libc++ standard library. As a result
building the QtWebKit shared library fails when linking
agains the ANGLE intermediate static library because libc++
misses some symbols needed by ANGLE.
Disable C++11 for the ANGLE intermediate library to fix the build.

ANGLE.pri:

7:51 AM Changeset in webkit [148203] by commit-queue@webkit.org

5 edits in trunk

Parent box with background-size auto and gradient image does not get properly repainted when child box is resized.
https://bugs.webkit.org/show_bug.cgi?id=114424

Source/WebCore:

Patch by Zalan Bujtas <Alan Bujtas> on 2013-04-11
Reviewed by Antti Koivisto.

Initiate full repaint on fill layer, when the image is generated and the background
property defines auto size.
http://www.w3.org/TR/css3-background/#background-size
'If the image has neither an intrinsic width nor an intrinsic height, its size is determined as for 'contain'.'

Extended fast/repaint/background-shorthand-with-gradient-and-height-changes.html
to cover this case too.

rendering/RenderObject.cpp:

(WebCore::mustRepaintFillLayers):

LayoutTests:

Patch by Zalan Bujtas <Alan Bujtas> on 2013-04-11
Reviewed by Antti Koivisto.

fast/repaint/background-shorthand-with-gradient-and-height-changes-expected.txt:
fast/repaint/background-shorthand-with-gradient-and-height-changes.html:

7:42 AM Changeset in webkit [148202] by commit-queue@webkit.org

2 edits in trunk/Source/WebCore

[GTK][AC] use a smart pointer for GList and ClutterCanvas
https://bugs.webkit.org/show_bug.cgi?id=114057

Patch by ChangSeok Oh <ChangSeok Oh> on 2013-04-11
Reviewed by Gustavo Noronha Silva.

We can make simple code & prevent possible memory leak by using a proper smart pointer.
Especially if we get children by clutter_actor_get_children, we should free it after using.

No functionality changed.

platform/graphics/clutter/GraphicsLayerActor.cpp:

(graphicsLayerActorUpdateTexture):
(graphicsLayerActorRemoveAll):

7:36 AM Changeset in webkit [148201] by anilsson@rim.com

3 edits in trunk/Source/WebCore

[BlackBerry][CSS Filters] Blur filter fails to recompute blur size when layer size changes
https://bugs.webkit.org/show_bug.cgi?id=114272

Reviewed by Rob Buis.

Blur appeared blocky or pixelated when surface changed size after
creating the filter actions. The initial image size was used to
determine the blur size (expressed in texture coordinate system).
Fixed by recomputing the blur size when the surface size changes, using
a new Uniform1f subclass that can use a functor to compute the uniform
value per-frame.

This fixes css3/filters/effect-blur-hw.html. This can only be verified
by manual inspection because the BlackBerry port is not currently using
pixel tests.

PR 323730

platform/graphics/blackberry/LayerFilterRenderer.cpp:

(WebCore):
(SurfaceFunctor):
(WebCore::SurfaceFunctor::SurfaceFunctor):
(InverseSurfaceWidth):
(WebCore::InverseSurfaceWidth::InverseSurfaceWidth):
(WebCore::InverseSurfaceWidth::operator()):
(InverseSurfaceHeight):
(WebCore::InverseSurfaceHeight::InverseSurfaceHeight):
(WebCore::LayerFilterRenderer::actionsForOperations):

platform/graphics/blackberry/LayerFilterRenderer.h:

(Uniform1f):
(WebCore):
(Uniform1fWithFunctor):
(WebCore::Uniform1fWithFunctor::create):
(WebCore::Uniform1fWithFunctor::Uniform1fWithFunctor):
(WebCore::Uniform1fWithFunctor::apply):
(WebCore::Uniform1f::createWithFunctor):

7:28 AM Changeset in webkit [148200] by anilsson@rim.com

4 edits in trunk/Source/WebCore

[BlackBerry] Accelerated compositing debug rectangle incorrectly drawn for layers with surfaces
https://bugs.webkit.org/show_bug.cgi?id=114275

Reviewed by Carlos Garcia Campos.

The drawDebugBorder method was updated to mirror the logic in
compositeLayersRecursive, i.e. if we're compositing a surface to the
screen, use the transformed bounds stored in the surface. The bounds
stored in the layer should only be used when drawing the layer to the
surface.

Speaking of which, also update the code to skip border drawing when
drawing a layer to a surface. Drawing the borders inside the surface
would only result in messing up the surface contents, especially when
CSS filters like blur were the reason for having a surface in the first
place - the border would be blurred and hard to discern.

Only manually testable, the debug border is disabled during layout
tests.

PR 323746

platform/graphics/blackberry/LayerRenderer.cpp:

(WebCore::LayerRenderer::drawDebugBorder):

platform/graphics/blackberry/LayerRendererSurface.cpp:

(WebCore::LayerRendererSurface::drawRect):
(WebCore::LayerRendererSurface::transformedBounds): Added.
(WebCore):

platform/graphics/blackberry/LayerRendererSurface.h:

(LayerRendererSurface):

7:27 AM Changeset in webkit [148199] by anilsson@rim.com

2 edits in trunk/Source/WebCore

[BlackBerry] Compiler warning in LayerWebKitThread.cpp
https://bugs.webkit.org/show_bug.cgi?id=114277

Reviewed by Carlos Garcia Campos.

All the relevant WTF::Vector APIs use size_t, storing the position in
an int resulted in a compiler warning. Fixed by switching to size_t.

No change in behavior, no new tests.

PR 323753

platform/graphics/blackberry/LayerWebKitThread.cpp:

(WebCore::LayerWebKitThread::remove):
(WebCore::LayerWebKitThread::replaceSublayer):

6:39 AM WebKitGTK/2.0.x edited by Carlos Garcia Campos

(diff)

6:36 AM Changeset in webkit [148198] by Carlos Garcia Campos

3 edits
8 adds in releases/WebKitGTK/webkit-2.0

Merge r148182 - FrameLoaderClient::assignIdentifierToInitialRequest() not called for the main resource when loaded from the memory cache
https://bugs.webkit.org/show_bug.cgi?id=112418

Source/WebCore:

Reviewed by Darin Adler.

When the main resource is loaded from the memory cache the
response is not added to the ResponseVector of the document
loader, so that when committing the provisional load the remaining
delegate messages are not called because the ResponseVector of the
document loader is empty. When the main resource is loaded from
the memory cache, there's no resource loader, and the client is
notified about the response received using
ResourceLoadNotifier::dispatchDidReceiveResponse() directly
instead of ResourceLoadNotifier::didReceiveResponse() which is the
one adding the response to the ResponseVector of the document
loader. So, the problem can be fixed by adding the response to the
ResponseVector before calling dispatchDidReceiveResponse() when
loading the main resource without a resource loade

Test: loader/go-back-cached-main-resource.html

loader/DocumentLoader.cpp:

(WebCore::DocumentLoader::responseReceived):

LayoutTests:

Patch by Manuel Rego Casasnovas <Manuel Rego Casasnovas> on 2013-04-11
Reviewed by Darin Adler.

loader/go-back-cached-main-resource-expected.txt: Added.
loader/go-back-cached-main-resource.html: Added.
loader/resources/first-page.html: Added.
loader/resources/other-page.html: Added.
platform/gtk-wk1/loader/go-back-cached-main-resource-expected.txt: Added.
platform/wk2/loader/go-back-cached-main-resource-expected.txt: Added.

6:16 AM Changeset in webkit [148197] by mikhail.pozdnyakov@intel.com

15 edits
2 adds in trunk/Source/WebCore

[CoordinatedGraphics] Use ScrollingStateTree to handle fixed elements positioning while scrolling
https://bugs.webkit.org/show_bug.cgi?id=114353

Reviewed by Noam Rosenthal.

ScrollingCoordinatorCoordinatedGraphics should implement threaded scrolling interface and
re-use threaded scrolling approach (ScrollingStateTree and ViewportConstraints) to handle
fixed/sticky elements positioning while scrolling. The rationals are below.

Before the change ScrollingCoordinatorCoordinatedGraphics was implementing Chromium-specific
interface which is going to be removed eventually as there is no ScrollingCoordinatorChromium
existing.

Tested by existing manual tests (fixed-position.html for example.

CMakeLists.txt:
Target.pri:

Included ScrollingStateTree files.

page/scrolling/ScrollingStateFixedNode.cpp:
page/scrolling/ScrollingStateFixedNode.h:
page/scrolling/ScrollingStateNode.cpp:
page/scrolling/ScrollingStateNode.h:
page/scrolling/ScrollingStateScrollingNode.cpp:
page/scrolling/ScrollingStateScrollingNode.h:
page/scrolling/ScrollingStateStickyNode.cpp:
page/scrolling/ScrollingStateStickyNode.h:
page/scrolling/ScrollingStateTree.cpp:
page/scrolling/ScrollingStateTree.h:

Removed ENABLE(THREADED_SCROLLING) code guards.

page/scrolling/coordinatedgraphics/ScrollingCoordinatorCoordinatedGraphics.cpp:

(WebCore::ScrollingCoordinatorCoordinatedGraphics::ScrollingCoordinatorCoordinatedGraphics):
(WebCore::ScrollingCoordinatorCoordinatedGraphics::~ScrollingCoordinatorCoordinatedGraphics):
(WebCore):
(WebCore::ScrollingCoordinatorCoordinatedGraphics::attachToStateTree):
(WebCore::ScrollingCoordinatorCoordinatedGraphics::detachFromStateTree):
(WebCore::ScrollingCoordinatorCoordinatedGraphics::clearStateTree):
(WebCore::ScrollingCoordinatorCoordinatedGraphics::updateViewportConstrainedNode):

page/scrolling/coordinatedgraphics/ScrollingCoordinatorCoordinatedGraphics.h:

ScrollingCoordinatorCoordinatedGraphics is using threaded scrolling approach to handle fixed
elements positioning while scrolling.

(WebCore):
(ScrollingCoordinatorCoordinatedGraphics):

page/scrolling/coordinatedgraphics/ScrollingStateNodeCoordinatedGraphics.cpp: Added.

(WebCore):
(WebCore::ScrollingStateNode::platformScrollLayer):
(WebCore::ScrollingStateNode::setScrollPlatformLayer):
(WebCore::ScrollingStateNode::setScrollLayer):

page/scrolling/coordinatedgraphics/ScrollingStateScrollingNodeCoordinatedGraphics.cpp: Added.

(WebCore):
(WebCore::ScrollingStateScrollingNode::counterScrollingPlatformLayer):
(WebCore::ScrollingStateScrollingNode::setCounterScrollingLayer):

5:49 AM Changeset in webkit [148196] by zarvai@inf.u-szeged.hu

1 edit
1 delete in trunk/LayoutTests

[Qt] Unreviewed gardening. Remove platform specific expected after r148195.

platform/qt/fast/events/backspace-navigates-back-expected.txt: Removed.

4:46 AM Changeset in webkit [148195] by allan.jensen@digia.com

7 edits in trunk/Source

[Qt] EventHandler should handle Space and BackSpace
https://bugs.webkit.org/show_bug.cgi?id=114428

Reviewed by Jocelyn Turcotte.

Source/WebCore:

Enable default space-handling for Qt.

page/EventHandler.cpp:

Source/WebKit/qt:

Remove our own handling of space and backspace.

WebCoreSupport/QWebPageAdapter.cpp:

(QWebPageAdapter::handleScrolling):

WidgetApi/qwebpage.cpp:

(QWebPagePrivate::keyPressEvent):

Source/WebKit2:

Remove our own handling of space and backspace.

WebProcess/WebPage/qt/WebPageQt.cpp:

(WebKit::WebPage::performDefaultBehaviorForKeyEvent):

4:37 AM Changeset in webkit [148194] by rakuco@webkit.org

2 edits in trunk/Source/WebCore

IconDatabase: Simplify boolean logic in assertion.
https://bugs.webkit.org/show_bug.cgi?id=114425

Reviewed by Andreas Kling.

Follow-up to r148097; drop the overzealous use of !! to turn a
PassRefPtr into a boolean.

loader/icon/IconDatabase.cpp:

(WebCore::IconDatabase::updateIconRecord):

4:28 AM Changeset in webkit [148193] by zarvai@inf.u-szeged.hu

2 edits
4 adds in trunk/LayoutTests

[Qt] Unreviewed gardending. Greening bots.

platform/qt-5.0-wk1/loader/go-back-cached-main-resource-expected.txt: Rebaselining after r148182.
platform/qt-5.0-wk2/loader/go-back-cached-main-resource-expected.txt: Rebaselining after r148182.
platform/qt/TestExpectations:

4:11 AM Changeset in webkit [148192] by anilsson@rim.com

14 edits in trunk/Source

[BlackBerry] LayerTexture refactoring
https://bugs.webkit.org/show_bug.cgi?id=114276

Reviewed by Carlos Garcia Campos.

Source/WebCore:

Remove the GpuHandle and HostType typedefs from LayerTexture, they
don't make any sense now that we removed the Skia code paths. With Skia
they used to be an OpenGL texture ID and an SkBitmap respectively, but
now it's just BlackBerry::Platform::Graphics::Buffer all around.

This allows us to rename textureId() into buffer() which is more to the
point. The texture ID concept is now up for grabs and can be used to
consolidate all the various code for extracting an OpenGL texture from
a BlackBerry::Platform::Graphics::Buffer. This opportunity is seized in
LayerTexture::platformTexture(), a name which makes more sense than
textureId() for anyone familiar with the GraphicsContext3D code. We can
also leverage the Platform3DObject typedef to avoid including gl2.h.

The vertex attribute state was getting trampled when interacting with
BlackBerry::Platform::Graphics in the LayerFilterRenderer. Some
additional GL state save/restore incantations when accessing the OpenGL
texture for a LayerTexture in the new LayerTexture::platformTexture()
method fixes various CSS Filters test cases.

Since we removed the Skia integration from LayerTiler, nobody cares
about the dirty rect, tile rect or opaqueness of the tile contents. So
remove those parameters from the updateContents code path.

No change in behavior except for fixing CSS Filters tests like
css3/filters/effect-blur-hw.html. This can only be verified by manually
looking at the tests, we don't have pixel test support to catch this
kind of regression.

PR 322882

platform/graphics/blackberry/LayerCompositingThread.cpp:

(WebCore::LayerCompositingThread::drawSurface):

platform/graphics/blackberry/LayerFilterRenderer.cpp:

(WebCore::LayerFilterRenderer::ping):
(WebCore::LayerFilterRenderer::pong):
(WebCore::LayerFilterRenderer::pushSnapshot):
(WebCore::LayerFilterRenderer::popSnapshot):
(WebCore::LayerFilterRenderer::applyActions):

platform/graphics/blackberry/LayerRenderer.cpp:

(WebCore::LayerRenderer::useSurface):

platform/graphics/blackberry/LayerTexture.cpp:

(WebCore::LayerTexture::LayerTexture):
(WebCore::LayerTexture::updateContents):
(WebCore::LayerTexture::setContentsToColor):
(WebCore::LayerTexture::protect):
(WebCore::LayerTexture::platformTexture):
(WebCore):

platform/graphics/blackberry/LayerTexture.h:

(WebCore::LayerTexture::isDirty):
(WebCore::LayerTexture::buffer):
(LayerTexture):
(WebCore::LayerTexture::setBuffer):

platform/graphics/blackberry/LayerTile.cpp:

(WebCore::LayerTile::setContents):
(WebCore::LayerTile::updateContents):

platform/graphics/blackberry/LayerTile.h:

(WebCore::LayerTile::hasTexture):
(LayerTile):

platform/graphics/blackberry/LayerTiler.cpp:

(WebCore::LayerTiler::updateTextureContentsIfNeeded):
(WebCore::LayerTiler::uploadTexturesIfNeeded):
(WebCore::LayerTiler::performTileJob):
(WebCore::LayerTiler::drawTile):

platform/graphics/blackberry/LayerTiler.h:

(WebCore::LayerTiler::TextureJob::TextureJob):
(WebCore::LayerTiler::TextureJob::setContents):
(WebCore::LayerTiler::TextureJob::updateContents):
(TextureJob):
(LayerTiler):

platform/graphics/blackberry/TextureCacheCompositingThread.cpp:

(WebCore::TextureCacheCompositingThread::createBuffer):
(WebCore::TextureCacheCompositingThread::collectGarbage):
(WebCore::TextureCacheCompositingThread::install):
(WebCore::TextureCacheCompositingThread::resizeTexture):
(WebCore::TextureCacheCompositingThread::evict):
(WebCore::TextureCacheCompositingThread::textureForContents):
(WebCore::TextureCacheCompositingThread::updateContents):

platform/graphics/blackberry/TextureCacheCompositingThread.h:

(TextureCacheCompositingThread):
(WebCore::TextureCacheCompositingThread::ZombieTexture::ZombieTexture):
(ZombieTexture):

Source/WebKit/blackberry:

Adapt to the LayerTexture refactoring down in WebCore.

PR 322882

Api/WebOverlay.cpp:

(BlackBerry::WebKit::WebOverlayLayerCompositingThreadClient::uploadTexturesIfNeeded):
(BlackBerry::WebKit::WebOverlayLayerCompositingThreadClient::drawTextures):

4:04 AM WebKitGTK/2.0.x edited by Carlos Garcia Campos

(diff)

4:00 AM Changeset in webkit [148191] by Carlos Garcia Campos

40 edits
2 moves
2 adds
2 deletes in releases/WebKitGTK/webkit-2.0/Source

Merge r147924 - [GTK] DOM objects created wrapping a base class have incorrect GObject type
https://bugs.webkit.org/show_bug.cgi?id=113132

Reviewed by Gustavo Noronha Silva.

Source/WebCore:

Some of the DOM methods can create a concrete instance of another
object, but return a base class. GObject bindings provide custom
kit implementations for Node, Element, Event and EvenTarget, so
any concrete HTMLElement wrapped as an HTMLElement ends up
wrapping the right WebCore object but in the wrong GObject type.
This affects not only HTMLElements, but any polymorphic object
wrapped using a base class.
This patch changes the way objects are created, so that all
classes have their own kit implementation, but for polymorphic
objects the kit implementation of the base class is always
used. The kit method of polymorphic base classes use a generic
wrap method instead of the wrapFoo method like all other
classes. In the static code we provide the implementation of the
generic wrap method for every polimorphic base class, falling back
to their wrapFoo method if the concrete object can't be used to
wrap the object.
Both WebKitDOMBinding and WebKitHTMLElementWrapperFactory now provide
implementations of wrap and have been renamed to WebKitDOMPrivate
and WebKitDOMHTMLPrivate.

bindings/gobject/GNUmakefile.am:
bindings/gobject/WebKitDOMBinding.cpp: Removed.
bindings/gobject/WebKitDOMEventTarget.cpp:

(WebKit::kit): Moved kit implementation from WebKitDOMBinding for
consistency with all other objects that have their own kit.
(WebKit::core): Updated to match the core implementation fo all
other objects.

bindings/gobject/WebKitDOMEventTargetPrivate.h:

(WebCore):
(WebKit):

bindings/gobject/WebKitDOMHTMLPrivate.cpp: Added.

(WebKit): Use preprocessor macros to define and set the wrap
functions of all HTML elements, removing a lot of boilerplate code
and making it easier to add new wrap functions in the future.
(WebKit::wrap):

bindings/gobject/WebKitDOMHTMLPrivate.h: Renamed from Source/WebCore/bindings/gobject/WebKitHTMLElementWrapperFactory.h.

(WebCore):
(WebKit):

bindings/gobject/WebKitDOMObject.cpp:
bindings/gobject/WebKitDOMPrivate.cpp: Added.

(WebKit):
(WebKit::wrap):

bindings/gobject/WebKitDOMPrivate.h: Renamed from Source/WebCore/bindings/gobject/WebKitDOMBinding.h.

(WebCore):
(WebKit):

bindings/gobject/WebKitHTMLElementWrapperFactory.cpp: Removed.
bindings/scripts/CodeGeneratorGObject.pm:

(IsBaseType): Helper function to check if a type is a base class.
(GetBaseClass): Helper function to get the base class for a given
parent type.
(GenerateFunction): Remove special case for EventTarget since the
kit prototype of EventTarget is in its private header like all
other objects.
(GenerateCFile): Add a kit implementation for all objects.
(IsPolymorphic): Helper function to check if the given type is the
base class of a polymorphic object.
(Generate): Include the private header of the base class for
polymorphic objects.
(WriteData): Remove the special case for WebKitDOMNode since now
all objects has a kit method.

bindings/scripts/test/GObject/WebKitDOMFloat64Array.cpp:

(WebKit::kit):

bindings/scripts/test/GObject/WebKitDOMFloat64ArrayPrivate.h:

(WebKit):

bindings/scripts/test/GObject/WebKitDOMTestActiveDOMObject.cpp:
bindings/scripts/test/GObject/WebKitDOMTestActiveDOMObjectPrivate.h:

(WebKit):

bindings/scripts/test/GObject/WebKitDOMTestCallback.cpp:
bindings/scripts/test/GObject/WebKitDOMTestCallbackPrivate.h:

(WebKit):

bindings/scripts/test/GObject/WebKitDOMTestCustomNamedGetter.cpp:
bindings/scripts/test/GObject/WebKitDOMTestCustomNamedGetterPrivate.h:

(WebKit):

bindings/scripts/test/GObject/WebKitDOMTestEventConstructor.cpp:
bindings/scripts/test/GObject/WebKitDOMTestEventConstructorPrivate.h:

(WebKit):

bindings/scripts/test/GObject/WebKitDOMTestEventTarget.cpp:
bindings/scripts/test/GObject/WebKitDOMTestEventTargetPrivate.h:

(WebKit):

bindings/scripts/test/GObject/WebKitDOMTestException.cpp:
bindings/scripts/test/GObject/WebKitDOMTestExceptionPrivate.h:

(WebKit):

bindings/scripts/test/GObject/WebKitDOMTestInterface.cpp:
bindings/scripts/test/GObject/WebKitDOMTestInterfacePrivate.h:

(WebKit):

bindings/scripts/test/GObject/WebKitDOMTestMediaQueryListListener.cpp:
bindings/scripts/test/GObject/WebKitDOMTestMediaQueryListListenerPrivate.h:

(WebKit):

bindings/scripts/test/GObject/WebKitDOMTestNamedConstructor.cpp:
bindings/scripts/test/GObject/WebKitDOMTestNamedConstructorPrivate.h:

(WebKit):

bindings/scripts/test/GObject/WebKitDOMTestNode.cpp:

(WebKit::kit):

bindings/scripts/test/GObject/WebKitDOMTestNodePrivate.h:

(WebKit):

bindings/scripts/test/GObject/WebKitDOMTestObj.cpp:
bindings/scripts/test/GObject/WebKitDOMTestObjPrivate.h:

(WebKit):

bindings/scripts/test/GObject/WebKitDOMTestOverloadedConstructors.cpp:
bindings/scripts/test/GObject/WebKitDOMTestOverloadedConstructorsPrivate.h:

(WebKit):

bindings/scripts/test/GObject/WebKitDOMTestSerializedScriptValueInterface.cpp:
bindings/scripts/test/GObject/WebKitDOMTestSerializedScriptValueInterfacePrivate.h:

(WebKit):

bindings/scripts/test/GObject/WebKitDOMTestTypedefs.cpp:
bindings/scripts/test/GObject/WebKitDOMTestTypedefsPrivate.h:

(WebKit):

Source/WebKit/gtk:

Remove WebKitDOMBindindg.h include from several files because it's
been renamed and it was not actually needed.

WebCoreSupport/ChromeClientGtk.cpp:
WebCoreSupport/EditorClientGtk.cpp:
webkit/webkithittestresult.cpp:

3:00 AM Changeset in webkit [148190] by rniwa@webkit.org

3 edits in trunk/Tools

Remove code for Chromium from wkbuild.py
https://bugs.webkit.org/show_bug.cgi?id=114423

Reviewed by Benjamin Poulain.

BuildSlaveSupport/build.webkit.org-config/wkbuild.py:

(_should_file_trigger_build):

BuildSlaveSupport/build.webkit.org-config/wkbuild_unittest.py:

(ShouldBuildTest):

2:49 AM Changeset in webkit [148189] by commit-queue@webkit.org

3 edits in trunk/Source/WebCore

[BlackBerry] RenderThemeBlackBerry: upstream missing code
https://bugs.webkit.org/show_bug.cgi?id=114343

Patch by Alberto Garcia <agarcia@igalia.com> on 2013-04-11
Reviewed by Xan Lopez.

Add implementations of sliderTickSize() and
sliderTickOffsetFromTrackCenter(), as well as a missing header.

platform/blackberry/RenderThemeBlackBerry.cpp:

(WebCore):
(WebCore::RenderThemeBlackBerry::sliderTickSize):
(WebCore::RenderThemeBlackBerry::sliderTickOffsetFromTrackCenter):

platform/blackberry/RenderThemeBlackBerry.h:

(RenderThemeBlackBerry):

2:17 AM Changeset in webkit [148188] by allan.jensen@digia.com

4 edits in trunk/Source/WebCore

HitTestRequest::AllowFrameScrollbars does not test main frame scrollbar
https://bugs.webkit.org/show_bug.cgi?id=112563

Reviewed by Antonio Gomes.

Move the frame scrollbar testing from RenderPart to RenderView, so that it is
tested by all frame and not just child frames. At the same time remove the
main frame test from the one current callsite using the option.

Tested by fast/events/touch/gesture/gesture-scrollbar.html.

page/EventHandler.cpp:

(WebCore::EventHandler::handleGestureEvent):

rendering/RenderPart.cpp:

(WebCore::RenderPart::nodeAtPoint):

rendering/RenderView.cpp:

(WebCore::RenderView::hitTest):

2:09 AM Changeset in webkit [148187] by Philippe Normand

2 edits in trunk/Tools

Unreviewed, update my IRC nickname.

Scripts/webkitpy/common/config/committers.py: I no longer use philn-tp.

2:00 AM Changeset in webkit [148186] by commit-queue@webkit.org

19 edits in trunk

Incorrect evaluation of resolution media queries
https://bugs.webkit.org/show_bug.cgi?id=114029

Patch by Rune Lillesveen <rune@opera.com> on 2013-04-11
Reviewed by Kenneth Rohde Christiansen.

Removed setResolutionOverride from exports.

Source/autotools/symbols.filter:

Source/WebCore:

The implementation used the physical resolution to evaluate the
resolution media features. Changed to use the actual CSS resolution,
also known as the device-pixel-ratio, instead. Unified the code for
evaluating the resolution and device-pixel-ratio media features.

No new tests, covered by existing tests.

WebCore.exp.in:
css/CSSPrimitiveValue.h:

(WebCore::CSSPrimitiveValue::isResolution):

css/MediaQueryEvaluator.cpp:

(WebCore::evalResolution):
(WebCore::device_pixel_ratioMediaFeatureEval):
(WebCore::resolutionMediaFeatureEval):

page/Screen.cpp:
page/Screen.h:
page/Settings.cpp:

(WebCore):

page/Settings.h:

(Settings):

testing/InternalSettings.cpp:

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

testing/InternalSettings.h:

(Backup):
(InternalSettings):

testing/InternalSettings.idl:

Source/WebKit:

Removed setResolutionOverride from exports.

WebKit.vcxproj/WebKitExportGenerator/WebKitExports.def.in:

Source/WebKit/win:

Removed setResolutionOverride from exports.

WebKit.vcproj/WebKitExports.def.in:

LayoutTests:

Modified tests to change CSS resolution instead of physical resolution.

fast/media/mq-resolution.html:

1:46 AM Changeset in webkit [148185] by rniwa@webkit.org

15 edits
1 delete in trunk/Tools

Remove the rest of Chromium code from webkitpy
https://bugs.webkit.org/show_bug.cgi?id=114420

Reviewed by Benjamin Poulain.

Scripts/webkitpy/common/config/urls.py:

(view_revision_url):

Scripts/webkitpy/common/host.py:

(Host._engage_awesome_locale_hacks):
(Host.initialize_scm):
(Host):

Scripts/webkitpy/common/host_mock.py:

(MockHost.init):
(MockHost.checkout):

Scripts/webkitpy/common/net/buildbot/chromiumbuildbot.py: Removed.
Scripts/webkitpy/layout_tests/layout_package/json_results_generator.py:

(JSONResultsGeneratorBase._insert_generic_metadata):

Scripts/webkitpy/layout_tests/models/test_expectations.py:

(TestExpectationParser._tokenize_line):

Scripts/webkitpy/layout_tests/models/test_expectations_unittest.py:

(ExpectationSyntaxTests.test_bare_name_and_bugs):
(ExpectationSyntaxTests.test_warnings):

Scripts/webkitpy/layout_tests/port/base.py:

(Port.skips_perf_test):
(Port.path_to_test_expectations_file):
(Port._port_flag_for_scripts):

Scripts/webkitpy/style/checkers/cpp.py:

(check_function_definition):
(check_include_line):

Scripts/webkitpy/style/checkers/cpp_unittest.py:

(OrderOfIncludesTest.test_check_wtf_includes):
(WebKitStyleTest.test_webkit_export_check):

Scripts/webkitpy/style/checkers/test_expectations_unittest.py:

(TestExpectationsTestCase.test_valid_expectations):

Scripts/webkitpy/tool/bot/irc_command.py:

(Rollout.execute):

Scripts/webkitpy/tool/bot/irc_command_unittest.py:

(IRCCommandTest.test_whois):

Scripts/webkitpy/tool/bot/ircbot_unittest.py:

(IRCBotTest.test_help):

Scripts/webkitpy/tool/commands/sheriffbot_unittest.py:

(SheriffBotTest.test_command_aliases):

1:29 AM Changeset in webkit [148184] by rniwa@webkit.org

2 edits in trunk/Tools

roll-chromium-deps doesn't roll DEPS
https://bugs.webkit.org/show_bug.cgi?id=114421

Reviewed by Antti Koivisto.

Roll DEPS. Rolling DEPS. Rolled DEPS.

Scripts/webkitpy/tool/bot/irc_command.py:

(RollChromiumDEPS.execute):

1:27 AM Changeset in webkit [148183] by glenn@skynav.com

3 edits in trunk/Tools

[webkitpy] Remove lingering return_exit_code usage in scm_mock and rebaselineserver.
https://bugs.webkit.org/show_bug.cgi?id=114418

Reviewed by Ryosuke Niwa.

Scripts/webkitpy/common/checkout/scm/scm_mock.py:

(MockSCM.add): Remove optional remove_exit_code argument.
(MockSCM.add_list): Remove optional remove_exit_code argument and return value.

Scripts/webkitpy/tool/servers/rebaselineserver.py:

(_rebaseline_test): Use ScriptError to capture SCM.add() failure and exit code.
(_move_test_baselines): Use ScriptError to capture SCM.add() failure and exit code.

1:25 AM Changeset in webkit [148182] by Carlos Garcia Campos

3 edits
8 adds in trunk

FrameLoaderClient::assignIdentifierToInitialRequest() not called for the main resource when loaded from the memory cache
https://bugs.webkit.org/show_bug.cgi?id=112418

Source/WebCore:

Reviewed by Darin Adler.

Test: loader/go-back-cached-main-resource.html

loader/DocumentLoader.cpp:

(WebCore::DocumentLoader::responseReceived):

LayoutTests:

Patch by Manuel Rego Casasnovas <Manuel Rego Casasnovas> on 2013-04-11
Reviewed by Darin Adler.

loader/go-back-cached-main-resource-expected.txt: Added.
loader/go-back-cached-main-resource.html: Added.
loader/resources/first-page.html: Added.
loader/resources/other-page.html: Added.
platform/gtk-wk1/loader/go-back-cached-main-resource-expected.txt: Added.
platform/wk2/loader/go-back-cached-main-resource-expected.txt: Added.

12:45 AM Changeset in webkit [148181] by rniwa@webkit.org

2 edits in trunk/Tools

Fix roll-chromium-deps and hi.

Scripts/webkitpy/tool/bot/irc_command.py:

(Hi.execute):
(RollChromiumDEPS.execute):

12:40 AM Changeset in webkit [148180] by rniwa@webkit.org

10 edits
7 deletes in trunk/Tools

Remove webkitpy code to roll Chromium DEPS
https://bugs.webkit.org/show_bug.cgi?id=114415

Reviewed by Benjamin Poulain.

Removed the code to roll Chromium DEPS in WebKit.

Scripts/webkitpy/common/checkout/checkout.py:

(Checkout.bug_id_for_this_commit):

Scripts/webkitpy/common/checkout/checkout_mock.py:

(MockCheckout.commit_message_for_this_commit):

Scripts/webkitpy/common/checkout/checkout_unittest.py:

(CheckoutTest.test_suggested_reviewers):

Scripts/webkitpy/common/checkout/deps.py: Removed.
Scripts/webkitpy/common/checkout/deps_mock.py: Removed.
Scripts/webkitpy/tool/bot/irc_command.py:

(RollChromiumDEPS): Teach webkitbot a sense of humor.
(RollChromiumDEPS.execute):

Scripts/webkitpy/tool/bot/irc_command_unittest.py:

(IRCCommandTest.test_create_bug):

Scripts/webkitpy/tool/bot/ircbot_unittest.py:

(IRCBotTest.test_revert):

Scripts/webkitpy/tool/bot/sheriff.py:

(Sheriff.post_rollout_patch):

Scripts/webkitpy/tool/commands/init.py:
Scripts/webkitpy/tool/commands/roll.py: Removed.
Scripts/webkitpy/tool/commands/roll_unittest.py: Removed.
Scripts/webkitpy/tool/steps/init.py:
Scripts/webkitpy/tool/steps/preparechangelogfordepsroll.py: Removed.
Scripts/webkitpy/tool/steps/updatechromiumdeps.py: Removed.
Scripts/webkitpy/tool/steps/updatechromiumdeps_unittest.py: Removed.

12:20 AM Changeset in webkit [148179] by Csaba Osztrogonác

3 edits in trunk/Source/WebKit2

Unreviewed Qt buildfixes after r148147, r148148 and r148154.

DerivedSources.pri:
Target.pri:

12:06 AM Changeset in webkit [148178] by Lucas Forschler

1 copy in tags/Safari-537.37

New Tag.

12:03 AM Changeset in webkit [148177] by glenn@skynav.com

3 edits in trunk/Tools

[webkitpy] SVNTest fails four tests when using subversion client 1.7 or later
https://bugs.webkit.org/show_bug.cgi?id=114386

Reviewed by Benjamin Poulain.

Subversion client 1.7 uses a new locking scheme that invalidates the technique used
in test_svn_lock(), so skip that test for 1.7 (or later) until a new technique
can be implemented. Further, 1.7 changed svn add to not add intermediate directories
by default and to return exit code 1 if a file/dir had already been added.

Scripts/webkitpy/common/checkout/scm/scm_unittest.py:

(test_svn_lock): Skip body of test if subversion client 1.7 or later.

Scripts/webkitpy/common/checkout/scm/svn.py:

(SVN.add_list): Use --parents option and handle exit code 1 if subversion client 1.7 or later.

Note: See TracTimeline for information about the timeline view.

Download in other formats:

RSS Feed