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

Timeline



Apr 26, 2007:

11:03 PM Changeset in webkit [21137] by bdash
  • 4 edits in trunk

2007-04-27 Mark Rowe <​mrowe@apple.com>

Reviewed by Oliver.

  • JavaScriptCore.xcodeproj/project.pbxproj: Improve dependencies in Xcode project by marking dftables as a dependency of Generate Derived Sources rather than of JavaScriptCore itself.

2007-04-27 Mark Rowe <​mrowe@apple.com>

Reviewed by Oliver.

  • JavaScriptGlue.xcodeproj/project.pbxproj: Improve dependencies in Xcode project by marking JavaScriptGlue as a dependency of testjsglue.
10:48 PM Changeset in webkit [21136] by bdash
  • 2 edits in trunk/WebCore

2007-04-26 Matt Perry <​mpComplete@gmail.com>

Reviewed by Maciej.

Added a version check to hackAroundLibXMLEntityBug, since that bug has
been fixed in recent versions of libxml2.

  • dom/XMLTokenizer.cpp: (WebCore::hackAroundLibXMLEntityBug):
10:36 PM Changeset in webkit [21135] by bdash
  • 1 edit
    7 adds in trunk/WebCore

2007-04-26 Christopher Brichford <​chrisb@adobe.com>

Reviewed by Timothy Hatcher.

  • WebCore.apolloproj/mac/WebCore.Debug.xcconfig: Added.
  • WebCore.apolloproj/mac/WebCore.Release.xcconfig: Added.
  • WebCore.apolloproj/mac/WebCore.xcconfig: Added.
  • WebCore.apolloproj/mac/WebCore.xcodeproj/project.pbxproj: Added.
10:11 PM Changeset in webkit [21134] by bdash
  • 2 edits in trunk/WebCore

2007-04-26 Alp Toker <​alp@atoker.com>

Reviewed by Maciej.

​http://bugs.webkit.org/show_bug.cgi?id=13505
Cast loses precision on x86_64

  • platform/graphics/cairo/GraphicsContextCairo.cpp: (WebCore::GraphicsContext::beginTransparencyLayer): (WebCore::GraphicsContext::endTransparencyLayer): Use a stack of opacity values instead of Cairo user data.
10:09 PM Changeset in webkit [21133] by bdash
  • 2 edits in trunk/WebKitTools

2007-04-26 Alp Toker <​alp@atoker.com>

Reviewed by Mark Rowe.

  • GdkLauncher/main.cpp: (main): Avoid use of deprecated API.
10:00 PM Changeset in webkit [21132] by bdash
  • 3 edits
    4 adds in trunk

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

Reviewed by Darin.

Test: fast/forms/listbox-scrollbar-incremental-load.html

The scroller position was not updated correctly during insertion, since
scrollbar proportions are updated only during layout

  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::selectionChanged): Delay scrolling to reveal the selection until after layout if m_optionsChanged is set. Layout is guaranteed to happen soon when the flag is set.

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

Reviewed by Darin.

  • fast/forms/listbox-scrollbar-incremental-load-expected.checksum: Added.
  • fast/forms/listbox-scrollbar-incremental-load-expected.png: Added.
  • fast/forms/listbox-scrollbar-incremental-load-expected.txt: Added.
  • fast/forms/listbox-scrollbar-incremental-load.html: Added.
6:41 PM Changeset in webkit [21131] by ggaren
  • 5 edits in trunk/JavaScriptCore

Build fix -- added #includes that we used to get implicitly through
JSStringRef.h.

  • API/JSNode.c:
  • API/JSNodeList.c:
  • API/minidom.c:
  • API/testapi.c:
6:15 PM Changeset in webkit [21130] by ggaren
  • 2 edits in trunk/JavaScriptCore

Reviewed by Maciej Stachowiak, Adam Roben.


