⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Timeline



Oct 30, 2010:

11:43 PM Changeset in webkit [70989] by pfeldman@chromium.org
  • 6 edits in trunk/WebCore

2010-10-30 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: new resources and network panel usability improvements.
https://bugs.webkit.org/show_bug.cgi?id=48680

By default, navigation clears network log, added 'preserve log' button,
that prevents log from being cleared.

Storing expanded state of all group items, restoring last selected item,
fall back to main frame's main resource by default.
Do not list XHRs and redirects in resources.

  • English.lproj/localizedStrings.js:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::didLoadResourceFromMemoryCache): (WebCore::InspectorController::identifierForInitialRequest):
  • inspector/front-end/NetworkPanel.js: (WebInspector.NetworkPanel.prototype.get statusBarItems): (WebInspector.NetworkPanel.prototype._positionSummaryBar): (WebInspector.NetworkPanel.prototype._updateFilter): (WebInspector.NetworkPanel.prototype._createStatusbarButtons): (WebInspector.NetworkPanel.prototype._onPreserveLogClicked): (WebInspector.NetworkPanel.prototype.reset): (WebInspector.NetworkPanel.prototype.refreshResource):
  • inspector/front-end/ResourceManager.js: (WebInspector.ResourceManager.prototype.identifierForInitialRequest): (WebInspector.ResourceManager.prototype.willSendRequest): (WebInspector.ResourceManager.prototype.didLoadResourceFromMemoryCache): (WebInspector.ResourceManager.prototype.setOverrideContent): (WebInspector.ResourceManager.prototype.didCreateWebSocket):
  • inspector/front-end/StoragePanel.js: (WebInspector.StoragePanel): (WebInspector.StoragePanel.prototype.show): (WebInspector.StoragePanel.prototype._initDefaultSelection): (WebInspector.StoragePanel.prototype.reset): (WebInspector.StoragePanel.prototype.addResourceToFrame): (WebInspector.StoragePanel.prototype.refreshResource): (WebInspector.StoragePanel.prototype.showDatabase): (WebInspector.BaseStorageTreeElement.prototype.onselect): (WebInspector.StorageCategoryTreeElement): (WebInspector.StorageCategoryTreeElement.prototype.get itemURL): (WebInspector.StorageCategoryTreeElement.prototype.onselect): (WebInspector.StorageCategoryTreeElement.prototype.onattach): (WebInspector.StorageCategoryTreeElement.prototype.onexpand): (WebInspector.StorageCategoryTreeElement.prototype.oncollapse): (WebInspector.FrameTreeElement.prototype.get itemURL): (WebInspector.FrameTreeElement.prototype.onselect): (WebInspector.FrameResourceTreeElement.prototype.get itemURL): (WebInspector.FrameResourceTreeElement.prototype.onselect): (WebInspector.DatabaseTreeElement.prototype.get itemURL): (WebInspector.DatabaseTreeElement.prototype.onselect): (WebInspector.DatabaseTableTreeElement.prototype.get itemURL): (WebInspector.DatabaseTableTreeElement.prototype.onselect): (WebInspector.DOMStorageTreeElement.prototype.get itemURL): (WebInspector.DOMStorageTreeElement.prototype.onselect): (WebInspector.CookieTreeElement.prototype.get itemURL): (WebInspector.CookieTreeElement.prototype.onselect): (WebInspector.ApplicationCacheTreeElement.prototype.get itemURL): (WebInspector.ApplicationCacheTreeElement.prototype.onselect):
4:08 PM Changeset in webkit [70988] by tonikitoo@webkit.org
  • 1 edit in trunk/WebKit/win/Interfaces/WebKit.idl

2010-10-30 Antonio Gomes <agomes@rim.com>

Unreviewed Windows bots build fix (take I).

Touching WebKit.idl to try forcing a clean build.

  • Interfaces/WebKit.idl:
2:00 PM Changeset in webkit [70987] by wsiegrist@apple.com
  • 1 edit in trunk/WebKitTools/BuildSlaveSupport/build.webkit.org-config/config.json

William Siegrist <wsiegrist@apple.com>

Change trigger name to match builder name change in r70871.

12:31 PM Changeset in webkit [70986] by Patrick Gansterer
  • 4 edits in trunk

2010-10-30 Patrick Gansterer <Patrick Gansterer>

Unreviewed, build fix after r70846.

  • platform/graphics/wince/ImageWinCE.cpp: (WebCore::RGBA32Buffer::asNewNativeImage):

2010-10-30 Patrick Gansterer <Patrick Gansterer>

Unreviewed, build fix after r70574.

  • WebCoreSupport/FrameLoaderClientWinCE.cpp: (WebKit::FrameLoaderClientWinCE::transferLoadingResourceFromPage):
11:48 AM Changeset in webkit [70985] by Dimitri Glazkov
  • 19 edits
    4 deletes in trunk

2010-10-30 Dimitri Glazkov <Dimitri Glazkov>

Unreviewed, rolling out r70984.
http://trac.webkit.org/changeset/70984
https://bugs.webkit.org/show_bug.cgi?id=46015

Made media/audio-delete-while-slider-thumb-clicked.html crash.

  • fast/events/shadow-boundary-crossing.html:

2010-10-30 Dimitri Glazkov <Dimitri Glazkov>

Unreviewed, rolling out r70984.
http://trac.webkit.org/changeset/70984
https://bugs.webkit.org/show_bug.cgi?id=46015

Made media/audio-delete-while-slider-thumb-clicked.html crash.

  • Android.mk:
  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.xcodeproj/project.pbxproj:
  • dom/ContainerNode.cpp: (WebCore::notifyChildInserted):
  • dom/EventContext.cpp: Removed.
  • dom/EventContext.h: Removed.
  • dom/Node.cpp: (WebCore::Node::markAncestorsWithChildNeedsStyleRecalc): (WebCore::Node::createRendererIfNeeded): (WebCore::Node::eventParentNode): (WebCore::Node::enclosingLinkEventParentOrSelf): (WebCore::eventTargetAsSVGElementInstance): (WebCore::eventTargetRespectingSVGTargetRules): (WebCore::Node::eventAncestors): (WebCore::Node::dispatchGenericEvent):
  • dom/Node.h:
  • dom/Text.cpp: (WebCore::Text::createRenderer):
  • dom/WindowEventContext.cpp: Removed.
  • dom/WindowEventContext.h: Removed.
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::getEventListenersForNode):
  • inspector/InspectorInstrumentation.cpp: (WebCore::eventHasListeners): (WebCore::InspectorInstrumentation::willDispatchEventImpl):
  • inspector/InspectorInstrumentation.h: (WebCore::InspectorInstrumentation::willDispatchEvent):
  • page/EventHandler.cpp: (WebCore::EventHandler::updateMouseEventTargetNode):
  • svg/SVGElement.cpp: (WebCore::SVGElement::eventParentNode):
  • svg/SVGElement.h:
10:32 AM Changeset in webkit [70984] by Dimitri Glazkov
  • 19 edits
    4 adds in trunk

2010-10-30 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Darin Adler.

Implement shadow DOM-aware event targeting and introduce EventContext to track the context of each event dispatch.
https://bugs.webkit.org/show_bug.cgi?id=46015

Tuned the test to better reflect its point: the event should indeed fire (it used to be swallowed),
but its target should be a non-shadow node.

  • fast/events/shadow-boundary-crossing.html: Modified the test.

2010-10-30 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Darin Adler.

Implement shadow DOM-aware event targeting and introduce EventContext to track the context of each event dispatch.
https://bugs.webkit.org/show_bug.cgi?id=46015

This patch adds the notion of EventContext (and a very similar-acting WindowEventContext, specifically
for DOMWindow), an abstraction that carries information around dispatching an event for any given Node.

This abstraction is necessary to ensure that events, fired from shadow DOM nodes are properly retargeted to
appear as if they are coming from their host, thus never exposing the shadow DOM nodes to the world outside.

  • Android.mk: Added EventContext, WindowEventContext files.
  • CMakeLists.txt: Ditto.
  • GNUmakefile.am: Ditto.
  • WebCore.gypi: Ditto.
  • WebCore.pro: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • dom/ContainerNode.cpp: (WebCore::notifyChildInserted): Changed to be shadow DOM-aware.
  • dom/EventContext.cpp: Added.
  • dom/EventContext.h: Added.
  • dom/Node.cpp: (WebCore::Node::markAncestorsWithChildNeedsStyleRecalc): Changed to be shadow DOM-aware. (WebCore::Node::createRendererIfNeeded): Ditto. (WebCore::Node::parentOrHostNode): Added new helper method. (WebCore::Node::enclosingLinkEventParentOrSelf): Changed to be shadow DOM-aware. (WebCore::eventTargetRespectingSVGTargetRules): Collapsed two helper methods into one. (WebCore::Node::eventAncestors): Refactored to collect a vector of EventContexts. (WebCore::Node::topEventContext): Added. (WebCore::eventHasListeners): Changed to use EventContexts. (WebCore::Node::dispatchGenericEvent): Ditto.
  • dom/Node.h: Removed eventParentNode that's no longer needed, added parentOrHostNode decl,

and changed signature of eventAncestors to use EventContexts.

  • dom/Text.cpp: (WebCore::Text::createRenderer): Changed to be shadow DOM-aware.
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::getEventListenersForNode): Changed to use EventContexts.
  • page/EventHandler.cpp: (WebCore::EventHandler::updateMouseEventTargetNode): Removed code that's no longer necessary.
  • svg/SVGElement.cpp: Removed eventParentNode that's no longer needed.
  • svg/SVGElement.h: Ditto.
  • dom/WindowEventContext.cpp: Added.
  • dom/WindowEventContext.h: Added.
8:17 AM Changeset in webkit [70983] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2010-10-30 Xan Lopez <xlopez@igalia.com>

Reviewed by Martin Robinson.

[GTK] Use new width for height APIs in GTK+ 3.x
https://bugs.webkit.org/show_bug.cgi?id=48709

Use the new width for height APIs in GTK+ 3.x, since size-request
is deprecated.

For now we just return the same value for preferred and minimum
width/height, which should match the 2.x behavior. Probably we
could do something smarter for the minimum values.

  • webkit/webkitwebview.cpp: (webkit_web_view_get_preferred_width): return our preferred width. (webkit_web_view_get_preferred_height): return our preferred height. (webkit_web_view_class_init): hook the new default handlers.
7:51 AM Changeset in webkit [70982] by andreas.kling@nokia.com
  • 2 edits in trunk/WebKit/qt

2010-10-30 Andreas Kling <kling@webkit.org>

Reviewed by Antonio Gomes.

[Qt] IM: Don't manipulate selection via RenderTextControl
https://bugs.webkit.org/show_bug.cgi?id=48700

Go directly through the Frame's SelectionController instead.

No new tests, this is covered by tst_QWebPage::inputMethods.

  • Api/qwebpage.cpp: (QWebPagePrivate::inputMethodEvent): (QWebPage::inputMethodQuery):
7:49 AM Changeset in webkit [70981] by andreas.kling@nokia.com
  • 2 edits in trunk/WebKit2

2010-10-30 Andreas Kling <kling@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt][WK2] Crash in drawUpdateChunkIntoBackingStore
https://bugs.webkit.org/show_bug.cgi?id=48707

UpdateChunk images have to be created using the QImage constructor
that takes a bytesPerLine (stride) value, or the data length won't
match UpdateChunk::size().

  • Shared/qt/UpdateChunk.cpp: (WebKit::UpdateChunk::createImage):
7:48 AM Changeset in webkit [70980] by andreas.kling@nokia.com
  • 3 edits in trunk/WebKitTools

2010-10-30 Andreas Kling <kling@webkit.org>

Reviewed by Antonio Gomes.

[Qt] QtTestBrowser: Switching view type moves the embedded inspector
https://bugs.webkit.org/show_bug.cgi?id=48705

Reinsert the embedded inspector into the splitter after changing
between QWebView/QGraphicsWebView.

  • QtTestBrowser/launcherwindow.cpp: (LauncherWindow::init): (LauncherWindow::initializeView):
  • QtTestBrowser/webinspector.h: (WebInspector::WebInspector):
6:23 AM Changeset in webkit [70979] by Nikolas Zimmermann
  • 50 edits
    10 adds in trunk

2010-10-30 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Rob Buis.

Convert SVGAnimatedNumber/SVGAnimatedNumberList to the new SVGAnimatedPropertyTearOff concept
https://bugs.webkit.org/show_bug.cgi?id=48686

Convert the next set of primitives to use the new SVG(Animated)PropertyTearOff concept.

Tests: svg/dom/SVGAnimatedNumber.html

svg/dom/SVGAnimatedNumberList.html
svg/dom/SVGNumber.html

  • GNUmakefile.am: Add SVGAnimatedNumber.h / SVGAnimatedNumberList.h to build.
  • WebCore.gypi: Ditto.
  • WebCore.pro: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • bindings/scripts/CodeGenerator.pm: Recognize SVGAnimatedNumber/SVGAnimatedNumberList as new style SVG animated properties. Remove SVGNumber from PODType hash.
  • bindings/scripts/CodeGeneratorJS.pm: Remove special SVGNumber handing, it's obsolete now.
  • bindings/scripts/CodeGeneratorObjC.pm: Add special SVGNumber handling, as it's the first of the converted dynamic SVGAnimatedProperty types that's not in WebCore namespace.
  • bindings/scripts/CodeGeneratorV8.pm: Use AvoidInclusionOfType method from CodeGenerator.pm, remove special SVGNumber handling, that's obsolete now.
  • rendering/svg/SVGTextLayoutAttributesBuilder.cpp: (WebCore::extractFloatValuesFromSVGNumberList): Adapt to SVGNumberList change, which is a plain Vector<float> now.
  • svg/DeprecatedSVGAnimatedPropertyTraits.h: Remove SVGAnimatedNumber/SVGAnimatedNumberList handling.
  • svg/DeprecatedSVGAnimatedTemplate.h: Ditto.
  • svg/SVGAnimatedNumber.h: Added.
  • svg/SVGAnimatedNumber.idl: Enable StrictTypeChecking.
  • svg/SVGAnimatedNumberList.h: Added.
  • svg/SVGAnimatedNumberList.idl: Ditto.
  • svg/SVGComponentTransferFunctionElement.cpp: Switch animated float properties to the new SVGAnimatedNumber(List). (WebCore::SVGComponentTransferFunctionElement::SVGComponentTransferFunctionElement): (WebCore::SVGComponentTransferFunctionElement::parseMappedAttribute): (WebCore::SVGComponentTransferFunctionElement::transferFunction):
  • svg/SVGComponentTransferFunctionElement.h:
  • svg/SVGFEColorMatrixElement.cpp: Ditto. (WebCore::SVGFEColorMatrixElement::SVGFEColorMatrixElement): (WebCore::SVGFEColorMatrixElement::parseMappedAttribute): (WebCore::SVGFEColorMatrixElement::build):
  • svg/SVGFEColorMatrixElement.h:
  • svg/SVGFECompositeElement.h: Ditto.
  • svg/SVGFEConvolveMatrixElement.cpp: Ditto. (WebCore::SVGFEConvolveMatrixElement::SVGFEConvolveMatrixElement): (WebCore::SVGFEConvolveMatrixElement::kernelUnitLengthXIdentifier): Added, replacing char[] hack by a static atomic string. (WebCore::SVGFEConvolveMatrixElement::kernelUnitLengthYIdentifier): Ditto. (WebCore::SVGFEConvolveMatrixElement::parseMappedAttribute): (WebCore::SVGFEConvolveMatrixElement::build):
  • svg/SVGFEConvolveMatrixElement.h: Switch animated float properties to the new SVGAnimatedNumber(List).
  • svg/SVGFEDiffuseLightingElement.cpp: Ditto. (WebCore::SVGFEDiffuseLightingElement::kernelUnitLengthXIdentifier): Remove trailing whitespace. (WebCore::SVGFEDiffuseLightingElement::kernelUnitLengthYIdentifier): Ditto.
  • svg/SVGFEDiffuseLightingElement.h: Switch animated float properties to the new SVGAnimatedNumber(List).
  • svg/SVGFEDisplacementMapElement.h: Ditto.
  • svg/SVGFEGaussianBlurElement.cpp: Ditto. (WebCore::SVGFEGaussianBlurElement::stdDeviationXIdentifier): Remove trailing whitespace. (WebCore::SVGFEGaussianBlurElement::stdDeviationYIdentifier): Ditto.
  • svg/SVGFEGaussianBlurElement.h: Switch animated float properties to the new SVGAnimatedNumber(List).
  • svg/SVGFELightElement.h: Ditto.
  • svg/SVGFEMorphologyElement.cpp: (WebCore::SVGFEMorphologyElement::radiusXIdentifier): Added, replacing char[] hack by a static atomic string. (WebCore::SVGFEMorphologyElement::radiusYIdentifier): Ditto.
  • svg/SVGFEMorphologyElement.h: Switch animated float properties to the new SVGAnimatedNumber(List).
  • svg/SVGFEOffsetElement.h: Ditto.
  • svg/SVGFESpecularLightingElement.cpp: (WebCore::SVGFESpecularLightingElement::kernelUnitLengthXIdentifier): Remove trailing whitespace. (WebCore::SVGFESpecularLightingElement::kernelUnitLengthYIdentifier): Ditto.
  • svg/SVGFESpecularLightingElement.h: Switch animated float properties to the new SVGAnimatedNumber(List).
  • svg/SVGFETurbulenceElement.cpp: (WebCore::SVGFETurbulenceElement::baseFrequencyXIdentifier): Remove trailing whitespace. (WebCore::SVGFETurbulenceElement::baseFrequencyYIdentifier): Ditto.
  • svg/SVGFETurbulenceElement.h: Switch animated float properties to the new SVGAnimatedNumber(List).
  • svg/SVGFilterElement.cpp: (WebCore::SVGFilterElement::filterResXIdentifier): Remove trailing whitespace. (WebCore::SVGFilterElement::filterResYIdentifier): Ditto.
  • svg/SVGLengthList.idl: Remove leftover [SVGListProperty] marker.
  • svg/SVGMarkerElement.cpp: (WebCore::SVGMarkerElement::orientTypeIdentifier): Remove trailing whitespace. (WebCore::SVGMarkerElement::orientAngleIdentifier): Ditto.
  • svg/SVGNumber.idl: Remove [PODType] marker, add StrictTypeChecking.
  • svg/SVGNumberList.cpp: Convert from SVGPODList<RefPtr<SVGPODListItem<float> > > to plain Vector<float>, just like it has been done for SVGLengthList before. (WebCore::SVGNumberList::parse): (WebCore::SVGNumberList::valueAsString):
  • svg/SVGNumberList.h: (WebCore::SVGNumberList::SVGNumberList):
  • svg/SVGPathElement.h: Switch animated float properties to the new SVGAnimatedNumber(List).
  • svg/SVGStopElement.h: Ditto.
  • svg/SVGTextPositioningElement.cpp: Switch animated float properties to the new SVGAnimatedNumber(List). (WebCore::SVGTextPositioningElement::SVGTextPositioningElement): (WebCore::SVGTextPositioningElement::parseMappedAttribute):
  • svg/SVGTextPositioningElement.h: Ditto.
  • svg/properties/SVGPropertyTraits.h: Add SVGNumberList handling.

2010-10-30 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Rob Buis.

Convert SVGAnimatedNumber/SVGAnimatedNumberList to the new SVGAnimatedPropertyTearOff concept
https://bugs.webkit.org/show_bug.cgi?id=48686

Fix invalid tests in svg/dynamic-updates, that relied on a bug in our SVGNumberList implementation. If an item was already
inserted in a list, and we tried to insert it somewhere else, we didn't remove it from the old list. Now that SVGNumberList
has been switched to SVGAnimatedListPropertyTearOff, the bug is fixed, and it behaves exactly like SVGLengthList.

Side effect: svg/dynamic-updates/SVGFETurbulenceElement-svgdom-baseFrequency-prop.html now works as expected (dynamic update is executed)

  • platform/mac-leopard/svg/dynamic-updates/SVGFETurbulenceElement-svgdom-baseFrequency-prop-expected.checksum:
  • platform/mac-leopard/svg/dynamic-updates/SVGFETurbulenceElement-svgdom-baseFrequency-prop-expected.png:
  • svg/dom/SVGAnimatedNumber-expected.txt: Added.
  • svg/dom/SVGAnimatedNumber.html: Added.
  • svg/dom/SVGAnimatedNumberList-expected.txt: Added.
  • svg/dom/SVGAnimatedNumberList.html: Added.
  • svg/dom/SVGNumber-expected.txt: Added.
  • svg/dom/SVGNumber.html: Added.
  • svg/dom/script-tests/SVGAnimatedNumber.js: Added.
  • svg/dom/script-tests/SVGAnimatedNumberList.js: Added.
  • svg/dom/script-tests/SVGNumber.js: Added.
  • svg/dynamic-updates/script-tests/SVGFEColorMatrixElement-svgdom-type-prop.js: Test now works in Firefox as well. (executeTest): SVGNumber.value only accepts numbers, not strings, now that StrictTypeChecking is enabled.
  • svg/dynamic-updates/script-tests/SVGFEColorMatrixElement-svgdom-values-prop.js: Test now works in Firefox as well. (executeTest): Don't call appendItem several times with the same SVGNumber, it will be removed if it was already inserted in a list.
  • svg/dynamic-updates/script-tests/SVGFETurbulenceElement-svgdom-baseFrequency-prop.js:
1:59 AM Changeset in webkit [70978] by Nikolas Zimmermann
  • 5 edits in trunk/LayoutTests

2010-10-30 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed.

Update SVGFEFloodElement mac-leopard baseline, which was forgotten.

  • platform/mac-leopard/svg/dynamic-updates/SVGFEFloodElement-dom-flood-opacity-attr-expected.checksum:
  • platform/mac-leopard/svg/dynamic-updates/SVGFEFloodElement-dom-flood-opacity-attr-expected.png:
  • platform/mac-leopard/svg/dynamic-updates/SVGFEFloodElement-svgdom-flood-opacity-prop-expected.checksum:
  • platform/mac-leopard/svg/dynamic-updates/SVGFEFloodElement-svgdom-flood-opacity-prop-expected.png:

Oct 29, 2010:

11:15 PM Changeset in webkit [70977] by abarth@webkit.org
  • 1 edit
    2 copies in branches/chromium/552

Merge 70718 - 2010-10-27 Adam Barth <abarth@webkit.org>

Reviewed by Ojan Vafai.

REGRESSION(66511): CORDYS Process Factory doesn't work
https://bugs.webkit.org/show_bug.cgi?id=48453

Test some appendChild cases that didn't have coverage before.

  • fast/dom/allowed-children-expected.txt: Added.
  • fast/dom/allowed-children.html: Added.

2010-10-27 Adam Barth <abarth@webkit.org>

Reviewed by Ojan Vafai.

REGRESSION(66511): CORDYS Process Factory doesn't work
https://bugs.webkit.org/show_bug.cgi?id=48453

When Ojan refactored this function in 66511, he missed an "else",
which causes us to generated HIERARCHY_REQUEST_ERR in some cases where
we're not supposed to. This patch restores the "else" behavior (using
an early return) and adds a test for the cases that caused problems in
the field.

Test: fast/dom/allowed-children.html

  • dom/Node.cpp: (WebCore::isChildTypeAllowed):

TBR=abarth@webkit.org

11:13 PM Changeset in webkit [70976] by dbates@webkit.org
  • 28 edits
    2 copies in trunk

2010-10-29 Daniel Bates <dbates@rim.com>

