Timeline



Feb 5, 2007:

11:52 PM Changeset in webkit [22903] by beidson
  • 4 edits in branches/WindowsMerge/WebKitWin

Add config.h to some .cpps

9:43 PM Changeset in webkit [19425] by ap
  • 4 edits
    2 adds in trunk

Reviewed by Geoff.

​http://bugs.webkit.org/show_bug.cgi?id=12619
Non-abbreviated XPaths don't work

Test: fast/xpath/ancestor-axis.html

  • xml/XPathParser.cpp: (WebCore::XPath::Parser::lex): Fix axis name parsing to actually work.
  • xml/XPathStep.cpp: (WebCore::XPath::Step::nodeTestMatches): Fix "*" node test to actually work.
8:10 PM Changeset in webkit [19424] by brmorris
  • 5 edits in S60/trunk/WebKit

bujtas, reviewed by Yongjun.

DESC: PartialImage() can end up in a dialog box which starts up a nested active scheduler JELE-6XSH3P

and since the page still being loaded in the background new image
data could come to the image decoder. that's a fatal re-entry on
the image decoder.

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

7:55 PM Changeset in webkit [19423] by brmorris
  • 5 edits in S60/branches/3.1m/WebKit

bujtas, reviewed by Yongjun.

DESC: PartialImage() can end up in a dialog box which starts up a nested active scheduler JELE-6XSH3P
​http://bugs.webkit.org/show_bug.cgi?id=12550

7:43 PM Changeset in webkit [19422] by ddkilzer
  • 5 edits in trunk

LayoutTests:

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=7266
Webarchive format saves duplicate WebSubresources to .webarchive file

  • webarchive/test-duplicate-resources-expected.txt: Duplicate resources removed.
  • webarchive/test-frameset-expected.txt: Ditto.

WebKit:

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=7266
Webarchive format saves duplicate WebSubresources to .webarchive file

Tests: webarchive/test-duplicate-resources.html

webarchive/test-frameset.html

  • WebView/WebArchiver.mm: (+[WebArchiver _archiveWithMarkupString:fromFrame:nodes:]): Use an NSMutableSet to prevent duplicate subresources from being added to the webarchive.
6:27 PM Changeset in webkit [19421] by bdash
  • 13 edits
    4 deletes in trunk

2007-02-06 Mark Rowe <​mrowe@apple.com>

Roll out incomplete support for font-stretch (r19350) at Dave Hyatt's request.
See ​http://bugs.webkit.org/show_bug.cgi?id=12530#c9 for more info.

  • css/CSSComputedStyleDeclaration.cpp: (WebCore::): (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
  • css/cssparser.cpp: (WebCore::CSSParser::parseValue):
  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::applyDeclarations): (WebCore::CSSStyleSelector::applyProperty):
  • platform/FontCache.cpp: (WebCore::FontPlatformDataCacheKey::FontPlatformDataCacheKey): (WebCore::FontPlatformDataCacheKey::operator==): (WebCore::computeHash): (WebCore::FontPlatformDataCacheKeyTraits::deletedValue): (WebCore::FontPlatformDataCacheKeyTraits::emptyValue): (WebCore::FontCache::getCachedFontPlatformData):
  • platform/FontDescription.h: (WebCore::FontDescription::FontDescription): (WebCore::FontDescription::weight): (WebCore::FontDescription::setWeight): (WebCore::FontDescription::operator==):
  • platform/mac/FontCacheMac.mm: (WebCore::FontCache::createFontPlatformData):
  • platform/mac/WebFontCache.mm: (acceptableChoice): (betterChoice): (+[WebFontCache fontWithFamily:traits:size:]):

2007-02-06 Mark Rowe <​mrowe@apple.com>

Roll out incomplete support for font-stretch (r19350) at Dave Hyatt's request.
See ​http://bugs.webkit.org/show_bug.cgi?id=12530#c9 for more info.

  • WebInspector/webInspector/inspector.js:
  • WebView/WebHTMLView.mm: (-[WebHTMLView _addToStyle:fontA:fontB:]):

2007-02-06 Mark Rowe <​mrowe@apple.com>

Roll out incomplete support for font-stretch (r19350) at Dave Hyatt's request.
See ​http://bugs.webkit.org/show_bug.cgi?id=12530#c9 for more info.

  • fast/css/computed-style-expected.txt:
  • fast/css/font-stretch-expected.checksum: Removed.
  • fast/css/font-stretch-expected.png: Removed.
  • fast/css/font-stretch-expected.txt: Removed.
  • fast/css/font-stretch.html: Removed.
6:12 PM Changeset in webkit [19420] by adele
  • 4 edits in trunk/WebCore

Reviewed by Darin.

A few small updates to my bidi patch.

  • rendering/RenderFileUploadControl.cpp: (WebCore::RenderFileUploadControl::paintObject): Moved the declaration of the CharacterBuffer outside the if statement since we use its data outside that scope.
  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::paintItemForeground): ditto.
  • rendering/bidi.cpp: (WebCore::RenderBlock::bidiReorderCharacters): Used a PassRefPtr for the BidiContext.
5:19 PM Changeset in webkit [19419] by justing
  • 3 edits in trunk/LayoutTests/editing/selection

Checked in old results. Fixed.

4:36 PM Changeset in webkit [19418] by adele
  • 4 edits in trunk/WebCore

Reviewed by Adam.

Fix for assertion failure in fast/events/objc-keyboard-event-creation.html

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::selectionStart): Check that the selection offsets are not equal to -1 (which is means the selection has never been set) instead of checking for >= 0. (WebCore::HTMLInputElement::selectionEnd): ditto. (WebCore::HTMLInputElement::setValue): ditto. (WebCore::HTMLInputElement::selection): Return an empty selection if the input is not a text field or if no selection has ever been set in the text field.
  • html/HTMLTextAreaElement.cpp: (WebCore::HTMLTextAreaElement::selectionStart): Check that the selection offsets are not equal to -1. (WebCore::HTMLTextAreaElement::selectionEnd): ditto. (WebCore::HTMLTextAreaElement::setValue): ditto. (WebCore::HTMLTextAreaElement::selection): Return an empty selection if no selection has ever been set in the textarea.
  • dom/Position.cpp: Added FIXMEs that we should catch the negative offset earlier. (WebCore::Position::previous): (WebCore::Position::next):
4:20 PM Changeset in webkit [19417] by justing
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by harrison


<rdar://problem/4947387>
REGRESSION: Shift Command Left should only select to the beginning of the line

  • editing/selection/4947387-expected.checksum: Added.
  • editing/selection/4947387-expected.png: Added.
  • editing/selection/4947387-expected.txt: Added.
  • editing/selection/4947387.html: Added.

WebCore:

Reviewed by harrison

<rdar://problem/4947387>
REGRESSION Shift Command Left should only select to the beginning of the line

  • editing/Editor.cpp: (WebCore::CommandEntry::): Never copy/paste code.
3:41 PM Changeset in webkit [19416] by darin
  • 4 edits
    2 adds in trunk

LayoutTests:

Reviewed by John Sullivan.

  • fast/forms/input-select-on-click-expected.txt: Added.
  • fast/forms/input-select-on-click.html: Added.

WebCore:

Reviewed by John Sullivan.

Test: fast/forms/input-select-on-click.html

  • page/EventHandler.h: Added m_mouseDownWasSingleClickInSelection boolean. Organized other booleans a bit more logically.
  • page/EventHandler.cpp: (WebCore::EventHandler::EventHandler): Rearrange order to match EventHandler.h, add m_mouseDownWasSingleClickInSelection initialization. (WebCore::EventHandler::handleMousePressEventSingleClick): Set m_mouseDownWasSingleClickInSelection to true in the appropriate special case. (WebCore::EventHandler::handleMousePressEvent): Set m_mouseDownWasSingleClickInSelection to false. (WebCore::EventHandler::handleMouseReleaseEvent): Change code to clear selection or set to caret to only fire if m_mouseDownWasSingleClickInSelection was true rather than firing for any m_mouseDownMayStartSelect case.
3:39 PM Changeset in webkit [19415] by staikos
  • 2 edits in trunk/WebCore

Make http cookies work

3:25 PM Changeset in webkit [19414] by adele
  • 7 edits
    8 adds in trunk

LayoutTests:

Reviewed by Mitz and Anders.

