Timeline



Mar 1, 2007:

11:59 PM Changeset in webkit [19940] by antti
  • 8 edits
    3 adds in trunk

LayoutTests:

Reviewed by Maciej.


HTTP layout test for ​http://bugs.webkit.org/show_bug.cgi?id=12690
REGRESSION: can not log in to bank of america with TOT webkit
<rdar://problem/4990044>

also for
​http://bugs.webkit.org/show_bug.cgi?id=12604
​http://bugs.webkit.org/show_bug.cgi?id=12020


  • http/tests/misc/multiple-submit-expected.txt: Added.
  • http/tests/misc/multiple-submit.html: Added.
  • http/tests/misc/resources/post-echo-and-notify-done.cgi: Added.

WebCore:

Reviewed by Maciej.


Fix ​http://bugs.webkit.org/show_bug.cgi?id=12690
REGRESSION: can not log in to bank of america with TOT webkit
<rdar://problem/4990044>


and ​http://bugs.webkit.org/show_bug.cgi?id=12604
REGRESSION: After closing the "Would you like to save password" sheet, the form fails
to submit automatically at ​http://www.mac.com/WebObjects/HomePage.woa
<rdar://problem/4871752>


and ​http://bugs.webkit.org/show_bug.cgi?id=12020
REGRESSION: Flickr uploading broken
<rdar://problem/4928662>


Turn protection against multiple forms submission back on. This approach is buggy
but it is way better than not having it at all. Not protecting against this
breaks number of major sites.


  • bridge/mac/WebCoreAXObject.mm: (-[WebCoreAXObject accessibilityPerformAction:]):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::resetMultipleFormSubmissionProtection): (WebCore::FrameLoader::submitForm): (WebCore::FrameLoader::receivedMainResourceError):
  • loader/FrameLoader.h:
  • page/EventHandler.cpp: (WebCore::EventHandler::keyEvent):
  • page/Frame.cpp: (WebCore::Frame::setView):
  • page/mac/EventHandlerMac.mm: (WebCore::EventHandler::mouseDown):
10:35 PM Changeset in webkit [19939] by bdash
  • 4 edits in tags/Safari-5522.1

Versioning.

9:26 PM Changeset in webkit [19938] by bdash
  • 2 edits in tags/Safari-5522.1/WebCore

Merge in r19928.

2007-03-01 David Harrison <​harrison@apple.com>

Reviewed by Darin.

<rdar://problem/5033905> Have the DOM secondary thread check raise an exception by default

  • platform/mac/ThreadCheck.mm: (WebCore::_WebCoreThreadViolationCheck): Initialize threadViolationIsException to true.
9:18 PM Changeset in webkit [19937] by bdash
  • 10 edits
    2 copies in tags/Safari-5522.1

Merge in r19921.

2007-02-28 Brady Eidson <​beidson@apple.com>

Reviewed by Beth

Move the Thread Safety Check functions into their own header for export to WebKit

  • WebCore.exp: Export the function
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/scripts/CodeGeneratorObjC.pm: Include the new header
  • platform/Logging.h: Move stuff to ThreadCheck.h
  • platform/ThreadCheck.h: Added.
  • platform/mac/LoggingMac.mm: Move stuff to ThreadCheck.mm
  • platform/mac/ThreadCheck.mm: Added. (WebCore::_WebCoreThreadViolationCheck): (WebCoreReportThreadViolation):

2007-02-28 Brady Eidson <​beidson@apple.com>

Reviewed by Beth

Start using the Thread Safety Check implemented in WebCore for the DOM bindings in the rest of
the WebKit API instead of the ASSERT_MAIN_THREAD() hack

  • History/WebBackForwardList.mm: (-[WebBackForwardList initWithWebCoreBackForwardList:]): (-[WebBackForwardList init]): (-[WebBackForwardList dealloc]): (-[WebBackForwardList finalize]):
  • History/WebHistoryItem.mm: (-[WebHistoryItem initWithURLString:title:lastVisitedTimeInterval:]): (-[WebHistoryItem dealloc]): (-[WebHistoryItem finalize]): (-[WebHistoryItem copyWithZone:]): (-[WebHistoryItem initWithWebCoreHistoryItem:]):
  • Misc/WebIconDatabase.mm: (-[WebIconDatabase init]):
8:47 PM Changeset in webkit [19936] by bdash
  • 1 copy in tags/Safari-5522.1

New tag.

7:16 PM Changeset in webkit [23018] by sfalken
  • 1 edit in branches/WindowsMerge/WebKitWin/WebKit.vcproj/VERSION

Bump version for submit

7:13 PM Changeset in webkit [19935] by sfalken
  • 1 copy in tags/Safari-521.35.5b

New tag.

6:42 PM Changeset in webkit [19934] by kmccullo
  • 3 edits
    3 adds in trunk

LayoutTests:

Reviewed by Adam.

  • Tests that the url of a document after an open is what it was before the open.
  • fast/loader/external-script-URL-location-expected.txt: Added.
  • fast/loader/external-script-URL-location.html: Added.
  • fast/loader/resources/external-script-URL-location.js: Added.

WebCore:

Reviewed by Adam.

  • Added the test case: external-script-URL-location.html
  • Fix an issue where the url of a document is null after an open if the document has no parent.
  • dom/Document.cpp: (WebCore::Document::open):
6:29 PM Changeset in webkit [19933] by bdash
  • 3 edits in tags/Safari-4522.1

Versioning.

6:23 PM Changeset in webkit [19932] by bdash
  • 1 copy in tags/Safari-4522.1

New tag.

6:17 PM Changeset in webkit [19931] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Darin.

<rdar://problem/4960250>
​http://bugs.webkit.org/show_bug.cgi?id=11627
REGRESSION: Reproducible crash at IMDb in WebCore::FrameLoader::stopLoadingSubframes


In rare cases, we could end up calling checkLoadComplete twice for the same frame. This would cause the
didFailProvisionalLoad delegate method to be called twice for the same frame, and also cause the provisional document loader
to be reset to null when other code wasn't expecting it.


This regressed in revision 10904 with the fix for <rdar://problem/4184719>. The fix is to only call stopLoading on the frame
if either the document loader is loading, or the document is still being parsed. I've verified that the bug is still fixed and
that no leaks occur.


  • loader/DocumentLoader.cpp: (WebCore::DocumentLoader::stopLoading):
5:58 PM Changeset in webkit [19930] by kjk
  • 2 edits in trunk/WebCore

Reviewed by Darin.

Fix ARM crash due to accessing non-4-byte-aligned memory
as 32-bit values.

  • platform/AtomicString.cpp: (WebCore::UCharBufferTranslator::equal):
5:57 PM Changeset in webkit [19929] by justing
  • 3 edits in trunk/WebKit

Reviewed by harrison


<rdar://problem/4838199>
Integrate Mail and WebKit paste operations


Provide subresources used to create the fragment as a
convenience.

  • WebView/WebHTMLView.mm: (-[WebHTMLView _documentFragmentFromPasteboard:inContext:allowPlainText:chosePlainText:]): Update the calls to the changed method. (-[WebHTMLView _documentFragmentFromPasteboard:forType:inContext:subresources:]): Give the caller the subresources in the WebArchive and RTF cases.
  • WebView/WebHTMLViewPrivate.h:
5:53 PM Changeset in webkit [19928] by harrison
  • 2 edits in trunk/WebCore

Reviewed by Darin.

<rdar://problem/5033905> Have the DOM secondary thread check raise an exception by default

  • platform/mac/ThreadCheck.mm: (WebCore::_WebCoreThreadViolationCheck): Initialize threadViolationIsException to true.
4:46 PM Changeset in webkit [23017] by aroben
  • 5 edits in branches/WindowsMerge/WebKitWin

Reviewed by Kevin M.

Add support for right-truncation to WebKitGraphics.

2:04 PM Changeset in webkit [23016] by andersca
  • 2 edits in branches/WindowsMerge/WebCore

Reviewed by Tim.

  • plugins/win/npfunctions.h: Fix tyop.
2:00 PM Changeset in webkit [19927] by beidson
  • 2 edits in trunk/WebCore

Reviewed by Darin

<rdar://problem/5030628> - Crash opening a new window with the
"New windows open to the same page" pref set

  • history/HistoryItem.cpp: (WebCore::HistoryItem::HistoryItem): Set the m_subItems vector capacity correctly
9:08 AM Changeset in webkit [19926] by ap
  • 3 edits
    2 adds in trunk

Reviewed by Maciej.

​http://bugs.webkit.org/show_bug.cgi?id=12801
Assertion failure in createMarkup() (root) when doing Select All, Copy in an SVG document

Test: editing/pasteboard/createMarkup-assert.xml

  • editing/markup.cpp: (WebCore::createMarkup): Removed the assertion, because it's wrong in a non-HTML world.

Feb 28, 2007:

11:43 PM Changeset in webkit [19925] by aroben
  • 3 edits in trunk/WebCore

Reviewed by Oliver and Hyatt.

Fix <rdar://problem/5024233> Crash while using Find on empty document

No layout test possible, as this involves a null document.

  • dom/Range.cpp: (WebCore::rangeOfContents): Added an ASSERT.
  • page/Frame.cpp: (WebCore::Frame::findString): Added a null-check for document(). (WebCore::Frame::markAllMatchesForText): Ditto.
8:42 PM Changeset in webkit [23015] by aroben
  • 9 edits in branches/WindowsMerge

WebCoreWin:

Reviewed by Anders.

Clean up WebCoreTextRenderer. WebCoreDrawTextAtPoint now just draws
text at a point. It doesn't center truncate, clip, or modify alignment.

No layout test possible.

  • platform/win/WebCoreTextRenderer.cpp: (WebCore::WebCoreDrawTextAtPoint): Use only WebCore types, and don't center truncate or modify alignment. (WebCore::WebCoreDrawDoubledTextAtPoint): Moved from WebKitGraphics.cpp. (WebCore::WebCoreTextFloatWidth): Added.
  • platform/win/WebCoreTextRenderer.h: Updated/added declarations.

WebKitWin:

Reviewed by Anders.

Made WebKitGraphics more flexible.

7:25 PM Changeset in webkit [19924] by oliver
  • 2 edits in trunk/WebKit

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

Reviewed by Maciej.

Fixes <rdar://problem/5012009>


When looking for a requested resource we should also check
the set of manually added subresources if WebCore can't find it.

  • WebView/WebDataSource.mm: (-[WebDataSource subresourceForURL:]):
5:30 PM Changeset in webkit [19923] by thatcher
  • 8 edits in trunk/WebKitTools

Reviewed by Tim H.

Fixes: ​http://bugs.webkit.org/show_bug.cgi?id=12887
Bug#12887: [Drosera] Add ability to close loaded files

  • Drosera/DebuggerDocument.h:
  • Drosera/DebuggerDocument.m: (-[DebuggerDocument closeCurrentFile:]): Adds delegation to call a JS script to close files
  • Drosera/Drosera.xcodeproj/project.pbxproj:
  • Drosera/English.lproj/MainMenu.nib/classes.nib:
  • Drosera/English.lproj/MainMenu.nib/info.nib:
  • Drosera/English.lproj/MainMenu.nib/keyedobjects.nib: Adds Close Current File menu item
  • Drosera/debugger.html: Changed "no files loaded" to "<No files loaded>" to match Xcode style
  • Drosera/debugger.js: Adds implementation of closeFile() to unload currently loaded file
4:47 PM Changeset in webkit [19922] by kjk
  • 2 edits in trunk/WebCore

Not reviewed - simple gdk build fix.

Add stubs for newly introduced localized strings.

  • platform/gdk/TemporaryLinkStubs.cpp: (WebCore::searchMenuNoRecentSearchesText): (WebCore::searchMenuRecentSearchesText): (WebCore::searchMenuClearRecentSearchesText):
4:01 PM Changeset in webkit [19921] by beidson
  • 10 edits
    2 adds in trunk