No review, rolling out 70971.
http://trac.webkit.org/changeset/70971
https://bugs.webkit.org/show_bug.cgi?id=6751

Rolling out changeset 70971 <http://trac.webkit.org/changeset/70971> since
it caused layout tests regressions on all bots. In particular, the
child count in a generated frame name differs after this patch. We need
to look into this further.

  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::name):

2010-10-29 Daniel Bates <dbates@rim.com>

No review, rolling out 70971.
http://trac.webkit.org/changeset/70971
https://bugs.webkit.org/show_bug.cgi?id=6751

Rolling out changeset 70971 <http://trac.webkit.org/changeset/70971> since
it caused layout tests regressions on all bots. In particular, the
child count in a generated frame name differs after this patch. We need
to look into this further.

  • Api/qwebframe.cpp: (QWebFrame::frameName):

2010-10-29 Daniel Bates <dbates@rim.com>

No review, rolling out 70971.
http://trac.webkit.org/changeset/70971
https://bugs.webkit.org/show_bug.cgi?id=6751

Rolling out changeset 70971 <http://trac.webkit.org/changeset/70971> since
it caused layout tests regressions on all bots. In particular, the
child count in a generated frame name differs after this patch. We need
to look into this further.

  • webkit/webkitwebframe.cpp: (webkit_web_frame_get_name):

2010-10-29 Daniel Bates <dbates@rim.com>

No review, rolling out 70971.
http://trac.webkit.org/changeset/70971
https://bugs.webkit.org/show_bug.cgi?id=6751

Rolling out changeset 70971 <http://trac.webkit.org/changeset/70971> since
it caused layout test failures on all bots. In particular, the
child count in a generated frame name differs after this patch. We need
to look into this further.

  • WebFrame.cpp: (WebFrame::name):

2010-10-29 Daniel Bates <dbates@rim.com>

No review, rolling out 70971.
http://trac.webkit.org/changeset/70971
https://bugs.webkit.org/show_bug.cgi?id=6751

Rolling out changeset 70971 <http://trac.webkit.org/changeset/70971> since
it caused layout test failures on all bots. In particular, the
child count in a generated frame name differs after this patch. We need
to look into this further.

  • WebView/WebFrame.mm: (-[WebFrame name]):

2010-10-29 Daniel Bates <dbates@rim.com>

No review, rolling out 70971.
http://trac.webkit.org/changeset/70971
https://bugs.webkit.org/show_bug.cgi?id=6751

Rolling out changeset 70971 <http://trac.webkit.org/changeset/70971> since
it caused layout test failures on all bots. In particular, the
child count in a generated frame name differs after this patch. We need
to look into this further.

  • ewk/ewk_frame.cpp: (ewk_frame_name_get):

2010-10-29 Daniel Bates <dbates@rim.com>

No review, rolling out 70971.
http://trac.webkit.org/changeset/70971
https://bugs.webkit.org/show_bug.cgi?id=6751

Rolling out changeset 70971 <http://trac.webkit.org/changeset/70971> since
it caused layout test failures on all bots. In particular, the
child count in a generated frame name differs after this patch. We need
to look into this further.

Test: fast/frames/iframe-set-inner-html.html

  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::openURL): (WebCore::HTMLFrameElementBase::setName):
  • html/HTMLFrameElementBase.h:
  • loader/DocumentLoader.cpp: (WebCore::DocumentLoader::mainResource):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::loadURLIntoChildFrame): (WebCore::FrameLoader::commitProvisionalLoad):
  • loader/HistoryController.cpp: (WebCore::HistoryController::saveDocumentState): (WebCore::HistoryController::restoreDocumentState): (WebCore::HistoryController::createItem): (WebCore::HistoryController::currentFramesMatchItem):
  • loader/ProgressTracker.cpp: (WebCore::ProgressTracker::progressStarted): (WebCore::ProgressTracker::progressCompleted):
  • loader/archive/cf/LegacyWebArchive.cpp: (WebCore::LegacyWebArchive::create):
  • page/FrameTree.cpp: (WebCore::FrameTree::setName): (WebCore::FrameTree::clearName): (WebCore::FrameTree::uniqueChildName): (WebCore::FrameTree::child): (WebCore::FrameTree::find):
  • page/FrameTree.h: (WebCore::FrameTree::name):

2010-10-29 Daniel Bates <dbates@rim.com>

No review, rolling out 70971.
http://trac.webkit.org/changeset/70971
https://bugs.webkit.org/show_bug.cgi?id=6751

Rolling out changeset 70971 <http://trac.webkit.org/changeset/70971> since
it caused layout test failures on all bots. In particular, the
child count in a generated frame name differs after this patch. We need
to look into this further.

  • fast/frames/frame-element-name-expected.txt:
  • fast/frames/iframe-set-inner-html-expected.txt: Copied from LayoutTests/fast/frames/iframe-set-inner-html-expected.txt.
  • fast/frames/iframe-set-inner-html.html: Copied from LayoutTests/fast/frames/iframe-set-inner-html.html.
  • fast/frames/resources/frame-element-name-left.html:
  • fast/frames/resources/frame-element-name-right.html:

2010-10-29 Daniel Bates <dbates@rim.com>

No review, rolling out 70971.
http://trac.webkit.org/changeset/70971
https://bugs.webkit.org/show_bug.cgi?id=6751

Rolling out changeset 70971 <http://trac.webkit.org/changeset/70971> since
it caused layout test failures on all bots. In particular, the
child count in a generated frame name differs after this patch. We need
to look into this further.

  • WebProcess/WebPage/WebFrame.cpp: (WebKit::WebFrame::name):
10:38 PM Changeset in webkit [70975] by tonikitoo@webkit.org
  • 21 edits in trunk

Needs a "LinuxEditingBehavior", perhaps with a better name
https://bugs.webkit.org/show_bug.cgi?id=36627

Reviewed by Ojan Vafai.
Patch by Antonio Gomes <agomes@rim.com>

WebCore:

Added EditingUnixBehavior editing type aim to cover the cases when
neither Mac or Windows behaviors are appropriated.

It was decided to name it Unix not Linux.

No new tests. The current editing tests that involve platform specific
bahaviors are coded so that they go through all editing behavior
types possible. There will be a follow up bug where all of them will
be updated to include the new editing behavior ("unix").

  • editing/EditingBehaviorTypes.h:
  • page/Settings.cpp:

(WebCore::editingBehaviorTypeForPlatform):
(WebCore::Settings::Settings):

WebKit/chromium:

Added the corresponding Chromium setting to WebCore's EditingUnixBehavior: EditingBehaviorUnix.

  • public/WebSettings.h:
  • src/AssertMatchingEnums.cpp:

WebKit/gtk:

Added the corresponding GTK+ setting to WebCore's EditingUnixBehavior: WEBKIT_EDITING_BEHAVIOR_UNIX.

  • webkit/webkitwebsettings.cpp:

(webkit_web_settings_class_init):

  • webkit/webkitwebsettings.h:

WebKit/mac:

Added the corresponding GTK+ setting to WebCore's EditingUnixBehavior: WebKitEditingUnixBehavior.

  • WebView/WebFrame.mm:

(core):

  • WebView/WebPreferencesPrivate.h:

WebKit/qt:

Adding support to Qt's DRTSupport class to test the newly introduced Unix editing behavior.

  • WebCoreSupport/DumpRenderTreeSupportQt.cpp:

(DumpRenderTreeSupportQt::setEditingBehavior):

WebKit/win:

Added the corresponding GTK+ setting to WebCore's EditingUnixBehavior: WebKitEditingUnixBehavior.

  • Interfaces/IWebPreferences.idl:

WebKitTools:

Adding support to Mac's, GTK+'s, Windows' and Chromium's LayoutTestController class to test the newly introduced Unix editing behavior.

  • DumpRenderTree/chromium/LayoutTestController.cpp:

(LayoutTestController::setEditingBehavior):

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:

(LayoutTestController::setEditingBehavior):

  • DumpRenderTree/mac/LayoutTestControllerMac.mm:

(LayoutTestController::setEditingBehavior):

  • DumpRenderTree/win/LayoutTestControllerWin.cpp:

(LayoutTestController::setEditingBehavior):

9:07 PM Changeset in webkit [70974] by mrowe@apple.com
  • 9 edits
    2 deletes in tags/Safari-534.11.1/WebCore

Merge r70970.

9:07 PM Changeset in webkit [70973] by mrowe@apple.com
  • 5 edits in tags/Safari-534.11.1

Versioning.

8:57 PM Changeset in webkit [70972] by mrowe@apple.com
  • 1 copy in tags/Safari-534.11.1

New tag.

8:49 PM Changeset in webkit [70971] by dbates@webkit.org
  • 28 edits
    2 deletes in trunk

2010-10-29 Daniel Bates <dbates@rim.com>

Reviewed by Adam Barth.

For unnamed frames, window.name returns a generated name
https://bugs.webkit.org/show_bug.cgi?id=6751

Modified Chromium-port to use FrameTree::uniqueName().

  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::name):

2010-10-29 Daniel Bates <dbates@rim.com>

Reviewed by Adam Barth.

For unnamed frames, window.name returns a generated name
https://bugs.webkit.org/show_bug.cgi?id=6751

Modified Qt-port to use FrameTree::uniqueName().

  • Api/qwebframe.cpp: (QWebFrame::frameName):

2010-10-29 Daniel Bates <dbates@rim.com>

Reviewed by Adam Barth.

For unnamed frames, window.name returns a generated name
https://bugs.webkit.org/show_bug.cgi?id=6751

Modified GTK-port to use FrameTree::uniqueName().

  • webkit/webkitwebframe.cpp: (webkit_web_frame_get_name):

2010-10-29 Daniel Bates <dbates@rim.com>

Reviewed by Adam Barth.

For unnamed frames, window.name returns a generated name
https://bugs.webkit.org/show_bug.cgi?id=6751

Modified Apple Windows-port to use FrameTree::uniqueName().

  • WebFrame.cpp: (WebFrame::name):

2010-10-29 Daniel Bates <dbates@rim.com>

Reviewed by Adam Barth.

For unnamed frames, window.name returns a generated name
https://bugs.webkit.org/show_bug.cgi?id=6751

Modified Mac-port to use FrameTree::uniqueName().

  • WebView/WebFrame.mm: (-[WebFrame name]):

2010-10-29 Daniel Bates <dbates@rim.com>

Reviewed by Adam Barth.

For unnamed frames, window.name returns a generated name
https://bugs.webkit.org/show_bug.cgi?id=6751

Modified EFL-port to use FrameTree::uniqueName().

  • ewk/ewk_frame.cpp: (ewk_frame_name_get):

2010-10-29 Daniel Bates <dbates@rim.com>

Reviewed by Adam Barth.

For unnamed frames, window.name returns a generated name
https://bugs.webkit.org/show_bug.cgi?id=6751

Fixes an issue where the window.name of an unnamed frame returns
a unique generated name that is used internally to identify the
frame. Instead, we should return an empty string as implied by
section 5.1.6 of the HTML5 spec. (http://www.w3.org/TR/html5/browsers.html#browsing-context-names).
This section describes that a browsing context can have no name or be
the empty string.

  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::openURL): Removed ASSERT for empty frame name since this is valid as per the HTML5 spec. (WebCore::HTMLFrameElementBase::setName): Removed the call to FrameTree::uniqueChildName() since the loader code no longer depends on the frame name being unique.
  • html/HTMLFrameElementBase.h:
  • loader/DocumentLoader.cpp: (WebCore::DocumentLoader::mainResource): Use FrameTree::uniqueName().
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::loadURLIntoChildFrame): Ditto. (WebCore::FrameLoader::commitProvisionalLoad): Ditto.
  • loader/HistoryController.cpp: (WebCore::HistoryController::saveDocumentState): Ditto. (WebCore::HistoryController::restoreDocumentState): Ditto. (WebCore::HistoryController::createItem): (WebCore::HistoryController::currentFramesMatchItem): Ditto.
  • loader/ProgressTracker.cpp: (WebCore::ProgressTracker::progressStarted): Ditto. (WebCore::ProgressTracker::progressCompleted): Ditto.
  • loader/archive/cf/LegacyWebArchive.cpp: (WebCore::LegacyWebArchive::create): Ditto.
  • page/FrameTree.cpp: (WebCore::FrameTree::setName): Modified to store the DOM-specified name of the frame. (WebCore::FrameTree::clearName): (WebCore::FrameTree::uniqueChildName): Use FrameTree::uniqueName(). (WebCore::FrameTree::child): Ditto. (WebCore::FrameTree::find): Ditto.
  • page/FrameTree.h: (WebCore::FrameTree::uniqueName): Added.

2010-10-29 Daniel Bates <dbates@rim.com>

Reviewed by Adam Barth.

For unnamed frames, window.name returns a generated name
https://bugs.webkit.org/show_bug.cgi?id=6751

Updated test case fast/frames/frame-element-name.html to check
the value of window.name.

  • fast/frames/frame-element-name-expected.txt: Updated result.
  • fast/frames/iframe-set-inner-html-expected.txt: Removed; This test ensured that the generated name for an unnamed frames was unique by looking at the value of window.name. However, the generated name should not be exposed to web developers. Instead, window.name should return the empty string for an unnamed frame as implied by section 5.1.6 of the HTML5 spec. We should consider exposing a DRT method for obtaining the internal frame name (which may be generated) and re-implementing this test to use it.
  • fast/frames/iframe-set-inner-html.html: Removed.
  • fast/frames/resources/frame-element-name-left.html: Added test for window.name.
  • fast/frames/resources/frame-element-name-right.html: Ditto.

2010-10-29 Daniel Bates <dbates@rim.com>

Reviewed by Adam Barth.

For unnamed frames, window.name returns a generated name
https://bugs.webkit.org/show_bug.cgi?id=6751

Modified WebKit2 to use FrameTree::uniqueName().

  • WebProcess/WebPage/WebFrame.cpp: (WebKit::WebFrame::name):
8:42 PM Changeset in webkit [70970] by mrowe@apple.com
  • 9 edits
    2 deletes in trunk/WebCore

Roll out r70847 as it causes crashes when typing text in both Safari and Mail: <http://webkit.org/b/48708>.

8:37 PM Changeset in webkit [70969] by mitz@apple.com
  • 2 edits in trunk/WebKitLibraries

Snow Leopard PowerPC build fix.

  • libWebKitSystemInterfaceSnowLeopard.a:
8:34 PM Changeset in webkit [70968] by mitz@apple.com
  • 2 edits in trunk/WebKitLibraries

Leopard PowerPC build fix.

  • libWebKitSystemInterfaceLeopard.a:
6:40 PM Changeset in webkit [70967] by jamesr@google.com
  • 495 edits
    494 copies in trunk/LayoutTests

2010-10-29 James Robinson <jamesr@chromium.org>

Update pixel expectations for Snow Leopard for tests in tables/mozilla/bugs.

Moves the leopard-specific mac baselines to platform/mac-leopard/ and adds new Snow Leopard pixel
expectations for tests that need them. List of files omitted due to length.

6:28 PM Changeset in webkit [70966] by weinig@apple.com
  • 2 edits in trunk/WebKit2

Implement WebProcess::platformSetCacheModel for windows
<rdar://problem/8605796>
https://bugs.webkit.org/show_bug.cgi?id=48703

Reviewed by Simon Fraser.

  • WebProcess/win/WebProcessWin.cpp:

(WebKit::memorySize):
(WebKit::volumeFreeSize):
(WebKit::WebProcess::platformSetCacheModel):
Add implemenation based on the version in WebKit/win/WebView.cpp.

6:23 PM Changeset in webkit [70965] by dimich@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Unreviewed test expectations update.
Enable a few tests that were fixed in revs 70891:70949.

  • platform/chromium/test_expectations.txt:
6:09 PM Changeset in webkit [70964] by andreas.kling@nokia.com
  • 2 edits in trunk/WebKit2

2010-10-29 Andreas Kling <kling@webkit.org>

Reviewed by Anders Carlsson.

WK2: Unbreak compilation of WebPlatformTouchPoint::decode()

  • Shared/WebPlatformTouchPoint.cpp: (WebKit::WebPlatformTouchPoint::decode):
6:07 PM Changeset in webkit [70963] by Darin Adler
  • 5 edits in trunk/WebCore

Try to fix build.

  • page/ContextMenuController.cpp: Added include of BackForwardController.h.
  • page/DOMWindow.cpp: Ditto.
  • page/History.cpp: Ditto.
  • platform/ContextMenu.cpp: Ditto.
5:58 PM Changeset in webkit [70962] by mrowe@apple.com
  • 5 edits in trunk

Versioning.

5:55 PM Changeset in webkit [70961] by mrowe@apple.com
  • 1 copy in tags/Safari-534.11

New tag.

5:44 PM Changeset in webkit [70960] by Darin Adler
  • 33 edits
    3 deletes in trunk

More back/forward refactoring

Reviewed by Sam Weinig.

WebCore:

  • WebCore.exp.in: Updated.
  • GNUmakefile.am: Removed BackForwardControllerClient.h.
  • WebCore.gypi: Ditto.
  • WebCore.pro: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • history/BackForwardControllerClient.h: Removed.
  • history/BackForwardController.cpp:

(WebCore::BackForwardController::BackForwardController):
Set up just m_client.
(WebCore::BackForwardController::~BackForwardController):
Removed call to backForwardControllerDestroyed.
(WebCore::BackForwardController::canGoBackOrForward): Added.
(WebCore::BackForwardController::goBackOrForward): Added.
(WebCore::BackForwardController::goBack): Added.
(WebCore::BackForwardController::goForward): Added.
(WebCore::BackForwardController::addItem): Added.
(WebCore::BackForwardController::setCurrentItem): Added.
(WebCore::BackForwardController::count): Added.
(WebCore::BackForwardController::backCount): Added.
(WebCore::BackForwardController::forwardCount): Added.
(WebCore::BackForwardController::itemAtIndex): Added.
(WebCore::BackForwardController::isActive): Added.
(WebCore::BackForwardController::close): Added.

  • history/BackForwardController.h: Added functions from Page

and BackForwardList that should be called here instead.
Note that BackForwardList is to be renamed BackForwardClient soon.

  • history/BackForwardList.h: Removed many now-unneeded

virtual functions.

  • history/BackForwardListImpl.h: Moved BackForwardListClient

in here.

  • history/PageCache.cpp:

(WebCore::logCanCachePageDecision): Use backForward().
(WebCore::PageCache::canCache): Ditto.

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::transitionToCommitted): Removed unneeded
null check of backFowardList, which is never null.
(WebCore::FrameLoader::checkLoadCompleteForThisFrame): Use backForward.
(WebCore::FrameLoader::didFirstLayout): Ditto.
(WebCore::FrameLoader::continueLoadAfterNavigationPolicy): Ditto.
(WebCore::FrameLoader::checkDidPerformFirstNavigation): Ditto.

  • loader/HistoryController.cpp:

(WebCore::HistoryController::goToItem): Ditto.
(WebCore::HistoryController::updateForStandardLoad): Ditto.
(WebCore::HistoryController::updateForRedirectWithLockedBackForwardList): Ditto.
(WebCore::HistoryController::updateBackForwardListClippedAtTarget): Ditto.
(WebCore::HistoryController::pushState): Ditto.

  • loader/NavigationScheduler.cpp:

(WebCore::ScheduledHistoryNavigation::fire): Ditto.
(WebCore::NavigationScheduler::scheduleHistoryNavigation): Ditto.

  • page/ContextMenuController.cpp:

(WebCore::ContextMenuController::contextMenuItemSelected): Ditto.

  • page/DOMWindow.cpp:

(WebCore::DOMWindow::close): Ditto.

  • page/History.cpp:

(WebCore::History::length): Ditto.

  • page/Page.cpp:

(WebCore::Page::Page): Use more adoptPtr. Use backForwardClient instead of
backForwardControllerClient.
(WebCore::Page::~Page): Use backForward.
(WebCore::Page::backForwardList): Ditto.
(WebCore::Page::goBack): Ditto.
(WebCore::Page::goForward): Ditto.
(WebCore::Page::canGoBackOrForward): Ditto.
(WebCore::Page::goBackOrForward): Ditto.
(WebCore::Page::getHistoryLength): Ditto.
(WebCore::Page::PageClients::PageClients): Made non-inline.
(WebCore::Page::PageClients::~PageClients): Ditto.

  • page/Page.h: Made PageClients constructor and destructor non-inline.

Added backForward function. Marked old functions as deprecated.

  • page/Settings.cpp:

(WebCore::Settings::setUsesPageCache): Use backForward.

  • platform/ContextMenu.cpp:

(WebCore::ContextMenu::populate): Ditto.
(WebCore::ContextMenu::checkOrEnableIfNeeded): Ditto.

  • wml/WMLDoElement.cpp:

(WebCore::WMLDoElement::defaultEventHandler): Ditto.

  • wml/WMLDocument.cpp:

(WebCore::WMLDocument::finishedParsing): Ditto.

  • wml/WMLPageState.cpp:

(WebCore::WMLPageState::reset): Ditto.
(WebCore::tryAccessHistoryURLs): Ditto.

  • wml/WMLPrevElement.cpp:

(WebCore::WMLPrevElement::executeTask): Ditto.

WebKit2:

  • WebKit2.pro: Removed WebBackForwardControllerClient.cpp and .h.
  • WebKit2.xcodeproj/project.pbxproj: Ditto.
  • WebProcess/WebCoreSupport/WebBackForwardControllerClient.cpp: Removed.
  • WebProcess/WebCoreSupport/WebBackForwardControllerClient.h: Removed.
  • WebProcess/InjectedBundle/InjectedBundleBackForwardList.cpp:

(WebKit::InjectedBundleBackForwardList::itemAtIndex): Use backForward.
(WebKit::InjectedBundleBackForwardList::backListCount): Ditto.
(WebKit::InjectedBundleBackForwardList::forwardListCount): Ditto.
(WebKit::InjectedBundleBackForwardList::clear): Ditto.

  • WebProcess/WebPage/WebBackForwardListProxy.h: Added newly-needed include.
  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::WebPage): Set up backForwardClient.

5:38 PM Changeset in webkit [70959] by hyatt@apple.com
  • 25 edits
    4 adds in trunk

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

Reviewed by Sam Weinig.

Make repaint invalidation work with vertical lines. Rewrite linesBoundingBox for RenderText and
RenderInline to give the correct rectangle back for vertical lines. Also patch linesVisibleOverflowBoundingBox
to give back the correct rectangle for vertical lines.

Fix bugs in the overflow accessors of InlineFlowBox when m_overflow was 0.

Added fast/repaint/inline-vertical-lr-overflow.html

WebCore:

  • rendering/InlineBox.cpp:

(WebCore::InlineBox::adjustForFlippedBlocksWritingMode):

  • rendering/InlineFlowBox.h:

(WebCore::InlineFlowBox::logicalLeftVisibleOverflow):
(WebCore::InlineFlowBox::logicalRightVisibleOverflow):
(WebCore::InlineFlowBox::bottomLayoutOverflow):
(WebCore::InlineFlowBox::rightLayoutOverflow):
(WebCore::InlineFlowBox::bottomVisualOverflow):
(WebCore::InlineFlowBox::rightVisualOverflow):

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::adjustForFlippedBlocksWritingMode):

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

(WebCore::RenderInline::linesBoundingBox):
(WebCore::RenderInline::linesVisibleOverflowBoundingBox):
(WebCore::RenderInline::clippedOverflowRectForRepaint):

  • rendering/RenderText.cpp:

(WebCore::RenderText::linesBoundingBox):