Tests for ​http://bugs.webkit.org/show_bug.cgi?id=11811
<rdar://problem/4947184> REGRESSION (r11783): Hebrew text in list boxes is reversed

  • fast/text/international/bidi-listbox-expected.checksum: Added.
  • fast/text/international/bidi-listbox-expected.png: Added.
  • fast/text/international/bidi-listbox-expected.txt: Added.
  • fast/text/international/bidi-listbox.html: Added.
  • fast/text/international/bidi-listbox-atsui-expected.checksum: Added.
  • fast/text/international/bidi-listbox-atsui-expected.png: Added.
  • fast/text/international/bidi-listbox-atsui-expected.txt: Added.
  • fast/text/international/bidi-listbox-atsui.html: Added.

WebCore:

Reviewed by Mitz and Anders.

Fix for ​http://bugs.webkit.org/show_bug.cgi?id=11811
<rdar://problem/4947184> REGRESSION (r11783): Hebrew text in list boxes is reversed

​http://bugs.webkit.org/show_bug.cgi?id=11812
<rdar://problem/4960269> REGRESSION (Native file upload): Hebrew filenames are reversed

Tests:
fast/text/international/bidi-listbox.html
fast/text/international/bidi-listbox-atsui.html

  • rendering/RenderBlock.h: Added static bidiReorderCharacters that will run the bidi algorithm on a character buffer.
  • rendering/bidi.cpp: (WebCore::RenderBlock::bidiReorderCharacters): Added. Creates an anonymous RenderBlock and RenderText to feed into bidiReorderLines. (WebCore::RenderBlock::constructLine): Uses the new convenience methods on BidiRun.
  • rendering/bidi.h: (WebCore::BidiRun::reversed): Added convenience method. (WebCore::BidiRun::dirOverride): ditto.
  • rendering/RenderFileUploadControl.cpp: (WebCore::RenderFileUploadControl::paintObject): Creates a buffer and calls bidiReorderCharacters to get a correctly ordered character buffer for drawText.
  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::paintItemForeground): ditto.
3:22 PM Changeset in webkit [22902] by adachan
  • 2 edits in branches/WindowsMerge/WebKitWin

2007-02-05 Ada Chan <​adachan@apple.com>

Reviewed by Anders.


<rdar://4945519> Crash when back/forward buttons are clicked several times
Clear loader's previous history item when frame load is completed (got that code from the mac side).

  • WebFrame.cpp: (WebFrame::frameLoadCompleted):
3:19 PM Changeset in webkit [19413] by weinig
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by Darin.

  • fast/repaint/selection-gap-overflow-scroll-expected.checksum: Added.
  • fast/repaint/selection-gap-overflow-scroll-expected.png: Added.
  • fast/repaint/selection-gap-overflow-scroll-expected.txt: Added.
  • fast/repaint/selection-gap-overflow-scroll.html: Added.

WebCore:

Reviewed by Darin.

Test: fast/repaint/selection-gap-overflow-scroll.html

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::selectionGapRects): Adjust for overflow scroll.
3:07 PM Changeset in webkit [19412] by weinig
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by Adele.

  • fast/forms/search-rtl-expected.checksum: Added.
  • fast/forms/search-rtl-expected.png: Added.
  • fast/forms/search-rtl-expected.txt: Added.
  • fast/forms/search-rtl.html: Added.

WebCore:

Reviewed by Adele.

Test: fast/forms/search-rtl.html

  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::createInnerBlockStyle): Always make the inner block LTR so that the results button will be on the left and the cancel button will be on the right. (WebCore::RenderTextControl::createInnerTextStyle): Inherit the direction property directly from the control so that it won't be masked by the inner block's LTR override. (WebCore::RenderTextControl::nodeAtPoint): Removed code that swapped the results and cancel buttons for RTL. (WebCore::RenderTextControl::forwardEvent): Ditto. (WebCore::RenderTextControl::clientPaddingLeft): Ditto. (WebCore::RenderTextControl::clientPaddingRight): Ditto.
2:55 PM Changeset in webkit [19411] by weinig
  • 4 edits
    2 adds in trunk

LayoutTests:

Reviewed by Adele.

  • fast/forms/search-cancel-button-mouseup-expected.txt: Added.
  • fast/forms/search-cancel-button-mouseup.html: Added.

WebCore:

Reviewed by Adele.

Test: fast/forms/search-cancel-button-mouseup.html

Made the cancel button capturing.

  • html/HTMLTextFieldInnerElement.cpp: (WebCore::HTMLSearchFieldCancelButtonElement::HTMLSearchFieldCancelButtonElement): (WebCore::HTMLSearchFieldCancelButtonElement::defaultEventHandler):
  • html/HTMLTextFieldInnerElement.h:
1:12 PM Changeset in webkit [19410] by ggaren
  • 1 edit
    1 add in trunk/WebKitTools

Reviewed by Darin Adler.


New script that allows you to ignore leaks by regular expression.

  • Scripts/run-leaks: Added.
1:10 PM Changeset in webkit [19409] by bdakin
  • 2 edits in trunk/WebKit

Reviewed by Adam.

Fix for <rdar://problem/4975161> REGRESSION: With BumperCar 2.1.1,
the contextual menu fails to appear when I ctrl-click on page

  • WebCoreSupport/WebContextMenuClient.mm: (WebContextMenuClient::getCustomMenuFromDefaultItems): If the delegate does not respond to contextMenuItemsForElement, return the default menu instead of nil.
1:08 PM Changeset in webkit [19408] by ggaren
  • 3 edits in trunk/LayoutTests

Reviewed by John Sullivan and Darin Adler.

Focused gc-8.html on the issue it was really trying to test (a crash)
to fix intermittent failures seen on buildbot.


  • fast/dom/gc-8.html: Don't test the value of our custom property, which (legitimately) can be cleared by garbage collection.
  • fast/dom/gc-8-expected.txt:
10:32 AM Changeset in webkit [19407] by ap
  • 4 edits
    1 add in trunk/WebCore

2007-02-05 Mitz Pettel <​mitz@webkit.org>

Reviewed by Darin.

No test possible since Mac OS X only ships with fonts that contain shaping
information.

  • icu/unicode/ushape.h: Added from ​http://www.opensource.apple.com/darwinsource/10.4.8.ppc/ICU-6.2.9/icuSources/common/unicode/
  • platform/FontData.h: (WebCore::FontData::shapesArabic): Added. Returns whether the font contains shaping information for Arabic or not.
  • platform/mac/FontDataMac.mm: (WebCore::FontData::platformInit): (WebCore::FontData::checkShapesArabic): Added. Checks whether the font contains a glyph metamorphosis table. The heuristic is that if a font contains Arabic glyphs and a metamorphosis table, then it includes shaping information for Arabic.
  • platform/mac/FontMac.mm: (WebCore::overrideLayoutOperation): Changed to use the same character buffer passed to ATSUI instead of the original character buffer and to give zero-width spaces zero width. (WebCore::isArabicLamWithAlefLigature): Added. Checks if the character is one of the Arabic presentation forms of Lam with Alef. (WebCore::shapeArabic): Added. This is a wrapper around the ICU Arabic shaping routine that replaces each space following a Lam with Alef ligature resulting from shaping with a zero-width space. (WebCore::ATSULayoutParameters::initialize): Changed to allocate the auxiliary buffer lazily for mirroring and Arabic shaping. Added a call to shapeArabic() when required.
9:24 AM Changeset in webkit [19406] by brmorris
  • 19 edits in S60/trunk

bujtas, merged of r19405 to s60/trunk by mbradley, reviewed by yongjun.

DESC: browser is crashing if it is closed while a page is loading and closing and opening a new window crashes the browser as well TMCN-6XRQP2
fix:

  1. cancel parsing when the document is being closed
  2. flush static image decoder queue. remove images associated with the the view being destroyed


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

9:07 AM Changeset in webkit [19405] by brmorris
  • 19 edits in S60/branches/3.1m

bujtas <​zbujtas@gmail.com>, reviewed by Yongjun.

DESC: browser is crashing if it is closed while a page is loading and closing and opening a new window crashes the browser as well TMCN-6XRQP2
fix:

  1. change the order of destroying webkit objects.
  2. cancel parsing when the document is being closed
  3. flush static image decoder queue. remove images associated with the the view being destroyed
  4. remove formfill dependency on webkitcontrol. formfill is static object while webkitcontrol is per window.
  5. do not create formfill manager unless browser capabilities say so


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

9:00 AM Changeset in webkit [19404] by brmorris
  • 3 edits in S60/branches/3.1m/WebKit

raalexan, Reviewed by Yongjun Zhang.

