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

Timeline



Jul 14, 2007:

4:39 PM Changeset in webkit [24295] by beidson
  • 3 edits in trunk/WebCore

Third shot at blind Qt/GDK build fix

  • platform/gdk/TemporaryLinkStubs.cpp: (FTPDirectoryDocument::createTokenizer):
  • platform/qt/TemporaryLinkStubs.cpp: (FTPDirectoryDocument::createTokenizer):
4:25 PM Changeset in webkit [24294] by beidson
  • 2 edits in trunk/WebCore/platform

RUBBERSTAMPED BY SAM WEINIG - Try to (actually) fix Qt and GDK

4:21 PM Changeset in webkit [24293] by beidson
  • 4 edits in trunk

Fix Windows build and set the pref to allow FTP directory listings in Windows nightlies

4:13 PM Changeset in webkit [24292] by zimmermann
  • 610 edits
    8 adds in branches/feature-branch

Fixes: ​http://bugs.webkit.org/show_bug.cgi?id=5940 (<svg> inside <body> incorrectly extends over top/bottom borders)
Fixes: ​http://bugs.webkit.org/show_bug.cgi?id=14145 (RenderSVGContainer should not inherit from RenderContainer)

Make RenderSVGContainer inherit from RenderObject directly, instead of RenderContainer.
Itdoes a lot of things that we don't need for SVG, and by removing this inheritance
we can finally fix the differences about SVG object positioning, as we don't need
parent translation (tx/ty). Fixes SVG text rendering within XHTML.

Fix requireLayer() function on RenderSVGRoot, as it can actually be a layer now.
This change is based on Rob's patch at 5940 - fixing z-order indexing.

Change RenderSVGRoot renderName() and update all layout tests, because of this change.

Fix absoluteRects() function for RenderSVGInlineText & RenderSVGTSpan.
Checked extensively using WebInspector, to assure all rects are correctly displayed.

Testcases fixed: junk-data.svg, missing-xlink.svg hixie/013.xml (all regressions!)
(junk-data.svg & missing-xlink.svg previously relied on SVG text painting out of <svg> overflow rect.

Fix these testcases, as within this new RenderSVGContainer concept, that bug is now fixed, and breaks them)

Testcases added: custom/absolute-sized-svg-in-xhtml.xhtml, custom/inline-svg-in-xhtml.xml (Mozilla XTech demo)

3:59 PM Changeset in webkit [24291] by weinig
  • 2 edits in trunk/WebCore

Fix Tiger build.

  • config.h:
3:58 PM Changeset in webkit [24290] by beidson
  • 3 edits in trunk/WebCore

Blind attempt at (hopefully) fixing the Qt and GDK builds

  • platform/gdk/TemporaryLinkStubs.cpp: (FTPDirectoryDocument::FTPDirectoryDocument):
  • platform/qt/TemporaryLinkStubs.cpp: (FTPDirectoryDocument::FTPDirectoryDocument):
3:25 PM Changeset in webkit [24289] by beidson
  • 22 edits
    4 adds in trunk

JavaScriptCore:

Reviewed by Sam Weinig

Initial check-in for <rdar://problem/3154486> - Supporting FTP directory listings in the browser

  • wtf/Platform.h: Add ENABLE_FTPDIR feature to handle building on platforms that don't have the proper network-layer support

WebCore:

Reviewed by Sam Weinig

Initial check-in for <rdar://problem/3154486> - Supporting FTP directory listings in the browser

Development has taken place on both Leopard and Windows (windows enabling patch will follow shortly)
Support will only exist on platforms whose network layers (ie, ResourceHandle) support piping the raw
FTP Directory data into the engine.

This is known to include Leopard, Windows, and CURL based platforms - probably others
Known incompatible platforms include Tiger.

The FTP listing page is built in HTML. An external HTML document can be provided as a template for the
directory listing. This document is parsed, and expected to have a <table> with a certain class in it.
As FTP directory listings come in, each entry is added to this table as a row.

If no HTML template is provided to work with, a very basic empty document is hard coded that contains only
this special table.

