Timeline



Jan 30, 2009:

10:24 PM Changeset in webkit [40448] by mitz@apple.com
  • 2 edits in trunk/WebKitLibraries

Reviewed by Timothy Hatcher.

  • <rdar://problem/6545912> expose the build number in autoversion.h
  • win/tools/scripts/auto-version.sh: Added a #define BUILD_NUMBER with the full build number.
9:12 PM Changeset in webkit [40447] by zecke@webkit.org
  • 2 edits in trunk/WebCore

2009-01-30 Holger Hans Peter Freyther <zecke@selfish.org>

Build fix.

Move the inline virtual destructor after a non-inline virtual
function so that the symbol for the vtable is not marked as a
weakly exported symbol.

This trick was previously used at revision 36122 in JSObject.h

  • loader/FrameLoaderClient.h: (WebCore::FrameLoaderClient::~FrameLoaderClient):
8:43 PM Changeset in webkit [40446] by oliver@apple.com
  • 3 edits in trunk/WebCore

Inline SimpleFontData::widthForGlyph for a minor page load improvement.

Reviewed by Stephanie Lewis

8:40 PM Changeset in webkit [40445] by zecke@webkit.org
  • 3 edits in trunk/WebCore

Build fix for WX and maybe even Windows.

8:12 PM Changeset in webkit [40444] by zecke@webkit.org
  • 9 edits in trunk

Move Frame::sendResizeEvent and Frame::sendScrollEvent to EventHandler

Carry out the move and catch up in two call sites.

7:51 PM Changeset in webkit [40443] by zecke@webkit.org
  • 11 edits in trunk

Move Frame::isFrameSet to Document::isFrameSet

Changed the FrameLoader callsite. I assume that as this
point m_frame.document() might return a null pointer. Change
the condition to either not having a document or not having a
frame set. This should be semantically equivalent to the
old code.

Make Document::body() const to be able to call it from within
Document::isFrameSet. Leave the isHTMLDocument() check as this
is a stronger condition than simply having a HTMLElement in the
DOM.

7:51 PM Changeset in webkit [40442] by zecke@webkit.org
  • 2 edits in trunk/WebCore

[GTK] Logic fixes and build fixes

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

Catch up with the internal GeoLocation API and fix the logic of
the previous version.

7:51 PM Changeset in webkit [40441] by zecke@webkit.org
  • 6 edits in trunk

[GTK] Implement GeolocationService using the Geoclue library

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

Untested implementation of the GeolocationService using the geoclue
library. Velocity handling is completely missing and the accuracy
handling might be wrong.

7:48 PM Changeset in webkit [40440] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Build fix. Declare preferredSize as static.

  • platform/text/cf/StringImplCF.cpp:

(WebCore::StringWrapperCFAllocator::preferredSize):

7:11 PM Changeset in webkit [40439] by mrowe@apple.com
  • 2 edits in trunk/JavaScriptCore

<rdar://problem/6391501> Enable the JIT on Mac OS X x86_64 as it passes all tests.

Rubber-stamped by Oliver Hunt.

  • wtf/Platform.h:
6:31 PM Changeset in webkit [40438] by Darin Adler
  • 3 edits in trunk/WebCore

2009-01-30 Darin Adler <Darin Adler>

Reviewed by Mark Rowe.

Bug 23622: create CFString and NSString objects from WebCore::String without copying the characters
https://bugs.webkit.org/show_bug.cgi?id=23622

  • platform/text/cf/StringImplCF.cpp: (WebCore::StringWrapperCFAllocator::allocator): Added. Returns the allocator. (WebCore::StringWrapperCFAllocator::retain): Added. Callback for allocator. (WebCore::StringWrapperCFAllocator::release): Ditto. (WebCore::StringWrapperCFAllocator::copyDescription): Ditto. (WebCore::StringWrapperCFAllocator::allocate): Ditto. (WebCore::StringWrapperCFAllocator::reallocate): Ditto. (WebCore::StringWrapperCFAllocator::deallocate): Ditto. (WebCore::StringWrapperCFAllocator::preferredSize): Ditto. (WebCore::StringWrapperCFAllocator::create): Added. Creates the allocator, but returns 0 if garbage collection is enabled. (WebCore::StringImpl::createCFString): Use StringWrapperCFAllocator if possible.
  • platform/text/mac/StringImplMac.mm: (WebCore::StringImpl::operator NSString *): Use CFString and toll-free bridging, rather than using NSString directly. This lets NSString benefit from the above.
6:31 PM Changeset in webkit [40437] by Darin Adler
  • 2 edits in trunk/WebCore

2009-01-30 Darin Adler <Darin Adler>

  • page/Frame.cpp: (WebCore::Frame::createView): Fix debug build by removing get() from assert.
6:13 PM Changeset in webkit [40436] by ggaren@apple.com
  • 9 edits in trunk/WebKit

WebKit/gtk:

2009-01-30 Geoffrey Garen <ggaren@apple.com>

Build fix.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::createFrame):

WebKit/mac:

2009-01-30 Geoffrey Garen <ggaren@apple.com>

Build fix.

  • WebView/WebFramePrivate.h:

WebKit/qt:

2009-01-30 Geoffrey Garen <ggaren@apple.com>

Build fix.

  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::createFrame):

WebKit/win:

2009-01-30 Geoffrey Garen <ggaren@apple.com>

Build fix.

  • Interfaces/IWebFramePrivate.idl:
  • WebCoreSupport/WebFrameLoaderClient.cpp: (WebFrameLoaderClient::loadURLIntoChild):
5:57 PM Changeset in webkit [40435] by zecke@webkit.org
  • 14 edits
    1 delete in trunk

Kill FrameLoaderClient.cpp, move the code over to Frame::createView

FrameLoaderClient is supposed to be an interface, move the
to be shared code to Frame which is a controller and is
allowed to create a FrameView.

5:47 PM Changeset in webkit [40434] by Simon Fraser
  • 2 edits
    4 adds in trunk/WebCore

2009-01-30 Simon Fraser <Simon Fraser>

Reviewed by Dave Hyatt

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

New files to support accelerated compositing at the RenderLayer
level.

  • WebCore.xcodeproj/project.pbxproj: New files added to project, and run through the sort script.
  • rendering/RenderLayerBacking.cpp: Added.
  • rendering/RenderLayerBacking.h: Added. New object to store compositing-related data for a single RenderLayer.
  • rendering/RenderLayerCompositor.cpp: Added.
  • rendering/RenderLayerCompositor.h: Added. Per-RenderView controller object for compositing hierarchy maintenance.
5:42 PM Changeset in webkit [40433] by Simon Fraser
  • 2 edits
    9 adds in trunk/WebCore

2009-01-30 Simon Fraser <Simon Fraser>

Reviewed by Dave Hyatt

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

Add new files related to accelerated compositing.

5:28 PM Changeset in webkit [40432] by ggaren@apple.com
  • 16 edits in trunk

WebCore:

2009-01-30 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.

Split "lockHistory" into "lockHistory" and "lockBackForwardList" in
preparation for setting them differently during a redirect.

  • WebCore.base.exp:
  • bindings/js/JSHTMLFormElementCustom.cpp: (WebCore::JSHTMLFormElement::submit):
  • bindings/js/JSLocationCustom.cpp: (WebCore::navigateIfAllowed): (WebCore::JSLocation::setHref): (WebCore::JSLocation::setProtocol): (WebCore::JSLocation::setHost): (WebCore::JSLocation::setHostname): (WebCore::JSLocation::setPort): (WebCore::JSLocation::setPathname): (WebCore::JSLocation::setSearch): (WebCore::JSLocation::setHash): (WebCore::JSLocation::replace): (WebCore::JSLocation::assign):
  • html/HTMLAnchorElement.cpp: (WebCore::HTMLAnchorElement::defaultEventHandler):
  • html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::submit):
  • html/HTMLFormElement.h:
  • loader/FrameLoader.cpp: (WebCore::FormSubmission::FormSubmission): (WebCore::ScheduledRedirection::ScheduledRedirection): (WebCore::isBackForwardLoadType): (WebCore::FrameLoader::createWindow): (WebCore::FrameLoader::changeLocation): (WebCore::FrameLoader::urlSelected): (WebCore::FrameLoader::submitFormAgain): (WebCore::FrameLoader::submitForm): (WebCore::FrameLoader::restoreDocumentState): (WebCore::FrameLoader::scheduleHTTPRedirection): (WebCore::FrameLoader::scheduleLocationChange): (WebCore::FrameLoader::scheduleRefresh): (WebCore::FrameLoader::redirectionTimerFired): (WebCore::FrameLoader::loadURLIntoChildFrame): (WebCore::FrameLoader::startRedirectionTimer): (WebCore::FrameLoader::loadFrameRequestWithFormAndValues): (WebCore::FrameLoader::transitionToCommitted): (WebCore::FrameLoader::clientRedirected): (WebCore::FrameLoader::continueFragmentScrollAfterNavigationPolicy): (WebCore::FrameLoader::loadItem):
  • loader/FrameLoader.h:
  • loader/FrameLoaderTypes.h: (WebCore::):
  • page/ContextMenuController.cpp: (WebCore::ContextMenuController::contextMenuItemSelected):
  • svg/SVGAElement.cpp: (WebCore::SVGAElement::defaultEventHandler):

WebKit/mac:

2009-01-30 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.

Split "lockHistory" into "lockHistory" and "lockBackForwardList" in
preparation for setting them differently during a redirect.

  • WebView/WebPDFView.mm: (-[WebPDFView PDFViewWillClickOnLink:withURL:]):

WebKit/qt:

2009-01-30 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.

Split "lockHistory" into "lockHistory" and "lockBackForwardList" in
preparation for setting them differently during a redirect.

  • Api/qwebpage.cpp: (QWebPage::triggerAction):
4:42 PM Changeset in webkit [40431] by weinig@apple.com
  • 3 edits
    2 adds in trunk

WebCore:

2009-01-30 Sam Weinig <sam@webkit.org>

Reviewed by Dan Bernstein.

Fix for <rdar://problem/6545095>

ASSERTION FAILED: RenderBlock.h:519: !o
o->isRenderBlock()

Test: fast/block/float/crash-replaced-display-block.html

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::layoutBlockChildren): Move RenderBlock only code into isRenderBlock if-statement.

LayoutTests:

2009-01-30 Sam Weinig <sam@webkit.org>

Reviewed by Dan Bernstein.

Test for <rdar://problem/6545095>

ASSERTION FAILED: RenderBlock.h:519: !o
o->isRenderBlock()
  • fast/block/float/crash-replaced-display-block-expected.txt: Added.
  • fast/block/float/crash-replaced-display-block.html: Added.
3:46 PM Changeset in webkit [40430] by Chris Fleizach
  • 6 edits in trunk

Bug 23656: AX hierarchy for iFrames is incorrect
https://bugs.webkit.org/show_bug.cgi?id=23656

The AX hierarchy when an iFrame was present was incorrect. A different scroll area was returned when going down compared to when going up.

3:43 PM Changeset in webkit [40429] by treat@webkit.org
  • 2 edits in trunk/WebCore

2009-01-30 Adam Treat <adam.treat@torchmobile.com>

Fix Qt build to include new class IdentifierRep introduced in revision 40412.

  • WebCore.pro:
3:32 PM Changeset in webkit [40428] by andersca@apple.com
  • 5 edits in trunk/WebKit/mac

2009-01-30 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Fix <rdar://problem/6544048>


Have NetscapePluginInstanceProxy keep track of all the ProxyInstance objects associated.


When the plug-in instance is destroyed, invalidate all proxy instances.


  • Plugins/Hosted/NetscapePluginInstanceProxy.h:
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::destroy): (WebKit::NetscapePluginInstanceProxy::addInstance): (WebKit::NetscapePluginInstanceProxy::removeInstance):
  • Plugins/Hosted/ProxyInstance.h:
  • Plugins/Hosted/ProxyInstance.mm: (WebKit::ProxyInstance::ProxyInstance): (WebKit::ProxyInstance::~ProxyInstance): (WebKit::ProxyInstance::invalidate):
3:16 PM Changeset in webkit [40427] by andersca@apple.com
  • 3 edits in trunk/WebCore

2009-01-30 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Make IdentifierRep.h a private header, and add symbols needed by WebKit to WebCore.LP64.exp.

  • WebCore.LP64.exp:
  • WebCore.xcodeproj/project.pbxproj:
3:13 PM Changeset in webkit [40426] by zecke@webkit.org
  • 2 edits in trunk/WebCore

[Gtk+] Use AccessibilityObject::stringValue for the AtkObject name

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

The API documentation at http://library.gnome.org/devel/atk/stable/AtkText.html
states that: "AtkObjects whose text content is simple, unattributed, and very
brief may expose that content via atk_object_get_name instead"

As we are currently not able to always provide a AtkTextInterface
implementation this will make the whole content available to accerciser,
and other tools.

3:11 PM Changeset in webkit [40425] by weinig@apple.com
  • 8 edits in trunk/WebCore

2009-01-30 Sam Weinig <sam@webkit.org>

Commit files I forgot to in the previous commit.

  • page/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::layoutCount):
  • page/Frame.cpp: (WebCore::Frame::contentRenderer): (WebCore::Frame::forceLayoutWithPageWidthRange):
  • page/FrameView.cpp: (WebCore::FrameView::layout):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::nodeAtPoint):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::repaintUsingContainer): (WebCore::RenderObject::repaint): (WebCore::RenderObject::repaintRectangle): (WebCore::RenderObject::view): (WebCore::RenderObject::scheduleRelayout): (WebCore::RenderObject::maximalOutlineSize):
  • rendering/SVGRenderSupport.cpp: (WebCore::clampImageBufferSizeToViewport):
  • svg/SVGLength.cpp: (WebCore::SVGLength::PercentageOfViewport):
3:09 PM Changeset in webkit [40424] by ggaren@apple.com
  • 57 edits
    6 adds in trunk

WebCore:

2009-01-30 Geoffrey Garen <ggaren@apple.com>

Reviewed by Darin Adler.


More scaffolding for redirect tracking. No behavior change yet.

  • bindings/js/JSHTMLFormElementCustom.cpp: (WebCore::JSHTMLFormElement::submit): Added scaffolding necessary for redirect checking in a form submission through JS. (Currently, we just specify "no redirect.")
  • bindings/js/ScriptController.h:
  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::processingUserGesture): (WebCore::ScriptController::processingUserGestureEvent): (WebCore::ScriptController::isJavaScriptAnchorNavigation): Refactored user gesture check to make it more clear.

(WebCore::ScriptController::pageIsProcessingUserGesture): Added a helper
for checking if there's a user gesture in any frame on the page (a more
relaxed rule than our popup blocking rule, which requires the user
gesture to occur in the frame that's executing JavaScript.)

  • html/HTMLFormElement.h:
  • html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::submit): Added scaffolding necessary for redirect checking in a form submission through the DOM. (Currently, we just specify "no redirect.")
  • html/HTMLFormElement.idl: Made submit custom so it can participate in redirect checking.
  • loader/FrameLoader.h:
  • loader/FrameLoader.cpp: (WebCore::FormSubmission::FormSubmission): Clarified some names, and added a "lockHistory" parameter to form submissions, to match other navigations.

(WebCore::FrameLoader::submitFormAgain):
(WebCore::FrameLoader::submitForm): Use the lockHistory parameter.

(WebCore::FrameLoader::userGestureHint): Refactored this function for
clarity, and added a FIXME for a change I was too scared to make myself.
(Really, we should just get rid of this function.)

(WebCore::FrameLoader::loadFrameRequestWithFormAndValues): Refactored
to pass a FrameLoadType to form posts, too, so they can participate in
redirect tracking.

(WebCore::FrameLoader::loadPostRequest):
(WebCore::FrameLoader::continueFragmentScrollAfterNavigationPolicy):
(WebCore::FrameLoader::loadItem): Use the lockHistory parameter.

LayoutTests:

2009-01-30 Geoffrey Garen <ggaren@apple.com>

Reviewed by Darin Adler.

More redirect layout test work.


Added tests (expected to fail) for form submission via JavaScript:


  • http/tests/history/redirect-js-form-submit-0-seconds-expected.txt: Added.
  • http/tests/history/redirect-js-form-submit-0-seconds.html: Added.
  • http/tests/history/redirect-js-form-submit-2-seconds-expected.txt: Added.
  • http/tests/history/redirect-js-form-submit-2-seconds.html: Added.
  • http/tests/history/redirect-js-form-submit-before-load-expected.txt: Added.
  • http/tests/history/redirect-js-form-submit-before-load.html: Added.


Updated these tests to dump their back/forward lists, to check for
back/forward regressions, and to use # instead of ? to communicate between
URLs, since, to my surprise, form submissions can't use query strings.

  • http/tests/history/redirect-200-refresh-0-seconds-expected.txt:
  • http/tests/history/redirect-200-refresh-0-seconds.pl:
  • http/tests/history/redirect-200-refresh-2-seconds-expected.txt:
  • http/tests/history/redirect-200-refresh-2-seconds.pl:
  • http/tests/history/redirect-301-expected.txt:
  • http/tests/history/redirect-301.pl:
  • http/tests/history/redirect-302-expected.txt:
  • http/tests/history/redirect-302.pl:
  • http/tests/history/redirect-303-expected.txt:
  • http/tests/history/redirect-303.pl:
  • http/tests/history/redirect-307-expected.txt:
  • http/tests/history/redirect-307.pl:
  • http/tests/history/redirect-js-document-location-0-seconds-expected.txt:
  • http/tests/history/redirect-js-document-location-0-seconds.html:
  • http/tests/history/redirect-js-document-location-2-seconds-expected.txt:
  • http/tests/history/redirect-js-document-location-2-seconds.html:
  • http/tests/history/redirect-js-document-location-before-load-expected.txt:
  • http/tests/history/redirect-js-document-location-before-load.html:
  • http/tests/history/redirect-js-location-0-seconds-expected.txt:
  • http/tests/history/redirect-js-location-0-seconds.html:
  • http/tests/history/redirect-js-location-2-seconds-expected.txt:
  • http/tests/history/redirect-js-location-2-seconds.html:
  • http/tests/history/redirect-js-location-assign-0-seconds-expected.txt:
  • http/tests/history/redirect-js-location-assign-0-seconds.html:
  • http/tests/history/redirect-js-location-assign-2-seconds-expected.txt:
  • http/tests/history/redirect-js-location-assign-2-seconds.html:
  • http/tests/history/redirect-js-location-assign-before-load-expected.txt:
  • http/tests/history/redirect-js-location-assign-before-load.html:
  • http/tests/history/redirect-js-location-before-load-expected.txt:
  • http/tests/history/redirect-js-location-before-load.html:
  • http/tests/history/redirect-js-location-href-0-seconds-expected.txt:
  • http/tests/history/redirect-js-location-href-0-seconds.html:
  • http/tests/history/redirect-js-location-href-2-seconds-expected.txt:
  • http/tests/history/redirect-js-location-href-2-seconds.html:
  • http/tests/history/redirect-js-location-href-before-load-expected.txt:
  • http/tests/history/redirect-js-location-href-before-load.html:
  • http/tests/history/redirect-js-location-replace-0-seconds-expected.txt:
  • http/tests/history/redirect-js-location-replace-0-seconds.html:
  • http/tests/history/redirect-js-location-replace-2-seconds-expected.txt:
  • http/tests/history/redirect-js-location-replace-2-seconds.html:
  • http/tests/history/redirect-js-location-replace-before-load-expected.txt:
  • http/tests/history/redirect-js-location-replace-before-load.html:
  • http/tests/history/redirect-meta-refresh-0-seconds-expected.txt:
  • http/tests/history/redirect-meta-refresh-0-seconds.html:
  • http/tests/history/redirect-meta-refresh-2-seconds-expected.txt:
  • http/tests/history/redirect-meta-refresh-2-seconds.html:
  • http/tests/history/resources/redirect-target.html:
3:00 PM Changeset in webkit [40423] by pol@apple.com
  • 1 edit
    13 adds in trunk/LayoutTests

2009-01-30 Pierre-Olivier Latour <pol@apple.com>