DESC: Enhanced Input feature not working in 3.1 [TSW TMCN-6XYSLY]
​http://bugs.webkit.org/show_bug.cgi?id=12533

8:51 AM Changeset in webkit [22901] by aroben
  • 2 edits in branches/WindowsMerge/WebCore

Build fix.

  • WebCore.vcproj/WebCore.vcproj: Add new files.
7:18 AM Changeset in webkit [19403] by ddkilzer
  • 1 edit in trunk/WebCore/ChangeLog

Clean up changelog entry. Also testing new svn-co-one-file script.

5:07 AM Changeset in webkit [19402] by eseidel
  • 2 edits in trunk/WebCore

2007-02-05 Eric Seidel <​eric@webkit.org>

No review, build fix only.

  • ksvg2/svg/SVGStyleElement.cpp: add forgotten header.
4:46 AM Changeset in webkit [19401] by eseidel
  • 3 edits in trunk/WebCore

2007-02-05 Eric Seidel <​eric@webkit.org>

Reviewed by bdash.

Fix style name testcase which I broke in my last commit.

  • ksvg2/svg/SVGStyleElement.cpp: (WebCore::SVGStyleElement::xmlspace): (WebCore::SVGStyleElement::title): override Element::title() (WebCore::SVGStyleElement::parseMappedAttribute):
  • ksvg2/svg/SVGStyleElement.h:
3:21 AM Changeset in webkit [19400] by eseidel
  • 11 edits
    1 add in trunk/WebCore

2007-02-04 Eric Seidel <​eric@webkit.org>

Reviewed by darin.

Fix hand cursor on link hover for SVG
Fix status text on link hover for SVG
Fix tooltips on link hover for SVG
Fix link dragging for SVG
​http://bugs.webkit.org/show_bug.cgi?id=12575


Test: added manual-tests/svg-links.svg

  • dom/Document.cpp: (WebCore::Document::prepareMouseEvent):
  • dom/Element.cpp: (WebCore::Element::title):
  • dom/Element.h:
  • html/HTMLElement.cpp: (WebCore::HTMLElement::setId): (WebCore::HTMLElement::title): (WebCore::HTMLElement::setTitle): (WebCore::HTMLElement::setLang):
  • html/HTMLElement.h:
  • ksvg2/svg/SVGAElement.cpp: (WebCore::SVGAElement::title): (WebCore::SVGAElement::parseMappedAttribute): (WebCore::SVGAElement::defaultEventHandler):
  • ksvg2/svg/SVGAElement.h:
  • ksvg2/svg/SVGTitleElement.cpp:
  • ksvg2/svg/SVGTitleElement.h:
  • manual-tests/svg-links.svg: Added.
  • rendering/HitTestResult.cpp: (WebCore::HitTestResult::title): (WebCore::HitTestResult::absoluteImageURL): (WebCore::HitTestResult::absoluteLinkURL): (WebCore::HitTestResult::isLiveLink): (WebCore::HitTestResult::titleDisplayString):

Feb 4, 2007:

10:59 PM Changeset in webkit [22900] by adachan
  • 2 edits in branches/WindowsMerge/WebKitWin

2007-02-04 Ada Chan <​adachan@apple.com>

Reviewed by Steve.

WebKitWin:

<rdar://4946475> Private browsing is not working correctly
Implemented FrameLoaderClient::privateBrowsingEnabled().

  • WebFrame.cpp: (WebFrame::privateBrowsingEnabled):


10:06 PM Changeset in webkit [19399] by ap
  • 3 edits
    2 adds in trunk

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=12602
XPath functions string(), normalize-space() and string-length() don't work
without arguments

Test: fast/xpath/implicit-node-args.html

  • xml/XPathFunctions.cpp: (WebCore::XPath::FunString::doEvaluate): (WebCore::XPath::FunStringLength::doEvaluate): (WebCore::XPath::FunNormalizeSpace::doEvaluate): Don't try to pass RefPtr<Node> to the Value ctor.
9:56 PM Changeset in webkit [19398] by ap
  • 11 edits
    14 adds in trunk

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=8791
XPath should support custom node resolvers

WebCore:

  • bindings/js/JSCustomXPathNSResolver.cpp: Added. (WebCore::JSCustomXPathNSResolver::create): (WebCore::JSCustomXPathNSResolver::JSCustomXPathNSResolver): (WebCore::JSCustomXPathNSResolver::~JSCustomXPathNSResolver): (WebCore::JSCustomXPathNSResolver::lookupNamespaceURI):
  • bindings/js/JSCustomXPathNSResolver.h: Added.
  • bindings/objc/DOMCustomXPathNSResolver.h: Added.
  • bindings/objc/DOMCustomXPathNSResolver.mm: Added. (WebCore::DOMCustomXPathNSResolver::DOMCustomXPathNSResolver): (WebCore::DOMCustomXPathNSResolver::~DOMCustomXPathNSResolver): (WebCore::DOMCustomXPathNSResolver::lookupNamespaceURI): Wrappers for JS/ObjC objects that can be used as resolvers.
  • bindings/scripts/CodeGeneratorJS.pm:
  • bindings/scripts/CodeGeneratorObjC.pm: Use the above wrappers to support custom resolvers.
  • xml/XPathNSResolver.cpp:
  • xml/XPathNSResolver.h: Turned this class into an abstract one.
  • xml/NativeXPathNSResolver.cpp: Added. (WebCore::NativeXPathNSResolver::NativeXPathNSResolver): (WebCore::NativeXPathNSResolver::~NativeXPathNSResolver): (WebCore::NativeXPathNSResolver::lookupNamespaceURI):
  • xml/NativeXPathNSResolver.h: Added. A XPathNSResolver subclass, doing what XPathNSResolver used to do.
  • xml/XPathEvaluator.cpp: (WebCore::XPathEvaluator::createNSResolver): Create a NativeXPathNSResolver.
  • WebCore.xcodeproj/project.pbxproj: Added new files.

LayoutTests:

  • fast/xpath/nsresolver-bad-object-expected.txt: Added.
  • fast/xpath/nsresolver-bad-object.xhtml: Added.
  • fast/xpath/nsresolver-exception-expected.txt: Added.
  • fast/xpath/nsresolver-exception.xhtml: Added.
  • fast/xpath/nsresolver-function-expected.txt: Added.
  • fast/xpath/nsresolver-function.xhtml: Added.
  • fast/xpath/nsresolver-object-expected.txt: Added.
  • fast/xpath/nsresolver-object.xhtml: Added.
9:55 PM Changeset in webkit [22899] by aliceli1
  • 4 edits in branches/WindowsMerge/WebKitWin

Reviewed by Steve.

Fix some memory leaks. This fixes WebFrame leak as well.
Can be considered a partial fix for <rdar://4974495> Boomer leaks while running HTML iBench

  • WebActionPropertyBag.cpp: (WebActionPropertyBag::WebActionPropertyBag):
  • WebDataSource.cpp: (WebDataSource::representation):
  • WebURLResponse.cpp: (:m_refCount):
7:52 PM Changeset in webkit [22898] by aroben
  • 3 edits in branches/WindowsMerge/WebKitWin

Reviewed by Steve.

Fix <rdar://problem/4972498> REGRESSION: Crash while closing View Source window

  • WebView.cpp: (WebView::WebView): Initialize new member. (WebViewWndProc): Set flag when we receive WM_DESTROY, and don't handle WM_SIZE if that flag is true. (WebView::mainFrame): Added argument check.
  • WebView.h: (WebView::setIsBeingDestroyed): Added. (WebView::isBeingDestroyed): Added.
6:54 PM Changeset in webkit [19397] by weinig
  • 19 edits in trunk/LayoutTests

Reviewed by Darin.

Patch for ​http://bugs.webkit.org/show_bug.cgi?id=12562
Fix test cases that are now spewing console errors that
they are not testing for

  • fast/events/iframe-object-onload-expected.txt:
  • fast/events/iframe-object-onload.html: Change nil to null.
  • fast/forms/display-none-in-onchange-keyboard-expected.txt:
  • fast/forms/display-none-in-onchange-keyboard.html: Add missing id to div, change its text, and change the resulting text.
  • fast/forms/input-text-click-inside-expected.checksum:
  • fast/forms/input-text-click-inside-expected.png:
  • fast/forms/input-text-click-inside-expected.txt:
  • fast/forms/input-text-click-inside.html: Change moveTo(x,y) to mouseMoveTo(x,y).
  • fast/forms/input-text-click-outside-expected.txt:
  • fast/forms/input-text-click-outside.html: Ditto.
  • fast/js/array-foreach-expected.txt:
  • fast/js/array-foreach.html: Add try-catch blocks to catch exceptions and print them and general cleanup.
  • fast/js/array-some-expected.txt:
  • fast/js/array-some.html: Ditto.
  • fast/table/click-near-anonymous-table-expected.checksum:
  • fast/table/click-near-anonymous-table-expected.png:
  • fast/table/click-near-anonymous-table-expected.txt:
  • fast/table/click-near-anonymous-table.html: Change timeForward(ms) to leapForward(ms) and change the amount of time from 500ms to 1000ms to avoid clicking.
