WebKit-https.git
10 years ago[Qt] Skipping test permanently because it is out of date to the current spec.
ossy@webkit.org [Wed, 19 Oct 2011 14:09:49 +0000 (14:09 +0000)]
[Qt] Skipping test permanently because it is out of date to the current spec.
https://bugs.webkit.org/show_bug.cgi?id=68796

Patch by Zoltan Arvai <zarvai@inf.u-szeged.hu> on 2011-10-19
Reviewed by Csaba Osztrogonác.

* platform/qt/Skipped:

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

10 years agoAX: buttons of number type <input> controls are not fully accessible
cfleizach@apple.com [Wed, 19 Oct 2011 14:02:21 +0000 (14:02 +0000)]
AX: buttons of number type <input> controls are not fully accessible
https://bugs.webkit.org/show_bug.cgi?id=70241

Unreviewed. Fix missing </File> tag in Windows WebCore project file.

* WebCore.vcproj/WebCore.vcproj:

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

10 years ago[GTK] Enable to use built-product-archive on GTK platform
rgabor@webkit.org [Wed, 19 Oct 2011 13:59:23 +0000 (13:59 +0000)]
[GTK] Enable to use built-product-archive on GTK platform
https://bugs.webkit.org/show_bug.cgi?id=70319

Reviewed by Csaba Osztrogonác.

* BuildSlaveSupport/built-product-archive:

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

10 years agoWeb Inspector: [Chromium] Add an ability to show the objects that were allocated...
loislo@chromium.org [Wed, 19 Oct 2011 13:36:47 +0000 (13:36 +0000)]
Web Inspector: [Chromium] Add an ability to show the objects that were allocated between snapshot N-2 and snapshot N-1 and still alive in snapshot N.
https://bugs.webkit.org/show_bug.cgi?id=61178

This ability will help us to see only the leaked objects.
Scenario:
1) do an action on inspected page which leaks javascript data;
2) make snapshot;
3) repeat first two steps three or more times;
4) select the last snapshot and select the filter 'Objects allocated between Snapshot 1 and Snapshot2' instead of 'All objects'.
The view shows the object that were allocated between snapshot N-2 and snapshot N-1 and still alive in snapshot N

Reviewed by Pavel Feldman.

* English.lproj/localizedStrings.js:
* inspector/front-end/DetailedHeapshotGridNodes.js:
(WebInspector.HeapSnapshotConstructorNode):
(WebInspector.HeapSnapshotConstructorNode.prototype._createNodesProvider):
* inspector/front-end/DetailedHeapshotView.js:
(WebInspector.HeapSnapshotConstructorsDataGrid):
(WebInspector.HeapSnapshotConstructorsDataGrid.prototype.setDataSource):
(WebInspector.HeapSnapshotConstructorsDataGrid.prototype.populateChildren):
(WebInspector.HeapSnapshotConstructorsDataGrid.prototype._filterSelectIndexChanged.firstSnapshotLoaded):
(WebInspector.HeapSnapshotConstructorsDataGrid.prototype._filterSelectIndexChanged.secondSnapshotLoaded):
(WebInspector.HeapSnapshotConstructorsDataGrid.prototype._filterSelectIndexChanged):
(WebInspector.HeapSnapshotDiffDataGrid.prototype._baseProfileIndexChanged):
(WebInspector.HeapSnapshotDiffDataGrid.prototype.populateChildren):
(WebInspector.DetailedHeapshotView.profileCallback):
(WebInspector.DetailedHeapshotView):
(WebInspector.DetailedHeapshotView.prototype.get statusBarItems):
(WebInspector.DetailedHeapshotView.prototype._changeBase):
(WebInspector.DetailedHeapshotView.prototype._changeFilter):
(WebInspector.DetailedHeapshotView.prototype._loadProfileByIndex):
(WebInspector.DetailedHeapshotView.prototype._changeView):
(WebInspector.DetailedHeapshotView.prototype._updateFilterOptions):
* inspector/front-end/HeapSnapshot.js:
(WebInspector.HeapSnapshot.prototype.dispose):
(WebInspector.HeapSnapshot.prototype.get maxNodeId):
(WebInspector.HeapSnapshot.prototype.aggregates):
(WebInspector.HeapSnapshot.prototype._buildAggregates):
(WebInspector.HeapSnapshot.prototype._sortAggregateIndexes):
(WebInspector.HeapSnapshot.prototype.createNodesProviderForClass):
(WebInspector.HeapSnapshot.prototype.updateStaticData):
* inspector/front-end/HeapSnapshotProxy.js:
(WebInspector.HeapSnapshotWorker):
(WebInspector.HeapSnapshotProxy.prototype.aggregates):
(WebInspector.HeapSnapshotProxy.prototype.createNodesProviderForClass):
(WebInspector.HeapSnapshotProxy.prototype.get maxNodeId):
(WebInspector.HeapSnapshotProxy.prototype.startLoading):

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

10 years agoStrip trailing whitespace in the WebCore/css C++ code.
apavlov@chromium.org [Wed, 19 Oct 2011 13:31:03 +0000 (13:31 +0000)]
Strip trailing whitespace in the WebCore/css C++ code.
https://bugs.webkit.org/show_bug.cgi?id=70410

Reviewed by Dirk Schulze.

Only affected file names retained below for brevity.
Multiline boolean expression were fixed where necessary to avoid patch style violations.

* css/CSSBorderImageSliceValue.cpp:
* css/CSSBorderImageSliceValue.h:
* css/CSSBorderImageValue.cpp:
* css/CSSCanvasValue.cpp:
* css/CSSCanvasValue.h:
* css/CSSComputedStyleDeclaration.cpp:
* css/CSSCursorImageValue.cpp:
* css/CSSFontFace.cpp:
* css/CSSFontFace.h:
* css/CSSFontFaceSource.cpp:
* css/CSSFontFaceSource.h:
* css/CSSFontFaceSrcValue.cpp:
* css/CSSFontFaceSrcValue.h:
* css/CSSFontSelector.cpp:
* css/CSSFontSelector.h:
* css/CSSFunctionValue.cpp:
* css/CSSFunctionValue.h:
* css/CSSGradientValue.cpp:
* css/CSSGradientValue.h:
* css/CSSImageGeneratorValue.cpp:
* css/CSSImageGeneratorValue.h:
* css/CSSImageValue.cpp:
* css/CSSImageValue.h:
* css/CSSImportRule.cpp:
* css/CSSInitialValue.cpp:
* css/CSSInitialValue.h:
* css/CSSMutableStyleDeclaration.cpp:
* css/CSSMutableStyleDeclaration.h:
* css/CSSParser.cpp:
* css/CSSParser.h:
* css/CSSParserValues.cpp:
* css/CSSParserValues.h:
* css/CSSPrimitiveValue.cpp:
* css/CSSPrimitiveValue.h:
* css/CSSPrimitiveValueCache.cpp:
* css/CSSPrimitiveValueCache.h:
* css/CSSPrimitiveValueMappings.h:
* css/CSSProperty.cpp:
* css/CSSProperty.h:
* css/CSSPropertyLonghand.cpp:
* css/CSSReflectValue.cpp:
* css/CSSReflectValue.h:
* css/CSSReflectionDirection.h:
* css/CSSSegmentedFontFace.cpp:
* css/CSSSegmentedFontFace.h:
* css/CSSSelector.cpp:
* css/CSSSelector.h:
* css/CSSSelectorList.cpp:
* css/CSSSelectorList.h:
* css/CSSStyleApplyProperty.cpp:
* css/CSSStyleDeclaration.h:
* css/CSSStyleSelector.cpp:
* css/CSSStyleSelector.h:
* css/CSSStyleSheet.cpp:
* css/CSSTimingFunctionValue.cpp:
* css/CSSTimingFunctionValue.h:
* css/CSSUnicodeRangeValue.cpp:
* css/CSSUnicodeRangeValue.h:
* css/CSSValueList.cpp:
* css/CSSWrapShapes.cpp:
* css/CSSWrapShapes.h:
* css/FontValue.cpp:
* css/MediaList.cpp:
* css/MediaQuery.cpp:
* css/MediaQuery.h:
* css/MediaQueryEvaluator.h:
* css/MediaQueryExp.cpp:
* css/MediaQueryExp.h:
* css/Pair.h:
* css/SVGCSSComputedStyleDeclaration.cpp:
* css/SVGCSSParser.cpp:
* css/SVGCSSStyleSelector.cpp:
* css/SelectorChecker.cpp:
* css/SelectorChecker.h:
* css/StyleBase.cpp:
* css/StyleMedia.cpp:
* css/StyleMedia.h:
* css/StyleSheetList.cpp:
* css/StyleSheetList.h:
* css/WebKitCSSFilterValue.cpp:
* css/WebKitCSSFilterValue.h:
* css/WebKitCSSKeyframeRule.cpp:
* css/WebKitCSSKeyframeRule.h:
* css/WebKitCSSKeyframesRule.cpp:
* css/WebKitCSSKeyframesRule.h:
* css/WebKitCSSMatrix.cpp:
* css/WebKitCSSMatrix.h:
* css/WebKitCSSTransformValue.cpp:
* css/WebKitCSSTransformValue.h:

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

10 years agoMake FilterEffect::apply() independent of image data management
krit@webkit.org [Wed, 19 Oct 2011 13:27:31 +0000 (13:27 +0000)]
Make FilterEffect::apply() independent of image data management
https://bugs.webkit.org/show_bug.cgi?id=70350

Reviewed by Zoltan Herczeg.

FilterEffect::apply() needs to be independent from the image data management in FilterEffect.
This allows implementing effective hardware accelerated alternatives, like OpenGL, CI or OpenCL
based filters, to the software rendering. The software rendering code will be used as fallback
and moves to platformApplySoftware().

The patch does not change the behavior of SVG filters. Did not add new test cases.

* platform/graphics/filters/FEBlend.cpp: Renamed apply() to platformApplyGeneric()
    and removed validity checks that are done by FilterEffect::apply(). This is a
    recurring pattern for all following effects.
(WebCore::FEBlend::platformApplySoftware):
* platform/graphics/filters/FEBlend.h:
* platform/graphics/filters/FEColorMatrix.cpp:
(WebCore::FEColorMatrix::platformApplySoftware):
* platform/graphics/filters/FEColorMatrix.h:
* platform/graphics/filters/FEComponentTransfer.cpp:
(WebCore::FEComponentTransfer::platformApplySoftware):
* platform/graphics/filters/FEComponentTransfer.h:
* platform/graphics/filters/FEComposite.cpp:
(WebCore::FEComposite::platformApplySoftware):
* platform/graphics/filters/FEComposite.h:
* platform/graphics/filters/FEConvolveMatrix.cpp:
(WebCore::FEConvolveMatrix::platformApplySoftware):
* platform/graphics/filters/FEConvolveMatrix.h:
* platform/graphics/filters/FEDisplacementMap.cpp:
(WebCore::FEDisplacementMap::platformApplySoftware):
* platform/graphics/filters/FEDisplacementMap.h:
* platform/graphics/filters/FEDropShadow.cpp:
(WebCore::FEDropShadow::platformApplySoftware):
* platform/graphics/filters/FEDropShadow.h:
* platform/graphics/filters/FEFlood.cpp:
(WebCore::FEFlood::platformApplySoftware):
* platform/graphics/filters/FEFlood.h:
* platform/graphics/filters/FEGaussianBlur.cpp:
(WebCore::FEGaussianBlur::platformApplySoftware):
* platform/graphics/filters/FEGaussianBlur.h:
* platform/graphics/filters/FELighting.cpp:
(WebCore::FELighting::platformApplySoftware):
* platform/graphics/filters/FELighting.h:
* platform/graphics/filters/FEMerge.cpp:
(WebCore::FEMerge::platformApplySoftware):
* platform/graphics/filters/FEMerge.h:
* platform/graphics/filters/FEMorphology.cpp:
(WebCore::FEMorphology::platformApplySoftware):
* platform/graphics/filters/FEMorphology.h:
* platform/graphics/filters/FEOffset.cpp:
(WebCore::FEOffset::platformApplySoftware):
* platform/graphics/filters/FEOffset.h:
* platform/graphics/filters/FETile.cpp:
(WebCore::FETile::platformApplySoftware):
* platform/graphics/filters/FETile.h:
* platform/graphics/filters/FETurbulence.cpp:
(WebCore::FETurbulence::platformApplySoftware):
* platform/graphics/filters/FETurbulence.h:
* platform/graphics/filters/FilterEffect.cpp: Added non-virtual apply() function. This will call the platform specific apply functions,
    which need to be virtual now.
(WebCore::FilterEffect::hasResult): Added a comment for new implementations. This function needs to be updated for every new port.
(WebCore::FilterEffect::apply):
(WebCore::FilterEffect::createImageBufferResult): Removed determineAbsolutePaintRect() call. This is done in apply() now.
(WebCore::FilterEffect::createUnmultipliedImageResult): Ditto.
(WebCore::FilterEffect::createPremultipliedImageResult): Ditto.
* platform/graphics/filters/FilterEffect.h:
* platform/graphics/filters/SourceAlpha.cpp:
(WebCore::SourceAlpha::platformApplySoftware):
* platform/graphics/filters/SourceAlpha.h:
* platform/graphics/filters/SourceGraphic.cpp:
(WebCore::SourceGraphic::platformApplySoftware):
* platform/graphics/filters/SourceGraphic.h:
* svg/graphics/filters/SVGFEImage.cpp:
(WebCore::FEImage::platformApplySoftware):
* svg/graphics/filters/SVGFEImage.h:

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

10 years ago[Qt] [WK2] Rename files and classes to follow conventions
caio.oliveira@openbossa.org [Wed, 19 Oct 2011 12:58:35 +0000 (12:58 +0000)]
[Qt] [WK2] Rename files and classes to follow conventions
https://bugs.webkit.org/show_bug.cgi?id=70391

Reviewed by Noam Rosenthal.

Rename the files and classes following the agreed conventions on
QtWebKit mailing list -- which are based on existing usage we have.

- Public API follow Qt conventions: qsomething.cpp qsomething.h with class named QSomething

- Private classes for public API classes follow Qt conventions:
  qsomething_p.h qsomething_p.cpp (if needed), with class named QSomethingPrivate

- Subclasses and Qt-specific implementations of WebKit/WebCore classes use Qt suffix:
  FrameLoaderClientQt.cpp

- Other classes created for using in our port use Qt prefix: QtFileDownloader, QtViewInterface

This is documented in https://trac.webkit.org/wiki/QtWebKitHacking