Updated the references images for the media layout tests on Tiger since they now differ from Leopard after installing QuickTime 7.6.

  • platform/mac-tiger/media: Added.
  • platform/mac-tiger/media/video-aspect-ratio-expected.checksum: Added.
  • platform/mac-tiger/media/video-aspect-ratio-expected.png: Added.
  • platform/mac-tiger/media/video-controls-rendering-expected.checksum: Added.
  • platform/mac-tiger/media/video-controls-rendering-expected.png: Added.
  • platform/mac-tiger/media/video-display-toggle-expected.checksum: Added.
  • platform/mac-tiger/media/video-display-toggle-expected.png: Added.
  • platform/mac-tiger/media/video-layer-crash-expected.checksum: Added.
  • platform/mac-tiger/media/video-layer-crash-expected.png: Added.
  • platform/mac-tiger/media/video-transformed-expected.checksum: Added.
  • platform/mac-tiger/media/video-transformed-expected.png: Added.
  • platform/mac-tiger/media/video-zoom-expected.checksum: Added.
  • platform/mac-tiger/media/video-zoom-expected.png: Added.
2:57 PM Changeset in webkit [40422] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

2009-01-30 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Fix <rdar://problem/6490778>.


Change the NPRuntime related functions to use IdentifierRep directly, and make sure to always
validate IdentifierReps before dereferencing them.


  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WKPCEvaluate): (WKPCGetStringIdentifier): (WKPCGetIntIdentifier): (identifierFromIdentifierRep): (WKPCInvoke): (WKPCGetProperty): (WKPCSetProperty): (WKPCRemoveProperty): (WKPCHasProperty): (WKPCHasMethod): (WKPCIdentifierInfo):
2:42 PM Changeset in webkit [40421] by Dimitri Glazkov
  • 1 edit
    1 delete in trunk/LayoutTests

2009-01-30 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Eric Seidel.

Correct mis-applied http://trac.webkit.org/changeset/40413 by actually
removing the test file.

  • fast/dom/Window/timeout-released-on-close.html: Removed.
2:19 PM Changeset in webkit [40420] by zecke@webkit.org
  • 2 edits in trunk/WebCore

Build fix for Gtk+

2:16 PM Changeset in webkit [40419] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

Finally fix load() to propagate exceptions correctly.

Reviewed by Mark Rowe and Sam Weinig.

2:02 PM Changeset in webkit [40418] by Dimitri Glazkov
  • 2 edits in trunk/JavaScriptCore

2009-01-30 David Levin <levin@chromium.org>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=23618
Templated worker tasks should be more error proof to use.
Fix Chromium build.

  • wtf/TypeTraits.h: (WTF::IsConvertibleToInteger::IsConvertibleToDouble): Avoid "possible loss of data" warning when using Microsoft's C++ compiler by avoiding an implicit conversion of int types to doubles.
1:57 PM Changeset in webkit [40417] by weinig@apple.com
  • 6 edits in trunk/WebCore

2009-01-30 Sam Weinig <sam@webkit.org>

Reviewed by Dan Bernstein.

Add toRenderView methods for casting RenderObjects to RenderViews. The methods will assert if the object
is not a RenderView.

  • dom/Document.cpp: (WebCore::Document::renderView):
  • editing/SelectionController.cpp: (WebCore::SelectionController::nodeWillBeRemoved): (WebCore::SelectionController::recomputeCaretRect): (WebCore::SelectionController::invalidateCaretRect): (WebCore::SelectionController::focusedOrActiveStateChanged):
  • editing/mac/SelectionControllerMac.mm: (WebCore::SelectionController::notifyAccessibilityForSelectionChange):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::styleDidChange): (WebCore::RenderBox::repaintLayerRectsForImage): (WebCore::RenderBox::availableHeightUsing):
  • rendering/RenderView.h: (WebCore::toRenderView):
1:48 PM Changeset in webkit [40416] by Simon Fraser
  • 2 edits in trunk/WebCore

2009-01-30 Simon Fraser <Simon Fraser>

Reviewed by Dave Hyatt

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

Really use the container-relative repaint rect methods
for repaint during layout

  • rendering/RenderObject.h: (WebCore::RenderObject::LayoutRepainter::LayoutRepainter):
1:47 PM Changeset in webkit [40415] by andersca@apple.com
  • 6 edits in trunk/WebCore

2009-01-30 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler.

Get rid of PrivateIdentifier and use IdentifierRep instead.

  • bridge/NP_jsobject.cpp: (_NPN_Invoke): (_NPN_GetProperty): (_NPN_SetProperty): (_NPN_RemoveProperty): (_NPN_HasProperty): (_NPN_HasMethod):
  • bridge/c/c_instance.cpp: (JSC::Bindings::CInstance::getPropertyNames):
  • bridge/c/c_utility.cpp: (JSC::Bindings::convertNPStringToUTF16):
  • bridge/c/c_utility.h:
  • bridge/npruntime.cpp: (_NPN_GetStringIdentifier): (_NPN_GetStringIdentifiers): (_NPN_GetIntIdentifier): (_NPN_IdentifierIsString): (_NPN_UTF8FromIdentifier): (_NPN_IntFromIdentifier):
1:25 PM Changeset in webkit [40414] by andersca@apple.com
  • 3 edits in trunk/WebCore

2009-01-30 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig and Darin Adler.

Make IdentifierRep a real class, add necessary class methods.

  • bridge/IdentifierRep.cpp: (WebCore::identifierSet): Returns a set of all identifiers.


(WebCore::IdentifierRep::isValid):
Return whether an identifier is valid, meaning that it's present in the
set of identifiers.


  • bridge/IdentifierRep.h: (WebCore::IdentifierRep::isString): (WebCore::IdentifierRep::number): (WebCore::IdentifierRep::string): (WebCore::IdentifierRep::IdentifierRep): (WebCore::IdentifierRep::):
12:17 PM Changeset in webkit [40413] by treat@webkit.org
  • 2 edits
    1 copy in trunk/LayoutTests

2009-01-30 Dmitry Titov <dimich@chromium.org>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=23650
LayoutTests/fast/dom/Window/timeout-released-on-close.html fails intermittently.
Disable this test to keep buildbots green. Need more time to investigate the cause.

  • fast/dom/Window/timeout-released-on-close.html: Removed.
  • fast/dom/Window/timeout-released-on-close.html-disabled: Copied from LayoutTests/fast/dom/Window/timeout-released-on-close.html.
11:57 AM Changeset in webkit [40412] by andersca@apple.com
  • 7 edits
    2 adds in trunk/WebCore

2009-01-30 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Add IdentifierRep which will be used by both plug-in implementations.

  • GNUmakefile.am:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bridge/IdentifierRep.cpp: Added. (WebCore::IdentifierRep::IdentifierRep): (WebCore::IdentifierRep::): (WebCore::intIdentifierMap): (WebCore::identifierRep): (WebCore::stringIdentifierMap):
  • bridge/IdentifierRep.h: Added.
  • bridge/c/c_utility.h:
  • platform/text/PlatformString.h:
  • platform/text/String.cpp: (WebCore::String::fromUTF8WithLatin1Fallback):
11:55 AM Changeset in webkit [40411] by weinig@apple.com
  • 13 edits in trunk/WebCore

2009-01-30 Sam Weinig <sam@webkit.org>

Reviewed by David Hyatt.

Add toRenderInline methods for casting RenderObjects to RenderInlines. The methods will assert if the object
is not a RenderInline.

  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::removeLineBoxFromRenderObject): (WebCore::InlineFlowBox::extractLineBoxFromRenderObject): (WebCore::InlineFlowBox::attachLineBoxToRenderObject): (WebCore::InlineFlowBox::rendererLineBoxes): (WebCore::InlineFlowBox::determineSpacingForFlowBoxes): (WebCore::InlineFlowBox::paint):
  • rendering/LayoutState.cpp: (WebCore::LayoutState::LayoutState):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paintObject): (WebCore::RenderBlock::addFocusRingRects):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::offsetFromContainer): (WebCore::RenderBox::computeRectForRepaint): (WebCore::RenderBox::containingBlockWidthForPositioned): (WebCore::RenderBox::containingBlockHeightForPositioned): (WebCore::RenderBox::calcAbsoluteHorizontalValues): (WebCore::RenderBox::calcAbsoluteHorizontalReplaced):
  • rendering/RenderContainer.cpp: (WebCore::RenderContainer::updateBeforeAfterContentForContainer):
  • rendering/RenderInline.cpp: (WebCore::RenderInline::inlineContinuation): (WebCore::nextContinuation): (WebCore::RenderInline::splitInlines):
  • rendering/RenderInline.h: (WebCore::toRenderInline):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateLayerPosition): (WebCore::RenderLayer::boundingBox):
  • rendering/RenderLineBoxList.cpp: (WebCore::RenderLineBoxList::dirtyLinesFromChangedChild):
  • rendering/RenderText.h:
  • rendering/RenderTreeAsText.cpp: (WebCore::operator<<):
  • rendering/bidi.cpp: (WebCore::RenderBlock::createLineBoxes): (WebCore::RenderBlock::layoutInlineChildren):
10:20 AM Changeset in webkit [40410] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-01-30 Darin Fisher <darin@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=23647
Fix PLATFORM(SKIA)'s ImageSource::clear method to match other ports

  • platform/graphics/skia/ImageSourceSkia.cpp: (WebCore::ImageSource::clear):
10:16 AM Changeset in webkit [40409] by hyatt@apple.com
  • 18 edits in trunk/WebCore

2009-01-30 David Hyatt <hyatt@apple.com>

Add toRenderBlock methods for casting RenderObjects to RenderBlocks. The methods will assert if the object
is not a RenderBlock. Also add a toRenderBlock method that takes a RenderBlock but returns void and that
is unimplemented. This method will catch anyone trying to do a cast when the object is already a RenderBlock.

Making this change caught a bad cast in RenderBlock::layoutBlockChildren, so that is also fixed by this
patch with a containsFloats() check.

Reviewed by Darin Adler

  • dom/PositionIterator.cpp: (WebCore::PositionIterator::isCandidate):
  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::addBlockPlaceholderIfNeeded):
  • editing/Editor.cpp: (WebCore::Editor::hasBidiSelection):
  • page/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::anchorElement): (WebCore::AccessibilityRenderObject::accessibilityIsIgnored):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::styleWillChange): (WebCore::RenderBlock::removeChild): (WebCore::RenderBlock::layoutBlock): (WebCore::RenderBlock::collapseMargins): (WebCore::RenderBlock::clearFloatsIfNeeded): (WebCore::RenderBlock::layoutBlockChildren): (WebCore::RenderBlock::fillBlockSelectionGaps): (WebCore::RenderBlock::markAllDescendantsWithFloatsForLayout): (WebCore::RenderBlock::firstLineBlock): (WebCore::getLineAtIndex): (WebCore::getHeightForLineCount): (WebCore::RenderBlock::lineCount): (WebCore::RenderBlock::adjustForBorderFit): (WebCore::RenderBlock::clearTruncation):
  • rendering/RenderBlock.h: (WebCore::toRenderBlock):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::offsetFromContainer): (WebCore::RenderBox::computeRectForRepaint): (WebCore::RenderBox::calcReplacedHeightUsing):
  • rendering/RenderFlexibleBox.cpp: (WebCore::RenderFlexibleBox::layoutHorizontalBox): (WebCore::RenderFlexibleBox::layoutVerticalBox):
  • rendering/RenderInline.cpp: (WebCore::RenderInline::inlineContinuation): (WebCore::nextContinuation): (WebCore::RenderInline::positionForCoordinates):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateScrollInfoAfterLayout):
  • rendering/RenderListItem.cpp: (WebCore::getParentOfFirstLineBox): (WebCore::RenderListItem::positionListMarker):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::containingBlock): (WebCore::RenderObject::computeRectForRepaint): (WebCore::RenderObject::removeFromObjectLists): (WebCore::RenderObject::getTextDecorationColors):
  • rendering/RenderTable.cpp: (WebCore::RenderTable::addChild): (WebCore::RenderTable::recalcSections):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::styleDidChange):
  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::adjustControlHeightBasedOnLineHeight):
  • rendering/RootInlineBox.cpp: (WebCore::RootInlineBox::block):
  • rendering/bidi.cpp: (WebCore::RenderBlock::createLineBoxes):
9:30 AM Changeset in webkit [40408] by beidson@apple.com
  • 10 edits in trunk

WebCore:

2009-01-30 Brady Eidson <beidson@apple.com>

Reviewed by Sam Weinig

  • Let CachedFrame handle the suspending/resuming of active DOM objects itself instead of relying on some outside force.
  • Clear the previous history item inside FrameLoader instead of relying on multiple FrameLoaderClients to do it.
  • history/CachedFrame.cpp: (WebCore::CachedFrame::CachedFrame): Suspend active DOM objects here. (WebCore::CachedFrame::restore): Resume those DOM objects here.
  • history/CachedFrame.h: (WebCore::CachedFrame::domWindow):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::commitProvisionalLoad): Let CachedFrame creation handle the suspending of active DOM objects. (WebCore::FrameLoader::open): Let CachedFrame::restore() handle resuming those DOM objects. (WebCore::FrameLoader::checkLoadCompleteForThisFrame): Call frameLoadCompleted() instead of performing a client call by itself. (WebCore::FrameLoader::frameLoadCompleted):

WebKit/mac:

2009-01-30 Brady Eidson <beidson@apple.com>

Reviewed by Sam Weinig

Remove FrameLoaderClient code that is now handled by FrameLoader itself

  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::frameLoadCompleted):

WebKit/qt:

2009-01-30 Brady Eidson <beidson@apple.com>

Reviewed by Sam Weinig

Remove FrameLoaderClient code that is now handled by FrameLoader itself

  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::frameLoadCompleted):

WebKit/win:

2009-01-30 Brady Eidson <beidson@apple.com>

Reviewed by Sam Weinig

Remove FrameLoaderClient code that is now handled by FrameLoader itself

  • WebFrame.cpp: (WebFrame::frameLoadCompleted):
9:13 AM Changeset in webkit [40407] by ap@webkit.org
  • 1 edit
    3 adds in trunk/LayoutTests

Reviewed by Darin Adler.

Test that documents loaded from application cache cannot load local resources.

  • http/tests/appcache/local-content-expected.txt: Added.
  • http/tests/appcache/local-content.html: Added.
  • http/tests/appcache/resources/local-content.manifest: Added.
9:11 AM Changeset in webkit [40406] by ap@webkit.org
  • 4 edits in trunk/WebCore

Reviewed by Darin Adler.

<rdar://problem/6448823> Application Cache is kept after a private browsing session

  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::selectCache): (WebCore::ApplicationCacheGroup::update): Cache update will not be started if private browsing is enabled. Existing cache version from disk database will be used.
  • loader/appcache/ApplicationCacheGroup.h: Made postListenerTask() static, in order to call it when refusing to update from selectCache().
  • loader/appcache/ApplicationCache.cpp: (WebCore::ApplicationCache::addDynamicEntry): (WebCore::ApplicationCache::removeDynamicEntry): Added reminders about private browsing to unimplemented methods.
9:06 AM Changeset in webkit [40405] by darin@chromium.org
  • 3 edits in trunk/LayoutTests

2009-01-30 Dmitry Titov <dimich@chromium.org>

Reviewed by Alexey Proskuryakov.

https://bugs.webkit.org/show_bug.cgi?id=23615
Try to make this test reliable on buildbots.

  • fast/dom/Window/resources/long_timeout.html: Allocate more objects to make detection more reliable.
  • fast/dom/Window/timeout-released-on-close.html: Use threshold to account for objects allocated by the running scripts. Add debug output in case of failure to help understand possible failures.
7:17 AM Changeset in webkit [40404] by Simon Hausmann
  • 6 edits in trunk

2009-01-30 Laszlo Gombos <Laszlo Gombos>

Reviewed by Simon Hausmann.

Bug 23580: GNU mode RVCT compilation support
<https://bugs.webkit.org/show_bug.cgi?id=23580>

7:17 AM Changeset in webkit [40403] by Simon Hausmann
  • 3 edits in trunk/WebKit/qt

2009-01-30 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Tor Arne Vestbø.

Hide the Offline Web Application Cache path API from the public API
for now.

5:15 AM Changeset in webkit [40402] by ap@webkit.org
  • 2 edits in trunk/LayoutTests

Reviewed by Mark Rowe.

https://bugs.webkit.org/show_bug.cgi?id=23639
Test that appcache ignores Cache-Control: no-store

  • http/tests/resources/network-simulator.php: Send no-store with all responses.
5:13 AM Changeset in webkit [40401] by ap@webkit.org
  • 2 edits
    4 adds in trunk/LayoutTests

Reviewed by Mark Rowe.

https://bugs.webkit.org/show_bug.cgi?id=23640
Test that non-HTML main resources work with application cache correctly

  • http/tests/appcache/non-html-expected.txt: Added.
  • http/tests/appcache/non-html.xhtml: Added.
  • http/tests/appcache/resources/abe.png: Added.
  • http/tests/appcache/resources/non-html.manifest: Added.
  • http/tests/resources/network-simulator.php: Support more file extensions.
3:12 AM QtWebKitFeatures45 created by Simon Hausmann
2:48 AM QtWebKit edited by Simon Hausmann
(diff)
12:38 AM Changeset in webkit [40400] by ap@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-01-30 David Levin <levin@chromium.org>

Reviewed by Alexey Proskuryakov.

Bug 23618: Templated worker tasks should be more error proof to use
<https://bugs.webkit.org/show_bug.cgi?id=23618>

Add the type traits needed for the generic worker tasks
and compile asserts for them.

Add a summary header to the TypeTraits.h file to explain what is in there.

Add a note to explain IsPod's deficiencies.

  • wtf/TypeTraits.h:
12:34 AM Changeset in webkit [40399] by ap@webkit.org
  • 9 edits
    2 adds in trunk

2009-01-30 David Levin <levin@chromium.org>

Reviewed by Alexey Proskuryakov.

Bug 23616: Various "template helpers" should be consolidated from isolated files in JavaScriptCore.
<https://bugs.webkit.org/show_bug.cgi?id=23616>

  • wtf/TypeTraits.h: Moved RemovePointer, IsPod, IsInteger to this file.
  • wtf/OwnPtr.h: Use RemovePointer from TypeTraits.h.
  • wtf/RetainPtr.h: Ditto.
  • wtf/HashTraits.h: Use IsInteger from TypeTraits.h.
  • wtf/VectorTraits.h: Use IsPod from TypeTraits.h.
12:18 AM Changeset in webkit [40398] by ap@webkit.org
  • 3 edits in trunk/WebCore

2009-01-30 David Levin <levin@chromium.org>

Reviewed by Alexey Proskuryakov.

Bug 23599: ResourceError needs to be copyable for use in another thread.
<https://bugs.webkit.org/show_bug.cgi?id=23599>

Add ResourceErrorBase::copy method. (Not yet used.)

  • platform/network/ResourceErrorBase.cpp: (WebCore::ResourceErrorBase::copy):
  • platform/network/ResourceErrorBase.h:

Jan 29, 2009:

10:40 PM Changeset in webkit [40397] by Stephanie Lewis
  • 6 edits in trunk

2009-01-29 Stephanie Lewis <Stephanie Lewis>

RS by Oliver Hunt.

Update the order files.

  • WebCore.order:
  • WebKit.order:
7:09 PM Changeset in webkit [40396] by cwzwarich@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-01-29 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Oliver Hunt.

Bug 23551: Crash on page load with profiler enabled and running
<https://bugs.webkit.org/show_bug.cgi?id=23551>
<rdar://problem/6529521>

Interpreter::execute(FunctionBodyNode*, ...) calls Profiler::didExecute()
with a stale CallFrame. If some part of the scope chain has already been
freed, Profiler::didExecute() will crash when attempting to get the lexical
global object. The fix is to make the didExecute() call use the caller's
CallFrame, not the one made for the function call. In this case, the
willExecute() call should also be changed to match.

Since this occurs in the actual inspector JS, it is difficult to reduce.
I couldn't make a layout test.

  • interpreter/Interpreter.cpp: (JSC::Interpreter::execute):
6:54 PM Changeset in webkit [40395] by hyatt@apple.com
  • 1 edit in trunk/WebCore/WebCore.xcodeproj/project.pbxproj

Fix mac bustage.

6:35 PM Changeset in webkit [40394] by alice.liu@apple.com
  • 1 edit in trunk/WebKit/win/WebHistoryItem.cpp

Windows build fix

6:01 PM Changeset in webkit [40393] by eric@webkit.org
  • 12 edits
    1 copy
    1 add in trunk/WebCore

2009-01-29 Dmitry Titov <dimich@chromium.org>

Reviewed by Darin Adler.

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

Make TimerBase thread-aware (for Workers).
Added new class ThreadTimers - it keeps a heap of all timers for a thread (previously kept in a static global).
Pointer to instance of ThreadTimers is stored in GlobalThreadData.
Most static methods of TimerBase went to ThreadTimers.

