Timeline



Aug 2, 2007:

8:17 PM Changeset in webkit [24835] by oliver
  • 6 edits in trunk/WebCore

Reviewed by Geoff

Fix for <rdar://problem/5369332> Xcode crashes while selecting a hyperlink within a AppleScript dictionary (WebCore::Font::drawGlyphBuffer)

There were many places where we were not correctly retaining/releasing the
NSFont object stored in the C++ PlatformFontData object, this resulted in
the GC incorrectly collecting the NSFont.

This patch fixes the problem by prevent direct modification of the PlatformFontData
font pointer, allowing us to enforce correct CFRetain/Release behaviour.

  • platform/FontData.h: (WebCore::FontData::getNSFont):
  • platform/mac/FontCacheMac.mm: (WebCore::FontCache::getFontDataForCharacters): (WebCore::FontCache::createFontPlatformData):
  • platform/mac/FontDataMac.mm: (WebCore::initFontData): (WebCore::FontData::platformInit): (WebCore::FontData::platformDestroy): (WebCore::FontData::smallCapsFontData): (WebCore::FontData::containsCharacters): (WebCore::FontData::determinePitch): (WebCore::FontData::platformWidthForGlyph): (WebCore::FontData::checkShapesArabic):
  • platform/mac/FontMac.mm: (WebCore::initializeATSUStyle): (WebCore::overrideLayoutOperation): (WebCore::Font::drawGlyphs):
  • platform/mac/FontPlatformData.h: (WebCore::FontPlatformData::FontPlatformData): (WebCore::FontPlatformData::~FontPlatformData): (WebCore::FontPlatformData::hash): (WebCore::FontPlatformData::operator==): (WebCore::FontPlatformData::font): (WebCore::FontPlatformData::setFont):
5:24 PM Changeset in webkit [24834] by weinig
  • 3 edits in trunk/LayoutTests

Reviewed by Kevin McCullough.

Change test to use shared shouldBe function to
show failures more clearly.

  • fast/dom/Window/window-resize-expected.txt:
  • fast/dom/Window/window-resize.html:
4:54 PM Changeset in webkit [24833] by antti
  • 2 edits in trunk/WebCore

Oops, this change wasn't supposed to be commited.

  • page/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge setBaseBackgroundColor:]):
4:45 PM Changeset in webkit [24832] by antti
  • 4 edits in trunk/WebCore

Reviewed by Darin.

<rdar://problem/5355951>
plainText() fragments TCMalloc heap badly on large pages


also likely fixes some cases of
<rdar://problem/5335382>
CrashTracer: [REGRESSION] 73 crashes in Safari at com.apple.WebCore: WebCore::DeprecatedStringData::increaseUnicodeSize + 52


If you load http://dscoder.com/test.txt with WebKit build with TCMalloc and system malloc you see that
Safari RPRVT with TCMalloc is 118.8MB
Safari RPRVT with system malloc is 69.7MB


Difference is almost entirely caused by heap fragmentation from a full document plainText() call (for indexing purposes).


The patch helps in two ways:

  • construct plainText string in pieces to avoid O(n2) reallocs
  • allocate buffers using system malloc so they can be returned back to OS and don't fragment and grow TCMalloc heap


This shrinks http://dscoder.com/test.txt RPRVT to 79.0MB and makes full document plainText() take 50ms instead of 500ms.
The benefits are not limited to extreme cases, web pages above ~200kB can show substantial improvement in RPRVT.

  • editing/TextIterator.cpp: (WebCore::plainTextToMallocAllocatedBuffer): (WebCore::plainText):
  • editing/TextIterator.h:
  • page/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge selectedString]): (-[WebCoreFrameBridge stringForRange:]):
3:50 PM Changeset in webkit [24831] by hyatt
  • 3 edits in trunk/WebCore

Fix for 5374437, allow comment nodes to be the child of a document.
Refine the check to always make a root element to check documentElement()
rather than firstChild(), since a comment node could be present as the
firstChild() now.

Reviewed by Tim Hatcher

  • html/HTMLDocument.cpp: (WebCore::HTMLDocument::childAllowed):
  • html/HTMLParser.cpp: (WebCore::HTMLParser::finished):
3:39 PM Changeset in webkit [24830] by kmccullo
  • 2 edits in trunk/WebKitTools

Reviewed by Tim.

  • It would help if I actually called the right function.
  • Drosera/DebuggerDocument.cpp: (DebuggerDocument::showConsole):
3:00 PM Changeset in webkit [24829] by antti
  • 6 edits
    5 adds in trunk

LayoutTests:

Reviewed by Darin.


<rdar://problem/5228138>
REGRESSION(Leopard): test failures: tests that test Georgian numbering


Correct counter test results for Georgian numbers.
Add leopard specific versions needed due to font changes (actual characters instead of boxes).

  • css2.1/t1202-counter-09-b-expected.txt:
  • css2.1/t1202-counters-09-b-expected.txt:
  • mac/leopard/Skipped:
  • mac/leopard/css2.1: Added.
  • mac/leopard/css2.1/t1202-counter-09-b-expected.txt: Added.
  • mac/leopard/css2.1/t1202-counters-09-b-expected.txt: Added.
  • mac/leopard/fast/lists: Added.
  • mac/leopard/fast/lists/w3-list-styles-expected.txt: Added.

WebCore:

Reviewed by Darin.


<rdar://problem/5228138>
REGRESSION(Leopard): test failures: tests that test Georgian numbering

Fix to Georgian number tables to get CSS2.1 test results right. Font has relevant characters now
which revealed that results were actually wrong.

  • rendering/RenderListMarker.cpp: (WebCore::toGeorgian):
2:41 PM Changeset in webkit [24828] by antti
  • 6 edits in trunk/LayoutTests

Reviewed by Geoff.


Fix <rdar://problem/5228233>
REGRESSION(Leopard): layout test failures: fast/table/absolute-table-at-bottom