Fixed
<rdar://problem/4885130> Remove #include of JSStringRefCF.h from JSStringRef.h
<rdar://problem/4885123> JavaScriptCore is not cross-platform -- JSStringRef.h references CF datatypes

  • API/JSStringRef.h: Removed #include -- no clients need it anymore.
5:36 PM Changeset in webkit [21129] by hyatt
  • 1 edit in trunk/LayoutTests/fast/dom/Range/surroundContents-1-expected.txt

Fix layout test results jnow that we no longer make empty render texts.

5:35 PM Changeset in webkit [21128] by hyatt
  • 1 edit in trunk/LayoutTests/fast/dom/delete-contents-expected.txt

Fix layout test results jnow that we no longer make empty render texts.

5:34 PM Changeset in webkit [21127] by hyatt
  • 3 edits in trunk/LayoutTests/editing/execCommand

Fix layout test results jnow that we no longer make empty render texts.

5:33 PM Changeset in webkit [21126] by hyatt
  • 3 edits in trunk/LayoutTests/editing/deleting

Fix layout test results jnow that we no longer make empty render texts.

5:30 PM Changeset in webkit [21125] by hyatt
  • 4 edits in trunk/WebCore

Fix for 13511, crash on a counters page. Make sure that RenderTexts
never get a null StringImpl. I would have added an assert that the length
had to be nonzero also, but unfortunately editing creates empty RenderTexts
because of its EditingText DOM subclass that always forces renderers to be
made. I suppose there is a good reason for this...

Reviewed by aroben

  • dom/CharacterData.cpp: (WebCore::CharacterData::setData): (WebCore::CharacterData::appendData): (WebCore::CharacterData::insertData): (WebCore::CharacterData::deleteData): (WebCore::CharacterData::replaceData):
  • rendering/RenderCounter.cpp: (WebCore::RenderCounter::RenderCounter):
  • rendering/RenderText.cpp: (WebCore::RenderText::RenderText): (WebCore::RenderText::trimmedPrefWidths): (WebCore::RenderText::setTextWithOffset): (WebCore::RenderText::setTextInternal): (WebCore::RenderText::setText):
4:21 PM Changeset in webkit [21124] by aliceli1
  • 2 edits in trunk/WebCore

Reviewed by Oliver.

  • editing/SelectionController.cpp: (WebCore::SelectionController::selectAll): adding null check
2:53 PM Changeset in webkit [21123] by justing
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by oliver


<rdar://problem/5120591> Crash when creating a link in Leopard wiki server

  • editing/execCommand/5120591-expected.checksum: Added.
  • editing/execCommand/5120591-expected.png: Added.
  • editing/execCommand/5120591-expected.txt: Added.
  • editing/execCommand/5120591.html: Added.

WebCore:

Reviewed by oliver

<rdar://problem/5120591>
Crash when creating a link in Leopard wiki server

  • editing/CreateLinkCommand.cpp: (WebCore::CreateLinkCommand::doApply): Use a RefPtr, setting the anchor's href can destroy it if there is an ancestor with a subtree modified event listener.
2:15 PM Changeset in webkit [21122] by ddkilzer
  • 2 edits in trunk/JavaScriptCore

2007-04-25 David Kilzer <​ddkilzer@apple.com>

Reviewed by Maciej.

Add assertions for debug builds.

  • kjs/JSLock.cpp: (KJS::JSLock::lock): Assert the return value of pthread_mutex_lock() in debug builds. (KJS::JSLock::unlock): Assert the return value of pthread_mutex_unlock() in debug builds.
1:37 PM Changeset in webkit [21121] by kmccullo
  • 2 edits in trunk/WebKitTools
  • Changed debugging statements for more information.
  • Scripts/run-webkit-tests:
12:51 PM Changeset in webkit [21120] by hyatt
  • 7 edits in trunk/WebCore

Reviewed by hyatt

