Timeline



Apr 24, 2009:

9:03 PM Changeset in webkit [42861] by kov@webkit.org
  • 2 edits in trunk/WebCore

2009-04-25 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Mark Rowe.

Only set the new URL once for the request. Doing it a second time
after the call to willSendRequest was causing crashes when
redirected requests got cancelled.

  • platform/network/soup/ResourceHandleSoup.cpp: (WebCore::restartedCallback):
8:44 PM Changeset in webkit [42860] by kevino@webkit.org
  • 8 edits in trunk

wx build fix. Switching JSCore from a static lib to a dynamic lib to match the Apple build and fix symbol exports.

8:19 PM Changeset in webkit [42859] by kevino@webkit.org
  • 2 edits in trunk/WebCore

wx build fix. Adding ScriptEventListener.cpp.

5:27 PM Changeset in webkit [42858] by jmalonzo@webkit.org
  • 2 edits
    1 move in trunk/JavaScriptCore

2009-04-24 Laszlo Gombos <Laszlo Gombos>

Rubber-stamped by Mark Rowe.

https://bugs.webkit.org/show_bug.cgi?id=25337
Move ThreadingQt.cpp under the qt directory.

5:18 PM Changeset in webkit [42857] by jmalonzo@webkit.org
  • 2 edits
    1 move in trunk/JavaScriptCore

2009-04-24 Laszlo Gombos <Laszlo Gombos>

Rubber-stamped by Mark Rowe.

https://bugs.webkit.org/show_bug.cgi?id=25338
Move ThreadingGtk.cpp under the gtk directory.

  • GNUmakefile.am:
  • wtf/ThreadingGtk.cpp: Removed.
  • wtf/gtk/ThreadingGtk.cpp: Copied from JavaScriptCore/wtf/ThreadingGtk.cpp.
5:05 PM Changeset in webkit [42856] by jmalonzo@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-24 Jan Michael Alonzo <jmalonzo@webkit.org>

Rubber-stamped by Gustavo Noronha.

Fix braces to comply with style guidelines.

  • WebCoreSupport/PasteboardHelperGtk.cpp: (WebKit::PasteboardHelperGtk::getClipboard): (WebKit::PasteboardHelperGtk::getCopyTargetList): (WebKit::PasteboardHelperGtk::getPasteTargetList):
5:05 PM Changeset in webkit [42855] by jmalonzo@webkit.org
  • 6 edits in trunk

2009-04-24 Sergio García-Cuevas <sergio_gcg@telefonica.net>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=17267
[GTK] Primary selection/clipboard support

Copy the link location to the primary selection as well as the
clipboard selection when using the "copy link selection" context
menu entry.

  • platform/gtk/PasteboardGtk.cpp: (WebCore::Pasteboard::writeURL):
  • platform/gtk/PasteboardHelper.h:

2009-04-24 Sergio García-Cuevas <sergio_gcg@telefonica.net>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=17267
[GTK] Primary selection/clipboard support

Add a method for getting the primary selection.

  • WebCoreSupport/PasteboardHelperGtk.cpp: (WebKit::PasteboardHelperGtk::getPrimary):
  • WebCoreSupport/PasteboardHelperGtk.h:
4:54 PM Changeset in webkit [42854] by eric@webkit.org
  • 7 edits in trunk

Reviewed by Simon Fraser.

https://bugs.webkit.org/show_bug.cgi?id=25383
Changing SVG size via JS does not update CSS width/height

Add an ugly hack to fix CSS width/height updates from JS.

We can't easily call setAttribute when width/height changes
as that would cause infinite recursion (as we'd try to set
the animation storage from the attributes)

For now, when we get notification that JS changed width/height
we copy what parseMappedAttribute does and update the CSS values as well.

A better fix for this would be:
https://bugs.webkit.org/show_bug.cgi?id=25383

  • svg/SVGSVGElement.cpp: (WebCore::updateCSSForAttribute): (WebCore::SVGSVGElement::svgAttributeChanged):
4:51 PM Changeset in webkit [42853] by barraclough@apple.com
  • 6 edits in trunk/JavaScriptCore

2009-04-24 Gavin Barraclough <barraclough@apple.com>

Reviewed by Sam "Wesley" Weinig.

Improve performance to YARR interpreter.
(From about 3x slower than PCRE on regex-dna to about 30% slower).

  • yarr/RegexCompiler.cpp: (JSC::Yarr::RegexPatternConstructor::setupAlternativeOffsets):
  • yarr/RegexInterpreter.cpp: (JSC::Yarr::Interpreter::checkCharacter): (JSC::Yarr::Interpreter::checkCasedCharacter): (JSC::Yarr::Interpreter::backtrackPatternCharacter): (JSC::Yarr::Interpreter::backtrackPatternCasedCharacter): (JSC::Yarr::Interpreter::matchParentheticalAssertionBegin): (JSC::Yarr::Interpreter::matchParentheticalAssertionEnd): (JSC::Yarr::Interpreter::backtrackParentheticalAssertionBegin): (JSC::Yarr::Interpreter::backtrackParentheticalAssertionEnd): (JSC::Yarr::Interpreter::matchDisjunction): (JSC::Yarr::Interpreter::interpret): (JSC::Yarr::ByteCompiler::atomPatternCharacter): (JSC::Yarr::ByteCompiler::atomParenthesesSubpatternBegin): (JSC::Yarr::ByteCompiler::atomParentheticalAssertionBegin): (JSC::Yarr::ByteCompiler::closeAlternative): (JSC::Yarr::ByteCompiler::closeBodyAlternative): (JSC::Yarr::ByteCompiler::atomParenthesesEnd): (JSC::Yarr::ByteCompiler::regexBegin): (JSC::Yarr::ByteCompiler::regexEnd): (JSC::Yarr::ByteCompiler::alterantiveBodyDisjunction): (JSC::Yarr::ByteCompiler::alterantiveDisjunction): (JSC::Yarr::ByteCompiler::emitDisjunction):
  • yarr/RegexInterpreter.h: (JSC::Yarr::ByteTerm::): (JSC::Yarr::ByteTerm::ByteTerm): (JSC::Yarr::ByteTerm::BodyAlternativeBegin): (JSC::Yarr::ByteTerm::BodyAlternativeDisjunction): (JSC::Yarr::ByteTerm::BodyAlternativeEnd): (JSC::Yarr::ByteTerm::AlternativeBegin): (JSC::Yarr::ByteTerm::AlternativeDisjunction): (JSC::Yarr::ByteTerm::AlternativeEnd): (JSC::Yarr::ByteTerm::SubpatternBegin): (JSC::Yarr::ByteTerm::SubpatternEnd):
  • yarr/RegexJIT.cpp: (JSC::Yarr::RegexGenerator::generateParentheticalAssertion):
  • yarr/RegexPattern.h:
4:41 PM Changeset in webkit [42852] by kmccullough@apple.com
  • 2 edits in trunk/WebCore

2009-04-24 Kevin McCullough <kmccullough@apple.com>

Reviewed by Dan Bernstein.

We should not show the deletion UI for blockquotes in mail but they do
show now that we only check for one visible border.

  • editing/DeleteButtonController.cpp: (WebCore::isDeletableElement):
4:28 PM Changeset in webkit [42851] by mrowe@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-04-24 Rob Raguet-Schofield <ragfield@gmail.com>

Rubber-stamped by Mark Rowe.

  • wtf/CurrentTime.h: Fix a typo in a comment.
4:05 PM Changeset in webkit [42850] by mitz@apple.com
  • 2 edits in trunk/WebCore
  • address Hyatt's review comments on the last patch
  • rendering/RenderLayer.cpp: (WebCore::performOverlapTests): (WebCore::RenderLayer::paintLayer):
4:00 PM Changeset in webkit [42849] by mitz@apple.com
  • 10 edits
    1 add in trunk/WebCore

Reviewed by Dave Hyatt.

  • WebCore.xcodeproj/project.pbxproj: Added OverlapTestRequestClient.h.
  • page/FrameView.cpp: (WebCore::FrameView::reset): Initialize m_isOverlapped. (WebCore::FrameView::useSlowRepaints): Also check m_isOverlapped. (WebCore::FrameView::removeSlowRepaintObject): Use useSlowRepaints() so overlapping is taken into account. (WebCore::FrameView::setIsOverlapped): Added. Sets m_isOverlapped and enables or disables copy-on-scroll accordingly.
  • page/FrameView.h: Added setIsOverlapped() and m_isOverlapped.
  • rendering/OverlapTestRequestClient.h: Added. During foreground painting, an OverlapTestRequestClient can make a request to test if any subsequently- painted layers overlap a rect. The test result is delivered to the requestClient via setOverlapTestResult(). (WebCore::OverlapTestRequestClient::~OverlapTestRequestClient):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paint): Pass an OverlapTestRequestMap to paintLayer(). When done painting, inform any remaining requestClients that they are not overlapped.

(WebCore::RenderLayer::paintLayer): Take an OverlapTestRequestMap and
pass it down to child and reflection layers. Inform requestClients if this
layer overlaps their requested rects.

  • rendering/RenderLayer.h:
  • rendering/RenderObject.h: (WebCore::RenderObject::PaintInfo::PaintInfo): Added an OverlapTestRequestMap member.
  • rendering/RenderReplica.cpp: (WebCore::RenderReplica::paint): For now, pass a 0 OverlapTestRequestMap to paintLayer().
  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::paint): For FrameViews, request an overlap test with the frame. (WebCore::RenderWidget::setOverlapTestResult): Call FrameView::setIsOverlapped() with the result of the overlap test.
  • rendering/RenderWidget.h: Made RenderWidget an OverlapTestRequestClient.
3:52 PM Changeset in webkit [42848] by Simon Fraser
  • 3 edits in trunk/LayoutTests

2009-04-24 Pierre d'Herbemont <pdherbemont@apple.com>

Reviewed by Simon Fraser

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

Fixed a race condition on the first 'played' attribute test start.

Previously we were starting nearly right away to play, and probably
before receiving the 'canplay' event. Now we ensure we'll wait for
that 'canplay' event before attempting to play.

  • media/video-played-expected.txt: Updated
  • media/video-played.html:
3:45 PM Changeset in webkit [42847] by ggaren@apple.com
  • 2 edits in trunk/LayoutTests

2009-04-24 Geoffrey Garen <ggaren@apple.com>

Rubber stamped by Eric Seidel.


Updated test to turn the buildbot green after Eric's last check-in.

  • platform/mac/svg/W3C-SVG-1.1/animate-elem-36-t-expected.txt:
3:30 PM Changeset in webkit [42846] by hyatt@apple.com
  • 22 edits
    2 deletes in trunk/WebCore

2009-04-24 David Hyatt <hyatt@apple.com>

Reviewed by Simon Fraser, Dan Bernstein.

Speed up the PLT by devirtualizing the height() function on InlineBox. virtualizing it in order to make the height() computation dynamic caused
a ~0.5% slowdown. This patch does the following to get the speed back:


(a) Devirtualizes isText and forces inline box creators to set the bit as needed. This actually resulted in simplified code, since ListMarkerBox could
then be removed.
(b) Reduces the height() call count. In some cases the code was repeatedly calling height(), which used to be fine when the function was just an inline
member variable access. The call sites have been patched to cut down on extra height() calls now that it is more expensive.
(c) Devirtualize height() except on SVG boxes. For all non-SVG, the height() function on InlineBox handles the computation. For SVG boxes, a new bit has
been set on InlineBoxes (isSVG()) that indicates that the virtual svgBoxHeight() function should be called to retrieve the height instead.

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • rendering/InlineBox.cpp: (WebCore::InlineBox::height):
  • rendering/InlineBox.h: (WebCore::InlineBox::InlineBox): (WebCore::InlineBox::isSVG): (WebCore::InlineBox::setIsSVG): (WebCore::InlineBox::isText): (WebCore::InlineBox::setIsText): (WebCore::InlineBox::svgBoxHeight):
  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::placeBoxesVertically): (WebCore::InlineFlowBox::paintBoxDecorations): (WebCore::InlineFlowBox::paintMask):
  • rendering/InlineFlowBox.h:
  • rendering/InlineTextBox.cpp:
  • rendering/InlineTextBox.h:
  • rendering/ListMarkerBox.cpp: Removed.
  • rendering/ListMarkerBox.h: Removed.
  • rendering/RenderListMarker.cpp: (WebCore::RenderListMarker::createInlineBox):
  • rendering/RenderSVGInline.cpp: (WebCore::RenderSVGInline::createFlowBox):
  • rendering/RenderSVGInlineText.cpp: (WebCore::RenderSVGInlineText::createTextBox):
  • rendering/RenderSVGText.cpp: (WebCore::RenderSVGText::createRootBox):
  • rendering/RenderText.cpp: (WebCore::RenderText::createInlineTextBox):
  • rendering/RootInlineBox.cpp:
  • rendering/RootInlineBox.h:
  • rendering/SVGInlineFlowBox.h: (WebCore::SVGInlineFlowBox::svgBoxHeight):
  • rendering/SVGInlineTextBox.h: (WebCore::SVGInlineTextBox::svgBoxHeight):
  • rendering/SVGRootInlineBox.h: (WebCore::SVGRootInlineBox::svgBoxHeight):
3:25 PM Changeset in webkit [42845] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

Reviewed by Adam Roben.

Add PrettyPatch support to run-webkit-tests

  • Scripts/run-webkit-tests:
3:09 PM Changeset in webkit [42844] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix

3:00 PM Changeset in webkit [42843] by Dimitri Glazkov
  • 24 edits
    4 adds in trunk/WebCore

2009-04-24 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=25370
Introduce ScriptEventListener to refactor away direct references to
JSLazyEventListener in WebCore code.

  • GNUmakefile.am: Added ScriptEventListener to project.
  • WebCore.pro: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • bindings/js/JSDOMGlobalObject.cpp: Removed unused JSLazyEventListener include.
  • bindings/js/JSLazyEventListener.cpp: Moved createAttributeEventListener helpers

to ScriptEventListener.

  • bindings/js/JSLazyEventListener.h: Ditto.
  • bindings/js/ScriptController.cpp: Removed unused JSLazyEventListener include.
  • bindings/js/ScriptEventListener.cpp: Added.
  • bindings/js/ScriptEventListener.h: Added.
  • bindings/v8/ScriptEventListener.cpp: Added.
  • bindings/v8/ScriptEventListener.h: Added.
  • dom/Document.cpp: Renamed JSLazyEventListener include to ScriptEventListener.
  • html/HTMLBodyElement.cpp: Ditto.
  • html/HTMLButtonElement.cpp: Ditto.
  • html/HTMLElement.cpp: Ditto.
  • html/HTMLFormElement.cpp: Ditto.
  • html/HTMLFrameElementBase.cpp: Ditto.
  • html/HTMLFrameSetElement.cpp: Ditto.
  • html/HTMLImageElement.cpp: Ditto.
  • html/HTMLInputElement.cpp: Ditto.
  • html/HTMLObjectElement.cpp: Ditto.
  • html/HTMLScriptElement.cpp: Ditto.
  • html/HTMLSelectElement.cpp: Ditto.
  • html/HTMLTextAreaElement.cpp: Ditto.
  • svg/SVGElement.cpp: Ditto.
  • svg/SVGSVGElement.cpp: Ditto.
2:53 PM Changeset in webkit [42842] by oliver@apple.com
  • 3 edits in trunk/JavaScriptCore

<rdar://problem/6050421> JavaScript register file should remap to release physical pages accumulated during deep recursion

Reviewed by Geoff Garen

We now track the maximum extent of the RegisterFile, and when we reach the final
return from JS (so the stack portion of the registerfile becomes empty) we see
if that extent is greater than maxExcessCapacity. If it is we use madvise or
VirtualFree to release the physical pages that were backing the excess.

2:52 PM Changeset in webkit [42841] by Darin Adler
  • 2 edits in trunk/WebCore

2009-04-24 Darin Adler <Darin Adler>

Reviewed by Sam Weinig.

Bug 25355: Crash when Creating New Tab or New Window when set to open Same Page
https://bugs.webkit.org/show_bug.cgi?id=25355
rdar://problem/6823543

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::loadItem): Added a null check.
2:34 PM Changeset in webkit [42840] by eric@webkit.org
  • 44 edits in trunk

Reviewed by Simon Fraser.

Fix RenderSVGImage to dump with full bounds and style information.
https://bugs.webkit.org/show_bug.cgi?id=25377

No functional changes, we're just now including more info in render tree dumps.

  • rendering/RenderObject.h: (WebCore::RenderObject::isSVGImage):
  • rendering/RenderSVGImage.h: (WebCore::RenderSVGImage::isSVGImage):
  • rendering/RenderTreeAsText.cpp: (WebCore::write):
  • rendering/SVGRenderTreeAsText.cpp: (WebCore::operator<<): (WebCore::write):
  • rendering/SVGRenderTreeAsText.h:
2:30 PM Changeset in webkit [42839] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

2009-04-24 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler.

Fix <rdar://problem/6761635>.


Make sure to keep an extra reference to the instance proxy in case the plug-in host crashes while
we're waiting for a reply.


  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::destroy):
2:18 PM Changeset in webkit [42838] by levin@chromium.org
  • 7 edits in trunk/WebCore

2009-04-24 David Levin <levin@chromium.org>

Reviewed by NOBODY (chromium build fix).

Rename "isInline" => "isAttribute" in v8 bindings (jsc was done in r42699).

  • bindings/v8/V8AbstractEventListener.cpp: (WebCore::V8AbstractEventListener::V8AbstractEventListener): (WebCore::V8AbstractEventListener::invokeEventHandler):
  • bindings/v8/V8AbstractEventListener.h:
  • bindings/v8/V8EventListenerList.cpp: (WebCore::V8EventListenerList::add): (WebCore::V8EventListenerList::remove): (WebCore::V8EventListenerList::clear):
  • bindings/v8/V8LazyEventListener.h: (WebCore::V8LazyEventListener::virtualisAttribute):
  • bindings/v8/custom/V8CustomEventListener.cpp: (WebCore::V8EventListener::V8EventListener):
  • bindings/v8/custom/V8CustomEventListener.h: (WebCore::V8EventListener::create): (WebCore::V8EventListener::virtualisAttribute):
2:16 PM Changeset in webkit [42837] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-24 Johan Bilien <jobi@via.ecp.fr>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=25373
Popup menu has the wrong size when popped up near the edge
of the screen

  • webkit/webkitwebview.cpp: (webkit_web_view_forward_context_menu_event): Let GTK place the popup menu instead of trying to do better and failing
1:48 PM Changeset in webkit [42836] by Simon Fraser
  • 19 edits in trunk/LayoutTests

2009-04-24 Pierre d'Herbemont <pdherbemont@apple.com>

Reviewed by Simon Fraser

<rdar://problem/6805928> Re-export layout test MPEG-4 file to remove unusual edit

Use better mp4 test file for media tests. The original mp4 file had an unusual
edit (non-zero start time) which is not supported by all media engines. The new
movie should also give more useful pixel test results.

  • media/content/test.mp4: Update the movie.
  • media/progress-event-total-expected.txt: Update test results to match movie duration and content changes.
  • media/progress-event-total.html: Ditto.
  • media/video-currentTime-set-expected.txt: Ditto.
  • media/video-loop-expected.txt: Ditto.
  • platform/mac/media/video-aspect-ratio-expected.checksum: Ditto.
  • platform/mac/media/video-aspect-ratio-expected.png: Ditto.
  • platform/mac/media/video-controls-rendering-expected.checksum: Ditto.
  • platform/mac/media/video-controls-rendering-expected.png: Ditto.
  • platform/mac/media/video-display-toggle-expected.checksum: Ditto.
  • platform/mac/media/video-display-toggle-expected.png: Ditto.
  • platform/mac/media/video-empty-source-expected.checksum: Ditto.
  • platform/mac/media/video-layer-crash-expected.checksum: Ditto.
  • platform/mac/media/video-layer-crash-expected.png: Ditto.
  • platform/mac/media/video-transformed-expected.checksum: Ditto.
  • platform/mac/media/video-transformed-expected.png: Ditto.
  • platform/mac/media/video-zoom-expected.checksum: Ditto.
  • platform/mac/media/video-zoom-expected.png: Ditto.
1:08 PM Changeset in webkit [42835] by sfalken@apple.com
  • 2 edits in trunk/WebKitTools

Fix nightly for Windows x64 users.
Installed browser location is written to a slightly different registry key in that case.


  • FindSafari/FindSafari.cpp: (getInstalledWebKitDirectory):
12:13 PM Changeset in webkit [42834] by kmccullough@apple.com
  • 1 edit
    2 adds in trunk/LayoutTests

2009-04-24 Kevin McCullough <kmccullough@apple.com>

When I moved the files I forgot to update the location of the image!

  • platform/mac/editing/deleting/deletionUI-differing-background-expected.txt: Added.
  • platform/mac/editing/deleting/deletionUI-differing-background.html: Added.
11:43 AM Changeset in webkit [42833] by Dimitri Glazkov
  • 2 edits in trunk/WebCore

2009-04-24 Nate Chapin <japhet@google.com>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25343
Fix Chromium/Skia bug where -webkit-box-shadow with 0,0 offset
ignores blur.

  • WebCore\platform\graphics\skia\GraphicsContextSkia.cpp (WebCore::GraphicsContext::setPlatformShadow): Add check against blur before clearing looper.
11:41 AM Changeset in webkit [42832] by kmccullough@apple.com
  • 1 edit
    3 deletes in trunk/LayoutTests

2009-04-24 Kevin McCullough <kmccullough@apple.com>

  • Build fix. Removed this test until I can figure out why it's failing.
  • platform/mac-leopard/editing/deleting/deletionUI-differing-background-expected.txt: Removed.
  • platform/mac/editing/deleting/deletionUI-differing-background-expected.txt: Removed.
  • platform/mac/editing/deleting/deletionUI-differing-background.html: Removed.
11:40 AM Changeset in webkit [42831] by Dimitri Glazkov
  • 3 edits
    2 adds in trunk

WebCore:

2009-04-24 Eric Roman <eroman@chromium.org>

Reviewed by Darin Adler.

Initialize TextIterator::m_textLength to 0.

This assures that TextIterator::length() will return 0 for cases when TextIterator's constructor returns early (because there is nothing to iterate over in the range).

<https://bugs.webkit.org/show_bug.cgi?id=25335>

Test: editing/selection/doubleclick-whitespace-img-crash.html

  • editing/TextIterator.cpp: (WebCore::TextIterator::TextIterator):

LayoutTests:

2009-04-24 Eric Roman <eroman@chromium.org>

Reviewed by Darin Adler.

Add a test for <https://bugs.webkit.org/show_bug.cgi?id=25335>.