Upgrades to the look and feel of the directory listings can take place largely in this HTML template which,
admittedly, is conspicuously absent in this initial checkin

  • WebCore.exp: New Settings symbols for WebKit
  • WebCore.xcodeproj/project.pbxproj:
  • config.h: If BUILDING_ON_TIGER, disable FTP directory listing support
  • dom/DOMImplementation.cpp: (WebCore::DOMImplementation::createDocument): Create an FTPDirectoryDocument if the mime type is "application/x-ftp-directory"
  • loader/FTPDirectoryDocument.cpp: Added. (WebCore::FTPDirectoryTokenizer::isWaitingForScripts): (WebCore::FTPDirectoryTokenizer::checkBuffer): (WebCore::FTPDirectoryTokenizer::FTPDirectoryTokenizer): Building on HTML tokenizer, this facilitates parsing FTP Directory listings in the engine (WebCore::FTPDirectoryTokenizer::appendEntry): Add a table row for a directory listing to the document (WebCore::FTPDirectoryTokenizer::createTDForFilename): Create the TD for the filename with the anchor in it

(WebCore::processFilesizeString): Prettify the filesize
(WebCore::wasLastDayOfMonth):
(WebCore::processFileDateString): Prettify the date
(WebCore::FTPDirectoryTokenizer::parseAndAppendOneLine):
(WebCore::FTPDirectoryTokenizer::loadDocumentTemplate): Loads the HTML template that FTP directory listings

can build on top of

(WebCore::FTPDirectoryTokenizer::createBasicDocument): Creates a most basic document (body and one table) to

append the rows to in case the document template is not loaded and parsed

(WebCore::FTPDirectoryTokenizer::write): Add the FTP listing to the buffer and parse entries out of it
(WebCore::FTPDirectoryTokenizer::finish):

(WebCore::FTPDirectoryDocument::FTPDirectoryDocument): Special document-type for FTP directory listings
(WebCore::FTPDirectoryDocument::createTokenizer):

  • loader/FTPDirectoryDocument.h: Added. (WebCore::FTPDirectoryDocument::isImageDocument):


  • loader/FTPDirectoryParser.cpp: Added. (WebCore::parseOneFTPLine): Adapted from ParseFTPList.cpp from Firefox - parses most known FTP directory listing styles into discrete FTP directory entries
  • loader/FTPDirectoryParser.h: Added. (WebCore::ListState::ListState): Maintains FTP Parser state (WebCore::ListResult::ListResult): Represents one FTP directory listing (WebCore::ListResult::clear):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::checkContentPolicy): If the Settings say to bypass the policy check for "application/x-ftp-directory", skip it. This will allow people to test this feature even if their browser/WebKit app would otherwise not allow it
  • page/Settings.cpp: (WebCore::Settings::Settings): (WebCore::Settings::setFTPDirectoryTemplatePath): Set the path to the FTP listing document template (WebCore::Settings::setForceFTPDirectoryListings): Set to bypass the policy delegate check for mime type "application/x-ftp-directory"
  • page/Settings.h: (WebCore::Settings::ftpDirectoryTemplatePath): (WebCore::Settings::forceFTPDirectoryListings):
  • platform/Logging.cpp: Add an "LogFTP" logging channel
  • platform/Logging.h:
  • platform/mac/LoggingMac.mm: (WebCore::InitializeLoggingChannelsIfNecessary):
  • platform/MimeTypeRegistry.cpp: (WebCore::initialiseSupportedNonImageMimeTypes): Add "application/x-ftp-directory"
  • platform/SharedBuffer.cpp: (WebCore::SharedBuffer::createWithContentsOfFile): Stub for now
  • platform/SharedBuffer.h:
  • platform/mac/SharedBufferMac.mm: (WebCore::SharedBuffer::createWithContentsOfFile): Load contents of the file into an NSData, then wrap that NSData

WebKit:

Reviewed by Sam Weinig

Initial check-in for <rdar://problem/3154486> - Supporting FTP directory listings in the browser

  • WebView/WebPreferenceKeysPrivate.h: Added preference keys for the FTP template location, as well as to force FTP directory listings, bypassing the policy delegate. This is necessary to test the new feature until browser policy delegate support is added.
  • WebView/WebPreferences.m: (-[WebPreferences _setFTPDirectoryTemplatePath:]): (-[WebPreferences _ftpDirectoryTemplatePath]): (-[WebPreferences _setForceFTPDirectoryListings:]): (-[WebPreferences _forceFTPDirectoryListings]):
  • WebView/WebPreferencesPrivate.h:
  • WebView/WebView.mm: (-[WebView _updateWebCoreSettingsFromPreferences:]):