* UIProcess/API/qt/qdesktopwebview.cpp:
(QDesktopWebViewPrivate::chooseFiles):
(toPolicyAction):
(QDesktopWebViewPrivate::navigationPolicyForURL):
* UIProcess/API/qt/qdesktopwebview_p.h:
* UIProcess/API/qt/qtouchwebpage.cpp:
(QTouchWebPage::updatePaintNode):
(QTouchWebPagePrivate::setPage):
* UIProcess/API/qt/qtouchwebpage.h:
* UIProcess/API/qt/qtouchwebpage_p.h:
* UIProcess/API/qt/qtouchwebview.cpp:
(QTouchWebViewPrivate::updateViewportConstraints):
* UIProcess/API/qt/qtouchwebview.h:
* UIProcess/API/qt/qtouchwebview_p.h:
* UIProcess/TiledDrawingAreaProxy.h:
* UIProcess/qt/ClientImpl.cpp:
(toQtViewInterface):
(toQtPolicyInterface):
(qt_wk_runJavaScriptAlert):
(qt_wk_runJavaScriptConfirm):
(qt_wk_runJavaScriptPrompt):
(qt_wk_setStatusText):
(qt_wk_runOpenPanel):
(qt_wk_mouseDidMoveOverElement):
(qt_wk_decidePolicyForNavigationAction):
* UIProcess/qt/QtDesktopWebPageProxy.cpp: Renamed from Source/WebKit2/UIProcess/qt/qdesktopwebpageproxy.cpp.
(dragOperationToDropAction):
(QtDesktopWebPageProxy::QtDesktopWebPageProxy):
(QtDesktopWebPageProxy::createDrawingAreaProxy):
(QtDesktopWebPageProxy::paintContent):
(QtDesktopWebPageProxy::setViewportArguments):
(QtDesktopWebPageProxy::doneWithTouchEvent):
(QtDesktopWebPageProxy::createPopupMenuProxy):
(QtDesktopWebPageProxy::handleEvent):
(QtDesktopWebPageProxy::handleMouseMoveEvent):
(QtDesktopWebPageProxy::handleMousePressEvent):
(QtDesktopWebPageProxy::handleMouseReleaseEvent):
(QtDesktopWebPageProxy::handleMouseDoubleClickEvent):
(QtDesktopWebPageProxy::handleWheelEvent):
(QtDesktopWebPageProxy::handleHoverLeaveEvent):
(QtDesktopWebPageProxy::handleHoverMoveEvent):
(QtDesktopWebPageProxy::handleDragEnterEvent):
(QtDesktopWebPageProxy::handleDragLeaveEvent):
(QtDesktopWebPageProxy::handleDragMoveEvent):
(QtDesktopWebPageProxy::handleDropEvent):
(QtDesktopWebPageProxy::timerEvent):
* UIProcess/qt/QtDesktopWebPageProxy.h: Renamed from Source/WebKit2/UIProcess/qt/qdesktopwebpageproxy.h.
* UIProcess/qt/QtGestureRecognizer.cpp:
(WebKit::QtGestureRecognizer::QtGestureRecognizer):
* UIProcess/qt/QtGestureRecognizer.h:
* UIProcess/qt/QtPanGestureRecognizer.cpp:
(WebKit::QtPanGestureRecognizer::QtPanGestureRecognizer):
* UIProcess/qt/QtPanGestureRecognizer.h:
* UIProcess/qt/QtPinchGestureRecognizer.cpp:
(WebKit::QtPinchGestureRecognizer::QtPinchGestureRecognizer):
* UIProcess/qt/QtPinchGestureRecognizer.h:
* UIProcess/qt/QtPolicyInterface.h: Renamed from Source/WebKit2/UIProcess/qt/PolicyInterface.h.
* UIProcess/qt/QtSGTileNode.cpp: Renamed from Source/WebKit2/UIProcess/qt/SGTileNode.cpp.
(WebKit::QtSGTileNode::QtSGTileNode):
(WebKit::QtSGTileNode::setBackBuffer):
(WebKit::QtSGTileNode::swapBuffersIfNeeded):
* UIProcess/qt/QtSGTileNode.h: Renamed from Source/WebKit2/UIProcess/qt/SGTileNode.h.
* UIProcess/qt/QtSGUpdateQueue.cpp: Renamed from Source/WebKit2/UIProcess/qt/SGUpdateQueue.cpp.
(WebKit::NodeUpdateCreateTile::NodeUpdateCreateTile):
(WebKit::NodeUpdateRemoveTile::NodeUpdateRemoveTile):
(WebKit::NodeUpdateSetBackBuffer::NodeUpdateSetBackBuffer):
(WebKit::NodeUpdateSwapTileBuffers::NodeUpdateSwapTileBuffers):
(WebKit::QtSGUpdateQueue::QtSGUpdateQueue):
(WebKit::QtSGUpdateQueue::createTileNode):
(WebKit::QtSGUpdateQueue::removeTileNode):
(WebKit::QtSGUpdateQueue::setNodeBackBuffer):
(WebKit::QtSGUpdateQueue::swapTileBuffers):
(WebKit::QtSGUpdateQueue::applyUpdates):
(WebKit::QtSGUpdateQueue::getScaleNode):
* UIProcess/qt/QtSGUpdateQueue.h: Renamed from Source/WebKit2/UIProcess/qt/SGUpdateQueue.h.
(WebKit::QtSGUpdateQueue::isSwapPending):
(WebKit::NodeUpdate::NodeUpdate):
(WebKit::NodeUpdate::~NodeUpdate):
* UIProcess/qt/QtTouchViewInterface.cpp: Renamed from Source/WebKit2/UIProcess/qt/TouchViewInterface.cpp.
(WebKit::QtTouchViewInterface::QtTouchViewInterface):
(WebKit::QtTouchViewInterface::didFindZoomableArea):
(WebKit::QtTouchViewInterface::sceneGraphUpdateQueue):
(WebKit::QtTouchViewInterface::setViewNeedsDisplay):
(WebKit::QtTouchViewInterface::drawingAreaSize):
(WebKit::QtTouchViewInterface::contentSizeChanged):
(WebKit::QtTouchViewInterface::isActive):
(WebKit::QtTouchViewInterface::hasFocus):
(WebKit::QtTouchViewInterface::isVisible):
(WebKit::QtTouchViewInterface::startDrag):
(WebKit::QtTouchViewInterface::didReceiveViewportArguments):
(WebKit::QtTouchViewInterface::didChangeUrl):
(WebKit::QtTouchViewInterface::didChangeTitle):
(WebKit::QtTouchViewInterface::didChangeToolTip):
(WebKit::QtTouchViewInterface::didChangeStatusText):
(WebKit::QtTouchViewInterface::didChangeCursor):
(WebKit::QtTouchViewInterface::loadDidBegin):
(WebKit::QtTouchViewInterface::loadDidCommit):
(WebKit::QtTouchViewInterface::loadDidSucceed):
(WebKit::QtTouchViewInterface::loadDidFail):
(WebKit::QtTouchViewInterface::didChangeLoadProgress):
(WebKit::QtTouchViewInterface::showContextMenu):
(WebKit::QtTouchViewInterface::hideContextMenu):
(WebKit::QtTouchViewInterface::runJavaScriptAlert):
(WebKit::QtTouchViewInterface::runJavaScriptConfirm):
(WebKit::QtTouchViewInterface::runJavaScriptPrompt):
(WebKit::QtTouchViewInterface::processDidCrash):
(WebKit::QtTouchViewInterface::didRelaunchProcess):
(WebKit::QtTouchViewInterface::engine):
* UIProcess/qt/QtTouchViewInterface.h: Renamed from Source/WebKit2/UIProcess/qt/TouchViewInterface.h.
(WebKit::QtTouchViewInterface::chooseFiles):
(WebKit::QtTouchViewInterface::didMouseMoveOverElement):
* UIProcess/qt/QtTouchWebPageProxy.cpp: Renamed from Source/WebKit2/UIProcess/qt/qtouchwebpageproxy.cpp.
(QtTouchWebPageProxy::QtTouchWebPageProxy):
(QtTouchWebPageProxy::createDrawingAreaProxy):
(QtTouchWebPageProxy::processDidCrash):
(QtTouchWebPageProxy::paintContent):
(QtTouchWebPageProxy::doneWithTouchEvent):
(QtTouchWebPageProxy::handleEvent):
(QtTouchWebPageProxy::setVisibleContentRectAndScale):
(QtTouchWebPageProxy::setVisibleContentRectTrajectoryVector):
(QtTouchWebPageProxy::setResizesToContentsUsingLayoutSize):
(QtTouchWebPageProxy::touchEvent):
(QtTouchWebPageProxy::findZoomableAreaForPoint):
(QtTouchWebPageProxy::renderNextFrame):
* UIProcess/qt/QtTouchWebPageProxy.h: Renamed from Source/WebKit2/UIProcess/qt/qtouchwebpageproxy.h.
(QtTouchWebPageProxy::touchViewInterface):
(QtTouchWebPageProxy::drawingArea):
* UIProcess/qt/QtViewInterface.h: Renamed from Source/WebKit2/UIProcess/qt/ViewInterface.h.
* UIProcess/qt/QtViewportInteractionEngine.cpp: Renamed from Source/WebKit2/UIProcess/qt/ViewportInteractionEngine.cpp.
(WebKit::visibleRectInContentCoordinate):
(WebKit::contentRectInViewportCoordinate):
(WebKit::ViewportUpdateGuard::ViewportUpdateGuard):
(WebKit::ViewportUpdateGuard::~ViewportUpdateGuard):
(WebKit::QtViewportInteractionEngine::QtViewportInteractionEngine):
(WebKit::QtViewportInteractionEngine::~QtViewportInteractionEngine):
(WebKit::QtViewportInteractionEngine::reset):
(WebKit::QtViewportInteractionEngine::setConstraints):
(WebKit::QtViewportInteractionEngine::panGestureStarted):
(WebKit::QtViewportInteractionEngine::panGestureRequestUpdate):
(WebKit::QtViewportInteractionEngine::panGestureCancelled):
(WebKit::QtViewportInteractionEngine::panGestureEnded):
(WebKit::QtViewportInteractionEngine::pinchGestureStarted):
(WebKit::QtViewportInteractionEngine::pinchGestureRequestUpdate):
(WebKit::QtViewportInteractionEngine::pinchGestureEnded):
(WebKit::QtViewportInteractionEngine::contentViewportChanged):
(WebKit::QtViewportInteractionEngine::updateContentIfNeeded):
(WebKit::QtViewportInteractionEngine::updateContentScaleIfNeeded):
(WebKit::QtViewportInteractionEngine::updateContentPositionIfNeeded):
(WebKit::QtViewportInteractionEngine::animateContentIntoBoundariesIfNeeded):
(WebKit::QtViewportInteractionEngine::animateContentPositionIntoBoundariesIfNeeded):
(WebKit::QtViewportInteractionEngine::animateContentScaleIntoBoundariesIfNeeded):
(WebKit::QtViewportInteractionEngine::scaleContent):
* UIProcess/qt/QtViewportInteractionEngine.h: Renamed from Source/WebKit2/UIProcess/qt/ViewportInteractionEngine.h.
(WebKit::QtViewportInteractionEngine::Constraints::Constraints):
(WebKit::operator==):
* UIProcess/qt/QtWebPageProxy.cpp:
(QtWebPageProxy::QtWebPageProxy):
(QtWebPageProxy::registerEditCommand):
* UIProcess/qt/QtWebPageProxy.h:
* UIProcess/qt/QtWebUndoCommand.cpp: Renamed from Source/WebKit2/UIProcess/qt/WebUndoCommandQt.cpp.
(QtWebUndoCommand::QtWebUndoCommand):
(QtWebUndoCommand::~QtWebUndoCommand):
(QtWebUndoCommand::redo):
(QtWebUndoCommand::undo):
* UIProcess/qt/QtWebUndoCommand.h: Renamed from Source/WebKit2/UIProcess/qt/WebUndoCommandQt.h.
(QtWebUndoCommand::inUndoRedo):
* UIProcess/qt/TiledDrawingAreaProxyQt.cpp:
(WebKit::TiledDrawingAreaProxy::updateWebView):
* UIProcess/qt/WebContextMenuProxyQt.cpp:
(WebKit::WebContextMenuProxyQt::WebContextMenuProxyQt):
(WebKit::WebContextMenuProxyQt::create):
* UIProcess/qt/WebContextMenuProxyQt.h:
* WebKit2.pro:

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

10 years agoWeb Inspector: Enable caseSensitive search / Regex support in advanced search.
vsevik@chromium.org [Wed, 19 Oct 2011 12:36:07 +0000 (12:36 +0000)]
Web Inspector: Enable caseSensitive search / Regex support in advanced search.
https://bugs.webkit.org/show_bug.cgi?id=70335

Reviewed by Pavel Feldman.

Source/WebCore:

* inspector/ContentSearchUtils.cpp:
(WebCore::ContentSearchUtils::createSearchRegex):
(WebCore::ContentSearchUtils::searchInTextByLines):
* inspector/ContentSearchUtils.h:
* inspector/Inspector.json:
* inspector/InspectorDebuggerAgent.cpp:
(WebCore::InspectorDebuggerAgent::searchInContent):
* inspector/InspectorDebuggerAgent.h:
* inspector/InspectorPageAgent.cpp:
(WebCore::InspectorPageAgent::searchInResource):
* inspector/InspectorPageAgent.h:
* inspector/front-end/AdvancedSearchController.js:
(WebInspector.SearchView):
(WebInspector.SearchView.prototype.get searchConfig):
(WebInspector.FileBasedSearchResultsPane.prototype._createSearchRegex):
(WebInspector.FileBasedSearchResultsPane.prototype.addSearchResult):
(WebInspector.FileBasedSearchResultsPane.SearchResult):
* inspector/front-end/ConsolePanel.js:
(WebInspector.ConsolePanel.prototype.performSearch):
* inspector/front-end/ContentProviders.js:
(WebInspector.ScriptContentProvider.prototype.searchInContent):
(WebInspector.ConcatenatedScriptsContentProvider.prototype.searchInContent.searchCallback):
(WebInspector.ConcatenatedScriptsContentProvider.prototype.searchInContent):
(WebInspector.ResourceContentProvider.prototype.searchInContent):
(WebInspector.CompilerSourceMappingContentProvider.prototype.searchInContent):
(WebInspector.StaticContentProvider.prototype.searchInContent):
* inspector/front-end/ElementsTreeOutline.js:
():
* inspector/front-end/NetworkPanel.js:
(WebInspector.NetworkLogView.prototype.performSearch):
* inspector/front-end/Resource.js:
(WebInspector.Resource.prototype.searchInContent):
* inspector/front-end/Script.js:
(WebInspector.Script.prototype.searchInContent):
* inspector/front-end/ScriptsSearchScope.js:
(WebInspector.ScriptsSearchScope.prototype.performSearch.searchCallbackWrapper):
(WebInspector.ScriptsSearchScope.prototype.performSearch):
* inspector/front-end/SourceFrame.js:
(WebInspector.SourceFrame.createSearchRegex):
* inspector/front-end/UISourceCode.js:
(WebInspector.UISourceCode.prototype.searchInContent):
(WebInspector.ContentProvider.prototype.searchInContent):
* inspector/front-end/inspector.css:
(.search-view .search-panel input[type="search"].search-config-search):
(.search-view .search-panel label.search-config-label):
(.search-view .search-panel input[type="checkbox"].search-config-checkbox):
* inspector/front-end/utilities.js:
():

LayoutTests:

* http/tests/inspector/search/search-in-concatenated-script.html:
* http/tests/inspector/search/search-in-resource-expected.txt:
* http/tests/inspector/search/search-in-resource.html:
* http/tests/inspector/search/search-in-script-expected.txt:
* http/tests/inspector/search/search-in-script.html:

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

10 years agoWeb Inspector: make extension tests pass on chromium
caseq@chromium.org [Wed, 19 Oct 2011 12:22:54 +0000 (12:22 +0000)]
Web Inspector: make extension tests pass on chromium
https://bugs.webkit.org/show_bug.cgi?id=70334

Reviewed by Pavel Feldman.

Source/WebKit/chromium:

* DEPS: roll chromium 105970:106056

LayoutTests:

* http/tests/inspector/extensions-headers.html:
* http/tests/inspector/resources/extension-main.js:
():
* inspector/extensions/extensions-audits.html:
* inspector/extensions/extensions-console.html:
* inspector/extensions/extensions-resources.html:
* platform/chromium/inspector/extensions/extensions-api-expected.txt: Added.
* platform/chromium/inspector/extensions/extensions-eval-expected.txt: Added.
* platform/chromium/test_expectations.txt:

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

10 years agoMediaList: Remove inheritance from StyleBase.
andreas.kling@nokia.com [Wed, 19 Oct 2011 12:11:40 +0000 (12:11 +0000)]
MediaList: Remove inheritance from StyleBase.
https://bugs.webkit.org/show_bug.cgi?id=70203

Reviewed by Darin Adler.

* css/MediaList.h:
(WebCore::MediaList::parentStyleSheet):
(WebCore::MediaList::setParentStyleSheet):
* css/MediaList.cpp:
(WebCore::MediaList::MediaList):
(WebCore::MediaList::notifyChanged):

    Change MediaList to no longer inherit from StyleBase. The parent()/setParent()
    mechanism was replaced by explicit parentStyleSheet()/setParentStyleSheet().

* css/CSSImportRule.cpp:
(WebCore::CSSImportRule::CSSImportRule):
(WebCore::CSSImportRule::~CSSImportRule):
* css/CSSMediaRule.cpp:
(WebCore::CSSMediaRule::CSSMediaRule):
(WebCore::CSSMediaRule::~CSSMediaRule):

    Change StyleBase::setParent() calls to MediaList::setParentStyleSheet().
    CSSRules are never moved between stylesheets, so the parent style sheet pointer
    is valid for the MediaList's lifetime.

* css/StyleSheet.cpp:
(WebCore::StyleSheet::~StyleSheet):

    Change StyleBase::setParent() calls to MediaList::setParentStyleSheet().

(WebCore::StyleSheet::setMedia):

    Change StyleBase::setParent() calls to MediaList::setParentStyleSheet().
    Assert that the parent sheet is either null, or this, as MediaLists should never
    be reparented to other sheets.
    Assert that the StyleSheet is indeed a CSSStyleSheet.  XSLStyleSheet inherits from
    StyleSheet, but it shouldn't be using the MediaList.

* bindings/js/JSDOMBinding.h:
(WebCore::root):

    Add MediaList* specific overload for root() since it can't fall back
    to root(StyleBase*) anymore.

* WebCore.xcodeproj/project.pbxproj:

    Add some PrivateHeaders to accomodate the JSDOMBinding.h changes.

* css/CSSMediaRule.h:

    Remove unnecessary forward-declaration of MediaList.

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

10 years agoUnreviewed, GTK rebaseline and GTK baselines for
philn@webkit.org [Wed, 19 Oct 2011 12:02:18 +0000 (12:02 +0000)]
Unreviewed, GTK rebaseline and GTK baselines for
fast/multicol/vertical-rl/rule-style.html

* platform/gtk/Skipped: Skip
fast/files/url-required-arguments.html, affected by bug 66045
* platform/gtk/fast/dom/Window/window-properties-expected.txt:
* platform/gtk/fast/multicol/vertical-rl/rule-style-expected.txt: Added.
* platform/gtk/svg/hixie/cascade/002-expected.txt:
* platform/gtk/tables/mozilla/marvin/backgr_simple-table-column-group-expected.txt:
* platform/gtk/tables/mozilla/marvin/backgr_simple-table-expected.txt:
* platform/gtk/tables/mozilla/marvin/backgr_simple-table-row-group-expected.txt:
* platform/gtk/tables/mozilla_expected_failures/marvin/backgr_layers-show-expected.txt:
* platform/gtk/tables/mozilla_expected_failures/marvin/backgr_position-table-cell-expected.txt:
* platform/gtk/tables/mozilla_expected_failures/marvin/backgr_position-table-column-expected.txt:

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

10 years agoWeb Inspector: Console messages for XHR logging should show call stack if available.
vsevik@chromium.org [Wed, 19 Oct 2011 11:29:27 +0000 (11:29 +0000)]
Web Inspector: Console messages for XHR logging should show call stack if available.
https://bugs.webkit.org/show_bug.cgi?id=70324

Reviewed by Pavel Feldman.

* inspector/front-end/ConsoleMessage.js:
(WebInspector.ConsoleMessageImpl.prototype._formatMessage):

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

10 years ago[Qt] Unreviewed fix after r97845.
ossy@webkit.org [Wed, 19 Oct 2011 11:18:01 +0000 (11:18 +0000)]
[Qt] Unreviewed fix after r97845.

* platform/qt/test_expectations.txt: Remove a skipped test, because NRWT can't handle duplicated entries.

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

10 years ago[Qt] Unreviewed gardening.
ossy@webkit.org [Wed, 19 Oct 2011 11:09:56 +0000 (11:09 +0000)]
[Qt] Unreviewed gardening.

inspector/cookie-parser.html is a flaky crash
https://bugs.webkit.org/show_bug.cgi?id=62662

[Qt] Assertion fail in CSSPrimitiveValue ctor
https://bugs.webkit.org/show_bug.cgi?id=69933

* platform/qt/Skipped: Skip failing tests to paint the bots green.

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

10 years ago[Qt] Unreviewed gardening. Skip new failing tests.
ossy@webkit.org [Wed, 19 Oct 2011 10:51:09 +0000 (10:51 +0000)]
[Qt] Unreviewed gardening. Skip new failing tests.

* platform/qt-mac/Skipped:
* platform/qt-wk2/Skipped:

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

