Reviewed by John
[WebKit-https.git] / WebCore / ChangeLog-2005-08-23
1 2005-01-04  Ken Kocienda  <kocienda@apple.com>
2
3         Reviewed by John
4
5         Fix for these two bugs:
6         
7         <rdar://problem/3938935> REGRESSION (Mail): Pasting into an empty document mangles content
8         <rdar://problem/3939148> REGRESSION (Mail): Pasting mistakenly reverses lines
9
10         * khtml/editing/htmlediting.cpp:
11         (khtml::ReplaceSelectionCommand::doApply): For 3938935, add one more case to handle an empty document; merge
12         neither start nor end. For 3939148, improve the code which adjusts the insertion point during 
13         the process of pasting. It formerly handled only one of the possible cases.
14         * layout-tests/editing/pasteboard/paste-text-015-expected.txt: Added.
15         * layout-tests/editing/pasteboard/paste-text-015.html: Added.
16
17 2005-01-04  David Hyatt  <hyatt@apple.com>
18
19         Fix for 3936571, placeholder attribute should work for normal inputs for Dashboard.
20         
21         Reviewed by john
22
23         * khtml/rendering/render_form.cpp:
24         (RenderLineEdit::updateFromElement):
25         * kwq/KWQLineEdit.mm:
26         (QLineEdit::setPlaceholderString):
27
28 2005-01-04  David Hyatt  <hyatt@apple.com>
29
30         Fix for 3830936, hang on changeforamerica.com.  Make sure to ignore the style not yet available option when
31         returning pseudo-styles.
32         
33         Reviewed by john
34
35         * khtml/css/cssstyleselector.cpp:
36         (khtml::CSSStyleSelector::matchRulesForList):
37         (khtml::CSSStyleSelector::pseudoStyleForElement):
38         * khtml/rendering/render_style.cpp:
39         (RenderStyle::addPseudoStyle):
40
41 2005-01-04  Darin Adler  <darin@apple.com>
42
43         - rolled out my custom tag name change again -- it caused a 1 ms PLT regression
44
45         * khtml/css/cssstyleselector.cpp:
46         * khtml/editing/htmlediting.cpp:
47         * khtml/editing/selection.cpp:
48         * khtml/editing/visible_position.cpp:
49         * khtml/html/dtd.cpp:
50         * khtml/html/htmlparser.cpp:
51         * khtml/html/htmlparser.h:
52         * khtml/html/htmltokenizer.cpp:
53         * khtml/misc/htmlhashes.cpp:
54         * khtml/misc/htmlhashes.h:
55         * khtml/misc/htmltags.c:
56         * khtml/misc/htmltags.h:
57         * khtml/misc/maketags:
58         * khtml/xml/dom_docimpl.cpp:
59         * khtml/xml/dom_docimpl.h:
60         * khtml/xml/dom_nodeimpl.cpp:
61         * khtml/xml/dom_position.cpp:
62         * kwq/KWQRenderTreeDebug.cpp:
63
64 2005-01-04  Ken Kocienda  <kocienda@apple.com>
65
66         Reviewed by John
67
68         Fix for this bug:
69         
70         <rdar://problem/3927554> Style info applied to remainder of document after a newline is entered
71
72         * khtml/editing/htmlediting.cpp:
73         (khtml::InsertParagraphSeparatorCommand::doApply): Clean up and simplification in code that inserts
74         a paragraph separator. The bug was all about applying styles to the new paragraph that did not need
75         to be applied. Now the code will detect when at the end of a style run and will not move and apply 
76         that ending style to the new paragraph, though it will place that style into the typing style. This
77         seems to match NSText behavior.
78         * layout-tests/editing/inserting/insert-div-021-expected.txt: This test result exhibited the bug fixed 
79         here. The only reason it was not noticed is that the erroneously copied inline was a span, and so did
80         not have any visible effect on the document.
81         * layout-tests/editing/inserting/insert-div-022-expected.txt: Added.
82         * layout-tests/editing/inserting/insert-div-022.html: Added.
83
84 2005-01-04  David Hyatt  <hyatt@apple.com>
85
86         Fix for 3904562, make sure to clear the outline dictionary when drawing so that random focus rings dont get drawn
87         all over the place.
88         
89         Reviewed by john
90
91         * khtml/rendering/render_flow.cpp:
92         (RenderFlow::paintLines):
93
94 2005-01-03  David Hyatt  <hyatt@apple.com>
95
96         Fix for float painting regressions 3932524, 3931664, and 3933068.  Make the noPaint flag setting more
97         robust and make it work regardless of which objects get a layout or not.
98         
99         Reviewed by mjs
100
101         * khtml/rendering/render_block.cpp:
102         (khtml::RenderBlock::insertFloatingObject):
103         (khtml::RenderBlock::addOverhangingFloats):
104         (khtml::RenderBlock::addIntrudingFloats):
105         * khtml/rendering/render_block.h:
106         (khtml::RenderBlock::FloatingObject::FloatingObject):
107
108 2005-01-03  Maciej Stachowiak  <mjs@apple.com>
109
110         Reviewed by Kevin.
111
112         <rdar://problem/3935390> Tiger 8A341: nil deref crash in DOM::NodeImpl::traverseNextNode
113         
114         * khtml/html/html_miscimpl.cpp:
115         (HTMLCollectionImpl::traverseNextItem): Assert that the starting point is not nil,
116         it should not ever be (now).
117         (HTMLCollectionImpl::item): When traversing items stop when we hit
118         nil, meaning the end to avoid triggering above assert (formerly crash).
119
120 2005-01-03  Maciej Stachowiak  <mjs@apple.com>
121
122         Reviewed by John and Kevin.
123
124         <rdar://problem/3870317> REGRESSION(125.9-125.11) broken behavior at test.profoundlearning.com - used to
125         
126         * khtml/ecma/kjs_window.cpp:
127         (Window::get): Look up frame names before buitin window properties
128         to match other browsers. This regressed because we added a builtin
129         "toolbar" property but this site had a frame with that name.
130
131 2004-12-21  Maciej Stachowiak  <mjs@apple.com>
132
133         Reviewed by Darin.
134
135         <rdar://problem/3888931> frame naming allows malicious site to bring up a window when you click on a link in another
136         
137         Added opener bridge method to help WebKit implement security check
138         for named frame visibility.
139         
140         * khtml/khtml_part.h:
141         * kwq/WebCoreBridge.h:
142         * kwq/WebCoreBridge.mm:
143         (-[WebCoreBridge opener]):
144
145 2005-01-03  Ken Kocienda  <kocienda@apple.com>
146
147         Reviewed by John
148
149         Fix for this bug:
150         
151         <rdar://problem/3933926> Tiger8A341: Mail crashes while forwarding embedded HTML message in -[WebCoreBridge ensureSelectionVisible]
152
153         * kwq/WebCoreBridge.mm:
154         (-[WebCoreBridge ensureSelectionVisible]): Put in some null checks to prevent crash experienced in bug.
155
156 2005-01-03  David Hyatt  <hyatt@apple.com>
157
158         Reviewed by NOBODY (OOPS!).
159
160         * khtml/rendering/render_block.cpp:
161         (khtml::RenderBlock::layoutPositionedObjects):
162
163 2005-01-03  Ken Kocienda  <kocienda@apple.com>
164
165         Reviewed by Harrison
166         
167         Fix for this bug:
168         
169         <rdar://problem/3928250> REGRESSION (Mail): Typing style lost after hitting return key
170
171         * khtml/editing/htmlediting.cpp:
172         (khtml::InsertLineBreakCommand::preservesTypingStyle): Now implemented, returning yes for this command.
173         (khtml::InsertParagraphSeparatorCommand::InsertParagraphSeparatorCommand): Initialize new typing style member variable.
174         (khtml::InsertParagraphSeparatorCommand::~InsertParagraphSeparatorCommand): Deref new typing style member variable.
175         (khtml::InsertParagraphSeparatorCommand::preservesTypingStyle): Now implemented, returning yes for this command.
176         (khtml::InsertParagraphSeparatorCommand::setFullTypingStyleBeforeInsertion): New function to set the typing style
177         (khtml::InsertParagraphSeparatorCommand::calculateAndSetTypingStyleAfterInsertion): Function called after the <p>
178         insertion is done. This function diffs the style created in setFullTypingStyleBeforeInsertion() with the style
179         of the new <p> and only sets those styles needed to preserve the style in effect before the insertion.
180         (khtml::InsertParagraphSeparatorCommand::doApply): Call new functions.
181         (khtml::TypingCommand::preservesTypingStyle): Now yes for inserting line breaks and paragraphs.
182          * khtml/editing/htmlediting.h: Declare new functions.
183
184 2004-12-25  Kevin Decker  <kdecker@apple.com>
185
186         Reviewed by Hyatt.
187
188         Fixed: <rdar://problem/3505072> hang in KHTMLParser::parseToken (consulting.soroos.net)
189         
190         * khtml/html/dtd.cpp: <label> needed a much higher priority such that it will close block elements.  Malformed HTML (ugh) is the only reason why we need to do this. 
191
192 2004-12-23  Darin Adler  <darin@apple.com>
193
194         Reviewed by Ken.
195
196         - re-fixed <rdar://problem/3760910> Request to include support for custom tag names in HTML (they already work in XML)
197
198         * khtml/css/cssstyleselector.cpp: (khtml::CSSStyleSelector::checkOneSelector): Changed
199         some code that used ID_LAST_TAG in a slightly wrong, but harmless, way.
200
201         * khtml/editing/htmlediting.cpp:
202         (khtml::debugPosition): Use nodeName rather than getTagName, since the latter works for
203         per-document tags and is just better all around for things like the document.
204         (khtml::debugNode): Ditto.
205         * khtml/editing/selection.cpp: (khtml::Selection::debugPosition): Ditto.
206         * khtml/editing/visible_position.cpp: (khtml::VisiblePosition::debugPosition): Ditto.
207         * khtml/xml/dom_nodeimpl.cpp: (NodeImpl::displayNode): Ditto.
208         * khtml/xml/dom_position.cpp: (DOM::Position::debugPosition): Ditto.
209
210         * khtml/html/dtd.cpp: (DOM::checkChild): Use ID_LAST_TAG rather than 1000 for the check that allows
211         non-HTML elements to be nested as desired.
212
213         * khtml/html/htmlparser.h: Change forbidden tag array to be allocated in the object rather than on
214         the heap. Also use ID_LAST_TAG for the array size; the old code used ID_CLOSE_TAG which is now a
215         much larger number.
216         * khtml/html/htmlparser.cpp:
217         (KHTMLParser::KHTMLParser): Remove code that creates the forbidden tag array on the heap; makes more
218         sense to just have the array be a member so we don't have to use new and delete on it. Also needed
219         to eliminate code that used ID_CLOSE_TAG for an array size.
220         (KHTMLParser::~KHTMLParser): Remove code that deletes the forbidden tag array. Also changed the code
221         that manages isindex to use deref instead of delete.
222         (KHTMLParser::reset): Use safer sizeof for clearing the forbidden tag array. Old code had the
223         mistake of using ID_CLOSE_TAG for the array size too.
224         (KHTMLParser::parseToken): Removed code that checks for bogus tags; it's not clear why it was ever
225         there and it would prevent custom tags from working. Added range check before using the forbidden
226         tag array with the token ID since custom tags will use index values past the end of the array.
227         (KHTMLParser::getElement): Removed most of the code that makes an element by ID; now the work is done
228         inside the DocumentImpl createHTMLElement. Any code that needs to be different than a dynamic
229         createElement call is still here. Last time I left out a few form element types from this switch;
230         fixed now.
231         * khtml/html/htmltokenizer.cpp: (khtml::HTMLTokenizer::parseTag): Call the tagId function in the
232         document if getTagID fails; this creates a unique per-document ID.
233
234         * khtml/misc/htmlhashes.h: Changed return types to unsigned short.
235         * khtml/misc/htmlhashes.cpp:
236         (khtml::getTagID): Changed return type to unsigned short.
237         (khtml::getAttrID): Ditto.
238
239         * khtml/misc/htmltags.c: Regenerated.
240         * khtml/misc/htmltags.h: Regenerated.
241
242         * khtml/misc/maketags: Changed the generated constants to use "const unsigned short" instead of "#define".
243         Changed ID_CLOSE_TAG to be a fixed value of 32000 rather than being just past the IDs of the HTML tags.
244         Also rewrote getTagName to work with the new scheme.
245
246         * khtml/xml/dom_docimpl.h: Added overload of createHTMLElement, made it non-virtual since it's not overriden.
247         * khtml/xml/dom_docimpl.cpp:
248         (DocumentImpl::createHTMLElement): Refactored into two separate functions, one that takes the tag ID.
249         Also updated for a few tags that the parser handled but this did not.
250
251         * kwq/KWQRenderTreeDebug.cpp:
252         (getTagName): Added. Works for custom nodes, because it calls nodeName rather than using getTagName on
253         the tag ID directly, which only works for standard nodes.
254         (operator<<): Update to call getTagName.
255         (nodePositionRelativeToRoot): Ditto.
256         (writeSelection): Ditto.
257
258 === Safari-177 ===
259
260 2004-12-22  Darin Adler  <darin@apple.com>
261
262         - rolled out my custom tag name change -- it broke amazon.com
263
264         * khtml/css/cssstyleselector.cpp:
265         * khtml/editing/htmlediting.cpp:
266         * khtml/editing/selection.cpp:
267         * khtml/editing/visible_position.cpp:
268         * khtml/html/dtd.cpp:
269         * khtml/html/htmlparser.cpp:
270         * khtml/html/htmlparser.h:
271         * khtml/html/htmltokenizer.cpp:
272         * khtml/misc/htmlhashes.cpp:
273         * khtml/misc/htmlhashes.h:
274         * khtml/misc/htmltags.c:
275         * khtml/misc/htmltags.h:
276         * khtml/misc/maketags:
277         * khtml/xml/dom_docimpl.cpp:
278         * khtml/xml/dom_docimpl.h:
279         * khtml/xml/dom_nodeimpl.cpp:
280         * khtml/xml/dom_position.cpp:
281         * kwq/KWQRenderTreeDebug.cpp:
282
283 2004-12-22  David Harrison  <harrison@apple.com>
284
285         Reviewed by Darin Adler.
286
287         * khtml/editing/selection.cpp:
288         (khtml::Selection::validate):
289         The selecting/deselecting bad behavior is because the Selection code that expands by words
290         had an inaccurate test for being at the end of the document (where double-clicking needs
291         to select the last word).  Fixed that check.
292
293 2004-12-22  Adele Amchan  <adele@apple.com>
294
295         Reviewed by Chris.
296
297         Fix for <rdar://problem/3911650> tabs at safeway.com stop working after a while
298
299         * kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::scrollToAnchor): removed call to cancelRedirection 
300           so that we match Firefox and WinIE behavior.
301
302 2004-12-22  Darin Adler  <darin@apple.com>
303
304         Reviewed by Ken.
305
306         - fixed <rdar://problem/3760910> Request to include support for custom tag names in HTML (they already work in XML)
307
308         * khtml/css/cssstyleselector.cpp: (khtml::CSSStyleSelector::checkOneSelector): Changed
309         some code that used ID_LAST_TAG in a slightly wrong, but harmless, way.
310
311         * khtml/editing/htmlediting.cpp:
312         (khtml::debugPosition): Use nodeName rather than getTagName, since the latter works for
313         per-document tags and is just better all around for things like the document.
314         (khtml::debugNode): Ditto.
315         * khtml/editing/selection.cpp: (khtml::Selection::debugPosition): Ditto.
316         * khtml/editing/visible_position.cpp: (khtml::VisiblePosition::debugPosition): Ditto.
317         * khtml/xml/dom_nodeimpl.cpp: (NodeImpl::displayNode): Ditto.
318         * khtml/xml/dom_position.cpp: (DOM::Position::debugPosition): Ditto.
319
320         * khtml/html/dtd.cpp: (DOM::checkChild): Use ID_LAST_TAG rather than 1000 for the check that allows
321         non-HTML elements to be nested as desired.
322
323         * khtml/html/htmlparser.h: Change forbidden tag array to be allocated in the object rather than on
324         the heap. Also use ID_LAST_TAG for the array size; the old code used ID_CLOSE_TAG which is now a
325         much larger number.
326         * khtml/html/htmlparser.cpp:
327         (KHTMLParser::KHTMLParser): Remove code that creates the forbidden tag array on the heap; makes more
328         sense to just have the array be a member so we don't have to use new and delete on it. Also needed
329         to eliminate code that used ID_CLOSE_TAG for an array size.
330         (KHTMLParser::~KHTMLParser): Remove code that deletes the forbidden tag array. Also changed the code
331         that manages isindex to use deref instead of delete.
332         (KHTMLParser::reset): Use safer sizeof for clearing the forbidden tag array. Old code had the
333         mistake of using ID_CLOSE_TAG for the array size too.
334         (KHTMLParser::parseToken): Removed code that checks for bogus tags; it's not clear why it was ever
335         there and it would prevent custom tags from working. Added range check before using the forbidden
336         tag array with the token ID since custom tags will use index values past the end of the array.
337         (KHTMLParser::getElement): Removed most of the code that makes an element by ID; now the work is done
338         inside the DocumentImpl createHTMLElement. Any code that needs to be different than a dynamic
339         createElement call is still here.
340         * khtml/html/htmltokenizer.cpp: (khtml::HTMLTokenizer::parseTag): Call the tagId function in the
341         document if getTagID fails; this creates a unique per-document ID.
342
343         * khtml/misc/htmlhashes.h: Changed return types to unsigned short.
344         * khtml/misc/htmlhashes.cpp:
345         (khtml::getTagID): Changed return type to unsigned short.
346         (khtml::getAttrID): Ditto.
347
348         * khtml/misc/htmltags.c: Regenerated.
349         * khtml/misc/htmltags.h: Regenerated.
350
351         * khtml/misc/maketags: Changed the generated constants to use "const unsigned short" instead of "#define".
352         Changed ID_CLOSE_TAG to be a fixed value of 32000 rather than being just past the IDs of the HTML tags.
353         Also rewrote getTagName to work with the new scheme.
354
355         * khtml/xml/dom_docimpl.h: Added overload of createHTMLElement, made it non-virtual since it's not overriden.
356         * khtml/xml/dom_docimpl.cpp:
357         (DocumentImpl::createHTMLElement): Refactored into two separate functions, one that takes the tag ID.
358         Also updated for a few tags that the parser handled but this did not.
359
360         * kwq/KWQRenderTreeDebug.cpp:
361         (getTagName): Added. Works for custom nodes, because it calls nodeName rather than using getTagName on
362         the tag ID directly, which only works for standard nodes.
363         (operator<<): Update to call getTagName.
364         (nodePositionRelativeToRoot): Ditto.
365         (writeSelection): Ditto.
366
367 2004-12-21  David Harrison  <harrison@apple.com>
368
369         Reviewed by Ken Kocienda.
370
371         <rdar://problem/3924934> REGRESSION: double click at end of line selects start of next line
372
373         Problem was the TextIterator was not handling exitNode() from a P block properly.
374         
375         * khtml/editing/visible_text.cpp:
376         (khtml::TextIterator::TextIterator):
377         Add new param that specifies whether the iterator is for content or for searching.
378         Search iterators do not prevent newlines at the beginning.
379         (khtml::TextIterator::advance):
380         Added some comments.
381         (khtml::TextIterator::handleTextNode):
382         Added some comments.
383         (khtml::TextIterator::exitNode):
384         Emit newline for P (and other) blocks with position following the block, instead of the m_lastTextNode.
385         (khtml::TextIterator::emitCharacter):
386         Added some comments.
387         (khtml::TextIterator::range):
388         Added some comments.
389         (khtml::CharacterIterator::CharacterIterator):
390         Specify search type TextIterator.
391         (khtml::CharacterIterator::advance):
392         * khtml/editing/visible_text.h:
393         (khtml::):
394         Add new TextIterator::TextIterator param that specifies whether the iterator is for content or for searching.
395         * khtml/editing/visible_units.cpp:
396         (khtml::nextWordBoundary):
397         Specify search type TextIterator.
398
399 2004-12-21  David Harrison  <harrison@apple.com>
400
401         Reviewed by Ken Kocienda.
402
403         <rdar://problem/3924695> REGRESSION (Mail): double-clicking past end of line shows no selection, should select to EOL
404         
405         Problem was that RenderText::setSelectionState did not handle the SelectionStart case where start and end are the end of the line.
406         Fixed by pretending the start == end-1 in that situation, as long as end > 0.
407         
408         * khtml/rendering/render_text.cpp:
409         (RenderText::setSelectionState)
410
411 2004-12-21  Maciej Stachowiak  <mjs@apple.com>
412
413         Reviewed by John.
414
415         <rdar://problem/3929187> WebKit needs to restrict access to certain window operations by domain
416         
417         * khtml/ecma/kjs_window.cpp:
418         (Window::get): Change most window functions to be restricted by
419         XSS domain check.
420
421 2004-12-21  Ken Kocienda  <kocienda@apple.com>
422
423         Reviewed by John
424
425         Fix for this bug:
426         
427         <rdar://problem/3928305> selecting an entire line and typing over causes new inserted text at top of document
428
429         * khtml/editing/htmlediting.cpp:
430         (khtml::DeleteSelectionCommand::insertPlaceholderForAncestorBlockContent): New function to detect case
431         formerly undetected and unhandled. This is the crux of the bug fix.
432         (khtml::DeleteSelectionCommand::doApply): Call insertPlaceholderForAncestorBlockContent() during
433         execution of command.
434         * khtml/editing/htmlediting.h: Declare new function.
435         * layout-tests/editing/deleting/delete-3928305-fix-expected.txt: Added.
436         * layout-tests/editing/deleting/delete-3928305-fix.html: Added.
437
438 2004-12-21  Ken Kocienda  <kocienda@apple.com>
439
440         Reviewed by me
441
442         * khtml/editing/htmlediting.cpp:
443         (khtml::CompositeEditCommand::removeBlockPlaceholderIfNeeded): Note to self: Must compile code before
444         checking in (aka must return false from function returning bool).
445
446 2004-12-21  Ken Kocienda  <kocienda@apple.com>
447
448         Reviewed by John
449         
450         Fix for this bug:
451         
452         <rdar://problem/3927752> Crash in khtml::CompositeEditCommand::removeBlockPlaceholderIfNeeded(DOM::NodeImpl*)
453
454         * khtml/editing/htmlediting.cpp:
455         (khtml::CompositeEditCommand::insertBlockPlaceholderIfNeeded): Added some null checks.
456         (khtml::CompositeEditCommand::removeBlockPlaceholderIfNeeded): Ditto.
457
458 2004-12-21  Ken Kocienda  <kocienda@apple.com>
459
460         Reviewed by Darin
461
462         * khtml/editing/htmlediting.cpp:
463         (khtml::ReplacementFragment::mergeStartNode): Refine concept of how this node is found based on
464         further experiements.
465         (khtml::ReplaceSelectionCommand::doApply): Add a special case for determining merges that need to
466         be done if the insertion point is in an empty block.
467         * layout-tests/editing/pasteboard/paste-text-012-expected.txt: Added.
468         * layout-tests/editing/pasteboard/paste-text-012.html: Added.
469         * layout-tests/editing/pasteboard/paste-text-013-expected.txt: Added.
470         * layout-tests/editing/pasteboard/paste-text-013.html: Added.
471         * layout-tests/editing/pasteboard/paste-text-014-expected.txt: Added.
472         * layout-tests/editing/pasteboard/paste-text-014.html: Added.
473
474 2004-12-21  Darin Adler  <darin@apple.com>
475
476         Reviewed by Ken.
477
478         - fixed <rdar://problem/3899133> text search in a Safari window takes a very long time on Tiger updates page (and some other pages)
479
480         * khtml/editing/visible_text.h: Add an "offset base node" parameter to emitCharacter, and also
481         add a field to track it. Must make a few things mutable so we can update them in the range accessor.
482         * khtml/editing/visible_text.cpp:
483         (khtml::TextIterator::advance): Pass in base node and offsets rather than computing actual offsets
484         using the node's index. We only compute the node index if actually asked for the range.
485         (khtml::TextIterator::handleTextNode): Pass 0 for base node and set base node to 0 when setting
486         up the offsets.
487         (khtml::TextIterator::handleTextBox): Ditto.
488         (khtml::TextIterator::handleReplacedElement): Pass base node and set base node instead of calling
489         nodeIndex.
490         (khtml::TextIterator::handleNonTextNode): Pass 0 for offset.
491         (khtml::TextIterator::exitNode): More of the same.
492         (khtml::TextIterator::emitCharacter): Ditto.
493         (khtml::TextIterator::range): If an offset base node is stored, then get its node index, and then
494         add that in to the offsets. Doing the work here guarantees it's done only once when doing a text search.
495
496         - another small fix
497
498         * khtml/khtml_part.cpp: Removed SPEED_DEBUG define. Not sure why it was on.
499
500 === Safari-176 ===
501
502 2004-12-20  Ken Kocienda  <kocienda@apple.com>
503
504         Reviewed by Darin
505
506         * khtml/editing/htmlediting.cpp:
507         (khtml::DeleteSelectionCommand::moveNodesAfterNode): My one-liner for this bug introduced layout test
508         regressions: <rdar://problem/3926142> REGRESSION (Mail): Deleting text decreases quote level
509         Rolling out until I can develop a real fix.
510
511 2004-12-20  David Harrison  <harrison@apple.com>
512
513         Reviewed by Dave Hyatt.
514         
515         Initial checkin of AXTextMarkerRef support.
516
517         * khtml/xml/dom_docimpl.cpp:
518         (DocumentImpl::updateSelection):
519         (DocumentImpl::setFocusNode):
520         * kwq/KWQAccObject.h:
521         * kwq/KWQAccObject.mm:
522         (-[KWQAccObject detach]):
523         (-[KWQAccObject anchorElement]):
524         (-[KWQAccObject addChildrenToArray:]):
525         (-[KWQAccObject accessibilityAttributeNames]):
526         (-[KWQAccObject accessibilityActionDescription:]):
527         (-[KWQAccObject accessibilityPerformAction:]):
528         (-[KWQAccObject textMarkerRangeFromMarkers:andEndMarker:]):
529         (-[KWQAccObject textMarkerForVisiblePosition:]):
530         (-[KWQAccObject visiblePositionForTextMarker:]):
531         (-[KWQAccObject AXTextMarkerRangeCopyStartMarkerWrapper:]):
532         (-[KWQAccObject AXTextMarkerRangeCopyEndMarkerWrapper:]):
533         (-[KWQAccObject visiblePositionForStartOfTextMarkerRange:]):
534         (-[KWQAccObject visiblePositionForEndOfTextMarkerRange:]):
535         (-[KWQAccObject accessibilityAttributeValue:]):
536         (-[KWQAccObject accessibilityParameterizedAttributeNames]):
537         (-[KWQAccObject textMarkerRangeFromVisiblePositions:andEndPos:]):
538         (-[KWQAccObject getSelectedTextMarkerRange]):
539         (-[KWQAccObject doAXLineForTextMarker:]):
540         (-[KWQAccObject doAXTextMarkerRangeForLine:]):
541         (-[KWQAccObject doAXStringForTextMarkerRange:]):
542         (-[KWQAccObject doAXNextTextMarkerForTextMarker:]):
543         (-[KWQAccObject doAXPreviousTextMarkerForTextMarker:]):
544         (-[KWQAccObject doAXLeftWordTextMarkerRangeForTextMarker:]):
545         (-[KWQAccObject doAXRightWordTextMarkerRangeForTextMarker:]):
546         (-[KWQAccObject doAXLeftLineTextMarkerRangeForTextMarker:]):
547         (-[KWQAccObject doAXRightLineTextMarkerRangeForTextMarker:]):
548         (-[KWQAccObject accessibilityAttributeValue:forParameter:]):
549         (-[KWQAccObject accessibilityFocusedUIElement]):
550         (-[KWQAccObject clearChildren]):
551         (-[KWQAccObject accObjectID]):
552         (-[KWQAccObject setAccObjectID:]):
553         (-[KWQAccObject removeAccObjectID]):
554         * kwq/KWQAccObjectCache.h:
555         * kwq/KWQAccObjectCache.mm:
556         (KWQAccObjectCache::KWQAccObjectCache):
557         (KWQAccObjectCache::~KWQAccObjectCache):
558         (KWQAccObjectCache::getAccObjectID):
559         (KWQAccObjectCache::removeAccObjectID):
560         (KWQAccObjectCache::textMarkerForVisiblePosition):
561         (KWQAccObjectCache::visiblePositionForTextMarker):
562
563 2004-12-19  Darin Adler  <darin@apple.com>
564
565         * kwq/KWQFoundationExtras.h: (KWQCFAutorelease): Added nil check.
566
567 2004-12-19  Darin Adler  <darin@apple.com>
568
569         Reviewed by Kevin.
570
571         - a garbage collection fix
572
573         * kwq/KWQFoundationExtras.h: (KWQCFAutorelease): Added a more-extensive comment and fixed the BUILDING_ON_PANTHER
574         #ifndef to use #if (in practice, either is OK).
575         * kwq/KWQKURL.mm: (KURL::getNSURL): Use KWQCFAutorelease instead of autorelease.
576
577 2004-12-17  David Hyatt  <hyatt@apple.com>
578
579         Fix for 3923255, specified percentage heights of divs with overflow auto inside tables not honored.
580         
581         Reviewed by kocienda
582
583         * khtml/rendering/render_box.cpp:
584         (RenderBox::calcPercentageHeight):
585         * khtml/rendering/render_table.cpp:
586         (RenderTableSection::layoutRows):
587
588 2004-12-17  David Harrison  <harrison@apple.com>
589
590         Reviewed by Ken Kocienda.
591
592         <rdar://problem/3924930> REGRESSION: triple click does not select to end of line
593         
594         * khtml/editing/visible_units.cpp:
595         (khtml::endOfParagraph):
596         When includeLineBreak is true, allow traversal to next node after enclosingBlockFlowElement.
597
598 2004-12-17  Ken Kocienda  <kocienda@apple.com>
599
600         Reviewed by Vicki
601
602         * khtml/editing/htmlediting.cpp:
603         (khtml::InsertParagraphSeparatorInQuotedContentCommand::doApply): Missed deleting some now-obsolete expcetion
604         codes when I added the createBreakElement() function.
605
606 2004-12-17  Richard Williamson   <rjw@apple.com>
607
608         Set the floor of max cacheable object size to 40K.  This restores
609         the long standing floor.  Lower floor deleteriously impacts the PLT.
610         Reviewed by Hyatt.
611
612         * khtml/khtml_part.cpp:
613         (KHTMLPart::checkCompleted):
614         * khtml/misc/loader.cpp:
615         (Cache::setSize):
616
617 2004-12-17  Ken Kocienda  <kocienda@apple.com>
618
619         Reviewed by Hyatt
620
621         Fix for this bug:
622         
623         <rdar://problem/3926142> REGRESSION (Mail): Deleting text decreases quote level
624
625         * khtml/editing/htmlediting.cpp:
626         (khtml::DeleteSelectionCommand::moveNodesAfterNode): Stop merging nodes when a <br> is hit.
627         Formerly checked only for block boundary.
628
629 2004-12-17  David Harrison  <harrison@apple.com>
630
631         Reviewed by Darin.
632
633                 Add KWQCFAutorelease for autoreleasing CF objects.
634         * kwq/KWQFoundationExtras.h:
635         (KWQCFAutorelease):
636         New.
637
638 2004-12-17  David Harrison  <harrison@apple.com>
639
640         Reviewed by Darin.
641
642                 Fix GC compatibility in getNSString.
643
644         * kwq/KWQString.mm:
645         (QString::getNSString):
646                 Use NSString allocator instead of CFString, so that autorelease works under GC.
647
648 2004-12-17  Ken Kocienda  <kocienda@apple.com>
649
650         Reviewed by Hyatt
651
652         Fix for this bug:
653         
654         <rdar://problem/3890973> REGRESSION (Mail): Deleting reorders remaining text
655
656         * khtml/editing/htmlediting.cpp:
657         (khtml::DeleteSelectionCommand::moveNodesAfterNode): Added check to ensure that moving content will not
658         move it after the <body> element.
659
660 2004-12-17  Ken Kocienda  <kocienda@apple.com>
661
662         Reviewed by me
663         
664         Added new layout tests covering cases from recent bug fixes.
665
666         * layout-tests/editing/inserting/insert-div-018-expected.txt: Added.
667         * layout-tests/editing/inserting/insert-div-018.html: Added.
668         * layout-tests/editing/inserting/insert-div-019-expected.txt: Added.
669         * layout-tests/editing/inserting/insert-div-019.html: Added.
670         * layout-tests/editing/inserting/insert-div-020-expected.txt: Added.
671         * layout-tests/editing/inserting/insert-div-020.html: Added.
672         * layout-tests/editing/inserting/insert-div-021-expected.txt: Added.
673         * layout-tests/editing/inserting/insert-div-021.html: Added.
674
675 2004-12-17  Ken Kocienda  <kocienda@apple.com>
676
677         Reviewed by me
678
679         Changes in layout resulting from giving <p> elements no margin rather than 0.1em margin.
680
681         * layout-tests/editing/inserting/insert-div-013-expected.txt
682         * layout-tests/editing/inserting/insert-div-014-expected.txt
683
684 2004-12-17  Ken Kocienda  <kocienda@apple.com>
685
686         Reviewed by John
687
688         Fix for this bug:
689         
690         <rdar://problem/3925317> Increase quote in email and with cursor below the quote the new line appears above the quote
691
692         * khtml/editing/htmlediting.cpp:
693         (khtml::InsertParagraphSeparatorCommand::doApply):
694         * khtml/editing/visible_position.cpp: Refine rules for whether to use the starting node or the starting
695         block as the reference node for the insertion of the new block.
696         (khtml::isLastVisiblePositionInBlock): Tweak rules again. Descendants of following blocks should answer true.
697         I worked this all out on the whiteboard this time. This should be the last tweak.
698
699 2004-12-17  Kevin Decker  <kdecker@apple.com>
700
701         Reviewed by Ken.
702
703         Fixed <rdar://problem/3824438> Need a clean way for Dashboard to detect when an XML parsing error occurs
704
705         * khtml/xml/xml_tokenizer.cpp:
706         (khtml::XMLTokenizer::insertErrorMessageBlock): In the xml error report, instead of a generic <div>, use <parsererror> to match Mozilla.
707
708 2004-12-16  Ken Kocienda  <kocienda@apple.com>
709
710         Reviewed by Chris
711
712         Fix for this bug:
713         
714         <rdar://problem/3924888> REGRESSION (Mail): Hitting return key at end of line does not insert visible newline
715
716         * khtml/editing/htmlediting.cpp:
717         (khtml::InsertParagraphSeparatorCommand::doApply): Tweaked code I added just before to fix 3924486, so that
718         it checks the downstream node for the starting position.
719
720 2004-12-16  Ken Kocienda  <kocienda@apple.com>
721
722         Reviewed by Chris
723
724         Added new createBreakElement() function that makes creating a <br> element a one-liner.
725         Converted all the code that used to call the DocumentImpl, and dealt with the exceptionCode,
726         over to this new helper.
727
728         * khtml/editing/htmlediting.cpp:
729         (khtml::DeleteSelectionCommand::moveNodesAfterNode): Now uses new helper function.
730         (khtml::InsertLineBreakCommand::doApply): Ditto.
731         (khtml::InsertParagraphSeparatorCommand::doApply): Ditto.
732         (khtml::InsertParagraphSeparatorInQuotedContentCommand::doApply): Ditto.
733         (khtml::ReplaceSelectionCommand::doApply): Ditto.
734         (khtml::createBreakElement): New helper.
735         * khtml/editing/htmlediting.h: Ditto.
736
737 2004-12-16  David Hyatt  <hyatt@apple.com>
738
739         Fix a bug where vertical-align values that depended on their parent's value went crazy if the parent was
740         vertical-align top or bottom.  The bug is 3771007, bankofamerica's ebills page.
741         
742         Reviewed by kocienda
743
744         * khtml/rendering/render_object.cpp:
745         (RenderObject::getVerticalPosition):
746
747 2004-12-16  Ken Kocienda  <kocienda@apple.com>
748
749         Reviewed by John
750
751         Fix for these bugs:
752         
753         <rdar://problem/3924486> REGRESSION (Mail): Hitting return key does not insert visible newline
754         <rdar://problem/3924579> REGRESSION (Mail): After deleting, hitting return key does not insert visible newline
755
756         * khtml/editing/htmlediting.cpp:
757         (khtml::InsertParagraphSeparatorCommand::doApply): For the first bug, detect when inserting a <p> will
758         make a visible <br> collapse. For the second bug, move the code that inserts the new <p> after
759         a check which may move the node used as the reference node for the insertion. Also change this code to
760         insert the <p> after the last sibling of the starting position, to move it past the nodes we're going
761         to want to shift into the new <p>.
762
763 2004-12-16  Darin Adler  <darin@apple.com>
764
765         Reviewed by Ken.
766
767         - use <p> elements rather than <br> when pasting plain text; also use margin of 0 rather than 0.1 em
768           after talking this over with Dave and Ken
769
770         * khtml/editing/htmlediting.h: Added createDefaultParagraphElement and createBlockPlaceholderElement.
771         * khtml/editing/htmlediting.cpp:
772         (khtml::CompositeEditCommand::insertBlockPlaceholderIfNeeded): Use new createBlockPlaceholderElement
773         helper so this can share code with the plain-text conversion code.
774         (khtml::InsertParagraphSeparatorCommand::createParagraphElement): Use new createDefaultParagraphElement
775         helper so this can share code with the plain-text conversion code.
776         (khtml::createDefaultParagraphElement): Added.
777         (khtml::createBlockPlaceholderElement): Added.
778
779         * khtml/editing/markup.cpp: (khtml::createFragmentFromText): Use paragraphs rather than
780         <br> elements for pasting plain text.
781
782         - regenerated these files with the newer gperf
783
784         * khtml/css/cssproperties.c: Regenerated.
785         * khtml/css/cssvalues.c: Regenerated.
786         * khtml/misc/htmlattrs.c: Regenerated.
787         * khtml/misc/htmltags.c: Regenerated.
788
789 2004-12-16  Ken Kocienda  <kocienda@apple.com>
790
791         Reviewed by Hyatt
792
793         Fix for this bug:
794         
795         <rdar://problem/3924291> REGRESSION (Mail): Crash deleting content following a <p> element
796
797         * khtml/xml/dom_nodeimpl.cpp:
798         (NodeImpl::enclosingInlineElement): Fixed problem in this function where recursive search for
799         parent that is not an inline would skip past previous siblings of nodes which were blocks. Now
800         it stops looking in this situation. This problem eventually caused the delete code to try to 
801         merge a node under a descendent.
802
803 2004-12-16  Adele Amchan  <adele@apple.com>
804
805         Change by Richard, reviewed by me.
806
807         Fix for: <rdar://problem/3923983> background image decoding prevents page with 0Kb image from finishing to load
808
809         * khtml/misc/loader.cpp: (CachedImage::data): Added call to notifyFinished so the 0Kb image case finishes loading.
810
811 2004-12-16  Adele Amchan  <adele@apple.com>
812
813         Reviewed by Chris.
814
815         Fix for: <rdar://problem/3534824> VIP: some navigation links on safeway.com don't work due to a JavaScript quirk
816
817         Added a call to ObjectImp::get before Window::get just returns Undefined.  
818         This was preventing us from getting the toString function from a Window object.
819
820         * khtml/ecma/kjs_window.cpp: (Window::get):
821
822 2004-12-16  Ken Kocienda  <kocienda@apple.com>
823
824         Reviewed by John
825         
826         Fix for this bug:
827         
828         <rdar://problem/3918351> REGRESSION (Mail, 173-175+): Return before first char of line leaves insertion point in wrong place
829         
830         * khtml/editing/htmlediting.cpp:
831         (khtml::InsertParagraphSeparatorCommand::doApply): Basically, did a rewrite of this function
832         to do a better job than it was doing before. Added several test cases to prove I am on a 
833         better track.
834         * khtml/editing/visible_position.cpp:
835         (khtml::isFirstVisiblePositionInBlock): Tweaked the rules a bit to fix an issue very similar to the
836         leaving-the-bar-node case problem I just fixed in a recent checkin (relevant markup: <p>foo</p>bar).
837         This function was returning true for the first position in "bar". Wrong. Also tightened up other 
838         rule: Should not report true when relationship between blocks cannot be determined.
839         (khtml::isLastVisiblePositionInBlock): Tightened up rule as above: Should not report true 
840         when relationship between blocks cannot be determined.
841         * layout-tests/editing/inserting/insert-div-010-expected.txt: Added.
842         * layout-tests/editing/inserting/insert-div-010.html: Added.
843         * layout-tests/editing/inserting/insert-div-011-expected.txt: Added.
844         * layout-tests/editing/inserting/insert-div-011.html: Added.
845         * layout-tests/editing/inserting/insert-div-012-expected.txt: Added.
846         * layout-tests/editing/inserting/insert-div-012.html: Added.
847         * layout-tests/editing/inserting/insert-div-013-expected.txt: Added.
848         * layout-tests/editing/inserting/insert-div-013.html: Added.
849         * layout-tests/editing/inserting/insert-div-014-expected.txt: Added.
850         * layout-tests/editing/inserting/insert-div-014.html: Added.
851         * layout-tests/editing/inserting/insert-div-015-expected.txt: Added.
852         * layout-tests/editing/inserting/insert-div-015.html: Added.
853         * layout-tests/editing/inserting/insert-div-016-expected.txt: Added.
854         * layout-tests/editing/inserting/insert-div-016.html: Added.
855         * layout-tests/editing/inserting/insert-div-017-expected.txt: Added.
856         * layout-tests/editing/inserting/insert-div-017.html: Added.
857             
858 2004-12-16  Ken Kocienda  <kocienda@apple.com>
859
860         Reviewed by me
861         
862         Added a layout test based on my last checkin.
863
864         * layout-tests/editing/selection/move-by-word-001-expected.txt: Added.
865         * layout-tests/editing/selection/move-by-word-001.html: Added.
866
867 2004-12-16  Ken Kocienda  <kocienda@apple.com>
868
869         Reviewed by John
870
871         Fix for this bug:
872         
873         <rdar://problem/3917929> REGRESSION (Mail): Command-left-arrow leaves insertion point too high after specific steps
874
875         * khtml/editing/visible_text.cpp:
876         (khtml::SimplifiedBackwardsTextIterator::advance): Add new check for leaving a text node and iterating
877         backwards into a different block that is an descendent of the block containing the text node (as in leaving 
878         the "bar" node in this example: <p>foo</p>bar).
879         (khtml::SimplifiedBackwardsTextIterator::handleNonTextNode): Call new emitNewlineForBROrText helper.
880         (khtml::SimplifiedBackwardsTextIterator::emitNewlineForBROrText): Factor out code from handleNonTextNode, since
881         it is called from there, and now also from advance().
882
883 2004-12-15  Darin Adler  <darin@apple.com>
884
885         Reviewed by Ken.
886         
887         - fixed problem where plain-text would put a blank line between each <p> even when they have no margins
888
889         * khtml/editing/visible_text.cpp: (khtml::TextIterator::exitNode): Add more checks and only set the
890         "add one more newline" flag if the margin is sufficient. A more complete fix would ignore the node
891         type altogether and use the render tree instead.
892
893 2004-12-14  John Sullivan  <sullivan@apple.com>
894
895         Reviewed by Ken.
896         
897         - rest of WebCore fix for <rdar://problem/3790011> undoable operations all say "Undo" 
898         in the menu, no specific action names
899
900         * khtml/editing/edit_actions.h: new header, contains EditAction enum
901         (renamed from HTMLEditAction, formerly in htmlediting.h)
902
903         * ForwardingHeaders/editing/edit_actions.h: new forwarding header
904         
905         * khtml/editing/htmlediting.h:
906         removed HTMLEditAction enum in favor of including edit_actions.h; added khtml::
907         namespace to EditAction usages; added m_editingAction ivar to ApplyStyleCommand
908
909         * khtml/editing/htmlediting.cpp:
910         (khtml::EditCommandPtr::editingAction):
911         update for enum name change
912         (khtml::EditCommand::editingAction):
913         ditto
914         (khtml::ApplyStyleCommand::ApplyStyleCommand):
915         added editingAction parameter to this constructor
916         (khtml::ApplyStyleCommand::editingAction):
917         return new ivar
918         (khtml::DeleteSelectionCommand::editingAction):
919         update for enum name change
920         (khtml::MoveSelectionCommand::editingAction):
921         ditto
922         (khtml::TypingCommand::editingAction):
923         ditto
924         (khtml::ReplaceSelectionCommand::editingAction):
925         ditto
926         
927         * khtml/khtml_part.h:
928         added EditAction parameter to applyStyle and computeAndSetTypingStyle
929         * khtml/khtml_part.cpp:
930         (KHTMLPart::computeAndSetTypingStyle):
931         added EditAction parameter
932         (KHTMLPart::applyStyle):
933         ditto
934
935         * kwq/KWQKHTMLPart.mm:
936         (KWQKHTMLPart::registerCommandForUndoOrRedo):
937         do the cast from EditAction to WebUndoAction a different way to match other code
938         
939         * kwq/WebCoreBridge.h:
940         * kwq/WebCoreBridge.mm:
941         (-[WebCoreBridge setTypingStyle:withUndoAction:]):
942         added WebUndoAction parameter, passed into ApplyStyleCommand constructor
943         (-[WebCoreBridge applyStyle:withUndoAction:]):
944         ditto
945
946         * WebCore.pbproj/project.pbxproj:
947         updated for new files
948
949 2004-12-14  David Hyatt  <hyatt@apple.com>
950
951         Fix for 3562458, rowspan and colspan converted to ints so that large values will work for them.  Remove
952         the ridiculous 1024 limit on the span values.
953
954         * khtml/html/html_tableimpl.cpp:
955         (HTMLTableCellElementImpl::parseHTMLAttribute):
956         * khtml/rendering/render_table.cpp:
957         (RenderTableCell::collapsedBottomBorder):
958         * khtml/rendering/render_table.h:
959         (khtml::RenderTableCell::colSpan):
960         (khtml::RenderTableCell::setColSpan):
961         (khtml::RenderTableCell::rowSpan):
962         (khtml::RenderTableCell::setRowSpan):
963         (khtml::RenderTableCol::span):
964         (khtml::RenderTableCol::setSpan):
965
966 2004-12-14  David Hyatt  <hyatt@apple.com>
967
968         Make sure <col> and <colgroup> can have spans updated dynamically as well.
969         
970         Reviewed by rjw
971
972         * khtml/html/html_tableimpl.cpp:
973         (HTMLTableColElementImpl::parseHTMLAttribute):
974         * khtml/rendering/render_table.cpp:
975         (RenderTableCell::updateFromElement):
976         (RenderTableCol::updateFromElement):
977
978 2004-12-14  David Hyatt  <hyatt@apple.com>
979
980         Fix for 3833123, setting a cell's colspan does not update rendering like it should.
981         
982         Reviewed by rjw
983
984         * khtml/html/html_tableimpl.cpp:
985         (HTMLTableCellElementImpl::parseHTMLAttribute):
986         * khtml/rendering/render_table.cpp:
987         (RenderTableCell::RenderTableCell):
988         (RenderTableCell::updateFromElement):
989
990 2004-12-14  Chris Blumenberg  <cblu@apple.com>
991
992         Fixed: <rdar://problem/3864536> crash copying text from other apps and pasting in a sticky in dashboard
993
994         Reviewed by hyatt.
995
996         * khtml/editing/markup.cpp:
997         (khtml::createFragmentFromText): ref and deref the fragment since calling appendChild can completely deref it
998
999 2004-12-14  David Hyatt  <hyatt@apple.com>
1000
1001         Fix for 3785211. Make sure to do a layout with the old position before doing a layout after changing to the new position.
1002         This is actually a regression from my positioned object DHTML optimization.
1003         
1004         Reviewed by kocienda
1005
1006         * khtml/rendering/render_object.cpp:
1007         (RenderObject::setStyle):
1008
1009 2004-12-14  David Hyatt  <hyatt@apple.com>
1010
1011         Make sure the class attribute works when newlines are present in the attribute.
1012
1013         Reviewed by kocienda
1014
1015         * khtml/html/html_elementimpl.cpp:
1016         (HTMLNamedAttrMapImpl::parseClassAttribute):
1017
1018 2004-12-14  David Hyatt  <hyatt@apple.com>
1019
1020         Fix for 3724938, float element is duplicated and paints twice.  The logic for when to paint floats was
1021         actually fairly screwed up.  This patch simplifies the logic and makes addOverhangingFloats easier to
1022         read by splitting it into two separate functions.
1023         
1024         Reviewed by kocienda
1025
1026         * khtml/rendering/render_block.cpp:
1027         (khtml::RenderBlock::layoutBlockChildren):
1028         (khtml::RenderBlock::clearFloats):
1029         (khtml::RenderBlock::addOverhangingFloats):
1030         (khtml::RenderBlock::addIntrudingFloats):
1031         * khtml/rendering/render_block.h:
1032
1033 2004-12-14  John Sullivan  <sullivan@apple.com>
1034
1035         Reviewed by Ken.
1036         
1037         - added Undo action names for Cut, Paste, and Drag
1038
1039         * khtml/editing/htmlediting.h:
1040         * khtml/editing/htmlediting.cpp:
1041         (khtml::DeleteSelectionCommand::editingAction):
1042         overridden to return HTMLEditActionCut
1043         (khtml::MoveSelectionCommand::editingAction):
1044         overridden to return HTMLEditActionDrag
1045         (khtml::ReplaceSelectionCommand::editingAction):
1046         overridden to return HTMLEditActionPaste
1047
1048 2004-12-14  John Sullivan  <sullivan@apple.com>
1049
1050         Reviewed by Ken.
1051         
1052         - architecture for WebCore part of fix for <rdar://problem/3790011> undoable operations all say "Undo" in the menu, 
1053         no specific action names
1054           
1055         The remaining step is to make each EditCommand subclass override editingAction() to return an
1056         appropriate value. (Unfortunately the mapping between subclass and user-distinguishable action
1057         is not completely straightforward, so this next step isn't trivial.)
1058
1059         * khtml/editing/htmlediting.h:
1060         new enum for HTMLEditAction
1061
1062         * khtml/editing/htmlediting.cpp:
1063         (khtml::EditCommandPtr::editingAction):
1064         new method, calls through to EditCommand
1065         (khtml::EditCommand::editingAction):
1066         new method for subclasses to override, returns HTMLEditActionUnspecified at this level
1067         (khtml::TypingCommand::editingAction):
1068         proof of concept override, returns HTMLEditActionTyping
1069                 
1070         * kwq/KWQKHTMLPart.h:
1071         declare new private bottleneck method registerCommandForUndoOrRedo
1072         * kwq/KWQKHTMLPart.mm:
1073         (KWQKHTMLPart::registerCommandForUndoOrRedo):
1074         new bottleneck method to reduce code duplication; now calls over the bridge
1075         to get the localized string to use for the Undo action name
1076         (KWQKHTMLPart::registerCommandForUndo):
1077         now calls new bottleneck method
1078         (KWQKHTMLPart::registerCommandForRedo):
1079         now calls new bottleneck method
1080         
1081         * kwq/WebCoreBridge.h:
1082         new enum for WebUndoAction, maps directly to HTMLEditAction.
1083         Declaration of nameForUndoAction:
1084
1085 2004-12-14  Darin Adler  <darin@apple.com>
1086
1087         Reviewed by John.
1088
1089         - added a bunch of missing nil checks; our old version of inherits used to work for nil (by accident)
1090
1091         * khtml/rendering/render_frames.cpp:
1092         (RenderPart::~RenderPart): Check widget for nil.
1093         (RenderPart::setWidget): Ditto.
1094         (RenderFrame::slotViewCleared): Ditto.
1095         (RenderPartObject::slotViewCleared): Ditto.
1096
1097 2004-12-13  Darin Adler  <darin@apple.com>
1098
1099         Reviewed by Ken.
1100
1101         - moved markup-related functions into new sources files in the editing directory
1102         - removed all of the uses of dynamic_cast, preparing to turn off RTTI to make our code smaller and slightly faster
1103
1104         * ForwardingHeaders/editing/markup.h: Added.
1105         * khtml/editing/markup.h: Added.
1106         * khtml/editing/markup.cpp: Added.
1107
1108         * WebCore.pbproj/project.pbxproj: Added markup.h/cpp.
1109
1110         * khtml/dom/dom_node.cpp: (Node::toHTML): Call createMarkup since there's no toHTML in NodeImpl any more.
1111         * khtml/html/html_elementimpl.cpp:
1112         (HTMLElementImpl::innerHTML): Changed to call createMarkup.
1113         (HTMLElementImpl::outerHTML): Ditto.
1114
1115         * khtml/ecma/kjs_window.cpp:
1116         (Window::retrieveWindow): Comment out assert that uses dynamic_cast.
1117         (Window::retrieveActive): Ditto.
1118
1119         * khtml/editing/htmlediting.h: Added forward class declaration needed now that I removed one elsewhere.
1120         * khtml/xml/dom_docimpl.h: Ditto.
1121
1122         * khtml/khtml_part.cpp:
1123         (KHTMLPart::slotDebugDOMTree): Use createMarkup instead of toHTML.
1124         (KHTMLPart::processObjectRequest): Use inherits instead of dynamic_cast.
1125
1126         * khtml/rendering/render_image.cpp: (RenderImage::paint): Add an explicit QChar conversion so this code
1127         still works even with the additional replace overloads added to QString.
1128         * kwq/KWQTextCodec.mm: (QTextCodec::fromUnicode): Ditto.
1129
1130         * khtml/rendering/render_object.h: Removed the version of arenaDelete that does not take an object
1131         base pointer, because it used dynamic_cast in its implementation. Made the other version public.
1132         * khtml/rendering/render_object.cpp: Ditto.
1133         * khtml/rendering/render_replaced.cpp: (RenderWidget::deref): Pass object base pointer to arenaDelete.
1134
1135         * khtml/xml/dom2_rangeimpl.h: Removed extra parameters from toHTML, and unneeded includes and declarations.
1136         * khtml/xml/dom2_rangeimpl.cpp: (DOM::RangeImpl::toHTML): Changed to call createMarkup, and moved all
1137         the support code into markup.cpp.
1138
1139         * khtml/xml/dom_nodeimpl.h: Moved toHTML and related functions into markup.cpp.
1140         * khtml/xml/dom_nodeimpl.cpp: Ditto.
1141
1142         * khtml/xml/dom_position.cpp:
1143         (DOM::startPosition): Implemented the version of this that takes a RangeImpl. Also added null checks
1144         so these return null positions rather than raising exceptions.
1145         (DOM::endPosition): Ditto.
1146
1147         * khtml/khtmlview.h: Added an APPLE_CHANGES function so inherits can detect this class without dynamic_cast.
1148         * kwq/KWQFrame.h: Ditto.
1149         * kwq/KWQFrame.mm: (QFrame::isQFrame): Ditto.
1150         * kwq/KWQKPartsPart.h: Ditto.
1151         * kwq/KWQKPartsPart.mm: (KParts::ReadOnlyPart::isKPartsReadOnlyPart): Ditto.
1152         * kwq/KWQScrollView.h: Ditto.
1153         * kwq/KWQScrollView.mm: (QScrollView::isQScrollView): Ditto.
1154         * kwq/KWQKHTMLPart.h: Ditto.
1155         * kwq/KWQKHTMLPart.mm:
1156         (KHTMLView::isKHTMLView): Ditto.
1157         (KWQKHTMLPart::setTitle): Added an explicit QChar conversion so this code still works even with the additional
1158         replace overloads added to QString.
1159         (KWQKHTMLPart::setStatusBarText): Ditto.
1160         (KWQKHTMLPart::runJavaScriptAlert): Ditto.
1161         (KWQKHTMLPart::runJavaScriptConfirm): Ditto.
1162         (KWQKHTMLPart::runJavaScriptPrompt): Ditto.
1163         (KWQKHTMLPart::attributedString): Ditto.
1164         (KWQKHTMLPart::isCharacterSmartReplaceExempt): Ditto.
1165         (KWQKHTMLPart::isKHTMLPart): That dynamic_cast thing (see above).
1166
1167         * kwq/KWQObject.h: Added virtual methods for the few cases where we need dynamic_cast-like behavior.
1168         * kwq/KWQObject.mm:
1169         (QObject::inherits): Changed to not use dynamic cast.
1170         (QObject::isKHTMLPart): Added. Returns false.
1171         (QObject::isKHTMLView): Ditto.
1172         (QObject::isKPartsReadOnlyPart): Ditto.
1173         (QObject::isQFrame): Ditto.
1174         (QObject::isQScrollView): Ditto.
1175
1176         * kwq/KWQRenderTreeDebug.cpp:
1177         (write): Changed to use inherits rather than dynamic_cast.
1178         (writeSelection): Ditto.
1179
1180         * kwq/KWQSlot.mm: (KWQSlot::call): Call through to the version with just a job pointer parameter rather
1181         than going straight on to the "no parameters at all" version.
1182
1183         * kwq/KWQString.h:
1184         * kwq/KWQString.mm: (QString::replace): Added overloads.
1185         * kwq/WebCoreBridge.mm:
1186         (-[WebCoreBridge markupStringFromNode:nodes:]): Changed to call functions in markup.h.
1187         (-[WebCoreBridge markupStringFromRange:nodes:]): Ditto.
1188         (-[WebCoreBridge selectedString]): Added an explicit QChar conversion so this code still works even with
1189         the additional replace overloads added to QString.
1190         (-[WebCoreBridge stringForRange:]): Ditto.
1191         (-[WebCoreBridge copyDOMNode:copier:]): Changed to call functions in markup.h.
1192         (-[WebCoreBridge elementAtPoint:]): QChar conversion.
1193         (-[WebCoreBridge documentFragmentWithMarkupString:baseURLString:]): Changed to call functions in markup.h.
1194         (-[WebCoreBridge documentFragmentWithText:]): Changed to call functions in markup.h.
1195
1196 2004-12-13  Ken Kocienda  <kocienda@apple.com>
1197
1198         Reviewed by John
1199
1200         Fix for this bug:
1201         
1202         <rdar://problem/3917956> REGRESSION (Mail): pasting can leave insertion point inside pasted text
1203
1204         * khtml/editing/htmlediting.cpp:
1205         (khtml::ReplaceSelectionCommand::doApply): Fix coding mistake. Calculations of bool flag based on 
1206         leading and trailing whitespace positions was reversed! I must have introduced this error recently
1207         when changing around this code.
1208
1209 2004-12-13  David Hyatt  <hyatt@apple.com>
1210
1211         Fix for 3915787, macobserver doesn't paint.  floatRect() needed to be const in the base class.  Also hit-testing
1212         and painting was using the wrong rect when setting up the x/y of the rect.
1213         
1214         Reviewed by mjs
1215
1216         * khtml/rendering/render_block.cpp:
1217         (khtml::RenderBlock::paint):
1218         (khtml::RenderBlock::nodeAtPoint):
1219         * khtml/rendering/render_object.h:
1220         (khtml::RenderObject::floatRect):
1221
1222 2004-12-13  Ken Kocienda  <kocienda@apple.com>
1223
1224         Reviewed by John
1225
1226         Fix for this bug:
1227         
1228         <rdar://problem/3917863> REGRESSION (Mail): pasting two lines of plain text copied from an RTF document results in two styles
1229
1230         Code to figuire out the end node to merge was missing the font tag in the second paragraph
1231         written out by AppKit convert-to-HTML function. I refined the algorithm to be smarter.
1232
1233         * khtml/editing/htmlediting.cpp:
1234         (khtml::ReplacementFragment::mergeEndNode): Refine algorithm used to walk through the fragment being pasted
1235         looking for the node that is the last inline in the last block of the fragment. The old algorithm was 
1236         insufficiently powerful.
1237         (khtml::ReplacementFragment::enclosingBlock): New helper function.
1238         * khtml/editing/htmlediting.h: Add declaration for new helper function.
1239         * layout-tests/editing/pasteboard/paste-text-011-expected.txt: Added.
1240         * layout-tests/editing/pasteboard/paste-text-011.html: Added.
1241
1242 2004-12-13  Ken Kocienda  <kocienda@apple.com>
1243
1244         Reviewed by John
1245
1246         WebCore side of fix for this bug:
1247         
1248         <rdar://problem/3768372> REGRESSION (Mail): paste of text ending in whitespace loses whitespace
1249
1250         Note that we are coordinating with Doug Davidson on the AppKit team to make a complete fix for this
1251         bug. This change involves our half of the needed changes.
1252         
1253         Note that a lot of this change has to do with changing code to use a <br> element instead of
1254         a comment node as the mechanism to annotate HTML with information used to fix the bug. In some
1255         other places, code to handle comments in markup can be removed since we do not use comments for
1256         such annotations after this change.
1257
1258         * khtml/editing/htmlediting.cpp: Remove isComment() helper; no longer needed.
1259         (khtml::ReplacementFragment::ReplacementFragment): Change m_hasInterchangeNewlineComment name to m_hasInterchangeNewline.
1260         (khtml::ReplacementFragment::isInterchangeNewlineNode): Name changed from isInterchangeNewlineComment.
1261         (khtml::ReplacementFragment::isInterchangeConvertedSpaceSpan): Local variable name convertedSpaceSpanClass changed to
1262         convertedSpaceSpanClassString to match other uses of the idiom used here.
1263         (khtml::ReplaceSelectionCommand::doApply): Change hasInterchangeNewlineComment() name to hasInterchangeNewline().
1264         * khtml/editing/htmlediting.h: Change names as noted in .cpp file. Remove isComment() helper; no longer needed.
1265         (khtml::ReplacementFragment::hasInterchangeNewline):  Change hasInterchangeNewlineComment() name to hasInterchangeNewline().
1266         * khtml/html/html_elementimpl.cpp:
1267         (HTMLElementImpl::createContextualFragment): No longer has includeCommentsInDOM flag; no longer needed as we do not
1268         annotate fragments with comments any longer.
1269         * khtml/html/html_elementimpl.h: Ditto.
1270         * khtml/xml/dom2_rangeimpl.cpp: Remove addCommentToHTMLMarkup() helper. No longer needed.
1271         (DOM::interchangeNewlineMarkupString): New helper to return <br> element markup we use to annotate content for interchange.
1272         (DOM::RangeImpl::toHTML): No longer uses addCommentToHTMLMarkup; now calls interchangeNewlineMarkupString(). Remove
1273         spurious semi-colon.
1274         * khtml/xml/dom2_rangeimpl.h: Remove obsolete addCommentToHTMLMarkup() function and EAddToMarkup enum.
1275         * kwq/WebCoreBridge.mm:
1276         (-[WebCoreBridge documentFragmentWithMarkupString:baseURLString:]): No longer pass bool to ask for including comments
1277         in DOM when calling createContextualFragment().
1278
1279 2004-12-10  John Sullivan  <sullivan@apple.com>
1280
1281         fixed deployment build bustage that John Louch ran into
1282
1283         * kwq/KWQTextEdit.mm:
1284         (QTextEdit::setScrollBarModes):
1285         move bool declaration inside exception-handling block to avoid obscure
1286         compiler error
1287
1288 2004-12-10  Maciej Stachowiak  <mjs@apple.com>
1289
1290         Reviewed by Richard.
1291
1292         <rdar://problem/3907484> REGRESSION (125-173): crash when KWQTextField is dealloc'ed while setting focus (profoundlearning.com)
1293         
1294         * kwq/KWQWidget.mm:
1295         (QWidget::setFocus): Handle the case where setting focus removed
1296         us from the superview - this can happen due to style changes on
1297         focus change.
1298
1299 2004-12-10  Ken Kocienda  <kocienda@apple.com>
1300
1301         Reviewed by Hyatt
1302
1303         Fix for this bug:
1304         
1305         <rdar://problem/3915008> REGRESSION (Mail): Too much white space between lines separated by carriage returns
1306
1307         There are a number of interesting things we could do to fix this bug, including SPI and involving
1308         the WebKit delegate, etc., however it seems reasonable to start with a hard-coded default that
1309         will fix the bug in the general case until such time as we can come up with more specific
1310         solutions. 
1311         
1312         So, I added a helper method to create <p> elements with an inline style that sets top and bottom margins
1313         to 0.1em.
1314
1315         * khtml/editing/htmlediting.cpp:
1316         (khtml::InsertParagraphSeparatorCommand::createParagraphElement): New factory method to create
1317         paragraph elements to insert. Also adds style information to keep the <p> from having "too-big" margins.
1318         (khtml::InsertParagraphSeparatorCommand::doApply): Call new factory method.
1319         * khtml/editing/htmlediting.h: Add createParagraphElement() declaration.
1320
1321 2004-12-10  Darin Adler  <darin@apple.com>
1322
1323         Reviewed by Hyatt.
1324
1325         - fixed <rdar://problem/3910419> setting style={overflow:hidden} for <textarea> does not prevent appearance of scrollbars
1326
1327         * khtml/rendering/render_form.h: Remove now-unneeded wrap parameter.
1328         * khtml/rendering/render_form.cpp:
1329         (RenderSubmitButton::rawText): Convert to QChar explicitly.
1330         (RenderLineEdit::updateFromElement): Ditto.
1331         (RenderLineEdit::slotTextChanged): Ditto.
1332         (RenderSelect::updateFromElement): Ditto.
1333         (TextAreaWidget::TextAreaWidget): Moved out most of the initialization since it's not something
1334         that requires a derived class. Now we don't use this class at all for WebCore, but they still
1335         have it for KDE.
1336         (TextAreaWidget::event): Moved out the ifdefs.
1337         (RenderTextArea::RenderTextArea): Moved setting code from TextAreaWidget here. Put a bunch that
1338         we don't need at all inside !APPLE_CHANGES, and removed the setting for scroll bars, since that's
1339         now done in setStyle.
1340         (RenderTextArea::handleFocusOut): Use type QTextEdit instead of TextAreaWidget since that's all
1341         that's needed and WebCore no longer has TextAreaWidget.
1342         (RenderTextArea::calcMinMaxWidth): Ditto.
1343         (RenderTextArea::setStyle): Add code to set scroll bar modes based on wrap setting combined with
1344         overflow style.
1345         (RenderTextArea::updateFromElement): Use type QTextEdit.
1346         (RenderTextArea::text): Ditto.
1347         (RenderTextArea::select): Ditto.
1348
1349         * kwq/KWQTextArea.mm:
1350         (-[KWQTextArea _configureTextViewForWordWrapMode]): Don't set horizontal scroller visibility here,
1351         since it's now handled by QTextEdit.
1352         (-[KWQTextArea initWithFrame:]): Don't set vertical scroller visibility or scroller auto-hiding.
1353
1354         * kwq/KWQTextEdit.h: Add setScrollBarModes function to be used instead of separate setter for
1355         the horizontal and vertical mode; needed because AppKit switches "autohide" for both at once.
1356         * kwq/KWQTextEdit.mm: (QTextEdit::setScrollBarModes): Added.
1357
1358 2004-12-10  Ken Kocienda  <kocienda@apple.com>
1359
1360         Reviewed by Darin
1361
1362         Fix for this bug:
1363         
1364         <rdar://problem/3915047> HItting return in empty document inserts <p> but 
1365         insertion point does not move
1366
1367         * khtml/editing/htmlediting.cpp:
1368         (khtml::InsertParagraphSeparatorCommand::doApply): The issue is that the
1369         code to insert the <p> element for the return is not detecting the fact
1370         that the document is empty. Inserting a <p> into an empty body will not
1371         "add a new line" as the user expects. With this change, we'll add a second 
1372         <p> when the root editable element has no rendered kids.
1373
1374 2004-12-10  Maciej Stachowiak  <mjs@apple.com>
1375
1376         Reviewed by John.
1377
1378         <rdar://problem/3912979> REGRESSION (125-173): repro crash in HTMLCollectionImpl code (www.clubtravel.ie)
1379
1380         * khtml/html/html_miscimpl.cpp:
1381         (HTMLCollectionImpl::traverseNextItem): Pass base when traversing
1382         the initial one step, otherwise we might inadvertantly step
1383         outside the collection base, thereby causing assertion failures or
1384         other badness later.
1385
1386 2004-12-10  Ken Kocienda  <kocienda@apple.com>
1387
1388         Reviewed by Chris
1389
1390         * khtml/editing/htmlediting.cpp:
1391         (khtml::InsertParagraphSeparatorCommand::doApply): There is a starting block which is supposed to 
1392         act as the root node for this operation. However, a loop was incorrectly coded, and a parent node
1393         search could escape this node. Also, one other piece to code to move nodes to the new <p> element
1394         should do nothing if the starting point for the selection is itself the starting block.
1395         (khtml::InsertParagraphSeparatorInQuotedContentCommand::doApply): Similar changes, in spirit, to the above
1396         function, though the names and concepts are slightly different.
1397
1398 2004-12-10  Maciej Stachowiak  <mjs@apple.com>
1399
1400         Reviewed by Ken.
1401
1402         <rdar://problem/3907705> REGRESSION (172-173): DHTML menus are broken at hrweb.apple.com
1403         
1404         * khtml/dom/html_document.cpp:
1405         (HTMLDocument::nameableItems): New method, wrapper for HTMLCollection creation.
1406         * khtml/dom/html_document.h:
1407         * khtml/ecma/kjs_html.cpp:
1408         (KJS::HTMLDocument::tryGet): use doc.nameableItems(), not doc.all()!
1409         * khtml/html/html_miscimpl.cpp:
1410         (HTMLCollectionImpl::traverseNextItem): Added new DOC_NAMEABLE_ITEMS type, this represents
1411         the items that can be accessed directly as a document propery, in particular forms, images,
1412         objects, applets and embeds.
1413         (HTMLCollectionImpl::updateNameCache): Fix some nameCache/idCache confusion.
1414         (HTMLFormCollectionImpl::updateNameCache): Ditto.
1415         * khtml/html/html_miscimpl.h:
1416         (DOM::HTMLCollectionImpl::): Added new type.
1417
1418 2004-12-10  Ken Kocienda  <kocienda@apple.com>
1419
1420         Reviewed by John
1421
1422         Fix for this bug:
1423         
1424         <rdar://problem/3914779> REGRESSION (Mail): Cannot arrow navigate to position before last character on text-wrapped line
1425
1426         * khtml/rendering/render_text.cpp:
1427         (RenderText::caretRect): Code was not detecting space at the end of a line correctly. Now it does.
1428
1429 2004-12-10  Ken Kocienda  <kocienda@apple.com>
1430
1431         Reviewed by John
1432
1433         Fix for this bug:
1434
1435         <rdar://problem/3914755> REGRESSION (Mail): Insertion point disappears after pasting paragraph
1436
1437         * khtml/editing/htmlediting.cpp:
1438         (khtml::ReplaceSelectionCommand::doApply): Selection could end up in a "placeholder" node
1439         that was removed from the document when pasting. This caused the disappearance. Now this
1440         is detected, and the selection is shifted to a node that is in the document.
1441
1442 2004-12-09  Richard Williamson   <rjw@apple.com>
1443
1444         Check to disable threaded decoding during
1445         layout tests wasn't invoking function, just checking address of 
1446         function, which would always return true.
1447         
1448         * kwq/WebCoreImageRendererFactory.m:
1449         (+[WebCoreImageRendererFactory shouldUseThreadedDecoding]):
1450
1451 2004-12-09  David Hyatt  <hyatt@apple.com>
1452
1453         Fix for 3892686, left/top overflow was not being propagated properly up to containing blocks because of a math
1454         error.
1455
1456         Also fix a bug I noticed on the same page where relative position offsets were not being added in properly for
1457         all inlines when repainting.
1458         
1459         Reviewed by rjw
1460
1461         * khtml/rendering/render_block.cpp:
1462         (khtml::RenderBlock::layoutBlockChildren):
1463         * khtml/rendering/render_flow.cpp:
1464         (RenderFlow::getAbsoluteRepaintRect):
1465
1466 2004-12-09  David Hyatt  <hyatt@apple.com>
1467
1468         Fix for 3867545, finance.yahoo.com lays out incorrectly.  Add a quirk that will prevent tables from moving
1469         down below floats when there is insufficient space.  Instead we will match Gecko and just spill out of the
1470         containing block to the right.  This appears to be the more common desired behavior, despite being wrong.
1471         WinIE sometimes wraps and sometimes doesn't, but it's really hard for us to match its inconsistency.
1472         
1473         Reviewed by john
1474
1475         * khtml/rendering/render_block.cpp:
1476         (khtml::RenderBlock::getClearDelta):
1477         * layout-tests/apple-only/base/www.cnn.com/index-expected.txt:
1478         * layout-tests/apple-only/base/www.ebay.com/index-expected.txt:
1479         * layout-tests/apple-only/base/www.excite.com/index-expected.txt:
1480         * layout-tests/fast/block/margin-collapse/102-expected.txt:
1481         * layout-tests/fast/block/margin-collapse/102.html:
1482
1483 2004-12-09  Richard Williamson   <rjw@apple.com>
1484
1485         Fixed <rdar://problem/3914078> worldclock crashing gc related
1486
1487         Use ProtectedValue for Context2D instance members.
1488
1489         Reviewed by Maciej.
1490
1491         * khtml/ecma/kjs_html.h:
1492
1493 2004-12-09  John Sullivan  <sullivan@apple.com>
1494
1495         Reviewed by Dave.
1496         
1497         - fixed <rdar://problem/3731099> Move AXTitle string for image elements to AXDescription
1498
1499         * kwq/KWQAccObject.mm:
1500         (-[KWQAccObject title]): moved image alt tag code out of here
1501         (-[KWQAccObject accessibilityDescription]): moved image alt tag code into this new method
1502         (-[KWQAccObject accessibilityAttributeNames]): include AXDescription in the set of attributes
1503         that ordinary elements return; this means that ordinary elements that aren't images will return
1504         a nil description, which isn't ideal, but is in keeping with the way the rest of these attributes
1505         work here.
1506         (-[KWQAccObject accessibilityAttributeValue:]):
1507         call accessibilityDescription when asked for AXDescription
1508
1509 2004-12-09  Ken Kocienda  <kocienda@apple.com>
1510
1511         Reviewed by Harrison
1512
1513         Fix for this bug:
1514         
1515         <rdar://problem/3910425> REGRESSION (Mail): Crash in ReplaceSelectionCommand; selection is empty, leading to null deref
1516
1517         * khtml/editing/htmlediting.cpp:
1518         (khtml::MoveSelectionCommand::doApply): The node representing the destination for the move may have
1519         been deleted. If this is the case, set the destination to the node the delete command provides in
1520         its ending selection.
1521
1522 === Safari-175 ===
1523
1524 2004-12-09  Ken Kocienda  <kocienda@apple.com>
1525
1526         Reviewed by John
1527         
1528         Fix for this bug:
1529         
1530         <rdar://problem/3912841> REGRESSION (173-TOT): Some images report 0x0 dimensions on layout tests, causes spurious test failures
1531
1532         The new threaded image decoding capability can throw off layout tests. The issue is that the decoding
1533         callback may not be delivered before the program asks for the dimensions of an image in order to
1534         wrote the layout dimensions. More generally, I think we need to ensure that there are no races in
1535         layout tests, so I have added a flag to the render tree debug code that we can set when debugging.
1536
1537         * kwq/KWQRenderTreeDebug.cpp:
1538         (debuggingRenderTree): New function. Returns flag which tells whether the program is debugging the render tree.
1539         (externalRepresentation): Sets debuggingRenderTree flag to true;
1540         * kwq/KWQRenderTreeDebug.h:
1541         * kwq/WebCoreImageRendererFactory.m:
1542         (+[WebCoreImageRendererFactory shouldUseThreadedDecoding]): Checks debuggingRenderTree flag and will not
1543         do threaded decoding in any case if the flag is set.
1544
1545 2004-12-09  Chris Blumenberg  <cblu@apple.com>
1546
1547         Fix for busting XMLHTTPRequest.
1548
1549         Reviewed by kocienda.
1550
1551         * khtml/misc/loader.cpp:
1552         (Loader::servePendingRequests): pass true for deliverAllData
1553         * kwq/KWQKJob.h:
1554         (KIO::get): take deliverAllData param
1555         (KIO::http_post): ditto
1556         * kwq/KWQKJobClasses.h:
1557         * kwq/KWQKJobClasses.mm:
1558         (KIO::TransferJob::TransferJob): if deliverAllData, create signal with data param
1559         (KIO::TransferJob::emitResult): if deliverAllData, call signal with data param
1560
1561 2004-12-09  Ken Kocienda  <kocienda@apple.com>
1562
1563         Reviewed by John
1564
1565         <rdar://problem/3911011> REGRESSION (Mail): Spaces at end of line causing word wrap lost when copied/pasted
1566
1567         * khtml/xml/dom_nodeimpl.cpp:
1568         (NodeImpl::renderedText): Fixed the code so that spaces at the end of lines are not skipped.
1569
1570 2004-12-07  Richard Williamson   <rjw@apple.com>
1571
1572         Support threaded image decoding on machines w/ > 2 CPUs.
1573
1574         Reviewed by Maciej and Chris.
1575
1576         * khtml/misc/loader.cpp:
1577         (CachedImageCallback::notifyUpdate):
1578         (CachedImageCallback::notifyFinished):
1579         (CachedImageCallback::notifyDecodingError):
1580         (CachedImageCallback::handleError):
1581         (CachedImageCallback::clear):
1582         (CachedImage::CachedImage):
1583         (CachedImage::clear):
1584         (CachedImage::data):
1585         (CachedImage::checkNotify):
1586         (Loader::servePendingRequests):
1587         (Loader::slotFinished):
1588         (Loader::numRequests):
1589         (Loader::cancelRequests):
1590         (Loader::removeBackgroundDecodingRequest):
1591         * khtml/misc/loader.h:
1592         (khtml::CachedImageCallback::CachedImageCallback):
1593         (khtml::CachedImageCallback::ref):
1594         (khtml::CachedImageCallback::deref):
1595         (khtml::CachedImage::decoderCallback):
1596         * khtml/rendering/render_object.cpp:
1597         (RenderObject::setPixmap):
1598         * kwq/KWQPixmap.h:
1599         * kwq/KWQPixmap.mm:
1600         (-[WebImageCallback initWithCallback:khtml::]):
1601         (-[WebImageCallback _commonTermination]):
1602         (-[WebImageCallback dealloc]):
1603         (-[WebImageCallback finalize]):
1604         (-[WebImageCallback notify]):
1605         (-[WebImageCallback setImageSourceStatus:]):
1606         (-[WebImageCallback status]):
1607         (QPixmap::shouldUseThreadedDecoding):
1608         (QPixmap::receivedData):
1609         * kwq/WebCoreImageRenderer.h:
1610         * kwq/WebCoreImageRendererFactory.h:
1611         * kwq/WebCoreImageRendererFactory.m:
1612         (+[WebCoreImageRendererFactory shouldUseThreadedDecoding]):
1613         (+[WebCoreImageRendererFactory setShouldUseThreadedDecoding:]):
1614
1615 2004-12-07  Ken Kocienda  <kocienda@apple.com>
1616
1617         Reviewed by John
1618
1619         * khtml/editing/htmlediting.cpp:
1620         (khtml::InsertParagraphSeparatorInQuotedContentCommand::doApply): Fix a problem with my
1621         change in thei code yesterday. Call to insertBlockPlaceholderIfNeeded() must be done
1622         after new block has been inserted, otherwise a crash can result. Shuffle down call 
1623         to insertBlockPlaceholderIfNeeded() a couple of lines (where the node is inserted), 
1624         and all is good.
1625
1626 2004-12-07  Ken Kocienda  <kocienda@apple.com>
1627
1628         Reviewed by John
1629
1630         Fix for this bug:
1631         
1632         <rdar://problem/3907422> REGRESSION (Mail): Pasting quoted content can place content after body element
1633
1634         * khtml/editing/htmlediting.cpp:
1635         (khtml::ReplaceSelectionCommand::doApply): Detect when the body element is the "reference block" used
1636         for determining the location for inserting content. Do not allow an insert before or after if the
1637         reference block is the body. Perform insertNodeAt(0) and appendNode, respectively, in the block-is-body case.
1638         * layout-tests/editing/inserting/insert-3907422-fix-expected.txt: Added.
1639         * layout-tests/editing/inserting/insert-3907422-fix.html: Added.
1640
1641 2004-12-07  Darin Adler  <darin@apple.com>
1642
1643         Reviewed by Don.
1644
1645         - fixed <rdar://problem/3908701> REGRESSION: Cursor does not change to "hand" over active links
1646
1647         * kwq/KWQEvent.mm: (positionForEvent): Add NSMouseMoved to list of events that have mouse location.
1648
1649 2004-12-07  Ken Kocienda  <kocienda@apple.com>
1650
1651         Reviewed by John
1652
1653         Changed name of constant from KHTMLInterchangeNewline to AppleInterchangeNewline.
1654         I discussed this with Hyatt and he agreed that going with "Apple" names was OK.
1655
1656         * khtml/editing/html_interchange.h: Name change, as described above.
1657         * khtml/editing/htmlediting.cpp:
1658         (khtml::ReplacementFragment::isInterchangeNewlineComment): Ditto.
1659         * khtml/xml/dom2_rangeimpl.cpp:
1660         (DOM::RangeImpl::toHTML): Ditto.
1661
1662 2004-12-07  Ken Kocienda  <kocienda@apple.com>
1663
1664         Reviewed by me
1665         
1666         Added a couple more layout tests.
1667
1668         * layout-tests/editing/deleting/delete-line-013-expected.txt: Added.
1669         * layout-tests/editing/deleting/delete-line-013.html: Added.
1670         * layout-tests/editing/deleting/delete-line-014-expected.txt: Added.
1671         * layout-tests/editing/deleting/delete-line-014.html: Added.
1672
1673 2004-12-06  Maciej Stachowiak  <mjs@apple.com>
1674
1675         Reviewed by Hyatt.
1676
1677         - fixed <rdar://problem/3906974> assertion failure in QWidget::beforeMouseDown clicking on <select multiple>
1678         
1679         * kwq/KWQListBox.mm:
1680         (QListBox::QListBox): Initialize KWQListBoxScrollView with this.
1681         (-[KWQListBoxScrollView initWithListBox:]): Make this class a KWQWidgetHolder.
1682         (-[KWQListBoxScrollView widget]): See above.
1683         (-[KWQTableView mouseDown:]): Pass outerView rather than self to beforeMouseDown and
1684         afterMouseDown, to avoid triggering an assertion failure.
1685
1686 2004-12-06  David Hyatt  <hyatt@apple.com>
1687
1688         Fix for 3615411, the linesAppended optimization was old and broken, and it's easier with the new code fixes
1689         made by me, kocienda and harrison to just remove it.
1690         
1691         Reviewed by mjs
1692
1693         * khtml/rendering/bidi.cpp:
1694         (khtml::RenderBlock::layoutInlineChildren):
1695         * khtml/rendering/render_block.cpp:
1696         (khtml:::RenderFlow):
1697         * khtml/rendering/render_block.h:
1698         * khtml/rendering/render_flow.cpp:
1699         (RenderFlow::dirtyLinesFromChangedChild):
1700
1701 2004-12-06  David Hyatt  <hyatt@apple.com>
1702
1703         Fix for 3787133, some web pages print with many blank pages.  Make sure to use the real page print rect and
1704         not a damage rect that can be changed when intersected with the clip regions of the web page.
1705         
1706         Reviewed by john
1707
1708         * khtml/rendering/render_canvas.h:
1709         (khtml::RenderCanvas::printRect):
1710         (khtml::RenderCanvas::setPrintRect):
1711         * khtml/rendering/render_flow.cpp:
1712         (RenderFlow::paintLines):
1713         * khtml/rendering/render_list.cpp:
1714         (RenderListMarker::paint):
1715         * kwq/KWQKHTMLPart.mm:
1716         (KWQKHTMLPart::adjustPageHeight):
1717
1718 2004-12-06  David Harrison  <harrison@apple.com>
1719
1720         Reviewed by Ken Kocienda and Dave Hyatt (OOPS!).
1721
1722                 <rdar://problem/3849947> Typing after pasting line does not appear until after window resize.
1723                 
1724                 
1725         * khtml/rendering/render_flow.cpp:
1726         (RenderFlow::dirtyLinesFromChangedChild):
1727                 Dirty the line above because new child can inval the cached line break position of previous line.
1728
1729 2004-12-06  David Hyatt  <hyatt@apple.com>
1730
1731         Fix for 3254464, radio buttons do not work for quiz on netscape.com.  Left/top overflow needed to be implemented.
1732         This also fixes 3106907, link hover color only partially set on rollover and the more general architecture bug
1733         3126929, handle top/left overflow.
1734
1735         This patch also fixes 3902891, scroll bar of position:fixed content moves when a page is scrolled.
1736
1737         Finally, the Emerson regression 3869718 (error involving computing the rightmost/lowest position of overflow:auto
1738         regions and web pages) has been fixed.
1739         
1740         Reviewed by mjs
1741
1742         * khtml/rendering/bidi.cpp:
1743         (khtml::RenderBlock::computeHorizontalPositionsForLine):
1744         (khtml::RenderBlock::checkLinesForOverflow):
1745         * khtml/rendering/render_block.cpp:
1746         (khtml:::RenderFlow):
1747         (khtml::RenderBlock::overflowHeight):
1748         (khtml::RenderBlock::overflowWidth):
1749         (khtml::RenderBlock::overflowLeft):
1750         (khtml::RenderBlock::overflowTop):
1751         (khtml::RenderBlock::overflowRect):
1752         (khtml::RenderBlock::layoutBlock):
1753         (khtml::RenderBlock::layoutBlockChildren):
1754         (khtml::RenderBlock::paint):
1755         (khtml::RenderBlock::floatRect):
1756         (khtml::RenderBlock::lowestPosition):
1757         (khtml::RenderBlock::rightmostPosition):
1758         (khtml::RenderBlock::leftmostPosition):
1759         (khtml::RenderBlock::nodeAtPoint):
1760         * khtml/rendering/render_block.h:
1761         * khtml/rendering/render_box.h:
1762         (khtml::RenderBox::borderBox):
1763         (khtml::RenderBox::borderTopExtra):
1764         (khtml::RenderBox::borderBottomExtra):
1765         * khtml/rendering/render_layer.cpp:
1766         (RenderLayer::paintScrollbars):
1767         (mustExamineRenderer):
1768         (RenderLayer::intersectsDamageRect):
1769         (RenderLayer::containsPoint):
1770         * khtml/rendering/render_line.cpp:
1771         (khtml::InlineFlowBox::placeBoxesHorizontally):
1772         (khtml::InlineFlowBox::verticallyAlignBoxes):
1773         * khtml/rendering/render_line.h:
1774         (khtml::InlineBox::leftOverflow):
1775         (khtml::InlineBox::rightOverflow):
1776         (khtml::InlineFlowBox::setVerticalOverflowPositions):
1777         (khtml::RootInlineBox::RootInlineBox):
1778         (khtml::RootInlineBox::leftOverflow):
1779         (khtml::RootInlineBox::rightOverflow):
1780         (khtml::RootInlineBox::setVerticalOverflowPositions):
1781         (khtml::RootInlineBox::setHorizontalOverflowPositions):
1782         * khtml/rendering/render_object.h:
1783         (khtml::RenderObject::borderBox):
1784         (khtml::RenderObject::overflowLeft):
1785         (khtml::RenderObject::overflowTop):
1786         (khtml::RenderObject::overflowRect):
1787         (khtml::RenderObject::floatRect):
1788         * khtml/rendering/render_table.cpp:
1789         (RenderTable::layout):
1790         (RenderTable::paint):
1791         (RenderTable::paintBoxDecorations):
1792         (RenderTable::calcMinMaxWidth):
1793         * khtml/rendering/render_table.h:
1794         (khtml::RenderTableCell::borderTopExtra):
1795         (khtml::RenderTableCell::borderBottomExtra):
1796         * kwq/KWQRect.h:
1797         * kwq/KWQRect.mm:
1798         (QRect::inflate):
1799
1800 2004-12-06  Maciej Stachowiak  <mjs@apple.com>
1801
1802         Reviewed by John.
1803
1804         - fixed <rdar://problem/3903797> scripts can cause other frames/windows to execute arbitrary script using javascript: URLs
1805         
1806         I changed all unprotected places that can navigate a different
1807         window or frame from script to check for a javascript: URL, and if
1808         found, to check for safety using cross-site-script rules. 
1809
1810         I considered a few other possible exploits and made no change:
1811
1812         - document.location is already protected because the document
1813         object itself is protected
1814
1815         - frame.src, frame.location, iframe.src and targetted links are
1816         all safe because setting the URL of a frame to a javascript: URL
1817         executes the script in the context of the parent
1818
1819         * khtml/ecma/kjs_window.cpp:
1820         (WindowFunc::tryCall):
1821         (Location::put):
1822         (LocationFunc::tryCall):
1823
1824 2004-12-06  Ken Kocienda  <kocienda@apple.com>
1825
1826         Reviewed by Maciej
1827
1828         Fix for this bug:
1829         
1830         <rdar://problem/3890955> 8A314: Forward delete sometimes fails to delete the selected quoted text
1831
1832         * khtml/editing/htmlediting.cpp:
1833         (khtml::DeleteSelectionCommand::handleSpecialCaseBRDelete): Fixed bonehead coding mistake in the
1834         check for one of the special cases being checked for in this function. The specific case 
1835         intends to check for a selection that is only a <br> after a block ends (as in </div><br>). If it
1836         sees such markup, it deletes only the <br> and bails. However, this code would run in *any*
1837         case where a selection ended in a <br> after a block and would not delete any part of the
1838         selection preceding the <br>. Bad. I have tightened the check to see that only a <br> is
1839         selected.
1840
1841         Fixing the bug above was accomplished with an additional call to DOM::Position::downstream. This
1842         new use of the function exposed this bug:
1843         
1844         <rdar://problem/3907666> Incorrectly coded loop in Position::downstream can lead to infinite loop
1845
1846         * khtml/xml/dom_position.cpp:
1847         (DOM::Position::downstream): I am ashamed of my first cut at this. Rewrote the loop so it does 
1848         not have this fatal flaw. It is a much better design as well.
1849         
1850         * layout-tests/editing/deleting/delete-3800834-fix-expected.txt: Changes made this test
1851         have what I consider to be a better result. Going with it.
1852
1853 2004-12-06  Chris Blumenberg  <cblu@apple.com>
1854
1855         Fixed: <rdar://problem/3871718> REGRESSION (125-168): text marked bold with font that does not have bold variant copies as non-bold
1856
1857         Reviewed by hyatt.
1858
1859         * kwq/DOM.mm:
1860         (-[DOMElement _font]): new SPI for AppKit
1861         * kwq/DOMPrivate.h:
1862
1863 2004-12-06  Darin Adler  <darin@apple.com>
1864
1865         Reviewed by Maciej.
1866
1867         - fixed <rdar://problem/3906327> Select All of a large document is slow (>15 secs on my machine for attached specimen)
1868
1869         * kwq/KWQScrollView.mm: (QScrollView::updateContents): Intersect with visibleRect before calling through
1870         to NSView to dirty; NSView could also be more efficient in this case (I filed 3906343).
1871
1872 2004-12-06  John Sullivan  <sullivan@apple.com>
1873
1874         Darin found what appears to be the real leak that we were falsely blaming
1875         on the 'leaks' tool (3880245). I made the change, and ran layout tests and PLT to make
1876         sure nothing barfed.
1877
1878         * khtml/css/cssparser.cpp:
1879         (CSSParser::parseValue):
1880         call clearProperties() instead of just setting numParsedProperties to 0
1881         (CSSParser::parseDeclaration):
1882         ditto
1883         (CSSParser::createStyleDeclaration):
1884         ditto
1885
1886 2004-12-06  Ken Kocienda  <kocienda@apple.com>
1887
1888         Reviewed by me
1889         
1890         New layout tests.
1891
1892         * layout-tests/editing/inserting/insert-div-001-expected.txt: Added.
1893         * layout-tests/editing/inserting/insert-div-001.html: Added.
1894         * layout-tests/editing/inserting/insert-div-002-expected.txt: Added.
1895         * layout-tests/editing/inserting/insert-div-002.html: Added.
1896         * layout-tests/editing/inserting/insert-div-003-expected.txt: Added.
1897         * layout-tests/editing/inserting/insert-div-003.html: Added.
1898         * layout-tests/editing/inserting/insert-div-004-expected.txt: Added.
1899         * layout-tests/editing/inserting/insert-div-004.html: Added.
1900         * layout-tests/editing/inserting/insert-div-005-expected.txt: Added.
1901         * layout-tests/editing/inserting/insert-div-005.html: Added.
1902         * layout-tests/editing/inserting/insert-div-006-expected.txt: Added.
1903         * layout-tests/editing/inserting/insert-div-006.html: Added.
1904         * layout-tests/editing/inserting/insert-div-007-expected.txt: Added.
1905         * layout-tests/editing/inserting/insert-div-007.html: Added.
1906         * layout-tests/editing/inserting/insert-div-008-expected.txt: Added.
1907         * layout-tests/editing/inserting/insert-div-008.html: Added.
1908         * layout-tests/editing/inserting/insert-div-009-expected.txt: Added.
1909         * layout-tests/editing/inserting/insert-div-009.html: Added.
1910
1911 2004-12-06  Ken Kocienda  <kocienda@apple.com>
1912
1913         Reviewed by Harrison
1914
1915         Fix for this bug:
1916         
1917         <rdar://problem/3906948> REGRESSION (Mail): Insert paragraph code can make the insertion point "stick" in place.
1918
1919         * khtml/editing/htmlediting.cpp:
1920         (khtml::InsertParagraphSeparatorCommand::doApply): Call insertBlockPlaceholderIfNeeded(), passing block
1921         being added to this function. This ensures that the added block has a height.
1922         (khtml::InsertParagraphSeparatorInQuotedContentCommand::doApply): Ditto.
1923
1924 2004-12-06  Ken Kocienda  <kocienda@apple.com>
1925  
1926         Reviewed by John
1927         
1928         * khtml/dom/dom_string.cpp:
1929         (DOM::DOMString::substring): Expose method already on DOMStrimgImpl.
1930         * khtml/dom/dom_string.h: Ditto.
1931         * khtml/editing/htmlediting.cpp:
1932         (khtml::CompositeEditCommand::rebalanceWhitespace): New helper to create and execute a
1933         RebalanceWhitespaceCommand instance.
1934         (khtml::DeleteSelectionCommand::doApply): Call rebalanceWhitespace() after running command.
1935         (khtml::InsertLineBreakCommand::doApply): Ditto.
1936         (khtml::InsertParagraphSeparatorCommand::doApply): Ditto.
1937         (khtml::InsertParagraphSeparatorInQuotedContentCommand::doApply): Ditto.
1938         (khtml::InsertTextCommand::input): Ditto.
1939         (khtml::RebalanceWhitespaceCommand::RebalanceWhitespaceCommand): New command.
1940         (khtml::RebalanceWhitespaceCommand::~RebalanceWhitespaceCommand): Ditto.
1941         (khtml::RebalanceWhitespaceCommand::doApply): Ditto.
1942         (khtml::RebalanceWhitespaceCommand::doUnapply): Ditto.
1943         (khtml::RebalanceWhitespaceCommand::preservesTypingStyle): Ditto.
1944         (khtml::ReplaceSelectionCommand::completeHTMLReplacement): Ditto.
1945         * khtml/editing/htmlediting.h: Ditto.
1946         (khtml::RebalanceWhitespaceCommand::): Ditto.
1947
1948 2004-12-05  Darin Adler  <darin@apple.com>
1949
1950         - fixed small problem in my check-in from yesterday
1951
1952         * kwq/KWQEvent.mm:
1953         (positionForEvent): Get location from event without raising exception if it's the wrong type.
1954         (clickCountForEvent): Same, for clickCount.
1955         (QMouseEvent::QMouseEvent): Use the new helper functions so this can be constructed even with
1956         the wrong type of NSEvent. Required for cases where a keyboard event causes a "click" and we need
1957         to synthesize a QMouseEvent for KHTML internal use, using the key down NSEvent.
1958
1959 2004-12-04  Darin Adler  <darin@apple.com>
1960
1961         Reviewed by John.
1962
1963         - fixed <rdar://problem/3878329> REGRESSION (169-170): colors are wrong for my.yahoo.com due to CSS background parsing changes
1964
1965         * khtml/css/cssparser.cpp: (CSSParser::parseBackgroundShorthand): Changed function so it doesn't rely on the position
1966         attribute being at the end of the array and then moved position attribute before color attribute so it takes precedence.
1967         Since "0" can be both the X coordinate of a position and a legal color (meaning black), we need to do position first.
1968
1969         - fixed <rdar://problem/3760869> click events for input type=button or type=checkbox don't have flags like shiftKey set
1970
1971         * khtml/rendering/render_form.h: Remove unused RenderFormElement fields.
1972         * khtml/rendering/render_form.cpp:
1973         (RenderFormElement::RenderFormElement): Take out code to set a bunch of unused fields.
1974         (RenderFormElement::slotClicked): Change to create the QMouseEvent from the actual mouse event rather than
1975         creating it with all the flags set to 0, using the new QMouseEvent constructor that does so.
1976
1977         * kwq/KWQEvent.h: Added constructor that takes no parameters which uses the current event from AppKit.
1978         Made the click count getter const and added an isDoubleClick that matches the logic used elsewhere.
1979         Added a fixState helper method so the constructors can save code.
1980         * kwq/KWQEvent.mm:
1981         (QMouseEvent::QMouseEvent): Factored out the state-fixing code that was in the two existing constructors
1982         and added a third constructor that uses the "current event" from AppKit (used above).
1983         (QMouseEvent::fixState): Compute state and click count based on event type.
1984
1985         - fixed first symptom of <rdar://problem/3830936> REGRESSION (125-165): crash due to null font family, hang at changeforamerica.com
1986
1987         * kwq/KWQFontFamily.mm: (KWQFontFamily::getNSFamily): Handle empty strings specially so we don't run into trouble when
1988         the family name is a null string. This prevents the crash, but there are still other problems that may have the same
1989         underlying cause in CSS.
1990
1991         - fixed <rdar://problem/3829808> Safari crashes when adding a DOM node that was removed from an XMLHTTP request result
1992
1993         * khtml/html/html_baseimpl.cpp: (HTMLBodyElementImpl::insertedIntoDocument):
1994         Added nil check before calling scheduleRelayout. This is new code so the nil-dereference is a recent regression.
1995
1996 2004-12-03  Chris Blumenberg  <cblu@apple.com>
1997
1998         New fixes for:
1999         <rdar://problem/3685766> WebDataSource is missing subresources when they use cached WebCore data
2000         <rdar://problem/3722434> REGRESSION?: Assertion failure trying to drag image in iframe (itapema.sc.gov.br)
2001         <rdar://problem/3903173> REGRESSION (172-TOT): assertion failure and crash in slotAllData logging into hotmail account
2002         <rdar://problem/3902749> REGRESSION (Tiger): missing image symbol does not appear
2003
2004         Reviewed by darin, rjw, kocienda.
2005
2006         * khtml/misc/loader.cpp:
2007         (CachedObject::~CachedObject):
2008         (CachedCSSStyleSheet::checkNotify):
2009         (Loader::servePendingRequests):
2010         (Loader::slotFinished):
2011         (Loader::slotReceivedResponse):
2012         (Cache::requestImage):
2013         (Cache::requestScript):
2014         * khtml/misc/loader.h:
2015         (khtml::CachedObject::CachedObject):
2016         (khtml::CachedObject::response):
2017         (khtml::CachedObject::allData):
2018         * kwq/KWQKJobClasses.h:
2019         * kwq/KWQKJobClasses.mm:
2020         (KIO::TransferJobPrivate::TransferJobPrivate):
2021         (KIO::TransferJobPrivate::~TransferJobPrivate):
2022         (KIO::TransferJob::TransferJob):
2023         (KIO::TransferJob::assembleResponseHeaders):
2024         (KIO::TransferJob::retrieveCharset):
2025         (KIO::TransferJob::emitResult):
2026         (KIO::TransferJob::emitReceivedResponse):
2027         * kwq/KWQLoader.h:
2028         * kwq/KWQLoader.mm:
2029         (KWQHeaderStringFromDictionary):
2030         (KWQCheckCacheObjectStatus):
2031         (KWQIsResponseURLEqualToURL):
2032         (KWQResponseURL):
2033         (KWQResponseMIMEType):
2034         (KWQCacheObjectExpiresTime):
2035         (khtml::CachedObject::setResponse):
2036         (khtml::CachedObject::setAllData):
2037         * kwq/KWQPixmap.h:
2038         * kwq/KWQPixmap.mm:
2039         (QPixmap::QPixmap):
2040         * kwq/KWQResourceLoader.mm:
2041         (-[KWQResourceLoader finishJobAndHandle:]):
2042         (-[KWQResourceLoader cancel]):
2043         (-[KWQResourceLoader reportError]):
2044         (-[KWQResourceLoader finishWithData:]):
2045         * kwq/KWQSignal.h:
2046         * kwq/KWQSignal.mm:
2047         (KWQSignal::call):
2048         * kwq/KWQSlot.h:
2049         * kwq/KWQSlot.mm:
2050         (KWQSlot::KWQSlot):
2051         (KWQSlot::call):
2052         * kwq/WebCoreBridge.h:
2053         * kwq/WebCoreResourceLoader.h:
2054
2055 2004-12-04  Darin Adler  <darin@apple.com>
2056
2057         Reviewed by John.
2058
2059         - fixed <rdar://problem/3876093> REGRESSION (166-167): Setting slider control's value from JavaScript has no effect (breaks RSS)
2060
2061         * khtml/rendering/render_form.cpp:
2062         (RenderSlider::updateFromElement): Call setValue to update the value of the DOM element rather than
2063         modifying the m_value data member directly. We don't use m_value at all for sliders now, and in fact
2064         the code relies on the fact that m_value is null. Setting m_value to a non-null value was causing the bug.
2065         (RenderSlider::slotSliderValueChanged): Ditto.
2066
2067 2004-12-03  John Sullivan  <sullivan@apple.com>
2068
2069         Reviewed by Ken.
2070         
2071         - fixed <rdar://problem/3889411> REGRESSION (125-172): repro crash in 
2072         khtml::BackgroundLayer::cullEmptyLayers
2073
2074         * khtml/rendering/render_style.cpp:
2075         (BackgroundLayer::cullEmptyLayers):
2076         added missing nil check
2077
2078 === Safari-173 ===
2079
2080 2004-12-03  Ken Kocienda  <kocienda@apple.com>
2081
2082         Reviewed by me
2083
2084         Roll out some recent changes by Chris that caused a performance regression.
2085         Fix is in hand, but it is a little risky this close to a submission. So,
2086         we have decided to roll back the change with the regression and roll in
2087         the new code after we submit.
2088
2089         * khtml/css/cssproperties.c:
2090         (hash_prop):
2091         (findProp):
2092         * khtml/css/cssvalues.c:
2093         (hash_val):
2094         (findValue):
2095         * khtml/misc/htmlattrs.c:
2096         (hash_attr):
2097         (findAttr):
2098         * khtml/misc/htmltags.c:
2099         (hash_tag):
2100         (findTag):
2101         * khtml/misc/loader.cpp:
2102         (CachedObject::~CachedObject):
2103         (CachedObject::setResponse):
2104         (CachedCSSStyleSheet::checkNotify):
2105         (Loader::servePendingRequests):
2106         (Loader::slotFinished):
2107         (Loader::slotReceivedResponse):
2108         (Cache::requestImage):
2109         (Cache::requestScript):
2110         * khtml/misc/loader.h:
2111         (khtml::CachedObject::CachedObject):
2112         (khtml::CachedObject::response):
2113         * kwq/KWQKJobClasses.h:
2114         * kwq/KWQKJobClasses.mm:
2115         (KIO::TransferJobPrivate::TransferJobPrivate):
2116         (KIO::TransferJobPrivate::~TransferJobPrivate):
2117         (KIO::TransferJob::TransferJob):
2118         (KIO::TransferJob::assembleResponseHeaders):
2119         (KIO::TransferJob::retrieveCharset):
2120         (KIO::TransferJob::emitResult):
2121         (KIO::TransferJob::emitReceivedResponse):
2122         * kwq/KWQLoader.h:
2123         * kwq/KWQLoader.mm:
2124         (KWQHeaderStringFromDictionary):
2125         (KWQCheckCacheObjectStatus):
2126         (KWQRetainResponse):
2127         (KWQReleaseResponse):
2128         (KWQIsResponseURLEqualToURL):
2129         (KWQResponseURL):
2130         (KWQResponseMIMEType):
2131         (KWQResponseTextEncodingName):
2132         (KWQResponseHeaderString):
2133         (KWQCacheObjectExpiresTime):
2134         (KWQLoader::KWQLoader):
2135         * kwq/KWQPixmap.h:
2136         * kwq/KWQPixmap.mm:
2137         (QPixmap::QPixmap):
2138         * kwq/KWQResourceLoader.mm:
2139         (-[KWQResourceLoader finishJobAndHandle]):
2140         (-[KWQResourceLoader cancel]):
2141         (-[KWQResourceLoader reportError]):
2142         (-[KWQResourceLoader finish]):
2143         * kwq/KWQSignal.h:
2144         * kwq/KWQSignal.mm:
2145         (KWQSignal::call):
2146         * kwq/KWQSlot.h:
2147         * kwq/KWQSlot.mm:
2148         (KWQSlot::KWQSlot):
2149         (KWQSlot::call):
2150         * kwq/WebCoreBridge.h:
2151         * kwq/WebCoreResourceLoader.h:
2152
2153 2004-12-03  John Sullivan  <sullivan@apple.com>
2154
2155         Reviewed by Chris.
2156         
2157         - fixed <rdar://problem/3903990> can't tab to all items on www.google.com any more (other pages too?)
2158
2159         * kwq/KWQKHTMLPart.mm:
2160         (KWQKHTMLPart::nextKeyViewInFrameHierarchy):
2161         when checking whether we moved the focus to another view, make sure we didn't "move" it to
2162         our documentView, because that's no move at all.
2163
2164 2004-12-03  Darin Adler  <darin@apple.com>
2165
2166         Reviewed by Vicki.
2167
2168         - fixed <rdar://problem/3901109> REGRESSION (171-172): repro crash in DOM::NodeImpl::setChanged at chick-fil-a.com
2169
2170         * khtml/css/css_valueimpl.cpp: (DOM::CSSMutableStyleDeclarationImpl::CSSMutableStyleDeclarationImpl):
2171         Added missing initialization for base class and node pointer.
2172
2173         - fixed a few places that could leave dangling node pointers
2174
2175         * khtml/html/html_baseimpl.cpp: (HTMLBodyElementImpl::~HTMLBodyElementImpl):
2176         Clear out the node pointer when the node is destroyed.
2177         * khtml/html/html_elementimpl.cpp: (HTMLElementImpl::~HTMLElementImpl): Ditto.
2178
2179 2004-12-03  Chris Blumenberg  <cblu@apple.com>
2180
2181         Fix for performance regression. My original patch added a signal for passing the data of a resource to its WebCore cache object. This patch passes the data with the preexisting "finished" symbol so we make less calls. 
2182         Fixed: <rdar://problem/3903173> REGRESSION (172-TOT): assertion failure and crash in slotAllData logging into hotmail account
2183
2184         Reviewed by rjw.
2185
2186         * khtml/misc/loader.cpp:
2187         (Loader::servePendingRequests): pass data param to slotFinished, removed allData signal
2188         (Loader::slotFinished): take data param
2189         * khtml/misc/loader.h:
2190         * kwq/KWQKJobClasses.h:
2191         * kwq/KWQKJobClasses.mm:
2192         (KIO::TransferJob::TransferJob): have m_result take a data param, removed m_allData
2193         (KIO::TransferJob::emitResult): take data param and pass it
2194         * kwq/KWQResourceLoader.mm:
2195         (-[KWQResourceLoader finishJobAndHandle:]): take data param and pass it
2196         (-[KWQResourceLoader cancel]): pass nil for data
2197         (-[KWQResourceLoader reportError]): ditto
2198         (-[KWQResourceLoader finishWithData:]): pass data
2199         * kwq/KWQSlot.mm:
2200         (KWQSlot::KWQSlot): pass data param to slotFinished
2201         (KWQSlot::call): added support for slotFinished_Loader, removed slotAllData
2202
2203 2004-12-03  Ken Kocienda  <kocienda@apple.com>
2204
2205         Reviewed by John
2206
2207         Did some clean up in the Position class as a result of trying to write some new layout
2208         tests and discovering a bug along the way.
2209
2210         I removed these three functions from the Position class:
2211         
2212         1. bool isFirstRenderedPositionOnLine() const;
2213         2. bool isLastRenderedPositionOnLine() const;
2214         3. static bool renderersOnDifferentLine(RenderObject *r1, long o1, RenderObject *r2, long o2);
2215         4. bool inFirstEditableInRootEditableElement() const;
2216
2217         The first two have replacements in the VisiblePosition class, and some code has been
2218         moved to use these new variants. The third function was a helper used only by these
2219         first two function, and can be removed as well. The fourth function was not used by anyone.
2220
2221         * khtml/editing/htmlediting.cpp:
2222         (khtml::InsertTextCommand::input): Change over to use VisiblePosition isFirstVisiblePositionOnLine().
2223         * khtml/editing/visible_position.cpp:
2224         (khtml::visiblePositionsOnDifferentLines): Added an additional check for blocks to this function.
2225         Incorrect results were being returned when asking about positions at the starts of blocks.
2226         * khtml/xml/dom_position.cpp:
2227         (DOM::Position::previousCharacterPosition): Change over to use VisiblePosition isFirstVisiblePositionOnLine().
2228         (DOM::Position::nextCharacterPosition): Change over to use VisiblePosition isLastVisiblePositionOnLine().
2229         (DOM::Position::rendersInDifferentPosition): Removed use of #3 helper in a log message. We can live without it.
2230         * khtml/xml/dom_position.h: Update header for deletions.
2231
2232 2004-12-03  Ken Kocienda  <kocienda@apple.com>
2233
2234         Reviewed by John
2235
2236         Terminology change in execCommand command identifiers. Specifically, the name of 
2237         "InsertNewline" command has been changed to "InsertLineBreak". This matches the 
2238         terminology used by AppKit. It is also more accurate, since the insertion of a
2239         "br" element is what the command does. The inspiration for this change is so the 
2240         -insertNewline AppKit method can be mapped to insert a new "div" element in 
2241         a document and avoid ambiguity with what the javascript editing command does.
2242         
2243         * khtml/editing/jsediting.cpp
2244         * layout-tests/editing/deleting/delete-tab-004.html
2245         * layout-tests/editing/editing.js
2246         * layout-tests/editing/inserting/insert-3654864-fix.html
2247         * layout-tests/editing/inserting/insert-3659587-fix.html
2248         * layout-tests/editing/inserting/insert-3775316-fix.html
2249         * layout-tests/editing/inserting/insert-3800346-fix.html
2250         * layout-tests/editing/inserting/insert-br-001.html
2251         * layout-tests/editing/inserting/insert-br-002.html
2252         * layout-tests/editing/inserting/insert-br-003.html
2253         * layout-tests/editing/inserting/insert-br-004.html
2254         * layout-tests/editing/inserting/insert-br-005.html
2255         * layout-tests/editing/inserting/insert-br-006.html
2256         * layout-tests/editing/inserting/insert-br-007.html
2257         * layout-tests/editing/inserting/insert-br-008.html
2258         * layout-tests/editing/inserting/insert-tab-004.html
2259         * layout-tests/editing/inserting/insert-text-with-newlines.html
2260         * layout-tests/editing/pasteboard/paste-text-010.html
2261
2262 2004-12-02  Ken Kocienda  <kocienda@apple.com>
2263
2264         Reviewed by John
2265
2266         Fix for this bug:
2267         
2268         <rdar://problem/3786362> REGRESSION (Mail): pasted text loses one newline
2269
2270         * khtml/editing/htmlediting.cpp:
2271         (khtml::InsertLineBreakCommand::doApply): Added check for strict mode before adding an extra br element
2272         at the end of a block. This is only necessary in quirks mode. Also, lower-case "br" used to make element.
2273         (khtml::ReplaceSelectionCommand::doApply): If the replacement adds a br element as the last element
2274         in a block and the document is in quirks mode, add an additional br to make the one in the 
2275         replacement content show up. This turns out to be much the same logic as is done in InsertLineBreakCommand.
2276         * layout-tests/editing/inserting/insert-3786362-fix-expected.txt: Added.
2277         * layout-tests/editing/inserting/insert-3786362-fix.html: Added.
2278
2279 2004-12-02  Richard Williamson   <rjw@apple.com>
2280
2281         Fixed <rdar://problem/3841332> REGRESSION (125.9-167u): repro crash in -[KWQPageState invalidate] involving .Mac images
2282
2283         Ensure that the document is cleared when leaving a non-HTML page.  This ensures that
2284         the b/f cache won't incorrectly trash the previous state when restoring.
2285
2286         Reviewed by John.
2287
2288         * kwq/WebCoreBridge.h:
2289         * kwq/WebCoreBridge.mm:
2290         (-[WebCoreBridge openURL:reload:contentType:refresh:lastModified:pageCache:]):
2291         (-[WebCoreBridge canCachePage]):
2292         (-[WebCoreBridge clear]):
2293
2294 2004-12-02  Ken Kocienda  <kocienda@apple.com>
2295
2296         Reviewed by Richard
2297
2298         Fix for this bug:
2299         
2300         <rdar://problem/3857775> 8A293: Mail.app crashes converting copy-pasted text into plain text
2301
2302         * khtml/xml/dom2_rangeimpl.cpp:
2303         (DOM::RangeImpl::commonAncestorContainer): Return the document element if no common ancestor container
2304         was found. This can happen in cases where the DOM was built from malformed markup (as in the case
2305         of this bug where there is content after the body tag). Did a little code clean up as well.
2306         (DOM::RangeImpl::compareBoundaryPoints): Made code more robust by adding some null checks.
2307
2308 2004-12-02  Ken Kocienda  <kocienda@apple.com>
2309
2310         Reviewed by Chris
2311
2312         Fix for this bug:
2313         
2314         <rdar://problem/3668157> REGRESSION (Mail): shift-click deselects when selection was created right-to-left
2315
2316         * khtml/khtml_part.cpp:
2317         (KHTMLPart::handleMousePressEventSingleClick): Use RangeImpl::compareBoundaryPoints
2318         to figure out which end of the selection to extend.
2319
2320 2004-12-02  David Harrison  <harrison@apple.com>
2321
2322         Reviewed by Ken Kocienda.
2323
2324                 <rdar://problem/3834917> REGRESSION (Mail): double-clicking blank line selects end of previous line
2325                 Fixed originally reported bug plus the case of double-clicking whitespace at the beginning of a line, which has a similar result.
2326
2327         * khtml/editing/visible_text.cpp:
2328         (khtml::SimplifiedBackwardsTextIterator::SimplifiedBackwardsTextIterator):
2329         (khtml::SimplifiedBackwardsTextIterator::handleTextNode):
2330         (khtml::SimplifiedBackwardsTextIterator::handleReplacedElement):
2331         (khtml::SimplifiedBackwardsTextIterator::handleNonTextNode):
2332         (khtml::SimplifiedBackwardsTextIterator::emitCharacter):
2333         Distinguish BR from whitespace.
2334         * khtml/editing/visible_text.h:
2335         Distinguish BR from whitespace.
2336         * khtml/editing/visible_units.cpp:
2337         (khtml::previousWordBoundary):
2338         Use UPSTREAM visible position now that SimplifiedBackwardsTextIterator distinguishes BR from whitespace.  Otherwise, double-clicking at end of line would result in caret selection at start of next line. 
2339
2340 2004-12-02  Ken Kocienda  <kocienda@apple.com>
2341
2342         Reviewed by John
2343
2344         Fix for this bug:
2345         
2346         <rdar://problem/3900996> Crash dragging past end of contentEditable DIV, at DOM::RangeImpl::pastEndNode() const + 24
2347
2348         * khtml/xml/dom_position.cpp:
2349         (DOM::Position::equivalentRangeCompliantPosition): Fixed this function so that it constrains the offset
2350         of the position to be >= 0 and <= number of kids of its node. Not doing this constraining led to a DOM
2351         exception trying to use a Position returned from this function to set the boundary point of a Range (which
2352         eventually led to the crash). Since this crash happened, it seems like this function was failing in its
2353         contract to return a range-compliant position, hence the need for this fix.
2354
2355 2004-12-01  Ken Kocienda  <kocienda@apple.com>
2356
2357         Reviewed by me
2358
2359         Moving code only.
2360
2361         * khtml/editing/htmlediting.cpp: Move ReplaceSelectionCommand into alphabetical order with
2362         regard to other editing commands. The class had a name change ages ago, and it was never
2363         moved.
2364         * khtml/editing/htmlediting.h: Ditto.
2365
2366 2004-12-01  Ken Kocienda  <kocienda@apple.com>
2367
2368         Reviewed by Hyatt
2369         
2370         Some improvements for paste, including some new code to annotate
2371         whitespace when writing to the pasteboard to ensure that the meaning
2372         of the markup on the pasteboard is unambiguous.
2373         
2374         There is also new code for reading this annotated markup from the pasteboard,
2375         removing the nodes that were added only to prevent ambiguity.
2376
2377         * WebCore.pbproj/project.pbxproj: Added html_interchange.h and html_interchange.cpp files.
2378         The header should have been added earlier, but I did not do so.
2379         * khtml/editing/html_interchange.cpp: Added.
2380         (convertHTMLTextToInterchangeFormat):
2381         * khtml/editing/html_interchange.h: Added some new constants for use with whitespace annotations.
2382         * khtml/editing/htmlediting.cpp:
2383         (khtml::ReplacementFragment::ReplacementFragment): Now looks for and removes annotations added for whitespace.
2384         Also fixed a bug in the code that counts blocks in a fragment.
2385         (khtml::ReplacementFragment::isInterchangeConvertedSpaceSpan): New helper. Recognizes annotation spans.
2386         (khtml::ReplacementFragment::insertNodeBefore): New helper.
2387         (khtml::ReplaceSelectionCommand::doApply): Fixed a bug in the code that sets the start position
2388         for the replacement after deleting. This was causing a bug when pasting at the end of a block.
2389         * khtml/editing/htmlediting.h: Add some new declarations.
2390         * khtml/xml/dom2_rangeimpl.cpp:
2391         (DOM::RangeImpl::toHTML): Calls to startMarkup now pass true for the new annotate flag.
2392         * khtml/xml/dom_nodeimpl.cpp:
2393         (NodeImpl::stringValueForRange): New helper.
2394         (NodeImpl::renderedText): New helper to return only the rendered text in a node.
2395         (NodeImpl::startMarkup): Now takes an additional flag to control whether interchange annotations
2396         should be added. Called by the paste code.
2397         * khtml/xml/dom_nodeimpl.h: Added and modified function declarations.
2398
2399         New test to check the khtml::ReplaceSelectionCommand::doApply fix.
2400         * layout-tests/editing/pasteboard/paste-text-010-expected.txt: Added.
2401         * layout-tests/editing/pasteboard/paste-text-010.html: Added.
2402
2403 2004-11-30  Chris Blumenberg  <cblu@apple.com>
2404
2405         * ChangeLog: removed conflict marker
2406
2407 2004-11-30  Chris Blumenberg  <cblu@apple.com>
2408
2409         Fixed:
2410         <rdar://problem/3685766> WebDataSource is missing subresources when they use cached WebCore data
2411         <rdar://problem/3722434> REGRESSION?: Assertion failure trying to drag image in iframe (itapema.sc.gov.br)
2412
2413         Reviewed by darin.
2414
2415         * khtml/misc/loader.cpp:
2416         (CachedObject::~CachedObject): release m_allData
2417         (CachedObject::setAllData): new
2418         (Loader::servePendingRequests): connect slotAllData
2419         (Loader::slotAllData): new
2420         (Cache::requestImage): tweak
2421         * khtml/misc/loader.h:
2422         (khtml::CachedObject::CachedObject): set allData to 0
2423         (khtml::CachedObject::allData): new
2424         * kwq/KWQKJobClasses.h:
2425         * kwq/KWQKJobClasses.mm:
2426         (KIO::TransferJob::TransferJob): set m_allData
2427         (KIO::TransferJob::emitAllData): new
2428         * kwq/KWQLoader.mm:
2429         (KWQCheckCacheObjectStatus): pass WebKit the data instead of the length of the resource
2430         * kwq/KWQResourceLoader.mm:
2431         (-[KWQResourceLoader finishWithData:]): renamed to pass all data for the resource
2432         * kwq/KWQSlot.mm:
2433         (KWQSlot::KWQSlot): support for slotAllData
2434         (KWQSlot::call):
2435         * kwq/WebCoreBridge.h:
2436         * kwq/WebCoreResourceLoader.h:
2437
2438 2004-11-30  Maciej Stachowiak  <mjs@apple.com>
2439
2440         Reviewed by John.
2441
2442 2004-11-30  Maciej Stachowiak  <mjs@apple.com>
2443
2444         Reviewed by John.
2445
2446         <rdar://problem/3805311> REGRESSION (159-163): onload in dynamically written document not called (causes blank search page at Japanese EPP site, many others)
2447
2448         * khtml/khtml_part.cpp:
2449         (KHTMLPart::begin): call setParsing on document here after opening
2450         - from now on we'll only set parsing to true for a document open
2451         caused by page loading, not a programmatic one.
2452         * khtml/xml/dom_docimpl.cpp:
2453         (DocumentImpl::open): don't setParsing to true here any more.
2454
2455 2004-11-30  Maciej Stachowiak  <mjs@apple.com>
2456
2457         Reviewed by John.
2458
2459         - fix recent regression from collection perf fixes.
2460         
2461         * khtml/html/html_miscimpl.cpp:
2462         (HTMLFormCollectionImpl::updateNameCache): Look up the name
2463         attribute in the name cache, not the id cache (d'oh!)
2464
2465 2004-11-30  Darin Adler  <darin@apple.com>
2466
2467         Reviewed by Ken.
2468
2469         - rolled in a KDE fix for a problem that may underlie a number of crashes
2470
2471         * khtml/xml/dom2_rangeimpl.cpp: (RangeImpl::compareBoundaryPoints): Rolled in a change from
2472         the KDE guys to fix a subtle problem. Code said "n = n =".
2473
2474         - rolled in a KDE fix for a containingBlock crash
2475
2476         * khtml/rendering/render_object.cpp: Roll in a change from KDE that adds frameset to the list of
2477         elements that can not be a containingBlock. They said this fixes a crash, although I did not look
2478         into that further.
2479
2480         - fixed <rdar://problem/3884660> 8A305: Repro crash in QScrollBar::setValue (affects Safari RSS)
2481
2482         * kwq/KWQButton.mm:
2483         (-[KWQButton initWithQButton:]): Set up target and action here instead of in caller.
2484         (-[KWQButton detachQButton]): Added.
2485         (-[KWQButton sendConsumedMouseUpIfNeeded]): Check button for nil instead of checking target.
2486         (-[KWQButton mouseDown:]): Add calls to QWidget::beforeMouseDown/afterMouseDown.
2487         (-[KWQButton widget]): Added.
2488         (-[KWQButton becomeFirstResponder]): Added check to handle when button is 0.
2489         (-[KWQButton resignFirstResponder]): Ditto.
2490         (-[KWQButton canBecomeKeyView]): Ditto.
2491         (QButton::QButton): Remove target and action setup; handled in KWQButton now.
2492         (QButton::~QButton): Call detachQButton instead of setTarget:nil.
2493
2494         * kwq/KWQComboBox.mm:
2495         (QComboBox::~QComboBox): Call detachQComboBox.
2496         (-[KWQPopUpButtonCell detachQComboBox]): Added.
2497         (-[KWQPopUpButtonCell trackMouse:inRect:ofView:untilMouseUp:]): Handle case where box is 0.
2498         (-[KWQPopUpButtonCell setHighlighted:]): Ditto.
2499         (-[KWQPopUpButton action:]): Ditto.
2500         (-[KWQPopUpButton widget]): Tweaked.
2501         (-[KWQPopUpButton mouseDown:]): Added. Calls QWidget::beforeMouseDown/afterMouseDown.
2502         (-[KWQPopUpButton becomeFirstResponder]): Handle case where widget is 0.
2503         (-[KWQPopUpButton resignFirstResponder]): Ditto.
2504         (-[KWQPopUpButton canBecomeKeyView]): Ditto.
2505
2506         * kwq/KWQLineEdit.mm: (QLineEdit::~QLineEdit): Updated to use new detachQLineEdit name.
2507
2508         * kwq/KWQListBox.mm:
2509         (-[KWQTableView mouseDown:]): Added. Calls QWidget::beforeMouseDown/afterMouseDown.
2510
2511         * kwq/KWQScrollBar.h: Removed m_scroller field.
2512         * kwq/KWQScrollBar.mm:
2513         (-[KWQScrollBar initWithQScrollBar:]): Rearranged a little bit.
2514         (-[KWQScrollBar detachQScrollBar]): Added.
2515         (-[KWQScrollBar widget]): Added.
2516         (-[KWQScrollBar mouseDown:]): Added.  Calls QWidget::beforeMouseDown and afterMouseDown.
2517         (QScrollBar::QScrollBar): Changed to no longer set m_scroller.
2518         (QScrollBar::~QScrollBar): Changed to call detachQScrollBar. No longer calls removeFromSuperview.
2519         (QScrollBar::setValue): Chagned to use getView instad of m_scrollBar.
2520         (QScrollBar::setKnobProportion): Ditto.
2521         (QScrollBar::scrollbarHit): Ditto.
2522
2523         * kwq/KWQScrollView.mm:
2524         (QScrollView::addChild): Changed to call QWidget to add to superview to accomodate the
2525         hack where we don't remove right away when doing mouse tracking.
2526         (QScrollView::removeChild): Changed to call QWidget to remove from superview to accomodate
2527         the hack where we don't add right away when doing mouse tracking.
2528
2529         * kwq/KWQSlider.h: Added destructor.
2530         * kwq/KWQSlider.mm:
2531         (-[KWQSlider initWithQSlider:]): Tweaked a little.
2532         (-[KWQSlider detachQSlider]): Added.
2533         (-[KWQSlider mouseDown:]): Added call to QWidget::beforeMouseDown/afterMouseDown.
2534         (-[KWQSlider widget]): Added.
2535         (QSlider::~QSlider): Added. Calls detachQSlider.
2536
2537         * kwq/KWQTextArea.h: Added detachQTextEdit method.
2538         * kwq/KWQTextArea.mm:
2539         (-[KWQTextArea detachQTextEdit]): Added.
2540         (-[KWQTextArea textDidChange:]): Added check for widget of 0.
2541         (-[KWQTextArea becomeFirstResponder]): Ditto.
2542         (-[KWQTextArea nextKeyView]): Ditto.
2543         (-[KWQTextArea previousKeyView]): Ditto.
2544         (-[KWQTextArea drawRect:]): Ditto.
2545         (-[KWQTextAreaTextView insertTab:]): Ditto.
2546         (-[KWQTextAreaTextView insertBacktab:]): Ditto.
2547         (-[KWQTextAreaTextView shouldDrawInsertionPoint]): Ditto.
2548         (-[KWQTextAreaTextView selectedTextAttributes]): Ditto.
2549         (-[KWQTextAreaTextView mouseDown:]): Ditto.
2550         (-[KWQTextAreaTextView keyDown:]): Ditto.
2551         (-[KWQTextAreaTextView keyUp:]): Ditto.
2552
2553         * kwq/KWQTextEdit.h: Added ~QTextEdit.
2554         * kwq/KWQTextEdit.mm: (QTextEdit::~QTextEdit): Added. Calls detachQTextEdit.
2555
2556         * kwq/KWQTextField.h: Changed invalidate to detachQLineEdit.
2557         * kwq/KWQTextField.mm: (-[KWQTextFieldController detachQLineEdit]): Changed.
2558
2559         * kwq/KWQWidget.h: Added addToSuperview/removeFromSuperview for use from QScrollView.
2560         Added beforeMouseDown and afterMouseDown for use in widget implementations.
2561         Removed unused hasMouseTracking function.
2562         * kwq/KWQWidget.mm:
2563         (QWidget::QWidget): Initialize two new fields.
2564         (QWidget::~QWidget): Added code to remove view when widget is destroyed.
2565         (QWidget::getOuterView): Remove unneeded exception blocking since we're just caling superview.
2566         (QWidget::addToSuperview): Added.
2567         (QWidget::removeFromSuperview): Added.
2568         (QWidget::beforeMouseDown): Added.
2569         (QWidget::afterMouseDown): Added.
2570
2571         * khtml/rendering/render_layer.cpp:
2572         (RenderLayer::setHasHorizontalScrollbar): Remove parent parameter; let addChild call addSubview:.
2573         (RenderLayer::setHasVerticalScrollbar): Ditto.
2574
2575 2004-11-30  Ken Kocienda  <kocienda@apple.com>
2576
2577         Reviewed by John
2578
2579         Fix for this bug:
2580         
2581         <rdar://problem/3863031> REGRESSION (Mail): caret continues flashing while mouse is down
2582
2583         * khtml/khtml_part.cpp:
2584         (KHTMLPart::timerEvent): Add a check for whether the mouse is down. Keep the caret drawn
2585         with no blink if it is.
2586
2587 2004-11-30  Ken Kocienda  <kocienda@apple.com>
2588
2589         Reviewed by John
2590
2591         Fix for this bug:
2592         
2593         <rdar://problem/3861602> cursor gets lost trying to backspace to delete a form control
2594
2595         * khtml/khtml_part.cpp:
2596         (KHTMLPart::setFocusNodeIfNeeded): This function would clear the selection if a <button>
2597         or <input type=image> was checked for focus since these elements are keyboard-focusable,
2598         but not mouse focusable. Also, this function did not work hard enough to set the focused
2599         node, and was content to clear it if the first element checked failed the test, rather
2600         than looking more at parents. This would have the effect of clearing, then resetting the
2601         focus on a DIV containing a button or image with content on either side of it in the
2602         process of arrowing over such content.
2603
2604 2004-11-30  Ken Kocienda  <kocienda@apple.com>
2605
2606         Reviewed by John
2607
2608         * khtml/editing/htmlediting.cpp:
2609         (khtml::ReplaceSelectionCommand::doApply): Fix smart replace, which I (knowingly) broke with yesterday's checkin.
2610         Also, call updateLayout() in one more place to prevent stale information being returned from caretMaxOffset().
2611         * khtml/khtml_part.cpp:
2612         (KHTMLPart::isCharacterSmartReplaceExempt): Make this virtual and always return true. This gets rid of an
2613         ugly APPLE_CHANGES block and use of KWQ(part) in ReplaceSelectionCommand.
2614         * khtml/khtml_part.h: To help out with the isCharacterSmartReplaceExempt cleanup, add declaration.
2615         * kwq/KWQKHTMLPart.h: To help out with the isCharacterSmartReplaceExempt cleanup, make 
2616         isCharacterSmartReplaceExempt virtual.
2617
2618 2004-11-30  Ken Kocienda  <kocienda@apple.com>
2619
2620         Reviewed by me
2621
2622         * khtml/editing/htmlediting.cpp:
2623         (khtml::ReplacementFragment::mergeEndNode): Fixed one-line coding mistake that created an endless loop.
2624         Seemed simple enough to land without review.
2625
2626 2004-11-29  Ken Kocienda  <kocienda@apple.com>
2627
2628         Reviewed by Chris
2629     
2630         Rewrite of paste code (specifically the ReplaceSelectionCommand class). Many more cases
2631         are handled correctly now, including selections that span multiple blocks, and cases
2632         where content on the pasteboard ends in newlines (or what appear to be newlines to a
2633         user, really block ends or BRs). I also made one small, but important change in the 
2634         copy code to annotate the markup written to the pasteboard to support these selections 
2635         ending in newlines.
2636
2637         New header that defines a couple of constants used in copying and pasting.
2638
2639         * ForwardingHeaders/editing/html_interchange.h: Added.
2640         * khtml/editing/html_interchange.h: Added.
2641         
2642         Rewrite of the ReplaceSelectionCommand. There are several new helper functions, as well
2643         as a new helper class, ReplacementFragment, which encapsulates information and functions
2644         pertaining to a document fragment that is being inserted into a document.
2645         
2646         * khtml/editing/htmlediting.cpp:
2647         (khtml::ReplacementFragment::ReplacementFragment):
2648         (khtml::ReplacementFragment::~ReplacementFragment):
2649         (khtml::ReplacementFragment::firstChild): Simple accessor.
2650         (khtml::ReplacementFragment::lastChild): Ditto.
2651         (khtml::ReplacementFragment::mergeStartNode): Looks at the nodes in a fragment and determines
2652         the starting node to use for merging into the block containing the start of the selection.
2653         (khtml::ReplacementFragment::mergeEndNode): Same as above, but for the end of the selection.
2654         (khtml::ReplacementFragment::pruneEmptyNodes): Simple helper.
2655         (khtml::ReplacementFragment::isInterchangeNewlineComment): Determines if a node is the
2656         special annotation comment added in by the copy code.
2657         (khtml::ReplacementFragment::removeNode): Simple helper.
2658         (khtml::isComment): Simple helper.
2659         (khtml::isProbablyBlock): Determines if a node is of a type that is usually rendered as a block.
2660         I would like to do better than this some day, but this check will hold us until I can do better.
2661         (khtml::ReplaceSelectionCommand::ReplaceSelectionCommand):
2662         (khtml::ReplaceSelectionCommand::~ReplaceSelectionCommand):
2663         (khtml::ReplaceSelectionCommand::doApply):
2664         (khtml::ReplaceSelectionCommand::completeHTMLReplacement): Figures out the right ending selection.
2665         * khtml/editing/htmlediting.h: Declarations for the new ReplacementFragment class.
2666         (khtml::ReplacementFragment::root):
2667         (khtml::ReplacementFragment::type):
2668         (khtml::ReplacementFragment::isEmpty):
2669         (khtml::ReplacementFragment::isSingleTextNode):
2670         (khtml::ReplacementFragment::isTreeFragment):
2671         (khtml::ReplacementFragment::hasMoreThanOneBlock):
2672         (khtml::ReplacementFragment::hasLogicalNewlineAtEnd):
2673         
2674         This smaller set of changes markup generation to add the newline annotation described in the
2675         comment at the start of this entry.
2676         
2677         * khtml/xml/dom2_rangeimpl.cpp:
2678         (DOM::RangeImpl::addCommentToHTMLMarkup): Simple helper.
2679         (DOM::RangeImpl::toHTML): Added new EAnnotateForInterchange default argument to control whether
2680         comment annotations are added to the markup generated.
2681         * khtml/xml/dom2_rangeimpl.h: Add some new declarations.
2682         * kwq/WebCoreBridge.mm:
2683         (-[WebCoreBridge markupStringFromRange:nodes:]): Request that markup resulting from call to 
2684         DOM::RangeImpl::toHTML uses annotations when generating.
2685         
2686         New tests.        
2687
2688         * layout-tests/editing/pasteboard/paste-text-001-expected.txt: Added.
2689         * layout-tests/editing/pasteboard/paste-text-001.html: Added.
2690         * layout-tests/editing/pasteboard/paste-text-002-expected.txt: Added.
2691         * layout-tests/editing/pasteboard/paste-text-002.html: Added.
2692         * layout-tests/editing/pasteboard/paste-text-003-expected.txt: Added.
2693         * layout-tests/editing/pasteboard/paste-text-003.html: Added.
2694         * layout-tests/editing/pasteboard/paste-text-004-expected.txt: Added.
2695         * layout-tests/editing/pasteboard/paste-text-004.html: Added.
2696         * layout-tests/editing/pasteboard/paste-text-005-expected.txt: Added.
2697         * layout-tests/editing/pasteboard/paste-text-005.html: Added.
2698         * layout-tests/editing/pasteboard/paste-text-006-expected.txt: Added.
2699         * layout-tests/editing/pasteboard/paste-text-006.html: Added.
2700         * layout-tests/editing/pasteboard/paste-text-007-expected.txt: Added.
2701         * layout-tests/editing/pasteboard/paste-text-007.html: Added.
2702         * layout-tests/editing/pasteboard/paste-text-008-expected.txt: Added.
2703         * layout-tests/editing/pasteboard/paste-text-008.html: Added.
2704         * layout-tests/editing/pasteboard/paste-text-009-expected.txt: Added.
2705         * layout-tests/editing/pasteboard/paste-text-009.html: Added.
2706
2707 2004-11-29  Ken Kocienda  <kocienda@apple.com>
2708
2709         Reviewed by Harrison
2710
2711         Made two small changes that make it possible for comments to have DOM nodes made for them
2712         when pasting. This relies on some earlier work I did some days ago.
2713
2714         * khtml/xml/dom_nodeimpl.cpp:
2715         (NodeImpl::startMarkup): Get the string from the comment.
2716         * kwq/WebCoreBridge.mm:
2717         (-[WebCoreBridge documentFragmentWithMarkupString:baseURLString:]): Did some very minor
2718         rearranging. Now passes a flag when creating a contextual fragment, requesting that comments
2719         be included in the DOM.
2720
2721 2004-11-29  Ken Kocienda  <kocienda@apple.com>
2722
2723         Reviewed by Harrison
2724
2725         Added some new helpers to the VisiblePosition class. I will begin to use these when I check in
2726         my improved paste code.
2727
2728         * khtml/editing/visible_position.cpp:
2729         (khtml::blockRelationship)
2730         (khtml::visiblePositionsInDifferentBlocks)
2731         (khtml::isFirstVisiblePositionInBlock)
2732         (khtml::isFirstVisiblePositionInNode)
2733         (khtml::isLastVisiblePositionInBlock)
2734         * khtml/editing/visible_position.h
2735
2736 2004-11-29  Ken Kocienda  <kocienda@apple.com>
2737
2738         Reviewed by Harrison
2739
2740         * khtml/xml/dom_position.cpp:
2741         (DOM::Position::downstream): Fix a bug in downstream that prevented a call with DoNotStayInBlock
2742         specified from obeying that directive. The old code would stop at an outer block boundary in
2743         the case where that block had a block as its first child. The correct behavior is to drill into
2744         that inner block (and continue on drilling down, if possible), to find the correct position.
2745
2746 2004-11-29  Ken Kocienda  <kocienda@apple.com>
2747
2748         Reviewed by Harrison
2749
2750         Small improvements to the node-display debugging helpers.
2751
2752         * khtml/xml/dom_nodeimpl.cpp:
2753         (NodeImpl::displayTree): Make the rootNode be this if there is no rootEditableElement.
2754         * khtml/xml/dom_nodeimpl.h: Make displayNode take a default argument of "" for its string.
2755
2756 2004-11-29  Ken Kocienda  <kocienda@apple.com>
2757
2758         Reviewed by Harrison
2759
2760         * khtml/editing/htmlediting.cpp:
2761         (khtml::DeleteSelectionCommand::handleGeneralDelete): The downstream position in this function 
2762         may need to be adjusted when deleting text off the front part of a text node. This fixes a problem 
2763         I discovered while improving the paste command, where the insertion poitn wound up in the wrong
2764         place after the delete. 
2765
2766 2004-11-29  Ken Kocienda  <kocienda@apple.com>
2767
2768         Reviewed by Harrison
2769
2770         Add a new helper function to insert a paragraph separator. Will be used in my
2771         upcoming paste improvments.
2772
2773         * khtml/editing/htmlediting.cpp: Added function
2774         (khtml::CompositeEditCommand::insertParagraphSeparator)
2775         * khtml/editing/htmlediting.h: Ditto.
2776
2777 2004-11-23  David Harrison  <harrison@apple.com>
2778
2779         Added various comments.
2780
2781         * khtml/editing/htmlediting.cpp:
2782         (khtml::StyleChange::init):
2783         (khtml::ApplyStyleCommand::doApply):
2784         (khtml::ApplyStyleCommand::applyBlockStyle):
2785         (khtml::ApplyStyleCommand::applyInlineStyle):
2786
2787 2004-11-23  David Hyatt  <hyatt@apple.com>
2788
2789         Hit testing in table cells with top/bottom space from vertical alignment didn't work.  I forgot about the
2790         super-secret yPos() lie that table cells do.  Use m_y instead of yPos().
2791
2792         * khtml/rendering/render_block.cpp:
2793         (khtml::RenderBlock::nodeAtPoint):
2794
2795 2004-11-22  David Hyatt  <hyatt@apple.com>
2796
2797         Make sure you can use document.createElement to make a <canvas> element.
2798
2799         * khtml/xml/dom_docimpl.cpp:
2800         (DocumentImpl::createHTMLElement):
2801
2802 2004-11-22  Maciej Stachowiak  <mjs@apple.com>
2803
2804         Reviewed by Dave.
2805
2806         <rdar://problem/3492044> performing JavaScript operations on form elements is slower than WinIE (HTMLFormCollection)
2807         <rdar://problem/3489679> selecting an item on the Apache bugzilla query page is very slow (HTMLFormCollection)
2808         <rdar://problem/3477810> checking 80 check boxes with JavaScript is 10x slower than in IE (HTMLFormCollection)
2809         <rdar://problem/3760962> JavaScript that toggles checkboxes is slow (HTMLCollection,HTMLFormCollection)
2810         
2811         * khtml/ecma/kjs_html.cpp:
2812         (KJS::HTMLDocument::tryGet):
2813         * khtml/html/html_formimpl.cpp:
2814         (DOM::HTMLFormElementImpl::HTMLFormElementImpl):
2815         (DOM::HTMLFormElementImpl::~HTMLFormElementImpl):
2816         * khtml/html/html_formimpl.h:
2817         * khtml/html/html_miscimpl.cpp:
2818         (HTMLCollectionImpl::HTMLCollectionImpl):
2819         (HTMLCollectionImpl::~HTMLCollectionImpl):
2820         (HTMLCollectionImpl::CollectionInfo::CollectionInfo):
2821         (HTMLCollectionImpl::CollectionInfo::reset):
2822         (HTMLCollectionImpl::resetCollectionInfo):
2823         (HTMLCollectionImpl::checkForNameMatch):
2824         (appendToVector):
2825         (HTMLCollectionImpl::updateNameCache):
2826         (HTMLCollectionImpl::namedItems):
2827         (HTMLFormCollectionImpl::HTMLFormCollectionImpl):
2828         (HTMLFormCollectionImpl::~HTMLFormCollectionImpl):
2829         (HTMLFormCollectionImpl::item):
2830         (HTMLFormCollectionImpl::updateNameCache):
2831         * khtml/html/html_miscimpl.h:
2832
2833 2004-11-22  David Hyatt  <hyatt@apple.com>
2834
2835         Improve the WebCore cache so that the maximum cacheable object size is scaled based off the total cache
2836         size.
2837         
2838         Reviewed by mjs
2839
2840         * khtml/misc/loader.cpp:
2841         (CachedObject::finish):
2842         (Cache::flush):
2843         (Cache::setSize):
2844         * khtml/misc/loader.h:
2845         (khtml::Cache::maxCacheableObjectSize):
2846
2847 2004-11-22  David Hyatt  <hyatt@apple.com>
2848
2849         Fix for 3673381, huge directory listing so slow it seems like a hang.  Rework painting and hit testing so that
2850         it crawls the line box tree instead of the render tree.  This allows more precise intersection/containment testing
2851         that lets us short circuit earlier when painting and hit testing.
2852         
2853         Reviewed by mjs
2854
2855         * khtml/khtml_part.cpp:
2856         (KHTMLPart::isPointInsideSelection):
2857         * khtml/rendering/render_block.cpp:
2858         (khtml::RenderBlock::paint):
2859         (khtml::RenderBlock::paintChildren):
2860         (khtml::RenderBlock::paintObject):
2861         (khtml::RenderBlock::paintFloats):
2862         (khtml::RenderBlock::nodeAtPoint):
2863         * khtml/rendering/render_block.h:
2864         * khtml/rendering/render_box.cpp:
2865         (RenderBox::nodeAtPoint):
2866         * khtml/rendering/render_box.h:
2867         * khtml/rendering/render_br.h:
2868         * khtml/rendering/render_canvas.cpp:
2869         (RenderCanvas::paint):
2870         * khtml/rendering/render_flow.cpp:
2871         (RenderFlow::paintLines):
2872         (RenderFlow::hitTestLines):
2873         (RenderFlow::caretRect):
2874         (RenderFlow::addFocusRingRects):
2875         (RenderFlow::paintFocusRing):
2876         (RenderFlow::paintOutlines):
2877         (RenderFlow::paintOutlineForLine):
2878         * khtml/rendering/render_flow.h:
2879         * khtml/rendering/render_frames.cpp:
2880         (RenderFrameSet::nodeAtPoint):
2881         * khtml/rendering/render_frames.h:
2882         * khtml/rendering/render_image.cpp:
2883         (RenderImage::nodeAtPoint):
2884         * khtml/rendering/render_image.h:
2885         * khtml/rendering/render_inline.cpp:
2886         (RenderInline::paint):
2887         (RenderInline::nodeAtPoint):
2888         * khtml/rendering/render_inline.h:
2889         * khtml/rendering/render_layer.cpp:
2890         (RenderLayer::paintLayer):
2891         (RenderLayer::hitTest):
2892         (RenderLayer::hitTestLayer):
2893         * khtml/rendering/render_layer.h:
2894         * khtml/rendering/render_line.cpp:
2895         (khtml::InlineBox::paint):
2896         (khtml::InlineBox::nodeAtPoint):
2897         (khtml::InlineFlowBox::flowObject):
2898         (khtml::InlineFlowBox::nodeAtPoint):
2899         (khtml::InlineFlowBox::paint):
2900         (khtml::InlineFlowBox::paintBackgrounds):
2901         (khtml::InlineFlowBox::paintBackground):
2902         (khtml::InlineFlowBox::paintBackgroundAndBorder):
2903         (khtml::InlineFlowBox::paintDecorations):
2904         (khtml::EllipsisBox::paint):
2905         (khtml::EllipsisBox::nodeAtPoint):
2906         (khtml::RootInlineBox::paintEllipsisBox):
2907         (khtml::RootInlineBox::paint):
2908         (khtml::RootInlineBox::nodeAtPoint):
2909         * khtml/rendering/render_line.h:
2910         (khtml::InlineRunBox::paintBackgroundAndBorder):
2911         * khtml/rendering/render_object.cpp:
2912         (RenderObject::hitTest):
2913         (RenderObject::setInnerNode):
2914         (RenderObject::nodeAtPoint):
2915         * khtml/rendering/render_object.h:
2916         (khtml::RenderObject::PaintInfo::PaintInfo):
2917         (khtml::RenderObject::PaintInfo::~PaintInfo):
2918         (khtml::RenderObject::paintingRootForChildren):
2919         (khtml::RenderObject::shouldPaintWithinRoot):
2920         * khtml/rendering/render_table.cpp:
2921         (RenderTable::layout):
2922         (RenderTable::paint):
2923         * khtml/rendering/render_text.cpp:
2924         (simpleDifferenceBetweenColors):
2925         (correctedTextColor):
2926         (InlineTextBox::nodeAtPoint):
2927         (InlineTextBox::paint):
2928         (InlineTextBox::selectionStartEnd):
2929         (InlineTextBox::paintSelection):
2930         (InlineTextBox::paintMarkedTextBackground):
2931         (InlineTextBox::paintDecoration):
2932         (RenderText::posOfChar):
2933         * khtml/rendering/render_text.h:
2934         (khtml::RenderText::paint):
2935         (khtml::RenderText::layout):
2936         (khtml::RenderText::nodeAtPoint):
2937         * khtml/xml/dom2_eventsimpl.cpp:
2938         (MouseEventImpl::computeLayerPos):
2939         * khtml/xml/dom_docimpl.cpp:
2940         (DocumentImpl::prepareMouseEvent):
2941         * kwq/KWQAccObject.mm:
2942         (-[KWQAccObject accessibilityHitTest:]):
2943         * kwq/KWQKHTMLPart.mm:
2944         (KWQKHTMLPart::scrollOverflowWithScrollWheelEvent):
2945         (KWQKHTMLPart::eventMayStartDrag):
2946         (KWQKHTMLPart::khtmlMouseMoveEvent):
2947         * kwq/WebCoreBridge.mm:
2948         (-[WebCoreBridge elementAtPoint:]):
2949         (-[WebCoreBridge _positionForPoint:]):
2950
2951 2004-11-22  Maciej Stachowiak  <mjs@apple.com>
2952
2953         Reviewed by Dave.
2954
2955         <rdar://problem/3890961> selecting an item on the Apache bugzilla query page can be sped up 10% (HTMLFormCollection)
2956         <rdar://problem/3890958> JavaScript that toggles checkboxes can be improved 73% (HTMLCollection,HTMLFormCollection)
2957
2958         This avoids the O(N^2) penalty for named item traversal for form collections.
2959
2960         It also combines the item traversal logic for all non-form
2961         collection operations into a single traverseNextItem
2962         function. This avoids having 5 copies of the big switch statement
2963         for this.
2964
2965         Also fixed a bug that prevented the last form element from being removed properly.
2966         
2967         * khtml/html/html_formimpl.cpp:
2968         (DOM::removeFromVector):
2969         * khtml/dom/html_misc.cpp:
2970         (HTMLCollection::namedItems):
2971         * khtml/dom/html_misc.h:
2972         * khtml/ecma/kjs_html.cpp:
2973         (KJS::HTMLCollection::getNamedItems):
2974         * khtml/html/html_miscimpl.cpp:
2975         (HTMLCollectionImpl::traverseNextItem):
2976         (HTMLCollectionImpl::calcLength):
2977         (HTMLCollectionImpl::length):
2978         (HTMLCollectionImpl::item):
2979         (HTMLCollectionImpl::nextItem):
2980         (HTMLCollectionImpl::checkForNameMatch):
2981         (HTMLCollectionImpl::namedItem):
2982         (HTMLCollectionImpl::namedItems):
2983         (HTMLCollectionImpl::nextNamedItem):
2984         (HTMLFormCollectionImpl::calcLength):
2985         (HTMLFormCollectionImpl::namedItem):
2986         (HTMLFormCollectionImpl::nextNamedItem):
2987         (HTMLFormCollectionImpl::namedItems):
2988         * khtml/html/html_miscimpl.h:
2989
2990 2004-11-22  Ken Kocienda  <kocienda@apple.com>
2991
2992         Reviewed by Harrison
2993
2994         Change around the way we block the Javascript "Paste" command identifier from
2995         being available. Formerly, this was done with an ifdef we never compiled in.
2996         Now, this is done with a couple of cheap runtime checks. The advantage is that
2997         we can now compile this command into development builds, and still yet switch
2998         on the command in deployment builds through the use of WebCore SPI so we can
2999         write and run layout tests with all of our builds.
3000
3001         * khtml/editing/jsediting.cpp:
3002         (DOM::JSEditor::queryCommandSupported): Checks state of paste command in case
3003         command being queried is the paste command. 
3004         (DOM::JSEditor::setSupportsPasteCommand): New SPI to turn on paste command.
3005         * khtml/editing/jsediting.h: Ditto.
3006         * khtml/khtml_part.cpp:
3007         (KHTMLPart::pasteFromPasteboard): Added.
3008         (KHTMLPart::canPaste): Added.
3009         * kwq/KWQKHTMLPart.mm:
3010         (KHTMLPart::canPaste): Added.
3011         * kwq/KWQRenderTreeDebug.cpp:
3012         (externalRepresentation): Turn on paste command.
3013         * kwq/WebCoreBridge.h: Add canPaste call so WebKit can fill in the answer.
3014
3015 2004-11-21  Maciej Stachowiak  <mjs@apple.com>
3016
3017         Reviewed by Richard.
3018
3019         <rdar://problem/3889655> HTMLCollectionImpl should use traverseNextNode to improve speed and save recursion
3020         
3021         * khtml/html/html_miscimpl.cpp:
3022         (HTMLCollectionImpl::calcLength):
3023         (HTMLCollectionImpl::getItem):
3024         (HTMLCollectionImpl::item):
3025         (HTMLCollectionImpl::nextItem):
3026         (HTMLCollectionImpl::getNamedItem):
3027         (HTMLCollectionImpl::namedItem):
3028         (HTMLCollectionImpl::nextNamedItemInternal):
3029         (HTMLFormCollectionImpl::nextNamedItemInternal):
3030
3031 2004-11-19  Maciej Stachowiak  <mjs@apple.com>
3032
3033         Reviewed by Darin.
3034
3035         <rdar://problem/3482935> JavaScript so slow it seems like a hang (hrweb.apple.com) (HTMLCollection?)
3036         <rdar://problem/3759149> PeopleSoft page in Safari twice as slow as Mozilla engine (HTMLFormCollection)
3037         <rdar://problem/3888368> selecting an item on the Apache bugzilla query page can be improved 95% (HTMLFormCollection)
3038
3039         Many optimizations to HTMLFormCollection. Iterating it should not
3040         be N^2 any more, though finding items by name could still be.
3041         
3042         * khtml/html/html_formimpl.cpp:
3043         (DOM::HTMLFormElementImpl::~HTMLFormElementImpl):
3044         (DOM::HTMLFormElementImpl::length):
3045         (DOM::HTMLFormElementImpl::submitClick):
3046         (DOM::HTMLFormElementImpl::formData):
3047         (DOM::HTMLFormElementImpl::submit):
3048         (DOM::HTMLFormElementImpl::reset):
3049         (DOM::HTMLFormElementImpl::radioClicked):
3050         (DOM::appendToVector):
3051         (DOM::removeFromVector):
3052         (DOM::HTMLFormElementImpl::registerFormElement):
3053         (DOM::HTMLFormElementImpl::removeFormElement):
3054         (DOM::HTMLFormElementImpl::makeFormElementDormant):
3055         (DOM::HTMLFormElementImpl::registerImgElement):
3056         (DOM::HTMLFormElementImpl::removeImgElement):
3057         * khtml/html/html_formimpl.h:
3058         * khtml/html/html_miscimpl.cpp:
3059         (HTMLFormCollectionImpl::FormCollectionInfo::FormCollectionInfo):
3060         (void::HTMLFormCollectionImpl::FormCollectionInfo::reset):
3061         (HTMLFormCollectionImpl::resetCollectionInfo):
3062         (HTMLFormCollectionImpl::calcLength):
3063         (HTMLFormCollectionImpl::item):
3064         (HTMLFormCollectionImpl::getNamedItem):
3065         (HTMLFormCollectionImpl::getNamedFormItem):
3066         (HTMLFormCollectionImpl::firstItem):
3067         (HTMLFormCollectionImpl::nextItem):
3068         * khtml/html/html_miscimpl.h:
3069         (DOM::HTMLFormCollectionImpl::~HTMLFormCollectionImpl):
3070         * khtml/xml/dom_elementimpl.cpp:
3071         (ElementImpl::setAttribute):
3072         (ElementImpl::setAttributeMap):
3073         * kwq/KWQPtrVector.h:
3074         (QPtrVector::findRef):
3075         * kwq/KWQVectorImpl.h:
3076         * kwq/KWQVectorImpl.mm:
3077         (KWQVectorImpl::findRef):
3078         * kwq/WebCoreBridge.mm:
3079         (-[WebCoreBridge elementWithName:inForm:]):
3080         (-[WebCoreBridge controlsInForm:]):
3081
3082 2004-11-19  David Harrison  <harrison@apple.com>
3083
3084         Reviewed by Ken and Darin.
3085
3086         <rdar://problem/3856215> Cannot remove bold from the beginning of a message
3087
3088         Problem is that KHTMLPart::computeAndSetTypingStyle always looked upstream
3089         for the existing style, but in this case (hitting cmd-B with caret at top of
3090         file) there is nothing upstream.  Changed this to use the VisiblePosition
3091         deepEquivalent instead.
3092         
3093         * khtml/khtml_part.cpp:
3094         (KHTMLPart::computeAndSetTypingStyle):
3095
3096 === Safari-172 ===
3097
3098 2004-11-19  Maciej Stachowiak  <mjs@apple.com>
3099
3100         Reviewed by Darin.
3101
3102         <rdar://problem/3864151> REGRESSION (125-167): Chrysler.com never stops loading
3103         
3104         * khtml/xml/dom_docimpl.cpp:
3105         (DocumentImpl::close): Don't fire the onload handler if there is a
3106         redirect pending. This is a very long-standing bug that was masked
3107         by our previously incorrect redirect logic. It used to be that an
3108         older redirect would always win. Recently we changed things so
3109         that a newer redirect would win, but a script that causes a
3110         redirect would stop parsing once complete (so if there are two
3111         redirects in the same script, the latter wins). However, we should
3112         have also prevented onload in this case. Testing with other
3113         browsers shows that onload handlers do not run at all when there
3114         is a pending redirect.
3115
3116 2004-11-19  Ken Kocienda  <kocienda@apple.com>
3117
3118         Reviewed by Harrison
3119
3120         Fix some object lifetime issues in these two commands. This fixes some crashes
3121         I am seeing in some new code I am working on, but have not yet reproduced otherwise.
3122
3123         * khtml/editing/htmlediting.cpp:
3124         (khtml::InsertParagraphSeparatorCommand::~InsertParagraphSeparatorCommand): No longer deref nodes
3125         in the ancestor list. They are not ref'ed when put on list. D'uh.
3126         (khtml::InsertParagraphSeparatorCommand::doApply): Ref all cloned nodes that are created by the command
3127         before putting them on the cloned nodes list. This are still deref'ed in the destructor.
3128         (khtml::InsertParagraphSeparatorInQuotedContentCommand::~InsertParagraphSeparatorInQuotedContentCommand): Ditto
3129         destructor comment.
3130         (khtml::InsertParagraphSeparatorInQuotedContentCommand::doApply): Ditto doApply comment.
3131
3132 2004-11-19  Ken Kocienda  <kocienda@apple.com>
3133
3134         Reviewed by Harrison
3135
3136         Fix for this bug:
3137         
3138         <rdar://problem/3655241> setTypingStyle: does not set the real typing style, and typingStyle does not return it
3139
3140         * khtml/khtml_part.cpp:
3141         (KHTMLPart::computeAndSetTypingStyle): New helper that does the work of reducing a passed-in style
3142         declaration given the current selection, and then sets the minimum necessary style as the typing
3143         style on the part.
3144         (KHTMLPart::applyStyle): Call new computeAndSetTypingStyle. The guts of computeAndSetTypingStyle used
3145         to be here in the selection-as-caret case. But now [WebCoreBridge setTypingStyle:] needs this code
3146         as well.
3147         * khtml/khtml_part.h: Declare new computeAndSetTypingStyle() function.
3148         * kwq/WebCoreBridge.h: Declare new typingStyle and setTypingStyle: methods.
3149         * kwq/WebCoreBridge.mm: 
3150         (-[WebCoreBridge typingStyle]): Calls through to the part to retrieve the typing style.
3151         (-[WebCoreBridge setTypingStyle:]): Calls through to the part to set the typing style.
3152
3153 2004-11-18  David Harrison  <harrison@apple.com>
3154
3155         Reviewed by Darin.
3156         
3157         Back out part of Darin's fix for <rdar://problem/3885729>, because the new exception gets triggered
3158         by Mail.app. Filed <rdar://problem/3886832> against Mail.app.
3159
3160         * kwq/DOM-CSS.mm:
3161         (-[DOMCSSStyleDeclaration setProperty:::]):
3162
3163 2004-11-18  Chris Blumenberg  <cblu@apple.com>
3164
3165         Fixed: <rdar://problem/3587481> Bug Reporter Login Page: Password AutoFill does not work reliably
3166
3167         Reviewed by john.
3168
3169         * kwq/KWQKHTMLPart.mm:
3170         (KWQKHTMLPart::currentForm): just return the current form, don't scan the entire document looking for a form if there is no current form
3171
3172 2004-11-18  Maciej Stachowiak  <mjs@apple.com>
3173
3174         Reviewed by Chris.
3175
3176         - fix recursive item traversal, use traverseNextNode() instead of
3177         the buggy hand-rolled traversal.
3178
3179         * khtml/xml/dom_nodeimpl.cpp:
3180         (NodeListImpl::recursiveItem):
3181
3182 2004-11-17  Darin Adler  <darin@apple.com>
3183
3184         Reviewed by Ken.
3185
3186         - fixed <rdar://problem/3885744> crash with XMLHttpRequest test page (reported by KDE folks)
3187
3188         * khtml/ecma/xmlhttprequest.cpp: (KJS::XMLHttpRequest::slotFinished):
3189         Rolled in fix from KDE; make sure to set job to 0 before calling changeState.
3190
3191         - fixed <rdar://problem/3885729> attempting to modify a computed style does nothing, but should raise a DOM exception
3192         - fixed <rdar://problem/3885731> style declarations use too many malloc blocks; switch to QValueList
3193         - fixed <rdar://problem/3885739> DOM::NodeImpl accessor in DOM::Node class is hot; should be inlined
3194         - changed NodeImpl calls like replaceChild to always ref/deref the parameter; this is a better way to fix
3195           an entire category of leaks we have been fixing one by one recently
3196         - changed computed styles so they hold a reference to the DOM node; the old code could end up with a
3197           stale RenderObject pointer, although I never saw it do that in practice
3198         - implemented the length and item methods for computed styles
3199         - implemented querying additional properties in computed styles (29 more)
3200
3201         * khtml/khtml_part.h: Update forward declaration of CSSMutableStyleDeclarationImpl since it's
3202         now a separate class rather than a typedef. Changed the parameter type of setTypingStyle to
3203         take a mutable style.
3204         * khtml/khtml_part.cpp:
3205         (KHTMLPart::setTypingStyle): Change parameter to take a mutable style.
3206         (KHTMLPart::applyStyle): Add code to make a mutable style in case we are passed
3207         a computed style; also change some types to mutable style.
3208         (updateState): Update iteration of CSSProperty objects in a style declaration to use
3209         the new valuesIterator interface.
3210         (KHTMLPart::selectionHasStyle): Add a call to makeMutable.
3211         (KHTMLPart::selectionStartHasStyle): Add call to makeMutable and update iteration.
3212         (editingStyle): Change type to mutable style, and simplify the style-creation calls,
3213         including accomodating the exception code that setCssText has now.
3214         (KHTMLPart::applyEditingStyleToElement): Change types to mutable style.
3215         (KHTMLPart::removeEditingStyleFromElement): Change code to call setChanged only if removing
3216         the style attributes really was a change, although it's not an important optimization it's
3217         good to do it right.
3218
3219         * khtml/css/css_base.h: Remove unneeded setParsedValue method.
3220         * khtml/css/css_base.cpp: Remove unneeded setParsedValue method. All the places that were
3221         calling it were already removing the old property explicitly, so the code in here to remove
3222         the property again was redundant.
3223
3224         * khtml/css/css_computedstyle.h: Updated virtual functions for changes to parameters in base class.
3225         Moved all the "set"-type functions so they are private. Store a node pointer instead of a renderer.
3226         * khtml/css/css_computedstyle.cpp:
3227         (DOM::CSSComputedStyleDeclarationImpl::CSSComputedStyleDeclarationImpl): Hold a reference to
3228         the node we compute style for, so we don't end up with a pointer to a deallocated RenderObject.
3229         Before we had no guarantee the object would outlast us.
3230         (DOM::CSSComputedStyleDeclarationImpl::setCssText): Add exception parameter, and set the
3231         exception to NO_MODIFICATION_ALLOWED_ERR.
3232         (DOM::CSSComputedStyleDeclarationImpl::getPropertyCSSValue): Update to use node pointer rather
3233         than renderer pointer. Added implementation for box-align, box-direction, box-flex, box-flex-group,
3234         box-lines, box-ordinal-group, box-orient, box-pack, caption-side, clear, cursor, direction,
3235         list-style-image, list-style-position, list-style-type, marquee-direction, marquee-repetition,
3236         marquee-style, user-modify, opacity, orphans, outline-style, page-break-after, page-break-before,
3237         page-break-inside, position, unicode-bidi, widows, z-index.
3238         (DOM::CSSComputedStyleDeclarationImpl::removeProperty): Add exception parameter, and set the
3239         exception to NO_MODIFICATION_ALLOWED_ERR.
3240         (DOM::CSSComputedStyleDeclarationImpl::setProperty): Ditto.
3241         (DOM::CSSComputedStyleDeclarationImpl::length): Implemented.
3242         (DOM::CSSComputedStyleDeclarationImpl::item): Implemented, calls getPropertyValue.
3243         (DOM::CSSComputedStyleDeclarationImpl::copyInheritableProperties): Changed return type to
3244         CSSMutableStyleDeclarationImpl.
3245         (DOM::CSSComputedStyleDeclarationImpl::copy): Added.
3246         (DOM::CSSComputedStyleDeclarationImpl::makeMutable): Added.
3247
3248         * khtml/css/css_ruleimpl.h: Update forward declaration of CSSMutableStyleDeclarationImpl since it's
3249         now a separate class rather than a typedef.
3250         * khtml/css/cssparser.h: Ditto.
3251
3252         * khtml/css/css_valueimpl.h: Refactor CSSStyleDeclarationImpl into two classes. New derived class
3253         CSSMutableStyleDeclarationImpl has the guts, and the base class has only some virtual functions.
3254         Removed a bunch of redundant stuff from other classes in this file too.
3255         (DOM::DashboardRegionImpl::setNext): Ref new before deref'ing old to handle the set-to-same case.
3256         (DOM::CSSProperty::CSSProperty): Added new overload so you can create a CSSProperty with initial values.
3257         (DOM::CSSProperty::operator=): Added.
3258         (DOM::CSSProperty::setValue): Use ref-before-deref pattern to simplify slightly.
3259
3260         * khtml/css/css_valueimpl.cpp:
3261         (DOM::CSSStyleDeclarationImpl::CSSStyleDeclarationImpl): Remove uneeded things.
3262         (DOM::CSSStyleDeclarationImpl::isStyleDeclaration): Put here now that it's no longer inline.
3263         (DOM::CSSMutableStyleDeclarationImpl::CSSMutableStyleDeclarationImpl): Added.
3264         (DOM::CSSMutableStyleDeclarationImpl::operator=): Added.
3265         (DOM::CSSMutableStyleDeclarationImpl::~CSSMutableStyleDeclarationImpl): Updated.
3266         (DOM::CSSMutableStyleDeclarationImpl::getPropertyValue): Removed now-uneeded check.
3267         (DOM::CSSMutableStyleDeclarationImpl::get4Values): Moved here from base class.
3268         (DOM::CSSMutableStyleDeclarationImpl::getShortHandValue): Ditto.
3269         (DOM::CSSMutableStyleDeclarationImpl::getPropertyCSSValue): Update to use QValueList instead of QPtrList.
3270         (DOM::CSSMutableStyleDeclarationImpl::removeProperty): Added exception parameter, updated for QValueList.
3271         (DOM::CSSMutableStyleDeclarationImpl::setChanged): Moved here from base class.
3272         (DOM::CSSMutableStyleDeclarationImpl::getPropertyPriority): Update to use QValueList.
3273         (DOM::CSSMutableStyleDeclarationImpl::setProperty): Added more overloads to match new parameters.
3274         (DOM::CSSMutableStyleDeclarationImpl::setStringProperty): Update to use QValueList.
3275         (DOM::CSSMutableStyleDeclarationImpl::setImageProperty): Ditto.
3276         (DOM::CSSMutableStyleDeclarationImpl::parseProperty): Remove unneeded initialization code due to QValueList.
3277         (DOM::CSSMutableStyleDeclarationImpl::addParsedProperties): Added.
3278         (DOM::CSSMutableStyleDeclarationImpl::setLengthProperty): Moved here from base class.
3279         (DOM::CSSMutableStyleDeclarationImpl::length): Update to use QValueList.
3280         (DOM::CSSMutableStyleDeclarationImpl::item): Moved here from base class.
3281         (DOM::CSSMutableStyleDeclarationImpl::cssText): Return empty string rather than null string when there are
3282         no styles in the list. Update to use QValueList.
3283         (DOM::CSSMutableStyleDeclarationImpl::setCssText): Update to use QValueList and to take an exceptionCode
3284         parameter and set it.
3285         (DOM::CSSMutableStyleDeclarationImpl::merge): Update to use QValueList.
3286         (DOM::CSSStyleDeclarationImpl::diff): Update to use QValueList.
3287         (DOM::CSSMutableStyleDeclarationImpl::copyBlockProperties): Moved here from base class. Change return type.
3288         (DOM::CSSStyleDeclarationImpl::copyPropertiesInSet): Update to use QValueList and use stack, not new/delete.
3289         (DOM::CSSMutableStyleDeclarationImpl::makeMutable): Added.
3290         (DOM::CSSMutableStyleDeclarationImpl::copy): Added.
3291
3292         * khtml/css/cssparser.cpp:
3293         (CSSParser::parseValue): Changed to use addParsedProperties.
3294         (CSSParser::parseDeclaration): Ditto.
3295         (CSSParser::createStyleDeclaration): Use new constructor to create declaration in a more efficient manner.
3296
3297         * khtml/css/cssproperties.in: Removed unused font-size-adjust and -khtml-flow-mode.
3298         * khtml/css/cssproperties.c: Regenerated.
3299         * khtml/css/cssproperties.h: Regenerated.
3300
3301         * khtml/css/cssstyleselector.cpp: (khtml::CSSStyleSelector::applyDeclarations):
3302         Updated to use QValueList interface to CSSMutableStyleDeclarationImpl.
3303
3304         * khtml/dom/css_value.cpp:
3305         (DOM::CSSStyleDeclaration::cssText): Removed unneeded cast.
3306         (DOM::CSSStyleDeclaration::setCssText): Added exception code handling.
3307         (DOM::CSSStyleDeclaration::getPropertyValue): Changed to call getPropertyValue directly instead of
3308         first doing getPropertyCSSValue and then doing cssText.
3309         (DOM::CSSStyleDeclaration::getPropertyCSSValue): Removed unneeded cast.
3310         (DOM::CSSStyleDeclaration::removeProperty): Added exception code handling. 
3311         (DOM::CSSStyleDeclaration::setProperty): Added exception code handling.
3312         (DOM::CSSStyleDeclaration::length): Removed unneeded cast.