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