Timeline


and

03/26/10:

22:44 Changeset [56658] by abarth@webkit.org

2010-03-26 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Add some basic IRC commands to sheriffbot
https://bugs.webkit.org/show_bug.cgi?id=36684

Adds support for sheriffbot to respond to a "hi" command and a
"last-green-revision" command. It's lame that we're rebuilding
MultiCommandTool, but as discussed in person we'll intergrate the two
once we see what the requirements are.

  • Scripts/webkitpy/tool/bot/irc_command.py: Added.
  • Scripts/webkitpy/tool/bot/queueengine.py:
  • Scripts/webkitpy/tool/bot/queueengine_unittest.py:
  • Scripts/webkitpy/tool/bot/sheriffircbot.py:
  • Scripts/webkitpy/tool/bot/sheriffircbot_unittest.py: Added.
  • Scripts/webkitpy/tool/commands/queues.py:
  • Scripts/webkitpy/tool/commands/queuestest.py:
  • Scripts/webkitpy/tool/commands/sheriffbot.py:
  • Scripts/webkitpy/tool/main.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/unittests.py:
22:42 Changeset [56657] by abarth@webkit.org

2010-03-26 Adam Barth <abarth@webkit.org>

Unreviewed. More errors in the IRC module. I have no idea how to test
ircbot.py, which is too bad. Hopefully we've abstracted it away enough
that we don't have to touch it very much after this patch.

  • Scripts/webkitpy/common/net/irc/ircbot.py:
22:37 Changeset [56656] by abarth@webkit.org

2010-03-26 Adam Barth <abarth@webkit.org>

Unreviewed. Add a missing "_".

  • Scripts/webkitpy/common/net/irc/ircproxy.py:
  • Scripts/webkitpy/common/net/irc/ircproxy_unittest.py: Added.
  • Scripts/webkitpy/common/net/irc/unittests.py:
22:08 Changeset [56655] by dbates@webkit.org

2010-03-26 Daniel Bates <dbates@rim.com>

Reviewed by Beth Dakin.

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