To pass, this test must not access invalid memory when run (won't necessarily manifest as a crash for failures).

  • editing/selection/doubleclick-whitespace-img-crash-expected.txt: Added.
  • editing/selection/doubleclick-whitespace-img-crash.html: Added.
11:38 AM Changeset in webkit [42830] by Simon Fraser
  • 3 edits in trunk/WebKitLibraries

2009-04-24 Simon Fraser <Simon Fraser>

Source changes reviewed by Darin Adler

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

Update WebKitSystemInterface for Mac with fixes for video controller drawing.

  • libWebKitSystemInterfaceLeopard.a:
  • libWebKitSystemInterfaceTiger.a:
11:36 AM Changeset in webkit [42829] by Dimitri Glazkov
  • 1 edit
    1 add in trunk/WebCore

2009-04-24 Fumitoshi Ukai <ukai@google.com>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25329
Add indexed access method in v8 binding of ClientRectList.

  • bindings/v8/custom/V8ClientRectListCustom.cpp: Added.
11:32 AM Changeset in webkit [42828] by Dimitri Glazkov
  • 4 edits
    3 adds in trunk

WebCore:

2009-04-24 Fumitoshi Ukai <ukai@google.com>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25257
Use null for unspecified event listener attributes of XMLHttpRequest
and XMLHttpRequestUpload in v8 binding.

cf. http://www.whatwg.org/specs/web-apps/current-work/#event-handler-attributes
An event handler attribute, unless otherwise specified, can either
have the value null or be set to a Function object.
Initially, an event handler attribute must be set to null.

Test: fast/xmlhttprequest/xmlhttprequest-default-attributes.html

  • bindings/v8/custom/V8XMLHttpRequestCustom.cpp: (WebCore::ACCESSOR_GETTER): Changed to use v8::Null().
  • bindings/v8/custom/V8XMLHttpRequestUploadCustom.cpp: (WebCore::ACCESSOR_GETTER): Ditto.

LayoutTests:

2009-04-24 Fumitoshi Ukai <ukai@google.com>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25257
Verify XMLHttpRequest has correct attribute value by default.

  • fast/xmlhttprequest/resources/xmlhttprequest-default-attributes.js: Added.
  • fast/xmlhttprequest/xmlhttprequest-default-attributes-expected.txt: Added.
  • fast/xmlhttprequest/xmlhttprequest-default-attributes.html: Added.
11:02 AM Changeset in webkit [42827] by christian@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-24 Christian Dywan <christian@twotoasts.de>

Rubber-stamped by Gustavo Noronha.

  • webkit/webkitwebsettings.cpp: (_WebKitWebSettingsPrivate::webkit_web_settings_class_init): Reword the description of the enable-spell-checking property and correct the mentioning of the default language used.
10:42 AM Changeset in webkit [42826] by kmccullough@apple.com
  • 1 edit
    3 adds in trunk/LayoutTests

2009-04-24 Kevin McCullough <kmccullough@apple.com>

Speculative fix for leopard build-bot until I have a fix on leopard.

  • platform/mac-leopard/editing: Added.
  • platform/mac-leopard/editing/deleting: Added.
  • platform/mac-leopard/editing/deleting/deletionUI-differing-background-expected.txt: Added.
9:38 AM Changeset in webkit [42825] by Darin Adler
  • 2 edits in trunk/WebCore

2009-04-24 Darin Adler <Darin Adler>

Reviewed by Adam Roben.

<rdar://problem/6663836> CrashTracer: crashes in Safari at com.apple.WebCore • WebCore::reportException + 1513

  • bindings/js/JSDOMBinding.cpp: (WebCore::reportException): Added a null check.
9:13 AM Changeset in webkit [42824] by beidson@apple.com
  • 2 edits in trunk/WebKit/mac

2009-04-24 Brady Eidson <beidson@apple.com>

Reviewed by Dan Bernstein

Currently working on a bug where a WebHistoryItem was being used after being dealloc'ed.
I added this assertion to help catch the case as soon as it happens instead of random
issues downstream. Figured it's worth checking in by itself.

  • History/WebHistoryItem.mm: (core): ASSERT that the WebCore::HistoryItem inside this WebHistoryItem is supposed to have this WebHistoryItem as a wrapper.
9:10 AM Changeset in webkit [42823] by Simon Fraser
  • 4 edits
    4 adds in trunk

2009-04-24 Simon Fraser <Simon Fraser>

Reviewed by Darin Adler

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

Add support for transtions and animations of background-position, -webkit-background-size,
-webkit-mask-position and -webkit-mask-size.

Tests: transitions/background-transitions.html

transitions/mask-transitions.html

  • page/animation/AnimationBase.cpp: (WebCore::blendFunc): Add a blend func for LengthSize.

(WebCore::ensurePropertyMap):
Add wrappers for the new properties.

(WebCore::addShorthandProperties):
Add a shorthand property for CSSPropertyBackgroundPosition to map to
CSSPropertyBackgroundPositionX and CSSPropertyBackgroundPositionY,
add CSSPropertyWebkitMask for CSSPropertyWebkitMaskPosition, and add
CSSPropertyWebkitMaskPosition for CSSPropertyWebkitMaskPositionX and
CSSPropertyWebkitMaskPositionY.

  • rendering/style/RenderStyle.h: (WebCore::InheritedFlags::setBackgroundXPosition): (WebCore::InheritedFlags::setBackgroundYPosition): (WebCore::InheritedFlags::setBackgroundSize): (WebCore::InheritedFlags::setMaskXPosition): (WebCore::InheritedFlags::setMaskYPosition): (WebCore::InheritedFlags::setMaskSize): Add setters for the properties that can be animated now.
8:55 AM Changeset in webkit [42822] by adele@apple.com
  • 7 edits in trunk/LayoutTests

2009-04-24 Adele Peterson <adele@apple.com>

Reviewed by Dan Bernstein.

Updating these pixel results to the generated results on Leopard.

  • platform/mac/editing/inserting/5510537-expected.checksum:
  • platform/mac/editing/inserting/5510537-expected.png:
  • platform/mac/editing/inserting/6703873-expected.checksum:
  • platform/mac/editing/inserting/6703873-expected.png:
  • platform/mac/editing/selection/select-all-iframe-expected.checksum:
  • platform/mac/editing/selection/select-all-iframe-expected.png:
8:39 AM Changeset in webkit [42821] by adele@apple.com
  • 7 edits
    4 adds in trunk

WebCore:

2009-04-24 Adele Peterson <adele@apple.com>

Reviewed by Darin Adler.

Fix for <rdar://problem/5089327> Newline gets stripped when pasting whole lines in certain markup

Test: editing/pasteboard/paste-blockquote-after-blockquote.html

When we have matching quote levels, its ok to merge the starts of the inserted and existing blocks more frequently.
But we should only merge here if the selection start was inside a mail blockquote. This prevents against removing a
blockquote from newly pasted quoted content that was pasted into an unquoted position. If that unquoted position happens
to be right after another blockquote, we don't want to merge and risk stripping a valid block (and newline) from the pasted content.

  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::shouldMergeStart): Also added an early return to always return false when we're already moving paragraphs. (WebCore::ReplaceSelectionCommand::doApply): Removed redundant check for when we're moving paragraphs.
  • editing/ReplaceSelectionCommand.h:

LayoutTests:

2009-04-24 Adele Peterson <adele@apple.com>

Reviewed by Darin Adler.

Test for <rdar://problem/5089327> Newline gets stripped when pasting whole lines in certain markup

  • editing/pasteboard/paste-blockquote-after-blockquote.html: Added.
  • platform/mac/editing/pasteboard/paste-blockquote-after-blockquote-expected.checksum: Added.
  • platform/mac/editing/pasteboard/paste-blockquote-after-blockquote-expected.png: Added.
  • platform/mac/editing/pasteboard/paste-blockquote-after-blockquote-expected.txt: Added.
6:16 AM Changeset in webkit [42820] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-24 Diego Escalante Urrelo <diegoe@gnome.org>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=15616
[GTK] Add spell checking

Implement EditorClient::ignoreWordInSpellDocument,
EditorClient::learnWord and EditorClient::getGuessesForWord. This
enables the corresponding user actions.

  • WebCoreSupport/EditorClientGtk.cpp: (WebKit::EditorClient::ignoreWordInSpellDocument): (WebKit::EditorClient::learnWord): (WebKit::EditorClient::getGuessesForWord):
6:16 AM Changeset in webkit [42819] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-24 Diego Escalante Urrelo <diegoe@gnome.org>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=15616
[GTK] Add spell checking

Implement EditorClient::checkSpellingOfString; enabling spell checking
to actually happen, this is the basis for other spelling functions.

  • WebCoreSupport/EditorClientGtk.cpp: (WebKit::EditorClient::checkSpellingOfString):
6:16 AM Changeset in webkit [42818] by kov@webkit.org
  • 6 edits in trunk

2009-04-24 Diego Escalante Urrelo <diegoe@gnome.org>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=15616
[GTK] Add spell checking

Add enchant support for spell-checking-languages property to work
properly.

  • GNUmakefile.am:
  • configure.ac:

WebKit/gtk

2009-04-24 Diego Escalante Urrelo <diegoe@gnome.org>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=15616
[GTK] Add spell checking

Add spell-checking-languages property to WebKitWebSettings.

  • webkit/webkitprivate.h:
  • webkit/webkitwebsettings.cpp: (_WebKitWebSettingsPrivate::): (_WebKitWebSettingsPrivate::webkit_web_settings_class_init): (_WebKitWebSettingsPrivate::webkit_web_settings_init): (_WebKitWebSettingsPrivate::webkit_web_settings_finalize): (_WebKitWebSettingsPrivate::webkit_web_settings_set_property): (_WebKitWebSettingsPrivate::webkit_web_settings_get_property): (_WebKitWebSettingsPrivate::webkit_web_settings_copy): (_WebKitWebSettingsPrivate::webkit_web_settings_get_spell_languages):
6:16 AM Changeset in webkit [42817] by kov@webkit.org
  • 3 edits in trunk/WebKit/gtk

2009-04-24 Diego Escalante Urrelo <diegoe@gnome.org>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=15616
[GTK] Add spell checking

Add enable-spell-checking property to WebKitWebSettings and implement
EditorClient::isContinuousSpellCheckingEnabled and
EditorClient::toggleContinuousSpellChecking to allow the UI to
enable check as you type spell checking.

  • WebCoreSupport/EditorClientGtk.cpp: (WebKit::EditorClient::isContinuousSpellCheckingEnabled): (WebKit::EditorClient::toggleContinuousSpellChecking):
  • webkit/webkitwebsettings.cpp: (_WebKitWebSettingsPrivate::): (_WebKitWebSettingsPrivate::webkit_web_settings_class_init): (_WebKitWebSettingsPrivate::webkit_web_settings_set_property): (_WebKitWebSettingsPrivate::webkit_web_settings_get_property): (_WebKitWebSettingsPrivate::webkit_web_settings_copy):
4:12 AM Changeset in webkit [42816] by ariya@webkit.org
  • 1 edit
    1 add in trunk/LayoutTests

2009-04-24 Ariya Hidayat <ariya.hidayat@nokia.com>

Reviewed by Simon Hausmann.

Added Qt-specific expected result for color conversion.

This is to compensate the lack of color profile in Qt to do
color conversion, hence CMYK 0,0,0,1 always give pure black (#000)
instead of very dark black.

  • platform/qt/fast/canvas/set-colors-expected.txt: Added.
4:08 AM Changeset in webkit [42815] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

2009-04-24 Simon Hausmann <simon.hausmann@nokia.com>

Rubber-stamped by Ariya Hidayat.

Fix qdoc warning about link to QAction::isEnabled.

3:30 AM Changeset in webkit [42814] by Simon Hausmann
  • 5 edits
    13 adds in trunk

2009-04-24 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Ariya Hidayat.

Added support for generating API docs in the Qt build using "make docs"

Added code snippets and overview from the Qt sources references in the API docs.

1:24 AM QtWebKitJournal edited by Simon Hausmann
(diff)

Apr 23, 2009:

11:38 PM Changeset in webkit [42813] by adele@apple.com
  • 5 edits in trunk/LayoutTests

2009-04-23 Adele Peterson <adele@apple.com>

Reviewed by Dan Bernstein.

Updating test to make it easier to see the blockquote in the results.

  • editing/pasteboard/paste-blockquote-into-blockquote-3.html:
  • platform/mac/editing/pasteboard/paste-blockquote-into-blockquote-3-expected.checksum:
  • platform/mac/editing/pasteboard/paste-blockquote-into-blockquote-3-expected.png:
  • platform/mac/editing/pasteboard/paste-blockquote-into-blockquote-3-expected.txt:
11:03 PM Changeset in webkit [42812] by mrowe@apple.com
  • 1 edit
    1 move
    1 add in trunk/LayoutTests

Move deletionUI-helpers.js in to the correct location so that the
tests that use it can find it.

  • platform/mac/editing/deleting/resources/deletionUI-helpers.js: Renamed from LayoutTests/editing/deleting/resources/deletionUI-helpers.js.
10:12 PM Changeset in webkit [42811] by mrowe@apple.com
  • 2 edits in trunk/JavaScriptCore

With great sadness and a heavy heart I switch us back from YARR to WREC in
order to restore greenness to the world once more.

  • wtf/Platform.h:
10:04 PM Changeset in webkit [42810] by mrowe@apple.com
  • 3 edits in trunk/JavaScriptCore

More Windows build fixage.

9:16 PM Changeset in webkit [42809] by mrowe@apple.com
  • 2 edits in trunk/JavaScriptCore

Attempt to fix the Windows build.

  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def: Remove a symbol that no longer exists.
8:47 PM Changeset in webkit [42808] by timothy@apple.com
  • 25 edits
    3 adds
    2 deletes in trunk

BUG 24604: WebKit profiler reports incorrect total times

JavaScriptCore:

2009-04-23 Francisco Tolmasky <francisco@280north.com>

BUG 24604: WebKit profiler reports incorrect total times
<https://bugs.webkit.org/show_bug.cgi?id=24604>

Reviewed by Timothy Hatcher and Kevin McCullough.

  • JavaScriptCore.exp:
  • JavaScriptCore.xcodeproj/project.pbxproj:
  • profiler/CallIdentifier.h: (JSC::CallIdentifier::Hash::hash): (JSC::CallIdentifier::Hash::equal): (JSC::CallIdentifier::hash): (WTF::):
  • profiler/HeavyProfile.cpp: Removed.
  • profiler/HeavyProfile.h: Removed.
  • profiler/Profile.cpp: No more need for TreeProfile/HeavyProfile (JSC::Profile::create):
  • profiler/Profile.h:
  • profiler/ProfileNode.cpp:
  • profiler/ProfileNode.h:
  • profiler/TreeProfile.cpp: Removed.
  • profiler/TreeProfile.h: Removed.

WebCore:

2009-04-23 Francisco Tolmasky <francisco@280north.com>

BUG 24604: WebKit profiler reports incorrect total times
<https://bugs.webkit.org/show_bug.cgi?id=24604>

Reviewed by Timothy Hatcher and Kevin McCullough.

Made it so that most of the profiler functions now match the behavior of Shark. Most notably, in the
heavy view, child nodes now represent the statistics of the root node. Each root node of heavy view
displays flattened statistics for a particular function that ran during the profile, and each child
of these root nodes represents a callpath that lead to it. Thus, the statistics for each of these child
nodes should show how much of the root nodes values came from it. For example, if you had the following to
stacks take place during the profile:

A ->calls 1 times-> B ->calls 2 times-> C
D ->calls 4 times-> C

The tree for the C root node would look like this:

C -> B -> A

-> D

The number of calls values would look like this:

C (6) -> B (2) -> A(2)

-> D (4)

What this means is that "2 of the total 6 C calls came from B", "2 of the total C calls came from A", and
"4 of the total C calls came from D". Notice that the "A ->calls 2 time->" is completely ignored. This becomes
particularly tricky during recursive calls, because each child note can represent multiple possible paths. This
is the reason that we would get things like 40000% previously with recursion.

This is also the way gprof works, and as close as we can get to Shark's behavior (Shark is not instrumented so it
can't know exactly how many calls came from where, etc).

  • English.lproj/localizedStrings.js: Added "Average" for average times in the profile.
  • inspector/JavaScriptProfile.cpp: (WebCore::ProfileClass):
  • inspector/JavaScriptProfileNode.cpp: (WebCore::getParent): (WebCore::getHead): (WebCore::getCallUID): (WebCore::ProfileNodeClass):
  • inspector/front-end/BottomUpProfileDataGridTree.js: Added. (WebInspector.BottomUpProfileDataGridTree): (WebInspector.BottomUpProfileDataGridTree.prototype.focus): (WebInspector.BottomUpProfileDataGridNode): (WebInspector.BottomUpProfileDataGridNode.prototype._takePropertiesFromProfileDataGridNode): (WebInspector.BottomUpProfileDataGridNode.prototype._keepOnlyChild): (WebInspector.BottomUpProfileDataGridNode.prototype._exclude): (WebInspector.BottomUpProfileDataGridNode.prototype._merge): (WebInspector.BottomUpProfileDataGridNode.prototype._populate):
  • inspector/front-end/DataGrid.js: (WebInspector.DataGrid.prototype.insertChild): (WebInspector.DataGrid.prototype.removeChild): (WebInspector.DataGrid.prototype.removeChildren): (WebInspector.DataGridNode.prototype.set hasChildren): (WebInspector.DataGridNode.prototype.get hasChildren):
  • inspector/front-end/ProfileDataGridTree.js: Added. (WebInspector.ProfileDataGridNode): (WebInspector.ProfileDataGridNode.prototype.get data.formatMilliseconds): (WebInspector.ProfileDataGridNode.prototype.get data): (WebInspector.ProfileDataGridNode.prototype.createCell): (WebInspector.ProfileDataGridNode.prototype.select): (WebInspector.ProfileDataGridNode.prototype.deselect): (WebInspector.ProfileDataGridNode.prototype.expand): (WebInspector.ProfileDataGridNode.prototype.insertChild): (WebInspector.ProfileDataGridNode.prototype.removeChild): (WebInspector.ProfileDataGridNode.prototype.removeChildren): (WebInspector.ProfileDataGridNode.prototype.findChild): (WebInspector.ProfileDataGridNode.prototype.get averageTime): (WebInspector.ProfileDataGridNode.prototype.get averagePercent): (WebInspector.ProfileDataGridNode.prototype.get selfPercent): (WebInspector.ProfileDataGridNode.prototype.get totalPercent): (WebInspector.ProfileDataGridNode.prototype._save): (WebInspector.ProfileDataGridNode.prototype._restore): (WebInspector.ProfileDataGridNode.prototype._merge): (WebInspector.ProfileDataGridTree): (WebInspector.ProfileDataGridTree.prototype.get expanded): (WebInspector.ProfileDataGridTree.prototype.appendChild): (WebInspector.ProfileDataGridTree.prototype.insertChild): (WebInspector.ProfileDataGridTree.prototype.removeChildren): (WebInspector.ProfileDataGridTree.prototype.findChild.WebInspector.ProfileDataGridNode.prototype.findChild.sort.WebInspector.ProfileDataGridNode.prototype.sort._save): (WebInspector.ProfileDataGridTree.propertyComparator.comparator): (WebInspector.ProfileDataGridTree.propertyComparator.else.comparator): (WebInspector.ProfileDataGridTree.propertyComparator):
  • inspector/front-end/ProfileView.js: (WebInspector.ProfileView): (WebInspector.ProfileView.prototype.set profile): (WebInspector.ProfileView.prototype.get bottomUpProfileDataGridTree): (WebInspector.ProfileView.prototype.get topDownProfileDataGridTree): (WebInspector.ProfileView.prototype.get currentTree): (WebInspector.ProfileView.prototype.set currentTree): (WebInspector.ProfileView.prototype.get topDownTree): (WebInspector.ProfileView.prototype.get bottomUpTree): (WebInspector.ProfileView.prototype.refresh): (WebInspector.ProfileView.prototype.refreshVisibleData): (WebInspector.ProfileView.prototype.refreshShowAsPercents): (WebInspector.ProfileView.prototype.performSearch.matchesQuery): (WebInspector.ProfileView.prototype.performSearch): (WebInspector.ProfileView.prototype._changeView): (WebInspector.ProfileView.prototype._focusClicked): (WebInspector.ProfileView.prototype._excludeClicked): (WebInspector.ProfileView.prototype._resetClicked): (WebInspector.ProfileView.prototype._sortProfile):
  • inspector/front-end/ProfilesPanel.js: (WebInspector.ProfilesPanel.prototype.showProfile): (WebInspector.ProfilesPanel.prototype.showView): (WebInspector.ProfilesPanel.prototype.searchMatchFound):
  • inspector/front-end/TopDownProfileDataGridTree.js: Added. (WebInspector.TopDownProfileDataGridNode): (WebInspector.TopDownProfileDataGridNode.prototype._populate): (WebInspector.TopDownProfileDataGridNode.prototype._exclude): (WebInspector.TopDownProfileDataGridTree): (WebInspector.TopDownProfileDataGridTree.prototype.focus): (WebInspector.TopDownProfileDataGridTree.prototype.exclude):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.css:
  • inspector/front-end/inspector.html:

LayoutTests:

2009-04-23 Francisco Tolmasky <francisco@280north.com>

BUG 24604: WebKit profiler reports incorrect total times
<https://bugs.webkit.org/show_bug.cgi?id=24604>

Reviewed by Timothy Hatcher and Kevin McCullough.

Changed profile.treeProfile to just profile, since these aren't generated in C++ anymore.
Removed heavy-view test since heavy-view isn't an actual tree that is generated in C++ land anymore,
but rather just a different display of the normal treeProfile in the JS data grid.

  • fast/profiler/heavy-view-expected.txt: Removed.
  • fast/profiler/heavy-view.html: Removed.
  • fast/profiler/resources/profiler-test-JS-resources.js: profiles[i].treeProfile -> profiles[i].treeProfile (printProfilesDataWithoutTime):
6:50 PM Changeset in webkit [42807] by eric@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by Sam Weinig.

Use static functions (and a couple templates)
to further reduce the amount of copy/paste code in SVGRenderTreeAsText

No test changes, only code cleanup.

  • rendering/SVGRenderTreeAsText.cpp: (WebCore::writeNameValuePair): (WebCore::writeNameAndQuotedValue): (WebCore::writeIfNotEmpty): (WebCore::writeIfNotDefault): (WebCore::writeStyle): (WebCore::writePositionAndStyle): (WebCore::operator<<):
6:42 PM Changeset in webkit [42806] by kmccullough@apple.com
  • 1 edit
    3 moves in trunk/LayoutTests

2009-04-23 Kevin McCullough <kmccullough@apple.com>

  • Windows build fix. Mac only tests go in platform/mac
  • editing/deleting/deletionUI-borders.html: Removed.
  • editing/deleting/deletionUI-differing-background.html: Removed.
  • editing/deleting/deletionUI-minimum-size.html: Removed.
  • platform/mac/editing/deleting/deletionUI-borders.html: Copied from editing/deleting/deletionUI-borders.html.
  • platform/mac/editing/deleting/deletionUI-differing-background.html: Copied from editing/deleting/deletionUI-differing-background.html.
  • platform/mac/editing/deleting/deletionUI-minimum-size.html: Copied from editing/deleting/deletionUI-minimum-size.html.
6:09 PM Changeset in webkit [42805] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-04-23 Gavin Barraclough <barraclough@apple.com>

Not Reviewed.

Speculative Windows build fix II.

  • yarr/RegexInterpreter.cpp:
5:53 PM Changeset in webkit [42804] by barraclough@apple.com
  • 3 edits in trunk/JavaScriptCore

2009-04-23 Gavin Barraclough <barraclough@apple.com>

Not Reviewed.

Speculative Windows build fix.

  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
  • runtime/RegExp.cpp:
5:50 PM Changeset in webkit [42803] by eric@webkit.org
  • 4 edits in trunk/WebCore

Reviewed by Simon Fraser.

SVGRenderTreeAsText cleanup/code sharing.

No test changes, only code cleanup.

  • rendering/RenderText.cpp: (WebCore::RenderText::firstRunOrigin):
  • rendering/RenderText.h:
  • rendering/SVGRenderTreeAsText.cpp: (WebCore::writeStandardPrefix): (WebCore::writeChildren): (WebCore::write): (WebCore::writeRenderResources):
5:39 PM Changeset in webkit [42802] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-04-23 Gavin Barraclough <barraclough@apple.com>

Rubber stamped by salty sea dogs Sam & Geoff.

Enable YARR_JIT by default (where supported), replacing WREC.

  • wtf/Platform.h:
5:37 PM Changeset in webkit [42801] by Darin Adler
  • 1 edit
    2 deletes in trunk/LayoutTests

2009-04-23 Darin Adler <Darin Adler>

Removed some stray files that Geoff missed when rolling out changes.

  • svg/custom/resources/svg-fonts-in-text-controls.js: Removed.
  • svg/custom/svg-fonts-in-text-controls-expected.txt: Removed.
5:32 PM Changeset in webkit [42800] by barraclough@apple.com
  • 8 edits in trunk

JavaScriptCore:

2009-04-23 Gavin Barraclough <barraclough@apple.com>

Reviewed by Geoff "Dread Pirate Roberts" Garen.

Various small fixes to YARR JIT, in preparation for enabling it by default.

  • Correctly index into the callframe when storing restart addresses for nested alternatives.
  • Allow backtracking back into matched alternatives of parentheses.
  • Fix callframe offset calculation for parenthetical assertions.
  • When a set of parenthese are quantified with a fixed and variable portion, and the variable portion is quantified once, this should not reset the pattern match on failure to match (the last match from the firxed portion should be preserved).
  • Up the pattern size limit to match PCRE's new limit.
  • Unlclosed parentheses should be reported with the message "missing )".
  • wtf/Platform.h:
  • yarr/RegexCompiler.cpp: (JSC::Yarr::RegexPatternConstructor::quantifyAtom): (JSC::Yarr::RegexPatternConstructor::setupAlternativeOffsets):
  • yarr/RegexInterpreter.cpp: (JSC::Yarr::Interpreter::matchParentheses): (JSC::Yarr::Interpreter::backtrackParentheses): (JSC::Yarr::ByteCompiler::emitDisjunction):
  • yarr/RegexJIT.cpp: (JSC::Yarr::RegexGenerator::loadFromFrameAndJump): (JSC::Yarr::RegexGenerator::generateParenthesesDisjunction): (JSC::Yarr::RegexGenerator::generateParentheticalAssertion): (JSC::Yarr::RegexGenerator::generateTerm): (JSC::Yarr::executeRegex):
  • yarr/RegexParser.h: (JSC::Yarr::Parser::): (JSC::Yarr::Parser::parseTokens): (JSC::Yarr::Parser::parse):
  • yarr/RegexPattern.h: (JSC::Yarr::PatternTerm::): (JSC::Yarr::PatternTerm::PatternTerm):

LayoutTests:

2009-04-23 Gavin Barraclough <barraclough@apple.com>

Reviewed by Geoff "Dread Pirate Roberts" Garen.

This test tries to force itself into PCRE; modify the
test so that as well as dodging WREC it can also avoid
YARR!

  • fast/js/resources/regexp-overflow-too-big.js:
5:25 PM Changeset in webkit [42799] by Beth Dakin
  • 5 edits in trunk

WebCore:

2009-04-23 Beth Dakin <Beth Dakin>

Reviewed by Darin Adler.

WebCore part of fix for <rdar://problem/6333461> REGRESSION
(r36864-r36869): Dragging stocks widget scrollbar drags the whole
widget

  • platform/Scrollbar.h: (WebCore::Scrollbar::isScrollbar):
  • platform/Widget.h: (WebCore::Widget::isScrollbar):

WebKit/mac:

2009-04-23 Beth Dakin <Beth Dakin>

Reviewed by Darin Adler.

Fix for <rdar://problem/6333461> REGRESSION (r36864-r36869):
Dragging stocks widget scrollbar drags the whole widget

Look for our new WebCore scrollbars in the WebHTMLView and add
proper Dashboard regions for them.

  • WebView/WebView.mm: (-[WebView _addScrollerDashboardRegionsForFrameView:dashboardRegions:]): (-[WebView _addScrollerDashboardRegions:from:]):
4:59 PM Changeset in webkit [42798] by kmccullough@apple.com
  • 3 edits
    8 adds in trunk

WebCore:

2009-04-23 Kevin McCullough <kmccullough@apple.com>

Reviewed by Adam Roben.

<rdar://problem/6808109> "Deletion UI" is not available for many
portions of HTML content

This patch makes the deletion UI show up in some new situations:
1) If a block's background color is different from its parent's
2) If a block has a background image.
3) If a block has a single visible border.
However the block must now not only be at least a minimum width and
height, but also exceed a minimum area. In practice this has led to
much better element selection.

  • editing/DeleteButtonController.cpp: (WebCore::isDeletableElement):

LayoutTests:

2009-04-23 Kevin McCullough <kmccullough@apple.com>

Reviewed by Adam Roben.

<rdar://problem/6808109> "Deletion UI" is not available for many
portions of HTML content

