Timeline



Oct 12, 2005:

6:41 PM Changeset in webkit [10841]
  • 4 copies
    7 deletes in tags/Safari-421

This commit was manufactured by cvs2svn to create tag 'Safari-421'.

6:41 PM Changeset in webkit [10840] by thatcher
  • 3 edits in branches/Safari-Leo-branch

Versioning to 421

6:08 PM Changeset in webkit [10839] by thatcher
  • 10 edits in trunk/WebKit

Reviewed by Darin.

Define WebNSInt and WebNSUInt to wrap around NSInt on Leopard and still build on Tiger
Once building on Tiger isn't needed we will drop WebNSInt and use NSInt

  • WebView.subproj/WebDefaultResourceLoadDelegate.m: (-[WebDefaultResourceLoadDelegate webView:resource:didReceiveContentLength:fromDataSource:]):
  • WebView.subproj/WebFrame.m: (-[WebFrame _sendRemainingDelegateMessagesWithIdentifier:response:length:error:]):
  • WebView.subproj/WebLoader.m: (-[NSURLProtocol didReceiveData:lengthReceived:]):
  • WebView.subproj/WebResourceLoadDelegate.h:
  • WebView.subproj/WebUIDelegate.h:
  • WebView.subproj/WebView.h:
  • WebView.subproj/WebView.m: (-[WebView _mouseDidMoveOverElement:modifierFlags:]): (-[WebView spellCheckerDocumentTag]):
  • WebView.subproj/WebViewInternal.h:
  • WebView.subproj/WebViewPrivate.h:
5:53 PM Changeset in webkit [10838] by vicki
  • 3 edits in trunk/LayoutTests
  • update expected results for <rdar://problem/4288266> createElement does not validate names when called in an HTML document (4566)
  • dom/html/level1/core/hc_documentinvalidcharacterexceptioncreateelement-expected.txt:
  • dom/html/level1/core/hc_documentinvalidcharacterexceptioncreateelement1-expected.txt:
5:49 PM Changeset in webkit [10837] by vicki
  • 2 edits in trunk/WebCore

Reviewed by Hyatt.

  • fix <rdar://problem/4288266> createElement does not validate names when called in an HTML document (4566)
  • khtml/html/html_documentimpl.cpp: (DOM::HTMLDocumentImpl::createElement):
5:35 PM Changeset in webkit [10836] by bdakin
  • 5 edits
    4 adds in trunk

* empty log message *

2:02 PM Changeset in webkit [10835] by darin
  • 2 edits in trunk/WebKit
  • WebView.subproj/WebPolicyDelegate.h: Fix a comment.
1:58 PM Changeset in webkit [10834] by darin
  • 1 edit
    4 deletes in trunk/LayoutTests
  • dom/html/level2/html/frame-expected.checksum: Removed.
  • dom/html/level2/html/frame-expected.png: Removed.
  • dom/html/level2/html/iframe-expected.checksum: Removed.
  • dom/html/level2/html/iframe-expected.png: Removed.
1:28 PM Changeset in webkit [12208] by thatcher
  • 2 edits in trunk/JavaScriptGlue

Setup the proper Framework search path for the Default config.
Prepping for a TOT submission and build by B&I.

  • JavaScriptGlue.xcodeproj/project.pbxproj:

Oct 11, 2005:

11:55 PM Changeset in webkit [10833]
  • 1 copy
    5 deletes in branches/Safari-Leo-branch

This commit was manufactured by cvs2svn to create branch
'Safari-Leo-branch'.

11:55 PM Changeset in webkit [10832] by thatcher
  • 2 edits in trunk/WebCore

Reviewed by Eric.

Crash: Drag element with CSS ::Before absolute positioning applied to it
​http://bugzilla.opendarwin.org/show_bug.cgi?id=4088

  • kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::khtmlMouseMoveEvent):
6:00 PM Changeset in webkit [10831] by thatcher
  • 2 edits in trunk/WebKit

Reviewed by Darin.

Test for 10.4 because of <rdar://problem/4243463>

  • WebView.subproj/WebHTMLView.m: (-[WebHTMLView conversationIdentifier]):