No tests added since there is no change to functionality. Several
tests in fast/repaint test repainting of objects that move during layout.

  • page/FrameView.cpp: (WebCore::FrameView::layout):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::layoutBlockChildren): Removed an early setNeedsLayout(false). Resetting the bit here prevents the full repaint that we rely on when we decide not to repaint children that move.
  • rendering/RenderBlock.h:
  • rendering/RenderObject.cpp:
  • rendering/RenderObject.h:
  • rendering/RenderTableSection.cpp: (WebCore::RenderTableSection::setCellWidths): Replaced the call to repaintObjectsBeforeLayout() if the cell didn't need layout with an unconditional repaint(). We don't bother with repainting only the delta between the old and new widths, since by marking the cell for layout results in a full repaint with the new width anyway.
12:24 PM Changeset in webkit [21119] by hyatt
  • 2 edits in trunk/WebCore

Fix for 13509. Don't recur to parent inline flows if they have been
marked as needing layout themselves.

Reviewed by adele

  • rendering/RenderFlow.cpp: (WebCore::RenderFlow::dirtyLinesFromChangedChild):
12:21 PM Changeset in webkit [21118] by ap
  • 3 edits
    3 adds in trunk

2007-04-26 Matt Perry <​mpComplete@gmail.com>

Reviewed and landed by Alexey.

​http://bugs.webkit.org/show_bug.cgi?id=13495
Crash when referencing an undefined entity in an XHTML document

Test: fast/parser/badentity.xhtml

  • dom/XMLTokenizer.cpp: (WebCore::getEntityHandler): Check for a NULL returned from getXHTMLEntity
11:37 AM Changeset in webkit [21117] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Hyatt.

<rdar://problem/4851944>
REGRESSION: With TOT, BMP images can no longer be opened in the browser window


On Tiger, the MIME type for the com.microsoft.bmp UTI isn't in the registry, so add it manually.


  • platform/MimeTypeRegistry.cpp: (WebCore::initialiseSupportedImageMIMETypes):
11:30 AM Changeset in webkit [21116] by hyatt
  • 4 edits in trunk/WebCore

Fix for bug 13503, invalidateContainingBlockPrefWidths is 15% of the
nesting PLT. Avoid O(n2) behavior by including inlines in the minmax
dirty chain.

Reviewed by kevin

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::calcInlinePrefWidths):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::setPrefWidthsDirty): (WebCore::RenderObject::invalidateContainerPrefWidths):
  • rendering/RenderObject.h:
11:11 AM Changeset in webkit [21115] by kmccullo
  • 2 edits in trunk/WebKitTools
  • Adding debugging statements to see why these tests fail. These will be removed.
  • Scripts/run-webkit-tests:
10:37 AM Changeset in webkit [21114] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Darin.

<rdar://problem/5067925>
Reproducible crash (infinite recursion) with a layout test that submits on unload (13078)


Ask the document loader whether it's loading the main resource, since there are cases where the frame
loader's variable isn't updated correctly.


In the distant future, the frame loader shouldn't have an instance variable but this is the least intrusive fix for now.


I couldn't come up with a good test case for this since it requires running three tests in a row and scheduleLoad didn't work. Also,
calling submit in the onunload prevents any new tests from loading!


  • loader/FrameLoader.cpp: (WebCore::FrameLoader::isLoading):
8:52 AM BuildingGdk edited by alp@atoker.com
(diff)
5:48 AM Changeset in webkit [21113] by zbujtas
  • 2 edits in S60/trunk/WebKit

yadavall, reviewed by <​zbujtas@gmail.com>

DESC: Fix build break
​http://bugs.webkit.org/show_bug.cgi?id=13497


  • group/EABI/browserengineU.def:
3:10 AM Changeset in webkit [21112] by bdash
  • 5 edits in tags/WebKit-419.1/WebKit

Merge r10259 and r15845.

2:41 AM Changeset in webkit [21111] by bdash
  • 3 edits in tags/WebKit-419.1/WebKit