This patch makes the deletion UI show up in some new situations:
1) If a block's background color is different from its parent's
2) If a block has a background image.
3) If a block has a single visible border.
However the block must now not only be at least a minimum width and
height, but also exceed a minimum area. In practice this has led to
much better element selection.

  • editing/deleting/deletionUI-borders.html: Added.
  • editing/deleting/deletionUI-differing-background.html: Added.
  • editing/deleting/deletionUI-minimum-size.html: Added.
  • editing/deleting/resources: Added.
  • editing/deleting/resources/deletionUI-helpers.js: Added. (debug): (determineDeletionUIExistence):
  • platform/mac/editing/deleting/deletionUI-borders-expected.txt: Added.
  • platform/mac/editing/deleting/deletionUI-differing-background-expected.txt: Added.
  • platform/mac/editing/deleting/deletionUI-minimum-size-expected.txt: Added.
4:53 PM Changeset in webkit [42797] by kevino@webkit.org
  • 2 edits in trunk/WebCore

wx build fix. Adding XMLTokenizer.cpp to the build.

4:32 PM Changeset in webkit [42796] by sullivan@apple.com
  • 2 edits in trunk/WebKit/mac

2009-04-23 John Sullivan <sullivan@apple.com>

fixed <rdar://problem/6822479> Assertion failure after Reset Safari in new history-writing code

Reviewed by Oliver Hunt

  • History/WebHistory.mm: (-[WebHistoryPrivate data]): Return nil immediately if there are no entries; this matches a recent Windows-platform fix.
4:14 PM Changeset in webkit [42795] by mrowe@apple.com
  • 2 edits in tags/Safari-6530.7/JavaScriptCore

Temporarily work around <rdar://problem/6812854> by forcing our 2GB allocation up in to high memory.

Rubber-stamped by Geoff Garen.

  • jit/ExecutableAllocatorFixedVMPool.cpp:

(JSC::FixedVMPoolAllocator::FixedVMPoolAllocator):

3:46 PM Changeset in webkit [42794] by adele@apple.com
  • 2 edits in trunk/WebCore

2009-04-23 Adele Peterson <adele@apple.com>

Reviewed by Darin Adler.

Fix for <rdar://problem/6788905> REGRESSION (41928?): hang in Position::upstream
https://bugs.webkit.org/show_bug.cgi?id=25312

I was unable to reproduce the problem, but I'm pretty sure this will fix it.

  • rendering/RenderObject.cpp: (WebCore::RenderObject::createVisiblePosition): Since VisiblePosition doesn't ensure the offset is good, we shouldn't pass max int as an offset.
3:38 PM Changeset in webkit [42793] by mrowe@apple.com
  • 4 edits in trunk

Versioning.

3:38 PM Changeset in webkit [42792] by mrowe@apple.com
  • 1 copy in tags/Safari-6530.7

New tag.

3:33 PM Changeset in webkit [42791] by Dimitri Glazkov
  • 5 edits
    2 adds in trunk

WebCore:

2009-04-23 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=25313
Missing scroll bars in GMail.

Test: fast/overflow/overflow-y-scroll.html

  • platform/ScrollView.cpp: (WebCore::ScrollView::updateScrollbars): Added check for the ScrollbarAlwaysOn scroll mode.

LayoutTests:

2009-04-23 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Darin Adler.
https://bugs.webkit.org/show_bug.cgi?id=25313
Missing scroll bars in GMail.

  • fast/overflow/overflow-y-scroll-expected.txt: Added.
  • fast/overflow/overflow-y-scroll.html: Added.

WebKit/mac:

2009-04-23 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=25313
Missing scroll bars in GMail.

  • WebView/WebDynamicScrollBarsView.m: (-[WebDynamicScrollBarsView updateScrollers]): Added check for the ScrollbarAlwaysOn

scroll mode.

3:17 PM Changeset in webkit [42790] by kdecker@apple.com
  • 2 edits in trunk/WebKit/mac

Fix the Tiger build

3:03 PM Changeset in webkit [42789] by andersca@apple.com
  • 3 edits in trunk/WebKit/mac

2009-04-23 Anders Carlsson <andersca@apple.com>

Reviewed by Geoffrey Garen.

Fix <rdar://problem/6821992>

Add a new m_inDestroy member variable. Set it to true when in destroy, and have all NPRuntime functions return false
when m_inDestroy is true.


  • Plugins/Hosted/NetscapePluginInstanceProxy.h:
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::NetscapePluginInstanceProxy): (WebKit::NetscapePluginInstanceProxy::destroy): (WebKit::NetscapePluginInstanceProxy::invoke): (WebKit::NetscapePluginInstanceProxy::invokeDefault): (WebKit::NetscapePluginInstanceProxy::construct): (WebKit::NetscapePluginInstanceProxy::getProperty): (WebKit::NetscapePluginInstanceProxy::setProperty): (WebKit::NetscapePluginInstanceProxy::removeProperty): (WebKit::NetscapePluginInstanceProxy::hasProperty): (WebKit::NetscapePluginInstanceProxy::hasMethod): (WebKit::NetscapePluginInstanceProxy::enumerate):
2:22 PM Changeset in webkit [42788] by mitz@apple.com
  • 4 edits
    2 adds in trunk

WebCore:

Reviewed by Justin Garcia.

  • fix <rdar://problem/6081309> Mail crash when pressing down arrow in some messages in WebCore::canHaveChildrenForEditing

Test: editing/selection/extend-by-line-anonymous-content-crash.html

  • editing/visible_units.cpp: (WebCore::previousLinePosition): Null-check node. If p is not an editable position, then closestLeafChildForXPos() may have returned a non-editable box, and in particular one belonging to anonymous content. If node is 0, fall back on RenderObject::positionForPoint, which finds the closest position in non-anonymous content. (WebCore::nextLinePosition): Ditto.
  • rendering/RenderObject.cpp: (WebCore::RenderObject::createVisiblePosition): Fixed a typo.

LayoutTests:

Reviewed by Justin Garcia.

  • test for <rdar://problem/6081309> Mail crash when pressing down arrow in some messages in WebCore::canHaveChildrenForEditing
  • editing/selection/extend-by-line-anonymous-content-crash-expected.txt: Added.
  • editing/selection/extend-by-line-anonymous-content-crash.html: Added.
1:49 PM Changeset in webkit [42787] by hyatt@apple.com
  • 3 edits in trunk/WebKit/mac

2009-04-23 David Hyatt <hyatt@apple.com>

Reviewed by Maciej.

Fix for <rdar://problem/6789879> REGRESSION (42464): Hitting assertion when loading message in Mail + TOT WebKit


Make the Mac platform the same as all the other platforms. Instead of (incorrectly) marking a FrameView for layout
when its underlying document view changes, just mark the outermost frame view for layout when the WebView's size changes.

  • WebView/WebHTMLView.mm: (-[WebHTMLView _frameOrBoundsChanged]): (-[WebHTMLView addSuperviewObservers]):
  • WebView/WebView.mm: (-[WebView _boundsChanged]): (-[WebView removeSizeObservers]): (-[WebView addSizeObservers]):
1:08 PM Changeset in webkit [42786] by kdecker@apple.com
  • 6 edits in trunk/WebKit/mac

Reviewed by Tim Hatcher.

Second part of the fix for <rdar://problem/6352982>

  • Plugins/WebBaseNetscapePluginView.h: Make this class conform to WebPluginContainerCheckController
  • Plugins/WebBaseNetscapePluginView.mm: Likewise. (-[WebBaseNetscapePluginView _webPluginContainerCancelCheckIfAllowedToLoadRequest:]): Added skeleton method.
  • Plugins/WebPluginContainerCheck.h: Added protocol for <WebPluginContainerCheckController>
  • Plugins/WebPluginContainerCheck.mm: (-[WebPluginContainerCheck initWithRequest:target:resultObject:selector:controller:]):
  • Plugins/WebPluginController.h: Make this class conform to WebPluginContainerCheckController
12:58 PM Changeset in webkit [42785] by ap@webkit.org
  • 3 edits
    2 adds in trunk

Reviewed by Adele Peterson.

<rdar://problem/6791439> REGRESSION: Get an error page instead of login page navigating back in gmail

Test: http/tests/history/back-to-post.php

  • platform/network/cf/ResourceRequestCFNet.cpp: (WebCore::ResourceRequest::doUpdatePlatformRequest): Apply a new cache policy if it changed after the platform request was first created. While at it, also made sure to update timeout.
12:02 PM Changeset in webkit [42784] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk/po

2009-04-23 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Rubber-stamped by Xan Lopez.

  • pt_BR.po: translation update.
11:45 AM Changeset in webkit [42783] by kov@webkit.org
  • 2 edits in trunk/WebCore

2009-04-23 Zan Dobersek <zandobersek@gmail.com>

Reviewed by Gustavo Noronha.

When creating a new drawable object, collect system's
information about the best depth it can provide and use it
in a new colormap that the new drawable requires.

  • platform/graphics/gtk/ImageGtk.cpp: (WebCore::BitmapImage::getGdkPixbuf):
11:06 AM Changeset in webkit [42782] by kdecker@apple.com
  • 8 edits
    2 adds in trunk/WebKit

Reviewed by Anders Carlsson.


First part of <rdar://problem/6352982>

  • Plugins/WebBaseNetscapePluginView.h: Imported #WebNetscapeContainerCheckPrivate.h; Added two new method: checkIfAllowedToLoadURL:url:frame:callbackFunc: and cancelCheckIfAllowedToLoadURL:
  • Plugins/WebBaseNetscapePluginView.mm: (-[WebBaseNetscapePluginView checkIfAllowedToLoadURL:frame:callbackFunc:]): Added skeleton method, does nothing interesting yet. (-[WebBaseNetscapePluginView cancelCheckIfAllowedToLoadURL:]): Likewise.
  • Plugins/WebNetscapeContainerCheckPrivate.h: Added.
  • Plugins/WebNetscapeContainerCheckPrivate.mm: Added. (browserContainerCheckFuncs): Added.
  • Plugins/WebNetscapePluginView.h: Imported #WebNetscapeContainerCheckPrivate.h;
  • Plugins/WebNetscapePluginView.mm: Imported #WebNetscapeContainerCheckPrivate.h; added WKN_CheckIfAllowedToLoadURL and WKN_CancelCheckIfAllowedToLoadURL functions. (-[WebNetscapePluginView getVariable:value:]): Return vtable for container check functions.
  • Plugins/npapi.mm: (WKN_CheckIfAllowedToLoadURL): Added new private function. (WKN_CancelCheckIfAllowedToLoadURL): Ditto.
9:42 AM Changeset in webkit [42781] by Adam Roben
  • 2 edits in trunk/LayoutTests

Skip a newly-added test that fails on Windows

See Bug 25344: http/tests/misc/redirect-to-external-url.html fails on
Windows
<https://bugs.webkit.org/show_bug.cgi?id=25344>

Reviewed by Brady Eidson.

  • platform/win/Skipped: Added http/tests/misc/redirect-to-external-url.html.
8:39 AM Changeset in webkit [42780] by kov@webkit.org
  • 4 edits in trunk/WebCore

2009-04-23 Zan Dobersek <zandobersek@gmail.com>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=15654
GdkPixbuf support for ImageCairo

Add support for converting a Cairo surface to a GdkPixbuf.

  • platform/graphics/BitmapImage.h:
  • platform/graphics/Image.h: (WebCore::Image::getGdkPixbuf):
  • platform/graphics/gtk/ImageGtk.cpp: (WebCore::BitmapImage::getGdkPixbuf):
8:07 AM Changeset in webkit [42779] by vestbo@webkit.org
  • 1 edit in trunk/WebKit/qt/Api/qwebelement.cpp

Fix Qt build warning

7:55 AM Changeset in webkit [42778] by ariya@webkit.org
  • 3 edits in trunk/WebCore

2009-04-23 Kenneth Rohde Christiansen <kenneth.christiansen@openbossa.org>

Reviewed by Simon Hausmann.

Make windowed plugins (on X11, Qt) behave nicer with wheel scrolling.

Basically, what we want is to send wheel events to the plugin *only*
when it has focus, or else send it to the parent frame.

This is a work around, and basically what we do, is to steal the
wheel events from the plugin, modify the window ID, and put the
event back into the Qt event loop.

  • plugins/qt/PluginContainerQt.cpp: (PluginClientWrapper::PluginClientWrapper): (PluginClientWrapper::~PluginClientWrapper): (PluginClientWrapper::x11Event): (PluginContainerQt::PluginContainerQt): (PluginContainerQt::~PluginContainerQt): (PluginContainerQt::on_clientClosed): (PluginContainerQt::on_clientIsEmbedded): (PluginContainerQt::redirectWheelEventsToParent): (PluginContainerQt::x11Event): (PluginContainerQt::focusInEvent):
  • plugins/qt/PluginContainerQt.h:
6:53 AM Changeset in webkit [42777] by ariya@webkit.org
  • 4 edits in trunk/WebKit/qt

2009-04-23 Antonio Gomes <antonio.gomes@openbossa.org>

Reviewed by Ariya Hidayat.

[Qt] Added QWebElement::computedStyleProperty method.

  • Api/qwebelement.cpp: (QWebElement::computedStyleProperty):
  • Api/qwebelement.h:
  • tests/qwebelement/tst_qwebelement.cpp: (tst_QWebElement::computedStyle):
5:46 AM QtWebKitTodo edited by vestbo@webkit.org
(diff)
5:36 AM QtWebKitJournal edited by vestbo@webkit.org
(diff)
4:30 AM Changeset in webkit [42776] by vestbo@webkit.org
  • 3 edits in trunk/WebKit/qt

2009-04-23 Tor Arne Vestbø <tor.arne.vestbo@nokia.com>

Rubber-stamped by Ariya Hidayat.

[Qt] Rename QWebElement arguments from html to markup

12:25 AM Changeset in webkit [42775] by mrowe@apple.com
  • 2 edits in trunk/WebKitTools

Fix the Windows DRT build.

  • DumpRenderTree/win/PolicyDelegate.h:
12:10 AM Changeset in webkit [42774] by beidson@apple.com
  • 2 edits in trunk/WebKitTools

2009-04-23 Brady Eidson <beidson@apple.com>

Fix Windows DRT build.

  • DumpRenderTree/win/PolicyDelegate.cpp: (PolicyDelegate::unableToImplementPolicyWithError):
12:09 AM Changeset in webkit [42773] by beidson@apple.com
  • 2 edits in trunk/LayoutTests

2009-04-23 Brady Eidson <beidson@apple.com>

Update my just-landed layout test to agree with Tiger's CFNetwork

  • http/tests/misc/resources/redirect-to-external-url.php:

Apr 22, 2009:

11:42 PM Changeset in webkit [42772] by beidson@apple.com
  • 7 edits
    8 adds in trunk

WebCore:

2009-04-22 Brady Eidson <beidson@apple.com>

Reviewed by Maciej Stachowiak

<rdar://problem/6786961> - "Are you sure you want to resubmit this form?" nag displays twice.

For most loads, we were consulting the navigation policy delegate twice. Once from FrameLoader before
the load started and once from MainResourceLoader in its willSendRequest callback.

In the past we tried removing MainResourceLoader's call altogether. This caused a regression where urls
that redirect to a url handled by an external application would no longer work in Safari. It probably
also broke other WebKit apps in subtle ways.

Changing MainResourceLoader to make the check only on redirects fixes both bugs. We now only call the
policy delegate once for most standard loads, but we do correctly call it a second time for redirects.

Tests: http/tests/misc/policy-delegate-called-twice.html

http/tests/misc/redirect-to-external-url.html

  • loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::willSendRequest): Call the navigation policy delegate only for redirects.

WebKitTools:

2009-04-22 Brady Eidson <beidson@apple.com>

Reviewed by Maciej Stachowiak

Add additional support for test:
<rdar://problem/5689748> - Cannot redirect to protocols handled by external applications.

  • DumpRenderTree/mac/PolicyDelegate.mm: (-[PolicyDelegate webView:unableToImplementPolicyWithError:frame:]): Dump formatted information when a policy cannot be implemented.
  • DumpRenderTree/win/PolicyDelegate.cpp: (PolicyDelegate::unableToImplementPolicyWithError): Ditto.
  • DumpRenderTree/win/PolicyDelegate.h:

LayoutTests:

2009-04-22 Brady Eidson <beidson@apple.com>

Reviewed by Maciej Stachowiak

With the fix for:
<rdar://problem/6786961> - "Are you sure you want to resubmit this form?" nag displays twice.
And also to make sure we never rebreak:
<rdar://problem/5689748> - Cannot redirect to protocols handled by external applications.

  • http/tests/misc/redirect-to-external-url-expected.txt: Added.
  • http/tests/misc/redirect-to-external-url.html: Added.
  • http/tests/misc/resources/redirect-to-external-url-iframe.html: Added.
  • http/tests/misc/resources/redirect-to-external-url.php: Added.
  • http/tests/misc/policy-delegate-called-twice-expected.txt: Added.
  • http/tests/misc/policy-delegate-called-twice.html: Added.
  • http/tests/misc/resources/iframe-policy-1.html: Added.
  • http/tests/misc/resources/iframe-policy-2.html: Added.
9:06 PM Changeset in webkit [42771] by mrowe@apple.com
  • 2 edits in trunk/JavaScriptCore

Add the m_ prefix on FixedVMPoolAllocator's member variables, and fix typos in a few comments.

Rubber-stamped by Gavin Barraclough.

  • jit/ExecutableAllocatorFixedVMPool.cpp:

(JSC::FixedVMPoolAllocator::addToFreeList):
(JSC::FixedVMPoolAllocator::coalesceFreeSpace):
(JSC::FixedVMPoolAllocator::FixedVMPoolAllocator):
(JSC::FixedVMPoolAllocator::alloc):
(JSC::FixedVMPoolAllocator::free):
(JSC::FixedVMPoolAllocator::isWithinVMPool):

9:06 PM Changeset in webkit [42770] by mrowe@apple.com
  • 2 edits in trunk/JavaScriptCore

Add some assertions to FixedVMPoolAllocator to guard against cases where we
attempt to free memory that didn't originate from the pool, or we attempt to
hand out a bogus address from alloc.

Rubber-stamped by Gavin Barraclough.

  • jit/ExecutableAllocatorFixedVMPool.cpp:

(JSC::FixedVMPoolAllocator::release):
(JSC::FixedVMPoolAllocator::FixedVMPoolAllocator):
(JSC::FixedVMPoolAllocator::alloc):
(JSC::FixedVMPoolAllocator::free):
(JSC::FixedVMPoolAllocator::isWithinVMPool):

7:41 PM Changeset in webkit [42769] by weinig@apple.com
  • 2 edits in trunk/WebCore

2009-04-22 Sam Weinig <sam@webkit.org>

Reviewed by Mark Rowe.

Fix for <rdar://problem/6800695>

Add a hack to only allow navigating (via a link from a http page)
to feed: urls (same for feeds: and feedsearch:) that map to http:
or https: via their nested protocol. This includes both feed://example.com
and feed:http://example.com.

  • loader/FrameLoader.cpp: (WebCore::isFeedWithNestedProtocolInHTTPFamily): (WebCore::FrameLoader::loadFrameRequest):
7:34 PM Changeset in webkit [42768] by weinig@apple.com
  • 7 edits in trunk/WebCore

2009-04-22 Sam Weinig <sam@webkit.org>

Reviewed by Anders Carlsson.

Autogenerate the XMLHttpRequest constructor getter/setter for the
WorkerContext and remove the now no longer needed JSWorkerContextBase.lut.h
file.

  • DerivedSources.make: Remove JSWorkerContextBase.lut.h
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • bindings/js/JSWorkerContextBase.cpp:
  • bindings/js/JSWorkerContextBase.h: Remove code to support hand written code for XMLHttpRequest constructor lookup and shadowing.
  • bindings/js/JSWorkerContextCustom.cpp: (WebCore::JSWorkerContext::customGetOwnPropertySlot): Remove JSC prefixes. (WebCore::JSWorkerContext::xmlHttpRequest): Added. Custom implementation of cached constructor lookup.
  • workers/WorkerContext.idl: Make the XMLHttpRequestConstructor accessor not V8 specific.
6:43 PM Changeset in webkit [42767] by adachan@apple.com
  • 6 edits in trunk

WebKitLibraries:

Update WebKitSystemInterface with new method that maps CFNetwork error code to localized description.


Reviewed by Darin Adler.

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

WebKit/win:

<rdar://problem/5115298> Don't get error msg when download is interrupted and cannot resume due to server unavailability
If we fail to get an error description back from CFNetwork, try to get the localized description
based on the error code.

Reviewed by Darin Adler.

  • WebError.cpp: (WebError::localizedDescription):
6:07 PM Changeset in webkit [42766] by Darin Adler
  • 2 edits in trunk/WebCore

2009-04-22 Darin Adler <Darin Adler>

  • page/Navigator.idl: Touch this file to fix build since Geolocation configuration was recently turned off.
6:02 PM Changeset in webkit [42765] by oliver@apple.com
  • 2 edits in trunk/LayoutTests

Include updated results

5:59 PM Changeset in webkit [42764] by oliver@apple.com
  • 2 edits in trunk/LayoutTests

Modify test to only record the important mouse events

Reviewed by Geoff Garen.

5:54 PM Changeset in webkit [42763] by justin.garcia@apple.com
  • 3 edits
    6 adds in trunk

WebCore:

2009-04-22 Justin Garcia <justin.garcia@apple.com>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=25322
Can't delete empty paragraph after a block image


Empty paragraph removal would have been handled by mergeParagraphs, but we stopped short
because of some code that avoided problems that would arise from Positions that would go
bad during deletion.


We have several checks that prevent us from using bad Positions, this one in particular
was added to avoid bad Positions that result from text removal. So, I removed the check
and started correctly updating m_downstreamEnd during text removal so that it doesn't go
bad. m_upstreamStart doesn't need to be updated during text removal, since only text
after it will ever be removed.

  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::deleteTextFromNode): (WebCore::DeleteSelectionCommand::handleGeneralDelete): (WebCore::DeleteSelectionCommand::mergeParagraphs):

LayoutTests:

2009-04-22 Justin Garcia <justin.garcia@apple.com>

Reviewed by Darin Adler.


https://bugs.webkit.org/show_bug.cgi?id=25322
Can't delete empty paragraph after a block image

  • editing/deleting/25322-1-expected.txt: Added.
  • editing/deleting/25322-1.html: Added.
  • editing/deleting/25322-2-expected.txt: Added.
  • editing/deleting/25322-2.html: Added.
  • editing/deleting/25322-3-expected.txt: Added.
  • editing/deleting/25322-3.html: Added.
5:49 PM Changeset in webkit [42762] by Darin Adler
  • 2 edits in trunk/LayoutTests

2009-04-22 Darin Adler <Darin Adler>

  • platform/mac/svg/W3C-SVG-1.1/animate-elem-36-t-expected.txt: Fixed one case Eric seems to have missed earlier today of RenderImage -> RenderSVGImage.
5:37 PM Changeset in webkit [42761] by oliver@apple.com
  • 7 edits in trunk

<rdar://problem/6757346> SAP: Prevent default on mouseDown does not stop iframe from capturing subsequent mouse moves

Reviewed by Darin Adler.

Make mouseUp target the correct frame when the original
mousedown made the drag non-capturing.

5:29 PM Changeset in webkit [42760] by oliver@apple.com
  • 5 edits
    3 adds in trunk

<rdar://problem/6757346> SAP: Prevent default on mouseDown does not stop iframe from capturing subsequent mouse moves

Reviewed by Darin Adler.

This is the first step of allowing drag events to match the behaviour
of mouse move events when the default action of the initial mouse down
is prevented. Remaining issue is that the final mouse up event still
targets the original root frame.

Test: fast/events/mouse-drag-from-frame-to-other-frame.html

5:01 PM Changeset in webkit [42759] by Simon Fraser
  • 2 edits in trunk/WebCore

2009-04-22 Simon Fraser <Simon Fraser>

Reviewed by Adele Peterson

<rdar://problem/6702882> QT movie's video fails to be displayed after switching between tabs
<rdar://problem/6754957> Resizing Web browser window and hitting play will cause video blocks and artifacting

When in a media document, MediaPlayerPrivateQTKit uses a QTMovieView which may get
layer backing under some circumstances. However, drawing the view via
displayRectIgnoringOpacity:inContext: bypasses any layer setup that AppKit normally performs.
So when in the media document, we draw via displayRectIgnoringOpacity:.

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::paint):
4:56 PM Changeset in webkit [42758] by mrowe@apple.com
  • 3 edits in trunk/LayoutTests

Update test results after Blackbeard's change in r42752 to disable geolocation.

  • fast/dom/Window/window-properties-expected.txt:
  • fast/dom/navigator-detached-no-crash-expected.txt:
3:59 PM Changeset in webkit [42757] by Simon Fraser
  • 2 edits
    2 adds in trunk

2009-04-22 Pierre d'Herbemont <pdherbemont@apple.com>

Reviewed by Simon Fraser.

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

Add tests for media 'played' attribute (missed in r42619).

  • media/video-played-expected.txt: Added.
  • media/video-played.html: Added.
3:51 PM Changeset in webkit [42756] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

2009-04-22 Anders Carlsson <andersca@apple.com>

Reviewed by Oliver Hunt.

Fix <rdar://problem/6792694>


When we're trying to instantiate a plug-in and the plug-in host has died, we need to invalidate the
instance so that it doesn't stick around and do bad things.


  • Plugins/Hosted/NetscapePluginHostManager.mm: (WebKit::NetscapePluginHostManager::instantiatePlugin):
2:52 PM Changeset in webkit [42755] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-04-22 Gavin Barraclough <barraclough@apple.com>

Rubber stamped by Sam "Blackbeard" Weinig.

