Timeline


and

07/11/06:

23:04 Changeset [15375] by thatcher

Build fix and versioning.

23:02 Changeset [15374] by thatcher

New good tag.

23:02 Changeset [15373] by thatcher

Bad tag.

23:01 Changeset [15372] by thatcher

new tag (part 2)

23:01 Changeset [15371] by thatcher

New tag.

23:00 Changeset [15370] by thatcher

More build fixes for the deprecated AGL functions.

21:26 Changeset [15369] by thatcher

Forgot to commit a file this morning that actually makes the Drosera console open. Thanks bdash!

20:31 Changeset [15368] by ddkilzer

WebCore:

Windows build fix. Reviewed by NOBODY.

  • WebCore.vcproj/WebCore/WebCore.vcproj: Added JSHTMLOptionsCollection.cpp/h and JSHTMLOptionsCollectionCuston.cpp. VC++ Express realphabetized the file list.
19:55 Changeset [15367] by darin

Reviewed by Adele.

  • html/HTMLOptionElement.cpp: (WebCore::HTMLOptionElement::index): Use a const reference for the list items, so we don't have to copy a vector.
  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::selectedIndex): Ditto. (WebCore::HTMLSelectElement::setSelectedIndex): Ditto. (WebCore::HTMLSelectElement::length): Ditto. (WebCore::HTMLSelectElement::remove): Ditto. (WebCore::HTMLSelectElement::value): Ditto. (WebCore::HTMLSelectElement::setValue): Ditto. (WebCore::HTMLSelectElement::stateValue): Ditto. (WebCore::HTMLSelectElement::restoreState): Ditto. (WebCore::HTMLSelectElement::appendFormData): Ditto. (WebCore::HTMLSelectElement::optionToListIndex): Ditto. (WebCore::HTMLSelectElement::listToOptionIndex): Ditto. (WebCore::HTMLSelectElement::recalcListItems): Made const, with the appropriate fields mutable. (WebCore::HTMLSelectElement::reset): Use a const reference for the list items, so we don't have to copy the vector. Remove the call to setSelectionChanged for the RenderMenuList case. (WebCore::HTMLSelectElement::notifyOptionSelected): Ditto, on both counts. (WebCore::HTMLSelectElement::defaultEventHandler): Call focus() before showing the pop-up.
  • html/HTMLSelectElement.h: The RenderMenuList class is no longer a friend. Changed the listItems function to return a const reference to the vector so it no longer copies the vector. Removed the const_cast to the call to recalcListItems and changed it to a const member function. Made m_recalcListItems mutable.
  • rendering/DeprecatedRenderSelect.cpp: (WebCore::DeprecatedRenderSelect::updateFromElement): Removed an unnecessary call to recalcListItems, which is called automatically. Use a const reference for the list items so we don't have to copy a vector. (WebCore::DeprecatedRenderSelect::layout): Ditto. (WebCore::DeprecatedRenderSelect::selectionChanged): Ditto. (WebCore::DeprecatedRenderSelect::updateSelection): Ditto.
  • rendering/RenderMenuList.cpp: (WebCore::RenderMenuList::RenderMenuList): Updated for renamed data members. (WebCore::RenderMenuList::createInnerBlock): Ditto. (WebCore::RenderMenuList::addChild): Ditto. (WebCore::RenderMenuList::removeChild): Ditto. (WebCore::RenderMenuList::setStyle): Ditto. Also removed code to set the style on the pop-up menu, because it's created with the correct style and destroyed before it a style change could occur. (WebCore::RenderMenuList::updateFromElement): Rearranged code to compute the maximum width in a simpler fashion, and to not bother trying to maintain the "selected" flags on the elements, since the HTMLSelectElement class takes care of that. Store the width as an int. Call setText to set the text based on the selected element's option text. (WebCore::RenderMenuList::paintObject): Don't check m_inner when setting up the clip -- always set up the clip. (WebCore::RenderMenuList::calcMinMaxWidth): Use m_optionsWidth directly instead of calling ceilf on m_longestWidth. (WebCore::RenderMenuList::showPopup): Don't use m_popupMenu to store the menu -- instead keep the pointer in a local variable. Get the selected index from the HTMLSelectElement. (WebCore::RenderMenuList::valueChanged): Call HTMLSelectElement::setSelectedIndex to do most of the work.
  • rendering/RenderMenuList.h: Renamed m_inner to m_innerBlock. Removed m_popupMenu, m_size, m_selectionChanged, and m_selectedIndex. Renamed m_longestWidth to m_optionsWidth and changed it to be an int. Removed unneeded override of removeLeftoverAnonymousBoxes function. Removed unneeded selectionChanged, setSelectionChanged, updateSelection, and hasPopupMenu functions. Removed extra includes.
  • rendering/RenderPopupMenu.cpp: (WebCore::RenderPopupMenu::populate): Change to iterate the list items instead of iterating all children of the select node.
  • rendering/RenderPopupMenu.h: Renamed getRenderMenuList to menuList.
  • rendering/RenderPopupMenuMac.mm: (WebCore::RenderPopupMenuMac::populate): Moved code to clear and create the pop-up here from the caller. Removed an extra retain that would cause the NSPopUpButtonCell to leak. (WebCore::RenderPopupMenuMac::showPopup): Removed unnecessary code to create the pop-up, which is now in populate, and also the call to the clear function, for the same reason. Reorganized code to make it a bit more readable. Removed an unnecessary if to check if frame is nil. Used a RefPtr to make sure we don't make a call on a frame after it's deleted. As part of the reorganization fixed a problem where we'd retain the event and then return early without releasing it in one case. (WebCore::RenderPopupMenuMac::addSeparator): Tweaked a little. (WebCore::RenderPopupMenuMac::addGroupLabel): Grouped all the code to manage the NSMenu at the bottom of the function. (WebCore::RenderPopupMenuMac::addOption): Ditto.
19:21 Changeset [15366] by justing

LayoutTests:

Reviewed by levi


<rdar://problem/4620686>
REGRESSION: Mispelling markers are still displayed when using the delete key to place caret into a misspelled word

  • editing/deleting/delete-after-span-ws-001-expected.checksum:
  • editing/deleting/delete-after-span-ws-001-expected.png:
  • editing/deleting/delete-after-span-ws-002-expected.checksum:
  • editing/deleting/delete-after-span-ws-002-expected.png:
  • editing/deleting/delete-after-span-ws-003-expected.checksum:
  • editing/deleting/delete-after-span-ws-003-expected.png:
  • editing/deleting/delete-block-merge-contents-010-expected.checksum:
  • editing/deleting/delete-block-merge-contents-010-expected.png:
  • editing/deleting/merge-unrendered-space-expected.checksum:
  • editing/deleting/merge-unrendered-space-expected.png:

WebCore:

Reviewed by levi


<rdar://problem/4620686>
REGRESSION: Mispelling markers are still displayed when using the delete key to place caret into a misspelled word

  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::respondToChangedSelection): Remove markers from newAdjacentWords even if oldAdjacentWords is equal to newAdjacentWords. This happens during a deletion.
17:45 Changeset [15365] by thatcher

better build fix since other projects that include DOMPrivate.h can't included JavaScriptCore/npruntime.h.

17:40 Changeset [15364] by tomernic

Reviewed by NOBODY (build fix)

  • bindings/objc/DOMPrivate.h: Don't @class NPObject; that breaks files that actually #import <JavaScriptCore/npruntime.h>, since the NPObject there is of a different type. Also clarified a highly misleading comment from my last commit. I meant to clarify the comment before landing.
17:27 Changeset [15363] by andersca

2006-07-12 Anders Carlsson <acarlsson@apple.com>

Reviewed by Geoff.

http://bugzilla.opendarwin.org/show_bug.cgi?id=9635
REGRESSION: Crash when adding to cart at <http://www.yemeksepeti.com/>


  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::scriptHandler): Don't load external scripts if the parser is stopped.


  • manual-tests/open-close-tokenizer-crash.html: Added.
  • manual-tests/resources/empty-file.js: Added.
  • manual-tests/resources/open-close-tokenizer-crash.html: Added. Add manual test.
17:00 Changeset [15362] by sullivan

Reviewed by Kevin and Tim O

  • added support for creating a selection image with white text
  • WebView/WebDocumentPrivate.h: added -selectionImageForcingWhiteText: and -selectionImageRect to the private <WebDocumentSelection> protocol
  • Misc/WebSearchableTextView.m: (-[NSString selectionImageForcingWhiteText:]): added stub for this new method to this obsolete class to satisfy the compiler (-[NSString selectionImageRect]): ditto


  • WebView/WebHTMLView.m: (-[WebHTMLView _selectionDraggingImage]): now calls -selectionImageForcingWhiteText:NO instead of just -selectionImage (-[WebHTMLView _selectionDraggingRect]): now calls selectionImageRect, to which the implementation moved (-[WebHTMLView selectionImageForcingWhiteText:]): implemented this new method by calling through to new bridge method selectionImageForcingWhiteText: (-[WebHTMLView selectionImageRect]): implemented this new method by using existing _selectionDraggingRect implementation


  • WebView/WebPDFView.m: (-[WebPDFView selectionImageForcingWhiteText:]): implemented by using code that was formerly in Safari (-[WebPDFView selectionImageRect]): implemented by returning selectionRect
16:59 Changeset [15361] by sullivan

Reviewed by Kevin, Timo O, Brady, Darin, and Hyatt (whee!)

  • added support for creating a selection image with white text
  • rendering/RenderObject.h: (WebCore::): add PaintRestriction enum (WebCore::RenderObject::PaintInfo::PaintInfo): add forceWhiteText boolean to PaintInfo struct
  • page/FramePrivate.h: (WebCore::FramePrivate::FramePrivate): replaced m_selectionOnly bool with m_paintRestriction, which also handles forcing the text to white
  • page/Frame.cpp: (WebCore::Frame::paint): updated to use & pass m_paintRestriction where it used to use & pass just m_selectionOnly
  • bridge/mac/FrameMac.h:
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::selectionImage): now takes forceWhiteText parameter, stored in FramePrivate as part of PaintRestriction
  • bridge/mac/WebCoreFrameBridge.h:
  • bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge selectionImageForcingWhiteText:]): renamed from selectionImage, now takes forceWhiteText parameter, which is passed down into Frame
  • rendering/RenderLayer.h:
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paint): use a PaintRestriction value where we used to use a selectionOnly bool (WebCore::RenderLayer::paintLayer): ditto; also stores forceWhiteText in PaintInfo struct
  • kcanvas/KCanvasResources.cpp: (WebCore::KCanvasMarker::draw): updated for changed signature of PaintInfo constructor
  • ksvg2/svg/SVGMaskElement.cpp: (WebCore::SVGMaskElement::drawMaskerContent): ditto
  • ksvg2/svg/SVGPatternElement.cpp: (WebCore::SVGPatternElement::drawPatternContentIntoTile): ditto
  • rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::paint): paint with white text color if forceWhiteText is set in PaintInfo struct
16:58 Changeset [15360] by tomernic

LayoutTests:

Reviewed by Geoff.

Test case for <http://bugzilla.opendarwin.org/show_bug.cgi?id=9843>:
Give Netscape plug-ins access to their own DOM element

  • plugins/netscape-dom-access-expected.checksum: Added.
  • plugins/netscape-dom-access-expected.png: Added.
  • plugins/netscape-dom-access-expected.txt: Added.
  • plugins/netscape-dom-access.html: Added.

WebCore:

Reviewed by Anders.

<http://bugzilla.opendarwin.org/show_bug.cgi?id=9843>:
Give Netscape plug-ins access to their own DOM element

  • html/HTMLPlugInElement.h: Just import JSC headers on Mac. The NPObject and Bindings::Instance stuff is only used on Mac anyway. Changed APPLE to PLATFORM(MAC). Added m_NPObject ivar.
  • html/HTMLPlugInElement.cpp: (WebCore::HTMLPlugInElement::HTMLPlugInElement): Initialize the NPObject. (WebCore::HTMLPlugInElement::~HTMLPlugInElement): Release the NPObject when the element is destroyed. (WebCore::HTMLPlugInElement::createNPObject): Creates an NPObject for the element. (WebCore::HTMLPlugInElement::getNPObject): Returns the NPObject for the element, creating and caching it if necessary.
  • html/HTMLAppletElement.h: Changed APPLE to PLATFORM(MAC).
  • html/HTMLAppletElement.cpp: ditto
  • html/HTMLEmbedElement.h: ditto
  • html/HTMLEmbedElement.cpp: ditto
  • html/HTMLObjectElement.h: ditto
  • html/HTMLObjectElement.cpp: ditto
  • bindings/objc/DOMPrivate.h: Added -[DOMElement _NPObject].
  • bindings/objc/DOM.mm: (-[DOMElement _NPObject]): Returns the NPObject for the element. Since this is only needed by the Netscape plug-in API, you can only get the NPObject for applet, embed, and object elements.

WebKit:

Reviewed by Geoff.

<http://bugzilla.opendarwin.org/show_bug.cgi?id=9843>:
Give Netscape plug-ins access to their own DOM element

  • Plugins/WebBaseNetscapePluginView.h:
  • Plugins/WebBaseNetscapePluginView.m: (-[WebBaseNetscapePluginView dealloc]): Release DOM element. (-[WebBaseNetscapePluginView getVariable:value:]): Return NPObject for plugin DOM element.
  • Plugins/WebNetscapePluginEmbeddedView.h:
  • Plugins/WebNetscapePluginEmbeddedView.m: (-[WebNetscapePluginEmbeddedView initWithFrame:plugin:URL:baseURL:MIMEType:attributeKeys:attributeValues:loadManually:DOMElement:]): Now takes a DOMElement, in much the same way that WebKit plug-in views take a DOMElement.
  • WebCoreSupport/WebFrameBridge.m: (-[WebFrameBridge viewForPluginWithURL:attributeNames:attributeValues:MIMEType:DOMElement:loadManually:]): Pass DOMElement to Netscape plug-ins. (-[WebFrameBridge viewForJavaAppletWithFrame:attributeNames:attributeValues:baseURL:DOMElement:]): ditto
16:32 Changeset [15359] by adele

LayoutTests:

Reviewed by Hyatt.

Tests for http://bugzilla.opendarwin.org/show_bug.cgi?id=9861
REGRESSION: Can't dynamically change list box to popup

  • fast/forms/select-change-listbox-to-popup-expected.checksum: Added.
  • fast/forms/select-change-listbox-to-popup-expected.png: Added.
  • fast/forms/select-change-listbox-to-popup-expected.txt: Added.
  • fast/forms/select-change-listbox-to-popup.html: Added.
  • fast/forms/select-change-popup-to-listbox.html:

WebCore:

Reviewed by Hyatt.


and updated fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=9859
REGRESSION: Can't dynamically change popup to list box

The original fix for 9859 was in the renderer, and I think we can catch the change
earlier in the element to detach and reattach.

I also did some cleanup to remove the PopupButton code path from DeprecatedRenderSelect

Test: fast/forms/select-change-listbox-to-popup.html

  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::parseMappedAttribute): If the new attribute value is going to require us to change from listbox to popup or vice versa, and we're already attached, then detach and reattach to create the correct renderer. If we're not attached, then we'll create the correct renderer when we attach. (WebCore::HTMLSelectElement::recalcStyle): Updated shouldUseMenuList since it no longer takes a RenderStyle. (WebCore::HTMLSelectElement::isKeyboardFocusable): ditto. (WebCore::HTMLSelectElement::isMouseFocusable): ditto. (WebCore::HTMLSelectElement::createRenderer): ditto. (WebCore::HTMLSelectElement::setRecalcListItems): ditto. (WebCore::HTMLSelectElement::reset): ditto. (WebCore::HTMLSelectElement::notifyOptionSelected): ditto. (WebCore::HTMLSelectElement::defaultEventHandler): ditto.
  • html/HTMLSelectElement.h: (WebCore::HTMLSelectElement::shouldUseMenuList): This method no longer takes a RenderStyle. That was leftover from when we used to use the appearance to determine whether or not to use the new menu list implementation.
  • rendering/DeprecatedRenderSelect.h: Removed PopupButton code path.
  • rendering/DeprecatedRenderSelect.cpp: (WebCore::DeprecatedRenderSelect::DeprecatedRenderSelect): (WebCore::DeprecatedRenderSelect::setWidgetWritingDirection): (WebCore::DeprecatedRenderSelect::updateFromElement): (WebCore::DeprecatedRenderSelect::baselinePosition): (WebCore::DeprecatedRenderSelect::layout): (WebCore::DeprecatedRenderSelect::updateSelection):
  • rendering/RenderMenuList.cpp: (WebCore::RenderMenuList::updateFromElement): Backed out previous fix since this is now done in HTMLSelectElement.
15:35 Changeset [15358] by justing

LayoutTests:

Reviewed by levi & thatcher


<http://bugzilla.opendarwin.org/show_bug.cgi?id=9642>
GMail Editor: Operations that use drop down menus blow away the selection

  • editing/selection/drag-to-contenteditable-iframe-expected.txt:
  • editing/selection/subframe-with-selection-expected.checksum: Added.
  • editing/selection/subframe-with-selection-expected.png: Added.
  • editing/selection/subframe-with-selection-expected.txt: Added.
  • editing/selection/subframe-with-selection.html: Added.

WebKit:

Reviewed by levi & thatcher


<http://bugzilla.opendarwin.org/show_bug.cgi?id=9642>
GMail Editor: Operations that use drop down menus blow away the selection


  • WebView/WebHTMLView.m: (-[NSArray maintainsInactiveSelection]): Maintain an inactive selection when resigning as first responder if the selection is editable or if the WebView tells us to.
  • WebView/WebView.m: (-[WebView maintainsInactiveSelection]): Just because a WebView is editable doesn't mean selections inside subframes will be. Return NO by default.
15:13 Changeset [15357] by thatcher

Tag for WebCore 315.14.5 (part 2 of 2.)

15:13 Changeset [15356] by thatcher

Tag for WebCore 315.14.5 (part 1 of 2.)

15:13 Changeset [15355] by thatcher

bad tag.

15:13 Changeset [15354] by thatcher

bad tag.

15:12 Changeset [15353] by thatcher

Tag for WebCore 315.14.4 (part 2 of 2.)

15:11 Changeset [15352] by thatcher

Tag for WebCore === (part 1 of 2.)

15:10 Changeset [15351] by thatcher

Versioning

15:08 Changeset [15350] by thatcher

Merged fix from Safari-2-0-branch to Safari-1-3-branch

2006-07-06 Adele Peterson <adele@apple.com>

Reviewed by Tim Hatcher.

  • Speculative fix for <rdar://problem/4605532> unrepro crash at cnn.com after upgrading to 10.4.7 (but happening with $
  • khtml/khtmlview.cpp: (KHTMLView::layout): Ref's the view during layout, since a recalcStyle could cause the view to $ If we get back from recalcStyle, and there's only one ref, return early since the the view will be deleted when we exi$
  • khtml/khtmlview.h: (KHTMLView::hasOneRef): Added.
