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