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