14:42 Changeset [15349] by adele

LayoutTests:

Reviewed by Hyatt.

Test for http://bugzilla.opendarwin.org/show_bug.cgi?id=9859
REGRESSION: Can't dynamically change popup to list box

  • fast/forms/select-change-popup-to-listbox-expected.checksum: Added.
  • fast/forms/select-change-popup-to-listbox-expected.png: Added.
  • fast/forms/select-change-popup-to-listbox-expected.txt: Added.
  • fast/forms/select-change-popup-to-listbox.html: Added.

WebCore:

Reviewed by Hyatt.

Fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=9859
REGRESSION: Can't dynamically change popup to list box

Test: fast/forms/select-change-popup-to-listbox.html

  • rendering/RenderMenuList.cpp: (WebCore::RenderMenuList::updateFromElement): If the select element has the multiple property set, or has a size > 1, then it really should be a list box, so we detach and attach the element so it creates the correct type of renderer.
14:31 Changeset [15348] by sullivan

Reviewed by Beth Dakin and Tim Omernick

  • fixed <rdar://problem/4622794> HiDPI: dragging the selection in Safari can show pixel cracks at non-integral scale factors
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::imageFromRect): round image rect in window coordinate space
14:30 Changeset [15347] by thatcher

New tag (part 2).

14:28 Changeset [15346] by thatcher

Tag for WebCore 418.20.1 (part 1 of 2.)

14:23 Changeset [15345] by thatcher

Versioning

14:08 Changeset [15344] by thatcher

Build fix.

14:06 Changeset [15343] by thatcher

New tag (part 2)

14:05 Changeset [15342] by thatcher

New tag.

13:58 Changeset [15341] by tomernic

Reviewed by Tim Hatcher.

<rdar://problem/4622748> WebKit now uses deprecated AGL functions

  • Plugins/WebBaseNetscapePluginView.m: (-[WebBaseNetscapePluginView _createWindowedAGLContext]): aglSetDrawable() is deprecated in AGL 3.0. Use aglSetWindowRef() instead. (-[WebBaseNetscapePluginView _createWindowlessAGLContext]): aglSetOffScreen() is deprecated in AGL 3.0. Use CGLSetOffScreen(), which does the same thing.
12:07 Changeset [15340] by ap

Reviewed by Darin.

The fix itself was landed on 2006-07-09, but I forgot about the test.

  • http/tests/misc/xhtml-expected.txt: Added.
  • http/tests/misc/xhtml.php: Added.
11:54 Changeset [15339] by bdakin

Reviewed by Adele.

Adele checked in new results for this test when you could style
selects, but the styling of selects is turned off for now, so back
to the original results.

  • fast/forms/001-expected.checksum:
  • fast/forms/001-expected.png:
  • fast/forms/001-expected.txt:
11:40 Changeset [15338] by thatcher

Adding the radar to a change.

11:21 Changeset [15337] by ap

Reviewed by Tim O.

WebKit:

  • Plugins/WebNetscapePluginStream.m: (-[WebNetscapePluginStream initWithRequest:pluginPointer:notifyData:sendNotification:]): Remove the early return when requesting an invalid (unsupported) URL.

WebKitTools:

  • DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.c: (pluginInvoke): treat getURL() with one parameter as if the second parameter were NULL - should open a new stream and deliver the data to the current instance.

LayoutTests:

  • plugins/get-empty-url-expected.txt: Added.
  • plugins/get-empty-url.html: Added.
11:13 Changeset [15336] by thatcher

Reviewed by Darin.

Bug 9598: [Drosera] add a JavaScript evaluator console
http://bugzilla.opendarwin.org/show_bug.cgi?id=9598

  • Drosera/DebuggerApplication.h:
  • Drosera/DebuggerApplication.m: (-[DebuggerApplication knownServers]):
  • Drosera/DebuggerDocument.h:
  • Drosera/DebuggerDocument.m: (-[WebScriptObject evaluateScript:inCallFrame:]): (-[WebScriptObject showConsole:]): (-[WebScriptObject toolbar:itemForItemIdentifier:willBeInsertedIntoToolbar:]): (-[WebScriptObject toolbarDefaultItemIdentifiers:]): (-[WebScriptObject toolbarAllowedItemIdentifiers:]): (-[WebScriptObject webView:createWebViewWithRequest:]): (-[WebScriptObject webViewShow:]): (-[WebScriptObject webViewAreToolbarsVisible:]): (-[WebScriptObject webView:setToolbarsVisible:]): (-[WebScriptObject webView:setResizable:]): (-[WebScriptObject webView:runJavaScriptAlertPanelWithMessage:initiatedByFrame:]): (-[WebScriptObject scriptConfirmSheetDidEnd:returnCode:contextInfo:]): (-[WebScriptObject webView:runJavaScriptConfirmPanelWithMessage:initiatedByFrame:]): (-[WebScriptObject webView:windowScriptObjectAvailable:]): (-[WebScriptObject webView:didFinishLoadForFrame:]): (-[WebScriptObject webView:didReceiveTitle:forFrame:]):
  • Drosera/Drosera.xcodeproj/project.pbxproj:
  • Drosera/Images/console.png: Added.
  • Drosera/console.css: Added.
  • Drosera/console.html: Added.
  • Drosera/console.js: Added.
11:02 Changeset [15335] by spadma

2006-07-11 zalan <zbujtas@gmail.com>

Reviewed by Sachin/David. Fix for:


http://bugzilla.opendarwin.org/show_bug.cgi?id=9401

Fix resource file directories. Missing #include <data_caging_paths.hrh>


08:00 Changeset [15334] by zbujtas

2006-07-07 brmorris <bradley.morrison@nokia.com>

Reviewed by zbujtas@gmail.com.

  • S60Internals/S60Headers.zip: data_caging_paths to \epoc32\include


Fix for:


http://bugzilla.opendarwin.org/show_bug.cgi?id=9776

07:21 Changeset [15333] by zbujtas

2006-07-07 brmorris <bradley.morrison@nokia.com>

Reviewed by zbujtas@gmail.com

00:51 Changeset [15332] by thatcher

Reviewed by a tired Geoff.

Bug 9597: [Drosera] hook up the variables table to show stack variables
http://bugzilla.opendarwin.org/show_bug.cgi?id=9597

  • Drosera/DebuggerDocument.m: (-[WebScriptObject isSelectorExcludedFromWebScript:]): (-[WebScriptObject webScriptAttributeKeysForScriptObject:]): (-[WebScriptObject localScopeVariableNamesForCallFrame:]): (-[WebScriptObject valueForScopeVariableNamed:inCallFrame:]): (-[WebScriptObject webView:didReceiveTitle:forFrame:]): (-[WebScriptObject webView:didLoadMainResourceForDataSource:]): (-[WebScriptObject webView:didParseSource:baseLineNumber:fromURL:sourceId:forWebFrame:]): (-[WebScriptObject webView:didEnterCallFrame:sourceId:line:forWebFrame:]): (-[WebScriptObject webView:willExecuteStatement:sourceId:line:forWebFrame:]): (-[WebScriptObject webView:willLeaveCallFrame:sourceId:line:forWebFrame:]):
  • Drosera/debugger.css:
  • Drosera/debugger.html:
  • Drosera/debugger.js:
00:30 Changeset [15331] by tomernic

Reviewed by Beth Dakin.

<http://bugzilla.opendarwin.org/show_bug.cgi?id=9844>:
Add DOM access test to DumpRenderTree's Netscape plug-in

  • DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.c: (testDOMAccess): (pluginInvoke):

07/10/06:

23:04 Changeset [15330] by darin

Reviewed by Geoff.

  • dom/Node.h:
  • dom/Node.cpp: (WebCore::Node::renderStyle): Made a const member function. The derived classes were overriding with const member functions, and I could change either this or the derived -- decided to do this.
22:48 Changeset [15329] by darin

Reviewed by Geoff.

  • rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::setFontFromControlSize): Code wants the family name, but was calling fontName, which is not the same thing. Also removed the unneeded conversion to DeprecatedString.
22:37 Changeset [15328] by ggaren

Reviewed by Darin.


  • Changed public header includes to the <JavaScriptCore/ style.
  • Changed instances of 'buffer' to 'string' since we decided on JSInternalString instead of JSStringBuffer.
  • API/JSContextRef.h:
  • API/JSInternalStringRef.cpp: (JSStringMake): (JSInternalStringRetain): (JSInternalStringRelease): (JSValueCopyStringValue): (JSInternalStringGetLength): (JSInternalStringGetCharactersPtr): (JSInternalStringGetCharacters): (JSInternalStringGetMaxLengthUTF8): (JSInternalStringGetCharactersUTF8): (CFStringCreateWithJSInternalString):
  • API/JSInternalStringRef.h:
  • API/JSNode.c: (JSNodePrototype_appendChild): (JSNode_getNodeType):
  • API/JSObjectRef.cpp: (JSObjectCallAsConstructor):
  • API/JSValueRef.h:
  • API/JavaScriptCore.h:
  • API/minidom.c: (main): (print):
  • API/testapi.c: (MyObject_getPropertyList): (myConstructor_callAsConstructor): (main): I noticed that we were prematurely releasing some string buffers, so I moved their release calls to the end of main(). I got rid of 'Buf' in *Buf (sometimes changing to 'IString', when necessary to differentiate a variable) to match the buffer->string change.
21:36 Changeset [15327] by ddkilzer

WebKitTools:

Reviewed by Darin.

Now track three states: initializing, running, and closed. If we are launched and the previous
state was initializing then we likely just experienced a crash on launch.

  • WebKitLauncher/WebKitNightlyEnabler.m: (myApplicationWillFinishLaunching): Improve wording of dialog. Update to set new running state. (myApplicationWillTerminate): Update to use new states. (cleanUpAfterOurselves): Display alert if previous state was initializing. Set state as initializing as early as practical. (symbol_lookup): Bring code up to speed with formatting guidelines. (GDSymbolLookup): Ditto.
20:40 Changeset [15326] by thatcher

New tag.

20:37 Changeset [15325] by darin
  • kjs/value.cpp: (KJS::JSValue::toInt32Inline): Added inline keyword one more place. Just in case.
20:36 Changeset [15324] by thatcher

Move the release tag.

20:34 Changeset [15323] by darin
  • fix the release build
  • kjs/value.h:
  • kjs/value.cpp: (KJS::JSValue::toInt32Inline): Move the code here to an inline. (KJS::JSValue::toInt32): Call the inline from both overloaded toInt32 functions.
20:33 Changeset [15322] by thatcher

Versioning

20:16 Changeset [15321] by ddkilzer

JavaScriptCore:

Reviewed by Darin.

  • JavaScriptCore.exp: Added overloaded KJS::JSValue::toInt32() method.
  • JavaScriptCore.xcodeproj/project.pbxproj: Altered attributes metadata for kjs/value.h to make it available as a forwarded header.
  • kjs/lookup.h: (KJS::lookupPut): Extracted a lookupPut() method from the existing lookupPut() method. The new method returns a boolean value if no entry is found in the lookup table.
  • kjs/value.cpp: (KJS::JSValue::toInt32): Overloaded toInt32() method with boolean "Ok" argument.
  • kjs/value.h: Ditto.

LayoutTests:

Reviewed by Darin.

  • fast/dom/select-selectedIndex-multiple-expected.txt: Updated test results.
  • fast/dom/select-selectedIndex-multiple.html: Updated to print comments between tests to make failures easier to track down.
  • fast/dom/select-selectedIndex-expected.txt: Mirrored updates from select-selectedIndex-multiple.html
  • fast/dom/select-selectedIndex.html: Ditto.
  • fast/js/resources/select-options-add.js: Added.
  • fast/js/select-options-add-expected.txt: Added.
  • fast/js/select-options-add.html: Added.

WebCore:

Reviewed by Darin.

Tests:

  • fast/dom/select-selectedIndex-multiple.html
  • fast/dom/select-selectedIndex.html
  • fast/js/select-options-add.html
  • DerivedSources.make: Added JSHTMLOptionsCollection.h.
  • ForwardingHeaders/kjs/operations.h: Added.
  • WebCore.xcodeproj/project.pbxproj: Added new source files.
  • bindings/js/JSHTMLOptionsCollectionCustom.cpp: Added. (WebCore::JSHTMLOptionsCollection::length): (WebCore::JSHTMLOptionsCollection::setLength): (WebCore::JSHTMLOptionsCollection::indexSetter):
  • bindings/js/kjs_html.cpp: Removed JSHTMLOptionsCollection implementation. Renamed classes and methods for consistency. (KJS::JSHTMLElement::selectGetter): (KJS::JSHTMLElement::put): (KJS::JSHTMLElement::selectSetter): (KJS::JSHTMLCollection::JSHTMLCollection): (KJS::JSHTMLCollectionProtoFunc::callAsFunction): (KJS::getHTMLOptionsCollection):
  • bindings/js/kjs_html.h: Ditto.
  • bindings/scripts/CodeGeneratorJS.pm: Added support for HasCustomIndexSetter class attribute. Added support for Optional parameter attribute, which makes generated code assume overloaded implementation methods are available for a JavaScript function with optional arguments. Changed local 'impl' variables to 'imp' so that impl() methods could be called without class designation.
  • html/HTMLOptionElement.idl: Added GenerateNativeConverter attribute.
  • html/HTMLOptionsCollection.cpp: Added methods used by generated JSHTMLOptionsCollection class. (WebCore::HTMLOptionsCollection::HTMLOptionsCollection): (WebCore::HTMLOptionsCollection::add): (WebCore::HTMLOptionsCollection::selectedIndex): (WebCore::HTMLOptionsCollection::setSelectedIndex): (WebCore::HTMLOptionsCollection::setLength):
  • html/HTMLOptionsCollection.h: Ditto.
  • html/HTMLOptionsCollection.idl: Added.
19:02 Changeset [15320] by harrison

Reviewed by Dave Hyatt.

<rdar://problem/4602408> -webkit-highlight needs to support images and elements other than text

  • rendering/RenderBox.cpp: (WebCore::RenderBox::paintCustomHighlight):
  • rendering/RenderBox.h: Utility for subclasses.


  • rendering/RenderImage.cpp: (WebCore::RenderImage::paint): Custom highlight in front of images.


  • rendering/RenderListMarker.cpp: (WebCore::RenderListMarker::paint): Custom highlight in front of list markers.
  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::paint): Custom highlight in front of objects.
18:22 Changeset [15319] by hyatt

Better fix for repainting issue with positioned objects when height
changes. Optimize when only a positioned child changes so that we don't
incorrectly do normal flow layout.

Reviewed by darin

  • WebCore.xcodeproj/project.pbxproj:
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::layoutBlock):
  • rendering/RenderFlexibleBox.cpp: (WebCore::RenderFlexibleBox::layoutBlock):
  • rendering/RenderTable.cpp: (WebCore::RenderTable::layout):
  • rendering/RenderView.cpp: (WebCore::RenderView::layout):
  • rendering/bidi.cpp: (WebCore::RenderBlock::layoutInlineChildren):
17:27 Changeset [15318] by ggaren

No review necessary. Removed bogus file I accidentally checked in before.

  • API/JSInternalSringRef.h: Removed.
17:26 Changeset [15317] by ggaren

Reviewed by Darin.


Added exception out parameter to API object callbacks, removed semi-bogus
JSContext(.*)Exception functions.


To make these calls syntactically simple, I added an exceptionSlot()
method to the ExecState class, which provides a JSValue slot in which to
store a JSValue* exception.

  • API/APICast.h: (toRef):
  • API/JSCallbackConstructor.cpp: (KJS::JSCallbackConstructor::construct):
  • API/JSCallbackFunction.cpp: (KJS::JSCallbackFunction::callAsFunction):
  • API/JSCallbackObject.cpp: (KJS::JSCallbackObject::init): (KJS::JSCallbackObject::getOwnPropertySlot): (KJS::JSCallbackObject::put): (KJS::JSCallbackObject::deleteProperty): (KJS::JSCallbackObject::construct): (KJS::JSCallbackObject::callAsFunction): (KJS::JSCallbackObject::getPropertyList): (KJS::JSCallbackObject::toBoolean): (KJS::JSCallbackObject::toNumber): (KJS::JSCallbackObject::toString): (KJS::JSCallbackObject::staticValueGetter): (KJS::JSCallbackObject::callbackGetter):
  • API/JSContextRef.cpp: (JSCheckSyntax):
  • API/JSContextRef.h:
  • API/JSNode.c: (JSNodePrototype_appendChild): (JSNodePrototype_removeChild): (JSNodePrototype_replaceChild): (JSNode_getNodeType): (JSNode_getChildNodes): (JSNode_getFirstChild): (JSNode_construct):
  • API/JSNode.h:
  • API/JSNodeList.c: (JSNodeListPrototype_item): (JSNodeList_length): (JSNodeList_getProperty):
  • API/JSObjectRef.h:
  • API/minidom.c: (print):
  • API/testapi.c: (MyObject_initialize): (MyObject_hasProperty): (MyObject_getProperty): (MyObject_setProperty): (MyObject_deleteProperty): (MyObject_getPropertyList): (MyObject_callAsFunction): (MyObject_callAsConstructor): (MyObject_convertToType): (print_callAsFunction): (myConstructor_callAsConstructor): (main):
  • JavaScriptCore.exp:
  • kjs/ExecState.h: (KJS::ExecState::exceptionHandle):
16:52 Changeset [15316] by adele

LayoutTests:

Reviewed by Beth.

Test for <rdar://problem/4621442> REGRESSION (NativePopup) : Popup menus aren't drawn correctly on page at orbitz.com; cheaptickets.com

  • fast/forms/select-style-expected.checksum: Added.
  • fast/forms/select-style-expected.png: Added.
  • fast/forms/select-style-expected.txt: Added.
  • fast/forms/select-style.html: Added.

WebCore:

Reviewed by Beth.

  • Fix for <rdar://problem/4621442> REGRESSION (NativePopup) : Popup menus aren't drawn correctly on page at orbitz.com; cheaptickets.com

Test: fast/forms/select-style.html

  • rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::isControlStyled): Don't let popups be style-able for now. We'll allow it later, when we have a good way to handle border & backgrounds and still having it look like a control.
16:50 Changeset [15315] by sullivan

