Context Navigation

← Previous Period
Next Period →

Timeline

Nov 11, 2003:

8:15 PM Changeset in webkit [5460] by vicki

2 edits in branches/Safari-1-1-Merlot-BuildFixes-branch/WebKit

rolled this fix onto the branch

2003-10-16 Chris Blumenberg <cblu@apple.com>

Fixed: <rdar://problem/3455176>: Assertion failure when loading non-existant plug-in content

Reviewed by rjw.

Misc.subproj/WebDownload.m: fixed build failure when using new Foundation. We are overriding and calling a renamed method. Continue to override the old method, override the new method and declare their interfaces to avoid build failures.

5:30 PM Changeset in webkit [5459] by mjs

2 edits in trunk/WebCore

Reviewed by John.

fixed 3479537 - reproducible crash in KJS::WindowFunc::tryCall

khtml/ecma/kjs_window.cpp: (WindowFunc::tryCall): Check part and doc impl for null.

3:16 PM Changeset in webkit [5458] by mjs

2 edits in trunk/WebCore

Reviewed by Ken.

fixed 3472330 - REGRESSION: onKeyPress handler not called when hitting keys in <input type=password> field

kwq/KWQTextField.mm: (-[KWQSecureTextField initWithQLineEdit:]): New initializer, so the secure text field knows about the widget. (-[KWQTextField setPasswordMode:]): Use new initializer for KWQSecureTextField. (-[KWQSecureTextField textView:shouldHandleEvent:]): New method, do the same thing as the non-secure text field to pass the event on. (-[KWQSecureTextField textView:didHandleEvent:]): Ditto.

1:58 PM Changeset in webkit [5457] by kocienda

8 edits in trunk/WebCore

Reviewed by Maciej

Fix for this bug:

<rdar://problem/3480173>: Too many names for programmatic click functions

Changed all instances of performClick() and simulateClick() to click().
Changed all instances of performSubmitClick() to submitClick().

Additionally, KWQButton had a simulateClick method, which was redundant
given that its superclass has a performClick: method.

khtml/html/html_formimpl.cpp: (HTMLFormElementImpl::submitClick): (HTMLInputElementImpl::defaultEventHandler): (HTMLSelectElementImpl::defaultEventHandler):
khtml/html/html_formimpl.h:
khtml/html/html_inlineimpl.cpp: (HTMLAnchorElementImpl::defaultEventHandler): (HTMLAnchorElementImpl::click):
khtml/html/html_inlineimpl.h:
khtml/rendering/render_form.cpp: (RenderLineEdit::slotReturnPressed):
kwq/KWQButton.h:
kwq/KWQButton.mm: (QButton::click):

1:51 PM Changeset in webkit [5456] by mjs

2 edits in trunk/WebCore

Reviewed by John.

fixed 3475272 - custom property on CSSStyleObject won't hold a value

khtml/ecma/kjs_css.cpp: (DOMCSSStyleDeclaration::tryPut): If the property is not a css property, save it as a custom object property in the normal way.

1:06 PM Changeset in webkit [5455] by sullivan

2 edits in trunk/WebCore

fixed <rdar://problem/3480421>: Focus rings around links are too thick, should use 3 px instead of 5 px

Reviewed by Don and Ken.

khtml/css/html4.css: use 3px instead of 5 px for :active, :focus

12:59 PM Changeset in webkit [5454] by kocienda

9 edits in trunk/WebCore

Reviewed by John

In the DOM Level 3 Events specification, keydown and keyup are
standard events. We can remove the KHTML component in the names
of these constants. Note that the keypress event is still
nonstandard and will retain the KHTML name component.

