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