11:02 AM Changeset in webkit [24288] by ap
  • 3 edits
    2 adds in trunk

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=14618
REGRESSION(13464): drivehq.com login form doesn't work

Test: http/tests/misc/post-submit-button.html

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::appendFormData): Append a submit even if its value is empty.
10:04 AM Changeset in webkit [24287] by bdash
  • 6 edits
    3 adds in trunk

2007-07-14 Cameron Zwarich <​cwzwarich@uwaterloo.ca>

Reviewed by Darin.

Fixes ​http://bugs.webkit.org/show_bug.cgi?id=13517,

​http://bugs.webkit.org/show_bug.cgi?id=14237, and
the failure of test js1_5/Scope/regress-185485.js

Bug 13517: DOM Exception 8 in finance.aol.com sub-page
Bug 14237: Javascript "var" statement interprets initialization in the topmost function scope

  • kjs/nodes.cpp: (VarDeclNode::evaluate):
  • tests/mozilla/expected.html:

2007-07-14 Cameron Zwarich <​cwzwarich@uwaterloo.ca>

Reviewed by Darin.

Added tests for the following bugs:
​http://bugs.webkit.org/show_bug.cgi?id=13517
​http://bugs.webkit.org/show_bug.cgi?id=14237

Bug 13517: DOM Exception 8 in finance.aol.com sub-page
Bug 14237: Javascript "var" statement interprets initialization in the topmost function scope

  • fast/js/resources/vardecl-blocks-init.js: Added.
  • fast/js/resources/vardecl-preserve-arguments.js: Updated.
  • fast/js/vardecl-blocks-init-expected.txt: Added.
  • fast/js/vardecl-blocks-init.html: Added.
  • fast/js/vardecl-preserve-arguments-expected.txt: Updated result.
9:53 AM Changeset in webkit [24286] by bdash
  • 2 edits in trunk/WebKitTools

2007-07-15 Mark Rowe <​mrowe@apple.com>

  • Scripts/generate-coverage-data: Don't fail if WebKitBuild directory does not exist.
1:21 AM Changeset in webkit [24285] by oliver
  • 4 edits in trunk/WebKit/win

Reviewed by Darin and Alexey.

Fix for <rdar://problem/5231528> Inline input of International text (IME)

​http://bugs.webkit.org/show_bug.cgi?id=14331