WebCore:

Reviewed by Beth

Move the Thread Safety Check functions into their own header for export to WebKit

  • WebCore.exp: Export the function
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/scripts/CodeGeneratorObjC.pm: Include the new header
  • platform/Logging.h: Move stuff to ThreadCheck.h
  • platform/ThreadCheck.h: Added.
  • platform/mac/LoggingMac.mm: Move stuff to ThreadCheck.mm
  • platform/mac/ThreadCheck.mm: Added. (WebCore::_WebCoreThreadViolationCheck): (WebCoreReportThreadViolation):

WebKit:

Reviewed by Beth

Start using the Thread Safety Check implemented in WebCore for the DOM bindings in the rest of
the WebKit API instead of the ASSERT_MAIN_THREAD() hack

  • History/WebBackForwardList.mm: (-[WebBackForwardList initWithWebCoreBackForwardList:]): (-[WebBackForwardList init]): (-[WebBackForwardList dealloc]): (-[WebBackForwardList finalize]):
  • History/WebHistoryItem.mm: (-[WebHistoryItem initWithURLString:title:lastVisitedTimeInterval:]): (-[WebHistoryItem dealloc]): (-[WebHistoryItem finalize]): (-[WebHistoryItem copyWithZone:]): (-[WebHistoryItem initWithWebCoreHistoryItem:]):
  • Misc/WebIconDatabase.mm: (-[WebIconDatabase init]):
3:59 PM Changeset in webkit [23014] by adele
  • 2 edits in branches/WindowsMerge/WebCore

Reviewed by Adam.

Fix for <rdar://problem/4887423> REGRESSION: search results popup menu strings are not localized
and <rdar://problem/3517227> accessibility-related strings in WebCore are not localized

  • platform/win/TemporaryLinkStubs.cpp: (WebCore::searchMenuNoRecentSearchesText): (WebCore::searchMenuRecentSearchesText): (WebCore::searchMenuClearRecentSearchesText): (WebCore::AXWebAreaText): (WebCore::AXLinkText): (WebCore::AXListMarkerText): (WebCore::AXImageMapText): (WebCore::AXHeadingText):
3:46 PM Changeset in webkit [19920] by bdakin
  • 2 edits in trunk/WebCore

Reviewed by Brady.

Fix for ​http://bugs.webkit.org/show_bug.cgi?id=12923 REGRESSION:
Assertion failure copying standalone image

  • platform/mac/PasteboardMac.mm: (WebCore::Pasteboard::writeImage): Use the CachedImage as the resource. This makes more sense anyway. There is no need to null- check the renderer or the CachedImage since we return early if there is no Image* and the HitTestResult::image() function checks for these things.
3:08 PM Changeset in webkit [19919] by adele
  • 8 edits in trunk

WebCore:

Reviewed by Beth.

Fix for <rdar://problem/4887423> REGRESSION: search results popup menu strings are not localized
and <rdar://problem/3517227> accessibility-related strings in WebCore are not localized

Use localized strings from WebKit instead of hard coded strings.

  • bridge/mac/WebCoreAXObject.mm: (-[WebCoreAXObject roleDescription]):
  • page/mac/WebCoreViewFactory.h:
  • platform/LocalizedStrings.h:
  • platform/mac/LocalizedStringsMac.mm: (WebCore::searchMenuNoRecentSearchesText): (WebCore::searchMenuRecentSearchesText): (WebCore::searchMenuClearRecentSearchesText): (WebCore::AXWebAreaText): (WebCore::AXLinkText): (WebCore::AXListMarkerText): (WebCore::AXImageMapText): (WebCore::AXHeadingText):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::itemText):

WebKit:

Reviewed by Beth.

Fix for <rdar://problem/4887423> REGRESSION: search results popup menu strings are not localized
and <rdar://problem/3517227> accessibility-related strings in WebCore are not localized

  • WebCoreSupport/WebViewFactory.mm: (-[WebViewFactory searchMenuNoRecentSearchesText]): (-[WebViewFactory searchMenuRecentSearchesText]): (-[WebViewFactory searchMenuClearRecentSearchesText]): (-[WebViewFactory AXWebAreaText]): (-[WebViewFactory AXLinkText]): (-[WebViewFactory AXListMarkerText]): (-[WebViewFactory AXImageMapText]): (-[WebViewFactory AXHeadingText]):
2:31 PM Changeset in webkit [19918] by brmorris
  • 2 edits in S60/trunk/JavaScriptCore

vbradley, reviewed by Yongjun

DESC: merge r19917 to s60/trunk. ALAA-6XSF7U : Reduce stack overflow counter
​http://bugs.webkit.org/show_bug.cgi?id=12928

2:26 PM Changeset in webkit [19917] by brmorris
  • 2 edits in S60/branches/3.1m/JavaScriptCore

vbradley, reviewed by Yongjun.

DESC: ALAA-6XSF7U : Reduce stack overflow counter
​http://bugs.webkit.org/show_bug.cgi?id=12928

2:23 PM Changeset in webkit [19916] by brmorris
  • 3 edits in S60/trunk/WebKit

yinxcai, reviewed by <Yongjun>

DESC: TMCN-6Y9Q7A --Browser ask permission for unsecure item load repeatedly
​http://bugs.webkit.org/show_bug.cgi?id=12901

2:17 PM Changeset in webkit [19915] by brmorris
  • 13 edits in S60/trunk/WebKit

yinxcai, reviewed by <Sachin>

DESC: merge of r19914 to s60/turnk: Browser may crash when selecting No for secure connections HMNN-6YLEHJ
​http://bugs.webkit.org/show_bug.cgi?id=12858

2:15 PM Changeset in webkit [19914] by brmorris
  • 13 edits in S60/branches/3.1m/WebKit

yinxcai, reviewed by <Sachin>

DESC Browser may crash when selecting No for secure connections HMNN-6YLEHJ
​http://bugs.webkit.org/show_bug.cgi?id=12858

2:11 PM Changeset in webkit [19913] by brmorris
  • 2 edits in S60/branches/3.1m/WebKit

yadavall, Reviewed by Sachin

DESC: Toolbar should be activated only on long keypress CR: 102-5945
​http://bugs.webkit.org/show_bug.cgi?id=12816

11:56 AM Changeset in webkit [19912] by ggaren
  • 1 edit
    1 move in trunk/LayoutTests

Reviewed by Bradeee Eidson.

Disabling this test for now because it fails on the buildbot but not on
my Intel or PPC machines. I've filed ​http://bugs.webkit.org/show_bug.cgi?id=12930
to track this issue.

  • fast/leaks/003.html: Removed.
7:46 AM Changeset in webkit [19911] by antti
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by Maciej.

  • fast/lists/ol-start-dynamic-expected.checksum: Added.
  • fast/lists/ol-start-dynamic-expected.png: Added.
  • fast/lists/ol-start-dynamic-expected.txt: Added.
  • fast/lists/ol-start-dynamic.html: Added.

WebCore:

Reviewed by Maciej.


Update list marker value when start attribute changes.

  • html/HTMLOListElement.cpp: (WebCore::HTMLOListElement::parseMappedAttribute):
4:29 AM Changeset in webkit [19910] by zimmermann
  • 8 edits in trunk

Reviewed by Maciej.

Corrected the use of the new ENABLE() macros in some ksvg2/svg files.
Add new build-webkit flags "--(no-)xpath" / "--(no-)xslt", to be able
to switch off build features easily. Also add "--(no-)svg-experimental-features"
flag, to be able to test filters/animations/use/foreignObject easily.

1:00 AM Changeset in webkit [19909] by bdash
  • 2 edits in trunk/WebKit

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

Reviewed by Maciej.

<rdar://problem/5028473> WebKit allocates a huge number of NSCalendarDates while loading history file

  • History/WebHistory.mm: (-[WebHistoryPrivate insertItem:atDateIndex:]): Use lastVisitedTimeInterval rather than _lastVisitedDate to avoid allocating NSCalendarDates.
12:42 AM Changeset in webkit [23013] by mjs
  • 2 edits in branches/WindowsMerge/WebCore

Reviewed by Adam.

  • platform/network/cf/ResourceRequestCFNet.cpp: (WebCore::ResourceRequest::doUpdatePlatformRequest): Use RetainPtr instead of explicit release, as this was caused by CFRelease on a null pointer, and REtainPtr is tolerant.
12:29 AM Changeset in webkit [19908] by rwlbuis
  • 34 edits
    4 adds in trunk

Reviewed by Maciej.

​http://bugs.webkit.org/show_bug.cgi?id=12609
Any SVG element will create renderers even when children of HTML elements

Allow creation of svg renderers only when parent is SVG, except for
the <svg> element.

12:10 AM Changeset in webkit [23012] by sfalken
  • 2 edits in branches/WindowsMerge/WebKitWin

Feb 27, 2007:

11:55 PM Changeset in webkit [19907] by rwlbuis
  • 3 edits
    4 adds in trunk

Reviewed by Maciej.

​http://bugs.webkit.org/show_bug.cgi?id=12913
Markers do not render in webkit when it misses markerWidth or markerHeight attribute

Set defaults for markerWidth/markerHeight so markers that do not specify them render.

9:33 PM Changeset in webkit [19906] by ggaren
  • 2 edits in trunk/LayoutTests

Extending the timeout on this test to see if it fixes the buildbot failure.
(I don't get a failure locally.)

  • fast/leaks/003.html:
9:13 PM Changeset in webkit [23011] by adachan
  • 7 edits
    4 adds in branches/WindowsMerge/WebKitWin
6:04 PM Changeset in webkit [23010] by aliceli1
  • 3 edits in branches/WindowsMerge/WebKitWin

WebKitWin:

Reviewed by Steve, Adam, Maciej, and their mothers.

This is a general leaks fix but helps the effort towards fixing
<rdar://4974495> Boomer leaks while running HTML iBench
However, this fix doens't close that bug.

  • WebFrame.cpp: (WebFrame::setTitle):

fix a leak of WebHistory

  • WebHistory.cpp: (_WebCoreHistoryProvider::containsItemForURLUnicode):

fix a leak of WebHistory

5:17 PM Changeset in webkit [23009] by sfalken
  • 1 edit in branches/WindowsMerge/WebKitWin/WebKit.vcproj/VERSION

Bump version for submit

5:16 PM Changeset in webkit [19905] by andersca
  • 2 edits in trunk/WebCore
  • loader/ResourceLoader.cpp: (WebCore::ResourceLoader::~ResourceLoader): Remove comment.
5:15 PM Changeset in webkit [19904] by sfalken
  • 1 copy in tags/Safari-521.35.3b

New tag.

5:12 PM Changeset in webkit [23008] by aroben
  • 2 edits in branches/WindowsMerge/WebKitWin

Reviewed by Anders.

Fix <rdar://problem/5004615> View source menu option is always disabled
when Netscape 8 is installed

The bug was caused by Netscape's npTrident.dll plugin, which registers
itself as a handler for text/html and text/xml.

  • WebFrame.cpp: (WebFrame::canProvideDocumentSource): Explicitly allow viewing the source of HTML and XML documents, rather than disallowing text, image, and plugin documents. Plugins can register themselves as handlers for HTML and XML MIME types, but we still want to display source in those cases.
5:08 PM Changeset in webkit [19903] by justing
  • 4 edits
    4 adds in trunk

LayoutTests:

Reviewed by oliver


<rdar://problem/5027300>
REGRESSION: Images inserted with align left/right are lost

  • editing/inserting/12882-expected.checksum: Added.
  • editing/inserting/12882-expected.png: Added.
  • editing/inserting/12882-expected.txt: Added.
  • editing/inserting/12882.html: Added.

WebCore:

Reviewed by oliver

<rdar://problem/5027300>
REGRESSION: Images inserted with align left/right are lost

  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::shouldMerge): Don't attempt to merge to or from a position before or after a block because it will be a no-op and lead to infinite recursion. In this case it instead resulted in content loss because of bugs in start/endOfParagraph (5027702).
  • editing/visible_units.cpp: Added two FIXMEs for the problems with start/endOfParagraph.