Merge r20042 and follow-ups from r20050, r20058, r20064, r20077 and r20668.

2:20 AM Changeset in webkit [21110] by oliver
  • 2 edits in trunk/WebKit

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

Reviewed by Adam.

Fix <rdar://problem/5061252> REGRESSION: In Gmail, image fails
to be inserted into message field after dragging

Don't try to create <img> tags for local image files as it results
in the potential to submit forms that look like they have an image,
when in reality they don't.

  • WebView/WebHTMLView.mm: (-[WebHTMLView _documentFragmentWithPaths:]):
2:19 AM Changeset in webkit [21109] by bdash
  • 1 copy in tags/WebKit-419.1

New tag.

12:09 AM Changeset in webkit [21108] by andersca
  • 2 edits in trunk/WebCore

... and fix the build.


  • loader/FrameLoader.cpp: (WebCore::FrameLoader::continueAfterNavigationPolicy):
12:06 AM Changeset in webkit [21107] by andersca
  • 2 edits in trunk/WebCore

Revert the previous change - activeDocumentLoader never returns the policy document loader.
Instead, add a null check for the policy document loader.


  • loader/FrameLoader.cpp: (WebCore::FrameLoader::continueAfterNavigationPolicy):
12:02 AM Changeset in webkit [21106] by andersca
  • 2 edits in trunk/WebCore

Use the active document loader instead of the policy document loader.


  • loader/FrameLoader.cpp: (WebCore::FrameLoader::continueAfterNavigationPolicy):

Apr 25, 2007:

11:46 PM Changeset in webkit [21105] by mjs
  • 2 edits in trunk/JavaScriptCore

Reviewed by Anders.


  • fix build problems
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj: Disable warning that gives often downright incorrect results based on guessing what will happen in 64-bit.
11:37 PM Changeset in webkit [21104] by andersca
  • 4 edits in trunk

WebCore:

Reviewed by Maciej.

<rdar://problem/5049099> documents no longer have a default base URL

Don't call canHandleRequest if we have valid substitute data and the protocol is "applewebdata".


  • loader/FrameLoader.cpp: (WebCore::FrameLoader::continueAfterNavigationPolicy):

WebKit:

Reviewed by Maciej.

<rdar://problem/5049099> documents no longer have a default base URL


If the base URL is nil, then create a unique applewebdata URL to match what Tiger WebKit does.


  • WebView/WebFrame.mm: (createUniqueWebDataURL): (-[WebFrame _loadData:MIMEType:textEncodingName:baseURL:unreachableURL:]):
7:10 PM Changeset in webkit [21103] by slewis
  • 2 edits in trunk/WebCore

Fix the build for platforms which overload the abs function

5:26 PM Changeset in webkit [21102] by oliver
  • 2 edits in trunk/WebKit

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

Rubber stamped by Adele.

Roll out WebKit changes from from r21052 to fix regression noted in
<rdar://problem/5159556> REGRESSION: In Mail, pressing option-command- ' doesn't decrease block quote in selection

  • WebView/WebHTMLView.mm: (-[WebHTMLView _interceptEditingKeyEvent:shouldSaveCommand:]): (-[WebHTMLView doCommandBySelector:]):
5:24 PM Changeset in webkit [21101] by zbujtas
  • 18 edits in S60/trunk

rathnasa, reviewed by zbujtas

DESC: widet menu api implementation
​http://bugs.webkit.org/show_bug.cgi?id=13494

4:39 PM Changeset in webkit [23334] by sfalken
  • 5 edits in branches/WindowsMerge

Build fix

4:23 PM BuildingOnWindows edited by pkasting@google.com
(diff)
3:04 PM Changeset in webkit [21100] by hyatt
  • 4 adds in trunk/LayoutTests/fast/block/float

Add another test case that is fixed by the new lazy minmax width code.

2:46 PM Changeset in webkit [21099] by hyatt
  • 3 edits in trunk/WebCore