Remove an arrow symbol unrelated to what is being tested. It has different metrics on Leopard.

  • fast/table/absolute-table-at-bottom-expected.checksum:
  • fast/table/absolute-table-at-bottom-expected.png:
  • fast/table/absolute-table-at-bottom-expected.txt:
  • fast/table/absolute-table-at-bottom.html:
  • mac/leopard/Skipped:
1:08 PM Changeset in webkit [24827] by beidson
  • 2 edits in trunk/WebKit

Reviewed by Tim

<rdar://problem/5381463> - setMIMETypesShownAsHTML mutates while enumerating

  • WebView/WebView.mm: (+[WebView setMIMETypesShownAsHTML:]): Copy the dictionary before we work with it.
11:24 AM Changeset in webkit [24826] by adachan
  • 8 edits in trunk

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

Reviewed by Steve.

WebCore:

<rdar://problem/5079175> Added parameters headerHeight and footerHeight to
computePageRectsForFrame() so we can account for the header and footer when
calculating page heights for this frame.

  • bridge/win/FrameWin.cpp: (WebCore::computePageRectsForFrame):
  • bridge/win/FrameWin.h:


WebKit/win:

<rdar://problem/5079175> Printing header and footer

  • Interfaces/IWebUIDelegate.idl: added methods for header/footer drawing.
  • WebFrame.cpp: (WebFrame::headerAndFooterHeights): ask client for the header and footer heights via IWebUIDelegate2 methods. (WebFrame::computePageRects): pass in header and footer heights when calculating page rect heights. (WebFrame::spoolPages): ask client to draw header and footer via IWebUIDelegate2 methods.
  • WebFrame.h:
  • WebKitGraphics.cpp: (DrawTextAtPoint): the code assumes color has 4 components - might as well assert it.
7:08 AM Changeset in webkit [24825] by yongjzha
  • 2 edits in S60/branches/3.1m/MemoryManager

2007-08-01 vmalaiya, <vikram.malaiya@nokia.com>

Reviewed by Yongjun
DESC: - Added error check in case an attempt to commit memory fails.
TSW Id - SKAR-74YPNV
http://bugs.webkit.org/show_bug.cgi?id=14844

  • Src/fast_malloc.cpp: (chunkMoreCore): (symbian_mmap):
7:02 AM Changeset in webkit [24824] by yongjzha
  • 2 edits in S60/trunk/MemoryManager

2007-08-01 vmalaiya, <vikram.malaiya@nokia.com>

Reviewed by Yongjun
DESC: - Added error check in case an attempt to commit memory fails.
TSW Id - SKAR-74YPNV
http://bugs.webkit.org/show_bug.cgi?id=14844

  • Src/fast_malloc.cpp: (chunkMoreCore): (symbian_mmap):
4:56 AM Changeset in webkit [24823] by staikos
  • 4 edits in trunk/WebKitQt

Add an interface for the useragent string

3:01 AM Changeset in webkit [24822] by bdash
  • 2 edits in trunk/LayoutTests

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

Layout test fix, rubber-stamped by Oliver.

  • fast/js/resources/string-concatenate-outofmemory.js: Don't let an out of memory exception during verification cause a failure. The exception is a perfectly valid success case as it shows that access the string did not cause a crash.
2:49 AM Changeset in webkit [24821] by bdash
  • 2 edits in trunk/JavaScriptCore

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

Build fix.

  • kjs/ustring.cpp: (KJS::UString::expandedSize): Use std::numeric_limits<size_t>::max() rather than the non-portable SIZE_T_MAX.
2:33 AM Changeset in webkit [24820] by bdash
  • 7 edits in trunk

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

Reviewed by Maciej.

<rdar://problem/5352887> "Out of memory" error during repeated JS string concatenation leaks hundreds of MBs of RAM

A call to fastRealloc was failing which lead to UString::expandCapacity leaking the buffer it was trying to reallocate.
It also resulted in the underlying UString::rep having both a null baseString and buf field, which meant that attempting
to access the contents of the string after the failed memory reallocation would crash.

A third issue is that expandedSize size was calculating the new length in a way that led to an integer overflow occurring.
Attempting to allocate a string more than 190,000,000 characters long would fail a the integer overflow would lead to a
memory allocation of around 3.6GB being attempted rather than the expected 390MB. Sizes that would lead to an overflow
are now returned as zero and callers are updated to treat this as though the memory allocation has failed.

  • kjs/array_object.cpp: (ArrayProtoFunc::callAsFunction): Check whether the append failed and raise an "Out of memory" exception if it did.
  • kjs/ustring.cpp: (KJS::allocChars): Wrapper around fastMalloc that takes a length in characters. It will return 0 when asked to allocate a zero-length buffer. (KJS::reallocChars): Wrapper around fastRealloc that takes a length in characters. It will return 0 when asked to allocate a zero-length buffer. (KJS::UString::expandedSize): Split the size calculation in two and guard against overflow during each step. (KJS::UString::expandCapacity): Don't leak r->buf if reallocation fails. Instead free the memory and use the null representation. (KJS::UString::expandPreCapacity): If fastMalloc fails then use the null representation rather than crashing in memcpy. (KJS::UString::UString): If calls to expandCapacity, expandPreCapacity or fastMalloc fail then use the null representation rather than crashing in memcpy. (KJS::UString::append): Ditto. (KJS::UString::operator=): Ditto.
  • kjs/ustring.h: Change return type of expandedSize from int to size_t.

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

Reviewed by Maciej.

<rdar://problem/5352887> "Out of memory" error during repeated JS string concatenation leaks hundreds of MBs of RAM

Update test to check that accessing the string after the "Out of memory" exception was raised does not crash.

  • fast/js/resources/string-concatenate-outofmemory.js:
  • fast/js/string-concatenate-outofmemory-expected.txt:
12:24 AM Changeset in webkit [24819] by aliceli1
  • 1 edit in trunk/WebKit/ChangeLog

forgot to add info about the bug being fixed

12:13 AM Changeset in webkit [24818] by aliceli1
  • 4 edits
    1 add in trunk

WebCore:

Reviewed by Kevin McCullough.