4:38 PM Changeset in webkit [19902] by andersca
  • 6 edits in trunk/WebCore

Reviewed by Geoff.

Make resource load delegate methods pass the right document loader.


  • loader/FrameLoader.cpp: (WebCore::FrameLoader::willSendRequest): (WebCore::FrameLoader::didReceiveResponse): (WebCore::FrameLoader::didReceiveData): (WebCore::FrameLoader::didFailToLoad): (WebCore::FrameLoader::didFinishLoad): (WebCore::FrameLoader::didReceiveAuthenticationChallenge): (WebCore::FrameLoader::didCancelAuthenticationChallenge): Use the resource loader's document loader instead of the active one.


  • loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::receivedError): Make it so we send the frame load delegate method before the resource load delegate method. This was a regression from 2.0 and was caused by the fix to rdar://problem/4609195. Because the way the loader now works, both delegate methods will be called.


  • loader/ResourceLoader.cpp: (WebCore::ResourceLoader::ResourceLoader):
  • loader/ResourceLoader.h: (WebCore::ResourceLoader::documentLoader): Add document loader pointer to ResourceLoader.
4:26 PM Changeset in webkit [19901] by ggaren
  • 4 edits in trunk/JavaScriptCore

Reviewed by Darin Adler.


Follow-up to fixing ​http://bugs.webkit.org/show_bug.cgi?id=12659 | <rdar://problem/4954306>
JS objects not collected after closing window @ ebay.com/maps.google.com


Changed Interpreter cache of global constructors and prototypes from
ProtectedPtrs to bare, marked pointers. ProtectedPtrs are inefficient,
and they increase the risk of reference cycles. Also, Darin said something
about ProtectedPtrs giving him warts.


Also changed data members to precise types from generic JSObject*'s.


Layout tests and JS tests pass.

  • kjs/SavedBuiltins.h:
  • kjs/interpreter.cpp: (KJS::Interpreter::init): (KJS::Interpreter::~Interpreter): (KJS::Interpreter::initGlobalObject): Moved Identifier::init() call to constructor, for clarity. (KJS::Interpreter::mark):
  • kjs/interpreter.h:
4:21 PM Changeset in webkit [19900] by ggaren
  • 1 edit in trunk/LayoutTests/fast/leaks/003-expected.txt

Oops! Forgot to check in this file.

4:04 PM Changeset in webkit [19899] by weinig
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by Hyatt.

  • fast/repaint/list-marker-expected.checksum: Added.
  • fast/repaint/list-marker-expected.png: Added.
  • fast/repaint/list-marker-expected.txt: Added.
  • fast/repaint/list-marker.html: Added.

WebCore:

Reviewed by Hyatt.

Test: fast/repaint/list-marker.html

  • rendering/RenderListItem.cpp: (WebCore::RenderListItem::positionListMarker): Add the marker to the visual overflow of all its ancestor blocks up to the list item. This needs to be done here since the marker is positioned only after those blocks have been laid out.
3:38 PM Changeset in webkit [19898] by bdash
  • 3 edits in trunk/WebKit

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

Reviewed by Tim Hatcher.

<rdar://problem/4985524> Problem with Blot and ToT WebKit (decoding WebCoreScrollView)

References to WebCoreScrollView as a subview of a WebHTMLView may be present in some NIB
files, so NSUnarchiver must be still able to look up the WebCoreScrollView class.

  • WebKit.exp: Export WebCoreScrollView symbol.
  • WebView/WebHTMLView.mm: Add empty WebCoreScrollView class.
3:27 PM Changeset in webkit [23007] by adachan
  • 1 edit in branches/WindowsMerge/WebKitWin/ChangeLog

Oops, ChangeLog wasn't saved last time.

3:24 PM Changeset in webkit [23006] by bdakin
  • 4 edits in branches/WindowsMerge

WebKitWin:

Reviewed by Adam.

Make msimg32.lib available to the engine so that we can use
AlphaBlend.

  • WebKit.vcproj/WebKit.vcproj:

WebCoreWin:

Reviewed by Adam.

<rdar://problem/4956528> Implement Pasteboard::writeImage for
Boomer context menus

  • platform/win/PasteboardWin.cpp: (WebCore::Pasteboard::writeImage): Copy a bitmap to the clipboard.
3:22 PM Changeset in webkit [23005] by adachan
  • 2 edits in branches/WindowsMerge/WebKitWin

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

Fix build.

  • WebFrame.cpp:
3:13 PM Changeset in webkit [19897] by antti
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by Hyatt.


  • fast/events/mouseout-dead-node-expected.checksum: Added.
  • fast/events/mouseout-dead-node-expected.png: Added.
  • fast/events/mouseout-dead-node-expected.txt: Added.
  • fast/events/mouseout-dead-node.html: Added.

WebCore:

Reviewed by Hyatt.