The TimerBase functionality did not change (it maintains the timer heap which computes the nearest firing).

Added new abstract class SharedTimer, so worker threads can provide their own implementation (will be implemented on WorkerRunLoop).

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.scons:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl: Added new ThreadTimers.cpp to build files.
  • platform/SharedTimer.h: (WebCore::SharedTimer::~SharedTimer): (WebCore::MainThreadSharedTimer::setFiredFunction): (WebCore::MainThreadSharedTimer::setFireTime): (WebCore::MainThreadSharedTimer::stop): Added abstract SharedTimer and MainThreadSharedTimer which redirects to the existing port-supplied functions.
  • platform/ThreadGlobalData.cpp: (WebCore::ThreadGlobalData::ThreadGlobalData): (WebCore::ThreadGlobalData::~ThreadGlobalData):
  • platform/ThreadGlobalData.h: (WebCore::ThreadGlobalData::threadTimers): Added ThreadTimers to ThreadGlobalData.
  • platform/ThreadTimers.cpp: Added. (WebCore::mainThreadSharedTimer): Static getter for a MainThreadSharedTimer singleton.

(WebCore::ThreadTimers::ThreadTimers):
(WebCore::ThreadTimers::setSharedTimer):
(WebCore::ThreadTimers::updateSharedTimer):
(WebCore::ThreadTimers::collectFiringTimers):
(WebCore::ThreadTimers::fireTimers):
(WebCore::ThreadTimers::sharedTimerFired):
(WebCore::ThreadTimers::sharedTimerFiredInternal):
(WebCore::ThreadTimers::fireTimersInNestedEventLoop):
ThreadTimers implementation. Most of the code is moved from previous static functions on TimerBase.
Need a new class so each thread may get ts own copy of the timer heap.

  • platform/ThreadTimers.h: Added. (WebCore::ThreadTimers::timerHeap): (WebCore::ThreadTimers::timersReadyToFire):
  • platform/Timer.cpp: (WebCore::timerHeap): (WebCore::timersReadyToFire): (WebCore::TimerHeapElement::TimerHeapElement): (WebCore::TimerHeapElement::checkConsistency): (WebCore::TimerHeapElement::operator=): (WebCore::TimerHeapIterator::checkConsistency): (WebCore::TimerBase::TimerBase): (WebCore::TimerBase::~TimerBase): (WebCore::TimerBase::isActive): (WebCore::TimerBase::checkHeapIndex): (WebCore::TimerBase::heapDelete): (WebCore::TimerBase::heapDeleteMin): (WebCore::TimerBase::heapInsert): (WebCore::TimerBase::heapPopMin): (WebCore::TimerBase::setNextFireTime): (WebCore::TimerBase::fireTimersInNestedEventLoop): Now instead of static timer heap these use accessor functions that pull thread-specific heap instance.
  • platform/Timer.h: static methods moved to ThreadTimers.
5:56 PM Changeset in webkit [40392] by weinig@apple.com
  • 2 edits in trunk/WebKit/win

2009-01-29 Sam Weinig <sam@webkit.org>

Fix Windows build

  • WebHistoryItem.cpp: (WebHistoryItem::initFromDictionaryRepresentation):
5:42 PM Changeset in webkit [40391] by hyatt@apple.com
  • 4 edits
    1 add in trunk/WebCore

2009-01-29 David Hyatt <hyatt@apple.com>

Beginning of work to eliminate RenderContainer and make containership "pluggable" into any spot in the render tree. The first step is to create
the new object that will handle children: RenderObjectChildList and move RenderContainer's member variables into it. Subsequent patches will begin
moving RenderContainer's functionality into RenderObjectChildList.

Reviewed by Eric Seidel

  • WebCore.xcodeproj/project.pbxproj:
  • rendering/RenderContainer.cpp: (WebCore::RenderContainer::RenderContainer): (WebCore::RenderContainer::destroyLeftoverChildren): (WebCore::RenderContainer::addChild): (WebCore::RenderContainer::removeChildNode): (WebCore::RenderContainer::updateBeforeAfterContentForContainer): (WebCore::RenderContainer::appendChildNode): (WebCore::RenderContainer::insertChildNode): (WebCore::RenderContainer::layout): (WebCore::RenderContainer::removeLeftoverAnonymousBlock): (WebCore::RenderContainer::positionForCoordinates): (WebCore::RenderContainer::addLineBoxRects): (WebCore::RenderContainer::collectAbsoluteLineBoxQuads):
  • rendering/RenderContainer.h: (WebCore::RenderContainer::firstChild): (WebCore::RenderContainer::lastChild): (WebCore::RenderContainer::firstChildBox): (WebCore::RenderContainer::lastChildBox):
  • rendering/RenderObjectChildList.h: Added. (WebCore::RenderObjectChildList::RenderObjectChildList): (WebCore::RenderObjectChildList::firstChild): (WebCore::RenderObjectChildList::lastChild): (WebCore::RenderObjectChildList::setFirstChild): (WebCore::RenderObjectChildList::setLastChild):
5:37 PM Changeset in webkit [40390] by hyatt@apple.com
  • 1 edit in trunk/WebCore/rendering/RenderThemeSafari.cpp

Fix windows build bustage.

5:14 PM Changeset in webkit [40389] by hyatt@apple.com
  • 1 edit in trunk/WebCore/rendering/RenderThemeSafari.cpp

Fix windows build bustage.

4:48 PM Changeset in webkit [40388] by darin@chromium.org
  • 4 edits in trunk/LayoutTests

2009-01-29 Tony Chang <tony@chromium.org>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=22894
Fix typo in fill-stroke-clip-reset-path.html where
layoutTestController.setUseDashboardCompatiblityMode is called instead of
layoutTestController.setUseDashboardCompatibilityMode.

  • fast/canvas/fill-stroke-clip-reset-path.html:
  • platform/mac/fast/canvas/fill-stroke-clip-reset-path-expected.checksum:
  • platform/mac/fast/canvas/fill-stroke-clip-reset-path-expected.png:
4:45 PM Changeset in webkit [40387] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-01-29 Jungshik Shin <jshin@chromium.org>

Reviewed by Alexey Proskuryakov.

http://bugs.webkit.org/show_bug.cgi?id=23598
Replace smart-quotes in UTF-8 with ASCII double-quotes to avoid
the compilation error on CJK Windows

  • wml/WMLTableElement.cpp: (WebCore::WMLTableElement::parseMappedAttribute):
4:42 PM Changeset in webkit [40386] by darin@chromium.org
  • 3 edits in trunk/LayoutTests

2009-01-29 Hironori Bono <hbono@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=18835
Rebaseline a layout test that verifies the InsertLineBreakCommand() function
doesn't delete existing markers.

  • platform/mac/editing/spelling/spelling-linebreak-expected.checksum:
  • platform/mac/editing/spelling/spelling-linebreak-expected.png:
4:39 PM Changeset in webkit [40385] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-01-29 Darin Fisher <darin@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=23627
Fix the PLATFORM(SKIA) build.

  • svg/graphics/SVGPaintServer.cpp: (WebCore::SVGPaintServer::teardown):
4:34 PM Changeset in webkit [40384] by eric@webkit.org
  • 4 edits in trunk

Reviewed by Darin Adler.

Remove <strong>/<em> tags when toggling bold/italic since IE inserts them instead of <b>/<i>
https://bugs.webkit.org/show_bug.cgi?id=23486

Test: editing/execCommand/toggle-styles.html

  • editing/ApplyStyleCommand.cpp: (WebCore::ApplyStyleCommand::isHTMLStyleNode):
4:23 PM Changeset in webkit [40383] by darin@chromium.org
  • 6 edits in trunk/WebCore

2009-01-29 Scott Violet <sky@google.com>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=23625
Skia platform doesn't render text to a canvas or support clipping to an image buffer

Fixes two bugs in Skia rendering to a canvas:
. Text was not rendered at all. This is because we never properly

fixed up the alpha values. The fix is to create a layer when
rendering text to a layer.

. We were not honoring clipping to an image buffer.

  • platform/graphics/chromium/FontChromiumWin.cpp: (WebCore::Font::drawGlyphs):
  • platform/graphics/skia/GraphicsContextSkia.cpp: (WebCore::GraphicsContext::clipToImageBuffer):
  • platform/graphics/skia/ImageBufferSkia.cpp: (WebCore::ImageBuffer::ImageBuffer):
  • platform/graphics/skia/PlatformContextSkia.cpp: (PlatformContextSkia::PlatformContextSkia): (PlatformContextSkia::setDrawingToImageBuffer): (PlatformContextSkia::isDrawingToImageBuffer): (PlatformContextSkia::beginLayerClippedToImage): (PlatformContextSkia::restore): (PlatformContextSkia::applyClipFromImage):
  • platform/graphics/skia/PlatformContextSkia.h:
4:19 PM Changeset in webkit [40382] by weinig@apple.com
  • 7 edits in trunk/WebKit

WebKit/mac:

2009-01-29 Sam Weinig <sam@webkit.org>

Reviewed by Anders Carlsson.

Second step in tracking the urls a HistoryItem was redirected through
Add SPI to access the array of redirect urls associated with a HistoryItem.

  • History/WebHistoryItem.mm: (-[WebHistoryItem dictionaryRepresentation]): (-[WebHistoryItem _redirectURLs]):
  • History/WebHistoryItemPrivate.h:

WebKit/win:

2009-01-29 Sam Weinig <sam@webkit.org>

Reviewed by Anders Carlsson.

Second step in tracking the urls a HistoryItem was redirected through
Add SPI to access the array of redirect urls associated with a HistoryItem.

  • Interfaces/IWebHistoryItemPrivate.idl:
  • WebHistoryItem.cpp: (WebHistoryItem::dictionaryRepresentation): (WebHistoryItem::redirectURLs):
  • WebHistoryItem.h:
3:52 PM Changeset in webkit [40381] by andersca@apple.com
  • 3 edits in trunk/WebKit/mac

2009-01-29 Anders Carlsson <andersca@apple.com>

Reviewed by Dan Bernstein.

Always activate the plug-in host process if we're in "modal mode" and are being told to activate.


  • Plugins/Hosted/NetscapePluginHostProxy.h:
  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WebKit::NetscapePluginHostProxy::pluginHostDied): Call endModal here.


(WebKit::NetscapePluginHostProxy::applicationDidBecomeActive):
If we're modal, we should always bring the plug-in host process to the front.


(WebKit::NetscapePluginHostProxy::beginModal):
Add an observer for the NSApplicationWillBecomeActiveNotification callback.


(WebKit::NetscapePluginHostProxy::endModal):
Remove the observer.

3:48 PM Changeset in webkit [40380] by weinig@apple.com
  • 10 edits in trunk

WebCore:

2009-01-29 Sam Weinig <sam@webkit.org>

Reviewed by Mark Rowe.

First step in tracking the urls a HistoryItem was redirected through.

  • WebCore.base.exp:
  • history/HistoryItem.cpp: (WebCore::HistoryItem::HistoryItem): (WebCore::HistoryItem::addRedirectURL): (WebCore::HistoryItem::redirectURLs): (WebCore::HistoryItem::setRedirectURLs):
  • history/HistoryItem.h:

WebKit/mac:

2009-01-29 Sam Weinig <sam@webkit.org>

Reviewed by Mark Rowe.

First step in tracking the urls a HistoryItem was redirected through.

  • History/WebHistoryItem.mm: (-[WebHistoryItem initFromDictionaryRepresentation:]): (-[WebHistoryItem dictionaryRepresentation]):
  • Misc/WebNSDictionaryExtras.h:
  • Misc/WebNSDictionaryExtras.m: (-[NSDictionary _webkit_arrayForKey:]): Add helper.

WebKit/win:

2009-01-29 Sam Weinig <sam@webkit.org>

Reviewed by Mark Rowe.

First step in tracking the urls a HistoryItem was redirected through.

  • WebHistoryItem.cpp: (WebHistoryItem::initFromDictionaryRepresentation): (WebHistoryItem::dictionaryRepresentation):
3:43 PM Changeset in webkit [40379] by krit@webkit.org
  • 7 edits
    4 adds in trunk

2009-01-29 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann, Eric Seidel.

Just transform the pattern directly instead of transforming the context fixes this
problem. Texts or strokes are no longer affected by transformations.

SVG pattern transformation/BoundingBox can cause ugly stroke thickness or text positions
https://bugs.webkit.org/show_bug.cgi?id=23472

Test: svg/custom/pattern-with-transformation.svg

  • platform/graphics/Pattern.h: (WebCore::Pattern::setPatternSpaceTransform):
  • platform/graphics/cairo/PatternCairo.cpp: (WebCore::Pattern::createPlatformPattern):
  • platform/graphics/cg/PatternCG.cpp: (WebCore::Pattern::createPlatformPattern):
  • platform/graphics/qt/PatternQt.cpp: (WebCore::Pattern::createPlatformPattern):
  • svg/graphics/SVGPaintServerPattern.cpp: (WebCore::SVGPaintServerPattern::setup):

LayoutTest

Transformations on SVG patterns caused ugly drawings. The pattern is transformed
directly now. Texts or strokes are no longer affected by transformations.

  • platform/mac/svg/custom/pattern-with-transformation-expected.checksum: Added.
  • platform/mac/svg/custom/pattern-with-transformation-expected.png: Added.
  • platform/mac/svg/custom/pattern-with-transformation-expected.txt: Added.
  • svg/custom/pattern-with-transformation.svg: Added.
2:49 PM Changeset in webkit [40378] by hyatt@apple.com
  • 22 edits
    2 deletes in trunk/WebCore

2009-01-29 David Hyatt <hyatt@apple.com>

Move the line box list member out of RenderFlow and down into RenderBlock and RenderInline. Eliminate RenderFlow from the tree.

Reviewed by Sam Weinig

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • html/HTMLAnchorElement.cpp:
  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::rendererLineBoxes): (WebCore::InlineFlowBox::determineSpacingForFlowBoxes):
  • rendering/InlineFlowBox.h:
  • rendering/LayoutState.cpp: (WebCore::LayoutState::LayoutState):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::RenderBlock): (WebCore::RenderBlock::destroy): (WebCore::RenderBlock::styleWillChange): (WebCore::RenderBlock::styleDidChange): (WebCore::RenderBlock::dirtyLineBoxes): (WebCore::RenderBlock::removeChild): (WebCore::RenderBlock::setSelectionState): (WebCore::RenderBlock::avoidsFloats): (WebCore::RenderBlock::positionForCoordinates): (WebCore::RenderBlock::calcInlinePrefWidths): (WebCore::RenderBlock::baselinePosition): (WebCore::RenderBlock::getBaselineOfFirstLineBox): (WebCore::RenderBlock::getBaselineOfLastLineBox): (WebCore::RenderBlock::rectWithOutlineForRepaint): (WebCore::RenderBlock::hoverAncestor): (WebCore::RenderBlock::updateDragState):
  • rendering/RenderBlock.h: (WebCore::RenderBlock::lineBoxes): (WebCore::RenderBlock::firstLineBox): (WebCore::RenderBlock::lastLineBox):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::offsetFromContainer): (WebCore::RenderBox::computeRectForRepaint): (WebCore::RenderBox::containingBlockWidthForPositioned): (WebCore::RenderBox::calcAbsoluteHorizontalValues): (WebCore::RenderBox::calcAbsoluteHorizontalReplaced):
  • rendering/RenderBox.h:
  • rendering/RenderFlow.cpp: Removed.
  • rendering/RenderFlow.h: Removed.
  • rendering/RenderInline.cpp: (WebCore::RenderInline::RenderInline): (WebCore::RenderInline::destroy): (WebCore::RenderInline::styleDidChange): (WebCore::RenderInline::addChildIgnoringContinuation): (WebCore::RenderInline::cloneInline): (WebCore::RenderInline::splitInlines): (WebCore::RenderInline::offsetLeft): (WebCore::RenderInline::offsetTop): (WebCore::RenderInline::positionForCoordinates): (WebCore::RenderInline::rectWithOutlineForRepaint): (WebCore::RenderInline::updateDragState): (WebCore::RenderInline::relativePositionedInlineOffset):
  • rendering/RenderInline.h: (WebCore::RenderInline::lineBoxes): (WebCore::RenderInline::firstLineBox): (WebCore::RenderInline::lastLineBox):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateLayerPosition):
  • rendering/RenderLineBoxList.cpp: (WebCore::RenderLineBoxList::dirtyLinesFromChangedChild):
  • rendering/RootInlineBox.cpp: (WebCore::RootInlineBox::rendererLineBoxes):
  • rendering/RootInlineBox.h:
  • rendering/bidi.cpp: (WebCore::RenderBlock::createLineBoxes):
  • wml/WMLAElement.cpp:
2:38 PM Changeset in webkit [40377] by andersca@apple.com
  • 6 edits in trunk/WebKit/mac

2009-01-29 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Pass the PSN of the client to the host, and get the PSN of the host back when checking in.


  • Plugins/Hosted/NetscapePluginHostManager.h:
  • Plugins/Hosted/NetscapePluginHostManager.mm: (WebKit::NetscapePluginHostManager::hostForPackage): Get the current PSN and pass it to spawnPluginHost.


(WebKit::NetscapePluginHostManager::spawnPluginHost):
Pass the PSN to the "check in" function.


  • Plugins/Hosted/NetscapePluginHostProxy.h:
  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WebKit::NetscapePluginHostProxy::NetscapePluginHostProxy):


(WebKit::NetscapePluginHostProxy::pluginHostDied):
Fix a bug noticed by Julien Chaffraix. Call endModal if necessary.


(WebKit::NetscapePluginHostProxy::beginModal):
(WebKit::NetscapePluginHostProxy::endModal):
(WebKit::NetscapePluginHostProxy::setModal):
Split out the code that does all of the work into beginModal and endModal methods.


  • Plugins/Hosted/WebKitPluginHost.defs:
  • Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView loadStream]):
2:05 PM Changeset in webkit [40376] by ggaren@apple.com
  • 2 edits in trunk/WebCore

2009-01-29 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.

Cleaned up some naming in ScheduledRedirection.

  • loader/FrameLoader.cpp: (WebCore::ScheduledRedirection::ScheduledRedirection):
2:03 PM Changeset in webkit [40375] by andersca@apple.com
  • 2 edits in trunk/WebCore

2009-01-29 Anders Carlsson <andersca@apple.com>

Reviewed by Dan Bernstein.

Update to use new API.

  • platform/graphics/mac/FontCustomPlatformData.cpp: (WebCore::FontCustomPlatformData::~FontCustomPlatformData): (WebCore::createFontCustomPlatformData):
1:53 PM Changeset in webkit [40374] by ap@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by Nikolas Zimmermann.

https://bugs.webkit.org/show_bug.cgi?id=23619
ASSERT(!error.isNull()) in DocumentLoader::mainReceivedError() when an SVG image load
is cancelled

Covered by existing tests (as a random crash).

  • loader/EmptyClients.h: (WebCore::EmptyFrameLoaderClient::cancelledError): (WebCore::EmptyFrameLoaderClient::blockedError): (WebCore::EmptyFrameLoaderClient::cannotShowURLError): (WebCore::EmptyFrameLoaderClient::interruptForPolicyChangeError): (WebCore::EmptyFrameLoaderClient::cannotShowMIMETypeError): (WebCore::EmptyFrameLoaderClient::fileDoesNotExistError): (WebCore::EmptyFrameLoaderClient::pluginWillHandleLoadError): Create non-null errors.
1:48 PM Changeset in webkit [40373] by ggaren@apple.com
  • 2 edits in trunk/WebCore

2009-01-29 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.

Removed an unused case of "lockHistory" to help make the other cases
clearer.

  • page/FrameLoadRequest.h: (WebCore::FrameLoadRequest::FrameLoadRequest):
1:39 PM Changeset in webkit [40372] by hyatt@apple.com
  • 8 edits in trunk/WebCore

2009-01-29 David Hyatt <hyatt@apple.com>

Move all of the contination insertion code from RenderFlow to RenderInline. addChildToFlow is renamed to addChildIgnoringContinuation, and it has been moved to
RenderContainer so that it can still be called on either blocks or inlines. The base class implementation in RenderContainer is what RenderBlock uses. RenderInline
subclasses it to do all the work it used to do in addChildToFlow.


RenderBlock's old addChildToFlow can just become addChild. This simplification is possible because addChild was actually never being called on anonymous
block continuations.


The code dealing with anonymous table parts in the old addChildWithContinuation method can now be removed as a result with no harmful side effects. Falling through to the
base class and ignoring a block continuation makes the right thing happen.