LayoutTests:

  • fast/repaint/inline-vertical-rl-overflow.html: Added.
  • platform/mac/fast/blockflow/background-vertical-lr-expected.txt:
  • platform/mac/fast/blockflow/background-vertical-rl-expected.txt:
  • platform/mac/fast/blockflow/basic-vertical-line-expected.txt:
  • platform/mac/fast/blockflow/border-image-vertical-lr-expected.txt:
  • platform/mac/fast/blockflow/border-image-vertical-rl-expected.txt:
  • platform/mac/fast/blockflow/border-radius-clipping-vertical-lr-expected.txt:
  • platform/mac/fast/blockflow/border-vertical-lr-expected.checksum:
  • platform/mac/fast/blockflow/border-vertical-lr-expected.png:
  • platform/mac/fast/blockflow/border-vertical-lr-expected.txt:
  • platform/mac/fast/blockflow/box-shadow-vertical-lr-expected.txt:
  • platform/mac/fast/blockflow/box-shadow-vertical-rl-expected.txt:
  • platform/mac/fast/blockflow/english-lr-text-expected.txt:
  • platform/mac/fast/blockflow/english-rl-text-expected.txt:
  • platform/mac/fast/blockflow/japanese-lr-text-expected.txt:
  • platform/mac/fast/blockflow/japanese-rl-text-expected.txt:
  • platform/mac/fast/repaint/inline-vertical-rl-overflow-expected.checksum: Added.
  • platform/mac/fast/repaint/inline-vertical-rl-overflow-expected.png: Added.
  • platform/mac/fast/repaint/inline-vertical-rl-overflow-expected.txt: Added.
  • platform/mac/fast/text/international/vertical-text-glyph-test-expected.txt:
5:37 PM Changeset in webkit [70958] by jamesr@google.com
  • 321 edits
    320 copies in trunk/LayoutTests

2010-10-29 James Robinson <jamesr@chromium.org>

Update pixel expectations for Snow Leopard for tests in tables/mozilla/marvin.

Moves the leopard-specific mac baselines to platform/mac-leopard/ and adds new Snow Leopard pixel
expectations for tests that need them. List of files omitted due to length.

5:13 PM Changeset in webkit [70957] by commit-queue@webkit.org
  • 5 edits
    1 delete in trunk

2010-10-29 Mike Lawther <mikelawther@chromium.org>

Reviewed by James Robinson.

Fix canvas/philip/tests/2d.imageData.get.source.negative
https://bugs.webkit.org/show_bug.cgi?id=48277

Remove canvas/philip/tests/2d.imageData.get.source.negative from chromium skipped.
Also remove spurious trailing newlines from 2d.imageData.get.source.negative-expected.txt

  • canvas/philip/tests/2d.imageData.get.source.negative-expected.txt:
  • platform/chromium/test_expectations.txt:

2010-10-29 Mike Lawther <mikelawther@chromium.org>

Reviewed by James Robinson.

Fix canvas/philip/tests/2d.imageData.get.source.negative
https://bugs.webkit.org/show_bug.cgi?id=48277

Handle negative width/height.
See http://www.whatwg.org/specs/web-apps/current-work/multipage/the-canvas-element.html#pixel-manipulation

  • html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::getImageData):
4:57 PM Changeset in webkit [70956] by commit-queue@webkit.org
  • 25 edits
    3 adds in trunk

2010-10-29 Kavita Kanetkar <kkanetkar@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: FileSystem integration
https://bugs.webkit.org/show_bug.cgi?id=45982

Adding filesystem support for Inspector under storage tab. This fetches root paths for
temporary and persistent filesystems and allows "reveal folder in OS" option that
launches native file browser. Currently this feature is disabled for non-chromium platforms.

  • CMakeLists.txt:
  • English.lproj/localizedStrings.js:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/CodeGeneratorInspector.pm:
  • inspector/Inspector.idl:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::connectFrontend): (WebCore::InspectorController::releaseFrontendLifetimeAgents):
  • inspector/InspectorController.h: (WebCore::InspectorController::fileSystemAgent):
  • inspector/InspectorFileSystemAgent.cpp: Added.
  • inspector/InspectorFileSystemAgent.h: Added.
  • inspector/front-end/FileSystemView.js: Added.
  • inspector/front-end/Settings.js:
  • inspector/front-end/StoragePanel.js: (WebInspector.StoragePanel): (WebInspector.StoragePanel.prototype.reset): (WebInspector.StoragePanel.prototype.addFileSystem): (WebInspector.StoragePanel.prototype.showFileSystem): (WebInspector.StoragePanel.prototype.updateFileSystemPath): (WebInspector.StoragePanel.prototype.updateFileSystemError): (WebInspector.FileSystemTreeElement): (WebInspector.FileSystemTreeElement.prototype.onselect):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.css: (.file-system-storage-tree-item .icon):
  • inspector/front-end/inspector.html:
  • inspector/front-end/inspector.js: (WebInspector.updateResource): (WebInspector._addFileSystemOrigin): (WebInspector.didGetFileSystemPath): (WebInspector.didGetFileSystemError): (WebInspector.reset):
  • platform/AsyncFileSystem.h: (WebCore::AsyncFileSystem::root):
  • platform/FileSystem.h:
  • platform/chromium/ChromiumBridge.h:
  • platform/chromium/FileSystemChromium.cpp: (WebCore::revealFolderInOS):

2010-10-29 Kavita Kanetkar <kkanetkar@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: FileSystem integration
https://bugs.webkit.org/show_bug.cgi?id=45982

  • public/WebFileUtilities.h: (WebKit::WebFileUtilities::revealFolderInOS):
  • src/ChromiumBridge.cpp: (WebCore::ChromiumBridge::revealFolderInOS):
4:53 PM Changeset in webkit [70955] by Martin Robinson
  • 2 edits in trunk/WebKit/gtk

2010-10-29 Martin Robinson <mrobinson@igalia.com>

Build fix for GTK+ after BackForwardListImpl introduction.

  • webkit/webkitwebbackforwardlist.cpp: (webkit_web_back_forward_list_new_with_web_view):
4:45 PM Changeset in webkit [70954] by Adam Roben
  • 8 edits in trunk/WebKit2

Stop using encode/decodeBytes for WebEvent subclasses

The use of encode/decodeBytes was making these classes sensitive to
differences in padding on different architectures (e.g., 32-bit vs.
64-bit). By encoding each data member explicitly we remove this
sensitivity, allowing these classes to be passed between processes
with different padding.

Fixes <http://webkit.org/b/48552> <rdar://problem/8606571> Mouse
events never reach Flash in WebKit2 on Mac (asserts in Debug builds)

Reviewed by Anders Carlsson.

  • Platform/CoreIPC/Arguments.h:

(CoreIPC::Arguments8::Arguments8):
(CoreIPC::Arguments8::encode):
(CoreIPC::Arguments8::decode):
(CoreIPC::In):
(CoreIPC::Out):
Added Arguments8, the biggest Arguments yet.

  • Shared/WebEvent.cpp:

(WebKit::WebEvent::encode):
(WebKit::WebEvent::decode):

  • Shared/WebKeyboardEvent.cpp:

(WebKit::WebKeyboardEvent::encode):
(WebKit::WebKeyboardEvent::decode):

  • Shared/WebMouseEvent.cpp:

(WebKit::WebMouseEvent::encode):
(WebKit::WebMouseEvent::decode):

  • Shared/WebPlatformTouchPoint.cpp:

(WebKit::WebPlatformTouchPoint::encode):
(WebKit::WebPlatformTouchPoint::decode):

  • Shared/WebTouchEvent.cpp:

(WebKit::WebTouchEvent::encode):
(WebKit::WebTouchEvent::decode):

  • Shared/WebWheelEvent.cpp:

(WebKit::WebWheelEvent::encode):
(WebKit::WebWheelEvent::decode):
Changed all encode/decode function to use CoreIPC::In/Out to
encode/decode data members individually.

4:44 PM Changeset in webkit [70953] by Adam Roben
  • 2 edits in trunk/WebKit2

Use only CoreIPC-sanctioned types in WebEvent subclasses

Fixes <http://webkit.org//48694> WebEvent subclasses should not use
architecture-dependent types

Reviewed by Anders Carlsson.

  • Shared/WebEvent.h:

(WebKit::WebEvent::type):
(WebKit::WebMouseEvent::button):
(WebKit::WebMouseEvent::clickCount):
(WebKit::WebWheelEvent::granularity):
(WebKit::WebPlatformTouchPoint::id):
(WebKit::WebPlatformTouchPoint::state):
Replaced enums, ints, and unsigneds with uint32_ts, int32_ts, and
uint32_ts, respectively.

4:44 PM Changeset in webkit [70952] by andreas.kling@nokia.com
  • 3 edits in trunk/LayoutTests

2010-10-29 Aaron Colwell <acolwell@chromium.org>

Reviewed by David Levin.

Disable media/video-canvas-alpha.html for Qt & GTK
https://bugs.webkit.org/show_bug.cgi?id=48688

  • platform/gtk/Skipped:
  • platform/qt/Skipped:
4:37 PM Changeset in webkit [70951] by andersca@apple.com
  • 3 edits in trunk/WebKit2

Generate a DelayedReply struct for sync messages marked delayed
https://bugs.webkit.org/show_bug.cgi?id=48696

Reviewed by Adam Roben.

  • Scripts/webkit2/messages.py:

If we encounter a sync message marked delayed, generate a DelayedReply struct with a send member funciton
that can be used for sending the reply.

  • Scripts/webkit2/messages_unittest.py:

Update expected results.

4:34 PM Changeset in webkit [70950] by jamesr@google.com
  • 89 edits
    88 copies in trunk/LayoutTests

2010-10-29 James Robinson <jamesr@chromium.org>

Update Snow Leopard pixel baselines for tests in tables/mozilla pt 1

This updates the pixel baselines in platform/mac to be valid for Snow Leopard for tests
in tables/mozilla/collapsing_borders, /core, /dom, and /other. Old baselines moved to
platform/mac-leopard.

4:31 PM Changeset in webkit [70949] by dimich@chromium.org
  • 3 edits in trunk/LayoutTests

[Chromium] Unreviewed, test expectations update.

  • platform/chromium-win/editing/deleting/5390681-2-expected.checksum:
  • platform/chromium-win/editing/deleting/5390681-2-expected.png:
4:24 PM Changeset in webkit [70948] by andreas.kling@nokia.com
  • 2 edits in trunk/WebKit/qt

2010-10-29 Andreas Kling <kling@webkit.org>

Reviewed by Ariya Hidayat.

[Qt] QWebPage's InputMethod event handling calls ShortcutOverride handler too
https://bugs.webkit.org/show_bug.cgi?id=48692

  • Api/qwebpage.cpp: (QWebPage::event): Add missing break statement.
4:22 PM Changeset in webkit [70947] by andreas.kling@nokia.com
  • 2 edits in trunk/WebKitTools

2010-10-29 Andreas Kling <kling@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] QtTestBrowser: Add keyboard shortcut to toggle full screen (F11)
https://bugs.webkit.org/show_bug.cgi?id=48695

  • QtTestBrowser/launcherwindow.cpp: (LauncherWindow::createChrome):
4:19 PM Changeset in webkit [70946] by andreas.kling@nokia.com
  • 3 edits in trunk/WebKitTools

2010-10-29 Andreas Kling <kling@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] QtTestBrowser: Fix uninitialized read in FpsTimer
https://bugs.webkit.org/show_bug.cgi?id=48675

FpsTimer::m_timer was never initialized and passed to QObject::killTimer()
on startup with in -graphicsbased mode.

  • QtTestBrowser/fpstimer.cpp: (FpsTimer::FpsTimer): (FpsTimer::numFrames): (FpsTimer::stop): (FpsTimer::timerEvent):
  • QtTestBrowser/fpstimer.h:
4:15 PM Changeset in webkit [70945] by rniwa@webkit.org
  • 8 edits in trunk

2010-10-29 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

Remove RenderTextControl::setSelectionRange
https://bugs.webkit.org/show_bug.cgi?id=47870

Converted RenderTextControll::setSelectionRange to a global function.

  • Api/qwebpage.cpp: (QWebPagePrivate::inputMethodEvent): Calls setSelectionRange.

2010-10-29 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

Remove RenderTextControl::setSelectionRange
https://bugs.webkit.org/show_bug.cgi?id=47870

Made RenderTextControl::setSelectionRange a global function.
Removed setSelectionStart, setSelectionEnd, and select from RenderTextControl.

No new tests are added since there is no behavioral change.

  • accessibility/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::setSelectedTextRange): Calls setSelectedTextRange.
  • dom/InputElement.cpp: (WebCore::InputElement::updateSelectionRange): Ditto.
  • html/HTMLFormControlElement.cpp: (WebCore::HTMLTextFormControlElement::setSelectionStart): Ditto. (WebCore::HTMLTextFormControlElement::setSelectionEnd): Ditto. (WebCore::HTMLTextFormControlElement::select): Ditto. (WebCore::HTMLTextFormControlElement::setSelectionRange): Ditto.
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::hasVisibleTextArea): Added. (WebCore::setSelectionRange): See above.
  • rendering/RenderTextControl.h:
3:48 PM Changeset in webkit [70944] by jamesr@google.com
  • 191 edits
    190 copies
    1 add in trunk/LayoutTests

2010-10-29 James Robinson <jamesr@chromium.org>

Update pixel expectations for Snow Leopard for tests in tables/mozilla_expected_failures.

Moves the leopard-specific mac baselines to platform/mac-leopard/ and adds new Snow Leopard pixel
expectations for 96 tests that need them. List of files omitted due to length.

3:46 PM Changeset in webkit [70943] by dpranke@chromium.org
  • 7 edits
    1 copy
    1 add in trunk/WebKitTools

2010-10-29 Dirk Pranke <dpranke@chromium.org>

Reviewed by Tony Chang.

new-run-webkit-tests: change TestResults to be serializable

In preparation for changing new-run-webkit-tests from
multithreaded to multiprocess, we need to make sure the data
going between the threads is easily serialized over a socket.

This change adds serialization/pickling for the TestResults and
TestFailure objects (using cPickle).

The TestFailure objects included a "has_wdiff" flag for Text
results, but the flag wasn't being used, so I've removed it,
simplifying the state to basically a set of enum objects with
associated methods.

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

  • Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py:
  • Scripts/webkitpy/layout_tests/layout_package/printing_unittest.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_failures.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_failures_unittest.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_results.py: Added.
  • Scripts/webkitpy/layout_tests/layout_package/test_results_unittest.py:Added.
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
  • Scripts/webkitpy/layout_tests/test_types/text_diff.py:
3:44 PM Changeset in webkit [70942] by dpranke@chromium.org
  • 3 edits in trunk/WebKitTools

2010-10-29 Dirk Pranke <dpranke@chromium.org>

Reviewed by Ojan Vafai.

new-run-webkit-tests: need to provide separate dashboard results for GPU tests
https://bugs.webkit.org/show_bug.cgi?id=48687

Modify the steps to upload the JSON files to the dashboards so
that the GPU tests don't conflict with the regular tests on a
port. We do this by modifying the --builder-name parameter to
append " - GPU", which should cause the app to treat the results
as a completely new builder. This is a little non-obvious, but
keeps us from having to restructure the app.

  • Scripts/webkitpy/layout_tests/port/chromium_gpu.py:
  • Scripts/webkitpy/layout_tests/port/chromium_gpu_unittest.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
3:39 PM Changeset in webkit [70941] by dimich@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Unreviewed test expectations update.

  • platform/chromium/test_expectations.txt:
3:36 PM Changeset in webkit [70940] by eric.carlson@apple.com
  • 3 edits in trunk/LayoutTests

2010-10-29 Eric Carlson <eric.carlson@apple.com>

Reviewed by Adam Roben.

Results for media test controls-drag-timebar.html are needlessly platform specific
https://bugs.webkit.org/show_bug.cgi?id=48662

  • media/controls-drag-timebar-expected.txt:
  • media/controls-drag-timebar.html: Remove some of the logging to avoid timing differences on slower machines.
3:32 PM Changeset in webkit [70939] by Michael Nordman
  • 4 edits in branches/chromium/552

Merge 70923 - 2010-10-28 Michael Nordman <Michael Nordman>

Reviewed by ap.

[Chrome] Fix an appcache regression introduced in r69226
https://bugs.webkit.org/show_bug.cgi?id=48592
Provide a noop method body ApplicationCacheHost::maybeLoadMainResourceForRedirect.

  • src/ApplicationCacheHost.cpp: (WebCore::ApplicationCacheHost::maybeLoadMainResourceForRedirect):

2010-10-28 Michael Nordman <Michael Nordman>

Reviewed by ap.

[Chrome] Fix an appcache regression introduced in r69226
https://bugs.webkit.org/show_bug.cgi?id=48592
Define a new ApplicationCacheHost method for the specific case of main resource redirect handling.

Tests: the existing layout test applies

  • loader/MainResourceLoader.cpp: use the newly introduced method (WebCore::MainResourceLoader::willSendRequest):
  • loader/appcache/ApplicationCacheHost.cpp: (WebCore::ApplicationCacheHost::maybeLoadMainResourceForRedirect):
  • loader/appcache/ApplicationCacheHost.h:

TBR=Michael Nordman

3:23 PM Changeset in webkit [70938] by Adam Roben
  • 5 edits in trunk/LayoutTests

Update test results for the presence of HTMLMeterElement

  • platform/win/fast/dom/Window/window-property-descriptors-expected.txt:
  • platform/win/fast/dom/prototype-inheritance-2-expected.txt:
  • platform/win/fast/dom/prototype-inheritance-expected.txt:
  • platform/win/fast/js/global-constructors-expected.txt:
3:23 PM Changeset in webkit [70937] by Adam Roben
  • 2 edits
    22 adds in trunk/LayoutTests

Stop skipping HTMLMeterElement tests on Windows and check in results for them

Fixes <http://webkit.org/b/48681>
fast/dom/HTMLMeterElement/meter-boundary-values.html fails on Windows

  • platform/win/Skipped: Removed HTMLMeterElement tests.
  • platform/win/fast/dom/HTMLMeterElement/meter-appearances-capacity-expected.checksum: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-appearances-capacity-expected.png: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-appearances-capacity-expected.txt: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-appearances-rating-relevancy-expected.checksum: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-appearances-rating-relevancy-expected.png: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-appearances-rating-relevancy-expected.txt: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-boundary-values-expected.checksum: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-boundary-values-expected.png: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-boundary-values-expected.txt: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-element-expected.checksum: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-element-expected.png: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-element-expected.txt: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-optimums-expected.checksum: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-optimums-expected.png: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-optimums-expected.txt: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-styles-changing-pseudo-expected.checksum: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-styles-changing-pseudo-expected.png: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-styles-changing-pseudo-expected.txt: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-styles-expected.checksum: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-styles-expected.png: Added.
  • platform/win/fast/dom/HTMLMeterElement/meter-styles-expected.txt: Added.
3:16 PM Changeset in webkit [70936] by andersca@apple.com
  • 8 edits in trunk/WebKit2

Use WebPage::send in a couple of more places.

Reviewed by Adam Roben.

  • WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::setWindowRect):
(WebKit::WebChromeClient::takeFocus):
(WebKit::WebChromeClient::setToolbarsVisible):
(WebKit::WebChromeClient::setStatusbarVisible):
(WebKit::WebChromeClient::setMenubarVisible):
(WebKit::WebChromeClient::setResizable):
(WebKit::WebChromeClient::setStatusbarText):
(WebKit::WebChromeClient::contentsSizeChanged):
(WebKit::WebChromeClient::mouseDidMoveOverElement):
(WebKit::WebChromeClient::setToolTip):
(WebKit::WebChromeClient::setCursor):
(WebKit::WebChromeClient::dispatchViewportDataDidChange):

  • WebProcess/WebCoreSupport/WebEditorClient.cpp:

(WebKit::WebEditorClient::registerCommandForUndo):
(WebKit::WebEditorClient::clearUndoRedoOperations):

  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::WebFrameLoaderClient::detachedFromParent2):
(WebKit::WebFrameLoaderClient::dispatchDidReceiveServerRedirectForProvisionalLoad):
(WebKit::WebFrameLoaderClient::dispatchDidStartProvisionalLoad):
(WebKit::WebFrameLoaderClient::dispatchDidReceiveTitle):
(WebKit::WebFrameLoaderClient::dispatchDidCommitLoad):
(WebKit::WebFrameLoaderClient::dispatchDidFailProvisionalLoad):
(WebKit::WebFrameLoaderClient::dispatchDidFailLoad):
(WebKit::WebFrameLoaderClient::dispatchDidFinishDocumentLoad):
(WebKit::WebFrameLoaderClient::dispatchDidFinishLoad):
(WebKit::WebFrameLoaderClient::dispatchDidFirstLayout):
(WebKit::WebFrameLoaderClient::dispatchDidFirstVisuallyNonEmptyLayout):
(WebKit::WebFrameLoaderClient::dispatchDecidePolicyForMIMEType):
(WebKit::WebFrameLoaderClient::dispatchDecidePolicyForNewWindowAction):
(WebKit::WebFrameLoaderClient::dispatchDecidePolicyForNavigationAction):
(WebKit::WebFrameLoaderClient::dispatchWillSubmitForm):
(WebKit::WebFrameLoaderClient::postProgressStartedNotification):
(WebKit::WebFrameLoaderClient::postProgressEstimateChangedNotification):
(WebKit::WebFrameLoaderClient::postProgressFinishedNotification):
(WebKit::WebFrameLoaderClient::didDisplayInsecureContent):
(WebKit::WebFrameLoaderClient::didRunInsecureContent):
(WebKit::WebFrameLoaderClient::dispatchDidBecomeFrameset):

  • WebProcess/WebPage/FindController.cpp:

(WebKit::FindController::findString):
(WebKit::FindController::hideFindIndicator):

  • WebProcess/WebPage/WebBackForwardListProxy.cpp:

(WebKit::WebBackForwardListProxy::addItem):
(WebKit::WebBackForwardListProxy::goToItem):
(WebKit::WebBackForwardListProxy::clear):

  • WebProcess/WebPage/WebFrame.cpp:

(WebKit::WebFrame::createMainFrame):
(WebKit::WebFrame::createSubframe):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::sendClose):
(WebKit::WebPage::pageDidScroll):
(WebKit::WebPage::mouseEvent):
(WebKit::WebPage::wheelEvent):
(WebKit::WebPage::keyEvent):
(WebKit::WebPage::validateMenuItem):
(WebKit::WebPage::touchEvent):
(WebKit::WebPage::show):
(WebKit::WebPage::runJavaScriptInMainFrame):
(WebKit::WebPage::getRenderTreeExternalRepresentation):
(WebKit::WebPage::getSourceForFrame):

3:15 PM Changeset in webkit [70935] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-10-29 Carlos Garcia Campos <cgarcia@igalia.com>

Reviewed by Andreas Kling.

[GTK] Crash in TextEncoding.cpp when using glib unicode
https://bugs.webkit.org/show_bug.cgi?id=48521

When the conversion to UTF-8 fails, try to encode the original string
without normalizing. Fixes tests cases:

fast/url/anchor.html
fast/url/query.html
fast/url/standard-url.html

  • platform/text/TextEncoding.cpp: (WebCore::TextEncoding::encode):
3:02 PM Changeset in webkit [70934] by andersca@apple.com
  • 8 edits
    1 add in trunk/WebKit2

Add a MessageSender class template
https://bugs.webkit.org/show_bug.cgi?id=48683

Reviewed by Adam Roben.

  • Platform/CoreIPC/MessageSender.h: Added.