khtml/ecma/kjs_binding.cpp: (ScriptInterpreter::wasRunByUserGesture)
khtml/ecma/kjs_dom.cpp: (DOMNode::getValueProperty) (DOMNode::putValue)
khtml/ecma/kjs_window.cpp: (Window::get) (Window::put)
khtml/html/html_elementimpl.cpp: (HTMLElementImpl::parseAttribute)
khtml/html/html_formimpl.cpp: (HTMLGenericFormElementImpl::defaultEventHandler)
khtml/html/html_inlineimpl.cpp: (HTMLAnchorElementImpl::defaultEventHandler)
khtml/xml/dom2_eventsimpl.cpp: (EventImpl::typeToId) (EventImpl::idToType) (KeyboardEventImpl::KeyboardEventImpl)
khtml/xml/dom2_eventsimpl.h: (DOM::EventImpl::)

11:30 AM Changeset in webkit [5453] by cblu

2 edits in trunk/WebCore

Fixed: <rdar://problem/3480281>: drag image of selected text and images include broken images

Reviewed by dave.

khtml/rendering/render_image.cpp: (RenderImage::paintObject): when drawing the drag image, don't draw the broken image

10:34 AM Changeset in webkit [5452] by vicki

2 edits in trunk/WebCore

Reviewed by sullivan.

rolled Merlot build fix onto the trunk

WebCorePrefix.h: added #include <sys/types.h>

Nov 10, 2003:

5:12 PM Changeset in webkit [5451] by hyatt

2 edits in trunk/WebCore

Fix for 3473531, text nested in hr inside h3 did not get the h3's style.

Reviewed by darin

khtml/html/dtd.cpp: (DOM::checkChild):

5:03 PM Changeset in webkit [5450] by hyatt

110 edits in trunk

Fix for 3478435, <hr> tags should be centered by default. Ditched the 1px margin as well.

Fix for 3479441, <h1>-<h6> tags should *not* be centered by default. They should also use bold instead of
bolder.

Reviewed by darin