addChildWithContinuation moved to RenderInline and has been renamed to addChildToContinuation.


Reviewed by Eric Seidel

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::addChild):
  • rendering/RenderBlock.h:
  • rendering/RenderContainer.h: (WebCore::RenderContainer::addChildIgnoringContinuation):
  • rendering/RenderFlow.cpp:
  • rendering/RenderFlow.h:
  • rendering/RenderInline.cpp: (WebCore::RenderInline::styleDidChange): (WebCore::RenderInline::addChild): (WebCore::nextContinuation): (WebCore::RenderInline::continuationBefore): (WebCore::RenderInline::addChildIgnoringContinuation): (WebCore::RenderInline::splitInlines): (WebCore::RenderInline::splitFlow): (WebCore::RenderInline::addChildToContinuation): (WebCore::RenderInline::childBecameNonInline):
  • rendering/RenderInline.h: (WebCore::RenderInline::continuation): (WebCore::RenderInline::setContinuation):
1:37 PM Changeset in webkit [40371] by beidson@apple.com
  • 3 edits
    3 adds in trunk

WebCore:

2009-01-29 Brady Eidson <beidson@apple.com>

Reviewed by Oliver Hunt

<rdar://problem/6337157> hyundaiusa.com closes window or tab during load

Test: fast/loader/subframe-self-close.html

  • page/DOMWindow.cpp: (WebCore::DOMWindow::close): Only allow mainframes to close the window.

LayoutTests:

2009-01-29 Brady Eidson <beidson@apple.com>

Reviewed by Oliver Hunt

<rdar://problem/6337157> hyundaiusa.com closes window or tab during load

  • fast/loader/resources/subframe-self-close.html: Added.
  • fast/loader/subframe-self-close-expected.txt: Added.
  • fast/loader/subframe-self-close.html: Added.
1:23 PM Changeset in webkit [40370] by Nikolas Zimmermann
  • 23 edits in trunk/WebCore

Reviewed by David Hyatt.

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

Remove several virtual functions from Node & Element related to form control / input elements,
that shouldn't reside there, but move to approriate locations, in the new form control /
input element abstract base classes.

Remove the isControl() method from Node, as it's redundant - isFormControlElement() provides the same information.
Move isEnabled( / isReadOnlyControl() / isTextControl() from Node to FormControlElement.
Move isAutofilled() / isChecked() / isIndeterminate() from Node to InputElement.
Move isInputTypeHidden() / isPasswordField() from Element to InputElement.

Querying these methods requires using the toInputElement/toFormControlElement casting helper functions
to cast Element pointers to InputElement/FormControlElement pointers.

1:19 PM Changeset in webkit [40369] by Nikolas Zimmermann
  • 2 edits in trunk/WebCore

Not reviewed. Fix WML build - apply same fix as HTMLAnchorElement received.

12:57 PM BuildingQtOnLinux edited by ben@meyerhome.net
(diff)
12:40 PM Changeset in webkit [40368] by ap@webkit.org
  • 6 edits
    7 adds in trunk

Reviewed by Anders Carlsson.

https://bugs.webkit.org/show_bug.cgi?id=23620
Crash in appcache/resource-redirect.html on Tiger

There was a bug in the test case - it didn't send a Location header, and on Tiger, this
resulted in a failure callback being sent. It turned out that proper redirects weren't
caught at all.

Tests: http/tests/appcache/manifest-redirect-2.html

http/tests/appcache/resource-redirect-2.html

  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::didFail): Fixed this function to not crash if didReceiveResponse() wasn't called for this request. (WebCore::ApplicationCacheGroup::didReceiveResponse): Fixed to check for redirects correctly. (WebCore::ApplicationCacheGroup::didReceiveManifestResponse): Ditto.
11:36 AM Changeset in webkit [40367] by hyatt@apple.com
  • 7 edits in trunk/WebCore

2009-01-29 David Hyatt <hyatt@apple.com>

Move dirtyLinesFromChangedChild into RenderLineBoxList so that it can be shared by RenderBlock and RenderInline.


Reviewed by Sam Weinig

  • rendering/RenderBlock.h: (WebCore::RenderBlock::dirtyLinesFromChangedChild):
  • rendering/RenderFlow.cpp:
  • rendering/RenderFlow.h:
  • rendering/RenderInline.h: (WebCore::RenderInline::dirtyLinesFromChangedChild):
  • rendering/RenderLineBoxList.cpp: (WebCore::RenderLineBoxList::dirtyLinesFromChangedChild):
  • rendering/RenderLineBoxList.h:
11:21 AM Changeset in webkit [40366] by Adam Roben
  • 11 edits in trunk

Fix Bug 23623: Windowed Flash instances aren't captured when a WebView receives a WM_PRINTCLIENT message

WebCore:

Fix Bug 23623: Windowed Flash instances aren't captured when a WebView
receives a WM_PRINTCLIENT message

<https://bugs.webkit.org/show_bug.cgi?id=23623>
<rdar://problem/6513921>
<rdar://problem/6536874>

Flash doesn't seem to respond to WM_PRINTCLIENT, so we change
WM_PRINTCLIENT messages into WM_PAINT messages and hook into the
BeginPaint/EndPaint APIs to return the HDC we received from
WM_PRINTCLIENT. This causes the plugin to paint into the
WM_PRINTCLIENT HDC via its normal WM_PAINT handler. This technique
came from <http://www.fengyuan.com/article/wmprint.html>.

No test possible.

Reviewed by Darin Adler.

  • platform/graphics/GraphicsContext.h: Added [set]ShouldIncludeChildWindows on Windows.
  • platform/graphics/cg/GraphicsContextPlatformPrivateCG.h: Added m_shouldIncludeChildWindows. (WebCore::GraphicsContextPlatformPrivate::GraphicsContextPlatformPrivate): Initialize the new member.
  • platform/graphics/win/GraphicsContextCGWin.cpp: (WebCore::GraphicsContext::setShouldIncludeChildWindows): (WebCore::GraphicsContext::shouldIncludeChildWindows): Added simple setter and getter.
  • plugins/PluginView.cpp: (WebCore::PluginView::PluginView): Initialize the new member.
  • plugins/PluginView.h: Added m_wmPrintHDC and various functions required for making WM_PRINTCLIENT work.
  • plugins/win/PluginViewWin.cpp: (WebCore::PluginView::hookedBeginPaint): (WebCore::PluginView::hookedEndPaint): (WebCore::hook): (WebCore::setUpOffscreenPaintingHooks): Added. Code was modified from code available on fengyuan.com.

(WebCore::PluginView::wndProc): Turn WM_PRINTCLIENT messages into
WM_PAINT messages and save off the HDC that we're supposed to paint
into.

(WebCore::PluginView::paintWindowedPluginIntoContext): Added. Gets an
HDC from the GraphicsContext, translates it into client coordinates,
gets the plugin to draw into the HDC by sending a WM_PRINTCLIENT
message, and returns the HDC to the GraphicsContext.

(WebCore::PluginView::paint): If we're trying to paint a windowed
plugin into the GraphicsContext, call paintWindowedPluginIntoContext.

(WebCore::PluginView::init): Call setUpOffscreenPaintingHooks to make
our WM_PRINTCLIENT trick work.

WebKit/win:

Fix Bug 23623: Windowed Flash instances aren't captured when a WebView
receives a WM_PRINTCLIENT message

<https://bugs.webkit.org/show_bug.cgi?id=23623>
<rdar://problem/6513921>
<rdar://problem/6536874>

Reviewed by Darin Adler.

  • WebFrame.cpp: (WebFrame::paintDocumentRectToContext): (WebFrame::spoolPages): Call GraphicsContext::setShouldIncludeChildWindows so that child windows will be painted into the GraphicsContext rather than painted directly to the screen.
  • WebView.cpp: (WebView::updateBackingStore): Added a windowsToPaint parameter, which we pass along to paintIntoBackingStore.

(WebView::paint): Tell updateBackingStore to paint child windows if
we're not painting to the screen.

(WebView::paintIntoBackingStore): Added a windowsToPaint paramter,
which we use to tell our GraphicsContext whether or not to include
child windows.

  • WebView.h: Added windowsToPaint parameters to paintIntoBackingStore and updateBackingStore (which I also made private).
11:20 AM Changeset in webkit [40365] by Adam Roben
  • 2 edits in trunk/WebKit/win

Build fix after r40353

  • WebCoreSupport/WebFrameLoaderClient.cpp: (WebFrameLoaderClient::loadURLIntoChild): Removed the check for FrameLoadTypeReloadAllowingStaleData.
10:00 AM Changeset in webkit [40364] by hyatt@apple.com
  • 5 edits in trunk/WebCore

2009-01-29 David Hyatt <hyatt@apple.com>

Move RenderFlow::destroy down into RenderBlock and RenderInline.

Reviewed by Anders Carlsson

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::destroy):
  • rendering/RenderFlow.cpp:
  • rendering/RenderFlow.h:
  • rendering/RenderInline.cpp: (WebCore::RenderInline::destroy):
7:46 AM Changeset in webkit [40363] by ddkilzer@apple.com
  • 18 edits in trunk

Remove semi-colons from the end of ObjC method implementations

Rubber-stamped by Adam Roben.

WebCore:

$ find WebCore -name \*.m -o -name \*.mm -exec perl -e 'undef $/; $s = <>; while ($s =~ m/[\n\r][-+].*;[\s\r\n]+\{/g) { print "$ARGV: $&\n"; }' {} \;

  • bridge/testbindings.mm:

WebKit/mac:

$ find WebKit -name \*.m -o -name \*.mm -exec perl -e 'undef $/; $s = <>; while ($s =~ m/[\n\r][-+].*;[\s\r\n]+\{/g) { print "$ARGV: $&\n"; }' {} \;

  • DefaultDelegates/WebDefaultUIDelegate.m: (-[WebDefaultUIDelegate webView:setResizable:]): (-[WebDefaultUIDelegate webView:dragDestinationActionMaskForDraggingInfo:]): (-[WebDefaultUIDelegate webView:dragSourceActionMaskForPoint:]): (-[WebDefaultUIDelegate webView:willPerformDragSourceAction:fromPoint:withPasteboard:]):
  • History/WebBackForwardList.mm: (-[WebBackForwardList addItem:]): (-[WebBackForwardList backListWithLimit:]): (-[WebBackForwardList forwardListWithLimit:]):
  • History/WebHistoryItem.mm: (-[WebHistoryItem alternateTitle]): (-[WebHistoryItem setViewState:]):
  • Misc/WebCoreStatistics.mm: (+[WebCoreStatistics garbageCollectJavaScriptObjectsOnAlternateThreadForDebugging:]):
  • Misc/WebKitNSStringExtras.m: (-[NSString _web_drawAtPoint:font:textColor:]):
  • Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView setAttributeKeys:andValues:]):
  • WebCoreSupport/WebEditorClient.mm: (-[WebEditCommand command]):
  • WebView/WebFrame.mm: (-[WebFrame _getVisibleRect:]):
  • WebView/WebHTMLRepresentation.mm: (-[WebHTMLRepresentation _redirectDataToManualLoader:forPluginView:]):
  • WebView/WebHTMLView.mm: (-[WebHTMLView elementAtPoint:allowShadowContent:]):
  • WebView/WebPreferences.mm: (-[WebPreferences setAllowsAnimatedImages:]): (-[WebPreferences setAutosaves:]): (-[WebPreferences PDFDisplayMode]):
  • WebView/WebView.mm: (+[WebView _viewClass:andRepresentationClass:forMIMEType:]): (-[WebView _viewClass:andRepresentationClass:forMIMEType:]): (+[WebView _unregisterViewClassAndRepresentationClassForMIMEType:]): (+[WebView _registerViewClass:representationClass:forURLScheme:]): (-[WebView _shouldChangeSelectedDOMRange:toDOMRange:affinity:stillSelecting:]): (-[WebView _insertNewlineInQuotedContent]):

WebKitTools:

$ find WebKitTools -name \*.m -o -name \*.mm -exec perl -e 'undef $/; $s = <>; while ($s =~ m/[\n\r][-+].*;[\s\r\n]+\{/g) { print "$ARGV: $&\n"; }' {} \;

  • DumpRenderTree/mac/FrameLoadDelegate.mm: (-[FrameLoadDelegate webView:didFailLoadWithError:forFrame:]): (-[FrameLoadDelegate webView:windowScriptObjectAvailable:]): (-[FrameLoadDelegate webView:didFinishDocumentLoadForFrame:]): (-[FrameLoadDelegate webView:didHandleOnloadEventsForFrame:]):
  • DumpRenderTree/mac/UIDelegate.mm: (-[UIDelegate webViewFrame:]):
6:35 AM Changeset in webkit [40362] by treat@webkit.org
  • 2 edits in trunk/WebCore

2009-01-29 Adam Treat <adam.treat@torchmobile.com>

Reviewed by Alexey Proskuryakov.

https://bugs.webkit.org/show_bug.cgi?id=23609
Use the two-arg constructor for implicit QUrl->KURL conversion as the
single argument ctors assume that KURL::parse has already been called
and assert if the parse results in a different string. This was causing
an assert for QUrl's with no path and without a trailing slash.

  • platform/qt/KURLQt.cpp: (WebCore::KURL::KURL):
5:09 AM Changeset in webkit [40361] by ddkilzer@apple.com
  • 3 edits in trunk/WebKit/wx

Build fix for Wx: Finish de-RefCount-ing FrameLoaderClientWx

  • WebKitSupport/FrameLoaderClientWx.cpp: (WebCore::FrameLoaderClientWx::ref): Removed. (WebCore::FrameLoaderClientWx::deref): Removed.
  • WebKitSupport/FrameLoaderClientWx.h: Ditto.
4:48 AM Changeset in webkit [40360] by ap@webkit.org
  • 5 edits in trunk/WebCore

Rubber-stamped by Darin Adler.

Renamed ApplicationCacheResource::Implicit to Master, because that's the word HTML5 uses,
and it describes the meaning better.

1:58 AM Changeset in webkit [40359] by ap@webkit.org
  • 3 edits in trunk/LayoutTests

Not reviewed (this was just a merge problem).

https://bugs.webkit.org/show_bug.cgi?id=23476
HTTP redirect should make application cache updating fail

  • http/tests/appcache/manifest-redirect.html:
  • http/tests/appcache/resource-redirect.html: Updated the tests to work with the new appcache event sending behavior.
1:27 AM Changeset in webkit [40358] by ap@webkit.org
  • 2 edits in trunk/WebCore

Release build fix.

  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::selectCache): Got rid of a variable that wasn't used in release builds.
12:50 AM Changeset in webkit [40357] by hyatt@apple.com
  • 4 edits in trunk/WebCore

2009-01-29 David Hyatt <hyatt@apple.com>

Get rid of createAnonymousFlow. It was only called in one spot, so just inline the code.

Reviewed by Oliver Hunt

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::updateFirstLetter):
  • rendering/RenderFlow.cpp:
  • rendering/RenderFlow.h:
12:35 AM Changeset in webkit [40356] by ap@webkit.org
  • 3 edits
    7 adds in trunk

Reviewed by Anders Carlsson.

https://bugs.webkit.org/show_bug.cgi?id=23476
HTTP redirect should make application cache updating fail

Tests: http/tests/appcache/manifest-redirect.html

http/tests/appcache/resource-redirect.html

  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::didReceiveResponse): (WebCore::ApplicationCacheGroup::didReceiveManifestResponse): Fail if response code isn't 2xx.
12:32 AM Changeset in webkit [40355] by ap@webkit.org
  • 6 edits
    3 adds in trunk

Reviewed by Anders Carlsson.

https://bugs.webkit.org/show_bug.cgi?id=23592
Update appcache manifest signature parsing

Test: http/tests/appcache/wrong-signature-2.html

WebCore:

  • loader/appcache/ManifestParser.cpp: (WebCore::parseManifest): Allow comments on signature line. Removed code that replaced nulls with 0xFFFD characters, because there is no such requirement in the spec.

LayoutTests:

  • http/tests/appcache/manifest-parsing.html: Fixed to point to a correct manifest (oops).
  • http/tests/appcache/resources/manifest-parsing.manifest: Added a comment at the end of signature line.
  • http/tests/appcache/resources/wrong-signature-2.manifest: Added.
  • http/tests/appcache/wrong-signature-2-expected.txt: Added.
  • http/tests/appcache/wrong-signature-2.html: Added.
  • http/tests/appcache/wrong-signature.html: Test that there must be whitespace before the comment.
12:26 AM Changeset in webkit [40354] by ap@webkit.org
  • 24 edits in trunk

Reviewed by Anders Carlsson.

https://bugs.webkit.org/show_bug.cgi?id=23584
Update application cache code for the latest round of spec changes

The changes mostly affect the update process - events are sent at a different
time, and caches are associated with documents earlier.

WebCore:

  • loader/DocumentLoader.cpp: (WebCore::DocumentLoader::mainReceivedError): Changed an always passing test to an assertion. (WebCore::DocumentLoader::shouldLoadResourceFromApplicationCache): Caches are now associated with DocumentLoader before being complete, but they only affect loading after being completed. (WebCore::DocumentLoader::getApplicationCacheFallbackResource): Ditto.
  • loader/appcache/ApplicationCache.cpp: (WebCore::ApplicationCache::setGroup): Allow setting the group to the same one, to simplify ApplicationCacheGroup code. (WebCore::ApplicationCache::isComplete): A new method that tells whether the cache is complete, as defined in HTML5.
  • loader/appcache/ApplicationCache.h: Added isComplete().
  • loader/appcache/ApplicationCacheGroup.h: (WebCore::ApplicationCacheUpdateOption): Per HTML5, cache updating should work differently when invoked with or without a browsing context (Frame). A Frame is currently always needed for updating in WebKit, so a new argument to update() tells whether we should pretend that there isn't one. (WebCore::ApplicationCacheGroup::cacheIsBeingUpdated): A helper for ApplicationCache::isComplete(). (WebCore::ApplicationCacheGroup::CompletionType): Cache update algorithm now waits for ne main resources to finish loading in more cases. A member variable of this type is used to distinguish between different cases when waiting. (WebCore::ApplicationCacheGroup::m_cacheCandidates): The new algorithm doesn't track cache candidates. Caches are associated with documents much earlier, but the ones whose main resources have not been loaded from a cache are being tracked in a pending master entry list.
  • loader/appcache/DOMApplicationCache.cpp: (WebCore::DOMApplicationCache::update): An explicit call to update() invokes the algorithm without a browsing context.
  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::ApplicationCacheGroup): (WebCore::ApplicationCacheGroup::~ApplicationCacheGroup): (WebCore::ApplicationCacheGroup::selectCache): (WebCore::ApplicationCacheGroup::selectCacheWithoutManifestURL): (WebCore::ApplicationCacheGroup::finishedLoadingMainResource): (WebCore::ApplicationCacheGroup::failedLoadingMainResource): (WebCore::ApplicationCacheGroup::stopLoading): (WebCore::ApplicationCacheGroup::disassociateDocumentLoader): (WebCore::ApplicationCacheGroup::cacheDestroyed): (WebCore::ApplicationCacheGroup::setNewestCache): (WebCore::ApplicationCacheGroup::update): (WebCore::ApplicationCacheGroup::didReceiveResponse): (WebCore::ApplicationCacheGroup::didFinishLoading): (WebCore::ApplicationCacheGroup::didFail): (WebCore::ApplicationCacheGroup::didFinishLoadingManifest): (WebCore::ApplicationCacheGroup::cacheUpdateFailed): (WebCore::ApplicationCacheGroup::manifestNotFound): (WebCore::ApplicationCacheGroup::checkIfLoadIsComplete): (WebCore::ApplicationCacheGroup::startLoadingEntry): (WebCore::ApplicationCacheGroup::deliverDelayedMainResources): (WebCore::ApplicationCacheGroup::addEntry): (WebCore::ApplicationCacheGroup::associateDocumentLoaderWithCache): (WebCore::CallCacheListenerTask::create): (WebCore::CallCacheListenerTask::performTask): (WebCore::CallCacheListenerTask::CallCacheListenerTask): (WebCore::ApplicationCacheGroup::postListenerTask): Rewrote the update algorithm.