This patch adds IME support to WebKit/win, it currently does not support
reconversion (<rdar://problem/5334818>) and has issues with the chinese
IMEs (<rdar://problem/5334826>)

  • WebEditorClient.cpp: (WebEditorClient::respondToChangedSelection): (WebEditorClient::handleInputMethodKeypress): Prevent the initial keydown for an IME from triggering a keypressed event
  • WebView.cpp: (WebView::WebView): (WebView::keyUp): (WebView::keyDown): (WebViewWndProc): (IMMDict::dict): (IMMDict::IMMDict):

Dynamic loader for IME libraries

(WebView::getIMMContext):
(WebView::releaseIMMContext):
(WebView::prepareCandidateWindow):
(selectionInsideMarkedText):
(setSelectionToEndOfRange):
(WebView::resetIME):
(WebView::updateSelectionForIME):
(WebView::selectionChanged):
(getCompositionString):
(compositionToUnderlines):

Helper functions

(WebView::onIMEStartComposition):
(WebView::onIMEComposition):
(WebView::onIMEEndComposition):
(WebView::onIMEChar):
(WebView::onIMENotify):
(WebView::onIMERequest):
(WebView::onIMESelect):
(WebView::onIMESetContext):

IME event handling, so far most of these are not implemented, but the bulk of functionality
is performed the the composition event handlers

  • WebView.h:

Jul 13, 2007:

11:40 PM Changeset in webkit [24284] by oliver
  • 3 edits in trunk/LayoutTests

Reviewed by Mark.

Correct expected pixel results

  • fast/forms/input-appearance-readonly-expected.checksum:
  • fast/forms/input-appearance-readonly-expected.png:
11:03 PM Changeset in webkit [24283] by oliver
  • 2 edits in trunk/WebCore

Reviewed by Sam.

Fix for

<rdar://problem/5335265> REGRESSION: WebCore::DragController crashes if WebDragSourceActionDHTML cleared
​http://bugs.webkit.org/show_bug.cgi?id=13652

Reorder logic for drag initiation: we can always preflight the clipboard, regardless as to whether
we are performing any dhtml drag handling

  • page/EventHandler.cpp: (WebCore::EventHandler::handleDrag):
9:51 PM Changeset in webkit [24282] by pewtermoose
  • 2 edits in trunk/WebCore

Reviewed by Tim Hatcher.

Bug 14401: Semi-regression: Can no longer copy anything from the inspector
​http://bugs.webkit.org/show_bug.cgi?id=14401

  • page/inspector/inspector.css:
6:49 PM Changeset in webkit [24281] by andersca
  • 2 edits in trunk/WebKit

Reviewed by Maciej.

<rdar://problem/5290103> Assert failure when loading page with multipart resource


Don't try to call the delegate method if the resource object doesn't exist in the
identifier map. When a multipart resource has finished loading one part, it is removed from the
web view identifier map.


This is not an ideal fix, a better fix would be to special-case multipart resources and not remove
them when the first part has finished loading. I've filed <rdar://problem/5335034> to track doing that.


  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::dispatchDidReceiveAuthenticationChallenge): (WebFrameLoaderClient::dispatchDidCancelAuthenticationChallenge): (WebFrameLoaderClient::dispatchDidReceiveResponse): (WebFrameLoaderClient::willCacheResponse): (WebFrameLoaderClient::dispatchDidReceiveContentLength): (WebFrameLoaderClient::dispatchDidFinishLoading): (WebFrameLoaderClient::dispatchDidFailLoading):
5:20 PM Changeset in webkit [24280] by weinig
  • 2 edits in trunk/LayoutTests

Fix results.

  • fast/text/word-break-run-rounding-expected.txt:
4:07 PM Changeset in webkit [24279] by bdakin
  • 23 edits in trunk/WebCore

Reviewed by Hyatt.

Fix for <rdar://problem/5304742> A hang occurs when attempting to
display web clip banner at ​http://www.signonsandiego.com/sports/
chargers/index.html

Here is the actual fix.

  • rendering/RenderInline.cpp: (WebCore::RenderInline::absoluteRects): Set topLevel to false when we recurse to get our children's absoluteRects.
  • rendering/RenderInline.h: New boolean parameter topLevel that defaults to true.
  • rendering/RenderObject.cpp: (WebCore::RenderObject::absoluteRects):
  • rendering/RenderObject.h:

And these are just other implementations/declarations of this
(virtual) function that have to add the parameter. No behavior
change here.

  • rendering/RenderPath.cpp: (WebCore::RenderPath::absoluteRects):
  • rendering/RenderPath.h:
  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::absoluteRects):
  • rendering/RenderSVGContainer.h:
  • rendering/RenderSVGHiddenContainer.cpp: (WebCore::RenderSVGHiddenContainer::absoluteRects):
  • rendering/RenderSVGHiddenContainer.h:
  • rendering/RenderSVGImage.cpp: (WebCore::RenderSVGImage::absoluteRects):
  • rendering/RenderSVGImage.h:
  • rendering/RenderSVGInlineText.cpp: (WebCore::RenderSVGInlineText::absoluteRects):
  • rendering/RenderSVGInlineText.h:
  • rendering/RenderSVGTSpan.cpp: (WebCore::RenderSVGTSpan::absoluteRects):
  • rendering/RenderSVGTSpan.h:
  • rendering/RenderSVGText.cpp: (WebCore::RenderSVGText::absoluteRects):
  • rendering/RenderSVGText.h:
  • rendering/RenderText.cpp: (WebCore::RenderText::absoluteRects):
  • rendering/RenderText.h:
  • rendering/RenderView.cpp: (WebCore::RenderView::absoluteRects):
  • rendering/RenderView.h:
3:45 PM Changeset in webkit [24278] by weinig
  • 4 edits
    4 adds in trunk

LayoutTests:

Reviewed by Hyatt.

