Timeline

View changes from
going back days
by author

Repository changesets Wiki changes

Nov 7, 2005:

4:49 PM Changeset in webkit [11095] by harrison

• 1 edit
  4 adds in trunk/LayoutTests

Layout test for ​http://bugzilla.opendarwin.org/show_bug.cgi?id=3739
Unreproducible - Assertion failure in isEqualIgnoringAffinity on double-click

• editing/selection/doubleclick-crash-expected.checksum: Added.
• editing/inserting/doubleclick-crash-expected.png: Added.
• editing/inserting/doubleclick-crash-expected.txt: Added.
• editing/inserting/doubleclick-crash.html: Added.

4:49 PM Changeset in webkit [11094] by harrison

• 3 edits in trunk/WebCore

Reviewed by Justin and Hyatt.

​http://bugzilla.opendarwin.org/show_bug.cgi?id=3739
Unreproducible - Assertion failure in isEqualIgnoringAffinity on double-click

Test case added:

• editing/inserting/doubleclick-crash.html

• khtml/rendering/render_text.cpp: (RenderText::atLineWrap): New utility function. (RenderText::caretRect): Use atLineWrap. Remove dead code. (RenderText::inlineBox): Use atLineWrap.
• khtml/rendering/render_text.h: Add atLineWrap.

3:24 PM Changeset in webkit [11093] by hyatt

• 3 edits in trunk/WebCore

Fix for sluggish loading of large pages due to excessive
repaints. Large pages still need a lot of work while
loading, but this helps.

Reviewed by darin

• khtml/rendering/render_object.cpp: (RenderObject::mustRepaintBackgroundOrBorder): (RenderObject::repaintAfterLayoutIfNeeded):
• khtml/rendering/render_object.h:

2:30 PM Changeset in webkit [11092] by adele

• 3 edits
  4 adds in trunk

Reviewed by Darin Adler.
Committed by Adele Peterson.

• fixed ​http://bugzilla.opendarwin.org/show_bug.cgi?id=5398 source visible when <script> used inside <option>

Added:

• fast/forms/option-select.html

• khtml/html/html_formimpl.h: (DOM::HTMLOptionElementImpl::checkDTD): Allow <script> tags to be children of option elements
• khtml/html/html_formimpl.cpp: (DOM::HTMLOptionElementImpl::text): Skip the script content when creating text string

1:32 PM Changeset in webkit [11091] by darin

• 2 edits in trunk/WebKitTools

• Scripts/cvs-apply: Fix case where the patch has files at the top level.

1:21 PM Changeset in webkit [11090] by ggaren

• 1 edit
  3 adds in trunk/WebCore

• Added manual test cases for <rdar://problem/4161660> window.close followed by window.print in onload handler crashes Safari in KJS::ScopeChain::bottom (redmccombstoyota.com)

• manual-tests/resources/window-close-during-parsing-popup1.html: Added.
• manual-tests/resources/window-close-during-parsing-popup2.html: Added.
• manual-tests/window-close-during-parsing.html: Added.

1:18 PM Changeset in webkit [11089] by ggaren

• 2 edits in trunk/WebKit

Darin reviewed this a while back.

• Fixed <rdar://problem/4161660> window.close followed by window.print in onload handler crashes Safari in KJS::ScopeChain::bottom (redmccombstoyota.com)

Added a call to stopLoading inside closeWindowSoon to prevent load
events from firing after a window has torn down.

Manual test case: WebCore/manual-tests/window-close-during-parsing.html

• WebCoreSupport.subproj/WebBridge.m: (-[WebBridge closeWindowSoon]):

12:52 PM Changeset in webkit [11088] by darin

• 12 edits in trunk/WebCore

Reviewed by Tim Omernick.

• speed up timers by using CFTimerRunLoopRef instead of NSTimer

• kwq/KWQObject.h: Removed pauseTimer, resumeTimers, and _addTimer, added timerIntervals and restartTimer. Made KWQObjectSenderScope functions be inline.
• kwq/KWQObject.mm: (timerFired): Added. Function called when the CFRunLoopTimerRef fires. Puts the timer into the deferred timer array if timers are deferred, otherwise makes a QTimerEvent and calls timerEvent on the target. (QObject::startTimer): Moved most of the code inside restartTimer. This now just picks a new timer ID and calls restartTimer. (QObject::restartTimer): Added. Creates a KWQObjectTimer and puts it in a timers dictionary. That includes creating a CFRunLoopTimer and scheduling it on the current run loop. (QObject::timerIntervals): Added. Looks up a timer and returns the intervals for next time to fire and the "fire again" interval. (deleteTimer): Added. Invalidates and then removes a timer. (QObject::killTimer): Changed to use deleteTimer to do most of its work. (deleteOneTimer): Added. Calls deleteTimer. (QObject::killTimers): Changed to use deleteOneTimer to do most of its work. (sendDeferredTimerEvent): Added. Creates a QTimerEvent and calls timerEvent on the target. (sendDeferredTimerEvents): Added. Calls sendDeferredTimerEvent in a loop on all the deferred timers. (QObject::setDefersTimers): Updated to call sendDeferredTimerEvents as needed.

