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