6:38 PM Changeset in webkit [22897] by oliver
  • 2 edits in branches/WindowsMerge/WebCore

2007-02-04 Oliver Hunt <​oliver@apple.com>

Reviewed by Adam.

Removed unimplemented EventHandler::handleMouseUp,
as a platform independent version is available now.

  • page/win/EventHandlerWin.cpp: (WebCore::EventHandler::eventActivatedView): Now we use this method, it needs to be implemented
6:37 PM Changeset in webkit [19396] by oliver
  • 6 edits in trunk/WebCore

2007-02-04 Oliver Hunt <​oliver@apple.com>

Reviewed by Adam.

Tidying up EventHandler, and moving more of
EventHandlerMac.mm to EventHandler.cpp

Also added EventHandler::eventLoopHandleMouse[Dragged,Up]
these are basically helper functions to allow correct event
handling on MacOS.

  • page/EventHandler.cpp: (WebCore::EventHandler::handleMouseUp): (WebCore::EventHandler::dragSourceMovedTo): (WebCore::EventHandler::dragSourceEndedAt): (WebCore::EventHandler::dispatchDragSrcEvent):

Moved from EventHandlerMac to EventHandler

  • page/EventHandler.h: (WebCore::EventHandler::eventLoopHandleMouseUp): (WebCore::EventHandler::eventLoopHandleMouseDragged): Null implementations of two Mac helper functions
  • page/mac/EventHandlerMac.mm: (WebCore::EventHandler::eventLoopHandleMouseDragged): (WebCore::EventHandler::handleDrag): Much tidying, reduced nesting, made less dependent on NS types (WebCore::EventHandler::eventLoopHandleMouseDragged): (WebCore::EventHandler::eventLoopHandleMouseUp): Helper functions to let us process events properly on MacOS
  • page/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge dragSourceEndedAt:operation:]):
5:08 PM Changeset in webkit [22896] by aroben
  • 4 edits in branches/WindowsMerge/WebKitWin

WebKitWin:

Reviewed by Hyatt.

Fix <rdar://problem/4968183> Cross-window frame targeting doesn't work

  • Interfaces/IWebView.idl: Pass RECT by value.
  • WebView.cpp: (WebView::initWithFrame): Uses the passed-in RECT for the frame, and sets the frame/group names.
  • WebView.h: Updated declaration.
6:15 AM Changeset in webkit [19395] by staikos
  • 3 edits in trunk/WebCore

Various Qt related hacks and cleanups to make forms work better

Feb 3, 2007:

11:56 PM Changeset in webkit [19394] by mjs
  • 1 edit
    1 delete in trunk/WebCore

Rubber stamped by Adam.

  • remove obsolete code that has an annoying space in the directory name
  • WebCore.vcproj/Image Viewer.cpp: Removed.
  • WebCore.vcproj/Image Viewer.h: Removed.
  • WebCore.vcproj/Image Viewer.ico: Removed.
  • WebCore.vcproj/Image Viewer.rc: Removed.
  • WebCore.vcproj/Image Viewer.reg: Removed.
  • WebCore.vcproj/Image Viewer.vcproj: Removed.
  • WebCore.vcproj/Image Viewer/ImageDocument.cpp: Removed.
  • WebCore.vcproj/Image Viewer/ImageDocument.h: Removed.
  • WebCore.vcproj/Image Viewer/ImageMainFrame.cpp: Removed.
  • WebCore.vcproj/Image Viewer/ImageMainFrame.h: Removed.
  • WebCore.vcproj/Image Viewer/ImageView.cpp: Removed.
  • WebCore.vcproj/Image Viewer/ImageView.h: Removed.
  • WebCore.vcproj/Image Viewer/Resource.h: Removed.
  • WebCore.vcproj/Image Viewer/res/ImageViewer.rc2: Removed.
  • WebCore.vcproj/Image Viewer/res/Toolbar.bmp: Removed.
  • WebCore.vcproj/Image Viewer/stdafx.cpp: Removed.
  • WebCore.vcproj/Image Viewer/stdafx.h: Removed.
  • WebCore.vcproj/Image iewerDoc.ico: Removed.
11:37 PM Changeset in webkit [19393] by staikos
  • 2 edits in trunk/JavaScriptCore

fix crash on JS that ends with an identifier

11:34 PM Changeset in webkit [19392] by oliver
  • 2 edits in trunk/WebCore

2007-02-04 Mitz Pettel <​mitz@webkit.org>

Reviewed by Oliver.

  • platform/mac/PlatformMouseEventMac.mm: (WebCore::eventNumberForEvent): Scroll wheel events don't have an event number.
10:50 PM Changeset in webkit [22895] by sfalken
  • 2 edits in branches/WindowsMerge/WebKitWin

2007-02-03 Steve Falkenburg <​sfalken@apple.com>

Reviewed by Adam.


<rdar://problem/4930947> Regression: History is not saved when boomer window is closed.


Fix boolean test causing lost history regression.

  • WebHistoryItem.cpp: (WebHistoryItem::hasURLString): isEmpty returns true if there is NO url, not if there is one.
6:07 PM Changeset in webkit [19391] by ggaren
  • 1 edit
    1 add in trunk/LayoutTests

Landing results that I think Maciej intended to land, to get the buildbot
back to green.

  • fast/forms/disabled-select-change-index-expected.txt: Added.
5:25 PM Changeset in webkit [19390] by zimmermann
  • 12 edits
    4 adds in trunk

Reviewed by Sam.

Add support for xml:space="preserve" for SVG text.
It can be modeled by using white-space="pre" and replace \n by spaces.

Handle xml:space="default" correctly.
Always remove tabs from SVG text - as demanded by spec.

Added test: svg/custom/text-whitespace-handling.svg

12:35 PM Changeset in webkit [19389] by ap
  • 5 edits in trunk

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=12560
W3C XPath test Text_Nodes.svg fails

Covered by dom/svg/level3/xpath/Text_Nodes.svg

  • xml/XPathStep.cpp: (WebCore::XPath::Step::nodeTestMatches): Only include the first text sibling in text() matches.
12:22 PM Changeset in webkit [19388] by ap
  • 4 edits
    2 adds in trunk

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=12485
XPath id() function doesn't work as root in paths

Test: fast/xpath/id-path.html

  • xml/XPathPath.cpp: (WebCore::XPath::Path::doEvaluate): Implemented. (WebCore::XPath::LocationPath::evaluate): A new overload that takes an initial NodeVector. (WebCore::XPath::LocationPath::doEvaluate): Moved most of the logic to the new evaluate().
  • xml/XPathPath.h: Added a new evaluate() overload and a using declaration to keep the generic one visible.
12:14 PM Changeset in webkit [19387] by ap
  • 2 edits in trunk/WebCore

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=12559
REGRESSION: Crash in checkForHeadCharset() under GuardMalloc

No tests possible.

  • loader/TextResourceDecoder.cpp: (WebCore::TextResourceDecoder::checkForHeadCharset): Fix end of buffer checks; sprinkle a number of new ones over the function.
7:00 AM Changeset in webkit [19386] by weinig
  • 2 edits in trunk/WebKitTools

Reviewed by Mark.

Patch for ​http://bugs.webkit.org/show_bug.cgi?id=12555
Drosera doesn't handle CR ( carriage returns ) well

  • Normalize all the line endings.
  • Drosera/debugger.js:
4:16 AM Changeset in webkit [19385] by ddkilzer
  • 2 edits in trunk/WebCore

WebCore:

Reviewed by NOBODY (no-svg build fix).

  • page/EventHandler.cpp: Include FloatPoint.h.
2:53 AM Changeset in webkit [19384] by ap
  • 3 edits
    2 adds in trunk

Reviewed by Mitz.

​http://bugs.webkit.org/show_bug.cgi?id=12549
Wrong character encoding at www.ravmilim.co.il