khtml/css/html4.css:
khtml/html/html_blockimpl.cpp: (HTMLHRElementImpl::parseAttribute):
layout-tests/apple-only/base/bugzilla.mozilla.org/index-expected.txt:
layout-tests/apple-only/base/home.netscape.com/index-expected.txt:
layout-tests/apple-only/base/www.altavista.com/index-expected.txt:
layout-tests/apple-only/base/www.cnn.com/index-expected.txt:
layout-tests/apple-only/base/www.digitalcity.com/index-expected.txt:
layout-tests/apple-only/base/www.ebay.com/index-expected.txt:
layout-tests/apple-only/base/www.excite.com/index-expected.txt:
layout-tests/apple-only/base/www.mapquest.com/index-expected.txt:
layout-tests/apple-only/base/www.microsoft.com/index-expected.txt:
layout-tests/apple-only/base/www.msnbc.com/index-expected.txt:
layout-tests/apple-only/base/www.nytimes.com/index-expected.txt:
layout-tests/apple-only/base/www.nytimes.com_Table/index-expected.txt:
layout-tests/apple-only/base/www.quicken.com/index-expected.txt:
layout-tests/apple-only/base/www.w3.org_DOML2Core/index-expected.txt:
layout-tests/css1/basic/class_as_selector-expected.txt:
layout-tests/css1/basic/comments-expected.txt:
layout-tests/css1/basic/containment-expected.txt:
layout-tests/css1/basic/contextual_selectors-expected.txt:
layout-tests/css1/basic/grouping-expected.txt:
layout-tests/css1/basic/id_as_selector-expected.txt:
layout-tests/css1/basic/inheritance-expected.txt:
layout-tests/css1/box_properties/border-expected.txt:
layout-tests/css1/box_properties/border_bottom-expected.txt:
layout-tests/css1/box_properties/border_bottom_inline-expected.txt:
layout-tests/css1/box_properties/border_bottom_width-expected.txt:
layout-tests/css1/box_properties/border_bottom_width_inline-expected.txt:
layout-tests/css1/box_properties/border_color-expected.txt:
layout-tests/css1/box_properties/border_color_inline-expected.txt:
layout-tests/css1/box_properties/border_inline-expected.txt:
layout-tests/css1/box_properties/border_left-expected.txt:
layout-tests/css1/box_properties/border_left_inline-expected.txt:
layout-tests/css1/box_properties/border_left_width-expected.txt:
layout-tests/css1/box_properties/border_left_width_inline-expected.txt:
layout-tests/css1/box_properties/border_right-expected.txt:
layout-tests/css1/box_properties/border_right_inline-expected.txt:
layout-tests/css1/box_properties/border_right_width-expected.txt:
layout-tests/css1/box_properties/border_right_width_inline-expected.txt:
layout-tests/css1/box_properties/border_style-expected.txt:
layout-tests/css1/box_properties/border_style_inline-expected.txt:
layout-tests/css1/box_properties/border_top-expected.txt:
layout-tests/css1/box_properties/border_top_inline-expected.txt:
layout-tests/css1/box_properties/border_top_width-expected.txt:
layout-tests/css1/box_properties/border_top_width_inline-expected.txt:
layout-tests/css1/box_properties/border_width-expected.txt:
layout-tests/css1/box_properties/border_width_inline-expected.txt:
layout-tests/css1/box_properties/clear-expected.txt:
layout-tests/css1/box_properties/clear_float-expected.txt:
layout-tests/css1/box_properties/float-expected.txt:
layout-tests/css1/box_properties/float_elements_in_series-expected.txt:
layout-tests/css1/box_properties/float_margin-expected.txt:
layout-tests/css1/box_properties/float_on_text_elements-expected.txt:
layout-tests/css1/box_properties/height-expected.txt:
layout-tests/css1/box_properties/margin-expected.txt:
layout-tests/css1/box_properties/margin_bottom-expected.txt:
layout-tests/css1/box_properties/margin_bottom_inline-expected.txt:
layout-tests/css1/box_properties/margin_inline-expected.txt:
layout-tests/css1/box_properties/margin_left-expected.txt:
layout-tests/css1/box_properties/margin_left_inline-expected.txt:
layout-tests/css1/box_properties/margin_right-expected.txt:
layout-tests/css1/box_properties/margin_right_inline-expected.txt:
layout-tests/css1/box_properties/margin_top-expected.txt:
layout-tests/css1/box_properties/margin_top_inline-expected.txt:
layout-tests/css1/box_properties/padding-expected.txt:
layout-tests/css1/box_properties/padding_bottom-expected.txt:
layout-tests/css1/box_properties/padding_bottom_inline-expected.txt:
layout-tests/css1/box_properties/padding_inline-expected.txt:
layout-tests/css1/box_properties/padding_left-expected.txt:
layout-tests/css1/box_properties/padding_left_inline-expected.txt:
layout-tests/css1/box_properties/padding_right-expected.txt:
layout-tests/css1/box_properties/padding_right_inline-expected.txt:
layout-tests/css1/box_properties/padding_top-expected.txt:
layout-tests/css1/box_properties/padding_top_inline-expected.txt:
layout-tests/css1/box_properties/width-expected.txt:
layout-tests/css1/cascade/cascade_order-expected.txt:
layout-tests/css1/cascade/important-expected.txt:
layout-tests/css1/classification/display-expected.txt:
layout-tests/css1/classification/list_style-expected.txt:
layout-tests/css1/classification/list_style_image-expected.txt:
layout-tests/css1/classification/list_style_position-expected.txt:
layout-tests/css1/classification/list_style_type-expected.txt:
layout-tests/css1/classification/white_space-expected.txt:
layout-tests/css1/color_and_background/background-expected.txt:
layout-tests/css1/color_and_background/background_attachment-expected.txt:
layout-tests/css1/color_and_background/background_color-expected.txt:
layout-tests/css1/color_and_background/background_image-expected.txt:
layout-tests/css1/color_and_background/background_position-expected.txt:
layout-tests/css1/color_and_background/background_repeat-expected.txt:
layout-tests/css1/color_and_background/color-expected.txt:
layout-tests/css1/conformance/forward_compatible_parsing-expected.txt:
layout-tests/css1/font_properties/font-expected.txt:
layout-tests/css1/font_properties/font_family-expected.txt:
layout-tests/css1/font_properties/font_size-expected.txt:
layout-tests/css1/font_properties/font_style-expected.txt:
layout-tests/css1/font_properties/font_variant-expected.txt:
layout-tests/css1/font_properties/font_weight-expected.txt:
layout-tests/css1/formatting_model/canvas-expected.txt:
layout-tests/css1/formatting_model/floating_elements-expected.txt:
layout-tests/css1/formatting_model/height_of_lines-expected.txt:
layout-tests/css1/formatting_model/inline_elements-expected.txt:
layout-tests/css1/formatting_model/replaced_elements-expected.txt:
layout-tests/css1/formatting_model/vertical_formatting-expected.txt:
layout-tests/css1/pseudo/anchor-expected.txt:
layout-tests/css1/pseudo/firstletter-expected.txt:
layout-tests/css1/pseudo/firstline-expected.txt:
layout-tests/css1/pseudo/multiple_pseudo_elements-expected.txt:
layout-tests/css1/pseudo/pseudo_elements_in_selectors-expected.txt:
layout-tests/css1/text_properties/letter_spacing-expected.txt:
layout-tests/css1/text_properties/line_height-expected.txt:
layout-tests/css1/text_properties/text-transform-expected.txt:
layout-tests/css1/text_properties/text_align-expected.txt:
layout-tests/css1/text_properties/text_decoration-expected.txt:
layout-tests/css1/text_properties/text_indent-expected.txt:
layout-tests/css1/text_properties/text_transform-expected.txt:
layout-tests/css1/text_properties/vertical_align-expected.txt:
layout-tests/css1/text_properties/word_spacing-expected.txt:
layout-tests/css1/units/color_units-expected.txt:
layout-tests/css1/units/length_units-expected.txt:
layout-tests/css1/units/percentage_units-expected.txt:
layout-tests/css1/units/urls-expected.txt:
layout-tests/fast/encoding/utf-16-big-endian-expected.txt:
layout-tests/fast/encoding/utf-16-little-endian-expected.txt:

4:00 PM Changeset in webkit [5449]

1 copy
326 deletes in branches/unlabeled-1.56.4

This commit was manufactured by cvs2svn to create branch
'unlabeled-1.56.4'.

4:00 PM Changeset in webkit [5448] by cblu

5 edits in trunk/WebCore

Fixed: <rdar://problem/3479266>: selected images don't obey CSS3 selection background-color attribute

Reviewed by dave.

khtml/rendering/render_image.cpp: (RenderImage::selectionTintColor): new, takes style into account (RenderImage::paintObject): use selectionTintColor
khtml/rendering/render_image.h:
kwq/KWQPainter.h: removed selectedImageTintColor
kwq/KWQPainter.mm:

3:52 PM Changeset in webkit [5447] by hyatt

9 edits in trunk/WebCore

Fix for 3479287, macro-ize the handling of the CSS 'inherit' value. This patch is preparation for
adding support for the 'initial' value, which will be used to fix the CSS parser's shorthand parsing
bugs.

Reviewed by kocienda

khtml/css/cssparser.cpp: (CSSParser::parseValue):
khtml/css/cssproperties.c: (hash_prop): (findProp):
khtml/css/cssproperties.h:
khtml/css/cssproperties.in:
khtml/css/cssstyleselector.cpp: (khtml::CSSStyleSelector::applyRule):
khtml/khtmlview.cpp: (KHTMLView::layout):

3:21 PM Changeset in webkit [5446] by cblu

5 edits in trunk/WebCore

Fixed:
<rdar://problem/3478883>: REGRESSION (113-114u): drag image for selected text includes unselected images
<rdar://problem/3479204>: selected images don't look selected

Reviewed by dave.