LayoutTests:

  • http/tests/appcache/404-manifest.html:
  • http/tests/appcache/404-resource.html: When initial cache attempt fails, the applicationCache singleton now gets an error event.
  • http/tests/appcache/foreign-iframe-main-expected.txt: Checking, downloading and progress events are now dispatched during initial cache attempt.
  • http/tests/appcache/idempotent-update-expected.txt:
  • http/tests/appcache/idempotent-update.html: Events are now posted asynchronously, so we cannot assume any status when handling an event. Rewrote the test to no longer rely on synchronous event dispatch.
  • http/tests/appcache/manifest-containing-itself.html: Changed the test to pass on reload, to simplify testing in browser (not related to any behavior changes).
  • http/tests/appcache/navigating-away-while-cache-attempt-in-progress.html: Fixed the test to not fail in browser due to missing layoutTestController.
  • http/tests/appcache/offline-access-expected.txt:
  • http/tests/appcache/offline-access.html: Removed a warning saying that output is timing dependent - I think this is not true now.
  • http/tests/appcache/remove-cache-expected.txt:
  • http/tests/appcache/remove-cache.html:
  • http/tests/appcache/resources/remove-cache-frame-2.html:
  • http/tests/appcache/resources/remove-cache-frame.html: An obsolete event is only dispatched if the document's main resource was already in cache, otherwise, an error event is dispatched. Changed the test to add the main resource to cache first. Also, updated it to make use of additional events that are now being posted.
  • http/tests/appcache/resources/offline-access-frame.html: Updated to dump frame events more completely.
  • http/tests/appcache/wrong-content-type.html:
  • http/tests/appcache/wrong-signature.html: Changed to expect and to make use of error event that is now being posted.

Jan 28, 2009:

11:38 PM Changeset in webkit [40353] by ggaren@apple.com
  • 9 edits in trunk

WebCore:

2009-01-28 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.


Merged FrameLoadTypeReloadAllowingStaleData with FrameLoadTypeReload.


Technically, selecting View->Text Encoding->[Non-Default Encoding] on
a page with frames may now be slightly slower than it used to be. Oh well.

  • loader/FrameLoader.cpp: (WebCore::isBackForwardLoadType): (WebCore::FrameLoader::restoreDocumentState): Subbed in FrameLoadTypeReload.

(WebCore::FrameLoader::loadURLIntoChildFrame): No need to account for
non-back-forward navigations anymore.

(WebCore::FrameLoader::canCachePage):
(WebCore::FrameLoader::logCanCachePageDecision):
(WebCore::FrameLoader::reloadWithOverrideEncoding):
(WebCore::FrameLoader::transitionToCommitted):
(WebCore::FrameLoader::loadItem): Subbed in FrameLoadTypeReload.

  • loader/FrameLoader.h: Renamed reloadAllowingStaleData => reloadWithOverrideEncoding, since that's what it actually does.

WebKit/mac:

2009-01-28 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.

Updated for WebCore rename.


  • WebView/WebView.mm: (-[WebView setCustomTextEncodingName:]):

WebKit/win:

2009-01-28 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.

Updated for WebCore rename.


  • WebView.cpp: (WebView::setCustomTextEncodingName):
10:55 PM Changeset in webkit [40352] by hyatt@apple.com
  • 7 edits in trunk/WebCore

2009-01-28 David Hyatt <hyatt@apple.com>

Move hit testing and painting of lines from RenderFlow into RenderLineBoxList.

Reviewed by Oliver Hunt

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paintContents): (WebCore::RenderBlock::hitTestContents):
  • rendering/RenderFlow.cpp:
  • rendering/RenderFlow.h:
  • rendering/RenderInline.cpp: (WebCore::RenderInline::paint): (WebCore::RenderInline::nodeAtPoint):
  • rendering/RenderLineBoxList.cpp: (WebCore::RenderLineBoxList::paint): (WebCore::RenderLineBoxList::hitTest):
  • rendering/RenderLineBoxList.h:
10:38 PM Changeset in webkit [40351] by mrowe@apple.com
  • 4 edits in trunk

Versioning.

10:24 PM Changeset in webkit [40350] by mrowe@apple.com
  • 1 copy in branches/Safari-6528

New branch.

9:59 PM Changeset in webkit [40349] by mrowe@apple.com
  • 4 edits in branches/Safari-3-2-branch

Versioning.

9:58 PM Changeset in webkit [40348] by mrowe@apple.com
  • 1 copy in tags/Safari-5525.28.2

New tag.

8:41 PM Changeset in webkit [40347] by hyatt@apple.com
  • 2 edits in trunk/WebCore

2009-01-28 David Hyatt <hyatt@apple.com>

Fix SVG pixel test regressions. I wish I could say that I understand this fix, but I don't. For
some reason the original code returned 0 for RenderSVGInlineText::yPos() and somehow my literal
replacement of yPos() with the body of that method changed the results. I am completely baffled,
but all SVG pixel tests pass again when I just take the code out.

Reviewed by Oliver Hunt

  • rendering/RenderSVGInlineText.cpp: (WebCore::RenderSVGInlineText::computeAbsoluteRectForRange):
7:53 PM Changeset in webkit [40346] by ddkilzer@apple.com
  • 2 edits in trunk/WebKit/mac

Add missing declaration for -[NSURL(WebNSURLExtras) _webkit_isFileURL]

Reviewed by Dan Bernstein.

  • Misc/WebNSURLExtras.h: (-[NSURL(WebNSURLExtras) _webkit_isFileURL]): Added missing declaration after the implementation was added in r9258.
6:24 PM Changeset in webkit [40345] by weinig@apple.com
  • 4 edits in trunk

JavaScriptCore:

2009-01-28 Sam Weinig <sam@webkit.org>

Reviewed by Gavin Barraclough.

Fix for <rdar://problem/6525537>
Hang occurs when closing Installer window (iTunes, Aperture)

JavaScriptGlue:

2009-01-28 Sam Weinig <sam@webkit.org>

Reviewed by Gavin Barraclough.

Fix for <rdar://problem/6525537>
Hang occurs when closing Installer window (iTunes, Aperture)

  • JSUtils.cpp: (getThreadGlobalObject): Use the shared JSGlobalData instance.
6:11 PM Changeset in webkit [40344] by hyatt@apple.com
  • 9 edits in trunk/WebCore

2009-01-28 David Hyatt <hyatt@apple.com>

Move dirtyLineBoxes from RenderFlow to RenderInline and RenderBlock.

Reviewed by Oliver Hunt

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::dirtyLineBoxes):
  • rendering/RenderBlock.h:
  • rendering/RenderFlow.cpp:
  • rendering/RenderFlow.h:
  • rendering/RenderInline.cpp: (WebCore::RenderInline::dirtyLineBoxes):
  • rendering/RenderInline.h:
  • rendering/RenderLineBoxList.cpp: (WebCore::RenderLineBoxList::dirtyLineBoxes):
  • rendering/RenderLineBoxList.h:
5:50 PM Changeset in webkit [40343] by hyatt@apple.com
  • 2 edits in trunk/WebCore

2009-01-28 David Hyatt <hyatt@apple.com>

Back out my change to scrollWidth/Height from an earlier checkin since it broke stuff. The current
behavior is not correct, but this will need to be investigated before being changed again.

  • rendering/RenderBox.cpp: (WebCore::RenderBox::scrollWidth): (WebCore::RenderBox::scrollHeight):
5:33 PM Changeset in webkit [40342] by hyatt@apple.com
  • 9 edits in trunk/WebCore

2009-01-28 David Hyatt <hyatt@apple.com>

Move paintOutline and paintOutlineWithLine from RenderFlow to RenderInline, since they only apply to inlines. Tighten up all of the code that
called paintOutline to use RenderInline as the type instead of RenderFlow.

Reviewed by cpst

  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::paint):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paintObject): (WebCore::RenderBlock::addContinuationWithOutline): (WebCore::RenderBlock::paintContinuationOutlines):
  • rendering/RenderFlow.cpp: (WebCore::RenderFlow::paintLines):
  • rendering/RenderFlow.h:
  • rendering/RenderInline.cpp: (WebCore::RenderInline::paintOutline): (WebCore::RenderInline::paintOutlineForLine):
  • rendering/RenderInline.h:
  • rendering/RenderObject.cpp: (WebCore::RenderObject::paintOutline):
  • rendering/RenderObject.h: (WebCore::RenderObject::PaintInfo::PaintInfo):
4:49 PM Changeset in webkit [40341] by darin@chromium.org
  • 1 edit
    3 adds in trunk/LayoutTests

2009-01-28 Darin Fisher <darin@chromium.org>

Fixing bustage.

https://bugs.webkit.org/show_bug.cgi?id=18835
Add missing layout test results.

  • platform/mac/editing/spelling/spelling-linebreak-expected.checksum: Added.
  • platform/mac/editing/spelling/spelling-linebreak-expected.png: Added.
  • platform/mac/editing/spelling/spelling-linebreak-expected.txt: Added.
4:34 PM Changeset in webkit [40340] by mrowe@apple.com
  • 4 edits in trunk

Versioning.

4:26 PM Changeset in webkit [40339] by weinig@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-01-28 Sam Weinig <sam@webkit.org>

Reviewed by Geoff Garen.

Initial patch by Mark Rowe.

<rdar://problem/6519356>
REGRESSION (r36006): "out of memory" alert running dromaeo on Windows

Report the cost of the ArrayStorage vector more accurately/often.

  • runtime/JSArray.cpp: (JSC::JSArray::JSArray): Report the extra cost even for a filled array because JSString using the single character optimization and immediates wont increase the cost themselves. (JSC::JSArray::putSlowCase): Update the cost when increasing the size of the array. (JSC::JSArray::increaseVectorLength): Ditto.
4:11 PM Changeset in webkit [40338] by mrowe@apple.com
  • 1 copy in tags/Safari-6528.11

New tag.

4:01 PM Changeset in webkit [40337] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-01-28 Darin Fisher <darin@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=23596
XMLHttpRequest.cpp no longer compiles without USE(JSC)

3:59 PM Changeset in webkit [40336] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-01-28 Darin Fisher <darin@chromium.org>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=23597
Make it possible to compile Database.cpp when USE(JSC) is not defined

  • storage/Database.cpp: (WebCore::Database::Database):
3:49 PM Changeset in webkit [40335] by mitz@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Darin Adler.

  • fix <rdar://problem/6077047> REGRESSION (r34544): Custom icon does not appear in address field on first visit to any site
  • loader/icon/IconLoader.cpp: (WebCore::IconLoader::finishLoading): Changed to set the icon data only after establishing a mapping from the page URL(s) to the icon. Doing things the other way around resulted in the data not being cached in the icon-URL to IconRecord map (because there was no interested page at the time) and subsequently having to be fetched asynchronously from the database when the page-to-icon mapping was established.
3:44 PM Changeset in webkit [40334] by hyatt@apple.com
  • 7 edits in trunk/WebCore

2009-01-28 David Hyatt <hyatt@apple.com>

Move addFocusRingRects from RenderFlow down into RenderBlock and RenderInline.

Reviewed by Oliver Hunt

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::addFocusRingRects):
  • rendering/RenderBlock.h:
  • rendering/RenderFlow.cpp:
  • rendering/RenderFlow.h:
  • rendering/RenderInline.cpp: (WebCore::RenderInline::addFocusRingRects):
  • rendering/RenderInline.h:
3:18 PM Changeset in webkit [40333] by hyatt@apple.com
  • 5 edits in trunk/WebCore

2009-01-28 David Hyatt <hyatt@apple.com>

Move localCaretRect from RenderFlow into RenderBlock, since it only applied to blocks anyway and
was misplaced from the start.

Reviewed by anttik

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::localCaretRect):
  • rendering/RenderBlock.h:
  • rendering/RenderFlow.cpp:
  • rendering/RenderFlow.h:
2:54 PM Changeset in webkit [40332] by weinig@apple.com
  • 13 edits
    3 adds in trunk

JavaScriptCore:

2009-01-28 Sam Weinig <sam@webkit.org>

Reviewed by Geoff Garen.

Fix for <rdar://problem/6129678>
REGRESSION (Safari 3-4): Local variable not accessible from Dashcode console or variables view

Iterating the properties of activation objects accessed through the WebKit debugging
APIs was broken by forced conversion of JSActivation to the global object. To fix this,
we use a proxy activation object that acts more like a normal JSObject.

  • debugger/DebuggerActivation.cpp: Added. (JSC::DebuggerActivation::DebuggerActivation): (JSC::DebuggerActivation::mark): (JSC::DebuggerActivation::className): (JSC::DebuggerActivation::getOwnPropertySlot): (JSC::DebuggerActivation::put): (JSC::DebuggerActivation::putWithAttributes): (JSC::DebuggerActivation::deleteProperty): (JSC::DebuggerActivation::getPropertyNames): (JSC::DebuggerActivation::getPropertyAttributes): (JSC::DebuggerActivation::defineGetter): (JSC::DebuggerActivation::defineSetter): (JSC::DebuggerActivation::lookupGetter): (JSC::DebuggerActivation::lookupSetter):
  • debugger/DebuggerActivation.h: Added. Proxy JSActivation object for Debugging.
  • runtime/JSActivation.h: (JSC::JSActivation::isActivationObject): Added.
  • runtime/JSObject.h: (JSC::JSObject::isActivationObject): Added.

WebCore:

2009-01-28 Sam Weinig <sam@webkit.org>

Reviewed by Geoff Garen.

Add forwarding header.

  • ForwardingHeaders/debugger/DebuggerActivation.h: Added.

WebKit/mac:

2009-01-28 Sam Weinig <sam@webkit.org>

Reviewed by Geoff Garen.

Fix for <rdar://problem/6129678>
REGRESSION (Safari 3-4): Local variable not accessible from Dashcode console or variables view

  • WebView/WebScriptDebugDelegate.mm: (-[WebScriptCallFrame scopeChain]): Wrap JSActivations in DebuggerActivations.
2:47 PM Changeset in webkit [40331] by ggaren@apple.com
  • 1 edit
    49 adds in trunk/LayoutTests

2009-01-28 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.


Added layout tests for how many history items are created during a
redirect. (Many of these results are expected failures for now.)

  • http/tests/history: Added.
  • http/tests/history/redirect-200-refresh-0-seconds-expected.txt: Added.
  • http/tests/history/redirect-200-refresh-0-seconds.pl: Added.
  • http/tests/history/redirect-200-refresh-2-seconds-expected.txt: Added.
  • http/tests/history/redirect-200-refresh-2-seconds.pl: Added.
  • http/tests/history/redirect-301-expected.txt: Added.
  • http/tests/history/redirect-301.pl: Added.
  • http/tests/history/redirect-302-expected.txt: Added.
  • http/tests/history/redirect-302.pl: Added.
  • http/tests/history/redirect-303-expected.txt: Added.
  • http/tests/history/redirect-303.pl: Added.
  • http/tests/history/redirect-307-expected.txt: Added.
  • http/tests/history/redirect-307.pl: Added.
  • http/tests/history/redirect-js-document-location-0-seconds-expected.txt: Added.
  • http/tests/history/redirect-js-document-location-0-seconds.html: Added.
  • http/tests/history/redirect-js-document-location-2-seconds-expected.txt: Added.
  • http/tests/history/redirect-js-document-location-2-seconds.html: Added.
  • http/tests/history/redirect-js-document-location-before-load-expected.txt: Added.
  • http/tests/history/redirect-js-document-location-before-load.html: Added.
  • http/tests/history/redirect-js-location-0-seconds-expected.txt: Added.
  • http/tests/history/redirect-js-location-0-seconds.html: Added.
  • http/tests/history/redirect-js-location-2-seconds-expected.txt: Added.
  • http/tests/history/redirect-js-location-2-seconds.html: Added.
  • http/tests/history/redirect-js-location-assign-0-seconds-expected.txt: Added.
  • http/tests/history/redirect-js-location-assign-0-seconds.html: Added.
  • http/tests/history/redirect-js-location-assign-2-seconds-expected.txt: Added.
  • http/tests/history/redirect-js-location-assign-2-seconds.html: Added.
  • http/tests/history/redirect-js-location-assign-before-load-expected.txt: Added.
  • http/tests/history/redirect-js-location-assign-before-load.html: Added.
  • http/tests/history/redirect-js-location-before-load-expected.txt: Added.
  • http/tests/history/redirect-js-location-before-load.html: Added.
  • http/tests/history/redirect-js-location-href-0-seconds-expected.txt: Added.
  • http/tests/history/redirect-js-location-href-0-seconds.html: Added.
  • http/tests/history/redirect-js-location-href-2-seconds-expected.txt: Added.
  • http/tests/history/redirect-js-location-href-2-seconds.html: Added.
  • http/tests/history/redirect-js-location-href-before-load-expected.txt: Added.
  • http/tests/history/redirect-js-location-href-before-load.html: Added.
  • http/tests/history/redirect-js-location-replace-0-seconds-expected.txt: Added.
  • http/tests/history/redirect-js-location-replace-0-seconds.html: Added.
  • http/tests/history/redirect-js-location-replace-2-seconds-expected.txt: Added.
  • http/tests/history/redirect-js-location-replace-2-seconds.html: Added.
  • http/tests/history/redirect-js-location-replace-before-load-expected.txt: Added.
  • http/tests/history/redirect-js-location-replace-before-load.html: Added.
  • http/tests/history/redirect-meta-refresh-0-seconds-expected.txt: Added.
  • http/tests/history/redirect-meta-refresh-0-seconds.html: Added.
  • http/tests/history/redirect-meta-refresh-2-seconds-expected.txt: Added.
  • http/tests/history/redirect-meta-refresh-2-seconds.html: Added.
  • http/tests/history/resources: Added.
  • http/tests/history/resources/redirect-target.html: Added.
2:43 PM Changeset in webkit [40330] by ggaren@apple.com
  • 2 edits in trunk/WebKitTools

2009-01-28 Geoffrey Garen <ggaren@apple.com>

Build fix for GTK.

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::webHistoryItemCount):
2:36 PM Changeset in webkit [40329] by ggaren@apple.com
  • 5 edits in trunk/WebKitTools

2009-01-28 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.


Added support for querying how many history items were created during a
layout test.

  • DumpRenderTree/LayoutTestController.cpp: (getWebHistoryItemCountCallback): (LayoutTestController::staticValues):
  • DumpRenderTree/LayoutTestController.h:
  • DumpRenderTree/mac/LayoutTestControllerMac.mm: (LayoutTestController::webHistoryItemCount):
  • DumpRenderTree/win/LayoutTestControllerWin.cpp: (LayoutTestController::webHistoryItemCount):
2:35 PM Changeset in webkit [40328] by hyatt@apple.com
  • 1 edit in trunk/WebCore/WebCore.xcodeproj/project.pbxproj

Fix build bustage.

1:58 PM Changeset in webkit [40327] by hyatt@apple.com
  • 1 edit in trunk/WebCore/rendering/RenderLineBoxList.h

Fix build bustage in release builds.

1:48 PM Changeset in webkit [40326] by hyatt@apple.com
  • 8 edits in trunk/WebCore

2009-01-28 David Hyatt <hyatt@apple.com>

Move createInlineBox out of RenderFlow and down into RenderInline and RenderBlock.

Reviewed by Beth Dakin

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::createInlineBox):
  • rendering/RenderBlock.h:
  • rendering/RenderFlow.cpp:
  • rendering/RenderFlow.h:
  • rendering/RenderInline.cpp: (WebCore::RenderInline::createInlineBox):
  • rendering/RenderInline.h:
  • rendering/bidi.cpp:
1:18 PM Changeset in webkit [40325] by adachan@apple.com
  • 4 edits in branches/Safari-3-2-branch/WebCore

Merge the rest of r33579.

Reviewed by Steve Falkenburg.

  • platform/network/cf/ResourceHandleCFNet.cpp: (WebCore::makeFinalRequest):
  • platform/network/cf/ResourceRequestCFNet.cpp: (WebCore::ResourceRequest::doUpdatePlatformRequest):
  • platform/network/win/CookieJarWin.cpp: (WebCore::setCookies): (WebCore::cookies): (WebCore::cookiesEnabled):
12:56 PM Changeset in webkit [40324] by hyatt@apple.com
  • 5 edits in trunk/WebCore

2009-01-28 David Hyatt <hyatt@apple.com>

Move calcMargins from RenderFlow to RenderInline, since it was only ever called on RenderInlines.

Reviewed by Oliver Hunt

  • rendering/RenderFlow.cpp:
  • rendering/RenderFlow.h:
  • rendering/RenderInline.h: (WebCore::RenderInline::calcMargins):
  • rendering/bidi.cpp: (WebCore::RenderBlock::layoutInlineChildren):
12:48 PM Changeset in webkit [40323] by hyatt@apple.com
  • 16 edits
    2 adds in trunk/WebCore

2009-01-28 David Hyatt <hyatt@apple.com>