Test: fast/encoding/meta-charset.html

  • loader/TextResourceDecoder.cpp: (WebCore::TextResourceDecoder::checkForHeadCharset): Fix an off by one error.

Feb 2, 2007:

9:54 PM Changeset in webkit [19383] by mjs
  • 3 edits
    1 add in trunk

LayoutTests:

Reviewed by Hyatt & Adele.

  • test case for <rdar://problem/4896972> TV guide listings page doesn't finish loading (11674)
  • fast/forms/disabled-select-change-index.html: Added.

WebCore:

Reviewed by Hyatt & Adele.


  • <rdar://problem/4896972> TV guide listings page doesn't finish loading (11674)


Test case: fast/forms/disabled-select-change-index.html


  • html/HTMLOptionElement.cpp: (WebCore::HTMLOptionElement::setSelected): Allow changing selected state of a disabled option.
8:16 PM Changeset in webkit [19382] by bdash
  • 3 edits in tags/Safari-521.34

Versioning.

8:11 PM Changeset in webkit [22894] by aliceli1
  • 4 edits in branches/WindowsMerge/WebKitWin

WebKitWin:

Reviewed by Maciej.

Fixed <rdar://4956286> Find: Deleting all text in the Find banner search field should clear the page's selection

  • Interfaces/IWebView.idl:
  • WebView.cpp: (WebView::clearSelection):
  • WebView.h:

Added a clearSelection function to webview so the findbanner can call it.

8:09 PM Changeset in webkit [19381] by bdash
  • 21 edits in tags/Safari-521.34

Roll out r19327.

7:58 PM Changeset in webkit [19380] by bdash
  • 1 copy in tags/Safari-521.34

New tag.

5:36 PM Changeset in webkit [19379] by zimmermann
  • 2 edits in trunk/WebCore

Build fix, not reviewed.
Blind attempt to fix Qt build.

5:33 PM Changeset in webkit [19378] by zimmermann
  • 14 edits
    16 adds in trunk

Reviewed by Eric & Maciej.

Fix serveral <use> issues:

  • When events are dispatch to a referenced element, the actual event.target

has to be set to the corresponding SVGElementInstance object

  • Node::shadowAncestorNode() should _not_ return the shadowParentNode for SVG elements

This is only needed for the HTML forms concept and breaks event dispatching for SVG shadow tree elements.

  • Add SVGElementInstance as possible EventTarget in bindings/js/kjs_dom.cpp

(Now event.target returns the actual SVGElementInstance object for a shadow tree element)

Fixes: ​http://bugs.webkit.org/show_bug.cgi?id=12502 (Reproducible crash when browsing SVG map.)
Fixes: ​http://bugs.webkit.org/show_bug.cgi?id=12511 (<use> has event dispatching issues)

Added test: svg/custom/use-event-handler-on-use-element.svg
Added test: svg/custom/use-event-handler-on-referenced-element.svg
Added test: svg/custom/use-elementInstance-event-target.svg
Added test: svg/custom/use-elementInstance-methods.svg

5:06 PM Changeset in webkit [19377] by ggaren
  • 3 edits in trunk/WebKitTools

Reviewed by Darin Adler.

Added some more known leaks to the leaks ignore list.

  • DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj: Why does XCode hate itself so much?
  • Scripts/run-webkit-tests:
4:30 PM Changeset in webkit [22893] by aroben
  • 2 edits in branches/WindowsMerge/WebKitWin

WebKitWin:

Reviewed by Anders.

Fix a crash when running the PLT.

  • WebView.cpp: (WebView::initWithFrame):

Added assertions to make catching this easier.

4:25 PM Changeset in webkit [19376] by oliver
  • 2 edits in trunk/WebCore

2007-02-02 Oliver Hunt <​oliver@apple.com>

Reviewed by Geoff.

Add guard around accessing [NSEvent eventNumber] to ensure
we don't attempt to access it on on non-mouse related NSEvent.

  • platform/mac/PlatformMouseEventMac.mm: (WebCore::eventNumberForEvent): (WebCore::PlatformMouseEvent::PlatformMouseEvent):
3:53 PM Changeset in webkit [19375] by aroben
  • 2 edits in trunk/WebCore

Reviewed by Anders.

Try to be a little less selfish about DocumentLoader.

Spotted while working on <rdar://problem/4957747>.

  • page/ContextMenuController.cpp: (WebCore::ContextMenuController::contextMenuItemSelected):

Everyone can open frames in new windows.

3:45 PM Changeset in webkit [19374] by zimmermann
  • 9 edits in trunk/WebCore

Reviewed by Eric.

Move xml:space/xml:lang from svgattrs.in to xmlattrs.in where they actually belong.
Now they are actually recognized by SVGLangSpace::parseMappedAttribute.
Patch implementing xml:space="preserve" coming soon, which acts as test for this stuff.

Always build XMLNames.cpp (not only for SVG) and replace all occourences of
"​http://www.w3.org/XML/1998/namespace" by "XMLNames::xmlNamespaceURI".

3:03 PM Changeset in webkit [19373] by bdakin
  • 2 edits in trunk/WebCore

Reviewed by Maciej.

Working on a layout test.

Fix for <rdar://problem/4966100> REGRESSION: Reproducible ASSERT
hit at Walgreens.com

  • page/Frame.cpp: (WebCore::Frame::setView): If we are setting the view to 0 and detaching the document, call unscheduleRealyout() on the old view before setting it to 0.
12:42 PM Changeset in webkit [19372] by brmorris
  • 26 edits in S60/branches/3.1m

yongjzha, Reviewed by zalan.

DESC: backport google.com/ig refreshing memory leak fix in <​http://bugs.webkit.org/show_bug.cgi?id=10773> PPEN-6QYG7L
improve stability under low memory condition.
​http://bugs.webkit.org/show_bug.cgi?id=12541

11:57 AM Changeset in webkit [19371] by ggaren
  • 1 edit in trunk/WebKitTools/DumpRenderTree/DumpRenderTree.m

add const back, too

11:56 AM Changeset in webkit [19370] by ggaren
  • 2 edits in trunk/WebKitTools

Even better build fix than the last.

  • ChangeLog:
  • DumpRenderTree/DumpRenderTree.m: (shouldIgnoreWebCoreNodeLeaks):
11:50 AM Changeset in webkit [19369] by brmorris
  • 2 edits in S60/branches/3.1m/WebCore

bujtas, Reviewed by yongjun.

Merge of r19368 to 3.1 by Brad.
DESC: browser crashes on www.tays.fi. when multiple layers are referring to the same background image, the render_object does not do proper deref on the image. MLIO-6XJAXG
fix: loop through the background layers and defer background images
​http://bugs.webkit.org/show_bug.cgi?id=12543

11:47 AM Changeset in webkit [19368] by brmorris
  • 2 edits in S60/trunk/WebCore

bujtas, Reviewed by yongjun.

DESC: browser crashes on www.tays.fi. when multiple layers are referring to the same background image, the render_object does not do proper deref on the image.
fix: loop through the background layers and defer background images
​http://bugs.webkit.org/show_bug.cgi?id=12543

10:57 AM Changeset in webkit [19367] by ggaren
  • 2 edits in trunk/WebKitTools

Fixed build. Added work-around for GCC bug.

  • DumpRenderTree/DumpRenderTree.m: (shouldIgnoreWebCoreNodeLeaks):
10:48 AM Changeset in webkit [22892] by sfalken
  • 2 edits in branches/WindowsMerge/WebCore

2007-02-02 Steve Falkenburg <​sfalken@apple.com>

Fix build.

  • WebCore.vcproj/WebCore.vcproj:
10:36 AM Changeset in webkit [19366] by ggaren
  • 1 edit in trunk/WebKitTools/DumpRenderTree/DumpRenderTree.m

build fix? i don't think so, but let's see...

10:04 AM Changeset in webkit [22891] by sfalken
  • 2 edits in branches/WindowsMerge/WebCore
9:57 AM Changeset in webkit [19365] by darin
  • 2 edits in trunk/JavaScriptCore

Reviewed by Anders.

  • fix copying and assigning a ListHashSet

No test because the code path with bugs I am fixing is not used yet.

  • wtf/ListHashSet.h: Tweaked ListHashSetNodeAllocator a little bit for clarity. Changed m_allocator to be an OwnPtr instead of doing an explicit delete. Fixed bug in copy constructor where we'd have an uninitialized m_allocator. Fixed bug in assignment operator where it would swap only the hash table, and not the head, tail, and allocator pointers.