Reviewed by John Sullivan.

  • WebView/WebDataSource.m: (-[WebDataSource _setPrimaryLoadComplete:]): Set our data only if the frame loader is has just loaded it (when coming from the back/forward cache, it hasn't).
15:29 Changeset [15314] by justing

Reviewed by harrison


<rdar://problem/4619260>
9A213 + Safari 521.15.1 Crash with To Do's alarm icon (WebCore::maxDeepOffset(WebCore::Node const*) )
<rdar://problem/4619841>
REGRESSION: Dragging selection over input field results in a crash (WebCore::maxDeepOffset(WebCore::Node const*)

  • WebCore.xcodeproj/project.pbxproj:
  • editing/Selection.cpp: (WebCore::Selection::adjustForEditableContent): Iterate using next/previousVisuallyDistinctCandidate instead of traverseNext/Previous node because we were skipping positions. Jump out of shadow trees. Migrate to isEditablePosition instead of isContentEditable.
15:25 Changeset [15313] by bdakin

Reviewed by Adele.

Shuffling comments around from my last check-in to make things read
more clearly.

  • page/DOMWindow.h:
  • page/DOMWindow.idl:
15:05 Changeset [15312] by bdakin

Reviewed by Darin.

Fix for <rdar://problem/4621095> The should be a way to
access the user interface scale factor through the DOM

window.devicePixelRatio returns the user interface scale factor.

  • page/DOMWindow.cpp: (WebCore::DOMWindow::devicePixelRatio):
  • page/DOMWindow.h:
  • page/DOMWindow.idl:
14:39 Changeset [15311] by spadma

2006-07-10 David Carson <david.carson@nokia.com>

Reviewed by Sachin/Zalan.

Fix for network connection, remove use of InternetConnectionManager
http://bugzilla.opendarwin.org/show_bug.cgi?id=9451


  • WebKit\BrowserView\src\HttpLoaderEventToUiListener.cpp: (CHttpLoaderEventToUiListener::ConnectionRequest) Added check fo NULL pointer as aConnectionPtr could be null after the cb
  • S60WebUi\WebUi\src\WebUiSessionAndSecurity.cpp
  • S60WebUi\WebUi\inc\WebUiSessionAndSecurity.h Removed dependancy on InternetConnectionManager, and removed ShowSessionInfo as the information is available from the Connection Manager application. Constructor no longer takes a InternetConnectionManager reference.


  • S60WebUi\WebUi\data\WebUi.rss
  • S60WebUi\WebUi\inc\WebUi.hrh Removed ShowSessionInfo menu option


  • S60WebUi\WebUi\src\WebUiWindowContainer.cpp
    • S60WebUi\WebUi\inc\WebUiWindowContainer.h (CWebUiWindowContainer::HandleCommandL): Removed ShowSessionInfo option (CWebUiWindowContainer::NetworkConnectionNeededL): Made function empty as Symbian default handler does the work.
14:17 Changeset [15310] by ggaren

Reviewed by Darin.

Improved type safety by implementing opaque JSValue/JSObject typing through
abuse of 'const', not void*. Also fixed an alarming number of bugs
exposed by this new type safety.


I made one design change in JavaScriptCore, which is that the JSObject
constructor should take a JSValue* as its prototype argument, not a JSObject*,
since we allow the prototype to be any JSValue*, including jsNull(), for
example.


  • API/APICast.h: (toJS):
  • API/JSBase.h:
  • API/JSCallbackConstructor.cpp: (KJS::JSCallbackConstructor::construct):
  • API/JSCallbackFunction.cpp: (KJS::JSCallbackFunction::callAsFunction):
  • API/JSCallbackObject.cpp: (KJS::JSCallbackObject::JSCallbackObject): (KJS::JSCallbackObject::getOwnPropertySlot): (KJS::JSCallbackObject::put): (KJS::JSCallbackObject::construct): (KJS::JSCallbackObject::callAsFunction): (KJS::JSCallbackObject::staticFunctionGetter):
  • API/JSCallbackObject.h:
  • API/JSContextRef.cpp: (JSEvaluate):
  • API/JSNode.c: (JSNodePrototype_appendChild): (JSNodePrototype_removeChild): (JSNodePrototype_replaceChild):
  • API/JSObjectRef.cpp: (JSObjectMake): (JSFunctionMakeWithBody): (JSObjectGetProperty): (JSObjectCallAsFunction): (JSObjectCallAsConstructor):
  • API/JSObjectRef.h:
  • API/testapi.c: (main):
  • ChangeLog:
  • kjs/object.h: (KJS::JSObject::JSObject):
13:53 Changeset [15309] by adele

http://bugzilla.opendarwin.org/show_bug.cgi?id=9830
More pixel test results need updating since switching to native popup

  • editing/selection/replaced-boundaries-3-expected.checksum:
  • editing/selection/replaced-boundaries-3-expected.png:
  • fast/block/margin-collapse/103-expected.checksum:
  • fast/block/margin-collapse/103-expected.png:
  • fast/forms/001-expected.checksum:
  • fast/forms/001-expected.png:
  • fast/forms/HTMLOptionElement_label01-expected.checksum:
  • fast/forms/HTMLOptionElement_label01-expected.png:
  • fast/forms/HTMLOptionElement_label02-expected.checksum:
  • fast/forms/HTMLOptionElement_label02-expected.png:
  • fast/forms/HTMLOptionElement_label03-expected.checksum:
  • fast/forms/HTMLOptionElement_label03-expected.png:
  • fast/forms/HTMLOptionElement_label04-expected.checksum:
  • fast/forms/HTMLOptionElement_label04-expected.png:
  • fast/forms/form-element-geometry-expected.checksum:
  • fast/forms/form-element-geometry-expected.png:
  • fast/forms/option-script-expected.checksum:
  • fast/forms/option-script-expected.png:
  • fast/forms/option-strip-whitespace-expected.checksum:
  • fast/forms/option-strip-whitespace-expected.png:
  • fast/invalid/014-expected.checksum:
  • fast/invalid/014-expected.png:
  • fast/parser/document-write-option-expected.checksum:
  • fast/parser/document-write-option-expected.png:
  • tables/mozilla/bugs/bug2479-3-expected.checksum:
  • tables/mozilla/bugs/bug2479-3-expected.png:
  • tables/mozilla/bugs/bug29326-expected.checksum:
  • tables/mozilla/bugs/bug29326-expected.png:
  • tables/mozilla/bugs/bug33855-expected.checksum:
  • tables/mozilla/bugs/bug33855-expected.png:
  • tables/mozilla/bugs/bug96334-expected.checksum:
  • tables/mozilla/bugs/bug96334-expected.png:
  • tables/mozilla/core/margins-expected.checksum:
  • tables/mozilla/core/margins-expected.png:
  • tables/mozilla/dom/tableDom-expected.checksum:
  • tables/mozilla/dom/tableDom-expected.png:
10:47 Changeset [15308] by beidson

WebCore:

Reviewed by Alexey

Resolved the console error messages people got from the new DB even if they didn't have it enabled

  • icon/IconDatabase.cpp: (WebCore::IconDatabase::pruneUnreferencedIcons): (WebCore::IconDatabase::pruneUnretainedIcons): Added quick check to bail if the DB isn't open

WebKit:

Reviewed by Alexey

Resolved the console error messages people got from the new DB even if they didn't have it enabled

  • Misc/WebIconDatabase.m: (-[WebIconDatabase init]): Disabled initializing the IconDatabaseBridge if user is living on the old DB
10:41 Changeset [15307] by ggaren

Approved by Maciej, Darin.


Renamed JSStringBufferRef to JSInternalStringRef. "Internal string" means the
JavaScript engine's internal string representation, which is the most
low-level and efficient representation to use when interfacing with JavaScript.

  • API/APICast.h: (toJS): (toRef):
  • API/JSBase.h:
  • API/JSCallbackObject.cpp: (KJS::JSCallbackObject::getOwnPropertySlot): (KJS::JSCallbackObject::put): (KJS::JSCallbackObject::deleteProperty): (KJS::JSCallbackObject::staticValueGetter): (KJS::JSCallbackObject::callbackGetter):
  • API/JSContextRef.cpp: (JSEvaluate): (JSCheckSyntax):
  • API/JSContextRef.h:
  • API/JSInternalStringRef.cpp: Added. (JSStringMake): (JSInternalStringCreate): (JSInternalStringCreateUTF8): (JSInternalStringRetain): (JSInternalStringRelease): (JSValueCopyStringValue): (JSInternalStringGetLength): (JSInternalStringGetCharactersPtr): (JSInternalStringGetCharacters): (JSInternalStringGetMaxLengthUTF8): (JSInternalStringGetCharactersUTF8): (JSInternalStringIsEqual): (JSInternalStringIsEqualUTF8): (JSInternalStringCreateCF): (CFStringCreateWithJSInternalString):
  • API/JSInternalStringRef.h: Added.
  • API/JSNode.c: (JSNodePrototype_appendChild): (JSNode_getNodeType): (JSNode_getChildNodes): (JSNode_getFirstChild):
  • API/JSNodeList.c: (JSNodeList_length): (JSNodeList_getProperty):
  • API/JSObjectRef.cpp: (JSFunctionMakeWithBody): (JSObjectGetDescription): (JSObjectHasProperty): (JSObjectGetProperty): (JSObjectSetProperty): (JSObjectDeleteProperty): (JSPropertyEnumeratorGetNext): (JSPropertyListAdd):
  • API/JSObjectRef.h:
  • API/JSStringBufferRef.cpp: Removed.
  • API/JSStringBufferRef.h: Removed.
  • API/JSValueRef.h:
  • API/JavaScriptCore.h:
  • API/minidom.c: (main): (print):
  • API/testapi.c: (assertEqualsAsUTF8String): (assertEqualsAsCharactersPtr): (assertEqualsAsCharacters): (MyObject_hasProperty): (MyObject_getProperty): (MyObject_setProperty): (MyObject_deleteProperty): (MyObject_getPropertyList): (print_callAsFunction): (myConstructor_callAsConstructor): (main):
  • JavaScriptCore.exp:
  • JavaScriptCore.xcodeproj/project.pbxproj:
09:48 Changeset [15306] by harrison
09:18 Changeset [15305] by darin

Reviewed by Darin.

  • manual-tests/ATSU-bad-layout.html: Added.
  • platform/mac/FontMac.mm: (WebCore::Font::floatWidthForComplexText): Return 0 immediately for empty runs.
08:54 Changeset [15304] by darin
  • try to fix the no-SVG build
  • ksvg2/misc/SVGImageLoader.cpp: Move namespace brace inside the #if.
08:53 Changeset [15303] by darin

Reviewed by Anders.

  • manual-tests/empty-title-popup.html: Added.
  • rendering/RenderMenuList.h: Add createInnerBlock.
  • rendering/RenderMenuList.cpp: (WebCore::RenderMenuList::createInnerBlock): Factored out of addChild. (WebCore::RenderMenuList::addChild): Call createInnerBlock. (WebCore::RenderMenuList::setText): Changed parameter type. (WebCore::RenderMenuList::showPopup): Call createInnerBlock before calling the parent class's addChild.
08:34 Changeset [15302] by darin
  • try to fix Windows build
  • DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.vcproj: Remove include paths with "khtml" in them. Add one for "html" subdir or WebCore.
08:00 Changeset [15301] by darin
  • tables/mozilla_expected_failures/bugs/bug2479-5-expected.txt: Updated one more result for Adele's 4463835 change last night.
05:57 Changeset [15300] by ddkilzer

WebCore:

Reviewed by Maciej via IRC. Patch by Rob Buis.

Make sure the attributes are calculated against viewport width value,
as defined in the spec. http://paste.lisp.org/display/22342

  • ksvg2/svg/SVGEllipseElement.cpp: (WebCore::SVGEllipseElement::rx): Changed LM_HEIGHT to LM_WIDTH.
  • ksvg2/svg/SVGLineElement.cpp: (SVGLineElement::x2): Ditto.
04:39 Changeset [15299] by bdakin

Reviewed by Maciej.

Fix for <rdar://problem/4610314> Support the CSS3 content property
for images

This patch provides initial support for the CSS3 content property.
It lacks support for any content other than images, and only allows
content for elements that seemed safe for now.

  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::applyProperty): Remove restrictions for just :before and :after
  • html/HTMLBRElement.cpp: (WebCore::HTMLBRElement::createRenderer):
  • html/HTMLFrameSetElement.cpp: (WebCore::HTMLFrameSetElement::createRenderer):
  • html/HTMLImageElement.cpp: (WebCore::HTMLImageElement::createRenderer):
  • html/HTMLLegendElement.cpp: (WebCore::HTMLLegendElement::createRenderer):
  • rendering/RenderImage.cpp: (WebCore::RenderImage::RenderImage): (WebCore::RenderImage::setCachedImage): Don't allow the image to be set this way if it was already set with the content property.
  • rendering/RenderImage.h: (WebCore::RenderImage::setIsAnonymousImage): Anonymous images are those set through the content property. (WebCore::RenderImage::isAnonymousImage):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::createObject): If there is content specified, create a RenderImage with the content and initialize the cached image to whatever was specified in the CSS.
03:31 Changeset [15298] by beidson

Reviewed by NOBODY (Build Fix!)

  • icon/SiteIcon.cpp: (SiteIcon::getImage): Release build made an "unused variable" warning, which is, of course, an error. fixed that.
02:57 Changeset [15297] by beidson

Reviewed by Maciej

SQLite Icon DB now fully replicates functionality of the old DB, including pruning to keep unused
information out to keep down disk usage. For now, it is still off by default, as the critical
feature of importing old icons into the new DB is still unrealized.
If you'd like to test, add #define ICONDEBUG to your WebKitPrefix.h

  • icon/IconDatabase.cpp: (WebCore::IconDatabase::IconDatabase): (WebCore::IconDatabase::open): -Sets up a timer for initial pruning (WebCore::IconDatabase::iconForPageURL): -Switched away from "dirty blob hack" as the problem is provisionally solved (WebCore::IconDatabase::retainIconForURL): (WebCore::IconDatabase::releaseIconForURL): -Same as found in WebKit (WebCore::IconDatabase::totalRetainCountForIconURL): -Adds up all retainers of icon (WebCore::IconDatabase::forgetIconForIconURLFromDatabase): -Wipes an Icon table entry out (WebCore::IconDatabase::establishTemporaryIconIDForIconURL): (WebCore::IconDatabase::establishTemporaryIconIDForEscapedIconURL): (WebCore::IconDatabase::establishIconIDForIconURL): (WebCore::IconDatabase::establishIconIDForEscapedIconURL): -Added ability to get an iconID without creating a new one if a lookup is all

that is genuinely wanted

(WebCore::IconDatabase::pruneUnreferencedIcons):
(WebCore::IconDatabase::pruneUnretainedIcons):

-Called on a timer on DB startup to clean it out

  • icon/IconDatabase.h:
  • icon/SiteIcon.cpp: (SiteIcon::SiteIcon): (SiteIcon::~SiteIcon): -Deletes the image on deletion (SiteIcon::getImage): -Cut down on no-longer-necessary debug info
02:02 Changeset [15296] by andersca

WebCore:

2006-07-10 Anders Carlsson <acarlsson@apple.com>

Reviewed by Adele.

http://bugzilla.opendarwin.org/show_bug.cgi?id=9245
Quirksmode: Incorrect handling of disabled BUTTON elements

  • html/HTMLButtonElement.cpp: (WebCore::HTMLButtonElement::defaultEventHandler): Don't call prepareSubmit if the element is disabled.
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::defaultEventHandler): Don't call prepareSubmit if the element is disabled.

LayoutTest:

2006-07-09 Anders Carlsson <acarlsson@apple.com>

Reviewed by Adele.

http://bugzilla.opendarwin.org/show_bug.cgi?id=9245
Quirksmode: Incorrect handling of disabled BUTTON elements


  • fast/forms/activate-and-disabled-elements-expected.txt: Added.
  • fast/forms/activate-and-disabled-elements.html: Added.
01:01 Changeset [15295] by darin
  • try to fix Windows build
  • platform/win/TemporaryLinkStubs.cpp: Put a few functions in the WebCore namespace.
  • platform/gdk/TemporaryLinkStubs.cpp: Ditto.
00:59 Changeset [15294] by darin

Reviewed by Geoff.

  • fix failures seen in layout tests
  • bindings/js/kjs_dom.cpp: (KJS::toJS): Restore one of the few WebCore:: prefixes that really needed to be there -- because we have two classes named JSHTMLDocument at the moment! (One in KJS and one in WebCore namespace.)
00:40 Changeset [15293] by darin
  • try to fix Windows build
  • rendering/RenderLayer.h: Correct a forward declaration of ScrollBar.
00:33 Changeset [15292] by darin
  • try to fix Windows build
  • COM/WebFrame.h: Qualify DeprecatedString and KURL with WebCore:: prefixes.
00:19 Changeset [15291] by adele

Reviewed by Maciej.

Updated results for:
<rdar://problem/4463835> Switch to use new popup menu implementation for <select>

  • editing/selection/replaced-boundaries-3-expected.txt:
  • editing/selection/select-box-expected.txt:
  • fast/block/margin-collapse/103-expected.txt:
  • fast/dom/option-properties-expected.txt:
  • fast/forms/003-expected.txt:
  • fast/forms/004-expected.txt:
  • fast/forms/HTMLOptionElement_label01-expected.txt:
  • fast/forms/HTMLOptionElement_label02-expected.txt:
  • fast/forms/HTMLOptionElement_label03-expected.txt:
  • fast/forms/HTMLOptionElement_label04-expected.txt:
  • fast/forms/HTMLOptionElement_label06-expected.txt:
  • fast/forms/HTMLOptionElement_label07-expected.txt:
  • fast/forms/form-element-geometry-expected.txt:
  • fast/forms/option-script-expected.txt:
  • fast/forms/option-strip-whitespace-expected.txt:
  • fast/forms/select-remove-option-expected.txt:
  • fast/forms/select-replace-option-expected.txt:
  • fast/forms/select-reset-expected.txt:
  • fast/invalid/014-expected.txt:
  • fast/parser/document-write-option-expected.txt:
  • fast/replaced/width100percent-expected.txt:
  • tables/mozilla/bugs/bug1188-expected.txt:
  • tables/mozilla/bugs/bug18359-expected.txt:
  • tables/mozilla/bugs/bug2479-3-expected.txt:
  • tables/mozilla/bugs/bug2479-4-expected.txt:
  • tables/mozilla/bugs/bug29326-expected.txt:
  • tables/mozilla/bugs/bug33855-expected.txt:
  • tables/mozilla/bugs/bug4382-expected.txt:
  • tables/mozilla/bugs/bug96334-expected.txt:
  • tables/mozilla/core/margins-expected.txt:
  • tables/mozilla/dom/tableDom-expected.txt:
00:19 Changeset [15290] by adele

Reviewed by Maciej.