fixed <rdar://problem/5310312> REGRESSION: javascript is mis-escaped at http://labs.zarate.org/passwd causing bookmarklet to break


  • WebCore.exp: expose some calls for WebKit to call.
  • manual-tests/JavaScript-bookmarklets.html: Added.

WebKit:

Reviewed by Kevin McCullough.

  • Misc/WebNSURLExtras.mm: (+[NSURL _web_URLWithUserTypedString:relativeToURL:]): (-[NSURL _web_userVisibleString]): (-[NSURL _web_URLWithLowercasedScheme]): (-[NSURL _web_dataForURLComponentType:]): These 4 changes are just casting changes.

(-[NSString _webkit_stringByReplacingValidPercentEscapes]):
This change replaces the call to an NSURL method with a webcore one that doesn't abort the escaping effort once an illegal character is encountered.

12:01 AM Changeset in webkit [24817] by aroben
  • 2 edits in trunk/WebKitTools

Don't delete the stderr file right after creating it

Reviewed by NOBODY.

  • Scripts/run-webkit-tests:

Aug 1, 2007:

11:24 PM Changeset in webkit [24816] by aroben
  • 2 edits in trunk/WebKitTools

When DRT crashes, record stderr and restart DRT

This prevents a DRT crash from causing the next few hundred tests to
"fail" because DRT is no longer running.

I also changed the terminology that run-webkit-tests uses in its
output a bit, so that crashing tests are referred to as "crashes"
instead of "failures".

Reviewed by Mark.

  • Scripts/run-webkit-tests: Detect a crash and record it as a tool failure. (sub openDumpTool): Use open3 so that we can access stderr. (sub dumpToolCrashed): Added. (sub printFailureMessageForTest): Added. (sub htmlForExpectedAndActualResults): Added. (sub deleteExpectedAndActualResults): Added. (sub recordActualResultsAndDiff): Added.
9:21 PM Changeset in webkit [24815] by treat
  • 5 edits
    2 adds in trunk

Add an interface to manage global history for clients

7:24 PM Changeset in webkit [24814] by treat
  • 3 edits in trunk/WebCore

Do not call update or repaint from inside a paint event.

6:43 PM Changeset in webkit [24813] by andersca
  • 2 edits in trunk/WebKit

Fix build.


  • Misc/WebNSURLExtras.mm: (+[NSURL _web_URLWithUserTypedString:relativeToURL:]): (-[NSURL _web_userVisibleString]): (-[NSURL _web_URLWithLowercasedScheme]): (-[NSURL _web_dataForURLComponentType:]):
6:18 PM Changeset in webkit [24812] by aliceli1
  • 2 edits
    1 move in trunk/WebKit

Reviewed by .

Making WebNSURLExtras objc++

  • Misc/WebNSURLExtras.m: Removed.
  • Misc/WebNSURLExtras.mm: Copied from WebKit/Misc/WebNSURLExtras.m.
  • WebKit.xcodeproj/project.pbxproj:
6:15 PM Changeset in webkit [24811] by darin
  • 3 edits in trunk/WebKit

Reviewed by Anders Carlsson and Kevin Decker.

  • fix <rdar://problem/5377432> Removal of MakeDataExecutable from 64-bit breaks WebKit build

The trick was to ifdef out more of the code that's only needed to support CFM, which exists
only for 32-bit PowerPC.

  • Plugins/WebNetscapePluginPackage.h: Define a SUPPORT_CFM symbol in this internal header when we support CFM. We support it only on 32-bit PowerPC. Only define the isBundle, isCFM, and connID fields when SUPPORT_CFM is on. Also use ResFileRefNum instead of SInt16.
  • Plugins/WebNetscapePluginPackage.m: Only compile the function pointer and transition vector functions when SUPPORT_CFM is on. (-[WebNetscapePluginPackage openResourceFile]): Put the non-bundle case inside a SUPPORT_CFM ifdef, since all non-CFM plug-ins are bundles. (-[WebNetscapePluginPackage closeResourceFile:]): Ditto. (-[WebNetscapePluginPackage _initWithPath:]): Use SUPPORT_CFM to compile out the code for non-bundle and bundle-based CFM plug-ins, and code that sets isBundle and isCFM. (-[WebNetscapePluginPackage executableType]): Put the CFM case inside SUPPORT_CFM. (-[WebNetscapePluginPackage load]): Put the non-bundle and CFM cases inside SUPPORT_CFM. There was a bit of dead code here. (-[WebNetscapePluginPackage _unloadWithShutdown:]): Put the non-bundle case inside SUPPORT_CFM.
5:35 PM Changeset in webkit [24810] by thatcher
  • 3 edits in trunk/WebCore

Reviewed by Justin.

<rdar://problem/5376156> Mail crash in DeleteButtonController::hide() when dropping selected image on DIV's border

Add the container element back so the selection can not touch the deletion UI nodes. The container
has style to prevent user selection, user drag and user modification.

  • editing/DeleteButtonController.cpp: (WebCore::DeleteButtonController::show): Make the container node, and append the button and outline elements. (WebCore::DeleteButtonController::hide): Remove the container elements and null out the other nodes.
  • editing/DeleteButtonController.h:
5:30 PM Changeset in webkit [24809] by darin
  • 4 edits in trunk/JavaScriptCore

Reviewed by Kevin McCullough.

  • fix <rdar://problem/5375186> pointers to pieces of class definition passed to JSClassCreate should all be const
  • API/JSObjectRef.h: Added const.
  • API/JSClassRef.cpp: (OpaqueJSClass::OpaqueJSClass): Added const. (OpaqueJSClass::create): Added const.
  • API/JSObjectRef.cpp: (JSClassCreate): Added const.
2:14 PM Changeset in webkit [24808] by sfalken
  • 8 edits in trunk

Build mod: Fix sln to match configs in vcproj.


Reviewed by Adam.

12:52 PM Changeset in webkit [24807] by zbujtas
  • 6 edits in S60/branches/3.1m

2007-07-31 yadavall <sriram.yadavalli@nokia.com>

Reviewed by Zalan Bujtas (zbujtas@gmail.com).
DESC: Fixes for Menu API