khtml/rendering/render_image.cpp: (RenderImage::RenderImage): init m_selectionState (RenderImage::paintObject): draw a tint over the image if selected, don't draw anything if not selected and the action is PaintActionSelection
khtml/rendering/render_image.h: (khtml::RenderImage::selectionState): new (khtml::RenderImage::setSelectionState): new
kwq/KWQPainter.h:
kwq/KWQPainter.mm: (QPainter::selectedImageTintColor): new

2:41 PM Changeset in webkit [5445] by kocienda

2 edits in trunk/WebCore

Reviewed by Hyatt

Fix for this bug:

<rdar://problem/3477795>: REGRESSION: can't submit form without
submit button by hitting return (cocoadev.com)

When I changed submitting forms using the return key in a text field
to go through a search for the correct submit or image input
element to use when submitting, I neglected to cover the case of
forms which have neither a submit or image input element. When
I encounter a form such as this now, I submit it anyway after
performing the submit or image input element and failing to find one.

khtml/html/html_formimpl.cpp: (HTMLFormElementImpl::performSubmitClick)

1:54 PM Changeset in webkit [5444] by rjw

2 edits in trunk

Forget to change NOBODY to Darin.

1:51 PM Changeset in webkit [5443] by rjw

12 edits in trunk

WebKit:

Fixed 3478765. Use ICU to access unicode properties.
Fixed 3478831. Unicode property/conversion functions should be 32 bit savvy.
Fixed 3478885. Remove dead arabic shaping code

Reviewed by NOBODY (OOPS!).

Misc.subproj/WebUnicode.h:
Misc.subproj/WebUnicode.m: (_unicodeDigitValue): (_unicodeDirection): (_unicodeMirrored): (_unicodeMirroredChar): (_unicodeLower): (_unicodeUpper): (WebKitInitializeUnicode): (shapeForNextCharacter): (initializeCharacterShapeIterator):
Misc.subproj/WebUnicodeTables.m:
WebCoreSupport.subproj/WebTextRenderer.m: (widthForNextCharacter): (fontContainsString):

WebCore:

Fixed 3478765. Use ICU to access unicode properties.
Fixed 3478831. Unicode property/conversion functions should be 32 bit savvy.

Reviewed by NOBODY (OOPS!).

WebCore-combined.exp:
WebCore.exp:
kwq/KWQChar.mm: (QChar::lower): (QChar::upper): (QChar::mirroredChar):
kwq/KWQString.h: (QChar::direction):
kwq/WebCoreUnicode.cpp:
kwq/WebCoreUnicode.h:

12:45 PM Changeset in webkit [5442] by rjw

1 add in trunk/LayoutTests/css1/text_properties/text-transform-expected.txt

Added expected results for text-transform test.

12:42 PM Changeset in webkit [5441] by rjw

1 edit in trunk/LayoutTests/css1/text_properties/text-transform.html

Removed Turkish and Deseret from the test case. These don't work yet.

11:58 AM Changeset in webkit [5440] by cblu

2 edits in trunk/WebKit

Fixed: <rdar://problem/3478351>: Safari: URL Alias on Dock failed to open the 2byte URL

Reviewed by dave.

Misc.subproj/WebNSURLExtras.m: (-[NSString _web_mapHostNameWithRange:encode:makeString:]): if the host name is percent-escaped, use CFURLCreateStringByReplacingPercentEscapes

10:49 AM Changeset in webkit [5439] by cblu

3 edits in trunk/WebCore

Fixed: <rdar://problem/3478781>: drag image of selected text and images doesn't include images

Reviewed by rjw.

khtml/rendering/render_image.cpp: (RenderImage::paintObject): paint when the paint action is PaintActionSelection
khtml/rendering/render_replaced.cpp: (RenderReplaced::paint): ditto

10:46 AM Changeset in webkit [5438] by kocienda

4 edits in trunk/WebCore

Reviewed by John

Fix for this bug:

<rdar://problem/3477624>: REGRESSION (113): Using input method to
enter non-ascii text submits the form

