1. 23 Jan, 2007 10 commits
    • staikos's avatar
      Temporary measure to make it possible to use the Qt port until the official Qt · 9fd5a74a
      staikos authored
      API is done
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19048 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      9fd5a74a
    • staikos's avatar
      Compile with a released Qt. · 309162be
      staikos authored
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19047 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      309162be
    • lars's avatar
      Reviewed by Zack · 441f8272
      lars authored
              Widget and ScrollView do not own the Qt widgets
              anymore. Make sure they can handle the case of
              a null QWidget and never delete them.
      
              Fix refcounting of FrameViews (they get created with a
              refcount of 1) and add a few safety checks to the 
              FrameLoaderClientQt.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19046 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      441f8272
    • zack's avatar
      Forgot to add this file to the build (fix the build for real) · c42e7c9f
      zack authored
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19045 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      c42e7c9f
    • zack's avatar
      Adjust the DragClient to latest changes. · c03f0c71
      zack authored
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19044 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      c03f0c71
    • zack's avatar
      Fix the Qt build · 1bd89127
      zack authored
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19043 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      1bd89127
    • lars's avatar
      Reviewed by Maciej · 1e1dad3d
      lars authored
              Make the last remaining pieces of the FrameLoader platform 
              independent. Move most of the code over to WebFrameLoaderClient.
              Some smaller cleanups in the WebFrameBridge, and moved some
              platform independent functionality over to the shared code
              in WebCore.
      
              Add support for Frames to the Qt build and fix some issues
              in the API classes.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19042 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      1e1dad3d
    • oliver's avatar
      2007-01-23 Oliver Hunt <oliver@apple.com> · 609a7d0e
      oliver authored
              Reviewed by Adam.
      
              Qt build stubs for Drop logic
      
              * WebCoreSupport/DragClientQt.cpp: Added.
              (WebCore::DragClientQt::actionMaskForDrag):
              (WebCore::DragClientQt::willPerformDragDestinationAction):
              * WebCoreSupport/DragClientQt.h: Added.
              * WebKitPart/WebKitPart.cpp:
              (WebKitPart::initView):
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19041 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      609a7d0e
    • oliver's avatar
      2007-01-23 Oliver Hunt <oliver@apple.com> · 67ff178d
      oliver authored
              Reviewed by Adam.
      
              Drop logic bindings for WebKit
      
              * WebCoreSupport/WebDragClient.h: Added.
              * WebCoreSupport/WebDragClient.mm: Added.
              (WebDragClient::WebDragClient):
              (WebDragClient::actionMaskForDrag):
              (WebDragClient::willPerformDragDestinationAction):
                Standard client impl
              * WebCoreSupport/WebFrameBridge.mm:
              (-[WebFrameBridge startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
                Updated to use DragController to track drag state
          
              * WebCoreSupport/WebPasteboardHelper.h: Added.
              (WebPasteboardHelper::WebPasteboardHelper):
                A *temporary* Helper class to access NSPasteboard access and 
                manipulation functions present in WebKit
              * WebCoreSupport/WebPasteboardHelper.mm: Added.
              (WebPasteboardHelper::urlFromPasteboard):
              (WebPasteboardHelper::plainTextFromPasteboard):
              (WebPasteboardHelper::fragmentFromPasteboard):
              (WebPasteboardHelper::insertablePasteboardTypes):
                See header comment
              * WebKit.xcodeproj/project.pbxproj:
              * WebView/WebDocumentInternal.h:
                Remove unnecessary protocol
              * WebView/WebFrameView.mm:
              (-[WebFrameView _setDocumentView:]):
                Updating to use DragController to track drag state
              * WebView/WebHTMLView.mm:
              (-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
                ditto
              (-[WebHTMLView dragImage:at:offset:event:pasteboard:source:slideBack:]):
                ditto
              (-[WebHTMLView draggingSourceOperationMaskForLocal:]):
                ditto
              (-[WebHTMLView draggedImage:endedAt:operation:]):
                ditto          
              (-[WebHTMLView _documentFragmentForPasteboard:]):
                Helper method to generate DocumentFragment from NSPasteboard without regressing 
              (-[WebHTMLView _canProcessDragWithDraggingInfo:]):
                Updating to use DragController to track drag state
              (-[WebHTMLView _isMoveDrag:]):
              (-[WebHTMLView _isNSColorDrag:]):
              * WebView/WebHTMLViewInternal.h:
                Removing unnecessary fields and methods
              * WebView/WebView.mm:
              (-[WebViewPrivate dealloc]):
                Remove obsolete ASSERT
              (-[WebView _setInitiatedDrag:]):
                Now passes directly through to DragController
              (-[WebView _commonInitializationWithFrameName:groupName:]):
              (-[WebView _loadingDragOperationForDraggingInfo:]):
                Removed
              (-[WebView draggingEntered:]):
                Updated to use DragController
              (-[WebView draggingUpdated:]):
                ditto
              (-[WebView draggingExited:]):
                ditto
              (-[WebView performDragOperation:]):
                ditto
              (-[WebView _hitTest:dragTypes:]):
              * WebView/WebViewInternal.h:
                remove unnecessary method def
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19040 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      67ff178d
    • oliver's avatar
      2007-01-23 Oliver Hunt <oliver@apple.com> · aeffa297
      oliver authored
              Reviewed by Adam.
      
              Added DragController to handled drag and drop state and control logic.
              Currently this only handles logic for Dragging and Dropping, and is 
              unable to initialise a drag itself.
      
              platform/DragData encapsulate the paltform Drag data (NSDraggingInfo on Mac, 
              QMimeType in Qt).  This provides methods to pull data out the drag data as a String, 
              a URL, a DocumentFragment (for rich content), or a Colour.
      
              * WebCore.exp:
              * WebCore.pro:
              * WebCore.xcodeproj/project.pbxproj:
              * dom/Clipboard.cpp: Added.
              (WebCore::Clipboard::setAccessPolicy):
              (WebCore::dragOpFromIEOp):
              (WebCore::IEOpFromDragOp):
              (WebCore::Clipboard::sourceOperation):
              (WebCore::Clipboard::destinationOperation):
              (WebCore::Clipboard::setSourceOperation):
              (WebCore::Clipboard::setDestinationOperation):
              (WebCore::Clipboard::setDropEffect):
              (WebCore::Clipboard::setEffectAllowed):
                move platform independent functions from ClipboardMac to Clipboard
      
              * dom/Clipboard.h:
              (WebCore::Clipboard::Clipboard):
              (WebCore::Clipboard::dropEffect):
              (WebCore::Clipboard::effectAllowed):
              (WebCore::Clipboard::policy):
                as above -- also removed virtual modifier from now 
                platform independent methods
      
              * editing/Editor.cpp:
              (WebCore::Editor::smartInsertDeleteEnabled): 
              (WebCore::Editor::shouldApplyStyle):
              * editing/Editor.h:
                adding calls through to EditorClient
               
              * page/DragActions.h: Added.
              (WebCore::):
                Replicates a number of NS* enums in the WebCore namespace
      
              * page/DragClient.h: Added.
              (WebCore::DragClient::~DragClient):
                Standard client interface for dragging UIDelegate
      
              * page/DragController.cpp: Added.
              (WebCore::createMouseEvent):
              (WebCore::DragController::DragController):
              (WebCore::DragController::~DragController):
              (WebCore::documentFragmentFromDragData):
              (WebCore::DragController::isMoveDrag):
              (WebCore::visiblePositionForPoint):
              (WebCore::DragController::cancelDrag):
              (WebCore::documentAtPoint):
              (WebCore::DragController::dragEntered):
              (WebCore::DragController::dragExited):
              (WebCore::DragController::dragUpdated):
              (WebCore::DragController::performDrag):
              (WebCore::DragController::dragEnteredOrUpdated):
              (WebCore::DragController::tryDocumentDrag):
              (WebCore::DragController::operationForLoad):
              (WebCore::DragController::concludeDrag):
              (WebCore::DragController::canProcessDrag):
              (WebCore::DragController::tryDHTMLDrag):
                Class to handle state and logic for dragging and dropping
      
              * page/DragController.h: Added.
              (WebCore::DragController::client):
              (WebCore::DragController::didInitiateDrag):
              (WebCore::DragController::isHandlingDrag):
              (WebCore::DragController::dragOperation):
              (WebCore::DragController::document):
              (WebCore::DragController::dragInitiator):
              (WebCore::DragController::dragDestinationAction):
              (WebCore::DragController::dragSourceAction):
              (WebCore::DragController::dragEnded):
              (WebCore::DragController::setIsHandlingDrag):
              (WebCore::DragController::setDidInitiateDrag):
              (WebCore::DragController::setDragOperation):
              (WebCore::DragController::setDragSourceAction):
              (WebCore::DragController::setDragInitiator):
                See above
      
              * page/Page.cpp:
              (WebCore::Page::Page):
                Update Page to have DragController
      
              * page/Page.h:
              (WebCore::Page::dragController):
                ditto
      
              * page/mac/DragControllerMac.mm: Added.
              (WebCore::DragController::isCopyKeyDown):
              (WebCore::DragController::dragOperation):
                Platform specific portions of the DragController
      
              * page/mac/EventHandlerMac.mm:
              (WebCore::EventHandler::handleDrag):
              (WebCore::EventHandler::dragSourceEndedAt):
                Convert from NSDragOperation to WebCore::DragOperation
      
              * page/mac/WebCoreFrameBridge.h:
              * page/mac/WebCoreFrameBridge.mm:
                Remove obsolete Drag functions
      
              * page/qt/DragControllerQt.cpp: Added.
              (WebCore::DragController::isCopyKeyDown):
              (WebCore::DragController::dragOperation):
                Basic stubs to maintain Qt  build
      
              * platform/DragData.cpp: Added.
              (WebCore::DragData::DragData):
              * platform/DragData.h: Added.
              (WebCore::DragData::clientPosition):
              (WebCore::DragData::globalPosition):
              (WebCore::DragData::platformData):
              (WebCore::DragData::draggingSourceOperationMask):
                DragData class to encapsulate platform drag data/event
      
              * platform/graphics/svg/SVGImage.cpp:
              (WebCore::SVGImage::setData):
                Addition of dummy DragClient
      
              * platform/graphics/svg/SVGImageEmptyClients.h:
              (WebCore::SVGEmptyDragClient::~SVGEmptyDragClient):
              (WebCore::SVGEmptyDragClient::willPerformDragDestinationAction):
              (WebCore::SVGEmptyDragClient::actionMaskForDrag):
              (WebCore::SVGEmptyDragClient::dragControllerDestroyed):
                ditto
      
              * platform/mac/ClipboardMac.h:
              * platform/mac/ClipboardMac.mm:
              (WebCore::ClipboardMac::ClipboardMac):
              (WebCore::ClipboardMac::clearData):
              (WebCore::ClipboardMac::clearAllData):
              (WebCore::ClipboardMac::getData):
              (WebCore::ClipboardMac::setData):
              (WebCore::ClipboardMac::types):
              (WebCore::ClipboardMac::setDragImage):
               Moving platform independent logic to Clipboard, and updating
               platform specific methods to use appropriate accessors rather than 
               directly manipulating data they no longer own
      
              * platform/mac/DragDataMac.mm: Added.
              (WebCore::DragData::DragData):
              (WebCore::DragData::canSmartReplace):
              (WebCore::DragData::containsColor):
              (WebCore::DragData::containsPlainText):
              (WebCore::DragData::asPlainText):
              (WebCore::DragData::asColor):
              (WebCore::DragData::createClipboard):
              (WebCore::imageExistsAtPaths):
              (WebCore::DragData::containsCompatibleContent):
              (WebCore::DragData::containsURL):
              (WebCore::DragData::asURL):
              (WebCore::DragData::asFragment):
                Mac implementations of DragData methods.  A number of these use
                a Helper class that accesses WebKit functionality.  That functionality 
                should be migrated to WebCore in the future.
      
              * platform/mac/PasteboardHelper.h: Added.
              (WebCore::PasteboardHelper::~PasteboardHelper):
                Temporary, and Mac only, helper class to access WebKit functionality 
                from WebCore
      
              * platform/qt/DragDataQt.cpp: Added.
              (WebCore::DragData::canSmartReplace):
              (WebCore::DragData::containsColor):
              (WebCore::DragData::containsPlainText):
              (WebCore::DragData::asPlainText):
              (WebCore::DragData::asColor):
              (WebCore::DragData::createClipboard):
              (WebCore::DragData::containsCompatibleContent):
              (WebCore::DragData::containsURL):
              (WebCore::DragData::asURL):
              (WebCore::DragData::asFragment):
               Basic stubs to maintain Qt build
      
              * rendering/HitTestResult.cpp:
              (WebCore::HitTestResult::HitTestResult):
               Correct HitTestResult copy contructor to copy localPoint
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19039 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      aeffa297
  2. 22 Jan, 2007 14 commits
    • bdash's avatar
      2007-01-22 Mark Rowe <mrowe@apple.com> · fa8c5eca
      bdash authored
              Reviewed by Maciej.
      
              http://bugs.webkit.org/show_bug.cgi?id=11692
              Bug 11692: REGRESSION(r17352): Disappearing page title
      
              * dom/Document.cpp:
              (WebCore::Document::setTitle): If the title has already been set explicitly via JavaScript,
              ignore any changes to it due to encountering <title> tags.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19038 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      fa8c5eca
    • weinig's avatar
      LayoutTests: · f98f37cd
      weinig authored
              Reviewed by Darin.
      
              - test for http://bugs.webkit.org/show_bug.cgi?id=12344
                NativeListBox: item hit testing does not account for top padding and border
      
              * fast/forms/listbox-selection-2-expected.checksum: Added.
              * fast/forms/listbox-selection-2-expected.png: Added.
              * fast/forms/listbox-selection-2-expected.txt: Added.
              * fast/forms/listbox-selection-2.html: Added.
      
      WebCore:
      
              Reviewed by Darin.
      
              - fix http://bugs.webkit.org/show_bug.cgi?id=12344
                NativeListBox: item hit testing does not account for top padding and border
      
              Test: fast/forms/listbox-selection-2.html
      
              * rendering/RenderListBox.cpp:
              (WebCore::RenderListBox::numVisibleItems): Changed height() to contentHeight()
              since items are visible only in the content box.
              (WebCore::RenderListBox::listIndexAtOffset): Adjusted for vertical padding and
              borders. Changed to return -1 instead of the last item's index if the given offset
              is below the last item.
              (WebCore::RenderListBox::autoscroll): Adjusted for vertical padding and borders.
              (WebCore::RenderListBox::controlClipRect): Changed to return the content box since
              items should not spill into the padding box. This change is the reason the test
              generates pixel results.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19037 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      f98f37cd
    • justing's avatar
      LayoutTests: · 0982ef50
      justing authored
              Reviewed by darin
      
              <rdar://problem/4944770>
              editing/pasteboard/4641033 layout test is broken (12328)
      
              The fixes not only make this test pass, but also fix a bug that 
              was reflected in the results checked in for this test.  The caret
              was in the wrong position:
              * editing/pasteboard/4641033-expected.checksum:
              * editing/pasteboard/4641033-expected.png:
              * editing/pasteboard/4641033-expected.txt:
              Added:
              * editing/pasteboard/4944770-1-expected.checksum: Added.
              * editing/pasteboard/4944770-1-expected.png: Added.
              * editing/pasteboard/4944770-1-expected.txt: Added.
              * editing/pasteboard/4944770-1.html: Added.
              * editing/pasteboard/4944770-2-expected.checksum: Added.
              * editing/pasteboard/4944770-2-expected.png: Added.
              * editing/pasteboard/4944770-2-expected.txt: Added.
              * editing/pasteboard/4944770-2.html: Added.
              * editing/pasteboard/4944770-expected.checksum: Added.
              * editing/pasteboard/4944770-expected.png: Added.
              * editing/pasteboard/4944770-expected.txt: Added.
              Equivalent render trees:
              * editing/execCommand/format-block-from-range-selection-expected.txt:
              * editing/pasteboard/drag-drop-modifies-page-expected.txt:
              Fixed:
              * editing/pasteboard/nested-blocks-with-text-area-expected.checksum:
              * editing/pasteboard/nested-blocks-with-text-area-expected.png:
              * editing/pasteboard/nested-blocks-with-text-area-expected.txt:
              * editing/pasteboard/nested-blocks-with-text-field-expected.checksum:
              * editing/pasteboard/nested-blocks-with-text-field-expected.png:
              * editing/pasteboard/nested-blocks-with-text-field-expected.txt:
      
      WebCore:
      
              Reviewed by darin
              
              <rdar://problem/4944770>
              editing/pasteboard/4641033 layout test is broken (12328)
              
              This layout test failure demonstrates three bugs:
              Smart replace shouldn't be turned on, but is because a word
              selection granularity from the previous test isn't cleared.
              Smart replace shouldn't add any spaces because the paste is performed
              in an empty paragraph.
              Smart replace spaces are added to the text of options inside
              the select element instead of before/after the select element.
              
              This patch fixes the second two problems.  During a 
              ReplaceSelectionCommand, the VisiblePosition for the end of 
              the inserted content is the last position in the last leaf 
              inserted.  This is a problem when the last leaf is inside a 
              select element, since VP creation is inconsistent there.  
              Second, smart spaces were inserted inside the last leaf, not
              at the endOfInsertedContent.
      
              * editing/ReplaceSelectionCommand.cpp:
              (WebCore::ReplaceSelectionCommand::shouldMergeStart): Renamed 
              m_lastNodeInserted to m_lastLeafInserted.  Use the 
              start/endOfInsertedContent getters.
              (WebCore::ReplaceSelectionCommand::shouldMergeEnd): Ditto.
              (WebCore::ReplaceSelectionCommand::removeNodePreservingChildren): Ditto.
              (WebCore::ReplaceSelectionCommand::removeRedundantStyles): Ditto.
              (WebCore::ReplaceSelectionCommand::positionAtEndOfInsertedContent): Added.
              Special case for when the last leaf inserted is inside a select 
              element: return the VisiblePosition after the select element.
              (WebCore::ReplaceSelectionCommand::positionAtStartOfInsertedContent): Added.
              (WebCore::ReplaceSelectionCommand::doApply): Did renaming.  Used
              the start/endOfInsertedContent getters.  Insert smart replace spaces
              into the right nodes.
              (WebCore::ReplaceSelectionCommand::shouldRemoveEndBR): Renaming.
              (WebCore::ReplaceSelectionCommand::completeHTMLReplacement): Ditto.
              (WebCore::ReplaceSelectionCommand::updateNodesInserted): Ditto.
              * editing/ReplaceSelectionCommand.h:
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19036 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      0982ef50
    • darin's avatar
      Reviewed by Adam. · 03488715
      darin authored
              - fix http://bugs.webkit.org/show_bug.cgi?id=12273
                REGRESSION: File input value invisible after removing and re-appending
      
              - made more of the file chooser code cross-platform
      
              - fixed an Objective-C garbage collection problem in FileChooser
      
              - tried to remove layering violations where FileChooser in the platform
                directory knows about the rendering and DOM trees (but wasn't able to
                do it entirely)
      
              * platform/FileChooser.cpp: Added.
              (WebCore::FileChooser::create): Changed parameters.
              (WebCore::FileChooser::chooseFile): Changed to call chooseIcon function.
              (WebCore::FileChooser::chooseIcon): Added.
      
              * platform/FileChooser.h: Added FileChooserClient so this file doesn't have
              to know about RenderFileUploadControl. Used RetainPtr instead of retain/release
              so this works properly with Objective-C GC. Replaced disconnectUploadControl
              with disconnectClient, which has a simple inline implementation. Moved the
              Document parameter from the constructor to openFileChooser, along with a comment
              about why it should not be there. Added a filename parameter to the constructor
              for the initial filename. Added a font parameter to basenameForWidth so we don't
              depend on having a pointer to the renderer. Removed the uploadControl() and
              document() functions since no one calls either of them.
      
              * platform/mac/FileChooserMac.mm: Removed code that is now cross-platform.
              (-[OpenPanelController beginSheetWithFrame:]): Added frame parameter, instead
              of storing a document pointer in the FileChooser.
              (WebCore::FileChooser::FileChooser): Updated for changed parameters. Also added
              code to choose the icon based on the initial filename. Pass the adopt parameter
              to the constructor of the RetainPtr.
              (WebCore::FileChooser::~FileChooser): Removed the release call, since the
              RetainPtr will take care of it. Added a call to disconnectFileChooser here.
              There's no need to disconnect at disconnectClient time, and that lets us have
              one more cross-platform function.
              (WebCore::FileChooser::openFileChooser): Added document parameter. Added code
              to get the frame from the document. And added a nil check since there's no
              ironclad guarantee the document won't have outlived its frame.
              (WebCore::FileChooser::basenameForWidth): Added font parameter. Use that instead
              of going at the upload control's style. The caller can handle that now.
      
              * rendering/RenderFileUploadControl.h: Removed unnecessary includes.
              Made class inherit privately from FileChooserClient. Changed constructor
              parameter to HTMLInputElement rather than Node. Made protected members be
              private instead, and made a couple function members const.
              * rendering/RenderFileUploadControl.cpp: Moved constants to the top of the file,
              but after the "using namespace"
              (WebCore::RenderFileUploadControl::RenderFileUploadControl): Updated for changes
              to the FileChooser::create function and the parameter types.
              (WebCore::RenderFileUploadControl::~RenderFileUploadControl): Removed unneeded
              null check -- there's no case where the FileChooser fails to be created.
              (WebCore::RenderFileUploadControl::click): Pass document to openFileChooser.
              (WebCore::RenderFileUploadControl::updateFromElement): Tweaked code a bit by
              using updateFromElement and setValue instead of casting the renderer to a
              RenderButton and calling setText directly. Put setInputType call here.
              (WebCore::RenderFileUploadControl::maxFilenameWidth): Made const. Broke long line.
              (WebCore::RenderFileUploadControl::createButtonStyle): Made const.
              (WebCore::RenderFileUploadControl::paintObject): Moved buttonShadowHeight constant
              to the top of the file. Call isEmpty instead of checking width and height of
              IntRect for 0. Computed font to pass to FileChooser::basenameForWidth. Broke
              long lines.
              (WebCore::HTMLFileUploadInnerButtonElement::HTMLFileUploadInnerButtonElement):
              Moved the setInputType call to the caller; concept here is that this class is
              as simple as possible, rather than doing as much of the button job as possible.
      
              * WebCore.xcodeproj/project.pbxproj: Added FileChooser.cpp.
              * WebCore.pro: Ditto.
              * CMakeLists.txt: Ditto.
              * WebCoreSources.bkl: Ditto.
      
              * platform/gdk/TemporaryLinkStubs.cpp: Updated stubs.
              * platform/qt/FileChooserQt.cpp: Updated stubs.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19035 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      03488715
    • bdakin's avatar
      Reviewed by Darin. · cc931d0a
      bdakin authored
              Darin and I looked into these tests that are adding an empty 
              RenderText, and we think they just need new results. Here they are!
      
              * editing/execCommand/find-after-replace-expected.txt:
              * fast/css/beforeSelectorOnCodeElement-expected.txt:
              * fast/forms/control-restrict-line-height-expected.txt:
              * fast/forms/select-size-expected.txt:
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19034 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      cc931d0a
    • weinig's avatar
      LayoutTests: · fb8a8a78
      weinig authored
              Reviewed by Darin.
      
              - pixel test for http://bugs.webkit.org/show_bug.cgi?id=12364
                REGRESSSION (NativeListBox): Selected option's background is not clipped to the list box
      
              * fast/forms/select-item-background-clip-expected.checksum: Added.
              * fast/forms/select-item-background-clip-expected.png: Added.
              * fast/forms/select-item-background-clip-expected.txt: Added.
              * fast/forms/select-item-background-clip.html: Added.
      
      WebCore:
      
              Reviewed by Darin.
      
              - fix http://bugs.webkit.org/show_bug.cgi?id=12364
                REGRESSSION (NativeListBox): Selected option's background is not clipped to the list box
      
              Test: fast/forms/select-item-background-clip.html
      
              * rendering/RenderListBox.cpp:
              (WebCore::RenderListBox::paintItemBackground): Intersect the item's rect with
              the list box's clip rect.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19033 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      fb8a8a78
    • sullivan's avatar
      * WebView/WebHTMLView.mm: · b394adc1
      sullivan authored
              (-[WebHTMLView _lookUpInDictionaryFromMenu:]):
              Tiger build fix: remove unused variable for return value of dictionaryServiceWindowShow
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19032 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      b394adc1
    • darin's avatar
      Reviewed by Beth. · ca718e15
      darin authored
              - fix http://bugs.webkit.org/show_bug.cgi?id=12369
                REGRESSION (r19027): amazon.com home page crashes WebKit r19028 debug build
      
              I caused this a few hours ago. Oops!
      
              * rendering/RenderText.cpp: (WebCore::RenderText::setStyle):
              Compute whether the font changed before calling RenderObject::setStyle.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19031 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      ca718e15
    • sullivan's avatar
      Reviewed by Adam and Darin · 97833a23
      sullivan authored
              - fixed <rdar://problem/4794320> "Look Up in Dictionary" does nothing in WebKit 
              (need to adopt new API)
      
              * Misc/WebNSURLExtras.m:
              (-[NSString _web_isUserVisibleURL]):
              random typo correction in comment
      
              * English.lproj/StringsNotToBeLocalized.txt:
              updated for these changes
      
              * WebView/WebHTMLView.mm:
              (coreGraphicsScreenPointForAppKitScreenPoint):
              new function to convert an AppKit screen point to a CG screen point
              (-[WebHTMLView _lookUpInDictionaryFromMenu:]):
              on Leopard now uses new API. There's something of an impedance mismatch between
              this API and WebKit, but that was true for the SPI we were using in Tiger also.
              Bug 4945808 covers the ways in which this is not perfect.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19030 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      97833a23
    • bdakin's avatar
      Reviewed by Maciej. · 5df49692
      bdakin authored
              Fix for <rdar://problem/4912129> REGRESSION: Crash occurs at 
              WebCore::Frame::eventHandler() after clicking to dismiss a 
              contextual menu on the page
      
              * page/EventHandler.cpp:
              (WebCore::EventHandler::stopAutoscrollTimer): Added nil-check.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19029 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      5df49692
    • darin's avatar
      Reviewed by Mitz. · 8a4fb2be
      darin authored
              - a couple tiny tweaks to make --reset-results work better
      
              * Scripts/run-webkit-tests: Always generate results when --reset-results is specified.
              Don't make a separate "new" entry while generating results when --reset-results
              is specified since new results are the norm in that case.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19028 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      8a4fb2be
    • darin's avatar
      LayoutTests: · 42563ac5
      darin authored
              Test by Andrew Wellington <proton@wiretapped.net>.
              Reviewed by Darin.
              
              - test for fix to http://bugs.webkit.org/show_bug.cgi?id=11197
                REGRESSION: Specifying a counter for a CODE tag's content style
                property on before or after causes a crash.
      
              * fast/css/beforeSelectorOnCodeElement-expected.checksum: Added.
              * fast/css/beforeSelectorOnCodeElement-expected.png: Added.
              * fast/css/beforeSelectorOnCodeElement-expected.txt: Added.
              * fast/css/beforeSelectorOnCodeElement.html: Added.
      
              Test by Mitz.
              Reviewed by Darin.
      
              - test for monospace problem mentioned in the above bug
              
              * fast/text/monospace-width-cache-expected.checksum: Added.
              * fast/text/monospace-width-cache-expected.png: Added.
              * fast/text/monospace-width-cache-expected.txt: Added.
              * fast/text/monospace-width-cache.html: Added.
      
      WebCore:
      
              Reviewed by Mitz.
      
              Inspired by a much simpler patch by Andrew Wellington <proton@wiretapped.net>.
      
              - fix http://bugs.webkit.org/show_bug.cgi?id=11197
                REGRESSION: Specifying a counter for a CODE tag's content style property
                on before or after causes a crash.
      
              Test: fast/css/beforeSelectorOnCodeElement.html
      
              - fix regression I recently introduced where the monospace cache could be
                used for text that was not ASCII, if text was changed after the style was set
      
              Test: fast/text/monospace-width-cache.html
      
              - streamline RenderText.h -- remove unneeded functions, make some needlessly
                virtual functions non-virtual, remove unneeded header includes, replace use
                of Font* with Font& for consistency with RenderStyle, use "text" consistently
                instead of "string"
      
              * rendering/RenderText.h: Removed unneeded include of "Text.h", declarations of
              SOFT_HYPHEN, DocumentMarker, InlineBox, Position, String. Got rid of friend
              declaration for InlineTextBox. Changed constructor to take a PassRefPtr since
              we do take owership of the passed-in text. Made deleteTextBoxes() private.
              Removed unneeded data(). Made override of length() private so people will use
              the faster textLength() instead. Renamed string() to text() so it matches up
              better with setText(). Same for stringLength(), originalString(), and
              setInternalString(), which are now textLength(), originalText(), and
              setTextInternal(). Renamed the old text() function to characters(), since it
              returns a pointer as StringImpl::characters does. Changed Font* to Font& everywhere.
              Renamed the internal calcMinMaxWidth() function to calcMinMaxWidthInternal() and
              made it non-virtual since it's not overriden anywhere. Also made the
              trimmedMinMaxWidth() function non-virtual. Made containsOnlyWhitespace() private.
              Removed both font() functions (one was needlessly virtual) and element().
              Renamed cacheWidths() to updateMonospaceCharacterWidth() and removed the
              shouldUseMonospaceCache() function. Renamed allAscii() to isAllASCII() and made
              it an inline function. Removed the m_allAsciiChecked flag and renamed the
              m_allAscii flag to m_isAllASCII.
              * rendering/RenderText.cpp:
              (WebCore::RenderText::RenderText): Removed initialization of m_allAsciiChecked.
              Initialize m_isAllASCII. Changed all occurrences of m_str->length() and
              m_str->characters () to use the stringLength() and characters() inline functions
              instead.
              (WebCore::RenderText::setStyle): Replaced the unconditional call to cacheWidths
              with a conditional call to updateMonospaceCharacterWidth. This speeds up the common
              case where style is changed without changing the font.
              (WebCore::RenderText::originalText): Changed code to not depend on overridden
              element() function with casts to a Text node -- this was one of only two call sites.
              Also renamed.
              (WebCore::RenderText::positionForCoordinates): Name change.
              (WebCore::RenderText::caretRect): Ditto.
              (WebCore::RenderText::updateMonospaceCharacterWidth): Renamed from cacheWidths and
              incorporate the check that was formerly in a separate shouldUseMonospaceCache
              function.
              (WebCore::RenderText::widthFromCache): Changed to take a Font& instead of Font*.
              Changed to take advantage of the fact that the string is known to be all ASCII in
              the m_monospaceCharacterWidth code path.
              (WebCore::RenderText::trimmedMinMaxWidth): Ditto. Also use style()->font() instead
              of font(false).
              (WebCore::RenderText::calcMinMaxWidth): Ditto. Also changed to use softHyphen instead
              of SOFT_HYPHEN.
              (WebCore::RenderText::containsOnlyWhitespace): More of the same.
              (WebCore::RenderText::setSelectionState): Ditto.
              (WebCore::RenderText::setTextWithOffset): Ditto.
              (WebCore::isInlineFlowOrEmptyText): Ditto.
              (WebCore::RenderText::setTextInternal): Updated for name changes. Also changed to
              set the m_isAllASCII flag every time, in the more-efficient way we use elsewhere
              (or'ing all the characters together), and to call updateMonospaceCharacterWidth
              if the "all ASCII"-ness of the text changes.
              (WebCore::RenderText::setText): Updated for name changes.
              (WebCore::RenderText::width): More like above.
              (WebCore::RenderText::selectionRect): Ditto.
              (WebCore::RenderText::caretMaxOffset): Ditto.
              (WebCore::RenderText::previousOffset): Ditto.
              (WebCore::RenderText::nextOffset): Ditto.
      
              * dom/Position.cpp:
              * dom/Range.cpp:
              * editing/CompositeEditCommand.cpp:
              * editing/DeleteSelectionCommand.cpp:
              * editing/VisiblePosition.cpp:
              * html/HTMLElement.cpp:
              Added now-needed include of "Text.h" which is no longer included by "RenderText.h".
      
              * editing/SelectionController.cpp:
              (WebCore::SelectionController::debugRenderer):
              * editing/TextIterator.cpp:
              (WebCore::TextIterator::handleTextNode):
              (WebCore::TextIterator::handleTextBox):
              (WebCore::SimplifiedBackwardsTextIterator::handleTextNode):
              Update for name changes of stringLength(), string(), and originalString() to
              textLength(), text(), and originalText(), and to always use textLength() instead
              of length() when we have a RenderText* rather than a RenderObject*.
              * editing/visible_units.cpp:
              (WebCore::startOfParagraph): Ditto.
              (WebCore::endOfParagraph): Ditto.
              * rendering/InlineFlowBox.cpp:
              (WebCore::InlineFlowBox::placeBoxesHorizontally): Ditto. Also removed use of the
              RenderObject::font() function and replaced it with explicit access to the style.
              (WebCore::InlineFlowBox::placeBoxesVertically): Ditto.
              * rendering/InlineTextBox.cpp:
              (WebCore::InlineTextBox::selectionRect): Ditto.
              (WebCore::InlineTextBox::isLineBreak): Ditto.
              (WebCore::InlineTextBox::paint): Ditto.
              (WebCore::InlineTextBox::selectionStartEnd): Ditto.
              (WebCore::InlineTextBox::paintSelection): Ditto.
              (WebCore::InlineTextBox::paintMarkedTextBackground): Ditto.
              (WebCore::InlineTextBox::paintSpellingOrGrammarMarker): Ditto.
              (WebCore::InlineTextBox::paintTextMatchMarker): Ditto.
              (WebCore::InlineTextBox::offsetForPosition): Ditto.
              (WebCore::InlineTextBox::positionForOffset): Ditto.
              * rendering/RenderContainer.cpp:
              (WebCore::RenderContainer::addChild): Ditto.
              * rendering/RenderCounter.cpp:
              (WebCore::RenderCounter::originalText): Ditto.
              (WebCore::RenderCounter::calcMinMaxWidth): Ditto.
              * rendering/RenderCounter.h: Ditto.
              * rendering/RenderTextFragment.cpp:
              (WebCore::RenderTextFragment::originalText): Ditto.
              * rendering/RenderTextFragment.h: Ditto.
              * rendering/SVGInlineFlowBox.cpp:
              (WebCore::placePositionedBoxesHorizontally): Ditto.
              (WebCore::placeBoxesVerticallyWithAbsBaseline): Ditto.
              * rendering/bidi.cpp:
              (WebCore::BidiIterator::increment): Ditto.
              (WebCore::BidiIterator::current): Ditto.
              (WebCore::BidiIterator::direction): Ditto.
              (WebCore::addRun): Ditto.
              (WebCore::checkMidpoints): Ditto.
              (WebCore::RenderBlock::computeHorizontalPositionsForLine): Ditto.
              (WebCore::RenderBlock::skipWhitespace): Ditto.
              (WebCore::RenderBlock::findNextLineBreak): Ditto.
      
              * rendering/RenderBR.h: Updated signature of width function which now takes a
              Font& instead of a Font*.
      
              * rendering/RenderBlock.cpp: (WebCore::stripTrailingSpace): Updated to no longer
              use the RenderObject::font() function, and added a comment about first-line, which
              is not handled properly here.
      
              * rendering/RenderMenuList.cpp: (WebCore::RenderMenuList::text):
              * rendering/RenderTreeAsText.cpp: (WebCore::writeTextRun):
              Eliminated use of RenderText::data().
      
              * rendering/RenderObject.h: Removed RenderObject::font(). This had the bad effect of
              having everyone pass in a boolean rather than using the cleaner style() and
              firstLineStyle() functions in cases where there's no a firstLineStyle boolean.
              * rendering/RenderObject.cpp:
              (WebCore::RenderObject::getVerticalPosition): Replaced use of RenderObject::font()
              with direct use of the style instead, which is arguably easier to understand anyway.
              (WebCore::RenderObject::baselinePosition): Ditto.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19027 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      42563ac5
    • weinig's avatar
      Reviewed by The Mitz. · 1c07d1d2
      weinig authored
              Move JSHTMLSelectElementCustom.cpp from bindings to bindings/js
              where it belongs.
      
              * WebCore.pro:
              * WebCore.xcodeproj/project.pbxproj:
              * bindings/JSHTMLSelectElementCustom.cpp: Removed.
              * bindings/js/JSHTMLSelectElementCustom.cpp: Added.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19026 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      1c07d1d2
    • zack's avatar
      Fix Qt build · a1a29ac9
      zack authored
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19025 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      a1a29ac9
  3. 21 Jan, 2007 16 commits
    • ap's avatar
      Reviewed by Darin. · 9ce7dcca
      ap authored
              http://bugs.webkit.org/show_bug.cgi?id=10934
              REGRESSION: prototype.js logs error (HTMLFormElement) on webkit builds
      
              Added constructors for most HTML elements; started auto-generating HTMLSelectElement JS binding.
      
              Test: fast/dom/Window/element-constructors-on-window.html
      
              * DerivedSources.make: Generate JSHTMLSelectElement.
              * WebCore.xcodeproj/project.pbxproj: Added JSHTMLSelectElement{Custom}.
              * bindings/js/JSHTMLElementWrapperFactory.cpp: Create a proper binding for SELECT.
      
              * bindings/js/kjs_html.cpp:
              (KJS::):
              (KJS::JSHTMLElement::classInfo):
              (KJS::JSHTMLElement::accessors):
              (KJS::JSHTMLElement::getOwnPropertySlot):
              (KJS::HTMLElementFunction::callAsFunction):
              (KJS::JSHTMLElement::put):
              * bindings/js/kjs_html.h:
              (KJS::JSHTMLElement::):
              Removed everything related to SELECT. Renamed getHTMLOptionsCollection to toJS for autogenerated code 
              to be happy, and rewrote it using cacheDOMObject.
      
              * bindings/scripts/CodeGeneratorJS.pm: Added HTMLElement to the list of types that cannot fail conversion.
      
              * html/HTMLAnchorElement.idl:
              * html/HTMLAppletElement.idl:
              * html/HTMLAreaElement.idl:
              * html/HTMLBRElement.idl:
              * html/HTMLBaseElement.idl:
              * html/HTMLBaseFontElement.idl:
              * html/HTMLBodyElement.idl:
              * html/HTMLButtonElement.idl:
              * html/HTMLCanvasElement.idl:
              * html/HTMLDListElement.idl:
              * html/HTMLDirectoryElement.idl:
              * html/HTMLDivElement.idl:
              * html/HTMLFieldSetElement.idl:
              * html/HTMLFontElement.idl:
              * html/HTMLFormElement.idl:
              * html/HTMLHRElement.idl:
              * html/HTMLHeadElement.idl:
              * html/HTMLHeadingElement.idl:
              * html/HTMLHtmlElement.idl:
              * html/HTMLImageElement.idl:
              * html/HTMLInputElement.idl:
              * html/HTMLIsIndexElement.idl:
              * html/HTMLLIElement.idl:
              * html/HTMLLabelElement.idl:
              * html/HTMLLegendElement.idl:
              * html/HTMLLinkElement.idl:
              * html/HTMLMapElement.idl:
              * html/HTMLMenuElement.idl:
              * html/HTMLMetaElement.idl:
              * html/HTMLModElement.idl:
              * html/HTMLOListElement.idl:
              * html/HTMLOptGroupElement.idl:
              * html/HTMLParagraphElement.idl:
              * html/HTMLParamElement.idl:
              * html/HTMLPreElement.idl:
              * html/HTMLQuoteElement.idl:
              * html/HTMLScriptElement.idl:
              * html/HTMLStyleElement.idl:
              * html/HTMLTextAreaElement.idl:
              * html/HTMLTitleElement.idl:
              * html/HTMLUListElement.idl:
              Generate constructors.
      
              * html/HTMLSelectElement.cpp:
              (WebCore::HTMLSelectElement::length):
              * html/HTMLSelectElement.h:
              Make length() return unsigned for auto-generated code to be happy.
      
              * html/HTMLSelectElement.idl: Adjusted to be usable for JS.
      
              * bindings/JSHTMLSelectElementCustom.cpp: Added. Implements remove().
      
              * page/DOMWindow.idl:
              Added properties for most elements' constructors.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19024 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      9ce7dcca
    • bdash's avatar
      2007-01-21 Mark Rowe <mrowe@apple.com> · 46f7ec9c
      bdash authored
              Reviewed by Maciej.
      
              http://bugs.webkit.org/show_bug.cgi?id=12357
              Bug 12357: Reproducible crash in WebCore::Settings::isJavaScriptEnabled in svg/custom/js-update-bounce.svg under guard-malloc
      
              * page/Frame.cpp:
              (WebCore::Frame::~Frame): Access the global object directly rather than via Window::retrieveWindow to prevent our reference to
              a deleted settings object being used.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19023 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      46f7ec9c
    • darin's avatar
      Reviewed by Maciej. · 4e5519d7
      darin authored
              - fix <rdar://problem/4930503> REGRESSION: Page changes title when
                command-clicking on named anchor link (12299)
      
              * loader/FrameLoader.cpp:
              (WebCore::FrameLoader::continueFragmentScrollAfterNavigationPolicy):
              Return if shouldContinue is false. This matches the other similar
              functions and prevents us from doing extra work in the "ignore" case.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19022 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      4e5519d7
    • bdash's avatar
      2007-01-21 Mark Rowe <mrowe@apple.com> · 41546709
      bdash authored
              Reviewed by Darin.
      
              http://bugs.webkit.org/show_bug.cgi?id=12355
              Bug 12355: Reproducible crash in WebCore::parseNumber in svg/custom/js-update-bounce.svg under guard-malloc
      
              * ksvg2/svg/SVGParserUtilities.cpp:
              (WebCore::parseNumber): Parenthesize to prevent reading past end of buffer.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19021 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      41546709
    • darin's avatar
      LayoutTests: · 311522cc
      darin authored
              Reviewed by Darin.
      
              - pixel test for http://bugs.webkit.org/show_bug.cgi?id=12345
                REGRESSION: Disabled pop-up text is not grayed out
      
              * fast/forms/select-disabled-appearance-expected.checksum: Added.
              * fast/forms/select-disabled-appearance-expected.png: Added.
              * fast/forms/select-disabled-appearance-expected.txt: Added.
              * fast/forms/select-disabled-appearance.html: Added.
      
      WebCore:
      
              Reviewed by Darin.
      
              - fix http://bugs.webkit.org/show_bug.cgi?id=12345
                REGRESSION: Disabled pop-up text is not grayed out
      
              Test: fast/forms/select-disabled-appearance.html
      
              * rendering/RenderThemeMac.mm:
              (WebCore::RenderThemeMac::adjustMenuListStyle):
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19020 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      311522cc
    • darin's avatar
      - fix non-Mac builds · ee3987d4
      darin authored
              * CMakeLists.txt: Added JSDocumentCustom.cpp.
              * WebCore.pro: Ditto.
              * WebCoreSources.bkl: Ditto.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19019 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      ee3987d4
    • darin's avatar
      LayoutTests: · 568a23ac
      darin authored
              Reviewed by Darin.
      
              - test for http://bugs.webkit.org/show_bug.cgi?id=12353
                REGRESSION: Crash on load (mutation event dispatch under the image element constructor deletes the element)
      
              * fast/dom/HTMLImageElement/constructor-mutation-event-dispatch-expected.txt: Added.
              * fast/dom/HTMLImageElement/constructor-mutation-event-dispatch.html: Added.
      
      WebCore:
      
              Reviewed by Darin.
      
              - fix http://bugs.webkit.org/show_bug.cgi?id=12353 <rdar://problem/4944599>
                REGRESSION: Crash on load (mutation event dispatch under the image element constructor deletes the element)
          
              Test: fast/dom/HTMLImageElement/constructor-mutation-event-dispatch.html
      
              * bindings/js/kjs_html.cpp:
              (KJS::ImageConstructorImp::construct): Protect the image element before setting
              its attributes.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19018 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      568a23ac
    • darin's avatar
      Reviewed by Tim H. · b68b6108
      darin authored
              * WebInspector/webInspector/inspector.css: Use row-resize for the splitter cursor
              instead of move. It's a horizontal splitter resizer.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19017 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      b68b6108
    • weinig's avatar
      LayoutTests: · 0585f0dd
      weinig authored
              Rolling out patch for http://bugs.webkit.org/show_bug.cgi?id=8360.
      
              * fast/layers/removed-by-scroll-handler-expected.checksum: Removed.
              * fast/layers/removed-by-scroll-handler-expected.png: Removed.
              * fast/layers/removed-by-scroll-handler-expected.txt: Removed.
              * fast/layers/removed-by-scroll-handler.html: Removed.
      
      WebCore:
      
              Rolling out patch for http://bugs.webkit.org/show_bug.cgi?id=8360.
      
              * rendering/RenderLayer.cpp:
              (WebCore::RenderLayer::RenderLayer):
              (WebCore::RenderLayer::scrollToOffset):
              (WebCore::RenderLayer::scrollRectToVisible):
              (WebCore::RenderLayer::updateScrollInfoAfterLayout):
              (WebCore::Marquee::start):
              * rendering/RenderLayer.h:
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19016 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      0585f0dd
    • ddkilzer's avatar
      LayoutTests: · a8971bca
      ddkilzer authored
              Reviewed by Darin.
      
              - test for http://bugs.webkit.org/show_bug.cgi?id=12280
                select element not available by name through document.all
                (dropdown list not available on smarthome.com checkout)
      
              * fast/dom/document-all-select-expected.txt: Added.
              * fast/dom/document-all-select.html: Added.
      
      WebCore:
      
              Reviewed by Darin.
      
              - fix http://bugs.webkit.org/show_bug.cgi?id=12280
                select element not available by name through document.all
                (dropdown list not available on smarthome.com checkout)
      
              Test: fast/dom/document-all-select.html
      
              * html/HTMLCollection.cpp:
              (WebCore::HTMLCollection::checkForNameMatch):
              (WebCore::HTMLCollection::updateNameCache):
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19015 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      a8971bca
    • weinig's avatar
      Reviewed by Darin. · 8b7eb291
      weinig authored
              - http://bugs.webkit.org/show_bug.cgi?id=10805
                REGRESSION (r15720): manual-tests/onblur-remove.html failing
      
              * manual-tests/onblur-remove.html: Changed the test to not require pressing
              Enter, since the correct behavior for Enter is to send a blur event.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19014 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      8b7eb291
    • weinig's avatar
      LayoutTests: · 74a19475
      weinig authored
              Reviewed by Darin.
      
              Test for: <rdar://problem/4928583> Memory usage grows when reloading google.com/ig
      
              * fast/dom/gc-10-expected.txt: Added.
              * fast/dom/gc-10.html: Added.
              * fast/dom/resources/gc-10-frame.html: Added.
      
      WebCore:
      
              Reviewed by Darin.
      
              Fix for: <rdar://problem/4928583> Memory usage grows when reloading google.com/ig
      
              JSDocuments are now responsible for marking the DOM wrappers associated with them, when they get marked.
              This fixes a JS object leak when a DOM wrapper has a reference to a JSDocument that is otherwise not reachable.
      
              Test: fast/dom/gc-10.html
      
              * WebCore.vcproj/WebCore/WebCore.vcproj:
              * WebCore.xcodeproj/project.pbxproj:
              * bindings/js/JSDocumentCustom.cpp: Added.
              (WebCore::JSDocument::mark):
              * bindings/js/kjs_binding.cpp:
              (KJS::ScriptInterpreter::markDOMNodesForDocument):
              (KJS::ScriptInterpreter::mark):
              * bindings/js/kjs_binding.h:
              * dom/Document.idl:
      
      WebKitTools:
      
              Reviewed by Darin.
      
              Test support for: <rdar://problem/4928583> Memory usage grows when reloading google.com/ig
      
              This adds a getJSObjectCount test-accessible function to allow test scripts to track JSObject usage.
      
              * DumpRenderTree/GCController.h:
              * DumpRenderTree/GCController.mm:
              (+[GCController isSelectorExcludedFromWebScript:]):
              (-[GCController getJSObjectCount]):
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19013 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      74a19475
    • weinig's avatar
      LayoutTests: · 8c67c07a
      weinig authored
              Reviewed by Darin.
      
              - test for http://bugs.webkit.org/show_bug.cgi?id=10692
                REGRESSION (r14847): Unnecessary scrollbars with position:relative; and overflow:auto;
      
              * fast/overflow/position-relative-expected.checksum: Added.
              * fast/overflow/position-relative-expected.png: Added.
              * fast/overflow/position-relative-expected.txt: Added.
              * fast/overflow/position-relative.html: Added.
      
      WebCore:
      
              Reviewed by Darin.
      
              - fix http://bugs.webkit.org/show_bug.cgi?id=10692
                REGRESSION (r14847): Unnecessary scrollbars with position:relative; and overflow:auto;
      
              Test: fast/overflow/position-relative.html
      
              * rendering/RenderBox.cpp:
              (WebCore::RenderBox::lowestPosition): Add the relative position offset only
              if 'includeSelf' is true.
              (WebCore::RenderBox::rightmostPosition): Ditto.
              (WebCore::RenderBox::leftmostPosition): Ditto.
              * rendering/RenderFlow.cpp:
              (WebCore::RenderFlow::lowestPosition): Ditto.
              (WebCore::RenderFlow::rightmostPosition): Ditto.
              (WebCore::RenderFlow::leftmostPosition): Ditto.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19012 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      8c67c07a
    • weinig's avatar
      LayoutTests: · 74ab2c6f
      weinig authored
              Reviewed by Darin.
      
              - test for http://bugs.webkit.org/show_bug.cgi?id=10472
                REGRESSION: "add" method of <select> object does not put the <option> object in the correct position
      
              * fast/dom/HTMLSelectElement/options-collection-detached-expected.txt: Added.
              * fast/dom/HTMLSelectElement/options-collection-detached.html: Added.
      
      WebCore:
      
              Reviewed by Darin.
      
              - fix http://bugs.webkit.org/show_bug.cgi?id=10472
                REGRESSION: "add" method of <select> object does not put the <option> object in the correct position
      
              Test: fast/dom/HTMLSelectElement/options-collection-detached.html
      
              * html/HTMLSelectElement.cpp:
              (WebCore::HTMLSelectElement::setRecalcListItems): Reset the options collection
              info if the <select> is not in the document. For in-document <select>s this
              happens anyway as a result of the DOM tree version increasing.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19011 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      74ab2c6f
    • weinig's avatar
      LayoutTests: · 0fcd7683
      weinig authored
              Reviewed by Darin.
      
              - updated results for http://bugs.webkit.org/show_bug.cgi?id=12349
                NativeFileUpload: Button text is not grayed out when the control is disabled
      
              * fast/forms/file-input-disabled-expected.checksum:
              * fast/forms/file-input-disabled-expected.png:
              * fast/forms/file-input-disabled-expected.txt:
      
      WebCore:
      
              Reviewed by Darin.
      
              - fix http://bugs.webkit.org/show_bug.cgi?id=12349
                NativeFileUpload: Button text is not grayed out when the control is disabled
      
              Covered by fast/forms/file-input-disabled.html 
      
              * css/html4.css: Changed selectors that had pseudo-classes after the -webkit-file-upload-button
              pseudo-element to have the pseudo-element at the end. Pseudo-elements must be at the end.
              * rendering/RenderFileUploadControl.cpp:
              (WebCore::RenderFileUploadControl::createButtonStyle): Changed to not overwrite the button's
              style with its parent's style. getPseudoStyle() handles inheritance properly.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19010 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      0fcd7683
    • weinig's avatar
      Reviewed by Darin. · 1537897b
      weinig authored
              - fix http://bugs.webkit.org/show_bug.cgi?id=12343
                crash in css2.1/t1202-counter-09-b.html
      
              * platform/Font.cpp:
              (WebCore::Font::glyphDataForCharacter):
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@19009 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      1537897b