12:47 PM Changeset in webkit [24806] by zbujtas
  • 2 edits in S60/trunk/WebCore

rathnasa, reviewed by zbujtas

DESC: TSW Id: ELTA-756J4Z When user focused the different links, the other sections are not hidden.
http://bugs.webkit.org/show_bug.cgi?id=14814

12:47 PM BuildingGdk edited by treat@kde.org
The two linux ports do not share the same dependencies. (diff)
12:45 PM Changeset in webkit [24805] by zbujtas
  • 2 edits in S60/branches/3.1m/WebCore

2007-07-30 rathnasa <sornalatha.rathnasamy@nokia.com>

Reviewed by zbujtas.
DESC: TSW Id: ELTA-756J4Z When user focused the different links, the other sections are not hidden.
http://bugs.webkit.org/show_bug.cgi?id=14814

12:43 PM BuildingOnLinux edited by treat@kde.org
Leave the buidling instructions for the pages specifically intended … (diff)
12:05 PM WikiStart edited by treat@kde.org
Go directly to the port's specific content page. We are talking about … (diff)
11:54 AM BuildingGdk edited by alp@atoker.com
Port is now maintained (diff)
9:53 AM Changeset in webkit [24804] by yongjzha
  • 2 edits in S60/trunk/WebCore

w3liu, reviewed by <yongjun.zhang@nokia.com>

DESC: FPER-74HDN2: Strange character are shown instead of Chinese on www.sina.com.cn
http://bugs.webkit.org/show_bug.cgi?id=14830

WARNING: NO TEST CASES ADDED OR CHANGED

  • khtml/khtml_part.cpp: (KHTMLPart::write):
7:22 AM BuildingOnLinux edited by samuel.verstraete@gmail.com
just fixed some "typo's" (diff)
7:16 AM BuildingOnLinux edited by Simon Hausmann
(diff)

Jul 31, 2007:

10:42 PM BuildingOnLinux edited by vprajan@gmail.com
(diff)
10:40 PM BuildingOnLinux edited by vprajan@gmail.com
(diff)
10:40 PM WikiStart edited by vprajan@gmail.com
(diff)
10:39 PM BuildingOnLinux edited by vprajan@gmail.com
(diff)
9:30 PM Changeset in webkit [24803] by oliver
  • 2 edits in trunk/LayoutTests

fast/encoding/char-encoding.html no longer needs to be in the Leopard skiplist

  • mac/leopard/Skipped:
5:40 PM Changeset in webkit [24802] by harrison
  • 2 edits in trunk/WebCore

Reviewed by Justin.

<rdar://problem/5362659> CrashTracer: [USER] 11 crashes in Mail at WebCore::InsertLineBreakCommand::doApply()

The problem was that deleting with the X control leaves the selection inside the fragment that was deleted.

  • editing/DeleteButtonController.cpp: (WebCore::DeleteButtonController::deleteTarget): Because the deletion UI only appears when the selection is entirely within the target, we unconditionally update the selection to be a caret where the target had been.
5:36 PM Changeset in webkit [24801] by adele
  • 17 edits
    4 adds
    4 deletes in trunk

LayoutTests:

Reviewed by Hyatt.

Updated tests for <rdar://problem/5339395> REGRESSION:http://sudokucraving.com does not render grid correctly

  • fast/forms/input-appearance-minWidth.html: Removed. This test is covered by fast/replaced/width100percent-textfield.html.
  • fast/forms/input-appearance-minWidth-expected.checksum: Removed.
  • fast/forms/input-appearance-minWidth-expected.png: Removed.
  • fast/forms/input-appearance-minWidth-expected.txt: Removed.
  • fast/forms/input-table-expected.checksum:
  • fast/forms/input-table-expected.png:
  • fast/forms/input-table-expected.txt:
  • fast/replaced/width100percent-menulist-expected.checksum:
  • fast/replaced/width100percent-menulist-expected.png:
  • fast/replaced/width100percent-menulist-expected.txt:
  • fast/replaced/width100percent-searchfield-expected.checksum: Added.
  • fast/replaced/width100percent-searchfield-expected.png: Added.
  • fast/replaced/width100percent-searchfield-expected.txt: Added.
  • fast/replaced/width100percent-searchfield.html: Added.
  • fast/replaced/width100percent-textarea-expected.checksum:
  • fast/replaced/width100percent-textarea-expected.png:
  • fast/replaced/width100percent-textarea-expected.txt:
  • fast/replaced/width100percent-textfield-expected.checksum:
  • fast/replaced/width100percent-textfield-expected.png:
  • fast/replaced/width100percent-textfield-expected.txt:

WebCore:

Reviewed by Hyatt.

Fix for <rdar://problem/5339395> REGRESSION:http://sudokucraving.com does not render grid correctly

  • rendering/RenderTextControl.h: (WebCore::RenderTextControl::hasControlClip): Clip for search fields.
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::controlClipRect): Added a control clip so the search field's cancel button and magnifier glass never draw outside the control's bounds. (WebCore::RenderTextControl::calcPrefWidths): Only include the inner box's padding when calculating the min/max width without using calcContentBoxWidth. Our old behavior was causing that inner padding to get counted twice. Also, no need to add in the border for an inner box that can't be controlled from outside this class.
  • rendering/RenderMenuList.cpp: (WebCore::RenderMenuList::calcPrefWidths): ditto.
5:20 PM Changeset in webkit [24800] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Geoff.

Speculative fix for <rdar://problem/5359695>
REGRESSION (Tiger Beta): Multiple crashes in WebCore::Widget::getView() const + 6


  • page/EventHandler.cpp: (WebCore::EventHandler::updateDragAndDrop): Null check the frame view.
2:17 PM Changeset in webkit [24799] by thatcher
  • 7 edits in trunk

WebCore:

Reviewed by Oliver and Beth.

<rdar://problem/5211271> ADOBE Leopard 9A410: At the first Launching InDesign after deactivate, EULA page gets blanked.