Although pirates do spell the word 'generate' as 'genertate',
webkit developers do not. Fixerate.

  • yarr/RegexJIT.cpp: (JSC::Yarr::RegexGenerator::generateAssertionBOL): (JSC::Yarr::RegexGenerator::generateAssertionEOL): (JSC::Yarr::RegexGenerator::generateAssertionWordBoundary): (JSC::Yarr::RegexGenerator::generatePatternCharacterSingle): (JSC::Yarr::RegexGenerator::generatePatternCharacterPair): (JSC::Yarr::RegexGenerator::generatePatternCharacterFixed): (JSC::Yarr::RegexGenerator::generatePatternCharacterGreedy): (JSC::Yarr::RegexGenerator::generatePatternCharacterNonGreedy): (JSC::Yarr::RegexGenerator::generateCharacterClassSingle): (JSC::Yarr::RegexGenerator::generateCharacterClassFixed): (JSC::Yarr::RegexGenerator::generateCharacterClassGreedy): (JSC::Yarr::RegexGenerator::generateCharacterClassNonGreedy): (JSC::Yarr::RegexGenerator::generateTerm):
2:44 PM Changeset in webkit [42754] by barraclough@apple.com
  • 4 edits in trunk/JavaScriptCore

2009-04-22 Gavin Barraclough <barraclough@apple.com>

Reviewed by Sam "Blackbeard" Weinig.

Improvements to YARR JIT. This patch expands support in three key areas:

  • Add (temporary) support for falling back to PCRE for expressions not supported.
  • Add support for x86_64 and Windows.
  • Add support for singly quantified parentheses (? and ??), alternatives within parentheses, and parenthetical assertions.
  • runtime/RegExp.cpp: (JSC::RegExp::match):
  • yarr/RegexJIT.cpp: (JSC::Yarr::RegexGenerator::storeToFrame): (JSC::Yarr::RegexGenerator::storeToFrameWithPatch): (JSC::Yarr::RegexGenerator::loadFromFrameAndJump): (JSC::Yarr::RegexGenerator::AlternativeBacktrackRecord::AlternativeBacktrackRecord): (JSC::Yarr::RegexGenerator::TermGenerationState::resetAlternative): (JSC::Yarr::RegexGenerator::TermGenerationState::resetTerm): (JSC::Yarr::RegexGenerator::TermGenerationState::jumpToBacktrack): (JSC::Yarr::RegexGenerator::TermGenerationState::plantJumpToBacktrackIfExists): (JSC::Yarr::RegexGenerator::TermGenerationState::addBacktrackJump): (JSC::Yarr::RegexGenerator::TermGenerationState::linkAlternativeBacktracks): (JSC::Yarr::RegexGenerator::TermGenerationState::propagateBacktrackingFrom): (JSC::Yarr::RegexGenerator::genertateAssertionBOL): (JSC::Yarr::RegexGenerator::genertateAssertionEOL): (JSC::Yarr::RegexGenerator::matchAssertionWordchar): (JSC::Yarr::RegexGenerator::genertateAssertionWordBoundary): (JSC::Yarr::RegexGenerator::genertatePatternCharacterSingle): (JSC::Yarr::RegexGenerator::genertatePatternCharacterPair): (JSC::Yarr::RegexGenerator::genertatePatternCharacterFixed): (JSC::Yarr::RegexGenerator::genertatePatternCharacterGreedy): (JSC::Yarr::RegexGenerator::genertatePatternCharacterNonGreedy): (JSC::Yarr::RegexGenerator::genertateCharacterClassSingle): (JSC::Yarr::RegexGenerator::genertateCharacterClassFixed): (JSC::Yarr::RegexGenerator::genertateCharacterClassGreedy): (JSC::Yarr::RegexGenerator::genertateCharacterClassNonGreedy): (JSC::Yarr::RegexGenerator::generateParenthesesDisjunction): (JSC::Yarr::RegexGenerator::generateParenthesesSingle): (JSC::Yarr::RegexGenerator::generateParentheticalAssertion): (JSC::Yarr::RegexGenerator::generateTerm): (JSC::Yarr::RegexGenerator::generateDisjunction): (JSC::Yarr::RegexGenerator::generateEnter): (JSC::Yarr::RegexGenerator::generateReturn): (JSC::Yarr::RegexGenerator::RegexGenerator): (JSC::Yarr::RegexGenerator::generate): (JSC::Yarr::RegexGenerator::compile): (JSC::Yarr::RegexGenerator::generationFailed): (JSC::Yarr::jitCompileRegex): (JSC::Yarr::executeRegex):
  • yarr/RegexJIT.h: (JSC::Yarr::RegexCodeBlock::RegexCodeBlock): (JSC::Yarr::RegexCodeBlock::~RegexCodeBlock):
2:24 PM Changeset in webkit [42753] by timothy@apple.com
  • 2 edits in trunk/WebCore

Fix a crash in Mail when forwarding a specific HTML email.
Caused by the fix for <rdar://problem/6512520> (r42096).
Unable to create a test case that works outside of Mail.

<rdar://problem/6796069>

Reviewed by Ada Chan.

  • editing/markup.cpp: (WebCore::createMarkup): Null check pastEnd before trying to call Range::compareBoundaryPoints.
1:44 PM QtWebKitJournal edited by tonikitoo@gmail.com
(diff)
12:09 PM Changeset in webkit [42752] by weinig@apple.com
  • 8 edits in trunk

JavaScriptCore:

2009-04-22 Sam Weinig <sam@webkit.org>

Rubber-stamped by Darin Adler.

Fix for <rdar://problem/6816957>
Turn off Geolocation by default

  • Configurations/FeatureDefines.xcconfig:

WebCore:

2009-04-22 Sam Weinig <sam@webkit.org>

Rubber-stamped by Darin Adler.

Fix for <rdar://problem/6816957>
Turn off Geolocation by default

  • Configurations/FeatureDefines.xcconfig:

WebKit/mac:

2009-04-22 Sam Weinig <sam@webkit.org>

Rubber-stamped by Darin Adler.

Fix for <rdar://problem/6816957>
Turn off Geolocation by default

  • Configurations/FeatureDefines.xcconfig:

WebKitTools:

2009-04-22 Sam Weinig <sam@webkit.org>

Rubber-stamped by Darin Adler.

Fix for <rdar://problem/6816957>
Turn off Geolocation by default

  • Scripts/build-webkit:
11:03 AM Changeset in webkit [42751] by eric@webkit.org
  • 43 edits in trunk

Rubber-stamped by David Hyatt.

Change RenderSVGImage to dump as RenderSVGImage instead of RenderImage

  • rendering/RenderSVGImage.h: (WebCore::RenderSVGImage::renderName):
8:48 AM Changeset in webkit [42750] by timothy@apple.com
  • 2 edits in trunk/WebCore

Fix a crash on Gmail when they remove a "before unload" event listener when
it was never added to the pending map in the first place.

<rdar://problem/6814144>

Reviewed by Darin Adler.

  • page/DOMWindow.cpp: (WebCore::allowsPendingBeforeUnloadListeners): Renamed from shouldAddPendingBeforeUnloadListener. (WebCore::DOMWindow::addEventListener): Call allowsPendingBeforeUnloadListeners now. (WebCore::DOMWindow::removeEventListener): Call allowsPendingBeforeUnloadListeners before removing the before unload listener. (WebCore::DOMWindow::clearAttributeEventListener): Ditto.
6:39 AM Changeset in webkit [42749] by ariya@webkit.org
  • 4 edits in trunk/WebCore

2009-04-22 Kenneth Rohde Christiansen <kenneth.christiansen@openbossa.org>

Reviewed by Ariya Hidayat.

Made windowed plugins move/resize synchronized with the painting, so
that windowed plugins are not moved before the rest of the parent
frame during scrolling.

  • plugins/qt/PluginContainerQt.cpp: (PluginContainerQt::PluginContainerQt): (PluginContainerQt::requestGeometry): (PluginContainerQt::adjustGeometry):
  • plugins/qt/PluginContainerQt.h:
  • plugins/qt/PluginViewQt.cpp: (WebCore::PluginView::updatePluginWidget): (WebCore::PluginView::paint):
5:34 AM Changeset in webkit [42748] by Simon Hausmann
  • 6 edits
    2 adds in trunk/WebCore

2009-04-22 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Ariya Hidayat.

Fix various bugs in the X11 Qt plugin focus handling.

Notify the focus controller on activation/deactivation of the plugin and
set focus on the plugin's element when it receives the window system
focus.

5:08 AM Changeset in webkit [42747] by ariya@webkit.org
  • 2 edits in trunk/WebCore

2009-04-22 Tamas Szirbucz <szirbucz.tamas@stud.u-szeged.hu>

Reviewed by Ariya Hidayat.

https://bugs.webkit.org/show_bug.cgi?id=25023
Delete reply in QNetworkReplyHandler::abort() to avoid leak.

  • platform/network/qt/QNetworkReplyHandler.cpp: (WebCore::QNetworkReplyHandler::abort):
3:44 AM QtWebKitJournal edited by Simon Hausmann
(diff)
1:52 AM Changeset in webkit [42746] by jhoneycutt@apple.com
  • 12 edits in trunk

2009-04-21 Jon Honeycutt <jhoneycutt@apple.com>

Allow the UI delegate to control cursor-setting.

Reviewed by Ada Chan.

WebCore/:

  • page/Chrome.cpp: (WebCore::Chrome::setCursor): Pass the call to the client.
  • page/Chrome.h:
  • page/ChromeClient.h: (WebCore::ChromeClient::setCursor): Stubbed; returns false.
  • platform/Cursor.h: Define PlatformCursorHandle for each platform.
  • platform/win/WidgetWin.cpp: Reordered some includes. Added an include of Page.h. (WebCore::Widget::setCursor): Fixed a typo in the comment about ignoreNextSetCursor. Get the Page, and call its Chrome's setCursor() method with the passed cursor. Fall back to SetCursor() if the Page is not available.
  • plugins/win/PluginViewWin.cpp: (WebCore::PluginView::handleMouseEvent): Remove an unused variable. Cleaned up some whitespace.

WebKit/win/:

  • Interfaces/IWebUIDelegatePrivate.idl: Added a function to the IWebUIDelegatePrivate5 interface.
  • WebCoreSupport/WebChromeClient.cpp: (WebChromeClient::setCursor): Try to call the UI delegate's webViewSetCursor() method. If it fails, fall back to calling SetCursor() with the passed cursor.
  • WebCoreSupport/WebChromeClient.h:
  • WebView.cpp: (WebViewWndProc): Call the Chrome's setCursor() method with the last set cursor.
12:22 AM Changeset in webkit [42745] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix

Apr 21, 2009:

10:42 PM Changeset in webkit [42744] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix

10:36 PM Changeset in webkit [42743] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

Improve String.replace performance slightly

Reviewed by Maciej Stachowiak

Apply our vm reentry caching logic to String.replace with global
regexes.

10:22 PM Changeset in webkit [42742] by mitz@apple.com
  • 2 edits in trunk/LayoutTests
  • updated another result with action element information
  • fast/encoding/mailto-always-utf-8-expected.txt:
10:05 PM Changeset in webkit [42741] by mitz@apple.com
  • 5 edits
    2 adds in trunk

WebKit/mac:

Reviewed by Jon Honeycutt.

  • Mac part of fixing for <rdar://problem/6755137> Action dictionary for policy decision is missing keys when full-page zoom is used
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::actionDictionary): Use absoluteLocation() instead of pageX() and pageY(), which are adjusted for zoom.

WebKit/win:

Reviewed by Jon Honeycutt.

  • Windows part of fixing <rdar://problem/6755137> Action dictionary for policy decision is missing keys when full-page zoom is used
  • WebActionPropertyBag.cpp: (WebActionPropertyBag::Read): Use absoluteLocation() instead of clientX() and clientY(), which are adjusted for zoom.

LayoutTests:

  • test for <rdar://problem/6755137> Action dictionary for policy decision is missing keys when full-page zoom is used
  • fast/loader/policy-delegate-action-hit-test-zoomed-expected.txt: Added.
  • fast/loader/policy-delegate-action-hit-test-zoomed.html: Added.
10:04 PM Changeset in webkit [42740] by mitz@apple.com
  • 8 edits in trunk

WebKit/win:

Reviewed by Jon Honeycutt.

  • implement DOMNode::nodeName()
  • DOMCoreClasses.cpp: (DOMNode::nodeName):

WebKitTools:

Reviewed by Jon Honeycutt.

  • dump the node in the element dictionary in the action information passed to the policy delegate
  • DumpRenderTree/mac/PolicyDelegate.mm: (-[PolicyDelegate webView:decidePolicyForNavigationAction:request:frame:decisionListener:]):
  • DumpRenderTree/win/PolicyDelegate.cpp: (dumpPath): (PolicyDelegate::decidePolicyForNavigationAction):

LayoutTests:

Reviewed by Jon Honeycutt.

  • updated result that shows the node in the element dictionary in the action information passed to the policy delegate
  • http/tests/security/feed-urls-from-remote-expected.txt:
9:20 PM Changeset in webkit [42739] by Dimitri Glazkov
  • 2 edits in trunk/WebCore

2009-04-21 John Abd-El-Malek <jam@google.com>

Reviewed by Dimitri Glazkov.

Add a method to purge the Chromium plugin list cache.
https://bugs.webkit.org/show_bug.cgi?id=25318

  • plugins/chromium/PluginDataChromium.cpp: (WebCore::resetChromiumPluginCache):
9:03 PM Changeset in webkit [42738] by mrowe@apple.com
  • 12 edits
    4 adds in trunk

Fix <https://bugs.webkit.org/show_bug.cgi?id=25164> / <rdar://problem/6786319>.

Reviewed by Darin Adler.

We need to ensure that XMLTokenizer sets the current DocLoader before calling in to
any libxml2 methods that may trigger a load. The presence of a DocLoader indicates
that the load was originated by WebCore's use of libxml2 and that we should enforce
the same-origin policy on it. XMLTokenizer::initializeParserContext,
XMLTokenizer::doWrite and XMLTokenizer::doEnd were three methods that were not setting
the current DocLoader when they should have.

The XMLTokenizerScope class is introduced to simplify the pattern of saving, setting and
restoring the current DocLoader and libxml2 error handlers. The DocLoader and error handlers
are saved and set when the scope is allocated, and restored to their previous values when
the scope is exited.

Test: http/tests/security/xss-DENIED-xml-external-entity.xhtml

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.scons:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • dom/XMLTokenizer.cpp:
  • dom/XMLTokenizer.h:
  • dom/XMLTokenizerLibxml2.cpp:

(WebCore::matchFunc):
(WebCore::shouldAllowExternalLoad):
(WebCore::openFunc):
(WebCore::XMLTokenizer::doWrite):
(WebCore::XMLTokenizer::initializeParserContext):
(WebCore::XMLTokenizer::doEnd):
(WebCore::xmlDocPtrForString):

  • dom/XMLTokenizerScope.cpp:

(WebCore::XMLTokenizerScope::XMLTokenizerScope):
(WebCore::XMLTokenizerScope::~XMLTokenizerScope):

  • dom/XMLTokenizerScope.h:
  • xml/XSLStyleSheet.cpp:

(WebCore::XSLStyleSheet::parseString):

  • xml/XSLTProcessor.cpp:
8:42 PM Changeset in webkit [42737] by weinig@apple.com
  • 6 edits in trunk

WebCore:

2009-04-21 Sam Weinig <sam@webkit.org>

Reviewed by Dan Bernstein.

Fix for <rdar://problem/5874009>
Add port 3659 (apple-sasl / PasswordServer) to the list of blocked ports.

  • platform/network/ResourceHandle.cpp: (WebCore::portAllowed):

LayoutTests:

2009-04-21 Sam Weinig <sam@webkit.org>

Reviewed by Dan Bernstein.

Update test and results for <rdar://problem/5874009>
Add port 3659 (apple-sasl / PasswordServer) to the list of blocked ports.

  • platform/mac-tiger/security/block-test-expected.txt:
  • platform/mac/security/block-test-expected.txt:
  • security/block-test.html:
7:18 PM Changeset in webkit [42736] by justin.garcia@apple.com
  • 3 edits in trunk/LayoutTests

Corrected a mistake in my last ChangeLog entry and updated some expected results.

6:53 PM Changeset in webkit [42735] by justin.garcia@apple.com
  • 3 edits
    2 adds in trunk

WebCore:

2009-04-21 Justin Garcia <justin.garcia@apple.com>

Reviewed by Dan Bernstein.

https://bugs.webkit.org/show_bug.cgi?id=25320
Crash when changing block styles in a root that contains no visible content (but contains nodes with renderers)

  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::moveParagraphContentsToNewBlockIfNecessary): If the enclosing block is the root editable element and it contains no visible content, create a new block but don't try and move content into it, since there's nothing for moveParagraphs to move.

LayoutTests:

2009-04-21 Justin Garcia <justin.garcia@apple.com>

Reviewed by Dan Bernstein.

https://bugs.webkit.org/show_bug.cgi?id=25320
Crash when changing block styles in a root that contains no visible content (but contains nodes with renderers)

  • editing/execCommand/25320-expected.txt: Added.
  • editing/execCommand/25320.html: Added.
  • platform/mac/fast/forms/caret-rtl-expected.checksum: Replaced.
  • platform/mac/fast/forms/caret-rtl-expected.txt: Replaced.
5:18 PM Changeset in webkit [42734] by ggaren@apple.com
  • 4 edits in trunk/JavaScriptCore

2009-04-21 Geoffrey Garen <ggaren@apple.com>

Reviewed by Cameron Zwarich and Oliver Hunt.


Re-Fixed <rdar://problem/6406045> REGRESSION: Stack overflow on PowerPC on
fast/workers/use-machine-stack.html (22531)


SunSpider reports no change.


Use a larger recursion limit on the main thread (because we can, and
there's some evidence that it may improve compatibility), and a smaller
recursion limit on secondary threads (because they tend to have smaller
stacks).

  • interpreter/Interpreter.cpp: (JSC::Interpreter::execute): (JSC::Interpreter::prepareForRepeatCall):
  • interpreter/Interpreter.h: (JSC::): Ditto. I wrote the recursion test slightly funny, so that the common case remains a simple compare to constant.
  • runtime/ArrayPrototype.cpp: (JSC::arrayProtoFuncToString): (JSC::arrayProtoFuncToLocaleString): (JSC::arrayProtoFuncJoin): Conservatively, set the array recursion limits to the lower, secondary thread limit. We can do something fancier if compatibility moves us, but this seems sufficient for now.
5:11 PM Changeset in webkit [42733] by sfalken@apple.com
  • 2 edits in trunk/WebKit/win

2009-04-21 Steve Falkenburg <sfalken@apple.com>

Minor clean-ups to WebKitCreateInstance change.


Reviewed by Jon Honeycutt.

  • WebKitCOMAPI.cpp: (classFactory): (WebKitCreateInstance):
4:30 PM Changeset in webkit [42732] by Darin Adler
  • 15 edits
    7 copies in trunk

WebCore:

2009-04-21 Darin Adler <Darin Adler>

Reviewed by Dan Bernstein.

Bug 25162: REGRESSION (r41176-r41242): Drag-selection above/below the line does not select to beginning/end
https://bugs.webkit.org/show_bug.cgi?id=25162
rdar://problem/6764354

  • page/Settings.cpp: (WebCore::Settings::Settings): Initialize editing behavior to Mac-style for PLATFORM(MAC). Later we may want to change this default to encompass other versions running on Mac.
  • page/Settings.h: Added EditingBehavior, setEditingBehavior, editingBehavior, and m_editingBehavior. Also moved m_maximumDecodedImageSize out of the middle of all the bit fields.
  • platform/graphics/IntPoint.h: Added a constructor to turn an IntSize into an IntPoint. I'm not sure the distinction here is serving us well at the moment. When converting from global to local coordinates you want to do IntPoint - IntPoint and have the result be another IntPoint, not an IntSize. And so on.
  • rendering/RenderBlock.cpp: (WebCore::positionForPointRespectingEditingBoundaries): Changed to use pointers more and separate coordinates less. (WebCore::RenderBlock::positionForPointWithInlineChildren): Turned into a member function. Added separate cases to support the Mac and Windows behavior. (WebCore::RenderBlock::positionForPoint): Updated for the change above. Also moved the computation of pointInContents closer to the place it's used.
  • rendering/RenderBlock.h: Updated for the new private member function.

LayoutTests:

2009-04-21 Darin Adler <Darin Adler>

Reviewed by Dan Bernstein.

Bug 25162: REGRESSION (r41176-r41242): Drag-selection above/below the line does not select to beginning/end
https://bugs.webkit.org/show_bug.cgi?id=25162
rdar://problem/6764354

  • editing/selection/resources/click-in-margins-inside-editable-div.js: Updated to expect different results on the Mac.
  • editing/selection/resources/click-in-padding-with-multiple-line-boxes.js: Ditto.
  • fast/css-generated-content/positioned-background-hit-test-crash.html: Clear the selection at the end of the test so the result won't include a caret.
  • platform/mac/editing/selection/click-in-margins-inside-editable-div-expected.txt:
  • platform/mac/editing/selection/click-in-padding-with-multiple-line-boxes-expected.txt: Copied from LayoutTests/editing/selection. Then changed to expect the different Mac results.
  • platform/mac/editing/selection/select-from-textfield-outwards-expected.checksum:
  • platform/mac/editing/selection/select-from-textfield-outwards-expected.png:
  • platform/mac/editing/selection/select-from-textfield-outwards-expected.txt: Changed to expect the different Mac results.
  • platform/mac/fast/css-generated-content/positioned-background-hit-test-crash-expected.txt: Changed to expect no caret.
  • platform/win/editing/selection/click-in-margins-inside-editable-div-expected.txt:
  • platform/win/editing/selection/click-in-padding-with-multiple-line-boxes-expected.txt: Copied from editing/selection.
  • platform/win/editing/selection/select-from-textfield-outwards-expected.checksum:
  • platform/win/editing/selection/select-from-textfield-outwards-expected.png:
  • platform/win/editing/selection/select-from-textfield-outwards-expected.txt: Copied from platform/mac/editing/selection.
2:58 PM Changeset in webkit [42731] by Adam Roben
  • 2 edits in trunk/WebKit

Update StringsNotToBeLocalized.txt for recent changes

Rubber-stamped by John Sullivan.

  • StringsNotToBeLocalized.txt:
2:35 PM Changeset in webkit [42730] by andersca@apple.com
  • 6 edits in trunk/WebKit/mac

2009-04-21 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler and Kevin Decker.

WebKit side of <rdar://problem/6781642>.


When we call resize with an actual changed size, block until the plug-in host is done.


  • Plugins/Hosted/NetscapePluginInstanceProxy.h:
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::resize):
  • Plugins/Hosted/WebHostedNetscapePluginView.h:
  • Plugins/Hosted/WebHostedNetscapePluginView.mm: (-[WebHostedNetscapePluginView updateAndSetWindow]):
  • Plugins/Hosted/WebKitPluginHost.defs:
2:10 PM Changeset in webkit [42729] by Adam Roben
  • 3 edits in trunk/WebKitTools

Windows build fix after r42726

  • DumpRenderTree/win/DumpRenderTree.cpp: Touched to force a rebuild.
  • DumpRenderTree/win/DumpRenderTree.vcproj: Delete any pre-r42726 manifest files during the pre-build event.
1:52 PM Changeset in webkit [42728] by timothy@apple.com
  • 2 edits in trunk/WebKit

Another attempt to fix the Windows and QT builds.

1:48 PM Changeset in webkit [42727] by timothy@apple.com
  • 2 edits in trunk/WebKit

Attempt to fix the Windows and QT builds.

1:42 PM Changeset in webkit [42726] by sfalken@apple.com
  • 10 edits
    2 adds in trunk

WebKit/win:

2009-04-21 Steve Falkenburg <sfalken@apple.com>

Add registry-free CoCreateInstance API for WebKit clients: WebKitCreateInstance.

Manifest-based registry-free COM continues to work for clients that have embedded
manifests with isolated COM information.


Reviewed by Adam Roben.

  • WebKit.vcproj/WebKit.def: Added WebKitCreateInstance.
  • WebKit.vcproj/WebKit.vcproj: Link against rpcrt4.dll for UUID hashing.
  • WebKit.vcproj/WebKit_debug.def: Added WebKitCreateInstance.
  • WebKitCOMAPI.cpp: Added. (CLSIDHash::hash): (CLSIDHash::equal): (CLSIDHashTraits::constructDeletedValue): (CLSIDHashTraits::isDeletedValue): (classFactory): (WebKitCreateInstance): New API.
  • WebKitCOMAPI.h: Added.

WebKitTools:

2009-04-21 Steve Falkenburg <sfalken@apple.com>

Use new WebKitCreateInstance API instead of CoCreateInstance
or COMPtr's createInstance to instantiate WebKit COM objects.

Reviewed by Adam Roben.

  • DumpRenderTree/win/DumpRenderTree.cpp: (initialize): (runTest): (createWebViewAndOffscreenWindow): (main):
  • DumpRenderTree/win/DumpRenderTree.vcproj:
  • DumpRenderTree/win/GCControllerWin.cpp: (GCController::collect): (GCController::collectOnAlternateThread): (GCController::getJSObjectCount):
  • DumpRenderTree/win/LayoutTestControllerWin.cpp: (LayoutTestController::keepWebHistory): (LayoutTestController::webHistoryItemCount): (LayoutTestController::setIconDatabaseEnabled):
  • DumpRenderTree/win/WorkQueueItemWin.cpp: (LoadItem::invoke):
1:21 PM Changeset in webkit [42725] by timothy@apple.com
  • 24 edits in trunk

Change pending unload and beforeunload counts to store the listeners in a single data structure that can be quickly iterated for dispatch.

WebCore:

2009-04-17 Timothy Hatcher <timothy@apple.com>

Change pending unload and beforeunload counts to store the listeners
in a single data structure that can be quickly iterated for dispatch.

<rdar://problem/6383352&6383379&6383940>