If a node dies while mouse is over it, it is still supposed to receive
mouseout event (wasn't case in Tiger webkit). However this event should
not propagate to any other nodes. This patch matches Firefox behavior in
this respect.

  • dom/EventTargetNode.cpp: (WebCore::EventTargetNode::dispatchGenericEvent):
3:10 PM Changeset in webkit [19896] by ggaren
  • 2 edits in trunk/LayoutTests

Bumped up the wiggle room on this test because it's failing on the buildbot.
(We're still well below the ~100 objects you would see in a real failure.)

  • fast/leaks/003.html:
3:09 PM Changeset in webkit [19895] by ggaren
  • 1 edit in trunk/LayoutTests/fast/dom/gc-10.html

Oops! Forgot to commit this file

3:03 PM Changeset in webkit [23004] by oliver
  • 4 edits in branches/WindowsMerge

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

Reviewed by Lou and Steve.


WebCoreWin:

Add a store for current Cursor to fix <rdar://problem/4788352>

  • platform/win/WidgetWin.cpp: (WebCore::Widget::setCursor):


WebKitWin:

<rdar://4983106> Stop/reload button never changes back to
reload when a click on a link spawns a download

  • WebError.cpp: (WebError::isPolicyChangeError): fixed string comparison
  • WebFrame.cpp: (WebFrame::interruptForPolicyChangeError): implemented the method


2:28 PM Changeset in webkit [19894] by ggaren
  • 10 edits
    2 adds in trunk

JavaScriptCore:

Reviewed by Maciej Stachowiak.


Fixed ​http://bugs.webkit.org/show_bug.cgi?id=12659 | <rdar://problem/4954306>
JS objects not collected after closing window @ ebay.com/maps.google.com

Don't GC in the Interpreter destructor. For that to work, the Interpreter
would have to NULL out all of its ProtectedPtrs before calling collect(). But
we've decided that we don't want things to work that way, anyway. We want the
client to be in charge of manual GC so that it can optimize cases when
it will be destroying many interpreters at once
(e.g., ​http://bugs.webkit.org/show_bug.cgi?id=12900).


Also removed Interpreter::collect() because it was redundant with
Collector::collect().

  • JavaScriptCore.exp:
  • kjs/interpreter.cpp: (KJS::Interpreter::~Interpreter):
  • kjs/testkjs.cpp: (TestFunctionImp::callAsFunction):

LayoutTests:

Reviewed by Maciej Stachowiak.


Test for ​http://bugs.webkit.org/show_bug.cgi?id=12659 | <rdar://problem/4954306>
JS objects not collected after closing window @ ebay.com/maps.google.com


  • fast/leaks/003-expected.txt: Added.
  • fast/leaks/003.html: Added.
  • fast/dom/gc-10.html: Bumped the wiggle room threshold on this test by 2 because it's failing on my machine with the patch applied but, according to the test notes, we're still well within the margin of error.

WebCore:

Reviewed by Maciej Stachowiak.


Fixed ​http://bugs.webkit.org/show_bug.cgi?id=12659 | <rdar://problem/4954306>
JS objects not collected after closing window @ ebay.com/maps.google.com

Garbage collect in the KJSProxy destructor, after clearing our reference
to the interpreter, because that's when the interpreter has torn down fully.

(Technically speaking, we can't *prove* that we have the only reference to
our interpreter, but that's how it works in practice, and manual garbage
collection is just an opportunistic optimization, so it's OK for it to
work in practice even if it can't be proven in theory.)


Layout tests pass. No leaks reported.

  • bindings/js/kjs_proxy.cpp: (WebCore::KJSProxy::~KJSProxy):
  • bindings/js/kjs_proxy.h:
  • page/Page.cpp: (WebCore::Page::~Page): Merged pageDestroyed() calls. Moved debug-only code to the bottom.
  • page/Frame.cpp: (WebCore::Frame::~Frame): Don't call getObject() because globalObject() returns a JSObject* already, and the call can leave a pointer to the Window object on the stack. Don't check for NULL because it is an invariant of JavaScriptCore that no JSObject* can be NULL. Do use a volatile pointer for w because the 'w = 0' assignment just screams to the compiler, "Don't generate any code for me!"
2:26 PM Changeset in webkit [23003] by adachan
  • 3 edits in branches/WindowsMerge/WebKitWin

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

Reviewed by Steve and Anders.

<rdar://4983106> Stop/reload button never changes back to reload when a click on a link spawns a download

WebKitWin:

  • WebError.cpp: (WebError::isPolicyChangeError): fixed string comparison
  • WebFrame.cpp: (WebFrame::interruptForPolicyChangeError): implemented the method
12:50 PM Changeset in webkit [19893] by aroben
  • 2 edits in trunk/WebKit

Reviewed by Beth.

Fix <rdar://problem/5011905> REGRESSION: "Open Link" contextual menu
item appears twice

  • WebCoreSupport/WebContextMenuClient.mm: (fixMenusToSendToOldClients): Remove the "Open Link" item from the default menu items array before sending it off to Tiger Mail. (WebContextMenuClient::getCustomMenuFromDefaultItems): Set the representedObject on every NSMenuItem to match our old (correct) API behavior.
12:31 PM Changeset in webkit [19892] by rwlbuis
  • 3 edits
    4 adds in trunk

Reviewed by David Hyatt.

​http://bugs.webkit.org/show_bug.cgi?id=4128
!important is ignored in inline styling.

Handle properties with !important flag better in inline
style declarations.

12:13 PM Changeset in webkit [19891] by hyatt
  • 2 edits in trunk/WebCore

Bug 11435. Make sure RenderViews always paintBoxDecorations. They used to before I removed RenderView's
paint method. This fix restores the original behavior.

Reviewed by mitz

  • rendering/RenderBox.cpp: (WebCore::RenderBox::setStyle):
11:51 AM Changeset in webkit [19890] by hyatt
  • 2 edits in trunk/WebCore

Fix for bug 12094, make sure setMinMaxKnown starts off false for the weird/rare case of empty
plaintext documents (which really are buggy and should be fixed to generate root elements).

Reviewed by mitz

  • rendering/RenderView.cpp: (WebCore::RenderView::RenderView):
10:45 AM Changeset in webkit [19889] by ap
  • 2 edits in trunk/WebCore

2007-02-27 Dex Deacon <​occupant4@gmail.com>

Reviewed by Darin.

Fixed the case where a BackForwardList of capacity==1 would grow without bound.

  • history/BackForwardList.cpp: (WebCore::BackForwardList::addItem):
10:10 AM Changeset in webkit [19888] by ap
  • 2 edits in trunk/WebCore

2007-02-22 Lars Naesbye Christensen <​lars@naesbye.dk>

Reviewed by Maciej.

​http://bugs.webkit.org/show_bug.cgi?id=12848
Help cursor should have a white outline

  • Resources/helpCursor.png:
10:00 AM Changeset in webkit [19887] by ap
  • 2 edits in trunk/WebKit

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

Reviewed by Darin.

  • WebCoreSupport/WebChromeClient.mm: (WebChromeClient::scaleFactor): If the view is not in a window, use the main screen's scale factor as a best guess.
9:48 AM Changeset in webkit [19886] by ap
  • 2 edits in trunk/WebCore

Reviewed by Maciej.

​http://bugs.webkit.org/show_bug.cgi?id=12594
REGRESSION: Strange highlight in active input area

No automated test possible.

  • rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::paint): Restore a check lost in r12792.
8:18 AM Changeset in webkit [19885] by ggaren
  • 2 edits in trunk/WebKitTools

Small tweak to run-webkit-tests.

  • Scripts/run-webkit-tests: Allow people with lots of RAM to run more than 1000 MallocStackLogging tests at a time.
6:08 AM Changeset in webkit [19884] by darin
  • 2 edits in trunk/WebCore

Reviewed by Mitz.

  • page/mac/EventHandlerMac.mm: (WebCore::EventHandler::focusDocumentView): Add null checks.
6:05 AM Changeset in webkit [19883] by darin
  • 3 edits in trunk/WebCore

Reviewed by Alexey.

  • platform/TextCodecICU.cpp: (WebCore::TextCodecICU::registerEncodingNames): Add "macroman" as an alias for "macintosh". We have seen that in use in mail, although I'm not sure it's used on the web.
  • platform/mac/mac-encodings.txt: Remove MacRoman line, which was added back as part of the patch for bug 4971226. But by adding it to the this file rather than the ICU codec, we get it only on Mac OS X. And long term we are trying to get rid of the use of TEC entirely, so we want this file to be as empty as possible.
6:05 AM Changeset in webkit [19882] by darin
  • 1 edit
    2 adds in trunk/LayoutTests
  • test for MacRoman encoding name alias
  • fast/encoding/charset-names-expected.txt: Added.
  • fast/encoding/charset-names.html: Added.
5:02 AM Changeset in webkit [19881] by darin
  • 2 edits in trunk/LayoutTests
  • fast/css/computed-style-expected.txt: Updated test result for Hyatt's addition of -webkit-border-fit.
2:12 AM Changeset in webkit [19880] by rwlbuis
  • 7 edits
    4 adds in trunk

Reviewed by Maciej.

​http://bugs.webkit.org/show_bug.cgi?id=12905
Not all svg shapes should support markers

Restrict marker usage to svg shapes polyline, polygon, line and path.

12:44 AM Changeset in webkit [19879] by adele
  • 6 edits in trunk

LayoutTests:

Reviewed by Lars.

Updated test for ​http://bugs.webkit.org/show_bug.cgi?id=12902
<rdar://problem/5012679> REGRESSION: Pressing return key doesn't
move caret to next line after applying a font color in GMail

  • fast/frames/iframe-window-focus-expected.txt:
  • fast/frames/iframe-window-focus.html: Updated test to include insertion of newlines.

WebCore:

Reviewed by Lars.

Fix for ​http://bugs.webkit.org/show_bug.cgi?id=12902
<rdar://problem/5012679> REGRESSION: Pressing return key doesn't
move caret to next line after applying a font color in GMail

Test: fast/frames/iframe-window-focus-2.html

  • page/EventHandler.h: Make focusDocumentView public.
  • page/Frame.cpp: (WebCore::Frame::focusWindow): Instead of just setting the focused frame, we should set also focus the document view. The bug here was that since the subframe view wasn't the first responder, the key down event was going to the wrong frame.

Feb 26, 2007:

11:54 PM Changeset in webkit [23002] by oliver
  • 2 edits in branches/WindowsMerge/WebCore

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

Reviewed by Maciej.

To stop the drag image for transparent images from looking
ugly (especially fully transparent images) we render the
image to a white background prior to passing the image on to
Windows


<rdar://problem/5015946>

  • platform/win/DragImageWin.cpp: (WebCore::createDragImageFromImage):
11:19 PM Changeset in webkit [19878] by mjs
  • 3 edits
    20 adds in trunk

LayoutTests:

Reviewed by Oliver.

  • svg/custom/circular-marker-reference-1-expected.checksum: Added.
  • svg/custom/circular-marker-reference-1-expected.png: Added.
  • svg/custom/circular-marker-reference-1-expected.txt: Added.
  • svg/custom/circular-marker-reference-1.svg: Added.
  • svg/custom/circular-marker-reference-2-expected.checksum: Added.
  • svg/custom/circular-marker-reference-2-expected.png: Added.
  • svg/custom/circular-marker-reference-2-expected.txt: Added.
  • svg/custom/circular-marker-reference-2.svg: Added.
  • svg/custom/circular-marker-reference-3-expected.checksum: Added.
  • svg/custom/circular-marker-reference-3-expected.png: Added.
  • svg/custom/circular-marker-reference-3-expected.txt: Added.
  • svg/custom/circular-marker-reference-3.svg: Added.
  • svg/custom/circular-marker-reference-4-expected.checksum: Added.
  • svg/custom/circular-marker-reference-4-expected.png: Added.
  • svg/custom/circular-marker-reference-4-expected.txt: Added.
  • svg/custom/circular-marker-reference-4.svg: Added.
  • svg/custom/non-circular-marker-reference-expected.checksum: Added.
  • svg/custom/non-circular-marker-reference-expected.png: Added.
  • svg/custom/non-circular-marker-reference-expected.txt: Added.
  • svg/custom/non-circular-marker-reference.svg: Added.

WebCore:

Reviewed by Oliver.

  • platform/graphics/svg/SVGResourceMarker.cpp: (WebCore::SVGResourceMarker::draw): Use a HashSet to guard against reference cycles.
9:30 PM Changeset in webkit [19877] by mjs
  • 4 edits in trunk

WebCore:

Reviewed by Kevin McCullough.

  • fix Qt build for earlier SVG changes.
  • platform/graphics/svg/qt/SVGResourceFilterQt.cpp:

WebKitQt:

Reviewed by Kevin McCullough.

  • fix Qt build for earlier SVG changes.
  • WebCoreSupport/EditCommandQt.cpp:
9:23 PM Changeset in webkit [19876] by oliver
  • 2 edits in trunk/WebCore

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

Reviewed by Maciej.

Fix for <rdar://problem/4827378>: Canvas with large height
uses lots of memory, computer almost stops responding


Put cap on maximum area of canvas, size is similar too the
maximum size allowed by firefox (firefox seems to to cut off
at area == 32767 * 9358).

Also protect renderer against the possibility of a null context
(this was triggering a CG warning)

  • html/HTMLCanvasElement.cpp: (WebCore::HTMLCanvasElement::createDrawingContext):

Apply maximum canvas area

(WebCore::HTMLCanvasElement::createPlatformImage):

Protect against null CG Context

9:13 PM Changeset in webkit [19875] by bdash
  • 3 edits
    4 adds in trunk

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

Reviewed by Antti Koivisto.

Test: fast/lists/list-style-type-dynamic-change.html

  • rendering/RenderListMarker.cpp: (WebCore::RenderListMarker::setStyle): Made changing the marker type trigger updating of the marker.

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

Reviewed by Antti Koivisto.

  • fast/lists/list-style-type-dynamic-change-expected.checksum: Added.
  • fast/lists/list-style-type-dynamic-change-expected.png: Added.
  • fast/lists/list-style-type-dynamic-change-expected.txt: Added.
  • fast/lists/list-style-type-dynamic-change.html: Added.
8:04 PM Changeset in webkit [19874] by mjs
  • 2 edits in trunk/WebKitTools

Reviewed by Lars.


  • set ENABLE_XSLT even when compiling without SVG support, since that is no longer hardcoded into config.h.
  • Scripts/build-webkit:
5:56 PM Changeset in webkit [19873] by kjk
  • 4 edits in trunk

JavaScriptCore:

Reviewed by Adam Roben.

Rename *_SUPPORT defines to ENABLE_*.

  • jscore.bkl:

WebKitTools:

Reviewed by Adam Roben.

Rename *_SUPPORT defines to ENABLE_*.

  • GdkLauncher/gdklauncher.bkl:
5:36 PM Changeset in webkit [19872] by adele
  • 5 edits
    2 adds in trunk

LayoutTests:

Reviewed by Maciej.

Test for <rdar://problem/5012761> REGRESSION: form submit invokes wrong event handler

  • fast/events/submit-reset-nested-bubble-expected.txt: Added.
  • fast/events/submit-reset-nested-bubble.html: Added.

WebCore:

Reviewed by Maciej.

Fix for <rdar://problem/5012761> REGRESSION: form submit invokes wrong event handler

Test: fast/events/submit-reset-nested-bubble.html

  • dom/EventTargetNode.h: Made handleLocalEvents virtual.
  • html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::handleLocalEvents): If we're not in the capture phase, and the target is a different form, and we're handling the submitEvent or the resetEvent, then stop propagation of the event. This matches Firefox behavior. You can only get in this situation if misnested tags cause forms to be nested.
  • html/HTMLFormElement.h: Added handleLocalEvents.
5:32 PM Changeset in webkit [19871] by darin
  • 2 edits in trunk/WebCore

Reviewed by Oliver Hunt.

  • <rdar://problem/5021555> TextCodecICU does not use fallback mappings; it should
  • platform/TextCodecICU.cpp: (WebCore::TextCodecICU::createICUConverter): Added a call to ucnv_setFallback(TRUE).
5:16 PM Changeset in webkit [19870] by oliver
  • 3 edits
    4 adds in trunk

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

Reviewed by Adam.

Fix for rdar://problem/5021127

Need to add null check to node -- original Obj-C didn't need
the null check due to the Obj-C null messaging semantics

  • editing/Editor.cpp: (WebCore::Editor::shouldInsertFragment):

Layout tests:

Layout test for rdar://problem/5021127

  • fast/events/standalone-image-drag-to-editable-expected.checksum: Added.
  • fast/events/standalone-image-drag-to-editable-expected.png: Added.
  • fast/events/standalone-image-drag-to-editable-expected.txt: Added.
  • fast/events/standalone-image-drag-to-editable.html: Added.
4:39 PM Changeset in webkit [23001] by aroben
  • 4 edits in branches/WindowsMerge

WebCoreWin:

Reviewed by Adele.

  • platform/win/GraphicsContextWin.cpp: Get rid of so-five-hours-ago SVG_SUPPORT.

WebKitWin:

Reviewed by Adele.

Stop bizarre crashes from happening on startup.

  • WebKit.vcproj/WebKit.vcproj: Use new ENABLE_* macros instead of *_SUPPORT.
4:33 PM Changeset in webkit [19869] by sullivan
  • 2 edits in trunk/WebKit

Reviewed by Darin and Geoff

  • WebView/WebHTMLView.mm: (coreGraphicsScreenPointForAppKitScreenPoint): This method was copied from WebBrowser, and it was wrong. Fixed it. This only affects the Dictionary pop-up panel.
4:09 PM Changeset in webkit [19868] by hyatt
  • 2 edits in trunk/WebCore

Make text files render using white-space: pre-wrap instead of white-space: pre.


Reviewed by aroben

  • loader/TextDocument.cpp: (WebCore::TextTokenizer::write):
3:53 PM Changeset in webkit [23000] by hyatt
  • 1 edit in branches/WindowsMerge/WebCore/platform/win/ScrollViewWin.cpp

Fix Boomer.

3:24 PM Changeset in webkit [19867] by hyatt
  • 3 edits in trunk/LayoutTests/fast/overflow

Update expected results for overflow test.

3:22 PM Changeset in webkit [19866] by hyatt
  • 7 edits in trunk/WebCore

Back out the change to add fullyClippedContentRect. The layout test
was actually showing more correct results.

  • page/Frame.cpp: (WebCore::Frame::visibleSelectionRect): (WebCore::Frame::setIsActive): (WebCore::Frame::markAllMatchesForText):
  • platform/ScrollView.h:
  • platform/mac/ScrollViewMac.mm:
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::scrollRectToVisible):
3:14 PM Changeset in webkit [22999] by aroben
  • 3 edits in branches/WindowsMerge/WebCore

Build fix.

  • config.h: Remove redundant #define.
  • WebCore.vcproj/WebCore.vcproj: Update to new macros.
2:55 PM Changeset in webkit [19865] by hyatt
  • 2 edits in trunk/WebKit

Update web inspector to account for border-fit.

Reviewed by darin

2:43 PM Changeset in webkit [19864] by ggaren
  • 4 edits in trunk/WebKit

Reviewed by Darin Adler.


Re-arranged things to put deprecated methods at the bottom.

  • Misc/WebCoreStatistics.h:
  • Misc/WebCoreStatistics.mm: (+[WebCoreStatistics javaScriptNoGCAllowedObjectsCount]): (+[WebCoreStatistics javaScriptReferencedObjectsCount]): (+[WebCoreStatistics javaScriptRootObjectClasses]):
  • WebKit.xcodeproj/project.pbxproj:
2:42 PM Changeset in webkit [19863] by hyatt
  • 1 edit
    8 adds in trunk/LayoutTests

Land layout test demonstrating border-fit.

Reviewed by darin

  • fast/borders/border-fit-expected.checksum: Added.
  • fast/borders/border-fit-expected.png: Added.
  • fast/borders/border-fit-expected.txt: Added.
  • fast/borders/border-fit.html: Added.
  • fast/borders/resources/Balloon_12641896.tiff: Added.
  • fast/borders/resources/Balloon_8107502.tiff: Added.
  • fast/borders/resources/largePerson.png: Added.
2:40 PM Changeset in webkit [22998] by aroben
  • 2 edits in branches/WindowsMerge/WebCore

Rubberstamped by Ada.

  • platform/graphics/svg/cg/SVGResourceFilterCg.cpp: Make #if match the header file.
2:36 PM Changeset in webkit [19862] by hyatt
  • 12 edits in trunk/WebCore

Add support for a new property called -webkit-border-fit. This property
has two values: border and lines. The "border" value means the border and
background paint normally at the border box level. The "lines" value indicates
that the border and background should shrink dynamically to snugly hug the
line boxes of normal flow descendants.

Reviewed by darin

fast/borders/border-fit.html

  • WebCore.xcodeproj/project.pbxproj:
  • css/CSSComputedStyleDeclaration.cpp: (WebCore::): (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
  • css/CSSPropertyNames.in:
  • css/CSSValueKeywords.in:
  • css/cssparser.cpp: (WebCore::CSSParser::parseValue):
  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::applyProperty):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::adjustForBorderFit): (WebCore::RenderBlock::borderFitAdjust):
  • rendering/RenderBlock.h:
  • rendering/RenderBox.cpp: (WebCore::RenderBox::paintBoxDecorations):
  • rendering/RenderBox.h: (WebCore::RenderBox::borderFitAdjust):
  • rendering/RenderStyle.cpp: (WebCore::StyleRareNonInheritedData::StyleRareNonInheritedData): (WebCore::StyleRareNonInheritedData::operator==): (WebCore::RenderStyle::diff):
  • rendering/RenderStyle.h: (WebCore::): (WebCore::RenderStyle::borderFit): (WebCore::RenderStyle::setBorderFit): (WebCore::RenderStyle::initialBorderFit):
