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

Timeline



Jan 14, 2007:

11:49 PM Changeset in webkit [18855] by hyatt
  • 2 edits in trunk/WebCore

Fix for bug 10356, positioning doesn't work with <svg> elements in XHTML. Change the requiresLayer
check to use the DOM, since the old check would fail if the parent block had no element.

Reviewed by Eric

Added svg/custom/relative-positioning.html (absolute/fixed too)

  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::requiresLayer):
11:05 PM Changeset in webkit [22765] by aliceli1
  • 4 edits in branches/WindowsMerge/WebKitWin

WebKitWin:

Reviewed by Lou.

Fixed: <rdar://problem/4888908> Find: "Use selection to find" and "Jump to selection" are not implemented

  • Interfaces/IWebView.idl:
  • WebView.cpp: (WebView::selectedText): (WebView::centerSelectionInVisibleArea):
  • WebView.h:
10:52 PM Changeset in webkit [18854] by eseidel
  • 3 edits
    4 adds in trunk

2007-01-14 Eric Seidel <​eric@webkit.org>

Reviewed by hyatt.


SVG does not correctly handle float: left
​http://bugs.webkit.org/show_bug.cgi?id=12210

Test: svg/custom/svg-float-border-padding.xml

  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::paint):
7:33 PM Changeset in webkit [18853] by brmorris
  • 2 edits in S60/trunk/WebCore

2007-01-14 brmorris 3.0 sdk build fix.

  • BWINS/WebCore_sdkU.DEF:
5:29 PM Changeset in webkit [18852] by hyatt
  • 4 adds in trunk/LayoutTests/fast/replaced

Add layout test for select line breaking.

5:28 PM Changeset in webkit [18851] by hyatt
  • 2 edits in trunk/WebCore

More work to fix 10145, regression where form elements don't break properly. Make sure to really use
the parent of the replaced element's whitespace value everywhere.

Reviewed by olliej

fast/replaced/three-selects-break.html

  • rendering/bidi.cpp: (WebCore::RenderBlock::findNextLineBreak):
5:08 PM Changeset in webkit [18850] by weinig
  • 33 edits in trunk/WebCore

Reviewed by Darin and then by Mitz.