Fix for <rdar://problem/4463835> Switch to use new popup menu implementation for <select>

  • css/html4.css: Added style for new selects, and for list boxes.
  • html/HTMLSelectElement.h: (WebCore::HTMLSelectElement::shouldUseMenuList): Removed check for appearance so new popups can be styled.


  • rendering/RenderMenuList.cpp: (WebCore::RenderMenuList::setStyle): Removed padding. This is now set by the theme. (WebCore::RenderMenuList::calcMinMaxWidth): Now takes minimum text size into account.
  • rendering/RenderMenuList.h: Removed baselinePosition since we're letting the theme calculate that.
  • rendering/RenderTheme.cpp: (WebCore::RenderTheme::isControlContainer): Added menuList as a ControlContainer, so RenderThemeMac will compute its baseline.
  • rendering/RenderTheme.h: (WebCore::RenderTheme::minimumTextSize): Added.
  • rendering/RenderThemeMac.h: Added minimumTextSize, setPopupPaddingFromControlSize, popupButtonSizes, popupButtonMargins, popupButtonPadding. Removed sizeOfArrowControl.
  • rendering/RenderThemeMac.mm: (WebCore::): Added enum for padding. (WebCore::RenderThemeMac::adjustRepaintRect): Now inflates rect to account for control shadow. (WebCore::RenderThemeMac::baselinePosition): Now calculates baseline for menulist. (WebCore::RenderThemeMac::popupButtonMargins): Tweaked values. (WebCore::RenderThemeMac::popupButtonSizes): Added. (WebCore::RenderThemeMac::popupButtonPadding): Added, so different padding can be set for different control sizes. (WebCore::RenderThemeMac::setPopupPaddingFromControlSize): Added. (WebCore::RenderThemeMac::paintMenuList): Now inflates rect to account for control shadow. (WebCore::RenderThemeMac::adjustMenuListStyle): Resets border, sets padding, sets control size. (WebCore::RenderThemeMac::setPopupButtonCellState): Removed unnecessary class name. (WebCore::RenderThemeMac::minimumTextSize): Added.

07/09/06:

23:16 Changeset [15289] by darin
  • add back "unused code" for plain text fields, still used by <isindex>!
  • platform/TextField.h:
  • platform/mac/TextFieldMac.mm:
  • platform/mac/WebCoreTextField.h:
  • platform/mac/WebCoreTextField.mm:
  • rendering/RenderLineEdit.cpp:
23:15 Changeset [15288] by darin
  • try to fix Windows build
  • COM/WebFrame.cpp: Rename QChar to DeprecatedChar.