Rename needsAcrobatFrameReloadingQuirk to needsAdobeFrameReloadingQuirk, since this now applies to more Adobe applications.

  • WebCore.exp:
  • page/Settings.cpp: (WebCore::Settings::Settings): (WebCore::Settings::setNeedsAdobeFrameReloadingQuirk):
  • page/Settings.h: (WebCore::Settings::needsAcrobatFrameReloadingQuirk):

WebKit:

Reviewed by Oliver and Beth.

<rdar://problem/5211271> ADOBE Leopard 9A410: At the first Launching InDesign after deactivate, EULA page gets blanked.

Check for more Adobe applications that need the frame reload quirk. Also cache the answer
so the version check dosen't happen more than once.

  • WebView/WebView.mm: (-[WebView _needsAdobeFrameReloadingQuirk]): (-[WebView _updateWebCoreSettingsFromPreferences:]):
1:29 PM Changeset in webkit [24798] by beidson
  • 7 edits
    2 adds in trunk

LayoutTests:

Reviewed by Brady

Test for http://bugs.webkit.org/show_bug.cgi?id=14757
HTMLTokenizer::processingData implementation is incorrect

  • http/tests/loading/slow-parsing-subframe-expected.txt: Added.
  • http/tests/loading/slow-parsing-subframe.html: Added.

WebCore:

Reviewed by Brady and Darin, tweaked by Brady, landed by Brady

Fix for http://bugs.webkit.org/show_bug.cgi?id=14757 and <rdar://problem/5364692>
HTMLTokenizer::processingData implementation is incorrect

  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::processingData): Made it also return true if the HTMLTokenizer was inside the write() call.
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::FrameLoader): (WebCore::FrameLoader::checkLoadCompleteTimerFired): (WebCore::FrameLoader::scheduleCheckLoadComplete): (WebCore::FrameLoader::stopForUserCancel): Changed a call to checkLoadComplete to be asynchronous, since stopForUserCancel can be called while parsing.
  • loader/FrameLoader.h:
12:54 PM Changeset in webkit [24797] by adele
  • 2 edits in trunk/WebKit

Reviewed by Oliver.

Fix for <rdar://problem/5308020> REGRESSION: Command-N with Dvorak-Qwerty keyboard layout stopped working inside web page text fields

  • WebView/WebHTMLView.mm: (-[WebHTMLView _handleStyleKeyEquivalent:]): The input method may have modified the character we get, so don't use charactersIgnoringModifiers to interpret the character we get.
12:08 PM Changeset in webkit [24796] by andersca
  • 3 edits in trunk/WebCore

Reviewed by Geoff.

<rdar://problem/5371582>
REGRESSION: PLT .5% slower due to r24451 (copying HTMLCollection objects)


Make the hash maps store CollectionInfo pointers to reduce amount of copying when
inserting/rehashing etc.


  • dom/Document.cpp: (WebCore::Document::~Document): (WebCore::Document::nameCollectionInfo):
  • dom/Document.h:
11:38 AM Changeset in webkit [24795] by adele
  • 4 edits in trunk/LayoutTests

Reviewed by Beth.

Updated this test to force display to happen earlier. This works around a bug in DRT that draws the scrollbars in a different way from Safari.

  • fast/replaced/width100percent-textarea-expected.checksum:
  • fast/replaced/width100percent-textarea-expected.png:
  • fast/replaced/width100percent-textarea.html:
11:27 AM Changeset in webkit [24794] by adele
  • 1 edit
    28 adds
    4 deletes in trunk/LayoutTests

Reviewed by Sam.

Breaking up fast/replaced/width100percent.html into separate tests for the different elements.

  • fast/replaced/width100percent-button-expected.checksum: Added.
  • fast/replaced/width100percent-button-expected.png: Added.
  • fast/replaced/width100percent-button-expected.txt: Added.
  • fast/replaced/width100percent-button.html: Added.
  • fast/replaced/width100percent-checkbox-expected.checksum: Added.
  • fast/replaced/width100percent-checkbox-expected.png: Added.
  • fast/replaced/width100percent-checkbox-expected.txt: Added.
  • fast/replaced/width100percent-checkbox.html: Added.
  • fast/replaced/width100percent-image-expected.checksum: Added.
  • fast/replaced/width100percent-image-expected.png: Added.
  • fast/replaced/width100percent-image-expected.txt: Added.
  • fast/replaced/width100percent-image.html: Added.
  • fast/replaced/width100percent-menulist-expected.checksum: Added.
  • fast/replaced/width100percent-menulist-expected.png: Added.
  • fast/replaced/width100percent-menulist-expected.txt: Added.
  • fast/replaced/width100percent-menulist.html: Added.
  • fast/replaced/width100percent-radio-expected.checksum: Added.
  • fast/replaced/width100percent-radio-expected.png: Added.
  • fast/replaced/width100percent-radio-expected.txt: Added.
  • fast/replaced/width100percent-radio.html: Added.
  • fast/replaced/width100percent-textarea-expected.checksum: Added.
  • fast/replaced/width100percent-textarea-expected.png: Added.
  • fast/replaced/width100percent-textarea-expected.txt: Added.
  • fast/replaced/width100percent-textarea.html: Added.
  • fast/replaced/width100percent-textfield-expected.checksum: Added.
  • fast/replaced/width100percent-textfield-expected.png: Added.
  • fast/replaced/width100percent-textfield-expected.txt: Added.
  • fast/replaced/width100percent-textfield.html: Added.
  • fast/replaced/width100percent.html: Removed.
  • fast/replaced/width100percent-expected.checksum: Removed.
  • fast/replaced/width100percent-expected.png: Removed.
  • fast/replaced/width100percent-expected.txt: Removed.
10:58 AM Changeset in webkit [24793] by weinig
  • 11 edits in trunk

LayoutTests:

Reviewed by Mitz.

Updated tests for http://bugs.webkit.org/show_bug.cgi?id=14825
Non-integer hsl() colours are ignored

  • fast/css/hsl-color-expected.checksum:
  • fast/css/hsl-color-expected.png:
  • fast/css/hsl-color-expected.txt:
  • fast/css/hsl-color.html: Add non-integer hue tests
  • fast/css/hsla-color-expected.checksum:
  • fast/css/hsla-color-expected.png:
  • fast/css/hsla-color-expected.txt:
  • fast/css/hsla-color.html: Add non-integer hue tests