Fix for bug 13373, make sure to invalidate the pref widths for objects
with percentage padding.

Reviewed by beth

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::layoutBlockChildren): (WebCore::RenderBlock::layoutPositionedObjects):
  • rendering/bidi.cpp: (WebCore::RenderBlock::layoutInlineChildren):
2:44 PM Changeset in webkit [21098] by hyatt
  • 4 adds in trunk/LayoutTests/fast/block/positioning

Add padding percent test.

2:36 PM Changeset in webkit [21097] by harrison
  • 2 edits in trunk/WebCore

Comment change. Correct the header name in the endif comment.


  • editing/FormatBlockCommand.h: Change InsertListCommand_h to FormatBlockCommand_h.
1:25 PM Changeset in webkit [21096] by darin
  • 2 edits in trunk/JavaScriptCore

Reviewed by Geoff.

  • tweak the allocator for a small speedup -- Shark showed this was a win, but I can't measure an improvement right now, but it's also clear these changes do no harm
  • wtf/FastMalloc.cpp: (WTF::LgFloor): Use ALWAYS_INLINE here; in testing I did a while back this was necessary to get this single-instruction function to be inlined. (WTF::SizeClass): Use ALWAYS_INLINE here too for the same reason. Also change the special case for a size of 0 to work without a branch for a bit of extra speed. (WTF::ByteSizeForClass): Use ALWAYS_INLINE here too for the same reason.
1:02 PM Changeset in webkit [21095] by darin
  • 2 edits in trunk/WebCore

Reviewed by Hyatt.

  • platform/AtomicString.cpp: (WebCore::AtomicString::add): Removed unnecessary use of strlen to check if a string is empty.
12:59 PM Changeset in webkit [21094] by ddkilzer
  • 2 edits in trunk/WebCore

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

Build fix.

  • rendering/RenderListMarker.h: Undeleted the m_text member.
12:24 PM GtkAPIProposal edited by Kimmo Kinnunen
(diff)
12:10 PM Changeset in webkit [21093] by hyatt
  • 40 edits in trunk/WebCore

Rearchitect calcPrefWidths. The calculation is now done lazily only when minPrefWidth
or maxPrefWidth are asked for. The result of the calculation is cached.

The new invalidation scheme for pref width invalidation follows the
containing block hierarchy and knows to halt at positioned objects, since
they cannot influence the size of their containers.