23:05 Changeset [15287] by darin
  • fix newlines to be consistent for all files in the COM directory (many had mixed style) and set the EOL style to "native" on them.
  • COM/*: Set properties and changed files.
22:20 Changeset [15286] by darin

Reviewed by Tim Hatcher.

  • put more into the WebCore namespace
  • removed unused code for plain text fields

Put more classes, class templates, enums, functions, and constants into
the WebCore namespace, including all the headers in the platform directory.

Removed lots of unneeded "WebCore::" qualifiers and "using" directives.

Added some "WebCore::" qualifiers in a few places.

Removed "uses WebCore::" in headers that we said we'd remove "when
everything is in the WebCore namespace".

  • bindings/js/JSDOMParser.cpp:
  • bindings/js/JSHTMLElementWrapperFactory.cpp:
  • bindings/js/JSXSLTProcessor.cpp:
  • bindings/js/kjs_binding.cpp:
  • bindings/js/kjs_css.cpp:
  • bindings/js/kjs_dom.cpp:
  • bindings/js/kjs_dom.h:
  • bindings/js/kjs_events.cpp:
  • bindings/js/kjs_proxy.h:
  • bindings/js/kjs_window.h:
  • bindings/objc/DOM.mm:
  • bridge/History.h:
  • bridge/JavaAppletWidget.h:
  • bridge/mac/FormDataMac.h:
  • bridge/mac/FormDataMac.mm:
  • bridge/mac/WebCoreAXObject.mm:
  • bridge/mac/WebCoreEncodings.mm:
  • bridge/mac/WebCoreFrameBridge.h:
  • bridge/mac/WebCoreIconDatabaseBridge.mm:
  • bridge/mac/WebCorePageState.h:
  • bridge/mac/WebCoreScriptDebugger.mm:
  • bridge/mac/WebCoreSettings.h:
  • css/CSSGrammar.y:
  • css/CSSPageRule.cpp:
  • css/CSSPageRule.h:
  • css/MediaList.cpp:
  • css/cssstyleselector.h:
  • dom/Clipboard.h:
  • dom/Document.cpp:
  • dom/Document.h:
  • dom/EventTargetNode.h:
  • dom/Node.cpp:
  • dom/Node.h:
  • dom/Position.h:
  • dom/Range.cpp:
  • dom/XMLTokenizer.cpp:
  • editing/AppendNodeCommand.h:
  • editing/ApplyStyleCommand.cpp:
  • editing/BreakBlockquoteCommand.cpp:
  • editing/BreakBlockquoteCommand.h:
  • editing/CompositeEditCommand.cpp:
  • editing/CompositeEditCommand.h:
  • editing/CreateLinkCommand.h:
  • editing/DeleteFromTextNodeCommand.h:
  • editing/DeleteSelectionCommand.h:
  • editing/FormatBlockCommand.h:
  • editing/HTMLInterchange.cpp:
  • editing/HTMLInterchange.h:
  • editing/IndentOutdentCommand.h:
  • editing/InsertIntoTextNodeCommand.h:
  • editing/InsertLineBreakCommand.h:
  • editing/InsertListCommand.h:
  • editing/InsertNodeBeforeCommand.h:
  • editing/InsertTextCommand.cpp:
  • editing/InsertTextCommand.h:
  • editing/JSEditor.cpp:
  • editing/JSEditor.h:
  • editing/JoinTextNodesCommand.h:
  • editing/MergeIdenticalElementsCommand.cpp:
  • editing/MergeIdenticalElementsCommand.h:
  • editing/ModifySelectionListLevel.h:
  • editing/MoveSelectionCommand.h:
  • editing/RebalanceWhitespaceCommand.h:
  • editing/RemoveNodeAttributeCommand.h:
  • editing/RemoveNodeCommand.h:
  • editing/RemoveNodePreservingChildrenCommand.h:
  • editing/ReplaceSelectionCommand.h:
  • editing/SetNodeAttributeCommand.h:
  • editing/SplitElementCommand.cpp:
  • editing/SplitElementCommand.h:
  • editing/SplitTextNodeCommand.cpp:
  • editing/SplitTextNodeCommand.h:
  • editing/SplitTextNodeContainingElementCommand.h:
  • editing/TypingCommand.cpp:
  • editing/TypingCommand.h:
  • editing/UnlinkCommand.h:
  • editing/VisiblePosition.cpp:
  • editing/WrapContentsInDummySpanCommand.cpp:
  • editing/WrapContentsInDummySpanCommand.h:
  • editing/markup.cpp:
  • editing/markup.h:
  • html/HTMLAnchorElement.cpp:
  • html/HTMLBodyElement.cpp:
  • html/HTMLButtonElement.h:
  • html/HTMLCollection.h:
  • html/HTMLDocument.h:
  • html/HTMLEmbedElement.cpp:
  • html/HTMLFormElement.cpp:
  • html/HTMLFormElement.h:
  • html/HTMLFrameElement.cpp:
  • html/HTMLFrameSetElement.cpp:
  • html/HTMLKeygenElement.cpp:
  • html/HTMLLabelElement.cpp:
  • html/HTMLObjectElement.cpp:
  • html/HTMLObjectElement.h:
  • html/HTMLTableElement.cpp:
  • html/HTMLTablePartElement.cpp:
  • icon/IconDatabase.h:
  • kcanvas/KCanvasPath.h:
  • kcanvas/KCanvasResources.cpp:
  • kcanvas/KCanvasResources.h:
  • kcanvas/KCanvasTreeDebug.cpp:
  • kcanvas/KCanvasTreeDebug.h:
  • kcanvas/RenderSVGContainer.cpp:
  • kcanvas/RenderSVGText.h:
  • kcanvas/device/KRenderingPaintServer.h:
  • kcanvas/device/KRenderingPaintServerGradient.h:
  • kcanvas/device/KRenderingPaintServerSolid.cpp:
  • ksvg2/misc/SVGImageLoader.cpp:
  • ksvg2/svg/SVGElement.cpp:
  • ksvg2/svg/SVGElement.h:
  • ksvg2/svg/SVGForeignObjectElement.cpp:
  • ksvg2/svg/SVGForeignObjectElement.h:
  • ksvg2/svg/SVGImageElement.cpp:
  • ksvg2/svg/SVGPoint.h:
  • ksvg2/svg/SVGStylable.h:
  • ksvg2/svg/SVGTRefElement.cpp:
  • ksvg2/svg/SVGTRefElement.h:
  • ksvg2/svg/SVGTSpanElement.cpp:
  • ksvg2/svg/SVGTSpanElement.h:
  • ksvg2/svg/SVGTextElement.h:
  • ksvg2/svg/svgpathparser.cpp:
  • ksvg2/svg/svgpathparser.h:
  • loader/Cache.h:
  • loader/DocLoader.h:
  • loader/LoaderFunctions.h:
  • loader/mac/LoaderFunctionsMac.mm:
  • page/Frame.h:
  • page/FrameView.h:
  • page/Page.h:
  • page/Settings.h:
  • platform/AffineTransform.cpp:
  • platform/AffineTransform.h:
  • platform/Arena.cpp:
  • platform/Arena.h:
  • platform/CookieJar.h:
  • platform/DeprecatedCString.cpp:
  • platform/DeprecatedCString.h:
  • platform/DeprecatedPtrList.h:
  • platform/DeprecatedPtrListImpl.cpp:
  • platform/DeprecatedPtrListImpl.h:
  • platform/DeprecatedPtrQueue.h:
  • platform/DeprecatedString.cpp:
  • platform/DeprecatedString.h:
  • platform/DeprecatedStringList.cpp:
  • platform/DeprecatedStringList.h:
  • platform/DeprecatedValueList.h:
  • platform/DeprecatedValueListImpl.cpp:
  • platform/DeprecatedValueListImpl.h:
  • platform/FileButton.h:
  • platform/FloatPoint.h:
  • platform/FloatRect.h:
  • platform/FloatSize.h:
  • platform/FontData.h:
  • platform/FontFallbackList.h:
  • platform/GraphicsContext.h:
  • platform/IntPoint.h:
  • platform/IntRect.h:
  • platform/IntSize.h:
  • platform/KURL.cpp:
  • platform/KURL.h:
  • platform/ListBox.h:
  • platform/Logging.cpp:
  • platform/Logging.h:
  • platform/Pen.h:
  • platform/PopUpButton.h:
  • platform/RegularExpression.cpp:
  • platform/RegularExpression.h:
  • platform/SSLKeyGenerator.h:
  • platform/ScrollBar.h:
  • platform/SegmentedString.h:
  • platform/Shared.h:
  • platform/Slider.h:
  • platform/StringImpl.h:
  • platform/TextBox.h:
  • platform/TextField.h:
  • platform/TextStream.cpp:
  • platform/TextStream.h:
  • platform/TransferJob.h:
  • platform/TransferJobClient.h:
  • platform/Widget.h:
  • platform/cg/AffineTransformCG.cpp:
  • platform/mac/ClipboardMac.h:
  • platform/mac/ClipboardMac.mm:
  • platform/mac/DeprecatedStringListMac.mm:
  • platform/mac/FontCacheMac.mm:
  • platform/mac/KURLMac.mm:
  • platform/mac/ListBoxMac.mm:
  • platform/mac/PopUpButtonMac.mm:
  • platform/mac/SSLKeyGeneratorMac.mm:
  • platform/mac/TextBoxMac.mm:
  • platform/mac/TextFieldMac.mm:
  • platform/mac/WebCoreTextArea.h:
  • platform/mac/WebCoreTextArea.mm:
  • platform/mac/WebCoreTextField.h:
  • platform/mac/WebCoreTextField.mm:
  • rendering/DeprecatedRenderSelect.h:
  • rendering/RenderArena.cpp:
  • rendering/RenderArena.h:
  • rendering/RenderBlock.cpp:
  • rendering/RenderBox.cpp:
  • rendering/RenderContainer.cpp:
  • rendering/RenderFlexibleBox.cpp:
  • rendering/RenderFlow.cpp:
  • rendering/RenderLayer.cpp:
  • rendering/RenderLineEdit.cpp:
  • rendering/RenderObject.h:
  • rendering/RenderStyle.cpp:
  • rendering/RenderStyle.h:
  • rendering/RenderText.cpp:
  • rendering/RenderTextFragment.cpp:
  • rendering/RenderTheme.h:
  • rendering/RenderTreeAsText.cpp:
  • rendering/RenderTreeAsText.h:
  • rendering/RenderView.h:
  • rendering/bidi.h:
  • xml/XSLStyleSheet.cpp:
  • xml/XSLTProcessor.cpp:
22:18 Changeset [15285] by ggaren

LayoutTests:

Reviewed by Beth.


Now that we garbage collect after reinitalizing the global object, not
before, we have more free memory, so a custom property that used to
get collected during the initialization phase of this test now avoids
getting collected until the collection phase.

  • fast/dom/gc-9-expected.txt:
  • fast/dom/gc-9.html:

WebCore:

Reviewed by Beth.

  • bindings/js/kjs_window.cpp: (KJS::Window::clear): Garbage collect after reinitalizing the global object, not before, since the reinitialization tends to create garbage.
21:45 Changeset [15284] by ddkilzer

WebKitTools:

Build fix. Reviewed by NOBODY.

  • Scripts/webkitdirs.pm: Backed out previous change.
21:25 Changeset [15283] by thatcher

Adding 2006 to the copyright years for headers I changed earlier.

21:18 Changeset [15282] by thatcher

FIx the copyright dates to include all years published.

21:05 Changeset [15281] by thatcher

Build fix for DumpRenderTree and other clients that don't use AppKit. Update the copyright date on headers I changed earlier.

21:00 Changeset [15280] by ddkilzer

LayoutTests:

Reviewed by Darin.

  • svg/custom/foreign-object-skew-expected.checksum: Updated.
  • svg/custom/foreign-object-skew-expected.png: Updated.
20:46 Changeset [15279] by ddkilzer

WebKitTools:

Reviewed by Darin.

  • Scripts/svn-apply: Set the ChangeLog entry date using a configurable timezone before applying the patch.
  • Scripts/svn-unapply: Reset the ChangeLog entry date before unapplying the patch.
20:26 Changeset [15278] by tomernic

Reviewed by Maciej.

<rdar://problem/4404652> Netscape plug-in mouse events broken in HiDPI

Multiply global mouse coordinates by the window scale factor so that plug-ins can use GlobalToLocal() in HiDPI.
This fixes many bugs involving plug-in mouse event handling in HiDPI. Most notably, the Flash player will now
correctly respond to clicks.

  • Plugins/WebBaseNetscapePluginView.m: (+[WebBaseNetscapePluginView getCarbonEvent:]): (-[WebBaseNetscapePluginView getCarbonEvent:withEvent:]):
20:13 Changeset [15277] by darin

Reviewed by Tim Hatcher.

  • fix assertion firing in plug-in layout tests
  • Plugins/WebBaseNetscapePluginView.m: (-[WebBaseNetscapePluginView restartNullEvents]): Don't start null events if the plug-in is not in the started state. This happens when the plug-in moves within its view hierarchy after it has been stopped.
20:02 Changeset [15276] by thatcher

WebCore:

Reviewed by Darin.

Bug 9820: Move new DOM API that has been through API review to public headers
http://bugzilla.opendarwin.org/show_bug.cgi?id=9820

  • bindings/objc/DOM.mm: (-[DOMElement focus]): (-[DOMElement blur]): (-[DOMRange text]): (-[DOMRange _text]):
  • bindings/objc/DOMCSS.mm: (-[DOMRGBColor color]): (-[DOMRGBColor _color]):
  • bindings/objc/DOMCore.h:
  • bindings/objc/DOMEvents.h:
  • bindings/objc/DOMExtensions.h:
  • bindings/objc/DOMHTML.mm: (-[DOMHTMLElement titleDisplayString]): (-[DOMHTMLDocument createDocumentFragmentWithMarkupString:baseURL:]): (-[DOMHTMLDocument createDocumentFragmentWithText:]): (-[DOMHTMLDocument _createDocumentFragmentWithMarkupString:baseURLString:]): (-[DOMHTMLDocument _createDocumentFragmentWithText:]):
  • bindings/objc/DOMPrivate.h:
  • bindings/objc/DOMXPath.h:
  • bindings/objc/DOMXPath.mm:

WebKit:

Reviewed by Darin.

Bug 9820: Move new DOM API that has been through API review to public headers
http://bugzilla.opendarwin.org/show_bug.cgi?id=9820

  • Misc/WebElementDictionary.m: include DOMExtensions.h
  • Misc/WebNSViewExtras.m: include DOMExtensions.h
  • WebKit.xcodeproj/project.pbxproj: make DOMXPath.h public
19:49 Changeset [15275] by thatcher

Reviewed by Kevin.

Bug 9818: move new UIDelegate API that has been through API review to public headers
http://bugzilla.opendarwin.org/show_bug.cgi?id=9818

<rdar://problem/4387541> API: Remove webView:setContentRect: & webViewContentRect: delegate methods?
The fix for 4310363 removed the only use of webViewContentRect: in our code. webView:setContentRect:
was never used to begin with. There's no harm in leaving these around in the API, but they'll cruft it up.

Also removes the never used webViewPrint: SPI that was replaced by webView:printFrameView:.

  • DefaultDelegates/WebDefaultUIDelegate.m:
  • WebCoreSupport/WebFrameBridge.m: (-[WebFrameBridge print]):
  • WebView/WebFrameView.h:
  • WebView/WebFrameView.m:
  • WebView/WebFrameViewPrivate.h:
  • WebView/WebUIDelegate.h:
  • WebView/WebUIDelegatePrivate.h:
19:43 Changeset [15274] by darin
  • fast/dom/delete-contents-expected.txt: Updated results that changed because we now keep comment nodes in the DOM.
19:31 Changeset [15273] by bdakin

Reviewed by Maciej.

Fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=9749 A
nested content editable div causes unnecessary screen redraws.


We were repainting too much because we were deciding that we needed
a full layout because we didn't have a first line box. Our code
used to hold a lot of assumptions that positioned elements would
always have line boxes, but I fixed a few bugs a while back that
appeared because this assumption isn't true. This assumption was
also causing the extra redraws because the check meant that we
would decide to do a full layout for any positioned element that
did not have any line boxes in the first place. Maciej and I talked
about this, and we couldn't figure out why the check for line boxes
was part of determining if we need a full layout in the first
place. After hunting around the code, we decided to assume that
this check is antiquated. Removing it fixes the bug and doesn't
seem to break any layout tests. So, fingers crossed!

Will add layout test so soon.

  • rendering/bidi.cpp: (WebCore::RenderBlock::layoutInlineChildren): Remove firstLineBox check.
17:57 Changeset [15272] by thatcher

Reviewed by Darin.

Bug 9814: Move new WebView API that has been through API review to public headers
http://bugzilla.opendarwin.org/show_bug.cgi?id=9814

  • WebView/WebView.h:
  • WebView/WebView.m: (-[WebView close]): (-[WebView setShouldCloseWithWindow:]): (-[WebView shouldCloseWithWindow]): (-[WebView selectedFrame]): (-[WebView setMainFrameURL:]): (-[WebView mainFrameURL]): (-[WebView isLoading]): (-[WebView mainFrameTitle]): (-[WebView mainFrameIcon]): (-[WebView mainFrameDocument]): (-[WebView setDrawsBackground:]): (-[WebView drawsBackground]): (-[WebView toggleSmartInsertDelete:]): (-[WebView toggleContinuousSpellChecking:]): (-[WebView canMakeTextStandardSize]): (-[WebView makeTextStandardSize:]): (-[WebView maintainsInactiveSelection]):
  • WebView/WebViewPrivate.h:
17:11 Changeset [15271] by thatcher

Reviewed by Maciej.

Bug 9487: The XPath section should be removed and/or moved.
http://bugzilla.opendarwin.org/show_bug.cgi?id=9487

16:18 Changeset [15270] by harrison

LayoutTests:

Reviewed by John Sullivan.

<rdar://problem/4598149> Deleting the contents of a DOMRange and then inserting a node into it crashes in WebCore::Font::canUseGlyphCache

  • fast/dom/delete-contents-expected.checksum: Added.
  • fast/dom/delete-contents-expected.png: Added.
  • fast/dom/delete-contents-expected.txt: Added.
  • fast/dom/delete-contents.html: Added.

WebCore:

Reviewed by John Sullivan.

<rdar://problem/4598149> Deleting the contents of a DOMRange and then inserting a node into it crashes in WebCore::Font::canUseGlyphCache

Test: fast/dom/delete-contents.html


  • dom/Range.cpp: (WebCore::Range::processContents):
15:48 Changeset [15269] by darin
  • another round of renaming; and KWQ is dead
  • WebCore.vcproj/WebCore/WebCore.vcproj: Remove paths for kwq and khtml directories. Move files to their new locations. Did rename.
  • WebCore.xcodeproj/project.pbxproj: Move files to their new locations. Did rename.
  • WebCoreSources.bkl: Did rename.
  • webcore-base.bkl: Removed kwq.
  • kwq/AccessibilityObjectCache.h: Removed.
  • kwq/AccessibilityObjectCache.mm: Removed.
  • kwq/FormDataMac.h: Removed.
  • kwq/FormDataMac.mm: Removed.
  • kwq/WebCoreAXObject.h: Removed.
  • kwq/WebCoreAXObject.mm: Removed.
  • kwq/WebCoreEditCommand.h: Removed.
  • kwq/WebCoreEditCommand.mm: Removed.
  • kwq/WebCorePageState.h: Removed.
  • kwq/WebCorePageState.mm: Removed.
  • kwq/WebCoreResourceLoaderImp.h: Removed.
  • kwq/WebCoreResourceLoaderImp.mm: Removed.
  • bridge/AXObjectCache.h: Added.
  • bridge/mac/AXObjectCacheMac.mm: Added.
  • bridge/mac/WebCoreAXObject.h: Added.
  • bridge/mac/WebCoreAXObject.mm: Added.
  • loader/CachedObject.cpp: Removed.
  • loader/CachedObject.h: Removed.
  • loader/CachedObjectClient.h: Removed.
  • loader/CachedObjectClientWalker.cpp: Removed.
  • loader/CachedObjectClientWalker.h: Removed.
  • loader/CachedResource.cpp: Added.
  • loader/CachedResource.h: Added.
  • loader/CachedResourceClient.h: Added.
  • loader/CachedResourceClientWalker.cpp: Added.
  • loader/CachedResourceClientWalker.h: Added.
  • bridge/mac/FrameMac.mm:
  • bridge/mac/WebCoreFrameBridge.mm:
  • css/CSSImageValue.h:
  • css/CSSImportRule.h:
  • dom/Document.cpp:
  • dom/Document.h:
  • dom/Notation.h:
  • dom/ProcessingInstruction.h:
  • dom/XMLTokenizer.cpp:
  • html/CanvasPattern.h:
  • html/HTMLDocument.h:
  • html/HTMLImageLoader.cpp:
  • html/HTMLImageLoader.h:
  • html/HTMLLinkElement.h:
  • html/HTMLScriptElement.cpp:
  • html/HTMLScriptElement.h:
  • html/HTMLTokenizer.cpp:
  • html/HTMLTokenizer.h:
  • kcanvas/KCanvasFilters.h:
  • ksvg2/svg/SVGCursorElement.cpp:
  • ksvg2/svg/SVGCursorElement.h:
  • ksvg2/svg/SVGFEImageElement.cpp:
  • ksvg2/svg/SVGFEImageElement.h:
  • loader/Cache.cpp:
  • loader/Cache.h:
  • loader/CachedCSSStyleSheet.cpp:
  • loader/CachedCSSStyleSheet.h:
  • loader/CachedImage.cpp:
  • loader/CachedImage.h:
  • loader/CachedScript.cpp:
  • loader/CachedScript.h:
  • loader/CachedXBLDocument.cpp:
  • loader/CachedXBLDocument.h:
  • loader/CachedXSLStyleSheet.cpp:
  • loader/CachedXSLStyleSheet.h:
  • loader/DocLoader.cpp:
  • loader/DocLoader.h:
  • loader/LoaderFunctions.h:
  • loader/Request.cpp:
  • loader/Request.h:
  • loader/loader.cpp:
  • loader/loader.h:
  • loader/mac/LoaderFunctionsMac.mm:
  • page/Frame.cpp:
  • page/FrameView.cpp:
  • platform/gdk/TemporaryLinkStubs.cpp:
  • platform/mac/ClipboardMac.h:
  • platform/win/TemporaryLinkStubs.cpp:
  • rendering/RenderBox.h:
  • rendering/RenderContainer.cpp:
  • rendering/RenderImage.cpp:
  • rendering/RenderImage.h:
  • rendering/RenderLayer.h:
  • rendering/RenderObject.cpp:
  • rendering/RenderObject.h:
  • rendering/RenderStyle.cpp:
  • rendering/RenderStyle.h:
  • xml/XSLImportRule.h: Did renaming.
  • platform/mac/can-convert.mm: Removed.
15:39 Changeset [15268] by darin
  • Scripts/do-webcore-rename: Final version of this round of renaming for posterity.
15:35 Changeset [15267] by ddkilzer

Remove conflict marker.

14:42 Changeset [15266] by andersca

Remove conflict marker. Thanks Mitz!

14:39 Changeset [15265] by andersca

WebCore:

2006-07-09 Anders Carlsson <acarlsson@apple.com>

Reviewed by Tim O.

  • WebCore.exp: Add wkPathFromFont.


  • platform/mac/FontDataMac.mm: (-[NSFont WebCore]): Use wkPathFromFont.


  • platform/mac/WebCoreSystemInterface.h:
  • platform/mac/WebCoreSystemInterface.mm: Declare wkPathFromFont.

WebKit:

2006-07-09 Anders Carlsson <acarlsson@apple.com>

Reviewed by Tim O.

  • WebCoreSupport/WebSystemInterface.m: (InitWebCoreSystemInterface): Initialize wkPathFromFont.
14:31 Changeset [15264] by andersca

2006-07-09 Anders Carlsson <acarlsson@apple.com>

Reviewed by Darin.

  • WebKitSystemInterface.h:
  • libWebKitSystemInterface.a: Add WKPathFromFont.
14:11 Changeset [15263] by darin
  • fix release build
  • Plugins/WebBaseNetscapePluginView.m: (-[WebBaseNetscapePluginView restorePortState:]): Cast inside the assertion so that we don't have an unused variable in versions with assertions disabled. The alternative would be to wrap the whole thing in an #if statement.
13:37 Changeset [15262] by tomernic

JavaScriptCore:

Reviewed by Maciej.

Added an OpenGL drawing model to the Netscape Plug-in API.

  • bindings/npapi.h:

WebKit:

Reviewed by John Sullivan.

  • Plugins/WebBaseNetscapePluginView.h:
  • Added ivars for OpenGL support. Someday it would be nice to refactor this class so that each drawing model is encapsulated in a class; this would allow WebBaseNetscapePluginView to make more efficient use of space, for example by not keeping OpenGL-related ivars for Quickdraw plug-ins.


  • Plugins/WebBaseNetscapePluginView.m:
  • Declared a bunch of internal methods for OpenGL support (see below).
  • Removed "forUpdate" from CoreGraphics port state struct; it was always set to "YES", so I just cleaned up the silly code that used it.
  • Declared OpenGL port state struct. (-[WebBaseNetscapePluginView saveAndSetNewPortStateForUpdate:]):
  • Moved a CoreGraphics-related assertion down to the big "switch" statement.
  • Don't set window.type here -- according to the Netscape Plug-in API docs, the plug-in should default to "windowed" mode, and may call NPN_SetValue() during its NPN_New() to request that the browser use a "windowless" (offscreen) context instead.
  • Moved the assertion from the top of this method here; removed a less restrictive assertion that is now obsolete.
  • Removed "forUpdate" flag from CoreGraphics port state struct.
  • Fill in OpenGL port state struct. Set up the viewport appropriately for both windowed and windowless OpenGL plug-ins. Windowed plug-ins need to have their GL viewport transformed by the amount the plug-in is clipped; windowless plug-ins are drawn off-screen into a surface whose geometry is never changed or clipped, so they may always draw with a viewport origin of (0, 0). (-[WebBaseNetscapePluginView restorePortState:]):
  • Removed "forUpdate" flag from CoreGraphics port state struct.
  • Restore the old OpenGL context saved by -saveAndSetNewPortStateForUpdate:. (-[WebBaseNetscapePluginView sendEvent:]):
  • Updated an assertion to also include OpenGL. To ensure that attached plug-in window movements happen atomically with web page redisplays, we assert that the plug-in's window is set only while the plug-in view is redrawing.
  • Same deal as with the assertion; only save/set port state when redrawing the plug-in view. Plug-ins that use the new drawing models are only allowed to draw when the web page draws. I might consider changing this for windowed OpenGL plug-ins, since they always obscure the page content anyway. (-[WebBaseNetscapePluginView isNewWindowEqualToOldWindow]):
  • Compare new NP_GLContext structs. (-[WebBaseNetscapePluginView updateAndSetWindow]):
  • In OpenGL mode, can only set window when updating plug-in view. (-[WebBaseNetscapePluginView setWindowIfNecessary]):
  • ditto
  • Updated logging for OpenGL drawing mode. (-[WebBaseNetscapePluginView addWindowObservers]):
  • No need to observe frame/bounds change notifications for this and all parent views. See -renewGState comments below. (-[WebBaseNetscapePluginView removeWindowObservers]):
  • Don't need to remove frame/bounds observers anymore. (-[WebBaseNetscapePluginView start]):
  • Plug-ins are "windowed" by default. This is not a change from our previous behavior, but this is a better place to set the default value as it allows the plug-in to override it later. (-[WebBaseNetscapePluginView stop]):
  • Destroy AGL context when the plug-in stops. (-[WebBaseNetscapePluginView dealloc]):
  • Assert that the AGL stuff has been cleaned up. (-[WebBaseNetscapePluginView drawRect:]):
  • If this is a windowless OpenGL plugin, blit its contents back into this view. (-[WebBaseNetscapePluginView renewGState]):
  • This method is called when the view or one of its parents is moved or resized (see comments). (-[WebBaseNetscapePluginView viewWillMoveToWindow:]):
  • Hide the AGL window if the plug-in view is about to be removed from its window. (-[WebBaseNetscapePluginView viewHasMoved:]):
  • Renamed and moved to the "Internal" category. (-[WebBaseNetscapePluginView invalidateRegion:]):
  • Style changes.
  • Add support for OpenGL (uses the same region type as CoreGraphics). (-[WebBaseNetscapePluginView getVariable:value:]):
  • Style changes.
  • Implemented NPNVsupportsOpenGLBool; returns YES since we now support the OpenGL drawing model. (-[WebBaseNetscapePluginView setVariable:value:]):
  • Implemented NPPVpluginWindowBool, which allows plug-ins to specify whether they should be rendered in "windowed" or "windowless" mode. This is an older part of the Netscape Plug-in API that was never implemented in WebKit. "Windowed" Quickdraw plug-ins do not actually reside in a separate window, and can already do many of the same things (such as transparency) that only "windowless" plug-ins can do on other platforms. However, we need the "windowed" vs. "windowless" distinction for OpenGL plug-ins so that they have some way of specifying whether they should be rendered on an accelerated overlay surface, composited into the browser window.
  • Support for setting the drawing model to OpenGL. (-[WebBaseNetscapePluginView _viewHasMoved]):
  • Renamed from -viewHasMoved:, and moved down in the file.
  • None of this work is necessary when the plug-in is not in a window; the plug-in's state will be properly restored when it is moved back into a window.
  • Reshape OpenGL surface window here. (-[WebBaseNetscapePluginView _createAGLContextIfNeeded]):
  • Creates the AGL context of the appropriate type (windowed/windowless). (-[WebBaseNetscapePluginView _createWindowedAGLContext]):
  • Creates a windowed AGL context, which is an AGL context attached to a child window. This is the only way to get true hardware acceleration. (-[WebBaseNetscapePluginView _createWindowlessAGLContext]):
  • Creates a windowless AGL context, which is an AGL context attached to an offscreen buffer. This buffer can then be blitted back into the browser window with a different alpha, or scaled, or whatever. (-[WebBaseNetscapePluginView _cglContext]):
  • Returns the underlying CGL context from the AGL context. We give the plug-in access to the CGL context because CGL is the more primitive of the GL drawable APIs and allows for finer control over the context. (-[WebBaseNetscapePluginView _getAGLOffscreenBuffer:width:height:]):
  • Returns the buffer allocated for the offscreen AGL context, if there is one. (-[WebBaseNetscapePluginView _destroyAGLContext]):
  • Destroys the AGL context, as well as the associated offscreen buffer or child window. (-[WebBaseNetscapePluginView _reshapeAGLWindow]):
  • Positions the AGL window over the browser window. (-[WebBaseNetscapePluginView _hideAGLWindow]):
  • Hides the AGL window. (-[WebBaseNetscapePluginView _aglOffscreenImageForDrawingInRect:]):
  • Returns an NSImage representation of the offscreen AGL context's framebuffer. This is used to draw the offscreen bits back into the plug-in view. This is kind of tricky because it has to convert the offscreen buffer in-place from BGRA to RGBA so that it can be wrapped in an NSBitmapImageRep. See comments.
  • WebKit.xcodeproj/project.pbxproj: Link OpenGL and AGL.
11:10 Changeset [15261] by beidson

Fixed a nearly unparsable ChangeLog entry

10:52 Changeset [15260] by beidson

Reviewed by Maciej

Set the stage to remove the workaround for the SQLite BLOB corruption. A few other small cleanups, and
preparation for pruning unreferenced and unretained icons.

  • icon/IconDatabase.cpp: (WebCore::IconDatabase::recreateDatabase): Added another trigger to assist in icon removal (WebCore::IconDatabase::deletePrivateTables): Cleaned up logging messages (WebCore::IconDatabase::imageDataForIconID): #ifdefed the blobbing hack for impending removal, use the real blob by default (WebCore::IconDatabase::imageDataForIconURL): same (WebCore::IconDatabase::imageDataForPageURL): same (WebCore::IconDatabase::pruneUnreferencedIcons): Will delete any icons and their data that are not referenced by any PageURL
  • icon/IconDatabase.h:
  • icon/SQLDatabase.h: Changed BlobAsVector to be unsigned char as thats the most reasonable type for a byte-buffer, and is what CFData expects
  • icon/SQLStatement.cpp: (WebCore::SQLStatement::getColumnBlobAsVector):
10:38 Changeset [15259] by beidson

Reviewed by Maciej

The ICONDEBUG flag no choose either/or the new icon database and the old one
No longer any need to live side by side to compare results

  • Misc/WebIconDatabase.m: (-[NSMutableDictionary iconURLForURL:]): (-[NSMutableDictionary retainIconForURL:]): (-[NSMutableDictionary releaseIconForURL:]): (-[WebIconDatabase _setHaveNoIconForIconURL:]): (-[WebIconDatabase _setIconURL:forURL:]): (-[WebIconDatabase _resetCachedWebPreferences:]):
10:03 Changeset [15258] by darin
  • move all but the last 12 files out of kwq directory
  • removed a few unused source files I discovered
  • WebCore.vcproj/WebCore/WebCore.vcproj: Update for new file locations.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • bridge/mac/FrameMac.mm:
  • dom/Element.h:
  • html/HTMLImageLoader.cpp:
  • loader/Cache.cpp:
  • loader/Cache.h:
  • loader/CachedCSSStyleSheet.cpp:
  • loader/CachedCSSStyleSheet.h:
  • loader/CachedImage.cpp:
  • loader/CachedImage.h:
  • loader/CachedObject.h:
  • loader/CachedScript.cpp:
  • loader/CachedScript.h:
  • loader/CachedXBLDocument.cpp:
  • loader/CachedXBLDocument.h:
  • loader/CachedXSLStyleSheet.cpp:
  • loader/CachedXSLStyleSheet.h:
  • loader/DocLoader.cpp:
  • loader/DocLoader.h:
  • page/Frame.cpp:
  • page/Frame.h:
  • page/FramePrivate.h:
  • platform/gdk/TemporaryLinkStubs.cpp:
  • platform/win/TemporaryLinkStubs.cpp:
  • rendering/DeprecatedSlider.cpp:
  • rendering/RenderObject.h:
  • xml/xmlhttprequest.cpp: Updated includes as needed for file location changes. Changed CachePolicy uses to match new filename and enum names.
  • loader/CachedResource.cpp: Removed.
  • loader/CachedResource.h: Removed.
  • loader/CachedResourceClient.h: Removed.
  • loader/CachedResourceClientWalker.cpp: Removed.
  • loader/CachedResourceClientWalker.h: Removed. The above files were checked in by accident a while back. The rename of CachedObject to CachedResource is now reflected in do-webcore-rename, and will be done soon.
  • kwq/AffineTransform.cpp: Removed.
  • kwq/AffineTransform.h: Removed.
  • kwq/CacheControl.h: Removed.
  • kwq/DeprecatedPtrQueue.h: Removed.
  • kwq/FileButton.h: Removed.
  • kwq/FileButton.mm: Removed.
  • kwq/ListBox.h: Removed.
  • kwq/ListBox.mm: Removed.
  • kwq/LoaderFunctions.h: Removed.
  • kwq/LoaderFunctions.mm: Removed.
  • kwq/PopUpButton.h: Removed.
  • kwq/PopUpButton.mm: Removed.
  • kwq/SSLKeyGenerator.h: Removed.
  • kwq/SSLKeyGenerator.mm: Removed.
  • kwq/Settings.h: Removed.
  • kwq/TextBox.h: Removed.
  • kwq/TextBox.mm: Removed.
  • kwq/TextField.h: Removed.
  • kwq/TextField.mm: Removed.
  • kwq/WebCoreScrollBar.h: Removed.
  • kwq/WebCoreScrollBar.mm: Removed.
  • kwq/WebCoreSlider.h: Removed.
  • kwq/WebCoreSlider.mm: Removed.
  • loader/CachePolicy.h: Added.
  • loader/LoaderFunctions.h: Added.
  • loader/mac/LoaderFunctionsMac.mm: Added.
  • page/Settings.h: Added.
  • platform/AffineTransform.cpp: Added.
  • platform/AffineTransform.h: Added.
  • platform/DeprecatedPtrQueue.h: Added.
  • platform/FileButton.h: Added.
  • platform/ListBox.h: Added.
  • platform/PopUpButton.h: Added.
  • platform/SSLKeyGenerator.h: Added.
  • platform/ScrollBar.h: Added.
  • platform/Slider.h: Added.
  • platform/TextBox.h: Added.
  • platform/TextField.h: Added.
  • platform/cg/AffineTransformCG.cpp: Added.
  • platform/mac/FileButtonMac.mm: Added.
  • platform/mac/ListBoxMac.mm: Added.
  • platform/mac/PopUpButtonMac.mm: Added.
  • platform/mac/SSLKeyGeneratorMac.mm: Added.
  • platform/mac/ScrollBarMac.mm: Added.
  • platform/mac/ScrollBarMac.mm: Added.
  • platform/mac/SliderMac.mm: Added.
  • platform/mac/SliderMac.mm: Added.
  • platform/mac/TextBoxMac.mm: Added.
  • platform/mac/TextFieldMac.mm: Added.
09:36 Changeset [15257] by darin
  • Scripts/do-webcore-rename: Next round of renaming. (Last round was done.)
08:53 Changeset [15256] by ddkilzer

LayoutTests:

Reviewed by Darin.

  • fast/js/resources/js-test-pre.js: Change description() to work with MSIE 6.
08:42 Changeset [15255] by darin
  • move 4 files out of kwq directory
  • make some minor project adjustments for the WebCore project
  • WebCore.xcodeproj/project.pbxproj: Link libsqlite3 the same way we do other libraries, rather than using a custom linker option. Moved the files from their old locations to the new ones. Removed "khtml" from the list of header search paths since that directory doesn't exist any more. Eliminated STYLE_LDFLAGS and just put -umbrella WebKit in OTHER_LDFLAGS in the Production configuration (was a leftover from "build styles" in Xcode 1). Changed to use EXPORTED_SYMBOLS_FILE instead of using -exported_symbols_list directly again now that Xcode has fixed the "strip twice" bug. Removed unneeded explicit SECTORDER_FLAGS = "" settings. Removed the explicit -Y,3 option now that Xcode does that by default.
  • WebCore.vcproj/WebCore/WebCore.vcproj: Removed some obsolete uneeded files, many of which didn't exist any more. Moved the files from their old locations to the new ones.
  • WebCoreSources.bkl: Moved the files from their old locations to the new ones.
  • kwq/DeprecatedCString.cpp: Removed.
  • kwq/DeprecatedCString.h: Removed.
  • kwq/TextStream.cpp: Removed.
  • kwq/TextStream.h: Removed.
  • platform/DeprecatedCString.cpp: Added.
  • platform/DeprecatedCString.h: Added.
  • platform/TextStream.cpp: Added.
  • platform/TextStream.h: Added.
08:23 Changeset [15254] by ddkilzer

WebKitTools:

Reviewed by NOBODY (build fix).

  • Scripts/webkitdirs.pm: Restore Hyatt's original logic to make buildbot work again. Add VC++ Express check as the fallback.
08:10 Changeset [15253] by darin

Rubber stamped by Maciej (kinda).

  • did the next pass of renaming (used do-webcore-rename script) this takes care of most of the remaining KWQ names (almost all)
  • WebCore+SVG/DOMList.h:
  • WebCore.vcproj/WebCore/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • bindings/js/kjs_dom.cpp:
  • bindings/js/kjs_window.cpp:
  • bindings/objc/DOMHTML.mm:
  • bridge/mac/FrameMac.h:
  • bridge/mac/FrameMac.mm:
  • bridge/mac/WebCoreFrameBridge.mm:
  • bridge/mac/WebCoreSettings.h:
  • bridge/mac/WebCoreSettings.mm:
  • bridge/win/FrameWin.cpp:
  • css/CSSPrimitiveValue.cpp:
  • css/CSSValueKeywords.in:
  • css/MediaQueryEvaluator.cpp:
  • css/cssparser.cpp:
  • css/cssstyleselector.cpp:
  • css/cssstyleselector.h:
  • dom/CharacterData.cpp:
  • dom/CharacterData.h:
  • dom/Document.cpp:
  • dom/Element.cpp:
  • dom/Element.h:
  • dom/EventTargetNode.cpp:
  • dom/EventTargetNode.h:
  • dom/Node.cpp:
  • dom/Node.h:
  • dom/Position.cpp:
  • dom/ProcessingInstruction.cpp:
  • dom/QualifiedName.cpp:
  • dom/QualifiedName.h:
  • dom/XMLTokenizer.cpp: Added.
  • dom/xml_tokenizer.cpp: Removed.
  • dom/xml_tokenizer.h: Removed.
  • editing/EditAction.h:
  • editing/HTMLInterchange.cpp:
  • editing/HTMLInterchange.h:
  • editing/Selection.h:
  • editing/SelectionController.h:
  • editing/TextAffinity.h:
  • editing/TextIterator.cpp:
  • editing/TextIterator.h:
  • editing/VisiblePosition.h:
  • editing/htmlediting.cpp:
  • editing/markup.cpp:
  • editing/markup.h:
  • editing/visible_units.cpp:
  • editing/visible_units.h:
  • html/HTMLDocument.cpp:
  • html/HTMLFontElement.cpp:
  • html/HTMLInputElement.cpp:
  • html/HTMLKeygenElement.cpp:
  • html/HTMLSelectElement.h:
  • html/HTMLTextAreaElement.cpp:
  • html/HTMLTextFieldInnerElement.cpp:
  • html/HTMLTokenizer.cpp:
  • html/HTMLTokenizer.h:
  • kcanvas/KCanvasContainer.cpp: Removed.
  • kcanvas/KCanvasContainer.h: Removed.
  • kcanvas/KCanvasCreator.cpp:
  • kcanvas/KCanvasFilters.cpp:
  • kcanvas/KCanvasFilters.h:
  • kcanvas/KCanvasMatrix.cpp:
  • kcanvas/KCanvasMatrix.h:
  • kcanvas/KCanvasPath.cpp:
  • kcanvas/KCanvasPath.h:
  • kcanvas/KCanvasResources.cpp:
  • kcanvas/KCanvasResources.h:
  • kcanvas/KCanvasTreeDebug.cpp:
  • kcanvas/KCanvasTreeDebug.h:
  • kcanvas/RenderForeignObject.cpp:
  • kcanvas/RenderForeignObject.h:
  • kcanvas/RenderPath.cpp:
  • kcanvas/RenderPath.h:
  • kcanvas/RenderSVGContainer.cpp: Added.
  • kcanvas/RenderSVGContainer.h: Added.
  • kcanvas/RenderSVGImage.cpp:
  • kcanvas/RenderSVGImage.h:
  • kcanvas/RenderSVGText.cpp:
  • kcanvas/RenderSVGText.h:
  • kcanvas/device/KRenderingPaintServer.h:
  • kcanvas/device/KRenderingPaintServerGradient.cpp:
  • kcanvas/device/KRenderingPaintServerGradient.h:
  • kcanvas/device/KRenderingPaintServerPattern.cpp:
  • kcanvas/device/KRenderingPaintServerPattern.h:
  • kcanvas/device/KRenderingPaintServerSolid.cpp:
  • kcanvas/device/KRenderingPaintServerSolid.h:
  • kcanvas/device/quartz/KCanvasFilterQuartz.mm:
  • kcanvas/device/quartz/KCanvasItemQuartz.h:
  • kcanvas/device/quartz/KCanvasResourcesQuartz.h:
  • kcanvas/device/quartz/KRenderingDeviceQuartz.mm:
  • kcanvas/device/quartz/KRenderingPaintServerGradientQuartz.mm:
  • kcanvas/device/quartz/KRenderingPaintServerQuartz.mm:
  • ksvg2/css/CSSValueKeywords.in:
  • ksvg2/css/SVGRenderStyleDefs.cpp:
  • ksvg2/misc/KCanvasRenderingStyle.cpp:
  • ksvg2/misc/KSVGTimeScheduler.cpp:
  • ksvg2/svg/SVGAElement.cpp:
  • ksvg2/svg/SVGAnimateTransformElement.cpp:
  • ksvg2/svg/SVGAnimateTransformElement.h:
  • ksvg2/svg/SVGGElement.cpp:
  • ksvg2/svg/SVGHelper.cpp:
  • ksvg2/svg/SVGHelper.h:
  • ksvg2/svg/SVGImageElement.cpp:
  • ksvg2/svg/SVGLinearGradientElement.cpp:
  • ksvg2/svg/SVGMarkerElement.cpp:
  • ksvg2/svg/SVGMaskElement.cpp:
  • ksvg2/svg/SVGMatrix.cpp:
  • ksvg2/svg/SVGMatrix.h:
  • ksvg2/svg/SVGPatternElement.cpp:
  • ksvg2/svg/SVGRadialGradientElement.cpp:
  • ksvg2/svg/SVGSVGElement.cpp:
  • ksvg2/svg/SVGStyledElement.h:
  • ksvg2/svg/SVGStyledTransformableElement.cpp:
  • ksvg2/svg/SVGStyledTransformableElement.h:
  • ksvg2/svg/SVGSwitchElement.cpp:
  • ksvg2/svg/SVGTextElement.cpp:
  • ksvg2/svg/SVGTransformable.h:
  • ksvg2/svg/SVGUseElement.cpp:
  • kwq/AffineTransform.cpp: Added.
  • kwq/AffineTransform.h: Added.
  • kwq/DeprecatedCString.cpp: Added.
  • kwq/DeprecatedCString.h: Added.
  • kwq/DeprecatedPtrQueue.h: Added.
  • kwq/FileButton.h: Added.
  • kwq/FileButton.mm: Added.
  • kwq/FormDataMac.mm: Added.
  • kwq/KWQCString.cpp: Removed.
  • kwq/KWQCString.h: Removed.
  • kwq/KWQComboBox.h: Removed.
  • kwq/KWQComboBox.mm: Removed.
  • kwq/KWQEditCommand.h: Removed.
  • kwq/KWQEditCommand.mm: Removed.
  • kwq/KWQFileButton.h: Removed.
  • kwq/KWQFileButton.mm: Removed.
  • kwq/KWQFormData.h: Removed.
  • kwq/KWQFormData.mm: Removed.
  • kwq/KWQKHTMLSettings.h: Removed.
  • kwq/KWQKIOGlobal.h: Removed.
  • kwq/KWQKSSLKeyGen.h: Removed.
  • kwq/KWQKSSLKeyGen.mm: Removed.
  • kwq/KWQLineEdit.h: Removed.
  • kwq/KWQLineEdit.mm: Removed.
  • kwq/KWQListBox.h: Removed.
  • kwq/KWQListBox.mm: Removed.
  • kwq/KWQLoader.h: Removed.
  • kwq/KWQLoader.mm: Removed.
  • kwq/KWQPageState.h: Removed.
  • kwq/KWQPageState.mm: Removed.
  • kwq/KWQPtrQueue.h: Removed.
  • kwq/KWQResourceLoader.h: Removed.
  • kwq/KWQResourceLoader.mm: Removed.
  • kwq/KWQScrollBar.h: Removed.
  • kwq/KWQScrollBar.mm: Removed.
  • kwq/KWQSlider.h: Removed.
  • kwq/KWQSlider.mm: Removed.
  • kwq/KWQTextEdit.h: Removed.
  • kwq/KWQTextEdit.mm: Removed.
  • kwq/KWQTextStream.cpp: Removed.
  • kwq/KWQTextStream.h: Removed.
  • kwq/KWQWMatrix.cpp: Removed.
  • kwq/KWQWMatrix.h: Removed.
  • kwq/ListBox.h: Added.
  • kwq/ListBox.mm: Added.
  • kwq/LoaderFunctions.h: Added.
  • kwq/LoaderFunctions.mm: Added.
  • kwq/PopUpButton.h: Added.
  • kwq/PopUpButton.mm: Added.
  • kwq/SSLKeyGenerator.h: Added.
  • kwq/SSLKeyGenerator.mm: Added.
  • kwq/Settings.h: Added.
  • kwq/TextBox.h: Added.
  • kwq/TextBox.mm: Added.
  • kwq/TextField.h: Added.
  • kwq/TextField.mm: Added.
  • kwq/TextStream.cpp: Added.
  • kwq/TextStream.h: Added.
  • kwq/WebCoreEditCommand.h: Added.
  • kwq/WebCoreEditCommand.mm: Added.
  • kwq/WebCorePageState.h: Added.
  • kwq/WebCorePageState.mm: Added.
  • kwq/WebCoreResourceLoaderImp.h: Added.
  • kwq/WebCoreResourceLoaderImp.mm: Added.
  • kwq/WebCoreScrollBar.h: Added.
  • kwq/WebCoreScrollBar.mm: Added.
  • kwq/WebCoreSlider.h: Added.
  • kwq/WebCoreSlider.mm: Added.
  • loader/Cache.h:
  • loader/CachedCSSStyleSheet.cpp:
  • loader/CachedImage.cpp:
  • loader/CachedObject.h:
  • loader/CachedObjectClientWalker.h:
  • loader/CachedResource.h:
  • loader/CachedResourceClientWalker.h:
  • loader/CachedScript.h:
  • loader/CachedXBLDocument.h:
  • loader/Decoder.h:
  • loader/DocLoader.cpp:
  • loader/DocLoader.h:
  • loader/FormData.h:
  • loader/ImageDocument.cpp:
  • loader/PluginDocument.cpp:
  • loader/TextDocument.cpp:
  • loader/loader.cpp:
  • loader/loader.h:
  • page/Frame.cpp:
  • page/Frame.h:
  • page/FramePrivate.h:
  • page/Page.cpp:
  • page/Page.h:
  • platform/DeprecatedPtrList.h:
  • platform/DeprecatedPtrListImpl.cpp:
  • platform/DeprecatedPtrListImpl.h:
  • platform/DeprecatedString.cpp:
  • platform/DeprecatedString.h:
  • platform/DeprecatedStringList.cpp:
  • platform/DeprecatedStringList.h:
  • platform/DeprecatedValueList.h:
  • platform/DeprecatedValueListImpl.cpp:
  • platform/DeprecatedValueListImpl.h:
  • platform/FloatSize.h:
  • platform/Font.cpp:
  • platform/KURL.cpp:
  • platform/RegularExpression.cpp:
  • platform/RegularExpression.h:
  • platform/SegmentedString.h:
  • platform/StreamingTextDecoder.cpp:
  • platform/String.cpp:
  • platform/StringImpl.cpp:
  • platform/TextEncoding.cpp:
  • platform/TransferJob.cpp:
  • platform/TransferJob.h:
  • platform/TransferJobInternal.h:
  • platform/gdk/FrameGdk.cpp:
  • platform/gdk/TemporaryLinkStubs.cpp:
  • platform/mac/ClipboardMac.h:
  • platform/mac/ClipboardMac.mm:
  • platform/mac/CursorMac.mm:
  • platform/mac/DeprecatedStringMac.mm: Added.
  • platform/mac/FontCacheMac.mm:
  • platform/mac/FontMac.mm:
  • platform/mac/FoundationExtras.h:
  • platform/mac/ImageMac.mm:
  • platform/mac/KURLMac.mm:
  • platform/mac/MouseEventMac.mm: Removed.
  • platform/mac/QStringListMac.mm: Removed.
  • platform/mac/QStringMac.mm: Removed.
  • platform/mac/TransferJobMac.mm:
  • platform/mac/WebCoreTextArea.h:
  • platform/mac/WebCoreTextArea.mm:
  • platform/mac/WebCoreTextField.h:
  • platform/mac/WebCoreTextField.mm:
  • platform/mac/WidgetMac.mm:
  • platform/win/TemporaryLinkStubs.cpp:
  • rendering/DataRef.h:
  • rendering/DeprecatedRenderSelect.cpp:
  • rendering/DeprecatedRenderSelect.h:
  • rendering/DeprecatedSlider.cpp:
  • rendering/InlineFlowBox.cpp:
  • rendering/RenderApplet.cpp:
  • rendering/RenderBlock.cpp:
  • rendering/RenderBlock.h:
  • rendering/RenderContainer.cpp:
  • rendering/RenderFileButton.cpp:
  • rendering/RenderFlexibleBox.cpp:
  • rendering/RenderFlow.cpp:
  • rendering/RenderFormElement.cpp:
  • rendering/RenderFrameSet.cpp:
  • rendering/RenderFrameSet.h:
  • rendering/RenderImage.cpp:
  • rendering/RenderInline.cpp:
  • rendering/RenderLayer.cpp:
  • rendering/RenderLayer.h:
  • rendering/RenderLineEdit.cpp:
  • rendering/RenderListItem.cpp:
  • rendering/RenderListMarker.cpp:
  • rendering/RenderObject.cpp:
  • rendering/RenderObject.h:
  • rendering/RenderPartObject.cpp:
  • rendering/RenderReplaced.cpp:
  • rendering/RenderStyle.cpp: Added.
  • rendering/RenderStyle.h: Added.
  • rendering/RenderTable.cpp:
  • rendering/RenderTable.h:
  • rendering/RenderTableCell.cpp:
  • rendering/RenderTableCell.h:
  • rendering/RenderTableCol.cpp:
  • rendering/RenderTableCol.h:
  • rendering/RenderTableRow.cpp:
  • rendering/RenderTableSection.cpp:
  • rendering/RenderTableSection.h:
  • rendering/RenderText.cpp:
  • rendering/RenderText.h:
  • rendering/RenderTextArea.cpp:
  • rendering/RenderTextControl.cpp: Added.
  • rendering/RenderTextControl.h: Added.
  • rendering/RenderTextField.cpp: Removed.
  • rendering/RenderTextField.h: Removed.
  • rendering/RenderTheme.cpp:
  • rendering/RenderThemeMac.mm:
  • rendering/RenderTreeAsText.cpp:
  • rendering/RenderTreeAsText.h:
  • rendering/RenderView.cpp:
  • rendering/RenderWidget.cpp:
  • rendering/bidi.cpp:
  • rendering/render_style.cpp: Removed.
  • rendering/render_style.h: Removed.
  • xml/XSLStyleSheet.cpp:
  • xml/XSLTProcessor.cpp:
  • xml/xmlhttprequest.cpp:
  • xpath/impl/XPathParser.cpp:
07:53 Changeset [15252] by ddkilzer

WebKitTools:

Reviewed by Timothy Hatcher.

  • Scripts/webkitdirs.pm: Make Windows build work with Visual C++ Express.
05:05 Changeset [15251] by jdevalk

Fixes http://bugzilla.opendarwin.org/show_bug.cgi?id=9804

In the congratulations message, "capatibilies" should be "capabilities".

04:29 Changeset [15250] by ddkilzer

Tweaked bug title in last comment to be correct.

04:09 Changeset [15249] by ddkilzer

WebCore:

Reviewed by Darin.

http://bugzilla.opendarwin.org/show_bug.cgi?id=8251
navigator.product incorrect in WebKit.app on Intel Macs

For navigator.platform return "MacIntel" for Intel processors, "MacPPC"
otherwise.

  • bindings/js/kjs_navigator.cpp: (KJS::Navigator::getValueProperty):
03:46 Changeset [15248] by ddkilzer

WebKitTools:

Reviewed by Timothy Hatcher.

  • Scripts/run-webkit-tests: Speed up test list generation, implement --ignore-tests feature, and minor clean up.
00:56 Changeset [15247] by thatcher

LayoutTests:

Reviewed by Maciej.

Bug 5312: comments aren't available via DOM
http://bugzilla.opendarwin.org/show_bug.cgi?id=5312

New tests for DOM comment nodes. Updated results to account for new
nodes in the DOM (editing offsets changed.)

  • editing/deleting/delete-line-013-expected.txt:
  • editing/deleting/delete-line-014-expected.txt:
  • editing/inserting/insert-3907422-fix-expected.txt:
  • editing/inserting/insert-div-009-expected.txt:
  • editing/inserting/insert-div-010-expected.txt:
  • editing/selection/extend-by-character-006-expected.txt:
  • editing/selection/move-between-blocks-no-001-expected.txt:
  • editing/selection/move-between-blocks-yes-001-expected.txt:
  • fast/dom/HTMLDocument/object-by-name-or-id-expected.txt:
  • fast/dom/HTMLDocument/object-by-name-or-id.html:
  • fast/dom/comment-document-fragment-expected.txt: Added.
  • fast/dom/comment-document-fragment.html: Added.
  • fast/dom/comment-dom-node-expected.txt: Added.
  • fast/dom/comment-dom-node.html: Added.
  • fast/dom/comment-not-documentElement-expected.checksum: Added.
  • fast/dom/comment-not-documentElement-expected.png: Added.
  • fast/dom/comment-not-documentElement-expected.txt: Added.
  • fast/dom/comment-not-documentElement.html: Added.
  • fast/text/atsui-pointtooffset-calls-cg-expected.txt:
  • fast/text/atsui-rtl-override-selection-expected.txt:

WebCore:

Reviewed by Maciej.

Bug 5312: comments aren't available via DOM
http://bugzilla.opendarwin.org/show_bug.cgi?id=5312

<rdar://problem/4564414> getting comments via DOM isn't working (5312)
<rdar://problem/4545691> DOM_COMMENT node masking out real DOM elements

  • dom/Document.cpp: (WebCore::Document::removedLastRef): set m_documentElement to 0 (WebCore::Document::childrenChanged): invalidate the document element we have cached in case it was replaced (WebCore::Document::documentElement): cache the first element as the document node if m_documentElement is 0
  • dom/Document.h: added m_documentElement
  • html/HTMLDocument.cpp: removed documentElement(), HTML documents just use Document's documentElement()
  • html/HTMLDocument.h: ditto
  • html/HTMLParser.cpp: (WebCore::HTMLParser::handleError): make a new HTML node is the document's firstChild is NULL or the firstChild is not a HTML element (like a comment)
  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::parseComment): removed the #ifdef to include comment nodes

WebKit:

Reviewed by Maciej.

Bug 5312: comments aren't available via DOM
http://bugzilla.opendarwin.org/show_bug.cgi?id=5312

Makes the Web Inspector show comment node contents.

  • WebInspector/WebInspector.m: (-[DOMNode _displayName]): return the contents of the comment
  • WebInspector/webInspector/inspector.js: check for comment nodes
00:04 Changeset [15246] by ap

Reviewed by Darin.

  • WebView/WebFrame.m: (-[WebFrame _addExtraFieldsToRequest:mainResource:alwaysFromRequest:]): Add an Accept header to main resource requests.
  • English.lproj/StringsNotToBeLocalized.txt: Added new strings.

07/08/06:

22:44 Changeset [15245] by thatcher

Update the pixel result to match.

22:13 Changeset [15244] by thatcher

Build fix.

22:03 Changeset [15243] by thatcher

New tag (part 2.)

22:02 Changeset [15242] by thatcher

New tag.

21:58 Changeset [15241] by thatcher

Reviewed by Maciej.

Moved KJS_GetCreatedJavaVMs to jni_utility.cpp.
Switched KJS_GetCreatedJavaVMs over to use dlopen and dlsym
now that NSAddImage, NSLookupSymbolInImage and NSAddressOfSymbol
are deprecated in Leopard.

  • JavaScriptCore.exp:
  • JavaScriptCore.xcodeproj/project.pbxproj:
  • bindings/jni/jni_utility.cpp: (KJS::Bindings::KJS_GetCreatedJavaVMs):
  • bindings/softlinking.c: Removed.
  • bindings/softlinking.h: Removed.
21:28 Changeset [15240] by mjs

Reviewed by Darin, tweaked slightly and landed by Maciej.

http://bugzilla.opendarwin.org/show_bug.cgi?id=9787
fast/frames tests failing (bad pointer to owner element) under MallocScribble


  • html/HTMLFrameElement.cpp: (WebCore::HTMLFrameElement::close): Disconnect the owner element of our frame.


  • html/HTMLIFrameElement.cpp: (WebCore::HTMLIFrameElement::attach): Don't call setFrame here.


  • html/HTMLPlugInElement.cpp: (WebCore::HTMLPlugInElement::detach): Disconnect the owner element of our frame.
  • html/HTMLPlugInElement.h: (WebCore::HTMLPlugInElement::setFrameName): New function.


  • page/Frame.cpp: (WebCore::Frame::Frame): Don't call setFrame.


(WebCore::Frame::~Frame):
Null out the owner element.


(WebCore::Frame::requestObject):
When an object creates a child frame, assign it a name so it can be
accessed from HTMLPlugInElement.


  • rendering/RenderPart.cpp: (WebCore::RenderPart::RenderPart): (WebCore::RenderPart::~RenderPart):
  • rendering/RenderPart.h: Get rid of setFrame.
13:04 Changeset [15239] by darin

Reviewed by Geoff.

  • bindings/js/kjs_window.cpp: (KJS::showModalDialog): Set the return value after returning from the function if the window is not cleared; this is a better way to handle the case where the window does not get cleared before returning, and handles some new cases created by slight changes in the latest Safari properly too. (KJS::Window::clear): Changed logic slightly so we always store the result of getDirect into the return value slot -- the old code left the storage untouched if it was 0. Also made it only overwrite the return value slot if it's 0.
  • bindings/js/kjs_proxy.h:
  • bindings/js/kjs_proxy.cpp:
  • bindings/js/kjs_window.h:
  • bindings/js/kjs_window.cpp:
  • page/Frame.cpp: Roll the previous fix out.
12:45 Changeset [15238] by darin

LayoutTests:

Reviewed by Geoff (well, half of it at least).

  • plugins/jsobjc-simple.html: Add an autorelease to fix a storage leak.

WebKitTools:

Reviewed by Geoff (well, half of it at least).

  • DumpRenderTree/DumpRenderTree.m: (-[LayoutTestController keepWebHistory]):
  • DumpRenderTree/ObjCPlugin.m: (-[ObjCPlugin removeBridgeRestrictions:]): Add a release to fix a storage leak.
12:38 Changeset [15237] by darin
  • Scripts/do-webcore-rename: A few more.
12:11 Changeset [15236] by darin
  • Scripts/do-webcore-rename: Another cut at renames. About ready to go (later today).
11:08 Changeset [15235] by ggaren

Reviewed by Anders.


  • Make JSObjectGetProperty return a JSValue or NULL, like JSEvaluate does.
  • API/JSObjectRef.cpp: (JSObjectGetProperty):
  • API/JSObjectRef.h:
  • API/testapi.c: (main):
10:53 Changeset [15234] by ggaren

Style change -- no review necessary.


Use 0 instead of NULL in API .cpp files, to match our style guidelines.

  • API/JSContextRef.cpp: (JSEvaluate):
  • API/JSObjectRef.cpp: (JSFunctionMakeWithBody): (JSObjectCallAsFunction): (JSObjectCallAsConstructor):
  • API/JSValueRef.cpp: (JSValueToObject):
10:42 Changeset [15233] by ggaren

Reviewed by TimO.

  • Added ability to pass NULL for thisObject when calling JSObjectCallAsFunction, to match JSEvaluate.


  • API/JSObjectRef.cpp: (JSObjectCallAsFunction):
  • API/JSObjectRef.h:
  • API/testapi.c: (main):
10:14 Changeset [15232] by darin
  • clear up a few minor housekeeping problems
  • fast/css/xml-stylesheet-pi-not-in-prolog-expected.checksum: Added.
  • fast/css/xml-stylesheet-pi-not-in-prolog-expected.png: Added.
  • fast/forms/input-appearance-height-expected.checksum: Added. Added missing test results.
  • fast/block/float/nopaint-after-layer-destruction-expected.checksum:
  • fast/block/positioning/absolute-positioned-overconstrained.html:
  • fast/css/ZeroOpacityLayers.html:
  • fast/css/ZeroOpacityLayers2.html:
  • fast/css/font-shorthand-weight-only-expected.checksum:
  • fast/css/hsl-color-expected.checksum:
  • fast/css/hsla-color-expected.checksum:
  • fast/css/resize-corner-tracking-expected.checksum:
  • fast/dom/HTMLLinkElement/pending-stylesheet-count-expected.checksum:
  • fast/dynamic/selection-highlight-adjust-expected.checksum:
  • fast/forms/button-generated-content-expected.checksum:
  • fast/frames/calculate-fixed.html:
  • fast/frames/calculate-order.html:
  • fast/frames/calculate-percentage.html:
  • fast/frames/calculate-relative.html:
  • fast/frames/calculate-round.html:
  • fast/history/clicked-link-is-visited-expected.checksum:
  • fast/layers/opacity-outline-expected.checksum:
  • fast/media/import-p-red.css:
  • fast/parser/broken-comments-vs-parsing-mode-expected.checksum:
  • fast/repaint/clipped-relative-expected.checksum:
  • fast/repaint/float-move-during-layout-expected.checksum:
  • fast/repaint/layer-child-outline-expected.checksum:
  • fast/repaint/outline-shrinking-expected.checksum:
  • fast/repaint/static-to-positioned-expected.checksum:
  • fast/repaint/table-collapsed-border-expected.checksum:
  • fast/table/absolute-table-at-bottom-expected.checksum:
  • fast/table/border-collapsing/border-collapsing-head-foot-expected.checksum:
  • fast/table/empty-section-crash-expected.checksum:
  • fast/table/rtl-cell-display-none-assert-expected.checksum:
  • fast/table/text-field-baseline-expected.checksum:
  • fast/text/capitalize-empty-generated-string-expected.checksum:
  • svg/custom/gradient-stroke-width-expected.checksum:
  • traversal/stay-within-root-expected.checksum: Removed "No newline at end of file" that snuck in as part of the "apply patch" process.
  • fast/block/positioning/absolute-positioned-overconstrained-expected.txt:
  • fast/css/ZeroOpacityLayers-expected.checksum:
  • fast/css/ZeroOpacityLayers-expected.png:
  • fast/css/ZeroOpacityLayers-expected.txt:
  • fast/css/ZeroOpacityLayers2-expected.checksum:
  • fast/css/ZeroOpacityLayers2-expected.png:
  • fast/css/ZeroOpacityLayers2-expected.txt: Updated tests results that changed due to the above.
09:54 Changeset [15231] by darin
  • DefaultDelegates/WebDefaultContextMenuDelegate.m: (-[WebDefaultUIDelegate contextMenuItemsForElement:defaultMenuItems:]): Removed misleading old comment.
07:30 Changeset [15230] by darin
  • try to fix Windows build
  • WebCore.vcproj/WebCore/WebCore.vcproj: Add missing source files.
02:34 Changeset [15229] by thatcher

New tag.

02:30 Changeset [15228] by thatcher

Versioning

07/07/06:

20:36 Changeset [15227] by ggaren
  • build fix
  • JSUtils.cpp: (KJSValueToCFTypeInternal):
  • JSValueWrapper.cpp: (JSValueWrapper::JSObjectCopyPropertyNames):
19:32 Changeset [15226] by justing

LayoutTests:

Reviewed by levi

  • editing/selection/editable-non-editable-crash-expected.checksum: Added.
  • editing/selection/editable-non-editable-crash-expected.png: Added.
  • editing/selection/editable-non-editable-crash-expected.txt: Added.
  • editing/selection/editable-non-editable-crash.html: Added.
  • editing/selection/skip-non-editable-1-expected.txt:
  • editing/selection/skip-non-editable-1.html:

WebCore:

Reviewed by levi


Fix Mail ToDo crashers.

  • dom/Range.cpp: (WebCore::Range::compareBoundaryPoints): Added an ASSERT that both containers are non-null and an early return.
  • editing/Selection.cpp: (WebCore::Selection::validate): Fix a dangling start/end. (WebCore::Selection::adjustForEditableContent): Added an early return if m_start or m_end are null. (WebCore::Selection::isContentEditable): Use isRichlyEditablePosition. (WebCore::Selection::isContentRichlyEditable): Ditto.
  • editing/Selection.h:
  • editing/VisiblePosition.cpp: (WebCore::VisiblePosition::next): Use the new highestEditableRoot. (WebCore::VisiblePosition::previous): Ditto.
  • editing/htmlediting.cpp: (WebCore::highestEditableRoot): Takes in a position. (WebCore::isEditablePosition): Added. (WebCore::isRichlyEditablePosition): Added. (WebCore::rootEditableElement): Added. (WebCore::nextCandidate): Moved and split out from nextVisiblePosition. (WebCore::nextVisuallyDistinctCandidate): Ditto. (WebCore::previousCandidate): Moved and split out from previousVisiblePosition. (WebCore::previousVisuallyDistinctCandidate): Ditto. (WebCore::firstEditablePositionAfterPositionInRoot): Iterate over positions, using nextVisuallyDistinctCandidate, skipping atomic nodes that are non-editable. (WebCore::lastEditablePositionBeforePositionInRoot): Ditto.
  • editing/htmlediting.h:
  • editing/visible_units.cpp: (WebCore::startOfWord): Added a FIXME.
19:25 Changeset [15225] by ggaren

Reviewed by Maciej.


  • Standardized which functions take a JSContext as an argument. The rule is: if you might execute JavaScript, you take a JSContext, otherwise you don't.


The FIXME in JSObjectRef.h requires refactoring some parts of Interpreter,
but not API changes, so I'm putting it off until later.

  • API/JSCallbackObject.cpp: (KJS::JSCallbackObject::JSCallbackObject): (KJS::JSCallbackObject::init):
  • API/JSCallbackObject.h:
  • API/JSContextRef.cpp: (JSContextCreate):
  • API/JSContextRef.h:
  • API/JSObjectRef.cpp: (JSObjectMake): (JSPropertyEnumeratorGetNext):
  • API/JSObjectRef.h:
  • API/testapi.c: (MyObject_initialize): (main):
  • JavaScriptCore.exp:
  • kjs/array_object.cpp: (ArrayInstance::setLength): (ArrayInstance::pushUndefinedObjectsToEnd):
  • kjs/nodes.cpp: (ForInNode::execute):
  • kjs/reference.cpp: (KJS::Reference::getPropertyName): (KJS::Reference::getValue):
  • kjs/reference.h:
  • kjs/scope_chain.cpp: (KJS::ScopeChain::print):
19:02 Changeset [15224] by ggaren

Reviewed by Maciej.


More API action.


  • Headerdoc finished

Semantic Changes:

  • Added a JSContextRef argument to many functions, because you need a JSContextRef for doing virtually anything. I expect to add this argument to even more functions in a future patch.


  • Removed the globalObjectPrototype argument to JSContextCreate because you can't create an object until you have a context, so it's impossible to pass a prototype object to JSContextCreate. That's OK because (1) there's no reason to give the global object a prototype and (2) if you really want to, you can just use a separate call to JSObjectSetPrototype.


  • Removed the JSClassRef argument to JSClassCreate because it was unnecessary, and you need to be able to make the global object's class before you've created a JSContext.


  • Added an optional exception parameter to JSFunctionMakeWithBody because anything less would be uncivilized.


  • Made the return value parameter to JSObjectGetProperty optional to match all other return value parameters in the API.


  • Made JSObjectSetPrivate/JSObjectGetPrivate work on JSCallbackFunctions and JSCallbackConstructors. You could use an abstract base class or strategic placement of m_privateData in the class structure to implement this, but the former seemed like overkill, and the latter seemed too dangerous.


  • Fixed a bug where JSPropertyEnumeratorGetNext would skip the first property.

Cosmetic Changes:

  • Reversed the logic of the JSChar #ifdef to avoid confusing headerdoc


  • Removed function names from @function declarations because headeroc can parse them automatically, and I wanted to rule out manual mismatch.
  • Changed Error::create to take a const UString& instead of a UString* because it was looking at me funny.


  • Renamed JSStringBufferCreateWithCFString to JSStringBufferCreateCF because the latter is more concise and it matches JSStringBufferCreateUTF8.


  • API/JSCallbackObject.cpp: (KJS::JSCallbackObject::getOwnPropertySlot): (KJS::JSCallbackObject::put): (KJS::JSCallbackObject::deleteProperty): (KJS::JSCallbackObject::getPropertyList): (KJS::JSCallbackObject::toBoolean): (KJS::JSCallbackObject::toNumber): (KJS::JSCallbackObject::toString):
  • API/JSClassRef.cpp: (JSClassCreate):
  • API/JSContextRef.cpp: (JSContextCreate): (JSContextSetException):
  • API/JSContextRef.h:
  • API/JSNode.c: (JSNodePrototype_class): (JSNode_class):
  • API/JSNodeList.c: (JSNodeListPrototype_class): (JSNodeList_class):
  • API/JSObjectRef.cpp: (JSObjectGetProperty): (JSObjectGetPrivate): (JSObjectSetPrivate): (JSObjectCallAsFunction): (JSObjectCallAsConstructor): (JSPropertyEnumeratorGetNext):
  • API/JSObjectRef.h:
  • API/JSStringBufferRef.cpp: (JSStringBufferCreateCF):
  • API/JSStringBufferRef.h:
  • API/JSValueRef.cpp: (JSValueIsInstanceOf):
  • API/JSValueRef.h:
  • API/minidom.c: (main):
  • API/minidom.js:
  • API/testapi.c: (MyObject_hasProperty): (MyObject_setProperty): (MyObject_deleteProperty): (MyObject_getPropertyList): (MyObject_convertToType): (MyObject_class): (main):
  • JavaScriptCore.exp:
18:45 Changeset [15223] by lweintraub

Reviewed by NOBODY

Missed a few in the last update...

  • fast/forms/input-delete-expected.txt:
  • fast/repaint/inline-outline-repaint-expected.txt:
  • fast/repaint/selection-after-delete-expected.txt:
17:46 Changeset [15222] by lweintraub

LayoutTests:

Reviewed by justin

Deletion editing deligates now show up!

  • editing/deleting/collapse-whitespace-3587601-fix-expected.txt:
  • editing/deleting/delete-3608430-fix-expected.txt:
  • editing/deleting/delete-3608445-fix-expected.txt:
  • editing/deleting/delete-3608462-fix-expected.txt:
  • editing/deleting/delete-3775172-fix-expected.txt:
  • editing/deleting/delete-3800834-fix-expected.txt:
  • editing/deleting/delete-3857753-fix-expected.txt:
  • editing/deleting/delete-3865854-fix-expected.txt:
  • editing/deleting/delete-3928305-fix-expected.txt:
  • editing/deleting/delete-3959464-fix-expected.txt:
  • editing/deleting/delete-4038408-fix-expected.txt:
  • editing/deleting/delete-4083333-fix-expected.txt:
  • editing/deleting/delete-after-span-ws-001-expected.txt:
  • editing/deleting/delete-after-span-ws-002-expected.txt:
  • editing/deleting/delete-after-span-ws-003-expected.txt:
  • editing/deleting/delete-all-text-in-text-field-assertion-expected.txt:
  • editing/deleting/delete-and-undo-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-001-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-002-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-003-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-004-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-005-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-006-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-007-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-008-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-009-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-010-expected.txt:
  • editing/deleting/delete-at-paragraph-boundaries-011-expected.txt:
  • editing/deleting/delete-block-contents-001-expected.txt:
  • editing/deleting/delete-block-contents-002-expected.txt:
  • editing/deleting/delete-block-contents-003-expected.txt:
  • editing/deleting/delete-block-merge-contents-001-expected.txt:
  • editing/deleting/delete-block-merge-contents-002-expected.txt:
  • editing/deleting/delete-block-merge-contents-003-expected.txt:
  • editing/deleting/delete-block-merge-contents-004-expected.txt:
  • editing/deleting/delete-block-merge-contents-005-expected.txt:
  • editing/deleting/delete-block-merge-contents-006-expected.txt:
  • editing/deleting/delete-block-merge-contents-007-expected.txt:
  • editing/deleting/delete-block-merge-contents-008-expected.txt:
  • editing/deleting/delete-block-merge-contents-009-expected.txt:
  • editing/deleting/delete-block-merge-contents-010-expected.txt:
  • editing/deleting/delete-block-merge-contents-012-expected.txt:
  • editing/deleting/delete-block-merge-contents-013-expected.txt:
  • editing/deleting/delete-block-merge-contents-014-expected.txt:
  • editing/deleting/delete-block-merge-contents-015-expected.txt:
  • editing/deleting/delete-block-merge-contents-016-expected.txt:
  • editing/deleting/delete-block-merge-contents-017-expected.txt:
  • editing/deleting/delete-block-merge-contents-018-expected.txt:
  • editing/deleting/delete-block-merge-contents-019-expected.txt:
  • editing/deleting/delete-block-merge-contents-020-expected.txt:
  • editing/deleting/delete-block-merge-contents-021-expected.txt:
  • editing/deleting/delete-block-merge-contents-022-expected.txt:
  • editing/deleting/delete-block-merge-contents-023-expected.txt:
  • editing/deleting/delete-block-merge-contents-024-expected.txt:
  • editing/deleting/delete-br-001-expected.txt:
  • editing/deleting/delete-br-002-expected.txt:
  • editing/deleting/delete-br-003-expected.txt:
  • editing/deleting/delete-br-004-expected.txt:
  • editing/deleting/delete-br-005-expected.txt:
  • editing/deleting/delete-br-006-expected.txt:
  • editing/deleting/delete-br-007-expected.txt:
  • editing/deleting/delete-br-008-expected.txt:
  • editing/deleting/delete-br-009-expected.txt:
  • editing/deleting/delete-br-010-expected.txt:
  • editing/deleting/delete-br-011-expected.txt:
  • editing/deleting/delete-character-001-expected.txt:
  • editing/deleting/delete-contiguous-ws-001-expected.txt:
  • editing/deleting/delete-image-001-expected.txt:
  • editing/deleting/delete-image-002-expected.txt:
  • editing/deleting/delete-image-003-expected.txt:
  • editing/deleting/delete-image-004-expected.txt:
  • editing/deleting/delete-leading-ws-001-expected.txt:
  • editing/deleting/delete-line-001-expected.txt:
  • editing/deleting/delete-line-002-expected.txt:
  • editing/deleting/delete-line-003-expected.txt:
  • editing/deleting/delete-line-004-expected.txt:
  • editing/deleting/delete-line-005-expected.txt:
  • editing/deleting/delete-line-006-expected.txt:
  • editing/deleting/delete-line-007-expected.txt:
  • editing/deleting/delete-line-008-expected.txt:
  • editing/deleting/delete-line-009-expected.txt:
  • editing/deleting/delete-line-010-expected.txt:
  • editing/deleting/delete-line-011-expected.txt:
  • editing/deleting/delete-line-012-expected.txt:
  • editing/deleting/delete-line-013-expected.txt:
  • editing/deleting/delete-line-014-expected.txt:
  • editing/deleting/delete-line-015-expected.txt:
  • editing/deleting/delete-line-016-expected.txt:
  • editing/deleting/delete-line-017-expected.txt:
  • editing/deleting/delete-line-end-ws-001-expected.txt:
  • editing/deleting/delete-line-end-ws-002-expected.txt:
  • editing/deleting/delete-listitem-001-expected.txt:
  • editing/deleting/delete-listitem-002-expected.txt:
  • editing/deleting/delete-select-all-001-expected.txt:
  • editing/deleting/delete-select-all-002-expected.txt:
  • editing/deleting/delete-select-all-003-expected.txt:
  • editing/deleting/delete-selection-001-expected.txt:
  • editing/deleting/delete-tab-001-expected.txt:
  • editing/deleting/delete-tab-002-expected.txt:
  • editing/deleting/delete-tab-003-expected.txt:
  • editing/deleting/delete-tab-004-expected.txt:
  • editing/deleting/delete-trailing-ws-001-expected.txt:
  • editing/deleting/delete-trailing-ws-002-expected.txt:
  • editing/deleting/delete-ws-fixup-001-expected.txt:
  • editing/deleting/delete-ws-fixup-002-expected.txt:
  • editing/deleting/delete-ws-fixup-003-expected.txt:
  • editing/deleting/delete-ws-fixup-004-expected.txt:
  • editing/deleting/forward-delete-expected.txt:
  • editing/deleting/merge-different-styles-expected.txt:
  • editing/deleting/merge-endOfParagraph-expected.txt:
  • editing/deleting/merge-into-empty-block-1-expected.txt:
  • editing/deleting/merge-into-empty-block-2-expected.txt:
  • editing/deleting/merge-no-br-expected.txt:
  • editing/deleting/merge-unrendered-space-expected.txt:
  • editing/deleting/pruning-after-merge-1-expected.txt:
  • editing/deleting/pruning-after-merge-2-expected.txt:
  • editing/deleting/smart-delete-001-expected.txt:
  • editing/deleting/smart-delete-002-expected.txt:
  • editing/deleting/table-cells-expected.txt:
  • editing/execCommand/insertHTML-expected.txt:
  • editing/input/text-input-controller-expected.txt:
  • editing/inserting/insert-3775316-fix-expected.txt:
  • editing/inserting/insert-3851164-fix-expected.txt:
  • editing/inserting/insert-after-delete-001-expected.txt:
  • editing/inserting/insert-div-021-expected.txt:
  • editing/inserting/insert-div-026-expected.txt:
  • editing/pasteboard/4076267-2-expected.txt:
  • editing/pasteboard/4076267-3-expected.txt:
  • editing/pasteboard/4076267-expected.txt:
  • editing/pasteboard/paste-table-002-expected.txt:
  • editing/pasteboard/paste-text-015-expected.txt:
  • editing/selection/move-between-blocks-no-001-expected.txt:
  • editing/selection/select-all-iframe-expected.txt:
  • editing/style/block-style-005-expected.txt:
  • editing/style/smoosh-styles-001-expected.txt:
  • editing/style/smoosh-styles-002-expected.txt:
  • editing/style/style-3681552-fix-002-expected.txt:
  • editing/style/typing-style-001-expected.txt:
  • editing/style/typing-style-002-expected.txt:
  • editing/unsupported-content/list-delete-001-expected.txt:
  • editing/unsupported-content/list-delete-002-expected.txt:
  • editing/unsupported-content/list-delete-003-expected.txt:
  • editing/unsupported-content/table-delete-001-expected.txt:
  • editing/unsupported-content/table-delete-002-expected.txt:
  • editing/unsupported-content/table-delete-003-expected.txt:
  • fast/forms/input-text-option-delete-expected.checksum:
  • fast/forms/input-text-option-delete-expected.txt:

WebCore:

Reviewed by justin

Allowed deletion typing commands to take in a granularity

  • bridge/mac/WebCoreFrameBridge.h: Pass granularity across the bridge
  • bridge/mac/WebCoreFrameBridge.mm: Ditto (-[WebCoreFrameBridge deleteKeyPressedWithSmartDelete:granularity:]): (-[WebCoreFrameBridge forwardDeleteKeyPressedWithSmartDelete:granularity:]):
  • editing/TypingCommand.cpp: Constructors take in a granularity (WebCore::TypingCommand::TypingCommand): (WebCore::TypingCommand::deleteKeyPressed): (WebCore::TypingCommand::forwardDeleteKeyPressed):
  • editing/TypingCommand.h: ditto
  • page/Frame.h: Fixed bug that shouldDeleteSelection wasn't virtual

WebKit:

Reviewed by justin

Finished moving deletion selection expansion across the bridge... say that 3 times fast.

  • WebView/WebHTMLView.m: Pass granularity to WebCore to handle expansion (-[WebHTMLView _deleteRange:killRing:prepend:smartDeleteOK:deletionAction:granularity:]): (-[WebHTMLView _deleteSelection]): (-[NSArray _deleteWithDirection:granularity:killRing:isTypingAction:]): (-[NSArray deleteToMark:]):
17:27 Changeset [15221] by beidson

Reviewed by Anders and Tim Hatcher

Added an accessor to a SQLite config option to change its disk-syncing behavior.
Used this accessor to significantly speed up writes to the Icon DB.

  • icon/IconDatabase.cpp: (WebCore::IconDatabase::open): Disable full syncing to the SQLite db to speed up writes to the icondb
  • icon/SQLDatabase.cpp: (SQLDatabase::setFullsync): (SQLDatabase::setSynchronous):
  • icon/SQLDatabase.h: (WebCore::SQLDatabase::): Added accessor to the SQLite config pragma "synchronous"
17:21 Changeset [15220] by darin

Reviewed by Maciej.

  • fixed a storage leak noticed by the buildbot
  • dom/Document.cpp: (WebCore::Document::removedLastRef): Call delete on the markers before clearing the map containing them.
16:46 Changeset [15219] by andersca

2006-07-08 Anders Carlsson <acarlsson@apple.com>

Reviewed by Brady.

http://bugzilla.opendarwin.org/show_bug.cgi?id=9777
Reproducible crash in Loading/Frames in ToT.


The problem was that the renderpart destructor resets the frame's
owner element. For iframes, this should only be done when the element is
going to be removed from the document.


  • html/HTMLIFrameElement.cpp: (WebCore::HTMLIFrameElement::~HTMLIFrameElement): (WebCore::HTMLIFrameElement::willRemove): (WebCore::HTMLIFrameElement::removedFromDocument): (WebCore::HTMLIFrameElement::detach):
  • html/HTMLIFrameElement.h: Remove the frame in willRemove instead.


  • rendering/RenderPart.cpp: (WebCore::RenderPart::RenderPart): (WebCore::RenderPart::~RenderPart):
  • rendering/RenderPart.h: Don't call setFrame(0) in the destructor when the element is an iframe element.
16:01 Changeset [15218] by darin

Reviewed by Anders.

  • fixed a storage leak noticed by the buildbot
  • page/FrameView.cpp: (WebCore::FrameViewPrivate::~FrameViewPrivate): Delete m_scheduledEvents.
15:39 Changeset [15217] by sullivan

Added a bug number to an existing entry.

15:33 Changeset [15216] by darin

Reviewed by Adele.

  • fixed a couple of storage leaks noticed by the buildbot
  • html/HTMLOptGroupElement.cpp: (WebCore::HTMLOptGroupElement::attach): Call deref on the style after calling setRenderStyle, since styleForRenderer returns us a "ref'd" RenderStyle object.
  • html/HTMLOptionElement.cpp: (WebCore::HTMLOptionElement::attach): Ditto.
15:18 Changeset [15215] by darin

Reviewed by Adele.

  • fix some license mistakes
  • LICENSE-APPLE: Added more years of publication to the copyright here.
  • html/CanvasGradient.idl: Corrected license on this file. It had the wrong one.
  • html/CanvasPattern.idl: Ditto.
  • html/CanvasRenderingContext2D.idl: Ditto.
  • html/HTMLCanvasElement.idl: Ditto.
14:13 Changeset [15214] by beidson

Reviewed by John

Changed an ASSERT to a LOG_ERROR for an error that could be handled gracefully, but
whose assertion was reproducibly causing a build bot failure

  • Misc/WebIconDatabase.m: (-[WebIconDatabase _releaseIconForIconURLString:]):
13:41 Changeset [15213] by ggaren

Reviewed by John.


  • Fixed a few crashes resulting from NULL parameters to JSClassCreate.
  • API/JSClassRef.cpp: (JSClassCreate): (JSClassRelease):
  • API/testapi.c: Added test for NULL parameters. (main):
13:39 Changeset [15212] by ggaren

Reviewed by John, mocked by Darin.


  • Changed JSEvaluate to take a JSObjectRef instead of a JSValueRef as "this," since "this" must be an object.
  • API/JSContextRef.cpp: (JSEvaluate):
  • API/JSContextRef.h:
13:38 Changeset [15211] by ggaren

Reviewed by John.


  • More headerdoc
12:57 Changeset [15210] by thatcher

New tag (part 2).

12:56 Changeset [15209] by thatcher

New tag.

12:55 Changeset [15208] by thatcher

Versioning

12:54 Changeset [15207] by thatcher

Merged fix from TOT to Safari-1-3-branch
This fixes <rdar://problem/4600166> REGRESSION: Safari crash while loading web pages ( http://www.yahoo.com/ )

2005-10-16 David Hyatt <hyatt@apple.com>

Clean up the deletion of anonymous boxes in the render tree.
Renamed methods to make it more clear and also added comments.


Reviewed by bdakin

  • khtml/rendering/render_container.cpp: (RenderContainer::detach): this was renamed on the branch, from RenderContainer::destroy (RenderContainer::detachLeftoverChildren): this was renamed on the branch, from RenderContainer::destroyLeftoverChildren
  • khtml/rendering/render_container.h:
  • khtml/rendering/render_flow.cpp: (RenderFlow::detach): this was renamed on the branch, from RenderContainer::destroy
12:33 Changeset [15206] by lweintraub

LayoutTests:

Reviewed by justin

Improved table editing

  • editing/deleting/delete-to-select-table-expected.checksum: Added.
  • editing/deleting/delete-to-select-table-expected.png: Added.
  • editing/deleting/delete-to-select-table-expected.txt: Added.
  • editing/deleting/delete-to-select-table.html: Added.
  • editing/input/text-input-controller-expected.txt:

WebCore:

Reviewed by justin

Improved table editing: pressing delete directly before or after a table will now select
the table first, then delete it, instead of just deleting it. Useful for when the table's
existence is non-obvious.

  • bridge/mac/FrameMac.h: Added shouldDeleteSelection, which allows WebCore to trigger deletion editing deligates
  • bridge/mac/FrameMac.mm: ditto (WebCore::FrameMac::shouldDeleteSelection):
  • bridge/mac/WebCoreFrameBridge.h: ditto
  • editing/TypingCommand.cpp: Implemented check for a caret position next to a table, and subsequent selection (as opposed to removal) (WebCore::TypingCommand::deleteKeyPressed): (WebCore::TypingCommand::forwardDeleteKeyPressed):
  • page/Frame.cpp: shouldDeleteSelection added (WebCore::Frame::shouldDeleteSelection):
  • page/Frame.h: ditto

WebKit:

Reviewed by justin

Improved table editing

  • WebCoreSupport/WebFrameBridge.m: Added method to allow WebCore to trigger deletion editing delegate (-[WebFrameBridge shouldDeleteSelectedDOMRange:]):
  • WebView/WebHTMLView.m: Moved code that expanded a selection when the delete key is pressed over to WebCore so we can be more intelligent about how to handle it (-[WebHTMLView _deleteRange:killRing:prepend:smartDeleteOK:deletionAction:]): (-[NSArray _deleteWithDirection:granularity:killRing:isTypingAction:]):
12:29 Changeset [15205] by thatcher

Versioning

12:21 Changeset [15204] by thatcher

New tag (part 2).

12:20 Changeset [15203] by thatcher

New tag.

12:19 Changeset [15202] by thatcher

Versioning

11:24 Changeset [15201] by weinig

Reviewed by Eric.

  • WebCore.xcodeproj/project.pbxproj:
  • xml/XSLImportRule.cpp: Added. (WebCore::XSLImportRule::XSLImportRule): (WebCore::XSLImportRule::parentStyleSheet): (WebCore::XSLImportRule::loadSheet):
  • xml/XSLImportRule.h: Added.
  • xml/XSLStyleSheet.cpp:
  • xml/XSLStyleSheet.h:
10:22 Changeset [15200] by andersca

WebCore:

2006-07-07 Anders Carlsson <acarlsson@apple.com>

Reviewed by Darin.

http://bugzilla.opendarwin.org/show_bug.cgi?id=8300
REGRESSION: First mouse click is randomly ignored when clicking on checkbox and radio buttons


  • css/html4.css: Set "-webkit-user-select: ignore" on radio buttons and check boxes .


LayoutTests:

2006-07-07 Anders Carlsson <acarlsson@apple.com>

Reviewed by Darin.

http://bugzilla.opendarwin.org/show_bug.cgi?id=8300
REGRESSION: First mouse click is randomly ignored when clicking on checkbox and radio buttons

  • fast/forms/radio-check-click-and-drag-expected.txt: Added.
  • fast/forms/radio-check-click-and-drag.html: Added.
10:09 Changeset [15199] by sullivan

Reviewed by Tim Hatcher

  • WebKit.exp: added surprisingly missing _WebPreferencesChangedNotification, defined in WebPreferences.h
01:36 Changeset [15198] by mjs

Reviewed by Tim Hatcher.

  • fix 20 Node leaks on fast/events tests - these should be the last ones
  • bindings/objc/DOMEvents.mm: (-[DOMEvent dealloc]): implement - deref the event! (-[DOMEvent finalize]): ditto
Note: See TracTimeline for information about the timeline view.