Reviewed by Darin Adler.

  • WebCore.base.exp: Change what symbols are exported.
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::stopLoading): Remove count updating code and move an if inside a block already checking m_frame->document().
  • page/Chrome.cpp: Remove disableSuddenTermination/enableSuddenTermination.
  • page/Chrome.h: Ditto.
  • page/ChromeClient.h: Ditto.
  • page/DOMWindow.cpp: (WebCore::pendingUnloadEventListenerMap): Create a static map. (WebCore::pendingBeforeUnloadEventListenerMap): Ditto. (WebCore::addPendingEventListener): Add the event listener to the map. (WebCore::removePendingEventListener): Remove the event listener from the map. (WebCore::removePendingEventListeners): Remove all listeners for the window. (WebCore::dispatchWindowEventToListeners): Dispatch the event to the given listeners. (WebCore::DOMWindow::dispatchAllPendingBeforeUnloadEvents): Send the beforeunload event. (WebCore::DOMWindow::pendingUnloadEventListeners): Return a count of pending listeners. (WebCore::DOMWindow::dispatchAllPendingUnloadEvents): Send the unload event. (WebCore::DOMWindow::~DOMWindow): Remove the window from both event listener maps. (WebCore::DOMWindow::addEventListener): Call addPendingEventListener when needed. (WebCore::DOMWindow::removeEventListener): Call removePendingEventListener when needed. (WebCore::DOMWindow::removeAllEventListeners): Call removePendingEventListeners. (WebCore::DOMWindow::removeInlineEventListenerForType): Call removePendingEventListener when needed.
  • page/DOMWindow.h: (WebCore::DOMWindow::frame): Changed to be const.
  • page/EventHandler.cpp: (WebCore::EventHandler::EventHandler): Remove count tracking code.
  • page/EventHandler.h: Ditto.
  • page/Page.cpp: (WebCore::Page::Page): Ditto.
  • page/Page.h: Ditto.

WebKit/mac:

2009-04-17 Timothy Hatcher <timothy@apple.com>

Change how sudden termination works with WebView teardown.

<rdar://problem/6383352&6383379&6383940>

Reviewed by Darin Adler.

  • WebCoreSupport/WebChromeClient.h: Remove disableSuddenTermination/enableSuddenTermination.
  • WebCoreSupport/WebChromeClient.mm: Ditto.
  • WebView/WebFrame.mm: (-[WebFrame _pendingFrameUnloadEventCount]): Ask the DOMWindow.
  • WebView/WebView.mm: (+[WebView canCloseAllWebViews]): Call DOMWindow::dispatchAllPendingBeforeUnloadEvents. (+[WebView closeAllWebViews]): Call DOMWindow::dispatchAllPendingUnloadEvents and call close on all the WebViews. (-[WebView _closeWithFastTeardown]): Remove code for unload event dispatch. (-[WebView _close]): Correct a comment. (+[WebView _applicationWillTerminate]): Call closeAllWebViews.
  • WebView/WebViewPrivate.h: Add canCloseAllWebViews and closeAllWebViews.
12:58 PM Changeset in webkit [42724] by ggaren@apple.com
  • 4 edits in trunk

JavaScriptCore:

2009-04-21 Geoffrey Garen <ggaren@apple.com>

Rubber-stamped by Adam Roben.


Disabled one more Mozilla JS test because it fails intermittently on Windows.
(See https://bugs.webkit.org/show_bug.cgi?id=25160.)

  • tests/mozilla/expected.html:

WebKitTools:

2009-04-21 Geoffrey Garen <ggaren@apple.com>

Rubber-stamped by Adam Roben.


Disabled one more Mozilla JS test because it fails intermittently on Windows.
(See https://bugs.webkit.org/show_bug.cgi?id=25160.)

  • Scripts/run-javascriptcore-tests:
12:38 PM Changeset in webkit [42723] by Darin Adler
  • 14 edits
    1 move in trunk/LayoutTests

2009-04-21 Darin Adler <Darin Adler>

Reviewed by Adele Peterson.

Fix some tests that are depending on Mac vs. Windows selection quirks accidentally.
Added failing tests to the skipped list for SnowLeopard.

  • editing/resources/select-all-iframe-src.html: Moved.
  • editing/selection/resources/select-all-iframe-src.html: Moved from editing/resources/select-all-iframe-src.html, since this is used by a test in the selection directory. Changed mouse coordinates to be farther to the right so we don't depend on the rule for what's selected when you click below text on the last line.
  • editing/selection/select-all-iframe.html: Updated for new location of file above.
  • fast/events/standalone-image-drag-to-editable.html: Changed mouse coordinates to be farther to the right so we don't depend on the rule for what's selected when you click below text on the last line.
  • platform/mac-snowleopard/Skipped: Added some tests to the Skipped list.
  • svg/custom/pointer-events-path.svg: Clear the selection at the end of the test so it's not in the test results. Also removed unneeded use of the layout test controller.
  • platform/mac/editing/selection/select-all-iframe-expected.checksum: Generated new results.
  • platform/mac/editing/selection/select-all-iframe-expected.png: Ditto.
  • platform/mac/editing/selection/select-all-iframe-expected.txt: Ditto.
  • platform/mac/fast/events/standalone-image-drag-to-editable-expected.checksum: Ditto.
  • platform/mac/fast/events/standalone-image-drag-to-editable-expected.png: Ditto.
  • platform/mac/fast/events/standalone-image-drag-to-editable-expected.txt: Ditto.
  • platform/mac/svg/custom/pointer-events-path-expected.checksum: Ditto.
  • platform/mac/svg/custom/pointer-events-path-expected.png: Ditto.
  • platform/mac/svg/custom/pointer-events-path-expected.txt: Ditto.
12:20 PM Changeset in webkit [42722] by justin.garcia@apple.com
  • 8 edits
    4 adds in trunk

WebCore:

2009-04-21 Justin Garcia <justin.garcia@apple.com>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=19644
Text copied with Select All pastes with a indent but shouldn't
<rdar://problem/6102483>
Pasting the content of an HTML message in Mail causes addition nested <div> elements to be added


To improve selectall/copy/paste fidelity of certain pages, we began wrapping copied content with
a div that held properties and attributes from the fully selected body. To fix the above issues,
only do this if if the body has certain properties or attributes. We'll begin adding to this list as
necessary. For now it's just background colors and images. Tested copy/paste of nytimes, wired,
arstechnica, and several others.


Massive nesting can still happen, it will just be much less common.

  • editing/markup.cpp: (WebCore::createMarkup):

LayoutTests:

2009-04-21 Justin Garcia <justin.garcia@apple.com>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=19644
Text copied with Select All pastes with a indent but shouldn't
<rdar://problem/6102483>
Pasting the content of an HTML message in Mail causes addition nested <div> elements to be added

  • editing/pasteboard/19644-1-expected.txt: Added.
  • editing/pasteboard/19644-1.html: Added.
  • editing/pasteboard/19644-2-expected.txt: Added.
  • editing/pasteboard/19644-2.html: Added.
  • platform/mac/editing/inserting/insert-3907422-fix-expected.txt:
  • platform/mac/editing/pasteboard/paste-text-015-expected.checksum:
  • platform/mac/editing/pasteboard/paste-text-015-expected.png:
  • platform/mac/editing/pasteboard/paste-text-015-expected.txt:
  • platform/mac/editing/style/font-family-with-space-expected.txt:
11:34 AM Changeset in webkit [42721] by pkasting@chromium.org
  • 9 edits in trunk/WebCore

2009-04-16 Peter Kasting <pkasting@google.com>

Reviewed by David Hyatt.

https://bugs.webkit.org/show_bug.cgi?id=25250
Allow platforms to snap the scroll thumb back to the drag origin
during a drag. Implement functions for Safari/Win and Chromium/Win
to do this snapping at distances approximating the native ones.

  • platform/Scrollbar.cpp: (WebCore::Scrollbar::Scrollbar): (WebCore::Scrollbar::setValue): (WebCore::Scrollbar::scroll): (WebCore::Scrollbar::moveThumb): (WebCore::Scrollbar::setCurrentPos): (WebCore::Scrollbar::mouseMoved): (WebCore::Scrollbar::mouseDown):
  • platform/Scrollbar.h:
  • platform/ScrollbarTheme.h: (WebCore::ScrollbarTheme::shouldSnapBackToDragOrigin):
  • platform/chromium/ScrollbarThemeChromium.h:
  • platform/chromium/ScrollbarThemeChromiumLinux.cpp: (WebCore::ScrollbarThemeChromium::shouldSnapBackToDragOrigin):
  • platform/chromium/ScrollbarThemeChromiumWin.cpp: (WebCore::ScrollbarThemeChromium::shouldSnapBackToDragOrigin):
  • platform/win/ScrollbarThemeWin.cpp: (WebCore::ScrollbarThemeWin::shouldCenterOnThumb): (WebCore::ScrollbarThemeWin::shouldSnapBackToDragOrigin):
  • platform/win/ScrollbarThemeWin.h:
11:15 AM Changeset in webkit [42720] by Adam Roben
  • 2 edits in trunk/WebCore

Windows build fix

  • bindings/js/JSLazyEventListener.cpp: (WebCore::eventParameterName): Remove unreachable code when SVG is enabled.
11:09 AM Changeset in webkit [42719] by Adam Roben
  • 3 edits in trunk/WebKitTools

Fix running run-safari and run-webkit-tests from the paths recommended on webkit.org

Reviewed by Steve Falkenburg and Sam Weinig.

  • Scripts/run-webkit-nightly.cmd: Moved quotes from variable definitions to variable uses. Otherwise cmd.exe can get confused about the quoting.
  • Scripts/run-webkit-tests: Changed to use an absolute path to execAppWithEnv so that run-webkit-tests will work when run from outside the source tree.
11:09 AM Changeset in webkit [42718] by Adam Roben
  • 8 edits in trunk

Rename JavaScriptCore_debug.dll to JavaScriptCore.dll in the Debug configuration

JavaScriptCore:

Rename JavaScriptCore_debug.dll to JavaScriptCore.dll in the Debug
configuration

This matches the naming scheme for WebKit.dll, and will be necessary
once Safari links against JavaScriptCore.dll. This change also causes
run-safari not to fail (because the launcher printed by FindSafari was
always looking for JavaScriptCore.dll, never
JavaScriptCore_debug.dll).

Part of Bug 25305: can't run safari or drt on windows
<https://bugs.webkit.org/show_bug.cgi?id=25305>

Reviewed by Steve Falkenburg and Sam Weinig.

WebKit/win:

Rename JavaScriptCore_debug.dll to JavaScriptCore.dll in the Debug
configuration

This matches the naming scheme for WebKit.dll, and will be necessary
once Safari links against JavaScriptCore.dll. This change also causes
run-safari not to fail (because the launcher printed by FindSafari was
always looking for JavaScriptCore.dll, never
JavaScriptCore_debug.dll).

Part of Bug 25305: can't run safari or drt on windows
<https://bugs.webkit.org/show_bug.cgi?id=25305>

Reviewed by Steve Falkenburg and Sam Weinig.

  • WebKit.vcproj/WebKit.vcproj: Use $(WebKitDLLConfigSuffix) for naming JavaScriptCore.{dll,lib}.

WebKitTools:

Rename JavaScriptCore_debug.dll to JavaScriptCore.dll in the Debug
configuration

This matches the naming scheme for WebKit.dll, and will be necessary
once Safari links against JavaScriptCore.dll. This change also causes
run-safari not to fail (because the launcher printed by FindSafari was
always looking for JavaScriptCore.dll, never
JavaScriptCore_debug.dll).

Part of Bug 25305: can't run safari or drt on windows
<https://bugs.webkit.org/show_bug.cgi?id=25305>

Reviewed by Steve Falkenburg and Sam Weinig.

  • DumpRenderTree/win/DumpRenderTree.vcproj: Use $(WebKitDLLConfigSuffix) for naming JavaScriptCore.{dll,lib}.
11:08 AM Changeset in webkit [42717] by Adam Roben
  • 2 edits in trunk/JavaScriptCore

Fix JavaScriptCore build on VC++ Express

Reviewed by Steve Falkenburg and Sam Weinig.

  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj: Link explicitly against gdi32.lib and oleaut32.lib.
11:04 AM BuildingOnWindows edited by Adam Roben
Removed now-obsolete comments about copying DLLs in order to run the … (diff)
10:26 AM Changeset in webkit [42716] by eric@webkit.org
  • 1 edit
    5 adds in trunk/LayoutTests

2009-04-21 Dave Moore <davemoore@google.com> and Eric Seidel <eric@webkit.org>

Reviewed by Darin Adler.

Add test for (as of yet not implemented) HTML5 named option access.
https://bugs.webkit.org/show_bug.cgi?id=25191

I believe this will require separate implementations
for V8 and JSC the way our current bindings are designed.

  • fast/dom/HTMLSelectElement/named-options-expected.txt: Added.
  • fast/dom/HTMLSelectElement/named-options.html: Added.
  • fast/dom/HTMLSelectElement/resources/TEMPLATE.html: Copied from LayoutTests/fast/dom/CSSStyleDeclaration/resources/TEMPLATE.html.
  • fast/dom/HTMLSelectElement/resources/named-options.js: Added.
10:20 AM Changeset in webkit [42715] by kevino@webkit.org
  • 2 edits in trunk/WebCore

!ENABLE(SVG) build fix.

10:00 AM Changeset in webkit [42714] by Dimitri Glazkov
  • 5 edits in trunk/WebCore

2009-04-21 Dave Moore <davemoore@google.com>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25309
Better use AtomicStrings when calling from the V8
bindings into WebCore code.

  • bindings/v8/custom/V8DOMWindowCustom.cpp:
  • bindings/v8/custom/V8HTMLCollectionCustom.cpp:
  • bindings/v8/custom/V8HTMLFormElementCustom.cpp:
  • bindings/v8/custom/V8HTMLFrameSetElementCustom.cpp:
9:19 AM Changeset in webkit [42713] by Chris Fleizach
  • 3 edits
    2 adds in trunk

Bug 25304: AX: Safari not identifying a secure text field
https://bugs.webkit.org/show_bug.cgi?id=25304

9:10 AM Changeset in webkit [42712] by mitz@apple.com
  • 2 edits in trunk/WebCore
  • fix the Tiger build
  • page/mac/EventHandlerMac.mm: (method_setImplementation):
8:48 AM Changeset in webkit [42711] by cwzwarich@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

2009-04-21 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Darin Adler.

Add a test for bug 3235: CSS2: Table cell coalescing in CSS broken
<https://bugs.webkit.org/show_bug.cgi?id=3235>
<rdar://3299883/>

This was fixed long ago, but we should add a test to prevent a regression.

  • fast/table/cell-coalescing.html: Added.
  • platform/mac/fast/table/cell-coalescing-expected.txt: Added.
8:15 AM Changeset in webkit [42710] by ariya@webkit.org
  • 7 edits in trunk/WebKit/qt

2009-04-21 Antonio Gomes <antonio.gomes@openbossa.org>

Reviewed by Ariya Hidayat.

QWebElementSelection renamed to QWebElementCollection.

  • Api/qwebelement.cpp: (QWebElement::findAll): (QWebElementCollectionPrivate::QWebElementCollectionPrivate): (QWebElementCollectionPrivate::create): (QWebElementCollection::QWebElementCollection): (QWebElementCollection::operator=): (QWebElementCollection::~QWebElementCollection): (QWebElementCollection::operator+): (QWebElementCollection::append): (QWebElementCollection::count): (QWebElementCollection::at): (QWebElementCollection::toList):
  • Api/qwebelement.h:
  • Api/qwebframe.cpp: (QWebFrame::findAllElements):
  • Api/qwebframe.h:
  • QtLauncher/main.cpp: (MainWindow::selectElements):
  • tests/qwebelement/tst_qwebelement.cpp: (tst_QWebElement::simpleCollection): (tst_QWebElement::iteration): (tst_QWebElement::emptyCollection): (tst_QWebElement::appendCollection): (tst_QWebElement::nullSelect):
6:46 AM QtWebKitTodo edited by Simon Hausmann
(diff)
6:18 AM Changeset in webkit [42709] by ariya@webkit.org
  • 3 edits in trunk/WebCore

2009-04-21 Laszlo Gombos <Laszlo Gombos>

Reviewed by Sam Weinig.

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

Fix the build with disabled DOM storage or disabled database support.

  • bindings/js/ScriptObjectQuarantine.cpp:
  • bindings/js/ScriptObjectQuarantine.h:
3:03 AM QtWebKitTodo edited by kenneth.christiansen@gmail.com
(diff)
2:27 AM Changeset in webkit [42708] by zecke@webkit.org
  • 2 edits in trunk/WebCore

Compile fix after r42633. const on a static method does not work

1:33 AM QtWebKitTodo edited by kenneth.christiansen@gmail.com
(diff)
12:52 AM Changeset in webkit [42707] by mitz@apple.com
  • 5 edits in trunk

Reviewed by Darin Adler and John Sullivan.

  • fix <rdar://problem/5914146> Crash at -[NSScrollView scrollWheel:]

-[NSScrollView scrollWheel:] runs a nested event-tracking run loop
in a mode that allows WebCore timers to fire and NSURLConnection
callbacks to be dispatched, which can release the NSScrollView and
cause it to be deallocated (one example is a DOM timer callback that
removes a subframe from the document). This leads to a crash in
-scrollView:.

The fix is to replace the implementation of -[NSScrollView scrollWheel:]
with one that optionally retains the receiver for the duration of the
call.

  • page/mac/EventHandlerMac.mm: (WebCore::nsScrollViewScrollWheelShouldRetainSelf): Added. (WebCore::setNSScrollViewScrollWheelShouldRetainSelf): Added. Replaces the implementation of -[NSScrollView scrollWheel:] with the self-retaining one, and sets a static boolean that tells it whether to retain the receiver around the call to the original implementation. (WebCore::selfRetainingNSScrollViewScrollWheel): Added. If setNSScrollViewScrollWheelShouldRetainSelf(true) was called and this function is executing on the main thread, it retains the NSScrollView, invokes the original -[NSScrollView scrollWheel:], then releases the view. Otherwise it just calls through to the original implementation. (WebCore::EventHandler::passWheelEventToWidget): Added calls to setNSScrollViewScrollWheelShouldRetainSelf() around the call to -scrollWheel:.
12:48 AM Changeset in webkit [42706] by sfalken@apple.com
  • 2 edits in trunk/WebKitTools

Update FindSafari for JavaScriptCore DLL changes.
Also removed code previously needed to support running nightlies against Safari 3 on Windows.


Reviewed by Mark Rowe.

  • FindSafari/FindSafari.cpp: (getStringValue): (getInstalledWebKitDirectory): (_tmain):
12:23 AM Changeset in webkit [42705] by ggaren@apple.com
  • 9 edits
    3 copies
    1 add in trunk

JavaScriptCore:

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Mark Rowe.


Tiger crash fix: Put VM tags in their own header file, and fixed up the
#ifdefs so they're not used on Tiger.

  • JavaScriptCore.xcodeproj/project.pbxproj:
  • interpreter/RegisterFile.h: (JSC::RegisterFile::RegisterFile):
  • jit/ExecutableAllocatorFixedVMPool.cpp: (JSC::FixedVMPoolAllocator::FixedVMPoolAllocator):
  • jit/ExecutableAllocatorPosix.cpp: (JSC::ExecutablePool::systemAlloc):
  • runtime/Collector.cpp: (JSC::allocateBlock):
  • wtf/VMTags.h: Added.

JavaScriptGlue:

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Mark Rowe.

Tiger crash fix: Put VM tags in their own header file, and fixed up the
#ifdefs so they're not used on Tiger.

  • ForwardingHeaders/wtf/VMTags.h: Copied from JavaScriptGlue/ForwardingHeaders/wtf/HashTraits.h.

WebCore:

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Mark Rowe.

Tiger crash fix: Put VM tags in their own header file, and fixed up the
#ifdefs so they're not used on Tiger.

  • ForwardingHeaders/wtf/VMTags.h: Copied from WebCore/ForwardingHeaders/wtf/HashTraits.h.

WebKit/mac:

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Mark Rowe.

Tiger crash fix: Put VM tags in their own header file, and fixed up the
#ifdefs so they're not used on Tiger.

  • ForwardingHeaders/wtf/VMTags.h: Copied from ForwardingHeaders/wtf/HashTraits.h.
12:02 AM Changeset in webkit [42704] by xan@webkit.org
  • 2 edits in trunk/WebCore

2009-04-21 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=21546
[GTK] ATK accessibility enhancements

Based on a patch by Alp Toker.

Implement AtkComponent interface.

  • page/gtk/AccessibilityObjectWrapperAtk.cpp: (core):
12:01 AM Changeset in webkit [42703] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-21 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=21546
[GTK] ATK accessibility enhancements

Implement window<->screen coordinate transformation functions.

  • WebCoreSupport/ChromeClientGtk.cpp: (WebKit::widgetScreenPosition): (WebKit::ChromeClient::windowToScreen): (WebKit::ChromeClient::screenToWindow):
12:01 AM Changeset in webkit [42702] by xan@webkit.org
  • 2 edits in trunk/WebCore

2009-04-21 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=21546
[GTK] ATK accessibility enhancements

Only implement AtkAction interface if we actually have an action.

  • page/gtk/AccessibilityObjectWrapperAtk.cpp:

Apr 20, 2009:

11:41 PM Changeset in webkit [42701] by sfalken@apple.com
  • 1 edit in trunk/JavaScriptCore/JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj

Windows build fix

11:34 PM Changeset in webkit [42700] by ap@webkit.org
  • 13 edits
    3 adds in trunk

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=25290
REGRESSION(r41732): Crash when constructing XMLHttpRequest in a detached document

Test: fast/dom/xmlhttprequest-constructor-in-detached-document.html

  • bindings/js/JSAudioConstructor.cpp: (WebCore::JSAudioConstructor::JSAudioConstructor):
  • bindings/js/JSAudioConstructor.h:
  • bindings/js/JSDOMGlobalObject.h: (WebCore::getDOMConstructor):
  • bindings/js/JSImageConstructor.cpp: (WebCore::JSImageConstructor::JSImageConstructor):
  • bindings/js/JSImageConstructor.h:
  • bindings/js/JSMessageChannelConstructor.cpp: (WebCore::JSMessageChannelConstructor::JSMessageChannelConstructor):
  • bindings/js/JSMessageChannelConstructor.h:
  • bindings/js/JSOptionConstructor.cpp: (WebCore::JSOptionConstructor::JSOptionConstructor):
  • bindings/js/JSOptionConstructor.h:
  • bindings/js/JSXMLHttpRequestConstructor.cpp: (WebCore::JSXMLHttpRequestConstructor::JSXMLHttpRequestConstructor):
  • bindings/js/JSXMLHttpRequestConstructor.h: Avoid accessing JSDOMGlobalObject via ScriptExecutionContext, since it may not work during frame teardown.
10:58 PM Changeset in webkit [42699] by ggaren@apple.com
  • 32 edits in trunk/WebCore

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Rubber stamped by Darin Adler and Sam Weinig.

Renamed "*InlineEventListener*" => "*AttributeEventListener*", and
"isInline" => "isAttribute".

  • bindings/js/JSDOMGlobalObject.cpp: (WebCore::JSDOMGlobalObject::createJSAttributeEventListener):
  • bindings/js/JSEventListener.cpp: (WebCore::JSEventListener::JSEventListener): (WebCore::JSEventListener::~JSEventListener): (WebCore::JSEventListener::handleEvent): (WebCore::JSEventListener::virtualisAttribute):
  • bindings/js/JSEventListener.h: (WebCore::JSEventListener::create): (WebCore::JSEventListener::isAttribute):
  • bindings/js/JSLazyEventListener.cpp: (WebCore::createAttributeEventListener):
  • bindings/js/JSLazyEventListener.h:
  • bindings/scripts/CodeGeneratorJS.pm:
  • dom/Element.cpp: (WebCore::Element::setWindowAttributeEventListener):
  • dom/Element.h:
  • dom/EventListener.h: (WebCore::EventListener::isAttribute): (WebCore::EventListener::virtualisAttribute):
  • dom/Node.cpp: (WebCore::Node::clearAttributeEventListener): (WebCore::Node::setAttributeEventListener): (WebCore::Node::getAttributeEventListener): (WebCore::Node::onabort): (WebCore::Node::setOnabort): (WebCore::Node::onblur): (WebCore::Node::setOnblur): (WebCore::Node::onchange): (WebCore::Node::setOnchange): (WebCore::Node::onclick): (WebCore::Node::setOnclick): (WebCore::Node::oncontextmenu): (WebCore::Node::setOncontextmenu): (WebCore::Node::ondblclick): (WebCore::Node::setOndblclick): (WebCore::Node::onerror): (WebCore::Node::setOnerror): (WebCore::Node::onfocus): (WebCore::Node::setOnfocus): (WebCore::Node::oninput): (WebCore::Node::setOninput): (WebCore::Node::onkeydown): (WebCore::Node::setOnkeydown): (WebCore::Node::onkeypress): (WebCore::Node::setOnkeypress): (WebCore::Node::onkeyup): (WebCore::Node::setOnkeyup): (WebCore::Node::onload): (WebCore::Node::setOnload): (WebCore::Node::onmousedown): (WebCore::Node::setOnmousedown): (WebCore::Node::onmousemove): (WebCore::Node::setOnmousemove): (WebCore::Node::onmouseout): (WebCore::Node::setOnmouseout): (WebCore::Node::onmouseover): (WebCore::Node::setOnmouseover): (WebCore::Node::onmouseup): (WebCore::Node::setOnmouseup): (WebCore::Node::onmousewheel): (WebCore::Node::setOnmousewheel): (WebCore::Node::onbeforecut): (WebCore::Node::setOnbeforecut): (WebCore::Node::oncut): (WebCore::Node::setOncut): (WebCore::Node::onbeforecopy): (WebCore::Node::setOnbeforecopy): (WebCore::Node::oncopy): (WebCore::Node::setOncopy): (WebCore::Node::onbeforepaste): (WebCore::Node::setOnbeforepaste): (WebCore::Node::onpaste): (WebCore::Node::setOnpaste): (WebCore::Node::ondragenter): (WebCore::Node::setOndragenter): (WebCore::Node::ondragover): (WebCore::Node::setOndragover): (WebCore::Node::ondragleave): (WebCore::Node::setOndragleave): (WebCore::Node::ondrop): (WebCore::Node::setOndrop): (WebCore::Node::ondragstart): (WebCore::Node::setOndragstart): (WebCore::Node::ondrag): (WebCore::Node::setOndrag): (WebCore::Node::ondragend): (WebCore::Node::setOndragend): (WebCore::Node::onreset): (WebCore::Node::setOnreset): (WebCore::Node::onresize): (WebCore::Node::setOnresize): (WebCore::Node::onscroll): (WebCore::Node::setOnscroll): (WebCore::Node::onsearch): (WebCore::Node::setOnsearch): (WebCore::Node::onselect): (WebCore::Node::setOnselect): (WebCore::Node::onselectstart): (WebCore::Node::setOnselectstart): (WebCore::Node::onsubmit): (WebCore::Node::setOnsubmit): (WebCore::Node::onunload): (WebCore::Node::setOnunload):
  • dom/Node.h:
  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplacementFragment::ReplacementFragment):
  • html/HTMLBodyElement.cpp: (WebCore::HTMLBodyElement::parseMappedAttribute):
  • html/HTMLButtonElement.cpp: (WebCore::HTMLButtonElement::parseMappedAttribute):
  • html/HTMLElement.cpp: (WebCore::HTMLElement::parseMappedAttribute):
  • html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::parseMappedAttribute):
  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::parseMappedAttribute):
  • html/HTMLFrameSetElement.cpp: (WebCore::HTMLFrameSetElement::parseMappedAttribute):
  • html/HTMLImageElement.cpp: (WebCore::HTMLImageElement::parseMappedAttribute):
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::parseMappedAttribute):
  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::parseMappedAttribute):
  • html/HTMLScriptElement.cpp: (WebCore::HTMLScriptElement::parseMappedAttribute):
  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::parseMappedAttribute):
  • html/HTMLTextAreaElement.cpp: (WebCore::HTMLTextAreaElement::parseMappedAttribute):
  • page/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::mouseButtonListener):
  • page/DOMWindow.cpp: (WebCore::DOMWindow::setAttributeEventListener): (WebCore::DOMWindow::clearAttributeEventListener): (WebCore::DOMWindow::getAttributeEventListener): (WebCore::DOMWindow::onabort): (WebCore::DOMWindow::setOnabort): (WebCore::DOMWindow::onblur): (WebCore::DOMWindow::setOnblur): (WebCore::DOMWindow::onchange): (WebCore::DOMWindow::setOnchange): (WebCore::DOMWindow::onclick): (WebCore::DOMWindow::setOnclick): (WebCore::DOMWindow::ondblclick): (WebCore::DOMWindow::setOndblclick): (WebCore::DOMWindow::onerror): (WebCore::DOMWindow::setOnerror): (WebCore::DOMWindow::onfocus): (WebCore::DOMWindow::setOnfocus): (WebCore::DOMWindow::onkeydown): (WebCore::DOMWindow::setOnkeydown): (WebCore::DOMWindow::onkeypress): (WebCore::DOMWindow::setOnkeypress): (WebCore::DOMWindow::onkeyup): (WebCore::DOMWindow::setOnkeyup): (WebCore::DOMWindow::onload): (WebCore::DOMWindow::setOnload): (WebCore::DOMWindow::onmousedown): (WebCore::DOMWindow::setOnmousedown): (WebCore::DOMWindow::onmousemove): (WebCore::DOMWindow::setOnmousemove): (WebCore::DOMWindow::onmouseout): (WebCore::DOMWindow::setOnmouseout): (WebCore::DOMWindow::onmouseover): (WebCore::DOMWindow::setOnmouseover): (WebCore::DOMWindow::onmouseup): (WebCore::DOMWindow::setOnmouseup): (WebCore::DOMWindow::onmousewheel): (WebCore::DOMWindow::setOnmousewheel): (WebCore::DOMWindow::onreset): (WebCore::DOMWindow::setOnreset): (WebCore::DOMWindow::onresize): (WebCore::DOMWindow::setOnresize): (WebCore::DOMWindow::onscroll): (WebCore::DOMWindow::setOnscroll): (WebCore::DOMWindow::onsearch): (WebCore::DOMWindow::setOnsearch): (WebCore::DOMWindow::onselect): (WebCore::DOMWindow::setOnselect): (WebCore::DOMWindow::onsubmit): (WebCore::DOMWindow::setOnsubmit): (WebCore::DOMWindow::onunload): (WebCore::DOMWindow::setOnunload): (WebCore::DOMWindow::onbeforeunload): (WebCore::DOMWindow::setOnbeforeunload): (WebCore::DOMWindow::onwebkitanimationstart): (WebCore::DOMWindow::setOnwebkitanimationstart): (WebCore::DOMWindow::onwebkitanimationiteration): (WebCore::DOMWindow::setOnwebkitanimationiteration): (WebCore::DOMWindow::onwebkitanimationend): (WebCore::DOMWindow::setOnwebkitanimationend): (WebCore::DOMWindow::onwebkittransitionend): (WebCore::DOMWindow::setOnwebkittransitionend):
  • page/DOMWindow.h:
  • svg/SVGElement.cpp: (WebCore::SVGElement::parseMappedAttribute):
  • svg/SVGElementInstance.cpp: (WebCore::SVGElementInstance::onabort): (WebCore::SVGElementInstance::setOnabort): (WebCore::SVGElementInstance::onblur): (WebCore::SVGElementInstance::setOnblur): (WebCore::SVGElementInstance::onchange): (WebCore::SVGElementInstance::setOnchange): (WebCore::SVGElementInstance::onclick): (WebCore::SVGElementInstance::setOnclick): (WebCore::SVGElementInstance::oncontextmenu): (WebCore::SVGElementInstance::setOncontextmenu): (WebCore::SVGElementInstance::ondblclick): (WebCore::SVGElementInstance::setOndblclick): (WebCore::SVGElementInstance::onerror): (WebCore::SVGElementInstance::setOnerror): (WebCore::SVGElementInstance::onfocus): (WebCore::SVGElementInstance::setOnfocus): (WebCore::SVGElementInstance::oninput): (WebCore::SVGElementInstance::setOninput): (WebCore::SVGElementInstance::onkeydown): (WebCore::SVGElementInstance::setOnkeydown): (WebCore::SVGElementInstance::onkeypress): (WebCore::SVGElementInstance::setOnkeypress): (WebCore::SVGElementInstance::onkeyup): (WebCore::SVGElementInstance::setOnkeyup): (WebCore::SVGElementInstance::onload): (WebCore::SVGElementInstance::setOnload): (WebCore::SVGElementInstance::onmousedown): (WebCore::SVGElementInstance::setOnmousedown): (WebCore::SVGElementInstance::onmousemove): (WebCore::SVGElementInstance::setOnmousemove): (WebCore::SVGElementInstance::onmouseout): (WebCore::SVGElementInstance::setOnmouseout): (WebCore::SVGElementInstance::onmouseover): (WebCore::SVGElementInstance::setOnmouseover): (WebCore::SVGElementInstance::onmouseup): (WebCore::SVGElementInstance::setOnmouseup): (WebCore::SVGElementInstance::onmousewheel): (WebCore::SVGElementInstance::setOnmousewheel): (WebCore::SVGElementInstance::onbeforecut): (WebCore::SVGElementInstance::setOnbeforecut): (WebCore::SVGElementInstance::oncut): (WebCore::SVGElementInstance::setOncut): (WebCore::SVGElementInstance::onbeforecopy): (WebCore::SVGElementInstance::setOnbeforecopy): (WebCore::SVGElementInstance::oncopy): (WebCore::SVGElementInstance::setOncopy): (WebCore::SVGElementInstance::onbeforepaste): (WebCore::SVGElementInstance::setOnbeforepaste): (WebCore::SVGElementInstance::onpaste): (WebCore::SVGElementInstance::setOnpaste): (WebCore::SVGElementInstance::ondragenter): (WebCore::SVGElementInstance::setOndragenter): (WebCore::SVGElementInstance::ondragover): (WebCore::SVGElementInstance::setOndragover): (WebCore::SVGElementInstance::ondragleave): (WebCore::SVGElementInstance::setOndragleave): (WebCore::SVGElementInstance::ondrop): (WebCore::SVGElementInstance::setOndrop): (WebCore::SVGElementInstance::ondragstart): (WebCore::SVGElementInstance::setOndragstart): (WebCore::SVGElementInstance::ondrag): (WebCore::SVGElementInstance::setOndrag): (WebCore::SVGElementInstance::ondragend): (WebCore::SVGElementInstance::setOndragend): (WebCore::SVGElementInstance::onreset): (WebCore::SVGElementInstance::setOnreset): (WebCore::SVGElementInstance::onresize): (WebCore::SVGElementInstance::setOnresize): (WebCore::SVGElementInstance::onscroll): (WebCore::SVGElementInstance::setOnscroll): (WebCore::SVGElementInstance::onsearch): (WebCore::SVGElementInstance::setOnsearch): (WebCore::SVGElementInstance::onselect): (WebCore::SVGElementInstance::setOnselect): (WebCore::SVGElementInstance::onselectstart): (WebCore::SVGElementInstance::setOnselectstart): (WebCore::SVGElementInstance::onsubmit): (WebCore::SVGElementInstance::setOnsubmit): (WebCore::SVGElementInstance::onunload): (WebCore::SVGElementInstance::setOnunload):
  • svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::parseMappedAttribute):