10 years agoWeb Inspector: move JS specific popover handling to JavaScriptSourceFrame.
podivilov@chromium.org [Wed, 19 Oct 2011 09:48:28 +0000 (09:48 +0000)]
Web Inspector: move JS specific popover handling to JavaScriptSourceFrame.
https://bugs.webkit.org/show_bug.cgi?id=70033

Reviewed by Pavel Feldman.

* inspector/front-end/DebuggerPresentationModel.js:
(WebInspector.DebuggerPresentationModel.prototype.uiSourceCodes):
(WebInspector.DebuggerPresentationModel.prototype.get paused):
* inspector/front-end/JavaScriptSourceFrame.js:
(WebInspector.JavaScriptSourceFrame):
(WebInspector.JavaScriptSourceFrame.prototype.shouldShowPopover):
(WebInspector.JavaScriptSourceFrame.prototype.onShowPopover.showObjectPopover):
(WebInspector.JavaScriptSourceFrame.prototype.onShowPopover):
(WebInspector.JavaScriptSourceFrame.prototype.onHidePopover):
(WebInspector.JavaScriptSourceFrame.prototype._highlightExpression):
* inspector/front-end/ScriptsPanel.js:
(WebInspector.ScriptsPanel.prototype._createSourceFrame):
(WebInspector.SourceFrameDelegateForScriptsPanel):
* inspector/front-end/SourceFrame.js:
(WebInspector.SourceFrame):
(WebInspector.SourceFrame.prototype.willHide):
(WebInspector.SourceFrame.prototype._initializeTextViewer):
(WebInspector.SourceFrame.prototype._getPopoverAnchor):
(WebInspector.SourceFrame.prototype.get readOnly):
(WebInspector.SourceFrame.prototype._setReadOnly):
(WebInspector.SourceFrame.prototype.contentChanged):
(WebInspector.SourceFrame.prototype.shouldShowPopover):
(WebInspector.SourceFrame.prototype.onShowPopover):
(WebInspector.SourceFrame.prototype.onHidePopover):

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

10 years ago[Qt] Unreviewed gardening. Add platform specific expected result for new tests.
rgabor@webkit.org [Wed, 19 Oct 2011 09:45:48 +0000 (09:45 +0000)]
[Qt] Unreviewed gardening. Add platform specific expected result for new tests.

* platform/qt/fast/multicol/vertical-rl/rule-style-expected.png: Added.
* platform/qt/fast/multicol/vertical-rl/rule-style-expected.txt: Added.
* platform/qt/fast/selectors/visited-descendant-expected.png: Added.
* platform/qt/fast/selectors/visited-descendant-expected.txt: Added.

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

10 years ago[EFL][WK2] Added missing files to build WebKit2 EFL
commit-queue@webkit.org [Wed, 19 Oct 2011 08:32:30 +0000 (08:32 +0000)]
[EFL][WK2] Added missing files to build WebKit2 EFL
https://bugs.webkit.org/show_bug.cgi?id=70312

Patch by Tomasz Morawski <t.morawski@samsung.com> on 2011-10-19
Reviewed by Ryosuke Niwa.

Added missing files to WebCore/CMakeListEfl.txt script:
- platform/network/soup/CredentialStorageSoup.cpp
- platform/network/soup/ProxyServerSoup.cpp
Fixes few linking time errors.

* CMakeListsEfl.txt:

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

10 years ago[EFL][WK2] Added missing dummy interpretKeyEvent function
commit-queue@webkit.org [Wed, 19 Oct 2011 07:53:02 +0000 (07:53 +0000)]
[EFL][WK2] Added missing dummy interpretKeyEvent function
https://bugs.webkit.org/show_bug.cgi?id=70311

Patch by Tomasz Morawski <t.morawski@samsung.com> on 2011-10-19
Reviewed by Ryosuke Niwa.

Added missing dummy interpretKeyEvent function needed to compile WK2.

* WebProcess/WebPage/efl/WebPageEfl.cpp:
(WebKit::WebPage::interpretKeyEvent):

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

10 years agoRename AllowAllocation to ConstructorMode
haraken@chromium.org [Wed, 19 Oct 2011 07:47:20 +0000 (07:47 +0000)]
Rename AllowAllocation to ConstructorMode
https://bugs.webkit.org/show_bug.cgi?id=70397

Reviewed by Adam Barth.

The name 'AllowAllocation' is confusing (See discussion here: bug 70015).
This patch renames 'AllowAllocation' to 'ConstructorMode' to clarify that
this class is for distinguishing the following two situations:
 1. A programmer allocates an object via "new X"; X is a function with a C++ constructCallback.
 2. C++ allocates an object via the function template and tries to wrap the object with a JS flavor.