Reviewed by darin

  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::init): (WebCore::CSSStyleSelector::initForStyleResolve):
  • page/FrameView.cpp: (WebCore::FrameView::layout):
  • rendering/RenderApplet.cpp: (WebCore::RenderApplet::layout):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::makeChildrenNonInline): (WebCore::RenderBlock::removeChild): (WebCore::RenderBlock::layout): (WebCore::RenderBlock::layoutBlock): (WebCore::RenderBlock::calcPrefWidths): (WebCore::InlineMinMaxIterator::endOfInline): (WebCore::shouldGrowTableCellForImage): (WebCore::RenderBlock::calcInlinePrefWidths): (WebCore::RenderBlock::calcBlockPrefWidths):
  • rendering/RenderBlock.h:
  • rendering/RenderBox.cpp: (WebCore::RenderBox::setStyle): (WebCore::RenderBox::minPrefWidth): (WebCore::RenderBox::maxPrefWidth): (WebCore::RenderBox::calcWidth): (WebCore::RenderBox::calcWidthUsing): (WebCore::RenderBox::calcAbsoluteHorizontal): (WebCore::RenderBox::calcAbsoluteHorizontalValues):
  • rendering/RenderBox.h:
  • rendering/RenderContainer.cpp: (WebCore::RenderContainer::removeChildNode): (WebCore::RenderContainer::appendChildNode): (WebCore::RenderContainer::insertChildNode): (WebCore::RenderContainer::layout):
  • rendering/RenderContainer.h: (WebCore::RenderContainer::moveChildNode):
  • rendering/RenderCounter.cpp: (WebCore::RenderCounter::dirtyLineBoxes): (WebCore::RenderCounter::calcPrefWidths):
  • rendering/RenderCounter.h:
  • rendering/RenderFileUploadControl.cpp: (WebCore::RenderFileUploadControl::calcPrefWidths):
  • rendering/RenderFlexibleBox.cpp: (WebCore::RenderFlexibleBox::calcHorizontalPrefWidths): (WebCore::RenderFlexibleBox::calcVerticalPrefWidths): (WebCore::RenderFlexibleBox::calcPrefWidths): (WebCore::RenderFlexibleBox::layoutBlock):
  • rendering/RenderForeignObject.cpp: (WebCore::RenderForeignObject::layout):
  • rendering/RenderFrameSet.cpp: (WebCore::RenderFrameSet::layout):
  • rendering/RenderHTMLCanvas.cpp: (WebCore::RenderHTMLCanvas::layout):
  • rendering/RenderImage.cpp: (WebCore::RenderImage::layout):
  • rendering/RenderInline.cpp:
  • rendering/RenderInline.h:
  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::updateFromElement): (WebCore::RenderListBox::calcPrefWidths):
  • rendering/RenderListItem.cpp: (WebCore::firstNonMarkerChild): (WebCore::RenderListItem::updateMarkerLocation): (WebCore::RenderListItem::calcPrefWidths): (WebCore::RenderListItem::layout):
  • rendering/RenderListMarker.cpp: (WebCore::RenderListMarker::layout): (WebCore::RenderListMarker::calcPrefWidths): (WebCore::RenderListMarker::updateMargins):
  • rendering/RenderListMarker.h:
  • rendering/RenderObject.cpp: (WebCore::RenderObject::RenderObject): (WebCore::RenderObject::removeChildNode): (WebCore::RenderObject::moveChildNode): (WebCore::RenderObject::appendChildNode): (WebCore::RenderObject::insertChildNode): (WebCore::RenderObject::setPrefWidthsDirty): (WebCore::RenderObject::invalidateContainingBlockPrefWidths): (WebCore::RenderObject::information): (WebCore::RenderObject::setStyle):
  • rendering/RenderObject.h: (WebCore::RenderObject::layer): (WebCore::RenderObject::hasLayer): (WebCore::RenderObject::prefWidthsDirty): (WebCore::RenderObject::setNeedsLayoutAndPrefWidthsRecalc): (WebCore::RenderObject::setHasLayer):
  • rendering/RenderPartObject.cpp: (WebCore::RenderPartObject::layout):
  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::layout):
  • rendering/RenderSVGContainer.h:
  • rendering/RenderSVGHiddenContainer.cpp:
  • rendering/RenderSVGHiddenContainer.h:
  • rendering/RenderSVGText.cpp: (WebCore::RenderSVGText::layout):
  • rendering/RenderTable.cpp: (WebCore::RenderTable::calcWidth): (WebCore::RenderTable::layout): (WebCore::RenderTable::removeChildNode):
  • rendering/RenderTable.h:
  • rendering/RenderTableRow.cpp: (WebCore::RenderTableRow::layout):
  • rendering/RenderTableSection.cpp: (WebCore::RenderTableSection::removeChildNode):
  • rendering/RenderTableSection.h:
  • rendering/RenderText.cpp: (WebCore::RenderText::trimmedPrefWidths): (WebCore::isSpaceAccordingToStyle): (WebCore::RenderText::minPrefWidth): (WebCore::RenderText::maxPrefWidth): (WebCore::RenderText::calcPrefWidths): (WebCore::RenderText::width):
  • rendering/RenderText.h:
  • rendering/RenderView.cpp: (WebCore::RenderView::RenderView): (WebCore::RenderView::calcPrefWidths): (WebCore::RenderView::layout):
  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::layout):