Patch for ​http://bugs.webkit.org/show_bug.cgi?id=12228
Cleanup assorted CSS classes

  • css/CSSBorderImageValue.cpp: (WebCore::CSSBorderImageValue::CSSBorderImageValue): (WebCore::CSSBorderImageValue::cssText):
  • css/CSSBorderImageValue.h:
  • css/CSSComputedStyleDeclaration.cpp: (WebCore::valueForLength): (WebCore::valueForBorderStyle): (WebCore::valueForTextAlign): (WebCore::valueForShadow): (WebCore::getPositionOffsetValue): (WebCore::CSSComputedStyleDeclaration::cssText): (WebCore::primitiveValueFromLength): (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): (WebCore::CSSComputedStyleDeclaration::getPropertyPriority): (WebCore::CSSComputedStyleDeclaration::removeProperty): (WebCore::CSSComputedStyleDeclaration::setProperty): (WebCore::CSSComputedStyleDeclaration::length): (WebCore::CSSComputedStyleDeclaration::item):
  • css/CSSComputedStyleDeclaration.h:
  • css/CSSCursorImageValue.cpp: (WebCore::CSSCursorImageValue::CSSCursorImageValue):
  • css/CSSCursorImageValue.h:
  • css/CSSGrammar.y:
  • css/CSSImageValue.cpp:
  • css/CSSImageValue.h:
  • css/CSSInheritedValue.cpp:
  • css/CSSInheritedValue.h:
  • css/CSSInitialValue.cpp:
  • css/CSSInitialValue.h:
  • css/CSSMutableStyleDeclaration.cpp: (WebCore::CSSMutableStyleDeclaration::CSSMutableStyleDeclaration): (WebCore::CSSMutableStyleDeclaration::operator=): (WebCore::CSSMutableStyleDeclaration::getPropertyValue): (WebCore::CSSMutableStyleDeclaration::get4Values): (WebCore::CSSMutableStyleDeclaration::getShorthandValue): (WebCore::CSSMutableStyleDeclaration::getPropertyCSSValue): (WebCore::CSSMutableStyleDeclaration::removeProperty): (WebCore::CSSMutableStyleDeclaration::setChanged): (WebCore::CSSMutableStyleDeclaration::getPropertyPriority): (WebCore::CSSMutableStyleDeclaration::getPropertyShorthand): (WebCore::CSSMutableStyleDeclaration::isPropertyImplicit): (WebCore::CSSMutableStyleDeclaration::setProperty): (WebCore::CSSMutableStyleDeclaration::setImageProperty): (WebCore::CSSMutableStyleDeclaration::parseDeclaration): (WebCore::CSSMutableStyleDeclaration::addParsedProperties): (WebCore::CSSMutableStyleDeclaration::setLengthProperty): (WebCore::CSSMutableStyleDeclaration::cssText): (WebCore::CSSMutableStyleDeclaration::merge): (WebCore::CSSMutableStyleDeclaration::removePropertiesInSet):
  • css/CSSMutableStyleDeclaration.h: (WebCore::CSSMutableStyleDeclaration::setProperty): (WebCore::CSSMutableStyleDeclaration::removeProperty):
  • css/CSSNamespace.h: (WebCore::CSSNamespace::CSSNamespace):
  • css/CSSPrimitiveValue.cpp: (WebCore::isCSSTokenizerIdentifier): (WebCore::CSSPrimitiveValue::CSSPrimitiveValue): (WebCore::CSSPrimitiveValue::cleanup): (WebCore::CSSPrimitiveValue::computeLengthInt): (WebCore::CSSPrimitiveValue::computeLengthIntForLength): (WebCore::CSSPrimitiveValue::computeLengthShort): (WebCore::CSSPrimitiveValue::computeLengthFloat): (WebCore::CSSPrimitiveValue::setFloatValue): (WebCore::scaleFactorForConversion): (WebCore::CSSPrimitiveValue::getFloatValue): (WebCore::CSSPrimitiveValue::setStringValue): (WebCore::CSSPrimitiveValue::getStringValue): (WebCore::CSSPrimitiveValue::parseString): (WebCore::CSSPrimitiveValue::getIdent): (WebCore::CSSPrimitiveValue::cssText):
  • css/CSSPrimitiveValue.h: (WebCore::CSSPrimitiveValue::getCounterValue): (WebCore::CSSPrimitiveValue::getRectValue): (WebCore::CSSPrimitiveValue::getRGBColorValue): (WebCore::CSSPrimitiveValue::getPairValue): (WebCore::CSSPrimitiveValue::getDashboardRegionValue):
  • css/CSSProperty.cpp: (WebCore::operator==):
  • css/CSSProperty.h: (WebCore::CSSProperty::operator=):
  • css/CSSQuirkPrimitiveValue.h: (WebCore::CSSQuirkPrimitiveValue::CSSQuirkPrimitiveValue):
  • css/CSSRuleList.cpp: (WebCore::CSSRuleList::CSSRuleList): (WebCore::CSSRuleList::deleteRule): (WebCore::CSSRuleList::insertRule):
  • css/CSSRuleList.h: (WebCore::CSSRuleList::item):
  • css/CSSSelector.cpp: (WebCore::CSSSelector::print): (WebCore::CSSSelector::specificity): (WebCore::CSSSelector::extractPseudoType): (WebCore::CSSSelector::operator==): (WebCore::CSSSelector::selectorText):
  • css/CSSSelector.h: (WebCore::CSSSelector::CSSSelector): (WebCore::CSSSelector::~CSSSelector): (WebCore::CSSSelector::append): (WebCore::CSSSelector::next): (WebCore::CSSSelector::pseudoType): (WebCore::CSSSelector::hasTag): (WebCore::CSSSelector::hasAttribute):
  • css/CSSStyleDeclaration.cpp: (WebCore::propertyID): (WebCore::CSSStyleDeclaration::getPropertyPriority): (WebCore::CSSStyleDeclaration::getPropertyShorthand): (WebCore::CSSStyleDeclaration::isPropertyImplicit): (WebCore::CSSStyleDeclaration::setProperty): (WebCore::CSSStyleDeclaration::isPropertyName): (WebCore::CSSStyleDeclaration::parentRule): (WebCore::CSSStyleDeclaration::diff): (WebCore::CSSStyleDeclaration::copyPropertiesInSet):
  • css/CSSStyleDeclaration.h:
  • css/Counter.h: (WebCore::Counter::Counter): (WebCore::Counter::listStyleNumber):
  • css/csshelper.cpp: (WebCore::parseURL):
  • css/csshelper.h:
  • css/cssparser.h: (WebCore::domString): (WebCore::atomicString): (WebCore::CSSParser::operator|):
  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::matchRulesForList): (WebCore::CSSStyleSelector::checkSelector): (WebCore::CSSStyleSelector::checkOneSelector): (WebCore::CSSRuleSet::addRule):