9:55 AM Changeset in webkit [19364] by ggaren
  • 12 edits in trunk

JavaScriptCore:

Reviewed by Maciej Stachowiak.


Use WTFLog instead of fprintf for logging KJS::Node leaks.

  • kjs/nodes.cpp: (NodeCounter::~NodeCounter): Changed count to unsigned, updated to match style guidelines.

WebCore:

Reviewed by Maciej Stachowiak.


Added support for selectively ignoring WebCore::Node leaks during layout
tests, so that we can ignore known leaks in other components.

  • WebCore.exp:
  • dom/Node.cpp: (WebCore::Node::isSupported): Moved isSupported up with the rest of the static functions.

(WebCore::): Added an ignoreSet, which collects WebCore::Nodes whose lifetime
we want to ignore. We need to track which nodes to ignore rather than, say,
just suspending the count, because node destruction depends on lots of
different variables, so it would be nearly impossible to know when exactly
to suspend the count and when exactly to resume it.

(WebCore::NodeCounter::~NodeCounter): Changed to use WTFLog instead of fprintf.

(WebCore::Node::startIgnoringLeaks): Do the ignoring.
(WebCore::Node::stopIgnoringLeaks): ditto
(WebCore::Node::Node): ditto
(WebCore::Node::~Node): ditto

  • dom/Node.h: Moved isSupported up with the rest of the static functions.
  • platform/mac/LoggingMac.mm: (WebCore::initializeWithUserDefault): Renamed from "initializeLoggingChannel" because the real goal here is to honor a user default -- WTFLoggingChannels need no run-time initialization. Also replaced "off by default, on if a user default says so" behavior with "only override existing setting if a user default says so" behavior. It seemed like a bug that you would specify a channel's on/off state in its definition, but this function would unconditionally blow that state away. (WebCore::InitializeLoggingChannelsIfNecessary):

WebKit:

Reviewed by Maciej Stachowiak.


Added support for selectively ignoring WebCore::Node leaks during layout
tests, so that we can ignore known leaks in other components.

  • Misc/WebCoreStatistics.h:
  • Misc/WebCoreStatistics.mm: (+[WebCoreStatistics startIgnoringWebCoreNodeLeaks]): (+[WebCoreStatistics stopIgnoringWebCoreNodeLeaks]):

WebKitTools:

Reviewed by Maciej Stachowiak.


Added support for selectively ignoring WebCore::Node leaks during layout
tests, so that we can ignore known leaks in other components.

  • DumpRenderTree/DumpRenderTree.m: (shouldIgnoreWebCoreNodeLeaks): Implements a black list of tests whose WebCore::Node leaks we have to ignore. Does this CFString gobbledy-gook confuse anyone else? (runTest):
8:47 AM Changeset in webkit [19363] by brmorris
  • 2 edits in S60/trunk/WebKit

yaharon, Reviewed by Yongjun.

DESC: [S60] FUTE_32: If web page included a link, which name is e.g. T0000900 user can't open it, cause phone starts calling to number 0000900. (KDIS-6XYCL5)
​http://bugs.webkit.org/show_bug.cgi?id=12534

8:28 AM Changeset in webkit [19362] by eseidel
  • 190 edits
    8 adds in trunk

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

Reviewed by mitz.

Fix gradient stop style resolution to be less of a hack, fixing style invalidation:
​http://bugs.webkit.org/show_bug.cgi?id=12193
and fix stop-color and stop-opacity not to inherit:
​http://bugs.webkit.org/show_bug.cgi?id=12193


Tests:

  • svg/custom/gradient-stops-style-change.svg: added
  • svg/W3C-SVG-1.1/pservers-grad-18-b.svg: updated
  • WebCore.xcodeproj/project.pbxproj:
  • ksvg2/css/SVGCSSStyleSelector.cpp: (WebCore::CSSStyleSelector::applySVGProperty):
  • ksvg2/css/SVGRenderStyle.cpp: (WebCore::SVGRenderStyle::inheritedNotEqual): (WebCore::SVGRenderStyle::inheritFrom):
  • ksvg2/css/SVGRenderStyle.h:
  • ksvg2/svg/SVGDefsElement.cpp: (WebCore::SVGDefsElement::createRenderer):
  • ksvg2/svg/SVGGradientElement.cpp: (WebCore::SVGGradientElement::createRenderer): (WebCore::SVGGradientElement::buildStops):
  • ksvg2/svg/SVGGradientElement.h: (WebCore::SVGGradientElement::rendererIsNeeded):
  • ksvg2/svg/SVGStopElement.cpp: (WebCore::SVGStopElement::createRenderer):
  • ksvg2/svg/SVGStopElement.h: (WebCore::SVGStopElement::rendererIsNeeded):
  • rendering/RenderSVGGradientStop.cpp: Added.
  • rendering/RenderSVGGradientStop.h: Added.
  • rendering/RenderSVGHiddenContainer.cpp: Added.
  • rendering/RenderSVGHiddenContainer.h: Added.
  • rendering/RenderSVGContainer.cpp:
  • rendering/RenderSVGContainer.h:
6:41 AM Changeset in webkit [19361] by zack
  • 2 edits in trunk/WebCore

Fix scrolling-on-selection bug by correctly
scrolling to the offset instead of trying to
ensure visibility with margins
r=Lars

6:09 AM Changeset in webkit [19360] by bdash
  • 3 edits in tags/Safari-419.1

Versioning.

6:07 AM Changeset in webkit [19359] by bdash
  • 1 copy in tags/Safari-419.1

New tag.

6:06 AM Changeset in webkit [19358] by bdash
  • 6 edits in branches/Safari-2-0-branch

Update bundle identifiers.

4:27 AM Changeset in webkit [22890] by aroben
  • 2 edits in branches/WindowsMerge/WebCore

Turn on Lucida Grande.

3:06 AM Changeset in webkit [22889] by aroben
  • 3 edits in branches/WindowsMerge/WebCore

WebCoreWin:

Reviewed by Maciej.

Implemented menulist drawing.

2:26 AM Changeset in webkit [19357] by zack
  • 1 edit in trunk/WebCore/platform/graphics/qt/GraphicsContextQt.cpp

actually set the pen on color setting

2:24 AM Changeset in webkit [19356] by zack
  • 2 edits in trunk/WebCore

Fix drawing of dashed and colored lines. Pen
has to be inherited for stroke to stay consistent.
r=Lars

1:49 AM Changeset in webkit [19355] by zack
  • 2 edits in trunk/WebKitQt

Compilation fix: forward declare d-ptr.

1:35 AM Changeset in webkit [19354] by mjs
  • 2 edits in trunk/JavaScriptCore
  • not reviewed, build fix
  • wtf/ListHashSet.h: (WTF::ListHashSetNodeAllocator::ListHashSetNodeAllocator): ummm, use union correctly
1:14 AM Changeset in webkit [19353] by mjs
  • 1 edit in trunk/JavaScriptCore/ChangeLog

Reviewed by Darin.


  • use a custom allocator for ListHashSet, to fix ~1% perf regression using it for form control
  • wtf/ListHashSet.h: (WTF::ListHashSetNodeAllocator::ListHashSetNodeAllocator): (WTF::ListHashSetNodeAllocator::allocate): (WTF::ListHashSetNodeAllocator::deallocate): (WTF::ListHashSetNode::operator new): (WTF::ListHashSetNode::operator delete): (WTF::ListHashSetNode::destroy): (WTF::ListHashSetTranslator::translate): (WTF::::ListHashSet): (WTF::::~ListHashSet): (WTF::::add): (WTF::::unlinkAndDelete): (WTF::::deleteAllNodes):
1:09 AM Changeset in webkit [19352] by mjs
  • 2 edits in trunk/JavaScriptCore

Reviewed by Darin.


  • use a custom allocator for ListHashSet, to fix ~1% per regression using it for form control
  • wtf/ListHashSet.h: (WTF::ListHashSetNodeAllocator::ListHashSetNodeAllocator): (WTF::ListHashSetNodeAllocator::allocate): (WTF::ListHashSetNodeAllocator::deallocate): (WTF::ListHashSetNode::operator new): (WTF::ListHashSetNode::operator delete): (WTF::ListHashSetNode::destroy): (WTF::ListHashSetTranslator::translate): (WTF::::ListHashSet): (WTF::::~ListHashSet): (WTF::::add): (WTF::::unlinkAndDelete): (WTF::::deleteAllNodes):
12:41 AM Changeset in webkit [19351] by zack
  • 3 edits in trunk/WebKitQt