• kwq/KWQTimer.h: (QTimer::QTimer): Made constructor inline and removed the optional parameter. (QTimer::isActive): Made inline. (QTimer::fireDate): Added. Inline function. Got rid of the NSTimer and added a CFRunLoopTimer.
• kwq/KWQTimer.mm: (timerFired): Added. Calls fire on the QTimer object. (QTimer::start): Changed to create a CFRunLoopTimer. (QTimer::stop): Changed to use CFRunLoopTimer functions. (QTimer::fire): Ditto. (deleteKWQSlot): Added. Used as delete function for the CFRunLoopTimerContext below. (singleShotTimerFired): Added. Used as the fire function for the CFRunLoopTimerContext below. (QTimer::singleShot): Changed to create, schedule, and release a CFRunLoopTimer.

• khtml/ecma/kjs_window.h: Added PausedTimeouts class. Moved WindowQObject class up in the file so it can be used inline in the implementation of the Window class, and changed to use the PausedTimeouts class intead of a QMap to store the paused timeouts.
• khtml/ecma/kjs_window.cpp: (KJS::ScheduledAction::execute): Put a check for nil part at the start; this replaces a check in the caller and a check partway down this function (after the part was already dereferenced). Fixed message handling so the message is not converted to ascii and back. Added an isObject() check so we won't crash if the passed function is a value that's not an object. Changed check for function vs. script to check for a function of nil instead of using a separate boolean. (KJS::WindowQObject::pauseTimeouts): Updated to create the new PausedTimeouts object, using the new QObject::timerIntervals function rather than the old QObject::pauseTimer function. This means that the paused timers are entirely stored in the PausedTimeouts object instead of being half in a QMap and the other half inside the QObject itself. (KJS::WindowQObject::resumeTimeouts): Ditto, using the new QObject::restartTimer function. (KJS::PausedTimeouts::~PausedTimeouts): Added. Deletes the stored timeouts, including the scheduled actions, unless the timeouts have already been resumed (in which case the m_array field will be 0).

• kwq/KWQKHTMLPart.h: Renamed pause/resumeActions to pause/resumeTimeouts and made them use the PausedTimeouts class and no longer take a key parameter.
• kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::redirectionTimerStartedOrStopped): Use the new fireDate() function rathe than the old getNSTimer() function to get the fire date. (KWQKHTMLPart::pauseTimeouts): Remove the call to hasTimeouts and the key parameter. (KWQKHTMLPart::resumeTimeouts): Remove the key parameter. (KWQKHTMLPart::openURLFromPageCache): Change the timeout handling to use PausedTimeouts instead of a QMap for the paused timeouts.

• kwq/KWQPageState.h: Changed to use PausedTimeouts instead of a QMap for timeouts. Added a pausedTimeouts parameter to the initializer and removed the setPausedActions: method.
• kwq/KWQPageState.mm: (-[KWQPageState initWithDocument:URL:windowProperties:locationProperties:interpreterBuiltins:pausedTimeouts:]): Added code where it sets the pausedTimouts. (-[KWQPageState pausedTimeouts]): Added. (-[KWQPageState clear]): Added code to delete the pausedTimeouts and to deref mousePressNode. (-[KWQPageState invalidate]): Fixed bug where document would not be deref'd if the view was 0. (-[KWQPageState dealloc]): Removed code to deref mousePressNode since that's now handled in the clear method. (-[KWQPageState finalize]): Set document to 0 after deref for consistency.

• kwq/WebCoreBridge.mm: (-[WebCoreBridge saveDocumentToPageCache]): Changed to pass in the paused timeouts using the new pauseTimeouts function.

11:59 AM Changeset in webkit [11087] by justing

• 6 edits
  8 adds in trunk/LayoutTests

Reviewed by harrison

New/updated layout tests for fixes for:

<rdar://problem/4125131> REGRESSION (Mail): after certain steps,
extra blank line appears when typing past end of reply-quoted text
<rdar://problem/4024996> Applying block styles can cause assertion
failure in inline style removal

• editing/deleting/delete-at-paragraph-boundaries-011-expected.txt:
• editing/deleting/delete-br-011-expected.checksum:
• editing/deleting/delete-br-011-expected.png:
• editing/deleting/delete-br-011-expected.txt:
• editing/inserting/insert-3786362-fix-expected.txt:
• editing/inserting/insert-at-end-01-expected.checksum: Added.
• editing/inserting/insert-at-end-01-expected.png: Added.
• editing/inserting/insert-at-end-01-expected.txt: Added.
• editing/inserting/insert-at-end-01.html: Added.
• editing/inserting/insert-at-end-02-expected.checksum: Added.
• editing/inserting/insert-at-end-02-expected.png: Added.
• editing/inserting/insert-at-end-02-expected.txt: Added.
• editing/inserting/insert-at-end-02.html: Added.

11:59 AM Changeset in webkit [11086] by justing

• 6 edits in trunk/WebCore

Reviewed by harrison

<rdar://problem/4125131> REGRESSION (Mail): after certain steps,
extra blank line appears when typing past end of reply-quoted text
<rdar://problem/4024996> Applying block styles can cause assertion
failure in inline style removal

Prevent VisiblePositions at [br, 1] at the end of root editable elements.

Layout tests added:

• inserting/insert-at-end-01.html
• inserting/insert-at-end-02.html

Layout tests changed (fixed):

• editing/deleting/delete-br-011.html
• editing/deleting/delete-at-paragraph-boundaries-011.html
• editing/inserting/insert-3786362-fix.html