5:44 PM Changeset in webkit [10830] by thatcher
  • 3 edits in trunk/WebCore

Reviewed by Darin.

workaround for <rdar://problem/4294625> to fix our build

  • kwq/KWQPointArray.h:
  • kwq/KWQSize.h:
4:14 PM Changeset in webkit [10829]
  • 209 copies
    15 deletes in tags/WebCore-416~11

This commit was manufactured by cvs2svn to create tag
'WebCore-416~11'.

4:14 PM Changeset in webkit [10828] by thatcher
  • 2 edits in branches/Safari-Den-branch/WebCore

Versioning to WebCore-416.11

3:37 PM Changeset in webkit [10827] by adele
  • 2 edits in trunk/WebKit

Rolling out fix for ​http://bugzilla.opendarwin.org/show_bug.cgi?id=5195
since it caused:

REGRESSION text areas draw focus ring around each glyph, no caret in text fields
​http://bugzilla.opendarwin.org/show_bug.cgi?id=5335

  • WebView.subproj/WebHTMLView.m: (-[WebHTMLView _propagateDirtyRectsToOpaqueAncestors]): (-[WebHTMLView _recursiveDisplayRectIfNeededIgnoringOpacity:isVisibleRect:rectIsVisibleRectForView:topView:]): (-[WebHTMLView _recursiveDisplayAllDirtyWithLockFocus:visRect:]):
3:26 PM Changeset in webkit [10826]
  • 1 copy
    7 deletes in tags/perf-search-end

This commit was manufactured by cvs2svn to create tag
'perf-search-end'.

3:26 PM Changeset in webkit [10825] by ggaren
  • 2 edits in trunk/JavaScriptCore

Added regexp_object.lut.h build phase from JavaScriptCore
to JavaScriptCore+SVG.

Reviewed by mitz.

3:01 PM Changeset in webkit [10824] by ggaren
  • 2 edits in trunk/JavaScriptCore

Fixed build bustage from last checkin (stray characters
in the project file).

Reviewed by mitz.

2:49 PM Changeset in webkit [10823] by ggaren
  • 2 edits in trunk/JavaScriptCore

New JavaScriptCore test results to reflect the last change.

  • tests/mozilla/expected.html:
2:12 PM Changeset in webkit [10822] by thatcher
  • 2 edits in branches/Safari-2-0-branch/WebCore

Merged fix from TOT to Safari-2-0-branch

2005-10-10 Justin Garcia <​justin.garcia@apple.com>

Reviewed by eric

<rdar://problem/4283218> M-JPEG / Server Push and JavaScript

Ignore multipart loads in numRequests, call checkCompleted() after each section is complete.

  • khtml/misc/loader.cpp:
2:00 PM Changeset in webkit [10821] by ggaren
  • 1 edit in trunk/JavaScriptCore/ChangeLog

Small fixup to last Changelog entry.

1:57 PM Changeset in webkit [10820] by ggaren
  • 1 edit
    2 adds in trunk/LayoutTests

Added a layout test for the new special properties of the global RegExp object.
(See JavaSriptCore changelog.)

  • fast/js/regexp-caching-expected.txt: Added.
  • fast/js/regexp-caching.html: Added.
1:56 PM Changeset in webkit [10819] by thatcher
  • 2 edits in branches/Safari-Den-branch/WebCore

Merged fix from TOT to Safari-Den-branch

2005-10-10 Justin Garcia <​justin.garcia@apple.com>

Reviewed by eric

<rdar://problem/4283218> M-JPEG / Server Push and JavaScript

Ignore multipart loads in numRequests, call checkCompleted() after each section is complete.

  • khtml/misc/loader.cpp:
1:43 PM Changeset in webkit [10818] by ggaren
  • 7 edits in trunk/JavaScriptCore
  • Implemented caching of match state inside the global RegExp object (lastParen, leftContext, rightContext, lastMatch, input).