Run rounding makes word-break:break-all/word not functional

  • fast/text/word-break-run-rounding-expected.checksum: Added.
  • fast/text/word-break-run-rounding-expected.png: Added.
  • fast/text/word-break-run-rounding-expected.txt: Added.
  • fast/text/word-break-run-rounding.html: Added.

WebCore:

Reviewed by Hyatt.

Run rounding makes word-break:break-all/word not functional

Test: fast/text/word-break-run-rounding.html

  • rendering/RenderText.cpp: (WebCore::RenderText::calcPrefWidths): Update the maximum width only on word boundaries to avoid rounding errors.
  • rendering/bidi.cpp: (WebCore::RenderBlock::findNextLineBreak): Integrated breakAll with the midWordBreak/wrapW mechanism. Also made the wrapW upper bound more accurate by resetting it when tmpW is committed.
3:37 PM Changeset in webkit [24277] by weinig
  • 14 edits
    8 adds in trunk

LayoutTests:

Reviewed by Geoff Garen.

Tests for <rdar://problem/5318394>
http site can read/write content of https site in same domain

Adds ssl configuration information and tests.

  • http/conf/apache2-httpd.conf:
  • http/conf/cygwin-httpd.conf:
  • http/conf/httpd.conf:
  • http/conf/webkit-httpd.pem: Added.
  • http/tests/security/cross-frame-access-expected.txt:
  • http/tests/security/cross-frame-access-first-time-expected.txt:
  • http/tests/security/cross-frame-access-protocol-expected.txt: Added.
  • http/tests/security/cross-frame-access-protocol.html: Added.
  • http/tests/security/cross-frame-access-put-expected.txt:
  • http/tests/ssl: Added.
  • http/tests/ssl/resources: Added.
  • http/tests/ssl/resources/cross-frame-access-protocol-iframe.html: Added.
  • http/tests/ssl/verify-ssl-enabled-expected.txt: Added.
  • http/tests/ssl/verify-ssl-enabled.php: Added.

WebCore:

Reviewed by Geoff Garen.

Patch for <rdar://problem/5318394>
http site can read/write content of https site in same domain

Add protocol and port checks in isSafeScript.

  • bindings/js/kjs_window.cpp: (KJS::Window::isSafeScript):

WebKitTools:

Reviewed by Geoff Garen.

Add support for running SSL tests over HTTPS.

  • DumpRenderTree/DumpRenderTree.m: (runTest):
  • DumpRenderTree/FrameLoadDelegate.m: (-[FrameLoadDelegate webView:didFailProvisionalLoadWithError:forFrame:]):
  • Scripts/run-webkit-httpd:
  • Scripts/run-webkit-tests:
3:20 PM Changeset in webkit [24276] by andersca
  • 3 edits
    1 add in trunk/WebCore

Reviewed by Mitz Pettel.

<rdar://problem/5298870>
REGRESSION: In Yahoo Mail modal dialog, OK and Cancel buttons do not work, and descriptive text is missing (14327)


​http://bugs.webkit.org/show_bug.cgi?id=14327
REGRESSION (r21367): dialogArguments not set in window generated by showModalDialog

When creating popup windows/dialogs, prevent the window properties from being cleared by the initial frame load.


  • loader/FrameLoader.cpp: (WebCore::FrameLoader::FrameLoader): (WebCore::FrameLoader::createWindow): (WebCore::FrameLoader::clear): (WebCore::FrameLoader::begin): (WebCore::FrameLoader::open):
  • loader/FrameLoader.h: (WebCore::FrameLoader::setShouldClearWindowProperties):
  • manual-tests/modal-dialog-arguments.html: Added.
2:45 PM Changeset in webkit [24275] by thatcher
  • 2 edits in trunk/WebKit

Reviewed by Oliver Hunt.

<rdar://problem/5333766> Can't include WebEditingDelegatePrivate.h

  • WebView/WebEditingDelegatePrivate.h: Changed the include to be a framework include, so other projects can use this header.
2:37 PM Changeset in webkit [24274] by kevino
  • 1 edit in branches/wx-port-alpha/trunk/WebKitTools/Scripts/webkitdirs.pm

The isGdk function got lost during merges. This restores scripts and such that expect this to be defined.