• khtml/editing/apply_style_command.cpp: (khtml::ApplyStyleCommand::applyInlineStyle): Do the layout before calculating start/end positions, not after, since upstream/downstream need to know who is rendered/unrendered. Don't do equivalentRangeCompliantPosition() on the start position in addition to downstream(), since it a) is confusing, b) frequently causes start/end to be equal, making removeInlineStyle a no-op and c) causes an assertion to fire. Only reset start/end using endingSelection() if splitTextElement was needed, since reseting start/end negates the work done above to swap start/end if they are backwards. When the start position points off the end of a node, that node should be skipped in all cases, not just the start.node() != end.node() case.

• khtml/editing/composite_edit_command.cpp: (khtml::CompositeEditCommand::appendBlockPlaceholder): (khtml::CompositeEditCommand::insertBlockPlaceholder): Placeholders should be allowed in nodes that aren't blockFlow, for example, deleting the b in <div><span>b</span></div> should insert a placeholder.

The assertion here should really be something like isBlockFlow

isInlineFlow, but I can't assert those until deletion improves (4244964).

• khtml/editing/delete_selection_command.cpp: (khtml::DeleteSelectionCommand::calculateTypingStyleAfterDelete): Don't try to select the placeholder when applying style to it. It isn't necessary, and it's now impossible to do at the end of the document in any case.

• khtml/editing/visible_position.cpp: (khtml::VisiblePosition::initDownstream): Don't create VisiblePositions at [br, 1] at the end of editable blocks, it produces strange/inconsistent editing behavior at the end of the document.

• khtml/khtml_part.cpp: Fixed a comment.

Nov 6, 2005:

7:07 PM Changeset in webkit [11085] by ggaren

• 1 edit in trunk/JavaScriptCore/ChangeLog

changelog fixup

1:47 PM Changeset in webkit [11084] by andersca

• 3 edits in trunk/WebCore

2005-11-06 Anders Carlsson <​andersca@mac.com>

Reviewed by Darin.

• kwq/KWQClipboard.mm: (KWQClipboard::dragNSImage): Get the NSImage from the image renderer.

• kwq/WebCoreImageRenderer.h: Add image method. This already exists in WebImageRenderer in WebKit.

12:44 PM Changeset in webkit [11083] by ggaren

• 1 edit
  4 adds
  2 deletes in trunk/LayoutTests

Layout test for ​http://bugzilla.opendarwin.org/show_bug.cgi?id=5571
REGRESSION (412.5-TOT): duplicated words/sentences at
shakespeer.sourceforge.net

Also renamed previous test for consistency.

• fast/js/regexp-big-unicode-ranges-expected.txt: Removed.
• fast/js/regexp-big-unicode-ranges.html: Removed.
• fast/js/regexp-extended-characters-crash-expected.txt: Added.
• fast/js/regexp-extended-characters-crash.html: Added.
• fast/js/regexp-extended-characters-match-expected.txt: Added.
• fast/js/regexp-extended-characters-match.html: Added.

12:41 PM Changeset in webkit [11082] by ggaren

• 3 edits in trunk/JavaScriptCore

Reviewed by Darin.

• Fixed ​http://bugzilla.opendarwin.org/show_bug.cgi?id=5571 REGRESSION (412.5-TOT): duplicated words/sentences at shakespeer.sourceforge.net

Our UTF16-modified PCRE didn't work with extended character classes
(classes involving characters > 255) because it used the GETCHARINC
macro to read through them. In UTF16 mode, GETCHARINC expects UTF16
input, but PCRE encodes character classes in UTF8 regardless of the
input mode of the subject string.

The fix is to explicitly define GETUTF8CHARINC, and to use it,
rather than GETCHARINC, when reading extended character classes.

In UTF8 mode, we simply define GETCHARINC to be GETUTF8CHARINC.

• pcre/pcre_internal.h:
• pcre/pcre_xclass.c: (_pcre_xclass):

Nov 5, 2005:

10:22 PM Changeset in webkit [11081] by ggaren

• 1 edit
  2 adds in trunk/LayoutTests

Layout test for ​http://bugzilla.opendarwin.org/show_bug.cgi?id=5357
REGRESSION: Scriptable plugin hides properties of OBJECT element

• fast/dom/object-plugin-hides-properties.html: Added.
• fast/js/object-plugin-hides-properties-expected.txt: Added.

10:20 PM Changeset in webkit [11080] by ggaren

• 2 edits in trunk/JavaScriptCore

Patch by Mitz Pettel, reviewed by Maciej.

• Fixed ​http://bugzilla.opendarwin.org/show_bug.cgi?id=5357 REGRESSION: Scriptable plugin hides properties of OBJECT element

• bindings/objc/objc_class.mm: (KJS::Bindings::ObjcClass::fallbackObject):

9:58 PM Changeset in webkit [11079] by ggaren

• 1 edit in trunk/LayoutTests/ChangeLog

changelog fixup

9:52 PM Changeset in webkit [11078] by ggaren

• 1 edit
  2 adds in trunk/LayoutTests

Layout test for ​http://bugzilla.opendarwin.org/show_bug.cgi?id=5409
slice() testcase doesn't pass

• fast/js/string-slice-abnormal-values.html: Added.
• fast/js/string-slice-abnormal-values-expected.txt: Added.

9:48 PM Changeset in webkit [11077] by ggaren

• 3 edits in trunk/JavaScriptCore

Reviewed by Darin.

• Fixed ​http://bugzilla.opendarwin.org/show_bug.cgi?id=5409 slice() testcase doesn't pass

Modified String.slice to deal with funky values.
Updated test results. We now pass <js1_2/String/slice.js>.