As per sections 14.2 (http://www.w3.org/TR/CSS2/colors.html#background)
and 17.5.1 (http://www.w3.org/TR/CSS2/tables.html#table-layers) of the
CSS 2.1 spec., the background-image of a table-row should span the
entire area of the table-row (i.e cover all the <td>'s). Moreover,
fixes background image painting with respect to row-groups, columns
and column-groups (*).

Currently, the background-image of a table row, row-group, and column-group
is inherited by the table cells. This directly contradicts section 14.2.
Instead, the background-image should cover the entire area of the table row,
row-group, column, or column-group, respectively as described in section 17.5.1.

(*) A follow up patch to compute the absolute content box of a column, and
column-group is needed so that background image positioning for these table
elements works. See bug #36104.

Tests: fast/table/table-background.html

fast/table/table-cell-border-draws-on-top-of-col-background.html
fast/table/table-cell-border-draws-on-top-of-col-group-background.html
fast/table/table-cell-border-draws-on-top-of-row-background.html
fast/table/table-cell-border-draws-on-top-of-row-group-background.html
fast/table/table-cell-overrides-row-background.html
fast/table/table-col-background.html
fast/table/table-col-group-background.html
fast/table/table-col-group-col-span-background.html
fast/table/table-col-group-span-background.html
fast/table/table-col-span-background.html
fast/table/table-row-background-left-50px.html
fast/table/table-row-background-right-100percent.html
fast/table/table-row-background-right-50px.html
fast/table/table-row-background.html
fast/table/table-row-group-background-positioned.html
fast/table/table-row-group-background.html

  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelObject::paintFillLayerExtended): (WebCore::RenderBoxModelObject::calculateBackgroundImageGeometry): Modified to call RenderObject::adjustBackgroundImagePosition.
  • rendering/RenderBoxModelObject.h:
  • rendering/RenderObject.h: (WebCore::RenderObject::adjustBackgroundImagePosition): Added.
  • rendering/RenderTableCell.cpp: (WebCore::RenderTableCell::adjustBackgroundImagePosition): Added.
  • rendering/RenderTableCell.h:

2010-03-26 Daniel Bates <dbates@rim.com>

Reviewed by Beth Dakin.

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

Tests that background-image support for table, table-cell, table-row,
row-group, table-column and column-group (*) conform to section 17.5.1
of the CSS 2.1 spec.

(*) A follow up patch to compute the absolute content box of a column, and
column-group is needed so that background image positioning for these table
elements works. See bug #36104.

  • fast/table/table-background.html: Added.
  • fast/table/table-cell-border-draws-on-top-of-col-background.html: Added.
  • fast/table/table-cell-border-draws-on-top-of-col-group-background.html: Added.
  • fast/table/table-cell-border-draws-on-top-of-row-background.html: Added.
  • fast/table/table-cell-border-draws-on-top-of-row-group-background.html: Added.
  • fast/table/table-cell-overrides-row-background.html: Added.
  • fast/table/table-col-background.html: Added.
  • fast/table/table-col-group-background.html: Added.
  • fast/table/table-col-group-col-span-background.html: Added.
  • fast/table/table-col-group-span-background.html: Added.
  • fast/table/table-col-span-background.html: Added.
  • fast/table/table-row-background-left-50px.html: Added.
  • fast/table/table-row-background-right-100percent.html: Added.
  • fast/table/table-row-background-right-50px.html: Added.
  • fast/table/table-row-background.html: Added.
  • fast/table/table-row-group-background-positioned.html: Added.
  • fast/table/table-row-group-background.html: Added.
  • platform/mac/fast/table/table-background-expected.checksum: Added.
  • platform/mac/fast/table/table-background-expected.png: Added.
  • platform/mac/fast/table/table-background-expected.txt: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-col-background-expected.checksum: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-col-background-expected.png: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-col-background-expected.txt: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-col-group-background-expected.checksum: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-col-group-background-expected.png: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-col-group-background-expected.txt: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-row-background-expected.checksum: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-row-background-expected.png: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-row-background-expected.txt: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-row-group-background-expected.checksum: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-row-group-background-expected.png: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-row-group-background-expected.txt: Added.
  • platform/mac/fast/table/table-cell-overrides-row-background-expected.checksum: Added.
  • platform/mac/fast/table/table-cell-overrides-row-background-expected.png: Added.
  • platform/mac/fast/table/table-cell-overrides-row-background-expected.txt: Added.
  • platform/mac/fast/table/table-col-background-expected.checksum: Added.
  • platform/mac/fast/table/table-col-background-expected.png: Added.
  • platform/mac/fast/table/table-col-background-expected.txt: Added.
  • platform/mac/fast/table/table-col-group-background-expected.checksum: Added.
  • platform/mac/fast/table/table-col-group-background-expected.png: Added.
  • platform/mac/fast/table/table-col-group-background-expected.txt: Added.
  • platform/mac/fast/table/table-col-group-col-span-background-expected.checksum: Added.
  • platform/mac/fast/table/table-col-group-col-span-background-expected.png: Added.
  • platform/mac/fast/table/table-col-group-col-span-background-expected.txt: Added.
  • platform/mac/fast/table/table-col-group-span-background-expected.checksum: Added.
  • platform/mac/fast/table/table-col-group-span-background-expected.png: Added.
  • platform/mac/fast/table/table-col-group-span-background-expected.txt: Added.
  • platform/mac/fast/table/table-col-span-background-expected.checksum: Added.
  • platform/mac/fast/table/table-col-span-background-expected.png: Added.
  • platform/mac/fast/table/table-col-span-background-expected.txt: Added.
  • platform/mac/fast/table/table-row-background-expected.checksum: Added.
  • platform/mac/fast/table/table-row-background-expected.png: Added.
  • platform/mac/fast/table/table-row-background-expected.txt: Added.
  • platform/mac/fast/table/table-row-background-left-50px-expected.checksum: Added.
  • platform/mac/fast/table/table-row-background-left-50px-expected.png: Added.
  • platform/mac/fast/table/table-row-background-left-50px-expected.txt: Added.
  • platform/mac/fast/table/table-row-background-right-100percent-expected.checksum: Added.
  • platform/mac/fast/table/table-row-background-right-100percent-expected.png: Added.
  • platform/mac/fast/table/table-row-background-right-100percent-expected.txt: Added.
  • platform/mac/fast/table/table-row-background-right-50px-expected.checksum: Added.
  • platform/mac/fast/table/table-row-background-right-50px-expected.png: Added.
  • platform/mac/fast/table/table-row-background-right-50px-expected.txt: Added.
  • platform/mac/fast/table/table-row-group-background-expected.checksum: Added.
  • platform/mac/fast/table/table-row-group-background-expected.png: Added.
  • platform/mac/fast/table/table-row-group-background-expected.txt: Added.
  • platform/mac/fast/table/table-row-group-background-positioned-expected.checksum: Added.
  • platform/mac/fast/table/table-row-group-background-positioned-expected.png: Added.
  • platform/mac/fast/table/table-row-group-background-positioned-expected.txt: Added.
22:00 Changeset [56654] by abarth@webkit.org

2010-03-26 Adam Barth <abarth@webkit.org>

Unreviewed. Fix a the commit queue after my recent change.

  • Scripts/webkitpy/tool/bot/queueengine.py:
21:31 Changeset [56653] by laszlo.1.gombos@nokia.com

2010-03-26 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Unreviewed, minor build fix.

[Gtk] Guard defined but not used function

Eliminate the "defined but not used" warning after r52684.
No new tests as there is no new functionality.

  • platform/ContextMenu.cpp:
19:38 Changeset [56652] by dpranke@chromium.org

2010-03-26 Dirk Pranke <dpranke@chromium.org>

Unreviewed fix.

Fix the return value for port/base.diff_image (changed from 1/0 to
True/False in bug 34826.

  • Scripts/webkitpy/layout_tests/port/base.py:
19:15 Changeset [56651] by abarth@webkit.org

2010-03-26 Justin Schuh <jschuh@chromium.org>

Reviewed by Adam Barth.

Security: iFrame.src accepts JavaScript URL via nodeValue or textContent
https://bugs.webkit.org/show_bug.cgi?id=36502

Overrode inherited nodeValue and textContent in Attr.idl so they proxy
to value, which performs a security check.

  • http/tests/security/xss-DENIED-iframe-src-alias-expected.txt: Added.
  • http/tests/security/xss-DENIED-iframe-src-alias.html: Added.

2010-03-26 Justin Schuh <jschuh@chromium.org>

Reviewed by Adam Barth.

Security: iFrame.src accepts JavaScript URL via nodeValue or textContent
https://bugs.webkit.org/show_bug.cgi?id=36502

Overrode inherited nodeValue and textContent in Attr.idl so they proxy
to value, which performs a security check.

Test: http/tests/security/xss-DENIED-iframe-src-alias.html

  • bindings/js/JSAttrCustom.cpp: (WebCore::JSAttr::nodeValue): (WebCore::JSAttr::setNodeValue): (WebCore::JSAttr::textContent): (WebCore::JSAttr::setTextContent):
  • bindings/v8/custom/V8AttrCustom.cpp: (WebCore::V8Attr::nodeValueAccessorSetter): (WebCore::V8Attr::nodeValueAccessorGetter): (WebCore::V8Attr::textContentAccessorSetter): (WebCore::V8Attr::textContentAccessorGetter):
  • dom/Attr.idl:
18:20 Changeset [56650] by eric.carlson@apple.com

2010-03-26 Eric Carlson <eric.carlson@apple.com>

<rdar://problem/7320584>
https://bugs.webkit.org/show_bug.cgi?id=36681
"new Audio()" doesn't work with plug-in backed media engine.

Don't require plug-in backed <video> and <audio> elements to be in the document to play.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::HTMLMediaElement): Initialize m_proxyWidget (WebCore::HTMLMediaElement::createRenderer): Set renderer widget if it has already been allocated. (WebCore::HTMLMediaElement::scheduleLoad): Call createMediaPlayerProxy to make sure we have a plug-in proxy. (WebCore::HTMLMediaElement::loadResource): Ditto. (WebCore::HTMLMediaElement::allocateMediaPlayerIfNecessary): Allocate media player. (WebCore::HTMLMediaElement::setMediaPlayerProxy): Call allocateMediaPlayerIfNecessary. (WebCore::HTMLMediaElement::getPluginProxyParams): New, return the url and params vectors. (WebCore::HTMLMediaElement::finishParsingChildren): Call createMediaPlayerProxy. (WebCore::HTMLMediaElement::createMediaPlayerProxy): New, allocate the proxy widget if necessary.
  • html/HTMLMediaElement.h:
  • loader/EmptyClients.h: (WebCore::EmptyFrameLoaderClient::createMediaPlayerProxyPlugin): Add declaration.
  • loader/FrameLoader.cpp: (WebCore::toPlugInElement): Don't allow an audio or video element to be cast to a plug-in element. (WebCore::FrameLoader::loadPlugin): (WebCore::FrameLoader::loadMediaPlayerProxyPlugin): New, allocate a media player proxy plug-in.
  • loader/FrameLoader.h: Declare loadMediaPlayerProxyPlugin.
  • loader/FrameLoaderClient.h: Declare createMediaPlayerProxyPlugin.
  • rendering/RenderEmbeddedObject.cpp: (WebCore::RenderEmbeddedObject::updateWidget): Call loadMediaPlayerProxyPlugin when passed an audio or video node.
  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::requiresCompositingForVideo):
  • rendering/RenderVideo.cpp: (WebCore::RenderVideo::RenderVideo): Get default width and height from defaultSize. (WebCore::RenderVideo::defaultSize): New, return the spec defined default width and height.
  • rendering/RenderVideo.h:
18:11 Changeset [56649] by dpranke@chromium.org

2010-03-26 Dirk Pranke <dpranke@chromium.org>

Reviewed by David Levin.

Update a bunch of incorrect image checksums, and suppress all of the
remaining IMAGE failures in platform/mac/test_expectations.txt.

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

  • platform/mac-leopard/compositing/geometry/root-layer-update-expected.checksum:
  • platform/mac-leopard/compositing/overflow/overflow-positioning-expected.checksum:
  • platform/mac-leopard/compositing/repaint/become-overlay-composited-layer-expected.checksum:
  • platform/mac-leopard/compositing/repaint/layer-repaint-rects-expected.checksum:
  • platform/mac-leopard/media/video-zoom-controls-expected.checksum:
  • platform/mac/animations/3d/change-transform-in-end-event-expected.checksum:
  • platform/mac/animations/3d/state-at-end-event-transform-expected.checksum:
  • platform/mac/animations/state-at-end-event-expected.checksum:
  • platform/mac/compositing/animation/state-at-end-event-transform-layer-expected.checksum:
  • platform/mac/compositing/color-matching/pdf-image-match-expected.checksum:
  • platform/mac/compositing/compositing-visible-descendant-expected.checksum:
  • platform/mac/compositing/generated-content-expected.checksum:
  • platform/mac/compositing/geometry/abs-position-inside-opacity-expected.checksum:
  • platform/mac/compositing/geometry/fixed-position-expected.checksum:
  • platform/mac/compositing/geometry/horizontal-scroll-composited-expected.checksum:
  • platform/mac/compositing/geometry/outline-change-expected.checksum:
  • platform/mac/compositing/geometry/partial-layout-update-expected.checksum:
  • platform/mac/compositing/geometry/root-layer-update-expected.checksum:
  • platform/mac/compositing/geometry/transfrom-origin-on-zero-size-layer-expected.checksum:
  • platform/mac/compositing/masks/direct-image-mask-expected.checksum:
  • platform/mac/compositing/masks/masked-ancestor-expected.checksum:
  • platform/mac/compositing/masks/multiple-masks-expected.checksum:
  • platform/mac/compositing/masks/simple-composited-mask-expected.checksum:
  • platform/mac/compositing/overflow/ancestor-overflow-expected.checksum:
  • platform/mac/compositing/overflow/fixed-position-ancestor-clip-expected.checksum:
  • platform/mac/compositing/overflow/overflow-positioning-expected.checksum:
  • platform/mac/compositing/overflow/parent-overflow-expected.checksum:
  • platform/mac/compositing/overflow/scrollbar-painting-expected.checksum:
  • platform/mac/compositing/reflections/reflection-in-composited-expected.checksum:
  • platform/mac/compositing/reflections/reflection-ordering-expected.checksum:
  • platform/mac/compositing/reflections/simple-composited-reflections-expected.checksum:
  • platform/mac/compositing/repaint/become-overlay-composited-layer-expected.checksum:
  • platform/mac/compositing/repaint/content-into-overflow-expected.checksum:
  • platform/mac/compositing/repaint/layer-repaint-expected.checksum:
  • platform/mac/compositing/repaint/layer-repaint-rects-expected.checksum:
  • platform/mac/compositing/repaint/opacity-between-absolute-expected.checksum:
  • platform/mac/compositing/repaint/opacity-between-absolute2-expected.checksum:
  • platform/mac/compositing/repaint/overflow-into-content-expected.checksum:
  • platform/mac/compositing/sibling-positioning-expected.checksum:
  • platform/mac/compositing/transitions/singular-scale-transition-expected.checksum:
  • platform/mac/css2.1/t0905-c5525-flthw-00-c-g-expected.checksum:
  • platform/mac/css2.1/t0905-c5526-flthw-00-c-g-expected.checksum:
  • platform/mac/editing/deleting/delete-to-select-table-expected.checksum:
  • platform/mac/editing/deleting/type-delete-after-quote-expected.checksum:
  • platform/mac/editing/inserting/5418891-expected.checksum:
  • platform/mac/editing/selection/4960116-expected.checksum:
  • platform/mac/editing/selection/caret-rtl-2-expected.checksum:
  • platform/mac/editing/selection/extend-by-word-002-expected.checksum:
  • platform/mac/editing/selection/select-all-002-expected.checksum:
  • platform/mac/editing/selection/select-text-overflow-ellipsis-expected.checksum:
  • platform/mac/editing/selection/transformed-selection-rects-expected.checksum:
  • platform/mac/fast/backgrounds/background-clip-text-expected.checksum:
  • platform/mac/fast/backgrounds/quirks-mode-line-box-backgrounds-expected.checksum:
  • platform/mac/fast/backgrounds/repeat/mask-negative-offset-repeat-expected.checksum:
  • platform/mac/fast/backgrounds/size/backgroundSize15-expected.checksum:
  • platform/mac/fast/backgrounds/size/backgroundSize16-expected.checksum:
  • platform/mac/fast/backgrounds/size/contain-and-cover-expected.checksum: Added.
  • platform/mac/fast/backgrounds/svg-as-background-3-expected.checksum:
  • platform/mac/fast/backgrounds/svg-as-background-5-expected.checksum:
  • platform/mac/fast/backgrounds/svg-as-background-6-expected.checksum:
  • platform/mac/fast/backgrounds/svg-as-mask-expected.checksum:
  • platform/mac/fast/block/float/avoidance-percent-width-compat-expected.checksum:
  • platform/mac/fast/block/float/avoidance-percent-width-strict-expected.checksum:
  • platform/mac/fast/block/margin-collapse/103-expected.checksum:
  • platform/mac/fast/box-shadow/basic-shadows-expected.checksum:
  • platform/mac/fast/box-shadow/transform-fringing-expected.checksum:
  • platform/mac/fast/css/apple-prefix-expected.checksum:
  • platform/mac/fast/css/color-correction-backgrounds-and-text-expected.checksum:
  • platform/mac/fast/css/color-correction-expected.checksum:
  • platform/mac/fast/css/color-correction-on-background-image-expected.checksum:
  • platform/mac/fast/css/color-correction-on-backgrounds-expected.checksum:
  • platform/mac/fast/css/color-correction-on-box-shadow-expected.checksum:
  • platform/mac/fast/css/color-correction-on-text-expected.checksum:
  • platform/mac/fast/css/color-correction-on-text-shadow-expected.checksum:
  • platform/mac/fast/css/color-correction-untagged-images-expected.checksum:
  • platform/mac/fast/css/namespaces/namespaces-comments-expected.checksum:
  • platform/mac/fast/css/namespaces/namespaces-empty-expected.checksum:
  • platform/mac/fast/css/namespaces/namespaces-escapes-expected.checksum:
  • platform/mac/fast/css/namespaces/namespaces-invalid-at-expected.checksum:
  • platform/mac/fast/css/variables/misplaced-import-test-expected.checksum:
  • platform/mac/fast/encoding/invalid-UTF-8-expected.checksum:
  • platform/mac/fast/events/standalone-image-drag-to-editable-expected.checksum:
  • platform/mac/fast/fast-mobile-scrolling/fixed-position-element-expected.checksum:
  • platform/mac/fast/fast-mobile-scrolling/no-fixed-position-elements-expected.checksum:
  • platform/mac/fast/forms/001-expected.checksum:
  • platform/mac/fast/forms/menulist-clip-expected.checksum:
  • platform/mac/fast/forms/menulist-narrow-width-expected.checksum:
  • platform/mac/fast/forms/menulist-option-wrap-expected.checksum:
  • platform/mac/fast/forms/menulist-restrict-line-height-expected.checksum:
  • platform/mac/fast/forms/menulist-style-color-expected.checksum:
  • platform/mac/fast/forms/select-baseline-expected.checksum:
  • platform/mac/fast/forms/textarea-scrollbar-expected.checksum:
  • platform/mac/fast/frames/viewsource-attribute-expected.checksum:
  • platform/mac/fast/inline/long-wrapped-line-expected.checksum:
  • platform/mac/fast/lists/anonymous-items-expected.checksum:
  • platform/mac/fast/lists/markers-in-selection-expected.checksum:
  • platform/mac/fast/multicol/client-rects-expected.checksum: Added.
  • platform/mac/fast/multicol/float-multicol-expected.checksum:
  • platform/mac/fast/overflow/line-clamp-expected.checksum:
  • platform/mac/fast/overflow/position-fixed-transform-clipping-expected.checksum:
  • platform/mac/fast/reflections/abs-position-in-reflection-expected.checksum:
  • platform/mac/fast/reflections/opacity-reflection-transform-expected.checksum:
  • platform/mac/fast/repaint/backgroundSizeRepaint-expected.checksum:
  • platform/mac/fast/repaint/box-shadow-h-expected.checksum:
  • platform/mac/fast/repaint/box-shadow-v-expected.checksum:
  • platform/mac/fast/repaint/line-in-scrolled-clipped-block-expected.checksum:
  • platform/mac/fast/repaint/no-caret-repaint-in-non-content-editable-element-expected.checksum:
  • platform/mac/fast/repaint/rel-positioned-inline-with-overflow-expected.checksum:
  • platform/mac/fast/repaint/selection-gap-overflow-scroll-2-expected.checksum:
  • platform/mac/fast/repaint/subtree-layoutstate-transform-expected.checksum:
  • platform/mac/fast/repaint/table-outer-border-expected.checksum:
  • platform/mac/fast/table/fixed-table-with-percent-inside-percent-table-expected.checksum:
  • platform/mac/fast/table/fixed-table-with-percent-width-inside-auto-table-expected.checksum:
  • platform/mac/fast/table/fixed-table-with-percent-width-inside-div-expected.checksum:
  • platform/mac/fast/table/fixed-table-with-percent-width-inside-extra-large-div-expected.checksum:
  • platform/mac/fast/table/fixed-table-with-percent-width-inside-fixed-width-table-expected.checksum:
  • platform/mac/fast/table/fixed-table-with-small-percent-width-expected.checksum:
  • platform/mac/fast/text/atsui-partial-selection-expected.checksum:
  • platform/mac/fast/text/capitalize-boundaries-expected.checksum:
  • platform/mac/fast/text/drawBidiText-expected.checksum:
  • platform/mac/fast/text/line-breaking-with-kerning-expected.checksum:
  • platform/mac/fast/text/sticky-typesetting-features-expected.checksum:
  • platform/mac/fast/transforms/shadows-expected.checksum:
  • platform/mac/http/tests/misc/slow-loading-mask-expected.checksum:
  • platform/mac/media/video-zoom-controls-expected.checksum:
  • platform/mac/plugins/embed-attributes-style-expected.checksum:
  • platform/mac/plugins/qt-qwidget-plugin-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug106158-1-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug106158-2-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug34176-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug6674-expected.checksum:
  • platform/mac/test_expectations.txt:
  • platform/mac/transforms/2d/transform-fixed-container-expected.checksum:
  • platform/mac/transitions/move-after-transition-expected.checksum:
  • platform/mac/transitions/transition-end-event-rendering-expected.checksum:
17:15 Changeset [56648] by dpranke@chromium.org

2010-03-25 Dirk Pranke <dpranke@chromium.org>

Reviewed by Eric Seidel.

Add a test_expectations.txt file for the mac platform so
new-run-webkit-tests can track IMAGE failures.

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

  • platform/mac/test_expectations.txt: Added.

2010-03-26 Dirk Pranke <dpranke@chromium.org>

Reviewed by Eric Seidel.

Change the Mac port of new-run-webkit-tests to look for a
test_expectations.txt file in addition to the Skipped files, so we
can track pixel failures.

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

  • Scripts/webkitpy/layout_tests/port/mac.py:
16:38 Changeset [56647] by dpranke@chromium.org

2010-03-26 Dirk Pranke <dpranke@chromium.org>

Reviewed by Eric Seidel.

Implement pixel tests (image diff) properly on the Mac port.

This change introduces a new "ServerPocess" class that can be used
to manage processes that the run-webkit-tests harness forks off and
expects to stay up for longer than a single request/response session.
Both DumpRenderTree and ImageDiff use this style of communication,
although the current code forks off a new ImageDiff for each diff
(We need to restructure other parts of the code to be able to do this
safely in a multi-threaded environment).

Also, now that the ServerProcess abstraction exists, we can probably
clean up and simplify some of the thread management logic in
test_shell_thread as well.

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

  • Scripts/webkitpy/layout_tests/port/mac.py:
  • Scripts/webkitpy/layout_tests/port/server_process.py:
  • Scripts/webkitpy/layout_tests/test_types/image_diff.py:
15:23 Changeset [56646] by simon.fraser@apple.com

2010-03-26 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein, Darin Adler.

Re-entrant layout via plug-ins may cause crashes with bad RenderWidgets
https://bugs.webkit.org/show_bug.cgi?id=36675
<rdar://problem/7787617>

Fix two places in the code where RenderWidgets can get destroyed while being iterated over.
This can happen when plug-ins execute script from NPP_SetWindow, and that script makes a change
to the page that destroys a RenderWidget.

Tests: plugins/reentrant-update-widget-positions.html

plugins/update-widgets-crash.html

  • page/FrameView.cpp: (WebCore::FrameView::updateWidgets): ref() the RenderEmbeddedObjects that are put into the vector before iterating of them, and deref() them at the end. Rather than checking the m_widgetUpdateSet to see if the RenderWidget is still live, test object->node().
  • rendering/RenderView.cpp: (WebCore::RenderView::RenderView): Initialize some data members to make it more obvious in the debugger that the object is not garbage. (WebCore::RenderView::updateWidgetPositions): Use a Vector of RenderWidget* to keep the RenderWidgets alive during iteration, by reffing and dereffing them.
  • rendering/RenderWidget.h: (WebCore::RenderWidget::ref): Make this and deref() public.
15:16 Changeset [56645] by eric@webkit.org

2010-03-26 Sergio Villar Senin <svillar@igalia.com>

Reviewed by Eric Seidel.

Print didFinishLoadForFrame outcome in DRT

[GTK] Improve reporting of frame loader callbacks in DRT
https://bugs.webkit.org/show_bug.cgi?id=36454

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (webViewLoadCommitted):
15:12 Changeset [56644] by bdakin@apple.com

WebCore: Partial fix for https://bugs.webkit.org/show_bug.cgi?id=36564
Performance regression for setting content of <text> in SVG
-and corresponding-
<rdar://problem/7693963>

Reviewed by Darin Adler.

toClipPath() is extremely inefficient, so this patch removes the
call to it from RenderSVGResourceClipper::resourceBoundingBox()
which is very hot code in SVG with clip-paths. It is sufficient to
call unite with the objectBoundingBox(), which will return early if
objectBoundingBox() is empty.

  • rendering/RenderSVGResourceClipper.cpp:

(WebCore::RenderSVGResourceClipper::resourceBoundingBox):

LayoutTests: Updated test for https://bugs.webkit.org/show_bug.cgi?id=36564
Performance regression for setting content of <text> in SVG
-and corresponding-
<rdar://problem/7693963>

Reviewed by Darin Adler.

  • platform/mac/svg/custom/clip-path-referencing-use2-expected.txt:
14:51 Changeset [56643] by eric@webkit.org

2010-03-26 Eric Seidel <eric@webkit.org>

Unreviewed test fix.
My change conflicted with one of Adam's causing test-webkitpy to fail.

Move commit_message_for_this_commit from scm to checkout
https://bugs.webkit.org/show_bug.cgi?id=36629

  • Scripts/webkitpy/common/checkout/api_unittest.py:
    • modified_changelogs is now on Checkout instead of scm.
14:45 Changeset [56642] by abarth@webkit.org

2010-03-26 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Refactor IRCBot controller logic to allow for commands
https://bugs.webkit.org/show_bug.cgi?id=36676

We need to move the controller logic into the tool package so it can
know about commands. The changes to queueengine could go in a
different patch, but we're going to need it anyway.

  • Scripts/webkitpy/common/config/irc.py: Added.
  • Scripts/webkitpy/common/net/irc/ircbot.py:
  • Scripts/webkitpy/common/net/irc/ircproxy.py:
  • Scripts/webkitpy/tool/bot/queueengine.py:
  • Scripts/webkitpy/tool/bot/sheriffircbot.py: Added.
  • Scripts/webkitpy/tool/commands/sheriffbot.py:
  • Scripts/webkitpy/tool/main.py:
  • Scripts/webkitpy/tool/mocktool.py:
13:44 Changeset [56641] by joepeck@webkit.org

2010-03-26 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by Dave Hyatt.

Generated run-in Content is Mistakenly Getting Deleted
https://bugs.webkit.org/show_bug.cgi?id=36505
<rdar://problem/7767161>

Test: fast/runin/generated2.html
Test: fast/runin/generated3.html
Test: fast/runin/generated4.html

Do not destroy :before/:after generated content with display run-in
when relaying out different nodes. Have their real owners correctly
handle them.

  • rendering/RenderObjectChildList.cpp: (WebCore::RenderObjectChildList::updateBeforeAfterContent): simplified logic (WebCore::RenderObjectChildList::beforeAfterContainer): skip generated run-ins when checking children, check for them in grandchildren

2010-03-26 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by Dave Hyatt.

Generated run-in Content is Mistakenly Getting Deleted
https://bugs.webkit.org/show_bug.cgi?id=36505
<rdar://problem/7767161>

Test dynamic update of the inner block, containing the generated
run-in, but not owning it with the :before rule, and owners.

  • fast/runin/generated2.html: Added.
  • platform/mac/fast/runin/generated2-expected.txt: Added.

Test dynamic update of the outer block, this block owns the
generated run-in, but it is actually contained in a child block.

  • fast/runin/generated3.html: Added.
  • platform/mac/fast/runin/generated3-expected.txt: Added.

Test dynamic updates affecting generated run-ins both
in relation to and not in relation to list markers.

  • fast/runin/generated4.html: Added.
  • platform/mac/fast/runin/generated4-expected.txt: Added.
13:22 Changeset [56640] by abarth@webkit.org

2010-03-26 Adam Barth <abarth@webkit.org>

Rubber stamped by Eric Seidel.

Move the threading code into its own module. (It has nothing to do with IRC.)

  • Scripts/webkitpy/common/net/irc/ircbot.py:
  • Scripts/webkitpy/common/net/irc/ircproxy.py:
  • Scripts/webkitpy/common/net/irc/messagepump.py: Removed.
  • Scripts/webkitpy/common/net/irc/messagepump_unittest.py: Removed.
  • Scripts/webkitpy/common/net/irc/threadedmessagequeue.py: Removed.
  • Scripts/webkitpy/common/net/irc/threadedmessagequeue_unittest.py: Removed.
  • Scripts/webkitpy/common/net/irc/unittests.py:
  • Scripts/webkitpy/common/thread: Added.
  • Scripts/webkitpy/common/thread/init.py: Added.
  • Scripts/webkitpy/common/thread/messagepump.py: Copied from Scripts/webkitpy/common/net/irc/messagepump.py.
  • Scripts/webkitpy/common/thread/messagepump_unittest.py: Copied from Scripts/webkitpy/common/net/irc/messagepump_unittest.py.
  • Scripts/webkitpy/common/thread/threadedmessagequeue.py: Copied from Scripts/webkitpy/common/net/irc/threadedmessagequeue.py.
  • Scripts/webkitpy/common/thread/threadedmessagequeue_unittest.py: Copied from Scripts/webkitpy/common/net/irc/threadedmessagequeue_unittest.py.
  • Scripts/webkitpy/common/thread/unittests.py: Added.
  • Scripts/webkitpy/common/unittests.py:
13:19 Changeset [56639] by ojan@chromium.org

2010-03-25 Ojan Vafai <ojan@chromium.org>

Reviewed by Darin Adler.

shift+home/end and cmd+shift+left/right don't extend the selection correctly
https://bugs.webkit.org/show_bug.cgi?id=36539

  • editing/selection/extend-after-mouse-selection.html:
  • editing/selection/extend-selection-expected.txt: Extending forward by lineboundary selects the whole line. Now extending backwards by lineboundary after the whole line is selected is a noop instead of collapsed the selection to the start.
  • editing/selection/move-begin-end.html: Just cleaned up this test.
  • platform/mac/editing/selection/extend-after-mouse-selection-expected.txt:

2010-03-25 Ojan Vafai <ojan@chromium.org>

Reviewed by Darin Adler.

shift+home/end and cmd+shift+left/right don't extend the selection correctly
https://bugs.webkit.org/show_bug.cgi?id=36539

  • WebCore.base.exp:
  • editing/SelectionController.cpp: (WebCore::SelectionController::modify):
  • editing/SelectionController.h:
12:47 Changeset [56638] by eric@webkit.org

2010-03-26 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Move commit_message_for_this_commit from scm to checkout
https://bugs.webkit.org/show_bug.cgi?id=36629

Finally add some basic unit testing for Checkout.commit_message_for_this_commit

  • Scripts/webkitpy/common/checkout/api.py:
  • Scripts/webkitpy/common/checkout/api_unittest.py: Added.
  • Scripts/webkitpy/common/unittests.py:
12:40 Changeset [56637] by jhawkins@chromium.org

2010-03-24 James Hawkins <jhawkins@chromium.org>

Reviewed by Darin Fisher.

[Chromium] Implement WebFormControlElement and WebSelectElement. Add
a getFormControlElements method to WebFormElement.

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

  • WebKit.gyp:
  • public/WebElement.h:
  • public/WebFormControlElement.h: Added.
  • public/WebFormElement.h:
  • public/WebInputElement.h: (WebKit::WebInputElement::WebInputElement): (WebKit::WebInputElement::operator=): (WebKit::WebInputElement::assign):
  • public/WebSelectElement.h: Added.
  • src/WebElement.cpp: (WebKit::WebElement::isFormControlElement):
  • src/WebFormControlElement.cpp: Added.
  • src/WebFormElement.cpp: (WebKit::WebFormElement::getFormControlElements):
  • src/WebInputElement.cpp: (WebKit::WebInputElement::WebInputElement):
  • src/WebSelectElement.cpp: Added.
12:30 Changeset [56636] by tonikitoo@webkit.org

User Agent Switcher on QtLauncher https://bugs.webkit.org/show_bug.cgi?id=36451

Patch by Diego Gonzalez <diego.gonzalez@openbossa.org> on 2010-03-26
Reviewed by Simon Hausmann.

Make possible to change the QtLauncher user agent via
a predefined list.

  • QtLauncher/QtLauncher.pro:
  • QtLauncher/QtLauncher.qrc: Added.
  • QtLauncher/main.cpp:

(LauncherWindow::showUserAgentDialog):
(LauncherWindow::createChrome):

  • QtLauncher/useragentlist.txt: Added.
  • QtLauncher/webpage.cpp:

(WebPage::userAgentForUrl):

  • QtLauncher/webpage.h:

(WebPage::setUserAgent):

11:21 QtWebKitBackportingFixes edited by laszlo.1.gombos@nokia.com
(diff)
11:14 Changeset [56635] by eric@webkit.org

2010-03-26 Victor Wang <victorw@chromium.org>

Reviewed by Adam Barth.

Host layout test flakiness dashboard in TestResultServer appengine.

Flakiness dashboard is a tool to monitor layout test status and
help layout test regression diagnostics.

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

  • TestResultServer/handlers/dashboardhandler.py: Added.
    • New handler to handle dashboard request
  • TestResultServer/handlers/menu.py:
    • Add new dashboard links
  • TestResultServer/handlers/testfilehandler.py:
    • Request routes refactory
  • TestResultServer/index.yaml:
  • TestResultServer/main.py:
    • Add new dashboard request routes and refactor test result file rountes.
  • TestResultServer/model/dashboardfile.py: Added.
    • Model to access datastore for dashboard files
  • TestResultServer/model/testfile.py:
  • TestResultServer/stylesheets/dashboardfile.css: Added.
  • TestResultServer/templates/dashboardfilelist.html: Added.
  • TestResultServer/templates/showfilelist.html:
11:06 Changeset [56634] by bweinstein@apple.com

Add fast/multicol/hit-test-above-or-below.html, this is failing in the same way as Qt, and is being
tracked by the same bug (https://bugs.webkit.org/show_bug.cgi?id=35973).

  • platform/win/Skipped:
10:38 Changeset [56633] by levin@chromium.org

Rubber-stamped by Nate Chapin.

[chromium] Updated test expectations due to recent failures,
added bugs, etc.

  • platform/chromium/test_expectations.txt:
10:37 FastMalloc Glossary created by ggaren@apple.com
10:32 Changeset [56632] by eric@webkit.org

2010-03-26 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Remove some evil statics from CommitInfo
https://bugs.webkit.org/show_bug.cgi?id=36637

These methods should really be on checkout. You can tell because they
know about ChangeLogs and take an SCM as an argument. :)

  • Scripts/webkitpy/common/checkout/api.py:
  • Scripts/webkitpy/common/checkout/changelog.py:
  • Scripts/webkitpy/common/checkout/commitinfo.py:
  • Scripts/webkitpy/tool/commands/queries.py:
10:32 WikiStart edited by ggaren@apple.com
(diff)
10:29 WikiStart edited by ggaren@apple.com
(diff)
10:04 Changeset [56631] by eric@webkit.org

2010-03-26 Janne Koskinen <janne.p.koskinen@digia.com>

Reviewed by Laszlo Gombos.

Don't undefine SKIP_STATIC_CONSTRUCTORS_ON_GCC for Symbian HW targets.
https://bugs.webkit.org/show_bug.cgi?id=34081

Defining StringImpl instances as globals will cause a crash on process exit as
StringImpl::Remove expects TLS which was already deleted at time of exiting main and ends up
constructing one exiting thread.

  • config.h:
09:34 Changeset [56630] by eric@webkit.org

2010-03-26 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Pavel Feldman.

Send worker resource content to inspector to enable display of web
workers in inspector's resource tab.
https://bugs.webkit.org/show_bug.cgi?id=36658

  • inspector/InspectorController.cpp: (WebCore::InspectorController::resourceRetrievedByXMLHttpRequest): (WebCore::InspectorController::scriptImported):
  • inspector/InspectorResource.cpp: (WebCore::InspectorResource::type): (WebCore::InspectorResource::setOverrideContent): (WebCore::InspectorResource::sourceString):
  • inspector/InspectorResource.h:
  • workers/DefaultSharedWorkerRepository.cpp: (WebCore::SharedWorkerScriptLoader::notifyFinished):
  • workers/Worker.cpp: (WebCore::Worker::notifyFinished):

2010-03-26 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Pavel Feldman.

Send worker resource content to inspector to enable display of web
workers in inspector's resource tab.
https://bugs.webkit.org/show_bug.cgi?id=36658

  • src/SharedWorkerRepository.cpp: (WebCore::SharedWorkerScriptLoader::notifyFinished):
08:42 Changeset [56629] by yael.aharon@nokia.com

WebCore: Fix the rendering of HTMLProgressElement
https://bugs.webkit.org/show_bug.cgi?id=36206

Reviewed by Antti Koivisto.

Update the rendering and css to conform to
http://www.whatwg.org/specs/web-apps/current-work/complete.html#the-progress-element-0.
Updated the tests and the expected results accordingly.

  • css/html.css:
  • rendering/RenderProgress.cpp:
  • rendering/RenderProgress.h:

LayoutTests: Fix the rendering of HTMLProgressElement
https://bugs.webkit.org/show_bug.cgi?id=36206
Add dir=rtl test and update the expected results.

Reviewed by Antti Koivisto.

  • fast/dom/HTMLProgressElement/progress-element.html:
  • platform/qt/fast/dom/HTMLProgressElement/progress-element-expected.checksum:
  • platform/qt/fast/dom/HTMLProgressElement/progress-element-expected.png:
  • platform/qt/fast/dom/HTMLProgressElement/progress-element-expected.txt:
08:30 Changeset [56628] by jchaffraix@webkit.org

Unreviewed build fix.

FontPlatformDataPango.cpp: prototype for
'WebCore::FontPlatformData::FontPlatformData(cairo_font_face_t*, int, bool, bool)'
does not match any in class
https://bugs.webkit.org/show_bug.cgi?id=36496

  • platform/graphics/gtk/FontPlatformDataPango.cpp:

(WebCore::FontPlatformData::FontPlatformData): The 'size' parameter
should be a float to match the declaration and the other definitions.

08:22 Changeset [56627] by eric@webkit.org

2010-03-26 Shu Chang <chang.shu@nokia.com>

Reviewed by Eric Seidel.

[Qt] Prevent referring d->m_job in the future because calling abort()
deletes the instance itself.
https://bugs.webkit.org/show_bug.cgi?id=36618

Test: http/tests/appcache/fallback.html

  • platform/network/qt/ResourceHandleQt.cpp: (WebCore::ResourceHandle::cancel):
07:49 QtWebKit edited by zoltan@webkit.org
(diff)
07:37 Changeset [56626] by hausmann@webkit.org

[Qt] Compile with QT_USE_FAST_OPERATOR_PLUS

Patch by Olivier Goffart <ogoffart@trolltech.com> on 2010-03-26
Reviewed by Simon Hausmann.

  • WebCoreSupport/InspectorClientQt.cpp:

(WebCore::InspectorClientQt::populateSetting):
(WebCore::InspectorClientQt::storeSetting):

07:35 Changeset [56625] by hausmann@webkit.org

[Qt] Doc: Simplified Commercial Editions for Qt 4.7.

Patch by David Boddie <dboddie@trolltech.com> on 2010-03-26
Reviewed by Simon Hausmann.

  • docs/qtwebkit.qdoc:
07:26 Changeset [56624] by senorblanco@chromium.org

2010-03-26 Stephen White <senorblanco@chromium.org>

Unreviewed; build fix.

Rolling out 56620; too many layout test failures.

Covered by fast/css/*.

  • css/CSSPrimitiveValue.cpp: (WebCore::CSSPrimitiveValue::cleanup): (WebCore::CSSPrimitiveValue::cssText):
  • css/CSSPrimitiveValue.h: (WebCore::CSSPrimitiveValue::):

2010-03-26 Stephen White <senorblanco@chromium.org>

Unreviewed; build fix.

Rolling out 56620; too many layout test failures.

  • fast/css/cssText-cache-expected.txt: Removed.
  • fast/css/cssText-cache.html: Removed.
07:05 Changeset [56623] by hausmann@webkit.org

[Qt] Build JavaScriptCore as a static library.
https://bugs.webkit.org/show_bug.cgi?id=36590

Patch by Jocelyn Turcotte <jocelyn.turcotte@nokia.com> on 2010-03-26
Reviewed by Simon Hausmann.

This patch takes what was left of the unused JavaScriptCore.pro
and moved the compilation logic from JavaScriptCore.pri to
JavaScriptCore.pro.

.:

  • WebKit.pro:

JavaScriptCore:

WebCore:

  • WebCore.pro:
06:44 Changeset [56622] by hausmann@webkit.org

[Qt] Fix compilation on Windows

Patch by Olivier Goffart <ogoffart@trolltech.com> on 2010-03-26
Reviewed by Simon Hausmann.

For some reason, the MSVC compiler choose the
operator+(const QString &, const QString &) instead of
operator+(const WebCore::String &, const WebCore::String &)
resulting in errors when QT_USE_FAST_OPERATOR_PLUS is used.

  • dom/XMLTokenizerQt.cpp:

(WebCore::handleElementNamespaces):

06:28 Changeset [56621] by philn@webkit.org

2010-03-26 Philippe Normand <pnormand@igalia.com>

fast/parser/xml-declaration-missing-ending-mark.html crashed on Gtk bot
https://bugs.webkit.org/show_bug.cgi?id=33027

Skipping test that fails on GTK+ Debug bots.

  • platform/gtk/Skipped:
06:18 Changeset [56620] by senorblanco@chromium.org

2010-03-24 Stephen White <senorblanco@chromium.org>

Reviewed by Darin Adler.

In order to speed up multiple calls to CSSPrimitiveValue::cssText(),
this CL caches the String result. When m_value is changed, the
cached string is cleared. This gives a good speedup on benchmarks
which do a lot of CSS property gets, such as Peacekeeper.
The processing cost should be negligible, since the strings are
refcounted. The memory cost is an additional 4 bytes per
CSSPrimitiveValue, and the extended lifetime of the computed string
(potentially, the same as the lifetime of the CSSPrimitiveValue).

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

Covered by fast/css/large-number-round-trip.html, and more.

  • css/CSSPrimitiveValue.cpp: (WebCore::CSSPrimitiveValue::CSSPrimitiveValue): (WebCore::CSSPrimitiveValue::cleanup): (WebCore::CSSPrimitiveValue::cssText):
  • css/CSSPrimitiveValue.h:

2010-03-25 Stephen White <senorblanco@chromium.org>

Reviewed by Darin Adler.

Added a test for CSSPrimitiveValue::cssText() string cacheing.

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

  • fast/css/cssText-cache-expected.txt: Added.
  • fast/css/cssText-cache.html: Added.
05:58 Changeset [56619] by eric@webkit.org

2010-03-26 Qi Zhang <qi.2.zhang@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] fast/canvas/drawImage-with-negative-source-destination.html failed
https://bugs.webkit.org/show_bug.cgi?id=35005

Enable fast/canvas/drawImage-with-negative-source-destination.html

  • platform/qt/Skipped:

2010-03-26 Qi Zhang <qi.2.zhang@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] fast/canvas/drawImage-with-negative-source-destination.html failed
https://bugs.webkit.org/show_bug.cgi?id=35005

To support negative width or height at context.drawImage

  • platform/graphics/qt/StillImageQt.cpp: (WebCore::StillImage::draw):
05:52 Changeset [56618] by tonikitoo@webkit.org

Reviewed by Simon Hausmann.

Add an option to QtLauncher to enable/disable a QGLWidget as Viewport
of the QGraphicsView when the launcher is running on graphicsview mode.

[Qt] QtLauncher needs an option to Enable/Disable a QGLWidget as Viewport
https://bugs.webkit.org/show_bug.cgi?id=36270

  • QtLauncher/QtLauncher.pro:
  • QtLauncher/main.cpp:

(LauncherWindow::toggleQGLWidgetViewport):
(LauncherWindow::createChrome):

04:11 Changeset [56617] by ager@chromium.org

2010-03-26 Mads Ager <ager@chromium.org>

Reviewed by Jeremy Orlow.

Tweak Chromium test expectations
https://bugs.webkit.org/show_bug.cgi?id=36651

fast/js/large-expressions.html started failing on Linux in release
mode occassionally. Also fails on other platforms.

  • platform/chromium/test_expectations.txt:
02:21 Changeset [56616] by ukai@chromium.org

Unreviewed, removing websocket/tests/long-invalid-header.html from Chromium test_expectations.txt

  • platform/chromium/test_expectations.txt: remove websocket/tests/long-invalid-header.html
01:58 Changeset [56615] by philn@webkit.org

2010-03-08 Philippe Normand <pnormand@igalia.com>

Reviewed by Gustavo Noronha.

[GStreamer] soften dependency on libsoup in the http src element
https://bugs.webkit.org/show_bug.cgi?id=35864

Replaced SoupURI calls with KURL and pause/resume internal soup
messages only if the element is compiled for a port depending on
libsoup.

  • platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp: (webkit_web_src_init): (webKitWebSrcSetUri): (webKitWebSrcNeedDataMainCb): (webKitWebSrcEnoughDataMainCb):
01:45 Changeset [56614] by ukai@chromium.org

2010-03-26 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by Shinichiro Hamaji.

[Chromium-Win] websocket_server.py failed to start
https://bugs.webkit.org/show_bug.cgi?id=36586

subproess.Popen() on Python 2.4/Windows with stdout,stdout, but no stdin will fail, because it uses return value of GetStdHandle(STD_INPUT_HANDLE), but DuplicateHandle requires integer, not the handle.
We don't need stdin, so use devnull as stdin.

  • Scripts/webkitpy/layout_tests/port/websocket_server.py: open os.devnull for stdin
01:05 Changeset [56613] by pfeldman@chromium.org

2010-03-25 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Shot record filter is implemented in Timeline Panel.
https://bugs.webkit.org/show_bug.cgi?id=36606

  • English.lproj/localizedStrings.js:
  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel): (WebInspector.TimelinePanel.prototype.get statusBarItems): (WebInspector.TimelinePanel.prototype._createStatusbarButtons): (WebInspector.TimelinePanel.prototype._toggleFilterButtonClicked): (WebInspector.TimelinePanel.prototype._innerAddRecordToTimeline): (WebInspector.TimelinePanel.prototype._addToRecordsWindow): (WebInspector.TimelineRecordGraphRow.prototype.update): (WebInspector.TimelinePanel.FormattedRecord.prototype.get _isLongEvent):
  • inspector/front-end/inspector.css:
00:58 Changeset [56612] by tkent@chromium.org

2010-03-26 Kent Tamura <tkent@chromium.org>

Reviewed by Dimitri Glazkov.

[DRT/Chromium] Add TestNavigationController and TestWebWorker
https://bugs.webkit.org/show_bug.cgi?id=36520

Add LayoutTestController class, which is going to be used by
DumpRenderTree Chromium port. These files are based on:

  • src/webkit/tools/test_shell/layout_test_controller.cc
  • src/webkit/tools/test_shell/layout_test_controller.h of Chromium rev.40492.
  • DumpRenderTree/chromium/LayoutTestController.cpp: Added.
  • DumpRenderTree/chromium/LayoutTestController.h: Added.
00:55 Changeset [56611] by hausmann@webkit.org

Symbian build fix.

[Qt] Updated the def file with new exports used by QtLauncher.

  • symbian/eabi/QtWebKitu.def:
00:43 Changeset [56610] by tkent@chromium.org

Rollout r56609. It was accidental commit.

00:34 Changeset [56609] by tkent@chromium.org

2009-10-20 Mikhail Naganov <mnaganov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: populate child nodes before sorting them.

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

  • inspector/front-end/BottomUpProfileDataGridTree.js: (WebInspector.BottomUpProfileDataGridNode): Swapped with BottomUpProfileDataGridTree to be consistent with TopDownProfileDataGridNode. (WebInspector.BottomUpProfileDataGridNode.prototype._takePropertiesFromProfileDataGridNode): (WebInspector.BottomUpProfileDataGridNode.prototype._keepOnlyChild): (WebInspector.BottomUpProfileDataGridNode.prototype._exclude): (WebInspector.BottomUpProfileDataGridNode.prototype._merge): (WebInspector.BottomUpProfileDataGridNode.prototype._sharedPopulate): (WebInspector.BottomUpProfileDataGridTree.prototype.exclude):
  • inspector/front-end/ProfileDataGridTree.js: (WebInspector.ProfileDataGridNode.prototype.sort): Added missing parentheses. (WebInspector.ProfileDataGridNode.prototype.get _parent): (WebInspector.ProfileDataGridNode.prototype._populate):
  • inspector/front-end/TopDownProfileDataGridTree.js: (WebInspector.TopDownProfileDataGridNode.prototype._sharedPopulate):
00:13 Changeset [56608] by pfeldman@chromium.org

2010-03-25 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Implement CSSStyleSelector::pseudoStyleForElement and use it in the InspectorDOMAgent.

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

This change provides pseudo elements information into the inspector styles sidebar pane.
Changes applied:

  • CSSStyleSelector::pseudoStyleRulesForElement implemented. I changed the signature to accept PseudoId instead of a String since it better reflected the needs;
  • Accompanied elementStyle checks with m_collectRulesOnly checks;
  • Removed pseudoStyleRulesForElement usage from DOMWindow::getMatchedCSSRules. It was anyways returning 0, while I would need to convert String to PseudoId in order to leave the call in place;
  • Needed to introduce AFTER_LAST_INTERNAL_PSEUDOID marker const in RenderStyleConstants' PseudoId enum
  • InspectorDOMAgent is now iterating over all pseudo ids to get all styles (including internal ones).
  • Brushed up front-end code so that proper overriding took place locally within pseudo id categories.
  • English.lproj/localizedStrings.js:
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::styleRulesForElement): (WebCore::CSSStyleSelector::pseudoStyleRulesForElement): (WebCore::CSSStyleSelector::SelectorChecker::checkSelector): (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector):
  • css/CSSStyleSelector.h:
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::getStyles): (WebCore::InspectorDOMAgent::buildObjectForAttributeStyles): (WebCore::InspectorDOMAgent::buildArrayForCSSRules): (WebCore::InspectorDOMAgent::buildArrayForPseudoElements):
  • inspector/InspectorDOMAgent.h:
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylesSidebarPane.prototype.update.callback): (WebInspector.StylesSidebarPane.prototype.update): (WebInspector.StylesSidebarPane.prototype._update): (WebInspector.StylesSidebarPane.prototype._refreshStyleRules): (WebInspector.StylesSidebarPane.prototype._rebuildStyleRules.insertInheritedNodeSeparator): (WebInspector.StylesSidebarPane.prototype._rebuildStyleRules): (WebInspector.StylesSidebarPane.prototype._markUsedProperties): (WebInspector.StylesSidebarPane.prototype._refreshSectionsForStyleRules): (WebInspector.StylesSidebarPane.prototype._rebuildSectionsForStyleRules): (WebInspector.StylesSidebarPane.prototype._changeColorFormat): (WebInspector.StylesSidebarPane.prototype.addBlankSection): (WebInspector.StylesSidebarPane.prototype.removeSection):
  • inspector/front-end/inspector.css: (.styles-sidebar-separator):
  • page/DOMWindow.cpp: (WebCore::DOMWindow::getMatchedCSSRules):
  • rendering/style/RenderStyleConstants.h: (WebCore::):
00:11 Changeset [56607] by mrowe@apple.com

Update WebCoreSQLite3 to SQLite v3.6.12.

Rubber-stamped by Dan Bernstein.

  • WebCoreSQLite3/sqlite3.h:
  • WebCoreSQLite3/sqlite3ext.h:
  • libWebCoreSQLite3.a:
  • libWebKitSystemInterfaceLeopard.a:
  • libWebKitSystemInterfaceSnowLeopard.a:
  • libWebKitSystemInterfaceTiger.a:

03/25/10:

23:34 Changeset [56606] by abarth@webkit.org

2010-03-25 Charlie Reis <creis@chromium.org>

Reviewed by Adam Barth.

onbeforeunload not called at window close + frame or iframe focused
https://bugs.webkit.org/show_bug.cgi?id=27481

Adds a layout test for bug 27481. Requires calling shouldClose on
the WebView, which is done with a new method on LayoutTestController.

  • fast/events/onbeforeunload-focused-iframe-expected.txt: Added.
  • fast/events/onbeforeunload-focused-iframe.html: Added.
  • fast/events/resources/onbeforeunload-focused-iframe-frame.html: Added.
  • platform/gtk/Skipped:
  • platform/qt/Skipped:

2010-03-25 Charlie Reis <creis@chromium.org>

Reviewed by Adam Barth.

onbeforeunload not called at window close + frame or iframe focused
https://bugs.webkit.org/show_bug.cgi?id=27481

Test: fast/events/onbeforeunload-focused-iframe.html

Replaces initial manual tests with a layout test.

  • manual-tests/onbeforeunload-focused-iframe.html: Removed.
  • manual-tests/resources/focused-iframe.html: Removed.

2010-03-25 Charlie Reis <creis@chromium.org>

Reviewed by Adam Barth.

onbeforeunload not called at window close + frame or iframe focused
https://bugs.webkit.org/show_bug.cgi?id=27481

Adds a callShouldCloseOnWebView method to LayoutTestController,
to allow automated testing for bug 27481.

  • DumpRenderTree/LayoutTestController.cpp: (callShouldCloseOnWebViewCallback): (LayoutTestController::staticFunctions):
  • DumpRenderTree/LayoutTestController.h:
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::callShouldCloseOnWebView):
  • DumpRenderTree/mac/LayoutTestControllerMac.mm: (LayoutTestController::callShouldCloseOnWebView):
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::callShouldCloseOnWebView):
  • DumpRenderTree/qt/LayoutTestControllerQt.h:
  • DumpRenderTree/win/LayoutTestControllerWin.cpp: (LayoutTestController::callShouldCloseOnWebView):
23:23 Changeset [56605] by eric@webkit.org

2010-03-25 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

REGRESSION: plugins/netscape-construct.html and plugins/mouse-events-fixedpos.html failing on Snow Leopard Bot
https://bugs.webkit.org/show_bug.cgi?id=36462

Adding tests to the Skipped list per Alexey's recommendation
until someone from Apple can update the version of Safari
on the Snow Leopard bot.

  • platform/mac-snowleopard/Skipped:
23:06 Changeset [56604] by eric@webkit.org

2010-03-25 Eric Uhrhane <ericu@chromium.org>

Reviewed by David Levin.

Missing lock in call to doneCreatingDatabase
https://bugs.webkit.org/show_bug.cgi?id=36473

Added new test storage/open-database-over-quota.html.

  • storage/open-database-over-quota.html: Added.
  • storage/open-database-over-quota-expected.txt: Added.

2010-03-25 Eric Uhrhane <ericu@chromium.org>

Reviewed by David Levin.

Missing lock in call to doneCreatingDatabase
https://bugs.webkit.org/show_bug.cgi?id=36473

Added new test storage/open-database-over-quota.html.

  • storage/DatabaseTracker.cpp: (WebCore::DatabaseTracker::canEstablishDatabase): Added missing lock call.
22:48 Changeset [56603] by eric@webkit.org

2010-03-25 Tony Chang <tony@chromium.org>

Reviewed by David Levin.

[chromium] correctly handle move drag operations
https://bugs.webkit.org/show_bug.cgi?id=36484

  • platform/chromium/test_expectations.txt:

2010-03-25 Tony Chang <tony@chromium.org>

Reviewed by David Levin.

[chromium] correctly handle move drag operations
https://bugs.webkit.org/show_bug.cgi?id=36484

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::dragTargetDragEnter): (WebKit::WebViewImpl::dragTargetDragOver): (WebKit::WebViewImpl::dragTargetDragEnterOrOver): Combine common code into a helper method

and properly mask against the drag effect.

(WebKit::WebViewImpl::createUniqueIdentifierForRequest):

  • src/WebViewImpl.h:
22:42 Changeset [56602] by abarth@webkit.org

2010-03-25 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Add webkit-patch lkgr for finding last known good revision
https://bugs.webkit.org/show_bug.cgi?id=36626

This is rather slow for now because the command
has to compute this information from the buildbot.
A better long-term solution would be to have a server
somewhere store a pre-computed LKGR and then any
script (like webkit-patch) could just fetch it.

  • Scripts/webkitpy/common/net/buildbot.py:
  • Scripts/webkitpy/tool/commands/queries.py:
22:38 Changeset [56601] by abarth@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Move modified_changelogs (and friends) from scm to checkout
https://bugs.webkit.org/show_bug.cgi?id=36636

These functions know about ChangeLogs, which is forbidden knowledge in
scm.py.

  • Scripts/webkitpy/common/checkout/api.py:
  • Scripts/webkitpy/common/checkout/changelog.py:
  • Scripts/webkitpy/common/checkout/scm.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/steps/abstractstep.py:
  • Scripts/webkitpy/tool/steps/preparechangelogforrevert.py:
  • Scripts/webkitpy/tool/steps/revertrevision.py:
  • Scripts/webkitpy/tool/steps/updatechangelogswithreviewer.py:
  • Scripts/webkitpy/tool/steps/validatereviewer.py:
22:32 Changeset [56600] by abarth@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Move apply_patch from scm to checkout
https://bugs.webkit.org/show_bug.cgi?id=36635

SCM shouldn't have any knowledge of WebKit scripts.

  • Scripts/webkitpy/common/checkout/api.py:
  • Scripts/webkitpy/common/checkout/scm.py:
  • Scripts/webkitpy/common/checkout/scm_unittest.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/steps/applypatch.py:
21:48 Changeset [56599] by eric@webkit.org

2010-03-25 Eric Seidel <eric@webkit.org>

Unreviewed build fix to un-break webkit-patch land.
Test-case coming in follow-up commit.

Move commit_message_for_this_commit from scm to checkout
https://bugs.webkit.org/show_bug.cgi?id=36629

  • Scripts/webkitpy/common/checkout/api.py: import scm.CommitMessage
21:40 Changeset [56598] by ojan@chromium.org

WebCore:
2010-03-25 Ojan Vafai <ojan@chromium.org>

Reviewed by Eric Seidel.

[chromium] REGRESSION: Some LayoutTests fail after r56567
https://bugs.webkit.org/show_bug.cgi?id=36630

Fix boolean logic goofup. This was a last minute code review
change that didn't get tested on Windows before commit.

  • editing/SelectionController.cpp: (WebCore::SelectionController::setIsDirectional):

LayoutTests:
2010-03-25 Ojan Vafai <ojan@chromium.org>

Reviewed by Eric Seidel.

[chromium] REGRESSION: Some LayoutTests fail after r56567
https://bugs.webkit.org/show_bug.cgi?id=36630

  • platform/chromium/test_expectations.txt:
21:35 Changeset [56597] by mitz@apple.com

<rdar://problem/7728903> Support color bitmap fonts

Reviewed by Darin Adler.

WebCore:

Test: platform/mac/fonts/color-bitmap.html

  • platform/graphics/mac/FontMac.mm:

(WebCore::showGlyphsWithAdvances): Added. Calls CGContextShowGlyphsWithAdvances()
or CTFontShowGlyphs() depending on whether the font is a color bitmap font.
(WebCore::Font::drawGlyphs): Use showGlyphsWithAdvances(). Don’t include the font’s matrix in
the context’s text matrix when drawing with a color bitmap font, because CTFontShowGlyphs()
factors it in.

  • platform/graphics/mac/FontPlatformData.h:

(WebCore::FontPlatformData::FontPlatformData): Added m_isColorBitmapFont member
and initialized it in constructors.
(WebCore::FontPlatformData::isColorBitmapFont): Added this getter.

  • platform/graphics/mac/FontPlatformDataMac.mm:

(WebCore::FontPlatformData::FontPlatformData): Initialize m_isColorBitmapFont.
(WebCore::FontPlatformData::operator=): Copy m_isColorBitmapFont.
(WebCore::FontPlatformData::setFont): Set m_isColorBitmapFont.

WebKitTools:

  • DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj: Added ColorBits.ttf

and ColorBits-A.png.

  • DumpRenderTree/fonts/ColorBits-A.png: Copied from WebCore/inspector/front-end/Images/successGreenDot.png.
  • DumpRenderTree/fonts/ColorBits.ttf: Added.
  • DumpRenderTree/mac/DumpRenderTree.mm:

(activateFonts): Activate ColorBits.ttf.

LayoutTests:

  • platform/mac-leopard/Skipped:
  • platform/mac-snowleopard/Skipped:
  • platform/mac-tiger/Skipped:
  • platform/mac/fonts/color-bitmap-expected.checksum: Added.
  • platform/mac/fonts/color-bitmap-expected.png: Added.
  • platform/mac/fonts/color-bitmap-expected.txt: Added.
  • platform/mac/fonts/color-bitmap.html: Added.
21:28 Changeset [56596] by dimich@chromium.org

Unreviewed, another attempt to unblock Chromium Win test bot.

  • platform/chromium/test_expectations.txt:
21:23 Changeset [56595] by dimich@chromium.org

Unreviewed, removing stale expectation file for Chromium.

The test was updated by http://trac.webkit.org/changeset/56506 to a script test.

  • platform/chromium-win/editing/inserting/6633727-expected.txt: Removed.
21:13 Changeset [56594] by dimich@chromium.org

Unreviewed, temporarily disabling test on Chromium bots.

http://crbug.com/39423

  • platform/chromium/test_expectations.txt:
20:04 Changeset [56593] by mrowe@apple.com

Remove a printf that was causing commit-log-editor to spew the name of the editor
to the terminal many times during a commit.

  • Scripts/commit-log-editor:
20:04 Changeset [56592] by eric@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Move commit_message_for_this_commit from scm to checkout
https://bugs.webkit.org/show_bug.cgi?id=36629

This function requires knowledge of ChangeLogs, but scm shouldn't know
about ChangeLogs.

  • Scripts/webkitpy/common/checkout/api.py:
  • Scripts/webkitpy/common/checkout/scm.py:
  • Scripts/webkitpy/tool/commands/upload.py:
  • Scripts/webkitpy/tool/commands/upload_unittest.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/steps/applypatchwithlocalcommit.py:
  • Scripts/webkitpy/tool/steps/commit.py:
19:46 Changeset [56591] by eric@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Reviewed by Sam Weinig.

Implement allow-top-navigation for HTML5 sandbox attribute
https://bugs.webkit.org/show_bug.cgi?id=36549

Test various cases relating to navigating top level frames and the
allow-top-navigation directive. This feature is tricky to test
exhaustively because there are a lot of permutations.

  • fast/frames/resources/fail-and-notify-done.html: Added.
  • fast/frames/resources/navigate-top-by-constant-name-to-pass.html: Added.
  • fast/frames/resources/navigate-top-by-constant-name2-to-pass.html: Added.
  • fast/frames/resources/navigate-top-by-name-to-fail.html: Added.
  • fast/frames/resources/navigate-top-by-name-to-pass.html: Added.
  • fast/frames/resources/navigate-top-to-fail.html: Added.
  • fast/frames/resources/navigate-top-to-pass.html: Added.
  • fast/frames/resources/pass-and-notify-done.html: Added.
  • fast/frames/sandboxed-iframe-navigation-top-by-constant-name-expected.txt: Added.
  • fast/frames/sandboxed-iframe-navigation-top-by-constant-name.html: Added.
  • fast/frames/sandboxed-iframe-navigation-top-by-constant-name2-expected.txt: Added.
  • fast/frames/sandboxed-iframe-navigation-top-by-constant-name2.html: Added.
  • fast/frames/sandboxed-iframe-navigation-top-by-name-denied-expected.txt: Added.
  • fast/frames/sandboxed-iframe-navigation-top-by-name-denied.html: Added.
  • fast/frames/sandboxed-iframe-navigation-top-by-name-expected.txt: Added.
  • fast/frames/sandboxed-iframe-navigation-top-by-name.html: Added.
  • fast/frames/sandboxed-iframe-navigation-top-denied-expected.txt: Added.
  • fast/frames/sandboxed-iframe-navigation-top-denied.html: Added.
  • fast/frames/sandboxed-iframe-navigation-top-expected.txt: Added.
  • fast/frames/sandboxed-iframe-navigation-top.html: Added.

2010-03-25 Adam Barth <abarth@webkit.org>

Reviewed by Sam Weinig.

Implement allow-top-navigation for HTML5 sandbox attribute
https://bugs.webkit.org/show_bug.cgi?id=36549

Tests: fast/frames/sandboxed-iframe-navigation-top-by-constant-name.html

fast/frames/sandboxed-iframe-navigation-top-by-constant-name2.html
fast/frames/sandboxed-iframe-navigation-top-by-name-denied.html
fast/frames/sandboxed-iframe-navigation-top-by-name.html
fast/frames/sandboxed-iframe-navigation-top-denied.html
fast/frames/sandboxed-iframe-navigation-top.html

  • bindings/generic/BindingDOMWindow.h: (WebCore::::createWindow):
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::createWindow): Moved the security check to FrameLoader because we need to allow window.open navigation to succeed sometimes when the SandboxNavigation bit is set.
  • html/HTMLIFrameElement.cpp: (WebCore::parseSandboxAttribute): Added allow-top-navigation to the parser.
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::createWindow): The SandboxNavigation check now occurs after we search for the frame by name. Notice that the named frame navigation case is caught by the generic navigation access check. (WebCore::FrameLoader::shouldAllowNavigation): Update the navigation access check to allow navigating of top unless it's forbidden.
  • loader/FrameLoaderTypes.h: Introduce a bit for SandboxedTopNavigation. (WebCore::):
19:32 Changeset [56590] by ap@apple.com

Reviewed by Mark Rowe.

  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WKPCRunSyncOpenPanel): Re-fetch host proxy in a way that works in a function that doesn't have a pluginID argument for some reason.
19:21 Changeset [56589] by eric@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

REGRESSION: webkit-patch land can't land "rubber-stamped" patches
https://bugs.webkit.org/show_bug.cgi?id=36582

Allow a "-" in rubber stamped.

  • Scripts/webkitpy/tool/steps/validatereviewer.py:
19:12 Changeset [56588] by dumi@chromium.org

Unreviewed, Chromium build fix.

WebCore:

V8CustomVoidCallback should not check if a return value is a
boolean. It should only convert the value to a boolean.

  • bindings/v8/custom/V8CustomVoidCallback.cpp:

(WebCore::invokeCallback):

LayoutTests:

Update the expectations for statement-error-callback.html for Chromium.

  • platform/chromium-mac/storage/statement-error-callback-expected.txt:
  • platform/chromium-win/storage/statement-error-callback-expected.txt:
18:48 Changeset [56587] by dimich@chromium.org

Revert "2010-03-25 Laszlo Gombos <laszlo.1.gombos@nokia.com>"

This reverts commit 2a3b308efc33f4829fe9717d93a4b245d3b25f6d.

18:26 Changeset [56586] by abarth@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Unreviewed. Fix some copyright lines to remove extraneous comma and
python directive.

  • Scripts/webkitpy/tool/bot/patchcollection.py:
  • Scripts/webkitpy/tool/bot/patchcollection_unittest.py:
  • Scripts/webkitpy/tool/bot/queueengine.py:
  • Scripts/webkitpy/tool/bot/queueengine_unittest.py:
  • Scripts/webkitpy/tool/commands/download.py:
  • Scripts/webkitpy/tool/commands/earlywarningsystem.py:
  • Scripts/webkitpy/tool/commands/queries.py:
  • Scripts/webkitpy/tool/commands/queues.py:
  • Scripts/webkitpy/tool/commands/sheriffbot.py:
  • Scripts/webkitpy/tool/comments.py:
  • Scripts/webkitpy/tool/grammar.py:
  • Scripts/webkitpy/tool/multicommandtool.py:
  • Scripts/webkitpy/tool/multicommandtool_unittest.py:
18:04 Changeset [56585] by eric@webkit.org

2010-03-25 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Darin Adler.

Fix build if NPAPI support is disabled
https://bugs.webkit.org/show_bug.cgi?id=36621

No new tests, this is a build fix.

  • plugins/PluginView.cpp: Guard getValueStatic() with NETSCAPE_PLUGIN_API (WebCore::PluginView::getValue):
  • plugins/PluginView.h: Guard getValue() with NETSCAPE_PLUGIN_API
  • plugins/PluginViewNone.cpp: Guard platformGetValue() and platformGetValueStatic with NETSCAPE_PLUGIN_API; Guard privateBrowsingStateChanged() and setJavaScriptPaused() with PLATFORM(MAC)
17:53 Changeset [56584] by dimich@chromium.org

Not reviewed, update pixel results for Chromium bots.

The rendering of these tests was changed by http://trac.webkit.org/changeset/56507.
Also updated Chromium's test_expectations.txt.

  • platform/chromium-linux/fast/dynamic/flash-replacement-test-expected.checksum:
  • platform/chromium-linux/fast/dynamic/flash-replacement-test-expected.png:
  • platform/chromium-linux/plugins/embed-attributes-style-expected.checksum:
  • platform/chromium-linux/plugins/embed-attributes-style-expected.png:
  • platform/chromium/test_expectations.txt: Use BUG_WKXXXX instead of WKXXXX notation to appease validation on the bot.
17:46 Changeset [56583] by simon.fraser@apple.com

2010-03-25 Simon Fraser <simon.fraser@apple.com>

Reviewed by Darin Adler, Alexey Proskuryakov.

https://bugs.webkit.org/show_bug.cgi?id=36631
Allow the test plugin to run scripts in response to NPP_SetWindow calls

Hook up the ability for the TestNetscapePlugIn to run JavaScript in
response to NPP_SetWindow.

  • DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp: (pluginAllocate):
  • DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.h:
  • DumpRenderTree/TestNetscapePlugIn.subproj/main.cpp: (NPP_New): (NPP_Destroy): (NPP_SetWindow):
17:26 Changeset [56582] by dimich@chromium.org

Unreviewed, fixing Chromium test bots.

[chromium] REGRESSION: Some LayoutTests fail after r56567
https://bugs.webkit.org/show_bug.cgi?id=36630

  • platform/chromium/test_expectations.txt: Update expectations.
17:24 Changeset [56581] by eric@webkit.org

2010-03-25 Yael Aharon <yael.aharon@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] QtLauncher crashes on Mac OS and Linux when exiting with QGraphicsView mode enabled
https://bugs.webkit.org/show_bug.cgi?id=35251

Followed the way QWebView registers for the signal QWebPage::destroyed(), to prevent
QGraphicsWebView from referencing QWebPage after it was deleted.

  • Api/qgraphicswebview.cpp: (QGraphicsWebViewPrivate::_q_pageDestroyed): (QGraphicsWebView::setPage):
  • Api/qgraphicswebview.h:
17:06 Changeset [56580] by eric@webkit.org

2010-03-25 Drew Wilson <atwilson@chromium.org>

Reviewed by Nate Chapin.

[v8] Error in getScriptExecutionContext() when worker context is terminating.
https://bugs.webkit.org/show_bug.cgi?id=36565

Removed WorkerScriptExecutionProxy::retrieve() and added WorkerScriptController::controllerForContext(). This allows
callers to differentiate between "the current context is shutting down" vs "the current context is not a worker context".

Test: Existing worker tests suffice.

  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::getConstructor): Changed to use WorkerScriptController::controllerForContext() instead of WorkerScriptExecutionProxy::retrieve().
  • bindings/v8/V8Utilities.cpp: (WebCore::getScriptExecutionContext): Changed to use WorkerScriptController::controllerForContext() instead of WorkerScriptExecutionProxy::retrieve().
  • bindings/v8/WorkerContextExecutionProxy.cpp: Removed WorkerScriptExecutionProxy::retrieve().
  • bindings/v8/WorkerContextExecutionProxy.h: Removed WorkerScriptExecutionProxy::retrieve().
  • bindings/v8/WorkerScriptController.cpp: (WebCore::WorkerScriptController::controllerForContext): Added helper function to get the WorkerScriptController for the current context.
  • bindings/v8/WorkerScriptController.h: Added declaration for controllerForContext().

2010-03-25 Drew Wilson <atwilson@chromium.org>

Reviewed by Nate Chapin.

[v8] Error in getScriptExecutionContext() when worker context is terminating
https://bugs.webkit.org/show_bug.cgi?id=36565

Test: Existing worker tests suffice.

  • src/WebWorkerClientImpl.cpp: (WebKit::WebWorkerClientImpl::createWorkerContextProxy): Changed to use WorkerScriptController::controllerForContext() instead of WorkerScriptExecutionProxy::retrieve().
16:48 Changeset [56579] by eric@webkit.org

2010-03-25 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Darin Fisher.

[Chromium] Add an ASSERT macro to the Chromium WebKit API
https://bugs.webkit.org/show_bug.cgi?id=36545

  • WebKit.gyp: Add WebCommon.cpp
  • public/WebCommon.h: Add the Macro.
  • public/WebPrivatePtr.h: (WebKit::WebPrivatePtr::~WebPrivatePtr): Verify the pointer is now 0.
  • src/WebCommon.cpp: Added. (WebKit::failedAssertion): Calls the WTF assert function and then crashes.
16:33 Changeset [56578] by kov@webkit.org

2010-03-25 Gustavo Noronha Silva <gns@gnome.org>

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

Skipping tests that started failing after 56567. We need to figure
out a proper fix, that may involve adding a new editing behavior
for our platform.

  • platform/gtk/Skipped:
  • platform/qt/Skipped:
16:30 Changeset [56577] by eric@webkit.org

2010-03-25 Kent Hansen <kent.hansen@nokia.com>

Reviewed by Simon Hausmann.

[Qt] QWebFrame::pageChanged() signal is not documented
https://bugs.webkit.org/show_bug.cgi?id=36609

  • Api/qwebframe.cpp:
16:12 Changeset [56576] by eric@webkit.org

2010-03-25 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Fix the package build on Maemo 5

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

Always embed the stylesheet for the no-listbox rendering. This way it'll be
included in the generated files for the package generation.

  • WebCore.pri:
16:09 Changeset [56575] by laszlo.1.gombos@nokia.com

2010-03-25 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Unreviewed, build fix.

[Qt] Fix QtLauncher guards.

  • QtLauncher/main.cpp: (LauncherWindow::showFPS): (LauncherWindow::updateFPS):
15:49 Changeset [56574] by kov@webkit.org

2010-03-25 Reinout van Schouwen <reinouts@gnome.org>

Reviewed by Gustavo Noronha.

Updated Dutch translation
https://bugs.webkit.org/show_bug.cgi?id=36432

  • nl.po:
15:38 Changeset [56573] by scherkus@chromium.org

2010-03-25 Andrew Scherkus <scherkus@chromium.org>

Unreviewed, expectations tweak.

Reorganized and updated comments for media test expectations.

  • platform/chromium/test_expectations.txt: Tweaked.
15:29 Changeset [56572] by dumi@chromium.org

WebCore: Changing the V8 and JSC implementations of
SQLStatementErrorCallback to interpret as false all results that
could be converted to a false boolean. Pretty much a revert of
r54981.

Reviewed by Dimitri Glazkov.

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

  • bindings/js/JSCustomSQLStatementErrorCallback.cpp:

(WebCore::JSCustomSQLStatementErrorCallback::handleEvent):

  • bindings/v8/custom/V8CustomSQLStatementErrorCallback.cpp:

(WebCore::V8CustomSQLStatementErrorCallback::handleEvent):

  • bindings/v8/custom/V8CustomVoidCallback.cpp:

(WebCore::invokeCallback):

  • bindings/v8/custom/V8CustomVoidCallback.h:

LayoutTests: Updating the test to conform to the latest spec changes.

Reviewed by Dimitri Glazkov.

  • storage/statement-error-callback-expected.txt:
  • storage/statement-error-callback.html:
15:25 Changeset [56571] by eric@webkit.org

2010-03-25 Jochen Eisinger <jochen@chromium.org>

Reviewed by Jeremy Orlow.

Remove ASSERT(isMainThread()) which is violated for workers and not necessary at this point.
https://bugs.webkit.org/show_bug.cgi?id=36614

  • src/DatabaseObserver.cpp: (WebCore::DatabaseObserver::canEstablishDatabase):
14:58 Changeset [56570] by dimich@chromium.org

Unreviewed, fix Chromium tests.

Recent http://trac.webkit.org/changeset/56489 made all urls in Chromium that
have unspecified port to return invalidPortNumber from KURL::port().

  • platform/KURLGoogle.cpp: Return 0 for unspecified ports, mimic WebKit KURL behavior for invalid ports.

(WebCore::KURL::port):

14:39 Changeset [56569] by krit@webkit.org

2010-03-25 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

This is a follow up of:
https://bugs.webkit.org/show_bug.cgi?id=30055
Bad DOM performance in large SVG files

Just save and restore the GraphicsContext, if we realy modify it.
(Cairo)Path::contains checks if a point is in the boundingBox
as a heuristic. The intention was to make the process faster for
points outside of this rect, but it rather causes the
opposite.

No new test cases added. No change in functionality.

  • platform/graphics/cairo/PathCairo.cpp: (WebCore::Path::contains):
  • rendering/RenderPath.cpp: (WebCore::RenderPath::paint):
14:33 Changeset [56568] by simon.fraser@apple.com

2010-03-25 Simon Fraser <simon.fraser@apple.com>

Revert r56565 which was based on an incorrect premise.

  • page/animation/CompositeAnimation.cpp: (WebCore::CompositeAnimation::clearRenderer): (WebCore::CompositeAnimation::updateKeyframeAnimations): (WebCore::CompositeAnimation::getAnimatedStyle): (WebCore::CompositeAnimation::setAnimating): (WebCore::CompositeAnimation::timeToNextService): (WebCore::CompositeAnimation::getAnimationForProperty): (WebCore::CompositeAnimation::suspendAnimations): (WebCore::CompositeAnimation::resumeAnimations): (WebCore::CompositeAnimation::isAnimatingProperty): (WebCore::CompositeAnimation::pauseAnimationAtTime): (WebCore::CompositeAnimation::numberOfActiveAnimations):
13:21 Changeset [56567] by ojan@chromium.org

2010-03-25 Ojan Vafai <ojan@chromium.org>

Reviewed by David Levin.

mouse-based selections are always directional on Window/Linux
https://bugs.webkit.org/show_bug.cgi?id=25195

The tests are modified to expect different things for Win/Linux vs. Mac.

  • editing/selection/5195166-1.html:
  • editing/selection/extend-after-mouse-selection.html:
  • editing/selection/extend-selection-after-double-click-expected.txt:
  • editing/selection/extend-selection-after-double-click.html:
  • platform/chromium-linux/editing/selection/5195166-1-expected.checksum: Removed.
  • platform/chromium-linux/editing/selection/5195166-1-expected.png: Removed.
  • platform/chromium-win/editing/selection/5195166-1-expected.checksum: Removed.
  • platform/chromium-win/editing/selection/5195166-1-expected.png: Removed.
  • platform/chromium-win/editing/selection/5195166-1-expected.txt: Removed.
  • platform/mac/editing/selection/5195166-1-expected.checksum: Removed.
  • platform/mac/editing/selection/5195166-1-expected.png: Removed.
  • platform/mac/editing/selection/5195166-1-expected.txt:
  • platform/qt/editing/selection/5195166-1-expected.txt: Removed.
  • platform/win/editing/selection/extend-after-mouse-selection-expected.txt:

2010-03-25 Ojan Vafai <ojan@chromium.org>

Reviewed by David Levin.

mouse-based selections are always directional on Windows/Linux
https://bugs.webkit.org/show_bug.cgi?id=25195

Change m_lastChangeWasHorizontalExtension to m_isDirectional
and make m_isDirectional always be true for Windows/Linux.

  • editing/SelectionController.cpp: (WebCore::SelectionController::SelectionController): (WebCore::SelectionController::setSelection): (WebCore::SelectionController::setIsDirectional): (WebCore::SelectionController::willBeModified): When double-clicking, the base/extent will be in the middle of the selection instead of the start/end of it. Changed to maintain that modifications after double-click still move the start/end of the selection, not the base/extent. (WebCore::SelectionController::modify):
  • editing/SelectionController.h:
  • page/EventHandler.cpp: (WebCore::EventHandler::handleMousePressEventSingleClick): (WebCore::EventHandler::updateSelectionForMouseDrag):
13:20 Changeset [56566] by yurys@chromium.org

2010-03-25 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Allow running tests with enabled developer extras and closed Web Inspector. Tests that have inspector-enabled/ in their path/url will have developer extras enabled.

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

  • DumpRenderTree/LayoutTestController.h:
  • DumpRenderTree/gtk/DumpRenderTree.cpp: (shouldEnableDeveloperExtras): (runTest):
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::setJavaScriptProfilingEnabled): (LayoutTestController::setDeveloperExtrasEnabled): (LayoutTestController::showWebInspector): (LayoutTestController::closeWebInspector):
  • DumpRenderTree/mac/DumpRenderTree.mm: (shouldEnableDeveloperExtras): (runTest):
  • DumpRenderTree/mac/LayoutTestControllerMac.mm: (LayoutTestController::setJavaScriptProfilingEnabled): (LayoutTestController::setDeveloperExtrasEnabled): (LayoutTestController::showWebInspector): (LayoutTestController::closeWebInspector):
  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::shouldEnableDeveloperExtras): (WebCore::DumpRenderTree::open):
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::closeWebInspector): (LayoutTestController::setDeveloperExtrasEnabled): (LayoutTestController::showWebInspector): (LayoutTestController::setJavaScriptProfilingEnabled):
  • DumpRenderTree/win/DumpRenderTree.cpp: (shouldEnableDeveloperExtras): (runTest):
  • DumpRenderTree/win/LayoutTestControllerWin.cpp: (LayoutTestController::setJavaScriptProfilingEnabled): (LayoutTestController::setDeveloperExtrasEnabled): (LayoutTestController::showWebInspector): (LayoutTestController::closeWebInspector):
12:57 Changeset [56565] by simon.fraser@apple.com

2010-03-25 Simon Fraser <simon.fraser@apple.com>

Reviewed by Eric Carlson.

Make calls to m_keyframeAnimations.checkConsistency() debug-only
https://bugs.webkit.org/show_bug.cgi?id=36555

Remove lots of calls to m_keyframeAnimations.checkConsistency(); which were left after some earlier
debugging. Leave one in CompositeAnimation::animate() to catch any future issues, which is OK since
these calls are no-ops in release builds.

  • page/animation/CompositeAnimation.cpp: (WebCore::CompositeAnimation::clearRenderer): (WebCore::CompositeAnimation::updateKeyframeAnimations): (WebCore::CompositeAnimation::getAnimatedStyle): (WebCore::CompositeAnimation::setAnimating): (WebCore::CompositeAnimation::timeToNextService): (WebCore::CompositeAnimation::getAnimationForProperty): (WebCore::CompositeAnimation::suspendAnimations): (WebCore::CompositeAnimation::resumeAnimations): (WebCore::CompositeAnimation::isAnimatingProperty): (WebCore::CompositeAnimation::pauseAnimationAtTime): (WebCore::CompositeAnimation::numberOfActiveAnimations):
12:07 Changeset [56564] by simon.fraser@apple.com

2010-03-25 Simon Fraser <simon.fraser@apple.com>

Build fix: no review.

Another c_str() -> data().

  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::enumerate):
11:58 Changeset [56563] by barraclough@apple.com
  • profiler/ProfileGenerator.cpp:

(JSC::ProfileGenerator::willExecute):
(JSC::ProfileGenerator::didExecute):

Reviewed by NOBODY (build fix).

11:48 Changeset [56562] by simon.fraser@apple.com

2010-03-25 Simon Fraser <simon.fraser@apple.com>

Reviewed by Geoffrey Garen.

https://bugs.webkit.org/show_bug.cgi?id=36557
Animations use the wrong timing-function sometimes

Fix an issue where animations picked the wrong style for their "unanimated" value.

Previously, the "unanimated" style was considered to be the style before the animation
was applied. This caused us to pick up stale values for animation timing functions, for
blending when a property is missing from the first keyframe, and for resuming transitions
once an animation ends.

Instead, we need to use the "current" style, which is the one that causes the animation
to kick off.

Tests: animations/longhand-timing-function.html

animations/transition-and-animation-3.html
animations/unanimated-style.html

  • page/animation/CompositeAnimation.cpp: (WebCore::CompositeAnimation::updateKeyframeAnimations):
11:37 WebKit Team edited by cjerdonek@webkit.org
Updated link to committers.py. (diff)
11:29 Changeset [56561] by ossy@webkit.org

Unreviewed expected file fix.

  • platform/mac/fast/frames/flattening/frameset-flattening-grid-expected.txt: Needless newline removed.
11:20 Changeset [56560] by barraclough@apple.com

Bug 36611 - Cleanup JSC::CString
Rename CString::c_str() -> CString::data(), CString::size() -> CString::length(),
remove UString::getCString() (all uses are wrong, should use UString::UTF8String()).

Reviewed by Geoff Garen.

JavaScriptCore:

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::printUnaryOp):
(JSC::CodeBlock::printBinaryOp):
(JSC::CodeBlock::printConditionalJump):
(JSC::CodeBlock::printGetByIdOp):
(JSC::CodeBlock::printPutByIdOp):
(JSC::printGlobalResolveInfo):
(JSC::printStructureStubInfo):
(JSC::CodeBlock::printStructure):
(JSC::CodeBlock::printStructures):
(JSC::CodeBlock::dump):

  • jsc.cpp:

(functionPrint):
(functionDebug):
(runInteractive):
(fillBufferWithContentsOfFile):

  • profiler/CallIdentifier.h:

(JSC::CallIdentifier::c_str):

  • profiler/Profile.cpp:

(JSC::Profile::debugPrintDataSampleStyle):

  • profiler/ProfileNode.cpp:

(JSC::ProfileNode::debugPrintData):
(JSC::ProfileNode::debugPrintDataSampleStyle):

  • runtime/DateConversion.cpp:

(JSC::parseDate):

  • runtime/JSGlobalObjectFunctions.cpp:

(JSC::encode):
(JSC::globalFuncJSCPrint):

  • runtime/UString.cpp:

(JSC::operator==):
(JSC::UString::toDouble):

  • runtime/UString.h:

(JSC::CString::length):
(JSC::CString::data):

WebCore:

  • bridge/NP_jsobject.cpp:

(_NPN_Enumerate):

  • bridge/c/c_utility.cpp:

(JSC::Bindings::convertValueToNPVariant):

  • bridge/jni/jsc/JNIBridgeJSC.cpp:

(JavaField::valueFromInstance):
(JavaField::setValueToInstance):

  • bridge/jni/jsc/JavaInstanceJSC.cpp:

(JavaInstance::invokeMethod):

  • bridge/jni/jsc/JavaStringJSC.h:

(JSC::Bindings::JavaStringImpl::UTF8String):

11:03 Changeset [56559] by jwieczorek@webkit.org

2010-03-25 Jakub Wieczorek <jwieczorek@webkit.org>

Rubber-stamped by Tor Arne Vestbø.

[Qt] Update test results after r56555.
https://bugs.webkit.org/show_bug.cgi?id=36413

  • platform/qt/fast/forms/basic-inputs-expected.txt:
  • platform/qt/fast/forms/isindex-placeholder-expected.txt:
  • platform/qt/fast/forms/text-style-color-expected.txt:
  • platform/qt/fast/forms/textarea-placeholder-pseudo-style-expected.txt:
  • platform/qt/fast/forms/textarea-placeholder-set-attribute-expected.txt:
  • platform/qt/fast/forms/textarea-placeholder-set-value-expected.txt:
11:00 Changeset [56558] by ap@apple.com

Reviewed by Mark Rowe.

Correctness fix after r56493.

  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WKPCRunSyncOpenPanel): We still need to update our hostProxy reference, even though we didn't use to have instanceProxy. Nothing guarantees that the host proxy won't go away while the open panel is up.
10:51 Changeset [56557] by tonikitoo@webkit.org

2010-03-25 Antonio Gomes <tonikitoo@webkit.org>

Unreview expectation result fix.

After http://trac.webkit.org/changeset/56547, the test needs new (and proper)
results.

  • platform/mac/fast/frames/flattening/frameset-flattening-grid-expected.txt:
10:39 Changeset [56556] by joepeck@webkit.org

2010-03-25 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by David Levin.

Asking Questions Page Grammar Fixes
https://bugs.webkit.org/show_bug.cgi?id=36608

  • asking_questions.html:
10:30 Changeset [56555] by jwieczorek@webkit.org

2010-03-25 Jakub Wieczorek <jwieczorek@webkit.org>

Reviewed by Eric Seidel.

[Qt] RenderTheme: fix size adjustment for text fields
https://bugs.webkit.org/show_bug.cgi?id=36413

Update test results after RenderTheme changes.

  • platform/qt/fast/forms/textarea-placeholder-pseudo-style-expected.txt:
  • platform/qt/fast/forms/textarea-placeholder-set-attribute-expected.txt:
  • platform/qt/fast/forms/textarea-placeholder-set-value-expected.txt:

2010-03-25 Jakub Wieczorek <jwieczorek@webkit.org>

Reviewed by Eric Seidel.

[Qt] RenderTheme: fix size adjustment for text fields
https://bugs.webkit.org/show_bug.cgi?id=36413

To match other ports, remove the hardcoded dimensions for text fields so that
the default ones from WebCore can be used.
Properly apply the correct padding to text areas - this fixes their appearance
with the Oxygen style.

  • platform/qt/RenderThemeQt.cpp: (WebCore::RenderThemeQt::computeSizeBasedOnStyle):
10:21 Changeset [56554] by abarth@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Rubber stamped by Eric Seidel.

Tweaks to sheriffbot to improve latency by keeping the working copy up
to date even when there's no build break. Also, officially move
sheriffbot to #webkit.

(Also teach ValidateReviewer to understand rubber stamps.)

  • Scripts/webkitpy/common/net/irc/ircbot.py:
  • Scripts/webkitpy/tool/commands/sheriffbot.py:
  • Scripts/webkitpy/tool/steps/validatereviewer.py:
10:18 Changeset [56553] by tonikitoo@webkit.org

Not displaying FPS info on the terminal. On S60 and Maemo the Window title will be used and Status bar will used on desktop.

Reviewed by Simon Hausmann.
Patch by Jesus Palencia <jesus.palencia@openbossa.org>

[Qt] QtLauncher's FPS info should not be displayed on the terminal
https://bugs.webkit.org/show_bug.cgi?id=36244

  • QtLauncher/main.cpp:

(LauncherWindow::initializeView):
(LauncherWindow::showFPS):
(LauncherWindow::updateFPS):

  • QtLauncher/webview.cpp:

(WebViewGraphicsBased::setFrameRateMeasurementEnabled):
(WebViewGraphicsBased::updateFrameRate):

  • QtLauncher/webview.h:

(WebViewGraphicsBased::frameRateMeasurementEnabled):

09:42 Changeset [56552] by vestbo@webkit.org

[Qt] Implement ChromeClient::windowResizerRect()

Reviewed by Simon Hausmann.

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

We assume the resize corner to be in the lower right corner of
the window and having the width and height of the scrollbars.

The helper function geometryRelativeToOwnerWidget() in the page
client is used to clip the resize rect to the actual size of the
viewport, not the size of the QGraphicsView.

09:31 Changeset [56551] by joepeck@webkit.org

2010-03-25 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by Timothy Hatcher.

New Asking Questions Page is not Valid HTML5
https://bugs.webkit.org/show_bug.cgi?id=36605

  • asking_questions.html: fixed issues raised by the validator.
09:18 Changeset [56550] by jchaffraix@webkit.org

[Qt][GTK] http/tests/xmlhttprequest/xmlhttprequest-onProgress-open-should-zero-length.html
fails after r56394
https://bugs.webkit.org/show_bug.cgi?id=36531

Reviewed by Alexey Proskuryakov.

The test was assuming that we would receive 3 progress events which may not be the case anymore
due to the event throttling added in r56394.

  • http/tests/xmlhttprequest/xmlhttprequest-onProgress-open-should-zero-length.html: Removed

the counting as it belonged to the original version of the test and instead use the position
to determine when to reset the XHR.

  • platform/gtk/Skipped:
  • platform/qt/Skipped:

Removed the test from the Skipped lists now that it should pass.

09:09 Changeset [56549] by joepeck@webkit.org

2010-03-25 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by Pavel Feldman.

Web Inspector: reloading debug target window with cmd-R from inspector window produces beep.
https://bugs.webkit.org/show_bug.cgi?id=36554

  • inspector/front-end/inspector.js: (WebInspector.documentKeyDown): prevent the default behavior, if handled, which would result in a system beep.
08:49 QtWebKitBackportingFixes edited by hausmann@webkit.org
(diff)
08:25 Changeset [56548] by eric@webkit.org

2010-03-25 Jochen Eisinger <jochen@chromium.org>

Reviewed by Jeremy Orlow.

Forward DatabaseTracker::canEstablishDatabase to chromium layer.
https://bugs.webkit.org/show_bug.cgi?id=36595

  • storage/chromium/DatabaseObserver.h:
  • storage/chromium/DatabaseTrackerChromium.cpp: (WebCore::DatabaseTracker::canEstablishDatabase):

2010-03-25 Jochen Eisinger <jochen@chromium.org>

Reviewed by Jeremy Orlow.

Forward DatabaseTracker::canEstablishDatabase to chromium layer.
https://bugs.webkit.org/show_bug.cgi?id=36595

  • public/WebFrameClient.h: (WebKit::WebFrameClient::allowDatabase):
  • src/DatabaseObserver.cpp: (WebCore::DatabaseObserver::canEstablishDatabase):
  • src/WebWorkerBase.h: (WebKit::WebWorkerBase::allowDatabase):
08:24 Changeset [56547] by tonikitoo@webkit.org

Added missing frame flattening activation on fast/frames/flattening/frameset-flattening-grid.html; removed unnecessary CONSOLE MESSAGE from the expected file; reset the setFrameSetFlatteningEnabled for each test.

Reviewed by Kenneth Christiansen.
Patch by Diego Gonzalez <diego.gonzalez@openbossa.org>

WebKitTools:

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp:

(WebCore::WebPage::resetSettings):

LayoutTests:

  • fast/frames/flattening/frameset-flattening-grid.html:
  • platform/qt/fast/frames/flattening/frameset-flattening-grid-expected.txt:
08:06 Changeset [56546] by eric@webkit.org

2010-03-25 yael aharon <yael.aharon@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] Windowed netscape plugins don't work with QGraphicsWebView on Symbian
https://bugs.webkit.org/show_bug.cgi?id=35112

Add a proxy widget when loading a QWidget based plugin in a QGraphicsWebView.

  • plugins/symbian/PluginContainerSymbian.cpp: (PluginContainerSymbian::PluginContainerSymbian): (PluginContainerSymbian::focusInEvent):
  • plugins/symbian/PluginContainerSymbian.h: (WebCore::PluginContainerSymbian::proxy):
  • plugins/symbian/PluginViewSymbian.cpp: (WebCore::PluginView::updatePluginWidget): (WebCore::PluginView::platformStart): (WebCore::PluginView::platformDestroy):
08:04 QtWebKitBackportingFixes edited by hausmann@webkit.org
(diff)
08:04 Changeset [56545] by cjerdonek@webkit.org

Moved deprecated_logging unit test import statement from
webkitpy/unittests.py to webkitpy/common/system/unittests.py.

Reviewed by Adam Barth.

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

This is part of the master bug to reorganize webkitpy.

  • Scripts/webkitpy/common/system/unittests.py:
  • Scripts/webkitpy/unittests.py:
07:46 Changeset [56544] by cjerdonek@webkit.org

Moved webkit_logging.py to common/system/deprecated_logging.py
inside webkitpy.

Reviewed by Adam Barth.

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

This is part of the master bug to reorganize webkitpy.

  • Scripts/validate-committer-lists:
  • Scripts/webkitpy/common/checkout/changelog.py:
  • Scripts/webkitpy/common/checkout/scm.py:
  • Scripts/webkitpy/common/net/bugzilla.py:
  • Scripts/webkitpy/common/net/buildbot.py:
  • Scripts/webkitpy/common/net/credentials.py:
  • Scripts/webkitpy/common/net/irc/ircproxy.py:
  • Scripts/webkitpy/common/net/networktransaction.py:
  • Scripts/webkitpy/common/net/statusserver.py:
  • Scripts/webkitpy/common/system/deprecated_logging.py: Copied from WebKitTools/Scripts/webkitpy/webkit_logging.py.
  • Scripts/webkitpy/common/system/deprecated_logging_unittest.py: Copied from WebKitTools/Scripts/webkitpy/webkit_logging_unittest.py.
  • Scripts/webkitpy/common/system/executive.py:
  • Scripts/webkitpy/tool/bot/queueengine.py:
  • Scripts/webkitpy/tool/commands/download.py:
  • Scripts/webkitpy/tool/commands/earlywarningsystem_unittest.py:
  • Scripts/webkitpy/tool/commands/openbugs.py:
  • Scripts/webkitpy/tool/commands/queries.py:
  • Scripts/webkitpy/tool/commands/queues.py:
  • Scripts/webkitpy/tool/commands/sheriffbot.py:
  • Scripts/webkitpy/tool/commands/stepsequence.py:
  • Scripts/webkitpy/tool/commands/upload.py:
  • Scripts/webkitpy/tool/main.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/multicommandtool.py:
  • Scripts/webkitpy/tool/steps/abstractstep.py:
  • Scripts/webkitpy/tool/steps/applypatch.py:
  • Scripts/webkitpy/tool/steps/build.py:
  • Scripts/webkitpy/tool/steps/checkstyle.py:
  • Scripts/webkitpy/tool/steps/closebug.py:
  • Scripts/webkitpy/tool/steps/closebugforlanddiff.py:
  • Scripts/webkitpy/tool/steps/confirmdiff.py:
  • Scripts/webkitpy/tool/steps/ensurebuildersaregreen.py:
  • Scripts/webkitpy/tool/steps/ensurelocalcommitifneeded.py:
  • Scripts/webkitpy/tool/steps/obsoletepatches.py:
  • Scripts/webkitpy/tool/steps/preparechangelog.py:
  • Scripts/webkitpy/tool/steps/reopenbugafterrollout.py:
  • Scripts/webkitpy/tool/steps/runtests.py:
  • Scripts/webkitpy/tool/steps/update.py:
  • Scripts/webkitpy/tool/steps/updatechangelogswithreviewer.py:
  • Scripts/webkitpy/tool/steps/validatereviewer.py:
  • Scripts/webkitpy/unittests.py:
  • Scripts/webkitpy/webkit_logging.py: Removed.
  • Scripts/webkitpy/webkit_logging_unittest.py: Removed.
07:31 Changeset [56543] by simon.fraser@apple.com

2010-03-25 Simon Fraser <simon.fraser@apple.com>

Reviewed by Adele Peterson.

https://bugs.webkit.org/show_bug.cgi?id=35714
Computed style on delayed transform animations is incorrect

When fetching computed style while a delayed accelerated animation is in effect,
we would get the value form the first keyframe, rather than getting the current
style.

Fix by checking to see whether we've in the delay phase (with no backwards-fill)
in KeyframeAnimation::getAnimatedStyle().

Test: compositing/animation/computed-style-during-delay.html

  • page/animation/KeyframeAnimation.cpp: (WebCore::KeyframeAnimation::getAnimatedStyle):
07:31 Changeset [56542] by simon.fraser@apple.com

Additional refptr/passrefptr workarounds for WINSCW compiler

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

Patch by Janne Koskinen <janne.p.koskinen@digia.com> on 2010-02-12

Reviewed by Tor Arne Vestbø.

  • wtf/PassRefPtr.h:

(WTF::refIfNotNull):
(WTF::PassRefPtr::PassRefPtr):
(WTF::PassRefPtr::~PassRefPtr):
(WTF::PassRefPtr::clear):
(WTF::::operator):

  • wtf/RefPtr.h:

(WTF::RefPtr::RefPtr):
(WTF::::operator):

07:27 Changeset [56541] by jchaffraix@webkit.org

Add a page on how to ask questions
https://bugs.webkit.org/show_bug.cgi?id=35223

Reviewed by David Levin.

  • asking_questions.html: Added.

Add a page on how to ask questions based on Ariya Hidayat's mail: how
to get help effectively
(http://article.gmane.org/gmane.os.opendarwin.webkit.user/145/match=get+help+effectively)

  • contact.html: Tweaked the wording to make it more clear what the

mailing list are used for. Also changed the formatting to better
underline the different mailing lists.

07:14 PythonGuidelines edited by cjerdonek@webkit.org
Added note on cleaning *.pyc files. (diff)
07:13 Changeset [56540] by jchaffraix@webkit.org

commit-log-editor can call itself in an infinite loop
https://bugs.webkit.org/show_bug.cgi?id=35291

Reviewed by Kenneth Rohde Christiansen.

if $editor ends up being commit-log-editor, the script will exec itself
in an infinite loop.

To avoid this, we now check that the $editor variable is not
commit-log-editor to avoid this case.

  • Scripts/commit-log-editor: Added an isCommitLogEditor method and

reworked the $editor setting to add this check.

07:04 Changeset [56539] by eric@webkit.org

2010-03-25 Jesus Sanchez-Palencia <jesus.palencia@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

Add a Toggle Frame Flattening option to QtLauncher.
It will be enabled by default on Maemo5 and S60 platforms.

[Qt] Add enable/disable Frame Flattening option to QtLauncher
https://bugs.webkit.org/show_bug.cgi?id=36558

  • QtLauncher/main.cpp: (LauncherWindow::applyPrefs): (LauncherWindow::toggleFrameFlattening): (LauncherWindow::createChrome):
06:45 Changeset [56538] by eric@webkit.org

2010-03-25 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Initialize Web Inspector title with inspected page URL when opening Web Inspector.

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

  • inspector/InspectorController.cpp: (WebCore::InspectorController::setFrontend):
06:43 Changeset [56537] by chang.shu@nokia.com

2010-03-25 Shu Chang <chang.shu@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Added documentation for delete button.
https://bugs.webkit.org/show_bug.cgi?id=31560

  • Api/qwebsettings.cpp:
06:21 Changeset [56536] by eric@webkit.org

2010-03-25 Tasuku Suzuki <tasuku.suzuki@nokia.com>

Reviewed by Eric Seidel.

[Qt]Fix compile error with QT_NO_DESKTOPSERVICES
https://bugs.webkit.org/show_bug.cgi?id=36533

  • Api/qwebsettings.cpp: (QWebSettings::enablePersistentStorage):
06:02 Changeset [56535] by eric@webkit.org

2010-03-25 Antti Koivisto <koivisto@iki.fi>

Reviewed by Kenneth Rohde Christiansen.

https://bugs.webkit.org/show_bug.cgi?id=36316
Tiles of selected content do not update if not in viewport

Don't clip repaints to viewport when tiling is enabled.

  • page/Frame.cpp: (WebCore::Frame::setView): (WebCore::Frame::setTiledBackingStoreEnabled):
05:43 Changeset [56534] by eric@webkit.org

2010-03-25 Diego Gonzalez <diego.gonzalez@openbossa.org>

Reviewed by Eric Seidel.

Missing forward ENABLE_DATALIST macro to JavaScript enabled features macros
on WebKit.pri

[Qt] Forward ENABLE_DATALIST macro to JavaScript enabled macros
https://bugs.webkit.org/show_bug.cgi?id=36547

  • platform/qt/Skipped:

2010-03-25 Diego Gonzalez <diego.gonzalez@openbossa.org>

Reviewed by Eric Seidel.

Missing forward ENABLE_DATALIST macro to JavaScript enabled features macros
on WebKit.pri

[Qt] Forward ENABLE_DATALIST macro to JavaScript enabled macros
https://bugs.webkit.org/show_bug.cgi?id=36547

  • WebCore.pri:
05:22 Changeset [56533] by eric@webkit.org

2010-03-25 Tasuku Suzuki <tasuku.suzuki@nokia.com>

Reviewed by Eric Seidel.

[Qt]Fix compile error with QT_NO_DEBUG_STREAM
https://bugs.webkit.org/show_bug.cgi?id=36533

  • Api/qgraphicswebview.cpp: (QGraphicsWebViewPrivate::graphicsItemVisibleRect):
05:19 Changeset [56532] by pfeldman@chromium.org

2010-03-25 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Expect console object wrapping to fail.

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

  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor):
05:03 Changeset [56531] by eric@webkit.org

2010-03-25 Sergio Villar Senín <svillar@igalia.com>

Reviewed by Xan Lopez.

Added new API webkit_web_back_forward_list_clear. This function
clears the back forward list

[GTK] http/history tests are failing
https://bugs.webkit.org/show_bug.cgi?id=36173

  • tests/testwebbackforwardlist.c: (test_webkit_web_back_forward_list_clear): (main): added new unit test for the new API
  • webkit/webkitwebbackforwardlist.cpp: (webkit_web_back_forward_list_clear):
  • webkit/webkitwebbackforwardlist.h: new function that clears the back forward list
04:44 Changeset [56530] by eric@webkit.org

2010-03-25 Joanmarie Diggs <joanmarie.diggs@gmail.com>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=36371
[Gtk] Elements with a title attribute should not be ignored

  • platform/gtk/accessibility/object-with-title.html: Added
  • platform/gtk/accessibility/object-with-title-expected.txt: Added

2010-03-25 Joanmarie Diggs <joanmarie.diggs@gmail.com>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=36371
[Gtk] Elements with a title attribute should not be ignored

  • accessibility/AccessibilityRenderObject.cpp: (AccessibilityRenderObject::accessibilityIsIgnored):
04:23 Changeset [56529] by eric@webkit.org

2010-03-25 Kenneth Rohde Christiansen <kenneth@webkit.org>

Reviewed by Eric Seidel.

Make QWebPage not depend on view() but use the client->ownerWidget()
instead. Also, handle the case where there is not page client.

  • Api/qwebpage.cpp: (QWebPagePrivate::createContextMenu): (QWebPagePrivate::keyPressEvent): (QWebPage::javaScriptAlert): (QWebPage::javaScriptConfirm): (QWebPage::javaScriptPrompt): (QWebPage::shouldInterruptJavaScript): (QWebPage::createWindow): (QWebPage::action): (QWebPage::extension): (QWebPage::chooseFile): (QWebPage::userAgentForUrl):
04:20 Changeset [56528] by jocelyn.turcotte@nokia.com

[Qt] Build fix on MSVC. Reverts r55633 for stdint.h

Reviewed by nobody, build fix.

This file gets included in generated moc files which don't
include the prefix header.

  • os-win32/stdint.h:
03:43 PythonGuidelines edited by cjerdonek@webkit.org
Added note about *.pyc files (diff)
03:40 Changeset [56527] by steveblock@google.com

2010-03-24 Steve Block <steveblock@google.com>

Reviewed by David Levin.

Adds a missing ENABLE(WORKERS) guards to the V8 bindings.
https://bugs.webkit.org/show_bug.cgi?id=36550

Build fix only, no new tests.

  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::instantiateV8Object):
03:33 Changeset [56526] by eric@webkit.org

2010-03-25 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Eric Seidel.

Add EFL-specific code to platform/ScrollView.cpp and
platform/Scrollbar.cpp.
http://webkit.org/b/36305

  • platform/ScrollView.cpp:
  • platform/Scrollbar.cpp:
03:30 PythonGuidelines edited by cjerdonek@webkit.org
Added Basics and Code Structure sub-sections (diff)
03:12 Changeset [56525] by eric@webkit.org

2010-03-25 Tasuku Suzuki <tasuku.suzuki@nokia.com>

Reviewed by Eric Seidel.

[Qt]Fix compile error with QT_NO_LIBRARY
https://bugs.webkit.org/show_bug.cgi?id=36533

  • platform/FileSystem.h:
  • platform/qt/FileSystemQt.cpp: (WebCore::unloadModule):
02:46 Changeset [56524] by zoltan@webkit.org

2010-03-25 Zoltan Horvath <zoltan@webkit.org>

Reviewed by Eric Seidel.

svn-create-patch prints a warning for large patches
https://bugs.webkit.org/show_bug.cgi?id=32582

svn-create-patch prints a warning message for larger patches than 20k.

  • Scripts/svn-create-patch:
02:40 Changeset [56523] by eric@webkit.org

2010-03-25 Tasuku Suzuki <tasuku.suzuki@nokia.com>

Reviewed by Eric Seidel.

[Qt]Fix compile error with QT_NO_ANIMATION
https://bugs.webkit.org/show_bug.cgi?id=36526

  • platform/graphics/qt/GraphicsLayerQt.cpp: (WebCore::GraphicsLayerQtImpl::~GraphicsLayerQtImpl):
  • platform/graphics/qt/GraphicsLayerQt.h:
02:32 Changeset [56522] by abarth@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Unreviewed. Renamed early_warning_system.py to earlywarningsystem.py.
https://bugs.webkit.org/show_bug.cgi?id=36093

  • Scripts/webkitpy/tool/commands/early_warning_system.py: Removed.
  • Scripts/webkitpy/tool/commands/early_warning_system_unittest.py: Removed.
  • Scripts/webkitpy/tool/commands/earlywarningsystem.py: Copied from Scripts/webkitpy/tool/commands/early_warning_system.py.
  • Scripts/webkitpy/tool/commands/earlywarningsystem_unittest.py: Copied from Scripts/webkitpy/tool/commands/early_warning_system_unittest.py.
  • Scripts/webkitpy/tool/main.py:
  • Scripts/webkitpy/tool/unittests.py:
02:30 Changeset [56521] by cjerdonek@webkit.org

In webkitpy, pushed the unit test import statements in
webkitpy/unittests.py into appropriate unittests.py files in the
new root-level packages beneath webkitpy.

Reviewed by Adam Barth.

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

This is part of the master bug to reorganize webkitpy.

  • Scripts/webkitpy/common/system/unittests.py:
  • Scripts/webkitpy/common/unittests.py: Added.
  • Scripts/webkitpy/python24/unittests.py: Added.
  • Scripts/webkitpy/tool/unittests.py: Added.
  • Scripts/webkitpy/unittests.py:
02:17 Changeset [56520] by abarth@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Unreviewed. Move WebKitCheckout into the webkitpy.common.checkout
package.
https://bugs.webkit.org/show_bug.cgi?id=36093

  • Scripts/webkitpy/common/checkout/api.py: Copied from Scripts/webkitpy/webkitcheckout.py.
  • Scripts/webkitpy/tool/main.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/webkitcheckout.py: Removed.
02:11 Changeset [56519] by abarth@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Unreviewed. Move OutputCapture to webkitpy.common.system.
https://bugs.webkit.org/show_bug.cgi?id=36093

  • Scripts/webkitpy/common/net/bugzilla_unittest.py:
  • Scripts/webkitpy/common/net/credentials_unittest.py:
  • Scripts/webkitpy/common/system/outputcapture.py: Copied from Scripts/webkitpy/outputcapture.py.
  • Scripts/webkitpy/outputcapture.py: Removed.
  • Scripts/webkitpy/tool/commands/commandtest.py:
  • Scripts/webkitpy/tool/commands/queues_unittest.py:
  • Scripts/webkitpy/tool/commands/queuestest.py:
  • Scripts/webkitpy/tool/multicommandtool_unittest.py:
  • Scripts/webkitpy/tool/steps/closebugforlanddiff_unittest.py:
  • Scripts/webkitpy/tool/steps/steps_unittest.py:
  • Scripts/webkitpy/tool/steps/updatechangelogswithreview_unittests.py:
02:05 Changeset [56518] by abarth@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Unreviewed. Move user.py to webkitpy.common.system.

  • Scripts/webkitpy/common/checkout/scm.py:
  • Scripts/webkitpy/common/net/bugzilla.py:
  • Scripts/webkitpy/common/net/credentials.py:
  • Scripts/webkitpy/common/system/user.py: Copied from Scripts/webkitpy/user.py.
  • Scripts/webkitpy/common/system/user_unittest.py: Copied from Scripts/webkitpy/user_unittest.py.
  • Scripts/webkitpy/tool/commands/upload.py:
  • Scripts/webkitpy/tool/main.py:
  • Scripts/webkitpy/unittests.py:
  • Scripts/webkitpy/user.py: Removed.
  • Scripts/webkitpy/user_unittest.py: Removed.
02:01 Changeset [56517] by abarth@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Unreviewed. Move executive.py to webkitpy.common.system.
https://bugs.webkit.org/show_bug.cgi?id=36093

  • Scripts/webkitpy/common/checkout/scm.py:
  • Scripts/webkitpy/common/checkout/scm_unittest.py:
  • Scripts/webkitpy/common/config/ports.py:
  • Scripts/webkitpy/common/config/ports_unittest.py:
  • Scripts/webkitpy/common/net/credentials.py:
  • Scripts/webkitpy/common/net/credentials_unittest.py:
  • Scripts/webkitpy/common/system/executive.py: Copied from Scripts/webkitpy/executive.py.
  • Scripts/webkitpy/common/system/executive_unittest.py: Copied from Scripts/webkitpy/executive_unittest.py.
  • Scripts/webkitpy/executive.py: Removed.
  • Scripts/webkitpy/executive_unittest.py: Removed.
  • Scripts/webkitpy/layout_tests/port/mac.py:
  • Scripts/webkitpy/tool/bot/queueengine.py:
  • Scripts/webkitpy/tool/bot/queueengine_unittest.py:
  • Scripts/webkitpy/tool/commands/download.py:
  • Scripts/webkitpy/tool/commands/early_warning_system.py:
  • Scripts/webkitpy/tool/commands/queues.py:
  • Scripts/webkitpy/tool/commands/stepsequence.py:
  • Scripts/webkitpy/tool/main.py:
  • Scripts/webkitpy/tool/steps/checkstyle.py:
  • Scripts/webkitpy/tool/steps/preparechangelog.py:
  • Scripts/webkitpy/unittests.py:
  • Scripts/webkitpy/webkit_logging_unittest.py:
01:50 Changeset [56516] by abarth@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Unreviewed. Move grammary.py into webkitpy.tool.
https://bugs.webkit.org/show_bug.cgi?id=36093

  • Scripts/webkitpy/grammar.py: Removed.
  • Scripts/webkitpy/grammar_unittest.py: Removed.
  • Scripts/webkitpy/tool/commands/download.py:
  • Scripts/webkitpy/tool/commands/queries.py:
  • Scripts/webkitpy/tool/commands/queues.py:
  • Scripts/webkitpy/tool/commands/upload.py:
  • Scripts/webkitpy/tool/grammar.py: Copied from Scripts/webkitpy/grammar.py.
  • Scripts/webkitpy/tool/grammar_unittest.py: Copied from Scripts/webkitpy/grammar_unittest.py.
  • Scripts/webkitpy/tool/multicommandtool.py:
  • Scripts/webkitpy/tool/steps/obsoletepatches.py:
  • Scripts/webkitpy/tool/steps/updatechangelogswithreviewer.py:
  • Scripts/webkitpy/unittests.py:
01:49 Changeset [56515] by jam@chromium.org

2010-03-10 John Abd-El-Malek <jam@chromium.org>

Reviewed by Darin Fisher.

[chromium] Fix up WebMouseWheelEventBuilder to properly calculate the units
https://bugs.webkit.org/show_bug.cgi?id=35989

  • src/WebInputEventConversion.cpp:
01:48 Changeset [56514] by cjerdonek@webkit.org

In webkitpy, moved init/ to common/system/.

Reviewed by Adam Barth.

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

This is part of the master bug to reorganize webkitpy.

  • Scripts/webkitpy/common/system: Copied from WebKitTools/Scripts/webkitpy/init.
  • Scripts/webkitpy/common/system/logutils.py:
  • Scripts/webkitpy/common/system/logutils_unittest.py:
  • Scripts/webkitpy/init: Removed.
  • Scripts/webkitpy/init/init.py: Removed.
  • Scripts/webkitpy/init/logtesting.py: Removed.
  • Scripts/webkitpy/init/logutils.py: Removed.
  • Scripts/webkitpy/init/logutils_unittest.py: Removed.
  • Scripts/webkitpy/init/unittests.py: Removed.
  • Scripts/webkitpy/python24/versioning_unittest.py:
  • Scripts/webkitpy/style_references.py:
  • Scripts/webkitpy/unittests.py:
01:46 Changeset [56513] by tony@chromium.org

2010-03-25 Tony Chang <tony@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=33697
Have dragOpFromIEOp("move") return DragOperationGeneric | DragOperationMove
because only returning Generic doesn't work for platforms that don't
have a generic operation.

Covered by existing tests (fast/events/drag-and-drop.html).

  • dom/Clipboard.cpp: (WebCore::dragOpFromIEOp): (WebCore::Clipboard::destinationOperation): (WebCore::Clipboard::setDestinationOperation):
01:42 Changeset [56512] by abarth@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Unreviewed. Move stepsequence to webkitpy.tool.commands.
https://bugs.webkit.org/show_bug.cgi?id=36093

  • Scripts/webkitpy/stepsequence.py: Removed.
  • Scripts/webkitpy/tool/commands/abstractsequencedcommand.py:
  • Scripts/webkitpy/tool/commands/download.py:
  • Scripts/webkitpy/tool/commands/queues.py:
  • Scripts/webkitpy/tool/commands/stepsequence.py: Copied from Scripts/webkitpy/stepsequence.py.
01:38 Changeset [56511] by ukai@chromium.org

2010-03-25 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by Eric Seidel.

[Qt] websocket/tests/long-invalid-header.html fails
https://bugs.webkit.org/show_bug.cgi?id=36492

  • websocket/tests/long-invalid-header-expected.txt: trim console message.

2010-03-25 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by Eric Seidel.

[Qt] websocket/tests/long-invalid-header.html fails
https://bugs.webkit.org/show_bug.cgi?id=36492

trim message at most 128 bytes from the current position.
I believe it's enough data to investigate the error reason and
it makes test not depend on buffered size.

  • websockets/WebSocketHandshake.cpp: (WebCore::trimConsoleMessage): (WebCore::WebSocketHandshake::readServerHandshake): (WebCore::WebSocketHandshake::readHTTPHeaders):
01:34 Changeset [56510] by abarth@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Unreviewed. Create webkitpy.common.checkout as described in
https://bugs.webkit.org/show_bug.cgi?id=36093

  • Scripts/webkitpy/changelogs.py: Removed.
  • Scripts/webkitpy/changelogs_unittest.py: Removed.
  • Scripts/webkitpy/commitinfo.py: Removed.
  • Scripts/webkitpy/commitinfo_unittest.py: Removed.
  • Scripts/webkitpy/common/checkout: Added.
  • Scripts/webkitpy/common/checkout/init.py: Copied from Scripts/webkitpy/common/init.py.
  • Scripts/webkitpy/common/checkout/changelog.py: Copied from Scripts/webkitpy/changelogs.py.
  • Scripts/webkitpy/common/checkout/changelog_unittest.py: Copied from Scripts/webkitpy/changelogs_unittest.py.
  • Scripts/webkitpy/common/checkout/commitinfo.py: Copied from Scripts/webkitpy/commitinfo.py.
  • Scripts/webkitpy/common/checkout/commitinfo_unittest.py: Copied from Scripts/webkitpy/commitinfo_unittest.py.
  • Scripts/webkitpy/common/checkout/diff_parser.py: Copied from Scripts/webkitpy/diff_parser.py.
  • Scripts/webkitpy/common/checkout/diff_parser_unittest.py: Copied from Scripts/webkitpy/diff_parser_unittest.py.
  • Scripts/webkitpy/common/checkout/scm.py: Copied from Scripts/webkitpy/scm.py.
  • Scripts/webkitpy/common/checkout/scm_unittest.py: Copied from Scripts/webkitpy/scm_unittest.py.
  • Scripts/webkitpy/common/net/credentials.py:
  • Scripts/webkitpy/diff_parser.py: Removed.
  • Scripts/webkitpy/diff_parser_unittest.py: Removed.
  • Scripts/webkitpy/layout_tests/port/base.py:
  • Scripts/webkitpy/scm.py: Removed.
  • Scripts/webkitpy/scm_unittest.py: Removed.
  • Scripts/webkitpy/stepsequence.py:
  • Scripts/webkitpy/style_references.py:
  • Scripts/webkitpy/tool/commands/download.py:
  • Scripts/webkitpy/tool/commands/queries.py:
  • Scripts/webkitpy/tool/comments.py:
  • Scripts/webkitpy/tool/main.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/steps/preparechangelogforrevert.py:
  • Scripts/webkitpy/tool/steps/updatechangelogswithreviewer.py:
  • Scripts/webkitpy/tool/steps/validatereviewer.py:
  • Scripts/webkitpy/unittests.py:
  • Scripts/webkitpy/webkitcheckout.py:
01:02 Changeset [56509] by cjerdonek@webkit.org

In webkitpy, moved init/versioning.py to python24/.

Reviewed by Adam Barth.

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

This is part of the master bug to reorganize webkitpy.

  • Scripts/test-webkitpy:
  • Scripts/webkit-patch:
  • Scripts/webkitpy/init/unittests.py:
  • Scripts/webkitpy/init/versioning.py: Removed.
  • Scripts/webkitpy/init/versioning_unittest.py: Removed.
  • Scripts/webkitpy/python24: Added.
  • Scripts/webkitpy/python24/init.py: Copied from WebKitTools/Scripts/webkitpy/tool/init.py.
  • Scripts/webkitpy/python24/versioning.py: Copied from WebKitTools/Scripts/webkitpy/init/versioning.py.
  • Scripts/webkitpy/python24/versioning_unittest.py: Copied from WebKitTools/Scripts/webkitpy/init/versioning_unittest.py.
  • Scripts/webkitpy/unittests.py:
00:52 Changeset [56508] by abarth@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Unreviewed.

Create webkitpy.common.net as described in
https://bugs.webkit.org/show_bug.cgi?id=36093

  • Scripts/webkitpy/bugzilla.py: Removed.
  • Scripts/webkitpy/bugzilla_unittest.py: Removed.
  • Scripts/webkitpy/buildbot.py: Removed.
  • Scripts/webkitpy/buildbot_unittest.py: Removed.
  • Scripts/webkitpy/commitinfo.py:
  • Scripts/webkitpy/common/net: Added.
  • Scripts/webkitpy/common/net/init.py: Added.
  • Scripts/webkitpy/common/net/bugzilla.py: Copied from Scripts/webkitpy/bugzilla.py.
  • Scripts/webkitpy/common/net/bugzilla_unittest.py: Copied from Scripts/webkitpy/bugzilla_unittest.py.
  • Scripts/webkitpy/common/net/buildbot.py: Copied from Scripts/webkitpy/buildbot.py.
  • Scripts/webkitpy/common/net/buildbot_unittest.py: Copied from Scripts/webkitpy/buildbot_unittest.py.
  • Scripts/webkitpy/common/net/credentials.py: Copied from Scripts/webkitpy/credentials.py.
  • Scripts/webkitpy/common/net/credentials_unittest.py: Copied from Scripts/webkitpy/credentials_unittest.py.
  • Scripts/webkitpy/common/net/irc: Copied from Scripts/webkitpy/irc.
  • Scripts/webkitpy/common/net/irc/ircbot.py:
  • Scripts/webkitpy/common/net/irc/ircproxy.py:
  • Scripts/webkitpy/common/net/irc/messagepump_unittest.py:
  • Scripts/webkitpy/common/net/irc/threadedmessagequeue_unittest.py:
  • Scripts/webkitpy/common/net/irc/unittests.py:
  • Scripts/webkitpy/common/net/networktransaction.py: Copied from Scripts/webkitpy/networktransaction.py.
  • Scripts/webkitpy/common/net/networktransaction_unittest.py: Copied from Scripts/webkitpy/networktransaction_unittest.py.
  • Scripts/webkitpy/common/net/statusserver.py: Copied from Scripts/webkitpy/statusserver.py.
  • Scripts/webkitpy/credentials.py: Removed.
  • Scripts/webkitpy/credentials_unittest.py: Removed.
  • Scripts/webkitpy/irc: Removed.
  • Scripts/webkitpy/irc/init.py: Removed.
  • Scripts/webkitpy/irc/ircbot.py: Removed.
  • Scripts/webkitpy/irc/ircproxy.py: Removed.
  • Scripts/webkitpy/irc/messagepump.py: Removed.
  • Scripts/webkitpy/irc/messagepump_unittest.py: Removed.
  • Scripts/webkitpy/irc/threadedmessagequeue.py: Removed.
  • Scripts/webkitpy/irc/threadedmessagequeue_unittest.py: Removed.
  • Scripts/webkitpy/irc/unittests.py: Removed.
  • Scripts/webkitpy/networktransaction.py: Removed.
  • Scripts/webkitpy/networktransaction_unittest.py: Removed.
  • Scripts/webkitpy/scm_unittest.py:
  • Scripts/webkitpy/statusserver.py: Removed.
  • Scripts/webkitpy/tool/bot/queueengine.py:
  • Scripts/webkitpy/tool/commands/download.py:
  • Scripts/webkitpy/tool/commands/queries.py:
  • Scripts/webkitpy/tool/commands/queries_unittest.py:
  • Scripts/webkitpy/tool/commands/queues.py:
  • Scripts/webkitpy/tool/commands/queues_unittest.py:
  • Scripts/webkitpy/tool/commands/queuestest.py:
  • Scripts/webkitpy/tool/commands/upload.py:
  • Scripts/webkitpy/tool/main.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/steps/postdiffforrevert.py:
  • Scripts/webkitpy/unittests.py:
00:37 Changeset [56507] by jhoneycutt@apple.com

<rdar://problem/7780798> Missing plug-ins should be represented by text
only, instead of lego block

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

Reviewed by Dan Bernstein.

WebCore:

  • rendering/RenderEmbeddedObject.cpp:

(WebCore::RenderEmbeddedObject::paintReplaced):
Set the font rendering mode and the computed size, so that the text will
draw properly on Windows.

WebKit/win:

  • WebCoreSupport/WebFrameLoaderClient.cpp:

(WebFrameLoaderClient::createPlugin):
Return 0 if we failed to initialize the plug-in, which causes the new
"missing plug-in" text to draw.

00:34 Changeset [56506] by eric@webkit.org

2010-03-25 Kinuko Yasuda <kinuko@chromium.org>

Reviewed by David Levin.

Rewrite the layout test editing/inserting/6633727 to use dumpAsText
https://bugs.webkit.org/show_bug.cgi?id=31579

  • editing/inserting/6633727-expected.txt: Added.
  • editing/inserting/6633727.html:
  • editing/inserting/script-tests/6633727.js: Added.
  • platform/gtk/editing/inserting/6633727-expected.txt: Removed.
  • platform/mac/editing/inserting/6633727-expected.checksum: Removed.
  • platform/mac/editing/inserting/6633727-expected.png: Removed.
  • platform/mac/editing/inserting/6633727-expected.txt: Removed.
00:24 Changeset [56505] by cjerdonek@webkit.org

In webkitpy, renamed MockBugzillaTool to MockTool.

Reviewed by Adam Barth.

  • Scripts/webkitpy/tool/commands/commandtest.py:
  • Scripts/webkitpy/tool/commands/queries_unittest.py:
  • Scripts/webkitpy/tool/commands/queues_unittest.py:
  • Scripts/webkitpy/tool/commands/queuestest.py:
  • Scripts/webkitpy/tool/commands/sheriffbot_unittest.py:
  • Scripts/webkitpy/tool/commands/upload_unittest.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/steps/closebugforlanddiff_unittest.py:
  • Scripts/webkitpy/tool/steps/steps_unittest.py:
  • Scripts/webkitpy/tool/steps/updatechangelogswithreview_unittests.py:
00:18 Changeset [56504] by cjerdonek@webkit.org

In Scripts/webkitpy, moved webkitport.py and committers.py into
common/config/ (also creating common/config/).

Reviewed by Adam Barth.

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

This is part of the master bug to reorganize webkitpy.

  • Scripts/validate-committer-lists:
  • Scripts/webkitpy/bugzilla.py:
  • Scripts/webkitpy/bugzilla_unittest.py:
  • Scripts/webkitpy/changelogs.py:
  • Scripts/webkitpy/commitinfo.py:
  • Scripts/webkitpy/commitinfo_unittest.py:
  • Scripts/webkitpy/committers.py: Removed.
  • Scripts/webkitpy/committers_unittest.py: Removed.
  • Scripts/webkitpy/common: Added.
  • Scripts/webkitpy/common/init.py: Copied from WebKitTools/Scripts/webkitpy/style/init.py.
  • Scripts/webkitpy/common/config: Added.
  • Scripts/webkitpy/common/config/init.py: Copied from WebKitTools/Scripts/webkitpy/style/init.py.
  • Scripts/webkitpy/common/config/committers.py: Copied from WebKitTools/Scripts/webkitpy/committers.py.
  • Scripts/webkitpy/common/config/committers_unittest.py: Copied from WebKitTools/Scripts/webkitpy/committers_unittest.py.
  • Scripts/webkitpy/common/config/ports.py: Copied from WebKitTools/Scripts/webkitpy/webkitport.py.
  • Scripts/webkitpy/common/config/ports_unittest.py: Copied from WebKitTools/Scripts/webkitpy/webkitport_unittest.py.
  • Scripts/webkitpy/mock_bugzillatool.py:
  • Scripts/webkitpy/tool/commands/early_warning_system.py:
  • Scripts/webkitpy/tool/commands/queries.py:
  • Scripts/webkitpy/tool/commands/sheriffbot.py:
  • Scripts/webkitpy/tool/commands/upload.py:
  • Scripts/webkitpy/tool/steps/abstractstep.py:
  • Scripts/webkitpy/unittests.py:
  • Scripts/webkitpy/webkitport.py: Removed.
  • Scripts/webkitpy/webkitport_unittest.py: Removed.
00:09 Changeset [56503] by abarth@webkit.org

2010-03-25 Adam Barth <abarth@webkit.org>

Unreviewed. Move mock_bugzillatool.py to tool/mocktool.py.

  • Scripts/webkitpy/mock_bugzillatool.py: Removed.
  • Scripts/webkitpy/tool/commands/commandtest.py:
  • Scripts/webkitpy/tool/commands/queries_unittest.py:
  • Scripts/webkitpy/tool/commands/queues_unittest.py:
  • Scripts/webkitpy/tool/commands/queuestest.py:
  • Scripts/webkitpy/tool/commands/sheriffbot_unittest.py:
  • Scripts/webkitpy/tool/commands/upload_unittest.py:
  • Scripts/webkitpy/tool/mocktool.py: Copied from Scripts/webkitpy/mock_bugzillatool.py.
  • Scripts/webkitpy/tool/steps/closebugforlanddiff_unittest.py:
  • Scripts/webkitpy/tool/steps/steps_unittest.py:
  • Scripts/webkitpy/tool/steps/updatechangelogswithreview_unittests.py:
00:07 Changeset [56502] by eric@webkit.org

2010-03-24 Vangelis Kokkevis <vangelis@chromium.org>

Reviewed by Dimitri Glazkov.

Removing unecessary redraws of LayerSkia contents:
https://bugs.webkit.org/show_bug.cgi?id=36470

Test: No new functionality added, no tests.

  • platform/graphics/skia/LayerRendererSkia.cpp: (WebCore::LayerRendererSkia::drawLayerInCanvasRecursive): Debug borders now rendered at composite time rather than update time.
  • platform/graphics/skia/LayerSkia.cpp: (WebCore::LayerSkia::updateContents): (WebCore::LayerSkia::drawDebugBorder): Debug border drawing removed from updateContents and split into a separate method that can be called at composite time. (WebCore::LayerSkia::setBounds): Resetting the bounds causes the layer to redraw its contents. (WebCore::LayerSkia::setNeedsDisplay):
  • platform/graphics/skia/LayerSkia.h: Added declaration for drawDebugBorders method.

2010-03-24 Vangelis Kokkevis <vangelis@chromium.org>

Reviewed by Dimitri Glazkov.

Eliminate unecessary redraws of GraphicsLayer contents when doing accelerated compositing:
https://bugs.webkit.org/show_bug.cgi?id=36470

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::paint): Remove call that forced redrawing the contents of the entire layer hierarchy.

03/24/10:

23:59 Changeset [56501] by abarth@webkit.org

2010-03-24 Adam Barth <abarth@webkit.org>

Unreviewed. Move comments.py and multicommandtool.py to their new
home.

  • Scripts/webkitpy/comments.py: Removed.
  • Scripts/webkitpy/multicommandtool.py: Removed.
  • Scripts/webkitpy/multicommandtool_unittest.py: Removed.
  • Scripts/webkitpy/tool/commands/abstractsequencedcommand.py:
  • Scripts/webkitpy/tool/commands/download.py:
  • Scripts/webkitpy/tool/commands/openbugs.py:
  • Scripts/webkitpy/tool/commands/queries.py:
  • Scripts/webkitpy/tool/commands/queues.py:
  • Scripts/webkitpy/tool/commands/upload.py:
  • Scripts/webkitpy/tool/comments.py: Copied from Scripts/webkitpy/comments.py.
  • Scripts/webkitpy/tool/main.py:
  • Scripts/webkitpy/tool/multicommandtool.py: Copied from Scripts/webkitpy/multicommandtool.py.
  • Scripts/webkitpy/tool/multicommandtool_unittest.py: Copied from Scripts/webkitpy/multicommandtool_unittest.py.
  • Scripts/webkitpy/tool/steps/closebugforlanddiff.py:
  • Scripts/webkitpy/tool/steps/closepatch.py:
  • Scripts/webkitpy/tool/steps/reopenbugafterrollout.py:
  • Scripts/webkitpy/unittests.py:
23:47 Changeset [56500] by abarth@webkit.org

2010-03-24 Adam Barth <abarth@webkit.org>

Unreviewed. Fixup one import statement I didn't find because the .pyc
masked the error.

  • Scripts/webkitpy/stepsequence.py:
23:44 Changeset [56499] by abarth@webkit.org

2010-03-24 Adam Barth <abarth@webkit.org>

Unreviewed. Move queueengine.py to its new home.

  • Scripts/webkitpy/queueengine.py: Removed.
  • Scripts/webkitpy/queueengine_unittest.py: Removed.
  • Scripts/webkitpy/tool/bot/queueengine.py: Copied from Scripts/webkitpy/queueengine.py.
  • Scripts/webkitpy/tool/bot/queueengine_unittest.py: Copied from Scripts/webkitpy/queueengine_unittest.py.
  • Scripts/webkitpy/tool/commands/early_warning_system.py:
  • Scripts/webkitpy/tool/commands/queues.py:
  • Scripts/webkitpy/unittests.py:
23:40 Changeset [56498] by abarth@webkit.org

2010-03-24 Adam Barth <abarth@webkit.org>

Unreviewed. Move patchcollection.py to its new home.

  • Scripts/webkitpy/patchcollection.py: Removed.
  • Scripts/webkitpy/patchcollection_unittest.py: Removed.
  • Scripts/webkitpy/tool/bot: Added.
  • Scripts/webkitpy/tool/bot/init.py: Added.
  • Scripts/webkitpy/tool/bot/patchcollection.py: Copied from WebKitTools/Scripts/webkitpy/patchcollection.py.
  • Scripts/webkitpy/tool/bot/patchcollection_unittest.py: Copied from WebKitTools/Scripts/webkitpy/patchcollection_unittest.py.
  • Scripts/webkitpy/tool/commands/queues.py:
  • Scripts/webkitpy/unittests.py:
23:27 Changeset [56497] by cjerdonek@webkit.org

In Scripts/webkitpy, moved steps_references.py and the steps
folder into webkitpy/patch.

Reviewed by Adam Barth.

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

  • Scripts/webkitpy/steps: Removed.
  • Scripts/webkitpy/steps/init.py: Removed.
  • Scripts/webkitpy/steps/abstractstep.py: Removed.
  • Scripts/webkitpy/steps/applypatch.py: Removed.
  • Scripts/webkitpy/steps/applypatchwithlocalcommit.py: Removed.
  • Scripts/webkitpy/steps/build.py: Removed.
  • Scripts/webkitpy/steps/checkstyle.py: Removed.
  • Scripts/webkitpy/steps/cleanworkingdirectory.py: Removed.
  • Scripts/webkitpy/steps/cleanworkingdirectorywithlocalcommits.py: Removed.
  • Scripts/webkitpy/steps/closebug.py: Removed.
  • Scripts/webkitpy/steps/closebugforlanddiff.py: Removed.
  • Scripts/webkitpy/steps/closebugforlanddiff_unittest.py: Removed.
  • Scripts/webkitpy/steps/closepatch.py: Removed.
  • Scripts/webkitpy/steps/commit.py: Removed.
  • Scripts/webkitpy/steps/confirmdiff.py: Removed.
  • Scripts/webkitpy/steps/createbug.py: Removed.
  • Scripts/webkitpy/steps/editchangelog.py: Removed.
  • Scripts/webkitpy/steps/ensurebuildersaregreen.py: Removed.
  • Scripts/webkitpy/steps/ensurelocalcommitifneeded.py: Removed.
  • Scripts/webkitpy/steps/metastep.py: Removed.
  • Scripts/webkitpy/steps/obsoletepatches.py: Removed.
  • Scripts/webkitpy/steps/options.py: Removed.
  • Scripts/webkitpy/steps/postdiff.py: Removed.
  • Scripts/webkitpy/steps/postdiffforcommit.py: Removed.
  • Scripts/webkitpy/steps/postdiffforrevert.py: Removed.
  • Scripts/webkitpy/steps/preparechangelog.py: Removed.
  • Scripts/webkitpy/steps/preparechangelogforrevert.py: Removed.
  • Scripts/webkitpy/steps/promptforbugortitle.py: Removed.
  • Scripts/webkitpy/steps/reopenbugafterrollout.py: Removed.
  • Scripts/webkitpy/steps/revertrevision.py: Removed.
  • Scripts/webkitpy/steps/runtests.py: Removed.
  • Scripts/webkitpy/steps/steps_unittest.py: Removed.
  • Scripts/webkitpy/steps/update.py: Removed.
  • Scripts/webkitpy/steps/updatechangelogswithreview_unittests.py: Removed.
  • Scripts/webkitpy/steps/updatechangelogswithreviewer.py: Removed.
  • Scripts/webkitpy/steps/validatereviewer.py: Removed.
  • Scripts/webkitpy/steps_references.py: Removed.
  • Scripts/webkitpy/stepsequence.py:
  • Scripts/webkitpy/tool/commands/download.py:
  • Scripts/webkitpy/tool/commands/upload.py:
  • Scripts/webkitpy/tool/steps: Copied from WebKitTools/Scripts/webkitpy/steps.
  • Scripts/webkitpy/tool/steps/init.py:
  • Scripts/webkitpy/tool/steps/applypatch.py:
  • Scripts/webkitpy/tool/steps/applypatchwithlocalcommit.py:
  • Scripts/webkitpy/tool/steps/build.py:
  • Scripts/webkitpy/tool/steps/checkstyle.py:
  • Scripts/webkitpy/tool/steps/cleanworkingdirectory.py:
  • Scripts/webkitpy/tool/steps/cleanworkingdirectorywithlocalcommits.py:
  • Scripts/webkitpy/tool/steps/closebug.py:
  • Scripts/webkitpy/tool/steps/closebugforlanddiff.py:
  • Scripts/webkitpy/tool/steps/closebugforlanddiff_unittest.py:
  • Scripts/webkitpy/tool/steps/closepatch.py:
  • Scripts/webkitpy/tool/steps/commit.py:
  • Scripts/webkitpy/tool/steps/confirmdiff.py:
  • Scripts/webkitpy/tool/steps/createbug.py:
  • Scripts/webkitpy/tool/steps/editchangelog.py:
  • Scripts/webkitpy/tool/steps/ensurebuildersaregreen.py:
  • Scripts/webkitpy/tool/steps/ensurelocalcommitifneeded.py:
  • Scripts/webkitpy/tool/steps/metastep.py:
  • Scripts/webkitpy/tool/steps/obsoletepatches.py:
  • Scripts/webkitpy/tool/steps/postdiff.py:
  • Scripts/webkitpy/tool/steps/postdiffforcommit.py:
  • Scripts/webkitpy/tool/steps/postdiffforrevert.py:
  • Scripts/webkitpy/tool/steps/preparechangelog.py:
  • Scripts/webkitpy/tool/steps/preparechangelogforrevert.py:
  • Scripts/webkitpy/tool/steps/promptforbugortitle.py:
  • Scripts/webkitpy/tool/steps/reopenbugafterrollout.py:
  • Scripts/webkitpy/tool/steps/revertrevision.py:
  • Scripts/webkitpy/tool/steps/runtests.py:
  • Scripts/webkitpy/tool/steps/steps_unittest.py:
  • Scripts/webkitpy/tool/steps/update.py:
  • Scripts/webkitpy/tool/steps/updatechangelogswithreview_unittests.py:
  • Scripts/webkitpy/tool/steps/updatechangelogswithreviewer.py:
  • Scripts/webkitpy/tool/steps/validatereviewer.py:
  • Scripts/webkitpy/tool/steps_references.py: Copied from WebKitTools/Scripts/webkitpy/steps_references.py.
  • Scripts/webkitpy/unittests.py:
23:22 Changeset [56496] by eric@webkit.org

2010-03-24 Eric Seidel <eric@webkit.org>

Unreviewed, build fix only.

Abstract LayoutTestResults logic for easier reuse
https://bugs.webkit.org/show_bug.cgi?id=36579

  • Scripts/webkitpy/buildbot_unittest.py: Add a missing import.
23:15 Changeset [56495] by abarth@webkit.org

2010-03-24 Adam Barth <abarth@webkit.org>

Unreviewed. Fix a stray comma to test landing an patch without review.

  • Scripts/webkitpy/commitinfo.py:
23:10 Changeset [56494] by cjerdonek@webkit.org

In Scripts/webkitpy, moved commands_references.py and the commands
folder into webkitpy/patch.

Reviewed by Adam Barth.

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

This is part of master bug 36093 to organize webkitpy.

  • Scripts/webkitpy/commands: Removed.
  • Scripts/webkitpy/commands/init.py: Removed.
  • Scripts/webkitpy/commands/abstractsequencedcommand.py: Removed.
  • Scripts/webkitpy/commands/commandtest.py: Removed.
  • Scripts/webkitpy/commands/download.py: Removed.
  • Scripts/webkitpy/commands/download_unittest.py: Removed.
  • Scripts/webkitpy/commands/early_warning_system.py: Removed.
  • Scripts/webkitpy/commands/early_warning_system_unittest.py: Removed.
  • Scripts/webkitpy/commands/openbugs.py: Removed.
  • Scripts/webkitpy/commands/openbugs_unittest.py: Removed.
  • Scripts/webkitpy/commands/queries.py: Removed.
  • Scripts/webkitpy/commands/queries_unittest.py: Removed.
  • Scripts/webkitpy/commands/queues.py: Removed.
  • Scripts/webkitpy/commands/queues_unittest.py: Removed.
  • Scripts/webkitpy/commands/queuestest.py: Removed.
  • Scripts/webkitpy/commands/sheriffbot.py: Removed.
  • Scripts/webkitpy/commands/sheriffbot_unittest.py: Removed.
  • Scripts/webkitpy/commands/upload.py: Removed.
  • Scripts/webkitpy/commands/upload_unittest.py: Removed.
  • Scripts/webkitpy/commands_references.py: Removed.
  • Scripts/webkitpy/tool/commands: Copied from WebKitTools/Scripts/webkitpy/commands.
  • Scripts/webkitpy/tool/commands/commandtest.py:
  • Scripts/webkitpy/tool/commands/download.py:
  • Scripts/webkitpy/tool/commands/download_unittest.py:
  • Scripts/webkitpy/tool/commands/early_warning_system.py:
  • Scripts/webkitpy/tool/commands/early_warning_system_unittest.py:
  • Scripts/webkitpy/tool/commands/openbugs_unittest.py:
  • Scripts/webkitpy/tool/commands/queries_unittest.py:
  • Scripts/webkitpy/tool/commands/queues_unittest.py:
  • Scripts/webkitpy/tool/commands/queuestest.py:
  • Scripts/webkitpy/tool/commands/sheriffbot.py:
  • Scripts/webkitpy/tool/commands/sheriffbot_unittest.py:
  • Scripts/webkitpy/tool/commands/upload.py:
  • Scripts/webkitpy/tool/commands/upload_unittest.py:
  • Scripts/webkitpy/tool/commands_references.py: Copied from WebKitTools/Scripts/webkitpy/commands_references.py.
  • Scripts/webkitpy/tool/main.py:
  • Scripts/webkitpy/unittests.py:
23:07 Changeset [56493] by mrowe@apple.com

Build fix after r56474.

  • Plugins/Hosted/NetscapePluginHostProxy.mm:

(WKPCRunSyncOpenPanel):

23:07 Changeset [56492] by mrowe@apple.com

Revert the portion of r56489 that dealt with port zero as it introduced some test failures.

WebCore:

  • platform/KURL.cpp:

(WebCore::KURL::port): Use the "ok" argument to charactersToUIntStrict to determine whether
it was able to successfully parse the string as an unsigned integer, rather than relying on
the fact it returned zero when it failed.

LayoutTests:

  • platform/mac/security/block-test-expected.txt:
  • security/block-test.html:
22:51 Changeset [56491] by abarth@webkit.org

2010-03-24 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Rename webkitpy.patch to webkitpy.tool
https://bugs.webkit.org/show_bug.cgi?id=36580

This is in preparation for the great webkitpy naming cleanup.

  • Scripts/webkit-patch:
  • Scripts/webkitpy/patch: Removed.
  • Scripts/webkitpy/patch/init.py: Removed.
  • Scripts/webkitpy/patch/patcher.py: Removed.
  • Scripts/webkitpy/tool: Copied from WebKitTools/Scripts/webkitpy/patch.
  • Scripts/webkitpy/tool/main.py: Copied from WebKitTools/Scripts/webkitpy/patch/patcher.py.
  • Scripts/webkitpy/tool/patcher.py: Removed.
22:36 Changeset [56490] by barraclough@apple.com
  • platform/text/String.cpp:

(WebCore::putUTF8Triple): Correct capitalization of function name.
(WebCore::String::utf8): Fix C-style casts.
(WebCore::String::fromUTF8): Remove unnecessary newline.
(WebCore::String::fromUTF8WithLatin1Fallback): Use early return.

Reviewed by Sam Weinig.

22:26 Changeset [56489] by mrowe@apple.com

WebKit should treat port numbers outside the valid range as being blacklisted
<http://webkit.org/b/36571> / <rdar://problem/7790908>

Reviewed by Darin Adler.

WebCore:

  • platform/KURL.cpp:

(WebCore::KURL::port): Map invalid port numbers to invalidPortNumber.
(WebCore::portAllowed): Add invalidPortNumber to the blacklist.

  • platform/KURLGoogle.cpp: invalid port numbers to invalidPortNumber.

(WebCore::KURL::port): Add invalidPortNumber to the blacklist.
Also bring this in to sync with KURL. Having this identical code in two places is stupid.

LayoutTests:

  • platform/mac/security/block-test-expected.txt:
  • security/block-test.html: Add tests of a few invalid port numbers.
22:25 Changeset [56488] by eric@webkit.org

2010-03-24 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Abstract LayoutTestResults logic for easier reuse
https://bugs.webkit.org/show_bug.cgi?id=36579

  • Scripts/webkitpy/buildbot.py:
    • Split out logic into new LayoutTestResults class.
  • Scripts/webkitpy/buildbot_unittest.py:
    • Rename the testing class to match.
  • Scripts/webkitpy/commands/queries.py:
    • Use the new LayoutTestResults class.
22:14 Changeset [56487] by eric@webkit.org

2010-03-24 Eric Seidel <eric@webkit.org>

Reviewed by Eric Seidel.

Move Bugzilla.prompt_for_component to User.prompt_with_list for re-use
https://bugs.webkit.org/show_bug.cgi?id=36577

  • Scripts/webkitpy/bugzilla.py:
    • Move prompt_for_component to User
  • Scripts/webkitpy/commands/queries.py:
    • Add a missing argument_names declaration.
  • Scripts/webkitpy/user.py:
    • Add prompt_with_list
    • Make staticmethods classmethods for easier mocking
21:58 Changeset [56486] by eric@webkit.org

2010-03-24 MORITA Hajime <morrita@google.com>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=36517
svg/dom/frame-related-api-during-load.html shoud be more self-descriptive.

  • svg/dom/frame-related-api-during-load-expected.txt:
  • svg/dom/frame-related-api-during-load.html:
21:28 Changeset [56485] by ddkilzer@apple.com

Fix bug URL.

21:25 Changeset [56484] by ddkilzer@apple.com

<htp://webkit.org/b/36572> commit-log-editor: thinks mergeChangeLogs.pl is a ChangeLog file

Reviewed by Eric Seidel.

Fixes the following error when committing a file with
"ChangeLog" in the name that isn't a ChangeLog (like
mergeChangeLogs.pl from r56471 and r56472):

$ git commit .
Can't open WebKitTools/Scripts/webkitperl/VCSUtils_unittest/mergeChangeLog at commit-log-editor line 132.
error: There was a problem with the editor 'commit-log-editor'.
Please supply the message using either -m or -F option.

  • Scripts/commit-log-editor: Added '$' to anchor "ChangeLog" to

the end of the file name when searching for ChangeLog files in a
commit.

21:15 Changeset [56483] by ddkilzer@apple.com

<http://webkit.org/b/36570> resolve-ChangeLogs: fall back to git-merge-file if ChangeLog can't be merged

Reviewed by Eric Seidel.

  • Scripts/resolve-ChangeLogs: Switched to exec git-merge-file if

the merge attempt fails.

21:15 Changeset [56482] by barraclough@apple.com

Switch String::latin1, String::utf8, String::fromUTF8 to
use WTF's Unicode conversion methods rather than TextEncoder.
These methods only perform simple conversion, and don't need
really require TextEncoder's full capability (to look up arbitrary
encodings by name), switching to only be dependent on WTF will
make it easier if we chose to move WebCore::String to WTF.

Reviewed by Sam Weinig.

21:08 Changeset [56481] by scherkus@chromium.org

2010-03-24 Andrew Scherkus <scherkus@chromium.org>

Reviewed by Eric Carlson, Eric Seidel and David Levin.

Update a few more media layout tests to choose test clips based on codec support.

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

  • media/audio-constructor-src.html: Switched to use findMediaFile().
  • media/audio-play-event-expected.txt: Updated results.
  • media/audio-play-event.html: Switched to use findMediaFile().
  • media/video-no-audio.html: Ditto.
  • media/video-source-add-src.html: Ditto.
  • media/video-src-change.html: Closed p tag.
  • platform/mac-leopard/media/video-no-audio-expected.checksum: New results due to changing source video.
  • platform/mac-leopard/media/video-no-audio-expected.png: Ditto.
  • platform/mac-leopard/media/video-no-audio-expected.txt: Ditto.
  • platform/mac/media/video-no-audio-expected.checksum: Ditto.
  • platform/mac/media/video-no-audio-expected.png: Ditto.
  • platform/mac/media/video-no-audio-expected.txt: Ditto.
21:07 Changeset [56480] by barraclough@apple.com

Switch String::latin1, String::utf8, String::fromUTF8 to
use WTF's Unicode conversion methods rather than TextEncoder.
These methods only perform simple conversion, and don't need
really require TextEncoder's full capability (to look up arbitrary
encodings by name), switching to only be dependent on WTF will
make it easier if we chose to move WebCore::String to WTF.

Reviewed by Sam Weinig.

20:56 Changeset [56479] by weinig@apple.com

Add some missing exports.

Reviewed by Mark Rowe.

  • WebCore.base.exp:
20:45 Changeset [56478] by abarth@webkit.org

2010-03-24 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

webkit-patch or pre-commit hook should validate reviewer lines before committing
https://bugs.webkit.org/show_bug.cgi?id=26927

Validate that patches have valid reivewers listed in their ChangeLogs
before landing. For patches without reviewers can be landed if their
ChangeLogs state that they are unreviewed.

  • Scripts/webkitpy/changelogs.py:
  • Scripts/webkitpy/commands/download.py:
  • Scripts/webkitpy/commitinfo.py:
  • Scripts/webkitpy/commitinfo_unittest.py:
  • Scripts/webkitpy/mock_bugzillatool.py:
  • Scripts/webkitpy/steps/init.py:
  • Scripts/webkitpy/steps/validatereviewer.py: Added.
20:43 Changeset [56477] by barraclough@apple.com

Switch String::latin1, String::utf8, String::fromUTF8 to
use WTF's Unicode conversion methods rather than TextEncoder.
These methods only perform simple conversion, and don't need
really require TextEncoder's full capability (to look up arbitrary
encodings by name), switching to only be dependent on WTF will
make it easier if we chose to move WebCore::String to WTF.

Reviewed by Sam Weinig.

  • platform/text/String.cpp:

(WebCore::String::latin1):
(WebCore::putUTF8triple):
(WebCore::String::utf8):
(WebCore::String::fromUTF8):

18:10 Changeset [56476] by eric@webkit.org

Restore peace and justice to the universe.

18:06 Changeset [56475] by zecke@webkit.org

Undefined names reported by pyflakes in python scripts

ttps://bugs.webkit.org/show_bug.cgi?id=36403

Attempt to use names that exist or can exist in the lexical
scope instead of not being available at all.

  • Scripts/webkitpy/changelogs.py: Use self._content.
  • Scripts/webkitpy/layout_tests/port/base.py: Use os.stat
  • Scripts/webkitpy/layout_tests/test_types/fuzzy_image_diff.py: Use self._port
  • Scripts/webkitpy/style/processors/cpp_unittest.py: Use expected_message_re
17:52 Changeset [56474] by ap@apple.com

Reviewed by Anders Carlsson.

https://bugs.webkit.org/show_bug.cgi?id=36563
A plug-in makes Safari crash on http://www.itscodingtime.com/

  • Plugins/Hosted/NetscapePluginHostProxy.h: (WebKit::NetscapePluginHostProxy::port): Assert that the object is still alive. This isn't beautifully systemic, but helped catch a bug, and may help catch more. (WebKit::NetscapePluginHostProxy::clientPort): Ditto. (WebKit::NetscapePluginHostProxy::isProcessingRequests): Changed m_processingRequests to a static. This doesn't change behavior much, but helps avoid writing into deallocated memory.
  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WebKit::NetscapePluginHostProxy::NetscapePluginHostProxy): Changed m_processingRequests to a static. (WebKit::NetscapePluginHostProxy::processRequests): Ditto. Changing m_processingRequests after destroying the object in pluginHostDied() was wrong, but reasonably harmless, as there wasn't much time for some other object to be allocated at this address. (WKPCEvaluate): Refetch host proxy, as it may have been destroyed. (WKPCInvoke): Ditto. (WKPCInvokeDefault): Ditto. (WKPCGetProperty): Ditto. (WKPCSetProperty): Ditto. (WKPCRemoveProperty): Ditto. (WKPCHasProperty): Ditto. (WKPCHasMethod): Ditto. (WKPCEnumerate): Ditto. (WKPCRunSyncOpenPanel): Ditto.
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::processRequestsAndWaitForReply): Bail out of the "event loop" if host proxy went away while processing a request.
17:02 Changeset [56473] by eric@webkit.org

2010-03-24 Jesus Sanchez-Palencia <jesus.palencia@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

Add the FrameFlatteningEnabled WebAttribute to QWebSettings.

[Qt] Missing QWebSettings for Frame Flattening
https://bugs.webkit.org/show_bug.cgi?id=36553

  • Api/qwebsettings.cpp: (QWebSettingsPrivate::apply): (QWebSettings::QWebSettings):
  • Api/qwebsettings.h:
16:50 Changeset [56472] by ddkilzer@apple.com

<http://webkit.org/b/36560> resolve-ChangeLogs: git-rebase fails when resolve-ChangeLogs can't merge

Reviewed by Eric Seidel.

When resolve-ChangeLogs fails to merge a patch while running as
a git merge driver, it deletes the original file, which causes
an internal failure and stops git mid-merge:

fatal: Failed to execute internal merge

The fix is to use the --force switch with patch so that it will
always attempt to apply the patch. (The change in
mergeChangeLogs() for the previous commit also fixed this, but
adding --force also prevents any potential user interaction that
patch may want to display.)

  • Scripts/VCSUtils.pm:

(mergeChangeLogs): Added --force switch to patch command. Also
changed to use the exit status from the patch command to
determine the return value for this method.

  • Scripts/webkitperl/VCSUtils_unittest/mergeChangeLogs.pl: Added

test to cover this bug.

16:50 Changeset [56471] by ddkilzer@apple.com

<http://webkit.org/b/36560> resolve-ChangeLogs: move mergeChanges() into VCSUtils package

Reviewed by Eric Seidel.

  • Scripts/VCSUtils.pm:

(mergeChangeLogs): Copied from mergeChanges() in
resolve-ChangeLogs and renamed. Added method documentation.
Fixed bug found by new tests where the original file to be
patched was deleted when cleaning up after a traditinal rejected
patch failed to apply.

  • Scripts/resolve-ChangeLogs: Switched to using

mergeChangeLogs().
(mergeChanges): Moved to VCSUtils.pm and renamed to
mergeChangeLogs().

  • Scripts/webkitperl/VCSUtils_unittest/mergeChangeLogs.pl: Added.
16:36 Changeset [56470] by eric@webkit.org

2010-03-24 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Add support for qt's unassigned list to webkit-patch assign-to-committer
https://bugs.webkit.org/show_bug.cgi?id=36559

  • Scripts/webkitpy/bugzilla.py:
    • Move Bugzilla.unassigned_email into Bug and make it a set.
  • Scripts/webkitpy/bugzilla_unittest.py:
    • Test the new Bug.is_unassigned method
  • Scripts/webkitpy/commands/upload.py:
    • Use the new Bug.is_unassigned method instead of an explicit ==
  • Scripts/webkitpy/mock_bugzillatool.py:
    • Bugzilla.unassigned_email no longer needs mocking
16:30 Changeset [56469] by dimich@chromium.org

2010-03-24 Dmitry Titov <dimich@chromium.org>

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

In Chromium port, it broke invalid-image-data-standalone.html
invalid-image-data.html multipart-wait-before-boundary.html
stop-crash.html win-boundary-crash.html

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::finishedLoadingDocument):

2010-03-24 Dmitry Titov <dimich@chromium.org>

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

In Chromium port, it broke invalid-image-data-standalone.html
invalid-image-data.html multipart-wait-before-boundary.html
stop-crash.html win-boundary-crash.html

  • src/ResourceHandle.cpp: (WebCore::ResourceHandleInternal::ResourceHandleInternal): (WebCore::ResourceHandleInternal::start): (WebCore::ResourceHandleInternal::cancel): (WebCore::ResourceHandleInternal::didReceiveResponse): (WebCore::ResourceHandleInternal::didReceiveData): (WebCore::ResourceHandleInternal::didFinishLoading): (WebCore::ResourceHandleInternal::didFail):
16:21 Changeset [56468] by ap@apple.com

Reviewed by Geoff Garen.

  • wtf/FastMalloc.h: Added a using directive for fastMallocSize, like we do for all public WTF symbols. Also sorted the list alphabetically.
16:18 Changeset [56467] by eric@webkit.org

2010-03-24 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Significant scroll speed degradation if Timeline has
an expanded record with significant number of children.
https://bugs.webkit.org/show_bug.cgi?id=36543

  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel.prototype._updateBoundaries): (WebInspector.TimelinePanel.prototype._addToRecordsWindow): (WebInspector.TimelinePanel.prototype._filterRecords): (WebInspector.TimelinePanel.prototype._refreshRecords):
16:08 Changeset [56466] by kov@webkit.org

2010-03-24 Yuri Chornoivan <yurchor@ukr.net>

Reviewed by Gustavo Noronha.

Ukrainian translation.

  • uk.po: Added.
16:02 Changeset [56465] by kov@webkit.org

2010-03-24 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Eric Carlson.

[GStreamer] Fails to go to Loaded state because of floating point discrepancies
https://bugs.webkit.org/show_bug.cgi?id=35891

Use the m_fillStatus variable, which is more reliable, instead of
using the calculation, for the completed case.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::fillTimerFired):
16:00 Changeset [56464] by dimich@chromium.org

Unreviewed, Chromium test fix.

Recent r56445 added CustomEvent and a test but it didn't work on Chromium bots.
Test crashed due to infinte recursion because the compiler did not have the right definition of toV8(CustomEvent*)
and was substituting toV8(Event*).

Fixing test fast/events/custom-event.html

  • bindings/v8/custom/V8EventCustom.cpp:
15:45 Changeset [56463] by eric@webkit.org

2010-03-24 MORITA Hajime <morrita@google.com>

Reviewed by Darin Adler.

Refactoring: HTMLFormControlElement should not have redundant null check
https://bugs.webkit.org/show_bug.cgi?id=36487

No new tests as there is no new functionality.

  • html/HTMLFormControlElement.cpp: (WebCore::HTMLFormControlElement::dispatchFocusEvent): (WebCore::HTMLFormControlElement::dispatchBlurEvent):
15:22 Changeset [56462] by eric@webkit.org

2010-03-24 Eric Uhrhane <ericu@chromium.org>

DOMCoreException needs NoStaticTables modifier
https://bugs.webkit.org/show_bug.cgi?id=36458

No new tests, but this shows up in tests of Database accesses from the
Worker thread that I'll be checking in soon. The symptom is that
DOMCoreException.toString() returns [object DOMException] instead of
something more helpful.

  • fast/workers/worker-context-multi-port-expected.txt: Adjusted expectation that was depending on the broken behavior.

2010-03-24 Eric Uhrhane <ericu@chromium.org>

Reviewed by Alexey Proskuryakov.

DOMCoreException needs NoStaticTables modifier
https://bugs.webkit.org/show_bug.cgi?id=36458

No new tests, but this shows up in tests of Database accesses from the
Worker thread that I'll be checking in soon. The symptom is that
DOMCoreException.toString() returns [object DOMException] instead of
something more helpful.

  • dom/DOMCoreException.idl: Added NoStaticTables.
15:05 Changeset [56461] by eric@webkit.org

2010-03-24 Drew Wilson <atwilson@chromium.org>

Reviewed by Dmitry Titov.

Failed assertion in V8Proxy::setDOMException() if worker is shutting down.
https://bugs.webkit.org/show_bug.cgi?id=36514

Test: none (existing worker tests suffice)

  • bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::setDOMException): Updated setDOMException() to check for an empty exception (due to no active context) and avoid throwing in that case.
14:37 Changeset [56460] by evan@chromium.org

2010-03-23 Evan Martin <evan@chromium.org>

Reviewed by Dmitry Titov.

[chromium] use integral glyph widths
https://bugs.webkit.org/show_bug.cgi?id=36510

Extend a Linux Chromium layout test to also cover slight-hinted text.

  • platform/chromium/fast/text/chromium-linux-fontconfig-renderstyle.html:

2010-03-23 Evan Martin <evan@chromium.org>

Reviewed by Dmitry Titov.

[chromium] use integral glyph widths
https://bugs.webkit.org/show_bug.cgi?id=36510

Despite WebKit (and Skia, to an extent) supporting non-integral
glyph widths, the font code path we hit in Skia only supports
integral glyph positions. This means that we would accumulate
offsets when drawing a sequence up non-integer-width glyphs
which would cause gaps when snapped to the pixel grid when drawing.

  • platform/graphics/chromium/SimpleFontDataLinux.cpp: (WebCore::SimpleFontData::platformWidthForGlyph): round glyph widths to integers.
14:16 Changeset [56459] by dino@apple.com

2010-03-24 Dean Jackson <dino@apple.com>

Reviewed by Kevin Decker.

<rdar://problem/7785305>
Fix regression caused by r55576. It turned out that ensuring
CoreAnimation always gets a correct fill parameter caused flashing
as non-filled animations ended (CA briefly showed the initial animation
value before the style system set the final value).

  • platform/graphics/mac/GraphicsLayerCA.mm: (WebCore::GraphicsLayerCA::setupAnimation):
14:12 Changeset [56458] by eric@webkit.org

2010-03-24 Viatcheslav Ostapenko <ostapenko.viatcheslav@nokia.com>

Reviewed by Laszlo Gombos.

Auto-uppercase and predictive text need to be disabled for S60 (as for maemo)
https://bugs.webkit.org/show_bug.cgi?id=33176

  • WebCoreSupport/EditorClientQt.cpp:
14:08 Changeset [56457] by jam@chromium.org

2010-03-24 John Abd-El-Malek <jam@chromium.org>

Reviewed by Darin Fisher.

[chromium]WebKit side of adding search support to Pepper.
https://bugs.webkit.org/show_bug.cgi?id=36434

  • WebKit.gyp:
  • public/WebDocument.h:
  • public/WebNode.h: (WebKit::WebNode::to): (WebKit::WebNode::toConst):
  • public/WebPluginDocument.h: Added. (WebKit::WebPluginDocument::WebPluginDocument): (WebKit::WebPluginDocument::operator=): (WebKit::WebPluginDocument::assign):
  • src/WebDocument.cpp: (WebKit::WebDocument::isPluginDocument):
  • src/WebPluginDocument.cpp: Added. (WebKit::WebPluginDocument::plugin): (WebKit::WebPluginDocument::WebPluginDocument): (WebKit::WebPluginDocument::operator=): (WebKit::WebPluginDocument::operator PassRefPtr<PluginDocument>):
13:55 Changeset [56456] by eric@webkit.org

2010-03-24 Kent Hansen <kent.hansen@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Rename QWebSettings::XSSAuditorEnabled to XSSAuditingEnabled
https://bugs.webkit.org/show_bug.cgi?id=36522

For consistency with other QWebSettings attributes.

  • Api/qwebsettings.cpp: (QWebSettingsPrivate::apply):
  • Api/qwebsettings.h:

2010-03-24 Kent Hansen <kent.hansen@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Rename QWebSettings::XSSAuditorEnabled to XSSAuditingEnabled
https://bugs.webkit.org/show_bug.cgi?id=36522

  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::setXSSAuditorEnabled): Use the new name.
13:50 Changeset [56455] by weinig@apple.com

Rubber-stamped by Anders Carlsson..

Update Security Group Members list.

  • security/security-group-members.html:
13:38 Changeset [56454] by eric@webkit.org

2010-03-24 Hayato Ito <hayato@chromium.org>

Reviewed by NOBODY (Need!).

Refactor computePageRects so that Mac can make use of it.
https://bugs.webkit.org/show_bug.cgi?id=36159

Refactoring only, so no new tests.

  • WebCore.base.exp:
  • page/PrintContext.cpp: (WebCore::PrintContext::computePageRects): (WebCore::PrintContext::computePageRectsWithPageSize): (WebCore::PrintContext::computePageRectsWithPageSizeInternal): (WebCore::PrintContext::pageNumberForElement): (WebCore::PrintContext::numberOfPages):
  • page/PrintContext.h:

2010-03-24 Hayato Ito <hayato@chromium.org>

Reviewed by Shinichiro Hamaji.

Refactor computePageRects so that Mac can make use of it.
https://bugs.webkit.org/show_bug.cgi?id=36159

Refactoring only, so no new tests.

  • WebView/WebFrame.mm: (-[WebFrame _computePageRectsWithPrintWidthScaleFactor:printHeight:]):
13:20 Changeset [56453] by eric@webkit.org

2010-03-24 Jeremy Moskovich <jeremy@chromium.org>

Reviewed by Jeremy Orlow.

Add some diagnostics to try to track down cause of crash in ArchiveFactory::isArchiveMimeType().

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

No new tests as there is no new functionality.

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::finishedLoadingDocument): Make copy of mimeType string to isolate crash.

2010-03-24 Jeremy Moskovich <jeremy@chromium.org>

Reviewed by Jeremy Orlow.

Add some diagnostics to try to track down cause of crash in ArchiveFactory::isArchiveMimeType().

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

  • src/ResourceHandle.cpp: Track state across ResourceHandle invocations. (WebCore::ResourceHandleInternal::ResourceHandleInternal): (WebCore::ResourceHandleInternal::): (WebCore::ResourceHandleInternal::start): (WebCore::ResourceHandleInternal::cancel): (WebCore::ResourceHandleInternal::didReceiveResponse): (WebCore::ResourceHandleInternal::didReceiveData): (WebCore::ResourceHandleInternal::didFinishLoading): (WebCore::ResourceHandleInternal::didFail):
13:02 Changeset [56452] by eric@webkit.org

2010-03-24 Anton Muhin <antonm@chromium.org>

Reviewed by Nate Chapin.

Use newly introduced SetPrototype method to deal with global objects.
https://bugs.webkit.org/show_bug.cgi?id=36497

No new tests, should be covered by the current test infrastructure.

  • bindings/v8/V8DOMWindowShell.cpp: (WebCore::V8DOMWindowShell::installDOMWindow):
  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::getConstructor):
  • bindings/v8/WorkerContextExecutionProxy.cpp: (WebCore::WorkerContextExecutionProxy::initContextIfNeeded):
12:23 Changeset [56451] by dimich@chromium.org

Unreviewed, Chromium build fix.

  • platform/chromium/PopupMenuChromium.cpp:

(WebCore::PopupContainer::PopupContainer): Fix order of initializers.
(WebCore::PopupContainer::layout): Remove unused variable.

12:05 Changeset [56450] by kenneth@webkit.org

WebCore: Add a way to check if the page client is making use of
a QWidget.

Reviewed by Simon Hausmann.

  • platform/qt/QWebPageClient.h:

(QWebPageClient::isQWidgetClient):

WebKit/qt: Calling setView on a QWebPage being shown by a QGraphicsWebView,
would uninstall the page client, resulting in not expected
behaviour.

Reviewed by Simon Hausmann.

  • Api/qwebpage.cpp:

(QWebPageWidgetClient::QWebPageWidgetClient):
(QWebPageWidgetClient::isQWidgetClient):
(QWebPageWidgetClient::screenNumber):
(QWebPage::QWebPage):
(QWebPage::setView):

11:41 Changeset [56449] by eric@webkit.org

2010-03-24 Jay Campan <jcampan@google.com>

Reviewed by Darin Fisher.

Making Chromium select popups not steal activation from the browser.
Select popups are now like autocomplete popups, shown in non-activated
windows.
https://bugs.webkit.org/show_bug.cgi?id=36062

  • page/chromium/ChromeClientChromium.h:
  • platform/chromium/PopupMenuChromium.cpp: (WebCore::): (WebCore::PopupListBox::setMaxHeight): (WebCore::PopupListBox::disconnectClient): (WebCore::PopupListBox::PopupListBox): (WebCore::PopupContainer::create): (WebCore::PopupContainer::PopupContainer): (WebCore::PopupContainer::showPopup): (WebCore::PopupContainer::showExternal): (WebCore::PopupContainer::hidePopup): (WebCore::PopupContainer::layout): (WebCore::PopupContainer::chromeClientChromium): (WebCore::PopupListBox::hidePopup): (WebCore::PopupListBox::layout): (WebCore::PopupMenu::~PopupMenu): (WebCore::PopupMenu::show):
  • platform/chromium/PopupMenuChromium.h: (WebCore::PopupItem::): (WebCore::PopupItem::PopupItem): (WebCore::PopupContainerSettings::): (WebCore::PopupContainer::): (WebCore::PopupContainer::listBox): (WebCore::PopupContainer::popupType):

2010-03-24 Jay Campan <jcampan@google.com>

Reviewed by Darin Fisher.

Making Chromium select popups not steal activation from the browser.
Select popups are now like autocomplete popups, shown in non-activated
windows.
https://bugs.webkit.org/show_bug.cgi?id=36062

  • public/WebViewClient.h: (WebKit::WebViewClient::createPopupMenu):
  • src/ChromeClientImpl.cpp: (WebKit::ChromeClientImpl::popupOpened): (WebKit::ChromeClientImpl::popupClosed):
  • src/ChromeClientImpl.h:
  • src/WebViewImpl.cpp: (WebKit::): (WebKit::WebViewImpl::mouseDown): (WebKit::WebViewImpl::keyEvent): (WebKit::WebViewImpl::selectPopupHandleKeyEvent): (WebKit::WebViewImpl::hideSelectPopup): (WebKit::WebViewImpl::popupOpened): (WebKit::WebViewImpl::popupClosed): (WebKit::WebViewImpl::setFocus): (WebKit::WebViewImpl::applyAutoFillSuggestions): (WebKit::WebViewImpl::applyAutocompleteSuggestions):
  • src/WebViewImpl.h:
11:11 Changeset [56448] by apavlov@chromium.org

2010-03-24 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: [Chromium] Audits never complete
https://bugs.webkit.org/show_bug.cgi?id=36544

  • src/js/InspectorControllerImpl.js: (devtools.InspectorBackendImpl):
10:50 Changeset [56447] by levin@chromium.org

Need to move items that CanvasRenderingContext2D depends on into CanvasSurface.
https://bugs.webkit.org/show_bug.cgi?id=35453

Reviewed by Oliver Hunt.

Prepartory changes to allow for an OffscreenCanvas which may be used in a worker
or outside of the DOM.

No change in functionality, so new tests.

  • dom/CanvasSurface.cpp: Moved functionality that CanvasRenderingContext2D depends on

into this class (and removed dependencies on document/html element).
(WebCore::CanvasSurface::CanvasSurface):
(WebCore::CanvasSurface::~CanvasSurface): Put the desctructor in the cpp file
to avoid needing access to ~ImageBuffer in the header file.
(WebCore::CanvasSurface::setSurfaceSize): Does basic items needed
when the size changes. It is protected to force outside callers to go
through HTMLCanvasElement::setSize.
(WebCore::CanvasSurface::toDataURL): Just moved from HTMLCanvasElement and
made a note about a method to fix for worker usage.
(WebCore::CanvasSurface::willDraw): Made this virtual to allow an overide
which uses the renderbox and tracks a dirtyRect.
(WebCore::CanvasSurface::convertLogicalToDevice): Moved and changed to
rely on a member variable for page scale (to avoid using the document).
(WebCore::CanvasSurface::createImageBuffer):
(WebCore::CanvasSurface::drawingContext): Simple move from HTMLCanvasElement.
(WebCore::CanvasSurface::buffer): Ditto.
(WebCore::CanvasSurface::baseTransform): Ditto.

  • dom/CanvasSurface.h:

(WebCore::CanvasSurface::width): Simple move from HTMLCanvasElement.
(WebCore::CanvasSurface::height): Ditto.
(WebCore::CanvasSurface::size): Ditto.
(WebCore::CanvasSurface::setOriginTainted): Ditto.
(WebCore::CanvasSurface::originClean): Ditto.
(WebCore::CanvasSurface::hasCreatedImageBuffer): Ditto (with small name change).

  • html/HTMLCanvasElement.cpp:

(WebCore::HTMLCanvasElement::HTMLCanvasElement): Pass in the scale factor to CanvasSurface
so it doesn't need the document.
(WebCore::HTMLCanvasElement::willDraw): Moved the relevant portion to CanvasSurface.
(WebCore::HTMLCanvasElement::reset): Small changes due to refactoring.
(WebCore::HTMLCanvasElement::paint): Ditto.

  • html/HTMLCanvasElement.h:

(WebCore::HTMLCanvasElement::setSize): Ditto.

  • platform/MIMETypeRegistry.cpp:

(WebCore::MIMETypeRegistry::isSupportedImageMIMETypeForEncoding): Added assert
to verify that this is only called on the main thread.

  • platform/graphics/Image.cpp:

(WebCore::Image::nullImage): Ditto.

  • platform/graphics/cg/ImageBufferCG.cpp:

(WebCore::utiFromMIMEType): Ditto.

10:21 Changeset [56446] by apavlov@chromium.org

2010-03-24 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Inspector: XML Tags should not be converted to lowercase
https://bugs.webkit.org/show_bug.cgi?id=28600

LayoutTests:

  • inspector/elements-panel-xhtml-structure-expected.txt:
  • inspector/elements-panel-xhtml-structure.xhtml:

WebCore:

  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.updateBreadcrumb): (WebInspector.ElementsPanel.prototype.decorateNodeLabel):
  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeOutline.prototype.nodeNameToCorrectCase): (WebInspector.ElementsTreeElement.prototype._updateChildren): (WebInspector.ElementsTreeElement.prototype._nodeTitleInfo):
10:15 Changeset [56445] by kenneth@webkit.org

WebCore: Add support for DOM Level 3 Custom Event
http://www.w3.org/TR/DOM-Level-3-Events

Reviewed by Antti Koivisto.

Test: fast/events/custom-event.html

  • DerivedSources.cpp:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pri:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSEventCustom.cpp:

(WebCore::toJS):

  • bindings/scripts/CodeGeneratorJS.pm:
  • bindings/scripts/CodeGeneratorV8.pm:
  • dom/CustomEvent.cpp: Added.

(WebCore::CustomEvent::CustomEvent):
(WebCore::CustomEvent::initCustomEvent):
(WebCore::CustomEvent::isCustomEvent):

  • dom/CustomEvent.h: Added.

(WebCore::CustomEvent::create):
(WebCore::CustomEvent::detail):

  • dom/Document.cpp:

(WebCore::Document::createEvent):

  • dom/Event.cpp:

(WebCore::Event::isCustomEvent):

  • dom/Event.h:

LayoutTests: Layout test for DOM Level 3 Custom Event
http://www.w3.org/TR/DOM-Level-3-Events

Reviewed by Antti Koivisto.

  • fast/events/custom-event-expected.txt: Added.
  • fast/events/custom-event.html: Added.
09:56 Changeset [56444] by apavlov@chromium.org

2010-03-24 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Error when highlighting JavaScript with an invalid mime type
https://bugs.webkit.org/show_bug.cgi?id=36530

  • inspector/front-end/SourceTokenizer.js: (WebInspector.SourceTokenizer.Registry):
  • inspector/front-end/SourceView.js: (WebInspector.SourceView.prototype._contentLoaded): (WebInspector.SourceView.prototype._canonicalMimeType):
  • inspector/front-end/TextEditorHighlighter.js: (WebInspector.TextEditorHighlighter):
09:33 QtWebKitBackportingFixes edited by hausmann@webkit.org
(diff)
09:28 QtWebKitRelease20 edited by hausmann@webkit.org
(diff)
09:25 Changeset [56443] by pfeldman@chromium.org

2010-03-24 Pavel Feldman <pfeldman@chromium.org>

Not reviewed. Chromium test fix: added a frameCount check to the
debugger function call.

  • bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::createUtilityContext):
08:47 Changeset [56442] by apavlov@chromium.org

2010-03-24 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

[Chromium] Fix VKEY_F10, VKEY_F11 translation in WebInputEvent.cpp
https://bugs.webkit.org/show_bug.cgi?id=36524

  • src/WebInputEvent.cpp: (WebKit::staticKeyIdentifiers):
08:18 Porting Macros plan edited by efidler@rim.com
(diff)
07:11 Changeset [56441] by ossy@webkit.org

Rubber-stamped by Simon Hausmann and Gustavo Noronha Silva.

[Qt][GTK] http/tests/xmlhttprequest/xmlhttprequest-onProgress-open-should-zero-length.html fails after r56394
https://bugs.webkit.org/show_bug.cgi?id=36531

  • platform/gtk/Skipped: http/tests/xmlhttprequest/xmlhttprequest-onProgress-open-should-zero-length.html skipped until fix.
  • platform/qt/Skipped: http/tests/xmlhttprequest/xmlhttprequest-onProgress-open-should-zero-length.html skipped until fix.
06:25 QtWebKitBugs edited by vestbo@webkit.org
(diff)
06:21 ImprovingLifeOnWindows edited by aroben@apple.com
Removed section about detecting a missing Platform SDK. We now do this as … (diff)
04:07 Changeset [56440] by hausmann@webkit.org

[Qt] Support for QT_LIBINFIX in Symbian builds

Patch by Miikka Heikkinen <miikka.heikkinen@digia.com> on 2010-03-19
Reviewed by Simon Hausmann.

Configuring Qt with -qtlibinfix parameter will enable installing
an alternate version of Qt on devices that already have it on ROM.
This patch provides support for infixed builds of Webkit.

  • WebCore.pro:
02:36 Changeset [56439] by tkent@chromium.org

2010-03-24 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Make Icon::createIconForFiles() optional.
https://bugs.webkit.org/show_bug.cgi?id=35072

r54923 made Icon::createIconForFiles() deprecated. However moving
existing icon loading code to outside of WebCore is not good. So,
we assume:

  • ChromeClient::chooseIconForFiles(), renamed from iconForFiles(), is the primary API to load icons.
  • Icon::createIconForFiles() is an optional API to help implementing ChromeClient::iconForFiles().

This patch removes a call to Icon::createIconForFiles() from
FileChooser::loadIcon(), and ChromeClient::chooseIconForFiles() of
non-Chromium ports calls Icon::createIconForFiles().

  • WebCore.base.exp: Export FileChooser::iconLoaded(), Icon::createIconForFiles(), and Icon::~Icon().
  • WebCore.xcodeproj/project.pbxproj: Export Icon.h
  • loader/EmptyClient.h: Rename iconForFiles() to chooseIconForFiles().
  • page/Chrome.cpp: ditto.
  • page/Chrome.h: ditto.
  • page/ChromeClient.h: ditto.
  • platform/FileChooser.cpp: (WebCore::FileChooser::loadIcon):
  • platform/graphics/Icon.h: Remove a comment on createIconForFiles().
  • platform/graphics/chromium/IconChromiumLinux.cpp: Remove createIconForFiles().
  • platform/graphics/chromium/IconChromiumMac.cpp: ditto.
  • platform/graphics/chromium/IconChromiumWin.cpp: ditto.
  • rendering/RenderFileUploadControl.cpp: Rename iconForFiles() to chooseIconForFiles().
  • rendering/RenderFileUploadControl.h: ditto.

2010-03-24 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Make Icon::createIconForFiles() optional.
https://bugs.webkit.org/show_bug.cgi?id=35072

  • Rename iconForFiles() to chooseIconForFiles().
  • Call Icon::createIconForFiles() from chooseIconForFiles().
  • src/ChromeClientImpl.cpp: (WebKit::ChromeClientImpl::chooseIconForFiles):
  • src/ChromeClientImpl.h:

2010-03-24 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Make Icon::createIconForFiles() optional.
https://bugs.webkit.org/show_bug.cgi?id=35072

  • Rename iconForFiles() to chooseIconForFiles().
  • Call Icon::createIconForFiles() from chooseIconForFiles().
  • WebCoreSupport/ChromeClientGtk.cpp:
  • WebCoreSupport/ChromeClientGtk.h:

2010-03-24 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Make Icon::createIconForFiles() optional.
https://bugs.webkit.org/show_bug.cgi?id=35072

  • Rename iconForFiles() to chooseIconForFiles().
  • Call Icon::createIconForFiles() from chooseIconForFiles().
  • WebCoreSupport/ChromeClientHaiku.cpp: (WebCore::ChromeClientHaiku::chooseIconForFiles):
  • WebCoreSupport/ChromeClientHaiku.h:

2010-03-24 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Make Icon::createIconForFiles() optional.
https://bugs.webkit.org/show_bug.cgi?id=35072

  • Rename iconForFiles() to chooseIconForFiles().
  • Call Icon::createIconForFiles() from chooseIconForFiles().
  • WebCoreSupport/WebChromeClient.h:
  • WebCoreSupport/WebChromeClient.mm: (WebChromeClient::chooseIconForFiles):

2010-03-24 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Make Icon::createIconForFiles() optional.
https://bugs.webkit.org/show_bug.cgi?id=35072

  • Rename iconForFiles() to chooseIconForFiles().
  • Call Icon::createIconForFiles() from chooseIconForFiles().
  • WebCoreSupport/ChromeClientQt.cpp: (WebCore::ChromeClientQt::chooseIconForFiles):
  • WebCoreSupport/ChromeClientQt.h:

2010-03-24 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Make Icon::createIconForFiles() optional.
https://bugs.webkit.org/show_bug.cgi?id=35072

  • Rename iconForFiles() to chooseIconForFiles().
  • Call Icon::createIconForFiles() from chooseIconForFiles().
  • WebCoreSupport/WebChromeClient.cpp: (WebChromeClient::chooseIconForFiles):
  • WebCoreSupport/WebChromeClient.h:

2010-03-24 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Make Icon::createIconForFiles() optional.
https://bugs.webkit.org/show_bug.cgi?id=35072

  • Rename iconForFiles() to chooseIconForFiles().
  • Call Icon::createIconForFiles() from chooseIconForFiles().
  • WebKitSupport/ChromeClientWx.cpp: (WebCore::ChromeClientWx::chooseIconForFiles):
  • WebKitSupport/ChromeClientWx.h:
01:43 QtWebKitFeaturePlanning edited by henry.haverinen@nokia.com
(diff)
01:36 QtWebKitFeaturePlanning created by henry.haverinen@nokia.com
01:35 Changeset [56438] by barraclough@apple.com

Bug 36519 - JSGlobalContextRelease is unnecessarily slow

Reviewed by Oliver Hunt.

Since [ http://trac.webkit.org/changeset/35917 ], calling
JSGlobalContextRelease always triggers a GC heap collection
(if not a full destroy). As per 35917's changelog "This is
only really necessary when the (JSGlobalObject's) last
reference is released, but there is no way to determine that,
and no harm in collecting slightly more often."

Well, we now know of cases of API clients who are harmed by
the performance penalty of collecting too often, so it's time
to add a way to determine whether a call to JSGlobalContextRelease
is removing the last protect from it's global object. If further
protects are retaining the global object (likely from other
JSGlobalContextRefs), then don't trigger a GC collection.

  • API/JSContextRef.cpp:
  • runtime/Collector.cpp:

(JSC::Heap::unprotect): return a boolean indicating that the value is now unprotected.

  • runtime/Collector.h:
  • wtf/HashCountedSet.h:

(WTF::::remove): return a boolean indicating whether the value was removed from the set.

00:50 Changeset [56437] by cjerdonek@webkit.org

Added to check-webkit-style support for a --debug flag.

Reviewed by Shinichiro Hamaji.

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

The --debug flag adjusts the logging level to DEBUG and
includes the logger name and level in each log message.

  • Scripts/check-webkit-style:
    • Changed the code to check for the --debug flag and pass the result to the configure_logging() method.
  • Scripts/webkitpy/style/checker.py:
    • Added an is_debug parameter to configure_logging().
    • Refactored configure_logging() by adding calls to the following two methods: _create_log_handlers() and _create_debug_log_handlers().
  • Scripts/webkitpy/style/checker_unittest.py:
    • Added unit tests for configure_logging() with is_debug True by splitting the ConfigureLoggingTest class into ConfigureLoggingTest and ConfigureLoggingTestBase, and adding ConfigureLoggingDebugTest.
  • Scripts/webkitpy/style/optparser.py:
    • Updated the usage string.
    • Added an is_debug data attribute to the CommandOptionValues class.
    • Added support for the --debug flag to the ArgumentParser.parse() method.
    • Also added extra error information to the parse() method in the case of an invalid flag.
  • Scripts/webkitpy/style/optparser_unittest.py:
    • Updated the unit tests as necessary.
    • Also fixed an issue with the CommandOptionValuesTest.test_eq() unit test.
00:46 Changeset [56436] by barraclough@apple.com

Bug 36519 - JSGlobalContextRelease is unnecessarily slow

Reviewed by Oliver Hunt.

Since [ http://trac.webkit.org/changeset/35917 ], calling
JSGlobalContextRelease always triggers a GC heap collection
(if not a full destroy). As per 35917's changelog "This is
only really necessary when the (JSGlobalObject's) last
reference is released, but there is no way to determine that,
and no harm in collecting slightly more often."

Well, we now know of cases of API clients who are harmed by
the performance penalty of collecting too often, so it's time
to add a way to determine whether a call to JSGlobalContextRelease
is removing the last protect from it's global object. If further
protects are retaining the global object (likely from other
JSGlobalContextRefs), then don't trigger a GC collection.

  • API/JSContextRef.cpp:
  • runtime/Collector.cpp:

(JSC::Heap::unprotect): return a boolean indicating that the value is now unprotected.

  • runtime/Collector.h:
  • wtf/HashCountedSet.h:

(WTF::::remove): return a boolean indicating whether the value was removed from the set.

00:42 QtWebKit edited by henry.haverinen@nokia.com
(diff)
00:11 Changeset [56435] by barraclough@apple.com

Bug 36519 - JSGlobalContextRelease is unnecessarily slow

Reviewed by Oliver Hunt.

Since [ http://trac.webkit.org/changeset/35917 ], calling
JSGlobalContextRelease always triggers a GC heap collection
(if not a full destroy). As per 35917's changelog "This is
only really necessary when the (JSGlobalObject's) last
reference is released, but there is no way to determine that,
and no harm in collecting slightly more often."

Well, we now know of cases of API clients who are harmed by
the performance penalty of collecting too often, so it's time
to add a way to determine whether a call to JSGlobalContextRelease
is removing the last protect from it's global object. If further
protects are retaining the global object (likely from other
JSGlobalContextRefs), then don't trigger a GC collection.

  • API/JSContextRef.cpp:
  • runtime/Collector.cpp:

(JSC::Heap::unprotect): return a boolean indicating that the value is now unprotected.

  • runtime/Collector.h:
  • wtf/HashCountedSet.h:

(WTF::::remove): return a boolean indicating whether the value was removed from the set.

03/23/10:

22:55 Changeset [56434] by tkent@chromium.org

2010-03-23 Kent Tamura <tkent@chromium.org>

Reviewed by Dimitri Glazkov.

[DRT/Chromium] Add TestNavigationController and TestWebWorker
https://bugs.webkit.org/show_bug.cgi?id=36489

Add TestNavigationController and TestWebWorker classes, which are
going to be used by DumpRenderTree Chromium port. These files are
based on:

  • src/webkit/tools/test_shell/test_navigation_controller.{cc,h}
  • src/webkit/tools/test_shell/test_web_worker.h of Chromium rev.40492.

TestNavigationController has non-style changes.

  • Change ContentState type: binary string -> WebHistoryItem
  • Remove TestShell dependency by introducing NavigationHost interface.
  • DumpRenderTree/chromium/TestNavigationController.cpp: Added.
  • DumpRenderTree/chromium/TestNavigationController.h: Added.
  • DumpRenderTree/chromium/TestWebWorker.h: Added.
21:27 Changeset [56433] by ap@apple.com

Skip plugins/destroy-plugin-from-callback.html, which has been failing on SL bots since it
was added in r56338.

  • platform/mac-snowleopard/Skipped:
20:17 Changeset [56432] by mrowe@apple.com

Build fix.

  • runtime/ArrayPrototype.cpp:

(JSC::arrayProtoFuncSplice): Some versions of GCC emit a warning about the implicit 64- to 32-bit truncation
that takes place here. An explicit cast is sufficient to silence it.

17:31 Changeset [56431] by mitz@apple.com

Reverted accidental change from r56429.

  • WebCoreSupport/WebContextMenuClient.mm:

(WebContextMenuClient::getCustomMenuFromDefaultItems):

17:29 Changeset [56430] by mitz@apple.com

Fixed typos.

  • manual-tests/plugin-visible-rect-change.html:
17:27 Changeset [56429] by mitz@apple.com

WebCore: WebCore part of
<rdar://problem/7197736> Plug-in clip rect does not update when overflow
clip changes
https://bugs.webkit.org/show_bug.cgi?id=36479.

Reviewed by John Sullivan.

  • manual-tests/plugin-visible-rect-change.html: Added.
  • platform/mac/WidgetMac.mm:

(WebCore::WidgetPrivate::WidgetPrivate): Added previousVisibleRect.
(WebCore::Widget::setFrameRect): If the visible rect changed but the
frame rect did not, send a -visibleRectDidChange message to the view,
if it responds to it.
(WebCore::Widget::releasePlatformWidget): Reset previousVisibleRect.

  • rendering/RenderWidget.cpp:

(WebCore::RenderWidget::setWidgetGeometry): Track changes to the clip rect
imposed by the enclosing layer. Call Widget::setFrameRect when it changes,
even if the frame rect did not.

  • rendering/RenderWidget.h:

(WebCore::RenderWidget::windowClipRect): Added this accessor.

WebKit/mac: WebKit part of
<rdar://problem/7197736> Plug-in clip rect does not update when overflow
clip changes
https://bugs.webkit.org/show_bug.cgi?id=36479.

Reviewed by John Sullivan.

  • Plugins/Hosted/WebHostedNetscapePluginView.mm:

(-[WebHostedNetscapePluginView visibleRectDidChange]): Added. Calls
WKSyncSurfaceToView().

  • Plugins/WebBaseNetscapePluginView.h:
  • Plugins/WebBaseNetscapePluginView.mm:

(-[WebBaseNetscapePluginView _windowClipRect]): Changed to use Widget::windowClipRect().
(-[WebBaseNetscapePluginView visibleRectDidChange]): Added. Invokes -renewGState.

WebKitLibraries: WebKitSystemInterface part of
<rdar://problem/7197736> Plug-in clip rect does not update when overflow
clip changes
https://bugs.webkit.org/show_bug.cgi?id=36479.

Reviewed by John Sullivan.

  • WebKitSystemInterface.h: Added WKSyncSurfaceToView().
  • libWebKitSystemInterfaceSnowLeopard.a: Updated
17:06 Changeset [56428] by simon.fraser@apple.com

2010-03-23 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

Assertion ASSERTION FAILED: rootLayer == m_clipRectsRoot at hulu.com
https://bugs.webkit.org/show_bug.cgi?id=34065

Fix another cause of assertions related to the clip rects root. Embeds
with zero size caused the overlap testing to fail, yet we require them to work
for correct compositing, so when the composited bounds is empty, use a 1x1 rect
for overlap testing.

Test: compositing/geometry/empty-embed-rects.html

  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::addToOverlapMap): (WebCore::RenderLayerCompositor::computeCompositingRequirements):
16:31 Changeset [56427] by ap@apple.com

Build fix.

  • runtime/ArrayPrototype.cpp: (JSC::arrayProtoFuncSplice): Fixed a typo - length doesn't need to be converted with toInteger().
16:28 Changeset [56426] by kbr@google.com

2010-03-23 Kenneth Russell <kbr@google.com>

Unreviewed build fix.

Add #include <stdio.h> to fix Chromium Linux build.

  • src/WebGraphicsContext3DDefaultImpl.cpp:
16:12 Changeset [56425] by ap@apple.com

Reviewed by Geoff Garen.

https://bugs.webkit.org/show_bug.cgi?id=36511
<rdar://problem/7753498> Safari freezes when using SPUTNIK JavaScript conformance check

Test: fast/js/sputnik-S15.4.4.12_A3_T3.html

  • runtime/ArrayPrototype.cpp: (JSC::arrayProtoFuncSplice): We were incorrectly computing the start offset, and iterated over (almost) all integers. Note that this can be fixed without using doubles, but the code would be much more complicated, and there is no important reason to stick to integers here.
15:57 Changeset [56424] by eric@webkit.org

2010-03-23 Luiz Agostini <luiz.agostini@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

QGraphicsWebView crash when calling setView on the QWebPage...
https://bugs.webkit.org/show_bug.cgi?id=36436

Checking for pageClient existance before showing popups.

  • WebCoreSupport/QtFallbackWebPopup.cpp: (WebCore::QtFallbackWebPopup::show):
15:40 Changeset [56423] by eric@webkit.org

2010-03-23 Anders Bakken <anders.bakken@nokia.com>, Jesus Sanchez-Palencia <jesus.palencia@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

Check if q->scene() is available before actually using it
on QGraphicsWebViewPrivate.

QGraphicsWebView crash
https://bugs.webkit.org/show_bug.cgi?id=32670

  • Api/qgraphicswebview.cpp: (QGraphicsWebViewPrivate::screenNumber): (QGraphicsWebViewPrivate::ownerWidget):
15:27 Changeset [56422] by levin@chromium.org

[chromium] XMLHttpRequest.send sends 'undefined' string when passed undefined value.
https://bugs.webkit.org/show_bug.cgi?id=36506

Reviewed by Nate Chapin.

WebCore:

Test: http/tests/xmlhttprequest/send-undefined-and-null.html

  • bindings/v8/custom/V8XMLHttpRequestCustom.cpp:

(WebCore::isDocumentType): Unrelated style fix: fixed the casing of the method.
(WebCore::V8XMLHttpRequest::sendCallback): Added check for null/undefined to do the send.
This mirrors what is done in the JSC bindings. Previously, sending 'null' worked because
the last case, which did "toWebCoreStringWithNullCheck", resulted in sending an empty
request body.

LayoutTests:

Added a test to verify that sending either undefined and null both result in an empty request body.

  • http/tests/xmlhttprequest/send-undefined-and-null-expected.txt: Added.
  • http/tests/xmlhttprequest/send-undefined-and-null.html: Added.
15:24 Changeset [56421] by eric@webkit.org

2010-03-23 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Add support for revision -> build lookup in buildbot.py and layout test result parsing
https://bugs.webkit.org/show_bug.cgi?id=36474

  • Scripts/webkitpy/bugzilla_unittest.py: Added a FIXME about sharing code.
  • Scripts/webkitpy/buildbot.py:
    • Add support for looking up builds by revision number.
    • Add support for fetching and parsing results.html files from buildbot.
    • build_for_revision has an allow_failed_lookups option to work around the fact that our buildbot's xmlrpc calls return failure on old revision numbers.
    • Add parsing support for twisted directory listings.
  • Scripts/webkitpy/buildbot_unittest.py:
    • Unit test all the new code.
  • Scripts/webkitpy/commands/queries.py:
    • Add a new results-for command which prints all the results for a given revision (very slow due to slow revision lookup)
15:07 Changeset [56420] by dglazkov@chromium.org

WebCore: Fix hard-to-see crash due to incorrect libxml API usage.

Patch by Chris Evans <cevans@chromium.org> on 2010-03-23
Reviewed by Darin Adler.

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

Test: fast/text/bad-encoding.html

  • dom/XMLTokenizerLibxml2.cpp:

(WebCore::XMLTokenizer::doEnd): Avoid operations on a closed context.

LayoutTests: Add test for crash due to calling into libxml in invalid state. Note
that it is unlikely that this will actually crash for you in the test
environment because it is highly intermittent. However, adding test in
order to exercise the code path and prevent related regression.
See https://bugs.webkit.org/show_bug.cgi?id=36000

Patch by Chris Evans <cevans@chromium.org> on 2010-03-23
Reviewed by Darin Adler.

  • fast/text/bad-encoding.html: Added.
  • fast/text/bad-encoding-expected.txt: Added.
  • fast/text/resources/bad-encoding.xml: Added.
14:47 Changeset [56419] by dbates@webkit.org

2010-03-23 Daniel Bates <dbates@rim.com>

Reviewed by Adam Roben.

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

Detect if the Windows Platform SDK is missing when building with
Visual C++ Express Edition and inform the user to download it.

  • Scripts/webkitdirs.pm:
14:38 Changeset [56418] by japhet@chromium.org

2010-03-23 Nate Chapin <japhet@chromium.org>

Unreviewed, Chromium test expectations update.

Mark fast/dom/error-to-string-stack-overflow.html as failing on
all Chromium platforms, and http/tests/xmlhttprequest/xmlhttprequest-50ms-download-dispatch.html
as failing on Windows.

  • platform/chromium/test_expectations.txt:
14:33 Changeset [56417] by darin@apple.com

Tell Subversion about more directories that expect to have .pyc files.

  • Scripts/webkitpy: Modified property svn:ignore.
  • Scripts/webkitpy/irc: Added property svn:ignore.
14:24 Changeset [56416] by japhet@chromium.org

2010-03-23 Nate Chapin <japhet@chromium.org>

Unreviewed, revert r56376.

This revision introduced a crash in a couple of layout tests
on Chromium Linux.

  • page/chromium/ChromeClientChromium.h:
  • platform/chromium/PopupMenuChromium.cpp: (WebCore::): (WebCore::PopupContainer::create): (WebCore::PopupContainer::PopupContainer): (WebCore::PopupContainer::showPopup): (WebCore::PopupContainer::showExternal): (WebCore::PopupContainer::hidePopup): (WebCore::PopupMenu::show):
  • platform/chromium/PopupMenuChromium.h: (WebCore::PopupItem::): (WebCore::PopupItem::PopupItem): (WebCore::PopupContainerSettings::): (WebCore::PopupContainer::listBox):
14:15 Changeset [56415] by eric@webkit.org

2010-03-23 Sergey Ulanov <sergeyu@chromium.org>

Reviewed by Dmitry Titov.

Changes needed to implement Show/Hide Controls command for <video> in
chrome: (1) added Controls action in WebMediaPlayerAction that toggles
controls for media player, (2) added MediaHasVideo in
WebContextMenuData so that controls can be made toggleable only for
video player but not for audio.
https://bugs.webkit.org/show_bug.cgi?id=36460

  • public/WebContextMenuData.h: (WebKit::WebContextMenuData::):
  • public/WebMediaPlayerAction.h: (WebKit::WebMediaPlayerAction::):
  • src/ContextMenuClientImpl.cpp: (WebKit::ContextMenuClientImpl::getCustomMenuFromDefaultItems):
  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::performMediaPlayerAction):
13:52 Changeset [56414] by hyatt@apple.com

https://bugs.webkit.org/show_bug.cgi?id=36272, make sure nth-child can work when the portion with "n" is omitted.

Reviewed by Simon Fraser and Darin Adler.

Added fast/css/nth-child-implied-step.html

  • css/CSSGrammar.y:
  • css/tokenizer.flex:
13:51 Changeset [56413] by dbates@webkit.org

2010-03-23 Daniel Bates <dbates@rim.com>

Reviewed by Adam Barth.

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

Import the GNU readline interface to modify the behavior
of raw_input so as to provide line editing support. In
particular this will prevent "delete" characters from
appearing in the returned value for function raw_input.

  • Scripts/webkitpy/user.py:
13:48 Changeset [56412] by darin@apple.com

2010-03-23 Darin Adler <darin@apple.com>

Reviewed by Sam Weinig.

Reduce and straighten internal use of DOMImplementation
https://bugs.webkit.org/show_bug.cgi?id=36501

  • WebCore.xcodeproj/project.pbxproj: Xcode decided to re-sort this file.
  • dom/Clipboard.cpp: Removed unneeded include of DOMImplementation.h.
  • dom/DOMImplementation.cpp: Removed unneeded createDocument and createHTMLDocument functions. These should be done directly instead of involving the DOMImplementation class.
  • dom/DOMImplementation.h: Ditto.
  • dom/DocumentType.cpp: Removed unneeded include of DOMImplementation.h.
  • html/HTMLViewSourceDocument.cpp: (WebCore::HTMLViewSourceDocument::createTokenizer): Don't allocate a DOMImplementation object just to use a class member function.
  • loader/CachedFont.cpp: Removed unneeded include of DOMImplementation.h.
  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::responseXML): Use Document::create to create a new document instead of involving DOMImplementation.
  • xml/XSLTProcessor.cpp: (WebCore::XSLTProcessor::createDocumentFromSource): Ditto. And in the case where we do need to call DOMImplementation to interpret the MIME type, don't allocate a DOMImplementation object just to use a class member function.
13:06 Changeset [56411] by eric@webkit.org

2010-03-23 Luiz Agostini <luiz.agostini@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Maemo5 theme - all <select> elements should be rendered as menu lists.
https://bugs.webkit.org/show_bug.cgi?id=36367

Enabling NO_LISTBOX_RENDERING in WebCore.pri for Maemo5.

  • WebCore.pri:
12:39 Changeset [56410] by eric@webkit.org

2010-03-23 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

When uncaught exception happens reveal Scripts panel only if reporting uncaught exceptions is on.

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

  • src/js/DebuggerAgent.js: (devtools.DebuggerAgent.prototype.handleExceptionEvent_):
12:37 Changeset [56409] by joepeck@webkit.org

Revert accidental ChangeLog commit r56407.

12:23 Changeset [56408] by joepeck@webkit.org

2010-03-22 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by David Kilzer.

Upstream iPhone KeyEvent platform code and share with Mac platform
https://bugs.webkit.org/show_bug.cgi?id=35870

  • platform/iphone/KeyEventCodesIPhone.h: Improper comment for #endif.
  • platform/iphone/KeyEventIPhone.mm: Fixed headers.
12:23 Changeset [56407] by joepeck@webkit.org

Reviewed by Simon Fraser.

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

  • rendering/RenderEmbeddedObject.cpp: (WebCore::RenderEmbeddedObject::RenderEmbeddedObject): Fix a find/repalce mistake from my earlier patch. Rename m_setShowsMissingPluginIndicator -> m_showsMissingPluginIndicator. (WebCore::RenderEmbeddedObject::updateWidget): Don't update the widget if we're showing the missing plug-in indicator. (WebCore::RenderEmbeddedObject::paint):Fix a find/repalce mistake from my earlier patch. Rename m_setShowsMissingPluginIndicator -> m_showsMissingPluginIndicator. (WebCore::RenderEmbeddedObject::paintReplaced): Ditto.
  • rendering/RenderEmbeddedObject.h: Ditto. (WebCore::RenderEmbeddedObject::setShowsMissingPluginIndicator): Ditto. (WebCore::RenderEmbeddedObject::showsMissingPluginIndicator): Ditto.
  • Plugins/WebNetscapePluginView.mm:
  • Plugins/WebNullPluginView.h: Removed.
  • Plugins/WebNullPluginView.mm: Removed.
  • Resources/nullplugin.tiff: Removed.
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::createPlugin): Invoke the resource load delegate if the plug-in failed to load.
12:22 Changeset [56406] by eric@webkit.org

2010-03-23 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: TimelinePanel stays in the recording state after reopening WebInspector.
https://bugs.webkit.org/show_bug.cgi?id=36503

  • src/WebDevToolsAgentImpl.cpp: (WebKit::WebDevToolsAgentImpl::detach):
11:51 Changeset [56405] by hyatt@apple.com

https://bugs.webkit.org/show_bug.cgi?id=36277, make sure nth-child supports the positive unary prefix (+) as well as the negative
one (-).

Reviewed by Jeremy Orlow.

Added fast/css/nth-child-unary-prefix.html

  • css/tokenizer.flex:
11:49 Changeset [56404] by eric@webkit.org

2010-03-23 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Dmitry Titov.

Change notification sent to inspector frontend on workers creation / destruction
to be posted asynchronously to avoid JS reenterability problems. Also, renamed
willDestroyWorker to didDestroyWorker to reflect that it gets called later and got
rid of explicit IDs for workers (use addresses instead).
https://bugs.webkit.org/show_bug.cgi?id=36213

  • inspector/InjectedScriptHost.cpp: (WebCore::InjectedScriptHost::didDestroyWorker):
  • inspector/InjectedScriptHost.h:
  • inspector/InjectedScriptHost.idl:
  • inspector/InspectorController.cpp: (WebCore::PostWorkerNotificationToFrontendTask::create): (WebCore::PostWorkerNotificationToFrontendTask::PostWorkerNotificationToFrontendTask): (WebCore::PostWorkerNotificationToFrontendTask::performTask): (WebCore::InspectorController::postWorkerNotificationToFrontend): (WebCore::InspectorController::didCreateWorker): (WebCore::InspectorController::didDestroyWorker):
  • inspector/InspectorController.h: (WebCore::InspectorController::):
  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::didDestroyWorker):
  • inspector/InspectorFrontend.h:
  • inspector/InspectorWorkerResource.h: (WebCore::InspectorWorkerResource::create): (WebCore::InspectorWorkerResource::id): (WebCore::InspectorWorkerResource::InspectorWorkerResource):
  • inspector/front-end/InjectedFakeWorker.js: (InjectedFakeWorker.FakeWorker.prototype.terminate):
  • inspector/front-end/WorkersSidebarPane.js: (WebInspector.didDestroyWorker):
  • workers/AbstractWorker.cpp: (WebCore::AbstractWorker::AbstractWorker): (WebCore::AbstractWorker::onDestroyWorker):
  • workers/AbstractWorker.h: (WebCore::AbstractWorker::asID):
  • workers/SharedWorker.cpp: (WebCore::SharedWorker::SharedWorker):
  • workers/Worker.cpp: (WebCore::Worker::Worker):
11:30 Changeset [56403] by eric@webkit.org

2010-03-23 Kent Hansen <kent.hansen@nokia.com>

Reviewed by Darin Adler.

Fix compilation on Itanium in 32-bit mode
https://bugs.webkit.org/show_bug.cgi?id=36494

  • wtf/Platform.h: Introduce CPU(IA64_32). Don't define WTF_USE_JSVALUE64 if the CPU is in 32-bit mode.
10:53 Changeset [56402] by hyatt@apple.com

https://bugs.webkit.org/show_bug.cgi?id=36266, support DOM Level 3 focusin/focusout events. Add support for
IE onfocusin/onfocusout extensions to HTML. The old DOM level 2 names (DOMFocusIn/DOMFocusOut are still supported).

Reviewed by Sam Weinig.

Added fast/events/focusinout.html

  • dom/Document.cpp:

(WebCore::Document::setFocusedNode):

  • dom/Event.cpp:

(WebCore::Event::aliasedType):
(WebCore::Event::hasAliasedType):

  • dom/Event.h:
  • dom/EventNames.h:
  • dom/EventTarget.cpp:

(WebCore::EventTarget::fireEventListeners):

  • dom/EventTarget.h:
  • dom/Node.cpp:

(WebCore::Node::dispatchUIEvent):

  • html/HTMLAttributeNames.in:
  • html/HTMLButtonElement.cpp:

(WebCore::HTMLButtonElement::parseMappedAttribute):

  • html/HTMLElement.cpp:

(WebCore::HTMLElement::parseMappedAttribute):

  • html/HTMLFormControlElement.cpp:

(WebCore::HTMLTextFormControlElement::parseMappedAttribute):

  • html/HTMLFrameSetElement.cpp:

(WebCore::HTMLFrameSetElement::parseMappedAttribute):

  • html/HTMLSelectElement.cpp:

(WebCore::HTMLSelectElement::parseMappedAttribute):

  • svg/SVGElement.cpp:

(WebCore::SVGElement::parseMappedAttribute):

10:36 QtWebKit edited by kenneth@webkit.org
(diff)
10:30 QtWebKitBackportingFixes edited by siddharth.mathur@nokia.com
Moved r56370 to Incoming list (diff)
10:10 Changeset [56401] by eric@webkit.org

2010-03-23 MORITA Hajime <morrita@google.com>

Reviewed by Alexey Proskuryakov.

WebCore::SelectionController::setSelection NULL pointer
https://bugs.webkit.org/show_bug.cgi?id=31545

  • svg/dom/frame-related-api-during-load-expected.txt: Added.
  • svg/dom/frame-related-api-during-load.html: Added.

2010-03-23 MORITA Hajime <morrita@google.com>

Reviewed by Alexey Proskuryakov.

WebCore::SelectionController::setSelection NULL pointer
https://bugs.webkit.org/show_bug.cgi?id=31545

Added missing NULL-check that other APIs have.

Test: svg/dom/frame-related-api-during-load.html

  • svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::deselectAll):
10:09 Changeset [56400] by ggaren@apple.com

Interpreter fix for <rdar://problem/7728196> REGRESSION (r46701): -(-2147483648)
evaluates to -2147483648 on 32 bit (35842)

Reviewed by Mark Rowe.

  • interpreter/Interpreter.cpp:

(JSC::Interpreter::privateExecute): Only take the fast negate path if
a bit other than bit 31 is set. If none of bits 0-30 are set, then the
value we're negating can only be 0 or -2147483648, and neither can be
negated in int space.

  • jit/JITArithmetic.cpp:

(JSC::JIT::emit_op_negate):
(JSC::JIT::emitSlow_op_negate): Updated the JIT implementation to match
the interpreter, since it's slightly simpler.

09:52 Changeset [56399] by eric@webkit.org

2010-03-23 David Leong <david.leong@nokia.com>

Reviewed by Laszlo Gombos.

Build fix for Symbian Def file.

  • symbian/eabi/QtWebKitu.def:
09:44 QtWebKitBackportingFixes edited by siddharth.mathur@nokia.com
added JSC collector memory improvement #34350 (diff)
09:34 Changeset [56398] by eric@webkit.org

2010-03-23 Qi Zhang <qi.2.zhang@nokia.com>

Reviewed by Simon Hausmann.

Qt] LayoutTests/fast/canvas/fillrect_gradient.html failed
https://bugs.webkit.org/show_bug.cgi?id=36444

Enable LayoutTests/fast/canvas/fillrect_gradient.html

  • platform/qt/Skipped:
  • platform/qt/fast/canvas/fillrect_gradient-expected.checksum: Added.
  • platform/qt/fast/canvas/fillrect_gradient-expected.png: Added.
  • platform/qt/fast/canvas/fillrect_gradient-expected.txt:

2010-03-23 Qi Zhang <qi.2.zhang@nokia.com>

Reviewed by Simon Hausmann.

[Qt] LayoutTests/fast/canvas/fillrect_gradient.html failed
https://bugs.webkit.org/show_bug.cgi?id=36444

Handle duplicated or unsorted colorStop at platformDestory

  • platform/graphics/qt/GradientQt.cpp: (WebCore::Gradient::platformGradient):
08:47 Changeset [56397] by jchaffraix@webkit.org

Unreviewed build fix.

Try to make the window buildbot happy.

  • xml/XMLHttpRequestProgressEventThrottle.cpp:
  • xml/XMLHttpRequestProgressEventThrottle.h: Move initialization of the

static const in the cpp file.

08:42 Changeset [56396] by pfeldman@chromium.org

2010-03-23 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: make gear menu appear on styles panel at all times.

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

  • inspector/front-end/inspector.css:
08:39 Changeset [56395] by pfeldman@chromium.org

2010-03-23 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: render nodes in inherited style bars as links.

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

  • English.lproj/localizedStrings.js:
  • inspector/front-end/BreakpointsSidebarPane.js: (WebInspector.BreakpointsSidebarPane.prototype._appendBreakpointElement.breakpointClicked): (WebInspector.BreakpointsSidebarPane.prototype._appendBreakpointElement):
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylesSidebarPane.prototype._update.else.insertInheritedNodeSeparator): (WebInspector.StylesSidebarPane.prototype._update): (WebInspector.StylesSidebarPane.prototype._selectNode):
  • inspector/front-end/inspector.js:
08:33 Changeset [56394] by jchaffraix@webkit.org

[XHR] onProgress event needs to be dispatched according to what the specification states
https://bugs.webkit.org/show_bug.cgi?id=18654

Reviewed by Alexey Proskuryakov.

WebCore:

Test: http/tests/xmlhttprequest/xmlhttprequest-50ms-download-dispatch.html

  • Android.mk:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:

Added XMLHttpRequestProgressEventThrottle to the build systems.

  • xml/XMLHttpRequest.cpp:

(WebCore::XMLHttpRequest::XMLHttpRequest): Created the throttle.
(WebCore::XMLHttpRequest::callReadyStateChangeListener): The 'readyState' event are
dispatched using the XMLHttpRequestProgressEventThrottle now. For the DONE state, we
need to flush any pending progress event as we do not want it to be dispatched after
the DONE readyState event.
(WebCore::XMLHttpRequest::createRequest): Made this event use the
XMLHttpRequestProgressEventThrottle for event dispatching.
(WebCore::XMLHttpRequest::abort): Ditto.
(WebCore::XMLHttpRequest::networkError): Ditto.
(WebCore::XMLHttpRequest::abortError): Ditto.
(WebCore::XMLHttpRequest::didReceiveData): Ditto. Also fixed a potential warning.
(WebCore::XMLHttpRequest::suspend):
(WebCore::XMLHttpRequest::resume): Implemented the logic for suspend / resume.

  • xml/XMLHttpRequest.h:
  • xml/XMLHttpRequestProgressEventThrottle.cpp: Added.

(WebCore::XMLHttpRequestProgressEventThrottle::XMLHttpRequestProgressEventThrottle):
(WebCore::XMLHttpRequestProgressEventThrottle::~XMLHttpRequestProgressEventThrottle):
(WebCore::XMLHttpRequestProgressEventThrottle::dispatchProgressEvent): Implemented the bulk
of the event throttling here: we use a timer to do so, as long as the timer is active it means
that events are coming faster than the throttling time and we coalesce them.
(WebCore::XMLHttpRequestProgressEventThrottle::dispatchEvent): Generic method to dispatch an event.
(WebCore::XMLHttpRequestProgressEventThrottle::flushProgressEvent): Called when we want to dispatch
any pending events and stopping any further dispatching.
(WebCore::XMLHttpRequestProgressEventThrottle::dispatchPausedEvent): Used to dispatch the event
that was queued due to the object being suspended.
(WebCore::XMLHttpRequestProgressEventThrottle::fired): Used to dispatch any coalesced event.
(WebCore::XMLHttpRequestProgressEventThrottle::hasEventToDispatch): Used to check if we have
an event to dispatch.
(WebCore::XMLHttpRequestProgressEventThrottle::suspend): Marked the object as suspended.
(WebCore::XMLHttpRequestProgressEventThrottle::resume): Dispatched the event that was queued.

  • xml/XMLHttpRequestProgressEventThrottle.h: Added.

(WebCore::):
(WebCore::XMLHttpRequestProgressEventThrottle::suspended):

LayoutTests:

Add a test to check that progress events are really throttled.

  • http/tests/xmlhttprequest/resources/download-with-delay.php: Added.
  • http/tests/xmlhttprequest/xmlhttprequest-50ms-download-dispatch-expected.txt: Added.
  • http/tests/xmlhttprequest/xmlhttprequest-50ms-download-dispatch.html: Added.
08:14 Changeset [56393] by pfeldman@chromium.org

2010-03-23 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Element inspector search funtion: Error dispatching: performSearch.

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

Test: inspector/elements-panel-search.html

  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.performSearch): (WebInspector.ElementsPanel.prototype.addNodesToSearchResult):
  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor.): (injectedScriptConstructor):
07:57 Changeset [56392] by laszlo.1.gombos@nokia.com

2010-03-23 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Unreviewed, fix the reviwer for r56391.

  • WebCore/ChangeLog
  • LayoutTests/ChangeLog
07:37 Changeset [56391] by laszlo.1.gombos@nokia.com

2010-03-23 Qi Zhang <qi.2.zhang@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] fast/canvas/patternfill-repeat.html failed.
https://bugs.webkit.org/show_bug.cgi?id=34477

Change Pattern to expose the repeatX and repeatY
at GraphicsContextQt to apply repeatX and repeatY

  • platform/graphics/Pattern.h: (WebCore::Pattern::repeatX): (WebCore::Pattern::repeatY):
  • platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::GraphicsContext::drawLine): (WebCore::GraphicsContext::fillRect): (WebCore::GraphicsContext::rotate):

2010-03-23 Qi Zhang <qi.2.zhang@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] fast/canvas/patternfill-repeat.html failed.
https://bugs.webkit.org/show_bug.cgi?id=34477

Enable LayoutTests/fast/canvas/patternfill-repeat.html.

  • platform/qt/Skipped:
  • platform/qt/fast/canvas/patternfill-repeat-expected.checksum: Added.
  • platform/qt/fast/canvas/patternfill-repeat-expected.png: Added.
  • platform/qt/fast/canvas/patternfill-repeat-expected.txt:
07:19 QtWebKitBugs edited by jocelyn.turcotte@nokia.com
minor (diff)
07:05 Changeset [56390] by eric@webkit.org

2010-03-23 Alex Milowski <alex@milowski.com>

Reviewed by Kenneth Rohde Christiansen.

Updated test results from removal of use of STIX fonts.

  • platform/mac/mathml/presentation/fractions-expected.checksum:
  • platform/mac/mathml/presentation/fractions-expected.png:
  • platform/mac/mathml/presentation/fractions-expected.txt:
  • platform/mac/mathml/presentation/fractions-vertical-alignment-expected.checksum:
  • platform/mac/mathml/presentation/fractions-vertical-alignment-expected.png:
  • platform/mac/mathml/presentation/fractions-vertical-alignment-expected.txt:
  • platform/mac/mathml/presentation/mo-expected.checksum:
  • platform/mac/mathml/presentation/mo-expected.png:
  • platform/mac/mathml/presentation/mo-expected.txt:
  • platform/mac/mathml/presentation/over-expected.checksum:
  • platform/mac/mathml/presentation/over-expected.png:
  • platform/mac/mathml/presentation/over-expected.txt:
  • platform/mac/mathml/presentation/row-expected.checksum:
  • platform/mac/mathml/presentation/row-expected.png:
  • platform/mac/mathml/presentation/row-expected.txt:
  • platform/mac/mathml/presentation/sub-expected.checksum:
  • platform/mac/mathml/presentation/sub-expected.png:
  • platform/mac/mathml/presentation/sub-expected.txt:
  • platform/mac/mathml/presentation/subsup-expected.checksum:
  • platform/mac/mathml/presentation/subsup-expected.png:
  • platform/mac/mathml/presentation/subsup-expected.txt:
  • platform/mac/mathml/presentation/sup-expected.checksum:
  • platform/mac/mathml/presentation/sup-expected.png:
  • platform/mac/mathml/presentation/tables-expected.checksum:
  • platform/mac/mathml/presentation/tables-expected.png:
  • platform/mac/mathml/presentation/tables-expected.txt:
  • platform/mac/mathml/presentation/under-expected.checksum:
  • platform/mac/mathml/presentation/under-expected.png:
  • platform/mac/mathml/presentation/under-expected.txt:
  • platform/mac/mathml/presentation/underover-expected.checksum:
  • platform/mac/mathml/presentation/underover-expected.png:
  • platform/mac/mathml/presentation/underover-expected.txt:

2010-03-23 Alex Milowski <alex@milowski.com>

Reviewed by Kenneth Rohde Christiansen.

Removed the use of beta STIX fonts.

  • css/mathml.css:
06:34 Changeset [56389] by ossy@webkit.org

Unreviewed.

[Qt] websocket/tests/long-invalid-header.html fails
https://bugs.webkit.org/show_bug.cgi?id=36492

  • platform/qt/Skipped: websocket/tests/long-invalid-header.html skipped until fix.
06:23 Changeset [56388] by kov@webkit.org

2010-03-23 Gustavo Noronha Silva <gns@gnome.org>

[GTK] Failing fast/css/zoom-body-scroll.html - sizes are off by one after zooming out
https://bugs.webkit.org/show_bug.cgi?id=36491

Skipping test that fails on 32 bits bots.

  • platform/gtk/Skipped:
06:23 Changeset [56387] by kov@webkit.org

WebCore

2010-03-23 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Holger Freyther.

[GTK] Does not build with latest GTK+ development release
https://bugs.webkit.org/show_bug.cgi?id=36398

Fix building with newest GTK+ versions.

  • GNUmakefile.am:
  • platform/gtk/GtkPluginWidget.cpp: (WebCore::GtkPluginWidget::invalidateRect): (WebCore::GtkPluginWidget::paint):
  • platform/gtk/GtkVersioning.h: Added.
  • platform/gtk/PlatformScreenGtk.cpp: (WebCore::getVisual): (WebCore::screenRect): (WebCore::screenAvailableRect):
  • platform/gtk/ScrollbarGtk.cpp: (ScrollbarGtk::paint):
  • plugins/gtk/gtk2xtbin.c: (gtk_xtbin_set_position): (gtk_xtbin_unrealize):

WebKit/gtk

2010-03-23 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Holger Freyther.

[GTK] Does not build with latest GTK+ development release
https://bugs.webkit.org/show_bug.cgi?id=36398

Fix building with newest GTK+ versions.

  • WebCoreSupport/ChromeClientGtk.cpp: (WebKit::ChromeClient::windowRect): (WebKit::ChromeClient::setWindowRect): (WebKit::ChromeClient::unfocus): (WebKit::ChromeClient::canTakeFocus): (WebKit::ChromeClient::contentsSizeChanged):
  • webkit/webkitprivate.cpp: (currentToplevelCallback):
  • webkit/webkitwebframe.cpp: (webkit_web_frame_print_full): (webkit_web_frame_print):
  • webkit/webkitwebview.cpp: (webkit_web_view_grab_focus): (webkit_web_view_focus_in_event): (webkit_web_view_script_dialog):
05:52 Changeset [56386] by eric@webkit.org

2010-03-23 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Remove support for Qt v4.4
https://bugs.webkit.org/show_bug.cgi?id=36389

No new tests as there is no new functionality.

  • WebCore.pri:
  • WebCore.pro:
  • platform/network/qt/ResourceHandleQt.cpp: (WebCore::ResourceHandle::willLoadFromCache):
  • platform/qt/CookieJarQt.cpp: (WebCore::setCookies): (WebCore::cookies):
  • platform/qt/KURLQt.cpp: (WebCore::KURL::operator QUrl):
  • platform/qt/ScrollbarThemeQt.cpp: (WebCore::ScrollbarThemeQt::paintScrollCorner):
  • plugins/mac/PluginViewMac.cpp:

2010-03-23 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Remove support for Qt v4.4
https://bugs.webkit.org/show_bug.cgi?id=36389

  • Api/qwebelement.cpp: (QWebElement::classes):
  • Api/qwebpage.cpp: (QWebPagePrivate::editorActionForKeyEvent): (QWebPage::userAgentForUrl):
  • WebCoreSupport/EditorClientQt.cpp: (WebCore::EditorClientQt::handleKeyboardEvent):
  • tests/tests.pro:

2010-03-23 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Remove support for Qt v4.4
https://bugs.webkit.org/show_bug.cgi?id=36389

  • DumpRenderTree/qt/main.cpp: (main):
  • QtLauncher/mainwindow.cpp: (MainWindow::MainWindow):
04:07 Changeset [56385] by tkent@chromium.org

2010-03-23 Kent Tamura <tkent@chromium.org>

Reviewed by Adam Barth.

Add checks if setNeedsWillValidateCheck() and
setNeedsValidityCheck() are called correctly.
https://bugs.webkit.org/show_bug.cgi?id=34924

Introduce HTMLFormControlElement::m_willValidate and
m_isValid. They are the caches of willValidate() and
isValidFormControlElement(). setNeedsWillValidateCheck() updates
m_willValidate and setNeedsValidityCheck() updates m_isValid.

willValidate() and isValidFormControlElement() have assertions to
check m_willvalidate or m_isValid has the correct state. If
setNeedsWillValidateCheck() or setNeedsValidityCheck() is needed
to be called and is not called, these assertions fail.

  • html/HTMLButtonElement.h: (WebCore::HTMLButtonElement::recalcWillValidate):
  • html/HTMLFieldSetElement.h: (WebCore::HTMLFieldSetElement::recalcWillValidate):
  • html/HTMLFormControlElement.cpp: (WebCore::HTMLFormControlElement::HTMLFormControlElement): (WebCore::HTMLFormControlElement::parseMappedAttribute): (WebCore::HTMLFormControlElement::recalcWillValidate): (WebCore::HTMLFormControlElement::willValidate): (WebCore::HTMLFormControlElement::setNeedsWillValidateCheck): (WebCore::HTMLFormControlElement::validationMessage): (WebCore::HTMLFormControlElement::isValidFormControlElement): (WebCore::HTMLFormControlElement::setNeedsValidityCheck):
  • html/HTMLFormControlElement.h:
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::setInputType): (WebCore::HTMLInputElement::parseMappedAttribute): (WebCore::HTMLInputElement::setValue): (WebCore::HTMLInputElement::recalcWillValidate):
  • html/HTMLInputElement.h:
  • html/HTMLTextAreaElement.cpp: (WebCore::HTMLTextAreaElement::setNonDirtyValue):
01:18 Changeset [56384] by ukai@chromium.org

Unreviewed, remove flaky tag in websocket tests.

flakiness in websocket tests (http://crbug.com/38397) introducted at
Chromium r41818 has been fixed at r42320.

  • platform/chromium/test_expectations.txt: remove BUG38397
00:07 Changeset [56383] by pfeldman@chromium.org

2010-03-22 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Dave Hyatt.

Web Inspector: display CSS selector source line in the styles sidebar pane.

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

This change adds a sourceLine field into the CSSStyleRule that is populated
from within the parser. CSSParser is now keeping track of the line numbers
and last selector line number that is being used while creating CSSStyleRules.

Test: inspector/styles-source-lines.html

  • css/CSSGrammar.y:
  • css/CSSParser.cpp: (WebCore::CSSParser::CSSParser): (WebCore::CSSParser::lex): (WebCore::CSSParser::countLines): (WebCore::CSSParser::createStyleRule):
  • css/CSSParser.h: (WebCore::CSSParser::updateLastSelectorLine):
  • css/CSSStyleRule.cpp: (WebCore::CSSStyleRule::CSSStyleRule):
  • css/CSSStyleRule.h: (WebCore::CSSStyleRule::create): (WebCore::CSSStyleRule::sourceLine):
  • css/tokenizer.flex:
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::buildObjectForRule):
  • inspector/front-end/DOMAgent.js: (WebInspector.CSSStyleDeclaration.parseRule):
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertiesSection):

03/22/10:

23:48 Changeset [56382] by eric@webkit.org

2010-03-22 Qi Zhang <qi.2.zhang@nokia.com>

Reviewed by Laszlo Gombos.

Qt] Fix arc function in canvas
https://bugs.webkit.org/show_bug.cgi?id=36296

  • platform/qt/Skipped:
  • platform/qt/fast/canvas/arc360-expected.checksum: Added.
  • platform/qt/fast/canvas/arc360-expected.png: Added.
  • platform/qt/fast/canvas/arc360-expected.txt: Added.

2010-03-22 Qi Zhang <qi.2.zhang@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] Fix arc function in canvas
https://bugs.webkit.org/show_bug.cgi?id=36296

Based on HTLM5 spec (4.8.10.1.8 Complex shapes), arc function should draw a line to previous point, not only the arc.

  • platform/graphics/qt/PathQt.cpp: (WebCore::Path::addArc):
23:30 Changeset [56381] by eric@webkit.org

2010-03-22 Kenneth Russell <kbr@google.com>

Reviewed by Darin Fisher.

Add GraphicsContext3D abstraction to WebKit API
https://bugs.webkit.org/show_bug.cgi?id=36262

Added WebGraphicsContext3D to the WebKit API and refactored Chromium's
GraphicsContext3D implementation to use it. All of the OpenGL calls have
been moved out of WebKit/chromium/src/GraphicsContext3D.cpp and into the
WebGraphicsContext3D implementation. GraphicsContext3D is still
responsible for the transfer of rendered output from the
WebGraphicsContext3D to the HTMLCanvasElement.

The GraphicsContext3DInternal class, which is a data member of
GraphicsContext3D for the Chromium port, remains. It is possible to
eliminate this class and thereby one level of delegation, but this is
being deferred.

The needed entry point for a Chrome implementation of
WebGraphicsContext3D has been added to WebKitClient, but it is not being
called yet by GraphicsContext3D. It will be once this patch lands and
Chromium is rolled forward to support this entry point.

This is a large patch, but the transformation is almost entirely
mechanical and there is no change in functionality. Nearly all of
GraphicsContext3D and GraphicsContext3DInternal has been moved to
WebGraphicsContext3DDefaultImpl. The only area where the splitting of
logic is less than mechanical is GraphicsContext3D::beginPaint() and its
callees.

Ran all WebGL layout tests and demos from Khronos site in Chromium
on Mac and Windows.

  • WebKit.gyp:
  • public/WebGraphicsContext3D.h: Added. (WebKit::WebGraphicsContext3D::Attributes::Attributes): (WebKit::WebGraphicsContext3D::~WebGraphicsContext3D):
  • public/WebKitClient.h: (WebKit::WebKitClient::createGraphicsContext3D):
  • src/GraphicsContext3D.cpp: (WebCore::GraphicsContext3DInternal::GraphicsContext3DInternal): (WebCore::GraphicsContext3DInternal::~GraphicsContext3DInternal): (WebCore::GraphicsContext3DInternal::initialize): (WebCore::GraphicsContext3DInternal::platformGraphicsContext3D): (WebCore::GraphicsContext3DInternal::platformTexture): (WebCore::GraphicsContext3DInternal::beginPaint): (WebCore::GraphicsContext3DInternal::endPaint): (WebCore::GraphicsContext3DInternal::reshape): (WebCore::GraphicsContext3DInternal::bindAttribLocation): (WebCore::GraphicsContext3DInternal::bindTexture): (WebCore::GraphicsContext3DInternal::bufferData): (WebCore::GraphicsContext3DInternal::bufferSubData): (WebCore::GraphicsContext3DInternal::getActiveAttrib): (WebCore::GraphicsContext3DInternal::getActiveUniform): (WebCore::GraphicsContext3DInternal::getAttribLocation): (WebCore::GraphicsContext3DInternal::getContextAttributes): (WebCore::GraphicsContext3DInternal::getProgramInfoLog): (WebCore::GraphicsContext3DInternal::getShaderInfoLog): (WebCore::GraphicsContext3DInternal::getShaderSource): (WebCore::GraphicsContext3DInternal::getString): (WebCore::GraphicsContext3DInternal::getUniformLocation): (WebCore::GraphicsContext3DInternal::readPixels): (WebCore::GraphicsContext3DInternal::shaderSource): (WebCore::GraphicsContext3DInternal::texImage2D): (WebCore::GraphicsContext3DInternal::texSubImage2D): (WebCore::GraphicsContext3DInternal::uniform1fv): (WebCore::GraphicsContext3DInternal::uniform1iv): (WebCore::GraphicsContext3DInternal::uniform2fv): (WebCore::GraphicsContext3DInternal::uniform2iv): (WebCore::GraphicsContext3DInternal::uniform3fv): (WebCore::GraphicsContext3DInternal::uniform3iv): (WebCore::GraphicsContext3DInternal::uniform4fv): (WebCore::GraphicsContext3DInternal::uniform4iv): (WebCore::GraphicsContext3DInternal::uniformMatrix2fv): (WebCore::GraphicsContext3DInternal::uniformMatrix3fv): (WebCore::GraphicsContext3DInternal::uniformMatrix4fv): (WebCore::GraphicsContext3D::GraphicsContext3D): (WebCore::GraphicsContext3D::~GraphicsContext3D): (WebCore::GraphicsContext3D::create): (WebCore::GraphicsContext3D::platformGraphicsContext3D): (WebCore::GraphicsContext3D::platformTexture): (WebCore::GraphicsContext3D::texImage2D): (WebCore::GraphicsContext3D::texSubImage2D):
  • src/WebGraphicsContext3D.cpp: Added. (WebKit::WebGraphicsContext3D::createDefault):
  • src/WebGraphicsContext3DDefaultImpl.cpp: Added. (WebKit::WebGraphicsContext3DDefaultImpl::GLConnection::create): (WebKit::WebGraphicsContext3DDefaultImpl::GLConnection::~GLConnection): (WebKit::WebGraphicsContext3DDefaultImpl::VertexAttribPointerState::VertexAttribPointerState): (WebKit::WebGraphicsContext3DDefaultImpl::WebGraphicsContext3DDefaultImpl): (WebKit::WebGraphicsContext3DDefaultImpl::~WebGraphicsContext3DDefaultImpl): (WebKit::WebGraphicsContext3DDefaultImpl::initialize): (WebKit::WebGraphicsContext3DDefaultImpl::makeContextCurrent): (WebKit::WebGraphicsContext3DDefaultImpl::width): (WebKit::WebGraphicsContext3DDefaultImpl::height): (WebKit::WebGraphicsContext3DDefaultImpl::sizeInBytes): (WebKit::createTextureObject): (WebKit::WebGraphicsContext3DDefaultImpl::reshape): (WebKit::WebGraphicsContext3DDefaultImpl::flipVertically): (WebKit::WebGraphicsContext3DDefaultImpl::readBackFramebuffer): (WebKit::WebGraphicsContext3DDefaultImpl::activeTexture): (WebKit::WebGraphicsContext3DDefaultImpl::bindBuffer): (WebKit::WebGraphicsContext3DDefaultImpl::bindFramebuffer): (WebKit::WebGraphicsContext3DDefaultImpl::disableVertexAttribArray): (WebKit::WebGraphicsContext3DDefaultImpl::drawElements): (WebKit::WebGraphicsContext3DDefaultImpl::enableVertexAttribArray): (WebKit::WebGraphicsContext3DDefaultImpl::generateMipmap): (WebKit::WebGraphicsContext3DDefaultImpl::getActiveAttrib): (WebKit::WebGraphicsContext3DDefaultImpl::getActiveUniform): (WebKit::WebGraphicsContext3DDefaultImpl::getContextAttributes): (WebKit::WebGraphicsContext3DDefaultImpl::getError): (WebKit::WebGraphicsContext3DDefaultImpl::getProgramInfoLog): (WebKit::WebGraphicsContext3DDefaultImpl::getShaderInfoLog): (WebKit::WebGraphicsContext3DDefaultImpl::getShaderSource): (WebKit::WebGraphicsContext3DDefaultImpl::getString): (WebKit::WebGraphicsContext3DDefaultImpl::getVertexAttribOffset): (WebKit::WebGraphicsContext3DDefaultImpl::releaseShaderCompiler): (WebKit::WebGraphicsContext3DDefaultImpl::shaderSource): (WebKit::WebGraphicsContext3DDefaultImpl::vertexAttribPointer): (WebKit::WebGraphicsContext3DDefaultImpl::createBuffer): (WebKit::WebGraphicsContext3DDefaultImpl::createFramebuffer): (WebKit::WebGraphicsContext3DDefaultImpl::createProgram): (WebKit::WebGraphicsContext3DDefaultImpl::createRenderbuffer): (WebKit::WebGraphicsContext3DDefaultImpl::createTexture): (WebKit::WebGraphicsContext3DDefaultImpl::deleteBuffer): (WebKit::WebGraphicsContext3DDefaultImpl::deleteFramebuffer): (WebKit::WebGraphicsContext3DDefaultImpl::deleteProgram): (WebKit::WebGraphicsContext3DDefaultImpl::deleteRenderbuffer): (WebKit::WebGraphicsContext3DDefaultImpl::deleteShader): (WebKit::WebGraphicsContext3DDefaultImpl::deleteTexture): (WebKit::WebGraphicsContext3DDefaultImpl::synthesizeGLError):
  • src/WebGraphicsContext3DDefaultImpl.h: Added. (WebKit::WebGraphicsContext3DDefaultImpl::): (WebKit::WebGraphicsContext3DDefaultImpl::GLConnection::chooseFBConfig): (WebKit::WebGraphicsContext3DDefaultImpl::GLConnection::createNewContext): (WebKit::WebGraphicsContext3DDefaultImpl::GLConnection::createPbuffer): (WebKit::WebGraphicsContext3DDefaultImpl::GLConnection::destroyPbuffer): (WebKit::WebGraphicsContext3DDefaultImpl::GLConnection::makeCurrent): (WebKit::WebGraphicsContext3DDefaultImpl::GLConnection::destroyContext): (WebKit::WebGraphicsContext3DDefaultImpl::GLConnection::getCurrentContext): (WebKit::WebGraphicsContext3DDefaultImpl::GLConnection::GLConnection):
23:08 Changeset [56380] by ukai@chromium.org

2010-03-19 Abhishek Arya <inferno@chromium.org>

Reviewed by Adam Barth.

https://bugs.webkit.org/show_bug.cgi?id=36339
This LayoutTest tests for webkit ability to process long invalid headers generated by misbehaving websockets servers.

  • websocket/tests/long-invalid-header-expected.txt: Added.
  • websocket/tests/long-invalid-header.html: Added.
  • websocket/tests/long-invalid-header_wsh.py: Added.
  • websocket/tests/script-tests/long-invalid-header.js: Added.

2010-03-19 Abhishek Arya <inferno@chromium.org>

Reviewed by Adam Barth.

https://bugs.webkit.org/show_bug.cgi?id=36339
Off-by-one memory corruption fix for long invalid websockets upgrade header

Test: websocket/tests/long-invalid-header.html

  • websockets/WebSocketHandshake.cpp: (WebCore::WebSocketHandshake::readServerHandshake):
22:51 Changeset [56379] by eric@webkit.org

2010-03-22 David Kilzer <ddkilzer@apple.com>

Remove http/tests/cookies/simple-cookies-expired.html from skip list after CFNetwork issue is resolved
<http://webkit.org/b/25861>
<rdar://problem/6901322>

Reviewed by Darin Adler.

  • platform/win/Skipped: Removed:
  • http/tests/cookies/simple-cookies-expired.html
22:32 Changeset [56378] by eric@webkit.org

2010-03-22 Eric Seidel <eric@webkit.org>

Rubber-stamped by Brady Eidson.

fast/loader/cancel-load-during-port-block-timer.html fails on Tiger bot
https://bugs.webkit.org/show_bug.cgi?id=36425

This test bleeds into the next test and causes failures. Unsure of the
cause. Brady suggests we disable it for now (to match the other disabled) api test.

  • fast/loader/api-test-go-to-current-back-forward-item.html-disabled: Renamed from LayoutTests/fast/loader/api-test-go-to-current-back-forward-item.html.
22:03 Changeset [56377] by ukai@chromium.org

2010-03-22 Fumitoshi Ukai <ukai@chromium.org>

Unreviewed build fix for Chromim Mac at r56376

Fix initializer order of PopupContainer.

  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupContainer::PopupContainer):
20:01 Changeset [56376] by eric@webkit.org

2010-03-22 Jay Campan <jcampan@google.com>

Reviewed by Darin Fisher.

Making Chromium select popups not steal activation from the browser.
Select popups are now like autocomplete popups, shown in non-activated
windows.
https://bugs.webkit.org/show_bug.cgi?id=36062

  • page/chromium/ChromeClientChromium.h:
  • platform/chromium/PopupMenuChromium.cpp: (WebCore::): (WebCore::PopupContainer::create): (WebCore::PopupContainer::PopupContainer): (WebCore::PopupContainer::~PopupContainer): (WebCore::PopupContainer::showPopup): (WebCore::PopupContainer::showExternal): (WebCore::PopupContainer::hidePopup): (WebCore::PopupMenu::show):
  • platform/chromium/PopupMenuChromium.h: (WebCore::PopupContainer::): (WebCore::PopupContainer::popupType):

2010-03-22 Jay Campan <jcampan@google.com>

Reviewed by Darin Fisher.

Making Chromium select popups not steal activation from the browser.
Select popups are now like autocomplete popups, shown in non-activated
windows.
https://bugs.webkit.org/show_bug.cgi?id=36062

  • public/WebViewClient.h: (WebKit::WebViewClient::createPopupMenu):
  • src/ChromeClientImpl.cpp: (WebKit::ChromeClientImpl::popupOpened): (WebKit::ChromeClientImpl::popupClosed):
  • src/ChromeClientImpl.h:
  • src/WebViewImpl.cpp: (WebKit::): (WebKit::WebViewImpl::mouseDown): (WebKit::WebViewImpl::keyEvent): (WebKit::WebViewImpl::selectPopupHandleKeyEvent): (WebKit::WebViewImpl::hideSelectPopup): (WebKit::WebViewImpl::popupOpened): (WebKit::WebViewImpl::popupClosed): (WebKit::WebViewImpl::setFocus): (WebKit::WebViewImpl::applyAutoFillSuggestions): (WebKit::WebViewImpl::applyAutocompleteSuggestions):
  • src/WebViewImpl.h:
19:21 Changeset [56375] by dimich@chromium.org

[v8] Crash if the worker is terminated before its initial script is executed.
https://bugs.webkit.org/show_bug.cgi?id=36336

Reviewed by Nate Chapin.

WebCore:

Test: fast/workers/termination-early.html

  • bindings/v8/WorkerContextExecutionProxy.cpp:

(WebCore::WorkerContextExecutionProxy::initContextIfNeeded): Check for 0 context. When terminated, the v8 heap initialization returns 0.
(WebCore::WorkerContextExecutionProxy::evaluate): Don't execute script if initialization failed.

  • bindings/v8/WorkerContextExecutionProxy.h:

LayoutTests:

  • fast/workers/termination-early-expected.txt: Added.
  • fast/workers/termination-early.html: Added.
19:11 Changeset [56374] by eric@webkit.org

2010-03-22 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Pavel Feldman.

Support creation of fake workers before document is loaded by deferring
attaching of fake workers iframe until document is loaded and buffering all
messages addressed to worker.
https://bugs.webkit.org/show_bug.cgi?id=36445

  • inspector/front-end/InjectedFakeWorker.js: (InjectedFakeWorker.FakeWorker.prototype.postMessage): (InjectedFakeWorker.FakeWorker.prototype._buildWorker): (InjectedFakeWorker.FakeWorker.prototype._attachWorkerFrameToDocument): (InjectedFakeWorker.FakeWorker.prototype._onWorkerFrameLoaded):
18:31 Changeset [56373] by eric@webkit.org

2010-03-22 Kent Hansen <kent.hansen@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] Fix qdoc warnings
https://bugs.webkit.org/show_bug.cgi?id=36447

  • Api/qgraphicswebview.cpp: Add missing ().
  • Api/qwebframe.cpp: Remove reference to non-existent parameter "url".
  • Api/qwebsettings.cpp: Document parameter "location".
18:13 Changeset [56372] by eric@webkit.org

2010-03-22 Csaba Osztrogonác <ossy@webkit.org>

Reviewed by Adam Barth.

Make build_webkit_command() pass MAKEFLAGS environment variable to make.
https://bugs.webkit.org/show_bug.cgi?id=36440

  • Scripts/webkitpy/webkitport.py:
  • Scripts/webkitpy/webkitport_unittest.py:
17:28 Changeset [56371] by mjs@apple.com

(Not reviewed.)

Add another example.

  • demos/accessible-ascii-art/butterfly-figure.html: Added.
17:28 Changeset [56370] by eric@webkit.org

2010-03-22 Siddharth Mathur <siddharth.mathur@nokia.com>

Reviewed by Laszlo Gombos.

[Symbian] More efficient aligned memory allocation for JSC Collector
https://bugs.webkit.org/show_bug.cgi?id=34350

  • runtime/Collector.cpp: Reduced port-specific code and added private data member (JSC::Heap::Heap): (JSC::Heap::~Heap): (JSC::Heap::destroy): (JSC::Heap::allocateBlock): (JSC::Heap::freeBlockPtr):
  • runtime/Collector.h: Added private data member
  • wtf/symbian: Added.
  • wtf/symbian/BlockAllocatorSymbian.cpp: Added. (WTF::AlignedBlockAllocator::AlignedBlockAllocator): Helper class to allocate aligned blocks more efficiently as required by Collector (WTF::AlignedBlockAllocator::alloc): (WTF::AlignedBlockAllocator::free): (WTF::AlignedBlockAllocator::destroy): (WTF::AlignedBlockAllocator::~AlignedBlockAllocator):
  • wtf/symbian/BlockAllocatorSymbian.h: Added.
16:51 Changeset [56369] by ossy@webkit.org

Unreviewed, skip new failing test.

fast/loader/stateobjects/replacestate-in-iframe.html caused infinite loop.
https://bugs.webkit.org/show_bug.cgi?id=36467

  • platform/gtk/Skipped: fast/loader/stateobjects/replacestate-in-iframe.html skipped until fix.
  • platform/qt/Skipped: fast/loader/stateobjects/replacestate-in-iframe.html skipped until fix.
16:43 Changeset [56368] by mjs@apple.com

(Not reviewed.)

Add example of accessible ASCII art for HTML WG.

  • demos/accessible-ascii-art: Added.
  • demos/accessible-ascii-art/butterfly.html: Added.
16:14 Changeset [56367] by japhet@chromium.org

2010-03-22 Nate Chapin <japhet@chromium.org>

Unreviewed, Chromium test fixes.

  • platform/chromium-linux/fast/dynamic/flash-replacement-test-expected.checksum:
  • platform/chromium-linux/fast/dynamic/flash-replacement-test-expected.png:
  • platform/chromium-linux/plugins/embed-attributes-style-expected.checksum:
  • platform/chromium-linux/plugins/embed-attributes-style-expected.png:
  • platform/chromium-mac/svg/filters/feColorMatrix-values-expected.checksum: Added.
  • platform/chromium-mac/svg/filters/feColorMatrix-values-expected.png: Added.
  • platform/chromium-mac/svg/filters/feColorMatrix-values-expected.txt: Added.
  • platform/chromium-win/svg/filters/feColorMatrix-values-expected.checksum: Added.
  • platform/chromium-win/svg/filters/feColorMatrix-values-expected.png: Added.
  • platform/chromium-win/svg/filters/feColorMatrix-values-expected.txt: Added.
16:04 Changeset [56366] by dpranke@chromium.org

2010-03-22 Dirk Pranke <dpranke@chromium.org>

Reviewed by Dimitri Glazkov.

Change baseline_path() to point to the upstream locations for the
Chromium ports. Also change the reabselining scripts to use the
correct functions to get the baseline directories, and fix the
script's sys.path to pull in simplejson correctly.

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

  • Scripts/rebaseline-chromium-webkit-tests:
    • fix sys.path to pick up simplejson properly
  • Scripts/webkitpy/layout_tests/port/chromium.py:
    • change baseline_path() to use webkit_baseline_path()
    • error out correctly if we can't find the chromium base dir
  • Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests.py:
    • call baseline_path(), not chromium_baseline_path()
15:57 Changeset [56365] by darin@chromium.org

2010-03-22 Darin Fisher <darin@chromium.org>

Reviewed by Brady Eidson.

HistoryController::replaceState() should modify m_currentItem
instead of the current HistoryItem of the BackForwardList.

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

Test: fast/loader/stateobjects/replacestate-in-iframe.html

  • loader/HistoryController.cpp: (WebCore::HistoryController::replaceState):
15:54 Changeset [56364] by hausmann@webkit.org

Oops, fix spurious angle brackets around name in patch I landed manually

15:31 Changeset [56363] by joepeck@webkit.org

2010-03-22 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by David Kilzer.

Upstream Part of the iPhone Platform sharing code with the Mac Platform
https://bugs.webkit.org/show_bug.cgi?id=35870
<rdar://problem/7707283> and <rdar://problem/7707318>

Part 3 of 3: Shared code between iphone and mac. The shared code was
refactored to be usable by both platforms without platform specific
typing, meaning no NSEvent versus WebEvent special cases.

Shared code usable by either mac or iphone platforms.

  • platform/cocoa/KeyEventCocoa.h:
  • platform/cocoa/KeyEventCocoa.mm: (WebCore::keyIdentifierForCharCode): (WebCore::windowsKeyCodeForKeyCode): (WebCore::windowsKeyCodeForCharCode):

Make use of the shared code. Keep anything platform specific in the
original file before calling the shared code.

  • platform/iphone/KeyEventIPhone.mm: (WebCore::keyIdentifierForKeyEvent):
  • platform/mac/KeyEventMac.mm: (WebCore::keyIdentifierForKeyEvent): (WebCore::windowsKeyCodeForKeyEvent):
15:17 Rebaseline edited by dglazkov@chromium.org
(diff)
15:15 Rebaseline edited by dglazkov@chromium.org
Added title. (diff)
15:09 Rebaseline edited by dglazkov@chromium.org
Clean up rebaseline doc. (diff)
15:05 Changeset [56362] by kdecker@apple.com

Reviewed by Simon Fraser.

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

  • rendering/RenderEmbeddedObject.cpp: (WebCore::RenderEmbeddedObject::RenderEmbeddedObject): Fix a find/repalce mistake from my earlier patch. Rename m_setShowsMissingPluginIndicator -> m_showsMissingPluginIndicator. (WebCore::RenderEmbeddedObject::updateWidget): Don't update the widget if we're showing the missing plug-in indicator. (WebCore::RenderEmbeddedObject::paint):Fix a find/repalce mistake from my earlier patch. Rename m_setShowsMissingPluginIndicator -> m_showsMissingPluginIndicator. (WebCore::RenderEmbeddedObject::paintReplaced): Ditto.
  • rendering/RenderEmbeddedObject.h: Ditto. (WebCore::RenderEmbeddedObject::setShowsMissingPluginIndicator): Ditto. (WebCore::RenderEmbeddedObject::showsMissingPluginIndicator): Ditto.
  • Plugins/WebNetscapePluginView.mm:
  • Plugins/WebNullPluginView.h: Removed.
  • Plugins/WebNullPluginView.mm: Removed.
  • Resources/nullplugin.tiff: Removed.
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::createPlugin): Invoke the resource load delegate if the plug-in failed to load.
14:56 Changeset [56361] by joepeck@webkit.org

2010-03-22 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by David Kilzer.

Upstream Part of the iPhone Platform sharing code with the Mac Platform
https://bugs.webkit.org/show_bug.cgi?id=35870
<rdar://problem/7707283>

Part 2 of 3: No changes other than moving code around.
Copied KeyEvent function from platform/mac to a platform/cocoa shared directory.

  • WebCore.xcodeproj/project.pbxproj: Added cocoa group and new files.
  • platform/cocoa/KeyEventCocoa.h: Added.
  • platform/cocoa/KeyEventCocoa.mm: Copied from WebCore/platform/mac/KeyEventMac.mm. (WebCore::keyIdentifierForKeyEvent): (WebCore::windowsKeyCodeForKeyEvent): Include the shared code.
  • platform/mac/KeyEventMac.mm:
14:24 Changeset [56360] by joepeck@webkit.org

2010-03-22 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by David Kilzer.

Part 1 of 3: Upstream iPhone KeyEvent platform code and share with Mac platform
https://bugs.webkit.org/show_bug.cgi?id=35870
<rdar://problem/7707283>

  • WebCore.xcodeproj/project.pbxproj: Added iphone group and new files.
  • platform/iphone: Added.
  • platform/iphone/KeyEventCodesIPhone.h: Added.
  • platform/iphone/KeyEventIPhone.mm: Added. Ensure PLATFORM(IPHONE)
  • platform/mac/KeyEventMac.mm: Ensure PLATFORM(MAC)
14:14 Changeset [56359] by kenneth@webkit.org

Unreviewed.

Skip editing test that fails

  • platform/qt/Skipped:
14:08 Changeset [56358] by krit@webkit.org

2010-03-22 Dirk Schulze <krit@webkit.org>

Out of bounds read in SVG feColorMatrix filter
https://bugs.webkit.org/show_bug.cgi?id=32714

SVG Test has text in it. This caused a wrong DRT result on Mac.
The other result was created on Windows.

  • platform/mac/svg/filters/feColorMatrix-values-expected.txt:
13:38 Changeset [56357] by kdecker@apple.com

Attempt to fix the Leopard build.

  • rendering/RenderEmbeddedObject.cpp: Add ".f" to contants.
13:28 Changeset [56356] by krit@webkit.org

2010-03-22 Dirk Schulze <krit@webkit.org>

Out of bounds read in SVG feColorMatrix filter
https://bugs.webkit.org/show_bug.cgi?id=32714

Add missing expected result.

  • platform/mac/svg/filters/feColorMatrix-values-expected.txt: Added.
13:24 QtWebKitBackportingFixes edited by laszlo.1.gombos@nokia.com
(diff)
12:46 Changeset [56355] by krit@webkit.org

2010-03-22 Justin Schuh <jschuh@chromium.org>

Reviewed by Dirk Schulze.

Out of bounds read in SVG feColorMatrix filter
https://bugs.webkit.org/show_bug.cgi?id=32714

Prevents an invalid read when a valid values attribute is not
supplied for an feColorMatrix SVG filter. Also fixes general
handling of missing or invalid values attribute.

Tests: svg/filters/feColorMatrix-invalid-value.svg

svg/filters/feColorMatrix-values.svg

  • svg/SVGFEColorMatrixElement.cpp: (WebCore::SVGFEColorMatrixElement::build):

2010-03-22 Justin Schuh <jschuh@chromium.org>

Reviewed by Dirk Schulze.

Out of bounds read in SVG feColorMatrix filter
https://bugs.webkit.org/show_bug.cgi?id=32714

Prevents an invalid read when a valid values attribute is not
supplied for an feColorMatrix SVG filter. Also fixes general
handling of missing or invalid values attribute.

  • platform/win/svg/filters: Added.
  • platform/win/svg/filters/feColorMatrix-values-expected.checksum: Added.
  • platform/win/svg/filters/feColorMatrix-values-expected.png: Added.
  • platform/win/svg/filters/feColorMatrix-values-expected.txt: Added.
  • svg/filters/feColorMatrix-invalid-value-expected.txt: Added.
  • svg/filters/feColorMatrix-invalid-value.svg: Added.
  • svg/filters/feColorMatrix-values.svg: Added.
12:37 Changeset [56354] by philn@webkit.org

2010-03-22 Philippe Normand <pnormand@igalia.com>

Reviewed by Eric Carlson.

[GStreamer] Use ImageBuffer API to do painting
https://bugs.webkit.org/show_bug.cgi?id=35783

New ImageGStreamer class abstracting the conversion of GStreamer
buffers to cairo surfaces. Using this makes the painting code of
the player more generic.

  • GNUmakefile.am:
  • platform/graphics/gstreamer/ImageGStreamer.h: Added. (WebCore::ImageGStreamer::image):
  • platform/graphics/gstreamer/ImageGStreamerCairo.cpp: Added. (ImageGStreamer::createImage): (ImageGStreamer::ImageGStreamer): (ImageGStreamer::~ImageGStreamer):
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::paint):
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h:
12:36 Changeset [56353] by bdakin@apple.com

Fix for <rdar://problem/7766437> With Web Inspector opened, a crash
occurs at Webcore:getMatchedCSSRules() when navigating to a
previous page

Reviewed by Darin Adler.

defaultView() can legitimately by null (as it is in this case), so
it must be null-checked.

  • inspector/InspectorDOMAgent.cpp:

(WebCore::InspectorDOMAgent::getStyles):
(WebCore::InspectorDOMAgent::getComputedStyle):
(WebCore::InspectorDOMAgent::getMatchedCSSRules):

12:30 Changeset [56352] by kdecker@apple.com

Reviewed by Darin.

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


  • loader/FrameLoader.cpp: (WebCore::FrameLoader::loadPlugin): Call setShowsMissingPluginIndicator(true) in cases that fail to generate a widget.
  • rendering/RenderEmbeddedObject.cpp: Added new missing plug-in related static constants (WebCore::RenderEmbeddedObject::RenderEmbeddedObject): Initialize newly boolean to false. (WebCore::RenderEmbeddedObject::paint): Added. (WebCore::RenderEmbeddedObject::paintReplaced): Added. Draws a subtle rounded rectangle containing the text "Missing Plug-in".
  • rendering/RenderEmbeddedObject.h: Added the following two methods below. (WebCore::RenderEmbeddedObject::setShowsMissingPluginIndicator): (WebCore::RenderEmbeddedObject::showsMissingPluginIndicator):
  • rendering/RenderWidget.h: Moved paint(PaintInfo&, int, int) from private to protected.
11:45 Changeset [56351] by ap@apple.com

Reviewed by John Sullivan.

https://bugs.webkit.org/show_bug.cgi?id=36455
Make WebKit more resistant against plug-in crashes

No tests, because crashing on build bots isn't good, even if it's only helper processes
that crash.

  • Plugins/Hosted/NetscapePluginInstanceProxy.h: (WebKit::NetscapePluginInstanceProxy::waitForReply): Protect "this", because this function needs it after waiting for reply. Some callers used to do this, but not all, and we really shouldn't depend on callers here.
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::wheelEvent): Don't protect the plug-in instance proxy, because this function doesn't use it after waiting for reply. (WebKit::NetscapePluginInstanceProxy::createBindingsInstance): Ditto.
  • Plugins/Hosted/ProxyInstance.mm: (WebKit::ProxyInstance::invoke): Added an m_instanceProxy null check for another code path.
11:44 Changeset [56350] by japhet@chromium.org

2010-03-22 Nate Chapin <japhet@chromium.org>

Unreviewed, Chromium expectations tweak.

Add fast/dom/error-to-string-stack-overflow.html to expected
failures due to http://trac.webkit.org/changeset/56329.

  • platform/chromium/test_expectations.txt:
11:40 Changeset [56349] by jwieczorek@webkit.org

2010-03-22 Jakub Wieczorek <faw217@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Phonon media backend: expose supported MIME types to WebCore

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

List all skipped media tests individually and unskip two fixed by the patch.

  • platform/qt/Skipped:

2010-03-22 Jakub Wieczorek <faw217@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Phonon media backend: expose supported MIME types to WebCore

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

MediaPlayerPrivate should expose the MIME types that are supported
by the underlying backend in Phonon.

  • platform/graphics/qt/MediaPlayerPrivatePhonon.cpp: (WebCore::MediaPlayerPrivate::supportedTypesCache): (WebCore::MediaPlayerPrivate::getSupportedTypes): (WebCore::MediaPlayerPrivate::supportsType):
  • platform/graphics/qt/MediaPlayerPrivatePhonon.h:
11:33 Changeset [56348] by ggaren@apple.com

JavaScriptCore: Fixed <rdar://problem/7728196> REGRESSION (r46701): -(-2147483648)
evaluates to -2147483648 on 32 bit (35842)

Reviewed by Sam Weinig.

Two ways to fix the same bug:

  1. Check for overflow when negating, since negating the largest negative

int causes overflow.

  1. Constant-fold even when negating a negative, since, like they say in

high school, "math works."

  • assembler/MacroAssemblerARM.h:

(JSC::MacroAssemblerARM::branchNeg32):

  • assembler/MacroAssemblerX86Common.h:

(JSC::MacroAssemblerX86Common::branchNeg32): Added a branching version
of the negate operator.

  • jit/JITArithmetic.cpp:

(JSC::JIT::emit_op_negate): Use the branching version of the negate
operator to check for overflow.

(JSC::JIT::emitSlow_op_negate): Link the check for overflow to a slow case.
(We could emit inline code for this, since we know what the result would
be, but that's probably just a waste of generated code.)

  • parser/Grammar.y: Constant fold even when negating a negative.

LayoutTests: Added a test for <rdar://problem/7728196> REGRESSION (r46701): -(-2147483648)
evaluates to -2147483648 on 32 bit (35842)

Reviewed by Sam Weinig.

  • fast/js/negate-overflow-expected.txt: Added.
  • fast/js/negate-overflow.html: Added.
11:18 Changeset [56347] by apavlov@chromium.org

2010-03-22 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: AuditRules still use getMatchedCSSRules as a part of the img-related audit.
https://bugs.webkit.org/show_bug.cgi?id=36424

  • inspector/front-end/AuditRules.js: (WebInspector.AuditRules.ImageDimensionsRule.prototype.doRun): (WebInspector.AuditRules.ImageDimensionsRule.prototype.doRun.receivedImages): (WebInspector.AuditRules.ImageDimensionsRule.prototype.doRun.pushImageNodes):
  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor):
11:17 Changeset [56346] by darin@chromium.org

2010-03-22 Darin Fisher <darin@chromium.org>

Reviewed by Nate Chapin.

[chromium] Renderer crashes when navigating to a reference fragment in
a frame that has no current HistoryItem.

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

Test: fast/loader/crash-replacing-location-before-load.html

  • src/FrameLoaderClientImpl.cpp: (WebKit::FrameLoaderClientImpl::dispatchDidNavigateWithinPage):
10:51 Changeset [56345] by darin@apple.com

2010-03-22 Darin Adler <darin@apple.com>

Reviewed by Dan Bernstein.

TextBreakIteratorICU.cpp is incompatible with new UBreakIterator type in ICU 4.4
https://bugs.webkit.org/show_bug.cgi?id=36381

  • platform/text/TextBreakIteratorICU.cpp: (WebCore::setUpIterator): Use reinterpret_cast instead of static_cast or relying on conversion to void*. (WebCore::textBreakFirst): Ditto. (WebCore::textBreakLast): Ditto. (WebCore::textBreakNext): Ditto. (WebCore::textBreakPrevious): Ditto. (WebCore::textBreakPreceding): Ditto. (WebCore::textBreakFollowing): Ditto. (WebCore::textBreakCurrent): Ditto. (WebCore::isTextBreak): Ditto. (WebCore::setUpIteratorWithRules): Ditto.
10:48 Changeset [56344] by eric.carlson@apple.com

2010-03-22 Eric Carlson <eric.carlson@apple.com>

Reviewed by Sam Weinig.

Typo in GPL license text
https://bugs.webkit.org/show_bug.cgi?id=36442

Fix typo - "aint with this library" -> "along with this library".

  • bindings/scripts/CodeGenerator.pm:
  • bindings/scripts/CodeGeneratorJS.pm:
  • bindings/scripts/CodeGeneratorObjC.pm:
  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/scripts/IDLParser.pm:
  • bindings/scripts/IDLStructure.pm:
  • bindings/scripts/generate-bindings.pl:
10:42 Changeset [56343] by jwieczorek@webkit.org

2010-03-22 Jakub Wieczorek <jwieczorek@webkit.org>

Reviewed by Simon Hausmann.

[Qt] Don't construct a QLineEdit every time when painting a text field
https://bugs.webkit.org/show_bug.cgi?id=36373

Instead, keep one instance per RenderTheme around.

  • platform/qt/RenderThemeQt.cpp: (WebCore::findFrameLineWidth):

2010-03-22 Jakub Wieczorek <jwieczorek@webkit.org>

Reviewed by Simon Hausmann.

[Qt] Don't construct a QLineEdit every time when painting a text field
https://bugs.webkit.org/show_bug.cgi?id=36373

Add a simple benchmark covering this area.

  • tests/benchmarks/painting/tst_painting.cpp: (tst_Painting::textAreas):
10:39 Changeset [56342] by ddkilzer@apple.com

<http://webkit.org/b/36431> Clean up 'int' use in UString.cpp after r54789

Reviewed by Darin Adler.

  • runtime/UString.cpp:

(JSC::UString::from): Changed argument type from 'unsigned int'
to 'unsigned' to match WebKit coding style.
(JSC::UString::find): Changed static_cast<int>() to
static_cast<unsigned>() now that this method returns unsigned.
(JSC::UString::rfind): Ditto.

  • runtime/UString.h:

(JSC::UString::from): Changed argument type from 'unsigned int'
to 'unsigned' to match WebKit coding style.

10:38 Changeset [56341] by eric@webkit.org

2010-03-22 Luiz Agostini <luiz.agostini@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Multiselect Popup - adjusting QtAbstractWebPopup
https://bugs.webkit.org/show_bug.cgi?id=36186

QtAbstractWebPopup must provide new methods to be used by its descendants that
wish to handle <select multiple> elements.

  • platform/qt/QtAbstractWebPopup.cpp: (WebCore::QtAbstractWebPopup::selectItem): (WebCore::QtAbstractWebPopup::multiple):
  • platform/qt/QtAbstractWebPopup.h:
10:20 Changeset [56340] by eric@webkit.org

2010-03-22 Alex Milowski <alex@milowski.com>

Reviewed by Kenneth Rohde Christiansen.

Basic tests of mfrac (fractions)

  • mathml/presentation/fractions.xhtml: Added.
  • mathml/presentation/fractions-vertical-alignment.xhtml: Added
  • platform/mac/mathml/presentation/fractions-expected.checksum: Added.
  • platform/mac/mathml/presentation/fractions-expected.png: Added.
  • platform/mac/mathml/presentation/fractions-expected.txt: Added.
  • platform/mac/mathml/presentation/fractions-vertical-alignment-expected.checksum: Added.
  • platform/mac/mathml/presentation/fractions-vertical-alignment-expected.png: Added.
  • platform/mac/mathml/presentation/fractions-vertical-alignment-expected.txt: Added.

2010-03-22 Alex Milowski <alex@milowski.com>

Reviewed by Kenneth Rohde Christiansen.

Added basic support for mfrac (fractions)

Test: mathml/presentation/fractions.xhtml

  • WebCore.xcodeproj/project.pbxproj:
  • mathml/MathMLInlineContainerElement.cpp: (WebCore::MathMLInlineContainerElement::createRenderer):
  • mathml/RenderMathMLFraction.cpp: Added. (WebCore::RenderMathMLFraction::RenderMathMLFraction): (WebCore::RenderMathMLFraction::updateFromElement): (WebCore::RenderMathMLFraction::addChild):
  • mathml/RenderMathMLFraction.h: Added.
09:47 Changeset [56339] by darin@apple.com

WebKit: * StringsNotToBeLocalized.txt: Updated for recent changes.

WebKit/win: * WebCoreLocalizedStrings.cpp:
(WebCore::missingPluginText): Fixed localization helper text to match the same
string from Mac WebKit.

09:46 Rebaseline created by dglazkov@chromium.org
Ported rebaseline tool documentation from Chromium wiki
09:45 Changeset [56338] by ap@apple.com

Reviewed by Oliver Hunt.

https://bugs.webkit.org/show_bug.cgi?id=36400
Add a test for r15750 (Flash crashes after it replaces itself via a document.write())

  • plugins/destroy-plugin-from-callback-expected.txt: Added.
  • plugins/destroy-plugin-from-callback.html: Added.
09:29 Changeset [56337] by kdecker@apple.com

Reviewed by John Sullivan.

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

  • page/mac/WebCoreViewFactory.h: Added -missingPluginText method
  • platform/LocalizedStrings.h: Added missingPluginText()
  • platform/gtk/LocalizedStringsGtk.cpp: Likewise.
  • platform/mac/LocalizedStringsMac.mm: Same here.
  • platform/qt/Localizations.cpp: Ditto.
  • English.lproj/Localizable.strings: Added "Missing Plug-in" string.
  • src/LocalizedStrings.cpp: (WebCore::missingPluginText): Added
  • WebCoreSupport/WebViewFactory.mm: (-[WebViewFactory missingPluginText]): Added.
09:24 Changeset [56336] by eric@webkit.org

2010-03-22 Chris Jerdonek <cjerdonek@webkit.org>

Reviewed by Adam Barth.

Refactored the cpu_count() code in executive.py.

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

  • Scripts/webkitpy/executive.py:
    • Moved the import of the multiprocessing module to the top of the file rather than importing from within a function.
08:24 Changeset [56335] by tonikitoo@webkit.org

Revert "2010-03-18 Jesus Sanchez-Palencia <jesus.palencia@openbossa.org>"

This reverts commit d76f355a659be5b3384f9c26bc6ad64f3256c9fb.

Conflicts:

WebKitTools/ChangeLog

08:09 Changeset [56334] by eric@webkit.org

2010-03-22 Yi Shen <shenyi2006@gmail.com>

Reviewed by Simon Hausmann.

https://bugs.webkit.org/show_bug.cgi?id=35933
[Qt] [Symbian] Can not backward select (highlight) text using virtual keyboard
Make sure the selection start index is smaller than the selection end index.

  • Api/qwebpage.cpp: (QWebPagePrivate::inputMethodEvent):
  • tests/qwebpage/tst_qwebpage.cpp: (tst_QWebPage::inputMethods):
07:37 Changeset [56333] by eric@webkit.org

2010-03-22 Jedrzej Nowacki <jedrzej.nowacki@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Add support for syntax checking in the QtScript API.

New class was created; the QScriptSyntaxCheckResult which main
responsibility is to provide results of the ECMA Script code
syntax check. The class is not fully functional as the JSC C API
doesn't expose an error column number, but it is a good start point
for a future development.

[Qt] QtScript functionality should be extended by syntax checking.
https://bugs.webkit.org/show_bug.cgi?id=36123

  • qt/api/QtScript.pro:
  • qt/api/qscriptengine.cpp: (QScriptEngine::checkSyntax):
  • qt/api/qscriptengine.h:
  • qt/api/qscriptengine_p.cpp: (QScriptEnginePrivate::checkSyntax):
  • qt/api/qscriptengine_p.h:
  • qt/api/qscriptsyntaxcheckresult.cpp: Added. (QScriptSyntaxCheckResult::QScriptSyntaxCheckResult): (QScriptSyntaxCheckResult::~QScriptSyntaxCheckResult): (QScriptSyntaxCheckResult::operator=): (QScriptSyntaxCheckResult::state): (QScriptSyntaxCheckResult::errorLineNumber): (QScriptSyntaxCheckResult::errorColumnNumber): (QScriptSyntaxCheckResult::errorMessage):
  • qt/api/qscriptsyntaxcheckresult.h: Added.
  • qt/api/qscriptsyntaxcheckresult_p.cpp: Added. (QScriptSyntaxCheckResultPrivate::~QScriptSyntaxCheckResultPrivate): (QScriptSyntaxCheckResultPrivate::errorMessage): (QScriptSyntaxCheckResultPrivate::errorLineNumber):
  • qt/api/qscriptsyntaxcheckresult_p.h: Added. (QScriptSyntaxCheckResultPrivate::get): (QScriptSyntaxCheckResultPrivate::QScriptSyntaxCheckResultPrivate): (QScriptSyntaxCheckResultPrivate::state): (QScriptSyntaxCheckResultPrivate::errorColumnNumber):
  • qt/tests/qscriptengine/tst_qscriptengine.cpp: (tst_QScriptEngine::checkSyntax_data): (tst_QScriptEngine::checkSyntax):
06:26 Changeset [56332] by yurys@chromium.org

2010-03-22 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Fix inspected Page crash in destructor when Web Inspector is open.

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

  • src/js/InjectDispatch.js: (inspectedPageDestroyed): Added stub for missing method.
05:49 Changeset [56331] by cjerdonek@webkit.org

Created a function for getting a module-specific logging.logger
based on the file value of the module.

Reviewed by Adam Barth.

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

This function allows us to get the module-specific logger for
a module without having to hard-code the fully-qualified name
of the module in the module itself. The code can be the same
in every case: "_log = logutils.get_logger(file)".

  • Scripts/webkitpy/init/logutils.py: Added.
    • Added a module with a get_logger() function to return a module-specific logger based on the module's file variable.
  • Scripts/webkitpy/init/logutils_unittest.py: Added.
    • Added unit tests for logutils.py.
  • Scripts/webkitpy/init/unittests.py:
    • Added logutils_unittest to the list of imports.
04:43 QtWebKitBackportingFixes edited by hausmann@webkit.org
(diff)
04:07 Changeset [56330] by eric@webkit.org

2010-03-22 Jochen Eisinger <jochen@chromium.org>

Reviewed by Jeremy Orlow.

Added methods to WebSecurityOrigin for invoking
SecurityOrigin::canAccess and SecurityOrigin::Create
https://bugs.webkit.org/show_bug.cgi?id=36356

  • public/WebSecurityOrigin.h:
  • src/WebSecurityOrigin.cpp: (WebKit::WebSecurityOrigin::create): (WebKit::WebSecurityOrigin::canAccess):
02:46 Changeset [56329] by yurys@chromium.org

2010-03-22 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Handle worker exceptions in V8MessageHandler like it's done in regular documents. This way all worker exceptions will be logged in the console not only those which happen in event listeners.

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

  • bindings/v8/V8AbstractEventListener.cpp: (WebCore::V8AbstractEventListener::invokeEventHandler): Removed explicit call to reportException.
  • bindings/v8/V8Utilities.cpp: reportException function was removed since it's not used. (WebCore::getScriptExecutionContext):
  • bindings/v8/V8Utilities.h:
  • bindings/v8/WorkerContextExecutionProxy.cpp: (WebCore::v8MessageHandler): (WebCore::WorkerContextExecutionProxy::initContextIfNeeded): Setup message handler when first worker context is created.

2010-03-22 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Test that uncaught exception thrown from setTimeout callback in a Worker is reported to the worker object.

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

  • fast/workers/resources/worker-exception-in-timeout-callback.js: Added.
  • fast/workers/worker-script-error-expected.txt:
  • fast/workers/worker-script-error.html:
02:43 Changeset [56328] by eric@webkit.org

2010-03-22 Kenneth Rohde Christiansen <kenneth@webkit.org>

Reviewed by Simon Hausmann.

[Qt] REGRESSION (r56209): fast/media/print-restores-previous-mediatype.htm crashes
https://bugs.webkit.org/show_bug.cgi?id=36386

  • platform/qt/Skipped: unskip the test as it now passes.

2010-03-22 Kenneth Rohde Christiansen <kenneth@webkit.org>

Reviewed by Simon Hausmann.

[Qt] REGRESSION (r56209): fast/media/print-restores-previous-mediatype.htm crashes
https://bugs.webkit.org/show_bug.cgi?id=36386

Fix the regression by implementing a null printer for Qt DRT.

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::NullPrinter::NullPaintEngine::begin): (WebCore::NullPrinter::NullPaintEngine::end): (WebCore::NullPrinter::NullPaintEngine::type): (WebCore::NullPrinter::NullPaintEngine::drawPixmap): (WebCore::NullPrinter::NullPaintEngine::updateState): (WebCore::NullPrinter::paintEngine): (WebCore::DumpRenderTree::dryRunPrint):
02:24 Changeset [56327] by eric@webkit.org

2010-03-22 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Simon Hausmann.

Add EFL-specific code to page/EventHandler.cpp.
http://webkit.org/b/36306

  • page/EventHandler.cpp:
02:06 Changeset [56326] by eric@webkit.org

2010-03-22 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Simon Hausmann.

Add EFL-specific code to platform/DragImage.h and
platform/DragData.h.
http://webkit.org/b/36310

  • platform/DragImage.h:
  • platform/DragData.h:
01:45 Changeset [56325] by eric@webkit.org

2010-03-22 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Simon Hausmann.

Add EFL-specific code to platform/FileSystem.h.
http://webkit.org/b/36313

  • platform/FileSystem.h:
01:26 Changeset [56324] by eric@webkit.org

2010-03-22 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Simon Hausmann.

Add EFL-specific code to platform/Cursor.h.
http://webkit.org/b/36319

  • platform/Cursor.h:
01:05 Changeset [56323] by eric@webkit.org

2010-03-22 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Simon Hausmann.

Add EFL-specific code to platform/NotImplemented.h.
http://webkit.org/b/36320

  • platform/NotImplemented.h:
Note: See TracTimeline for information about the timeline view.