Make the frames white by default and get the
files that WebCore can handle render inside
the frames.
r=Lars

Feb 1, 2007:

11:43 PM Changeset in webkit [19350] by bdash
  • 13 edits
    4 adds in trunk

2007-02-01 Nicholas Shanks <​webkit@nickshanks.com>

Reviewed by Mark.

Add support for CSS2 font-stretch property.

  • css/CSSComputedStyleDeclaration.cpp: (WebCore::): (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
  • css/cssparser.cpp: (WebCore::CSSParser::parseValue):
  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::applyDeclarations): (WebCore::CSSStyleSelector::applyProperty):
  • platform/FontCache.cpp: (WebCore::FontPlatformDataCacheKey::FontPlatformDataCacheKey): (WebCore::FontPlatformDataCacheKey::operator==): (WebCore::computeHash): (WebCore::FontPlatformDataCacheKeyTraits::deletedValue): (WebCore::FontPlatformDataCacheKeyTraits::emptyValue): (WebCore::FontCache::getCachedFontPlatformData):
  • platform/FontDescription.h: (WebCore::): (WebCore::FontDescription::FontDescription): (WebCore::FontDescription::stretch): (WebCore::FontDescription::narrowerStretch): (WebCore::FontDescription::widerStretch): (WebCore::FontDescription::setStretch): (WebCore::FontDescription::operator==):
  • platform/mac/FontCacheMac.mm: (WebCore::FontCache::createFontPlatformData):
  • platform/mac/WebFontCache.mm: (acceptableChoice): (betterChoice): (+[WebFontCache fontWithFamily:traits:size:]):

2007-02-01 Nicholas Shanks <​webkit@nickshanks.com>

Reviewed by Mark.

CSS2 font-stretch property; uses American Typewriter and Helvetica Neue fonts,
requiring Regular, Condensed, Bold and Bold Condensed faces.

  • fast/css/computed-style-expected.txt: Updated.
  • fast/css/font-stretch-expected.checksum: Added.
  • fast/css/font-stretch-expected.png: Added.
  • fast/css/font-stretch-expected.txt: Added.
  • fast/css/font-stretch.html: Added.

2007-02-01 Nicholas Shanks <​webkit@nickshanks.com>

Reviewed by Mark.

Add support for CSS2 font-stretch property.

  • WebInspector/webInspector/inspector.js:
  • WebView/WebHTMLView.mm: (-[WebHTMLView _addToStyle:fontA:fontB:]):
10:57 PM Changeset in webkit [19349] by thatcher
  • 2 edits in trunk/WebCore

Reviewed by Mark Rowe.

<rdar://problem/4969201> REGRESSION: Safari applies the wrong font to BODY element at ​http://www.apple.com/downloads/dashboard/

  • platform/mac/WebFontCache.mm: (+[WebFontCache fontWithFamily:traits:size:]): Don't call [NSFont fontWithName:] with the desiredFamily on Tiger.
9:52 PM Changeset in webkit [19348] by ap
  • 4 edits in trunk

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=12506
REGRESSION: Safari doesn't display hebrew text on a web page,
displayed correctly on Tiger

WebCore:

  • loader/TextResourceDecoder.cpp: (WebCore::TextResourceDecoder::checkForHeadCharset): Advance the pointer when necessary.

LayoutTests:

  • fast/encoding/pseudo-tags-in-attributes-expected.txt: Changing back to success.
9:44 PM Changeset in webkit [19347] by staikos
  • 4 edits
    2 adds in trunk

Add a cookie interface for the Qt build.

7:11 PM Changeset in webkit [19346] by bdakin
  • 1530 edits in trunk/LayoutTests/tables/mozilla/bugs

un-doing accidental change that made all of the files in tables/mozilla/bugs allow tabs. whoopsie!

6:40 PM Changeset in webkit [19345] by mjs
  • 3 edits in trunk/WebCore

Reviewed by Adele.


  • fixed <rdar://problem/4922764> ASSERTION: (!m_contentFunction) in PolicyCheck::call trying to visit feed URL (CNN, Apple, etc.)
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::stopPolicyCheck): Call new cancel method instead of call(false) on old check. (WebCore::PolicyCheck::cancel): call any callback function that is set with false or PolicyIgnore as appropriate.
  • loader/FrameLoader.h:
5:55 PM Changeset in webkit [22888] by sfalken
  • 1 edit in branches/WindowsMerge/WebKitWin/WebKit.vcproj/VERSION

Bump version for submit

5:55 PM Changeset in webkit [19344] by sfalken
  • 1 copy in tags/Safari-521.33.1b

New tag.

5:41 PM Changeset in webkit [22887] by adachan
  • 2 edits in branches/WindowsMerge/WebKitWin

2007-02-01 Ada Chan <​adachan@apple.com>

Reviewed by Anders.


Forgot to assign document to 0 in case of failure in WebView::mainFrameDocument().

  • WebView.cpp: (WebView::mainFrameDocument):
5:34 PM Changeset in webkit [19343] by andersca
  • 74 edits in trunk

LayoutTests:

Reviewed by Darin.

Update test results that have console messages.


  • dom/html/level2/html/HTMLFrameElement09-expected.txt:
  • dom/html/level2/html/HTMLIFrameElement11-expected.txt:
  • dom/xhtml/level2/html/HTMLBaseElement01-expected.txt:
  • dom/xhtml/level2/html/HTMLBaseElement02-expected.txt:
  • dom/xhtml/level2/html/frame-expected.txt:
  • dom/xhtml/level2/html/iframe-expected.txt:
  • editing/execCommand/4920488-expected.txt:
  • editing/selection/contenteditable-click-inside-expected.txt:
  • editing/selection/contenteditable-click-outside-expected.txt:
  • fast/AppleScript/001-expected.txt:
  • fast/dynamic/015-expected.txt:
  • fast/events/iframe-object-onload-expected.txt:
  • fast/events/updateLayoutForHitTest-expected.txt:
  • fast/forms/display-none-in-onchange-keyboard-expected.txt:
  • fast/forms/input-text-click-inside-expected.txt:
  • fast/forms/input-text-click-outside-expected.txt:
  • fast/forms/selected-index-assert-expected.txt:
  • fast/js/array-foreach-expected.txt:
  • fast/js/array-some-expected.txt:
  • fast/js/kde/exception_propagation-expected.txt:
  • fast/js/kde/garbage-n-expected.txt:
  • fast/js/kde/string-1-n-expected.txt:
  • fast/js/kde/string-2-n-expected.txt:
  • fast/parser/script-tag-with-trailing-slash-expected.txt:
  • fast/table/click-near-anonymous-table-expected.txt:
  • fast/tokenizer/002-expected.txt:
  • fast/tokenizer/external-script-document-write_2-expected.txt:
  • fast/tokenizer/script_extra_close-expected.txt:
  • fast/xsl/transform-xhr-doc-expected.txt:
  • http/tests/navigation/error404-goback-expected.txt:
  • http/tests/xmlhttprequest/set-dangerous-headers-expected.txt:
  • svg/W3C-SVG-1.1/animate-elem-30-t-expected.txt:
  • svg/W3C-SVG-1.1/animate-elem-33-t-expected.txt:
  • svg/W3C-SVG-1.1/animate-elem-34-t-expected.txt:
  • svg/W3C-SVG-1.1/animate-elem-37-t-expected.txt:
  • svg/W3C-SVG-1.1/animate-elem-41-t-expected.txt:
  • svg/W3C-SVG-1.1/animate-elem-78-t-expected.txt:
  • svg/W3C-SVG-1.1/color-prop-03-t-expected.txt:
  • svg/W3C-SVG-1.1/interact-cursor-01-f-expected.txt:
  • svg/W3C-SVG-1.1/linking-uri-01-b-expected.txt:
  • svg/W3C-SVG-1.1/linking-uri-02-b-expected.txt:
  • svg/W3C-SVG-1.1/metadata-example-01-b-expected.txt:
  • svg/W3C-SVG-1.1/painting-marker-03-f-expected.txt:
  • svg/W3C-SVG-1.1/shapes-polygon-01-t-expected.txt:
  • svg/W3C-SVG-1.1/shapes-polyline-01-t-expected.txt:
  • svg/W3C-SVG-1.1/struct-group-03-t-expected.txt:
  • svg/W3C-SVG-1.1/struct-use-01-t-expected.txt:
  • svg/W3C-SVG-1.1/styling-css-01-b-expected.txt:
  • svg/W3C-SVG-1.1/styling-css-02-b-expected.txt:
  • svg/W3C-SVG-1.1/styling-css-03-b-expected.txt:
  • svg/custom/dynamic-empty-path-expected.txt:
  • svg/custom/js-update-container-expected.txt:
  • svg/custom/js-update-gradient-expected.txt:
  • svg/custom/js-update-polygon-changes-expected.txt:
  • svg/custom/js-update-polygon-removal-expected.txt:
  • svg/custom/path-bad-data-expected.txt:
  • svg/custom/polyline-hittest-expected.txt:
  • svg/custom/polyline-invalid-points-expected.txt:
  • svg/custom/polyline-setattribute-points-null-expected.txt:
  • svg/custom/svgpolyparser-extra-space-expected.txt:
  • svg/custom/use-modify-target-symbol-expected.txt:
  • svg/custom/viewbox-syntax-expected.txt:
  • tables/mozilla/bugs/bug53690-1-expected.txt:
  • tables/mozilla/bugs/bug9024-expected.txt:
  • tables/mozilla_expected_failures/bugs/bug3517-expected.txt:
  • tables/mozilla_expected_failures/bugs/bug92868_1-expected.txt:
  • tables/mozilla_expected_failures/core/captions1-expected.txt:
  • tables/mozilla_expected_failures/dom/appendCells1-expected.txt:
  • tables/mozilla_expected_failures/dom/appendCellsRebuild1-expected.txt:
  • tables/mozilla_expected_failures/dom/insertTbodyExpand1-expected.txt:
  • tables/mozilla_expected_failures/dom/insertTbodyRebuild1-expected.txt:

WebKitTools:

Reviewed by Darin.

  • DumpRenderTree/UIDelegate.m: (-[UIDelegate webView:addMessageToConsole:]): Dump console messages.
4:52 PM Changeset in webkit [19342] by bdakin
  • 1525 edits
    3 moves
    13 adds
    9 deletes in trunk

WebCore:

Reviewed by Hyatt.

Fix for ​http://bugs.webkit.org/show_bug.cgi?id=10517 REGRESSION
(r12065-r12082): Navigation graphic wraps to the next line on
duart.com and <rdar://problem/4960505> REGRESSION: The Awesome
Windows Vista page has a layout error in the header table (10517)

The problem is that we never supported a quirk that other browsers
support where, under very specific circumstances, a table cell will
grow to fit an image inside of it. This is only a regression
because a bug was fixed were we were 1 pixel off in the calculation
of overflow in some cases.

  • rendering/RenderBlock.cpp: (WebCore::shouldGrowTableCellForImage): (WebCore::RenderBlock::calcInlineMinMaxWidth):
4:48 PM Changeset in webkit [22886] by adachan
  • 3 edits in branches/WindowsMerge/WebKitWin

2007-02-01 Ada Chan <​adachan@apple.com>

Reviewed by Steve, Geoff.

WebKitWin:

<rdar://4969341> Regression: crash: when Resetting Safari
<rdar://4953390> Closing my two open boomer windows produced a crash
Fixed crash when closing second window and doing the close
all safari window option in Reset Safari.

  • WebFrame.cpp: WebFramePrivate has a pointer to frameView but does not hold a reference to it. The frame holds a ref to the frameView anyway, so it's safest to get the frameView through the frame. The crash happened because frame already has cleared out its frameView but WebFrame was still accessing the old stale frameView through that data member in WebFramePrivate. (WebFrame::WebFramePrivate::m_policyFunction): (WebFrame::WebFramePrivate::frameView): (WebFrame::initWithWebFrameView): (WebFrame::layoutIfNeeded): (WebFrame::hasFrameView):
  • WebView.cpp: (WebView::paintIntoBackingStore): bail if frameView is null. (WebView::setHostWindow): don't set the view window's parent to 0. It'll end up creating a standalone window.


3:35 PM Changeset in webkit [22885] by andersca
  • 2 edits in branches/WindowsMerge/WebCore

Reviewed by Brady.

  • platform/network/cf/ResourceHandleCFNet.cpp: (WebCore::willSendRequest): Make sure to always retain the request we return. (In practice, we were already doing this since client can never be null).


(WebCore::didReceiveResponse):
Remove client null check.

3:06 PM Changeset in webkit [22884] by sfalken
  • 2 edits in branches/WindowsMerge/WebCore

2007-02-01 Steve Falkenburg <​sfalken@apple.com>

B&I build fix

  • WebCore.vcproj/WebCore.vcproj:
11:57 AM Changeset in webkit [22883] by andersca
  • 2 edits in branches/WindowsMerge/WebKitWin

Reviewed by Adam.

<rdar://problem/4969348>
REGRESSION: Crash loading pdf


  • WebView.cpp: (WebView::canShowMIMEType): Return true if the MIME type can be handled by a plug-in.
11:57 AM Changeset in webkit [19341] by lars
  • 2 edits in trunk/WebKitQt

Reviewed by Adam.

Fix form loading for the Qt build.

11:53 AM Changeset in webkit [19340] by brmorris
  • 2 edits in S60/trunk/WebCore

brmorris, reviewed by zalan

DESC: Updated def file

  • BWINS/WEBCOREU.DEF:
9:34 AM Changeset in webkit [19339] by darin
  • 2 edits in trunk/LayoutTests
  • fast/encoding/pseudo-tags-in-attributes-expected.txt: Change test result to reflect failure because bug 12506 isn't fixed yet; we'll change it back once the bug is re-fixed.
4:27 AM Changeset in webkit [19338] by zack
  • 5 edits in trunk/WebCore

Implement native look for comboboxes
and fix/implement popup menus. Also add the m_timestamp
and m_eventType setting to PlatformMouseEvent
r=Lars

12:56 AM Changeset in webkit [19337] by mjs
  • 2 edits in trunk/WebKit

Reviewed by Adam.


<rdar://problem/4730469> REGRESSION: Assertion failure in -[WebDataSource(WebInternal) _bridge] when forwarding message

  • WebView/WebDataSource.mm: (-[WebDataSource subresources]): Check for being uncommitted and return emtpy result. (-[WebDataSource subresourceForURL:]): ditto
12:44 AM Changeset in webkit [22882] by aroben
  • 7 edits in branches/WindowsMerge/WebKitWin

WebKitWin:

Reviewed by Steve.

Fix <rdar://problem/4849601> Implement printing of individual subframes

This just moves some functions from IWebViewPrivate to IWebFramePrivate.

  • Interfaces/IWebFramePrivate.idl: Added declarations.
  • Interfaces/IWebViewPrivate.idl: Removed declarations.
  • WebFrame.cpp: Added functions from WebView.cpp. (WebFrame::computePageRects): (WebFrame::getPrintedPageCount): (WebFrame::spoolPages):
  • WebFrame.h: Added declarations.
  • WebView.cpp: Moved functions to WebFrame.cpp.
  • WebView.h: Removed declarations.
12:36 AM Changeset in webkit [19336] by darin
  • 5 edits
    2 adds in trunk

LayoutTests:

Reviewed by Maciej.

  • test for <rdar://problem/4887428> REGRESSION: Implement slight delay for firing incremental onSearch event
  • fast/forms/search-event-delay-expected.txt: Added.
  • fast/forms/search-event-delay.html: Added.

WebCore:

Reviewed by Maciej.

  • fix <rdar://problem/4887428> REGRESSION: Implement slight delay for firing incremental onSearch event
  • rendering/RenderTextControl.h:
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::RenderTextControl): Set up timer. (WebCore::RenderTextControl::subtreeHasChanged): Start timer here instead of immediately sending event. (WebCore::RenderTextControl::searchEventTimerFired): Added. Sends search event. (WebCore::RenderTextControl::stopSearchEventTimer): Added. (WebCore::RenderTextControl::startSearchEventTimer): Added. Sends search event right away if there is no text. If there is some text, sets the timer using the same delay rule as NSSearchField. If you keep typing, then the timer keeps getting reset 0.2 seconds into the future until you pause.
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::onSearch): Tell the renderer to stop the timer, since we're sending a search event. This helps when a caller other than the timer decides to send an explicit search event.
12:23 AM Changeset in webkit [19335] by mjs
  • 2 edits in trunk/WebCore

Fix by Darin, reviewed by Maciej.

  • fix ~10% performance regression from encoding fix
  • loader/TextResourceDecoder.cpp: (WebCore::TextResourceDecoder::checkForHeadCharset): Fix a logic error which made huge strings accidentally.
Note: See TracTimeline for information about the timeline view.