10:17 PM Changeset in webkit [42698] by ggaren@apple.com
  • 549 edits
    8 deletes in trunk

WebCore:

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Approved by Mark Rowe, Cameron Zwarich, Oliver Hunt, and Ojan Vafai.


Used svn merge to roll out revisions 42678, 42690, 42694, 42697 because
they broke the Tiger and Windows buildbots.

  • css/html4.css:
  • css/themeWin.css:
  • platform/graphics/SimpleFontData.cpp:
  • platform/graphics/SimpleFontData.h:
  • platform/graphics/chromium/SimpleFontDataChromiumWin.cpp:
  • platform/graphics/chromium/SimpleFontDataLinux.cpp:
  • platform/graphics/gtk/SimpleFontDataGtk.cpp:
  • platform/graphics/gtk/SimpleFontDataPango.cpp:
  • platform/graphics/mac/SimpleFontDataMac.mm:
  • platform/graphics/qt/SimpleFontDataQt.cpp:
  • platform/graphics/win/SimpleFontDataCGWin.cpp:
  • platform/graphics/win/SimpleFontDataCairoWin.cpp:
  • platform/graphics/win/SimpleFontDataWin.cpp:
  • platform/graphics/wx/SimpleFontDataWx.cpp:
  • rendering/RenderTextControl.cpp:
  • rendering/RenderTextControlMultiLine.cpp:
  • rendering/RenderTextControlSingleLine.cpp:

LayoutTests:

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Approved by Mark Rowe, Cameron Zwarich, Oliver Hunt, and Ojan Vafai.