2:33 PM Changeset in webkit [22997] by aroben
  • 4 edits in branches/WindowsMerge/WebCore

Reviewed by Darin.

Build fix.

  • WebCore.vcproj/build-generated-files.sh: Updated FEATURE_DEFINES.
  • platform/graphics/svg/cg/SVGResourceFilterCg.cpp: Use shiny new ENABLE(SVG).
  • platform/graphics/svg/cg/SVGResourceMaskerCg.cpp: Ditto.
2:32 PM Changeset in webkit [19861] by mjs
  • 3 edits in trunk/WebKit

Rubber stamped by Mitz.


  • fix layout tests by fixing discrepancy in feature macros.
  • WebInspector/WebInspector.m: (-_highlightNode:):
  • WebKit.xcodeproj/project.pbxproj:
2:30 PM Changeset in webkit [19860] by aroben
  • 2 edits in trunk/WebCore

Reviewed by Darin.

Build fix.

  • rendering/RenderTreeAsText.cpp: Can't use #ifndef with ENABLE() macro.
1:49 PM Changeset in webkit [19859] by adele
  • 3 edits
    2 adds in trunk

LayoutTests:

Reviewed by Adam.

Test for <rdar://problem/4990700> Safari always crashes when attempting to edit/view
Yahoo pipes in WebCore::HTMLSelectElement::optionToListIndex

  • fast/forms/select-out-of-bounds-index-expected.txt: Added.
  • fast/forms/select-out-of-bounds-index.html: Added.

WebCore:

Reviewed by Adam.

Fix for <rdar://problem/4990700> Safari always crashes when attempting to edit/view
Yahoo pipes in WebCore::HTMLSelectElement::optionToListIndex

Test: fast/forms/select-out-of-bounds-index.html

  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::setSelectedIndex): If we're about to deselect all options, then set m_lastOnChangeIndex to -1. (WebCore::HTMLSelectElement::optionToListIndex): Moved listSize to a local variable. Rewrote using a simpler for-loop to prevent out-of-bounds errors.
1:11 PM Changeset in webkit [19858] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Geoff.

<rdar://problem/4816376>
REGRESSION: NetNewsWire 3.0 - Crashes in WebDocumentLoaderMac::attachToFrame() (12674)


  • loader/FrameLoader.cpp: (WebCore::FrameLoader::reload): Remove call to setPolicyDocumentLoader here, it's set by the ::load function that we end up calling.
12:49 PM Changeset in webkit [19857] by brmorris
  • 2 edits in S60/branches/3.1m/WebCore

vbradley, reviewed by Zalan

DESC: [S60] MLIO-6YCACU: Browser freeze and panic in symbianoggplay.sourceforge.com
​http://bugs.webkit.org/show_bug.cgi?id=12831

12:49 PM Changeset in webkit [19856] by brmorris
  • 2 edits in S60/trunk/WebCore

vbradley, reviewed by Zalan

DESC: [S60] MLIO-6YCACU: Browser freeze and panic in symbianoggplay.sourceforge.com
​http://bugs.webkit.org/show_bug.cgi?id=12831

11:29 AM Changeset in webkit [19855] by mjs
  • 676 edits in trunk

JavaScriptCore:

Reviewed by Lars.


  • <rdar://problem/5021698> Disable experimental SVG features (12883)
  • wtf/Platform.h: Add ENABLE() macro similar to HAVE() and USE(), to allow nicer handling of optional WebKit features.

WebCore:

Reviewed by Lars.

  • fixed <rdar://problem/5021698> Disable experimental SVG features (12883)


I added an ENABLE_SVG_EXPERIMENTAL_FEATURES define to guard all use of the experimental
features, and used it to guard relevant tag names, elements, JS bindings and renderers.