exec(), test(), match(), search(), and replace() now dipatch regular
expression matching through the RegExp object's performMatch function,
to facilitate caching. This replaces registerRegexp and
setSubPatterns.

  • Implemented the special '$' aliases (e.g. RegExp.input aliases to RegExp.$_).
  • Moved support for backreferences into the new static hash table used for other special RegExp properties. Truncated backreferences at $9 to match IE, FF, and the "What's New in Netscape 1.2?" doc. (String.replace still supports double-digit backreferences.)
  • Tweaked RegExp.prototype.exec to handle ginormous values in lastIndex.

Fixes 11 -- count em, 11 -- JavaScriptCore tests.

Reviewed by NOBODY (OOPS!).

  • JavaScriptCore.xcodeproj/project.pbxproj: Added regexp_object.lut.h
  • kjs/create_hash_table: Tweaked to allow for more exotic characters.

We now rely on the compiler to catch illegal
identifiers.

  • kjs/regexp.cpp: (KJS::RegExp::RegExp):
  • kjs/regexp_object.cpp: (RegExpProtoFuncImp::callAsFunction): (RegExpObjectImp::RegExpObjectImp): (RegExpObjectImp::performMatch): (RegExpObjectImp::arrayOfMatches): (RegExpObjectImp::backrefGetter): (RegExpObjectImp::getLastMatch): (RegExpObjectImp::getLastParen): (RegExpObjectImp::getLeftContext): (RegExpObjectImp::getRightContext): (RegExpObjectImp::getOwnPropertySlot): (RegExpObjectImp::getValueProperty): (RegExpObjectImp::put): (RegExpObjectImp::putValueProperty):
  • kjs/regexp_object.h: (KJS::RegExpObjectImp::):
  • kjs/string_object.cpp: (substituteBackreferences): (replace): (StringProtoFuncImp::callAsFunction):
1:33 PM Changeset in webkit [10817] by justing
  • 2 edits in trunk/WebCore

Reviewed by eric

<rdar://problem/4283218> M-JPEG / Server Push and JavaScript

Ignore multipart loads in numRequests, call checkCompleted() after each section is complete.

  • khtml/misc/loader.cpp:
12:16 PM Changeset in webkit [10816] by adele
  • 10 edits in trunk

LayoutTests:

Rolling out Darin's previous change.

  • fast/clip/014-expected.checksum: Updated.
  • fast/clip/014-expected.png: Updated.

WebCore:

Rolling out fix for ​http://bugzilla.opendarwin.org/show_bug.cgi?id=5195
since it caused:

REGRESSION text areas draw focus ring around each glyph, no caret in text fields
​http://bugzilla.opendarwin.org/show_bug.cgi?id=5335

And updated expected results.

  • fast/clip/014-expected.checksum: Updated.
  • fast/clip/014-expected.png: Updated.
  • kwq/KWQTextArea.h:
  • kwq/KWQTextArea.mm: (-[KWQTextAreaTextView displayRectIgnoringOpacity:]): (-[KWQTextAreaTextView textView:shouldDrawInsertionPointInRect:color:turnedOn:]):
  • kwq/KWQTextField.h:
  • kwq/KWQTextField.mm: (-[KWQTextFieldController setInDrawingMachinery:]): (-[KWQTextFieldController textView:shouldDrawInsertionPointInRect:color:turnedOn:]): (-[KWQTextField displayRectIgnoringOpacity:]): (-[KWQSecureTextField displayRectIgnoringOpacity:]): (-[KWQSearchField displayRectIgnoringOpacity:]):
  • kwq/KWQWidget.h:
  • kwq/KWQWidget.mm: (QWidget::QWidget): (QWidget::~QWidget): (QWidget::paint): (QWidget::setDeferFirstResponderChanges):
2:33 AM Changeset in webkit [10815] by eseidel
  • 2 edits in trunk/WebKitTools

Bug #: none
Submitted by: eseidel
Reviewed by: hyatt

Checks for leaks in ImageDiff too.

  • Scripts/run-webkit-tests:
2:19 AM Changeset in webkit [10814] by eseidel
  • 2 edits in trunk/WebCore