2:29 PM Changeset in webkit [24273] by sfalken
  • 1 copy in tags/Safari-522.14.2b

New tag.

2:29 PM Changeset in webkit [24272] by sfalken
  • 1 edit in trunk/WebKit/win/WebKit.vcproj/VERSION

Bump version for submit

1:26 PM Changeset in webkit [24271] by yongjzha
  • 3 edits in S60/branches/3.1m/JavaScriptCore

ligman, reviewed by Yongjun

DESC: stop the recursion in ArgumentListNode's destructor.
​http://bugs.webkit.org/show_bug.cgi?id=14578

  • kjs/nodes.cpp: (ArgumentListNode::~ArgumentListNode):
  • kjs/nodes.h:
1:24 PM Changeset in webkit [24270] by yongjzha
  • 3 edits in S60/trunk/JavaScriptCore

ligman, reviewed by Yongjun

DESC: stop the recursion in ArgumentListNode's destructor.
​http://bugs.webkit.org/show_bug.cgi?id=14578

  • kjs/nodes.cpp: (ArgumentListNode::~ArgumentListNode):
  • kjs/nodes.h:
1:17 PM Changeset in webkit [24269] by sfalken
  • 3 edits in trunk/WebCore

Update ResourceError related code to account for changes in CF interfaces.

Reviewed by Darin, Ada.

  • platform/network/ResourceError.h: Change Mac conditionals to CF to share more code.
  • platform/network/cf/ResourceErrorCF.cpp: (WebCore::ResourceError::ResourceError): Modified based on CFErrorRef changes (WebCore::ResourceError::unpackPlatformError): Ported from ResourceErrorMac.mm. (WebCore::ResourceError::operator CFErrorRef): Ported from ResourceErrorMac.mm. (WebCore::ResourceError::operator CFStreamError): Modified based on CFErrorRef changes.
12:06 PM Changeset in webkit [24268] by andersca
  • 3 edits in trunk/WebKit

Reviewed by Darin and Maciej.

<rdar://problem/5271096> panic after Safari stress test, caused by port leak

Replace uses of -[NSObject performSelector:withObject:afterDelay:] with CFRunLoopTimer. performSelector
causes the target (the WebHTMLView in this case) to be retained until the timer fires. Furthermore, when
running the PLT or iBench, the timers will not fire until the main loop is entered (usually after running all tests).


This means that the timers and ports will not be released until after the test has finished running.


  • WebView/WebHTMLView.mm: (-[WebHTMLViewPrivate dealloc]): (-[WebHTMLView _cancelUpdateMouseoverTimer]): (-[WebHTMLView _updateMouseoverWithFakeEvent]): (-[WebHTMLView _updateMouseoverTimerCallback:]): (-[WebHTMLView _frameOrBoundsChanged]): (-[WebHTMLView _updateActiveState]): (-[WebHTMLView _updateActiveStateTimerCallback:]): (-[WebHTMLView viewWillMoveToWindow:]): (-[WebHTMLView viewDidMoveToWindow]): (-[WebHTMLView mouseDown:]): (-[WebTextCompleteController dealloc]):
  • WebView/WebHTMLViewInternal.h:
11:55 AM Changeset in webkit [24267] by antti
  • 5 edits
    2 adds in trunk

LayoutTests:

Reviewed by Adele.