Refactor the first and last line box member variables from RenderFlow. Encapsulate them into a new class called RenderLineBoxList that manages all modifications to
the list. This change will make it easier to give RenderInline and RenderBlock separate versions of the line box list without having to duplicate very much code.

Reviewed by Oliver Hunt

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::deleteLine): (WebCore::InlineFlowBox::removeLineBoxFromRenderObject): (WebCore::InlineFlowBox::extractLine): (WebCore::InlineFlowBox::extractLineBoxFromRenderObject): (WebCore::InlineFlowBox::attachLine): (WebCore::InlineFlowBox::attachLineBoxToRenderObject):
  • rendering/InlineFlowBox.h:
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::deleteLineBoxTree):
  • rendering/RenderFlow.cpp: (WebCore::RenderFlow::destroy): (WebCore::RenderFlow::dirtyLineBoxes): (WebCore::RenderFlow::createInlineBox):
  • rendering/RenderFlow.h: (WebCore::RenderFlow::RenderFlow): (WebCore::RenderFlow::lineBoxes): (WebCore::RenderFlow::firstLineBox): (WebCore::RenderFlow::lastLineBox):
  • rendering/RenderLineBoxList.cpp: Added. (WebCore::RenderLineBoxList::~RenderLineBoxList): (WebCore::RenderLineBoxList::appendLineBox): (WebCore::RenderLineBoxList::deleteLineBoxTree): (WebCore::RenderLineBoxList::extractLineBox): (WebCore::RenderLineBoxList::attachLineBox): (WebCore::RenderLineBoxList::removeLineBox): (WebCore::RenderLineBoxList::deleteLineBoxes): (WebCore::RenderLineBoxList::checkConsistency):
  • rendering/RenderLineBoxList.h: Added. (WebCore::RenderLineBoxList::RenderLineBoxList): (WebCore::RenderLineBoxList::firstLineBox): (WebCore::RenderLineBoxList::lastLineBox): (WebCore::RenderLineBoxList::checkConsistency):
  • rendering/RenderSVGInline.cpp: (WebCore::RenderSVGInline::createInlineBox):
  • rendering/RenderSVGText.cpp: (WebCore::RenderSVGText::createInlineBox):
  • rendering/RootInlineBox.cpp: (WebCore::RootInlineBox::removeLineBoxFromRenderObject): (WebCore::RootInlineBox::extractLineBoxFromRenderObject): (WebCore::RootInlineBox::attachLineBoxToRenderObject):
  • rendering/RootInlineBox.h:
  • rendering/bidi.cpp: (WebCore::RenderBlock::layoutInlineChildren):
12:46 PM Changeset in webkit [40322] by ddkilzer@apple.com
  • 3 edits in trunk/WebCore

Clean up FontPlatformData constructor

Reviewed by David Hyatt.

  • platform/graphics/mac/FontPlatformData.h: Updated copyright. (WebCore::FontPlatformData::FontPlatformData): Provide readable names for constructor arguments and reformat member variables.
  • platform/graphics/mac/FontPlatformDataMac.mm: Updated copyright. (WebCore::FontPlatformData::FontPlatformData): Provide readable names for constructor arguments and reformat member variables.
12:37 PM Changeset in webkit [40321] by darin@chromium.org
  • 3 edits in trunk/WebCore

2009-01-28 Tony Chang <tony@chromium.org>

Reviewed by David Hyatt.

https://bugs.webkit.org/show_bug.cgi?id=23593
Compile fixes for RenderThemeChromiumMac and RenderThemeChromiumWin.
This is a follow up for hyatt's RenderBox changes.

  • rendering/RenderThemeChromiumMac.mm: (WebCore::RenderThemeChromiumMac::baselinePosition):
  • rendering/RenderThemeChromiumWin.cpp: (WebCore::RenderThemeChromiumWin::paintMenuList):
12:26 PM Changeset in webkit [40320] by ddkilzer@apple.com
  • 2 edits in trunk

Fixed ChangeLog dates.

12:07 PM Changeset in webkit [40319] by ddkilzer@apple.com
  • 6 edits in trunk

Bug 23490: Remove initialRefCount argument from RefCounted class

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

Reviewed by Darin Adler.

JavaScriptCore:

RefCountedBase now always starts with a ref count of 1, so there
is no need to pass the initialRefCount into the class anymore.

  • wtf/ByteArray.h: (WTF::ByteArray::ByteArray): Removed call to RefCounted(1).
  • wtf/RefCounted.h: (WTF::RefCountedBase::RefCountedBase): Changed to start with a ref count of 1. (WTF::RefCounted::RefCounted): Removed initialRefCount argument and removed call to RefCounted(1).

WebKit/wx:

FrameLoaderClientWx is no longer a RefCounted class.

  • WebKitSupport/FrameLoaderClientWx.cpp: (WebCore::FrameLoaderClientWx::FrameLoaderClientWx): Removed call to the RefCounted<FrameLoaderClientWx>(0) super constructor.
  • WebKitSupport/FrameLoaderClientWx.h: Don't include RefCounted.h and stop inheriting from RefCounted<FrameLoaderClientWx>.
11:40 AM Changeset in webkit [40318] by Simon Fraser
  • 5 edits
    8 adds in trunk

2009-01-28 Simon Fraser <Simon Fraser>

Reviewed by Dave Hyatt

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

Fix assertions when inline elements have both opacity (or something
else that creates a RenderLayer), and transform by ensuring that
setHasTransform() is only called for non-inline or replaced elements.
We also have to ensure that RenderReplicas will get transforms applied
to them by lying about them being replaced.

Tests: fast/transforms/transform-on-inline.html

fast/transforms/transform-table-row.html

  • rendering/RenderBox.cpp: (WebCore::RenderBox::styleDidChange):
  • rendering/RenderReplica.cpp: (WebCore::RenderReplica::RenderReplica):
  • rendering/RenderTableRow.h: (WebCore::RenderTableRow::requiresLayer):
11:38 AM Changeset in webkit [40317] by darin@chromium.org
  • 3 edits
    1 add in trunk

2009-01-28 Hironori Bono <hbono@chromium.org>

Reviewed by Justin Garcia.

https://bugs.webkit.org/show_bug.cgi?id=18835
Spelling underlines disappear when they shouldn't

The WebCore::InsertLineBreakCommand::doApply() function splits a text node
without copying existing markers. To fix this bug, change code which
manually splits a text node into a CompositeEditCommand::splitTextNode() call.

Test: editing/spelling/spelling-linebreak.html

  • editing/InsertLineBreakCommand.cpp: (WebCore::InsertLineBreakCommand::doApply):
11:31 AM Changeset in webkit [40316] by hyatt@apple.com
  • 9 edits in trunk/WebCore

2009-01-28 David Hyatt <hyatt@apple.com>

Move the m_lineHeight member variable out of RenderFlow and down into RenderBlock and RenderInline. Shift the line height computation into RenderStyle so
that it isn't repeated in three places.

Reviewed by Sam Weinig

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::RenderBlock): (WebCore::RenderBlock::lineHeight):
  • rendering/RenderBlock.h:
  • rendering/RenderFlow.cpp:
  • rendering/RenderFlow.h: (WebCore::RenderFlow::RenderFlow):
  • rendering/RenderInline.cpp: (WebCore::RenderInline::RenderInline): (WebCore::RenderInline::lineHeight):
  • rendering/RenderInline.h:
  • rendering/RenderObject.cpp: (WebCore::RenderObject::lineHeight):
  • rendering/style/RenderStyle.h: (WebCore::InheritedFlags::computedLineHeight):
11:19 AM Changeset in webkit [40315] by darin@chromium.org
  • 4 edits in trunk/WebCore

2009-01-28 Zachary Kuznia <zork@chromiuum.org>

Reviewed by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=22070
Add an option to allow scripts to close windows.

  • page/Settings.cpp: (WebCore::Settings::Settings): (WebCore::Settings::setAllowScriptsToCloseWindows):
  • page/Settings.h: (WebCore::Settings::allowScriptsToCloseWindows):
10:52 AM Changeset in webkit [40314] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-01-28 Adam Langley <agl@google.com>

Reviewed by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=23573
Make PlatformWidget on chromium an intptr_t to match changes made in Chromium

  • platform/chromium/PlatformWidget.h:
9:39 AM Changeset in webkit [40313] by hyatt@apple.com
  • 2 edits in trunk/WebCore

2009-01-28 David Hyatt <hyatt@apple.com>

Fix for

<rdar://problem/6531287> REGRESSION: CrashTracer: [USER] 12 crashes in Safari at com.apple.WebCore • WebCore::RenderBlock::removeFloatingObject + 59

My tightening up of markAllDescendantsWithFloatsForLayout exposed a buggy check in that code that could
lead to a bad cast.

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::markAllDescendantsWithFloatsForLayout):
9:30 AM Changeset in webkit [40312] by hyatt@apple.com
  • 29 edits in trunk/WebCore

2009-01-27 David Hyatt <hyatt@apple.com>

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

Work towards eliminating RenderFlow from the tree.


Move the m_continuation variable down from RenderFlow into RenderInline and RenderBlock. Since a block can only have an inline continuation following it, the type
of the member and methods reflect this (inlineContinuation() and m_inlineContinuation). Since an inline can have either a block or an inline continuation following it,
a base class of RenderBox* is used for the type of object returned (in anticipation of the removal of RenderFlow).


Since moving the continuation variable down into the subclasses increased the size of RenderInline and RenderBlock by 4 bytes, this patch also moves all of the bitfield
members of RenderFlow up to RenderObject. Since they fit within the available bits on RenderObject, this patch actually results in a net savings of 4 bytes on RenderInlines
and RenderBlocks!


One bitfield member was eliminated rather than moved: m_firstLine. This was really more of a state variable used during line layout only, so I removed it as a member
and just passed firstLine down to various methods as needed. Doing so uncovered some potential bugs where the first line state was not being respected when querying for
line offsets.

Continuations have been completely hidden from all files except for RenderFlow, RenderBlock and RenderInline. All of the code that referenced continuations directly
from base classes has been refactored to use virtual methods on RenderBlock and RenderInline instead.


RenderFlow still has a common addChildWithContinuation method that is used by both blocks and inlines. Since refactoring that method will be pretty tricky, I've held
off on that for a later patch.

Reviewed by Oliver Hunt

  • html/HTMLAnchorElement.cpp: (WebCore::HTMLAnchorElement::isKeyboardFocusable):
  • page/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::anchorElement): (WebCore::AccessibilityRenderObject::boundingBoxRect):
  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::determineSpacingForFlowBoxes): (WebCore::InlineFlowBox::paint):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::RenderBlock): (WebCore::RenderBlock::~RenderBlock): (WebCore::RenderBlock::destroy): (WebCore::RenderBlock::addChildToFlow): (WebCore::RenderBlock::makeChildrenNonInline): (WebCore::RenderBlock::removeChild): (WebCore::RenderBlock::layoutBlock): (WebCore::RenderBlock::expandsToEncloseOverhangingFloats): (WebCore::RenderBlock::collapseMargins): (WebCore::RenderBlock::determineHorizontalPosition): (WebCore::RenderBlock::setCollapsedBottomMargin): (WebCore::RenderBlock::layoutOnlyPositionedObjects): (WebCore::RenderBlock::paintObject): (WebCore::RenderBlock::addContinuationWithOutline): (WebCore::RenderBlock::setSelectionState): (WebCore::RenderBlock::shouldPaintSelectionGaps): (WebCore::RenderBlock::fillSelectionGaps): (WebCore::RenderBlock::leftSelectionOffset): (WebCore::RenderBlock::rightSelectionOffset): (WebCore::RenderBlock::leftOffset): (WebCore::RenderBlock::leftRelOffset): (WebCore::RenderBlock::rightOffset): (WebCore::RenderBlock::rightRelOffset): (WebCore::RenderBlock::lineWidth): (WebCore::RenderBlock::lowestPosition): (WebCore::RenderBlock::rightmostPosition): (WebCore::RenderBlock::leftmostPosition): (WebCore::RenderBlock::getClearDelta): (WebCore::RenderBlock::nodeAtPoint): (WebCore::RenderBlock::offsetForContents): (WebCore::RenderBlock::availableWidth): (WebCore::RenderBlock::setDesiredColumnCountAndWidth): (WebCore::RenderBlock::desiredColumnWidth): (WebCore::RenderBlock::desiredColumnCount): (WebCore::RenderBlock::columnRects): (WebCore::RenderBlock::layoutColumns): (WebCore::RenderBlock::adjustPointToColumnContents): (WebCore::RenderBlock::adjustRectForColumns): (WebCore::RenderBlock::absoluteRects): (WebCore::RenderBlock::absoluteQuads): (WebCore::RenderBlock::rectWithOutlineForRepaint): (WebCore::RenderBlock::hoverAncestor): (WebCore::RenderBlock::updateDragState): (WebCore::RenderBlock::outlineStyleForRepaint): (WebCore::RenderBlock::childBecameNonInline): (WebCore::RenderBlock::updateHitTestResult):
  • rendering/RenderBlock.h: (WebCore::RenderBlock::rightOffset): (WebCore::RenderBlock::leftOffset): (WebCore::RenderBlock::inlineContinuation): (WebCore::RenderBlock::setInlineContinuation):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::absoluteRects): (WebCore::RenderBox::absoluteQuads): (WebCore::RenderBox::addFocusRingRects): (WebCore::RenderBox::containingBlockWidth):
  • rendering/RenderBox.h: (WebCore::RenderBox::collapsedMarginBottom): (WebCore::RenderBox::childBecameNonInline):
  • rendering/RenderContainer.cpp: (WebCore::RenderContainer::updateBeforeAfterContentForContainer): (WebCore::RenderContainer::removeLeftoverAnonymousBlock):
  • rendering/RenderFlexibleBox.cpp: (WebCore::RenderFlexibleBox::layoutVerticalBox):
  • rendering/RenderFlow.cpp: (WebCore::nextContinuation): (WebCore::RenderFlow::continuationBefore): (WebCore::RenderFlow::addChildWithContinuation): (WebCore::RenderFlow::addChild): (WebCore::RenderFlow::destroy): (WebCore::RenderFlow::addFocusRingRects):
  • rendering/RenderFlow.h: (WebCore::RenderFlow::RenderFlow):
  • rendering/RenderInline.cpp: (WebCore::RenderInline::RenderInline): (WebCore::RenderInline::destroy): (WebCore::RenderInline::inlineContinuation): (WebCore::RenderInline::styleDidChange): (WebCore::RenderInline::addChildToFlow): (WebCore::RenderInline::cloneInline): (WebCore::RenderInline::splitInlines): (WebCore::RenderInline::splitFlow): (WebCore::RenderInline::positionForCoordinates): (WebCore::RenderInline::rectWithOutlineForRepaint): (WebCore::RenderInline::updateDragState): (WebCore::RenderInline::childBecameNonInline): (WebCore::RenderInline::updateHitTestResult):
  • rendering/RenderInline.h: (WebCore::RenderInline::isRenderInline): (WebCore::RenderInline::continuation): (WebCore::RenderInline::setContinuation):
  • rendering/RenderListItem.cpp: (WebCore::RenderListItem::positionListMarker):
  • rendering/RenderListMarker.cpp: (WebCore::RenderListMarker::RenderListMarker): (WebCore::RenderListMarker::setSelectionState):
  • rendering/RenderListMarker.h:
  • rendering/RenderObject.cpp: (WebCore::RenderObject::RenderObject): (WebCore::RenderObject::repaintAfterLayoutIfNeeded): (WebCore::RenderObject::rectWithOutlineForRepaint): (WebCore::RenderObject::handleDynamicFloatPositionChange): (WebCore::RenderObject::updateDragState): (WebCore::RenderObject::updateHitTestResult): (WebCore::RenderObject::getTextDecorationColors): (WebCore::RenderObject::adjustRectForOutlineAndShadow):
  • rendering/RenderObject.h: (WebCore::RenderObject::isInlineBlockOrInlineTable): (WebCore::RenderObject::childrenInline): (WebCore::RenderObject::setChildrenInline): (WebCore::RenderObject::hasColumns): (WebCore::RenderObject::setHasColumns): (WebCore::RenderObject::cellWidthChanged): (WebCore::RenderObject::setCellWidthChanged): (WebCore::RenderObject::isInlineContinuation): (WebCore::RenderObject::hoverAncestor): (WebCore::RenderObject::outlineStyleForRepaint): (WebCore::RenderObject::setHasMarkupTruncation): (WebCore::RenderObject::hasMarkupTruncation): (WebCore::RenderObject::selectionState): (WebCore::RenderObject::setSelectionState): (WebCore::RenderObject::hasSelectedChildren): (WebCore::RenderObject::isTopMarginQuirk): (WebCore::RenderObject::isBottomMarginQuirk): (WebCore::RenderObject::setTopMarginQuirk): (WebCore::RenderObject::setBottomMarginQuirk):
  • rendering/RenderReplaced.cpp: (WebCore::RenderReplaced::RenderReplaced): (WebCore::RenderReplaced::setSelectionState):
  • rendering/RenderReplaced.h: (WebCore::RenderReplaced::canBeSelectionLeaf):
  • rendering/RenderTable.cpp: (WebCore::RenderTable::calcWidth):
  • rendering/RenderTableCell.cpp: (WebCore::RenderTableCell::updateWidth): (WebCore::RenderTableCell::layout):
  • rendering/RenderText.cpp: (WebCore::RenderText::RenderText): (WebCore::RenderText::localCaretRect): (WebCore::RenderText::setSelectionState):
  • rendering/RenderText.h: (WebCore::RenderText::canBeSelectionLeaf):
  • rendering/RenderView.cpp: (WebCore::RenderView::updateHitTestResult):
  • rendering/RenderView.h:
  • rendering/RootInlineBox.cpp: (WebCore::RootInlineBox::selectionTop):
  • rendering/bidi.cpp: (WebCore::RenderBlock::createLineBoxes): (WebCore::RenderBlock::constructLine): (WebCore::RenderBlock::computeHorizontalPositionsForLine): (WebCore::RenderBlock::layoutInlineChildren): (WebCore::RenderBlock::determineStartPosition): (WebCore::RenderBlock::skipTrailingWhitespace): (WebCore::RenderBlock::skipLeadingWhitespace): (WebCore::RenderBlock::fitBelowFloats): (WebCore::RenderBlock::findNextLineBreak): (WebCore::RenderBlock::checkLinesForTextOverflow):
8:39 AM Changeset in webkit [40311] by treat@webkit.org
  • 9 edits in trunk

Reviewed by Nikolas Zimmermann and George Staikos.

https://bugs.webkit.org/show_bug.cgi?id=23557
Allow option of hit testing frame content without clipping to the visible
viewport.

8:26 AM Changeset in webkit [40310] by Simon Hausmann
  • 2 edits in trunk/WebCore

2009-01-28 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Holger Freyther.

Fix the build with Qt on Windows.

Make sure that the QTWEBKIT_EXPORT macro becomes the Q_DECL_EXPORT macro
instead of the import macro when building the dll, by defining
QT_MAKEDLL.

SystemTimeWin.cpp is needed on Windows for userIdleTime(), which for the
other Qt platforms comes from TemporaryLinkStubs.cpp.

5:42 AM Changeset in webkit [40309] by ariya@webkit.org
  • 2 edits in trunk/WebKit/qt

2009-01-28 Ariya Hidayat <ariya.hidayat@trolltech.com>

Rubber-stamped by Simon Hausmann.

Provide default implementations of JavaScript dialog boxes like in
most web browsers, i.e. indicate that is from JavaScript and show the
frame URL.

  • Api/qwebpage.cpp: (QWebPage::javaScriptAlert): (QWebPage::javaScriptConfirm): (QWebPage::javaScriptPrompt):
5:19 AM Changeset in webkit [40308] by ariya@webkit.org
  • 2 edits in trunk/WebCore

2009-01-28 Ariya Hidayat <ariya.hidayat@trolltech.com>

Reviewed by Simon Hausmann.

[Qt] Speed-up clipping: avoid calling expensive QPainter::clipRegion().
Beside, the check is not necessary since QPainter::setClipRect() will
work just fine if there is no clip region yet.

  • platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::GraphicsContext::clip):
12:19 AM Changeset in webkit [40307] by darin@chromium.org
  • 2 edits
    2 adds in trunk/WebCore

2009-01-28 Darin Fisher <darin@chromium.org>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=23546
Upstream GoogleURL implementation of KURL

  • platform/KURL.h:
  • platform/KURLGoogle.cpp: Added.
  • platform/KURLGooglePrivate.h: Added.