12:01 PM Changeset in webkit [21092] by sfalken
  • 10 edits in trunk

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

Reviewed by Adam.

Mac callbacks for new spelling methods in WebEditorClient.
Not used yet.

  • WebCoreSupport/WebEditorClient.h: (WebEditorClient::ignoreWordInSpellDocument): (WebEditorClient::learnWord): (WebEditorClient::checkSpellingOfString): (WebEditorClient::checkGrammarOfString): (WebEditorClient::updateSpellingUIWithGrammarString): (WebEditorClient::updateSpellingUIWithMisspelledWord): (WebEditorClient::showSpellingUI): (WebEditorClient::spellingUIIsShowing): (WebEditorClient::getGuessesForWord):

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

Reviewed by Adam.

Fix spelling error in spelling method name.
Expose new spelling callback methods to all platforms.

  • bridge/EditorClient.h:
  • platform/gdk/EditorClientGdk.cpp: (WebCore::EditorClientGdk::updateSpellingUIWithGrammarString):
  • platform/gdk/EditorClientGdk.h:
  • platform/graphics/svg/SVGImageEmptyClients.h: (WebCore::SVGEmptyEditorClient::updateSpellingUIWithGrammarString): (WebCore::SVGEmptyEditorClient::getGuessesForWord):

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

Reviewed by Adam.


Fix spelling error in spelling method name.

  • WebCoreSupport/EditorClientQt.cpp: (WebCore::EditorClientQt::updateSpellingUIWithGrammarString):
  • WebCoreSupport/EditorClientQt.h:
11:42 AM Changeset in webkit [21091] by ggaren
  • 10 edits in trunk

WebCore:

Reviewed by Darin Adler, Tim Hatcher.

Fixed a few NSAutoreleasePool issues I noticed while reviewing Brady's patch.

  • platform/graphics/mac/GraphicsContextMac.mm: (WebCore::GraphicsContext::setCompositeOperation): Call -drain instead of -release, since -release is a no-op in a GC world.

WebKit:

Reviewed by Darin Adler, Tim Hatcher.

Fixed a few NSAutoreleasePool issues I noticed while reviewing Brady's patch.

  • Carbon/CarbonUtils.m: (PoolCleaner): Call -drain instead of -release, since -release is a no-op in a GC world.
  • Misc/WebKitErrors.m: (registerErrors): Condensed onto one line.
  • Plugins/WebPluginDatabase.m: (-[WebPluginDatabase refresh]): Call -drain instead of -release, since -release is a no-op in a GC world.
  • WebCoreSupport/WebChromeClient.mm: (WebChromeClient::setStatusbarText):
  • WebInspector/WebNodeHighlightView.m: (-[WebNodeHighlightView initWithHighlight:andRects:forView:]): Don't drain and then release because drain deallocates the receiver, so the release is an over-release.
  • WebView/WebView.mm: (-[WebView rectsForTextMatches]): Re-allocate the pool after draining it, because drain deallocates the receiver, so the drain would leave you without any autorelease pool, causing a leak and then an over-release at the bottom of the loop.
11:21 AM Changeset in webkit [21090] by ap
  • 3 edits in trunk/WebCore

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=13445
NodeList access by index is slow

  • dom/ChildNodeList.cpp: (WebCore::ChildNodeList::item): Support iterating backwards from the last accessed node or from the last child.
  • dom/NodeList.h: Make cachedLength unsigned.
11:10 AM BuildingOnWindows edited by pkasting@google.com
(diff)
10:34 AM Changeset in webkit [21089] by bdash
  • 1 edit in trunk/WebCore/platform/gdk/TemporaryLinkStubs.cpp

Don't try and return a value from a stub function with return type of void.

10:32 AM Changeset in webkit [21088] by bdash
  • 2 edits in trunk/WebCore

2007-04-26 Mark Rowe <​mrowe@apple.com>