• kjs/string_object.cpp: (StringProtoFuncImp::callAsFunction):
• tests/mozilla/expected.html:

9:07 PM Changeset in webkit [11076] by ggaren

• 2 edits in branches/Ti-2005-009-branch/JavaScriptCore

• Rolled out the pcre changes in my previous checkin because I applied them to the wrong branch.

Reviewed by TimH.

• pcre/pcre.c: (compile_branch):

8:09 PM Changeset in webkit [11075] by ggaren

• 1 edit in branches/Ti-2005-009-branch/JavaScriptCore/ChangeLog

changelog fixup

8:05 PM Changeset in webkit [11074] by ggaren

• 3 edits in branches/Ti-2005-009-branch/JavaScriptCore

Reviewed by NOBODY (OOPS!).

• Fixed <rdar://problem/4323773> REGRESSION (412.2-416.12): xxx appears on page due to JS regexp with char > 255 (5597) (tiddlywiki.com)

Previously, we fixed a buffer overflow by returning an error when
compiling character classes with characters > 255. But that broke
the internet.

Plan B here is to work around such classes, skipping single characters

255, skipping character ranges beginning > 255, and truncating

character ranges at 255.

I removed the early returns we added and the early returns in the
original library. Now, funky expressions compile normally, tweaked
by our special rules.

I also merged the UTF8 and UTF16 checks for out of range characters.
I remember we kept them separate before because we were purist about
marking off our code changes inside UTF16 #ifdefs. Since this patch
applies equally to UTF8 and UTF16, we've already lost that innocence,
so why not have a little fun?

• ChangeLog:
• pcre/pcre.c: (compile_branch):
• tests/mozilla/jsDriver.pl: Oh, by the way, I modified this to look inside of $SYMROOTS/Development. Not perfect, but certainly better than $SYMROOTS/, which is always wrong with XCode 2.1.

Nov 4, 2005:

10:49 PM Changeset in webkit [11073] by darin

• 1 edit
  32 adds in trunk/LayoutTests

• added pixel results for a bunch of tests that didn't have them

• fast/block/basic/021-expected.checksum: Added.
• fast/block/basic/021-expected.png: Added.
• fast/block/positioning/062-expected.checksum: Added.
• fast/block/positioning/062-expected.png: Added.
• fast/box-sizing/box-sizing-expected.checksum: Added.
• fast/box-sizing/box-sizing-expected.png: Added.
• fast/box-sizing/panels-one-expected.checksum: Added.
• fast/box-sizing/panels-one-expected.png: Added.
• fast/box-sizing/panels-two-expected.checksum: Added.
• fast/box-sizing/panels-two-expected.png: Added.
• fast/box-sizing/percentage-height-expected.checksum: Added.
• fast/box-sizing/percentage-height-expected.png: Added.
• fast/css-generated-content/014-expected.checksum: Added.
• fast/css-generated-content/014-expected.png: Added.
• fast/events/event-targets-expected.checksum: Added.
• fast/events/event-targets-expected.png: Added.
• fast/forms/blankbuttons-expected.checksum: Added.
• fast/forms/blankbuttons-expected.png: Added.
• fast/forms/button-positioned-expected.checksum: Added.
• fast/forms/button-positioned-expected.png: Added.
• fast/forms/indeterminate-expected.checksum: Added.
• fast/forms/indeterminate-expected.png: Added.
• fast/frames/calculate-fixed-expected.checksum: Added.
• fast/frames/calculate-fixed-expected.png: Added.
• fast/frames/calculate-order-expected.checksum: Added.
• fast/frames/calculate-order-expected.png: Added.
• fast/frames/calculate-percentage-expected.checksum: Added.
• fast/frames/calculate-percentage-expected.png: Added.
• fast/frames/calculate-relative-expected.checksum: Added.
• fast/frames/calculate-relative-expected.png: Added.
• fast/frames/calculate-round-expected.checksum: Added.
• fast/frames/calculate-round-expected.png: Added.

4:20 PM Changeset in webkit [11072] by hyatt

• 1 edit in trunk/WebCore/khtml/rendering/render_block.cpp

Fix build bustage.

4:13 PM Changeset in webkit [11071]

• 82 copies
  5 deletes in tags/JavaScriptCore-416~14

This commit was manufactured by cvs2svn to create tag
'JavaScriptCore-416~14'.

4:13 PM Changeset in webkit [11070] by thatcher

• 2 edits in branches/Ti-2005-009-branch/JavaScriptCore

Versioning to JavaScriptCore-416.14

4:05 PM Changeset in webkit [11069]

• 8 copies
  3 deletes in tags/JavaScriptCore-312~1~1

This commit was manufactured by cvs2svn to create tag
'JavaScriptCore-312~1~1'.

4:05 PM Changeset in webkit [11068] by thatcher

• 2 edits in branches/Safari-1-3-branch/JavaScriptCore

Versioning JavaScriptCore-312.1.1

4:02 PM Changeset in webkit [11067] by thatcher

• 2 edits in branches/Safari-1-3-branch/JavaScriptCore

Merging fix from Ti-2005-009-branch to Safari-1-3-branch

2005-11-04 Geoffrey Garen <​ggaren@apple.com>

Reviewed by darin.

• Fixed <rdar://problem/4317701> *SecUpd: Tiger* PCRE heap overflow in Safari JavaScriptCore