(CoreIPC::MessageSender::send):
Add send overloads for simple message sending.

  • WebKit2.pro:
  • WebKit2.xcodeproj/project.pbxproj:

Add MessageSender.h

  • WebProcess/WebPage/FindController.cpp:

(WebKit::FindController::countStringMatches):
(WebKit::FindController::findString):
(WebKit::FindController::updateFindIndicator):
Call send on the web page.

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::connection):
Add connection(), used by the MessageSender.

  • WebProcess/WebPage/WebPage.h:

(WebKit::WebPage::destinationID):
Add destinationID(), used by the MessageSender.

  • win/WebKit2.vcproj:

Add MessageSender.h

3:01 PM Changeset in webkit [70933] by benjamin.poulain@nokia.com
  • 2 edits in trunk/WebCore

2010-10-29 Benjamin Poulain <benjamin.poulain@nokia.com>

Reviewed by Antonio Gomes.

TextureMapperGL get the driver info log but do not use it
https://bugs.webkit.org/show_bug.cgi?id=48247

Remove the call to glGetProgramInfoLog().

  • platform/graphics/opengl/TextureMapperGL.cpp: (WebCore::TexmapShaderInfo::createShaderProgram):
2:47 PM Changeset in webkit [70932] by commit-queue@webkit.org
  • 10 edits in trunk

2010-10-29 Benjamin Kalman <kalman@google.com>

Reviewed by Tony Chang.

Deleting contenteditable text to the left of a non-contenteditable span inserts unnecessary placeholder <br/>
https://bugs.webkit.org/show_bug.cgi?id=48658

  • editing/deleting/5390681-expected.txt: Update test for changed behaviour.
  • editing/deleting/5390681.html: Ditto (update comment).
  • editing/deleting/delete-mixed-editable-content-001-expected.txt: Ditto.
  • editing/pasteboard/copy-backslash-with-euc-expected.txt: Ditto.

2010-10-29 Benjamin Kalman <kalman@google.com>

Reviewed by Tony Chang.

Deleting contenteditable text to the left of a non-contenteditable span inserts unnecessary placeholder <br/>
https://bugs.webkit.org/show_bug.cgi?id=48658

  • WebCore.exp.in: Update is{Start,End}OfParagraph symbols.
  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::doApply): Cross editability boundaries when determining whether to insert placeholder.
  • editing/visible_units.cpp: Allow editability boundary to be crossed in {start,end,isStart,isEnd}OfParagraph. (WebCore::startOfParagraph): Also, avoid returning a position where renderer can't accept caret, like in endOfParagraph. (WebCore::endOfParagraph): (WebCore::isStartOfParagraph): (WebCore::isEndOfParagraph):
  • editing/visible_units.h: Ditto.
2:38 PM Changeset in webkit [70931] by Darin Adler
  • 23 edits in trunk/WebKit

2010-10-29 Darin Adler <Darin Adler>

Reviewed by Sam Weinig.

Change BackForwardList clients to use BackForwardListImpl to prepare for further refactoring
https://bugs.webkit.org/show_bug.cgi?id=48574

  • src/BackForwardListClientImpl.h: Use BackForwardListImpl.h.
  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::WebViewImpl): Cast to BackForwardList* before calling setClient.

2010-10-29 Darin Adler <Darin Adler>

Reviewed by Sam Weinig.

Change BackForwardList clients to use BackForwardListImpl to prepare for further refactoring
https://bugs.webkit.org/show_bug.cgi?id=48574

  • Api/qwebhistory.cpp: (QWebHistory::clear):
  • Api/qwebhistory_p.h:
  • Api/qwebpage.cpp: (QWebPagePrivate::QWebPagePrivate): Use BackForwardListImpl.

2010-10-29 Darin Adler <Darin Adler>

Reviewed by Sam Weinig.

Change BackForwardList clients to use BackForwardListImpl to prepare for further refactoring
https://bugs.webkit.org/show_bug.cgi?id=48574

  • webkit/webkitprivate.h:
  • webkit/webkitwebbackforwardlist.cpp: (webkit_web_back_forward_list_dispose): (webkit_web_back_forward_list_new_with_web_view): (webkit_web_back_forward_list_go_forward): (webkit_web_back_forward_list_go_back): (webkit_web_back_forward_list_contains_item): (webkit_web_back_forward_list_go_to_item): (webkit_web_back_forward_list_get_forward_list_with_limit): (webkit_web_back_forward_list_get_back_list_with_limit): (webkit_web_back_forward_list_get_back_item): (webkit_web_back_forward_list_get_current_item): (webkit_web_back_forward_list_get_forward_item): (webkit_web_back_forward_list_get_nth_item): (webkit_web_back_forward_list_get_back_length): (webkit_web_back_forward_list_get_forward_length): (webkit_web_back_forward_list_get_limit): (webkit_web_back_forward_list_set_limit): (webkit_web_back_forward_list_add_item): (webkit_web_back_forward_list_clear): (WebKit::core):
  • webkit/webkitwebview.cpp: (webkit_web_view_set_maintains_back_forward_list): (webkit_web_view_get_back_forward_list): Use BackForwardListImpl.

2010-10-29 Darin Adler <Darin Adler>

Reviewed by Sam Weinig.

Change BackForwardList clients to use BackForwardListImpl to prepare for further refactoring
https://bugs.webkit.org/show_bug.cgi?id=48574

  • WebBackForwardList.cpp: (backForwardListWrappers): (WebBackForwardList::WebBackForwardList): (WebBackForwardList::createInstance):
  • WebBackForwardList.h:
  • WebView.cpp: (WebView::backForwardList): Use BackForwardListImpl.

2010-10-29 Darin Adler <Darin Adler>

Reviewed by Sam Weinig.

Change BackForwardList clients to use BackForwardListImpl to prepare for further refactoring
https://bugs.webkit.org/show_bug.cgi?id=48574

  • WebFrame.cpp: (wxWebFrame::CanGoBack): Use canGoBackOrForward. (wxWebFrame::CanGoForward): Ditto.

2010-10-29 Darin Adler <Darin Adler>

Reviewed by Sam Weinig.

Change BackForwardList clients to use BackForwardListImpl to prepare for further refactoring
https://bugs.webkit.org/show_bug.cgi?id=48574

  • History/WebBackForwardList.mm: Use BackForwardListImpl.

2010-10-29 Darin Adler <Darin Adler>

Reviewed by Sam Weinig.

Change BackForwardList clients to use BackForwardListImpl to prepare for further refactoring
https://bugs.webkit.org/show_bug.cgi?id=48574

  • ewk/ewk_history.cpp: (ewk_history_new):
  • ewk/ewk_private.h:
  • ewk/ewk_view.cpp: (_ewk_view_priv_new): (ewk_view_history_enable_get): (ewk_view_history_enable_set): (ewk_view_history_get): Use BackForwardListImpl.
2:31 PM Changeset in webkit [70930] by andersca@apple.com
  • 2 edits in trunk/WebKit2

Add the plug-in proxy to the map before sending the CreatePlugin message
https://bugs.webkit.org/show_bug.cgi?id=48678

Reviewed by Adam Roben.

  • WebProcess/Plugins/PluginProxy.cpp:

(WebKit::PluginProxy::initialize):

2:28 PM Changeset in webkit [70929] by pfeldman@chromium.org
  • 7 edits in trunk/WebCore

2010-10-29 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: last active panel is not restored upon opening inspector.
https://bugs.webkit.org/show_bug.cgi?id=48668

  • inspector/Inspector.idl:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::inspect): (WebCore::InspectorController::showPanel): (WebCore::InspectorController::disconnectFrontend): (WebCore::InspectorController::populateScriptObjects):
  • inspector/InspectorController.h:
  • inspector/InspectorState.cpp: (WebCore::InspectorState::InspectorState):
  • inspector/front-end/Settings.js: (WebInspector.Settings):
  • inspector/front-end/inspector.js: (WebInspector.set currentPanel): (WebInspector.doLoadedDone.onPopulateScriptObjects): (WebInspector.doLoadedDone): (WebInspector.documentClick.followLink): (WebInspector.documentClick):
2:26 PM Changeset in webkit [70928] by Adam Roben
  • 3 edits in trunk/WebKit2

Remove redundant data members from WebTouchEvent

WebEvent already stores the Type and timestamp.

Fixes <http://webkit.org/b/48671> WebTouchEvent duplicates data from
WebEvent

Reviewed by Anders Carlsson.

  • Shared/WebEvent.h:
  • Shared/WebTouchEvent.cpp:

(WebKit::WebTouchEvent::WebTouchEvent):
Removed m_type and m_timestamp (which was never even getting
initialized!).

2:24 PM Changeset in webkit [70927] by andersca@apple.com
  • 5 edits in trunk/WebKit2

Add GetWindowScriptNPObject message
https://bugs.webkit.org/show_bug.cgi?id=48670

Reviewed by Adam Roben.

  • PluginProcess/PluginControllerProxy.cpp:

(WebKit::PluginControllerProxy::windowScriptNPObject):
Send the GetWindowScriptNPObject message.

  • WebProcess/Plugins/PluginProxy.cpp:

(WebKit::PluginProxy::getWindowScriptNPObject):
Add stub.

  • WebProcess/Plugins/PluginProxy.messages.in:

Add GetWindowScriptNPObject message.

2:18 PM Changeset in webkit [70926] by eric.carlson@apple.com
  • 3 edits
    1 add
    7 deletes in trunk/LayoutTests

2010-10-29 Eric Carlson <eric.carlson@apple.com>

Reviewed by Adam Roben.

Results for media test controls-drag-timebar.html are needlessly platform specific
https://bugs.webkit.org/show_bug.cgi?id=48662

  • media/controls-drag-timebar-expected.txt: Added.
  • media/controls-drag-timebar.html: Remove all platform specific logging.
  • platform/chromium-mac/media/controls-drag-timebar-expected.txt: Removed.
  • platform/chromium-win/media/controls-drag-timebar-expected.txt: Removed.
  • platform/gtk/media/controls-drag-timebar-expected.txt: Removed.
  • platform/mac-leopard/media/controls-drag-timebar-expected.txt: Removed.
  • platform/mac-snowleopard/Skipped: Unskip test.
  • platform/mac-tiger/media/controls-drag-timebar-expected.txt: Removed.
  • platform/mac/media/controls-drag-timebar-expected.txt: Removed.
  • platform/win/media/controls-drag-timebar-expected.txt: Removed.
2:07 PM Changeset in webkit [70925] by dimich@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Unreviewed test expectations update.

  • platform/chromium/test_expectations.txt:
2:06 PM Changeset in webkit [70924] by ojan@chromium.org
  • 2 edits in trunk/WebKitTools

2010-10-29 Ojan Vafai <ojan@chromium.org>

Reviewed by Tony Chang.

[chromium] add debug logging to help diagnose flakiness dashboard issues
https://bugs.webkit.org/show_bug.cgi?id=48657

The appengine app thinks it's getting empty files uploaded. Add some logging
to see if new-run-webkit-tests agrees.

  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
2:06 PM Changeset in webkit [70923] by Michael Nordman
  • 6 edits in trunk

2010-10-28 Michael Nordman <Michael Nordman>

Reviewed by ap.

[Chrome] Fix an appcache regression introduced in r69226
https://bugs.webkit.org/show_bug.cgi?id=48592
Provide a noop method body ApplicationCacheHost::maybeLoadMainResourceForRedirect.

  • src/ApplicationCacheHost.cpp: (WebCore::ApplicationCacheHost::maybeLoadMainResourceForRedirect):

2010-10-28 Michael Nordman <Michael Nordman>

Reviewed by ap.

[Chrome] Fix an appcache regression introduced in r69226
https://bugs.webkit.org/show_bug.cgi?id=48592
Define a new ApplicationCacheHost method for the specific case of main resource redirect handling.

Tests: the existing layout test applies

  • loader/MainResourceLoader.cpp: use the newly introduced method (WebCore::MainResourceLoader::willSendRequest):
  • loader/appcache/ApplicationCacheHost.cpp: (WebCore::ApplicationCacheHost::maybeLoadMainResourceForRedirect):
  • loader/appcache/ApplicationCacheHost.h:
2:00 PM Changeset in webkit [70922] by eric.carlson@apple.com
  • 3 edits
    2 adds in trunk

2010-10-29 Eric Carlson <eric.carlson@apple.com>

Reviewed by Adam Roben.

Navigating to a .wav file creates a plugin document but should create a media document
https://bugs.webkit.org/show_bug.cgi?id=48546
<rdar://problem/8606105>

Test: media/media-can-play-wav-audio.html

  • platform/MIMETypeRegistry.cpp: Add mapping from ".wav" to "audio/x-wav".

2010-10-28 Eric Carlson <eric.carlson@apple.com>

Reviewed by Adam Roben.

Navigating to a .wav file creates a plugin document but should create a media document
https://bugs.webkit.org/show_bug.cgi?id=48546
<rdar://problem/8606105>

  • media/media-can-play-wav-audio-expected.txt: Added.
  • media/media-can-play-wav-audio.html: Added.
1:52 PM Changeset in webkit [70921] by mihaip@chromium.org
  • 96 edits
    94 copies in trunk/LayoutTests

2010-10-29 Mihai Parparita <mihaip@chromium.org>

Update the pixel expectations for fast/block/float for the Mac port.

For the tests that currently fail on Snow Leopard with --pixel-tests
--tolerance 0, this moves the current pixel results to mac-leopard and
adds new Snow Leopard-specific results to platform/mac.

List of files omitted due to length but composed entirely of moves from
platform/mac to platform/mac-leopard and newly added files in
platform/mac.

platform/mac/test_expectations.txt was also modified to remove a
now-passing test.

1:51 PM Changeset in webkit [70920] by andersca@apple.com
  • 12 edits in trunk/WebKit2

Change the Connection::sendSync timeout parameter to have the NoTimeout argument as a default
argument and update all the call sites.

Reviewed by Dan Bernstein.

  • Platform/CoreIPC/Connection.h:
  • PluginProcess/PluginControllerProxy.cpp:

(WebKit::PluginControllerProxy::proxiesForURL):
(WebKit::PluginControllerProxy::cookiesForURL):

  • WebProcess/InjectedBundle/InjectedBundle.cpp:

(WebKit::InjectedBundle::postSynchronousMessage):

  • WebProcess/Plugins/PluginProcessConnectionManager.cpp:

(WebKit::PluginProcessConnectionManager::getPluginProcessConnection):

  • WebProcess/Plugins/PluginProxy.cpp:

(WebKit::PluginProxy::initialize):
(WebKit::PluginProxy::destroy):
(WebKit::PluginProxy::paint):
(WebKit::PluginProxy::handleMouseEvent):
(WebKit::PluginProxy::handleWheelEvent):
(WebKit::PluginProxy::handleMouseEnterEvent):
(WebKit::PluginProxy::handleMouseLeaveEvent):
(WebKit::PluginProxy::handleKeyboardEvent):

  • WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::windowRect):
(WebKit::WebChromeClient::createWindow):
(WebKit::WebChromeClient::toolbarsVisible):
(WebKit::WebChromeClient::statusbarVisible):
(WebKit::WebChromeClient::menubarVisible):
(WebKit::WebChromeClient::canRunBeforeUnloadConfirmPanel):
(WebKit::WebChromeClient::runBeforeUnloadConfirmPanel):
(WebKit::WebChromeClient::runJavaScriptAlert):
(WebKit::WebChromeClient::runJavaScriptConfirm):
(WebKit::WebChromeClient::runJavaScriptPrompt):

  • WebProcess/WebCoreSupport/WebPlatformStrategies.cpp:

(WebKit::WebPlatformStrategies::populatePluginCache):

  • WebProcess/WebPage/WebBackForwardListProxy.cpp:

(WebKit::WebBackForwardListProxy::itemAtIndex):
(WebKit::WebBackForwardListProxy::backListCount):
(WebKit::WebBackForwardListProxy::forwardListCount):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::createPlugin):
(WebKit::WebPage::changeAcceleratedCompositingMode):

  • WebProcess/WebPage/mac/LayerBackedDrawingAreaMac.mm:

(WebKit::LayerBackedDrawingArea::attachCompositingContext):

  • WebProcess/WebPage/mac/WebPageMac.mm:

(WebKit::WebPage::interceptEditingKeyboardEvent):

1:49 PM Changeset in webkit [70919] by commit-queue@webkit.org
  • 7 edits
    6 adds in trunk

2010-10-29 Aaron Colwell <acolwell@chromium.org>

Reviewed by James Robinson.

Added a layout test for verifying that globalAlpha is properly handled
when copying a video frame to a 2D canvas context.
https://bugs.webkit.org/show_bug.cgi?id=48094

  • media/video-canvas-alpha-expected.txt: Added.
  • media/video-canvas-alpha.html: Added.
  • platform/chromium-mac/media/video-canvas-alpha-expected.checksum: Added.
  • platform/chromium-mac/media/video-canvas-alpha-expected.png: Added.
  • platform/chromium-win/media/video-canvas-alpha-expected.checksum: Added.
  • platform/chromium-win/media/video-canvas-alpha-expected.png: Added.

2010-10-29 Aaron Colwell <acolwell@chromium.org>

Reviewed by James Robinson.

Created helper method WebCore::PlatformContextSkia::getNormalizedAlpha()
to centralize alpha rounding & clipping logic.
https://bugs.webkit.org/show_bug.cgi?id=48094

Test: media/video-canvas-alpha.html

  • platform/graphics/skia/ImageSkia.cpp: (WebCore::paintSkBitmap):
  • platform/graphics/skia/PlatformContextSkia.cpp: (WebCore::PlatformContextSkia::getNormalizedAlpha):
  • platform/graphics/skia/PlatformContextSkia.h:

2010-10-29 Aaron Colwell <acolwell@chromium.org>

Reviewed by James Robinson.

Fix globalAlpha support when using drawImage() to copy a video frame
to a 2D canvas context.
https://bugs.webkit.org/show_bug.cgi?id=48094

This fix applies the globalAlpha value to the canvas before passing it
down to the lower layers that don't have access to the graphics
context. This makes sure that any drawing on the canvas will have the
proper global alpha value applied.

  • src/WebMediaPlayerClientImpl.cpp: (WebKit::WebMediaPlayerClientImpl::paint):
1:33 PM Changeset in webkit [70918] by Nikolas Zimmermann
  • 8 edits
    2 adds in trunk

2010-10-29 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Dirk Schulze.

ASSERT when SVGAnimatedProperty instance is assigned to multiple elements.
https://bugs.webkit.org/show_bug.cgi?id=48555

Test: svg/custom/insertItemBefore-from-non-list-origin.svg

  • bindings/scripts/CodeGeneratorJS.pm: Include SVGAnimatedPropertyTearOff.h instead of SVGPropertyTearOff.h
  • bindings/scripts/CodeGeneratorObjC.pm: Ditto.
  • bindings/scripts/CodeGeneratorV8.pm: Ditto.
  • svg/properties/SVGAnimatedListPropertyTearOff.h: (WebCore::SVGAnimatedListPropertyTearOff::isAnimatedListTearOff): Return true. (WebCore::SVGAnimatedListPropertyTearOff::removeItemFromList): Devirtualize function.
  • svg/properties/SVGAnimatedProperty.h: (WebCore::SVGAnimatedProperty::isAnimatedListTearOff): Return false.
  • svg/properties/SVGListPropertyTearOff.h: (WebCore::SVGListPropertyTearOff::removeItemFromListIfNeeded): Only try to remove the passed item from its list, if its actually a list and not just a SVGAnimatedProperty.

2010-10-29 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Dirk Schulze.

ASSERT when SVGAnimatedProperty instance is assigned to multiple elements.
https://bugs.webkit.org/show_bug.cgi?id=48555

  • svg/custom/insertItemBefore-from-non-list-origin-expected.txt: Added.
  • svg/custom/insertItemBefore-from-non-list-origin.svg: Added.
1:25 PM Changeset in webkit [70917] by hyatt@apple.com
  • 9 edits in trunk/WebCore

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

Reviewed by Adam Roben.

Fix the dirty rect checks in RenderBoxLineList, InlineFlowBox, InlineTextBox. They were still assuming horizontal-tb writing-mode.
This patch makes the checks directionally abstract and also refactors the checks in RenderLineBoxList to share code.

No tests yet, since the invalidation code for inlines still assumes horizontal-tb. That will be fixed in a follow-up bug,
and then repaint tests will be easy to write.

  • rendering/InlineBox.cpp:

(WebCore::InlineBox::adjustForFlippedBlocksWritingMode):

  • rendering/InlineBox.h:
  • rendering/InlineFlowBox.cpp:

(WebCore::InlineFlowBox::paint):

  • rendering/InlineTextBox.cpp:

(WebCore::InlineTextBox::paint):

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::convertFromFlippedWritingMode):

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

(WebCore::RenderLineBoxList::rangeIntersectsDirtyRect):
(WebCore::RenderLineBoxList::anyLineIntersectsDirtyRect):
(WebCore::RenderLineBoxList::lineIntersectsDirtyRect):
(WebCore::RenderLineBoxList::paint):
(WebCore::RenderLineBoxList::hitTest):

  • rendering/RenderLineBoxList.h:
1:06 PM Changeset in webkit [70916] by sullivan@apple.com
  • 5 edits in trunk/WebKit2

https://bugs.webkit.org/show_bug.cgi?id=48666
WebKit2 should provide counterpart getter to WKBundleNodeHandleSetHTMLInputElementAutofilled

Reviewed by Anders Carlsson.

  • WebProcess/InjectedBundle/API/c/WKBundleNodeHandle.cpp:

(WKBundleNodeHandleGetHTMLInputElementAutofilled):
New function, calls through to InjectedBundleNodeHandle.

  • WebProcess/InjectedBundle/API/c/WKBundleNodeHandlePrivate.h:

Declared new function.

  • WebProcess/InjectedBundle/DOM/InjectedBundleNodeHandle.cpp:

(WebKit::InjectedBundleNodeHandle::isHTMLInputElementAutofilled):
New function, calls through to HTMLInputElement.

  • WebProcess/InjectedBundle/DOM/InjectedBundleNodeHandle.h:

Declared new function.

12:30 PM Changeset in webkit [70915] by jberlin@webkit.org
  • 2 edits in trunk/WebKit2

ASSERTION FAILURE in WebProcessProxy::frameDestroyed when a Page is closed before it can send
the DidCreateMainFrame messsage to the UIProcess.
https://bugs.webkit.org/show_bug.cgi?id=48654

Reviewed by Anders Carlsson.

WebProcessProxy::frameDestroyed was asserting that the frameID it was called with was
registered in WebProcessProxy::frameCreated (which is called when the WebPageProxy is
created). However, in the case where we close the page before it's had the chance to send
the DidCreateMainFrame message back to the UI process, the UI Process will get the
DidDestroyFrame message because it's sent directly to the WebProcessProxy and doesn't go
through WebPageProxy.

Remove the assert because it is invalid in this case.

  • UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::frameDestroyed):

12:13 PM Changeset in webkit [70914] by Patrick Gansterer
  • 2 edits in trunk/WebCore

2010-10-29 Patrick Gansterer <Patrick Gansterer>

Reviewed by Adam Roben.

Cleanup createGlobalImageFileDescriptor in ClipboardWin
https://bugs.webkit.org/show_bug.cgi?id=48189

  • platform/win/ClipboardWin.cpp: (WebCore::createGlobalURLContent): (WebCore::createGlobalUrlFileDescriptor): (WebCore::ClipboardWin::writeURL):
12:00 PM Changeset in webkit [70913] by mihaip@chromium.org
  • 4 edits in trunk/LayoutTests