4:53 PM Changeset in webkit [18849] by bdash
  • 5 edits in trunk/WebKitTools

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

Reviewed by Adam.

<rdar://problem/4908909> Need to create Leopard nightly build

  • WebKitLauncher/WebKitLauncher.xcodeproj/project.pbxproj: Always use the 10.4 Universal SDK. The disk images don't mount pre-10.4 so we aren't losing anything here.
  • WebKitLauncher/WebKitNightlyEnabler.m: (poseAsWebKitApp): Simplify CoreFoundation-related hackery by using _CFGetProcessPath instead of walking the mach-o symbol tables. The smaller timeframe where the CFProcessPath environment variable is set allows this to work correctly on Leopard where the old code failed. (enableWebKitNightlyBehaviour):
  • WebKitLauncher/main.m: (main): Pass executable path as WebKitAppPath rather than CFProcessPath to prevent it being picked up too early by CoreFoundation.
  • Drosera/launcher.m: (main): Ditto
4:39 PM Changeset in webkit [18848] by ddkilzer
  • 6 edits
    2 adds in trunk

LayoutTests:

Reviewed by Adam. Patch by Mitz.

  • fast/tokenizer/ignore-tags-in-iframe-expected.txt: Added.
  • fast/tokenizer/ignore-tags-in-iframe.html: Added.

WebCore:

Reviewed by Adam. Patch by Mitz.

Test: fast/tokenizer/ignore-tags-in-iframe.html

  • html/HTMLParser.cpp: (WebCore::HTMLParser::reset): Removed unused inSelect member variable. (WebCore::HTMLParser::selectCreateErrorCheck): Ditto. (WebCore::HTMLParser::processCloseTag): Ditto.
  • html/HTMLParser.h: Ditto.
  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::parseTag): Changed not to process <pre>, <listing>, <script>, <style>, <title>, <textarea> and <xmp> if the parser is in skip mode (waiting for a close tag of an element whose contents are not parsed, such as iframe).
  • html/HTMLTokenizer.h: (WebCore::HTMLTokenizer::State::setInStyle): Removed unused bit. (WebCore::HTMLTokenizer::State::): Ditto.
3:58 PM Changeset in webkit [18847] by brmorris
  • 2 edits in S60/trunk/WebKit

2007-01-14 brmorris <​bradley.morrison@nokia.com>

DESC: reverted revision 18809

  • BrowserCore/Formcontrols/src/FInputlFormattedEditor.cpp
1:45 PM Changeset in webkit [18846] by ddkilzer
  • 2 edits in trunk/WebKit

WebKit:

Reviewed by Darin.

  • WebView/WebView.mm: (-[WebView _objectForIdentifier:]): Removed assertion. (-[WebView _removeObjectForIdentifier:]): Removed assertion.
12:13 PM Changeset in webkit [22764] by aroben
  • 11 edits
    1 copy
    3 adds in branches/WindowsMerge/WebKitWin

WebKitWin:

Reviewed by Steve.

Fix: <rdar://4831292> Implement WebKitStatistics

Added object counting:

  • WebDataSource.cpp: (WebDataSource::WebDataSource): (WebDataSource::~WebDataSource):
  • WebFrame.cpp: (WebFrame::WebFrame): (WebFrame::~WebFrame):
  • WebHTMLRepresentation.cpp: (WebHTMLRepresentation::WebHTMLRepresentation): (WebHTMLRepresentation::~WebHTMLRepresentation):
  • WebView.cpp: (WebView::WebView): Initialize m_page member. (WebView::~WebView):
10:46 AM Changeset in webkit [18845] by ap
  • 2 edits in trunk/LayoutTests

Reviewed by Mitz.

  • fast/multicol/float-multicol.html: Fixed an URL not to point to a remote resource (we already have a local copy).
10:08 AM Changeset in webkit [18844] by ap
  • 12 edits in trunk

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=12258
fast/tokenizer/002 test broken

