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