Bug #: 5278
Submitted by: Rob Buis <​rwlbuis@xs4all.nl>
Reviewed by: eseidel

Fix SVGSymbolElementImpl to not warn when inheriting viewbox.
​http://bugzilla.opendarwin.org/show_bug.cgi?id=5278

  • ksvg2/svg/SVGUseElementImpl.cpp: (SVGUseElementImpl::close):
2:11 AM Changeset in webkit [10813] by eseidel
  • 4 edits in trunk/WebCore

Bug #: 5301
Submitted by: Julien Palmas <​julien.palmas@mac.com>
Reviewed by: eseidel

Fixes WebCore+SVG build under gcc 3.3
​http://bugzilla.opendarwin.org/show_bug.cgi?id=5301

  • WebCore+SVG/DrawCanvasItem.mm:
  • WebCore+SVG/DrawView.mm: (-[DrawDocument drawRect:initWithDrawView:]):
  • kwq/KWQSlot.mm:

Oct 10, 2005:

11:55 PM Changeset in webkit [10812] by ggaren
  • 2 edits in trunk/WebCore
  • Fixed <rdar://problem/4245682> Regression: CrashTracer: 923 crashes in Safari at com.apple.WebCore: KJS::Window::clear + 132

Reviewed by mjs.

No test case because this crash isn't reliably reproducible.

  • khtml/ecma/kjs_window.cpp: (KJS::Window::Window): Initialize m_returnValueSlot to 0, so that we know not to write to it until it's explicitly set to a valid address.
8:45 PM Changeset in webkit [10811] by darin
  • 3 edits in trunk/LayoutTests

The old results were incorrectly clipped. This always worked fine in the
browser, but didn't draw correctly in DumpRenderTree until the bug was fixed.

  • fast/clip/014-expected.checksum: Updated.
  • fast/clip/014-expected.png: Updated.
7:49 PM Changeset in webkit [10810] by darin
  • 7 edits in trunk/WebCore

Reviewed by Maciej.

The first half of the fix alone broke DumpRenderTree. Better now.

  • kwq/KWQWidget.h: Made paint non-virtual (it had no reason to be virtual in KWQ) and added an isPainting function.
  • kwq/KWQWidget.mm: (QWidget::QWidget): Added painting boolean (actually a count). (QWidget::~QWidget): Check that painting is 0 when the widget is destroyed. (QWidget::paint): Use SPI to draw in a way that works no matter what context we are in. (QWidget::isPainting): Return true if inside the paint function.
  • kwq/KWQTextArea.h: Remove inDrawingMachinery field since we can use isPainting now.
  • kwq/KWQTextArea.mm: (-[KWQTextAreaTextView textView:shouldDrawInsertionPointInRect:color:turnedOn:]): Use isPainting() instead of inDrawingMachinery. inDrawingMachinery required that we use displayRectIgnoringOpacity: to paint, which we no longer do.
  • kwq/KWQTextField.h: More of the same.
  • kwq/KWQTextField.mm: (-[KWQTextFieldController textView:shouldDrawInsertionPointInRect:color:turnedOn:]): Ditto.
4:48 PM Changeset in webkit [10809] by eseidel
  • 5 edits in trunk/WebCore

Bug #: 5320
Submitted by: darin
Reviewed by: eseidel

Mem leak fixes in xslt code.
​http://bugzilla.opendarwin.org/show_bug.cgi?id=5320

  • khtml/xml/dom_docimpl.cpp: (DocumentImpl::applyXSLTransform): (DocumentImpl::setTransformSourceDocument):
  • khtml/xml/xml_tokenizer.cpp: (khtml::parseXMLDocumentFragment):
  • khtml/xsl/xslt_processorimpl.cpp: (DOM::XSLTProcessorImpl::XSLTProcessorImpl): (DOM::XSLTProcessorImpl::~XSLTProcessorImpl): (DOM::stylesheetLoadFunc): (DOM::XSLTProcessorImpl::transformDocument): (DOM::XSLTProcessorImpl::documentFromXMLDocPtr):
  • khtml/xsl/xslt_processorimpl.h: (DOM::XSLTProcessorImpl::stylesheet): (DOM::XSLTProcessorImpl::sourceDocument):