WebCore:

  • loader/FrameLoader.h: Added a flush parameter to write().
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::write): Ditto. (WebCore::FrameLoader::endIfNotLoading): Instead of poorly duplicating the logic in write(), call it directly.

LayoutTests:

  • editing/selection/designmode-no-caret-expected.txt:
  • fast/dom/HTMLHeadElement/textInHead4-expected.txt:
  • fast/parser/broken-comments-vs-parsing-mode-expected.txt:
  • fast/parser/tabs-in-scripts-expected.txt:
  • fast/tokenizer/external-script-document-write-expected.txt:
  • fast/tokenizer/missing-title-end-tag-1-expected.txt:
  • tables/mozilla/images/adforce_imgis_com-expected.txt:
  • tables/mozilla_expected_failures/images/adforce_imgis_com-expected.txt: Updated the results (inconsequential changes only).
2:04 AM Changeset in webkit [18843] by aroben
  • 8 edits
    2 adds in trunk

LayoutTests:

Reviewed by Maciej.

  • editing/deleting/4845371.html: Removed bogus "Javascript" type.
  • editing/selection/4397952.html: Ditto.
  • fast/html/script-allowed-types-languages-expected.txt: Added.
  • fast/html/script-allowed-types-languages.html: Added. Tests type/language whitelisting.

WebCore:

Reviewed by Maciej.

Make sure our whitelisting of the type and language attributes of the
<script> element is enforced in all HTMLTokenizer/HTMLScriptElement
code paths.

All layout tests pass.

  • html/HTMLScriptElement.cpp: (WebCore::HTMLScriptElement::shouldExecuteAsJavaScript): New method to determine whether the script should be executed, given its type and language attributes. (WebCore::HTMLScriptElement::evaluateScript): Check type/language before executing.
  • html/HTMLScriptElement.h: Added new declarations.
  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::begin): Made scriptSrc a String. (WebCore::HTMLTokenizer::scriptHandler): Check shouldExecuteAsJavaScript before executing. (WebCore::HTMLTokenizer::notifyFinished): Ditto. (WebCore::HTMLTokenizer::parseTag): Moved type/language checking from here to HTMLScriptElement::shouldExecuteAsJavaScript.
  • html/HTMLTokenizer.h: Made scriptSrc a String, and removed the javascript member.
1:48 AM Changeset in webkit [18842] by hyatt
  • 4 adds in trunk/LayoutTests/fast/replaced

Add a second breaking test for replaced and text mixtures.

1:21 AM Changeset in webkit [18841] by hyatt
  • 4 adds in trunk/LayoutTests/fast/multicol

Add multicol float avoidance test.

1:16 AM Changeset in webkit [18840] by hyatt
  • 2 edits in trunk/WebCore

Make sure floats are avoided if you have non-auto column property values. Fixes the crazy layout
on ​http://weblogs.mozillazine.org/roc.

Reviewed by mitz

Added fast/multicol/float-avoidance.html

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::avoidsFloats):
12:25 AM Changeset in webkit [18839] by hyatt
  • 4 adds in trunk/LayoutTests/fast/replaced

Add layout test for replaced breaking.

12:22 AM Changeset in webkit [18838] by hyatt
  • 13 edits in trunk/WebCore

Fix for 10145, regression where line breaks are not being honored between adjacent replaced elements.
When checking for break opportunities where replaced elements are involved, we ignore their white-space
values and use their parents' instead. The assumption/idea behind this change is that because the element
is replaced, its white-space value is irrelevant except as a means of applying style to its interior.

The CSS spec does not cover this situation, so we're implementing this based off what other browsers seem to
do.

Reviewed by aroben

  • css/CSSComputedStyleDeclaration.cpp: (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
  • css/CSSValueKeywords.in:
  • css/cssparser.cpp: (WebCore::CSSParser::parseValue):
  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::applyProperty):
  • page/EventHandler.cpp: (WebCore::EventHandler::EventHandler): (WebCore::EventHandler::clear): (WebCore::EventHandler::handleWheelEvent):
  • page/EventHandler.h:
  • rendering/RenderBox.cpp: (WebCore::RenderBox::calcHeight):
  • rendering/RenderLayer.cpp: (WebCore::Marquee::Marquee): (WebCore::Marquee::start): (WebCore::Marquee::updateMarqueePosition): (WebCore::Marquee::updateMarqueeStyle): (WebCore::Marquee::timerFired):
  • rendering/RenderLayer.h:
  • rendering/RenderPartObject.cpp: (WebCore::RenderPartObject::viewCleared):
  • rendering/RenderStyle.h: (WebCore::):
  • rendering/bidi.cpp: (WebCore::RenderBlock::findNextLineBreak):