WebCore:

Reviewed by Mitz.

Fix for http://bugs.webkit.org/show_bug.cgi?id=14825
Non-integer hsl() colours are ignored

  • css/CSSParser.cpp: (WebCore::CSSParser::parseHSLParameters): Parse hue as a Number, not an Integer.
1:32 AM BuildingQtOnWindows edited by Simon Hausmann
(diff)

Jul 30, 2007:

8:54 PM Changeset in webkit [24792] by justing
  • 6 edits
    4 adds in trunk

LayoutTests:

Reviewed by Darin.


<rdar://problem/5369009>
Crash due to infinite recursion in moveParagraphs deleting line from a Mail message

  • editing/deleting/5369009-expected.checksum: Added.
  • editing/deleting/5369009-expected.png: Added.
  • editing/deleting/5369009-expected.txt: Added.
  • editing/deleting/5369009.html: Added.

WebCore:

Reviewed by Darin.

<rdar://problem/5369009> Crash due to infinite recursion in moveParagraphs on delete


After the delete, we moved content into the previous block, and a style
rule turned a style span in the moved content into a block, throwing
moveParagraphs into infinite recursion, as it continually tried and failed
to get the style span into the same paragraph as the content just before it.


Added a method to ReplaceSelectionCommand to keep inserted style spans
from turning into blocks because of style rules. Will add code to prevent
other kinds of content from changing appearance because of style rules,
post-Leopard (5371536). Also added a bool to ReplaceSelectionCommand's
constructor to guard against infinite recursion.

  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::moveParagraphs):
  • editing/JSEditor.cpp:
  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::ReplaceSelectionCommand): (WebCore::ReplaceSelectionCommand::negateStyleRulesThatEffectAppearance): (WebCore::ReplaceSelectionCommand::doApply):
  • editing/ReplaceSelectionCommand.h:
8:44 PM Changeset in webkit [24791] by treat
  • 3 edits in trunk/WebKitQt

Fix build.

8:18 PM Changeset in webkit [24790] by sullivan
  • 3 edits in trunk/WebKit

Reviewed by Darin


  • fixed <rdar://problem/5216176> Need WebKit SPI to allow clients using embedded WebViews to avoid clipping ends of some printed pages


This provides Mail, and other clients that print views that embed WebViews, a way to ensure that
the HTML is laid out for printing before pagination occurs.

  • WebView/WebHTMLViewPrivate.h: new SPI method -_layoutForPrinting


  • WebView/WebHTMLView.mm: (-[WebHTMLView _web_setPrintingModeRecursiveAndAdjustViewSize]): new method, just like existing _web_setPrintingModeRecursive except passes YES for adjustViewSize (-[WebHTMLView _layoutForPrinting]): new SPI method, sets printing mode temporarily to adjust the view size for printing (-[NSView _web_setPrintingModeRecursiveAndAdjustViewSize]): new helper method to do the recursion
5:48 PM Changeset in webkit [24789] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Brady.

<rdar://problem/5371515>
BitmapImage::getTIFFRepresentation() bug results in favicons not working at many sites

Don't bail out if a single frame is invalid.


  • platform/graphics/mac/ImageMac.mm: (WebCore::BitmapImage::getTIFFRepresentation):
5:29 PM Changeset in webkit [24788] by darin
  • 1 edit in trunk/WebCore/ChangeLog-2006-05-10

Fix a bug number on an old check-in.

5:09 PM Changeset in webkit [24787] by adele
  • 2 edits in trunk/WebKit

Reviewed by Darin.

Fix for <rdar://problem/5367919> A crash occurs at WebCore::Frame::isFrameSet() when attempting to print a iframe before it loads at http://www.monster.com/

  • WebView/WebHTMLView.mm: (-[WebHTMLView knowsPageRange:]): Nil check for frame.
4:56 PM Changeset in webkit [24786] by adele
  • 2 edits in trunk/WebCore

Reviewed by Darin.

Fix for <rdar://problem/5274937> Safari 3 breaks Missing Sync's WebKit code by crashing

  • loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::continueAfterContentPolicy): Nil check for the FrameLoader.
4:24 PM Changeset in webkit [24785] by andersca
  • 6 edits in trunk

WebCore:

Reviewed by Darin.

<rdar://problem/5370710>
REGRESSION: After switching from Bookmark view, the Find Banner won't appear while displaying a PDF file

Add a hasHTMLView to FrameLoaderClient. This is only useful for clients who can show different views for different
MIME types and URL schemes.


  • loader/FrameLoader.cpp: (WebCore::FrameLoader::transitionToCommitted): Call delegate methods here if the current view is not a HTML view.


  • loader/FrameLoaderClient.h: (WebCore::FrameLoaderClient::hasHTMLView):

WebKit:

Reviewed by Darin.

<rdar://problem/5370710>
REGRESSION: After switching from Bookmark view, the Find Banner won't appear while displaying a PDF file

Implement hasHTMLView.


  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::hasHTMLView):
4:05 PM Changeset in webkit [24784] by darin
  • 1 edit in trunk/WebKitTools/ChangeLog

Hatcher!

4:05 PM Changeset in webkit [24783] by darin
  • 2 edits in trunk/WebKitTools

Reviewed by Tim Matcher.

  • DumpRenderTree/DumpRenderTree.m: (dump): Fix dumping for documents that include null characters. This turned out not to be needed for the test case that motivated me to do it, but it's nice to have this for the future.
3:29 PM Changeset in webkit [24782] by justing
  • 1 edit in trunk/WebKit/WebView/WebEditingDelegatePrivate.h

Fixed a typo in my last checkin.

11:29 AM Changeset in webkit [24781] by weinig
  • 14 edits
    2 adds
    4 deletes in trunk

LayoutTests:

Reviewed by Maciej.