4:32 PM Changeset in webkit [12207]
  • 1 copy in branches/Safari-2-0-branch/JavaScriptGlue/Info.plist

This commit was manufactured by cvs2svn to create branch
'Safari-2-0-branch'.

4:32 PM Changeset in webkit [12206] by darin
  • 16 edits in trunk/JavaScriptGlue

Reviewed by Eric.

  • converted tabs to spaces, NULL to 0, added some (void), got rid of prefixes in headers (reserved for compiler/library)
  • Info.plist:
  • JSBase.cpp:
  • JSBase.h:
  • JSObject.cpp:
  • JSObject.h:
  • JSRun.cpp:
  • JSRun.h:
  • JSUtils.cpp:
  • JSUtils.h:
  • JSValueWrapper.cpp:
  • JSValueWrapper.h:
  • JavaScriptGlue.cpp:
  • JavaScriptGlue.h:
  • UserObjectImp.cpp:
  • UserObjectImp.h:
11:54 AM Changeset in webkit [12205] by darin
  • 2 edits
    1 add in trunk/JavaScriptGlue
  • .cvsignore: Added. Ignore Makefile.in and Makefile.
12:16 AM Changeset in webkit [10808] by hyatt
  • 1 edit in trunk/WebCore/khtml/rendering/bidi.cpp

Bug #:
Submitted by:
Reviewed by:

12:15 AM Changeset in webkit [10807] by hyatt
  • 1 edit in branches/Safari-2-0-branch/WebCore/khtml/rendering/bidi.cpp

Bug #:
Submitted by:
Reviewed by:

Oct 9, 2005:

11:35 PM Changeset in webkit [10806] by darin
  • 5 edits in trunk/WebKitTools
  • Scripts/check-dom-results: Dump ".xhtml" for tests in the xhtml directory.
  • Scripts/cvs-apply: Handle added/deleted files in the current directory.
  • Scripts/cvs-unapply: Ditto.
  • Scripts/run-webkit-tests: Add a few more false-positive leaks.
8:50 PM Changeset in webkit [10805] by hyatt
  • 2 edits in /

Land fix for bugzilla bug 4974, make sure to only move below floats in line layout if white-space is normal.
One case was using !isPre and including nowrap incorrectly as a result. Patch from koivisto.

Reviewed by hyatt

  • khtml/rendering/bidi.cpp: (khtml::RenderBlock::findNextLineBreak):
6:28 PM Changeset in webkit [10804] by darin
  • 2 edits in trunk/WebKit

Reviewed by Maciej.

This was actually a Tiger regression. When AppKit added a new code path for rendering
NSView, our special hack for doing layout when we draw didn't work any more. So we
were able to fix this without adding any API.

  • WebView.subproj/WebHTMLView.m: (-[WebHTMLView _recursiveDisplayRectIgnoringOpacity:inContext:topView:]): Added. Does the same thing that other _recursiveDisplay methods do.
4:15 PM Changeset in webkit [10803] by thatcher
  • 1 edit in trunk/WebKitTools/DumpRenderTree/TextInputController.m

Fixes the following build error when building Deployment:
DumpRenderTree/TextInputController.m: In function '-[TextInputController textInput]':
DumpRenderTree/TextInputController.m:87: warning: class 'NSView <WebDocumentView>' does not implement the 'NSTextInput' protocol

3:57 PM Changeset in webkit [10802] by darin
  • 1 edit in trunk/JavaScriptCore/ChangeLog

Updated change log to mention one more bug fixed.

3:56 PM Changeset in webkit [10801] by darin
  • 5 edits
    2 adds in trunk

JavaScriptCore:

Reviewed by Maciej; some changes done after review.