Tests: fast/events/constructors/*

* bindings/scripts/CodeGeneratorV8.pm: Renamed AllowAllocation to ConstructorMode.
(GenerateConstructorCallback):
* bindings/scripts/test/V8/V8TestInterface.cpp: Updated a run-binding-tests result.
(WebCore::V8TestInterface::constructorCallback):
* bindings/v8/V8Binding.cpp: Renamed AllowAllocation to ConstructorMode.
(WebCore::V8BindingPerIsolateData::V8BindingPerIsolateData):
* bindings/v8/V8Binding.h: Ditto.
(WebCore::ConstructorMode::ConstructorMode):
(WebCore::ConstructorMode::~ConstructorMode):
(WebCore::ConstructorMode::current):
(WebCore::SafeAllocation::newInstance):
* bindings/v8/V8Proxy.cpp: Ditto.
(WebCore::V8Proxy::checkNewLegal):
* bindings/v8/custom/V8EventConstructors.cpp: Ditto.
(WebCore::constructV8Event):

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

10 years ago[chromium] Test expectation change.
yuzo@google.com [Wed, 19 Oct 2011 07:32:14 +0000 (07:32 +0000)]
[chromium] Test expectation change.

Unreviewed.

* platform/chromium/test_expectations.txt:

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

10 years ago[EFL] Added missing dummy platformDescription function
commit-queue@webkit.org [Wed, 19 Oct 2011 07:18:01 +0000 (07:18 +0000)]
[EFL] Added missing dummy platformDescription function
https://bugs.webkit.org/show_bug.cgi?id=70310

Patch by Tomasz Morawski <t.morawski@samsung.com> on 2011-10-19
Reviewed by Ryosuke Niwa.

Added dummy platformDescription function needed to compile WK2.

* platform/efl/ContextMenuEfl.cpp:
(WebCore::ContextMenu::platformDescription):

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

10 years agoGenerate a SharedWorker constructor of V8 using [Constructor] IDL
haraken@chromium.org [Wed, 19 Oct 2011 06:57:36 +0000 (06:57 +0000)]
Generate a SharedWorker constructor of V8 using [Constructor] IDL
https://bugs.webkit.org/show_bug.cgi?id=67879

Reviewed by Hajime Morita.

Source/WebCore:

Spec: http://dev.w3.org/html5/workers/#shared-workers-and-the-sharedworker-interface
This patch changed SharedWorker::create(..., context, ec) to
SharedWorker::create(context, ..., ec), since a parameter specified by [CallWith]
should come at the beginning (c.f. bug 69799).

Test: ui_tests:WorkerTest.FLAKY_SharedWorkerFastConstructor
      ui_tests:WorkerTest.FLAKY_SharedWorkerFastName

* WebCore.gypi: Removed bindings/v8/custom/V8SharedWorkerCustom.cpp
* WebCore.pro: Ditto.
* bindings/js/JSSharedWorkerCustom.cpp: Moved ScriptExecutionContext parameter to the beginning.
(WebCore::JSSharedWorkerConstructor::constructJSSharedWorker):
* bindings/v8/custom/V8SharedWorkerCustom.cpp: Removed.
* workers/SharedWorker.cpp: Moved ScriptExecutionContext parameter to the beginning.
(WebCore::SharedWorker::create): Ditto.
* workers/SharedWorker.h: Ditto.
* workers/SharedWorker.idl: Added [Constructor] IDL.

LayoutTests:

Added test cases for an undefined name and a null name on SharedWorker constructor.

* fast/workers/resources/shared-worker-name.js:
(test7.try.worker.port.onmessage):
(test7): A test case for a null name.
(test8.worker.port.onmessage):
(test8): Ditto.
(test9.try.worker.port.onmessage):
(test9): A test case for an undefined name.
(test10.worker.port.onmessage):
(test10): Ditto.
* fast/workers/shared-worker-constructor-expected.txt:
* fast/workers/shared-worker-constructor.html:
* fast/workers/shared-worker-name-expected.txt:
* platform/chromium-win/fast/workers/shared-worker-constructor-expected.txt: Updated SyntaxError with TypeError. The reason why ui_tests has been so far working without updating this error type is that shared-worker-constructor.html is marked FLAKY in chromium.

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

10 years ago[chromium] Attempt to fix Linux Clang compilation error.
yuzo@google.com [Wed, 19 Oct 2011 06:42:29 +0000 (06:42 +0000)]
[chromium] Attempt to fix Linux Clang compilation error.

Unreviewed.

* tests/CCLayerTreeHostTest.cpp:
(WTF::CCLayerTreeHostTestThreadOnly::runTestThreaded):
(WTF::TEST_F):

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

10 years agoRolled out last Windows build fix because it was wrong.
ggaren@apple.com [Wed, 19 Oct 2011 05:36:11 +0000 (05:36 +0000)]
Rolled out last Windows build fix because it was wrong.

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

10 years agoSource/WebCore: Implement NSProcessInfo::systemUptime on Mac Leopard.
jnd@chromium.org [Wed, 19 Oct 2011 04:32:28 +0000 (04:32 +0000)]
Source/WebCore: Implement NSProcessInfo::systemUptime on Mac Leopard.
https://bugs.webkit.org/show_bug.cgi?id=66577

Reviewed by Tony Chang.

* WebCore.gyp/WebCore.gyp:
* platform/chromium/ScrollAnimatorChromiumMac.mm:
(-[NSProcessInfo systemUptime]):

LayoutTests: Enable touch tests on Mac Leopard.
https://bugs.webkit.org/show_bug.cgi?id=66577

Reviewed by Tony Chang.

* platform/chromium/test_expectations.txt:

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

10 years agoRolled out last Windows build fix because it was wrong.
ggaren@apple.com [Wed, 19 Oct 2011 03:55:25 +0000 (03:55 +0000)]
Rolled out last Windows build fix because it was wrong.

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

10 years agoTry to fix part of the Windows build.
ggaren@apple.com [Wed, 19 Oct 2011 03:47:12 +0000 (03:47 +0000)]
Try to fix part of the Windows build.

Export!

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

10 years agoUnreviewed. New expected results after http://trac.webkit.org/changeset/97707.
ojan@chromium.org [Wed, 19 Oct 2011 03:10:12 +0000 (03:10 +0000)]
Unreviewed. New expected results after trac.webkit.org/changeset/97707.

* fast/table/height-percent-test-vertical-expected.txt: Renamed from LayoutTests/platform/gtk/fast/table/height-percent-test-vertical-expected.txt.
* platform/chromium-linux/fast/table/028-vertical-expected.png:
* platform/chromium-linux/fast/table/height-percent-test-vertical-expected.png:
* platform/chromium-mac-snowleopard/fast/writing-mode/fallback-orientation-expected.png: Renamed from LayoutTests/platform/chromium-mac/fast/writing-mode/fallback-orientation-expected.png.
* platform/chromium-win-vista/fast/writing-mode/fallback-orientation-expected.png: Removed.
* platform/chromium-win-xp/fast/writing-mode/fallback-orientation-expected.png:
* platform/chromium-win/fast/table/028-vertical-expected.png:
* platform/chromium-win/fast/table/028-vertical-expected.txt:
* platform/chromium-win/fast/table/height-percent-test-vertical-expected.png:
* platform/chromium-win/fast/writing-mode/fallback-orientation-expected.png:
* platform/chromium-win/fast/writing-mode/fallback-orientation-expected.txt:
* platform/chromium/test_expectations.txt:
* platform/qt/fast/table/height-percent-test-vertical-expected.txt: Removed.

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

10 years agoSwitched ropes from malloc memory to GC memory
ggaren@apple.com [Wed, 19 Oct 2011 02:54:29 +0000 (02:54 +0000)]
Switched ropes from malloc memory to GC memory
https://bugs.webkit.org/show_bug.cgi?id=70364

Reviewed by Gavin Barraclough.

~1% SunSpider speedup. Neutral elsewhere. Removes one cause for strings
having C++ destructors.

* heap/MarkStack.cpp:
(JSC::visitChildren): Call the JSString visitChildren function now,
since it's no longer a no-op.

* runtime/JSString.cpp:
(JSC::JSString::~JSString): Moved this destructor out of line because
it's called virtually, so there's no value to inlining.

(JSC::JSString::RopeBuilder::expand): Switched RopeBuilder to be a thin
initializing wrapper around JSString. JSString now represents ropes
directly, rather than relying on an underlying malloc object.

(JSC::JSString::visitChildren): Visit our rope fibers, since they're GC
objects now.

(JSC::JSString::resolveRope):
(JSC::JSString::resolveRopeSlowCase):
(JSC::JSString::outOfMemory): Updated for operating on JSStrings instead
of malloc objects.

(JSC::JSString::replaceCharacter): Removed optimizations for substringing
ropes and replacing subsections of ropes. We want to reimplement versions
of these optimizations in the future, but this patch already has good
performance without them.

* runtime/JSString.h:
(JSC::RopeBuilder::JSString):
(JSC::RopeBuilder::finishCreation):
(JSC::RopeBuilder::createNull):
(JSC::RopeBuilder::create):
(JSC::RopeBuilder::createHasOtherOwner):
(JSC::jsSingleCharacterString):
(JSC::jsSingleCharacterSubstring):
(JSC::jsNontrivialString):
(JSC::jsString):
(JSC::jsSubstring):
(JSC::jsOwnedString): Lots of mechanical changes here. The two important
things are: (1) The fibers in JSString::m_fibers are JSStrings now, not
malloc objects; (2) I simplified the JSString constructor interface to
only accept PassRefPtr<StringImpl>, instead of variations on that like
UString, reducing refcount churn.

* runtime/JSValue.h:
* runtime/JSValue.cpp:
(JSC::JSValue::toPrimitiveString): Updated this function to return a
JSString instead of a UString, since that's what clients want now.

* runtime/Operations.cpp:
(JSC::jsAddSlowCase):
* runtime/Operations.h:
(JSC::jsString):
* runtime/SmallStrings.cpp:
(JSC::SmallStrings::createEmptyString): Updated for interface changes above.

* runtime/StringConstructor.cpp:
(JSC::constructWithStringConstructor):
* runtime/StringObject.h:
(JSC::StringObject::create): Don't create a new JSString if we already
have a JSString.

* runtime/StringPrototype.cpp:
(JSC::stringProtoFuncConcat): Updated for interface changes above.

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

10 years agoErrrk, fix partial commit of r97825!
barraclough@apple.com [Wed, 19 Oct 2011 01:50:14 +0000 (01:50 +0000)]
Errrk, fix partial commit of r97825!

* runtime/DatePrototype.cpp:
(JSC::dateProtoFuncToISOString):

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

10 years agoDate.prototype.toISOString fails to throw exception
barraclough@apple.com [Wed, 19 Oct 2011 01:49:13 +0000 (01:49 +0000)]
Date.prototype.toISOString fails to throw exception
https://bugs.webkit.org/show_bug.cgi?id=70394

Reviewed by Sam Weinig.

Source/JavaScriptCore:

* runtime/DatePrototype.cpp:
(JSC::dateProtoFuncToISOString):
    - Should throw a range error if the internal value is not finite.

LayoutTests:

* fast/js/date-toisostring-expected.txt:
* fast/js/script-tests/date-toisostring.js:
(throwsRangeError):
    - fixed test case.

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

10 years agofix baseline for fast/history/history-back-twice-with-subframes-assert.html
dpranke@chromium.org [Wed, 19 Oct 2011 01:37:31 +0000 (01:37 +0000)]
fix baseline for fast/history/history-back-twice-with-subframes-assert.html
https://bugs.webkit.org/show_bug.cgi?id=70393

Reviewed by James Robinson.

the baseline added in r97821 appears to have had a couple of
extra lines in it.

* fast/history/history-back-twice-with-subframes-assert-expected.txt:

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

10 years agoTry to jostle the windows build back to life.
weinig@apple.com [Wed, 19 Oct 2011 01:37:04 +0000 (01:37 +0000)]
Try to jostle the windows build back to life.

* rendering/HitTestResult.h:

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

10 years agoUpdate chromium leopard cg baseline.
dpranke@chromium.org [Wed, 19 Oct 2011 01:16:28 +0000 (01:16 +0000)]
Update chromium leopard cg baseline.

Unreviewed, baseline update.

* platform/chromium-cg-mac-leopard/fullscreen/full-screen-remove-ancestor-after-expected.png: Added.

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

10 years agoAssertion failure when going back in page with navigated subframes
andersca@apple.com [Wed, 19 Oct 2011 00:52:24 +0000 (00:52 +0000)]
Assertion failure when going back in page with navigated subframes
https://bugs.webkit.org/show_bug.cgi?id=70389
<rdar://problem/8988444>

Reviewed by Darin Adler.

Source/WebCore:

Test: fast/history/history-back-twice-with-subframes-assert.html

If a single navigation ends up loading multiple frame, the first committed frame will
end up calling recursiveUpdateForCommit on the main frame which will null out the provisional item
for all frames on the page. This means that it can null out the provisional item for any frames
that are still yet to be committed which causes the aforementioned assertion failure.

Fix this by only nulling out the provisional history item (and saving/restoring the scroll position and
some other things) for frames that already contain the URL that the item requested. If a frame is being loaded,
it will null out its provisional history item when it's committed.

* loader/HistoryController.cpp:
(WebCore::HistoryController::recursiveUpdateForCommit):

LayoutTests:

* fast/history/history-back-twice-with-subframes-assert-expected.txt: Added.
* fast/history/history-back-twice-with-subframes-assert.html: Added.

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

10 years agowatchlist: Should try to run if it can and not throw on mistakes.
levin@chromium.org [Wed, 19 Oct 2011 00:47:26 +0000 (00:47 +0000)]
watchlist: Should try to run if it can and not throw on mistakes.
https://bugs.webkit.org/show_bug.cgi?id=70358

Reviewed by Adam Barth.

* Scripts/webkitpy/common/system/outputcapture.py: Add the ability to capture log output.
* Scripts/webkitpy/common/watchlist/watchlistloader_unittest.py: Adapt to the logging of errors.
* Scripts/webkitpy/common/watchlist/watchlistparser.py: Change to log problems and fix problems when found.
* Scripts/webkitpy/common/watchlist/watchlistparser_unittest.py: Adapt to the logging of errors.
* Scripts/webkitpy/common/watchlist/watchlistrule.py: Expose a way to remove instructions.
* Scripts/webkitpy/common/watchlist/watchlistrule_unittest.py: Test the new function.

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

10 years agoSource/WebKit2: Move uses of C SPI out of WKView.h and into WKViewPrivate.h
weinig@apple.com [Wed, 19 Oct 2011 00:40:30 +0000 (00:40 +0000)]
Source/WebKit2: Move uses of C SPI out of WKView.h and into WKViewPrivate.h
https://bugs.webkit.org/show_bug.cgi?id=70387

Reviewed by Dan Bernstein.

* UIProcess/API/mac/WKViewPrivate.h:
Move declarations here.

* UIProcess/API/mac/WKView.h:
* UIProcess/API/mac/WKView.mm:
(-[WKView _updateRemoteAccessibilityRegistration:]):
(-[WKView initWithFrame:]):
(-[WKView initWithFrame:contextRef:]):
(-[WKView _registerDraggedTypes]):
(-[WKView initWithFrame:contextRef:pageGroupRef:]):
(-[WKView pageRef]):
(-[WKView canChangeFrameLayout:]):
(-[WKView printOperationWithPrintInfo:forFrame:]):
(-[WKView setFrame:andScrollBy:]):
Move definitions to private category.

* UIProcess/API/mac/PDFViewController.mm:
* UIProcess/mac/WKFullScreenWindowController.mm:
* UIProcess/mac/WebInspectorProxyMac.mm:
Update to include WKViewPrivate.h where necessary.

Tools: Move uses of C-SPI out of WKView.h and into WKViewPrivate.h
https://bugs.webkit.org/show_bug.cgi?id=70387

Reviewed by Dan Bernstein.

* MiniBrowser/mac/BrowserWindowController.m:
* TestWebKitAPI/Tests/mac/DeviceScaleFactorOnBack.mm:
* TestWebKitAPI/mac/JavaScriptTestMac.mm:
* TestWebKitAPI/mac/PlatformWebViewMac.mm:
* TestWebKitAPI/mac/WebKitAgnosticTest.mm:
* WebKitTestRunner/mac/PlatformWebViewMac.mm:
Update to include WKViewPrivate.h where necessary.

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

10 years agoUnreviewed, rolling out r97765.
dpranke@chromium.org [Wed, 19 Oct 2011 00:15:30 +0000 (00:15 +0000)]
Unreviewed, rolling out r97765.
http://trac.webkit.org/changeset/97765
https://bugs.webkit.org/show_bug.cgi?id=70388

"broke webkit_unit_tests, http/tests/xmlhttprequest
/xmlhttprequest-test-send-flag.html on linux debug" (Requested
by dpranke on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2011-10-18

Source/WebCore:

* WebCore.gypi:
* loader/DocumentThreadableLoader.cpp:
(WebCore::DocumentThreadableLoader::create):
(WebCore::DocumentThreadableLoader::~DocumentThreadableLoader):
(WebCore::DocumentThreadableLoader::cancel):
(WebCore::DocumentThreadableLoader::setDefersLoading):
(WebCore::DocumentThreadableLoader::willSendRequest):
(WebCore::DocumentThreadableLoader::didSendData):
(WebCore::DocumentThreadableLoader::didReceiveResponse):
(WebCore::DocumentThreadableLoader::didReceiveData):
(WebCore::DocumentThreadableLoader::didReceiveCachedMetadata):
(WebCore::DocumentThreadableLoader::didFinishLoading):
(WebCore::DocumentThreadableLoader::didFail):
(WebCore::DocumentThreadableLoader::preflightSuccess):
(WebCore::DocumentThreadableLoader::loadRequest):
* loader/DocumentThreadableLoader.h:
* loader/cache/CachedRawResource.cpp:
(WebCore::CachedRawResource::CachedRawResource):
(WebCore::CachedRawResource::data):
* loader/cache/CachedRawResource.h:
* loader/cache/CachedResource.cpp:
(WebCore::CachedResource::CachedResource):
* loader/cache/CachedResource.h:
(WebCore::CachedResource::setRequestedFromNetworkingLayer):
* loader/cache/CachedResourceClient.h:
* loader/cache/CachedResourceRequest.cpp:
(WebCore::CachedResourceRequest::~CachedResourceRequest):
(WebCore::CachedResourceRequest::willSendRequest):
(WebCore::CachedResourceRequest::didFinishLoading):
(WebCore::CachedResourceRequest::didReceiveResponse):
(WebCore::CachedResourceRequest::didReceiveData):
* loader/cache/CachedResourceRequest.h:
* loader/chromium/CachedRawResourceChromium.cpp: Removed.
* loader/chromium/CachedResourceRequestChromium.cpp: Removed.
* loader/chromium/DocumentThreadableLoaderChromium.cpp:
(WebCore::DocumentThreadableLoader::didDownloadData):
* loader/icon/IconLoader.h:
* platform/network/BlobResourceHandle.cpp:
(WebCore::BlobResourceHandle::notifyResponse):

LayoutTests:

* http/tests/inspector/network/network-content-replacement-xhr-expected.txt:

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

10 years agoRename static put to putByIndex
mhahnenberg@apple.com [Tue, 18 Oct 2011 23:53:38 +0000 (23:53 +0000)]
Rename static put to putByIndex
https://bugs.webkit.org/show_bug.cgi?id=70281

Reviewed by Geoffrey Garen.

Source/JavaScriptCore:

Renaming versions of deleteProperty that use an unsigned as the property
name to "deletePropertyByIndex" in preparation for adding them to the
MethodTable, which requires unique names for each method.

* dfg/DFGOperations.cpp:
(JSC::DFG::putByVal):
* jit/JITStubs.cpp:
(JSC::DEFINE_STUB_FUNCTION):
* runtime/Arguments.cpp:
(JSC::Arguments::putVirtual):
(JSC::Arguments::putByIndex):
* runtime/Arguments.h:
* runtime/ArrayPrototype.cpp:
(JSC::arrayProtoFuncMap):
* runtime/JSArray.cpp:
(JSC::JSArray::put):
(JSC::JSArray::putVirtual):
(JSC::JSArray::putByIndex):
* runtime/JSArray.h:
* runtime/JSByteArray.cpp:
(JSC::JSByteArray::putVirtual):
(JSC::JSByteArray::putByIndex):
* runtime/JSByteArray.h:
* runtime/JSCell.cpp:
(JSC::JSCell::putVirtual):
(JSC::JSCell::putByIndex):
* runtime/JSCell.h:
* runtime/JSNotAnObject.cpp:
(JSC::JSNotAnObject::putVirtual):
(JSC::JSNotAnObject::putByIndex):
* runtime/JSNotAnObject.h:
* runtime/JSObject.cpp:
(JSC::JSObject::putVirtual):
(JSC::JSObject::putByIndex):
* runtime/JSObject.h:
* runtime/RegExpConstructor.cpp:
(JSC::RegExpMatchesArray::fillArrayInstance):
* runtime/RegExpMatchesArray.h:
(JSC::RegExpMatchesArray::putVirtual):
(JSC::RegExpMatchesArray::putByIndex):

Source/WebCore:

No new tests.

Renaming versions of deleteProperty that use an unsigned as the property
name to "deletePropertyByIndex" in preparation for adding them to the
MethodTable, which requires unique names for each method.

* bindings/scripts/CodeGeneratorJS.pm:
(GenerateHeader):
(GenerateImplementation):
* bridge/runtime_array.cpp:
(JSC::RuntimeArray::putVirtual):
(JSC::RuntimeArray::putByIndex):
* bridge/runtime_array.h:

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

10 years agoMark storage/domstorage/events/documentURI as a flaky crash
dpranke@chromium.org [Tue, 18 Oct 2011 23:53:33 +0000 (23:53 +0000)]
Mark storage/domstorage/events/documentURI as a flaky crash
https://bugs.webkit.org/show_bug.cgi?id=70386

Unreviewed, expectations change.

on chromium snowleopard.

* platform/chromium/test_expectations.txt:

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

10 years agoAdd a new chromium baseline for fast/multicol/vertical-rl/rule-style.html.
dpranke@chromium.org [Tue, 18 Oct 2011 23:44:12 +0000 (23:44 +0000)]
Add a new chromium baseline for fast/multicol/vertical-rl/rule-style.html.

Unreviewed, baseline update.

* platform/chromium-cg-mac-snowleopard/fast/multicol/vertical-rl/rule-style-expected.png: Added.
* platform/chromium-win/fast/multicol/vertical-rl/rule-style-expected.png: Added.
* platform/chromium-win/fast/multicol/vertical-rl/rule-style-expected.txt: Added.

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

10 years agoUnreviewed, expectations update.
dpranke@chromium.org [Tue, 18 Oct 2011 23:40:11 +0000 (23:40 +0000)]
Unreviewed, expectations update.

* platform/chromium/test_expectations.txt:

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

10 years agoFix the build.
weinig@apple.com [Tue, 18 Oct 2011 23:38:05 +0000 (23:38 +0000)]
Fix the build.

* rendering/HitTestResult.cpp:
(WebCore::HitTestResult::absolutePDFURL):
Fix typo.

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

10 years agoUnreviewed warning fix after https://bugs.webkit.org/show_bug.cgi?id=70269
alexis.menard@openbossa.org [Tue, 18 Oct 2011 23:36:36 +0000 (23:36 +0000)]
Unreviewed warning fix after https://bugs.webkit.org/show_bug.cgi?id=70269

* WebCore.pro:

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

10 years agoStop leaking internal names in WKView API header
weinig@apple.com [Tue, 18 Oct 2011 23:25:41 +0000 (23:25 +0000)]
Stop leaking internal names in WKView API header
https://bugs.webkit.org/show_bug.cgi?id=70380

Reviewed by Anders Carlsson.

Move _frameSizeUpdatesDisabledCount to WKViewData. Since we cannot change the size of
WKView due to the fragile Objective-C ABI, we must rename the existing variable unused. If
drop 32-bit support, we will be able to remove it.

* UIProcess/API/mac/WKView.h:
* UIProcess/API/mac/WKView.mm:
(-[WKView disableFrameSizeUpdates]):
(-[WKView enableFrameSizeUpdates]):
(-[WKView frameSizeUpdatesDisabled]):
* WebKit2.xcodeproj/project.pbxproj:

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

10 years ago Expose PDF information in hit test result
ap@apple.com [Tue, 18 Oct 2011 23:20:08 +0000 (23:20 +0000)]
    Expose PDF information in hit test result
        https://bugs.webkit.org/show_bug.cgi?id=70353

        Reviewed by Sam Weinig.

        * WebCore.exp.in:
        * rendering/HitTestResult.cpp:
        (WebCore::HitTestResult::absolutePDFURL):
        * rendering/HitTestResult.h:
        Added HitTestResult::absolutePDFURL(). It will contain a URL when over a PDF object or embed;
        not sure if we want this for PDFImageDocuments.

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

10 years ago[chromium] Preserve offscreen tiles instead of immediately recycling them
enne@google.com [Tue, 18 Oct 2011 23:19:15 +0000 (23:19 +0000)]
[chromium] Preserve offscreen tiles instead of immediately recycling them
https://bugs.webkit.org/show_bug.cgi?id=70352

Reviewed by James Robinson.

Tested by existing layout tests.

Prior to having a TextureManager class, tiled layers recycled their
textures internally to avoid reallocation. Unfortunately, it recycled
these tiles as soon as they went offscreen, even if they were still
valid. Instead, keep tiles around forever, letting the TextureManager
decide (via LRU) when to reclaim the underlying textures. This will
improve scrolling performance due to not needing to repaint tiles.

This change does lead to more texture reallocation for any textures
that get reclaimed, but the command buffer implementation already
pools and reuses texture ids, so it should not introduce additional
flush syncs. If there's any performance penalty, it'll be at the
driver level, but I'd prefer to measure that there's a problem before
prematurely optimizing and adding complexity to the TextureManager.

* platform/graphics/chromium/TiledLayerChromium.cpp:
(WebCore::TiledLayerChromium::cleanupResources):
(WebCore::TiledLayerChromium::createTile):
(WebCore::TiledLayerChromium::prepareToUpdate):
* platform/graphics/chromium/TiledLayerChromium.h:

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

10 years agoCrash in WebProcess at com.apple.AppKit: -[NSFilePromiseDragSource draggedImage:ended...
enrica@apple.com [Tue, 18 Oct 2011 23:17:26 +0000 (23:17 +0000)]
Crash in WebProcess at com.apple.AppKit: -[NSFilePromiseDragSource draggedImage:endedAt:operation:] + 101
https://bugs.webkit.org/show_bug.cgi?id=70340
<rdar://problem/9548174>

We have seen a number of crash reports for this issue that seem to occurr when
the user drags an image from a web page onto the desktop.
We have not been able to reproduce this crash, therefore this is a speculative fix,
but the theory is that the dragging source has been already released when draggedImage
is called.
This patch attempts a fix for the problem by retaining the dragging source when
NSFilePromiseDragSource is created.

Reviewed by Darin Adler.

* WebProcess/WebCoreSupport/mac/WebDragClientMac.mm:
(-[WKPasteboardFilePromiseOwner initWithSource:]):
(-[WKPasteboardFilePromiseOwner dealloc]):

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

10 years ago [WK2] Crash after storing credentials in Keychain
ap@apple.com [Tue, 18 Oct 2011 23:14:48 +0000 (23:14 +0000)]
    [WK2] Crash after storing credentials in Keychain
        https://bugs.webkit.org/show_bug.cgi?id=70376
        <rdar://problem/9647503>

        Reviewed by Dan Bernstein.

        * Shared/mac/SecKeychainItemResponseData.cpp:
        (WebKit::SecKeychainItemResponseData::encode):
        (WebKit::SecKeychainItemResponseData::decode):
        Actually send keychain item over, too. In shimSecKeychainItemCreateFromContent(), we are
        going to use it to override SecKeychainItemCreateFromContent(), and we cannot return noErr
        from this function without returning a non-null keychain item.

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

10 years agoThis revised fix doesn't break SL build.
weinig@apple.com [Tue, 18 Oct 2011 23:08:56 +0000 (23:08 +0000)]
This revised fix doesn't break SL build.
https://bugs.webkit.org/show_bug.cgi?id=70345

Reviewed and landed by Sam Weinig.

Patch by Damian Kaleta <dkaleta@apple.com> on 2011-10-18

* UIProcess/API/mac/WKView.mm:
(+[WKView hideWordDefinitionWindow]): Hides the definition window.
* UIProcess/API/mac/WKViewPrivate.h:

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

10 years ago[MutationObservers] Implement WebKitMutationObserver.observe for characterData changes
adamk@chromium.org [Tue, 18 Oct 2011 23:06:48 +0000 (23:06 +0000)]
[MutationObservers] Implement WebKitMutationObserver.observe for characterData changes
https://bugs.webkit.org/show_bug.cgi?id=68957

Reviewed by Ryosuke Niwa.

Source/WebCore:

Test: fast/mutation/observe-characterdata.html

* dom/CharacterData.cpp:
(WebCore::CharacterData::dispatchModifiedEvent):

LayoutTests:

* fast/mutation/observe-characterdata-expected.txt: Added.
* fast/mutation/observe-characterdata.html: Added.

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

10 years agoMove spellCheckerDocumentTag and handleCorrectionPanelResult from WKViewPrivate to...
weinig@apple.com [Tue, 18 Oct 2011 23:06:31 +0000 (23:06 +0000)]
Move spellCheckerDocumentTag and handleCorrectionPanelResult from WKViewPrivate to WKViewInternal
https://bugs.webkit.org/show_bug.cgi?id=70377

Reviewed by Anders Carlsson.

* UIProcess/API/mac/WKView.mm:
(-[WKView spellCheckerDocumentTag]):
(-[WKView handleCorrectionPanelResult:]):
Move definition of methods to (Internal) area.

* UIProcess/API/mac/WKViewInternal.h:
Add methods.

* UIProcess/API/mac/WKViewPrivate.h:
Remove methods.

* UIProcess/cf/WebBackForwardListCF.cpp:
Add missing newline.

* UIProcess/mac/CorrectionPanel.mm:
Change #include to WKViewInternal.h

* WebKit2.xcodeproj/project.pbxproj:
Sort some files.

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

10 years agoScroll animator tracing
commit-queue@webkit.org [Tue, 18 Oct 2011 23:05:10 +0000 (23:05 +0000)]
Scroll animator tracing
https://bugs.webkit.org/show_bug.cgi?id=70280

Patch by Scott Byer <scottbyer@chromium.org> on 2011-10-18
Reviewed by James Robinson.

No functional difference, no test needed.

* platform/ScrollAnimatorNone.cpp:
(WebCore::ScrollAnimatorNone::scroll):
(WebCore::ScrollAnimatorNone::animationTimerFired):

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

10 years ago[chromium] Remove USE(THREADED_COMPOSITING) from the tree, this is controlled by...
jamesr@google.com [Tue, 18 Oct 2011 23:02:02 +0000 (23:02 +0000)]
[chromium] Remove USE(THREADED_COMPOSITING) from the tree, this is controlled by a runtime setting
https://bugs.webkit.org/show_bug.cgi?id=70356

Reviewed by Kenneth Russell.

Source/WebCore:

* platform/graphics/chromium/cc/CCLayerTreeHost.h:

Source/WebKit/chromium:

* features.gypi:
* src/WebLayerTreeViewImpl.cpp:
(WebKit::WebLayerTreeViewImpl::scheduleComposite):
* src/WebLayerTreeViewImpl.h:
* src/WebViewImpl.h:

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

10 years agocleanup some FIXMEs in RenderFlexibleBox
ojan@chromium.org [Tue, 18 Oct 2011 22:58:38 +0000 (22:58 +0000)]
cleanup some FIXMEs in RenderFlexibleBox
https://bugs.webkit.org/show_bug.cgi?id=70374

Reviewed by Darin Adler.

No new tests. Existing tests cover all changes.

* rendering/RenderFlexibleBox.cpp:
(WebCore::RenderFlexibleBox::layoutBlock):
(WebCore::RenderFlexibleBox::flowAwareBorderAndPaddingLogicalHeight):
(WebCore::RenderFlexibleBox::flowAwareMarginLogicalHeightForChild):
* rendering/RenderFlexibleBox.h:
Removed the FIXME to implement computePreferredLogicalWidths. It doesn't
seem to be necessary, at least not with the current test cases.

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

10 years agoMove WKView helper files into a sub-group
weinig@apple.com [Tue, 18 Oct 2011 22:57:03 +0000 (22:57 +0000)]
Move WKView helper files into a sub-group
https://bugs.webkit.org/show_bug.cgi?id=70375

Reviewed by Darin Adler.

* WebKit2.xcodeproj/project.pbxproj:
Move some files around.

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

10 years agoRemove expectations for tests that are no longer failing.
dpranke@chromium.org [Tue, 18 Oct 2011 22:48:50 +0000 (22:48 +0000)]
Remove expectations for tests that are no longer failing.

Unreviewed, expectations change.

* platform/chromium/test_expectations.txt:

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

10 years ago<http://webkit.org/b/70365> Web Inspector: Some localizedStrings.js Updates
joepeck@webkit.org [Tue, 18 Oct 2011 22:47:21 +0000 (22:47 +0000)]
<webkit.org/b/70365> Web Inspector: Some localizedStrings.js Updates

Reviewed by Darin Adler.

Source/WebCore:

Fixed syntax issue, added missing strings, and removed a duplicate.

* English.lproj/localizedStrings.js:

Tools:

* Scripts/check-inspector-strings:
Updated to the new webkitpy way to determine checkout root.

* Scripts/webkitdirs.pm:
(copyInspectorFrontendFiles):
Made it so build-webkit --inspector-frontend copies over
the localizedStrings.js file as well as the frontend files.

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

10 years ago[EFL] DRT: Clear list of visited pages before each test.
commit-queue@webkit.org [Tue, 18 Oct 2011 22:45:35 +0000 (22:45 +0000)]
[EFL] DRT: Clear list of visited pages before each test.
https://bugs.webkit.org/show_bug.cgi?id=70355

Patch by Raphael Kubo da Costa <kubo@profusion.mobi> on 2011-10-18
Reviewed by Antonio Gomes.

Source/WebKit/efl:

Add ewk_history_clear as a way to clear the list of visited pages in a
Ewk_History, in a similar fashion to Qt's QWebHistory::clear().

* ewk/ewk_history.cpp:
(ewk_history_clear):
* ewk/ewk_history.h:

Tools:

r97596 uncovered a bug in the implementation: the list of visited
links was not cleared between tests, which caused tests such as
fast/repaint/outline-repaint-glitch.html to be flaky (the link color
changed depending on whether fast/repaint/border-repaint-glitch.html
was visited before or not).

Call ewk_history_clear to make sure the visited links list does not
persist between tests.

* DumpRenderTree/efl/DumpRenderTreeChrome.cpp:
(DumpRenderTreeChrome::resetDefaultsToConsistentValues):

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

10 years agofix unused-but-set-variable warnings in WebCore
tony@chromium.org [Tue, 18 Oct 2011 22:38:29 +0000 (22:38 +0000)]
fix unused-but-set-variable warnings in WebCore
https://bugs.webkit.org/show_bug.cgi?id=70344

Reviewed by Adam Barth.

This is a new compiler warning in gcc 4.6.

* bindings/v8/V8DOMWrapper.cpp:
(WebCore::V8DOMWrapper::isWrapperOfType):
* platform/graphics/chromium/ComplexTextControllerLinux.cpp:
(WebCore::ComplexTextController::normalizeSpacesAndMirrorChars):

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

10 years agoEnable out-of-thread compositing in WebKit compositor API
commit-queue@webkit.org [Tue, 18 Oct 2011 22:37:40 +0000 (22:37 +0000)]
Enable out-of-thread compositing in WebKit compositor API
https://bugs.webkit.org/show_bug.cgi?id=69850

Patch by Antoine Labour <piman@chromium.org> on 2011-10-18
Reviewed by James Robinson.

* src/WebLayerTreeView.cpp:
(WebKit::WebLayerTreeView::Settings::operator CCSettings):
(WebKit::WebLayerTreeView::composite):

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

10 years agoCertain column-rule styles render incorrectly in vertical writing modes
mitz@apple.com [Tue, 18 Oct 2011 22:14:00 +0000 (22:14 +0000)]
Certain column-rule styles render incorrectly in vertical writing modes
https://bugs.webkit.org/show_bug.cgi?id=70362

Reviewed by Darin Adler.

Source/WebCore:

Test: fast/multicol/vertical-rl/rule-style.html

* rendering/RenderBlock.cpp:
(WebCore::RenderBlock::paintColumnRules): Use the correct border sides for vertical writing modes.

LayoutTests:

* fast/multicol/vertical-rl/rule-style-expected.png: Added.
* fast/multicol/vertical-rl/rule-style-expected.txt: Added.
* fast/multicol/vertical-rl/rule-style.html: Added.

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

10 years agoFix broken binding tests by changing G_MAXUSHORT to G_MAXUINT in WebKitDOMTestObj...
annacc@chromium.org [Tue, 18 Oct 2011 22:07:23 +0000 (22:07 +0000)]
Fix broken binding tests by changing G_MAXUSHORT to G_MAXUINT in WebKitDOMTestObj.cpp.
https://bugs.webkit.org/show_bug.cgi?id=70267

Reviewed by Daniel Bates.

Tests: fixing WebKitDOMTestObj.cpp

* bindings/scripts/test/GObject/WebKitDOMTestObj.cpp:
(webkit_dom_test_obj_class_init): Change G_MAXUSHORT to G_MAXUINT

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

10 years agoUnreviewed, rolling out r97788.
inferno@chromium.org [Tue, 18 Oct 2011 22:04:53 +0000 (22:04 +0000)]
Unreviewed, rolling out r97788.
http://trac.webkit.org/changeset/97788
https://bugs.webkit.org/show_bug.cgi?id=70366

broke compile (Requested by inferno-sec on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2011-10-18

* UIProcess/API/mac/WKView.mm:
* UIProcess/API/mac/WKViewPrivate.h:

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

10 years agoArray.prototype methods missing exception checks
barraclough@apple.com [Tue, 18 Oct 2011 21:57:59 +0000 (21:57 +0000)]
Array.prototype methods missing exception checks
https://bugs.webkit.org/show_bug.cgi?id=70360

Reviewed by Geoff Garen.

Missing exception checks after calls to the static getProperty helper,
these may result in the wrong exception being thrown (or an ASSERT being hit,
as is currently the case running test-262).

Source/JavaScriptCore:

No performance impact.

* runtime/ArrayPrototype.cpp:
(JSC::arrayProtoFuncConcat):
(JSC::arrayProtoFuncReverse):
(JSC::arrayProtoFuncShift):
(JSC::arrayProtoFuncSlice):
(JSC::arrayProtoFuncSplice):
(JSC::arrayProtoFuncUnShift):
(JSC::arrayProtoFuncReduce):
(JSC::arrayProtoFuncReduceRight):
(JSC::arrayProtoFuncIndexOf):
(JSC::arrayProtoFuncLastIndexOf):

LayoutTests:

* fast/js/array-proto-func-property-getter-except-expected.txt:
* fast/js/script-tests/array-proto-func-property-getter-except.js:
    - The test results that change were only testing for an absence of a crash; the new results are correct
      (the first exception thrown should be the one returned).
* sputnik/Conformance/15_Native_Objects/15.5_String/15.5.4/15.5.4.10_String.prototype.match/S15.5.4.10_A1_T4-expected.txt:
* sputnik/Conformance/15_Native_Objects/15.5_String/15.5.4/15.5.4.10_String.prototype.match/S15.5.4.10_A1_T6-expected.txt:
* sputnik/Conformance/15_Native_Objects/15.5_String/15.5.4/15.5.4.10_String.prototype.match/S15.5.4.10_A1_T7-expected.txt:
* sputnik/Conformance/15_Native_Objects/15.5_String/15.5.4/15.5.4.10_String.prototype.match/S15.5.4.10_A1_T8-expected.txt:
* sputnik/Conformance/15_Native_Objects/15.5_String/15.5.4/15.5.4.10_String.prototype.match/S15.5.4.10_A1_T9-expected.txt:
* sputnik/Conformance/15_Native_Objects/15.5_String/15.5.4/15.5.4.11_String.prototype.replace/S15.5.4.11_A1_T11-expected.txt:
* sputnik/Conformance/15_Native_Objects/15.5_String/15.5.4/15.5.4.11_String.prototype.replace/S15.5.4.11_A1_T12-expected.txt:
* sputnik/Conformance/15_Native_Objects/15.5_String/15.5.4/15.5.4.12_String.prototype.search/S15.5.4.12_A1_T4-expected.txt:
    - Check in passing results.

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

10 years agoAdd missing baselines for fast/selectors/visited-descendant.
dpranke@chromium.org [Tue, 18 Oct 2011 21:07:01 +0000 (21:07 +0000)]
Add missing baselines for fast/selectors/visited-descendant.

Unreviewed, expectations change.

* platform/chromium-cg-mac-leopard/fast/selectors/visited-descendant-expected.png: Added.
* platform/chromium-cg-mac-snowleopard/fast/selectors/visited-descendant-expected.png: Added.
* platform/chromium-linux/fast/selectors/visited-descendant-expected.png: Added.
* platform/chromium-mac-leopard/fast/selectors/visited-descendant-expected.png: Added.
* platform/chromium-mac-snowleopard/fast/selectors/visited-descendant-expected.png: Added.
* platform/chromium-win/fast/selectors/visited-descendant-expected.png: Added.
* platform/chromium-win/fast/selectors/visited-descendant-expected.txt: Added.

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

10 years agoCorrected bad cut-and-paste in MutationObserver test description.
adamk@chromium.org [Tue, 18 Oct 2011 21:01:17 +0000 (21:01 +0000)]
Corrected bad cut-and-paste in MutationObserver test description.

Unreviewed.

* fast/mutation/observe-attributes-expected.txt:
* fast/mutation/observe-attributes.html:

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

10 years ago2011-10-18 Damian Kaleta <dkaleta@apple.com>
kdecker@apple.com [Tue, 18 Oct 2011 20:50:34 +0000 (20:50 +0000)]
2011-10-18  Damian Kaleta  <dkaleta@apple.com>

        Add WebKit API to hide the definition window
        https://bugs.webkit.org/show_bug.cgi?id=70345

        Reviewed by Anders Carlsson.

        * UIProcess/API/mac/WKView.mm:
        (-[WKView hideWordDefinitionWindow]): Hides the definition window.
        * UIProcess/API/mac/WKViewPrivate.h:

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

10 years agoCrash in RenderDeprecatedFlexibleBox::layoutHorizontalBox
inferno@chromium.org [Tue, 18 Oct 2011 20:42:23 +0000 (20:42 +0000)]
Crash in RenderDeprecatedFlexibleBox::layoutHorizontalBox
https://bugs.webkit.org/show_bug.cgi?id=70183

Patch by Julien Chaffraix <jchaffraix@webkit.org> on 2011-10-18
Reviewed by David Hyatt.

Source/WebCore:

Test: fast/flexbox/layoutHorizontal-crash.html

The deferred scroll information update logic was not updated when an object was destroy'ed.

The fix is very simple. As the deferred logic was only used with deprecated flexbox, it is a
pretty uncommon cases so let's optimize the new code for the common case.

* rendering/RenderBlock.cpp:
(WebCore::RenderBlock::willBeDestroyed):

LayoutTests:

* fast/flexbox/layoutHorizontal-crash-expected.txt: Added.
* fast/flexbox/layoutHorizontal-crash.html: Added.

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

10 years ago[MutationObservers] Clean up MutationObserver code in Node
adamk@chromium.org [Tue, 18 Oct 2011 20:09:02 +0000 (20:09 +0000)]
[MutationObservers] Clean up MutationObserver code in Node
https://bugs.webkit.org/show_bug.cgi?id=70349

Reviewed by Ryosuke Niwa.

Three small cleanups.

No new tests, no behavior change.

* dom/Node.cpp:
(WebCore::Node::clearRareData): Use indices instead of iterator in for loop; fix typo.
(WebCore::Node::registerMutationObserver): Inline single call to Node::ensureMutationObserverEntries.
* dom/Node.h: Make mutationObserverEntries private.

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

10 years ago[chromium] Fix shutdown race when posting main thread task to CCThreadProxy and enabl...
jamesr@google.com [Tue, 18 Oct 2011 19:55:24 +0000 (19:55 +0000)]
[chromium] Fix shutdown race when posting main thread task to CCThreadProxy and enable tests
https://bugs.webkit.org/show_bug.cgi?id=70161

Reviewed by David Levin.

Source/WebCore:

Adds a weak pointer mechanism to cancel main thread tasks posted to CCThreadProxy instances from the compositor
thread. Previously there was a race condition where main thread tasks could run even after the CCThreadProxy was
destroyed.

This race does not exist in the other direction because when tearing down a CCThreadProxy we first post a quit
task to the compositor thread and then suspend execution of the main thread until all compositor tasks for the
CCThreadProxy have been drained.

Covered by the now-enabled CCLayerTreeHostTest* unit tests.

* WebCore.gypi:
* platform/graphics/chromium/cc/CCScopedMainThreadProxy.h: Added.
(WebCore::CCScopedMainThreadProxy::create):
(WebCore::CCScopedMainThreadProxy::postTask):
(WebCore::CCScopedMainThreadProxy::shutdown):
(WebCore::CCScopedMainThreadProxy::CCScopedMainThreadProxy):
(WebCore::CCScopedMainThreadProxy::runTaskIfNotShutdown):
* platform/graphics/chromium/cc/CCThreadProxy.cpp:
(WebCore::CCThreadProxy::CCThreadProxy):
(WebCore::CCThreadProxy::~CCThreadProxy):
(WebCore::CCThreadProxy::createBeginFrameAndCommitTaskOnCCThread):
* platform/graphics/chromium/cc/CCThreadProxy.h:

Source/WebKit/chromium:

Enables the CCLayerTreeHostTest* tests by default. Most tests are run twice in a single thread and multiple
thread configuration. Some tests run only in the multiple thread configuration if they depend on the compositor
thread scheduling draws by itself.

* tests/CCLayerTreeHostTest.cpp:
(::CCLayerTreeHostTest::timeout):
(::CCLayerTreeHostTest::clearTimeout):
(::CCLayerTreeHostTest::CCLayerTreeHostTest):
(::CCLayerTreeHostTest::onEndTest):
(::CCLayerTreeHostTest::TimeoutTask::TimeoutTask):
(::CCLayerTreeHostTest::TimeoutTask::clearTest):
(::CCLayerTreeHostTest::TimeoutTask::~TimeoutTask):
(::CCLayerTreeHostTest::TimeoutTask::Run):
(::CCLayerTreeHostTest::runTest):
(::CCLayerTreeHostTest::doBeginTest):
(::CCLayerTreeHostTestThreadOnly::runTest):
(::CCLayerTreeHostTestSetNeedsRedraw::commitCompleteOnCCThread):

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

10 years agoimplement flex-flow:column
ojan@chromium.org [Tue, 18 Oct 2011 19:35:14 +0000 (19:35 +0000)]
implement flex-flow:column
https://bugs.webkit.org/show_bug.cgi?id=70082

Reviewed by David Hyatt.

Source/WebCore:

There's still a bug with a FIXME where we don't compute the right
size for the container of the flexbox in the presence of
orthogonal flows. That's the cause of all the failing cases
in the tests.

Tests: css3/flexbox/flex-flow-border.html
       css3/flexbox/flex-flow-margins.html
       css3/flexbox/flex-flow-orientations.html
       css3/flexbox/flex-flow-overflow.html
       css3/flexbox/flex-flow-padding.html
       css3/flexbox/flex-flow.html

* rendering/RenderFlexibleBox.cpp:
(WebCore::RenderFlexibleBox::layoutBlock):
(WebCore::RenderFlexibleBox::hasOrthogonalFlow):
(WebCore::RenderFlexibleBox::isColumnFlow):
(WebCore::RenderFlexibleBox::isHorizontalFlow):
(WebCore::RenderFlexibleBox::isLeftToRightFlow):
Use isHorizontalFlow and isLeftToRightFlow so that methods like
flowAwareBorderStart look exactly like borderStart with
isHorizontalWritingMode and isLeftToRightDirection replaced.

(WebCore::RenderFlexibleBox::setFlowAwareLogicalHeight):
(WebCore::RenderFlexibleBox::flowAwareLogicalHeightForChild):
(WebCore::RenderFlexibleBox::flowAwareLogicalWidthForChild):
(WebCore::RenderFlexibleBox::flowAwareLogicalHeight):
(WebCore::RenderFlexibleBox::flowAwareLogicalWidth):
(WebCore::RenderFlexibleBox::flowAwareContentLogicalHeight):
(WebCore::RenderFlexibleBox::flowAwareContentLogicalWidth):
(WebCore::RenderFlexibleBox::transformedWritingMode):
Transform the writing-mode based of the flex-flow and direction
values. That methods like flowAwareBorderBefore look exactly like
borderBefore, except it switches over a different value.

(WebCore::RenderFlexibleBox::flowAwareBorderStart):
(WebCore::RenderFlexibleBox::flowAwareBorderBefore):
(WebCore::RenderFlexibleBox::flowAwareBorderAfter):
(WebCore::RenderFlexibleBox::flowAwarePaddingStart):
(WebCore::RenderFlexibleBox::flowAwarePaddingBefore):
(WebCore::RenderFlexibleBox::flowAwarePaddingAfter):
(WebCore::RenderFlexibleBox::flowAwareMarginStartForChild):
(WebCore::RenderFlexibleBox::flowAwareMarginEndForChild):
(WebCore::RenderFlexibleBox::flowAwareMarginBeforeForChild):
(WebCore::RenderFlexibleBox::flowAwareMarginAfterForChild):
(WebCore::RenderFlexibleBox::setFlowAwareMarginStartForChild):
(WebCore::RenderFlexibleBox::setFlowAwareMarginEndForChild):
(WebCore::RenderFlexibleBox::layoutAndPlaceChildrenInlineDirection):
(WebCore::RenderFlexibleBox::alignChildrenBlockDirection):
* rendering/RenderFlexibleBox.h:

LayoutTests:

* css3/flexbox/flex-flow-border-expected.txt: Added.
* css3/flexbox/flex-flow-border.html: Added.
Failures are due to not handling orthogonal flows correctly.
* css3/flexbox/flex-flow-expected.txt: Added.
* css3/flexbox/flex-flow-margins-expected.txt: Added.
* css3/flexbox/flex-flow-margins.html: Added.
* css3/flexbox/flex-flow-orientations-expected.txt: Added.
* css3/flexbox/flex-flow-orientations.html: Added.
* css3/flexbox/flex-flow-overflow-expected.txt: Added.
* css3/flexbox/flex-flow-overflow.html: Added.
* css3/flexbox/flex-flow-padding-expected.txt: Added.
* css3/flexbox/flex-flow-padding.html: Added.
Failures are due to not handling orthogonal flows correctly.
* css3/flexbox/flex-flow.html: Added.
* css3/flexbox/resources/flexbox.js:
Added the ability to check offsets that include clientLeft/clientTop
and for checking clientWidth/clientHeight. This is neede dfor the border tests.

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

10 years agoSplit TestExpectationsEditor into its own file.
dglazkov@chromium.org [Tue, 18 Oct 2011 19:31:13 +0000 (19:31 +0000)]
Split TestExpectationsEditor into its own file.
https://bugs.webkit.org/show_bug.cgi?id=70348

Reviewed by Adam Barth.

* Scripts/webkitpy/layout_tests/controllers/test_expectations_editor.py: Copied from Tools/Scripts/webkitpy/layout_tests/models/test_expectations.py.
* Scripts/webkitpy/layout_tests/controllers/test_expectations_editor_unittest.py: Copied from Tools/Scripts/webkitpy/layout_tests/models/test_expectations_unittest.py.
* Scripts/webkitpy/layout_tests/models/test_expectations.py: Adjusted for change.
* Scripts/webkitpy/layout_tests/models/test_expectations_unittest.py: Ditto.
* Scripts/webkitpy/tool/servers/gardeningserver.py: Ditto.

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

10 years ago[Chromium] buildbot runs nrwt with --chromium instead of --chromium-cg-mac
rniwa@webkit.org [Tue, 18 Oct 2011 19:24:40 +0000 (19:24 +0000)]
[Chromium] buildbot runs nrwt with --chromium instead of --chromium-cg-mac
https://bugs.webkit.org/show_bug.cgi?id=70313

Reviewed by Adam Barth.

Fixed the bug by passing --platform --chromium-cg-mac in RunWebKitTests. We can't always add
this parameter because build-webkit and other scripts don't accept it.

* BuildSlaveSupport/build.webkit.org-config/master.cfg:

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

10 years ago[chromium] Tracking update rects in LayerChromium and CCLayerImpl.
commit-queue@webkit.org [Tue, 18 Oct 2011 19:10:17 +0000 (19:10 +0000)]
[chromium] Tracking update rects in LayerChromium and CCLayerImpl.
https://bugs.webkit.org/show_bug.cgi?id=69441

Patch by Shawn Singh <shawnsingh@chromium.org> on 2011-10-18
Reviewed by James Robinson.

This patch does not do much on its own, but is just part 1 of a
string of other patches which will use these rects for
visualization and scissoring.  The appropriate testing will be
associated with those patches.

* platform/graphics/chromium/Canvas2DLayerChromium.cpp:
(WebCore::Canvas2DLayerChromium::updateCompositorResources):
* platform/graphics/chromium/LayerChromium.cpp:
(WebCore::LayerChromium::pushPropertiesTo):
* platform/graphics/chromium/LayerChromium.h:
* platform/graphics/chromium/TiledLayerChromium.cpp:
(WebCore::TiledLayerChromium::cleanupResources):
(WebCore::TiledLayerChromium::updateCompositorResources):
(WebCore::TiledLayerChromium::pushPropertiesTo):
(WebCore::TiledLayerChromium::prepareToUpdate):
* platform/graphics/chromium/TiledLayerChromium.h:
* platform/graphics/chromium/VideoLayerChromium.cpp:
(WebCore::VideoLayerChromium::updateCompositorResources):
* platform/graphics/chromium/WebGLLayerChromium.cpp:
(WebCore::WebGLLayerChromium::updateCompositorResources):
* platform/graphics/chromium/cc/CCLayerImpl.h:
(WebCore::CCLayerImpl::updateRect):
(WebCore::CCLayerImpl::setUpdateRect):

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

10 years agoChange CodeGeneratorGObject.pm to use g_value_get_uint() instead of g_value_get_ushort()
annacc@chromium.org [Tue, 18 Oct 2011 18:18:36 +0000 (18:18 +0000)]
Change CodeGeneratorGObject.pm to use g_value_get_uint() instead of g_value_get_ushort()
for IDL attributes of type unsigned short.
https://bugs.webkit.org/show_bug.cgi?id=70267

Reviewed by Xan Lopez.

Tests: small change to WebKitDOMTestObj.cpp

* bindings/scripts/CodeGeneratorGObject.pm:
(GetGValueTypeName): convert unsigned short to uint instead of ushort
* bindings/scripts/test/GObject/WebKitDOMTestObj.cpp:
(webkit_dom_test_obj_set_property): expect g_value_get_uint instead of g_value_get_ushort
(webkit_dom_test_obj_class_init): expect G_MAXUSHORT instead of G_MAXUINT16

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

10 years agoWeb Inspector: ScriptsPanel's should use similar logic to decide whether it could...
vsevik@chromium.org [Tue, 18 Oct 2011 18:06:01 +0000 (18:06 +0000)]
Web Inspector: ScriptsPanel's should use similar logic to decide whether it could show an anchor location and actually showing it.
https://bugs.webkit.org/show_bug.cgi?id=70322

Reviewed by Pavel Feldman.

* inspector/front-end/ScriptsPanel.js:
(WebInspector.ScriptsPanel.prototype.canShowAnchorLocation):
(WebInspector.ScriptsPanel.prototype._showSourceLine):

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

10 years agoSubscribe myself to WebIDL changes
abarth@webkit.org [Tue, 18 Oct 2011 17:56:33 +0000 (17:56 +0000)]
Subscribe myself to WebIDL changes
https://bugs.webkit.org/show_bug.cgi?id=70341

Reviewed by David Levin.

I've also added some documentation and refactored my previous
subscriptions to make it eaiser for others to subscribe as well.

* Scripts/webkitpy/common/config/watchlist:

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

10 years agoRebaseline most remaining IMAGE diffs for chromium-mac (Skia)
caryclark@google.com [Tue, 18 Oct 2011 17:39:50 +0000 (17:39 +0000)]
Rebaseline most remaining IMAGE diffs for chromium-mac (Skia)
on SnowLeopard (10.6).

Unreviewed gardening.

* platform/chromium-mac-snowleopard/editing/deleting/delete-3959464-fix-expected.png:
* platform/chromium-mac-snowleopard/editing/deleting/delete-block-contents-003-expected.png:
* platform/chromium-mac-snowleopard/editing/deleting/delete-contiguous-ws-001-expected.png:
* platform/chromium-mac-snowleopard/editing/deleting/delete-tab-002-expected.png:
* platform/chromium-mac-snowleopard/editing/deleting/delete-tab-003-expected.png:
* platform/chromium-mac-snowleopard/editing/deleting/delete-to-select-table-expected.png:
* platform/chromium-mac-snowleopard/editing/deleting/delete-trailing-ws-002-expected.png:
* platform/chromium-mac-snowleopard/editing/deleting/delete-ws-fixup-001-expected.png:
* platform/chromium-mac-snowleopard/editing/deleting/delete-ws-fixup-002-expected.png:
* platform/chromium-mac-snowleopard/editing/execCommand/5569741-expected.png:
* platform/chromium-mac-snowleopard/editing/execCommand/insert-list-and-stitch-expected.png:
* platform/chromium-mac-snowleopard/editing/execCommand/insertHorizontalRule-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/5418891-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/editing-empty-divs-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/insert-br-004-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/insert-br-005-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/insert-br-007-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/insert-div-025-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/insert-paragraph-02-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/insert-paragraph-03-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/insert-tab-002-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/insert-tab-003-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/insert-text-with-newlines-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/paragraph-separator-01-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/paragraph-separator-02-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/paragraph-separator-03-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/return-key-with-selection-001-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/return-key-with-selection-002-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/return-key-with-selection-003-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/typing-002-expected.png:
* platform/chromium-mac-snowleopard/editing/inserting/typing-around-br-001-expected.png:
* platform/chromium-mac-snowleopard/editing/pasteboard/4242293-expected.png:
* platform/chromium-mac-snowleopard/editing/pasteboard/8145-2-expected.png:
* platform/chromium-mac-snowleopard/editing/pasteboard/cut-text-001-expected.png:
* platform/chromium-mac-snowleopard/editing/pasteboard/merge-after-delete-1-expected.png:
* platform/chromium-mac-snowleopard/editing/pasteboard/merge-after-delete-2-expected.png:
* platform/chromium-mac-snowleopard/editing/pasteboard/merge-after-delete-expected.png:
* platform/chromium-mac-snowleopard/editing/pasteboard/merge-end-blockquote-expected.png:
* platform/chromium-mac-snowleopard/editing/pasteboard/paste-text-019-expected.png:
* platform/chromium-mac-snowleopard/editing/selection/13804-expected.png:
* platform/chromium-mac-snowleopard/editing/selection/5234383-1-expected.png:
* platform/chromium-mac-snowleopard/editing/selection/5234383-2-expected.png:
* platform/chromium-mac-snowleopard/editing/selection/extend-by-word-001-expected.png:
* platform/chromium-mac-snowleopard/editing/selection/move-backwords-by-word-001-expected.png:
* platform/chromium-mac-snowleopard/editing/selection/move-by-character-001-expected.png:
* platform/chromium-mac-snowleopard/editing/selection/move-by-line-001-expected.png:
* platform/chromium-mac-snowleopard/editing/selection/move-by-word-001-expected.png:
* platform/chromium-mac-snowleopard/editing/spelling/spelling-expected.png:
* platform/chromium-mac-snowleopard/editing/style/block-styles-007-expected.png:
* platform/chromium-mac/css2.1/20110323/block-replaced-width-001-expected.png:
* platform/chromium-mac/css2.1/t1602-c43-center-00-d-ag-expected.png:
* platform/chromium-mac/editing/deleting/delete-3959464-fix-expected.png: Added.
* platform/chromium-mac/editing/deleting/delete-after-span-ws-001-expected.png:
* platform/chromium-mac/editing/deleting/delete-after-span-ws-002-expected.png:
* platform/chromium-mac/editing/deleting/delete-after-span-ws-003-expected.png:
* platform/chromium-mac/editing/deleting/delete-and-undo-expected.png: Added.
* platform/chromium-mac/editing/deleting/delete-block-contents-003-expected.png: Added.
* platform/chromium-mac/editing/deleting/delete-contiguous-ws-001-expected.png: Added.
* platform/chromium-mac/editing/deleting/delete-line-end-ws-001-expected.png:
* platform/chromium-mac/editing/deleting/delete-line-end-ws-002-expected.png:
* platform/chromium-mac/editing/deleting/delete-tab-002-expected.png: Added.
* platform/chromium-mac/editing/deleting/delete-tab-003-expected.png: Added.
* platform/chromium-mac/editing/deleting/delete-to-select-table-expected.png: Added.
* platform/chromium-mac/editing/deleting/delete-trailing-ws-002-expected.png: Added.
* platform/chromium-mac/editing/deleting/delete-ws-fixup-001-expected.png: Added.
* platform/chromium-mac/editing/deleting/delete-ws-fixup-002-expected.png: Added.
* platform/chromium-mac/editing/execCommand/5569741-expected.png: Added.
* platform/chromium-mac/editing/execCommand/insert-list-and-stitch-expected.png: Added.
* platform/chromium-mac/editing/execCommand/insertHorizontalRule-expected.png: Added.
* platform/chromium-mac/editing/inserting/5418891-expected.png: Added.
* platform/chromium-mac/editing/inserting/editable-inline-element-expected.png:
* platform/chromium-mac/editing/inserting/editing-empty-divs-expected.png: Added.
* platform/chromium-mac/editing/inserting/insert-br-004-expected.png: Added.
* platform/chromium-mac/editing/inserting/insert-br-005-expected.png: Added.
* platform/chromium-mac/editing/inserting/insert-br-007-expected.png: Added.
* platform/chromium-mac/editing/inserting/insert-div-025-expected.png: Added.
* platform/chromium-mac/editing/inserting/insert-paragraph-02-expected.png: Added.
* platform/chromium-mac/editing/inserting/insert-paragraph-03-expected.png: Added.
* platform/chromium-mac/editing/inserting/insert-tab-002-expected.png: Added.
* platform/chromium-mac/editing/inserting/insert-tab-003-expected.png: Added.
* platform/chromium-mac/editing/inserting/insert-text-with-newlines-expected.png: Added.
* platform/chromium-mac/editing/inserting/paragraph-separator-01-expected.png: Added.
* platform/chromium-mac/editing/inserting/paragraph-separator-02-expected.png: Added.
* platform/chromium-mac/editing/inserting/paragraph-separator-03-expected.png: Added.
* platform/chromium-mac/editing/inserting/return-key-with-selection-001-expected.png: Added.
* platform/chromium-mac/editing/inserting/return-key-with-selection-002-expected.png: Added.
* platform/chromium-mac/editing/inserting/return-key-with-selection-003-expected.png: Added.
* platform/chromium-mac/editing/inserting/typing-002-expected.png: Added.
* platform/chromium-mac/editing/inserting/typing-around-br-001-expected.png: Added.
* platform/chromium-mac/editing/pasteboard/4242293-expected.png: Added.
* platform/chromium-mac/editing/pasteboard/4641033-expected.png:
* platform/chromium-mac/editing/pasteboard/4944770-1-expected.png:
* platform/chromium-mac/editing/pasteboard/4944770-2-expected.png:
* platform/chromium-mac/editing/pasteboard/5478250-expected.png:
* platform/chromium-mac/editing/pasteboard/8145-2-expected.png: Added.
* platform/chromium-mac/editing/pasteboard/cut-text-001-expected.png: Added.
* platform/chromium-mac/editing/pasteboard/drop-text-without-selection-expected.png:
* platform/chromium-mac/editing/pasteboard/merge-after-delete-1-expected.png: Added.
* platform/chromium-mac/editing/pasteboard/merge-after-delete-2-expected.png: Added.
* platform/chromium-mac/editing/pasteboard/merge-after-delete-expected.png: Added.
* platform/chromium-mac/editing/pasteboard/merge-end-blockquote-expected.png: Added.
* platform/chromium-mac/editing/pasteboard/paste-text-019-expected.png: Added.
* platform/chromium-mac/editing/pasteboard/pasting-tabs-expected.png:
* platform/chromium-mac/editing/selection/13804-expected.png: Added.
* platform/chromium-mac/editing/selection/3690703-2-expected.png:
* platform/chromium-mac/editing/selection/3690703-expected.png:
* platform/chromium-mac/editing/selection/3690719-expected.png:
* platform/chromium-mac/editing/selection/4397952-expected.png:
* platform/chromium-mac/editing/selection/5234383-1-expected.png: Added.
* platform/chromium-mac/editing/selection/5234383-2-expected.png: Added.
* platform/chromium-mac/editing/selection/caret-before-select-expected.png:
* platform/chromium-mac/editing/selection/caret-rtl-2-expected.png:
* platform/chromium-mac/editing/selection/caret-rtl-2-left-expected.png:
* platform/chromium-mac/editing/selection/caret-rtl-expected.png:
* platform/chromium-mac/editing/selection/caret-rtl-right-expected.png:
* platform/chromium-mac/editing/selection/extend-by-word-001-expected.png: Added.
* platform/chromium-mac/editing/selection/extend-selection-bidi-expected.png:
* platform/chromium-mac/editing/selection/fake-doubleclick-expected.png:
* platform/chromium-mac/editing/selection/iframe-expected.png:
* platform/chromium-mac/editing/selection/move-backwords-by-word-001-expected.png: Added.
* platform/chromium-mac/editing/selection/move-by-character-001-expected.png: Added.
* platform/chromium-mac/editing/selection/move-by-line-001-expected.png: Added.
* platform/chromium-mac/editing/selection/move-by-word-001-expected.png: Added.
* platform/chromium-mac/editing/selection/move-past-trailing-space-expected.png:
* platform/chromium-mac/editing/selection/replaced-boundaries-3-expected.png:
* platform/chromium-mac/editing/selection/select-all-001-expected.png:
* platform/chromium-mac/editing/selection/select-all-002-expected.png:
* platform/chromium-mac/editing/selection/select-box-expected.png:
* platform/chromium-mac/editing/selection/select-element-paragraph-boundary-expected.png:
* platform/chromium-mac/editing/selection/selection-3748164-fix-expected.png:
* platform/chromium-mac/editing/selection/unrendered-001-expected.png:
* platform/chromium-mac/editing/selection/unrendered-003-expected.png:
* platform/chromium-mac/editing/selection/unrendered-004-expected.png:
* platform/chromium-mac/editing/selection/unrendered-005-expected.png:
* platform/chromium-mac/editing/spelling/spelling-expected.png: Added.
* platform/chromium-mac/editing/style/5046875-1-expected.png:
* platform/chromium-mac/editing/style/block-styles-007-expected.png: Added.
* platform/chromium-mac/fast/css-generated-content/table-parts-before-and-after-expected.png: Added.
* platform/chromium-mac/fast/css/background-shorthand-invalid-url-expected.png:
* platform/chromium-mac/fast/css/color-correction-expected.png:
* platform/chromium-mac/fast/css/color-correction-on-backgrounds-expected.png:
* platform/chromium-mac/fast/css/focus-ring-outline-width-expected.png:
* platform/chromium-mac/fast/css/outline-auto-location-expected.png:
* platform/chromium-mac/fast/css/shadow-multiple-expected.png:
* platform/chromium-mac/fast/dom/focus-contenteditable-expected.png:
* platform/chromium-mac/fast/events/autoscroll-expected.png:
* platform/chromium-mac/fast/events/context-no-deselect-expected.png:
* platform/chromium-mac/fast/forms/input-appearance-focus-expected.png:
* platform/chromium-mac/fast/forms/input-appearance-number-rtl-expected.png:
* platform/chromium-mac/fast/forms/input-appearance-selection-expected.png:
* platform/chromium-mac/fast/forms/input-double-click-selection-gap-bug-expected.png:
* platform/chromium-mac/fast/forms/input-placeholder-visibility-2-expected.png:
* platform/chromium-mac/fast/forms/input-text-double-click-expected.png:
* platform/chromium-mac/fast/forms/input-text-drag-down-expected.png:
* platform/chromium-mac/fast/forms/input-text-option-delete-expected.png:
* platform/chromium-mac/fast/forms/input-text-scroll-left-on-blur-expected.png:
* platform/chromium-mac/fast/forms/listbox-hit-test-zoomed-expected.png:
* platform/chromium-mac/fast/forms/plaintext-mode-2-expected.png:
* platform/chromium-mac/fast/forms/select-listbox-multiple-no-focusring-expected.png:
* platform/chromium-mac/fast/forms/tabbing-input-iframe-expected.png:
* platform/chromium-mac/fast/forms/textarea-scrollbar-expected.png:
* platform/chromium-mac/fast/forms/textarea-scrolled-type-expected.png:
* platform/chromium-mac/fast/forms/textfield-outline-expected.png:
* platform/chromium-mac/fast/forms/validation-message-appearance-expected.png:
* platform/chromium-mac/fast/frames/take-focus-from-iframe-expected.png:
* platform/chromium-mac/fast/inline/continuation-outlines-expected.png:
* platform/chromium-mac/fast/inline/continuation-outlines-with-layers-2-expected.png:
* platform/chromium-mac/fast/inline/continuation-outlines-with-layers-expected.png:
* platform/chromium-mac/fast/media/mq-min-pixel-ratio-expected.png:
* platform/chromium-mac/fast/multicol/shadow-breaking-expected.png:
* platform/chromium-mac/fast/multicol/vertical-lr/float-avoidance-expected.png:
* platform/chromium-mac/fast/multicol/vertical-rl/float-avoidance-expected.png:
* platform/chromium-mac/fast/reflections/reflection-direction-expected.png:
* platform/chromium-mac/fast/reflections/reflection-masks-expected.png:
* platform/chromium-mac/fast/reflections/reflection-masks-opacity-expected.png:
* platform/chromium-mac/fast/repaint/japanese-rl-selection-repaint-in-regions-expected.png:
* platform/chromium-mac/fast/repaint/shadow-multiple-horizontal-expected.png:
* platform/chromium-mac/fast/repaint/shadow-multiple-strict-horizontal-expected.png:
* platform/chromium-mac/fast/repaint/shadow-multiple-strict-vertical-expected.png:
* platform/chromium-mac/fast/repaint/shadow-multiple-vertical-expected.png:
* platform/chromium-mac/fast/text/international/rtl-caret-expected.png:
* platform/chromium-mac/fast/text/justify-nbsp-expected.png:
* platform/chromium-mac/fast/text/justify-padding-distribution-expected.png:
* platform/chromium-mac/fast/transforms/transformed-caret-expected.png:
* platform/chromium-mac/scrollbars/listbox-scrollbar-combinations-expected.png:
* platform/chromium-mac/scrollbars/overflow-scrollbar-combinations-expected.png:
* platform/chromium-mac/svg/clip-path/clip-path-transform-1-expected.png: Added.
* platform/chromium-mac/svg/clip-path/clip-path-transform-2-expected.png: Added.
* platform/chromium-mac/svg/css/group-with-shadow-expected.png:
* platform/chromium-mac/svg/custom/alignment-baseline-modes-expected.png:
* platform/chromium-mac/svg/custom/createImageElement-expected.png:
* platform/chromium-mac/svg/custom/dominant-baseline-modes-expected.png:
* platform/chromium-mac/svg/custom/external-paintserver-reference-expected.png: Added.
* platform/chromium-mac/svg/custom/feComponentTransfer-Discrete-expected.png:
* platform/chromium-mac/svg/custom/feComponentTransfer-Gamma-expected.png:
* platform/chromium-mac/svg/custom/feComponentTransfer-Linear-expected.png:
* platform/chromium-mac/svg/custom/feComponentTransfer-Table-expected.png:
* platform/chromium-mac/svg/custom/focus-ring-expected.png:
* platform/chromium-mac/svg/custom/font-face-cascade-order-expected.png:
* platform/chromium-mac/svg/custom/font-face-simple-expected.png:
* platform/chromium-mac/svg/custom/glyph-selection-lang-attribute-expected.png:
* platform/chromium-mac/svg/custom/group-opacity-expected.png:
* platform/chromium-mac/svg/custom/image-parent-translation-expected.png:
* platform/chromium-mac/svg/custom/image-small-width-height-expected.png:
* platform/chromium-mac/svg/custom/image-with-transform-clip-filter-expected.png:
* platform/chromium-mac/svg/custom/invalid-css-expected.png:
* platform/chromium-mac/svg/custom/js-late-gradient-and-object-creation-expected.png:
* platform/chromium-mac/svg/custom/js-update-image-and-display-expected.png:
* platform/chromium-mac/svg/custom/js-update-image-and-display2-expected.png:
* platform/chromium-mac/svg/custom/js-update-image-and-display3-expected.png:
* platform/chromium-mac/svg/custom/js-update-image-expected.png:
* platform/chromium-mac/svg/custom/linking-base-external-reference-expected.png: Added.
* platform/chromium-mac/svg/custom/pointer-events-image-expected.png:
* platform/chromium-mac/svg/custom/text-image-opacity-expected.png:
* platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-height-attr-expected.png:
* platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-preserveAspectRatio-attr-expected.png:
* platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-width-attr-expected.png:
* platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-x-attr-expected.png:
* platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-y-attr-expected.png:
* platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-height-prop-expected.png:
* platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-preserveAspectRatio-prop-expected.png:
* platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-width-prop-expected.png:
* platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-x-prop-expected.png:
* platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-y-prop-expected.png:
* platform/chromium-mac/svg/filters/filter-source-position-expected.png:
* platform/chromium-mac/svg/transforms/text-with-pattern-inside-transformed-html-expected.png:
* platform/chromium-mac/svg/transforms/text-with-pattern-with-svg-transform-expected.png:
* platform/chromium-mac/transforms/2d/compound-transforms-vs-containers-expected.png:

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

10 years agoAdd myself to the newfangled watchlist plumbing.
dglazkov@chromium.org [Tue, 18 Oct 2011 17:37:03 +0000 (17:37 +0000)]
Add myself to the newfangled watchlist plumbing.
https://bugs.webkit.org/show_bug.cgi?id=70338

Reviewed by David Levin.

* Scripts/webkitpy/common/config/watchlist: Added sheepishly.

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

10 years ago2011-10-18 Eric Carlson <eric.carlson@apple.com>
eric.carlson@apple.com [Tue, 18 Oct 2011 17:34:27 +0000 (17:34 +0000)]
2011-10-18  Eric Carlson  <eric.carlson@apple.com>

        Use the new cached cue loader
        https://bugs.webkit.org/show_bug.cgi?id=70269

        Reviewed by Antti Koivisto.

        No new tests, no functionality changed.

        * CMakeLists.txt: Remove CueParser.cpp/h and CueParserPrivate.h.
        * GNUmakefile.list.am: Ditto.
        * WebCore.gypi: Ditto.
        * WebCore.xcodeproj/project.pbxproj: Ditto.

        * html/HTMLMediaElement.cpp:
        (WebCore::HTMLMediaElement::loadTextTracks):
        (WebCore::HTMLMediaElement::textTrackReadyStateChanged): New, not implemented yet.
        (WebCore::HTMLMediaElement::textTrackModeChanged): Ditto.
        (WebCore::HTMLMediaElement::textTrackCreated): Ditto.
        (WebCore::HTMLMediaElement::textTrackAddCues): Ditto.
        (WebCore::HTMLMediaElement::textTrackRemoveCues): Ditto.
        (WebCore::HTMLMediaElement::textTrackAddCue): Ditto.
        (WebCore::HTMLMediaElement::textTrackRemoveCue): Ditto.
        * html/HTMLMediaElement.h:

        * html/HTMLTrackElement.cpp:
        (WebCore::HTMLTrackElement::load): Update for interface change
        (WebCore::HTMLTrackElement::textTrackLoadingCompleted): New, dispatch 'load' or 'error' event.
        * html/HTMLTrackElement.h:

        * html/LoadableTextTrack.cpp:
        (WebCore::LoadableTextTrack::LoadableTextTrack): Add TextTrackClient and TextTrackLoadingClient
            parameters.
        (WebCore::LoadableTextTrack::load): Make url parameter KURL instead of String. Create the
            cue loader, trigger the load.
        (WebCore::LoadableTextTrack::newCuesAvailable): New callback from the loader. Not implemented yet.
        (WebCore::LoadableTextTrack::cueLoadingStarted): Ditto.
        (WebCore::LoadableTextTrack::cueLoadingCompleted): Ditto.
        * html/LoadableTextTrack.h:

        * html/MutableTextTrack.cpp:
        (WebCore::MutableTextTrack::MutableTextTrack): Add TextTrackClient parameter.
        * html/MutableTextTrack.h:
        (WebCore::MutableTextTrack::create):

        * html/TextTrack.cpp:
        (WebCore::TextTrack::TextTrack): Add TextTrackClient parameter.
        (WebCore::TextTrack::~TextTrack): Tell the client to remove all cues, forget the client.
        (WebCore::TextTrack::setReadyState): Tell client about ready state change.
        (WebCore::TextTrack::setMode): Tell client about mode change.
        * html/TextTrack.h:
        (WebCore::TextTrackClient::textTrackAddCues): New client interface.
        (WebCore::TextTrackClient::textTrackRemoveCues): Ditto.
        (WebCore::TextTrackClient::textTrackAddCue): Ditto.
        (WebCore::TextTrackClient::textTrackRemoveCue): Ditto.
        (WebCore::TextTrack::create):
        (WebCore::TextTrack::trackType): New.
        (WebCore::TextTrack::client): Ditto.
        (WebCore::TextTrack::setClient): Ditto.

        * html/track/CueParser.cpp: Removed.
        * html/track/CueParser.h: Removed.
        * html/track/CueParserPrivate.h: Removed.

        * html/track/WebVTTParser.cpp:
        (WebCore::WebVTTParser::fileIdentifierMaximumLength): New, return length of WebVTT file
            magic number.
        (WebCore::WebVTTParser::hasRequiredFileIdentifier): Update to use constants instead of hard
            coded lengths.
        (WebCore::WebVTTParser::WebVTTParser): Initialize "m_client" in the initilization list
            instead of setting it in the constructor.
        (WebCore::WebVTTParser::getNewCues): Renamed from fetchParsedCues.
        * html/track/WebVTTParser.h:
        (WebCore::WebVTTParser::create): Take a WebVTTParserClient* instead of a CueParserPrivateClient*.

        * loader/CueLoader.cpp:
        (WebCore::CueLoader::CueLoader):
        (WebCore::CueLoader::~CueLoader): Remove client.
        (WebCore::CueLoader::cueLoadTimerFired): Tell client about new clues and/or that the load
            has finished.
        (WebCore::CueLoader::processNewCueData): Process newly loaded data.
        (WebCore::CueLoader::didReceiveData): Ditto.
        (WebCore::CueLoader::notifyFinished): Do final processing as necessary, prime the timer
            to call client.
        (WebCore::CueLoader::load): Create resource loader, initiate loading.
        (WebCore::CueLoader::newCuesParsed): Prime timer to tell client about newly parsed cues.
        (WebCore::CueLoader::getNewCues): Get new cues from parser.
        * loader/CueLoader.h:
        (WebCore::CueLoader::create):

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

10 years agoFix a unit test in TestFailures.
dglazkov@chromium.org [Tue, 18 Oct 2011 17:33:24 +0000 (17:33 +0000)]
Fix a unit test in TestFailures.
https://bugs.webkit.org/show_bug.cgi?id=70337

Reviewed by Adam Barth.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/NewBugForm_unittests.js: Turns out,
    the value of a method property should always be lowercase.

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

10 years agoAlways enable ENABLE(XPATH)
abarth@webkit.org [Tue, 18 Oct 2011 17:21:28 +0000 (17:21 +0000)]
Always enable ENABLE(XPATH)
https://bugs.webkit.org/show_bug.cgi?id=70217

Reviewed by Eric Seidel.

.:

* Source/cmake/OptionsEfl.cmake:
* Source/cmake/OptionsWinCE.cmake:
* Source/cmakeconfig.h.cmake:
* configure.ac:

Source/JavaScriptCore:

* Configurations/FeatureDefines.xcconfig:

Source/WebCore:

As discussed on webkit-dev, we're reducing the complexity of WebKit by
removing unneeded configuration options.  ENABLE(XPATH) is enabled on
every port, so we are removing the ability to disable the feature.

* CMakeLists.txt:
* Configurations/FeatureDefines.xcconfig:
* GNUmakefile.am:
* GNUmakefile.list.am:
* UseJSC.cmake:
* WebCore.pro:
* bindings/js/JSCustomXPathNSResolver.cpp:
* bindings/js/JSCustomXPathNSResolver.h:
* bindings/js/JSDOMBinding.cpp:
(WebCore::setDOMException):
* bindings/js/JSExceptionBase.cpp:
(WebCore::toExceptionBase):
* bindings/js/JSXPathResultCustom.cpp:
* bindings/objc/DOMCustomXPathNSResolver.h:
* bindings/objc/DOMCustomXPathNSResolver.mm:
* bindings/objc/DOMInternal.h:
* bindings/objc/DOMUtility.mm:
(JSC::createDOMWrapper):
* bindings/objc/DOMXPath.mm:
(kit):
* bindings/objc/PublicDOMInterfaces.h:
* bindings/v8/V8DOMWrapper.cpp:
(WebCore::V8DOMWrapper::getXPathNSResolver):
* bindings/v8/V8DOMWrapper.h:
* bindings/v8/V8Proxy.cpp:
(WebCore::V8Proxy::setDOMException):
* bindings/v8/V8Proxy.h:
* bindings/v8/custom/V8CustomXPathNSResolver.cpp:
* bindings/v8/custom/V8CustomXPathNSResolver.h:
* bindings/v8/custom/V8DocumentCustom.cpp:
(WebCore::V8Document::evaluateCallback):
* dom/Document.cpp:
* dom/Document.h:
* dom/Document.idl:
* dom/ExceptionCode.cpp:
(WebCore::getExceptionCodeDescription):
* dom/ExceptionCode.h:
* features.pri:
* inspector/InspectorDOMAgent.cpp:
* page/DOMWindow.idl:
* xml/NativeXPathNSResolver.cpp:
* xml/NativeXPathNSResolver.h:
* xml/XPathEvaluator.cpp:
* xml/XPathEvaluator.h:
* xml/XPathEvaluator.idl:
* xml/XPathException.h:
* xml/XPathException.idl:
* xml/XPathExpression.cpp:
* xml/XPathExpression.h:
* xml/XPathExpression.idl:
* xml/XPathExpressionNode.cpp:
* xml/XPathExpressionNode.h:
* xml/XPathFunctions.cpp:
* xml/XPathFunctions.h:
* xml/XPathGrammar.y:
* xml/XPathNSResolver.cpp:
* xml/XPathNSResolver.h:
* xml/XPathNSResolver.idl:
* xml/XPathNodeSet.cpp:
* xml/XPathNodeSet.h:
* xml/XPathParser.cpp:
* xml/XPathParser.h:
* xml/XPathPath.cpp:
* xml/XPathPath.h:
* xml/XPathPredicate.cpp:
* xml/XPathPredicate.h:
* xml/XPathResult.cpp:
* xml/XPathResult.h:
* xml/XPathResult.idl:
* xml/XPathStep.cpp:
* xml/XPathStep.h:
* xml/XPathUtil.cpp:
* xml/XPathUtil.h:
* xml/XPathValue.cpp:
* xml/XPathValue.h:
* xml/XPathVariableReference.cpp:
* xml/XPathVariableReference.h:

Source/WebKit/chromium:

* features.gypi:

Source/WebKit/mac:

* Configurations/FeatureDefines.xcconfig:

Source/WebKit/win:

* Interfaces/DOMWindow.idl:

Source/WebKit2:

* Configurations/FeatureDefines.xcconfig:

Tools:

* Scripts/build-webkit:

WebKitLibraries:

* win/tools/vsprops/FeatureDefines.vsprops:
* win/tools/vsprops/FeatureDefinesCairo.vsprops:

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

10 years agogarden-o-matic should not report flaking in expected failures as failures.
dglazkov@chromium.org [Tue, 18 Oct 2011 17:17:50 +0000 (17:17 +0000)]
garden-o-matic should not report flaking in expected failures as failures.
https://bugs.webkit.org/show_bug.cgi?id=70279

Reviewed by Adam Barth.

On the bots, flakes that are already expected to fail aren't reported as failures (they keep the bot green).
For example, if TEXT failure is expected, a TEXT TIMEOUT result will not cause the redness. We should do the
same for garden-o-matic.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/results.js: Spuriously changed to use Array iterators rather than jQuery APIs,
    rewrote the logic for determining expected or unexpected failures as a central analysis tool.
* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/results_unittests.js: Added unit tests.

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

10 years agoAX: buttons of number type <input> controls are not fully accessible
cfleizach@apple.com [Tue, 18 Oct 2011 17:06:52 +0000 (17:06 +0000)]
AX: buttons of number type <input> controls are not fully accessible
https://bugs.webkit.org/show_bug.cgi?id=70241

Source/WebCore:

Staged commit, part 6.

Reviewed by Darin Adler.

Test: platform/mac/accessibility/html5-input-number.html

* accessibility/AccessibilityRenderObject.cpp:
(WebCore::AccessibilityRenderObject::addImageMapChildren):
(WebCore::AccessibilityRenderObject::addTextFieldChildren):
(WebCore::AccessibilityRenderObject::addChildren):
* accessibility/AccessibilityRenderObject.h:

LayoutTests:

Reviewed by Darin Adler.

* platform/mac/accessibility/html5-input-number-expected.txt: Added.
* platform/mac/accessibility/html5-input-number.html: Added.

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

10 years agoIndexed arguments on the Arguments object should be enumerable.
barraclough@apple.com [Tue, 18 Oct 2011 17:04:32 +0000 (17:04 +0000)]
Indexed arguments on the Arguments object should be enumerable.
https://bugs.webkit.org/show_bug.cgi?id=70302

Reviewed by Sam Weinig.

See ECMA-262 5.1 chapter 10.6 step 11b.
This is visible through a number of means, including Object.keys, Object.getOwnPropertyDescriptor, and operator in.

* fast/js/kde/function-expected.txt:
* fast/js/kde/script-tests/function.js:
(foo2):
    - This tests using 'in'.
* fast/js/arguments-expected.txt:
* fast/js/script-tests/arguments.js:
(descriptor):
    - This tests using 'Object.getOwnPropertyDescriptor'.
* ietestcenter/Javascript/15.2.3.14-3-4-expected.txt:
    - This tests using 'Object.keys'.

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

10 years agoIndexed arguments on the Arguments object should be enumerable.
barraclough@apple.com [Tue, 18 Oct 2011 17:03:55 +0000 (17:03 +0000)]
Indexed arguments on the Arguments object should be enumerable.
https://bugs.webkit.org/show_bug.cgi?id=70302

Reviewed by Sam Weinig.

See ECMA-262 5.1 chapter 10.6 step 11b.
This is visible through a number of means, including Object.keys, Object.getOwnPropertyDescriptor, and operator in.

* runtime/Arguments.cpp:
(JSC::Arguments::getOwnPropertyDescriptor):
    - The 'enumerable' property should be true for indexed arguments.
(JSC::Arguments::getOwnPropertyNames):
    - Don't guard the adding of indexed properties with 'IncludeDontEnumProperties'.

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

10 years ago[Qt] Merge QML plugin for WebKit2 with the original WebKit1 plugin
alexis.menard@openbossa.org [Tue, 18 Oct 2011 16:59:44 +0000 (16:59 +0000)]
[Qt] Merge QML plugin for WebKit2 with the original WebKit1 plugin

.:

We don't need a sepearate plugin, as we no longer expose the WK2
APIs under QtWebKit.experimental, but now use QtWebKit. The version
was also changed to 3.0 instead of 5.0, as the original plugin was
versioned 1.0, and bumping to 3.0 fits the scheme used by QtWebKit releases.

Patch by Tor Arne Vestbø <tor.arne.vestbo@nokia.com> on 2011-10-18
Reviewed by Andreas Kling.

* Source/WebKit.pro:

Source/WebKit/qt:

We don't need a separate plugin, as we no longer expose the WK2
APIs under QtWebKit.experimental, but now use QtWebKit. The version
was also changed to 3.0 instead of 5.0, as the original plugin was
versioned 1.0, and bumping to 3.0 fits the scheme used by QtWebKit releases.

Patch by Tor Arne Vestbø <tor.arne.vestbo@nokia.com> on 2011-10-18
Reviewed by Andreas Kling.

* declarative/declarative.pro:
* declarative/plugin.cpp:
(WebKitQmlPlugin::registerTypes):

Source/WebKit2:

We don't need a separate plugin, as we no longer expose the WK2
APIs under QtWebKit.experimental, but now use QtWebKit. The version
was also changed to 3.0 instead of 5.0, as the original plugin was
versioned 1.0, and bumping to 3.0 fits the scheme used by QtWebKit releases.

Patch by Tor Arne Vestbø <tor.arne.vestbo@nokia.com> on 2011-10-18
Reviewed by Andreas Kling.

* UIProcess/API/qt/qmlplugin/plugin.cpp: Removed.
* UIProcess/API/qt/qmlplugin/qmldir: Removed.
* UIProcess/API/qt/qmlplugin/qmlplugin.pro: Removed.
* UIProcess/API/qt/tests/qmltests/DesktopWebView/tst_linkHovered.qml:
* UIProcess/API/qt/tests/qmltests/DesktopWebView/tst_loadFail.qml:
* UIProcess/API/qt/tests/qmltests/DesktopWebView/tst_loadProgress.qml:
* UIProcess/API/qt/tests/qmltests/DesktopWebView/tst_loadProgressSignal.qml:
* UIProcess/API/qt/tests/qmltests/DesktopWebView/tst_navigationPolicyForUrl.qml:
* UIProcess/API/qt/tests/qmltests/DesktopWebView/tst_preferences.qml:
* UIProcess/API/qt/tests/qmltests/DesktopWebView/tst_properties.qml:
* UIProcess/API/qt/tests/qmltests/TouchWebView/tst_load.qml:
* UIProcess/API/qt/tests/qmltests/TouchWebView/tst_loadFail.qml:
* UIProcess/API/qt/tests/qmltests/TouchWebView/tst_loadNegativeSizeView.qml:
* UIProcess/API/qt/tests/qmltests/TouchWebView/tst_loadProgress.qml:
* UIProcess/API/qt/tests/qmltests/TouchWebView/tst_loadProgressSignal.qml:
* UIProcess/API/qt/tests/qmltests/TouchWebView/tst_loadZeroSizeView.qml:
* UIProcess/API/qt/tests/qmltests/TouchWebView/tst_properties.qml:

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

10 years agoSource/WebCore: Make DocumentThreadableLoader a CachedResourceClient.
japhet@chromium.org [Tue, 18 Oct 2011 16:31:32 +0000 (16:31 +0000)]
Source/WebCore: Make DocumentThreadableLoader a CachedResourceClient.
This will allow us to cache ThreadableLoaderClients as
appropriate in a later patch.
https://bugs.webkit.org/show_bug.cgi?id=61225

Reviewed by Antti Koivisto.

No new tests, no functionality change intended.

* WebCore.gypi:
* loader/DocumentThreadableLoader.cpp:
(WebCore::DocumentThreadableLoader::notifyFinished): CachedResourceClients don't send separate callbacks
    for didFail() and didFinishLoading(), so we need to differentiate here.
(WebCore::DocumentThreadableLoader::loadRequest): Call CachedResourceLoader::requestRawResource()
    instead of ResourceLoadScheduler::scheduleSubresourceLoad().
* loader/DocumentThreadableLoader.h:
* loader/cache/CachedRawResource.cpp:
(WebCore::CachedRawResource::data): Handle the complexity of some ThreadableLoaderClients setting
    DoNotBufferData in ResoureLoaderOptions. If the DataBufferingPolicy is BufferData, then the
    'data' input parameter will contain all data received so far, and we need to determine the
    incremental data to forward to the clients. If the policy is DoNotBufferData, 'data' is just
    the incremental data.
* loader/cache/CachedRawResource.h:
* loader/cache/CachedResource.cpp:
* loader/cache/CachedResource.h:
(WebCore::CachedResource::preloadResult):
(WebCore::CachedResource::didSendData):
(WebCore::CachedResource::didDownloadData):
(WebCore::CachedResource::setLoadFinishTime):
(WebCore::CachedResource::loadFinishTime):
* loader/cache/CachedResourceClient.h: Add several new callbacks needed by DocumentThreadableLoader.
* loader/cache/CachedResourceRequest.cpp:
(WebCore::CachedResourceRequest::didReceiveResponse): CachedResource::setResponse() can now cancel
    the request, so protect appropriately.
* loader/cache/CachedResourceRequest.h:
* loader/chromium/CachedRawResourceChromium.cpp: Added (plumb didDownloadData() to DocumentThreadableLoader).
* loader/chromium/CachedResourceRequestChromium.cpp: Added (plumb didDownloadData() to DocumentThreadableLoader).
* loader/chromium/DocumentThreadableLoaderChromium.cpp:
* platform/network/BlobResourceHandle.cpp:
(WebCore::BlobResourceHandle::notifyResponse): Protect before setting response, since it might kill us.

LayoutTests: Test update for https://bugs.webkit.org/show_bug.cgi?id=61225.
Due to changes in the implementation of DocumentThreadableLoader (and
therefore XHR), this test will have empty (rather than null) content
if we go over the length limit.

Reviewed by Antti Koivisto.

* http/tests/inspector/network/network-content-replacement-xhr-expected.txt:

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

10 years agoUnreviewed, rolling out r97752.
commit-queue@webkit.org [Tue, 18 Oct 2011 16:15:48 +0000 (16:15 +0000)]
Unreviewed, rolling out r97752.
http://trac.webkit.org/changeset/97752
https://bugs.webkit.org/show_bug.cgi?id=70332

Caused assertion failures in MediaList::setParentStyleSheet()
on the apple bots. (Requested by kling on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2011-10-18

* WebCore.xcodeproj/project.pbxproj:
* bindings/js/JSDOMBinding.h:
* css/CSSImportRule.cpp:
(WebCore::CSSImportRule::CSSImportRule):
(WebCore::CSSImportRule::~CSSImportRule):
* css/CSSMediaRule.cpp:
(WebCore::CSSMediaRule::CSSMediaRule):
(WebCore::CSSMediaRule::~CSSMediaRule):
* css/CSSMediaRule.h:
* css/MediaList.cpp:
(WebCore::MediaList::MediaList):
(WebCore::MediaList::notifyChanged):
* css/MediaList.h:
* css/StyleSheet.cpp:
(WebCore::StyleSheet::~StyleSheet):
(WebCore::StyleSheet::setMedia):

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

10 years agoChange TextTrack to use the word DISABLED instead of OFF for mode
annacc@chromium.org [Tue, 18 Oct 2011 16:07:06 +0000 (16:07 +0000)]
Change TextTrack to use the word DISABLED instead of OFF for mode
https://bugs.webkit.org/show_bug.cgi?id=70268

Reviewed by Daniel Bates.

No new tests. No new functionality.

* html/TextTrack.cpp:
(WebCore::TextTrack::setMode): use Disabled instead of Off
* html/TextTrack.h: update Mode enum to use Disabled instead of Off

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

10 years ago[Qt][WK2] ASSERT when displaying context menu in input field
commit-queue@webkit.org [Tue, 18 Oct 2011 15:51:32 +0000 (15:51 +0000)]
[Qt][WK2] ASSERT when displaying context menu in input field
https://bugs.webkit.org/show_bug.cgi?id=69859

Patch by Dinu Jacob <dinu.jacob@nokia.com> on 2011-10-18
Reviewed by Simon Hausmann.

When creating a submenu in the context menu, the submenu of type
QWidget is statically cast to QObject before calling setParent on
it. QObject::setParent asserts if the object is a QWidget. So, call
setParent of QWidget itself.

* UIProcess/qt/WebContextMenuProxyQt.cpp:
(WebKit::WebContextMenuProxyQt::createContextMenu):

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

10 years agoAX: buttons of number type <input> controls are not fully accessible
cfleizach@apple.com [Tue, 18 Oct 2011 15:50:25 +0000 (15:50 +0000)]
AX: buttons of number type <input> controls are not fully accessible
https://bugs.webkit.org/show_bug.cgi?id=70241

Staged commit, part 5.

Reviewed by Darin Adler.

* accessibility/AXObjectCache.cpp:
(WebCore::AXObjectCache::getOrCreate):
* accessibility/AccessibilityRenderObject.cpp:
(WebCore::AccessibilityRenderObject::boundingBoxRect):

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

10 years agoMove default stylesheet loading to a separate function
antti@apple.com [Tue, 18 Oct 2011 15:48:47 +0000 (15:48 +0000)]
Move default stylesheet loading to a separate function
https://bugs.webkit.org/show_bug.cgi?id=70331

Reviewed by Andreas Kling.

- Move default stylesheet loading out from CSSStyleSelector::styleForElement()
- Rename e -> element
- Remove pointless reffing of the placeholder style.

* css/CSSStyleSelector.cpp:
(WebCore::ensureDefaultStyleSheetsForElement):
(WebCore::isAtShadowBoundary):
(WebCore::CSSStyleSelector::styleForElement):

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

10 years agoAX: buttons of number type <input> controls are not fully accessible
cfleizach@apple.com [Tue, 18 Oct 2011 15:43:50 +0000 (15:43 +0000)]
AX: buttons of number type <input> controls are not fully accessible
https://bugs.webkit.org/show_bug.cgi?id=70241

Staged commit part 4.

Reviewed by Darin Adler.

* accessibility/AccessibilityMockObject.h:
(WebCore::AccessibilityMockObject::isMockObject):
(WebCore::AccessibilityMockObject::detachFromParent):
* accessibility/AccessibilityObject.cpp:
(WebCore::AccessibilityObject::elementAccessibilityHitTest):
* accessibility/mac/WebAccessibilityObjectWrapper.mm:
(-[WebAccessibilityObjectWrapper accessibilityActionNames]):
(-[WebAccessibilityObjectWrapper accessibilityAttributeNames]):
(-[WebAccessibilityObjectWrapper subrole]):
(-[WebAccessibilityObjectWrapper accessibilityAttributeValue:]):

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

10 years agoAX: buttons of number type <input> controls are not fully accessible
cfleizach@apple.com [Tue, 18 Oct 2011 15:38:29 +0000 (15:38 +0000)]
AX: buttons of number type <input> controls are not fully accessible
https://bugs.webkit.org/show_bug.cgi?id=70241

Staged commit part 3. Committing the new spin button classes.

Reviewed by Darin Adler.

* CMakeLists.txt:
* GNUmakefile.list.am:
* WebCore.gypi:
* WebCore.pro:
* WebCore.vcproj/WebCore.vcproj:
* WebCore.xcodeproj/project.pbxproj:
* accessibility/AccessibilitySpinButton.cpp: Added.
(WebCore::AccessibilitySpinButton::create):
(WebCore::AccessibilitySpinButton::AccessibilitySpinButton):
(WebCore::AccessibilitySpinButton::~AccessibilitySpinButton):
(WebCore::AccessibilitySpinButton::incrementButton):
(WebCore::AccessibilitySpinButton::decrementButton):
(WebCore::AccessibilitySpinButton::elementRect):
(WebCore::AccessibilitySpinButton::addChildren):
(WebCore::AccessibilitySpinButton::step):
(WebCore::AccessibilitySpinButtonPart::AccessibilitySpinButtonPart):
(WebCore::AccessibilitySpinButtonPart::create):
(WebCore::AccessibilitySpinButtonPart::elementRect):
(WebCore::AccessibilitySpinButtonPart::press):
* accessibility/AccessibilitySpinButton.h: Added.
(WebCore::AccessibilitySpinButton::setSpinButtonElement):
(WebCore::AccessibilitySpinButton::roleValue):
(WebCore::AccessibilitySpinButton::accessibilityIsIgnored):
(WebCore::AccessibilitySpinButton::isSpinButton):
(WebCore::AccessibilitySpinButtonPart::~AccessibilitySpinButtonPart):
(WebCore::AccessibilitySpinButtonPart::isIncrementor):
(WebCore::AccessibilitySpinButtonPart::setIsIncrementor):
(WebCore::AccessibilitySpinButtonPart::roleValue):
(WebCore::AccessibilitySpinButtonPart::isSpinButtonPart):
(WebCore::AccessibilitySpinButtonPart::accessibilityIsIgnored):
(WebCore::toAccessibilitySpinButton):
(WebCore::toAccessibilitySpinButtonPart):

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

10 years agoAX: buttons of number type <input> controls are not fully accessible
cfleizach@apple.com [Tue, 18 Oct 2011 15:28:11 +0000 (15:28 +0000)]
AX: buttons of number type <input> controls are not fully accessible
https://bugs.webkit.org/show_bug.cgi?id=70241

../WebCore:

Phase 2 of the staged commit.

Reviewed by Darin Adler.

* accessibility/AccessibilityObject.cpp:
(WebCore::AccessibilityObject::boundingBoxForQuads):
* accessibility/AccessibilityObject.h:
(WebCore::AccessibilityObject::isSpinButton):
(WebCore::AccessibilityObject::isSpinButtonPart):
(WebCore::AccessibilityObject::isMockObject):

../WebKit/chromium:

Reviewed by Darin Adler.

* public/WebAccessibilityRole.h:
* src/AssertMatchingEnums.cpp:

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

10 years agoFix distcheck.
kov@webkit.org [Tue, 18 Oct 2011 15:26:47 +0000 (15:26 +0000)]
Fix distcheck.

Patch by Gustavo Noronha Silva <gns@gnome.org> on 2011-10-18

Source/JavaScriptCore
* GNUmakefile.list.am: fix a typo and add a missing header to the
list.

Source/WebKit/gtk/po
* GNUmakefile.am: create po directory when it's not there yet both
when updating the pot and the po files.

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

10 years agoAX: buttons of number type <input> controls are not fully accessible
cfleizach@apple.com [Tue, 18 Oct 2011 15:22:09 +0000 (15:22 +0000)]
AX: buttons of number type <input> controls are not fully accessible
https://bugs.webkit.org/show_bug.cgi?id=70241

Expose the spin button object to the AX hierarchy. Since there is no distinct
render object that backs this object, a mock object needs to be created for the entire
spin button and the actual increment and decrement buttons inside.

This will be a staged commit since the first time around Windows layout tests started crashing for an undeterminable reason.

Reviewed by Darin Adler.

* html/shadow/TextControlInnerElements.cpp:
(WebCore::SpinButtonElement::step):
(WebCore::SpinButtonElement::repeatingTimerFired):
* html/shadow/TextControlInnerElements.h:

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