Jan 27, 2009:

8:08 PM Changeset in webkit [40306] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

2009-01-27 Anders Carlsson <andersca@apple.com>

Reviewed by Oliver Hunt.

Fix two bugs with Core Animation based plug-ins.


  1. The plug-in view was marked as opaque even though it's not. (This would leave garbage in the plug-in view).
  2. The plug-in layer needs to have autoresizing turned on.


  • Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView setLayer:]):
5:54 PM Changeset in webkit [40305] by Darin Adler
  • 2 edits in trunk/WebCore

2009-01-27 Darin Adler <Darin Adler>

Reviewed by Sam Weinig.

REGRESSION (r40234): Launching iTunes from a link displays a
"Safari can't find the Internet plug-in" sheet in Safari
rdar://problem/6533678

  • rendering/RenderPartObject.cpp: (WebCore::havePlugin): Put the check for empty string in here. (WebCore::serviceTypeForClassId): Only return activeXType() if we have a plug-in that can handle that type. While it might be logical to return an Active X type for any <object> with a classId in it, even if we don't have a plug-in that implements Active X, existing clients, such as Safari, don't expect that.
5:10 PM Changeset in webkit [40304] by mitz@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Steve Falkenburg.

  • fix <rdar://problem/6524351> Text looks bad when font smoothing is forced off
  • platform/graphics/win/FontCGWin.cpp: (WebCore::Font::drawGlyphs): Changed to use GDI rather than Core Graphics if the client has turned font smoothing off.
3:05 PM Changeset in webkit [40303] by Simon Fraser
  • 17 edits in trunk/WebCore

2009-01-27 Simon Fraser <Simon Fraser>

Reviewed by Dave Hyatt

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

Use the new container-relative repaint methods to compute
post-layout repaints, and repaints using RenderLayer's
cached repaintRect() relative to that container, rather than
using absolute coords.

Replaced lots of boilerplate old/new rect code in various
layout methods with a stack-based LayoutRepainter class, and
fixed a bug in RenderTable which set didFullRepaint to
'true' every time.

2:40 PM Changeset in webkit [40302] by beidson@apple.com
  • 6 edits in trunk/WebCore

2009-01-27 Brady Eidson <beidson@apple.com>

Reviewed by Darin Adler

<rdar://problem/3541409> - More groundwork and cleanup for CachedFrames

  • history/CachedFrame.cpp: (WebCore::CachedFrame::CachedFrame): Make the savePlatformDataToCachedFrame() client call here, and add the code for caching subframes. (WebCore::CachedFrame::clear):
  • history/CachedFrame.h: Change to be RefCounted (WebCore::CachedFrame::create): (WebCore::CachedFrame::url):
  • history/CachedPage.cpp: (WebCore::CachedPage::CachedPage): (WebCore::CachedPage::restore):
  • history/CachedPage.h: (WebCore::CachedPage::document): (WebCore::CachedPage::documentLoader): (WebCore::CachedPage::view): (WebCore::CachedPage::url): (WebCore::CachedPage::domWindow): (WebCore::CachedPage::cachedMainFrame):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::cachePageForHistoryItem): Move the platform data saving into the CachedFrame itself.
2:09 PM Changeset in webkit [40301] by zecke@webkit.org
  • 2 edits in trunk/LayoutTests

2009-01-27 Holger Hans Peter Freyther <zecke@selfish.org>

Rubber stamped by Darin Adler.

Get the build bot green as discussed on the mailinglist. There is
only one crash after unloading a page with plugins. For the remaining
failing tests DumpRenderTree needs to be looked at.

  • platform/gtk/Skipped:
1:54 PM Changeset in webkit [40300] by Simon Fraser
  • 13 edits in trunk/WebCore

2009-01-27 Simon Fraser <Simon Fraser>

Reviewed by Dave Hyatt

Flip the arguments of computeRectForRepaint() from

computeRectForRepaint(IntRect&, RenderBox* repaintContainer, bool)

to

computeRectForRepaint(RenderBox* repaintContainer, IntRect&, bool)

1:44 PM Changeset in webkit [40299] by ddkilzer@apple.com
  • 2 edits in trunk/WebCore

Add missing header guards to AccessibilityObjectWrapper.h

Reviewed by Anders Carlsson.

  • page/mac/AccessibilityObjectWrapper.h: Added missing header guards.
1:28 PM Changeset in webkit [40298] by Simon Fraser
  • 5 edits in trunk/WebCore

2009-01-27 Simon Fraser <Simon Fraser>

Reviewed by Dave Hyatt

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

Fix crash of svg/custom/stop-crash.svg testcase because of
infinite recursion, as I missed replacing absoluteClippedOverflowRect()
with clippedOverflowRectForRepaint() in RenderSVGGradientStop.

Also fix a missed conversion in RenderTableCol, and add a couple
of comments.

11:24 AM Changeset in webkit [40297] by adele@apple.com
  • 2 edits in trunk/WebCore

2009-01-27 Adele Peterson <adele@apple.com>

RS by Dave Hyatt

Use the textfield appearance for search fields in themeWin.css so the border/background
styling will work correctly on these fields.

  • css/themeWin.css:
10:45 AM Changeset in webkit [40296] by mitz@apple.com
  • 2 edits in trunk/WebCore
  • build fix
  • bindings/js/JSHTMLInputElementCustom.cpp: (WebCore::JSHTMLInputElement::setSelectionStart): (WebCore::JSHTMLInputElement::setSelectionEnd): (WebCore::JSHTMLInputElement::setSelectionRange):
10:26 AM Changeset in webkit [40295] by mitz@apple.com
  • 2 edits in trunk/WebCore
  • build fix
  • platform/graphics/Gradient.cpp: (WebCore::Gradient::setSpreadMethod):
9:30 AM Changeset in webkit [40294] by Dimitri Glazkov
  • 4 edits in trunk/WebCore

2009-01-27 Paul Godavari <paul@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=23561
Explicitly send the MIME type from an image dragged from a web
page to the desktop, to Chromium so that it can properly generate
a file name.

  • platform/chromium/ChromiumDataObject.cpp: (WebCore::ChromiumDataObject::clear): (WebCore::ChromiumDataObject::hasData):
  • platform/chromium/ChromiumDataObject.h:
  • platform/chromium/ClipboardChromium.cpp: (WebCore::writeImageToDataObject):
9:24 AM Changeset in webkit [40293] by Dimitri Glazkov
  • 6 edits in trunk

WebCore:

2009-01-27 Mads Ager <ager@chromium.org>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=23300
Follow Firefox in throwing exceptions when using selection
functions on HTMLInputElements that cannot have selection.

This requires custom setSelectionRange, setSelectionStart and
setSelectionEnd implementations, but it gets rid of the custom
getOwnPropertySlot.

  • bindings/js/JSHTMLInputElementCustom.cpp: (WebCore::JSHTMLInputElement::selectionStart): (WebCore::JSHTMLInputElement::selectionEnd): (WebCore::JSHTMLInputElement::setSelectionRange):
  • html/HTMLInputElement.idl:

LayoutTests:

2009-01-27 Mads Ager <ager@chromium.org>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=23300
Update test of selection functions on HTMLInputElements to expect
the selection functions to throw exceptions when used on buttons.
This is consistent with the Firefox behavior.

  • fast/forms/selection-functions-expected.txt:
  • fast/forms/selection-functions.html:
9:13 AM Changeset in webkit [40292] by Dimitri Glazkov
  • 16 edits in trunk/WebCore

2009-01-27 Evan Stade <estade@chromium.org>

Reviewed by Nikolas Zimmermann.

https://bugs.webkit.org/show_bug.cgi?id=23435
Make spreadMethod a member of Gradient rather than GraphicsContext.
Implement spreadMethod for Skia.

  • platform/graphics/Gradient.cpp: (WebCore::Gradient::Gradient):
  • platform/graphics/Gradient.h: (WebCore::Gradient::setSpreadMethod): (WebCore::Gradient::spreadMethod):
  • platform/graphics/GraphicsContext.cpp:
  • platform/graphics/GraphicsContext.h:
  • platform/graphics/GraphicsContextPrivate.h:
  • platform/graphics/GraphicsTypes.h: (WebCore::):
  • platform/graphics/cairo/GradientCairo.cpp: (WebCore::Gradient::platformGradient):
  • platform/graphics/cairo/GraphicsContextCairo.cpp: (WebCore::GraphicsContext::fillPath): (WebCore::GraphicsContext::strokePath):
  • platform/graphics/qt/GradientQt.cpp: (WebCore::Gradient::platformGradient):
  • platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::GraphicsContext::fillPath): (WebCore::GraphicsContext::strokePath):
  • platform/graphics/skia/GradientSkia.cpp: (WebCore::Gradient::platformGradient):
  • svg/SVGLinearGradientElement.cpp: (WebCore::SVGLinearGradientElement::buildGradient):
  • svg/SVGRadialGradientElement.cpp: (WebCore::SVGRadialGradientElement::buildGradient):
  • svg/graphics/SVGPaintServerGradient.cpp: (WebCore::SVGPaintServerGradient::SVGPaintServerGradient): (WebCore::SVGPaintServerGradient::setup): (WebCore::SVGPaintServerGradient::externalRepresentation):
  • svg/graphics/SVGPaintServerGradient.h:
8:03 AM Changeset in webkit [40291] by ariya@webkit.org
  • 2 edits in trunk/WebCore

2009-01-27 Ariya Hidayat <ariya.hidayat@trolltech.com>

Rubber-stamped by Simon Hausmann.

http://www.qtsoftware.com/developer/task-tracker/index_html?id=238662&method=entry

[Qt] Map function keys F1 to F24.

  • platform/qt/PlatformKeyboardEventQt.cpp: (WebCore::windowsKeyCodeForKeyEvent):
7:26 AM Changeset in webkit [40290] by ariya@webkit.org
  • 2 edits in trunk/WebKit/qt

2009-01-27 Ariya Hidayat <ariya.hidayat@trolltech.com>

Reviewed by Simon Hausmann.

http://www.qtsoftware.com/developer/task-tracker/index_html?id=238391&method=entry

[Qt] If QPainter fails to start on a QPrinter instance, do not
continue printing.

  • Api/qwebframe.cpp: (QWebFrame::print):
12:05 AM Changeset in webkit [40289] by beidson@apple.com
  • 26 edits in trunk

WebCore:

2009-01-27 Brady Eidson <beidson@apple.com>

Reviewed by Dan Bernstein

Rework FrameLoaderClient to work on a CachedFrame basis instead of CachedPage

  • WebCore.base.exp:
  • history/CachedPage.cpp:
  • history/CachedPage.h: (WebCore::CachedPage::cachedMainFrame):
  • loader/EmptyClients.h: (WebCore::EmptyFrameLoaderClient::savePlatformDataToCachedFrame): (WebCore::EmptyFrameLoaderClient::transitionToCommittedFromCachedFrame):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::restoreDocumentState): (WebCore::FrameLoader::commitProvisionalLoad): (WebCore::FrameLoader::transitionToCommitted): (WebCore::FrameLoader::cachePageForHistoryItem):
  • loader/FrameLoaderClient.h:

WebKit/gtk:

2009-01-27 Brady Eidson <beidson@apple.com>

Reviewed by Dan Bernstein

Rework FrameLoaderClient to work on a CachedFrame basis instead of CachedPage

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::savePlatformDataToCachedFrame): (WebKit::FrameLoaderClient::transitionToCommittedFromCachedFrame):
  • WebCoreSupport/FrameLoaderClientGtk.h:

WebKit/mac:

2009-01-27 Brady Eidson <beidson@apple.com>

Reviewed by Dan Bernstein

Rework FrameLoaderClient to work on a CachedFrame basis instead of CachedPage

  • History/WebHistoryItem.mm:
  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::savePlatformDataToCachedFrame): (WebFrameLoaderClient::transitionToCommittedFromCachedFrame):
  • WebKit.order:

WebKit/qt:

2009-01-27 Brady Eidson <beidson@apple.com>

Reviewed by Dan Bernstein

Rework FrameLoaderClient to work on a CachedFrame basis instead of CachedPage

  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::savePlatformDataToCachedFrame): (WebCore::FrameLoaderClientQt::transitionToCommittedFromCachedFrame):
  • WebCoreSupport/FrameLoaderClientQt.h:

WebKit/win:

2009-01-27 Brady Eidson <beidson@apple.com>

Reviewed by Dan Bernstein

Rework FrameLoaderClient to work on a CachedFrame basis instead of CachedPage

  • WebCoreSupport/WebFrameLoaderClient.cpp: (WebFrameLoaderClient::savePlatformDataToCachedFrame): (WebFrameLoaderClient::transitionToCommittedFromCachedFrame):
  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebFrame.cpp:
  • WebFrame.h:

WebKit/wx:

2009-01-27 Brady Eidson <beidson@apple.com>

Reviewed by Dan Bernstein

Rework FrameLoaderClient to work on a CachedFrame basis instead of CachedPage

  • WebKitSupport/FrameLoaderClientWx.cpp: (WebCore::FrameLoaderClientWx::savePlatformDataToCachedFrame): (WebCore::FrameLoaderClientWx::transitionToCommittedFromCachedFrame):
  • WebKitSupport/FrameLoaderClientWx.h:

Jan 26, 2009:

10:58 PM Changeset in webkit [40288] by Simon Fraser
  • 33 edits in trunk/WebCore

2009-01-26 Simon Fraser <Simon Fraser>

Reviewed by Dave Hyatt

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

Add variants of all the methods that compute dirty rects for repainting
with versions that compute those rects relative to some container
RenderBox that acts as a painting root.

8:40 PM Changeset in webkit [40287] by Simon Fraser
  • 7 edits in trunk

2009-01-26 Simon Fraser <Simon Fraser>

Reviewed by David Hyatt

Back out r40285, because it was checked in with no bug number, no
testcase, is rendering change that did not get thorough review,
and broke the Mac build.

7:49 PM Changeset in webkit [40286] by beidson@apple.com
  • 4 edits in trunk/WebCore

2009-01-26 Brady Eidson <beidson@apple.com>

Reviewed by Darin Adler

-Make the recently added back/forward cache logging much better by actually

walking the entire frame tree and indenting the resulting output.

-Fix a null-termination bug in HistoryItem tree logging

  • history/HistoryItem.cpp: (WebCore::HistoryItem::showTreeWithIndent):
  • loader/FrameLoader.cpp: (WebCore::pageCacheLogPrefix): (WebCore::pageCacheLog): (WebCore::FrameLoader::logCanCachePageDecision): (WebCore::FrameLoader::logCanCacheFrameDecision):
  • loader/FrameLoader.h:
7:06 PM Changeset in webkit [40285] by treat@webkit.org
  • 7 edits in trunk

Allow option of hit testing frame content without clipping to the visible
viewport.

6:44 PM Changeset in webkit [40284] by ddkilzer@apple.com
  • 5 edits in trunk/LayoutTests

Fix misspelling of "lowercase" in font-face-locally-installed.html

Reviewed by Dan Bernstein.

  • fast/css/font-face-locally-installed.html: Updated.
  • platform/mac/fast/css/font-face-locally-installed-expected.checksum: Updated.
  • platform/mac/fast/css/font-face-locally-installed-expected.png: Updated.
  • platform/mac/fast/css/font-face-locally-installed-expected.txt: Updated.
5:59 PM Changeset in webkit [40283] by andersca@apple.com
  • 1 edit in trunk/WebKit/mac/ChangeLog

Sam did _not_ review this

5:57 PM Changeset in webkit [40282] by adele@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-01-26 Adele Peterson <adele@apple.com>

Build fix.

  • debugger/Debugger.cpp:
5:56 PM Changeset in webkit [40281] by andersca@apple.com
  • 4 edits in trunk/WebKit/mac

2009-01-26 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Add the ability for plug-ins to make WebKit operate in "modal mode"


  • Plugins/Hosted/NetscapePluginHostProxy.h:
  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WebKit::NetscapePluginHostProxy::NetscapePluginHostProxy): (WebKit::NetscapePluginHostProxy::pluginHostDied): If the plug-in crashes while we're modal, make sure to leave the modal mode.


(WebKit::NetscapePluginHostProxy::setModal):
(WKPCSetModal):

  • Plugins/Hosted/WebKitPluginClient.defs:
5:33 PM Changeset in webkit [40280] by sullivan@apple.com
  • 3 edits in trunk/WebKit/mac

2009-01-26 John Sullivan <sullivan@apple.com>

fixed <rdar://problem/6530053> REGRESSION (Leopard): Shift-tab in http authentication window gets
stuck in the Name field rather than cycling around

Reviewed by Dan Bernstein

  • Panels/English.lproj/WebAuthenticationPanel.nib/designable.nib:
  • Panels/English.lproj/WebAuthenticationPanel.nib/keyedobjects.nib: The two static text fields and the last button all had their "next key view" outlets set to the name field, which caused shift-tab from the name field to do the wrong thing. Fixed by making each selectable view have exactly one "next key view" set to it.
5:08 PM Changeset in webkit [40279] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-01-26 Gavin Barraclough <barraclough@apple.com>

Reviewed by Darin Adler.

Fixes for eq null & neq null, on 64-bit JIT.
https://bugs.webkit.org/show_bug.cgi?id=23559

This patch degrades 64-bit JIT performance on some benchmarks,
due to the whole not-being-incorrect thing.

  • jit/JIT.cpp: (JSC::JIT::privateCompileMainPass):
4:45 PM Changeset in webkit [40278] by weinig@apple.com
  • 6 edits
    3 adds in trunk

WebCore:

2009-01-26 Sam Weinig <sam@webkit.org>

Reviewed by Dan Bernstein.

Fix for https://bugs.webkit.org/show_bug.cgi?id=18141
Acid3 tests 77 and 78 fail on reload due to SVG font race
<rdar://problem/6504899>

Match Opera's behavior of loading SVGFonts as soon as <font-face-uri>
tag is inserted into the document and not at the first use of the font
as is done for CSS web fonts.

This fixes a race condition noticeable by some in the Acid3 test that
happened because the onload event handler for an iframe containing a
<font-face-uri> with an external reference was not delayed by the pending
load of the font.

Test: http/tests/misc/SVGFont-delayed-load.html

  • svg/SVGFontFaceUriElement.cpp: (WebCore::SVGFontFaceUriElement::~SVGFontFaceUriElement): (WebCore::SVGFontFaceUriElement::parseMappedAttribute): Call loadFont if the href attribute changes. (WebCore::SVGFontFaceUriElement::insertedIntoDocument): Call loadFont. (WebCore::SVGFontFaceUriElement::loadFont): Force a load of the font specified in href attribute.
  • svg/SVGFontFaceUriElement.h:

LayoutTests:

2009-01-26 Sam Weinig <sam@webkit.org>

Reviewed by Dan Bernstein.

Test for https://bugs.webkit.org/show_bug.cgi?id=18141
Acid3 tests 77 and 78 fail on reload due to SVG font race
<rdar://problem/6504899>

  • http/tests/misc/SVGFont-delayed-load-expected.txt: Added.
  • http/tests/misc/SVGFont-delayed-load.html: Added.
  • http/tests/misc/resources/SVGFont-delayed-loader.svg: Added.
  • svg/custom/acid3-test-77.html:
  • svg/custom/resources/Acid3Font-loader.svg:
4:38 PM Changeset in webkit [40277] by andersca@apple.com
  • 4 edits in trunk/WebKit/mac

2009-01-26 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Add the ability for a plug-in to show or hide the menu bar.


  • Plugins/Hosted/NetscapePluginHostProxy.h:
  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WebKit::NetscapePluginHostProxy::NetscapePluginHostProxy): (WebKit::NetscapePluginHostProxy::pluginHostDied): (WebKit::NetscapePluginHostProxy::setMenuBarVisible): (WKPCSetMenuBarVisible):
  • Plugins/Hosted/WebKitPluginClient.defs:
4:09 PM Changeset in webkit [40276] by pol@apple.com
  • 13 edits in trunk/LayoutTests

2009-01-26 Pierre-Olivier Latour <pol@apple.com>

Updated the reference images for the video tests that rely on the H264 test movie,
as QuickTime 7.6 apparently slightly changed the results from the 7.5.5 version.

  • platform/mac/media/video-aspect-ratio-expected.checksum:
  • platform/mac/media/video-aspect-ratio-expected.png:
  • platform/mac/media/video-controls-rendering-expected.checksum:
  • platform/mac/media/video-controls-rendering-expected.png:
  • platform/mac/media/video-display-toggle-expected.checksum:
  • platform/mac/media/video-display-toggle-expected.png:
  • platform/mac/media/video-layer-crash-expected.checksum:
  • platform/mac/media/video-layer-crash-expected.png:
  • platform/mac/media/video-transformed-expected.checksum:
  • platform/mac/media/video-transformed-expected.png:
  • platform/mac/media/video-zoom-expected.checksum:
  • platform/mac/media/video-zoom-expected.png:
4:00 PM Changeset in webkit [40275] by timothy@apple.com
  • 2 edits in trunk/WebCore

Change Web Archive serialization back to the binary property list format.

<rdar://problem/6127865> Web archives saved with Safari 4 cannot be read by Safari 3

Reviewed by Brady Eidson.

  • loader/archive/cf/LegacyWebArchive.cpp: (WebCore::LegacyWebArchive::rawDataRepresentation):
3:57 PM Changeset in webkit [40274] by cwzwarich@webkit.org
  • 8 edits
    1 add in trunk

2009-01-26 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Gavin Barraclough.

Bug 23552: Dashcode evaluator no longer works after making ExecStates actual call frames
<https://bugs.webkit.org/show_bug.cgi?id=23552>
<rdar://problem/6398839>

Dashcode will crash when using the evaluator because it saves a global call
frame, even after global code has finished executing, and then uses this as
a launching pad to execute new JS in the evaluator. The fix is to detect
when Dashcode is attempting to do this and execute code from a global call
frame instead.

JavaScriptCore:

  • JavaScriptCore.exp:
  • debugger/Debugger.cpp: (JSC::evaluateInGlobalCallFrame): Added so that WebScriptCallFrame can evaluate JS starting from a global call frame.
  • debugger/Debugger.h:

WebKit/mac:

  • ForwardingHeaders/runtime/Protect.h: Added.
  • WebView/WebScriptDebugDelegate.mm: (-[WebScriptCallFrame _initWithGlobalObject:debugger:caller:debuggerCallFrame:]): Added debugger, a WebScriptDebugger* argument. (-[WebScriptCallFrame evaluateWebScript:]): Detect when Dashcode is using a stale WebScriptCallFrame to execute new JS and evaluate it starting from the global object's global call frame instead.
  • WebView/WebScriptDebugger.h: (WebScriptDebugger::globalObject): Added. (WebScriptDebugger::globalCallFrame): Added.
  • WebView/WebScriptDebugger.mm: (WebScriptDebugger::WebScriptDebugger): Initialize m_globalObject. (WebScriptDebugger::initGlobalCallFrame): Created as a clone of callEvent so that the global call frame can be saved immediately after being created. (WebScriptDebugger::callEvent): Pass 'this' as the debugger argument of WebScriptCallFrame's _initWithGlobalObject method.
3:27 PM Changeset in webkit [40273] by andersca@apple.com
  • 3 edits in trunk/WebKit/mac

2009-01-26 Anders Carlsson <andersca@apple.com>

Reviewed by Oliver Hunt.

Make WKPCInvoke a simpleroutine.


  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WKPCInvoke):
  • Plugins/Hosted/WebKitPluginClient.defs:
3:16 PM Changeset in webkit [40272] by andersca@apple.com
  • 2 edits in trunk/WebCore

2009-01-26 Anders Carlsson <andersca@apple.com>

Fix 64-bit build.


  • WebCore.LP64.exp:
3:14 PM Changeset in webkit [40271] by Dimitri Glazkov
  • 7 edits
    2 moves in trunk/WebCore

2009-01-26 Dmitry Titov <dimich@chromium.org>

Reviewed by Alexey Proskuryakov.

https://bugs.webkit.org/show_bug.cgi?id=23511
Move DOMTimer from bindings/js to page/.
This was prepared by moving all JSC dependencies from DOMTimer
to ScheduledAction earlier, and helps unfork the Chromuim version
of the DOMTimer.

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.scons:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • bindings/js/DOMTimer.cpp: Removed.
  • bindings/js/DOMTimer.h: Removed.
  • page/DOMTimer.cpp: Copied from WebCore/bindings/js/DOMTimer.cpp.
  • page/DOMTimer.h: Copied from WebCore/bindings/js/DOMTimer.h.
3:07 PM Changeset in webkit [40270] by hyatt@apple.com
  • 4 edits in trunk/WebCore

2009-01-26 David Hyatt <hyatt@apple.com>

Eliminate rightmostPosition, lowestPosition and leftmostPosition from RenderFlow. Move the code into
RenderBlock's methods.

Reviewed by Eric Seidel

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::lowestPosition): (WebCore::RenderBlock::rightmostPosition): (WebCore::RenderBlock::leftmostPosition):
  • rendering/RenderFlow.cpp:
  • rendering/RenderFlow.h:
3:06 PM Changeset in webkit [40269] by darin@chromium.org
  • 3 edits in trunk/WebCore

2009-01-26 Darin Fisher <darin@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=23553
Upstream changes from jcampan to fix chromium bugs 6692 and 6694.

  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupListBox::setLoopSelectionNavigation): (WebCore::PopupListBox::PopupListBox): (WebCore::PopupContainer::setLoopSelectionNavigation): (WebCore::PopupListBox::handleKeyEvent): (WebCore::PopupListBox::paintRow): (WebCore::PopupListBox::clearSelection): (WebCore::PopupListBox::selectNextRow): (WebCore::PopupListBox::selectPreviousRow):
  • platform/chromium/PopupMenuChromium.h:
2:54 PM Changeset in webkit [40268] by Dimitri Glazkov
  • 1 edit in trunk/WebCore/WebCore.xcodeproj/project.pbxproj

Remove extraneous WebCore XCode project setting, introduced in previous commit.

2:52 PM Changeset in webkit [40267] by beidson@apple.com
  • 4 edits in trunk/WebCore

2009-01-26 Brady Eidson <beidson@apple.com>

Reviewed by Sam Weinig

Remove some historical CachedPage time stamp fluff.

  • history/CachedPage.cpp: (WebCore::CachedPage::CachedPage):
  • history/CachedPage.h: (WebCore::CachedPage::timeStamp):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::cachePageForHistoryItem):
2:45 PM Changeset in webkit [40266] by hyatt@apple.com
  • 8 edits in trunk/WebCore

2009-01-26 David Hyatt <hyatt@apple.com>

Move absoluteClippedOverflowRect, linesBoundingBox and borderBoundingBox from RenderFlow to RenderInline.

Reviewed by Eric Seidel

  • rendering/RenderBox.cpp: (WebCore::RenderBox::containingBlockHeightForPositioned):
  • rendering/RenderFlow.cpp:
  • rendering/RenderFlow.h:
  • rendering/RenderInline.cpp: (WebCore::RenderInline::linesBoundingBox): (WebCore::RenderInline::absoluteClippedOverflowRect):
  • rendering/RenderInline.h: (WebCore::RenderInline::borderBoundingBox):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateLayerPosition):
  • rendering/RenderTreeAsText.cpp: (WebCore::operator<<):
2:40 PM Changeset in webkit [40265] by Dimitri Glazkov
  • 5 edits in trunk/WebCore

2009-01-26 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=22650
Implement support for "Immutable" attribute to CodeGeneratorJS.pm, thus
allowing to explicitly mark methods as immutable (i.e. returning a new
instance rather than a reference to an existing instance).

  • bindings/js/JSSVGMatrixCustom.cpp: Removed custom methods.
  • bindings/scripts/CodeGeneratorJS.pm: Added check for Immutable

attribute when generating function call.

  • svg/SVGMatrix.idl: Replaced Custom attribute with Immutable on

methods.

2:29 PM Changeset in webkit [40264] by hyatt@apple.com
  • 22 edits in trunk/WebCore

2009-01-26 David Hyatt <hyatt@apple.com>

Eliminate isInlineFlow() now that a RenderBlock can never be an inline flow. isInlineFlow() is now
equivalent to isRenderInline(), so all calls to isInlineFlow() have been replaced with isRenderInline().

Reviewed by Eric Seidel

  • dom/Text.cpp: (WebCore::Text::rendererIsNeeded):
  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::placeBoxesHorizontally): (WebCore::shouldDrawTextDecoration):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::hitTestContents): (WebCore::InlineMinMaxIterator::next): (WebCore::RenderBlock::calcInlinePrefWidths):
  • rendering/RenderBlock.h: (WebCore::RenderBlock::isBlockFlow):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::clientWidth): (WebCore::RenderBox::clientHeight): (WebCore::RenderBox::nodeAtPoint): (WebCore::RenderBox::imageChanged): (WebCore::RenderBox::offsetForPositionedInContainer): (WebCore::RenderBox::containingBlockWidthForPositioned): (WebCore::RenderBox::containingBlockHeightForPositioned):
  • rendering/RenderBox.h: (WebCore::RenderBox::width): (WebCore::RenderBox::height): (WebCore::RenderBox::size): (WebCore::RenderBox::frameRect):
  • rendering/RenderContainer.cpp: (WebCore::RenderContainer::updateBeforeAfterContentForContainer):
  • rendering/RenderFlow.cpp: (WebCore::RenderFlow::dirtyLinesFromChangedChild): (WebCore::RenderFlow::createInlineBox): (WebCore::RenderFlow::paintLines): (WebCore::RenderFlow::hitTestLines): (WebCore::RenderFlow::absoluteClippedOverflowRect): (WebCore::RenderFlow::lowestPosition): (WebCore::RenderFlow::rightmostPosition): (WebCore::RenderFlow::leftmostPosition):
  • rendering/RenderFlow.h: (WebCore::RenderFlow::borderBoundingBox):
  • rendering/RenderInline.h: (WebCore::RenderInline::isRenderInline):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateLayerPosition): (WebCore::RenderLayer::intersectsDamageRect): (WebCore::RenderLayer::boundingBox):
  • rendering/RenderListItem.cpp: (WebCore::getParentOfFirstLineBox):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::getAbsoluteRepaintRectWithOutline): (WebCore::RenderObject::styleWillChange): (WebCore::RenderObject::firstLineStyle):
  • rendering/RenderObject.h:
  • rendering/RenderSVGInline.cpp: (WebCore::RenderSVGInline::createInlineBox):
  • rendering/RenderSVGText.cpp: (WebCore::RenderSVGText::createInlineBox):
  • rendering/RenderTableRow.cpp: (WebCore::RenderTableRow::nodeAtPoint):
  • rendering/RenderTableSection.cpp: (WebCore::RenderTableSection::nodeAtPoint):
  • rendering/RenderText.cpp: (WebCore::isInlineFlowOrEmptyText):
  • rendering/RenderTreeAsText.cpp: (WebCore::operator<<):
  • rendering/bidi.cpp: (WebCore::bidiNext): (WebCore::bidiFirst): (WebCore::appendRunsForObject): (WebCore::RenderBlock::createLineBoxes): (WebCore::RenderBlock::computeHorizontalPositionsForLine): (WebCore::RenderBlock::layoutInlineChildren): (WebCore::inlineFlowRequiresLineBox): (WebCore::requiresLineBox): (WebCore::RenderBlock::skipTrailingWhitespace): (WebCore::RenderBlock::skipLeadingWhitespace): (WebCore::RenderBlock::findNextLineBreak):
2:27 PM Changeset in webkit [40263] by andersca@apple.com
  • 9 edits in trunk/WebKit/mac

2009-01-26 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Implement using plug-in objects as constructors, and setting and getting properties from a plug-in object.


  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WKPCBooleanAndDataReply):
  • Plugins/Hosted/NetscapePluginInstanceProxy.h: (WebKit::NetscapePluginInstanceProxy::Reply::): (WebKit::NetscapePluginInstanceProxy::BooleanAndDataReply::BooleanAndDataReply): Rename NPObjectInvokeReply to BooleanAndDataReply.


  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::addValueToArray): Fix a cut and paste error.


(WebKit::NetscapePluginInstanceProxy::demarshalValueFromArray):
Handle NPObjects.


  • Plugins/Hosted/ProxyInstance.h:
  • Plugins/Hosted/ProxyInstance.mm: (WebKit::ProxyField::valueFromInstance): (WebKit::ProxyField::setValueToInstance): Call the ProxyInstance method.


(WebKit::ProxyInstance::~ProxyInstance):
Release the NPObject.


(WebKit::ProxyInstance::supportsConstruct):
Ask the plug-in host if an instance supports construct.


(WebKit::ProxyInstance::fieldValue):
(WebKit::ProxyInstance::setFieldValue):
Call the plug-in host methods.


  • Plugins/Hosted/WebKitPluginHostTypes.h: Rename ObjectValueType to JSObjectValueType, and add NPObjectValueType.
1:53 PM Changeset in webkit [40262] by hyatt@apple.com
  • 1 edit in trunk/LayoutTests/platform/mac/fast/table/form-with-table-style-expected.txt

Land new layout tests now that display:compact support has been removed.

1:52 PM Changeset in webkit [40261] by hyatt@apple.com
  • 1 edit in trunk/LayoutTests/fast/leaks/001-expected.txt

Land new layout tests now that display:compact support has been removed.

1:51 PM Changeset in webkit [40260] by hyatt@apple.com
  • 1 edit in trunk/LayoutTests/platform/mac/fast/forms/form-hides-table-expected.txt

Land new layout tests now that display:compact support has been removed.

1:51 PM Changeset in webkit [40259] by hyatt@apple.com
  • 7 edits in trunk/LayoutTests/platform/mac/fast/compact

Land new layout tests now that display:compact support has been removed.

1:50 PM Changeset in webkit [40258] by hyatt@apple.com
  • 10 edits in trunk/WebCore

2009-01-26 David Hyatt <hyatt@apple.com>

Drop support for display:compact.

(1) display:compact is no longer part of CSS2.1.
(2) Shipping Safari has completely broken compact support with severely broken rendering
(all of the compact text just piles on top of the text in the inserted block and on top of itself as well).
(3) This has been broken in WebKit for years (ever since blocks converted to painting lines directly) without
anyone even noticing.

Reviewed by Eric Seidel

  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::placeBoxesHorizontally):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::handleSpecialChild): (WebCore::RenderBlock::handleRunInChild): (WebCore::RenderBlock::layoutBlockChildren): (WebCore::RenderBlock::paint): (WebCore::RenderBlock::paintObject): (WebCore::RenderBlock::nodeAtPoint): (WebCore::RenderBlock::positionForCoordinates): (WebCore::shouldCheckLines): (WebCore::getHeightForLineCount): (WebCore::RenderBlock::renderName):
  • rendering/RenderBlock.h:
  • rendering/RenderBox.cpp: (WebCore::RenderBox::sizesToIntrinsicWidth):
  • rendering/RenderBox.h: (WebCore::RenderBox::width): (WebCore::RenderBox::height): (WebCore::RenderBox::size): (WebCore::RenderBox::frameRect):
  • rendering/RenderFlow.cpp: (WebCore::RenderFlow::paintLines): (WebCore::RenderFlow::hitTestLines): (WebCore::RenderFlow::absoluteClippedOverflowRect): (WebCore::RenderFlow::linesBoundingBox):
  • rendering/RenderObject.h: (WebCore::RenderObject::isInline):
  • rendering/bidi.cpp: (WebCore::RenderBlock::computeHorizontalPositionsForLine): (WebCore::RenderBlock::layoutInlineChildren):
  • rendering/bidi.h: (WebCore::BidiRun::BidiRun):
1:44 PM Changeset in webkit [40257] by mrowe@apple.com
  • 2 edits in trunk/WebKit/mac

Fix the build.

Remove -Wformat=2 from the warning flags as newer versions of GCC emit
warnings about non-literal format strings for uses of our UI_STRING macro.

  • Configurations/Base.xcconfig:
1:42 PM Changeset in webkit [40256] by pol@apple.com
  • 2 edits in trunk/WebKitTools

2009-01-26 Pierre-Olivier Latour <pol@apple.com>

Tweaked again earlier fix, this time just to print a warning and not abort if
attempting to generate pixel results and Perian is installed.

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

  • Scripts/run-webkit-tests:
12:55 PM Changeset in webkit [40255] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-01-26 Adam Langley <agl@google.com>

Reviewed by Nikolas Zimmermann.

gtk2drawing: fix compiler warnings

g_object_set_data expects a gpointer as the last argument. Cast the

TRUE/FALSE values to avoid a warning.

gtk_tree_view_column_new doesn't return a GtkWidget.

  • platform/gtk/gtk2drawing.c: (setup_widget_prototype): (moz_gtk_get_combo_box_inner_button): (moz_gtk_get_combo_box_button_inner_widgets): (ensure_combo_box_widgets): (moz_gtk_get_combo_box_entry_inner_widgets): (moz_gtk_get_combo_box_entry_arrow): (ensure_combo_box_entry_widgets): (ensure_toolbar_widget): (ensure_menu_bar_item_widget): (ensure_menu_popup_widget): (ensure_menu_item_widget): (ensure_image_menu_item_widget): (ensure_menu_separator_widget): (ensure_check_menu_item_widget): (ensure_tree_header_cell_widget): (moz_gtk_entry_paint):
12:14 PM Changeset in webkit [40254] by hyatt@apple.com
  • 4 edits in trunk

2009-01-26 David Hyatt <hyatt@apple.com>

Change display:run-in handling so that we no longer reuse RenderBlock as though it is an inline flow.
Instead we create a new RenderInline, move the RenderBlock's children underneath it, and then set the
inline as the new element's renderer.

Reviewed by Dan Bernstein

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::handleRunInChild):
  • rendering/RenderInline.cpp: (WebCore::RenderInline::renderName):
11:43 AM Changeset in webkit [40253] by mrowe@apple.com
  • 5 edits in trunk/WebKit

Clean up after r40240.

Rubber-stamped by Sam Weinig.

  • WebKit.xcodeproj/project.pbxproj: Reinstate WebKitPluginHost.defs.
  • Configurations/Base.xcconfig: Don't dead code strip in debug builds for now as it leads to link errors.
  • Plugins/Hosted/HostedNetscapePluginStream.mm: Revert change that is no longer needed now that WebKitPluginHost.defs

is back in the build.

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

Fix an assertion failure when Safari loads an error page

Reviewed by Alexey Proskuryakov.

  • WebFrame.cpp: (WebFrame::loadData): Use the two-argument version of KURL and MarshallingHelpers::BSTRToKURL, for reasons mentioned in r40248.
10:46 AM Changeset in webkit [40251] by christian@webkit.org
  • 2 edits in trunk/WebKitTools

2009-01-26 Christian Dywan <christian@twotoasts.de>

Rubber stamped by Holger Freyther.

  • GtkLauncher/main.c: (main): Initialize threads, which is required for libSoup.
8:46 AM Changeset in webkit [40250] by Simon Hausmann
  • 3 edits in trunk/WebKit/qt

2009-01-26 David Boddie <dboddie@trolltech.com>

Reviewed by Simon Hausmann.

Doc: qdoc fix for a link to a property and added a basic description of the offline caching and storage features from HTML 5.

8:44 AM Changeset in webkit [40249] by Dimitri Glazkov
  • 2 edits in trunk/WebCore

2009-01-26 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=23480
Add missing method stubs to BackForwardListChromium.

  • history/BackForwardListChromium.cpp: (WebCore::BackForwardList::backItem): (WebCore::BackForwardList::forwardItem):
7:19 AM Changeset in webkit [40248] by Adam Roben
  • 5 edits in trunk/WebKit/win

Fix an assertion on launch in KURL::KURL

All URL strings passed in to the WebKit API need to be passed to
MarshallingHelpers::BSTRToKURL so that they may be properly parsed.

Reviewed by Alexey Proskuryakov.

  • MarshallingHelpers.cpp: (MarshallingHelpers::BSTRToKURL): Changed to use the two-argument constructor for KURL so that the strings will be parsed rather than assumed to be in the correct encoding/form.
  • WebResource.cpp: (WebResource::initWithData):
  • WebURLResponse.cpp: (WebURLResponse::initWithURL):
  • WebView.cpp: (WebView::userAgentForURL): (WebView::copyURL): Changed to use MarshallingHelpers::BSTRToKURL instead of trying to do the work manually.
6:03 AM QtWebKitJournal edited by Simon Hausmann
(diff)
2:16 AM Changeset in webkit [40247] by Darin Adler
  • 3 edits in trunk/LayoutTests

2009-01-26 Darin Adler <Darin Adler>

Requested by David Hyatt.

  • platform/mac/fast/table/form-with-table-style-expected.checksum: Updated to reflect the progression; we now render things we used to fail to render.
  • platform/mac/fast/table/form-with-table-style-expected.png: Ditto.
Note: See TracTimeline for information about the timeline view.