This regression has been in the tree for a couple of versions now,
since we improved the way key events are processed. The solution
is to ask the text input manager if it has any marked text when
the return key (or enter key) is pressed, because if it does, the key
needs to work in the "accept" role for the input manager. Fixing
in this way has the pleasant effect of making return/enter key
behavior "do the right thing" based on context.

khtml/html/html_formimpl.cpp: (HTMLInputElementImpl::defaultEventHandler): Add check to see if text input manager has marked text before submitting text and password fields. Broke out key handling for checkboxes and radios, since the old code for these widgets worked fine.
kwq/KWQLineEdit.h: Added hasMarkedText function.
kwq/KWQLineEdit.mm: (QLineEdit::hasMarkedText): Returns whether the current input manager has marked text or not.

10:00 AM Changeset in webkit [5437] by rjw

2 edits in trunk/WebKit

Use ICU for upper/lower conversion. Fixed 3477157, 3478455, 3478456, 3478457, 3478486.
Remaining issues with surrogates (3477159) and Turkish I (3478482).

Reviewed by Ken.

Misc.subproj/WebUnicode.m: (_unicodeLower): (_unicodeUpper):

8:31 AM Changeset in webkit [5436] by darin

2 edits in trunk/JavaScriptCore

Reviewed by John.

fixed 3477528 -- array.sort(function) fails if the function returns a non-zero value that rounds to zero

kjs/array_object.cpp: (compareByStringForQSort): Added checks for undefined values to match what the specification calls for. (compareWithCompareFunctionForQSort): Added checks for undefined values as above, and also changed the code that looks at the compare function result to look at the number returned without rounding to an integer. (ArrayProtoFuncImp::call): Changed the code that looks at the compare function result to look at the number returned without rounding to an integer.

12:26 AM Changeset in webkit [5435] by darin

9 edits in trunk/WebCore

Reviewed by Dave.

fixed 3478173 -- REGRESSION (112-113): getElementById is failing for many types of elements

The problem was that ElementImpl::attach was being skipped for many objects with classes
derived from ElementImpl. By changing them all to call up to their base classes instead of
skipping levels, the bug went away. This was possible (and easy to do) because of the
rendererIsNeeded() change I made a while back.

khtml/html/html_baseimpl.cpp: (HTMLFrameElementImpl::attach): Call base class's attach() instead of creating renderer explicitly and then calling NodeBaseImpl::attach(). This was a remnant of the old way of changing the logic about when to create the renderer, not needed now that we have rendererIsNeeded(). (HTMLFrameSetElementImpl::attach): Ditto. (HTMLIFrameElementImpl::attach): Ditto.

khtml/html/html_formimpl.h: Removed many unused attach() overrides.
khtml/html/html_formimpl.cpp: (HTMLGenericFormElementImpl::attach): Ditto. (HTMLButtonElementImpl::attach): Removed, because now the base class's attach does the right thing. (HTMLFieldSetElementImpl::attach): Ditto. (HTMLInputElementImpl::attach): Removed explicit createRendererIfNeeded call, not needed now that HTMLGenericFormElementImpl calls through to its base class. (HTMLLegendElementImpl::attach): Removed, because now the base class's attach does the right thing. (HTMLSelectElementImpl::attach): Ditto. (HTMLTextAreaElementImpl::attach): Ditto.

khtml/html/html_imageimpl.cpp: (HTMLImageElementImpl::attach): Call base class's attach() instead of creating renderer explicitly and then calling NodeBaseImpl::attach(). (HTMLImageElementImpl::detach): Call base class's detach() instead of NodeBaseImpl::detach(). No need to skip the ElementImpl::detach() function, and important not to for the new "id" optimization.

khtml/html/html_inlineimpl.h: Removed HTMLBRElementImpl::attach override.
khtml/html/html_inlineimpl.cpp: (HTMLBRElementImpl::attach): Call base class's attach() instead of creating renderer explicitly and then calling NodeBaseImpl::attach().