Test for <rdar://problem/5192256>
click() in onchange handler causes another change event (causes hang at ​http://forums.whirlpool.net.au/)

  • fast/events/onchange-click-hang-expected.txt: Added.
  • fast/events/onchange-click-hang.html: Added.

WebCore:

Reviewed by Adele.


Fix <rdar://problem/5192256>
click() in onchange handler causes another change event (causes hang at ​http://forums.whirlpool.net.au/)


Guard against generating a simulated event from within a simulated event in the same node. Try to
match Firefox behavior.

  • dom/EventTargetNode.cpp: (WebCore::EventTargetNode::dispatchSimulatedMouseEvent): (WebCore::EventTargetNode::dispatchSimulatedClick):
  • dom/Node.cpp: (WebCore::Node::Node):
  • dom/Node.h:
11:29 AM Changeset in webkit [24266] by antti
  • 3 edits
    2 adds in trunk

LayoutTests:

Reviewed by Darin.


Test for x <rdar://problem/5333387>
Combination of selection and click() on checkbox crashes

  • fast/dynamic/checkbox-selection-crash-expected.txt: Added.
  • fast/dynamic/checkbox-selection-crash.html: Added.

WebCore:

Reviewed by Darin.


Fix <rdar://problem/5333387>
Combination of selection and click() on checkbox crashes


Ensure there is no pending style update before doing synchronous paint. Under certain
circumstances this ends up doing style recalc in middle of paint() which may
for example tear down the rendering tree being painted, with bad results.

  • dom/ContainerNode.cpp: (WebCore::ContainerNode::setActive):
11:16 AM Changeset in webkit [24265] by ddkilzer
  • 2 edits in trunk/WebKitTools

2007-07-13 David Kilzer <​ddkilzer@apple.com>

Reviewed by Darin.

Added support for -hhelp argument and verify that all command line arguments parse

correctly.

  • Scripts/run-webkit-httpd:
11:13 AM Changeset in webkit [24264] by ddkilzer
  • 1 edit in trunk/LayoutTests/ChangeLog

Fixed comment.

11:11 AM Changeset in webkit [24263] by antti
  • 3 edits
    2 adds in trunk

LayoutTests:

Reviewed by Adele.


Test for <rdar://problem/5333365>
Calling click() is very slow

  • fast/forms/slow-click-expected.txt: Added.
  • fast/forms/slow-click.html: Added.

WebCore:

Reviewed by Adele.


Fix <rdar://problem/5333365>
Calling click() is very slow


There is a 100ms delay in ContainerNode::setActive() to do an activation effect. It is used for
simulated mouse activations (using keyboard for example). However, it shouldn't be done when calling from javascript
(through click() interface) since it hangs script execution.

  • html/HTMLElement.cpp: (WebCore::HTMLElement::click):
10:21 AM Changeset in webkit [24262] by bdash
  • 4 edits in trunk

Update versioning of trunk.

9:34 AM Changeset in webkit [24261] by bdash
  • 3 edits in trunk/LayoutTests

2007-07-14 Mark Rowe <​mrowe@apple.com>

Reviewed by Sam.

Apply the change to Apache 2 configuration from r24204 to the other two Apache configuration files.

  • http/conf/cygwin-httpd.conf:
  • http/conf/httpd.conf:
7:16 AM Changeset in webkit [24260] by bdash
  • 2 edits in branches/Safari-2.0-dashboard

Versioning.

7:10 AM Changeset in webkit [24259] by bdash
  • 2 edits in branches/Safari-522

Versioning.

7:10 AM Changeset in webkit [24258] by bdash
  • 2 edits in branches/Safari-522/WebKit

Merge r23963.

Reviewed by Darin.

Fix for: <rdar://problem/5292988> domain names shouldn't contain ignorable characters

  • Misc/WebNSURLExtras.m: (isLookalikeCharacter): Renamed. Also excludes any non-printable character, any character considered as whitespace that isn't already converted to a space by ICU, any ignorable character, and any character excluded in Mozilla's blacklist: ​http://kb.mozillazine.org/Network.IDN.blacklist_chars (allCharactersInIDNScriptWhiteList):
7:10 AM Changeset in webkit [24257] by bdash
  • 4 edits in branches/Safari-522/WebCore

Merge r23529.

Reviewed by Geoff.

<rdar://problem/5211677>
-[WebPreferences setJavaEnabled:] only disables java inside <applet>, not <object>

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::requestObject): If the object is a Java MIME type and Java is disabled, don't load the plug-in.
  • platform/MimeTypeRegistry.cpp: (WebCore::MimeTypeRegistry::isJavaAppletMIMEType): Clean this up and add another applet MIME type.
5:00 AM Changeset in webkit [24256] by bdash
  • 15 edits in trunk

2007-07-13 Mitz Pettel <​mitz@webkit.org>

Reviewed by Dave Hyatt.

Tests: fast/repaint/box-shadow-h.html

fast/repaint/box-shadow-v.html

  • rendering/RenderHTMLCanvas.cpp: (WebCore::RenderHTMLCanvas::layout): Added a call to adjustOverflowForBoxShadow().
  • rendering/RenderImage.cpp: (WebCore::RenderImage::layout): Ditto.
  • rendering/RenderPartObject.cpp: (WebCore::RenderPartObject::layout): Ditto.
  • rendering/RenderReplaced.cpp: (WebCore::RenderReplaced::RenderReplaced): Initialize m_hasOverflow. (WebCore::RenderReplaced::~RenderReplaced): Added. Removes this object from the overflow rect map if it has overflow. (WebCore::RenderReplaced::shouldPaint): Account for overflow. (WebCore::RenderReplaced::adjustOverflowForBoxShadow): Added. Creates or removes an entry for this object in the global overflow rect map. (WebCore::RenderReplaced::overflowHeight): Added. (WebCore::RenderReplaced::overflowWidth): Added. (WebCore::RenderReplaced::overflowLeft): Added. (WebCore::RenderReplaced::overflowTop): Added. (WebCore::RenderReplaced::overflowRect): Added.
  • rendering/RenderReplaced.h:

2007-07-13 Mitz Pettel <​mitz@webkit.org>

Reviewed by Dave Hyatt.

  • fast/repaint/box-shadow-h-expected.checksum:
  • fast/repaint/box-shadow-h-expected.png:
  • fast/repaint/box-shadow-h-expected.txt:
  • fast/repaint/box-shadow-h.html:
  • fast/repaint/box-shadow-v-expected.checksum:
  • fast/repaint/box-shadow-v-expected.png:
  • fast/repaint/box-shadow-v-expected.txt:
  • fast/repaint/box-shadow-v.html:
4:51 AM Changeset in webkit [24255] by bdash
  • 8 edits
    4 adds in trunk

2007-07-13 Mitz Pettel <​mitz@webkit.org>

Reviewed by Dave Hyatt.

Test: fast/lists/marker-before-empty-inline.html

  • rendering/RenderBlock.h:
  • rendering/RenderListItem.cpp: (WebCore::getParentOfFirstLineBox): Changed to skip empty inline flows that do not actually generate any line boxes. Limited the nested lists quirk to the case where the list is a child of the list item, to match Firefox.
  • rendering/bidi.cpp: (WebCore::requiresLineBox): Added. Factored out of skipWhitespace. (WebCore::RenderBlock::generatesLineBoxesForInlineChild): Added. Checkes whether the child or any of its siblings following it will generate a line box in the flow. (WebCore::RenderBlock::skipWhitespace):

2007-07-13 Mitz Pettel <​mitz@webkit.org>

Reviewed by Dave Hyatt.

  • editing/pasteboard/4861080-expected.checksum:
  • editing/pasteboard/4861080-expected.png:
  • editing/pasteboard/4861080-expected.txt:
  • fast/lists/marker-before-empty-inline-expected.checksum: Added.
  • fast/lists/marker-before-empty-inline-expected.png: Added.
  • fast/lists/marker-before-empty-inline-expected.txt: Added.
  • fast/lists/marker-before-empty-inline.html: Added.
4:47 AM Changeset in webkit [24254] by bdash
  • 5 edits in trunk/WebCore

2007-07-13 Holger Hans Peter Freyther <​zecke@selfish.org>

Reviewed by Alexey.

Move HTTP methods to parse Content-Type from xmlhttprequest.cpp
which is LGPL to HTTPParsers.cpp. As both files were written by the
same author and the advice to move them came from him I think this move
is what is meant to be done.

Move filenameFromHTTPContentDisposition to HTTPParsers as well. This parse
function uses Vector<String>::split and doesn't honor quoting. This needs
to be fixed in later versions.

​http://bugs.webkit.org/show_bug.cgi?id=5954 and ​http://bugs.webkit.org/show_bug.cgi?id=14059
benefit from this change.

  • platform/network/HTTPParsers.cpp: (WebCore::filenameFromHTTPContentDisposition): (WebCore::extractMIMETypeFromMediaType): (WebCore::extractCharsetFromMediaType):
  • platform/network/HTTPParsers.h:
  • platform/network/cf/ResourceResponseCFNet.cpp:
  • xml/xmlhttprequest.cpp: (WebCore::XMLHttpRequest::responseMIMEType): (WebCore::XMLHttpRequest::didReceiveResponse):
Note: See TracTimeline for information about the timeline view.