I also converted all the existing optional feature defines to
ENABLE_FOO instead of FOO_SUPPORT since that is the more standard
way to do it in open source projects and works with the handy new
ENABLE() macro.

  • CMakeLists.txt:
  • DerivedSources.make:
  • Projects/gdk/webcore-gdk.bkl:
  • WebCore.pro:
  • WebCore.vcproj/WebCore/WebCore.vcproj:
  • WebCore.vcproj/WebCore/build-generated-files.sh:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSCustomXPathNSResolver.cpp:
  • bindings/js/JSCustomXPathNSResolver.h:
  • bindings/js/JSSVGElementWrapperFactory.cpp:
  • bindings/js/JSSVGElementWrapperFactory.h:
  • bindings/js/JSSVGMatrixCustom.cpp:
  • bindings/js/JSSVGPODTypeWrapper.h:
  • bindings/js/JSSVGPathSegCustom.cpp:
  • bindings/js/JSSVGPathSegListCustom.cpp:
  • bindings/js/JSSVGPointListCustom.cpp:
  • bindings/js/JSXSLTProcessor.cpp:
  • bindings/js/JSXSLTProcessor.h:
  • bindings/js/kjs_binding.cpp: (KJS::setDOMException):
  • bindings/js/kjs_css.cpp: (KJS::toJS):
  • bindings/js/kjs_dom.cpp: (KJS::toJS):
  • bindings/js/kjs_html.cpp: (KJS::HTMLElementFunction::callAsFunction):
  • bindings/js/kjs_proxy.cpp:
  • bindings/js/kjs_proxy.h:
  • bindings/js/kjs_window.cpp: (KJS::Window::getValueProperty):
  • bindings/objc/DOM.mm: (WebCore::createElementClassMap): (+[DOMNode _nodeWith:WebCore::]):
  • bindings/objc/DOMCSS.mm: (+[DOMCSSValue _CSSValueWith:WebCore::]):
  • bindings/objc/DOMCustomXPathNSResolver.h:
  • bindings/objc/DOMCustomXPathNSResolver.mm:
  • bindings/objc/DOMEvents.mm: (+[DOMEvent _eventWith:WebCore::]):
  • bindings/objc/DOMInternal.h:
  • bindings/objc/DOMSVGPathSegInternal.mm:
  • bindings/objc/DOMXPath.mm:
  • bindings/objc/ExceptionHandlers.mm: (WebCore::raiseDOMException):
  • bindings/objc/PublicDOMInterfaces.h:
  • bindings/scripts/CodeGeneratorJS.pm:
  • bindings/scripts/CodeGeneratorObjC.pm:
  • config.h:
  • css/CSSGrammar.y:
  • css/CSSStyleDeclaration.cpp: (WebCore::propertyID):
  • css/StyleBase.h:
  • css/cssparser.cpp: (WebCore::CSSParser::parseValue):
  • css/cssparser.h:
  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::loadDefaultStyle): (WebCore::CSSStyleSelector::initForStyleResolve): (WebCore::checkPseudoState): (WebCore::CSSStyleSelector::adjustRenderStyle): (WebCore::CSSStyleSelector::applyProperty):
  • css/cssstyleselector.h:
  • dom/DOMImplementation.cpp: (WebCore::DOMImplementation::hasFeature): (WebCore::DOMImplementation::createDocument):
  • dom/Document.cpp: (WebCore::Document::Document): (WebCore::Document::~Document): (WebCore::Document::createElement): (WebCore::Document::implicitClose): (WebCore::Document::recalcStyleSelector): (WebCore::Document::createEvent):
  • dom/Document.h:
  • dom/Document.idl:
  • dom/Event.cpp:
  • dom/Event.h:
  • dom/EventTarget.cpp:
  • dom/EventTarget.h:
  • dom/MappedAttributeEntry.h: (WebCore::):
  • dom/Node.cpp: (WebCore::Node::createRendererIfNeeded): (WebCore::Node::shadowAncestorNode):
  • dom/Node.h:
  • dom/ProcessingInstruction.cpp: (WebCore::ProcessingInstruction::ProcessingInstruction): (WebCore::ProcessingInstruction::checkStyleSheet): (WebCore::ProcessingInstruction::setCSSStyleSheet):
  • dom/ProcessingInstruction.h:
  • dom/Text.cpp: (WebCore::Text::createRenderer):
  • dom/XMLTokenizer.cpp: (WebCore::XMLTokenizer::startElementNs): (WebCore::XMLTokenizer::endElementNs): (WebCore::XMLTokenizer::processingInstruction): (WebCore::XMLTokenizer::end): (WebCore::XMLTokenizer::insertErrorMessageBlock):
  • dom/XMLTokenizer.h:
  • html/HTMLEmbedElement.cpp:
  • html/HTMLEmbedElement.h:
  • html/HTMLObjectElement.cpp:
  • html/HTMLObjectElement.h:
  • ksvg2/css/SVGCSSParser.cpp:
  • ksvg2/css/SVGCSSStyleSelector.cpp:
  • ksvg2/css/SVGRenderStyle.cpp:
  • ksvg2/css/SVGRenderStyle.h:
  • ksvg2/css/SVGRenderStyleDefs.cpp:
  • ksvg2/css/SVGRenderStyleDefs.h:
  • ksvg2/events/JSSVGLazyEventListener.cpp:
  • ksvg2/events/JSSVGLazyEventListener.h:
  • ksvg2/events/SVGZoomEvent.cpp:
  • ksvg2/events/SVGZoomEvent.h:
  • ksvg2/misc/KCanvasRenderingStyle.cpp:
  • ksvg2/misc/KCanvasRenderingStyle.h:
  • ksvg2/misc/PointerEventsHitRules.cpp:
  • ksvg2/misc/PointerEventsHitRules.h:
  • ksvg2/misc/SVGDocumentExtensions.cpp:
  • ksvg2/misc/SVGDocumentExtensions.h:
  • ksvg2/misc/SVGImageLoader.cpp:
  • ksvg2/misc/SVGImageLoader.h:
  • ksvg2/misc/SVGTimer.cpp:
  • ksvg2/misc/SVGTimer.h:
  • ksvg2/misc/TimeScheduler.cpp:
  • ksvg2/misc/TimeScheduler.h:
  • ksvg2/scripts/make_names.pl:
  • ksvg2/svg/ColorDistance.cpp:
  • ksvg2/svg/ColorDistance.h:
  • ksvg2/svg/GradientAttributes.h:
  • ksvg2/svg/LinearGradientAttributes.h:
  • ksvg2/svg/PatternAttributes.h:
  • ksvg2/svg/RadialGradientAttributes.h:
  • ksvg2/svg/SVGAElement.cpp:
  • ksvg2/svg/SVGAElement.h:
  • ksvg2/svg/SVGAngle.cpp:
  • ksvg2/svg/SVGAngle.h:
  • ksvg2/svg/SVGAnimateColorElement.cpp:
  • ksvg2/svg/SVGAnimateColorElement.h:
  • ksvg2/svg/SVGAnimateElement.cpp:
  • ksvg2/svg/SVGAnimateElement.h:
  • ksvg2/svg/SVGAnimateElement.idl:
  • ksvg2/svg/SVGAnimateMotionElement.cpp:
  • ksvg2/svg/SVGAnimateMotionElement.h:
  • ksvg2/svg/SVGAnimateTransformElement.cpp:
  • ksvg2/svg/SVGAnimateTransformElement.h:
  • ksvg2/svg/SVGAnimateTransformElement.idl:
  • ksvg2/svg/SVGAnimatedPathData.cpp:
  • ksvg2/svg/SVGAnimatedPathData.h:
  • ksvg2/svg/SVGAnimatedPoints.cpp:
  • ksvg2/svg/SVGAnimatedPoints.h:
  • ksvg2/svg/SVGAnimatedTemplate.h:
  • ksvg2/svg/SVGAnimationElement.cpp:
  • ksvg2/svg/SVGAnimationElement.h:
  • ksvg2/svg/SVGCircleElement.cpp:
  • ksvg2/svg/SVGCircleElement.h:
  • ksvg2/svg/SVGClipPathElement.cpp:
  • ksvg2/svg/SVGClipPathElement.h:
  • ksvg2/svg/SVGColor.cpp:
  • ksvg2/svg/SVGColor.h:
  • ksvg2/svg/SVGComponentTransferFunctionElement.cpp:
  • ksvg2/svg/SVGComponentTransferFunctionElement.h:
  • ksvg2/svg/SVGCursorElement.cpp:
  • ksvg2/svg/SVGCursorElement.h:
  • ksvg2/svg/SVGDefsElement.cpp:
  • ksvg2/svg/SVGDefsElement.h:
  • ksvg2/svg/SVGDescElement.cpp:
  • ksvg2/svg/SVGDescElement.h:
  • ksvg2/svg/SVGDocument.cpp:
  • ksvg2/svg/SVGDocument.h:
  • ksvg2/svg/SVGElement.cpp: (WebCore::SVGElement::dispatchEvent):
  • ksvg2/svg/SVGElement.h:
  • ksvg2/svg/SVGElementInstance.cpp:
  • ksvg2/svg/SVGElementInstance.h:
  • ksvg2/svg/SVGElementInstance.idl:
  • ksvg2/svg/SVGElementInstanceList.cpp:
  • ksvg2/svg/SVGElementInstanceList.h:
  • ksvg2/svg/SVGElementInstanceList.idl:
  • ksvg2/svg/SVGEllipseElement.cpp:
  • ksvg2/svg/SVGEllipseElement.h:
  • ksvg2/svg/SVGException.h:
  • ksvg2/svg/SVGExternalResourcesRequired.cpp:
  • ksvg2/svg/SVGExternalResourcesRequired.h:
  • ksvg2/svg/SVGFEBlendElement.cpp:
  • ksvg2/svg/SVGFEBlendElement.h:
  • ksvg2/svg/SVGFEBlendElement.idl:
  • ksvg2/svg/SVGFEColorMatrixElement.cpp:
  • ksvg2/svg/SVGFEColorMatrixElement.h:
  • ksvg2/svg/SVGFEColorMatrixElement.idl:
  • ksvg2/svg/SVGFEComponentTransferElement.cpp:
  • ksvg2/svg/SVGFEComponentTransferElement.h:
  • ksvg2/svg/SVGFEComponentTransferElement.idl:
  • ksvg2/svg/SVGFECompositeElement.cpp:
  • ksvg2/svg/SVGFECompositeElement.h:
  • ksvg2/svg/SVGFECompositeElement.idl:
  • ksvg2/svg/SVGFEDiffuseLightingElement.cpp:
  • ksvg2/svg/SVGFEDiffuseLightingElement.h:
  • ksvg2/svg/SVGFEDiffuseLightingElement.idl:
  • ksvg2/svg/SVGFEDisplacementMapElement.cpp:
  • ksvg2/svg/SVGFEDisplacementMapElement.h:
  • ksvg2/svg/SVGFEDisplacementMapElement.idl:
  • ksvg2/svg/SVGFEDistantLightElement.cpp:
  • ksvg2/svg/SVGFEDistantLightElement.h:
  • ksvg2/svg/SVGFEDistantLightElement.idl:
  • ksvg2/svg/SVGFEFloodElement.cpp:
  • ksvg2/svg/SVGFEFloodElement.h:
  • ksvg2/svg/SVGFEFloodElement.idl:
  • ksvg2/svg/SVGFEFuncAElement.cpp:
  • ksvg2/svg/SVGFEFuncAElement.h:
  • ksvg2/svg/SVGFEFuncAElement.idl:
  • ksvg2/svg/SVGFEFuncBElement.cpp:
  • ksvg2/svg/SVGFEFuncBElement.h:
  • ksvg2/svg/SVGFEFuncBElement.idl:
  • ksvg2/svg/SVGFEFuncGElement.cpp:
  • ksvg2/svg/SVGFEFuncGElement.h:
  • ksvg2/svg/SVGFEFuncGElement.idl:
  • ksvg2/svg/SVGFEFuncRElement.cpp:
  • ksvg2/svg/SVGFEFuncRElement.h:
  • ksvg2/svg/SVGFEFuncRElement.idl:
  • ksvg2/svg/SVGFEGaussianBlurElement.cpp:
  • ksvg2/svg/SVGFEGaussianBlurElement.h:
  • ksvg2/svg/SVGFEGaussianBlurElement.idl:
  • ksvg2/svg/SVGFEImageElement.cpp:
  • ksvg2/svg/SVGFEImageElement.h:
  • ksvg2/svg/SVGFEImageElement.idl:
  • ksvg2/svg/SVGFELightElement.cpp:
  • ksvg2/svg/SVGFELightElement.h:
  • ksvg2/svg/SVGFEMergeElement.cpp:
  • ksvg2/svg/SVGFEMergeElement.h:
  • ksvg2/svg/SVGFEMergeElement.idl:
  • ksvg2/svg/SVGFEMergeNodeElement.cpp:
  • ksvg2/svg/SVGFEMergeNodeElement.h:
  • ksvg2/svg/SVGFEMergeNodeElement.idl:
  • ksvg2/svg/SVGFEOffsetElement.cpp:
  • ksvg2/svg/SVGFEOffsetElement.h:
  • ksvg2/svg/SVGFEOffsetElement.idl:
  • ksvg2/svg/SVGFEPointLightElement.cpp:
  • ksvg2/svg/SVGFEPointLightElement.h:
  • ksvg2/svg/SVGFEPointLightElement.idl:
  • ksvg2/svg/SVGFESpecularLightingElement.cpp:
  • ksvg2/svg/SVGFESpecularLightingElement.h:
  • ksvg2/svg/SVGFESpecularLightingElement.idl:
  • ksvg2/svg/SVGFESpotLightElement.cpp:
  • ksvg2/svg/SVGFESpotLightElement.h:
  • ksvg2/svg/SVGFESpotLightElement.idl:
  • ksvg2/svg/SVGFETileElement.cpp:
  • ksvg2/svg/SVGFETileElement.h:
  • ksvg2/svg/SVGFETileElement.idl:
  • ksvg2/svg/SVGFETurbulenceElement.cpp:
  • ksvg2/svg/SVGFETurbulenceElement.h:
  • ksvg2/svg/SVGFETurbulenceElement.idl:
  • ksvg2/svg/SVGFilterElement.cpp:
  • ksvg2/svg/SVGFilterElement.h:
  • ksvg2/svg/SVGFilterElement.idl:
  • ksvg2/svg/SVGFilterPrimitiveStandardAttributes.cpp:
  • ksvg2/svg/SVGFilterPrimitiveStandardAttributes.h:
  • ksvg2/svg/SVGFitToViewBox.cpp:
  • ksvg2/svg/SVGFitToViewBox.h:
  • ksvg2/svg/SVGForeignObjectElement.cpp:
  • ksvg2/svg/SVGForeignObjectElement.h:
  • ksvg2/svg/SVGForeignObjectElement.idl:
  • ksvg2/svg/SVGGElement.cpp:
  • ksvg2/svg/SVGGElement.h:
  • ksvg2/svg/SVGGradientElement.cpp:
  • ksvg2/svg/SVGGradientElement.h:
  • ksvg2/svg/SVGImageElement.cpp:
  • ksvg2/svg/SVGImageElement.h:
  • ksvg2/svg/SVGLangSpace.cpp:
  • ksvg2/svg/SVGLangSpace.h:
  • ksvg2/svg/SVGLength.cpp:
  • ksvg2/svg/SVGLength.h:
  • ksvg2/svg/SVGLengthList.cpp:
  • ksvg2/svg/SVGLengthList.h:
  • ksvg2/svg/SVGLineElement.cpp:
  • ksvg2/svg/SVGLineElement.h:
  • ksvg2/svg/SVGLinearGradientElement.cpp:
  • ksvg2/svg/SVGLinearGradientElement.h:
  • ksvg2/svg/SVGList.h:
  • ksvg2/svg/SVGListTraits.h:
  • ksvg2/svg/SVGLocatable.cpp: (WebCore::SVGLocatable::nearestViewportElement): (WebCore::SVGLocatable::farthestViewportElement):
  • ksvg2/svg/SVGLocatable.h:
  • ksvg2/svg/SVGMPathElement.cpp:
  • ksvg2/svg/SVGMPathElement.h:
  • ksvg2/svg/SVGMarkerElement.cpp:
  • ksvg2/svg/SVGMarkerElement.h:
  • ksvg2/svg/SVGMaskElement.cpp:
  • ksvg2/svg/SVGMaskElement.h:
  • ksvg2/svg/SVGMetadataElement.cpp:
  • ksvg2/svg/SVGMetadataElement.h:
  • ksvg2/svg/SVGNumberList.cpp:
  • ksvg2/svg/SVGNumberList.h:
  • ksvg2/svg/SVGPaint.cpp:
  • ksvg2/svg/SVGPaint.h:
  • ksvg2/svg/SVGParserUtilities.cpp:
  • ksvg2/svg/SVGParserUtilities.h:
  • ksvg2/svg/SVGPathElement.cpp:
  • ksvg2/svg/SVGPathElement.h:
  • ksvg2/svg/SVGPathSeg.h:
  • ksvg2/svg/SVGPathSegArc.cpp:
  • ksvg2/svg/SVGPathSegArc.h:
  • ksvg2/svg/SVGPathSegClosePath.cpp:
  • ksvg2/svg/SVGPathSegClosePath.h:
  • ksvg2/svg/SVGPathSegCurvetoCubic.cpp:
  • ksvg2/svg/SVGPathSegCurvetoCubic.h:
  • ksvg2/svg/SVGPathSegCurvetoCubicSmooth.cpp:
  • ksvg2/svg/SVGPathSegCurvetoCubicSmooth.h:
  • ksvg2/svg/SVGPathSegCurvetoQuadratic.cpp:
  • ksvg2/svg/SVGPathSegCurvetoQuadratic.h:
  • ksvg2/svg/SVGPathSegCurvetoQuadraticSmooth.cpp:
  • ksvg2/svg/SVGPathSegCurvetoQuadraticSmooth.h:
  • ksvg2/svg/SVGPathSegLineto.cpp:
  • ksvg2/svg/SVGPathSegLineto.h:
  • ksvg2/svg/SVGPathSegLinetoHorizontal.cpp:
  • ksvg2/svg/SVGPathSegLinetoHorizontal.h:
  • ksvg2/svg/SVGPathSegLinetoVertical.cpp:
  • ksvg2/svg/SVGPathSegLinetoVertical.h:
  • ksvg2/svg/SVGPathSegList.cpp:
  • ksvg2/svg/SVGPathSegList.h:
  • ksvg2/svg/SVGPathSegMoveto.cpp:
  • ksvg2/svg/SVGPathSegMoveto.h:
  • ksvg2/svg/SVGPatternElement.cpp:
  • ksvg2/svg/SVGPatternElement.h:
  • ksvg2/svg/SVGPointList.cpp:
  • ksvg2/svg/SVGPointList.h:
  • ksvg2/svg/SVGPolyElement.cpp:
  • ksvg2/svg/SVGPolyElement.h:
  • ksvg2/svg/SVGPolygonElement.cpp:
  • ksvg2/svg/SVGPolygonElement.h:
  • ksvg2/svg/SVGPolylineElement.cpp:
  • ksvg2/svg/SVGPolylineElement.h:
  • ksvg2/svg/SVGPreserveAspectRatio.cpp:
  • ksvg2/svg/SVGPreserveAspectRatio.h:
  • ksvg2/svg/SVGRadialGradientElement.cpp:
  • ksvg2/svg/SVGRadialGradientElement.h:
  • ksvg2/svg/SVGRectElement.cpp:
  • ksvg2/svg/SVGRectElement.h:
  • ksvg2/svg/SVGRenderingIntent.h:
  • ksvg2/svg/SVGSVGElement.cpp:
  • ksvg2/svg/SVGSVGElement.h:
  • ksvg2/svg/SVGScriptElement.cpp:
  • ksvg2/svg/SVGScriptElement.h:
  • ksvg2/svg/SVGSetElement.cpp:
  • ksvg2/svg/SVGSetElement.h:
  • ksvg2/svg/SVGStopElement.cpp:
  • ksvg2/svg/SVGStopElement.h:
  • ksvg2/svg/SVGStringList.cpp:
  • ksvg2/svg/SVGStringList.h:
  • ksvg2/svg/SVGStylable.cpp:
  • ksvg2/svg/SVGStylable.h:
  • ksvg2/svg/SVGStyleElement.cpp:
  • ksvg2/svg/SVGStyleElement.h:
  • ksvg2/svg/SVGStyledElement.cpp: (WebCore::SVGStyledElement::notifyAttributeChange):
  • ksvg2/svg/SVGStyledElement.h:
  • ksvg2/svg/SVGStyledLocatableElement.cpp:
  • ksvg2/svg/SVGStyledLocatableElement.h:
  • ksvg2/svg/SVGStyledTransformableElement.cpp:
  • ksvg2/svg/SVGStyledTransformableElement.h:
  • ksvg2/svg/SVGSwitchElement.cpp:
  • ksvg2/svg/SVGSwitchElement.h:
  • ksvg2/svg/SVGSymbolElement.cpp:
  • ksvg2/svg/SVGSymbolElement.h:
  • ksvg2/svg/SVGTRefElement.cpp:
  • ksvg2/svg/SVGTRefElement.h:
  • ksvg2/svg/SVGTSpanElement.cpp:
  • ksvg2/svg/SVGTSpanElement.h:
  • ksvg2/svg/SVGTests.cpp:
  • ksvg2/svg/SVGTests.h:
  • ksvg2/svg/SVGTextContentElement.cpp:
  • ksvg2/svg/SVGTextContentElement.h:
  • ksvg2/svg/SVGTextElement.cpp:
  • ksvg2/svg/SVGTextElement.h:
  • ksvg2/svg/SVGTextPositioningElement.cpp:
  • ksvg2/svg/SVGTextPositioningElement.h:
  • ksvg2/svg/SVGTitleElement.cpp:
  • ksvg2/svg/SVGTitleElement.h:
  • ksvg2/svg/SVGTransform.cpp:
  • ksvg2/svg/SVGTransform.h:
  • ksvg2/svg/SVGTransformDistance.cpp:
  • ksvg2/svg/SVGTransformDistance.h:
  • ksvg2/svg/SVGTransformList.cpp:
  • ksvg2/svg/SVGTransformList.h:
  • ksvg2/svg/SVGTransformable.cpp:
  • ksvg2/svg/SVGTransformable.h:
  • ksvg2/svg/SVGURIReference.cpp:
  • ksvg2/svg/SVGURIReference.h:
  • ksvg2/svg/SVGUnitTypes.h:
  • ksvg2/svg/SVGUseElement.cpp:
  • ksvg2/svg/SVGUseElement.h:
  • ksvg2/svg/SVGUseElement.idl:
  • ksvg2/svg/SVGViewElement.cpp:
  • ksvg2/svg/SVGViewElement.h:
  • ksvg2/svg/SVGZoomAndPan.cpp:
  • ksvg2/svg/SVGZoomAndPan.h:
  • ksvg2/svg/svgtags.in:
  • loader/Cache.cpp: (WebCore::createResource): (WebCore::Cache::getStatistics):
  • loader/Cache.h:
  • loader/CachedImage.cpp: (WebCore::CachedImage::createImage):
  • loader/CachedResource.h: (WebCore::CachedResource::):
  • loader/CachedResourceClient.h:
  • loader/CachedXBLDocument.cpp:
  • loader/CachedXBLDocument.h:
  • loader/CachedXSLStyleSheet.cpp:
  • loader/CachedXSLStyleSheet.h:
  • loader/DocLoader.cpp:
  • loader/DocLoader.h:
  • page/DOMWindow.idl:
  • page/EventHandler.cpp: (WebCore::EventHandler::handleMouseMoveEvent): (WebCore::selectCursor):
  • page/Frame.cpp: (WebCore::Frame::Frame):
  • page/PageState.cpp: (WebCore::PageState::PageState): (WebCore::PageState::restore):
  • platform/MimeTypeRegistry.cpp: (WebCore::initialiseSupportedNonImageMimeTypes):
  • platform/graphics/FloatPoint3D.cpp:
  • platform/graphics/FloatPoint3D.h:
  • platform/graphics/ImageBuffer.cpp: (WebCore::ImageBuffer::renderSubtreeToImage):
  • platform/graphics/svg/SVGImage.cpp:
  • platform/graphics/svg/SVGImage.h:
  • platform/graphics/svg/SVGImageEmptyClients.h:
  • platform/graphics/svg/SVGPaintServer.cpp:
  • platform/graphics/svg/SVGPaintServer.h:
  • platform/graphics/svg/SVGPaintServerGradient.cpp:
  • platform/graphics/svg/SVGPaintServerGradient.h:
  • platform/graphics/svg/SVGPaintServerLinearGradient.cpp:
  • platform/graphics/svg/SVGPaintServerLinearGradient.h:
  • platform/graphics/svg/SVGPaintServerPattern.cpp:
  • platform/graphics/svg/SVGPaintServerPattern.h:
  • platform/graphics/svg/SVGPaintServerRadialGradient.cpp:
  • platform/graphics/svg/SVGPaintServerRadialGradient.h:
  • platform/graphics/svg/SVGPaintServerSolid.cpp:
  • platform/graphics/svg/SVGPaintServerSolid.h:
  • platform/graphics/svg/SVGResource.cpp:
  • platform/graphics/svg/SVGResource.h:
  • platform/graphics/svg/SVGResourceClipper.cpp:
  • platform/graphics/svg/SVGResourceClipper.h:
  • platform/graphics/svg/SVGResourceFilter.cpp:
  • platform/graphics/svg/SVGResourceFilter.h:
  • platform/graphics/svg/SVGResourceMarker.cpp:
  • platform/graphics/svg/SVGResourceMarker.h:
  • platform/graphics/svg/SVGResourceMasker.cpp:
  • platform/graphics/svg/SVGResourceMasker.h:
  • platform/graphics/svg/cg/CgSupport.cpp:
  • platform/graphics/svg/cg/CgSupport.h:
  • platform/graphics/svg/cg/RenderPathCg.cpp:
  • platform/graphics/svg/cg/SVGPaintServerCg.cpp:
  • platform/graphics/svg/cg/SVGPaintServerGradientCg.cpp:
  • platform/graphics/svg/cg/SVGPaintServerPatternCg.cpp:
  • platform/graphics/svg/cg/SVGPaintServerSolidCg.cpp:
  • platform/graphics/svg/cg/SVGResourceClipperCg.cpp:
  • platform/graphics/svg/cg/SVGResourceFilterCg.mm:
  • platform/graphics/svg/cg/SVGResourceMaskerCg.mm:
  • platform/graphics/svg/filters/SVGDistantLightSource.h:
  • platform/graphics/svg/filters/SVGFEBlend.cpp:
  • platform/graphics/svg/filters/SVGFEBlend.h:
  • platform/graphics/svg/filters/SVGFEColorMatrix.cpp:
  • platform/graphics/svg/filters/SVGFEColorMatrix.h:
  • platform/graphics/svg/filters/SVGFEComponentTransfer.cpp:
  • platform/graphics/svg/filters/SVGFEComponentTransfer.h:
  • platform/graphics/svg/filters/SVGFEComposite.cpp:
  • platform/graphics/svg/filters/SVGFEComposite.h:
  • platform/graphics/svg/filters/SVGFEConvolveMatrix.cpp:
  • platform/graphics/svg/filters/SVGFEConvolveMatrix.h:
  • platform/graphics/svg/filters/SVGFEDiffuseLighting.cpp:
  • platform/graphics/svg/filters/SVGFEDiffuseLighting.h:
  • platform/graphics/svg/filters/SVGFEDisplacementMap.cpp:
  • platform/graphics/svg/filters/SVGFEDisplacementMap.h:
  • platform/graphics/svg/filters/SVGFEFlood.cpp:
  • platform/graphics/svg/filters/SVGFEFlood.h:
  • platform/graphics/svg/filters/SVGFEGaussianBlur.cpp:
  • platform/graphics/svg/filters/SVGFEGaussianBlur.h:
  • platform/graphics/svg/filters/SVGFEImage.cpp:
  • platform/graphics/svg/filters/SVGFEImage.h:
  • platform/graphics/svg/filters/SVGFEMerge.cpp:
  • platform/graphics/svg/filters/SVGFEMerge.h:
  • platform/graphics/svg/filters/SVGFEMorphology.cpp:
  • platform/graphics/svg/filters/SVGFEMorphology.h:
  • platform/graphics/svg/filters/SVGFEOffset.cpp:
  • platform/graphics/svg/filters/SVGFEOffset.h:
  • platform/graphics/svg/filters/SVGFESpecularLighting.cpp:
  • platform/graphics/svg/filters/SVGFESpecularLighting.h:
  • platform/graphics/svg/filters/SVGFETile.h:
  • platform/graphics/svg/filters/SVGFETurbulence.cpp:
  • platform/graphics/svg/filters/SVGFETurbulence.h:
  • platform/graphics/svg/filters/SVGFilterEffect.cpp:
  • platform/graphics/svg/filters/SVGFilterEffect.h:
  • platform/graphics/svg/filters/SVGLightSource.cpp:
  • platform/graphics/svg/filters/SVGLightSource.h:
  • platform/graphics/svg/filters/SVGPointLightSource.h:
  • platform/graphics/svg/filters/SVGSpotLightSource.h:
  • platform/graphics/svg/filters/cg/SVGFEBlendCg.mm:
  • platform/graphics/svg/filters/cg/SVGFEColorMatrixCg.mm:
  • platform/graphics/svg/filters/cg/SVGFEComponentTransferCg.mm:
  • platform/graphics/svg/filters/cg/SVGFECompositeCg.mm:
  • platform/graphics/svg/filters/cg/SVGFEDiffuseLightingCg.mm:
  • platform/graphics/svg/filters/cg/SVGFEDisplacementMapCg.mm:
  • platform/graphics/svg/filters/cg/SVGFEFloodCg.mm:
  • platform/graphics/svg/filters/cg/SVGFEGaussianBlurCg.mm:
  • platform/graphics/svg/filters/cg/SVGFEHelpersCg.h:
  • platform/graphics/svg/filters/cg/SVGFEHelpersCg.mm:
  • platform/graphics/svg/filters/cg/SVGFEImageCg.mm:
  • platform/graphics/svg/filters/cg/SVGFEMergeCg.mm:
  • platform/graphics/svg/filters/cg/SVGFEOffsetCg.mm:
  • platform/graphics/svg/filters/cg/SVGFESpecularLightingCg.mm:
  • platform/graphics/svg/filters/cg/SVGFETileCg.mm:
  • platform/graphics/svg/filters/cg/SVGFilterEffectCg.mm:
  • platform/graphics/svg/qt/SVGPaintServerGradientQt.cpp:
  • platform/graphics/svg/qt/SVGPaintServerLinearGradientQt.cpp:
  • platform/graphics/svg/qt/SVGPaintServerPatternQt.cpp:
  • platform/graphics/svg/qt/SVGPaintServerQt.cpp:
  • platform/graphics/svg/qt/SVGPaintServerRadialGradientQt.cpp:
  • platform/graphics/svg/qt/SVGPaintServerSolidQt.cpp:
  • platform/graphics/svg/qt/SVGResourceClipperQt.cpp:
  • platform/graphics/svg/qt/SVGResourceFilterQt.cpp:
  • platform/graphics/svg/qt/SVGResourceMaskerQt.cpp:
  • rendering/HitTestResult.cpp: (WebCore::HitTestResult::absoluteImageURL): (WebCore::HitTestResult::absoluteLinkURL): (WebCore::HitTestResult::isLiveLink):
  • rendering/RenderForeignObject.cpp:
  • rendering/RenderForeignObject.h:
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::isTransparent):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::containingBlock):
  • rendering/RenderObject.h:
  • rendering/RenderPath.cpp:
  • rendering/RenderPath.h:
  • rendering/RenderSVGBlock.cpp:
  • rendering/RenderSVGBlock.h:
  • rendering/RenderSVGContainer.cpp:
  • rendering/RenderSVGContainer.h:
  • rendering/RenderSVGGradientStop.cpp:
  • rendering/RenderSVGGradientStop.h:
  • rendering/RenderSVGHiddenContainer.cpp:
  • rendering/RenderSVGHiddenContainer.h:
  • rendering/RenderSVGImage.cpp:
  • rendering/RenderSVGImage.h:
  • rendering/RenderSVGInline.cpp:
  • rendering/RenderSVGInline.h:
  • rendering/RenderSVGInlineText.cpp:
  • rendering/RenderSVGInlineText.h:
  • rendering/RenderSVGTSpan.cpp:
  • rendering/RenderSVGTSpan.h:
  • rendering/RenderSVGText.cpp:
  • rendering/RenderSVGText.h:
  • rendering/RenderStyle.cpp: (WebCore::StyleRareNonInheritedData::StyleRareNonInheritedData): (WebCore::StyleRareNonInheritedData::~StyleRareNonInheritedData): (WebCore::StyleRareNonInheritedData::operator==): (WebCore::RenderStyle::RenderStyle): (WebCore::RenderStyle::inheritFrom): (WebCore::RenderStyle::operator==): (WebCore::RenderStyle::inheritedNotEqual): (WebCore::RenderStyle::diff):
  • rendering/RenderStyle.h:
  • rendering/RenderText.cpp: (WebCore::RenderText::setStyle): (WebCore::RenderText::setTextInternal):
  • rendering/RenderTreeAsText.cpp: (WebCore::write): (WebCore::externalRepresentation):
  • rendering/SVGInlineFlowBox.cpp:
  • rendering/SVGInlineFlowBox.h:
  • rendering/SVGRenderTreeAsText.cpp:
  • rendering/SVGRenderTreeAsText.h:
  • rendering/SVGRootInlineBox.cpp:
  • rendering/SVGRootInlineBox.h:
  • rendering/bidi.cpp: (WebCore::shouldPreserveNewline): (WebCore::RenderBlock::findNextLineBreak):
  • webcore-base.bkl:
  • xml/NativeXPathNSResolver.cpp:
  • xml/NativeXPathNSResolver.h:
  • xml/XPathEvaluator.cpp:
  • xml/XPathEvaluator.h:
  • xml/XPathExpression.cpp:
  • xml/XPathExpression.h:
  • xml/XPathExpressionNode.cpp:
  • xml/XPathExpressionNode.h:
  • xml/XPathFunctions.cpp:
  • xml/XPathFunctions.h:
  • xml/XPathGrammar.y:
  • xml/XPathNSResolver.cpp:
  • xml/XPathNSResolver.h:
  • xml/XPathNamespace.cpp:
  • xml/XPathNamespace.h:
  • xml/XPathParser.cpp:
  • xml/XPathParser.h:
  • xml/XPathPath.cpp:
  • xml/XPathPath.h:
  • xml/XPathPredicate.cpp:
  • xml/XPathPredicate.h:
  • xml/XPathResult.cpp:
  • xml/XPathResult.h:
  • xml/XPathStep.cpp:
  • xml/XPathStep.h:
  • xml/XPathUtil.cpp:
  • xml/XPathUtil.h:
  • xml/XPathValue.cpp:
  • xml/XPathValue.h:
  • xml/XPathVariableReference.cpp:
  • xml/XPathVariableReference.h:
  • xml/XSLImportRule.cpp:
  • xml/XSLImportRule.h:
  • xml/XSLStyleSheet.cpp:
  • xml/XSLStyleSheet.h:
  • xml/XSLTProcessor.cpp:
  • xml/XSLTProcessor.h:
11:18 AM Changeset in webkit [19854] by brmorris
  • 2 edits in S60/branches/3.1m/WebKit

spadma, reviewed by Yongjun

DESC: Urls are doubly encoded HIIN-6YMGYV

11:16 AM Changeset in webkit [19853] by brmorris
  • 2 edits in S60/trunk/WebKit

spadma, reviewed by Yongjun

DESC: Urls are doubly encoded HIIN-6YMGYV

Feb 25, 2007:

9:31 PM Changeset in webkit [19852] by bdash
  • 3 edits
    4 adds in trunk

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

Reviewed by David Hyatt.

Test: fast/dynamic/float-withdrawal.html

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::layoutBlockChildren): Detect when a child block had intruding floats in a previous layout, and in that case force it to relayout.

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

Reviewed by David Hyatt.

  • fast/dynamic/float-withdrawal-expected.checksum: Added.
  • fast/dynamic/float-withdrawal-expected.png: Added.
  • fast/dynamic/float-withdrawal-expected.txt: Added.
  • fast/dynamic/float-withdrawal.html: Added.
7:00 PM Changeset in webkit [19851] by hyatt
  • 3 edits in trunk/WebCore

Revert my change to the default fill color. Just set the color to transparent explicitly when
drawing textarea resizer frames.

  • platform/graphics/GraphicsContext.cpp: (WebCore::GraphicsContextState::GraphicsContextState):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintOverflowControls):