khtml/html/html_objectimpl.h: Removed HTMLAppletElementImpl::attach override.
khtml/html/html_objectimpl.cpp: (HTMLAppletElementImpl::attach): Removed, because the base class's attach does the right thing. (HTMLEmbedElementImpl::attach): Call base class's attach() instead of creating renderer explicitly and then calling NodeBaseImpl::attach(). (HTMLObjectElementImpl::attach): Ditto.

Nov 9, 2003:

3:43 PM Changeset in webkit [5434] by hyatt

2 adds in trunk/LayoutTests/fast/text/basic

Add layout test for letter/word spacing bug.

3:41 PM Changeset in webkit [5433] by hyatt

3 edits in trunk/WebCore

Fix for 3478317, letter/word-spacing don't inherit properly through italic/bold elements.

Reviewed by darin

khtml/rendering/font.h: (khtml::Font::Font):
khtml/rendering/render_style.h: (khtml::RenderStyle::setFontDef):

2:30 PM Changeset in webkit [5432] by hyatt

2 edits in trunk/WebCore

Fix for 3478309, make sure backgrounds paint in the border box and not just in the padding box.

Reviewed by darin

Also making sure that <frame>s aren't transparent, since other browsers seem to only make <iframe>s/<object>s
be transparent.

khtml/rendering/render_box.cpp: (RenderBox::paintRootBoxDecorations): (RenderBox::paintBackgroundExtended):

Nov 8, 2003:

9:10 PM Changeset in webkit [5431] by darin

2 edits in trunk/WebCore

Reviewed by John.

fixed 3478025 -- links with non-Latin-1 characters in the hostname are not getting IDN-encoded

kwq/KWQKURL.mm: (KURL::parse): Changed existing code to take advantage of the fact that a NUL character is classified as a path segment end character. (KURL::findHostnameInHierarchicalURL): Don't treat NUL characters as terminating characters for hostnames, because QChar::latin1() turns all non-Latin-1 characters into NUL characters.

3:24 PM Changeset in webkit [5430] by darin

2 edits in trunk/WebCore

Reviewed by John.

fixed 3477509 -- REGRESSION (112-113): crash in KHTMLPart::processObjectRequest at espn.com, chosun.com

khtml/khtml_part.cpp: (KHTMLPart::processObjectRequest): Use a dynamic_cast to check the type of the part rather than assuming it's always a KHTMLPart. The crash was caused by casting a KWQPluginPart to KHTMLPart.

9:40 AM Changeset in webkit [5429] by cblu

2 edits in trunk/WebKit

Fixed: <rdar://problem/3240778>: add "save" menu item to contextual menu for text pages

Reviewed by darin.

WebView.subproj/WebTextView.m: (-[WebTextView menuForEvent:]): Made WebTextView's context menu behavior like WebHTMLView's context menu behavior with regards to selection. If the control-click was on a selection, show menu options for the selection like copy. If it was not on a selection, show menu options such as save and print. Don't select anything when control-clicking.

9:40 AM Changeset in webkit [5428] by cblu

2 edits in trunk/WebKit

Fixed: <rdar://problem/3478022>: assertion failure while loading WMP content

Reviewed by darin.

ChangeLog:
Plugins.subproj/WebNetscapePluginStream.m: (-[WebNetscapePluginConnectionDelegate connection:didReceiveResponse:]): Don't continue if the stream is cancelled in startStreamWithResponse.

9:39 AM Changeset in webkit [5427] by darin

6 edits in trunk/WebCore

Reviewed by Richard.

event cleanup motivated by questions on khtml-devel

khtml/dom/dom2_events.h: Removed eventModuleName.
khtml/dom/dom2_events.cpp: Removed eventModuleName.