• pcre/pcre.c: (read_repeat_counts): We now return an error for negative repeat counts in regular expressions. This matches Firefox exactly, and Win IE by effect (Win IE doesn't return an error, but it does refuse to match anything).

3:26 PM Changeset in webkit [11066] by ggaren

• 2 edits in branches/Ti-2005-009-branch/JavaScriptCore

Reviewed by darin.

• Fixed <rdar://problem/4317701> *SecUpd: Tiger* PCRE heap overflow in Safari JavaScriptCore

• pcre/pcre.c: (read_repeat_counts): We now return an error for negative repeat counts in regular expressions. This matches Firefox exactly, and Win IE by effect (Win IE doesn't return an error, but it does refuse to match anything).

1:13 PM Changeset in webkit [11065] by hyatt

• 2 edits in trunk/WebCore

Fix for bug 5626, ghosted float steals mouse events. Make sure
that the floatRect() method deals with overflow.

Reviewed by john

• khtml/rendering/render_block.cpp: (khtml::RenderBlock::paint): (khtml::RenderBlock::floatRect): (khtml::RenderBlock::nodeAtPoint):

1:02 PM Changeset in webkit [11064] by harrison

• 2 edits
  1 add in trunk/WebCore

Reviewed by John Sullivan.

Test case added:

• manual-tests/whitespace-pre-affinity.html

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

This patch addresses the root cause of the problem by making
RenderText::inlineBox() cope with the fact that in white-space:pre text the
newline characters are not part of any InlineTextBox... they lie "between"
them. Now DOWNSTREAM affinity selects the next text box only if the offset
that is past the current box is actually _in_ the next box.

• khtml/rendering/render_text.cpp: (RenderText::inlineBox):

12:40 PM Changeset in webkit [11063] by darin

• 1 edit
  7 deletes in trunk/WebCore

• removed unused duplicate copy of hash table code

• khtml/misc/hashfunctions.h: Removed.
• khtml/misc/hashmap.h: Removed.
• khtml/misc/hashset.h: Removed.
• khtml/misc/hashtable.cpp: Removed.
• khtml/misc/hashtable.h: Removed.
• khtml/misc/hashtraits.h: Removed.
• khtml/misc/pointerhash.h: Removed.

12:12 PM Changeset in webkit [11062]

• 22 copies
  3 deletes in tags/WebKit-312~5~2

This commit was manufactured by cvs2svn to create tag
'WebKit-312~5~2'.

12:12 PM Changeset in webkit [11061]

• 70 copies
  3 deletes in tags/WebCore-315~10~1

This commit was manufactured by cvs2svn to create tag
'WebCore-315~10~1'.

12:12 PM Changeset in webkit [11060] by thatcher

• 4 edits in branches/Safari-1-3-branch

Versioning to WebKit-312.5.2 and WebCore-315.10.1

12:04 PM Changeset in webkit [11059]

• 53 copies
  22 deletes in tags/WebKit-416~12

This commit was manufactured by cvs2svn to create tag
'WebKit-416~12'.

12:04 PM Changeset in webkit [11058]

• 210 copies
  15 deletes in tags/WebCore-416~14

This commit was manufactured by cvs2svn to create tag
'WebCore-416~14'.

12:04 PM Changeset in webkit [11057] by thatcher

• 4 edits in branches/Ti-2005-009-branch

Versioning to WebCore-416.14 and WebKit-416.12

9:45 AM Changeset in webkit [11056]

• 10 copies
  7 deletes in tags/Safari-521

This commit was manufactured by cvs2svn to create tag 'Safari-521'.

9:45 AM Changeset in webkit [11055] by thatcher

• 1 edit in branches/Safari-Leo-branch/JavaScriptCore/Info.plist

Versioning to 521.1

9:30 AM Changeset in webkit [11054] by darin

• 2 edits in trunk/JavaScriptCore

Reviewed by Tim Hatcher.

• kxmlcore/HashSet.h: Fixed case of "hashfunctions.h" -- needs to be "HashFunctions.h".

9:02 AM Changeset in webkit [12211] by thatcher

• 2 edits in trunk/JavaScriptGlue

Reviewed by Darin.

• JavaScriptGlue.xcodeproj/project.pbxproj: Fixed the FRAMEWORK_SEARCH_PATHS to look in the WebKit Frameworks for the Default config.

12:07 AM Changeset in webkit [11053] by thatcher

• 4 edits in branches/Safari-1-3-branch/WebKit

Merged fix from TOT to Safari-1-3-branch

2005-10-07 John Sullivan <​sullivan@apple.com>

Reviewed by Tim Omernick.

WebKit support for allowing clients to know which frame originated a particular JavaScript alert/dialog.

• WebView.subproj/WebUIDelegatePrivate.h: New optional delegate methods for the three JavaScript alert/dialogs. These are just like the existing ones in WebUIDelegate.h except that each adds a parameter specifying the frame that the JavaScript was running in. Eventually we'll deprecate the old three methods in favor of these in the public API.

• WebCoreSupport.subproj/WebBridge.m: (-[WebBridge runJavaScriptAlertPanelWithMessage:]): Call version of the delegate method that has the frame parameter if the delegate supports it. (-[WebBridge runJavaScriptConfirmPanelWithMessage:]): ditto (-[WebBridge runJavaScriptTextInputPanelWithPrompt:defaultText:returningText:]): ditto

• WebView.subproj/WebDefaultUIDelegate.m: (-[WebDefaultUIDelegate webView:runJavaScriptAlertPanelWithMessage:initiatedByFrame:]): Now implements the new version of the delegate method that includes the frame parameter. (Still doesn't do anything though.) (-[WebDefaultUIDelegate webView:runJavaScriptConfirmPanelWithMessage:initiatedByFrame:]): Now implements the new version of the delegate method that includes the frame parameter. (Still doesn't do anything though.) (-[WebDefaultUIDelegate webView:runJavaScriptTextInputPanelWithPrompt:defaultText:initiatedByFrame:]): Now implements the new version of the delegate method that includes the frame parameter. Doesn't actually use the frame parameter here yet though.

Nov 3, 2005:

11:47 PM Changeset in webkit [11052] by thatcher

• 4 edits in branches/Ti-2005-009-branch/WebKit

Merged fix from TOT to Ti-2005-009-branch

2005-10-07 John Sullivan <​sullivan@apple.com>

Reviewed by Tim Omernick.

WebKit support for allowing clients to know which frame originated a particular JavaScript alert/dialog.

• WebView.subproj/WebUIDelegatePrivate.h: New optional delegate methods for the three JavaScript alert/dialogs. These are just like the existing ones in WebUIDelegate.h except that each adds a parameter specifying the frame that the JavaScript was running in. Eventually we'll deprecate the old three methods in favor of these in the public API.

• WebCoreSupport.subproj/WebBridge.m: (-[WebBridge runJavaScriptAlertPanelWithMessage:]): Call version of the delegate method that has the frame parameter if the delegate supports it. (-[WebBridge runJavaScriptConfirmPanelWithMessage:]): ditto (-[WebBridge runJavaScriptTextInputPanelWithPrompt:defaultText:returningText:]): ditto

• WebView.subproj/WebDefaultUIDelegate.m: (-[WebDefaultUIDelegate webView:runJavaScriptAlertPanelWithMessage:initiatedByFrame:]): Now implements the new version of the delegate method that includes the frame parameter. (Still doesn't do anything though.) (-[WebDefaultUIDelegate webView:runJavaScriptConfirmPanelWithMessage:initiatedByFrame:]): Now implements the new version of the delegate method that includes the frame parameter. (Still doesn't do anything though.) (-[WebDefaultUIDelegate webView:runJavaScriptTextInputPanelWithPrompt:defaultText:initiatedByFrame:]): Now implements the new version of the delegate method that includes the frame parameter. Doesn't actually use the frame parameter here yet though.

10:05 PM Changeset in webkit [11051] by thatcher

• 1 edit
  1 add in branches/Safari-1-3-branch/WebCore

Merged fix from TOT to Safari-1-3-branch

2005-10-06 Beth Dakin <Beth Dakin>

Reviewed by Vicki.

Fix for <rdar://problem/4145535> Crash in khtml::RenderBlock::addOverhangingFloats
with simple HTML test file.

• khtml/rendering/render_block.cpp: Added nil check (khtml::RenderBlock::addOverhangingFloats):

9:59 PM Changeset in webkit [11050] by thatcher

• 2 edits in branches/Ti-2005-009-branch/WebCore

Merged fix from TOT to Ti-2005-009-branch

2005-10-06 Beth Dakin <Beth Dakin>

Reviewed by Vicki.

Fix for <rdar://problem/4145535> Crash in khtml::RenderBlock::addOverhangingFloats
with simple HTML test file.

• khtml/rendering/render_block.cpp: Added nil check (khtml::RenderBlock::addOverhangingFloats):

9:58 PM Changeset in webkit [11049] by thatcher

• in /

file ChangeLog was added on branch Safari-1-3-branch on 2005-11-04 06:04:56 +0000

9:58 PM Changeset in webkit [11048] by justing

• 2 edits in trunk/WebCore

Bugzilla 5604: plainText (visible_text.cpp) performance improvement

Reviewed by justin and darin

Optimized plainText for a ~100% speedup in most cases.

No test cases added, change only effects performance.

• khtml/editing/visible_text.cpp: (khtml::TextIterator::rangeLength): Removed an extraneous comment. (khtml::plainText): The length of the string to be created was pre-calculated to avoid the expense of many QString::append() calls. Removed this since TextIterator::advance() is much more expensive than append.

6:33 PM Changeset in webkit [11047] by ggaren

• 1 edit in trunk/JavaScriptCore/ChangeLog

changelog fixup in response to email from Darin

5:38 PM Changeset in webkit [11046] by thatcher

• 6 edits in trunk

JavaScriptCore:

Reviewed by Darin and Vicki.

• JavaScriptCore.xcodeproj/project.pbxproj: Change to use $(SYSTEM_LIBRARY_DIR) consistently and place $(NEXT_ROOT) in a few spots to make build-root work.

WebCore:

Reviewed by Darin and Vicki.

• WebCore.xcodeproj/project.pbxproj: Change to use $(SYSTEM_LIBRARY_DIR) consistently and place $(NEXT_ROOT) in a few spots to make build-root work.

WebKit:

Reviewed by Darin and Vicki.

• WebKit.xcodeproj/project.pbxproj: Change to use $(SYSTEM_LIBRARY_DIR) consistently and place $(NEXT_ROOT) in a few spots to make build-root work.

5:36 PM Changeset in webkit [12210] by thatcher

• 2 edits in trunk/JavaScriptGlue

Tools:

Reviewed by Darin and Vicki.

• Scripts/build-root: now builds in a sandbox and wont replace your system WebKit or Safari will now also build JavaScriptGlue and WebKitSystemInterface

JavaScriptGlue:

Reviewed by Darin and Vicki.

• JavaScriptGlue.xcodeproj/project.pbxproj: Change to use $(SYSTEM_LIBRARY_DIR) consistently and place $(NEXT_ROOT) in a few spots to make build-root work.

WebBrowser:

Reviewed by Darin and Vicki.

• WebBrowser.xcodeproj/project.pbxproj: Change to use $(SYSTEM_LIBRARY_DIR) consistently and place $(NEXT_ROOT) in a few spots to make build-root work.

WebKitSystemInterface:

Reviewed by Darin and Vicki.

• WebKitSystemInterface.xcodeproj/project.pbxproj: Change to use $(SYSTEM_LIBRARY_DIR) consistently and place $(NEXT_ROOT) in a few spots to make build-root work.

4:28 PM Changeset in webkit [11045]

• 18 copies in branches/kdom-dom-merger/WebCore

This commit was manufactured by cvs2svn to create branch
'kdom-dom-merger'.

4:28 PM Changeset in webkit [11044] by eseidel

• 4 edits
  8 deletes in /

Submitted by: eseidel
Reviewed by: none, SVG merger branch only.

3:53 PM Changeset in webkit [11043] by hyatt

• 2 edits in trunk/WebCore

Cleanup of bidi.cpp. This is bug 5532. I merged it nearly as is, although I
did one additional rename of my own (BidiIterator's "par" -> "block") and
I did not merge the * changes, since that particular style guideline is
"in dispute." :)

Reviewed by darin

• khtml/rendering/bidi.cpp: (khtml::BidiIterator::BidiIterator): (khtml::BidiIterator::pos): (khtml::BidiState::BidiState): (khtml::getBPMWidth): (khtml::BidiContext::BidiContext): (khtml::BidiContext::~BidiContext): (khtml::BidiContext::deref): (khtml::operator==): (khtml::operator!=): (khtml::bidiNext): (khtml::bidiFirst): (khtml::BidiIterator::increment): (khtml::BidiIterator::atEnd): (khtml::BidiIterator::current): (khtml::chopMidpointsAt): (khtml::checkMidpoints): (khtml::appendRunsForObject): (khtml::appendRun): (khtml::embed): (khtml::RenderBlock::constructLine): (khtml::RenderBlock::tabWidth): (khtml::RenderBlock::computeHorizontalPositionsForLine): (khtml::RenderBlock::bidiReorderLine): (khtml::buildCompactRuns): (khtml::RenderBlock::layoutInlineChildren): (khtml::RenderBlock::determineStartPosition): (khtml::RenderBlock::skipWhitespace): (khtml::RenderBlock::findNextLineBreak):

1:05 PM Changeset in webkit [11042] by hyatt

• 3 edits in trunk/WebCore

Make pre-wrap collapse away spaces at the start of a line. There are
still several bugs, namely with overflow and with selection. This is bugzilla
bug 5593.

Reviewed by adele

fast/text/whitespace/pre-wrap.html

• khtml/rendering/bidi.cpp: (khtml::skipNonBreakingSpace): (khtml::shouldCollapseWhiteSpace): (khtml::RenderBlock::skipWhitespace):
• khtml/rendering/render_block.h:

12:04 PM Changeset in webkit [11041] by thatcher

• 3 edits in branches/Safari-Leo-branch

Versioning to 521.

12:00 PM Changeset in webkit [11040] by adele

• 3 edits in trunk/WebCore

Reviewed by Darin.

• khtml/rendering/render_layer.cpp: (khtml::): Made global ScrollAlignment members const (khtml::RenderLayer::scrollRectToVisible): Changed ScrollAlignment parameters to const ScrollAlignment& (khtml::RenderLayer::getRectToExpose): ditto.
• khtml/rendering/render_layer.h: (khtml::RenderLayer::): removed typedefs (khtml::RenderLayer::getVisibleBehavior): Made this static. (khtml::RenderLayer::getPartialBehavior): ditto. (khtml::RenderLayer::getHiddenBehavior): ditto.

11:31 AM Changeset in webkit [11039] by ggaren

• 2 edits in trunk/JavaScriptCore

• Updated JavaScriptCore test results to reflect recent fixes.

• tests/mozilla/expected.html:

11:12 AM Changeset in webkit [11038] by adele

• 11 edits
  5 adds in trunk

LayoutTests:

Test for...
<rdar://problem/3612121> setting focus, by tabbing, searching, or with JavaScript does not scroll overflow:auto/scroll/overlay to reveal focused element (3480)

• fast/overflow/resources/scrollButton1.html: Added.
• fast/overflow/scrollRevealButton-expected.checksum: Added.
• fast/overflow/scrollRevealButton-expected.png: Added.
• fast/overflow/scrollRevealButton-expected.txt: Added.
• fast/overflow/scrollRevealButton.html: Added.

WebCore:

Reviewed by Hyatt.

Clean up of layer scrolling code. Now we can clearly define what to do if a rectangle is fully visible, partially visible, or hidden.
This also fixes a bug with the previous implementation where nested layers/frames would get passed a rect that was too large.

Added fast/overflow/scrollRevealButton.html

• khtml/rendering/render_layer.h: (khtml::RenderLayer::): Added ScrollBehavior enum, ScrollAlignment struct, and ScrollAlignment static members to describe specific scrolling behaviors depending on how visible the rectangle is. In the future, we can add other statics to easily describe desired behaviors.
• khtml/rendering/render_layer.cpp: (khtml::): initialize ScrollAlignment static members: alignCenterIfNeeded, alignToEdgeIfNeeded, alignCenterAlways, alignTopAlways, alignBottomAlways (khtml::RenderLayer::scrollRectToVisible): alignCenterIfNeeded is the default behavior for both directions. Pass the original rect (adjusted if scrolling has occurred) when recursively calling this function. (khtml::RenderLayer::getRectToExpose): Adjusted for new ScrollAlignment parameters.
• khtml/khtml_part.cpp: (KHTMLPart::gotoAnchor): Use the node's rect to determine where to scroll. We used to just use the origin, but this helps us match other browsers better. (KHTMLPart::setActiveNode): Removed scrolling code, since this is now also done in setFocusNode
• khtml/xml/dom_elementimpl.cpp: (ElementImpl::scrollIntoView): Use new ScrollAlignment values to describe scrolling behavior.
• kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::nextKeyViewInFrame): Removed scrolling code, since this is now also done in setFocusNode (KWQKHTMLPart::centerSelectionInVisibleArea): Use new ScrollAlignment values to describe scrolling behavior.
• kwq/WebCoreBridge.mm: (-[WebCoreBridge ensureSelectionVisible]): ditto.
• kwq/KWQScrollView.h: Removed ensureRectVisible, since we no longer use this.
• kwq/KWQScrollView.mm: ditto.
• khtml/khtmlview.cpp: (KHTMLView::doAutoScroll): Removed unnecessary nil checks for enclosingLayer (KHTMLView::focusNextPrevNode): ditto.

10:47 AM Changeset in webkit [11037] by sullivan

• 3 edits in trunk/WebKitTools

Reviewed by Darin.

• Scripts/run-safari: changed message to say "Starting Safari" instead of odd "Start Safari"
• Scripts/run-webkit-tests: changed the way we pass the file parameter to use -NSOpen rather than relying on unlabeled arguments being treated as files. This was failing on some machines that (mysteriously) had NSTreatUnknownArgumentsAsOpen set to NO in com.apple.Safari.plist.

10:16 AM Changeset in webkit [11036] by harrison

• 2 edits
  6 adds in trunk/LayoutTests

Other files that should have been included with my previous commit.

• editing/editing.js:
• editing/inserting/insert-br-quoted-001-expected.checksum: Added.
• editing/inserting/insert-br-quoted-002-expected.checksum: Added.
• editing/inserting/insert-br-quoted-003-expected.checksum: Added.
• editing/inserting/insert-br-quoted-004-expected.checksum: Added.
• editing/inserting/insert-br-quoted-005-expected.checksum: Added.
• editing/inserting/insert-br-quoted-006-expected.checksum: Added.

10:13 AM Changeset in webkit [11035] by ggaren

• 1 edit
  2 adds in trunk/LayoutTests

Layout test for ​http://bugzilla.opendarwin.org/show_bug.cgi?id=5602
REGRESSION: RegExp("[
^{s$]+", "g") returns extra matches}

• fast/js/regexp-lastindex.html: Added.
• fast/js/regexp-lastindex-expected.txt: Added.

10:11 AM Changeset in webkit [11034] by ggaren

• 2 edits in trunk/JavaScriptCore

Reviewed by darin.

• Fixed ​http://bugzilla.opendarwin.org/show_bug.cgi?id=5602 REGRESSION: RegExp("[
  ^{s$]+", "g") returns extra matches}

We now update lastIndex relative to the start of the last match,
rather than the start of the last search. We used to assume that
the two were equal, but that may not be the case in $ matches.

• kjs/regexp_object.cpp: (RegExpProtoFuncImp::callAsFunction):

2:28 AM Changeset in webkit [11033] by eseidel

• 187 edits
  70 adds
  46 deletes in /

Bug #: 3249
Submitted by: eseidel
Reviewed by: none, intermediate branch only.

first steps in KDOM -> DOM merger.

Integration issues:

• Headers (temporarily every kdom header maps to all webcore headers)
• Namespace mapping (KDOM -> DOM, khtml)
• Various KDOM stub classes (temporary)
• Various DOMStringImpl -> DOMString changes
• DOMString::string() -> DOMString::qstring()
• DOMString::handle() -> DOMString::impl()
• DocumentImpl, DocumentType ::impl() -> ::implentation()
• Moving NodeImpl::Id to QualifiedString/AtomicString
• SVGNames.*, SVGElementFactory.* (and perl script to generate them)
• EventImpl::id() -> type() with AtomicString, removal of SVGEventImpl
• Adding svg document and event creation in xml_docimpl.h
• Cleanup event dispatch, in SVGDocumentImpl, using SharedPtr
• exceptioncode additions, for appendNode, createEvent, dispatchEvent, etc.
• Mapped KSVGPart, KDOMPart, KSVGView, KDOMView to KHTMLPart, KHTMLView
• Removed all throw calls, replacing with exceptioncode parameters
• SVGElement derives from StyledElementImpl for now.
• Removed CDFInterface (for now)

Outstanding:

• DocumentBuilder vs. Tokenizer
• Merger of the Ecma systems
• CSS Parsing systems
• Event dispatch/Listening issues
• Merge KSVGView properties into KHTMLView
• requestDocument & CachedDocument vs. requestFrame and CachedObject...

2:28 AM Changeset in webkit [11032]

• 1 copy
  11 deletes in branches/kdom-dom-merger

This commit was manufactured by cvs2svn to create branch
'kdom-dom-merger'.