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

Timeline



Dec 30, 2005:

4:14 PM Changeset in webkit [11847] by darin
  • 23 edits in trunk/WebCore

Reviewed by Maciej.

Added new functions named equal and equalIgnoringCase.

  • khtml/xml/dom_stringimpl.h: Changed the type of the length parameter to computeHash to unsigned to match the rest of the interface. Added equal and equalIgnoringCase functions. Added a version of the hash function that works on char* to CaseInsensitiveHash as preparation for an atomic string change.
  • khtml/xml/dom_stringimpl.cpp: Moved the QT_ALLOC_QCHAR_VEC and QT_DELETE_QCHAR_VEC macros here, and changed them into inline functions. (DOM::equal): Implemented. (DOM::equalIgnoringCase): Implemented. (DOM::DOMStringImpl::computeHash): Changed parameter type to unsigned to match other similar functions.
  • khtml/dom/dom_string.h: Removed unnecessary friend declarations. Changed operator overloads for == and != to call through inline to DOMStringImpl. Replaced strcasecmp with equalIgnoringCase, overloaded for one more combination of parameters. Removed strcmp.
  • khtml/dom/dom_string.cpp: Removed functions that are now implemented in DOMStringImpl*.
  • khtml/xml/dom_atomicstring.h: Removed a redundant const. Removed unnecessary friend declarations. Removed AtomicString::equal. Filled out a larger set of == and != operator implementations, mostly with inlines calling through to DOMStringImpl. Renamed equalsIgnoreCase to equalIgnoringCase and filled out a larger set as above.
  • khtml/xml/dom_atomicstring.cpp: Renamed AtomicString::equal to just ==. Removed some implementation of == and equalsIgnoreCase that are replaced by inlines.
  • khtml/html/html_listimpl.cpp: (DOM::HTMLOListElementImpl::parseMappedAttribute): (DOM::HTMLLIElementImpl::parseMappedAttribute): Use the "==" operator instead of strcmp.
  • khtml/css/cssstyleselector.cpp: (khtml::CSSStyleSelector::checkOneSelector): Use equalIgnoringCase instead of equalsIgnoreCase.
  • khtml/rendering/render_form.cpp: (RenderSlider::updateFromElement): Fixed bug where a temporary DOMString was put into a const DOMString&. Changed some DOMString to AtomicString as appropriate, and also used equalIgnoringCase instead of strcasecmp. (RenderSlider::slotSliderValueChanged): More of the same.
  • khtml/xml/dom_docimpl.cpp: (DocumentImpl::processHttpEquiv): Replaced use of strcasecmp with equalIgnoringCase. Added parentheses to fix incorrect operator precedence in one case
(&& binds tighter than
) which fixes a crash when using a http-equiv pragma when there

is no part (not sure how to reproduce that).

  • khtml/khtml_part.cpp: (updateState): (KHTMLPart::selectionStartHasStyle):
  • khtml/ecma/kjs_html.cpp: (KJS::KJS::HTMLDocument::putValueProperty):
  • khtml/editing/apply_style_command.cpp: (khtml::StyleChange::checkForLegacyHTMLStyleChange): (khtml::StyleChange::currentlyHasStyle): (khtml::hasTextDecorationProperty): (khtml::ApplyStyleCommand::extractTextDecorationStyle): (khtml::ApplyStyleCommand::extractAndNegateTextDecorationStyle):
  • khtml/html/html_baseimpl.cpp: (HTMLBodyElementImpl::parseMappedAttribute): (HTMLFrameElementImpl::parseMappedAttribute):
  • khtml/html/html_blockimpl.cpp: (HTMLDivElementImpl::parseMappedAttribute): (HTMLHRElementImpl::parseMappedAttribute): (HTMLParagraphElementImpl::parseMappedAttribute): (HTMLMarqueeElementImpl::parseMappedAttribute):
  • khtml/html/html_elementimpl.cpp: (HTMLElementImpl::parseMappedAttribute): (HTMLElementImpl::addHTMLAlignment): (HTMLElementImpl::setContentEditable):
  • khtml/html/html_formimpl.cpp: (DOM::HTMLFormElementImpl::parseMappedAttribute): (DOM::HTMLButtonElementImpl::parseMappedAttribute): (DOM::HTMLInputElementImpl::setInputType): (DOM::HTMLInputElementImpl::parseMappedAttribute): (DOM::HTMLKeygenElementImpl::appendFormData): (DOM::HTMLTextAreaElementImpl::parseMappedAttribute):
  • khtml/html/html_imageimpl.cpp: (DOM::HTMLAreaElementImpl::parseMappedAttribute):
  • khtml/html/html_inlineimpl.cpp: (DOM::HTMLBRElementImpl::parseMappedAttribute):
  • khtml/html/html_tableimpl.cpp: (DOM::HTMLTablePartElementImpl::parseMappedAttribute):
  • khtml/html/htmlparser.cpp: (HTMLParser::handleError): Use equalIgnoringCase instead of strcasecmp.