Jan 13, 2007:

11:53 PM Changeset in webkit [18837] by bdash
  • 9 edits in trunk

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

Reviewed by Mitz.

Minor fixes to JavaScript pretty-printing.

  • JavaScriptCore.exp:
  • kjs/Parser.cpp: (KJS::Parser::prettyPrint): Return line number and error message if parsing fails.
  • kjs/Parser.h:
  • kjs/nodes2string.cpp: (ElementNode::streamTo): Include comma delimiters in array literals. (PropertyNameNode::streamTo): Quote property names in object literals to handle the case when the property name is not a valid identifier.
  • kjs/testkjs.cpp: (doIt): Print any errors encountered while pretty-printing.

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

Reviewed by Mitz.

Layout tests for fixes to JavaScript pretty-printing.

  • fast/js/pretty-print-expected.txt:
  • fast/js/resources/pretty-print.js: Test handling of object literal with non-identifier property name, and of array literals.
1:07 PM Changeset in webkit [18836] by hyatt
  • 2 edits in trunk/WebCore

While optimizing form controls and fixing their clipping, I caused this text zoom regression. The layout
of text is necessary because it needs to recalc minmaxwidth. This is still a really unfortunate
bit of code... since a lot of properties don't apply to text (e.g., borders), we're marking text renderers
as dirty when we don't have to.

Reviewed by mitz

  • rendering/RenderObject.cpp: (WebCore::RenderObject::setStyle):
12:01 PM Changeset in webkit [18835] by lars
  • 2 edits in trunk/WebCore

Reviewed by Alexey

String::format() was relying on undefined behaviour, which
happened to work on lots of systems (Mac and 32bit Linux at least)
but broke badly on 64bit Linux.

10:48 AM Changeset in webkit [18834] by lars
  • 2 edits in trunk/WebCore

Fix for bug 11924: WebCore would crash on any HTML content
with a non-empty <table> tag in it on a 64-bit machine. The
size_t (64bit) vs int (32bit) cast is now fixed in ensureRows,
and the clients of this function have been updated to check
for a false return code.

Reviewed by Darin

8:33 AM Changeset in webkit [18833] by ap
  • 3 edits
    2 adds in trunk

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=12165
REGRESSION: text encoding problem at jn.sapo.pt

Test: fast/encoding/script-in-head.html

  • loader/TextResourceDecoder.cpp: (WebCore::TextResourceDecoder::checkForHeadCharset): Ignore tags within <script> elements in head, just like we do for <title>.
3:21 AM Changeset in webkit [18832] by lars
  • 2 edits in trunk/WebCore

Reviewed by Maciej

Don't crash when constructing a GraphicsContext with a 0
PlatformGraphicsContext and don't do anything if painting
is disabled.

3:02 AM Changeset in webkit [18831] by lars
  • 3 edits in trunk/WebKitQt

Make the Qt port compile again.

1:49 AM Changeset in webkit [18830] by eseidel
  • 7 edits
    4 adds in trunk

2007-01-13 Eric Seidel <​eric@eseidel.com>

Reviewed by hyatt.

RichDraw demo does not live-update in Safari
(SVG elements should ignore CSS position)
​http://bugs.webkit.org/show_bug.cgi?id=10364

  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::adjustRenderStyle): ignore position for everything but <svg>
  • ksvg2/svg/SVGStyledElement.cpp: (WebCore::SVGStyledElement::rebuildRenderer): move path generation into layout()
  • rendering/RenderContainer.cpp: (WebCore::RenderContainer::layout): ASSERT kids marked as having completed layout
  • rendering/RenderPath.cpp: (WebCore::RenderPath::layout): move path generation into layout()
  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::layout): ASSERT kids marked as having completed layout
12:16 AM Changeset in webkit [18829] by hyatt
  • 1 edit in trunk/WebCore/rendering/RenderBlock.cpp

Fix leak of column vector.

Note: See TracTimeline for information about the timeline view.