Used svn merge to roll out revisions 42678, 42690, 42694, 42697 because
they broke the Tiger and Windows buildbots.

  • fast/forms/drag-out-of-textarea.html:
  • fast/forms/resources/TEMPLATE.html: Removed.
  • fast/forms/resources/text-control-intrinsic-widths.js: Removed.
  • fast/forms/resources/textarea-metrics.js: Removed.
  • fast/forms/search-abs-pos-cancel-button.html:
  • fast/forms/text-control-intrinsic-widths-expected.txt: Removed.
  • fast/forms/text-control-intrinsic-widths.html: Removed.
  • fast/forms/textarea-metrics-expected.txt: Removed.
  • fast/forms/textarea-metrics.html: Removed.
  • fast/forms/textarea-width.html:
  • fast/replaced/table-percent-height-expected.txt:
  • fast/replaced/table-percent-height.html:
  • platform/mac-leopard/fast/forms/input-text-maxlength-expected.checksum:
  • platform/mac-leopard/fast/forms/input-text-maxlength-expected.png:
  • platform/mac-leopard/fast/forms/input-text-maxlength-expected.txt:
  • platform/mac-leopard/fast/forms/input-text-paste-maxlength-expected.checksum:
  • platform/mac-leopard/fast/forms/input-text-paste-maxlength-expected.png:
  • platform/mac-leopard/fast/forms/input-text-paste-maxlength-expected.txt:
  • platform/mac/editing/deleting/5168598-expected.checksum:
  • platform/mac/editing/deleting/5168598-expected.png:
  • platform/mac/editing/deleting/5168598-expected.txt:
  • platform/mac/editing/inserting/4960120-1-expected.checksum:
  • platform/mac/editing/inserting/4960120-1-expected.png:
  • platform/mac/editing/inserting/4960120-1-expected.txt:
  • platform/mac/editing/inserting/5607069-2-expected.checksum:
  • platform/mac/editing/inserting/5607069-2-expected.png:
  • platform/mac/editing/inserting/5607069-2-expected.txt:
  • platform/mac/editing/inserting/5607069-3-expected.checksum:
  • platform/mac/editing/inserting/5607069-3-expected.png:
  • platform/mac/editing/inserting/5607069-3-expected.txt:
  • platform/mac/editing/inserting/before-after-input-element-expected.checksum:
  • platform/mac/editing/inserting/before-after-input-element-expected.png:
  • platform/mac/editing/inserting/before-after-input-element-expected.txt:
  • platform/mac/editing/inserting/typing-tab-designmode-forms-expected.checksum:
  • platform/mac/editing/inserting/typing-tab-designmode-forms-expected.png:
  • platform/mac/editing/inserting/typing-tab-designmode-forms-expected.txt:
  • platform/mac/editing/pasteboard/4806874-expected.checksum:
  • platform/mac/editing/pasteboard/4806874-expected.png:
  • platform/mac/editing/pasteboard/4806874-expected.txt:
  • platform/mac/editing/pasteboard/drop-text-without-selection-expected.checksum:
  • platform/mac/editing/pasteboard/drop-text-without-selection-expected.png:
  • platform/mac/editing/pasteboard/drop-text-without-selection-expected.txt:
  • platform/mac/editing/pasteboard/input-field-1-expected.checksum:
  • platform/mac/editing/pasteboard/input-field-1-expected.png:
  • platform/mac/editing/pasteboard/input-field-1-expected.txt:
  • platform/mac/editing/pasteboard/nested-blocks-with-text-area-expected.checksum:
  • platform/mac/editing/pasteboard/nested-blocks-with-text-area-expected.png:
  • platform/mac/editing/pasteboard/nested-blocks-with-text-area-expected.txt:
  • platform/mac/editing/pasteboard/nested-blocks-with-text-field-expected.checksum:
  • platform/mac/editing/pasteboard/nested-blocks-with-text-field-expected.png:
  • platform/mac/editing/pasteboard/nested-blocks-with-text-field-expected.txt:
  • platform/mac/editing/pasteboard/pasting-tabs-expected.checksum:
  • platform/mac/editing/pasteboard/pasting-tabs-expected.png:
  • platform/mac/editing/pasteboard/pasting-tabs-expected.txt:
  • platform/mac/editing/selection/3690703-2-expected.checksum:
  • platform/mac/editing/selection/3690703-2-expected.png:
  • platform/mac/editing/selection/3690703-2-expected.txt:
  • platform/mac/editing/selection/3690703-expected.checksum:
  • platform/mac/editing/selection/3690703-expected.png:
  • platform/mac/editing/selection/3690703-expected.txt:
  • platform/mac/editing/selection/3690719-expected.checksum:
  • platform/mac/editing/selection/3690719-expected.png:
  • platform/mac/editing/selection/3690719-expected.txt:
  • platform/mac/editing/selection/4895428-3-expected.checksum:
  • platform/mac/editing/selection/4895428-3-expected.png:
  • platform/mac/editing/selection/4895428-3-expected.txt:
  • platform/mac/editing/selection/4975120-expected.checksum:
  • platform/mac/editing/selection/4975120-expected.png:
  • platform/mac/editing/selection/4975120-expected.txt:
  • platform/mac/editing/selection/5136696-expected.checksum:
  • platform/mac/editing/selection/5136696-expected.png:
  • platform/mac/editing/selection/5136696-expected.txt:
  • platform/mac/editing/selection/5213963-expected.checksum:
  • platform/mac/editing/selection/5213963-expected.png:
  • platform/mac/editing/selection/5213963-expected.txt:
  • platform/mac/editing/selection/drag-select-1-expected.checksum:
  • platform/mac/editing/selection/drag-select-1-expected.png:
  • platform/mac/editing/selection/drag-select-1-expected.txt:
  • platform/mac/editing/selection/drag-text-delay-expected.checksum:
  • platform/mac/editing/selection/drag-text-delay-expected.png:
  • platform/mac/editing/selection/drag-text-delay-expected.txt:
  • platform/mac/editing/selection/select-from-textfield-outwards-expected.checksum:
  • platform/mac/editing/selection/select-from-textfield-outwards-expected.png:
  • platform/mac/editing/selection/select-from-textfield-outwards-expected.txt:
  • platform/mac/fast/block/margin-collapse/103-expected.txt:
  • platform/mac/fast/css/line-height-expected.checksum:
  • platform/mac/fast/css/line-height-expected.png:
  • platform/mac/fast/css/line-height-expected.txt:
  • platform/mac/fast/css/resize-corner-tracking-expected.checksum:
  • platform/mac/fast/css/resize-corner-tracking-expected.png:
  • platform/mac/fast/css/resize-corner-tracking-expected.txt:
  • platform/mac/fast/dom/HTMLTextAreaElement/reset-textarea-expected.checksum:
  • platform/mac/fast/dom/HTMLTextAreaElement/reset-textarea-expected.png:
  • platform/mac/fast/dom/HTMLTextAreaElement/reset-textarea-expected.txt:
  • platform/mac/fast/dom/isindex-002-expected.checksum:
  • platform/mac/fast/dom/isindex-002-expected.png:
  • platform/mac/fast/dom/isindex-002-expected.txt:
  • platform/mac/fast/dynamic/008-expected.checksum:
  • platform/mac/fast/dynamic/008-expected.png:
  • platform/mac/fast/dynamic/008-expected.txt:
  • platform/mac/fast/events/autoscroll-expected.checksum:
  • platform/mac/fast/events/autoscroll-expected.png:
  • platform/mac/fast/events/autoscroll-expected.txt:
  • platform/mac/fast/events/context-no-deselect-expected.checksum:
  • platform/mac/fast/events/context-no-deselect-expected.png:
  • platform/mac/fast/events/context-no-deselect-expected.txt:
  • platform/mac/fast/events/label-focus-expected.checksum:
  • platform/mac/fast/events/label-focus-expected.png:
  • platform/mac/fast/events/label-focus-expected.txt:
  • platform/mac/fast/forms/basic-textareas-expected.checksum:
  • platform/mac/fast/forms/basic-textareas-expected.png:
  • platform/mac/fast/forms/basic-textareas-expected.txt:
  • platform/mac/fast/forms/box-shadow-override-expected.checksum:
  • platform/mac/fast/forms/box-shadow-override-expected.png:
  • platform/mac/fast/forms/box-shadow-override-expected.txt:
  • platform/mac/fast/forms/control-restrict-line-height-expected.checksum:
  • platform/mac/fast/forms/control-restrict-line-height-expected.png:
  • platform/mac/fast/forms/control-restrict-line-height-expected.txt:
  • platform/mac/fast/forms/encoding-test-expected.checksum:
  • platform/mac/fast/forms/encoding-test-expected.png:
  • platform/mac/fast/forms/encoding-test-expected.txt:
  • platform/mac/fast/forms/fieldset-align-expected.checksum:
  • platform/mac/fast/forms/fieldset-align-expected.png:
  • platform/mac/fast/forms/fieldset-align-expected.txt:
  • platform/mac/fast/forms/form-element-geometry-expected.checksum:
  • platform/mac/fast/forms/form-element-geometry-expected.png:
  • platform/mac/fast/forms/form-element-geometry-expected.txt:
  • platform/mac/fast/forms/input-align-expected.checksum:
  • platform/mac/fast/forms/input-align-expected.png:
  • platform/mac/fast/forms/input-align-expected.txt:
  • platform/mac/fast/forms/input-appearance-bkcolor-expected.checksum:
  • platform/mac/fast/forms/input-appearance-bkcolor-expected.png:
  • platform/mac/fast/forms/input-appearance-bkcolor-expected.txt:
  • platform/mac/fast/forms/input-appearance-default-bkcolor-expected.checksum:
  • platform/mac/fast/forms/input-appearance-default-bkcolor-expected.png:
  • platform/mac/fast/forms/input-appearance-default-bkcolor-expected.txt:
  • platform/mac/fast/forms/input-appearance-disabled-expected.checksum:
  • platform/mac/fast/forms/input-appearance-disabled-expected.png:
  • platform/mac/fast/forms/input-appearance-disabled-expected.txt:
  • platform/mac/fast/forms/input-appearance-focus-expected.checksum:
  • platform/mac/fast/forms/input-appearance-focus-expected.png:
  • platform/mac/fast/forms/input-appearance-focus-expected.txt:
  • platform/mac/fast/forms/input-appearance-height-expected.checksum:
  • platform/mac/fast/forms/input-appearance-height-expected.png:
  • platform/mac/fast/forms/input-appearance-height-expected.txt:
  • platform/mac/fast/forms/input-appearance-preventDefault-expected.checksum:
  • platform/mac/fast/forms/input-appearance-preventDefault-expected.png:
  • platform/mac/fast/forms/input-appearance-preventDefault-expected.txt:
  • platform/mac/fast/forms/input-appearance-readonly-expected.checksum:
  • platform/mac/fast/forms/input-appearance-readonly-expected.png:
  • platform/mac/fast/forms/input-appearance-readonly-expected.txt:
  • platform/mac/fast/forms/input-appearance-selection-expected.checksum:
  • platform/mac/fast/forms/input-appearance-selection-expected.png:
  • platform/mac/fast/forms/input-appearance-selection-expected.txt:
  • platform/mac/fast/forms/input-appearance-visibility-expected.checksum:
  • platform/mac/fast/forms/input-appearance-visibility-expected.png:
  • platform/mac/fast/forms/input-appearance-visibility-expected.txt:
  • platform/mac/fast/forms/input-appearance-width-expected.checksum:
  • platform/mac/fast/forms/input-appearance-width-expected.png:
  • platform/mac/fast/forms/input-appearance-width-expected.txt:
  • platform/mac/fast/forms/input-baseline-expected.checksum:
  • platform/mac/fast/forms/input-baseline-expected.png:
  • platform/mac/fast/forms/input-baseline-expected.txt:
  • platform/mac/fast/forms/input-disabled-color-expected.checksum:
  • platform/mac/fast/forms/input-disabled-color-expected.png:
  • platform/mac/fast/forms/input-disabled-color-expected.txt:
  • platform/mac/fast/forms/input-double-click-selection-gap-bug-expected.checksum:
  • platform/mac/fast/forms/input-double-click-selection-gap-bug-expected.png:
  • platform/mac/fast/forms/input-double-click-selection-gap-bug-expected.txt:
  • platform/mac/fast/forms/input-paste-undo-expected.checksum:
  • platform/mac/fast/forms/input-paste-undo-expected.png:
  • platform/mac/fast/forms/input-paste-undo-expected.txt:
  • platform/mac/fast/forms/input-readonly-autoscroll-expected.checksum:
  • platform/mac/fast/forms/input-readonly-autoscroll-expected.png:
  • platform/mac/fast/forms/input-readonly-autoscroll-expected.txt:
  • platform/mac/fast/forms/input-readonly-dimmed-expected.checksum:
  • platform/mac/fast/forms/input-readonly-dimmed-expected.png:
  • platform/mac/fast/forms/input-readonly-dimmed-expected.txt:
  • platform/mac/fast/forms/input-readonly-empty-expected.checksum:
  • platform/mac/fast/forms/input-readonly-empty-expected.png:
  • platform/mac/fast/forms/input-readonly-empty-expected.txt:
  • platform/mac/fast/forms/input-spaces-expected.checksum:
  • platform/mac/fast/forms/input-spaces-expected.png:
  • platform/mac/fast/forms/input-spaces-expected.txt:
  • platform/mac/fast/forms/input-table-expected.txt:
  • platform/mac/fast/forms/input-text-click-inside-expected.checksum:
  • platform/mac/fast/forms/input-text-click-inside-expected.png:
  • platform/mac/fast/forms/input-text-click-inside-expected.txt:
  • platform/mac/fast/forms/input-text-click-outside-expected.checksum:
  • platform/mac/fast/forms/input-text-click-outside-expected.png:
  • platform/mac/fast/forms/input-text-click-outside-expected.txt:
  • platform/mac/fast/forms/input-text-double-click-expected.checksum:
  • platform/mac/fast/forms/input-text-double-click-expected.png:
  • platform/mac/fast/forms/input-text-double-click-expected.txt:
  • platform/mac/fast/forms/input-text-drag-down-expected.checksum:
  • platform/mac/fast/forms/input-text-drag-down-expected.png:
  • platform/mac/fast/forms/input-text-drag-down-expected.txt:
  • platform/mac/fast/forms/input-text-option-delete-expected.checksum:
  • platform/mac/fast/forms/input-text-option-delete-expected.png:
  • platform/mac/fast/forms/input-text-option-delete-expected.txt:
  • platform/mac/fast/forms/input-text-scroll-left-on-blur-expected.checksum:
  • platform/mac/fast/forms/input-text-scroll-left-on-blur-expected.png:
  • platform/mac/fast/forms/input-text-scroll-left-on-blur-expected.txt:
  • platform/mac/fast/forms/input-text-self-emptying-click-expected.checksum:
  • platform/mac/fast/forms/input-text-self-emptying-click-expected.png:
  • platform/mac/fast/forms/input-text-self-emptying-click-expected.txt:
  • platform/mac/fast/forms/input-text-word-wrap-expected.checksum:
  • platform/mac/fast/forms/input-text-word-wrap-expected.png:
  • platform/mac/fast/forms/input-text-word-wrap-expected.txt:
  • platform/mac/fast/forms/input-type-change2-expected.checksum:
  • platform/mac/fast/forms/input-type-change2-expected.png:
  • platform/mac/fast/forms/input-type-change2-expected.txt:
  • platform/mac/fast/forms/input-type-text-min-width-expected.checksum:
  • platform/mac/fast/forms/input-type-text-min-width-expected.png:
  • platform/mac/fast/forms/input-type-text-min-width-expected.txt:
  • platform/mac/fast/forms/input-value-expected.checksum:
  • platform/mac/fast/forms/input-value-expected.png:
  • platform/mac/fast/forms/input-value-expected.txt:
  • platform/mac/fast/forms/input-width-expected.checksum:
  • platform/mac/fast/forms/input-width-expected.png:
  • platform/mac/fast/forms/input-width-expected.txt:
  • platform/mac/fast/forms/linebox-overflow-in-textarea-padding-expected.checksum:
  • platform/mac/fast/forms/linebox-overflow-in-textarea-padding-expected.png:
  • platform/mac/fast/forms/linebox-overflow-in-textarea-padding-expected.txt:
  • platform/mac/fast/forms/negativeLineHeight-expected.checksum:
  • platform/mac/fast/forms/negativeLineHeight-expected.png:
  • platform/mac/fast/forms/negativeLineHeight-expected.txt:
  • platform/mac/fast/forms/password-placeholder-expected.checksum:
  • platform/mac/fast/forms/password-placeholder-expected.png:
  • platform/mac/fast/forms/password-placeholder-expected.txt:
  • platform/mac/fast/forms/password-placeholder-text-security-expected.checksum:
  • platform/mac/fast/forms/password-placeholder-text-security-expected.png:
  • platform/mac/fast/forms/password-placeholder-text-security-expected.txt:
  • platform/mac/fast/forms/placeholder-pseudo-style-expected.checksum:
  • platform/mac/fast/forms/placeholder-pseudo-style-expected.png:
  • platform/mac/fast/forms/placeholder-pseudo-style-expected.txt:
  • platform/mac/fast/forms/placeholder-set-attribute-expected.checksum:
  • platform/mac/fast/forms/placeholder-set-attribute-expected.png:
  • platform/mac/fast/forms/placeholder-set-attribute-expected.txt:
  • platform/mac/fast/forms/placeholder-set-value-expected.checksum:
  • platform/mac/fast/forms/placeholder-set-value-expected.png:
  • platform/mac/fast/forms/placeholder-set-value-expected.txt:
  • platform/mac/fast/forms/search-cancel-button-style-sharing-expected.checksum:
  • platform/mac/fast/forms/search-cancel-button-style-sharing-expected.png:
  • platform/mac/fast/forms/search-cancel-button-style-sharing-expected.txt:
  • platform/mac/fast/forms/search-display-none-cancel-button-expected.checksum:
  • platform/mac/fast/forms/search-display-none-cancel-button-expected.png:
  • platform/mac/fast/forms/search-display-none-cancel-button-expected.txt:
  • platform/mac/fast/forms/search-placeholder-value-changed-expected.checksum:
  • platform/mac/fast/forms/search-placeholder-value-changed-expected.png:
  • platform/mac/fast/forms/search-placeholder-value-changed-expected.txt:
  • platform/mac/fast/forms/search-rtl-expected.checksum:
  • platform/mac/fast/forms/search-rtl-expected.png:
  • platform/mac/fast/forms/search-rtl-expected.txt:
  • platform/mac/fast/forms/search-transformed-expected.checksum:
  • platform/mac/fast/forms/search-transformed-expected.png:
  • platform/mac/fast/forms/search-transformed-expected.txt:
  • platform/mac/fast/forms/search-zoomed-expected.checksum:
  • platform/mac/fast/forms/search-zoomed-expected.png:
  • platform/mac/fast/forms/search-zoomed-expected.txt:
  • platform/mac/fast/forms/searchfield-heights-expected.checksum:
  • platform/mac/fast/forms/searchfield-heights-expected.png:
  • platform/mac/fast/forms/searchfield-heights-expected.txt:
  • platform/mac/fast/forms/tabbing-input-iframe-expected.checksum:
  • platform/mac/fast/forms/tabbing-input-iframe-expected.png:
  • platform/mac/fast/forms/tabbing-input-iframe-expected.txt:
  • platform/mac/fast/forms/textAreaLineHeight-expected.checksum:
  • platform/mac/fast/forms/textAreaLineHeight-expected.png:
  • platform/mac/fast/forms/textAreaLineHeight-expected.txt:
  • platform/mac/fast/forms/textarea-align-expected.checksum:
  • platform/mac/fast/forms/textarea-align-expected.png:
  • platform/mac/fast/forms/textarea-align-expected.txt:
  • platform/mac/fast/forms/textarea-rows-cols-expected.checksum:
  • platform/mac/fast/forms/textarea-rows-cols-expected.png:
  • platform/mac/fast/forms/textarea-rows-cols-expected.txt:
  • platform/mac/fast/forms/textarea-scroll-height-expected.checksum:
  • platform/mac/fast/forms/textarea-scroll-height-expected.png:
  • platform/mac/fast/forms/textarea-scroll-height-expected.txt:
  • platform/mac/fast/forms/textarea-scrollbar-expected.checksum:
  • platform/mac/fast/forms/textarea-scrollbar-expected.png:
  • platform/mac/fast/forms/textarea-scrollbar-expected.txt:
  • platform/mac/fast/forms/textarea-scrolled-type-expected.checksum:
  • platform/mac/fast/forms/textarea-scrolled-type-expected.png:
  • platform/mac/fast/forms/textarea-scrolled-type-expected.txt:
  • platform/mac/fast/forms/textarea-setinnerhtml-expected.checksum:
  • platform/mac/fast/forms/textarea-setinnerhtml-expected.png:
  • platform/mac/fast/forms/textarea-setinnerhtml-expected.txt:
  • platform/mac/fast/forms/textarea-width-expected.checksum:
  • platform/mac/fast/forms/textarea-width-expected.png:
  • platform/mac/fast/forms/textarea-width-expected.txt:
  • platform/mac/fast/forms/textfield-drag-into-disabled-expected.checksum:
  • platform/mac/fast/forms/textfield-drag-into-disabled-expected.png:
  • platform/mac/fast/forms/textfield-drag-into-disabled-expected.txt:
  • platform/mac/fast/forms/textfield-outline-expected.checksum:
  • platform/mac/fast/forms/textfield-outline-expected.png:
  • platform/mac/fast/forms/textfield-outline-expected.txt:
  • platform/mac/fast/forms/textfield-overflow-expected.checksum:
  • platform/mac/fast/forms/textfield-overflow-expected.png:
  • platform/mac/fast/forms/textfield-overflow-expected.txt:
  • platform/mac/fast/forms/visual-hebrew-text-field-expected.txt:
  • platform/mac/fast/invalid/residual-style-expected.txt:
  • platform/mac/fast/lists/dynamic-marker-crash-expected.checksum:
  • platform/mac/fast/lists/dynamic-marker-crash-expected.png:
  • platform/mac/fast/lists/dynamic-marker-crash-expected.txt:
  • platform/mac/fast/overflow/overflow-x-y-expected.checksum:
  • platform/mac/fast/overflow/overflow-x-y-expected.png:
  • platform/mac/fast/overflow/overflow-x-y-expected.txt:
  • platform/mac/fast/parser/comment-in-textarea-expected.checksum:
  • platform/mac/fast/parser/comment-in-textarea-expected.png:
  • platform/mac/fast/parser/comment-in-textarea-expected.txt:
  • platform/mac/fast/parser/entity-comment-in-textarea-expected.checksum:
  • platform/mac/fast/parser/entity-comment-in-textarea-expected.png:
  • platform/mac/fast/parser/entity-comment-in-textarea-expected.txt:
  • platform/mac/fast/parser/open-comment-in-textarea-expected.checksum:
  • platform/mac/fast/parser/open-comment-in-textarea-expected.png:
  • platform/mac/fast/parser/open-comment-in-textarea-expected.txt:
  • platform/mac/fast/repaint/renderer-destruction-by-invalidateSelection-crash-expected.checksum:
  • platform/mac/fast/repaint/renderer-destruction-by-invalidateSelection-crash-expected.png:
  • platform/mac/fast/repaint/renderer-destruction-by-invalidateSelection-crash-expected.txt:
  • platform/mac/fast/repaint/search-field-cancel-expected.checksum:
  • platform/mac/fast/repaint/search-field-cancel-expected.png:
  • platform/mac/fast/repaint/search-field-cancel-expected.txt:
  • platform/mac/fast/repaint/subtree-root-skipped-expected.checksum:
  • platform/mac/fast/repaint/subtree-root-skipped-expected.png:
  • platform/mac/fast/repaint/subtree-root-skipped-expected.txt:
  • platform/mac/fast/replaced/replaced-breaking-expected.checksum:
  • platform/mac/fast/replaced/replaced-breaking-expected.png:
  • platform/mac/fast/replaced/replaced-breaking-expected.txt:
  • platform/mac/fast/replaced/replaced-breaking-mixture-expected.checksum:
  • platform/mac/fast/replaced/replaced-breaking-mixture-expected.png:
  • platform/mac/fast/replaced/replaced-breaking-mixture-expected.txt:
  • platform/mac/fast/replaced/width100percent-searchfield-expected.checksum:
  • platform/mac/fast/replaced/width100percent-searchfield-expected.png:
  • platform/mac/fast/replaced/width100percent-searchfield-expected.txt:
  • platform/mac/fast/replaced/width100percent-textarea-expected.checksum:
  • platform/mac/fast/replaced/width100percent-textarea-expected.png:
  • platform/mac/fast/replaced/width100percent-textarea-expected.txt:
  • platform/mac/fast/replaced/width100percent-textfield-expected.checksum:
  • platform/mac/fast/replaced/width100percent-textfield-expected.png:
  • platform/mac/fast/replaced/width100percent-textfield-expected.txt:
  • platform/mac/fast/table/003-expected.checksum:
  • platform/mac/fast/table/003-expected.png:
  • platform/mac/fast/table/003-expected.txt:
  • platform/mac/fast/table/colspanMinWidth-expected.checksum:
  • platform/mac/fast/table/colspanMinWidth-expected.png:
  • platform/mac/fast/table/colspanMinWidth-expected.txt:
  • platform/mac/fast/table/spanOverlapRepaint-expected.checksum:
  • platform/mac/fast/table/spanOverlapRepaint-expected.png:
  • platform/mac/fast/table/spanOverlapRepaint-expected.txt:
  • platform/mac/fast/table/text-field-baseline-expected.checksum:
  • platform/mac/fast/table/text-field-baseline-expected.png:
  • platform/mac/fast/table/text-field-baseline-expected.txt:
  • platform/mac/fast/text/international/rtl-white-space-pre-wrap-expected.checksum:
  • platform/mac/fast/text/international/rtl-white-space-pre-wrap-expected.png:
  • platform/mac/fast/text/international/rtl-white-space-pre-wrap-expected.txt:
  • platform/mac/fast/text/textIteratorNilRenderer-expected.checksum:
  • platform/mac/fast/text/textIteratorNilRenderer-expected.png:
  • platform/mac/fast/text/textIteratorNilRenderer-expected.txt:
  • platform/mac/fast/transforms/transformed-focused-text-input-expected.checksum:
  • platform/mac/fast/transforms/transformed-focused-text-input-expected.png:
  • platform/mac/fast/transforms/transformed-focused-text-input-expected.txt:
  • platform/mac/http/tests/navigation/anchor-basic-expected.checksum:
  • platform/mac/http/tests/navigation/anchor-basic-expected.png:
  • platform/mac/http/tests/navigation/anchor-basic-expected.txt:
  • platform/mac/http/tests/navigation/anchor-frames-expected.txt:
  • platform/mac/http/tests/navigation/anchor-goback-expected.checksum:
  • platform/mac/http/tests/navigation/anchor-goback-expected.png:
  • platform/mac/http/tests/navigation/anchor-goback-expected.txt:
  • platform/mac/http/tests/navigation/anchor-subframeload-expected.txt:
  • platform/mac/http/tests/navigation/javascriptlink-basic-expected.checksum:
  • platform/mac/http/tests/navigation/javascriptlink-basic-expected.png:
  • platform/mac/http/tests/navigation/javascriptlink-basic-expected.txt:
  • platform/mac/http/tests/navigation/javascriptlink-frames-expected.checksum:
  • platform/mac/http/tests/navigation/javascriptlink-frames-expected.png:
  • platform/mac/http/tests/navigation/javascriptlink-frames-expected.txt:
  • platform/mac/http/tests/navigation/javascriptlink-goback-expected.checksum:
  • platform/mac/http/tests/navigation/javascriptlink-goback-expected.png:
  • platform/mac/http/tests/navigation/javascriptlink-goback-expected.txt:
  • platform/mac/http/tests/navigation/javascriptlink-subframeload-expected.checksum:
  • platform/mac/http/tests/navigation/javascriptlink-subframeload-expected.png:
  • platform/mac/http/tests/navigation/javascriptlink-subframeload-expected.txt:
  • platform/mac/http/tests/navigation/metaredirect-basic-expected.checksum:
  • platform/mac/http/tests/navigation/metaredirect-basic-expected.png:
  • platform/mac/http/tests/navigation/metaredirect-basic-expected.txt:
  • platform/mac/http/tests/navigation/metaredirect-frames-expected.checksum:
  • platform/mac/http/tests/navigation/metaredirect-frames-expected.png:
  • platform/mac/http/tests/navigation/metaredirect-frames-expected.txt:
  • platform/mac/http/tests/navigation/metaredirect-goback-expected.checksum:
  • platform/mac/http/tests/navigation/metaredirect-goback-expected.png:
  • platform/mac/http/tests/navigation/metaredirect-goback-expected.txt:
  • platform/mac/http/tests/navigation/metaredirect-subframeload-expected.checksum:
  • platform/mac/http/tests/navigation/metaredirect-subframeload-expected.png:
  • platform/mac/http/tests/navigation/metaredirect-subframeload-expected.txt:
  • platform/mac/http/tests/navigation/post-goback2-expected.checksum:
  • platform/mac/http/tests/navigation/post-goback2-expected.png:
  • platform/mac/http/tests/navigation/post-goback2-expected.txt:
  • platform/mac/http/tests/navigation/postredirect-goback2-expected.checksum:
  • platform/mac/http/tests/navigation/postredirect-goback2-expected.png:
  • platform/mac/http/tests/navigation/postredirect-goback2-expected.txt:
  • platform/mac/http/tests/navigation/redirect302-basic-expected.checksum:
  • platform/mac/http/tests/navigation/redirect302-basic-expected.png:
  • platform/mac/http/tests/navigation/redirect302-basic-expected.txt:
  • platform/mac/http/tests/navigation/redirect302-frames-expected.checksum:
  • platform/mac/http/tests/navigation/redirect302-frames-expected.png:
  • platform/mac/http/tests/navigation/redirect302-frames-expected.txt:
  • platform/mac/http/tests/navigation/redirect302-goback-expected.checksum:
  • platform/mac/http/tests/navigation/redirect302-goback-expected.png:
  • platform/mac/http/tests/navigation/redirect302-goback-expected.txt:
  • platform/mac/http/tests/navigation/redirect302-subframeload-expected.checksum:
  • platform/mac/http/tests/navigation/redirect302-subframeload-expected.png:
  • platform/mac/http/tests/navigation/redirect302-subframeload-expected.txt:
  • platform/mac/http/tests/navigation/relativeanchor-basic-expected.checksum:
  • platform/mac/http/tests/navigation/relativeanchor-basic-expected.png:
  • platform/mac/http/tests/navigation/relativeanchor-basic-expected.txt:
  • platform/mac/http/tests/navigation/relativeanchor-frames-expected.txt:
  • platform/mac/http/tests/navigation/relativeanchor-goback-expected.checksum:
  • platform/mac/http/tests/navigation/relativeanchor-goback-expected.png:
  • platform/mac/http/tests/navigation/relativeanchor-goback-expected.txt:
  • platform/mac/http/tests/navigation/slowmetaredirect-basic-expected.checksum:
  • platform/mac/http/tests/navigation/slowmetaredirect-basic-expected.png:
  • platform/mac/http/tests/navigation/slowmetaredirect-basic-expected.txt:
  • platform/mac/http/tests/navigation/slowtimerredirect-basic-expected.checksum:
  • platform/mac/http/tests/navigation/slowtimerredirect-basic-expected.png:
  • platform/mac/http/tests/navigation/slowtimerredirect-basic-expected.txt:
  • platform/mac/http/tests/navigation/success200-basic-expected.checksum:
  • platform/mac/http/tests/navigation/success200-basic-expected.png:
  • platform/mac/http/tests/navigation/success200-basic-expected.txt:
  • platform/mac/http/tests/navigation/success200-frames-expected.checksum:
  • platform/mac/http/tests/navigation/success200-frames-expected.png:
  • platform/mac/http/tests/navigation/success200-frames-expected.txt:
  • platform/mac/http/tests/navigation/success200-frames-loadsame-expected.checksum:
  • platform/mac/http/tests/navigation/success200-frames-loadsame-expected.png:
  • platform/mac/http/tests/navigation/success200-frames-loadsame-expected.txt:
  • platform/mac/http/tests/navigation/success200-goback-expected.checksum:
  • platform/mac/http/tests/navigation/success200-goback-expected.png:
  • platform/mac/http/tests/navigation/success200-goback-expected.txt:
  • platform/mac/http/tests/navigation/success200-loadsame-expected.checksum:
  • platform/mac/http/tests/navigation/success200-loadsame-expected.png:
  • platform/mac/http/tests/navigation/success200-loadsame-expected.txt:
  • platform/mac/http/tests/navigation/success200-reload-expected.checksum:
  • platform/mac/http/tests/navigation/success200-reload-expected.png:
  • platform/mac/http/tests/navigation/success200-reload-expected.txt:
  • platform/mac/http/tests/navigation/success200-subframeload-expected.checksum:
  • platform/mac/http/tests/navigation/success200-subframeload-expected.png:
  • platform/mac/http/tests/navigation/success200-subframeload-expected.txt:
  • platform/mac/http/tests/navigation/timerredirect-basic-expected.checksum:
  • platform/mac/http/tests/navigation/timerredirect-basic-expected.png:
  • platform/mac/http/tests/navigation/timerredirect-basic-expected.txt:
  • platform/mac/http/tests/navigation/timerredirect-frames-expected.checksum:
  • platform/mac/http/tests/navigation/timerredirect-frames-expected.png:
  • platform/mac/http/tests/navigation/timerredirect-frames-expected.txt:
  • platform/mac/http/tests/navigation/timerredirect-goback-expected.checksum:
  • platform/mac/http/tests/navigation/timerredirect-goback-expected.png:
  • platform/mac/http/tests/navigation/timerredirect-goback-expected.txt:
  • platform/mac/http/tests/navigation/timerredirect-subframeload-expected.checksum:
  • platform/mac/http/tests/navigation/timerredirect-subframeload-expected.png:
  • platform/mac/http/tests/navigation/timerredirect-subframeload-expected.txt:
  • platform/mac/svg/custom/inline-svg-in-xhtml-expected.checksum:
  • platform/mac/svg/custom/inline-svg-in-xhtml-expected.png:
  • platform/mac/svg/custom/inline-svg-in-xhtml-expected.txt:
  • platform/mac/svg/hixie/mixed/003-expected.checksum:
  • platform/mac/svg/hixie/mixed/003-expected.png:
  • platform/mac/svg/hixie/mixed/003-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug1188-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug1188-expected.png:
  • platform/mac/tables/mozilla/bugs/bug1188-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug12384-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug12384-expected.png:
  • platform/mac/tables/mozilla/bugs/bug12384-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug18359-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug18359-expected.png:
  • platform/mac/tables/mozilla/bugs/bug18359-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug194024-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug194024-expected.png:
  • platform/mac/tables/mozilla/bugs/bug194024-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug24200-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug24200-expected.png:
  • platform/mac/tables/mozilla/bugs/bug24200-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug2479-2-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug2479-2-expected.png:
  • platform/mac/tables/mozilla/bugs/bug2479-2-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug2479-3-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug2479-3-expected.png:
  • platform/mac/tables/mozilla/bugs/bug2479-3-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug2479-4-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug28928-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug28928-expected.png:
  • platform/mac/tables/mozilla/bugs/bug28928-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug30559-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug30559-expected.png:
  • platform/mac/tables/mozilla/bugs/bug30559-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug30692-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug30692-expected.png:
  • platform/mac/tables/mozilla/bugs/bug30692-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug4382-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug4382-expected.png:
  • platform/mac/tables/mozilla/bugs/bug4382-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug4527-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug4527-expected.png:
  • platform/mac/tables/mozilla/bugs/bug4527-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug46368-1-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug46368-1-expected.png:
  • platform/mac/tables/mozilla/bugs/bug46368-1-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug46368-2-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug46368-2-expected.png:
  • platform/mac/tables/mozilla/bugs/bug46368-2-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug51037-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug51037-expected.png:
  • platform/mac/tables/mozilla/bugs/bug51037-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug55545-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug55545-expected.png:
  • platform/mac/tables/mozilla/bugs/bug55545-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug59354-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug59354-expected.png:
  • platform/mac/tables/mozilla/bugs/bug59354-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug7342-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug7342-expected.png:
  • platform/mac/tables/mozilla/bugs/bug7342-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug96334-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug96334-expected.png:
  • platform/mac/tables/mozilla/bugs/bug96334-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug99948-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug99948-expected.png:
  • platform/mac/tables/mozilla/bugs/bug99948-expected.txt:
  • platform/mac/tables/mozilla/dom/tableDom-expected.checksum:
  • platform/mac/tables/mozilla/dom/tableDom-expected.png:
  • platform/mac/tables/mozilla/dom/tableDom-expected.txt:
  • platform/mac/tables/mozilla/other/move_row-expected.checksum:
  • platform/mac/tables/mozilla/other/move_row-expected.png:
  • platform/mac/tables/mozilla/other/move_row-expected.txt:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug2479-5-expected.txt:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug92647-1-expected.checksum:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug92647-1-expected.png:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug92647-1-expected.txt:
  • platform/win/Skipped:
  • svg/custom/resources/svg-fonts-in-text-controls.js:
  • svg/custom/svg-fonts-in-text-controls-expected.txt:
  • svg/custom/svg-fonts-in-text-controls.html: Removed.
9:26 PM Changeset in webkit [42697] by Dimitri Glazkov
  • 2 edits in trunk/LayoutTests

2009-04-20 Dimitri Glazkov <Dimitri Glazkov>

Not reviewed, typo fix.

Rename erroneous lines in Skipped to actual test URLs.

  • platform/win/Skipped:
8:50 PM Changeset in webkit [42696] by Adam Roben
  • 2 edits in trunk/WebKit/win

Build fix

  • MemoryStream.cpp: (MemoryStream::Clone): Fix typo introduced when implementing suggestions from patch review.
8:48 PM Changeset in webkit [42695] by ggaren@apple.com
  • 2 edits in trunk/WebCore

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Adam Roben.

One last loose end, discovered using COLLECT_ON_EVERY_ALLOCATION, for
https://bugs.webkit.org/show_bug.cgi?id=21260
Unbounded memory growth when churning elements with anonymous event handler functions


  • bindings/js/JSSVGElementInstanceCustom.cpp: (WebCore::JSSVGElementInstance::mark): Don't ASSERT that an ElementInstance's correspondingElement has a JS wrapper. If a GC falls exactly between the allocation of the ElementInstance wrapper and the correspondingElement wrapper, the correspondingElement won't have a wrapper at the time we mark the ElementInstance's wrapper.
8:45 PM Changeset in webkit [42694] by Dimitri Glazkov
  • 2 edits in trunk/LayoutTests

2009-04-20 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Roben.

Add Windows failures temporarily to the Skipped list after r42678.
See https://bugs.webkit.org/show_bug.cgi?id=25306 for more
information. This is just to get the tree green while the tests
are fixed.

  • platform/win/Skipped:
8:42 PM Changeset in webkit [42693] by Adam Roben
  • 3 edits in trunk/WebKit/win

First part of <rdar://6395825> It takes over 20 sec to launch Safari with 500KB history file

This ports the changes that were made to WebKit/mac's WebHistory
implementation in r25275 to WebKit/win. WebHistory now stores a
HashMap from CFAbsoluteTime (stored as int64_t) to CFArray of
IWebHistoryItem*. The HashMap lets us look up the CFArray for a
particular day's history in constant time rather than linear time. The
precise reasons why we store the CFAbsoluteTime as an int64_t are lost
to the mists of time, but it is likely because these CFAbsoluteTimes
never have a decimal part, and integer comparisons are faster than
floating-point comparisons, so storing them as int64_t should improve
performance without losing precision. We also now use a binary search
instead of a linear search when determining the index at which to
insert a WebHistoryItem into its day's CFArray.

This patch reduces the time needed to load a 100,000-item
History.plist from ~15 seconds to ~7.5 seconds. Further improvements
could likely be made by reducing the number of string conversions,
reducing calls to CFTimeZone functions, and removing all the
WebHistoryItemsAdded notifications that are sent (Mac doesn't send any
while loading history).