Update tests for <rdar://problem/5363896>
REGRESSION: Setting document.domain does not override port-based cross-frame security checks -- breaks SAP NetWeaver

  • Also changes frame access test to use shared resource iframes.
  • http/tests/security/cross-frame-access-port-expected.txt:
  • http/tests/security/cross-frame-access-port-explicit-domain-expected.txt:
  • http/tests/security/cross-frame-access-port-explicit-domain.html:
  • http/tests/security/cross-frame-access-port.html:
  • http/tests/security/cross-frame-access-protocol-expected.txt:
  • http/tests/security/cross-frame-access-protocol-explicit-domain-expected.txt:
  • http/tests/security/cross-frame-access-protocol-explicit-domain.html:
  • http/tests/security/cross-frame-access-protocol.html:
  • http/tests/security/resources/cross-frame-iframe-for-port-explicit-domain-test.html: Removed.
  • http/tests/security/resources/cross-frame-iframe-for-port-test.html: Removed.
  • http/tests/security/resources/cross-frame-iframe-for-protocol-explicit-domain-test.html: Removed.
  • http/tests/security/resources/cross-frame-iframe-for-protocol-test.html: Removed.
  • http/tests/security/resources/cross-frame-iframe-with-explicit-domain-set.html: Added.
  • http/tests/security/resources/cross-frame-iframe.html: Added.

WebCore:

Reviewed by Maciej.

Fix for <rdar://problem/5363896>
REGRESSION: Setting document.domain does not override port-based cross-frame security checks -- breaks SAP NetWeaver

Updates the results for:

http/tests/security/cross-frame-access-port-explicit-domain.html
http/tests/security/cross-frame-access-protocol-explicit-domain.html

  • bindings/js/kjs_window.cpp: (KJS::createWindow): Use the new setDomainInternal method. (KJS::Window::isSafeScript): Don't set check the port or protocol if both documents have explicitly set document.domain in the DOM and the those domains are equal.
  • dom/Document.cpp: (WebCore::Document::Document): (WebCore::Document::setDomain): Remove force parameter and change all calls that relied on it use the new setDomainInternal method which does the same thing. (WebCore::Document::setDomainInternal):
  • dom/Document.h: Add new m_domainWasSetInDOM variable and accessor. (WebCore::Document::domainWasSetInDOM):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::checkCallImplicitClose): Use the new setDomainInternal method.
11:25 AM Changeset in webkit [24780] by justing
  • 14 edits in trunk

WebCore:

Reviewed by Tristan.


<rdar://problem/5098931> Attachments are lost when they are moved into a ToDo after a delete


Added an editor client method for asking whether or not it is
OK to merge content after a delete.

  • bridge/EditorClient.h:
  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::mergeParagraphs): Ask if the merge is allowed.
  • platform/gdk/EditorClientGdk.cpp: (WebCore::EditorClientGdk::shouldMoveRangeAfterDelete): Added a method stub.
  • platform/gdk/EditorClientGdk.h:
  • platform/graphics/svg/SVGImageEmptyClients.h: (WebCore::SVGEmptyEditorClient::shouldMoveRangeAfterDelete): Ditto.

WebKit:

Reviewed by Tristan.


<rdar://problem/5098931> Attachments are lost when they are moved into a ToDo after a delete


Mail needs to be asked if it is OK to do the content movement that happens
after a deleting in a situation like this one:
<div contenteditable="plaintext-only">foo</div><div>bar</div>
so that they can prevent the move or so that they can save content that
will be stripped by the move. This could have been done with shouldInsertNode
and a new WebViewInsertAction for "moves", but WebKit clients like Mail and
DashCode think that a shouldInsert* means that the user pasted something and
perform actions only appropriate for pastes. This change is less risky because
it won't require those clients to change their code.

  • DefaultDelegates/WebDefaultEditingDelegate.m: (-[WebDefaultEditingDelegate webView:shouldMoveRangeAfterDelete:replacingRange:]):
  • WebCoreSupport/WebEditorClient.h:
  • WebCoreSupport/WebEditorClient.mm: (WebEditorClient::shouldMoveRangeAfterDelete):
  • WebView/WebEditingDelegatePrivate.h:

win:

Reviewed by Tristan.


<rdar://problem/5098931> Attachments are lost when they are moved into a ToDo after a delete

  • WebEditorClient.cpp: (WebEditorClient::shouldMoveRangeAfterDelete): Method stub.
  • WebEditorClient.h:
11:15 AM Changeset in webkit [24779] by andersca
  • 4 edits
    4 adds in trunk

LayoutTests:

Reviewed by Anders.

<rdar://problem/5352526>
http://bugs.webkit.org/show_bug.cgi?id=14704
REGRESSION: sync XMLHttpRequest.send() raises an exception for failed authentication

  • http/tests/xmlhttprequest/failed-auth-expected.txt: Added.
  • http/tests/xmlhttprequest/failed-auth.html: Added. This test has other cases from bug 13075 commented out, as they still fail.
  • http/tests/xmlhttprequest/xmlhttprequest-missing-file-exception-expected.txt: Added.
  • http/tests/xmlhttprequest/xmlhttprequest-missing-file-exception.html: Added. This test already passed, but I think it's useful to cover this case, too.

WebCore:

Reviewed by Anders.

<rdar://problem/5352526>
http://bugs.webkit.org/show_bug.cgi?id=14704
REGRESSION: sync XMLHttpRequest.send() raises an exception for failed authentication

Test: http/tests/xmlhttprequest/failed-auth.html

  • platform/network/mac/ResourceHandleMac.mm: (WebCore::ResourceHandle::loadResourceSynchronously): Make the fake response better match the real one in case of failed authentication. Unfortunately, NSURLConnection doesn't give us the real response.
  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::send): If the response has an HTTP code in it, then the error wasn't a network one, and an exception shouldn't be raised.
10:48 AM Changeset in webkit [24778] by tristan
  • 1 edit in trunk/LayoutTests/http/tests/navigation/relativeanchor-frames-expected.txt

Updating layout test results with new expected results.

10:28 AM Changeset in webkit [24777] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Darin.

<rdar://problem/4942372> REGRESSION: Anchor links in Mail don't work anymore


