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