Gdk build fix. Add missing stub.

  • platform/gdk/TemporaryLinkStubs.cpp: (Editor::advanceToNextMisspelling):
10:21 AM Changeset in webkit [21087] by rwlbuis
  • 3 edits
    4 adds in trunk

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=12163
SVG text does not appear in a scrolling <div>

Make sure SVG text is painted when scrolling by taking into account the

relative offset of the non-SVG parent in the paintInfo rect.

10:08 AM Changeset in webkit [21086] by bdash
  • 5 edits in trunk/WebCore

2007-04-26 Mark Rowe <​mrowe@apple.com>

Fix non-Mac builds after r21081.

Stub out new Editor methods for Qt and Gdk, and new SVGEmptyEditorClient methods
for all non-Mac builds.

  • editing/qt/EditorQt.cpp: (WebCore::Editor::showSpellingGuessPanel): (WebCore::Editor::spellingPanelIsShowing):
  • page/qt/EventHandlerQt.cpp: (WebCore::EventHandler::passWheelEventToWidget): (WebCore::EventHandler::passMousePressEventToScrollbar):
  • platform/gdk/TemporaryLinkStubs.cpp: (Editor::showSpellingGuessPanel): (Editor::spellingPanelIsShowing):
  • platform/graphics/svg/SVGImageEmptyClients.h: (WebCore::SVGEmptyEditorClient::ignoreWordInSpellDocument): (WebCore::SVGEmptyEditorClient::learnWord): (WebCore::SVGEmptyEditorClient::checkSpellingOfString): (WebCore::SVGEmptyEditorClient::checkGrammarOfString): (WebCore::SVGEmptyEditorClient::udpateSpellingUIWithGrammarString): (WebCore::SVGEmptyEditorClient::updateSpellingUIWithMisspelledWord): (WebCore::SVGEmptyEditorClient::showSpellingUI): (WebCore::SVGEmptyEditorClient::spellingUIIsShowing): (WebCore::SVGEmptyEditorClient::getGuessesForWord):
9:35 AM Changeset in webkit [21085] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Ada.

<rdar://problem/5122153>
Crash occurs at WebCore::Frame::loader() when switching to Bookmarks view while loading standalone image

Don't call writeRawData if the data length is 0. (This is checked by the decoder for the non-raw case).

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::write):
6:57 AM Changeset in webkit [21084] by bdash
  • 2 edits in trunk/WebCore

2007-04-26 Mark Rowe <​mrowe@apple.com>

Gdk build fix courtesy of Alp Toker.

  • make-generated-sources.sh: Remove conditional logic to ensure the script exits with the correct exit code.
5:28 AM BuildingGdk edited by alp@atoker.com
List of build dependencies (diff)
12:47 AM Changeset in webkit [21083] by andersca
  • 5 edits
    2 adds in trunk

LayoutTests:

Reviewed by Mitz.

<rdar://problem/4869095>
default content type for XMLHttpRequest POSTs changed (breaks Flickrator 0.1 widget)

  • http/tests/xmlhttprequest/default-content-type-dashboard-expected.txt: Added.
  • http/tests/xmlhttprequest/default-content-type-dashboard.html: Added.

WebCore:

Reviewed by Mitz.

<rdar://problem/4869095>
default content type for XMLHttpRequest POSTs changed (breaks Flickrator 0.1 widget)


Use the right MIME type.


  • xml/xmlhttprequest.cpp: (WebCore::XMLHttpRequest::send):

WebKitTools:

Reviewed by Mitz.

Add a setUseDashboardCompatibilityMode method to LayoutTestController which is used to toggle
the dashboard compatibility mode.


  • DumpRenderTree/DumpRenderTree.m: (+[LayoutTestController isSelectorExcludedFromWebScript:]): (+[LayoutTestController webScriptNameForSelector:]): (-[LayoutTestController setUseDashboardCompatibilityMode:]): (runTest):
  • DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj:
Note: See TracTimeline for information about the timeline view.