2:17 PM Changeset in webkit [11846] by ap
  • 10 edits
    4 adds in trunk

Reviewed by Darin, landed by ap.

WebCore:

  • khtml/rendering/font.h:
  • khtml/rendering/font.cpp: (khtml::Font::selectionRectForText): Added.
  • khtml/rendering/render_text.cpp: (kthml::InlineTextBox::selectionRect): Use selectionRectForText. This works for justified text as well, and avoids intermediate rounding which resulted in selection rects narrower than AppKit's. (khtml::InlineTextBox::positionForOffset): Use selectionRectForText, which works for justified text as well.
  • kwq/KWQFontMetrics.h:
  • kwq/KWQFontMetrics.mm: (QFontMetrics::selectionRectForText): Added.
  • kwq/WebCoreTextRenderer.h:

WebKit:

  • WebCoreSupport.subproj/WebTextRenderer.m: (-[WebTextRenderer selectionRectForRun:style:geometry:]): Added. (CG_drawHighlight): Use new function CG_selectionRect. (CG_selectionRect): New function to compute the selection rect. Eliminated rounding hackery that was required for keeping the highlight rect within the selection rect computed by InlineTextBox::selectionRect, since the latter uses this function now. The new selection rect is wider and matches AppKit more closely, although the right hand side is roundf()ed instead of cielf()ed for optimal caret positioning. (ATSU_drawHighlight): Use new function ATSU_selectionRect. (ATSU_selectionRect): New function to compute the selection rect. Much like CG_selectionRect.

LayoutTests:

  • fast/text/justified-text-rect-expected.checksum: Added.
  • fast/text/justified-text-rect-expected.png: Added.
  • fast/text/justified-text-rect-expected.txt: Added.
  • fast/text/justified-text-rect.html: Added.
1:23 PM Changeset in webkit [11845] by ap
  • 2 edits in trunk/WebCore
  • khtml/editing/visible_text.cpp: (khtml::TextIterator::rangeFromLocationAndLength): Only special-case (0, 0) ranges for empty documents.
1:22 PM Changeset in webkit [11844] by andersca
  • 1 edit in trunk/LayoutTests/fast/dom/dom-parse-serialize-expected.txt

Forgot to commit this

1:16 PM Changeset in webkit [11843] by eseidel
  • 2 edits in trunk/LayoutTests

Bug #: 6305
Submitted by: eseidel
Reviewed by: ggaren

Missing CSS Style sheets in one of the XHTML tests in WebCore
​http://bugzilla.opendarwin.org/show_bug.cgi?id=6305

  • fast/parser/entities-in-xhtml.xhtml: removed <link> to css sheet
12:43 PM Changeset in webkit [11842] by ggaren
  • 1 edit in trunk/WebCore/ChangeLog

Remove conflict marker from ChangeLog

12:42 PM Changeset in webkit [11841] by ggaren
  • 3 edits
    1 add in trunk

WebCore:

Added test case for <rdar://problem/4268278> Submitting a form in onUnload event
handler causes crash in -[WebDataSource(WebPrivate) _commitIfReady:]

  • manual-tests/onunload-form-submit-crash.html: Added.

WebKit:

Reviewed by Eric.

Manual testcase added:
WebCore/manual-tests/onunload-form-submit-crash.html

  • Fixed <rdar://problem/4268278> Submitting a form in onUnload event handler causes crash in -[WebDataSource(WebPrivate) _commitIfReady:]

The problem is that the form submission in the unload event kicks off
a new load in the midst of the load that caused the unload event to
fire in the first place, so the two loads stomp each other.