2010-10-29 Mihai Parparita <mihaip@chromium.org>

Reviewed by Tony Chang.

[Chromium] fast/loader/stateobjects/state-api-on-detached-frame-crash.html does not pass on Chromium
https://bugs.webkit.org/show_bug.cgi?id=48647

Chromium throws an exception if calling push/replaceState on a detached
frame, while the mac port fails silently (presumably a JSC vs. V8
difference). Either behavior is correct (as long as we don't crash),
make the test handle both.

  • fast/loader/stateobjects/state-api-on-detached-frame-crash-expected.txt:
  • fast/loader/stateobjects/state-api-on-detached-frame-crash.html:
  • platform/chromium/test_expectations.txt:
12:00 PM Changeset in webkit [70912] by dimich@chromium.org
  • 2 edits in trunk/WebCore

[Chromium clang] Unreviewed fix removing compiler warning.
If GIFImageReader is a struct, don't call it a class.

  • platform/image-decoders/gif/GIFImageDecoder.h:
11:56 AM Changeset in webkit [70911] by ap@apple.com
  • 2 edits in trunk/WebKit/mac

Rubber-stamped by Sam Weinig.

Comment for usesDocumentViews explains it backwards. This flag is true in ToT, and we do
create views for subframes.

  • WebView/WebViewData.h: Double negation is better than a lie.
11:55 AM Changeset in webkit [70910] by oliver@apple.com
  • 4 edits
    3 adds in trunk

2010-10-29 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

REGRESSION: r69429-r69611: Crash in JSC::Interpreter::privateExecute
https://bugs.webkit.org/show_bug.cgi?id=47573

I think the interpreter portion of this was introduced by
an incorrect but silent merge when I updated prior to committing.
The JIT change is basically just a correctness fix, but it is
needed to prevent the testcase from asserting in debug builds.

The basic problem is incorrectly setting the activation object
on an arguments object. The crash was due to us setting a null
activation in the interpreter, in the jit we were setting the
activation of a strict mode arguments object.

  • interpreter/Interpreter.cpp: (JSC::Interpreter::privateExecute):
  • jit/JITStubs.cpp: (JSC::DEFINE_STUB_FUNCTION):
  • wtf/Platform.h:

2010-10-29 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

REGRESSION: r69429-r69611: Crash in JSC::Interpreter::privateExecute
https://bugs.webkit.org/show_bug.cgi?id=47573

Add a test to ensure that we don't incorrectly set a null activation
as an argument object's activation.

  • fast/js/interpreter-no-activation-expected.txt: Added.
  • fast/js/interpreter-no-activation.html: Added.
  • fast/js/script-tests/interpreter-no-activation.js: Added.
11:48 AM Changeset in webkit [70909] by jamesr@google.com
  • 5 edits
    2 copies in branches/chromium/517

Merge 70748 - 2010-10-27 Andy Estes <aestes@apple.com>

Reviewed by Adam Barth.

r66156 broke AtlasCT library, formerly affected http://map.d.co.il/
https://bugs.webkit.org/show_bug.cgi?id=45679

This patch removes WebKit's map of classid values to MIME types. It also
changes the behavior of object elements to render fallback content when
a non-empty classid attribute is specified, which is the behavior HTML5
specifies when a UA encounters a classid it doesn't understand.

Test: fast/replaced/object-with-non-empty-classid-triggers-fallback.html

  • html/HTMLObjectElement.cpp: Remove serviceTypeForClassId(), createClassIdToTypeMap(), and the ClassIdToTypeMap typedef. (WebCore::HTMLObjectElement::updateWidget): Do not call serviceTypeForClassId() when there is no type attribute, and render fallback content if the classid attribute is non-empty.

2010-10-27 Andy Estes <aestes@apple.com>

Reviewed by Adam Barth.

r66156 broke AtlasCT library, formerly affected http://map.d.co.il/
https://bugs.webkit.org/show_bug.cgi?id=45679

  • fast/dom/object-embed-plugin-scripting.html: Changed classid attributes to type attributes.
  • fast/dom/object-plugin-hides-properties.html: Ditto.
  • fast/replaced/object-with-non-empty-classid-triggers-fallback-expected.txt: Added.
  • fast/replaced/object-with-non-empty-classid-triggers-fallback.html: Added.

TBR=cpu@google.com
BUG=57986

Review URL: http://codereview.chromium.org/4214004

11:43 AM Changeset in webkit [70908] by mihaip@chromium.org
  • 2 edits in trunk/LayoutTests

2010-10-29 Mihai Parparita <mihaip@chromium.org>

Unreviewed cleanp of platform/chromium/test_expectations.txt

Group http/tests/navigation failures together, and differentiate between
flaky/failing tests and those that need updated baselines (though really
they shouldn't be pixel tests in the first place).

Also enable http/tests/navigation/post-goback-repost-policy.html, it
seems to be passing now (we've enabled sessionStorage for a while).

  • platform/chromium/test_expectations.txt:
11:40 AM Changeset in webkit [70907] by ap@apple.com
  • 2 edits in trunk/WebCore

Fix the world (d'oh!)

https://bugs.webkit.org/show_bug.cgi?id=48576
Let WebKit2 client know when a frame is a frameset

  • html/HTMLFrameSetElement.cpp: (WebCore::HTMLFrameSetElement::insertedIntoDocument): (WebCore::HTMLFrameSetElement::removedFromDocument): Call base class version.
11:36 AM Changeset in webkit [70906] by jianli@chromium.org
  • 2 edits in trunk/WebCore

Unreviewed. Fix chromium build break.

  • bindings/v8/custom/V8FileReaderCustom.cpp:

(WebCore::V8FileReader::resultAccessorGetter):

11:28 AM Changeset in webkit [70905] by Dimitri Glazkov
  • 2 edits in trunk/WebKitTools

2010-10-29 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Ojan Vafai.

[Chromium/DRT] Add master-name flag to new-run-webkit-tests.
https://bugs.webkit.org/show_bug.cgi?id=48649

  • BuildSlaveSupport/build.webkit.org-config/master.cfg: Added flag.
11:23 AM Changeset in webkit [70904] by jianli@chromium.org
  • 26 edits
    2 copies
    2 adds in trunk

Support readAsArrayBuffer in FileReader and FileReaderSync
https://bugs.webkit.org/show_bug.cgi?id=47691

Reviewed by David Levin.

WebCore:

Merge the similar code in FileReader and FileReaderSync into
FileReaderLoader.

Test cases have been added to read-file-test-cases.js.

  • Android.jscbindings.mk:
  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/v8/custom/V8FileReaderCustom.cpp: Added.
  • fileapi/FileReader.cpp:

(WebCore::FileReader::FileReader):
(WebCore::FileReader::readAsArrayBuffer):
(WebCore::FileReader::readAsBinaryString):
(WebCore::FileReader::readAsText):
(WebCore::FileReader::readAsDataURL):
(WebCore::FileReader::readInternal):
(WebCore::FileReader::doAbort):
(WebCore::FileReader::terminate):
(WebCore::FileReader::start):
(WebCore::FileReader::didStartLoading):
(WebCore::FileReader::didReceiveData):
(WebCore::FileReader::didFinishLoading):
(WebCore::FileReader::didFail):
(WebCore::FileReader::fireEvent):
(WebCore::FileReader::arrayBufferResult):
(WebCore::FileReader::stringResult):

  • fileapi/FileReader.h:

(WebCore::FileReader::readType):

  • fileapi/FileReader.idl:
  • fileapi/FileReaderLoader.cpp: Added.
  • fileapi/FileReaderLoader.h: Added.
  • fileapi/FileReaderLoaderClient.h: Added.
  • fileapi/FileReaderSync.cpp:

(WebCore::FileReaderSync::FileReaderSync):
(WebCore::FileReaderSync::readAsArrayBuffer):
(WebCore::FileReaderSync::readAsBinaryString):
(WebCore::FileReaderSync::readAsText):
(WebCore::FileReaderSync::readAsDataURL):
(WebCore::FileReaderSync::startLoading):

  • fileapi/FileReaderSync.h:
  • fileapi/FileReaderSync.idl:
  • html/canvas/ArrayBuffer.cpp:

(WebCore::ArrayBuffer::create):

  • html/canvas/ArrayBuffer.h:
  • loader/ThreadableLoaderClient.h:

LayoutTests:

  • fast/files/read-file-async-expected.txt:
  • fast/files/resources/read-common.js:

(readBlobAsArrayBuffer):
(_readBlobAsArrayBufferAsync):
(_readBlobAsArrayBufferSync):
(_readBlobAsBinaryStringSync):
(_readBlobAsTextSync):
(_readBlobAsDataURLSync):
(logResult):
(loaded):
(_toString):
(_isASCIIString):
(_toHexadecimal):
(_getElementAt):

  • fast/files/resources/read-file-test-cases.js:

(testReadingNonExistentFileAsArrayBuffer):
(testReadingEmptyFileAsArrayBuffer):
(testReadingUTF8EncodedFileAsArrayBuffer):
(testReadingBinaryFileAsArrayBuffer):
(testMultipleReads):

  • fast/files/workers/resources/worker-read-file-sync.js:

(testMultipleReadsSync):

  • fast/files/workers/worker-read-blob-sync-expected.txt:
  • fast/files/workers/worker-read-file-async-expected.txt:
  • fast/files/workers/worker-read-file-sync-expected.txt:
11:00 AM Changeset in webkit [70903] by Adam Roben
  • 2 edits in trunk/WebKit2

Build fix when Carbon plugins are disabled

  • WebProcess/Plugins/Netscape/mac/NetscapePluginMac.mm:

(WebKit::NetscapePlugin::platformHandleMouseEvent):

10:57 AM Changeset in webkit [70902] by Adam Roben
  • 2 edits in trunk/WebKit/win

Windows build fix

  • WebKitPrefix.h: Touched to force a rebuild.
10:53 AM Changeset in webkit [70901] by jer.noble@apple.com
  • 3 edits in trunk/WebKit/mac

2010-10-27 Jer Noble <jer.noble@apple.com>

Reviewed by Darin Adler.

Full screen video in Safari (still) exits full-screen if you switch to another
app on a multi-display system.
https://bugs.webkit.org/show_bug.cgi?id=47364
<rdar://problem/8382299>


The FullScreen window does not technically close when it deactivates, but the
two NSWindows that make up the FullScreen feature should not hide when deactivated.


  • WebView/WebVideoFullscreenController.mm: (createBackgroundFullscreenWindow): Do not set hidesOnDeactivate. (-[WebVideoFullscreenWindow initWithContentRect:styleMask:backing:defer:]): Ditto.
  • WebView/WebVideoFullscreenHUDWindowController.mm: (-[WebVideoFullscreenHUDWindow initWithContentRect:styleMask:backing:defer:]): Ditto.
10:50 AM Changeset in webkit [70900] by jschuh@chromium.org
  • 1 edit in branches/chromium/552/WebCore/html/HTMLMediaElement.cpp

Merge 70065 - 2010-10-19 Eric Carlson <eric.carlson@apple.com>

Not reviewed. Build fix.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::selectNextSourceChild): Initialize variable to avoid Windows "potentially uninitialized local variable" warning.

BUG=59593
TBR=jschuh@chromium.org

10:47 AM Changeset in webkit [70899] by jschuh@chromium.org
  • 5 edits
    4 copies in branches/chromium/552

Merge 70063 - 2010-10-19 Eric Carlson <eric.carlson@apple.com>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=46763
CRASH in WebCore::ThreadTimers::sharedTimerFiredInternal

Fix crashes caused by moving and deleting <source> element(s) of active media element.

Tests: media/video-source-moved.html

media/video-source-removed.html

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::HTMLMediaElement): Add logging. Initialize selectNextSourceChild. (WebCore::HTMLMediaElement::~HTMLMediaElement): Ditto. (WebCore::HTMLMediaElement::insertedIntoDocument): Ditto. (WebCore::HTMLMediaElement::removedFromDocument): Ditto. (WebCore::HTMLMediaElement::scheduleLoad): Ditto. (WebCore::HTMLMediaElement::setNetworkState): Deal with m_currentSourceNode being null when the media engine signals a failure by skipping the error message and continuing as usual. (WebCore::HTMLMediaElement::setVolume): Fix logging typo. (WebCore::HTMLMediaElement::havePotentialSourceChild): Save and restore m_nextChildNodeToConsider around call to selectNextSourceChild because they are both significant. (WebCore::HTMLMediaElement::selectNextSourceChild): Use m_nextChildNodeToConsider to pick the first node to consider. Bail immediately if it signals that we have already processed every <source> node. Stach the node following the current source element in m_nextChildNodeToConsider so we can resume the search even if m_currentSourceNode is removed while it is being processed. (WebCore::HTMLMediaElement::sourceWasAdded): New, move logic from HTMLSourceElement::insertedIntoTree here and correct it to deal with a <source> node being inserted immediately after the current <source> node and a new <source> node being inserted at the end of the list after all candidates have failed. (WebCore::HTMLMediaElement::sourceWillBeRemoved): New, deal with current source node and next potential node being removed.
  • html/HTMLMediaElement.h: (WebCore::HTMLMediaElement::sourceChildEndOfListValue): New, define sentinal value used to indicate that all nodes have been processed.
  • html/HTMLSourceElement.cpp: (WebCore::HTMLSourceElement::HTMLSourceElement): Add logging. (WebCore::HTMLSourceElement::insertedIntoTree): Call mediaElement->sourceWasAdded instead of having logic here. (WebCore::HTMLSourceElement::willRemove): New, call mediaElement->sourceWillBeRemoved (WebCore::HTMLSourceElement::scheduleErrorEvent): Add logging. (WebCore::HTMLSourceElement::cancelPendingErrorEvent): Add logging.
  • html/HTMLSourceElement.h:

2010-10-19 Eric Carlson <eric.carlson@apple.com>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=46763
CRASH in WebCore::ThreadTimers::sharedTimerFiredInternal

Test moving and deleting <source> element(s) of active <video>.

  • media/media-file.js: (mimeTypeForExtension): New, added to make it possible to set 'type' attribute on <source>.
  • media/video-source-moved-expected.txt: Added.
  • media/video-source-moved.html: Added.
  • media/video-source-removed-expected.txt: Added.
  • media/video-source-removed.html: Added.

BUG=59593
TBR=jschuh@chromium.org

10:45 AM Changeset in webkit [70898] by Adam Roben
  • 4 edits
    10 moves in trunk/WebKit2

Move NPRuntime-related files into WebProcess/Plugins/Netscape

Fixes <http://webkit.org/b/48645>.

Reviewed by Anders Carlsson.

  • WebKit2.pro:
  • WebKit2.xcodeproj/project.pbxproj:
  • win/WebKit2.vcproj:

Updated the files' paths.

  • WebProcess/Plugins/Netscape/JSNPMethod.cpp: Renamed from WebKit2/WebProcess/Plugins/JSNPMethod.cpp.
  • WebProcess/Plugins/Netscape/JSNPMethod.h: Renamed from WebKit2/WebProcess/Plugins/JSNPMethod.h.
  • WebProcess/Plugins/Netscape/JSNPObject.cpp: Renamed from WebKit2/WebProcess/Plugins/JSNPObject.cpp.
  • WebProcess/Plugins/Netscape/JSNPObject.h: Renamed from WebKit2/WebProcess/Plugins/JSNPObject.h.
  • WebProcess/Plugins/Netscape/NPJSObject.cpp: Renamed from WebKit2/WebProcess/Plugins/NPJSObject.cpp.
  • WebProcess/Plugins/Netscape/NPJSObject.h: Renamed from WebKit2/WebProcess/Plugins/NPJSObject.h.
  • WebProcess/Plugins/Netscape/NPRuntimeObjectMap.cpp: Renamed from WebKit2/WebProcess/Plugins/NPRuntimeObjectMap.cpp.
  • WebProcess/Plugins/Netscape/NPRuntimeObjectMap.h: Renamed from WebKit2/WebProcess/Plugins/NPRuntimeObjectMap.h.
  • WebProcess/Plugins/Netscape/NPRuntimeUtilities.cpp: Renamed from WebKit2/WebProcess/Plugins/NPRuntimeUtilities.cpp.
  • WebProcess/Plugins/Netscape/NPRuntimeUtilities.h: Renamed from WebKit2/WebProcess/Plugins/NPRuntimeUtilities.h.
10:44 AM Changeset in webkit [70897] by jschuh@chromium.org
  • 1 edit in branches/chromium/552/WebCore/html/HTMLMediaElement.cpp

Merge 69514 - 2010-10-11 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Eric Carlson.

CRASH at WebCore::HTMLSourceElement::scheduleErrorEvent
https://bugs.webkit.org/show_bug.cgi?id=46777

Another case that could lead to an improper set of states between
m_currentNode and m_loadingState. Also added an ASSERT and early
return to prevent crashes if this case can still happen in other ways.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::prepareForLoad):

BUG=59593
TBR=jschuh@chromium.org

10:32 AM Changeset in webkit [70896] by tony@chromium.org
  • 7 edits in trunk/WebCore

2010-10-29 Tony Chang <tony@chromium.org>

Unreviewed, rolling out r70850.
http://trac.webkit.org/changeset/70850
https://bugs.webkit.org/show_bug.cgi?id=45640

chromium and qt pixel test regressions

  • dom/Document.cpp:
  • dom/Document.h:
  • page/FrameView.cpp: (WebCore::FrameView::invalidateRect): (WebCore::FrameView::repaintContentRectangle): (WebCore::FrameView::doDeferredRepaints):
  • page/FrameView.h:
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paintContents):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintLayer):
10:28 AM Changeset in webkit [70895] by zmo@google.com
  • 2 edits in trunk/LayoutTests

2010-10-29 Zhenyao Mo <zmo@google.com>

Unreviewed, webgl test expectations update.

  • platform/chromium/test_expectations.txt:
10:23 AM Changeset in webkit [70894] by ap@apple.com
  • 43 edits in trunk

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=48576
Let WebKit2 client know when a frame is a frameset

This can change in two cases, when document changes, and when frame is navigated to another
document.

10:22 AM Changeset in webkit [70893] by andersca@apple.com
  • 3 edits in trunk/WebKit2

Connection::sendSyncMessage needs to dispatch incoming sync messages
https://bugs.webkit.org/show_bug.cgi?id=48606

Reviewed by Adam Roben.

  • Platform/CoreIPC/Connection.cpp:

(CoreIPC::Connection::sendSyncMessage):
Protect the connection when waiting for a reply. Schedule for any remaining incoming sync messages
to be sent once the bottom-most waitForSyncReply calls returns.

(CoreIPC::Connection::waitForSyncReply):
Make sure to dispatch any incoming sync messages.

(CoreIPC::Connection::processIncomingMessage):
If the incoming message is a sync message and we're waiting for a sync reply, add the message to the
queue of "sync messages received while waiting for sync reply" and wake up the client thread.

  • Platform/CoreIPC/Connection.h:
10:06 AM Changeset in webkit [70892] by Adam Roben
  • 11 edits in trunk/WebKit2

Use IntPoint/FloatSize in WebEvent subclasses

This is nicer than passing around pairs of ints/floats.

Fixes <http://webkit.org/b/48611>.

Reviewed by Anders Carlsson.

  • Shared/WebEvent.h:
  • Shared/WebMouseEvent.cpp:
  • Shared/WebPlatformTouchPoint.cpp:
  • Shared/WebWheelEvent.cpp:

Changed all x/y pairs to either use IntPoint or FloatSize, as
appropriate.

  • Shared/WebEventConversion.cpp:
  • Shared/mac/WebEventFactory.mm:
  • Shared/qt/WebEventFactoryQt.cpp:
  • Shared/win/WebEventFactory.cpp:
  • WebProcess/Plugins/Netscape/mac/NetscapePluginMac.mm:
  • WebProcess/Plugins/Netscape/win/NetscapePluginWin.cpp:

Updated for WebEvent changes.

10:05 AM Changeset in webkit [70891] by mnaganov@chromium.org
  • 2 edits in trunk/LayoutTests

2010-10-29 Mikhail Naganov <mnaganov@chromium.org>

[Chromium] Unreviewed test_expectations update.

Fix a duplicate for Mac, address r70887 tests miss.y

  • platform/chromium/test_expectations.txt:
10:01 AM Changeset in webkit [70890] by Adam Roben
  • 2 edits in trunk/WebKit2

Touch WebKit2Prefix.h to fix incremental builds on Windows

  • WebKit2Prefix.h:
9:59 AM Changeset in webkit [70889] by Adam Roben
  • 2 edits in trunk/WebKitSite

Instruct developers to install the Feburary 2010 DirectX SDK, not just any version

The February 2010 SDK is the last one with support for Visual Studio
2005.

I also removed some unnecessary instructions regarding the DirectX
SDK.

Fixes <http://webkit.org/b/48642> webkit.org should recommend a
specific version of the DirectX SDK

Reviewed by Steve Falkenburg.

  • building/tools.html:
9:57 AM Changeset in webkit [70888] by jocelyn.turcotte@nokia.com
  • 2 edits in trunk/WebKit/qt

2010-10-29 Jocelyn Turcotte <jocelyn.turcotte@nokia.com>

Reviewed by Andreas Kling.

[Qt] Fix tst_QWebFrame on XVFB.
https://bugs.webkit.org/show_bug.cgi?id=48637

These tests relied on a window manager to activate the window.

Fixes:

  • tst_QWebFrame::popupFocus()
  • tst_QwebFrame::inputFieldFocus()
  • tests/qwebframe/tst_qwebframe.cpp:
9:37 AM Changeset in webkit [70887] by jorlow@chromium.org
  • 1 edit
    2 adds in trunk/LayoutTests

2010-10-29 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Steve Block.

Add IndexedDB test/tutorial
https://bugs.webkit.org/show_bug.cgi?id=48067

This document is useful in terms of demonstrating IndexedDB and
documenting what we do and don't support currently, since I'll be
maintaining it for at least some time.

  • storage/indexeddb/tutorial-expected.txt: Added.
  • storage/indexeddb/tutorial.html: Added.
9:31 AM Changeset in webkit [70886] by Martin Robinson
  • 2 edits in trunk/WebKitTools

2010-10-29 Nicolas Dufresne <nicolas.dufresne@collabora.co.uk>

Reviewed by Martin Robinson.

Enable popup window in GtkLauncher
https://bugs.webkit.org/show_bug.cgi?id=48335

The GtkLauncher application does not support opening new window when
a link with "target=_blank" is clicked or similar call to
window.open(). Instead, GtkLauncher does nothing which breaks
navigation of some websites.

  • GtkLauncher/main.c: (activate_uri_entry_cb): (update_title): (link_hover_cb): (notify_title_cb): (notify_load_status_cb): (notify_progress_cb): (destroy_cb): (go_back_cb): (go_forward_cb): (create_web_view_cb): (web_view_ready_cb): (close_web_view_cb): (create_browser): (create_statusbar): (create_toolbar): (create_window): (main):
9:27 AM Changeset in webkit [70885] by Adam Roben
  • 21 edits in trunk/WebKit2

Remove unnecessary #includes from WebEvent.h

Fixes <http://webkit.org/b/48609> WebEvent.h #includes things it
doesn't need

Reviewed by Sam Weinig.

  • Scripts/webkit2/messages.py:

(MessageReceiver.iterparameters): Merged iterreplyparameters into this
function. All callers to iterparameters are really interested in the
reply parameters, too. This will cause us to, e.g., add
forward-declarations for types used in sync replies.
(headers_for_type): Added a special case for WebCore::KeypressCommand.

  • Scripts/webkit2/messages_unittest.py: Added a test to cover the