6:28 PM Changeset in webkit [19850] by weinig
  • 2 edits in trunk/WebCore

Update forgotten files.

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateLayerPosition):
6:07 PM Changeset in webkit [19849] by hyatt
  • 1 edit in trunk/WebCore/page/FrameView.cpp

Forgot to land FrameView.cpp in previous commit.

5:29 PM Changeset in webkit [19848] by hyatt
  • 5 edits in trunk/WebCore

Fix for regression caused by changing the containing block of positioned elements with no positioned ancestor
to be the initial containing block (represented in our tree by the RenderView). Rework RenderView's layout
to have very few special cases. Now it will just relayout its children when the width/height of the
view actually changes. Positioned elements no longer get a special additional layout, since width/height
adjustments are caught by the base class already anyway.

Reviewed by mjs, darin

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::layoutBlock):
  • rendering/RenderFlexibleBox.cpp: (WebCore::RenderFlexibleBox::layoutBlock): (WebCore::RenderFlexibleBox::layoutVerticalBox):
  • rendering/RenderView.cpp: (WebCore::RenderView::RenderView): (WebCore::RenderView::layout):
  • rendering/RenderView.h:
4:47 PM Changeset in webkit [22996] by mjs
  • 2 edits in branches/WindowsMerge/WebCore

Reviewed by Adam.

  • fixed layout test failure in fast/overflow/scrollRevealButton


I added a new ScrollView method, fullyClippedContentRect, that vies the content rect
as clipped by all ancestor scroll views, not just this one. Currently it only works
correctly on mac however. visibleContentRect, only accounting for the clipping from
this particular scroll view, is sufficient for most purposes.

  • platform/win/ScrollViewWin.cpp: (WebCore::ScrolView::fullyClippedContentRect):
4:41 PM Changeset in webkit [19847] by mjs
  • 3 edits in trunk/WebCore

Not reviewed, build fix.

  • fix build breakage
  • platform/gdk/ScrollViewGdk.cpp: (WebCore::ScrollView::fullyClippedContentRect):
  • platform/qt/ScrollViewQt.cpp: (WebCore::ScrollView::fullyClippedContentRect):
3:44 PM Changeset in webkit [19846] by mjs
  • 7 edits in trunk/WebCore

Reviewed by Adam.

  • fixed layout test failure in fast/overflow/scrollRevealButton


I added a new ScrollView method, fullyClippedContentRect, that vies the content rect
as clipped by all ancestor scroll views, not just this one. Currently it only works
correctly on mac however. visibleContentRect, only accounting for the clipping from
this particular scroll view, is sufficient for most purposes.

  • page/Frame.cpp: (WebCore::Frame::visibleSelectionRect): (WebCore::Frame::setIsActive): (WebCore::Frame::markAllMatchesForText):
  • platform/ScrollView.h:
  • platform/gdk/ScrollViewGdk.cpp: (WebCore::ScrolView::fullyClippedContentRect):
  • platform/mac/ScrollViewMac.mm: (WebCore::ScrollView::fullyClippedContentRect):
  • platform/qt/ScrollViewQt.cpp: (WebCore::ScrolView::fullyClippedContentRect):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::scrollRectToVisible):
2:23 PM Changeset in webkit [19845] by andersca
  • 2 edits in trunk/WebKitSite

Reviewed by Maciej.

  • building/checkout.html: Add Trac link to our SVN repository,
Note: See TracTimeline for information about the timeline view.