The solution is to cancel the first load and let the unload handler's
load win. (Firefox does the same.)

  • WebView.subproj/WebFrame.m: (-[WebFrame _transitionToCommitted:]): Moved call to -closeURL up the call stack to _continueLoadRequest. (See below.) This has the side-effect of always firing the unload event, even if the new datasource never becomes committed, which seems like a good thing.

(-[WebFrame _continueLoadRequestAfterNavigationPolicy:formState:]):
Call -closeURL here, instead of in _transitionToCommitted, so that the
unload handler can fire before we initialize any part of the load.

Check provisionalDataSource for nil to discover if the unload event
kicked off its own load.

Cleared up some coments.

(-[WebFrame _detachFromParent]):
It turns out that if you close the window instead of just navigating
to a new page, you get an alternate assertion failure/crash because
the load kicked off by the unload event handler generates resource
loader callbacks after the associated WebFrame/WebView has disappeared.

The nifty solution here is just to reverse the order of calls to
-stopLoading and -closeURL, thus guaranteeing that -stopLoading has the
last word when you close a window.

6:53 AM Changeset in webkit [11840] by andersca
  • 6 edits in trunk/LayoutTests

2005-12-30 Anders Carlsson <​andersca@mac.com>

Reviewed by Eric.

XMLSerializer should use createMarkup

  • fast/dom/XMLSerializer.html:
  • fast/dom/XMLSerializer-expected.txt: Create a document fragment and serialize it.
  • fast/dom/dom-parse-serialize.html:
  • fast/dom/dom-parse-serialize-expected.txt: Add some entities to attributes and text nodes.
  • fast/xsl/xslt-processer-expected.txt: Update test results.
6:51 AM Changeset in webkit [11839] by andersca
  • 3 edits in trunk/WebCore

2005-12-30 Anders Carlsson <​andersca@mac.com>

Reviewed by Eric.

  • khtml/ecma/xmlserializer.cpp: (KJS::XMLSerializerProtoFunc::callAsFunction):

Use createMarkup instead of NodeImpl::toString.

  • khtml/editing/markup.cpp: (khtml::startMarkup):

escape attribute values and ignore document fragment nodes.

1:57 AM Changeset in webkit [11838] by mjs
  • 2 edits in trunk/JavaScriptCore

No review, just test result update.

  • tests/mozilla/expected.html: Updated for newly passing test from recent fixes.
1:48 AM Changeset in webkit [11837] by eseidel
  • 7 edits in trunk/WebCore

Bug #: 6299
Submitted by: eseidel
Reviewed by: mjs

No functional changes, thus no layout test.

Just removes debug NSLog() calls from ksvg2/kcanvas.
​http://bugzilla.opendarwin.org/show_bug.cgi?id=6299

  • kcanvas/device/quartz/KCanvasFilterQuartz.mm: (KCanvasFilterQuartz::prepareFilter): (KCanvasFilterQuartz::applyFilter): (KCanvasFilterQuartz::getCIFilterStack): (KCanvasFilterQuartz::imageForName): (KCanvasFEBlendQuartz::getCIFilter): (KCanvasFEColorMatrixQuartz::getCIFilter): (KCanvasFEGaussianBlurQuartz::getCIFilter):
  • kcanvas/device/quartz/KCanvasResourcesQuartz.mm: (KCanvasClipperQuartz::applyClip):
  • kcanvas/device/quartz/KRenderingDeviceQuartz.mm: (KRenderingDeviceContextQuartz::mapFromVisual): (KRenderingDeviceContextQuartz::mapToVisual): (KRenderingDeviceQuartz::createResource): (KRenderingDeviceQuartz::createFilterEffect):
  • kcanvas/device/quartz/KRenderingPaintServerGradientQuartz.mm: (drawShadingWithStyle): (KRenderingPaintServerGradientQuartz::updateQuartzGradientCache):
  • kcanvas/device/quartz/KRenderingPaintServerQuartz.mm: (KRenderingPaintServerPatternQuartz::draw):
  • ksvg2/svg/svgpathparser.cpp: (SVGPathParser::parseSVG):
1:44 AM Changeset in webkit [11836] by andersca
  • 2 edits in trunk/JavaScriptCore

2005-12-30 Anders Carlsson <​andersca@mac.com>