failing case that inspired the above changes.

  • Shared/WebEvent.h: Replaced #includes with forward-declarations.
  • Shared/API/c/WKSharedAPICast.h:
  • Shared/NativeWebKeyboardEvent.h:
  • Shared/WebEvent.cpp:
  • Shared/WebKeyboardEvent.cpp:
  • Shared/WebMouseEvent.cpp:
  • Shared/WebPlatformTouchPoint.cpp:
  • Shared/WebTouchEvent.cpp:
  • Shared/WebWheelEvent.cpp:
  • UIProcess/LayerBackedDrawingAreaProxy.cpp:
  • UIProcess/WebUIClient.cpp:
  • UIProcess/WebUIClient.h:
  • WebProcess/WebPage/FindController.cpp:
  • WebProcess/WebPage/mac/WebPageMac.mm:

Added now-needed #includes and declarations.

8:45 AM Changeset in webkit [70884] by Csaba Osztrogonác
  • 2 edits in trunk/WebCore

Unreviewed fix. Touch WebCorePrefix.h to fix incremental build failure after r70878.

  • WebCorePrefix.h:
8:45 AM Changeset in webkit [70883] by Adam Roben
  • 2 edits in trunk/WebKitTools

Teach check-webkit-style about WebKit2's idiosyncracies

Fixes <http://webkit.org/b/48638> Style bot complains about a number
of WebKit2 conventions

Reviewed by Anders Carlsson.

  • Scripts/webkitpy/style/checker.py: Excluded some rules for various

WebKit2-related files. Also updated the excluded rules for
WebKitAPITest to match the current code.

8:41 AM Changeset in webkit [70882] by mnaganov@chromium.org
  • 2 edits in trunk/LayoutTests

2010-10-29 Mikhail Naganov <mnaganov@chromium.org>

[Chromium] Unreviewed test_expectations fix.

Remove duplicate for body-propagation\overflow\004-xhtml.xhtml.

  • platform/chromium/test_expectations.txt:
8:39 AM Changeset in webkit [70881] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-10-29 Carlos Garcia Campos <cgarcia@igalia.com>

Reviewed by Martin Robinson.

[GTK] Handle surrogate pairs in TextBreakIteratorGtk
https://bugs.webkit.org/show_bug.cgi?id=48227

TextBreakIteratorGtk uses UTF-8 because it's what pango expects, but we
need to return indices for the given input string that are in UTF-16.
The number of characters is the same for both UTF-16 and UTF-8 except when
the input string contains surrogate pairs. We need to keep both, the
index for the UTF-8 string to be used internally, and the index for the
UTF-16 string to be used as return value of the iterator interface. It
fixes test fast/forms/textarea-maxlength.html

  • platform/text/gtk/TextBreakIteratorGtk.cpp: (WebCore::CharacterIterator::getText): (WebCore::CharacterIterator::getLength): (WebCore::CharacterIterator::getSize): (WebCore::CharacterIterator::getIndex): (WebCore::CharacterIterator::getUTF16Index): (WebCore::CharacterIterator::getUTF16Length): (WebCore::CharacterIterator::characterSize): (WebCore::CharacterIterator::setText): (WebCore::CharacterIterator::setIndex): (WebCore::CharacterIterator::setUTF16Index): (WebCore::CharacterIterator::first): (WebCore::CharacterIterator::last): (WebCore::CharacterIterator::next): (WebCore::CharacterIterator::previous): (WebCore::setUpIterator): (WebCore::textBreakFirst): (WebCore::textBreakLast): (WebCore::textBreakNext): (WebCore::textBreakPrevious): (WebCore::textBreakPreceding): (WebCore::textBreakFollowing): (WebCore::textBreakCurrent): (WebCore::isTextBreak):
8:34 AM Changeset in webkit [70880] by apavlov@chromium.org
  • 3 edits
    2 adds in trunk

2010-10-29 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: [REGRESSION] Addition of a new (blank) style property does not work
https://bugs.webkit.org/show_bug.cgi?id=48631

Test: inspector/styles-add-blank-property.html

WebCore:

  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertiesSection.prototype.addNewBlankProperty): (WebInspector.ComputedStylePropertiesSection.prototype.onpopulate):

LayoutTests:

  • inspector/styles-add-blank-property-expected.txt: Added.
  • inspector/styles-add-blank-property.html: Added.
8:25 AM Changeset in webkit [70879] by pfeldman@chromium.org
  • 11 edits in trunk

2010-10-29 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: add close button to network view, make tests pass for
resource manager on all ports.
https://bugs.webkit.org/show_bug.cgi?id=48628

  • inspector/InspectorController.cpp: (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::frameDetachedFromParent): (WebCore::InspectorController::didLoadResourceFromMemoryCache): (WebCore::InspectorController::identifierForInitialRequest): (WebCore::InspectorController::willSendRequest): (WebCore::InspectorController::markResourceAsCached): (WebCore::InspectorController::didReceiveResponse): (WebCore::InspectorController::didReceiveContentLength): (WebCore::InspectorController::didFinishLoading): (WebCore::InspectorController::didFailLoading): (WebCore::InspectorController::resourceRetrievedByXMLHttpRequest): (WebCore::InspectorController::scriptImported): (WebCore::InspectorController::didCreateWebSocket): (WebCore::InspectorController::willSendWebSocketHandshakeRequest): (WebCore::InspectorController::didReceiveWebSocketHandshakeResponse): (WebCore::InspectorController::didCloseWebSocket):
  • inspector/front-end/NetworkPanel.js: (WebInspector.NetworkPanel):
  • inspector/front-end/ResourceManager.js: (WebInspector.ResourceTreeModel.prototype.didCommitLoadForFrame):
  • inspector/front-end/StoragePanel.js: (WebInspector.StoragePanel.prototype.addResourceToFrame):
  • inspector/front-end/inspector.js: (WebInspector._createPanels):
  • inspector/front-end/networkPanel.css: (.network-close-button):

2010-10-29 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: add close button to network view, make tests pass
for resource manager on all ports.
https://bugs.webkit.org/show_bug.cgi?id=48628

  • src/js/Tests.js:

2010-10-29 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: add close button to network view, make tests pass
for resource manager on all ports.
https://bugs.webkit.org/show_bug.cgi?id=48628

  • inspector/audits-panel-functional.html:
8:23 AM Changeset in webkit [70878] by Csaba Osztrogonác
  • 17 edits in trunk

Fix and cleanup of build systems
https://bugs.webkit.org/show_bug.cgi?id=48342

Reviewed by Adam Roben and David Kilzer.

.:

  • cmake/OptionsEfl.cmake: Remove unnecessary ENABLE_SANDBOX.
  • cmakeconfig.h.cmake:
  • configure.ac:

JavaScriptCore:

  • Configurations/FeatureDefines.xcconfig: Add missing ENABLE_FULLSCREEN_API

WebKit/chromium:

  • features.gypi: Remove unnecessary ENABLE_SANDBOX.

WebKit/mac:

  • Configurations/FeatureDefines.xcconfig: Remove unnecessary ENABLE_SANDBOX.

WebKit2:

  • Configurations/FeatureDefines.xcconfig: Remove unnecessary ENABLE_SANDBOX and add missing features.

WebKitLibraries:

Remove unnecessary ENABLE_SANDBOX.
Add missing features to converge to FeatureDefines.xcconfig.

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

WebKitTools:

  • Scripts/build-webkit:
    • Remove unnecessary ENABLE_SANDBOX option.
    • Add ENABLE_FULLSCREEN_API option.
8:19 AM Changeset in webkit [70877] by mnaganov@chromium.org
  • 2 edits in trunk/LayoutTests

2010-10-29 Mikhail Naganov <mnaganov@chromium.org>

[Chromium] Unreviewed test expectations update.

Add 4 more suspicious tests to bug 48634.
https://bugs.webkit.org/show_bug.cgi?id=48634

  • platform/chromium/test_expectations.txt:
7:43 AM Changeset in webkit [70876] by Adam Roben
  • 5 edits in trunk/LayoutTests

Update Windows test results after r70854

  • platform/win/fast/dom/Window/window-property-descriptors-expected.txt:
  • platform/win/fast/dom/prototype-inheritance-2-expected.txt:
  • platform/win/fast/dom/prototype-inheritance-expected.txt:
  • platform/win/fast/js/global-constructors-expected.txt:
7:33 AM Changeset in webkit [70875] by mnaganov@chromium.org
  • 2 edits in trunk/LayoutTests

2010-10-29 Mikhail Naganov <mnaganov@chromium.org>

[Chromium] Unreviewed, disable tests after r70867.

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

  • platform/chromium/test_expectations.txt:
7:19 AM Changeset in webkit [70874] by mnaganov@chromium.org
  • 2 edits in trunk/LayoutTests

2010-10-29 Mikhail Naganov <mnaganov@chromium.org>

[Chromium] Unreviewed, disable tests.

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

  • platform/chromium/test_expectations.txt:
6:52 AM Changeset in webkit [70873] by mnaganov@chromium.org
  • 1 edit
    6 adds in trunk/LayoutTests

2010-10-29 Mikhail Naganov <mnaganov@chromium.org>

[Chromium] Unreviewed rebaselines, probably leftover from r70858.

  • platform/chromium-mac/fast/images/icon-decoding-expected.checksum: Added.
  • platform/chromium-mac/fast/images/icon-decoding-expected.txt: Added.
  • platform/chromium-mac/fast/images/pdf-as-background-expected.checksum: Added.
  • platform/chromium-mac/fast/images/pdf-as-background-expected.png: Added.
  • platform/chromium-mac/svg/custom/createImageElement-expected.checksum: Added.
  • platform/chromium-mac/svg/custom/createImageElement-expected.png: Added.
6:50 AM Changeset in webkit [70872] by Nikolas Zimmermann
  • 37 edits
    2 adds in trunk/WebCore

2010-10-29 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Dirk Schulze.

Convert SVGAnimatedEnumeration/Integer to the new SVGAnimatedPropertyTearOff concept
https://bugs.webkit.org/show_bug.cgi?id=48623

Besides converting SVGAnimatedEnumeration/Integer to the new animated property tear off concept, fix support
for multiple wrappers within the new concept, which was broken - covered by existing svg/dynamic-updates/ tests.

Replace all 'extern char[]' usage when using the multiple wrappers macro, by static AtomicStrings.

  • GNUmakefile.am: Add SVGAnimatedEnumeration.h / SVGAnimatedInteger.h to build.
  • WebCore.gypi: Ditto.
  • WebCore.pro: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • bindings/scripts/CodeGenerator.pm: Recognize SVGAnimatedEnumeration / SVGAnimatedInteger as animated types using the new concept.
  • bindings/scripts/CodeGeneratorObjC.pm: Touched, to force regenerating ObjC bindings, dependency tracking is obviously broken.
  • svg/DeprecatedSVGAnimatedPropertyTraits.h: Remove SVGAnimatedBoolean/Enumeration/Integer handling.
  • svg/DeprecatedSVGAnimatedTemplate.h: Ditto.
  • svg/SVGAnimatedEnumeration.h: Added.
  • svg/SVGAnimatedEnumeration.idl:
  • svg/SVGAnimatedInteger.h: Added.
  • svg/SVGClipPathElement.h: Adapted all code to create new style animated property tear offs for long & int types.
  • svg/SVGComponentTransferFunctionElement.h:
  • svg/SVGFEBlendElement.h:
  • svg/SVGFEColorMatrixElement.h:
  • svg/SVGFECompositeElement.h:
  • svg/SVGFEConvolveMatrixElement.h:
  • svg/SVGFEDiffuseLightingElement.cpp: (WebCore::SVGFEDiffuseLightingElement::kernelUnitLengthXIdentifier): Replaced extern[] char hack using a static AtomicString. (WebCore::SVGFEDiffuseLightingElement::kernelUnitLengthYIdentifier): Ditto.
  • svg/SVGFEDiffuseLightingElement.h:
  • svg/SVGFEDisplacementMapElement.h:
  • svg/SVGFEGaussianBlurElement.cpp: (WebCore::SVGFEGaussianBlurElement::stdDeviationXIdentifier): Ditto. (WebCore::SVGFEGaussianBlurElement::stdDeviationYIdentifier): Ditto.
  • svg/SVGFEGaussianBlurElement.h:
  • svg/SVGFEMorphologyElement.h:
  • svg/SVGFESpecularLightingElement.cpp: (WebCore::SVGFESpecularLightingElement::kernelUnitLengthXIdentifier): Ditto. (WebCore::SVGFESpecularLightingElement::kernelUnitLengthYIdentifier): Ditto.
  • svg/SVGFESpecularLightingElement.h:
  • svg/SVGFETurbulenceElement.cpp: (WebCore::SVGFETurbulenceElement::baseFrequencyXIdentifier): Ditto. (WebCore::SVGFETurbulenceElement::baseFrequencyYIdentifier): Ditto.
  • svg/SVGFETurbulenceElement.h:
  • svg/SVGFilterElement.cpp: (WebCore::SVGFilterElement::filterResXIdentifier): Ditto. (WebCore::SVGFilterElement::filterResYIdentifier): Ditto.
  • svg/SVGFilterElement.h:
  • svg/SVGGradientElement.h:
  • svg/SVGMarkerElement.cpp: (WebCore::SVGMarkerElement::orientTypeIdentifier): Ditto. (WebCore::SVGMarkerElement::orientAngleIdentifier): Ditto.
  • svg/SVGMarkerElement.h:
  • svg/SVGMaskElement.h:
  • svg/SVGPatternElement.h:
  • svg/SVGTextContentElement.h:
  • svg/SVGTextPathElement.h:
  • svg/properties/SVGAnimatedPropertyMacros.h: Add new macro for multiple wrappers using static properties.
6:45 AM Changeset in webkit [70871] by Adam Roben
  • 2 edits in trunk/WebKitTools

Switch the Windows WebKit2 bot to the Release configuration

We only have one machine testing WebKit2 on Windows right now, and
Debug is just too slow for it to keep up.

Fixes (hopefully!) <http://webkit.org/b/48615> Windows WebKit2 bot is
always way behind

Reviewed by Jon Honeycutt.

  • BuildSlaveSupport/build.webkit.org-config/config.json:
6:44 AM Changeset in webkit [70870] by andreas.kling@nokia.com
  • 2 edits in trunk/WebCore

2010-10-29 Andreas Kling <kling@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Make GraphicsContext cheaper to construct
https://bugs.webkit.org/show_bug.cgi?id=48626

Do less unnecessary things when creating the Qt GraphicsContext:

  • Don't create a black QBrush that's immediately discarded.
  • Don't call setPlatform*(), set the painter brush and pen directly.
  • Call setRenderHints() once instead of multiple setRenderHint().
  • platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::GraphicsContextPlatformPrivate::GraphicsContextPlatformPrivate): (WebCore::GraphicsContext::GraphicsContext):
6:41 AM Changeset in webkit [70869] by satish@chromium.org
  • 2 edits in branches/chromium/517/WebCore

Merge 70644 - 2010-10-27 Satish Sampath <satish@chromium.org>

Reviewed by Steve Block.

Add runtime flag check before enabling speech input in code.
https://bugs.webkit.org/show_bug.cgi?id=48426

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::isSpeechEnabled): Added runtime flag check.

TBR=satish@chromium.org
Review URL: http://codereview.chromium.org/4111010

6:39 AM Changeset in webkit [70868] by andreas.kling@nokia.com
  • 2 edits in trunk/WebCore

2010-10-29 Andreas Kling <kling@webkit.org>

Reviewed by Simon Hausmann.

[Qt] GraphicsLayer: Don't notifySyncRequired() more than necessary
https://bugs.webkit.org/show_bug.cgi?id=48624

Don't queue multiple meta method invocations for notifySyncRequired()

4% speedup on <http://ariya.github.com/clock.htm>

  • platform/graphics/qt/GraphicsLayerQt.cpp: (WebCore::GraphicsLayerQtImpl::GraphicsLayerQtImpl): (WebCore::GraphicsLayerQtImpl::notifySyncRequired): (WebCore::GraphicsLayerQtImpl::notifyChange):
6:35 AM Changeset in webkit [70867] by zmo@google.com
  • 7 edits in trunk

2010-10-29 Zhenyao Mo <zmo@google.com>

Reviewed by Chris Marrin.

drawArrays/drawElements with first/offset out of range but size==0 should generate no error
https://bugs.webkit.org/show_bug.cgi?id=48601

  • fast/canvas/webgl/draw-arrays-out-of-bounds-expected.txt:
  • fast/canvas/webgl/draw-arrays-out-of-bounds.html:
  • fast/canvas/webgl/draw-elements-out-of-bounds-expected.txt:
  • fast/canvas/webgl/draw-elements-out-of-bounds.html:

2010-10-29 Zhenyao Mo <zmo@google.com>

Reviewed by Chris Marrin.

drawArrays/drawElements with first/offset out of range but size==0 should generate no error
https://bugs.webkit.org/show_bug.cgi?id=48601

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::drawArrays): (WebCore::WebGLRenderingContext::drawElements):
6:26 AM Changeset in webkit [70866] by mnaganov@chromium.org
  • 2 edits
    3 adds in trunk/LayoutTests

2010-10-29 Mikhail Naganov <mnaganov@chromium.org>

[Chromium] Unreviewed tests checksums update. Also, fix duplicates introduced in r70865.

  • platform/chromium-mac/fast/blockflow/background-horizontal-bt-expected.checksum: Added.
  • platform/chromium-mac/fast/blockflow/background-vertical-lr-expected.checksum: Added.
  • platform/chromium-mac/fast/blockflow/background-vertical-rl-expected.checksum: Added.
  • platform/chromium/test_expectations.txt:
6:06 AM Changeset in webkit [70865] by mnaganov@chromium.org
  • 2 edits in trunk/LayoutTests

2010-10-29 Mikhail Naganov <mnaganov@chromium.org>

[Chromium] Unreviewed test expectations update.

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

  • platform/chromium/test_expectations.txt:
5:08 AM Changeset in webkit [70864] by mnaganov@chromium.org
  • 15 edits
    12 adds in trunk/LayoutTests

2010-10-29 Mikhail Naganov <mnaganov@chromium.org>

[Chromium] Unreviewed tests rebaselines after r70843, r70847, r70856

  • platform/chromium-linux/fast/blockflow/border-radius-clipping-vertical-lr-expected.checksum: Added.
  • platform/chromium-linux/fast/blockflow/border-radius-clipping-vertical-lr-expected.png: Added.
  • platform/chromium-linux/fast/dom/HTMLMeterElement/meter-appearances-capacity-expected.checksum:
  • platform/chromium-linux/fast/dom/HTMLMeterElement/meter-appearances-capacity-expected.png:
  • platform/chromium-linux/fast/dom/HTMLMeterElement/meter-appearances-rating-relevancy-expected.checksum:
  • platform/chromium-linux/fast/dom/HTMLMeterElement/meter-appearances-rating-relevancy-expected.png:
  • platform/chromium-linux/fast/dom/HTMLMeterElement/meter-boundary-values-expected.checksum: Added.
  • platform/chromium-linux/fast/dom/HTMLMeterElement/meter-boundary-values-expected.png: Added.
  • platform/chromium-linux/fast/dom/HTMLMeterElement/meter-optimums-expected.checksum:
  • platform/chromium-linux/fast/dom/HTMLMeterElement/meter-optimums-expected.png:
  • platform/chromium-mac/fast/blockflow/border-radius-clipping-vertical-lr-expected.checksum: Added.
  • platform/chromium-mac/fast/blockflow/box-shadow-vertical-lr-expected.checksum:
  • platform/chromium-mac/fast/blockflow/box-shadow-vertical-rl-expected.checksum:
  • platform/chromium-mac/fast/dom/HTMLMeterElement/meter-boundary-values-expected.checksum: Added.
  • platform/chromium-win/fast/blockflow/border-radius-clipping-vertical-lr-expected.checksum: Added.
  • platform/chromium-win/fast/blockflow/border-radius-clipping-vertical-lr-expected.png: Added.
  • platform/chromium-win/fast/blockflow/border-radius-clipping-vertical-lr-expected.txt: Added.
  • platform/chromium-win/fast/dom/HTMLMeterElement/meter-appearances-capacity-expected.checksum:
  • platform/chromium-win/fast/dom/HTMLMeterElement/meter-appearances-capacity-expected.png:
  • platform/chromium-win/fast/dom/HTMLMeterElement/meter-appearances-rating-relevancy-expected.checksum:
  • platform/chromium-win/fast/dom/HTMLMeterElement/meter-appearances-rating-relevancy-expected.png:
  • platform/chromium-win/fast/dom/HTMLMeterElement/meter-boundary-values-expected.checksum: Added.
  • platform/chromium-win/fast/dom/HTMLMeterElement/meter-boundary-values-expected.png: Added.
  • platform/chromium-win/fast/dom/HTMLMeterElement/meter-boundary-values-expected.txt: Added.
  • platform/chromium-win/fast/dom/HTMLMeterElement/meter-optimums-expected.checksum:
  • platform/chromium-win/fast/dom/HTMLMeterElement/meter-optimums-expected.png:
5:00 AM Changeset in webkit [70863] by satish@chromium.org
  • 30 edits
    2 adds in trunk

2010-10-29 Leandro Gracia Gil <leandrogracia@google.com>

Reviewed by Jeremy Orlow.

Added a layout test for the language tag in speech input. Updated
the speech button tests to include the new language parameter in the
mock result.
https://bugs.webkit.org/show_bug.cgi?id=47089

  • fast/speech/input-text-language-tag-expected.txt: Added.
  • fast/speech/input-text-language-tag.html: Added.
  • fast/speech/input-text-speechbutton.html:
  • fast/speech/speech-button-ignore-generated-events.html:
  • platform/chromium/test_expectations.txt:

2010-10-29 Leandro Gracia Gil <leandrogracia@google.com>

Reviewed by Jeremy Orlow.

Patch the current speech input implementation to use the nearest
language tag. The language is now passed to the startRecognition
methods so that language-specific recognition could be used. Also added
a second parameter to setMockSpeechInputResult for the language used in
speech recognition.
https://bugs.webkit.org/show_bug.cgi?id=47089

This is the 2nd of a 4-sided patch in Chromium and WebKit. For more
details see http://codereview.chromium.org/3615005/show,
http://codereview.chromium.org/3595018/show and
https://bugs.webkit.org/show_bug.cgi?id=47420.

Test: fast/speech/input-text-language-tag.html

  • page/SpeechInput.cpp: (WebCore::SpeechInput::startRecognition):
  • page/SpeechInput.h:
  • page/SpeechInputClient.h:
  • platform/mock/SpeechInputClientMock.cpp: (WebCore::SpeechInputClientMock::startRecognition): (WebCore::SpeechInputClientMock::setRecognitionResult): (WebCore::SpeechInputClientMock::timerFired):
  • platform/mock/SpeechInputClientMock.h:
  • rendering/TextControlInnerElements.cpp: (WebCore::TextControlInnerElement::attachInnerElement): (WebCore::SearchFieldCancelButtonElement::detach): (WebCore::SpinButtonElement::defaultEventHandler): (WebCore::InputFieldSpeechButtonElement::defaultEventHandler): (WebCore::InputFieldSpeechButtonElement::detach):

2010-10-29 Leandro Gracia Gil <leandrogracia@google.com>

Reviewed by Jeremy Orlow.

Patch the current speech input implementation to use the nearest
language tag. The language is now passed to the startRecognition
methods so that language-specific recognition could be used. Also added
a second parameter to setMockSpeechInputResult for the language used in
speech recognition.
https://bugs.webkit.org/show_bug.cgi?id=47089