Make it possible for KURL to handle addig fragments to non-hierarchical URLs
such as mailto: and cid:.


  • platform/KURL.cpp: (WebCore::KURL::init): If the base URL is not hierarchical but the relative URL is a fragment, then allow parsing it.


(WebCore::KURL::parse):
If the URL is not hierarchical, set the fragment start and end positions correctly.

9:10 AM Changeset in webkit [24776] by hausmann
  • 2 edits in trunk/WebCore

On Windows make sure the dll also ends up in $$OUTPUT_DIR/bin, so that QtLauncher can find it.

9:10 AM Changeset in webkit [24775] by hausmann
  • 4 edits in trunk

Link QtLauncher into $$OUTPUT_DIR/bin

8:30 AM Changeset in webkit [24774] by hausmann
  • 5 edits in trunk

Removed the BUILDING_QT ifdef in JSStringRef.h and changed UChar for the Qt build to use wchar_t on Windows.

8:30 AM Changeset in webkit [24773] by hausmann
  • 2 edits in trunk/WebCore

Don't link against libxml/libxslt unless we really need it.

6:57 AM Changeset in webkit [24772] by zimmermann
  • 72 edits in branches/feature-branch

Reviewed by Oliver & Rob.

Enable SVG textPath selection. Actually build chunks for them instead of
ignoring them - was easier than I thought. Respect per-character transformation
when calculating the chunk boundaries for the selection rectangles. Also fixes
selection rect drawing of rotated characters.

Next thing todo is to unify the glyph size calculations, as per-glyph transformations
are not respected when calculating flow box sizes in SVGRootInlineBox.

6:51 AM BuildingQtOnWindows edited by Simon Hausmann
(diff)
6:46 AM BuildingQtOnWindows edited by Simon Hausmann
(diff)
6:39 AM BuildingQtOnWindows edited by Simon Hausmann
(diff)
6:34 AM BuildingQtOnWindows edited by Simon Hausmann
(diff)
6:31 AM BuildingQtOnWindows edited by Simon Hausmann
(diff)

Jul 29, 2007:

11:33 PM Changeset in webkit [24771] by tristan
  • 3 edits in trunk/WebCore

Reviewed by Oliver Hunt.

<rdar://problem/5369282> REGRESSION: DOMHTMLInput throwing exception due to missing _isEdited method

  • bindings/objc/DOMHTML.mm: (-[DOMHTMLInputElement _isEdited]): (-[DOMHTMLTextAreaElement _isEdited]):
  • bindings/objc/DOMPrivate.h: Renamed _isUserEdited to _isEdited.
9:46 PM Changeset in webkit [24770] by alp
  • 2 edits in trunk/WebKit

2007-07-30 Diego Escalante Urrelo <diegoe@gnome.org>

Reviewed by Adam.

http://bugs.webkit.org/show_bug.cgi?id=14806
Implement can_go_backward and can_go_forward in webkitgtkpage.cpp

  • gtk/Api/webkitgtkpage.cpp: Implement webkit_gtk_page_can_go_backward() and webkit_gtk_page_can_go_forward() functions.
9:07 PM Changeset in webkit [24769] by alp
  • 3 edits in trunk/WebKit

2007-07-30 Diego Escalante Urrelo <diegoe@gnome.org>

Reviewed by Alp Toker.

http://bugs.webkit.org/show_bug.cgi?id=14810
Bad n_params on load_finished's g_signal_new in webkitgtkpage.cpp

This is the proper fix for #14810.

  • gtk/Api/webkitgtkframe.cpp:
  • gtk/Api/webkitgtkpage.cpp: Fix the marshaller type of the "load_finished" signal and correct the class_offset parameters.
8:12 PM Changeset in webkit [24768] by alp
  • 2 edits in trunk/WebKit

2007-07-29 Diego Escalante Urrelo <diegoe@gnome.org>

Reviewed by Alp Toker.

http://bugs.webkit.org/show_bug.cgi?id=14810
Bad n_params on load_finished's g_signal_new in webkitgtkpage.cpp

  • gtk/Api/webkitgtkpage.cpp: Use a correct n_params value and the corresponding types for them in the "load_finished" signal.
7:40 PM Changeset in webkit [24767] by treat
  • 6 edits in trunk

Paint and scrolling performance fixes and improvements.

4:25 PM Changeset in webkit [24766] by adele
  • 6 edits in trunk

WebCore:

Reviewed by John.

WebCore part of fix for <rdar://problem/5102522> REGRESSION: Can't tab to webview that doesn't have editable content (affects Safari, preview pane in Mail)

  • WebCore.exp:
  • page/FocusController.cpp: (WebCore::FocusController::setInitialFocus): Added. (WebCore::FocusController::advanceFocus): Don't give the focus back to the chrome if this is the initial focus.
  • page/FocusController.h:

WebKit:

Reviewed by John.

WebKit part of fix for <rdar://problem/5102522> REGRESSION: Can't tab to webview that doesn't have editable content

  • WebView/WebHTMLView.mm: (-[WebHTMLView becomeFirstResponder]): Call new setInitialFocus method instead of advanceFocus.
2:51 PM Changeset in webkit [24765] by zecke
  • 2 edits in trunk

2007-07-29 Brian Mastenbrook <brian@mastenbrook.net>

Build fix for WebKit/Gtk to compile on Fedora 7 as described in
http://bugs.webkit.org/show_bug.cgi?id=14557

  • WebKit.pri: cairo-ft is used by the WebKit/Gtk port
8:18 AM Changeset in webkit [24764] by treat
  • 2 edits in trunk/WebKitQt

Respect the margins when creating frames.
Set the scroll mode to always off like we did before the rendered
frames patch and the other ports do now.

7:42 AM Changeset in webkit [24763] by treat
  • 2 edits in trunk/WebCore

WebCore::Widget::setParent should be virtual.

7:22 AM Changeset in webkit [24762] by treat
  • 3 edits in trunk/LayoutTests/qt

We actually pass the mac versions expected results here.

Note: See TracTimeline for information about the timeline view.