WebKit-https.git
17 years ago Patch contributed by Carsten Guenther <cguenther@gmail.com>.
ggaren [Tue, 28 Jun 2005 22:25:43 +0000 (22:25 +0000)]
Patch contributed by Carsten Guenther <cguenther@gmail.com>.

-fixes http://bugzilla.opendarwin.org/show_bug.cgi?id=3477
some US-centric date formats not parsed by JavaScript (clock at news8austin.com)

        Reviewed by darin.

        * kjs/date_object.cpp:
        (formatLocaleDate):
        (day):
        (dayFromYear):
        (daysInYear):
        (timeFromYear):
        (yearFromTime):
        (weekDay):
        (timeZoneOffset):
        (DateProtoFuncImp::call):
        (DateObjectImp::construct):
        (KJS::parseDate):
        (ymdhms_to_seconds):
        (KJS::makeTime):
        (findMonth):
        (KJS::KRFCDate_parseDate):
        * kjs/date_object.h:
        * tests/mozilla/expected.html: updated expected test results to reflect fix

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9516 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Patch by Kevin Ballard <kevin@sb.org>
justing [Tue, 28 Jun 2005 22:18:27 +0000 (22:18 +0000)]
    Patch by Kevin Ballard <kevin@sb.org>

        Reviewed by Darin

        - Fixes <http://bugzilla.opendarwin.org/show_bug.cgi?id=3654>
        Updating the value of a textarea wasn't saving the cursor position
        in a line ending-agnostic fashion.

        Test cases added:
        * layout-tests/fast/forms/cursor-position-expected.txt: Added.
        * layout-tests/fast/forms/cursor-position.html: Added.

        * kwq/KWQTextArea.mm:
        (-[KWQTextArea getCursorPositionAsIndex:inParagraph:]):
        (RangeOfParagraph):
        (-[KWQTextArea setCursorPositionToIndex:inParagraph:]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9515 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago*** empty log message ***
mjs [Tue, 28 Jun 2005 21:18:17 +0000 (21:18 +0000)]
*** empty log message ***

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9514 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Patch by Alexey Proskuryakov <ap@nypop.com>
justing [Tue, 28 Jun 2005 18:47:12 +0000 (18:47 +0000)]
    Patch by Alexey Proskuryakov <ap@nypop.com>

        - <http://bugzilla.opendarwin.org/show_bug.cgi?id=3727>
        When a DECODE_DEBUG preprocessor symbol is defined in khtml/misc/decoder.cpp,
        compilation fails because a non-existent variable "force" is used.

        Reviewed by Darin

        Test cases added: None required

        * khtml/misc/decoder.cpp:
        (Decoder::setEncoding):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9513 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Chris Blumenberg.
harrison [Tue, 28 Jun 2005 16:22:04 +0000 (16:22 +0000)]
    Reviewed by Chris Blumenberg.

        Test cases added: (NONE)

        (continued) <rdar://problem/4140688> assertion failure double-clicking text to select
        Fixed my checkin from yesterday.

        * khtml/rendering/render_text.cpp:
        (InlineTextBox::paintMarkedTextBackground):
        (InlineTextBox::positionForOffset):
        Changed tabs to spaces.

        (RenderText::positionForCoordinates):
        Make sure affinity is downstream when in mid-line.
        Changed tabs to spaces.

        (RenderText::caretRect):
        (RenderText::cacheWidths):
        (RenderText::trimmedMinMaxWidth):
        (RenderText::minXPos):
        (RenderText::width):
        (RenderText::caretMaxOffset):
        Changed tabs to spaces.

        * layout-tests/editing/style/create-block-for-style-004-expected.txt:
        Expected ending selection affinity is downstream.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9512 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Remove accidentally committed code that breaks the build. Oops!
mjs [Tue, 28 Jun 2005 05:47:26 +0000 (05:47 +0000)]
Remove accidentally committed code that breaks the build. Oops!

        * khtml/xml/dom_atomicstring.cpp:
        (DOM::AtomicString::expand):
        (DOM::AtomicString::shrink):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9511 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years agoRemoved accidental change from last checkin.
mjs [Tue, 28 Jun 2005 01:57:42 +0000 (01:57 +0000)]
Removed accidental change from last checkin.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9510 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Darin.
mjs [Tue, 28 Jun 2005 01:54:35 +0000 (01:54 +0000)]
    Reviewed by Darin.

- replaced all our hash functions with the state of the art in hashing
- ~1% speedup on cvs-base

        No test cases added, perf effects only.

        * khtml/xml/dom_atomicstring.cpp:
        (DOM::AtomicStringStatisticsExitLogger::~AtomicStringStatisticsExitLogger):
Improved stats gathering to track collisions in more detail and count reinserts
on remove.
        (DOM::addToCollisionCount): ditto
        (DOM::AtomicString::add): ditto
        (DOM::AtomicString::insert): ditto
        (DOM::AtomicString::remove): ditto
        * khtml/xml/dom_stringimpl.cpp:
        (DOM::DOMStringImpl::computeHash): Replace with SuperFastHash algorithm.
        * kwq/KWQCharsets.mm:
        (encodingNameHash): Clean up mistaken shift in the wrong direction.
        * kwq/KWQString.h: Removed unused hashing code.
        * kwq/KWQString.mm: ditto
        * kwq/KWQTextCodec.mm:
        (QTextCodec::hash): Use a variant of the SuperFastHash algorithm.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9509 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Darin.
mjs [Tue, 28 Jun 2005 00:02:08 +0000 (00:02 +0000)]
    Reviewed by Darin.

- replace hash functions  with better ones

        * JavaScriptCore.pbproj/project.pbxproj: Add new file to build.
        * kjs/interpreter_map.cpp:
        (KJS::InterpreterMap::computeHash): Use shared pointer hash.
        * kjs/pointer_hash.h: Added.
        (KJS::pointerHash): Pointer hash based on 32-bit mix and 64-bit mix hashes.
        * kjs/protected_values.cpp:
        (KJS::ProtectedValues::computeHash): Use shared pointer hash.
        * kjs/ustring.cpp:
        (KJS::UString::Rep::computeHash): Use SuperFastHash algorithm.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9501 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Patch by Anders Carlsson <andersca@mac.com>
justing [Mon, 27 Jun 2005 20:23:34 +0000 (20:23 +0000)]
    Patch by Anders Carlsson <andersca@mac.com>

        Reviewed by Darin.

        - Fixes <http://bugzilla.opendarwin.org/show_bug.cgi?id=3489>
        WebView's setSelectedDOMRange doesn't not implement clearing the selection as described in the WebView documentation:
        <http://developer.apple.com/documentation/Cocoa/Reference/WebKit/ObjC_classic/Classes/WebView.html>

        * WebView.subproj/WebView.m:
        (-[WebView setSelectedDOMRange:affinity:]):
        If range is nil, call deselectText.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9496 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Ken.
harrison [Mon, 27 Jun 2005 19:12:34 +0000 (19:12 +0000)]
    Reviewed by Ken.

        Test cases added: None because this relies on mouse clicking.

<rdar://problem/4140688> assertion failure double-clicking text to select

        * khtml/rendering/render_text.cpp:
        (RenderText::positionForCoordinates):
Use setAffinityUsingLinePosition.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9490 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago * Scripts/build-dumprendertree: Pass -project option so that having a second
darin [Mon, 27 Jun 2005 05:33:53 +0000 (05:33 +0000)]
    * Scripts/build-dumprendertree: Pass -project option so that having a second
        copy of the project (like the one Xcode 2.1 offers to make for you) that's
        out of date won't screw you up.
        * Scripts/build-webkit: Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9488 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago - updated a layout test so that it doesn't depend on a remote resource via...
darin [Sat, 25 Jun 2005 17:21:59 +0000 (17:21 +0000)]
    - updated a layout test so that it doesn't depend on a remote resource via http

        * layout-tests/fast/backgrounds/001.html: Change image URL to be relative.
        * layout-tests/fast/backgrounds/resources/bg_position_larger.gif: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9487 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Changes by Mitz Pettel.
darin [Sat, 25 Jun 2005 05:17:33 +0000 (05:17 +0000)]
    Changes by Mitz Pettel.
        Reviewed by me.

        Test cases added:
        * layout-tests/fast/text/international/bidi-explicit-embedding-expected.txt: Added.
        * layout-tests/fast/text/international/bidi-explicit-embedding.html: Added.

        - fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3599>
          Incorrect layout of explicit embedding levels

        * khtml/rendering/bidi.cpp:
        (khtml::embed): Set up the new run correctly following an embed/pop.
        (khtml::RenderBlock::bidiReorderLine): Close the "dangling" run when we see an explicit
        embedding character (or PDF).

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9486 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Changes by Mitz Pettel.
darin [Sat, 25 Jun 2005 04:47:05 +0000 (04:47 +0000)]
    Changes by Mitz Pettel.
        Reviewed by me.

        Test cases added:
        * layout-tests/fast/text/international/bidi-CS-after-AN-expected.txt: Added.
        * layout-tests/fast/text/international/bidi-CS-after-AN.html: Added.

        - fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3693>
          Bug_72578 in KHTML: bidi algorithm misbehaves with CS after AN

        * khtml/rendering/bidi.cpp: (khtml::RenderBlock::bidiReorderLine): Rolled in the
        patch from the KDE tree.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9485 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago - <http://bugzilla.opendarwin.org/show_bug.cgi?id=3673>
justing [Fri, 24 Jun 2005 21:05:11 +0000 (21:05 +0000)]
    - <bugzilla.opendarwin.org/show_bug.cgi?id=3673>
            Adds QBuffer::setBuffer(QByteArray) support to KWQBuffer, and adds a QBuffer(QByteArray) constructor.

        Reviewed by Darin

        Test cases added: Non required

        * kwq/KWQBuffer.h:
        (QBuffer::QBuffer):
        * kwq/KWQBuffer.mm:
        (QBuffer::setBuffer):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9484 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Patch contributed by Duncan Wilcox <duncan@mclink.it>
justing [Fri, 24 Jun 2005 20:48:59 +0000 (20:48 +0000)]
    Patch contributed by Duncan Wilcox <duncan@mclink.it>

        Reviewed by Darin

        - Fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3535>
            Spelling suggestions in the context menu don't call the should* delegate methods

        * WebView.subproj/WebHTMLView.m:
        (-[WebHTMLView _changeSpellingFromMenu:]):
        give delegate's webView:shouldInsertText:replacingDOMRange:givenAction: a chance
        to prevent replacing of selected text

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9483 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago - <http://bugzilla.opendarwin.org/show_bug.cgi?id=3672> KWQRect -- CGRect...
justing [Fri, 24 Jun 2005 18:57:22 +0000 (18:57 +0000)]
    - <bugzilla.opendarwin.org/show_bug.cgi?id=3672> KWQRect -- CGRect and other small additions
        - <http://bugzilla.opendarwin.org/show_bug.cgi?id=3670> Add remove and byteOrderMark support to KWQString for kdom

        Reviewed by Darin

        Test cases added: None required

        * kwq/KWQRect.h:
        (QRect::contains):
        * kwq/KWQRect.mm:
        (QRect::QRect):
        (QRect::topRight):
        (QRect::bottomLeft):
        (QRect::normalize):
        (QRect::operator CGRect):
        * kwq/KWQString.h:
        (QChar::):
        (QString::remove):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9482 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Justin Garcia.
darin [Fri, 24 Jun 2005 16:48:39 +0000 (16:48 +0000)]
    Reviewed by Justin Garcia.

        - fixed <rdar://problem/4151619> Safari changes "\" characters to "/" in the query part of links that start with "?"

        Test cases added:
        * layout-tests/fast/dom/anchor-backslash-expected.txt: Added.
        * layout-tests/fast/dom/anchor-backslash.html: Added.

        * kwq/KWQKURL.mm: (substituteBackslashes): Fix error where we'd ignore
        "?" and "#" characters if they were the first character in the string.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9481 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago * khtml/html/html_formimpl.cpp: (DOM::HTMLTextAreaElementImpl::setValue):
darin [Fri, 24 Jun 2005 05:59:09 +0000 (05:59 +0000)]
    * khtml/html/html_formimpl.cpp: (DOM::HTMLTextAreaElementImpl::setValue):
        Took out an unneeded type cast (meant to do that before committing).

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9480 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed and landed by Darin Adler.
darin [Fri, 24 Jun 2005 05:53:31 +0000 (05:53 +0000)]
    Reviewed and landed by Darin Adler.

        Test cases added:
        * layout-tests/fast/forms/selection-functions-expected.txt: Added.
        * layout-tests/fast/forms/selection-functions.html: Added.
        * layout-tests/fast/forms/textarea-crlf-expected.txt: Added.
        * layout-tests/fast/forms/textarea-crlf.html: Added.

        - Fixes bug <http://bugzilla.opendarwin.org/show_bug.cgi?id=3401>.
          Also fixes issue where setting text of a textarea in code and then
          retrieving it would leave the \r\n sequences in the text, despite
          the textarea translating to \n in all other cases.

        * khtml/ecma/kjs_html.h:
        * khtml/ecma/kjs_html.cpp:
        (KJS::getInputSelectionStart):
        (KJS::getInputSelectionEnd):
        (KJS::KJS::HTMLElement::getValueProperty):
        (KJS::KJS::HTMLElement::hasProperty):
        (KJS::KJS::HTMLElementFunction::tryCall):
        (KJS::HTMLElement::putValue):
        * khtml/html/html_formimpl.cpp:
        (DOM::HTMLInputElementImpl::canHaveSelection):
        (DOM::HTMLInputElementImpl::selectionStart):
        (DOM::HTMLInputElementImpl::selectionEnd):
        (DOM::HTMLInputElementImpl::setSelectionStart):
        (DOM::HTMLInputElementImpl::setSelectionEnd):
        (DOM::HTMLInputElementImpl::setSelectionRange):
        (DOM::HTMLInputElementImpl::setValue):
        (DOM::HTMLTextAreaElementImpl::selectionStart):
        (DOM::HTMLTextAreaElementImpl::selectionEnd):
        (DOM::HTMLTextAreaElementImpl::setSelectionStart):
        (DOM::HTMLTextAreaElementImpl::setSelectionEnd):
        (DOM::HTMLTextAreaElementImpl::setSelectionRange):
        (DOM::HTMLTextAreaElementImpl::setValue):
        * khtml/html/html_formimpl.h:
        * khtml/rendering/render_form.cpp:
        (RenderLineEdit::selectionStart):
        (RenderLineEdit::selectionEnd):
        (RenderLineEdit::setSelectionStart):
        (RenderLineEdit::setSelectionEnd):
        (RenderLineEdit::setSelectionRange):
        (RenderTextArea::selectionStart):
        (RenderTextArea::selectionEnd):
        (RenderTextArea::setSelectionStart):
        (RenderTextArea::setSelectionEnd):
        (RenderTextArea::select):
        (RenderTextArea::setSelectionRange):
        * khtml/rendering/render_form.h:
        * kwq/KWQLineEdit.h:
        * kwq/KWQLineEdit.mm:
        (QLineEdit::setCursorPosition):
        (QLineEdit::cursorPosition):
        (QLineEdit::selectAll):
        (QLineEdit::selectionStart):
        (QLineEdit::selectedText):
        (QLineEdit::setSelection):
        * kwq/KWQTextArea.h:
        * kwq/KWQTextArea.mm:
        (-[KWQTextArea setSelectedRange:]):
        (-[KWQTextArea selectedRange]):
        (-[KWQTextArea getCursorPositionAsIndex:inParagraph:]):
        (RangeOfParagraph):
        * kwq/KWQTextEdit.h:
        * kwq/KWQTextEdit.mm:
        (QTextEdit::getCursorPosition):
        (QTextEdit::selectionStart):
        (QTextEdit::selectionEnd):
        (QTextEdit::setSelectionStart):
        (QTextEdit::setSelectionEnd):
        (QTextEdit::selectAll):
        (QTextEdit::setSelectionRange):
        * kwq/KWQTextField.h:
        * kwq/KWQTextField.mm:
        (-[KWQTextFieldController selectedRange]):
        (-[KWQTextFieldController setSelectedRange:]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9479 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Patch by Eric Seidel <macdome@opendarwin.org>
justing [Thu, 23 Jun 2005 21:52:17 +0000 (21:52 +0000)]
    Patch by Eric Seidel <macdome@opendarwin.org>

        Adds QTextIStream, QTextOStream to KWQ

        Reviewed by Darin

        Test cases added: None required

        * kwq/KWQTextStream.h:
        (QTextIStream::QTextIStream):
        (QTextOStream::QTextOStream):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9478 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Patch by Eric Seidel <macdome@opendarwin.org>
justing [Thu, 23 Jun 2005 21:25:49 +0000 (21:25 +0000)]
    Patch by Eric Seidel <macdome@opendarwin.org>

        This patch contains several small additions to KWQ

        Reviewed by Darin

        Test cases added: None required

        * kwq/KWQApplication.h:
        (QApplication::palette):
        * kwq/KWQDateTime.h:
        (QDateTime::time):
        * kwq/KWQDict.h:
        (QDict::operator[]):
        (QDictIterator::operator*):
        * kwq/KWQKDebug.h:
        (kdbgstream::operator<<):
        * kwq/KWQMemArray.h:
        (QMemArray::duplicate):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9477 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Test cases added: None needed for this debug-only utility code.
harrison [Thu, 23 Jun 2005 19:47:50 +0000 (19:47 +0000)]
    Test cases added: None needed for this debug-only utility code.

        - Make showTree() const.
        - Add showTree() for InlineBox.
        - Add showTree() for RenderObject.

        * khtml/editing/selection.cpp:
        (khtml::Selection::showTree):
        * khtml/editing/selection.h:
        * khtml/editing/visible_position.cpp:
        (khtml::VisiblePosition::showTree):
        * khtml/editing/visible_position.h:
        Make showTree() const.

        * khtml/rendering/render_line.cpp:
        (khtml::InlineBox::showTree):
        * khtml/rendering/render_line.h:
        Add showTree() for InlineBox.

        * khtml/rendering/render_object.cpp:
        (RenderObject::showTree):
        * khtml/rendering/render_object.h:
        Add showTree() for RenderObject.

        * khtml/xml/dom_nodeimpl.cpp:
        (appendAttributeDesc):
        (NodeImpl::showNode):
        (NodeImpl::showTree):
        (NodeImpl::showTreeAndMark):
        * khtml/xml/dom_nodeimpl.h:
        Make showTree() const.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9476 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago - added template to be used when writing manual tests
vicki [Thu, 23 Jun 2005 19:13:57 +0000 (19:13 +0000)]
- added template to be used when writing manual tests

        * manual-tests/template.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9475 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago * Scripts/webkitdirs.pm: Fixed a regular expression in the code I just landed.
darin [Thu, 23 Jun 2005 15:00:36 +0000 (15:00 +0000)]
    * Scripts/webkitdirs.pm: Fixed a regular expression in the code I just landed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9474 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Changes based on input from Michael Kahl.
darin [Thu, 23 Jun 2005 14:53:16 +0000 (14:53 +0000)]
    Changes based on input from Michael Kahl.

        * Scripts/cvs-create-patch: Added code to handle getting changes in the top-level
        directory passed in.

        * Scripts/webkitdirs.pm: Eliminate use of changing the current directory and using
        getcwd() in the code to find the base product dir. Added code to handle unusual
        base product directory values that use SRCROOT.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9473 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago - added forwarding headers as needed for Eric Seidel's KDOM work
darin [Thu, 23 Jun 2005 03:15:27 +0000 (03:15 +0000)]
    - added forwarding headers as needed for Eric Seidel's KDOM work
          see <http://bugzilla.opendarwin.org/show_bug.cgi?id=3655>

        * ForwardingHeaders/kjs/value.h: Added.
        * ForwardingHeaders/qintdict.h: Added an include.
        * ForwardingHeaders/qptrcollection.h: Added.
        * ForwardingHeaders/qsize.h: Added.
        * kwq/KWQIntDict.h: Added. Empty file.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9472 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Chris Blumenberg.
sullivan [Wed, 22 Jun 2005 21:32:09 +0000 (21:32 +0000)]
    Reviewed by Chris Blumenberg.

        - fixed <rdar://problem/3764645> please add a way to allow WebKit clients to override the WebPDFView context menu

        * PublicHeaderChangesFromTiger.txt: Added.
        New file to keep track of changes made to public headers that haven't been through API review yet.
        Initially lists the WebMenuItem enum tags added to WebUIDelegate.h as part of this change.

        * WebView.subproj/WebDefaultContextMenuDelegate.m:
        (-[WebDefaultUIDelegate appendDefaultItems:toArray:]):
        new method, handles initial separator
        (-[WebDefaultUIDelegate contextMenuItemsForElement:defaultMenuItems:]):
        now has defaultMenuItems: parameter. Any menu items in this array are appended at the end of
        the standard set.
        (-[WebDefaultUIDelegate editingContextMenuItemsForElement:defaultMenuItems:]):
        ditto
        (-[WebDefaultUIDelegate webView:contextMenuItemsForElement:defaultMenuItems:]):
        This had a defaultMenuItems parameter before but it was always nil. Now it passes the defaultMenuItems
        parameter on to the two methods that construct lists (one for editing, the other for viewing). Also
        tweaked variable name and type for clarity.

        * WebView.subproj/WebHTMLView.m:
        (-[WebHTMLView menuForEvent:]):
        passes nil for new defaultItems parameter of _menuForElement:

        * WebView.subproj/WebImageView.m:
        (-[WebImageView menuForEvent:]):
        ditto

        * WebView.subproj/WebTextView.m:
        (-[WebTextView menuForEvent:]):
        ditto

        * WebView.subproj/WebPDFView.m:
        (-[WebPDFView elementAtPoint:]):
        new method to create the element dictionary needed for _menuForElement:defaultItems:. Only supplies the
        webFrame at this point.
        (-[WebPDFView _menuItemsFromPDFKitForEvent:]):
        new method to return copies of the menu items that PDFKit would include in the context menu, with
        WebKit tags applied
        (-[WebPDFView menuForEvent:]):
        now calls standard WebKit context menu mechanism, so clients' delegates can modify the context menu as
        desired. The initial set of items are the ones WebKit was already displaying for PDF context menus.

        * WebView.subproj/WebUIDelegate.h:
        added enum values for the menu items in the PDF context menu

        * WebView.subproj/WebViewPrivate.h:
        * WebView.subproj/WebView.m:
        (-[WebView _menuForElement:defaultItems:]):
        Added the defaultItems: parameter to this method, which is then passed along to WebDefaultUIDelegate.
        All callers pass nil except for WebPDFView, at least for now.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9471 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Vicki.
darin [Wed, 22 Jun 2005 21:05:59 +0000 (21:05 +0000)]
    Reviewed by Vicki.

        Test cases added:
        * layout-tests/fast/css/apple-prefix-expected.txt: Added.
        * layout-tests/fast/css/apple-prefix.html: Added.

        - fixed <rdar://problem/4141650> REGRESSION: Safari RSS slider broken with TOT WebCore

        * khtml/css/parser.y:
        (getPropertyID): Fix lifetime problem on the QString. Also construct the QString
        using length, since the passed-in char * is not necessarily null-terminated. Also do
        the mapping only if it's a "-apple-" prefix -- the old code would do it for, e.g.,
        "-applex-" too. The old code would not successfully map the "apple" in "appleLineClamp"
        to "khtml", and Dave changed things so that the internal name of all CSS attributes
        is "khtml" back in his 2005-05-03 check-in, breaking the use of appleLineClamp in RSS.
        (getValueID): Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9470 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years agoAdded more bug numbers to the change log.
darin [Wed, 22 Jun 2005 18:23:46 +0000 (18:23 +0000)]
Added more bug numbers to the change log.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9467 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed and landed by Darin Adler.
darin [Wed, 22 Jun 2005 18:22:03 +0000 (18:22 +0000)]
    Reviewed and landed by Darin Adler.

        Test cases added:
        * layout-tests/fast/dom/title-text-property-expected.txt: Added
        * layout-tests/fast/dom/title-text-property.html: Added
        * layout-tests/fast/dom/title-text-property-expected-2.txt: Added
        * layout-tests/fast/dom/title-text-property-2.html: Added

        Fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3586>
        Setting and getting title's text attribute does not work

        * khtml/dom/html_head.cpp:
        (HTMLTitleElement::text):
        (HTMLTitleElement::setText):
        Call HTMLTitleElementImpl::text() and ::setText().

        * khtml/html/html_headimpl.cpp:
        (HTMLTitleElementImpl::HTMLTitleElementImpl):
        (HTMLTitleElementImpl::insertedIntoDocument):
        (HTMLTitleElementImpl::removedFromDocument):
        (HTMLTitleElementImpl::childrenChanged):
        Pass the current title node to DocumentImpl::setTitle
        and ::removeTitle.

        (HTMLTitleElementImpl::text):
        Return the title.

        (HTMLTitleElementImpl::setText):
        Set the new title.

        * khtml/xml/dom_docimpl.cpp:
        (DocumentImpl::DocumentImpl):
        (DocumentImpl::~DocumentImpl):
        (DocumentImpl::updateTitle):
        (DocumentImpl::setTitle):
        (DocumentImpl::removeTitle):
        * khtml/xml/dom_docimpl.h:
        Only update the title if it's done by setting document.title
        or by changing the child nodes of the title element.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9466 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Change by Anders Carlsson.
darin [Wed, 22 Jun 2005 18:01:02 +0000 (18:01 +0000)]
    Change by Anders Carlsson.

        - added support for dumping title changes

        * DumpRenderTree/DumpRenderTree.m:
        (-[WaitUntilDoneDelegate webView:didReceiveTitle:forFrame:]): Added. Dump title change
        if requested by JavaScript.
        (+[LayoutTestController isSelectorExcludedFromWebScript:]): Added dumpTitleChanges.
        (-[LayoutTestController dumpTitleChanges]): Added, sets flag.
        (dumpRenderTree): Start flag as NO.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9465 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Changes by Mitz Pettel.
darin [Wed, 22 Jun 2005 17:39:53 +0000 (17:39 +0000)]
    Changes by Mitz Pettel.
        Reviewed by me.

        - fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3633>
          Layout order of bidi level runs is incorrect

        Test cases added:
        * layout-tests/fast/text/international/bidi-L2-run-reordering-expected.txt: Added.
        * layout-tests/fast/text/international/bidi-L2-run-reordering.html: Added.

        * khtml/rendering/bidi.cpp: (khtml::RenderBlock::bidiReorderLine): Fixed mistake
        where we'd advance too far after reversing a run.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9464 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago - added test case for JavaScriptCore fix
darin [Wed, 22 Jun 2005 17:29:28 +0000 (17:29 +0000)]
    - added test case for JavaScriptCore fix

        * layout-tests/fast/js/string-replace-2-expected.txt: Added.
        * layout-tests/fast/js/string-replace-2.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9463 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Change by Anders Carlsson.
darin [Wed, 22 Jun 2005 17:26:17 +0000 (17:26 +0000)]
    Change by Anders Carlsson.
        Reviewed by me.

        - fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3294>
          String.prototype.replace() fails with function as second param

        * kjs/string_object.cpp: (replace): Added code to handle functions.

        * tests/mozilla/expected.html: Updated since ecma_3/RegExp/regress-209067.js is fixed now.

        * tests/mozilla/run-mozilla-tests: Fix a minor coding style issue that leads to a warning each
        time we run the tests.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9462 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Change by Mitz Pettel.
darin [Wed, 22 Jun 2005 16:44:08 +0000 (16:44 +0000)]
    Change by Mitz Pettel.
        Reviewed by me.

        - fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3618>
          RTL runs drawn by CG not reversed properly

        * WebCoreSupport.subproj/WebTextRenderer.m: (-[WebTextRenderer _CG_drawRun:style:geometry:]):
        Fix off-by-one mistake in order-swapping loops.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9461 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by John Sullivan.
darin [Wed, 22 Jun 2005 16:38:03 +0000 (16:38 +0000)]
    Reviewed by John Sullivan.

        Test cases added:
        * layout-tests/fast/dom/css-set-property-exception-expected.txt: Added.
        * layout-tests/fast/dom/css-set-property-exception.html: Added.

        - fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3616>
          RSS search field, Dashboard widgets failing due to CSS exception

        * khtml/ecma/kjs_css.cpp: (KJS::DOMCSSStyleDeclaration::tryPut):
        Ignore exceptions when setting properties.

        * layout-tests/fast/js/exception-linenums.html: Changed test case that
        involved setting style to one that involved setting another property that
        raises an exception when you try to set it (setting prefix on a text node).

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9460 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Change by Michael Gaiman.
darin [Wed, 22 Jun 2005 16:20:01 +0000 (16:20 +0000)]
    Change by Michael Gaiman.
        Reviewed by me.

        - fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3436>
          Missing implementation of -[NSData(WebNSDateExtras) _webkit_parseRFC822HeaderFields]

        * Misc.subproj/WebNSDataExtras.h: Fixed name of category say NSData, not NSDate.
        * Misc.subproj/WebNSDataExtras.m: (-[NSData _webkit_parseRFC822HeaderFields]): Fixed method name.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9459 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Change proposed by Eric Seidel.
darin [Wed, 22 Jun 2005 15:31:57 +0000 (15:31 +0000)]
    Change proposed by Eric Seidel.

        * WebCore.pbproj/project.pbxproj: Fix build rule to match the one in JavaScriptCore.
        The one here accidentally had hard-coded paths in it and also had the $() vs. ${}
        issue that broke building on Xcode 2.1 for no good reason (it didn't need $() or ${}).

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9458 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Patch from KevinBallard <kevin@sb.org>, reviewed by me.
mjs [Wed, 22 Jun 2005 04:38:53 +0000 (04:38 +0000)]
    Patch from KevinBallard <kevin@sb.org>, reviewed by me.

- fix build for Xcode 2.1, use ${} for INPUT_FILE_BASE to fix build with Xcode 2.1

        * WebCore.pbproj/project.pbxproj:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9457 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago - Fixes <rdar://problem/4114192>.
justing [Wed, 22 Jun 2005 01:15:43 +0000 (01:15 +0000)]
- Fixes <rdar://problem/4114192>.
- Fixes a crash in TOT Mail/Blot if you Edit -> Add Hyperlink on a Selection
   that ends at the end of the document
- Fixes a problem in TOT Mail/Blot where Edit -> Add Hyperlink creates
   a link one character beyond what is expected.

In replacement, during insertion, some special case code was being called inappropriately.

        Reviewed by Harrison

        Test cases added: None, requires manual interaction

        * khtml/editing/replace_selection_command.cpp:
        (khtml::ReplaceSelectionCommand::doApply): In "step 2" of a replacement, we should
        only next() the insertion point if the special pre-insertion-merge actually occured.
        mergeStart used to be the correct check for the occurence of a pre-insertion-merge, but
        a bug fix made this not the case.  A non-null m_lastNodeInserted correctly signals
        that some pre-insertion merging occured.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9456 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years agoJavaScriptCore:
adele [Wed, 22 Jun 2005 00:48:49 +0000 (00:48 +0000)]
JavaScriptCore:

        rolling out fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=3293, since it caused layout test failures.
fast/forms/element-by-name
        fast/loader/loadInProgress

        * ChangeLog:
        * bindings/objc/objc_runtime.h:
        * bindings/objc/objc_runtime.mm:
        (ObjcFallbackObjectImp::hasProperty):
        * bindings/runtime_array.cpp:
        (RuntimeArrayImp::hasProperty):
        * bindings/runtime_array.h:
        * bindings/runtime_object.cpp:
        (RuntimeObjectImp::hasProperty):
        * bindings/runtime_object.h:
        * kjs/array_instance.h:
        * kjs/array_object.cpp:
        (ArrayInstanceImp::hasProperty):
        * kjs/function.cpp:
        (KJS::FunctionImp::hasProperty):
        (KJS::ActivationImp::hasProperty):
        * kjs/function.h:
        * kjs/object.cpp:
        (KJS::ObjectImp::hasProperty):
        * kjs/object.h:
        * kjs/object_object.cpp:
        (ObjectPrototypeImp::ObjectPrototypeImp):
        (ObjectProtoFuncImp::call):
        * kjs/object_object.h:
        (KJS::ObjectProtoFuncImp::):
        * kjs/string_object.cpp:
        (StringInstanceImp::hasProperty):
        * kjs/string_object.h:
        * tests/mozilla/expected.html:

WebCore:

        rolling out fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=3293, since it caused layout test failures.
fast/forms/element-by-name
        fast/loader/loadInProgress

        * ChangeLog:
        * khtml/ecma/kjs_binding.h:
        * khtml/ecma/kjs_css.cpp:
        (KJS::DOMCSSStyleDeclaration::hasProperty):
        * khtml/ecma/kjs_css.h:
        * khtml/ecma/kjs_dom.cpp:
        (KJS::DOMNodeList::hasProperty):
        (KJS::DOMNamedNodeMap::hasProperty):
        * khtml/ecma/kjs_dom.h:
        * khtml/ecma/kjs_html.cpp:
        (KJS::HTMLDocument::hasProperty):
        (KJS::KJS::HTMLElement::hasProperty):
        (KJS::KJS::HTMLCollection::hasProperty):
        * khtml/ecma/kjs_html.h:
        * khtml/ecma/kjs_window.cpp:
        (KJS::Window::hasProperty):
        (KJS::Konqueror::hasProperty):
        * khtml/ecma/kjs_window.h:
        * layout-tests/fast/js/eval-var-decl-expected.txt: Removed. This test used hasOwnProperty
        * layout-tests/fast/js/eval-var-decl.html: Removed. This test used hasOwnProperty
        * layout-tests/fast/js/hasOwnProperty-expected.txt: Removed.
        * layout-tests/fast/js/hasOwnProperty.html: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9455 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago * WebCore.pbproj/project.pbxproj: Switched to a build rule rather than a...
darin [Wed, 22 Jun 2005 00:23:09 +0000 (00:23 +0000)]
    * WebCore.pbproj/project.pbxproj: Switched to a build rule rather than a build phase for the
        .y files -- this gets rid of the problem where modifying the .y file would not cause sufficient
        compilation.

        * khtml/css/parser_wrapper.cpp: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9454 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago * JavaScriptCore.pbproj/project.pbxproj: Switched to a build rule rather...
darin [Wed, 22 Jun 2005 00:22:55 +0000 (00:22 +0000)]
    * JavaScriptCore.pbproj/project.pbxproj: Switched to a build rule rather than a build phase for
        .y files -- this gets rid of the problem where modifying the .y file would not cause sufficient
        compilation.

        * kjs/grammar_wrapper.cpp: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9453 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Patch from Anders Carlsson <andersca@mac.com>, reviewed by Darin.
adele [Tue, 21 Jun 2005 23:02:19 +0000 (23:02 +0000)]
    Patch from Anders Carlsson <andersca@mac.com>, reviewed by Darin.

        Fixed: <http://bugzilla.opendarwin.org/show_bug.cgi?id=3450>
        <rdar://problem/3881901> String.replace() method not working when regex pattern contains {n, m}

        * pcre/pcre.c: (pcre_compile): Remember the last char length so it can be subtracted correctly if needed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9452 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Test cases for:
adele [Tue, 21 Jun 2005 23:02:07 +0000 (23:02 +0000)]
    Test cases for:
        <http://bugzilla.opendarwin.org/show_bug.cgi?id=3450>
        <rdar://problem/3881901> String.replace() method not working when regex pattern contains {n, m}

        * layout-tests/fast/js/string_replace-expected.txt: Added.
        * layout-tests/fast/js/string_replace.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9451 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Patch contributed by Niels Leenheer <niels.leenheer@gmail.com>
ggaren [Tue, 21 Jun 2005 22:27:55 +0000 (22:27 +0000)]
    Patch contributed by Niels Leenheer <niels.leenheer@gmail.com>

        - fixed http://bugzilla.opendarwin.org/show_bug.cgi?id=3370
        (Safari can't handle strange frame dimensions)

        Reviewed by Darin.

        * khtml/dom/dom_string.cpp:
        (DOM::DOMString::toCoordsArray):
        * khtml/dom/dom_string.h:
        * khtml/html/html_imageimpl.cpp:
        (HTMLAreaElementImpl::parseMappedAttribute):
        * khtml/xml/dom_atomicstring.h:
        (DOM::AtomicString::toCoordsArray):
        * khtml/xml/dom_stringimpl.cpp:
        (DOM::parseLength):
        (DOM::DOMStringImpl::toCoordsArray):
        (DOM::DOMStringImpl::toLengthArray):
        * khtml/xml/dom_stringimpl.h:

        Test cases added:
        * layout-tests/fast/frames/invalid-expected.txt: Added.
        * layout-tests/fast/frames/invalid.html: Added.
        * layout-tests/fast/frames/valid-expected.txt: Added.
        * layout-tests/fast/frames/valid.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9450 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago - fixed <rdar://problem/4155049> javascript function named as "opener" doesn...
ggaren [Tue, 21 Jun 2005 18:44:58 +0000 (18:44 +0000)]
    - fixed <rdar://problem/4155049> javascript function named as "opener" doesn't get called because of window.opener property

        Test cases only, fix is in JavaScriptCore

        Reviewed by cblu.

        Test cases added:

        * layout-tests/fast/js/global-opener-function-expected.txt: Added.
        * layout-tests/fast/js/global-opener-function.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9449 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago - fixed <rdar://problem/4155532> 'delete' succeeds on functions
ggaren [Tue, 21 Jun 2005 18:42:30 +0000 (18:42 +0000)]
    - fixed <rdar://problem/4155532> 'delete' succeeds on functions
        - fixed <rdar://problem/4155049> javascript function named as "opener" doesn't get called because of window.opener property

        Reviewed by cblu.

        * kjs/nodes.cpp:
        (FuncDeclNode::processFuncDecl): Functions now have DontDelete and Internal attributes set when appropriate.

        Test cases:
        * tests/mozilla/expected.html: Updated for one new success.
        - see also test case added in WebCore.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9448 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Justin.
harrison [Tue, 21 Jun 2005 18:11:10 +0000 (18:11 +0000)]
    Reviewed by Justin.

        - show class and style attributes for element nodes
        - nil check the node being showTree'd
        - add Position::showTree()

        Test cases added: None needed for this debug-only utility code.

        * khtml/editing/selection.cpp:
        (khtml::Selection::showTree):
        - nil check the node
        * khtml/editing/visible_position.cpp:
        (khtml::VisiblePosition::previousVisiblePosition):
        - comments
        (khtml::VisiblePosition::showTree):
        - nil check the node
        * khtml/xml/dom_nodeimpl.cpp:
        (appendAttributeDesc):
        (NodeImpl::showNode):
        - show class and style attributes for element nodes
        * khtml/xml/dom_position.cpp:
        (DOM::Position::showTree):
        * khtml/xml/dom_position.h:
        - add Position::showTree()

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9447 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Vicki Murley
sullivan [Tue, 21 Jun 2005 18:05:57 +0000 (18:05 +0000)]
    Reviewed by Vicki Murley

        - fixed assertion failure Vicki ran into

        * Misc.subproj/WebIconDatabase.m:
        (-[WebIconDatabase _forgetIconForIconURLString:]):
        Handle the case where there are no associated page URLs for the icon URL

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9446 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years agoJavaScriptCore:
mjs [Tue, 21 Jun 2005 08:25:26 +0000 (08:25 +0000)]
JavaScriptCore:

        Reviewed by Darin(first pass) and Hyatt.

        - fixed http://bugzilla.opendarwin.org/show_bug.cgi?id=3576
        (roll in support for "const" keyword from KDE tree)
        - make processVarDecls handle deletability of variables declared
        in an eval block the same as evaluate would
        - make eval() call processVarDecls - needed to match mozilla and
        to make the second change testable

        I started with the KDE implementation of const but I ended up changing it a bit
        to avoid the use of a global variable. Now instead of the global variable it distinguishes
        const and var at the grammar level so the appropriate node can know the right kind of
        declaration.

        Test cases:
        * tests/mozilla/expected.html: Updated for one new test that is
        failing - we used to bail on it entirely because it checks for
        const support before starting.
        - see also test cases added in WebCore

        * kjs/grammar.y: Add rules for const declarations.
        * kjs/keywords.table: Add const keyword.
        * kjs/nodes.cpp:
        (VarDeclNode::VarDeclNode): Add parameter.
        (VarDeclNode::evaluate): Add const support.
        (VarDeclNode::processVarDecls): Add const support.
        (VarStatementNode::execute): Irrelevant change.
        (ForInNode::ForInNode): Tell our variable node that it's a variable.
        * kjs/nodes.h:
        (KJS::VarDeclNode::): Add declaration of type enum, extra constructor parameter.
        (KJS::VarStatementNode::VarStatementNode): Irrelevant change.
        * kjs/function.cpp:
        (KJS::GlobalFuncImp::call): Process var decls before evaluating.

WebCore:

        Reviewed by Darin(first pass) and Hyatt.

        - fixed http://bugzilla.opendarwin.org/show_bug.cgi?id=3576
        (roll in support for "const" keyword from KDE tree)
        - make processVarDecls handle deletability of variables declared
        in an eval block the same as evaluate would
        - make eval() call processVarDecls - needed to match mozilla and
        to make the second change testable

Test cases only, fix is in JavaScriptCore

        Test cases added:
        * layout-tests/fast/js/const-expected.txt: Added.
        * layout-tests/fast/js/const.html: Added.
        * layout-tests/fast/js/eval-var-decl-expected.txt: Added.
        * layout-tests/fast/js/eval-var-decl.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9445 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Chris Blumenberg.
sullivan [Tue, 21 Jun 2005 00:08:37 +0000 (00:08 +0000)]
    Reviewed by Chris Blumenberg.

        - fixed <rdar://problem/4155182> icon database retain counts can be incorrect after removing all icons

        Replaced the concept of "future retain count per page URL" with a simpler concept of "retain count per
        page URL", which is maintained even after the icon is actually loaded (unlike the future retain count).
        The total retain count for an icon is now the sum of the retain counts per page URL along with any retain
        count not associated with a page URL -- this is still needed for some internal housekeeping purposes.

        * Misc.subproj/WebIconDatabasePrivate.h:
        renamed iconURLToURLs -> iconURLToPageURLs for clarity
        renamed URLToIconURL -> pageURLToIconURL for clarity
        renamed futureURLToRetainCount -> pageURLToRetainCount (there's no more "future" aspect)
        renamed iconURLToRetainCount -> iconURLToExtraRetainCount (it now maintains only some of the retain count)

        * Misc.subproj/WebIconDatabase.m:
        (+[WebIconDatabase sharedIconDatabase]):
        updated for name changes only
        (-[WebIconDatabase init]):
        ditto
        (-[WebIconDatabase iconForURL:withSize:cache:]):
        ditto
        (-[WebIconDatabase iconURLForURL:]):
        ditto
        (-[WebIconDatabase retainIconForURL:]):
        just bump the retain count in pageURLToRetainCount, instead of behaving differently based on whether
        an icon had been loaded for this URL; this let me delete the internal method _retainFutureIconForURL:
        (-[WebIconDatabase releaseIconForURL:]):
        decrement the retain count in pageURLToRetainCount, then handle the case where the retain count for
        this page has gone to zero. I deleted the internal method _releaseFutureIconForURL: formerly called here.
        (-[WebIconDatabase removeAllIcons]):
        remove all the code that dealt with retain counts; this operation no longer affects retain counts
        (-[WebIconDatabase _setIconURL:forURL:]):
        remove the code that consolidated multiple retain counts for different page URLs into a single retain
        count; the multiple retain counts are now maintained even after the icon is loaded
        (-[WebIconDatabase _clearDictionaries]):
        updated for name changes only
        (-[WebIconDatabase _loadIconDictionaries]):
        ditto
        (-[WebIconDatabase _updateFileDatabase]):
        ditto
        (-[WebIconDatabase _totalRetainCountForIconURLString:]):
        new method, sums the retain counts associated with specific page URLs and the extra retain count
        not associated with specific page URLs
        (-[WebIconDatabase _retainIconForIconURLString:]):
        updated for name changes
        (-[WebIconDatabase _forgetIconForIconURLString:]):
        no longer affects retain counts at all; this is up to callers
        (-[WebIconDatabase _releaseIconForIconURLString:]):
        this now distinguishes the case where the retain count not associated with any page URLs hits
        zero from the case where the total retain count hits zero, and handles both

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9444 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Justin Garcia.
darin [Mon, 20 Jun 2005 22:40:44 +0000 (22:40 +0000)]
    Reviewed by Justin Garcia.

        * Scripts/run-webkit-tests: Report number of tests when they succeed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9443 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Chris Blumenberg.
sullivan [Mon, 20 Jun 2005 22:00:41 +0000 (22:00 +0000)]
    Reviewed by Chris Blumenberg.

        - added support for emptying the icon database

        * Misc.subproj/WebIconDatabase.h: just fixed a typo
        * Misc.subproj/WebIconDatabasePrivate.h:
        added WebPendingPublic category with method removeAllIcons, and
        declared WebIconDatabaseDidRemoveAllIconsNotification string.

        * Misc.subproj/WebIconDatabase.m:
        (-[WebIconDatabase removeAllIcons]):
        new method, removes all known icons from memory and disk. There's one loose end,
        covered by radar bug 4155182, where it's possible for the icon database's retain
        counts to get off after this operation. I plan to fix this next.
        (-[WebIconDatabase _setIconURL:forURL:]):
        just fixed some extra whitespace
        (-[WebIconDatabase _forgetIconForIconURLString:]):
        new method, extracted from _releaseIconForIconURLString
        (-[WebIconDatabase _releaseIconForIconURLString:]):
        now calls extracted method

        * WebKit.exp:
        added _WebIconDatabaseDidRemoveAllIconsNotification

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9442 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years agoJavaScriptCore:
mjs [Mon, 20 Jun 2005 21:59:39 +0000 (21:59 +0000)]
JavaScriptCore:

        Patch from Mark Rowe <opendarwin.org@bdash.net.nz>, reviewed by me.

- fixed http://bugzilla.opendarwin.org/attachment.cgi?id=2483

        Test cases added:
        * tests/mozilla/expected.html: Updated for two fixed tests.
- also added a layout test

* bindings/objc/objc_runtime.h:
        * bindings/objc/objc_runtime.mm:
        (ObjcFallbackObjectImp::hasOwnProperty):
        * bindings/runtime_array.cpp:
        (RuntimeArrayImp::hasOwnProperty):
        * bindings/runtime_array.h:
        * bindings/runtime_object.cpp:
        (RuntimeObjectImp::hasOwnProperty):
        * bindings/runtime_object.h:
        * kjs/array_instance.h:
        * kjs/array_object.cpp:
        (ArrayInstanceImp::hasOwnProperty):
        * kjs/function.cpp:
        (KJS::FunctionImp::hasOwnProperty):
        (KJS::ActivationImp::hasOwnProperty):
        * kjs/function.h:
        * kjs/object.cpp:
        (KJS::ObjectImp::hasProperty):
        (KJS::ObjectImp::hasOwnProperty):
        * kjs/object.h:
        (KJS::Object::hasOwnProperty):
        * kjs/object_object.cpp:
        (ObjectPrototypeImp::ObjectPrototypeImp):
        (ObjectProtoFuncImp::call):
        * kjs/object_object.h:
        (KJS::ObjectProtoFuncImp::):
        * kjs/string_object.cpp:
        (StringInstanceImp::hasOwnProperty):
        * kjs/string_object.h:

WebCore:

        Patch from Mark Rowe <opendarwin.org@bdash.net.nz>, reviewed by me.

- fixed http://bugzilla.opendarwin.org/attachment.cgi?id=2483

        Test cases added:
        * layout-tests/fast/js/hasOwnProperty-expected.txt: Added.
        * layout-tests/fast/js/hasOwnProperty.html: Added.
- also fixed two mozilla JS tests

        * khtml/ecma/kjs_binding.h:
        * khtml/ecma/kjs_css.cpp:
        (KJS::DOMCSSStyleDeclaration::hasOwnProperty):
        * khtml/ecma/kjs_css.h:
        * khtml/ecma/kjs_dom.cpp:
        (KJS::DOMNodeList::hasOwnProperty):
        (KJS::DOMNamedNodeMap::hasOwnProperty):
        * khtml/ecma/kjs_dom.h:
        * khtml/ecma/kjs_html.cpp:
        (KJS::HTMLDocument::hasOwnProperty):
        (KJS::KJS::HTMLElement::hasOwnProperty):
        (KJS::KJS::HTMLCollection::hasOwnProperty):
        * khtml/ecma/kjs_html.h:
        * khtml/ecma/kjs_window.cpp:
        (KJS::Window::hasOwnProperty):
        (KJS::Konqueror::hasOwnProperty):
        * khtml/ecma/kjs_window.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9441 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Changes by Darin. Layout tests by me. Reviewed by me.
ggaren [Mon, 20 Jun 2005 18:20:04 +0000 (18:20 +0000)]
    Changes by Darin. Layout tests by me. Reviewed by me.

Fixed <rdar://problem/4152454> crash in KJS::ExprStatementNode::execute(KJS::ExecState*) in World Clock widget with TOT.

        Test cases added:
        * layout-tests/fast/dom/gc-4-expected.txt: Added.
        * layout-tests/fast/dom/gc-4.html: Added.

        * khtml/ecma/kjs_html.cpp:
        (KJS::Context2D::mark): mark now (1) checks for NULL pointers and (2)
calls mark on its parent class (DOMObject::mark).

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9440 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Changes by both Anders Carlsson and me.
darin [Mon, 20 Jun 2005 05:52:16 +0000 (05:52 +0000)]
    Changes by both Anders Carlsson and me.
        Reviewed by Maciej and me.

        Test cases added:
        * layout-tests/fast/forms/element-order-expected.txt: Added.
        * layout-tests/fast/forms/element-order.html: Added.

        - fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3503>
          form.elements[] not order-preserving when elements added via DOM

        * khtml/html/html_formimpl.h: Added formElementIndex.
        * khtml/html/html_formimpl.cpp:
        (DOM::insertIntoVector): Added.
        (DOM::HTMLFormElementImpl::formElementIndex): Added. Computes an appropriate
        index for a form element, given the document position relative to the other elements.
        (DOM::HTMLFormElementImpl::registerFormElement): Use formElementIndex to determine
        where to insert the element in the formElements array.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9439 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Changes by Anders Carlsson and me.
darin [Mon, 20 Jun 2005 05:24:27 +0000 (05:24 +0000)]
    Changes by Anders Carlsson and me.
        Reviewed by Maciej and me.

        Test cases added: None. Could not figure out how to automate a test without
        making enhancements to DumpRenderTree.

        - fixed <rdar://problem/4141141> Onselect event is ignored with TEXTAREA and INPUT
          also <http://bugzilla.opendarwin.org/show_bug.cgi?id=3256>

        * khtml/html/html_formimpl.cpp: (DOM::HTMLTextAreaElementImpl::select):
        Remove unneeded explicit call to onSelect() here -- now handled by the renderer.

        * khtml/rendering/render_form.h: Added declarations as needed below.
        * khtml/rendering/render_form.cpp:
        (RenderFormElement::slotSelectionChanged): Add base class implementation that does nothing.
        (RenderLineEdit::RenderLineEdit): Connect the selection changed signal to our slot.
        (RenderLineEdit::slotSelectionChanged): Call onSelect when selection changes to something
        that's a selection (not a caret).
        (RenderTextArea::RenderTextArea): Connect as above.
        (RenderTextArea::slotSelectionChanged): Call as above.

        * kwq/KWQLineEdit.h: Added declarations as needed below.
        * kwq/KWQLineEdit.mm:
        (QLineEdit::QLineEdit): Added selectionChanged signal.
        (QLineEdit::hasSelectedText): Added.

        * kwq/KWQSlot.mm:
        (KWQSlot::KWQSlot): Make selectionChanged work on any form element.
        (KWQSlot::call): ditto.

        * kwq/KWQTextArea.h: Added hasSelection method.
        * kwq/KWQTextArea.mm:
        (-[KWQTextArea textViewDidChangeSelection:]): Added. Sends selectionChanged signal, except if
        we're in the middle of gaining or losing first responder.
        (-[KWQTextArea hasSelection]): Added.
        (-[KWQTextAreaTextView becomeFirstResponder]): Added. Sets flag to tell us we're in a responder
        change (using a count to avoid trouble if we reenter).
        (-[KWQTextAreaTextView resignFirstResponder]): Ditto.
        (-[KWQTextAreaTextView inResponderChange]): Added.

        * kwq/KWQTextEdit.h: Added declarations as needed below.
        * kwq/KWQTextEdit.mm:
        (QTextEdit::QTextEdit): Added selectionChanged signal.
        (QTextEdit::hasSelectedText): Added.

        * kwq/KWQTextField.h: Added declarations as needed below.
        * kwq/KWQTextField.mm:
        (-[KWQTextFieldController initWithTextField:QLineEdit:]): Fixed up to do the right
        thing with the result of the init function we call.
        (-[KWQTextFieldController controlTextDidEndEditing:]): Removed call to setHasFocus:NO
        since it's a little too late by the time the control is called; now handled by the
        text view delegate methods below.
        (-[KWQTextFieldController hasSelection]): Added.
        (-[KWQTextFieldController setHasFocus:]): Sets the new hasFocusAndSelectionSet after
        new focus and selection are both set up.
        (-[KWQTextFieldController textViewDidChangeSelection:]): Added. Sends the selection
        changed signal.
        (-[KWQTextField initWithQLineEdit:]): Fixed up to do the right thing with the result
        of the init function we call.
        (-[KWQTextField textViewDidChangeSelection:]): Added. Calls super and then calls the
        controller.
        (-[KWQTextField textDidEndEditing:]): Added. Calls setHasFocus:NO and then calls super.
        (-[KWQSecureTextField initWithQLineEdit:]): Same as above, but for secure text fields.
        (-[KWQSecureTextField textViewDidChangeSelection:]): Ditto.
        (-[KWQSecureTextField textDidEndEditing:]): Ditto.
        (-[KWQSearchField initWithQLineEdit:]): Same as above, but for search fields.
        (-[KWQSearchField textViewDidChangeSelection:]): Ditto.
        (-[KWQSearchField textDidEndEditing:]): Ditto.
        (-[KWQTextFieldFormatter init]): Fixed up to do the right thing with the result of
        the init function we call.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9438 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Changes by Mitz Pettel
darin [Mon, 20 Jun 2005 01:20:01 +0000 (01:20 +0000)]
Changes by Mitz Pettel
        Reviewed by me.

        - fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3466>
          ATSUI text doesn't render at coordinates greater than 32K

        * WebCoreSupport.subproj/WebTextRenderer.m: (-[WebTextRenderer _ATSU_drawRun:style:geometry:]):
        Translate the matrix of the CGContext instead of passing the appropriate coordinates to ATSU.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9437 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago * DumpRenderTree/DumpRenderTree.m: (dumpRenderTree): Add more auto-release...
darin [Sun, 19 Jun 2005 06:42:30 +0000 (06:42 +0000)]
    * DumpRenderTree/DumpRenderTree.m: (dumpRenderTree): Add more auto-release pools
        in the hope of making the tool use less memory and run faster.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9436 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Changes by Anders Carlsson.
darin [Sun, 19 Jun 2005 06:38:09 +0000 (06:38 +0000)]
    Changes by Anders Carlsson.
        Reviewed by me.

        Test cases added:
        * layout-tests/fast/dom/css-dom-read-expected.txt: Added.
        * layout-tests/fast/dom/css-dom-read.html: Added.

        - fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3534>
          CSSMutableStyleDeclarationImpl::item is unimplemented

        * khtml/css/css_valueimpl.cpp: (DOM::CSSMutableStyleDeclarationImpl::item):
        Get the style from the m_values list.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9435 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Eric Seidel.
darin [Sun, 19 Jun 2005 06:21:31 +0000 (06:21 +0000)]
    Reviewed by Eric Seidel.

        * pcre/get.c: (pcre_get_substring): Fix some computations so this works for UTF-16.
        This is unused in the current JavaScriptCore, but still good to fix.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9434 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago - undid a last minute change I made that broke a patch I landed
darin [Sun, 19 Jun 2005 06:03:41 +0000 (06:03 +0000)]
    - undid a last minute change I made that broke a patch I landed
          (layout tests caught the mistake, all should work again now)

        * khtml/html/html_formimpl.cpp:
        (DOM::HTMLLabelElementImpl::formElement): Don't include this when iterating.
        (DOM::HTMLLegendElementImpl::formElement): Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9433 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago - disabling the applet tests, since they are causing other tests to fail
darin [Sun, 19 Jun 2005 05:50:24 +0000 (05:50 +0000)]
    - disabling the applet tests, since they are causing other tests to fail

        * layout-tests/fast/replaced/applet-hspace-expected.txt: Removed.
        * layout-tests/fast/replaced/applet-hspace.html: Removed.
        * layout-tests/fast/replaced/applet-vspace-expected.txt: Removed.
        * layout-tests/fast/replaced/applet-vspace.html: Removed.
        * layout-tests/fast/replaced/resources/applet.class: Removed.
        * layout-tests/fast/replaced/resources/applet.java: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9432 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Changes by Anders Carlsson.
darin [Sun, 19 Jun 2005 05:44:29 +0000 (05:44 +0000)]
    Changes by Anders Carlsson.
        Reviewed by me.

        Test cases added:
        * layout-tests/fast/dom/resources/script3.js: Added.
        * layout-tests/fast/dom/resources/script4.js: Added.
        * layout-tests/fast/dom/resources/script5.js: Added.
        * layout-tests/fast/dom/script-add-expected.txt: Added.
        * layout-tests/fast/dom/script-add.html: Added.

        - fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3526>
          improve support for dynamically added <script> elements

        * khtml/html/html_headimpl.h: Added override of childrenChanged, new evaluateScript
        function, and m_evaluated boolean.
        * khtml/html/html_headimpl.cpp:
        (HTMLScriptElementImpl::HTMLScriptElementImpl): Init m_evaluated to false.
        (HTMLScriptElementImpl::childrenChanged): Added code to evaluate the script the first
        time a child is added (except for nodes created by parsing).
        (HTMLScriptElementImpl::insertedIntoDocument): Refactored to use the new text and
        evaluateScript functions. Also changed to not try to evaluate the script if the element
        is entirely empty. This allows you to declare an empty <script> element and fill the
        content in later using DHTML.
        (HTMLScriptElementImpl::notifyFinished): Refactored to use the new evaluateScript function.
        (HTMLScriptElementImpl::evaluateScript): Added. Checks and sets a new m_evaluated boolean
        to implement the "evaluate each script only once" rule.
        (HTMLScriptElementImpl::text): Replaced bogus implementation with one that works.
        (HTMLScriptElementImpl::setText): Ditto.

        * khtml/dom/html_head.cpp:
        (HTMLScriptElement::text): Replaced bogus implementation with one that calls through to
        the implementation class. This change doesn't really matter since we don't compile this file.
        (HTMLScriptElement::setText): Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9431 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago * khtml/html/html_formimpl.cpp: (DOM::HTMLLabelElementImpl::formElement):
darin [Sun, 19 Jun 2005 05:31:59 +0000 (05:31 +0000)]
    * khtml/html/html_formimpl.cpp: (DOM::HTMLLabelElementImpl::formElement):
        Oops, checked in a version that didn't even compile! Fixed now.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9430 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Changes by Anders Carlsson.
darin [Sun, 19 Jun 2005 05:00:25 +0000 (05:00 +0000)]
    Changes by Anders Carlsson.
        Reviewed by me.

        Test cases added:
        * layout-tests/fast/forms/focus-expected.txt: Added.
        * layout-tests/fast/forms/focus.html: Added.
        * layout-tests/fast/forms/legend-access-key-expected.txt: Added.
        * layout-tests/fast/forms/legend-access-key.html: Added.

        - fixed <rdar://problem/3481846> HTML4: accesskey not supported by HTML legend elements
          also <http://bugzilla.opendarwin.org/show_bug.cgi?id=3407>
        - added focus methods for <label> and <legend> elements
        - fixed crash when using JavaScript-created events that prevented tests from running

        * khtml/ecma/kjs_html.h: Added focus methods.
        * khtml/ecma/kjs_html.cpp: (KJS::KJS::HTMLElementFunction::tryCall): Added bindings for
        focus for <label> and <legend>. (Sorry about the merging, Dave.)

        * khtml/html/html_formimpl.h: Added new functions.
        * khtml/html/html_formimpl.cpp:
        (DOM::HTMLLabelElementImpl::focus): Added.
        (DOM::HTMLLegendElementImpl::formElement): Added.
        (DOM::HTMLLegendElementImpl::focus): Added.
        (DOM::HTMLLegendElementImpl::accessKeyAction): Added.

        * khtml/xml/dom_docimpl.cpp: (DocumentImpl::defaultEventHandler): If there's no QKeyEvent,
        then get the key name from keyIdentifier() instead of the QKeyEvent's unmodifiedText().
        The old code would just do a nil-deref.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9429 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Change by Niels Leenheer (also done by Dave Hyatt on stage at WWDC).
darin [Sun, 19 Jun 2005 03:39:56 +0000 (03:39 +0000)]
    Change by Niels Leenheer (also done by Dave Hyatt on stage at WWDC).
        Reviewed by me.

        Test cases added:
        * layout-tests/fast/replaced/object-align-hspace-vspace-expected.txt: Added.
        * layout-tests/fast/replaced/object-align-hspace-vspace.html: Added.
        * layout-tests/fast/replaced/resources/webkit-background.png: Added.

        * khtml/html/html_objectimpl.cpp:
        (DOM::HTMLObjectElementImpl::mapToEntry): Add cases for ATTR_ALIGN, ATTR_VSPACE and ATTR_HSPACE.
        (DOM::HTMLObjectElementImpl::parseMappedAttribute): Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9428 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Change by Finlay Dobbie.
darin [Sun, 19 Jun 2005 02:57:29 +0000 (02:57 +0000)]
    Change by Finlay Dobbie.
        Reviewed by me.

        - fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3331>
          10.3.9 Build Failure: NSString may not respond to `+stringWithCString:encoding:'

        * bindings/objc/WebScriptObject.mm: (-[WebScriptObject stringRepresentation]):
        Undo change we did a while back to work around the gcc 3.3 compiler error.
        It no longer seems to happen, and the workaround code was 10.4-specific.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9427 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years agoOops, forgot these files.
darin [Sun, 19 Jun 2005 02:28:37 +0000 (02:28 +0000)]
Oops, forgot these files.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9426 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Change by Niels Leenheer (also done by Dave Hyatt on stage at WWDC).
darin [Sun, 19 Jun 2005 02:27:36 +0000 (02:27 +0000)]
    Change by Niels Leenheer (also done by Dave Hyatt on stage at WWDC).
        Reviewed by me.

        Test cases added:
        * layout-tests/fast/replaced/applet-hspace-expected.txt: Added.
        * layout-tests/fast/replaced/applet-hspace.html: Added.
        * layout-tests/fast/replaced/applet-vspace-expected.txt: Added.
        * layout-tests/fast/replaced/applet-vspace.html: Added.
        * layout-tests/fast/replaced/resources/applet.class: Added.
        * layout-tests/fast/replaced/resources/applet.java: Added.

        - fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3246>
          HSPACE and VSPACE attributes are ignored in APPLET element

        * khtml/html/html_objectimpl.cpp:
        (DOM::HTMLAppletElementImpl::mapToEntry): Add cases for ATTR_VSPACE and ATTR_HSPACE.
        (DOM::HTMLAppletElementImpl::parseMappedAttribute): Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9425 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago * Scripts/cvs-apply: Improve handling of patches with CR characters in them.
darin [Sun, 19 Jun 2005 02:08:10 +0000 (02:08 +0000)]
    * Scripts/cvs-apply: Improve handling of patches with CR characters in them.
        * Scripts/cvs-unapply: Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9424 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago - removed some unused code from one of the KWQ headers
darin [Sun, 19 Jun 2005 01:58:36 +0000 (01:58 +0000)]
    - removed some unused code from one of the KWQ headers

        * kwq/KWQKHTMLPartBrowserExtension.h:

        - added a layout test for a bug that's already fixed

        * layout-tests/fast/forms/element-by-name-expected.txt: Added.
        * layout-tests/fast/forms/element-by-name.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9423 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years agoTiny formatting tweak.
darin [Sun, 19 Jun 2005 01:49:17 +0000 (01:49 +0000)]
Tiny formatting tweak.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9422 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Maciej.
vicki [Fri, 17 Jun 2005 23:02:48 +0000 (23:02 +0000)]
    Reviewed by Maciej.

        Test cases added: None, since this test requires manual interaction.

- fixed <rdar://problem/4086793> Selects wrapped by Span or Label elements
do not resize when options are added via javascript

* khtml/rendering/render_form.cpp:
        (RenderSelect::layout): remove call setting setNeedsLayout to true.  setNeedsLayout
is already true at this point, and this addtional call has the negative effect of
marking m_normalChildNeedsLayout on the element's parent, which keeps layout from
propagating all the way up to the root later on.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9421 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years agoBug #:
justing [Fri, 17 Jun 2005 21:55:39 +0000 (21:55 +0000)]
Bug #:
Submitted by:
Reviewed by:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9420 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Maciej.
adele [Fri, 17 Jun 2005 20:00:11 +0000 (20:00 +0000)]
    Reviewed by Maciej.

        - fixed http://bugzilla.opendarwin.org/show_bug.cgi?id=3419
        <rdar://problem/4111752> REGRESSION (125-312/412): Safari doesn't display edmunds.com pages first time through

        * khtml/html/htmltokenizer.cpp: (khtml::HTMLTokenizer::parseTag): set 'script' flag when
          parsing <script src="foo" /> case.  In this case, parseTag calls the scriptHandler function which expects
          that flag to be set, and sets it back to false at the appropriate time.

        Test cases added: These still don't test the cached script cases, but they're good regression
        tests for the tokenizer.  We'll have to come up with a good way to test the cached cases later.

        * layout-tests/fast/tokenizer/external-script-document-write_2-expected.txt: Added.
        * layout-tests/fast/tokenizer/external-script-document-write_2.html: Added.
        * layout-tests/fast/tokenizer/resources/empty_script.js: Added.
        * layout-tests/fast/tokenizer/script_extra_close-expected.txt: Added.
        * layout-tests/fast/tokenizer/script_extra_close.html: Added.
        * layout-tests/fast/tokenizer/script_quirk-expected.txt: Added.
        * layout-tests/fast/tokenizer/script_quirk.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9414 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years agoBug #:
justing [Fri, 17 Jun 2005 18:50:29 +0000 (18:50 +0000)]
Bug #:
Submitted by:
Reviewed by:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9413 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Patch contributed by Anders Carlsson<andersca@mac.com>.
ggaren [Fri, 17 Jun 2005 18:42:48 +0000 (18:42 +0000)]
Patch contributed by Anders Carlsson<andersca@mac.com>.

Fixes: http://bugzilla.opendarwin.org/show_bug.cgi?id=3408
        Reviewed by Darin.

        Test cases added: None. Test requires manual interaction.
Manual test available at: http://bugzilla.opendarwin.org/attachment.cgi?id=2207

        * khtml/html/html_formimpl.cpp: Eliminated HTMLButtonElementImpl::click. HTMLButtonElementImpl now inherits HTMLElementImpl::click.
        * khtml/html/html_formimpl.h: Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9412 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Patch contributed by Joos de Valk <webkit@joostdevalk.nl>.
ggaren [Fri, 17 Jun 2005 18:14:27 +0000 (18:14 +0000)]
Patch contributed by Joos de Valk <webkit@joostdevalk.nl>.

Fixes: http://bugzilla.opendarwin.org/show_bug.cgi?id=3408
        Reviewed by Darin.

        Test cases added: None. Test requires manual interaction.
Manual test available at: http://bugzilla.opendarwin.org/attachment.cgi?id=2207

        * khtml/html/html_formimpl.cpp: Eliminated HTMLButtonElementImpl::click. HTMLButtonElementImpl now inherits HTMLElementImpl::click.
        * khtml/html/html_formimpl.h: Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9410 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years agoBug #:
rjw [Fri, 17 Jun 2005 17:09:36 +0000 (17:09 +0000)]
Bug #:
Submitted by:
Reviewed by:
Changes by Mitz Pettel
        Reviewed by Richard Williamson.

Fixed http://bugzilla.opendarwin.org/show_bug.cgi?id=3545

        * WebCoreSupport.subproj/WebTextRenderer.m:
        (reverseCharactersInRun):
        (-[WebTextRenderer _ATSU_pointToOffset:style:position:reversed:includePartialGlyphs:]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9409 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by John.
darin [Fri, 17 Jun 2005 16:41:05 +0000 (16:41 +0000)]
    Reviewed by John.

        - fixed http://bugzilla.opendarwin.org/show_bug.cgi?id=3556
          black diamond question mark shown for invalid UTF-8 sequences

        Test cases added:
        * layout-tests/fast/encoding/invalid-UTF-8-expected.txt: Added.
        * layout-tests/fast/encoding/invalid-UTF-8.html: Added.

        * kwq/KWQTextCodec.mm:
        (unwanted): Added. Returns true for BOM, replacement, and null characters.
        (KWQTextDecoder::appendOmittingUnwanted): Renamed from appendOmittingNullsAndBOMs and
        changed to use the new "unwanted" function, which causes it to omit replacement characters.
        (KWQTextDecoder::convertUsingTEC): Call append function by its new name.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9408 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Chris.
sullivan [Fri, 17 Jun 2005 15:06:58 +0000 (15:06 +0000)]
    Reviewed by Chris.

        - fixed <rdar://problem/4151001> Reloading javascript-spawned window with no URL erases its contents

        * WebView.subproj/WebFrame.m:
        (-[WebFrame reload]):
        do nothing if URL is zero-length

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9407 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago - added prepare-ChangeLog script which we use internally to make ChangeLogs...
mjs [Fri, 17 Jun 2005 07:49:31 +0000 (07:49 +0000)]
    - added prepare-ChangeLog script which we use internally to make ChangeLogs, for
benefit of all WebKit hackerdom

        * Scripts/prepare-ChangeLog: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9406 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years agoAdded a few debugging methods to highlight Selections, VisiblePositions and Nodes...
justing [Fri, 17 Jun 2005 02:13:12 +0000 (02:13 +0000)]
Added a few debugging methods to highlight Selections, VisiblePositions and Nodes in the DOM tree.

Reviewed by Dave Harrison

Test cases added: None, changes are only to aid debugging

* khtml/editing/selection.cpp:
(khtml::Selection::showTree):
* khtml/editing/selection.h:
* khtml/editing/visible_position.cpp:
(khtml::VisiblePosition::showTree):
* khtml/editing/visible_position.h:
* khtml/xml/dom_nodeimpl.cpp:
(NodeImpl::showNode):
(NodeImpl::showTree):
(NodeImpl::showTreeAndMark):
* khtml/xml/dom_nodeimpl.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9405 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Fixed: <rdar://problem/4151759> 'delete' fails on variables declared inside 'eval...
ggaren [Fri, 17 Jun 2005 00:57:37 +0000 (00:57 +0000)]
Fixed: <rdar://problem/4151759> 'delete' fails on variables declared inside 'eval' statements.

        Reviewed by cblu.

        * kjs/context.h:
        (KJS::ContextImp::codeType): Added code type accessor for execution context objects.
        * kjs/internal.cpp:
        (ContextImp::ContextImp): Reflects change to ContextImp::codeType.
        * kjs/nodes.cpp:
        (VarDeclNode::evaluate): Added separate code path for variable declarations inside 'eval' statements.
        * tests/mozilla/expected.html: Updated expected test results to reflect fix.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9404 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years agoFix build bustage
hyatt [Thu, 16 Jun 2005 21:06:46 +0000 (21:06 +0000)]
Fix build bustage

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9403 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years agoFix build bustage
hyatt [Thu, 16 Jun 2005 20:33:57 +0000 (20:33 +0000)]
Fix build bustage

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9402 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Add an initial implementation of QualifiedName and HTMLNames. These classes...
hyatt [Thu, 16 Jun 2005 19:42:20 +0000 (19:42 +0000)]
    Add an initial implementation of QualifiedName and HTMLNames.  These classes are not used by anyone yet.
Progress being recorded in bugzilla bug 3405.

        Reviewed by mjs

        Test cases added: (None, does not affect build yet)

        * WebCore.pbproj/project.pbxproj:
        * khtml/xml/dom_qname.h,.cpp
        * khtml/html/htmlnames.h,.cpp

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9401 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago - recent changes to XcodeOptions made it depend on the current directory
darin [Thu, 16 Jun 2005 16:45:33 +0000 (16:45 +0000)]
    - recent changes to XcodeOptions made it depend on the current directory
          The intent was to have the WebKitBuild directory be next to WebKitTools,
          not inside the various build directories. Workaround for now is to call
          XcodeOptions when the directory is set to the WebKit directory.

        * Scripts/build-dumprendertree: Use a local variable for XcodeOptions and get it
        at the start of the script.
        * Scripts/build-webkit: Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9400 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago * Scripts/webkitdirs.pm: Recognize all 1.X versions of Xcode as old too,...
darin [Thu, 16 Jun 2005 03:41:24 +0000 (03:41 +0000)]
    * Scripts/webkitdirs.pm: Recognize all 1.X versions of Xcode as old too, not just 2.0.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9399 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago The behavior of <pre> is broken so that the newline
hyatt [Tue, 14 Jun 2005 22:37:11 +0000 (22:37 +0000)]
The behavior of <pre> is broken so that the newline
is not always properly being eaten after the open of the tag.  In addition we were eating the first newline
after all other tags.  This patch stops eating newlines after other tags and refines the behavior for pre
to catch all cases.

        Reviewed by sullivan

        Test cases added: fast/text/whitespace/030.html

        * khtml/html/htmltokenizer.cpp:
        (khtml::HTMLTokenizer::parseTag):
        (khtml::HTMLTokenizer::write):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9396 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Changes by Anders Carlsson.
darin [Tue, 14 Jun 2005 18:45:48 +0000 (18:45 +0000)]
    Changes by Anders Carlsson.
        Reviewed by me.

        - fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3496>
          Add gdb-safari script to launch Safari under gdb

        * Scripts/gdb-safari: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9395 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Changes by Devin Lane.
sullivan [Tue, 14 Jun 2005 18:06:00 +0000 (18:06 +0000)]
    Changes by Devin Lane.
        Reviewed by me.

        - fixed <rdar://problem/3766909> PDF viewing could use a zoom control other than the one in the context menu

        * WebView.subproj/WebPDFView.h:
        now implements protocol _web_WebDocumentTextSizing
        * WebView.subproj/WebPDFView.m:
        (-[WebPDFView _updateScalingToReflectTextSize]):
        new method, sets the PDF scaling from the text size multiplier
        (-[WebPDFView setDataSource:]):
        call _updateScalingToReflectTextSize
        (-[WebPDFView _web_textSizeMultiplierChanged]):
        implementation of protocol _web_WebDocumentTextSizing, calls _updateScalingToReflectTextSize

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9393 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Updated expected.html to reflect fix to <rdar://problem/4147745>.
ggaren [Tue, 14 Jun 2005 17:59:32 +0000 (17:59 +0000)]
Updated expected.html to reflect fix to <rdar://problem/4147745>.

        Reviewed by cblu.

        * tests/mozilla/expected.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9392 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Reviewed by Dave Harrison.
sullivan [Tue, 14 Jun 2005 17:36:32 +0000 (17:36 +0000)]
    Reviewed by Dave Harrison.

        * WebView.subproj/WebHTMLView.m:
        (-[WebHTMLView _delayedEndPrintMode:]):
        After discussing this with Dave, I made this method both more debugger-friendly with asserts for
        the cases we don't think could ever happen, and more paranoid by handling these cases in deployment
        builds.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9391 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago Fixed: <rdar://problem/4147745> JavaScript discards locally defined "arguments"...
ggaren [Tue, 14 Jun 2005 16:36:11 +0000 (16:36 +0000)]
Fixed: <rdar://problem/4147745> JavaScript discards locally defined "arguments" property

No layout tests added because this change fixes existing tests:
ecma/ExecutionContexts/10.1.6.js
        ecma_3/Function/regress-94506.js
        js1_4/Functions/function-001.js

        Reviewed by cblu.

        * kjs/function.cpp:
        (KJS::ActivationImp::get): get now checks for an "arguments" property defined in the local variable object
before trying to return the built-in arguments array.

        * kjs/function.h: ActivationImp::put no longer overrides ObjectImp::put

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9390 268f45cc-cd09-0410-ab3c-d52691b4dbfc

17 years ago - fixed build for Xcode 2.1
darin [Tue, 14 Jun 2005 16:19:16 +0000 (16:19 +0000)]
    - fixed build for Xcode 2.1

        * WebKit.pbproj/project.pbxproj: Use BUILT_PRODUCTS_DIR instead of SYMROOT to search for the
        WebKitSystemInterface.h file. We could re-jigger this again later, but for now this is consistent
        with both the .a file's location and where build-webkit puts the file.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@9389 268f45cc-cd09-0410-ab3c-d52691b4dbfc