This is the 2nd of a 4-sided patch in Chromium and WebKit. For more
details see http://codereview.chromium.org/3615005/show,
http://codereview.chromium.org/3595018/show and
https://bugs.webkit.org/show_bug.cgi?id=47420.

  • public/WebSpeechInputController.h: (WebKit::WebSpeechInputController::startRecognition):
  • public/WebSpeechInputControllerMock.h:
  • src/SpeechInputClientImpl.cpp: (WebKit::SpeechInputClientImpl::create): (WebKit::SpeechInputClientImpl::startRecognition):
  • src/SpeechInputClientImpl.h:
  • src/WebSpeechInputControllerMockImpl.cpp: (WebKit::WebSpeechInputControllerMockImpl::setMockRecognitionResult): (WebKit::WebSpeechInputControllerMockImpl::startRecognition):
  • src/WebSpeechInputControllerMockImpl.h:
  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::WebViewImpl):
  • src/WebViewImpl.h:

2010-10-29 Leandro Gracia Gil <leandrogracia@google.com>

Reviewed by Jeremy Orlow.

Added a second parameter to setMockSpeechInputResult for
the language used in speech input.
https://bugs.webkit.org/show_bug.cgi?id=47089

  • DumpRenderTree/LayoutTestController.cpp: (setMockSpeechInputResultCallback):
  • DumpRenderTree/LayoutTestController.h:
  • DumpRenderTree/chromium/LayoutTestController.cpp: (LayoutTestController::setMockSpeechInputResult):
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::setMockSpeechInputResult):
  • DumpRenderTree/mac/LayoutTestControllerMac.mm: (LayoutTestController::setMockSpeechInputResult):
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::setMockSpeechInputResult):
  • DumpRenderTree/qt/LayoutTestControllerQt.h:
  • DumpRenderTree/win/LayoutTestControllerWin.cpp: (LayoutTestController::setMockSpeechInputResult):
  • DumpRenderTree/wx/LayoutTestControllerWx.cpp: (LayoutTestController::setMockSpeechInputResult): (LayoutTestController::pageSizeAndMarginsInPixels):
4:50 AM Changeset in webkit [70862] by ager@chromium.org
  • 3 edits
    3 adds in trunk

2010-10-29 Mads Ager <ager@chromium.org>

Reviewed by Pavel Feldman.

[V8] The DateExtension cannot handle overwritten Date
https://bugs.webkit.org/show_bug.cgi?id=48621

Check that the Date property is an object before attempting to get
hidden properties from it.

Test: fast/dom/Window/slow-unload-handler-overwritten-date.html

  • bindings/v8/DateExtension.cpp: (WebCore::DateExtension::setAllowSleep):

2010-10-29 Mads Ager <ager@chromium.org>

Reviewed by Pavel Feldman.

[V8] The DateExtension cannot handle overwritten Date
https://bugs.webkit.org/show_bug.cgi?id=48621

  • fast/dom/Window/resources/slow-unload-handler-overwritten-date1.html: Added.
  • fast/dom/Window/slow-unload-handler-overwritten-date-expected.txt: Added.
  • fast/dom/Window/slow-unload-handler-overwritten-date.html: Added.
4:25 AM Changeset in webkit [70861] by alex
  • 3 edits in trunk/WebKit/gtk

2010-10-29 Alejandro G. Castro <alex@igalia.com>

Reviewed by Martin Robinson.

[GTK] Add the GtkScrollablePolicy property to the webview
https://bugs.webkit.org/show_bug.cgi?id=48357

Added the GtkScrollablePolicy properties to the webview widget, it
is now required for GtkScrollable widgets in GTK+ 3.x.

  • webkit/webkitprivate.h:
  • webkit/webkitwebview.cpp: (setHorizontalScrollPolicy): Added. (setVerticalScrollPolicy): Added. (getHorizontalScrollPolicy): Added. (getVerticalScrollPolicy): Added. (webkit_web_view_get_property): (webkit_web_view_set_property): (webkit_web_view_class_init):
3:32 AM Changeset in webkit [70860] by Nikolas Zimmermann
  • 2 edits in trunk/LayoutTests

2010-10-29 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed.

Oh my, remove a left-over alert(), that I accidently landed.

  • svg/dom/SVGScriptElement/resources/script-load4.js:
3:31 AM Changeset in webkit [70859] by tkent@chromium.org
  • 8 edits in trunk/LayoutTests

2010-10-29 Kent Tamura <tkent@chromium.org>

Unreviewed, test expectation update.

Update Qt and GTK expectations for r70854.

  • platform/gtk/fast/dom/Window/window-property-descriptors-expected.txt:
  • platform/gtk/fast/dom/prototype-inheritance-expected.txt:
  • platform/gtk/fast/js/global-constructors-expected.txt:
  • platform/qt/fast/dom/Window/window-properties-expected.txt:
  • platform/qt/fast/dom/Window/window-property-descriptors-expected.txt:
  • platform/qt/fast/dom/prototype-inheritance-expected.txt:
  • platform/qt/fast/js/global-constructors-expected.txt:
3:11 AM Changeset in webkit [70858] by abarth@webkit.org
  • 66 edits
    192 adds
    16 deletes in trunk/LayoutTests

2010-10-29 Adam Barth <abarth@webkit.org>

Unreviewed.

[chromium] Chromium Mac should use WebKit's image decoders
https://bugs.webkit.org/show_bug.cgi?id=47974

Update chromium-mac expectations for new image decoders. There are a
lot of changed images here, but I think they're correct.

  • platform/chromium-mac/css2.1/t0804-c5510-padn-00-b-ag-expected.checksum:
  • platform/chromium-mac/css2.1/t0804-c5510-padn-00-b-ag-expected.png: Removed.
  • platform/chromium-mac/css2.1/t0905-c414-flt-01-d-g-expected.checksum: Added.
  • platform/chromium-mac/css2.1/t0905-c414-flt-wrap-01-d-g-expected.checksum: Added.
  • platform/chromium-mac/css2.1/t100801-c544-valgn-02-d-agi-expected.checksum: Added.
  • platform/chromium-mac/css2.1/t100801-c544-valgn-03-d-agi-expected.checksum: Added.
  • platform/chromium-mac/css2.1/t100801-c544-valgn-04-d-agi-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/repeat/negative-offset-repeat-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/repeat/negative-offset-repeat-transformed-expected.checksum:
  • platform/chromium-mac/fast/backgrounds/repeat/negative-offset-repeat-transformed-expected.png:
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize01-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize01-expected.png: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize05-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize05-expected.png: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize06-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize07-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize08-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize09-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize10-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize11-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize12-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize12-expected.png: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize16-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize16-expected.png: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize17-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize17-expected.png: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize18-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize18-expected.png: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize19-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize19-expected.png: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize20-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize20-expected.png: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize21-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize21-expected.png: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize22-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/backgroundSize22-expected.png: Added.
  • platform/chromium-mac/fast/backgrounds/size/zero-expected.checksum: Added.
  • platform/chromium-mac/fast/backgrounds/size/zero-expected.png: Added.
  • platform/chromium-mac/fast/block/float/014-expected.checksum: Added.
  • platform/chromium-mac/fast/block/positioning/replaced-inside-fixed-top-bottom-expected.checksum: Added.
  • platform/chromium-mac/fast/block/positioning/replaced-inside-fixed-top-bottom-expected.png: Added.
  • platform/chromium-mac/fast/borders/border-fit-expected.checksum: Added.
  • platform/chromium-mac/fast/borders/border-fit-expected.png: Added.
  • platform/chromium-mac/fast/borders/border-image-omit-right-slice-expected.checksum: Added.
  • platform/chromium-mac/fast/borders/border-image-omit-right-slice-expected.png: Added.
  • platform/chromium-mac/fast/borders/border-image-rotate-transform-expected.checksum:
  • platform/chromium-mac/fast/borders/border-image-rotate-transform-expected.png:
  • platform/chromium-mac/fast/borders/svg-as-border-image-2-expected.checksum: Added.
  • platform/chromium-mac/fast/borders/svg-as-border-image-2-expected.png: Added.
  • platform/chromium-mac/fast/borders/svg-as-border-image-expected.checksum: Added.
  • platform/chromium-mac/fast/borders/svg-as-border-image-expected.png: Added.
  • platform/chromium-mac/fast/canvas/image-pattern-rotate-expected.checksum: Added.
  • platform/chromium-mac/fast/css/background-shorthand-invalid-url-expected.checksum: Added.
  • platform/chromium-mac/fast/css/background-shorthand-invalid-url-expected.png: Added.
  • platform/chromium-mac/fast/css/contentDiv-expected.checksum: Added.
  • platform/chromium-mac/fast/css/contentDiv-expected.png: Added.
  • platform/chromium-mac/fast/css/contentDivWithChildren-expected.checksum: Added.
  • platform/chromium-mac/fast/css/contentDivWithChildren-expected.png: Added.
  • platform/chromium-mac/fast/css/contentImage-expected.checksum: Added.
  • platform/chromium-mac/fast/css/contentImage-expected.png: Added.
  • platform/chromium-mac/fast/css/value-list-out-of-bounds-crash-expected.checksum: Added.
  • platform/chromium-mac/fast/forms/input-align-image-expected.checksum: Added.
  • platform/chromium-mac/fast/forms/input-appearance-bkcolor-expected.checksum: Added.
  • platform/chromium-mac/fast/forms/input-type-change-expected.checksum: Added.
  • platform/chromium-mac/fast/forms/preserveFormDuringResidualStyle-expected.checksum: Added.
  • platform/chromium-mac/fast/images/pixel-crack-image-background-webkit-transform-scale-expected.checksum:
  • platform/chromium-mac/fast/images/pixel-crack-image-background-webkit-transform-scale-expected.png: Removed.
  • platform/chromium-mac/fast/media/mq-min-pixel-ratio-expected.checksum: Added.
  • platform/chromium-mac/fast/media/mq-min-pixel-ratio-expected.png: Added.
  • platform/chromium-mac/fast/reflections/reflection-masks-expected.checksum: Added.
  • platform/chromium-mac/fast/reflections/reflection-masks-expected.png: Added.
  • platform/chromium-mac/fast/reflections/reflection-masks-opacity-expected.checksum: Added.
  • platform/chromium-mac/fast/repaint/block-layout-inline-children-replaced-expected.checksum:
  • platform/chromium-mac/fast/repaint/block-layout-inline-children-replaced-expected.png:
  • platform/chromium-mac/fast/replaced/absolute-image-sizing-expected.checksum: Added.
  • platform/chromium-mac/fast/replaced/image-onload-expected.checksum: Added.
  • platform/chromium-mac/fast/replaced/image-sizing-expected.checksum: Added.
  • platform/chromium-mac/fast/replaced/image-tag-expected.checksum: Added.
  • platform/chromium-mac/fast/speech/input-appearance-numberandspeech-expected.checksum:
  • platform/chromium-mac/fast/speech/input-appearance-numberandspeech-expected.png:
  • platform/chromium-mac/fast/speech/input-appearance-searchandspeech-expected.checksum:
  • platform/chromium-mac/fast/speech/input-appearance-searchandspeech-expected.png:
  • platform/chromium-mac/fast/speech/input-appearance-speechbutton-expected.checksum:
  • platform/chromium-mac/fast/speech/input-appearance-speechbutton-expected.png:
  • platform/chromium-mac/http/tests/misc/slow-loading-image-in-pattern-expected.checksum:
  • platform/chromium-mac/http/tests/misc/slow-loading-image-in-pattern-expected.png:
  • platform/chromium-mac/scrollbars/listbox-scrollbar-combinations-expected.checksum: Added.
  • platform/chromium-mac/scrollbars/overflow-scrollbar-combinations-expected.checksum: Added.
  • platform/chromium-mac/scrollbars/overflow-scrollbar-combinations-expected.png: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1-SE/filters-image-03-f-expected.checksum:
  • platform/chromium-mac/svg/W3C-SVG-1.1-SE/filters-image-03-f-expected.png:
  • platform/chromium-mac/svg/W3C-SVG-1.1/animate-elem-30-t-expected.checksum:
  • platform/chromium-mac/svg/W3C-SVG-1.1/animate-elem-30-t-expected.png: Removed.
  • platform/chromium-mac/svg/W3C-SVG-1.1/animate-elem-36-t-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/animate-elem-36-t-expected.png: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/animate-elem-39-t-expected.checksum:
  • platform/chromium-mac/svg/W3C-SVG-1.1/animate-elem-39-t-expected.png: Removed.
  • platform/chromium-mac/svg/W3C-SVG-1.1/animate-elem-40-t-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/animate-elem-40-t-expected.png: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/coords-viewattr-02-b-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/coords-viewattr-02-b-expected.png: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/filters-blend-01-b-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/filters-blend-01-b-expected.png: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/filters-conv-01-f-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/filters-displace-01-f-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/filters-displace-01-f-expected.png: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/filters-image-01-b-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/filters-image-01-b-expected.png: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/masking-path-04-b-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/masking-path-04-b-expected.png: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/render-groups-01-b-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/render-groups-01-b-expected.png: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/render-groups-03-t-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/render-groups-03-t-expected.png: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/struct-image-01-t-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/struct-image-03-t-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/struct-image-03-t-expected.png: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/struct-image-04-t-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/struct-image-04-t-expected.png: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/struct-image-06-t-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/struct-image-06-t-expected.png: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/struct-image-07-t-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/struct-image-07-t-expected.png:
  • platform/chromium-mac/svg/W3C-SVG-1.1/struct-image-08-t-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/struct-image-08-t-expected.png: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/struct-image-09-t-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/struct-image-09-t-expected.png: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/struct-image-10-t-expected.checksum: Added.
  • platform/chromium-mac/svg/W3C-SVG-1.1/struct-image-10-t-expected.png: Added.
  • platform/chromium-mac/svg/carto.net/scrollbar-expected.checksum: Added.
  • platform/chromium-mac/svg/carto.net/scrollbar-expected.png: Added.
  • platform/chromium-mac/svg/carto.net/selectionlist-expected.checksum: Added.
  • platform/chromium-mac/svg/carto.net/selectionlist-expected.png: Added.
  • platform/chromium-mac/svg/custom/convolution-crash-expected.checksum: Added.
  • platform/chromium-mac/svg/custom/focus-ring-expected.checksum:
  • platform/chromium-mac/svg/custom/focus-ring-expected.png:
  • platform/chromium-mac/svg/custom/group-opacity-expected.checksum: Added.
  • platform/chromium-mac/svg/custom/group-opacity-expected.png: Added.
  • platform/chromium-mac/svg/custom/image-parent-translation-expected.checksum: Added.
  • platform/chromium-mac/svg/custom/image-parent-translation-expected.png: Added.
  • platform/chromium-mac/svg/custom/image-small-width-height-expected.checksum: Added.
  • platform/chromium-mac/svg/custom/image-small-width-height-expected.png: Added.
  • platform/chromium-mac/svg/custom/image-with-transform-clip-filter-expected.checksum: Added.
  • platform/chromium-mac/svg/custom/js-update-image-and-display-expected.checksum: Added.
  • platform/chromium-mac/svg/custom/js-update-image-and-display-expected.png: Added.
  • platform/chromium-mac/svg/custom/js-update-image-and-display2-expected.checksum: Added.
  • platform/chromium-mac/svg/custom/js-update-image-and-display2-expected.png: Added.
  • platform/chromium-mac/svg/custom/js-update-image-and-display3-expected.checksum: Added.
  • platform/chromium-mac/svg/custom/js-update-image-and-display3-expected.png: Added.
  • platform/chromium-mac/svg/custom/js-update-image-expected.checksum: Added.
  • platform/chromium-mac/svg/custom/js-update-image-expected.png: Added.
  • platform/chromium-mac/svg/custom/pointer-events-image-expected.checksum: Added.
  • platform/chromium-mac/svg/custom/pointer-events-image-expected.png: Added.
  • platform/chromium-mac/svg/custom/scrolling-embedded-svg-file-image-repaint-problem-expected.checksum:
  • platform/chromium-mac/svg/custom/scrolling-embedded-svg-file-image-repaint-problem-expected.png:
  • platform/chromium-mac/svg/custom/text-image-opacity-expected.checksum: Added.
  • platform/chromium-mac/svg/custom/text-image-opacity-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEBlendElement-dom-in-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGFEBlendElement-dom-in-attr-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEBlendElement-dom-in2-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGFEBlendElement-dom-in2-attr-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEBlendElement-dom-mode-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGFEBlendElement-dom-mode-attr-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEBlendElement-svgdom-in-prop-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGFEBlendElement-svgdom-in-prop-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEBlendElement-svgdom-in2-prop-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGFEBlendElement-svgdom-in2-prop-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEBlendElement-svgdom-mode-prop-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGFEBlendElement-svgdom-mode-prop-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-bias-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-bias-attr-expected.png: Removed.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-divisor-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-divisor-attr-expected.png: Removed.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-edgeMode-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-edgeMode-attr-expected.png: Removed.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-in-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-in-attr-expected.png: Removed.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-kernelMatrix-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-kernelMatrix-attr-expected.png: Removed.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-kernelUnitLength-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-kernelUnitLength-attr-expected.png: Removed.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-order-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-order-attr-expected.png: Removed.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-preserveAlpha-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-preserveAlpha-attr-expected.png: Removed.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-targetX-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-targetX-attr-expected.png: Removed.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-targetY-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-targetY-attr-expected.png: Removed.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-height-attr-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-height-attr-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-preserveAspectRatio-attr-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-preserveAspectRatio-attr-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-width-attr-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-width-attr-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-x-attr-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-x-attr-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-y-attr-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-y-attr-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-height-prop-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-height-prop-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-preserveAspectRatio-prop-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-preserveAspectRatio-prop-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-width-prop-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-width-prop-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-x-prop-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-x-prop-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-y-prop-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-y-prop-expected.png: Added.
  • platform/chromium-mac/svg/filters/feDisplacementMap-expected.checksum: Added.
  • platform/chromium-mac/svg/filters/filterRes-expected.checksum: Added.
  • platform/chromium-mac/svg/filters/filteredImage-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug10565-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug11026-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug1188-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug120107-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug1271-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug12908-1-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug12908-2-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug12910-2-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug1296-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug1296-expected.png: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug13169-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug1430-expected.checksum:
  • platform/chromium-mac/tables/mozilla/bugs/bug1430-expected.png:
  • platform/chromium-mac/tables/mozilla/bugs/bug14929-expected.checksum:
  • platform/chromium-mac/tables/mozilla/bugs/bug15544-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug17138-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug196870-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug2981-2-expected.checksum:
  • platform/chromium-mac/tables/mozilla/bugs/bug4093-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug4284-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug4427-expected.checksum:
  • platform/chromium-mac/tables/mozilla/bugs/bug4427-expected.png: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug4523-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug50695-2-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug56563-expected.checksum:
  • platform/chromium-mac/tables/mozilla/bugs/bug56563-expected.png: Removed.
  • platform/chromium-mac/tables/mozilla/bugs/bug6404-expected.checksum:
  • platform/chromium-mac/tables/mozilla/bugs/bug82946-2-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug82946-2-expected.png: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug86708-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug97383-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/bugs/bug97383-expected.png: Added.
  • platform/chromium-mac/tables/mozilla/core/col_widths_auto_autoFix-expected.checksum:
  • platform/chromium-mac/tables/mozilla/core/col_widths_auto_autoFix-expected.png: Removed.
  • platform/chromium-mac/tables/mozilla/core/misc-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/marvin/tbody_valign_baseline-expected.checksum:
  • platform/chromium-mac/tables/mozilla/marvin/tbody_valign_bottom-expected.checksum:
  • platform/chromium-mac/tables/mozilla/marvin/tbody_valign_middle-expected.checksum:
  • platform/chromium-mac/tables/mozilla/marvin/tbody_valign_top-expected.checksum:
  • platform/chromium-mac/tables/mozilla/marvin/td_valign_baseline-expected.checksum:
  • platform/chromium-mac/tables/mozilla/marvin/td_valign_bottom-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/marvin/td_valign_middle-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/marvin/td_valign_top-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/marvin/tfoot_valign_baseline-expected.checksum:
  • platform/chromium-mac/tables/mozilla/marvin/tfoot_valign_bottom-expected.checksum:
  • platform/chromium-mac/tables/mozilla/marvin/tfoot_valign_middle-expected.checksum:
  • platform/chromium-mac/tables/mozilla/marvin/tfoot_valign_top-expected.checksum:
  • platform/chromium-mac/tables/mozilla/marvin/th_valign_baseline-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/marvin/th_valign_bottom-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/marvin/th_valign_middle-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/marvin/th_valign_top-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/marvin/thead_valign_baseline-expected.checksum:
  • platform/chromium-mac/tables/mozilla/marvin/thead_valign_bottom-expected.checksum:
  • platform/chromium-mac/tables/mozilla/marvin/thead_valign_middle-expected.checksum:
  • platform/chromium-mac/tables/mozilla/marvin/thead_valign_top-expected.checksum:
  • platform/chromium-mac/tables/mozilla/marvin/tr_valign_baseline-expected.checksum:
  • platform/chromium-mac/tables/mozilla/marvin/tr_valign_bottom-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/marvin/tr_valign_middle-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla/marvin/tr_valign_top-expected.checksum: Added.
  • platform/chromium-mac/tables/mozilla_expected_failures/bugs/bug6933-expected.checksum: Added.
  • platform/chromium/test_expectations.txt:
    • Now that we now fail a handful of tests more consistently across platforms.
2:54 AM Changeset in webkit [70857] by Nikolas Zimmermann
  • 52 edits
    5 adds in trunk

2010-10-27 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Dirk Schulze.

Convert SVGAnimatedBoolean to the new SVGAnimatedPropertyTearOff concept
https://bugs.webkit.org/show_bug.cgi?id=48469

Convert the next SVG animated primitive type to use the SVGAnimatedPropertyTearOff concept: SVGAnimatedBoolean.
Added new test verifying that StrictTypeChecking is enabled for SVGAnimatedBooleans. It also tests that this type is "static", see LayoutTests/ChangeLog for details.

Add support for StrictTypeChecking for boolean types.