Retested all tests to be sure nothing broke; added layout test for bug 5280.

  • kjs/config.h: Removed TIME_WITH_SYS_TIME define. Also set HAVE_SYS_TIMEB_H for the APPLE case (the latter is accurate but irrelevant).
  • kjs/date_object.h: Reformatted. Removed unnecessary include of "function_object.h". Moved declarations of helper classes and functions into the cpp file.
  • kjs/date_object.cpp: Removed code at top to define macros to use CoreFoundation instead of POSIX date functions. (KJS::styleFromArgString): Tweaked to return early instead of using a variable. (KJS::formatLocaleDate): Tweaked to check for undefined rather than checking argument count. (KJS::formatDate): Made parameter const. (KJS::formatDateUTCVariant): Ditto. (KJS::formatTime): Ditto. (KJS::DateProtoFuncImp::callAsFunction): Use gmtime_r and localtime_r instead of gmtime and localtime. (KJS::DateObjectImp::callAsFunction): Use localtime_r instead of localtime. (KJS::ymdhmsToSeconds): Renamed from ymdhms_to_seconds. Changed computation to avoid possible overflow if year is an extremely large or small number. (KJS::makeTime): Removed code to move large month numbers from tm_mon to tm_year; this was to accomodate CFGregorianDate, which is no longer used (and didn't handle negative values). (KJS::parseDate): Renamed from KRFCDate_parseDate; changed to return a value in milliseconds rather than in seconds. Reformatted the code. Changed to use UTF8String() instead of ascii(), since ascii() is not thread safe. Changed some variables back from int to long to avoid trouble if the result of strtol does not fit in an int (64-bit issue only).

LayoutTests:

  • fast/js/date-negative-setmonth-expected.txt: Added.
  • fast/js/date-negative-setmonth.html: Added.

Oct 8, 2005:

9:45 PM Changeset in webkit [10800] by darin
  • 3 edits
    2 adds in trunk

JavaScriptCore:

Reviewed by Geoff.
Tweaked and landed by Darin.

  • kjs/date_object.cpp: (KJS::skipSpacesAndComments): Take a pointer, and advance it past spaces, and also past anything enclosed in parentheses. (KJS::KRFCDate_parseDate): Use skipSpacesAndComments wherever we formerly had code to skip spaces.

LayoutTests:

  • fast/js/date-parse-comments-test-expected.txt: Added.
  • fast/js/date-parse-comments-test.html: Added.
9:36 PM Changeset in webkit [10799] by darin
  • 1 edit
    3 adds in trunk/LayoutTests
  • fast/xsl/xslt-entity-enc.xml: Added.
  • fast/xsl/resources/xslt-entity-enc.xsl: Added.
  • fast/xsl/xslt-entity-enc-expected.txt: Added.
9:25 PM Changeset in webkit [10798] by darin
  • 3 edits in trunk/JavaScriptCore

Reviewed, tweaked, and landed by Darin.

  • kjs/config.h: Make sure HAVE_MMAP and HAVE_SBRK are off for Win32. Turn HAVE_ERRNO_H on for Mac OS X. Sort defines so they are easy to compare with each other. Remove #undef of DEBUG_COLLECTOR.
  • pcre/pcre_exec.c: (match): Work around strange MSVC complaint by splitting the definition of a local variable into a separate declaration and initialization.
9:17 PM Changeset in webkit [10797] by darin
  • 7 edits
    2 adds in trunk

WebCore:

Reviewed by Maciej.

No layout test added: only affects plain text view and requires loading the
file in multiple chunks.

  • WebCore.exp: Added WebCoreTextDecoder.
  • kwq/WebCoreTextDecoder.h: Added.
  • kwq/WebCoreTextDecoder.mm: Added.
  • WebCore.xcodeproj/project.pbxproj: Added files, made header private (SPI).

WebKit:

Reviewed by Maciej.

  • WebView.subproj/WebTextRepresentation.m: (-[WebTextRepresentation finishedLoadingWithDataSource:]): Call flushReceivedData on the WebTextView so it can decode any final bytes.
  • WebView.subproj/WebTextView.h: Added WebCoreTextDecoder field and flushReceivedData method.
  • WebView.subproj/WebTextView.m: (-[WebTextView dealloc]): Release WebCoreTextDecoder. (-[WebTextView appendReceivedData:fromDataSource:]): Create a WebCoreTextDecoder to decode the text; use the textEncodingName from the data source. Use it to decode instead of the data source's stringWithData. (-[WebTextView flushReceivedData]): Call flush on the decoder and append any last bytes to the text view.
8:50 PM Changeset in webkit [10796] by darin
  • 1 edit
    4 adds in trunk/LayoutTests

Reviewed by Hyatt.

  • fast/css/acid2-pixel-expected.checksum: Added.
  • fast/css/acid2-pixel-expected.png: Added.
  • fast/css/acid2-pixel-expected.txt: Added.
  • fast/css/acid2-pixel.html: Added.
8:37 PM Changeset in webkit [10795] by darin
  • 3 edits in trunk/LayoutTests

Reviewed by Geoff Garen.

  • fast/js/date-big-setmonth.html: Changed to compare expected result with a Date value rather than with a constant. Also write out final result as a date rather than as a constant.
  • fast/js/date-big-setmonth-expected.txt: Updated.
8:31 PM Changeset in webkit [10794] by darin
  • 5 edits
    2 adds in trunk

LayoutTests:

  • fast/forms/form-data-encoding-expected.txt: Added.
  • fast/forms/form-data-encoding.html: Added.

WebCore:

Reviewed by Maciej.

Added one layout test.

  • khtml/html/html_formimpl.cpp: (DOM::HTMLFormElementImpl::formData): Pass true to the fromUnicode function to request entity encoding. (DOM::FormDataList::appendString): Ditto.
  • kwq/KWQTextCodec.h: Add boolean parameter to request entity-style encoding for characters that can't be expressed in the desired encoding.
  • kwq/KWQTextCodec.mm: (QTextCodec::fromUnicode): Added code to entity-encode characters that can't be expressed in the desired encoding.
7:57 PM Changeset in webkit [10793] by darin
  • 4 edits
    5 adds in trunk

LayoutTests:

Reviewed by Darin.

  • editing/input/text-input-controller-expected.txt: Added.
  • editing/input/text-input-controller.html: Added.

WebKitTools:

Reviewed, rearranged and landed by Darin.

  • DumpRenderTree/DumpRenderTree.m: (-[WaitUntilDoneDelegate webView:windowScriptObjectAvailable:]): Create a text input controller and put it in a property of the window object.
  • DumpRenderTree/TextInputController.h: Added.
  • DumpRenderTree/TextInputController.m: Added.
  • DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj: Added files.
7:36 PM Changeset in webkit [10792] by darin
  • 2 edits
    1 add in trunk/WebCore

Reviewed by Darin.

  • kwq/KWQListBox.mm: (KWQTableViewTypeSelectCallback): Added. (-[KWQTableView finalize]): Deallocate the UCTypeSelect object. (-[KWQTableView dealloc]): Ditto. (-[KWQTableView keyUp:]): If character typed is a graphic character, pass it along to the UCTypeSelect object and use it to type select, otherwise flush the object.
  • manual-tests/select-element-type-select.html: Added.
7:14 PM Changeset in webkit [10791] by darin
  • 12 edits in trunk

LayoutTests:

  • fast/dom/HTMLElement/bdo.html: Updated test, no longer says to expect failure.
  • fast/dom/HTMLElement/bdo-expected.checksum: Correct results.
  • fast/dom/HTMLElement/bdo-expected.png: Ditto.
  • fast/dom/HTMLElement/bdo-expected.txt: Ditto.
  • fast/text/international/bidi-LDB-2-HTML-expected.checksum: Updated to expect correct results for the section done with <bdo>.
  • fast/text/international/bidi-LDB-2-HTML-expected.png: Ditto.
  • fast/text/international/bidi-LDB-2-HTML-expected.txt: Ditto.

WebCore:

Reviewed by Darin.

Fixes the results of some existing layout tests.

  • khtml/xml/dom_elementimpl.h: Add eBDO to the list of entry types so there's a way to keep <bdo> dir attribute mappings separate from those of other elements.
  • khtml/html/html_elementimpl.cpp: (HTMLElementImpl::mapToEntry): Add special case to use eBDO for the dir attribute on elements with the <bdo> tag. (HTMLElementImpl::parseMappedAttribute): Use bidi-override for the unicode-bidi CSS property when setting it based on the dir attribute on a <bdo> tag. Along with the change above this obviates the need for an HTMLElementImpl subclass for <bdo>, but we could refactor it that way later if we decide there's an advantage to doing it that way.
  • khtml/css/html4.css: Remove the rules about <bdo>. They are insufficient, and now they are also unnecessary with the changes above.
6:46 PM Changeset in webkit [10790] by darin
  • 8 edits
    12 adds in trunk

LayoutTests:

Test cases for ​http://bugzilla.opendarwin.org/show_bug.cgi?id=4898
(Failures in dbaron's bidi ordering tests).

  • fast/dom/HTMLElement/bdo-expected.txt: Updated for differently-breaking text runs.
  • fast/text/international/bidi-override-expected.txt: Updated because override characters no longer show up in the render tree.
  • fast/text/international/bidi-LDB-2-CSS-expected.checksum: Added.
  • fast/text/international/bidi-LDB-2-CSS-expected.png: Added.
  • fast/text/international/bidi-LDB-2-CSS-expected.txt: Added.
  • fast/text/international/bidi-LDB-2-CSS.html: Added.
  • fast/text/international/bidi-LDB-2-HTML-expected.checksum: Added.
  • fast/text/international/bidi-LDB-2-HTML-expected.png: Added.
  • fast/text/international/bidi-LDB-2-HTML-expected.txt: Added.
  • fast/text/international/bidi-LDB-2-HTML.html: Added.
  • fast/text/international/bidi-LDB-2-formatting-characters-expected.checksum: Added.
  • fast/text/international/bidi-LDB-2-formatting-characters-expected.png: Added.
  • fast/text/international/bidi-LDB-2-formatting-characters-expected.txt: Added.
  • fast/text/international/bidi-LDB-2-formatting-characters.html: Added.
  • fast/text/international/bidi-explicit-embedding-expected.checksum: Updated to now-correct results.
  • fast/text/international/bidi-explicit-embedding-expected.png: Ditto.
  • fast/text/international/bidi-explicit-embedding-expected.txt: Ditto.

WebCore:

Reviewed by Darin.

Added 3 dbaron bidi ordering tests to LayoutTests.

  • khtml/rendering/bidi.cpp: (khtml::deleteBidiRuns): (khtml::Bidinext): (khtml::appendRunsForObject): (khtml::embed): (khtml::RenderBlock::bidiReorderLine):
3:45 AM Changeset in webkit [10789] by eseidel
  • 1 edit
    4 adds in trunk

Bug #: 5240
Submitted by: Rob Buis <​rwlbuis@xs4all.nl>
Reviewed by: eseidel

Almost forgot the layout test for the last commit. ;)
​http://bugzilla.opendarwin.org/show_bug.cgi?id=5240

  • svg-tests/custom/use-forward-refs-expected.checksum: Added.
  • svg-tests/custom/use-forward-refs-expected.png: Added.
  • svg-tests/custom/use-forward-refs-expected.txt: Added.
  • svg-tests/custom/use-forward-refs.svg: Added.
2:18 AM Changeset in webkit [10788] by eseidel
  • 4 edits in trunk/WebCore

Bug #: 5240
Submitted by: Rob Buis <​rwlbuis@xs4all.nl>
Reviewed by: eseidel

Fix to handle forward referencing <use> elements.
​http://bugzilla.opendarwin.org/show_bug.cgi?id=5240

  • ksvg2/svg/SVGDocumentImpl.cpp: (SVGDocumentImpl::executeScripts): delayed close. (SVGDocumentImpl::addForwardReference): new function.
  • ksvg2/svg/SVGDocumentImpl.h:
  • ksvg2/svg/SVGUseElementImpl.cpp: (SVGUseElementImpl::close): delay close, if missing reference.
Note: See TracTimeline for information about the timeline view.