Reviewed by Darin Adler.

  • WebHistory.cpp: (WebHistoryWriter::WebHistoryWriter): Changed to take a DateToEntriesMap instead of a CFArrayRef. Initialize m_dateKeys to contain the keys from m_entriesByDate in ascending order. (WebHistoryWriter::writeHistoryItems): Now loops through m_dateKeys in reverse order and gets the entries array from the DateToEntriesMap. (WebHistory::WebHistory): Removed code to initialize m_datesWithEntries (which has been removed) and m_entriesByDate (which has been converted to a HashMap). (WebHistory::removeAllItems): Clear out m_entriesByDate and m_orderedLastVisitedDays. (WebHistory::orderedLastVisitedDays): If we don't already have a cached m_orderedLastVisitedDays array, create one by converting the keys from m_entriesByDate to DATEs and sorting them in descending order. Then copy m_orderedLastVisitedDays to the output buffer. (WebHistory::orderedItemsLastVisitedOnDay): Updated to use findKey instead of findIndex and to treat m_entriesByDate as a HashMap. Now that the items in each day's CFArray are stored in descending order (newest to oldest), we don't have to reverse them when filling in the output buffer. (The old comment about putting the items in oldest-to-newest order was incorrect -- it was putting them in newest-to-oldest order.)

(WebHistory::addItemToDateCaches):
(WebHistory::removeItemFromDateCaches):
Changed to use findKey instead of findIndex and to treat
m_entriesByDate as a HashMap. If we're adding a key to or removing a
key from m_entriesByDate, also clear m_orderedLastVisitedDays so that
we will regenerate it for the current set of keys the next time it is
needed.

(timeIntervalForBeginningOfDay): Added. Returns the CFAbsoluteTime
that corresponds to the beginning of the day in which the passed-in
DATE occurs.
(WebHistory::findKey): Returns the key used in m_entriesByDate to
store the CFArray that should contain the IWebHistoryItem that
corresponds to the passed-in DATE. Replaces findIndex.
(WebHistory::insertItem): Changed to treat m_entriesByDate as a
HashMap rather than a CFArray. We now optimize for inserting at the
beginning and end of the array, and use a binary rather than linear
search when inserting into the middle of the array.
(WebHistory::data): Changed to treat m_entriesByDate as a HashMap.

  • WebHistory.h: Changed m_entriesByDate to a DateToEntriesMap, removed m_datesWithEntries, added m_orderedLastVisitedDays.
8:42 PM Changeset in webkit [42692] by Adam Roben
  • 9 edits in trunk/WebKit/win

Change MemoryStream::createInstance to return a COMPtr

Part of Bug 25294: All WebKit/win classes should return COMPtrs from
their static constructor members
<https://bugs.webkit.org/show_bug.cgi?id=25294>

Reviewed by Darin Adler.

  • MemoryStream.cpp: (MemoryStream::createInstance): Changed to return a COMPtr. (MemoryStream::Clone): Updated for createInstance change.
  • MemoryStream.h: Changed createInstance to return a COMPtr.
  • WebArchive.cpp: (WebArchive::data):
  • WebCoreSupport/EmbeddedWidget.cpp: (EmbeddedWidget::didReceiveData):
  • WebDataSource.cpp: (WebDataSource::data):
  • WebHistory.cpp: (WebHistory::data):
  • WebIconFetcher.cpp: (WebIconFetcherClient::finishedFetchingIcon):
  • WebResource.cpp: (WebResource::createInstance): Updated for changes to MemoryStream::createInstance.
8:42 PM Changeset in webkit [42691] by Adam Roben
  • 8 edits
    7 deletes in trunk/WebKit/win

Fix Bug 25295: Remove WebKit/win code that was used for Drosera support

<https://bugs.webkit.org/show_bug.cgi?id=25295>

Reviewed by Sam Weinig.

  • ForEachCoClass.h: Removed WebScriptDebugServer.
  • Interfaces/WebKit.idl: Removed WebScriptDebugServer and references to removed IDL files.
  • WebKit.vcproj/Interfaces.vcproj:
  • WebKit.vcproj/WebKit.vcproj: Removed IDL and source files for IWebScriptCallFrame, IWebScriptDebugListener, IWebScriptDebugServer, WebScriptCallFrame, and WebScriptDebugServer.
  • WebCoreSupport/WebFrameLoaderClient.cpp:
  • WebKitClassFactory.cpp:
  • WebKitDLL.cpp: Removed #includes of WebScriptDebugServer.h.
  • Interfaces/IWebScriptCallFrame.idl: Removed.
  • Interfaces/IWebScriptDebugListener.idl: Removed.
  • Interfaces/IWebScriptDebugServer.idl: Removed.
  • WebScriptCallFrame.cpp: Removed.
  • WebScriptCallFrame.h: Removed.
  • WebScriptDebugServer.cpp: Removed.
  • WebScriptDebugServer.h: Removed.
5:57 PM Changeset in webkit [42690] by levin@chromium.org
  • 3 edits in trunk/LayoutTests

2009-04-20 Ojan Vafai <ojan@chromium.org>

Reviewed by Eric Seidel

Fix svg-fonts-in-text-controls.js to measure the
widths after the SVG font has loaded.

For now the expected results have the success line
printed before the metrics being measured. I'll fix
this in a following commit. See
https://bugs.webkit.org/show_bug.cgi?id=25302
for more details.

  • svg/custom/resources/svg-fonts-in-text-controls.js:
  • svg/custom/svg-fonts-in-text-controls-expected.txt:
4:36 PM Changeset in webkit [42689] by sfalken@apple.com
  • 3 edits
    3 adds in trunk/JavaScriptCore

More Windows build fixes.

3:32 PM Changeset in webkit [42688] by beidson@apple.com
  • 2 edits in trunk/WebCore

2009-04-20 Brady Eidson <beidson@apple.com>

Rubberstamped by Mark Rowe

Clean builds are broken on case-sensitive filesystems. Let's fix, shall we?

  • platform/graphics/SimpleFontData.cpp: Remove "String.h" include. Ojan probably intended to include "PlatformString.h" which was unnecessary because it is pulled in through other headers. This wasn't a build failure on case-insensitive file systems because those systems would find the system <string.h>, averting the warning.
3:32 PM Changeset in webkit [42687] by ggaren@apple.com
  • 6 edits in trunk/WebCore

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Darin Adler.

Last patch for https://bugs.webkit.org/show_bug.cgi?id=21260
Unbounded memory growth when churning elements with anonymous event handler functions


Converted "lazy" event listeners to be unprotected, just like all the others.

  • bindings/js/JSEventListener.cpp: (WebCore::JSEventListener::JSEventListener): (WebCore::JSEventListener::~JSEventListener): (WebCore::JSEventListener::jsFunction): (WebCore::JSEventListener::markJSFunction): (WebCore::JSEventListener::handleEvent): (WebCore::JSEventListener::virtualIsInline):
  • bindings/js/JSEventListener.h: (WebCore::JSEventListener::isInline): Merged JSAbstractEventListener into JSEventListener. Now that the only difference between JSEventListener and JSLazyEventListener is that JSLazyEventListener compiles lazily, there's no need for an abstract base class.
  • bindings/js/JSLazyEventListener.cpp: Converted JSLazyEventListener to inherit from JSEventListener and match its un-GC-protected behavior. (WebCore::JSLazyEventListener::JSLazyEventListener): ditto (WebCore::JSLazyEventListener::parseCode): ditto (WebCore::createInlineEventListener): When creating a lazy event listener, ensure that the related node has a JS wrapper to mark the listener. Since the parser makes these listeners, it's possible that no JS reference has been made to the node yet.
  • bindings/js/JSLazyEventListener.h: ditto
  • dom/EventListener.h: (WebCore::EventListener::clearJSFunction): Removed an usused function.
3:30 PM Changeset in webkit [42686] by justin.garcia@apple.com
  • 1 edit in trunk/WebCore/rendering/RenderText.cpp

Fixed typo.

3:28 PM Changeset in webkit [42685] by justin.garcia@apple.com
  • 2 edits in trunk/WebCore

2009-04-20 Justin Garcia <justin.garcia@apple.com>

Reviewed by Dan Bernstein.

https://bugs.webkit.org/show_bug.cgi?id=25299
Don't bound caret to root line box if it is still inside its containing block


Can't yet write tests for behavior of non-standard caret widths.

  • rendering/RenderText.cpp: (WebCore::RenderText::localCaretRect):
2:47 PM Changeset in webkit [42684] by sfalken@apple.com
  • 1 edit in trunk/WebKit/mac/WebKitPrefix.h

Build fix

2:47 PM Changeset in webkit [42683] by sfalken@apple.com
  • 1 edit in trunk/WebCore/config.h

Build fix

2:46 PM Changeset in webkit [42682] by sfalken@apple.com
  • 1 edit in trunk/JavaScriptGlue/config.h

Build fix

2:36 PM Changeset in webkit [42681] by Darin Adler
  • 5 edits
    2 adds in trunk

WebKitTools:

2009-04-20 Darin Adler <Darin Adler>

Reviewed by Sam Weinig.

  • DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj: Added PlainTextController.h and PlainTextController.mm.
  • DumpRenderTree/mac/FrameLoadDelegate.mm: (-[FrameLoadDelegate webView:didClearWindowObject:forFrame:]): Added PlainTextController as "window.plainText".
  • DumpRenderTree/mac/PlainTextController.h: Added.
  • DumpRenderTree/mac/PlainTextController.mm: Added.

LayoutTests:

2009-04-20 Darin Adler <Darin Adler>

Reviewed by Sam Weinig.

  • fast/dom/Window/window-properties.html: Added a new DumpRenderTree window object property, plainText.
2:25 PM Changeset in webkit [42680] by sfalken@apple.com
  • 23 edits
    2 adds in trunk

JavaScriptCore:

2009-04-20 Steve Falkenburg <sfalken@apple.com>

Separate JavaScriptCore.dll from WebKit.dll.
Slight performance improvement or no change on benchmarks.


Allows us to break a circular dependency between CFNetwork and WebKit on Windows,
and simplifies standalone JavaScriptCore builds.

Reviewed by Oliver Hunt.

  • API/JSBase.h: Export symbols with JS_EXPORT when using MSVC.
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
  • JavaScriptCore.vcproj/WTF/WTF.vcproj: Build JavaScriptCore as a DLL instead of a static library.
  • config.h: Specify declspec(dllexport/dllimport) appropriately when exporting data.
  • runtime/InternalFunction.h: Specify JS_EXPORTDATA on exported data.
  • runtime/JSArray.h: Specify JS_EXPORTDATA on exported data.
  • runtime/JSFunction.h: Specify JS_EXPORTDATA on exported data.
  • runtime/StringObject.h: Specify JS_EXPORTDATA on exported data.
  • runtime/UString.h: Specify JS_EXPORTDATA on exported data.

WebCore:

2009-04-20 Steve Falkenburg <sfalken@apple.com>

Separate JavaScriptCore.dll from WebKit.dll.
Slight performance improvement or no change on benchmarks.


Allows us to break a circular dependency between CFNetwork and WebKit on Windows,
and simplifies standalone JavaScriptCore builds.

Reviewed by Oliver Hunt.

  • config.h: Specify declspec(dllexport/dllimport) appropriately when exporting data.

WebKit/cf:

2009-04-20 Steve Falkenburg <sfalken@apple.com>

Separate JavaScriptCore.dll from WebKit.dll.
Slight performance improvement or no change on benchmarks.


Allows us to break a circular dependency between CFNetwork and WebKit on Windows,
and simplifies standalone JavaScriptCore builds.


Reviewed by Oliver Hunt.

  • WebCoreSupport/WebInspectorClientCF.cpp: Workaround for lack of config.h in shared WebKit file (WebKit/win uses config.h from WebCore).

WebKit/win:

2009-04-20 Steve Falkenburg <sfalken@apple.com>

Separate JavaScriptCore.dll from WebKit.dll.
Slight performance improvement or no change on benchmarks.


Allows us to break a circular dependency between CFNetwork and WebKit on Windows,
and simplifies standalone JavaScriptCore builds.

Reviewed by Oliver Hunt.

  • WebKit.vcproj/WebKit.def: Mark JavaScriptCore re-exports as deprecated.
  • WebKit.vcproj/WebKit.vcproj: Link against JavaScriptCore.dll
  • WebKit.vcproj/WebKit_debug.def: Mark JavaScriptCore re-exports as deprecated.

WebKitLibraries:

2009-04-20 Steve Falkenburg <sfalken@apple.com>

Separate JavaScriptCore.dll from WebKit.dll.
Slight performance improvement or no change on benchmarks.


Allows us to break a circular dependency between CFNetwork and WebKit on Windows,
and simplifies standalone JavaScriptCore builds.

Reviewed by Oliver Hunt.

  • win/tools/vsprops/common.vsprops: Add BUILDING_{project} preprocessor define.

WebKitTools:

2009-04-20 Steve Falkenburg <sfalken@apple.com>

Separate JavaScriptCore.dll from WebKit.dll.
Slight performance improvement or no change on benchmarks.


Allows us to break a circular dependency between CFNetwork and WebKit on Windows,
and simplifies standalone JavaScriptCore builds.

Reviewed by Oliver Hunt.

  • DumpRenderTree/config.h: Specify JS, WebKit exported data as declspec(dllimport)
  • DumpRenderTree/win/DumpRenderTree.vcproj: Link against JavaScriptCore.dll
1:41 PM Changeset in webkit [42679] by andersca@apple.com
  • 18 edits in trunk

WebCore:

2009-04-20 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

<rdar://problem/6722845> In the Cocoa event model, NPWindow's window field should be null

  • bridge/npapi.h: (_NPCocoaEvent::): Add a CGContextRef member to the draw struct. Add an NPCoordinateSpace enum.


  • plugins/npfunctions.h: Add a prototype for NPN_ConvertPoint.

WebKit/mac:

2009-04-17 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

<rdar://problem/6722845>
In the Cocoa event model, NPWindow's window field should be null


  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WKPCConvertPoint): Get the instance proxy and call it's convertPoint function.


  • Plugins/Hosted/NetscapePluginInstanceProxy.h:
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::convertPoint): Call convertPoint on the plug-in view.


  • Plugins/Hosted/WebKitPluginClient.defs: Add PCConvertPoint.


  • Plugins/WebBaseNetscapePluginView.h: Add a declaration for convertFromX:andY:space:toX:andY:space:.


  • Plugins/WebBaseNetscapePluginView.mm: (-[WebBaseNetscapePluginView convertFromX:andY:space:toX:andY:space:]): Convert a point from one coordinate system to another.


  • Plugins/WebNetscapePluginEventHandler.h:
  • Plugins/WebNetscapePluginEventHandlerCarbon.h:
  • Plugins/WebNetscapePluginEventHandlerCarbon.mm: (WebNetscapePluginEventHandlerCarbon::drawRect):
  • Plugins/WebNetscapePluginEventHandlerCocoa.h: Add CGContextRef to drawRect.
  • Plugins/WebNetscapePluginEventHandlerCocoa.mm: (WebNetscapePluginEventHandlerCocoa::drawRect): Set the passed in context.


  • Plugins/WebNetscapePluginPackage.mm: (-[WebNetscapePluginPackage _tryLoad]): Add NPN_ConvertPoint to the browserFuncs vtable.


  • Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView saveAndSetNewPortStateForUpdate:]): Only set the window for the Carbon event model.


(-[WebNetscapePluginView restorePortState:]):
It's OK for the window context to be null.


(-[WebNetscapePluginView sendDrawRectEvent:]):
Pass the CGContextRef to drawRect.

  • Plugins/npapi.mm: (NPN_ConvertPoint): Call the plug-in view method.
12:52 PM QtWebKitTodo edited by kenneth.christiansen@gmail.com
(diff)
12:49 PM Changeset in webkit [42678] by eric@webkit.org
  • 546 edits
    10 adds in trunk

Reviewed by Dan Bernstein.

Make textarea and text input metrics more closely match IEs.
https://bugs.webkit.org/show_bug.cgi?id=15312

This involves:
-set text input width to size*avgCharWidth + maxCharWidth - avgCharWidth
-set textarea width to cols*avgCharWidth
-Make default CSS match IEs
-Correctly initializing m_avgCharWidth and m_maxCharWidth for each platform and SVG.

Those values for textarea and inputs were derived by doing a ton of manual
testing of IE's width values for various textareas and fonts.

To get the average and max character width of a font, we do the following
for each platform:
-Win: TextMetrics expose avgCharWidth and maxCharWidth
-SVG: avgCharWidth = width of an '0', fallback on width of a space glyph, then m_xHeight

maxCharWidth = width of a 'W' for roman fonts, fallback on m_ascent

-Linux: avgCharWidth = width of an '0', fallback on m_xHeight

maxCharWidth = max of avgCharWidth and m_ascent

-Mac: look in the OS/2 table for avgCharWidth and grab the maxCharWidth off the font.

If either one is not there, then calculate the value using the Linux approach.

Linux ports could probably dig into the OS/2 table as well, but I'll leave
that up to them to implement.

Tests: fast/forms/text-control-intrinsic-widths.html

fast/forms/textarea-metrics.html
svg/custom/svg-fonts-in-text-controls.html

  • css/html4.css:
  • css/themeWin.css:
  • platform/graphics/SimpleFontData.cpp: (WebCore::SimpleFontData::SimpleFontData): (WebCore::SimpleFontData::initCharWidths):
  • platform/graphics/SimpleFontData.h: (WebCore::SimpleFontData::maxCharWidth): (WebCore::SimpleFontData::avgCharWidth):
  • platform/graphics/chromium/SimpleFontDataChromiumWin.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/chromium/SimpleFontDataLinux.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/gtk/SimpleFontDataGtk.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/gtk/SimpleFontDataPango.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/mac/SimpleFontDataMac.mm: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/qt/SimpleFontDataQt.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/win/SimpleFontDataCGWin.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/win/SimpleFontDataCairoWin.cpp: (WebCore::SimpleFontData::platformInit): (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/win/SimpleFontDataWin.cpp: (WebCore::SimpleFontData::initGDIFont):
  • platform/graphics/wx/SimpleFontDataWx.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::calcPrefWidths):
  • rendering/RenderTextControlMultiLine.cpp: (WebCore::RenderTextControlMultiLine::createInnerTextStyle):
  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::preferredContentWidth):
12:13 PM QtWebKitContrib edited by kenneth.christiansen@gmail.com
(diff)
12:10 PM Changeset in webkit [42677] by eric@webkit.org
  • 10 edits in trunk/WebCore

Reviewed by Simon Fraser.

Remove m_absoluteBounds hack from SVG renderers and move outlineBoundsForRepaint into RenderSVGModelObject
https://bugs.webkit.org/show_bug.cgi?id=25276

This also exposed a buggy paint check in RenderSVGContainer::layout()
we should repaint if we are self painting OR if our kids changed, not AND.

Writing real outlineBoundsForRepaint required writing a mapLocalToContainer() function

No functional changes, thus no tests.

  • rendering/RenderPath.cpp: (WebCore::RenderPath::localToParentTransform): (WebCore::RenderPath::layout):
  • rendering/RenderPath.h:
  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::layout):
  • rendering/RenderSVGContainer.h:
  • rendering/RenderSVGModelObject.cpp: (WebCore::RenderSVGModelObject::mapLocalToContainer): (WebCore::RenderSVGModelObject::outlineBoundsForRepaint):
  • rendering/RenderSVGModelObject.h:
  • rendering/RenderSVGRoot.cpp: (WebCore::RenderSVGRoot::layout): (WebCore::RenderSVGRoot::paint): (WebCore::RenderSVGRoot::localToParentTransformWithoutCSSParentOffset): (WebCore::RenderSVGRoot::localToParentTransform): (WebCore::RenderSVGRoot::computeRectForRepaint):
  • rendering/RenderSVGRoot.h:
  • rendering/RenderSVGViewportContainer.cpp: (WebCore::RenderSVGViewportContainer::layout):
11:12 AM Changeset in webkit [42676] by weinig@apple.com
  • 5 edits in trunk/JavaScriptCore

2009-04-20 Sam Weinig <sam@webkit.org>

Reviewed by Kevin McCullough.

Always tag mmaped memory on darwin and clean up #defines
now that they are a little bigger.

  • interpreter/RegisterFile.h: (JSC::RegisterFile::RegisterFile):
  • jit/ExecutableAllocatorFixedVMPool.cpp: (JSC::FixedVMPoolAllocator::FixedVMPoolAllocator):
  • jit/ExecutableAllocatorPosix.cpp: (JSC::ExecutablePool::systemAlloc):
  • runtime/Collector.cpp: (JSC::allocateBlock):
11:11 AM Changeset in webkit [42675] by Simon Fraser
  • 3 edits
    10 adds in trunk

2009-04-20 Simon Fraser <Simon Fraser>

Reviewed by Eric Seidel

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

outlineBoundsForRepaint() should compute a quad relative to the
repaintContainer. This fixes the repaint issues originally fixed in
https://bugs.webkit.org/show_bug.cgi?id=12885 for elements in compositing layers.
Failure was only apparent when building with ACCELERATED_COMPOSITING enabled.

Tests: compositing/repaint/content-into-overflow.html

compositing/repaint/overflow-into-content.html

  • rendering/RenderBox.cpp: (WebCore::RenderBox::outlineBoundsForRepaint):
11:06 AM Changeset in webkit [42674] by weinig@apple.com
  • 29 edits in trunk

JavaScriptCore:

2009-04-20 Sam Weinig <sam@webkit.org>

Rubber-stamped by Tim Hatcher.

Add licenses for xcconfig files.

  • Configurations/Base.xcconfig:
  • Configurations/DebugRelease.xcconfig:
  • Configurations/FeatureDefines.xcconfig:
  • Configurations/JavaScriptCore.xcconfig:
  • Configurations/Version.xcconfig:

JavaScriptGlue:

2009-04-20 Sam Weinig <sam@webkit.org>

Rubber-stamped by Tim Hatcher.

Add licenses for xcconfig files.

  • Configurations/Base.xcconfig:
  • Configurations/DebugRelease.xcconfig:
  • Configurations/JavaScriptGlue.xcconfig:
  • Configurations/Version.xcconfig:

WebCore:

2009-04-20 Sam Weinig <sam@webkit.org>

Rubber-stamped by Tim Hatcher.

Add licenses for xcconfig files.

  • Configurations/Base.xcconfig:
  • Configurations/DebugRelease.xcconfig:
  • Configurations/FeatureDefines.xcconfig:
  • Configurations/Version.xcconfig:
  • Configurations/WebCore.xcconfig:

WebKit/mac:

2009-04-20 Sam Weinig <sam@webkit.org>

Rubber-stamped by Tim Hatcher.

Add licenses for xcconfig files.


  • Configurations/Base.xcconfig:
  • Configurations/DebugRelease.xcconfig:
  • Configurations/FeatureDefines.xcconfig:
  • Configurations/Version.xcconfig:
  • Configurations/WebKit.xcconfig:

WebKitTools:

2009-04-20 Sam Weinig <sam@webkit.org>

Rubber-stamped by Tim Hatcher.

Add licenses for xcconfig files.

  • DumpRenderTree/mac/Configurations/Base.xcconfig:
  • DumpRenderTree/mac/Configurations/DebugRelease.xcconfig:
  • DumpRenderTree/mac/Configurations/DumpRenderTree.xcconfig:
  • DumpRenderTree/mac/Configurations/ImageDiff.xcconfig:
  • DumpRenderTree/mac/Configurations/TestNetscapePlugIn.xcconfig:
10:31 AM Changeset in webkit [42673] by andersca@apple.com
  • 5 edits in trunk/WebKit/mac

2009-04-20 Anders Carlsson <andersca@apple.com>

Reviewed by Dan Bernstein.

WebKit side of <rdar://problem/6781302>


  • Plugins/Hosted/NetscapePluginInstanceProxy.h:
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::keyEvent): Pass the event keyChar.


(WebKit::NetscapePluginInstanceProxy::syntheticKeyDownWithCommandModifier):
Ditto.


(WebKit::NetscapePluginInstanceProxy::flagsChanged):
Pass a 0 keyChar.


  • Plugins/Hosted/WebHostedNetscapePluginView.mm: (-[WebHostedNetscapePluginView flagsChanged:]): Call NetscapePluginInstanceProxy::flagsChanged.


  • Plugins/Hosted/WebKitPluginHost.defs: Add a keyChar argument.
10:16 AM Changeset in webkit [42672] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-20 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=24786
WebKitDownload sometimes suggests peculiar filenames

Remove query and ref from the URI that is to be downloaded when
using it to provide a file name suggestion.

  • webkit/webkitdownload.cpp: (_WebKitDownloadPrivate::webkit_download_set_property):
8:27 AM Changeset in webkit [42671] by Dimitri Glazkov
  • 3 edits in trunk/WebCore

2009-04-20 Eric Roman <eroman@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25261
Implement the V8 binding for DOMWindow.event similarly to JSC, by using
the custom getter boilerplate from the IDL file.
Also, stub out DOMWindow.crypto which is defined by the idl.

  • bindings/v8/V8AbstractEventListener.cpp: (WebCore::V8AbstractEventListener::invokeEventHandler):
  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::ACCESSOR_GETTER):
8:23 AM Changeset in webkit [42670] by kov@webkit.org
  • 4 edits in trunk

WebCore

2009-04-20 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=25243
Crash when data:// loads are cancelled

Properly handle cancellation of the load for data:// loads. This
fixes crashing in the followin test:

plugins/return-error-from-new-stream-callback-in-full-frame-plugin.html

  • platform/network/soup/ResourceHandleSoup.cpp: (WebCore::parseDataUrl):

WebKit/gtk

2009-04-20 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

Handle the case when the plugin view is destroyed during
didReceiveResponse, to avoid crashing.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::committedLoad):
7:30 AM QtWebKitTodo edited by kenneth.christiansen@gmail.com
(diff)
7:11 AM QtWebKitTodo edited by ariya@webkit.org
(diff)
6:24 AM QtWebKitJournal edited by Simon Hausmann
(diff)
6:17 AM QtWebKitTodo edited by Simon Hausmann
(diff)
2:27 AM Changeset in webkit [42669] by ariya@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-04-20 Ariya Hidayat <ariya.hidayat@nokia.com>

Build fix for Qt port (after r42646). Not reviewed.

  • wtf/unicode/qt4/UnicodeQt4.h: Added U16_PREV.
Note: See TracTimeline for information about the timeline view.