Test: svg/dom/SVGExternalResourcesRequired.html

  • GNUmakefile.am: Add SVGAnimatedBoolean.h and SVGAnimatedStaticPropertyTearOff.h to build.
  • WebCore.gypi: Ditto.
  • WebCore.pro: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • bindings/scripts/CodeGenerator.pm: Add SVGAnimatedBoolean to the new style svg animated bindings list.
  • bindings/scripts/CodeGeneratorJS.pm: Add StrictTypeChecking support for boolean types.
  • bindings/scripts/CodeGeneratorObjC.pm: Ditto. Only generate kit() usage for new style animated types, if the wrapped type needs a tear off.
  • bindings/scripts/CodeGeneratorV8.pm: Ditto.
  • svg/DeprecatedSVGAnimatedTemplate.h: Remove SVGAnimatedBoolean handling.
  • svg/SVGAElement.h: s/DECLARE_ANIMATED_PROPERTY/DECLARE_ANIMATED_STATIC_PROPERTY_NEW/ for boolean types everywhere. Include SVGAnimatedPropertyMacros.h where needed.
  • svg/SVGAnimatedBoolean.h: Added.
  • svg/SVGAnimatedBoolean.idl:
  • svg/SVGAnimationElement.h:
  • svg/SVGCircleElement.h:
  • svg/SVGClipPathElement.h:
  • svg/SVGCursorElement.h:
  • svg/SVGDefsElement.h:
  • svg/SVGEllipseElement.h:
  • svg/SVGExternalResourcesRequired.cpp:
  • svg/SVGExternalResourcesRequired.h: (WebCore::SVGExternalResourcesRequired::~SVGExternalResourcesRequired):
  • svg/SVGFEConvolveMatrixElement.h:
  • svg/SVGFEImageElement.h:
  • svg/SVGFilterElement.h:
  • svg/SVGFontElement.h:
  • svg/SVGForeignObjectElement.h:
  • svg/SVGGElement.h:
  • svg/SVGGradientElement.h:
  • svg/SVGImageElement.h:
  • svg/SVGLineElement.h:
  • svg/SVGMPathElement.h:
  • svg/SVGMarkerElement.h:
  • svg/SVGMaskElement.h:
  • svg/SVGPathElement.h:
  • svg/SVGPatternElement.h:
  • svg/SVGPolyElement.h:
  • svg/SVGRectElement.h:
  • svg/SVGSVGElement.h:
  • svg/SVGScriptElement.h:
  • svg/SVGSwitchElement.h:
  • svg/SVGSymbolElement.h:
  • svg/SVGTextContentElement.h:
  • svg/SVGUseElement.h:
  • svg/SVGViewElement.h:
  • svg/properties/SVGAnimatedListPropertyTearOff.h: Moved RefPtr<SVGProperty> m_baseVal/m_animVal from SVGAnimatedProperty, as SVGAnimatedStaticPropertyTearOff doesn't need them.
  • svg/properties/SVGAnimatedProperty.h: Remove m_baseVal/m_animVal. (WebCore::SVGAnimatedProperty::commitChange): New method to share code between the SVGAnimated*PropertyTearOff classes.
  • svg/properties/SVGAnimatedPropertyMacros.h: Add new DECLARE_ANIMATED_STATIC_PROPERTY_NEW macro which creates SVGAnimatedStaticPropertyTearOff types.
  • svg/properties/SVGAnimatedPropertyTearOff.h: Moved RefPtr<SVGProperty> m_baseVal/m_animVal from SVGAnimatedProperty, as SVGAnimatedStaticPropertyTearOff doesn't need them.
  • svg/properties/SVGAnimatedStaticPropertyTearOff.h: Added. (WebCore::SVGAnimatedStaticPropertyTearOff::baseVal): Directly returns the stored m_property as reference, no tear off is created. (WebCore::SVGAnimatedStaticPropertyTearOff::animVal): Ditto. (WebCore::SVGAnimatedStaticPropertyTearOff::setBaseVal): Directly updates the stored m_property reference, and invalidates the SVG attributes. (WebCore::SVGAnimatedStaticPropertyTearOff::setAnimVal): Not implemented. (WebCore::SVGAnimatedStaticPropertyTearOff::create): (WebCore::SVGAnimatedStaticPropertyTearOff::SVGAnimatedStaticPropertyTearOff):
  • svg/properties/SVGListPropertyTearOff.h: (WebCore::SVGListPropertyTearOff::commitChange): Share code, use SVGAnimatedProperty::commitChange.
  • svg/properties/SVGPropertyTearOff.h: (WebCore::SVGPropertyTearOff::commitChange): Ditto.

2010-10-27 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Dirk Schulze.

Convert SVGAnimatedBoolean to the new SVGAnimatedPropertyTearOff concept
https://bugs.webkit.org/show_bug.cgi?id=48469

Add simple test that SVGAnimatedBooleans aren't creating tear off when accessing baseVal/animVal.
For example with SVGLength you can do:
var foo = rect.x.baseVal; SVGRectElement -> SVGAnimatedLength -> SVGLength
foo.value = 50;
affects the SVGRectElement

For SVGExternalResourcesRequired (one of the two SVGAnimatedBoolean types), this is not possible, as baseVal returns a POD type (boolean).
var bar = rect.externalResourcesRequired.baseVal; bar is a boolean
bar = false;
does NOT effect the rect.externalResourcesRequired object

  • svg/dom/SVGExternalResourcesRequired-expected.txt: Added.
  • svg/dom/SVGExternalResourcesRequired.html: Added.
  • svg/dom/script-tests/SVGExternalResourcesRequired.js: Added.
2:47 AM Changeset in webkit [70856] by morrita@google.com
  • 7 edits
    4 adds in trunk

2010-10-28 MORITA Hajime <morrita@google.com>

Reviewed by Kent Tamura.

<meter min=0 max=100 value=100> should be optimum.
https://bugs.webkit.org/show_bug.cgi?id=48613

  • fast/dom/HTMLMeterElement/meter-boundary-values.html: Added.
  • platform/mac/fast/dom/HTMLMeterElement/meter-appearances-capacity-expected.checksum:
  • platform/mac/fast/dom/HTMLMeterElement/meter-appearances-capacity-expected.png:
  • platform/mac/fast/dom/HTMLMeterElement/meter-boundary-values-expected.checksum: Added.
  • platform/mac/fast/dom/HTMLMeterElement/meter-boundary-values-expected.png: Added.
  • platform/mac/fast/dom/HTMLMeterElement/meter-boundary-values-expected.txt: Added.
  • platform/mac/fast/dom/HTMLMeterElement/meter-optimums-expected.checksum:
  • platform/mac/fast/dom/HTMLMeterElement/meter-optimums-expected.png:

2010-10-28 MORITA Hajime <morrita@google.com>

Reviewed by Kent Tamura.

<meter min=0 max=100 value=100> should be optimum.
https://bugs.webkit.org/show_bug.cgi?id=48613

Changed to make the condition inclusive for better case consistently.

  • Values on the border between optimum and subsequntial go optimum.
  • Values on the border between suboptimal and even-less-than-good go suboptimal.

Test: fast/dom/HTMLMeterElement/meter-boundary-values.html

  • html/HTMLMeterElement.cpp: (WebCore::HTMLMeterElement::gaugeRegion):
2:43 AM Changeset in webkit [70855] by mnaganov@chromium.org
  • 19 edits
    17 adds in trunk/LayoutTests

2010-10-29 Mikhail Naganov <mnaganov@chromium.org>

[Chromium] Unreviewed test expectations update after r70843.

  • platform/chromium-linux/fast/blockflow/box-shadow-horizontal-bt-expected.checksum: Added.
  • platform/chromium-linux/fast/blockflow/box-shadow-horizontal-bt-expected.png: Added.
  • platform/chromium-linux/fast/blockflow/box-shadow-vertical-lr-expected.checksum: Added.
  • platform/chromium-linux/fast/blockflow/box-shadow-vertical-lr-expected.png: Added.
  • platform/chromium-linux/fast/blockflow/box-shadow-vertical-rl-expected.checksum: Added.
  • platform/chromium-linux/fast/blockflow/box-shadow-vertical-rl-expected.png: Added.
  • platform/chromium-linux/fast/blockflow/english-lr-text-expected.checksum:
  • platform/chromium-linux/fast/blockflow/english-rl-text-expected.checksum:
  • platform/chromium-linux/fast/blockflow/japanese-lr-text-expected.checksum:
  • platform/chromium-linux/fast/blockflow/japanese-lr-text-expected.png:
  • platform/chromium-linux/fast/blockflow/japanese-rl-text-expected.checksum:
  • platform/chromium-linux/fast/blockflow/japanese-rl-text-expected.png:
  • platform/chromium-mac/fast/blockflow/border-vertical-lr-expected.checksum:
  • platform/chromium-mac/fast/blockflow/box-shadow-vertical-lr-expected.checksum: Added.
  • platform/chromium-mac/fast/blockflow/box-shadow-vertical-rl-expected.checksum: Added.
  • platform/chromium-mac/fast/blockflow/english-lr-text-expected.checksum:
  • platform/chromium-mac/fast/blockflow/japanese-lr-text-expected.checksum:
  • platform/chromium-mac/fast/blockflow/japanese-rl-text-expected.checksum:
  • platform/chromium-win/fast/blockflow/box-shadow-horizontal-bt-expected.checksum: Added.
  • platform/chromium-win/fast/blockflow/box-shadow-horizontal-bt-expected.png: Added.
  • platform/chromium-win/fast/blockflow/box-shadow-horizontal-bt-expected.txt: Added.
  • platform/chromium-win/fast/blockflow/box-shadow-vertical-lr-expected.checksum: Added.
  • platform/chromium-win/fast/blockflow/box-shadow-vertical-lr-expected.png: Added.
  • platform/chromium-win/fast/blockflow/box-shadow-vertical-lr-expected.txt: Added.
  • platform/chromium-win/fast/blockflow/box-shadow-vertical-rl-expected.checksum: Added.
  • platform/chromium-win/fast/blockflow/box-shadow-vertical-rl-expected.png: Added.
  • platform/chromium-win/fast/blockflow/box-shadow-vertical-rl-expected.txt: Added.
  • platform/chromium-win/fast/blockflow/english-lr-text-expected.checksum:
  • platform/chromium-win/fast/blockflow/english-lr-text-expected.png:
  • platform/chromium-win/fast/blockflow/english-rl-text-expected.checksum:
  • platform/chromium-win/fast/blockflow/english-rl-text-expected.png:
  • platform/chromium-win/fast/blockflow/japanese-lr-text-expected.checksum:
  • platform/chromium-win/fast/blockflow/japanese-lr-text-expected.png:
  • platform/chromium-win/fast/blockflow/japanese-rl-text-expected.checksum:
  • platform/chromium-win/fast/blockflow/japanese-rl-text-expected.png:
2:32 AM Changeset in webkit [70854] by commit-queue@webkit.org
  • 32 edits
    4 copies
    2 adds in trunk

2010-10-29 Kenichi Ishibashi <bashi@google.com>

Reviewed by Kent Tamura.

[HTML5] Add DOMSettableTokenList
https://bugs.webkit.org/show_bug.cgi?id=47810

  • fast/dom/Window/window-properties-expected.txt: Updated expectations.
  • fast/dom/Window/window-property-descriptors-expected.txt: Updated expectations.
  • fast/dom/prototype-inheritance-2-expected.txt: Updated expectations.
  • fast/dom/prototype-inheritance-expected.txt: Updated expectations.
  • fast/js/global-constructors-expected.txt: Updated expectations.

2010-10-29 Kenichi Ishibashi <bashi@google.com>

Reviewed by Kent Tamura.

[HTML5] Add DOMSettableTokenList
https://bugs.webkit.org/show_bug.cgi?id=47810

Adds DOMSettableTokenList interface to support <output> element.
The DOMTokenList interface was refactored since its was tightly coupled
with HTMLElement.classList property.

  • Android.derived.jscbindings.mk: Added JSDOMSettableTokenList.h file.
  • Android.derived.v8bindings.mk: Added V8DOMSettableTokenList.h file.
  • Android.jscbindings.mk: Added JSDOMSettableTokenList.cpp file.
  • Android.mk: Added ClassList.cpp file.
  • CMakeLists.txt: Added DOMSettableTokenList.{cpp,idl} and ClassList.cpp files.
  • DerivedSources.cpp: Added JSDOMSettableTokenList.cpp file.
  • DerivedSources.make: Added DOMSettableTokenList entry.
  • GNUmakefile.am: Added JSDOMSettableTokenList.{h,cpp} and ClassList.{h,cpp} files.
  • WebCore.gypi: Added DOMSettableTokenList.{h,cpp,idl}, V8DOMSettableTokenListCustom.cpp and ClassList.{h,cpp} files.
  • WebCore.pri: Added DOMSettableTokenList.idl file.
  • WebCore.pro: Added V8DOMSettableTokenListCustom.cpp, ClassList.{h,cpp} and DOMSettableTokenList.{h,cpp} files.
  • WebCore.vcproj/WebCore.vcproj: Added JSDOMSettableTokenList.{h,cpp}, ClassList.{h,cpp} and DOMSettableTokenList.{h,cpp} files.
  • WebCore.xcodeproj/project.pbxproj: Added ClassList.{h,cpp}, DOMSettabletokenList.{h,cpp} and JSDOMSettableTokenList.{h,cpp}
  • bindings/gobject/GNUmakefile.am: Added WebKitDOMDOMSettableTokenList.{h,cpp} and WebKitDOMDOMSettableTokenListPrivate.cpp files.
  • bindings/scripts/CodeGeneratorJS.pm: Modified IndexGetterReturnsStrings() returns 1 for DOMSettableTokenList.
  • bindings/v8/custom/V8DOMSettableTokenListCustom.cpp: Added. (WebCore::V8DOMSettableTokenList::indexedPropertyGetter):
  • dom/Element.cpp: Replaced DOMTokenList to ClassList. (WebCore::Element::classList):
  • dom/ElementRareData.h: ditto.
  • dom/SpaceSplitString.cpp: Added add() and remove() functions. (WebCore::SpaceSplitStringData::add): (WebCore::SpaceSplitStringData::remove): (WebCore::SpaceSplitString::add): (WebCore::SpaceSplitString::remove):
  • dom/SpaceSplitString.h: Added add() and remove() functions.
  • dom/StyledElement.cpp: Replaced DOMTokenList to ClassList. (WebCore::StyledElement::classAttributeChanged):
  • dom/StyledElement.h: ditto.
  • html/ClassList.cpp: Added, mostly from original DOMTokenList.cpp. (WebCore::ClassList::ClassList): (WebCore::ClassList::ref): (WebCore::ClassList::deref): (WebCore::ClassList::length): (WebCore::ClassList::item): (WebCore::ClassList::contains): (WebCore::ClassList::containsInternal): (WebCore::ClassList::add): (WebCore::ClassList::addInternal): (WebCore::ClassList::remove): (WebCore::ClassList::removeInternal): (WebCore::ClassList::toggle): (WebCore::ClassList::toString): (WebCore::ClassList::reset): (WebCore::ClassList::classNames):
  • html/ClassList.h: Added, mostly from original DOMTokenList.h. (WebCore::ClassList::create): (WebCore::ClassList::element):
  • html/DOMSettableTokenList.cpp: Added. (WebCore::DOMSettableTokenList::DOMSettableTokenList): (WebCore::DOMSettableTokenList::~DOMSettableTokenList): (WebCore::DOMSettableTokenList::item): (WebCore::DOMSettableTokenList::contains): (WebCore::DOMSettableTokenList::add): (WebCore::DOMSettableTokenList::addInternal): (WebCore::DOMSettableTokenList::remove): (WebCore::DOMSettableTokenList::removeInternal): (WebCore::DOMSettableTokenList::toggle): (WebCore::DOMSettableTokenList::setValue):
  • html/DOMSettableTokenList.h: Added. (WebCore::DOMSettableTokenList::create): (WebCore::DOMSettableTokenList::ref): (WebCore::DOMSettableTokenList::deref): (WebCore::DOMSettableTokenList::length): (WebCore::DOMSettableTokenList::toString): (WebCore::DOMSettableTokenList::value):
  • html/DOMSettableTokenList.idl: Added.
  • html/DOMTokenList.cpp: Refactored to be an abstract interface for both of ClassList and DOMSettableTokenList. (WebCore::DOMTokenList::validateToken): Changed to be a static member function of DOMTokenList. (WebCore::DOMTokenList::addToken): Added, mostly from original DOMTokenList::addInternal(). (WebCore::DOMTokenList::removeToken): Added, mostly from original DOMTokenList::removeInternal().
  • html/DOMTokenList.h: Refactored to be an abstract interface for both of ClassList and DOMSettableTokenList. (WebCore::DOMTokenList::~DOMTokenList): (WebCore::DOMTokenList::element):
  • page/DOMWindow.cpp: Added an entry for DOMSettableTokenList.
  • page/DOMWindow.idl: ditto.
2:00 AM Changeset in webkit [70853] by pfeldman@chromium.org
  • 11 edits in trunk/WebCore

2010-10-29 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: make legacy resource tracking optional in the native code,
make cookeis work against new resource manager.
https://bugs.webkit.org/show_bug.cgi?id=48532

  • inspector/InspectorCSSStore.cpp: (WebCore::InspectorCSSStore::resourceStyleSheetText):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::~InspectorController): (WebCore::InspectorController::connectFrontend): (WebCore::InspectorController::releaseFrontendLifetimeAgents): (WebCore::InspectorController::populateScriptObjects): (WebCore::InspectorController::unbindAllResources): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::frameDetachedFromParent): (WebCore::InspectorController::didLoadResourceFromMemoryCache): (WebCore::InspectorController::identifierForInitialRequest): (WebCore::InspectorController::willSendRequest): (WebCore::InspectorController::markResourceAsCached): (WebCore::InspectorController::didReceiveResponse): (WebCore::InspectorController::didReceiveContentLength): (WebCore::InspectorController::didFinishLoading): (WebCore::InspectorController::didFailLoading): (WebCore::InspectorController::resourceRetrievedByXMLHttpRequest): (WebCore::InspectorController::scriptImported): (WebCore::InspectorController::setResourceTrackingEnabled): (WebCore::InspectorController::getCookies): (WebCore::InspectorController::deleteCookie): (WebCore::InspectorController::didCreateWebSocket): (WebCore::InspectorController::willSendWebSocketHandshakeRequest): (WebCore::InspectorController::didReceiveWebSocketHandshakeResponse): (WebCore::InspectorController::didCloseWebSocket): (WebCore::InspectorController::breakpointsSettingKey): (WebCore::InspectorController::getResourceContent):
  • inspector/InspectorController.h:
  • inspector/InspectorResource.cpp: (WebCore::InspectorResource::updateScriptObject): (WebCore::InspectorResource::markFailed):
  • inspector/InspectorResource.h:
  • inspector/InspectorResourceAgent.cpp: (WebCore::populateObjectWithFrameResources):
  • inspector/front-end/CookieItemsView.js: (WebInspector.CookieItemsView.prototype._updateWithCookies): (WebInspector.CookieItemsView.prototype._filterCookiesForDomain):
  • inspector/front-end/Resource.js: (WebInspector.Resource.prototype._mimeTypeIsConsistentWithType): (WebInspector.Resource.prototype._checkWarning):
  • inspector/front-end/ResourceManager.js: (WebInspector.ResourceManager.prototype._updateResourceWithResponse): (WebInspector.ResourceManager.prototype.didFailLoading):
  • inspector/front-end/inspector.js:
1:23 AM Changeset in webkit [70852] by morrita@google.com
  • 2 edits in trunk/WebCore

2010-10-29 MORITA Hajime <morrita@google.com>

Unreviewed attempt for fixing build break for http://trac.webkit.org/changeset/70847

  • editing/EditingAllInOne.cpp:
1:08 AM Changeset in webkit [70851] by mihaip@chromium.org
  • 4 edits in trunk

2010-10-29 Mihai Parparita <mihaip@chromium.org>

Reviewed by Adam Barth.

Support RegExp in V8 implementation of SerializedScriptValue
https://bugs.webkit.org/show_bug.cgi?id=44809

fast/loader/stateobjects/pushstate-object-types.html now passes.

  • platform/chromium/test_expectations.txt:

2010-10-29 Mihai Parparita <mihaip@chromium.org>

Reviewed by Adam Barth.

Support RegExp in V8 implementation of SerializedScriptValue
https://bugs.webkit.org/show_bug.cgi?id=44809

Add RegExp serialization/deserialization now that V8 provides API access
to the RegExp type.

  • bindings/v8/SerializedScriptValue.cpp: (WebCore::ZigZag::Writer::writeRegExp): (WebCore::ZigZag::Serializer::writeRegExp): (WebCore::ZigZag::Serializer::doSerialize): (WebCore::ZigZag::Reader::read): (WebCore::ZigZag::Reader::readRegExp):
12:49 AM Changeset in webkit [70850] by morrita@google.com
  • 7 edits in trunk/WebCore

2010-10-13 MORITA Hajime <morrita@google.com>

Reviewed by David Hyatt.

Navigating dark background websites results in blinding white flashes between pages.
https://bugs.webkit.org/show_bug.cgi?id=45640

This FOUC is caused by an early layout request before the <body> is ready,
and the page's background style given for <body>, instead of <html>.
So many sites have such stylesheets that we should care them.

  • Some DOM operation such as 'element.offsetLeft' causes page layout.
  • The page layout results page repaint
  • The page page repaint makes a white screen. because there is nothing to paint before <body> is available.

This change:

  • extracted existing FOUC check on RenderBlock and RenderLayer to Document::mayCauseFlashOfUnstyledContent(),
  • checked <body> availability on mayCauseFlashOfUnstyledContent(), and
  • added FOUC guards before requesting reapint on FrameView.

No new tests. The data loading speed matters and it cannot be
captured by DRT.

  • dom/Document.cpp: (WebCore::Document::mayCauseFlashOfUnstyledContent): Added.
  • dom/Document.h:
  • page/FrameView.cpp: (WebCore::FrameView::invalidateRect): Added a guard. (WebCore::FrameView::repaintContentRectangle): Added a guard. (WebCore::FrameView::doDeferredRepaints): Added a guard. (WebCore::FrameView::shouldUpdate): Added.
  • page/FrameView.h:
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paintContents): Replaced FOUC check to use mayCauseFlashOfUnstyledContent
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintLayer): Replaced FOUC check to use mayCauseFlashOfUnstyledContent
12:16 AM Changeset in webkit [70849] by pfeldman@chromium.org
  • 2 edits in branches/chromium/552/WebCore/inspector

Merge 70799 - 2010-10-28 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: fix pausing on next JavaScript statement
https://bugs.webkit.org/show_bug.cgi?id=48534

  • inspector/InspectorDebuggerAgent.cpp: (WebCore::InspectorDebuggerAgent::InspectorDebuggerAgent): (WebCore::InspectorDebuggerAgent::schedulePauseOnNextStatement): (WebCore::InspectorDebuggerAgent::cancelPauseOnNextStatement): (WebCore::InspectorDebuggerAgent::pause): (WebCore::InspectorDebuggerAgent::didPause):
  • inspector/InspectorDebuggerAgent.h:

TBR=podivilov@chromium.org
BUG=61195

12:07 AM Changeset in webkit [70848] by hyatt@apple.com
  • 8 edits
    4 adds in trunk

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

Reviewed by Dan Bernstein.

WebCore:

Backgrounds should clip to the border radius properly for vertical lines. The code that does this still assumes horizontal lines.
The testcase to cover this used nested spans, which also exposed a bug in the propagation of the vertical bit through the line
box tree.

Added fast/blockflow/border-radius-clipping-vertical-lr.html.

  • rendering/RenderBlockLineLayout.cpp:

(WebCore::RenderBlock::createLineBoxes):

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::paintFillLayerExtended):

LayoutTests:

Backgrounds should clip to the border radius properly for vertical lines. The code that does this still assumes horizontal lines.
The testcase to cover this used nested spans, which also exposed a bug in the propagation of the vertical bit through the line
box tree.

Added fast/blockflow/border-radius-clipping-vertical-lr.html.

  • fast/blockflow/border-radius-clipping-vertical-lr.html: Added.
  • platform/mac/fast/blockflow/border-radius-clipping-vertical-lr-expected.checksum: Added.
  • platform/mac/fast/blockflow/border-radius-clipping-vertical-lr-expected.png: Added.
  • platform/mac/fast/blockflow/border-radius-clipping-vertical-lr-expected.txt: Added.
  • platform/mac/fast/blockflow/box-shadow-vertical-lr-expected.checksum:
  • platform/mac/fast/blockflow/box-shadow-vertical-lr-expected.png:
  • platform/mac/fast/blockflow/box-shadow-vertical-rl-expected.checksum:
  • platform/mac/fast/blockflow/box-shadow-vertical-rl-expected.png:
Note: See TracTimeline for information about the timeline view.