Reviewed by Maciej.

  • kjs/object.cpp: (KJS::JSObject::put):

Rework the getter setter part. We now walk the prototype chain, checking for
getter/setter properties and only take the slow path if any are found.

1:27 AM Changeset in webkit [11835] by eseidel
  • 2 edits
    2 adds in trunk/LayoutTests

Bug #: none
Submitted by: eseidel
Reviewed by: No review, just adding a couple missing results.

One missing pixel test.
One result which never was updated properly after rounded rect fix:
​http://bugzilla.opendarwin.org/show_bug.cgi?id=6157

  • svg/W3C-SVG-1.1/shapes-rect-01-t-expected.txt:
  • fast/parser/xhtml-alternate-entities-expected.checksum: Added.
  • fast/parser/xhtml-alternate-entities-expected.png: Added.
12:51 AM Changeset in webkit [11834] by mjs
  • 1 edit
    4 adds in trunk/LayoutTests

Reviewed by Darin, landed by Maciej.

  • fast/overflow/overflow-auto-table-expected.checksum: Added.
  • fast/overflow/overflow-auto-table-expected.png: Added.
  • fast/overflow/overflow-auto-table-expected.txt: Added.
  • fast/overflow/overflow-auto-table.html: Added.
12:49 AM Changeset in webkit [11833] by eseidel
  • 37 edits in trunk/LayoutTests

Bug #: none
Submitted by: eseidel
Reviewed by: none, just updating stale results

No review, updating stale results.
I went over all the interesting ones with mjs and hyatt.

  • css2.1/t040304-c64-uri-00-a-g-expected.checksum:
  • css2.1/t040304-c64-uri-00-a-g-expected.png:
  • css2.1/t0905-c414-flt-wrap-01-d-g-expected.checksum:
  • css2.1/t0905-c414-flt-wrap-01-d-g-expected.png:
  • css2.1/t100801-c544-valgn-02-d-agi-expected.checksum:
  • css2.1/t100801-c544-valgn-02-d-agi-expected.png:
  • dom/xhtml/level2/html/HTMLBaseElement02-expected.checksum:
  • dom/xhtml/level2/html/HTMLBaseElement02-expected.png:
  • dom/xhtml/level2/html/frame-expected.checksum:
  • dom/xhtml/level2/html/frame-expected.png:
  • dom/xhtml/level2/html/iframe-expected.checksum:
  • dom/xhtml/level2/html/iframe-expected.png:
  • editing/selection/extend-by-word-002-expected.checksum:
  • editing/selection/extend-by-word-002-expected.png:
  • fast/css/imageTileOpacity-expected.checksum:
  • fast/css/imageTileOpacity-expected.png:
  • fast/forms/001-expected.checksum:
  • fast/forms/001-expected.png:
  • fast/frames/empty-frame-src-expected.checksum:
  • fast/frames/empty-frame-src-expected.png:
  • fast/frames/invalid-expected.checksum:
  • fast/frames/invalid-expected.png:
  • fast/frames/valid-expected.checksum:
  • fast/frames/valid-expected.png:
  • svg/custom/conditional-processing-outside-switch-expected.txt:
  • svg/custom/control-points-for-S-and-T-expected.txt:
  • svg/custom/fill-update-expected.txt:
  • svg/custom/junk-data-expected.txt:
  • svg/custom/missing-xlink-expected.txt:
  • svg/custom/path-bad-data-expected.txt:
  • svg/custom/path-update-expected.txt:
  • svg/custom/simpleCDF-expected.txt:
  • svg/custom/transform-ignore-after-invalid-expected.txt:
  • svg/custom/transform-invalid-expected.txt:
  • svg/custom/transform-with-ending-space-expected.txt:
  • svg/custom/use-forward-refs-expected.txt:
12:18 AM Changeset in webkit [11832] by mjs
  • 2 edits in trunk/LayoutTests

No review.

  • fast/js/kde/Number-expected.txt:
12:14 AM Changeset in webkit [11831] by mjs
  • 2 edits in trunk/JavaScriptCore

Reviewed and committed by Maciej.

(Merged from KJS, original work by Harri Porten)

  • kjs/number_object.cpp: (NumberProtoFunc::callAsFunction): rewrote Number.toString(radix) to work with negative numbers, floating point and very large numbers.
Note: See TracTimeline for information about the timeline view.