khtml/xml/dom2_eventsimpl.h: Removed eventModuleName, made many trivially simple functions inline, made virtual functions no longer inline. Also made some functions const. (DOM::EventImpl::propagationStopped): Made this non-virtual; there was no reason for it to be virtual I could see. (DOM::EventImpl::defaultPrevented): Ditto. (DOM::EventImpl::setDefaultHandled): Ditto. (DOM::UIEventImpl::view): Made this inline. (DOM::UIEventImpl::detail): Ditto. (DOM::MouseEventImpl::screenX): Ditto. (DOM::MouseEventImpl::screenY): Ditto. (DOM::MouseEventImpl::clientX): Ditto. (DOM::MouseEventImpl::clientY): Ditto. (DOM::MouseEventImpl::layerX): Ditto. (DOM::MouseEventImpl::layerY): Ditto. (DOM::MouseEventImpl::ctrlKey): Ditto. (DOM::MouseEventImpl::shiftKey): Ditto. (DOM::MouseEventImpl::altKey): Ditto. (DOM::MouseEventImpl::metaKey): Ditto. (DOM::MouseEventImpl::button): Ditto. (DOM::MouseEventImpl::relatedTarget): Ditto. (DOM::MutationEventImpl::relatedNode): Ditto. (DOM::MutationEventImpl::prevValue): Ditto. (DOM::MutationEventImpl::newValue): Ditto. (DOM::MutationEventImpl::attrName): Ditto. (DOM::MutationEventImpl::attrChange): Ditto.

khtml/ecma/kjs_events.cpp: (KJS::getDOMEvent): Changed implementation to no longer require eventModuleName(), and removed some unneeded casts.

khtml/xml/dom2_eventsimpl.cpp: Made lots of functions inline. (EventImpl::isUIEvent): Since this is virtual, make it no longer inline. (EventImpl::isMouseEvent): Ditto. (EventImpl::isMutationEvent): Ditto. (EventImpl::isKeyboardEvent): Ditto. (UIEventImpl::isUIEvent): Ditto. (MouseEventImpl::isMouseEvent): Ditto. (KeyboardEventImpl::isKeyboardEvent): Ditto. (MutationEventImpl::isMutationEvent): Ditto.

Nov 7, 2003:

5:08 PM Changeset in webkit [5426] by cblu

2 edits in trunk/WebCore

Fixed: <rdar://problem/3020196>: links with href="" don't show hand cursor when you mouse over them

Reviewed by darin.

khtml/khtmlview.cpp: (KHTMLView::viewportMouseMoveEvent): check if the URL on the event is non-NULL, not if it is non-empty

4:48 PM Changeset in webkit [5425]

9 copies
2 deletes in tags/Safari-100~0~1

This commit was manufactured by cvs2svn to create tag
'Safari-100~0~1'.

4:48 PM Changeset in webkit [5424] by vicki

1 edit in branches/Safari-1-1-Merlot-BuildFixes-branch/WebCore/ChangeLog-2005-08-23

moved the release marker in the ChangeLog

4:40 PM Changeset in webkit [5423] by vicki

2 edits in branches/Safari-1-1-Merlot-BuildFixes-branch/WebCore

Reviewed by kocienda.

WebCorePrefix.h: fix Merlot build breakage

3:36 PM Changeset in webkit [5422] by vicki

1 edit in branches/Safari-1-1-Merlot-BuildFixes-branch/WebCore/ChangeLog-2005-08-23

Safari v100.0.1 stamp

3:24 PM Changeset in webkit [5421] by vicki

4 edits in branches/Safari-1-1-Merlot-BuildFixes-branch/WebCore

rolled this fix onto the Safari-1-1-Merlot-BuildFixes-branch

2003-10-26 Darin Adler <Darin Adler>

Reviewed by Maciej.

fixed 3463586 -- WebCore does not compile with new definition of NULL

khtml/khtml_part.cpp: (findWordBoundary): Changed NULL to 0 and did a little code cleanup of the surrounding code.
khtml/rendering/break_lines.cpp: (khtml::isBreakable): Changed NULL to 0 and did a tiny bit of cleanup.