Remove CURL support in the Gtk port, the HTTP backend is SOUP now.
authorchristian@webkit.org <christian@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 23 Feb 2009 11:15:30 +0000 (11:15 +0000)
committerchristian@webkit.org <christian@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 23 Feb 2009 11:15:30 +0000 (11:15 +0000)
https://bugs.webkit.org/show_bug.cgi?id=22624
[SOUP][GTK] Need API to get SoupSession from WebKit.

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

ChangeLog
GNUmakefile.am
WebCore/ChangeLog
WebCore/GNUmakefile.am
WebKit/gtk/ChangeLog
WebKit/gtk/webkit.pc.in
WebKitTools/ChangeLog
WebKitTools/GNUmakefile.am
configure.ac

index ad35878..c60eb9f 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2009-02-23  Xan Lopez  <xan@gnome.org>
+
+        Reviewed by Alexey Proskuryakov.
+
+        https://bugs.webkit.org/show_bug.cgi?id=22624
+        [SOUP][GTK] Need API to get SoupSession from WebKit.
+
+        Remove CURL support, the only supported HTTP backend is SOUP now.
+
+        * GNUmakefile.am:
+        * configure.ac:
+
 2009-02-19  Christian Dywan  <christian@twotoasts.de>
 
         Rubber-stamped by Holger Freyther.
 2008-11-16  Geoffrey Garen  <ggaren@apple.com>
 
         Not reviewed.
-        
+
         Try to fix gtk build.
 
         * configure.ac:
         Reviewed by Mark.
 
         Add x86_64 rule.
-        
+
         * Makefile:
 
 2008-05-09  Simon Hausmann  <hausmann@webkit.org>
         Reviewed by Holger.
 
         Removed explicit linkage against libxml and libxslt on Qt/Mac builds.
-        
+
         This dependency is completely unnecessary here and creates only problems by
         propagating through WebCore.pro over libQtWebKit.prl right now customer
         applications.
-        
+
 
         * WebKit.pri:
 
         Reviewed by Mark.
 
         Add an "x86_64" make rule.
-        
+
         * Makefile.shared:
 
 2008-05-02  Jan Michael Alonzo  <jmalonzo@unpluggable.com>
         http://bugs.webkit.org/show_bug.cgi?id=16476
 
         Add support for multiple http backends, and add soup backend (off by default).
-        
+
         * GNUmakefile.am:
         * configure.ac:
 
         Add separator '\' after libJavaScriptCore_la_LIBADD and cleanup
         whitespaces introduced in the previous commit.
 
-        * GNUmakefile.am: 
+        * GNUmakefile.am:
 
 2008-02-23  Jan Michael Alonzo  <jmalonzo@unpluggable.com>
 
         Reviewed by Lars.
 
         Don't compile the ICO plugin when building against Qt >= 4.4
-        
+
 
         * WebKit.pro:
 
index dbbb157..fb2deb5 100644 (file)
@@ -174,7 +174,6 @@ libWebCore_la_CFLAGS = \
        $(PANGO_CFLAGS) \
        $(GTK_CFLAGS) \
        $(XT_CFLAGS) \
-       $(LIBCURL_CFLAGS) \
        $(LIBSOUP_CFLAGS) \
        $(FREETYPE_CFLAGS) \
        $(SQLITE3_CFLAGS) \
@@ -201,7 +200,6 @@ libWebCore_la_LIBADD = \
        $(PANGO_LIBS) \
        $(GTK_LIBS) \
        $(XT_LIBS) \
-       $(LIBCURL_LIBS) \
        $(LIBSOUP_LIBS) \
        $(FREETYPE_LIBS) \
        $(UNICODE_LIBS) \
@@ -483,7 +481,8 @@ Programs_UnitTests_CFLAGS = \
        -O2 \
        $(global_cflags) \
        $(GLIB_CFLAGS) \
-       $(GTK_CFLAGS)
+       $(GTK_CFLAGS) \
+       $(LIBSOUP_CFLAGS)
 
 Programs_UnitTests_LDADD = \
        libwebkit-1.0.la \
index d668c84..021bb41 100644 (file)
@@ -1,3 +1,14 @@
+2009-02-23  Xan Lopez  <xan@gnome.org>
+
+        Reviewed by Alexey Proskuryakov.
+
+        https://bugs.webkit.org/show_bug.cgi?id=22624
+        [SOUP][GTK] Need API to get SoupSession from WebKit.
+
+        Remove CURL support.
+
+        * GNUmakefile.am:
+
 2009-02-23  Alexey Proskuryakov  <ap@webkit.org>
 
         Reviewed by Mark Rowe.
 
         https://bugs.webkit.org/show_bug.cgi?id=24042
         Bug 24042: MediaPlayer should cache plug-in proxy
-        
+
         The changes in https://bugs.webkit.org/show_bug.cgi?id=23917 assume that
         MediaPlayer will always have created the private media player object before
         the plug-in is instantiated and calls back with the proxy object. This is not
         Make it possible for a port to require a user gesture to play/pause an <audio> or <video> element
 
         * html/HTMLMediaElement.cpp:
-        (WebCore::HTMLMediaElement::HTMLMediaElement):  Rename m_loadRestrictions to m_Restrictions. 
+        (WebCore::HTMLMediaElement::HTMLMediaElement):  Rename m_loadRestrictions to m_Restrictions.
         Initialize m_internalCall.
         (WebCore::HTMLMediaElement::loadTimerFired): Increment m_internalCall around call to load().
         (WebCore::HTMLMediaElement::load): Call loadInternal if restrictions check out.
 
         * GNUmakefile.am: Added ContentType sources.
         * WebCore.pro: Ditto.
-        * WebCore.scons: Ditto. 
+        * WebCore.scons: Ditto.
         * WebCore.vcproj/WebCore.vcproj: Ditto.
         * WebCore.xcodeproj/project.pbxproj: Ditto.
         * WebCoreSources.bkl: Ditto.
 
         Reviewed by Dave Hyatt.
 
-        Fix for <rdar://problem/6077775> Should be able to specify 
+        Fix for <rdar://problem/6077775> Should be able to specify
         inactive ::selection color
 
-        This patch makes the ::selction pseudo-element work with 
-        the :window-inactive pseudo type. This was, a user can specify a 
+        This patch makes the ::selction pseudo-element work with
+        the :window-inactive pseudo type. This was, a user can specify a
         different ::selection style when a window is inactive.
 
         * css/CSSStyleSelector.cpp:
 2009-02-19  Simon Fraser  <simon.fraser@apple.com>
 
         Reviewed by Adele Peterson.
-        
+
         Call documentWillBecomeInactive() from Document::detach to ensure that
         media elements are shut down, because they may be kept alive by references
         from JS past document teardown.
-        
+
         documentWillBecomeInactive() calls  renderView()->willMoveOffscreen(), so no
         need to do that in detach() as well.
-        
+
         We no longer need to call documentWillBecomeInactive() from ~Page() (which
         was added for <https://bugs.webkit.org/show_bug.cgi?id=21116>), since this
         supersedes that change.
         Reviewed by Dave Kilzer.
 
         <rdar://problem/6592446> dynamically updating page doesn't seem to draw when updated
-        
+
         Need update after callback.
 
         * bindings/js/JSCustomPositionCallback.cpp:
         <source> element to be used as the initial url for the proxy.
         (WebCore::HTMLMediaElement::finishParsingChildren): New, allocate MediaPlayer and update widget.
         * html/HTMLMediaElement.h: Declare new methods for proxy, add m_needWidgetUpdate.
-        (WebCore::HTMLMediaElement::setNeedWidgetUpdate): 
+        (WebCore::HTMLMediaElement::setNeedWidgetUpdate):
 
         * html/HTMLVideoElement.cpp:
         (WebCore::HTMLVideoElement::attach): Poster image is handled by proxy when PLUGIN_PROXY_FOR_VIDEO.
 
         * rendering/RenderPartObject.cpp:
         (WebCore::RenderPartObject::RenderPartObject): Ditto.
-        (WebCore::RenderPartObject::updateWidget): Package params for proxy plug-in when element is 
+        (WebCore::RenderPartObject::updateWidget): Package params for proxy plug-in when element is
         <video> or <audio>.
         * rendering/RenderPartObject.h:
 
 
         Reviewed by Simon Fraser.
 
-        https://bugs.webkit.org/attachment.cgi?id=27666 
+        https://bugs.webkit.org/attachment.cgi?id=27666
         Fix Chromium build build: forgotten include in RenderObject, sync
         RenderTheme to the recent changes in RenderObject.
 
 
         Fix for <rdar://problem/6386623>
 
-        I made m_numParsedProperties and m_maxParsedProperties unsigned 
-        instead of int, and then added an early return from addPropery() if 
+        I made m_numParsedProperties and m_maxParsedProperties unsigned
+        instead of int, and then added an early return from addPropery() if
         m_maxParsedProperties exceeds the max value.
 
         * css/CSSParser.cpp:
 
         https://bugs.webkit.org/show_bug.cgi?id=16309
         HTML5: The third execCommand argument for insert{un}orderedlist should be ignored
-        
+
         When we originally implemented this command, MSDN documentation said that IE set the
         id attribute of inserted lists to the third argument to execCommand, but IE doesn't
         do this nor do any other browsers.
         * editing/CompositeEditCommand.cpp:
         (WebCore::CompositeEditCommand::moveParagraphContentsToNewBlockIfNecessary): Avoid modifying not just
         the body element, but any root editable element.
-        
+
 2009-02-16  Chris Fleizach  <cfleizach@apple.com>
 
         Reviewed by Adele Peterson.
+
         Bug 23979: AX: alt tag not returned for <input type="image">
         https://bugs.webkit.org/show_bug.cgi?id=23979
 
         Reviewed by Kevin Ollivier.
 
         The ref data is not properly set unless we use the CreateMatrix function.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=23978
 
         * platform/graphics/wx/TransformationMatrixWx.cpp:
         Reviewed by Darin Adler and Dan Bernstein.
 
         <rdar://problem/6583187>
-        
+
         * page/NavigatorBase.cpp:
         Handle PPC64 and X86_64 as well.
 
         to have a tighter type: RenderWidget instead of RenderObject.  Devirtualize
         updateWidgetPosition, since all RenderWidgets (except for applets) are now
         RenderParts also.
-        
+
         * rendering/RenderObject.cpp:
         * rendering/RenderObject.h:
         * rendering/RenderPart.cpp:
 
         Add isTextControl() to RenderObject.  Patch call sites thats asked isTextField() || isTextArea() to just use
         isTextControl() instead.
-        
+
         Add a toRenderTextControl converter for doing checked casting to RenderTextControls.
 
         * dom/Document.cpp:
         Get rid of isEdited/setEdited on RenderObject.  Devirtualize isEdited/setEdited on RenderTextControl.
         Callers were already asking if the renderer was a text field (or a text area), so it was especially
         pointless to waste a second virtual function call after making one just to determine the object type.
-    
+
         Reviewed by Simon Fraser
 
         * dom/Document.cpp:
 
         toRenderBox()->x() asserts for a layer on an inline; use
         m_owningLayer->renderBoxX() instead.
-        
+
         * rendering/RenderLayerBacking.cpp:
         (WebCore::RenderLayerBacking::paintIntoLayer):
 
 2009-02-12  Simon Fraser  <simon.fraser@apple.com>
 
         Fix non-Mac builds by adding Matrix3DTransformOperation.* and
-        PerspectiveTransformOperation.*. Cleaned out some cruft in 
+        PerspectiveTransformOperation.*. Cleaned out some cruft in
         the WebCore.vcproj fle.
 
         * GNUmakefile.am:
 2009-02-12  David Smith  <catfish.man@gmail.com>
 
         Reviewed by Dave Hyatt.
-        
+
         Fix https://bugs.webkit.org/show_bug.cgi?id=23935
         Nested :not() and non-simple selectors in :not() should be invalid
 
 2009-02-12  Simon Fraser  <simon.fraser@apple.com>
 
         Fix build with ACCELERATED_COMPOSITING turned on.
-        
+
         Need to cast toRenderBox() in order to call overflowClipRect().
 
         * rendering/RenderLayerBacking.cpp:
         Supporting this requires that we know when we are processing a media engine callback so
         we don't turn around and tell the media player to change the volume when it is reporting
         a volume change.
-        
+
         * html/HTMLMediaElement.cpp:
         (WebCore::HTMLMediaElement::HTMLMediaElement): initialize m_processingMediaPlayerCallback
         (WebCore::HTMLMediaElement::mediaPlayerNetworkStateChanged): call begin/endProcessingMediaPlayerCallback
-        (WebCore::HTMLMediaElement::setNetworkState): logic split out of mediaPlayerNetworkStateChanged 
+        (WebCore::HTMLMediaElement::setNetworkState): logic split out of mediaPlayerNetworkStateChanged
         to make it easier to maintain m_processingMediaPlayerCallback.
         (WebCore::HTMLMediaElement::mediaPlayerReadyStateChanged): call begin/endProcessingMediaPlayerCallback
         (WebCore::HTMLMediaElement::mediaPlayerTimeChanged): Ditto.
         * platform/graphics/win/QTMovieWin.cpp:
         (QTMovieWin::disableUnsupportedTracks):
 
-        * platform/graphics/win/MediaPlayerPrivateQuickTimeWin.h: fix typo added in r40925 
+        * platform/graphics/win/MediaPlayerPrivateQuickTimeWin.h: fix typo added in r40925
 
 2009-02-12  David Hyatt  <hyatt@apple.com>
 
         (WebCore::RenderObject::isFloatingOrPositioned):
         * rendering/RenderView.h:
 
-2009-02-12  Chris Marrin  <cmarrin@apple.com>        
+2009-02-12  Chris Marrin  <cmarrin@apple.com>
 
         Reviewed by Dave Hyatt
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=23908
-        
+
         Added parsing of 3d transform functions and properties
-        (perspective, perspective-origin, transform-style-3d, 
-        and backface-visibility).   
-        
+        (perspective, perspective-origin, transform-style-3d,
+        and backface-visibility).
+
         Test: transforms/3d/cssmatrix-3d-interface.xhtml
-        
+
         * css/CSSComputedStyleDeclaration.cpp:
         (WebCore::):
         (WebCore::computedTransform):
         (WebCore::TransformOperations::has3DOperation):
         * platform/graphics/transforms/TranslateTransformOperation.h:
         (WebCore::TranslateTransformOperation::TranslateTransformOperation):
-        
+
 2009-02-12  Kevin Ollivier  <kevino@theolliviers.com>
 
         wx build fix.
         the single caller.
 
         Fix build error in MediaPlayerPrivateQTKit.h.
-        
+
         * css/CSSStyleSelector.cpp:
         (WebCore::CSSStyleSelector::mapAnimationDirection):
         * platform/animation/Animation.h:
 2009-02-12  Eric Carlson  <eric.carlson@apple.com>
 
         Darin Adler, Antti Koivisto, Holger Hans Peter Freyther
-        
+
         Bug 23797: A platform should be able to use more than one media engine for <video> and <audio>
         https://bugs.webkit.org/show_bug.cgi?id=23797
 
         (WebCore::HTMLMediaElement::load): selectMediaURL now also returns a MIME type, pass it to
         MediaPlayer::load so it can choose the right media engine
         (WebCore::HTMLMediaElement::selectMediaURL): renamed from pickMedia. use MIMETypeRegistry
-        functions to strip params from the url and extract the codecs parameter. call 
+        functions to strip params from the url and extract the codecs parameter. call
         MediaPlayer::supportsType to see if we can open the file since only a media engine can
         answer questions about supported type+codecs.
         * html/HTMLMediaElement.h:
         (WebCore::MediaPlayerPrivateInterface::~MediaPlayerPrivateInterface):
         (WebCore::MediaPlayerPrivateInterface::totalBytesKnown):
 
-        * platform/graphics/chromium/MediaPlayerPrivateChromium.h: Add support for engine factory 
+        * platform/graphics/chromium/MediaPlayerPrivateChromium.h: Add support for engine factory
 
         * platform/graphics/gtk/MediaPlayerPrivateGStreamer.cpp:
         (WebCore::MediaPlayerPrivate::create): New, media engine factory function
         Reviewed by Dave Hyatt
 
         https://bugs.webkit.org/show_bug.cgi?id=23188
-        
+
         Define enum values for Animation::direction(), and one for
         IterationCountInfinite to make the code more self-documenting.
-        
+
         Order the member vars of Animation to optimize packing.
-        
+
         * css/CSSComputedStyleDeclaration.cpp:
         (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
         * platform/animation/Animation.cpp:
 
         https://bugs.webkit.org/show_bug.cgi?id=23905
 
-        Adds support for 3D CSS properties (transform-style-3d, perspective, 
+        Adds support for 3D CSS properties (transform-style-3d, perspective,
         perspective-origin, and backface-visibility) to RenderStyle and friends
 
         * platform/graphics/transforms/TransformOperation.h
         Reviewed by Dave Hyatt
 
         https://bugs.webkit.org/show_bug.cgi?id=23862
-        
+
         Add a bit on RenderStyle that gets set when running accelerated
         transitions of transform or opacity. These ensure that styles
         compare as different during the transition, so that interruption
 
         Eliminate the virtual position() function from RenderObject.  Rename the position() overrides
         in RenderText and RenderBox to positionLineBox and devirtualize them.
-        
+
         Patch the one call site to just check for isText() and isBox() and call the methods directly.
-        
+
         Remove some unnecessary overrides of position() on table sections and rows.
-    
+
         Reviewed by Simon Fraser
 
         * rendering/RenderBox.cpp:
 
 2009-02-11  David Hyatt  <hyatt@apple.com>
 
-        Combine RenderObject::element() and RenderObject::node() into a single function.  
+        Combine RenderObject::element() and RenderObject::node() into a single function.
         node() now has the "anonymous bit" checking behavior of element() and will
-        return 0 for anonymous content.  This patch switches all callers of element() 
+        return 0 for anonymous content.  This patch switches all callers of element()
         to node() and patches old callers of node() to deal with situations where they
-        did not expect node() to be 0.  A bunch of node() calls were calling stuff on 
+        did not expect node() to be 0.  A bunch of node() calls were calling stuff on
         Document that they clearly didn't intend, so overall this is a nice improvement.
-    
+
         Reviewed by Sam Weinig
 
         * dom/ContainerNode.cpp:
 
         When opacity or transform change on an object which has a compositing layer,
         avoid repainting the layer.
-        
+
         Added a new StyleDifference value, StyleDifferenceRecompositeLayer, which indicates
         that the only thing styleChanged() has to do is to update composited properties of
         the layer. RenderStyle::diff() now has an out param for a bitmask of "context sensitive"
         properties, currently for opacity and transform. When one of these changes, we need
         to see if we have a compositing layer before we decide whether to layout/repaint,
         or just update the composited layer, via adjustStyleDifference().
-        
+
         * rendering/RenderObject.cpp:
         (WebCore::RenderObject::adjustStyleDifference):
         (WebCore::RenderObject::setStyle):
         Fix for pixel test regression in fast/text.  Make sure not to add in
         the borderTop and paddingTop of the block to the baseline when painting
         strict mode text decorations.
-        
+
         Reviewed by Mark Rowe
-    
+
         * rendering/InlineFlowBox.cpp:
         (WebCore::InlineFlowBox::paintTextDecorations):
         * rendering/InlineFlowBox.h:
 2009-02-10  David Hyatt  <hyatt@apple.com>
 
         Some renames on line boxes.  xPos()/yPos() -> x()/y().  setXPos()/setYPos() -> setX()/setY().  m_object/object() -> m_renderer/renderer().  textObject() -> textRenderer().
-        
+
         Reviewed by Sam Weinig
 
         * dom/Position.cpp:
 
         Clean up "fallbackAnimating" logic in AnimationBase. This flag indicates
         that animation of an accelerated property must run in software for some reason.
-        
+
         Also remove use of private headers in GraphicsLayerCA related to a case
         where we may have to fall back on software animation of transform.
 
         relates to the RenderLayer z-order/overflow structure, rather than the render tree
         parent chain. Add a convenience method, ancestorCompositingLayer(), which finds
         the enclosing layer excluding self.
-        
+
         Fix enclosingCompositingLayer() to correctly look at stacking context and overflow
         lists.
-        
+
         Check for documentBeingDestroyed() in a few places to avoid work on document
         teardown.
-        
+
         * rendering/RenderLayer.cpp:
         (WebCore::RenderLayer::enclosingCompositingLayer):
         (WebCore::RenderLayer::setParent):
 
         Shrink the size of all RenderObjects (except for RenderInlines) by 4 bytes.  This patch moves the cached vertical position member to RenderInlines,
         since they were the only objects being queried across multiple lines.
-        
+
         Reviewed by Sam Weinig
 
         * rendering/InlineFlowBox.cpp:
         * html/HTMLMediaElement.cpp:
         (WebCore::HTMLMediaElement::HTMLMediaElement): Initialize m_loadRestrictions.
         (WebCore::HTMLMediaElement::attributeChanged): Only one attribute can change so put an "else" between tests.
-        (WebCore::HTMLMediaElement::load): Return INVALID_STATE_ERR if RequireUserGestureLoadRestriction is set 
+        (WebCore::HTMLMediaElement::load): Return INVALID_STATE_ERR if RequireUserGestureLoadRestriction is set
         and we are not preocessing a user gesture.
         (WebCore::HTMLMediaElement::mediaPlayerNetworkStateChanged): Remove unnecessary white space.
         (WebCore::HTMLMediaElement::play): Ditto.
         height() on InlineBox is now a virtual function that does a dynamic computation (including shrinking boxes
         with no text children).  For SVG boxes there is also a non-virtual setHeight function for updating the
         m_height member variable like before.
-    
+
         Reviewed by Darin Adler
 
         * rendering/EllipsisBox.cpp:
 2009-02-09  Eric Carlson  <eric.carlson@apple.com>
 
         Reviewed by Antti Koivisto
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=23737
         Make it possible to control media element playback without knowing so many
         internal implementation details.
         (WebCore::HTMLVideoElement::hasVideo): New.
 
         * rendering/MediaControlElements.cpp:
-        (WebCore::MediaControlPlayButtonElement::defaultEventHandler): Use new media element 
+        (WebCore::MediaControlPlayButtonElement::defaultEventHandler): Use new media element
         togglePlayState method.
         (WebCore::MediaControlTimelineElement::defaultEventHandler): Tell media element when scrubbing
         begins and ends so it can deal with pausing logic. Don't call setCurrentTime unless the time
 2009-02-08  Dirk Schulze  <krit@webkit.org>
 
         Unreviewed Qt build fix.
-        
+
         TransformationMatrix changes in r40761 broke QT build.
 
         * platform/graphics/qt/GraphicsContextQt.cpp:
 
         Reviewed by Eric Seidel <eric@webkit.org>
 
-        Put #if ENABLE(PAN_SCROLLING) around EventHandler::setPanScrollCursor. 
+        Put #if ENABLE(PAN_SCROLLING) around EventHandler::setPanScrollCursor.
         https://bugs.webkit.org/show_bug.cgi?id=23574
 
         * page/EventHandler.cpp:
 2009-02-07  Zalan Bujtas  <zbujtas@gmail.com>
 
         Reviewed by Rob Buis <rwlbuis@gmail.com>
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=23788
-        call style()->isOriginalDisplayInlineType() on RenderBox instead of RenderObject. 
-        
+        call style()->isOriginalDisplayInlineType() on RenderBox instead of RenderObject.
+
         * rendering/bidi.cpp:
         (WebCore::RenderBlock::skipLeadingWhitespace):
 
 
         Reviewed by Dan Bernstein
 
-        <rdar://problem/6563402> - Workaround a case where ::didReceiveData() might be called on a ResourceHandleClient 
+        <rdar://problem/6563402> - Workaround a case where ::didReceiveData() might be called on a ResourceHandleClient
         before ::didReceiveResponse().
 
         No layout test, as we don't have a concrete way to reproduce these conditions.  To that effect, I've added an ASSERT
         this case and can come up with something more reproducible.
 
         * loader/MainResourceLoader.cpp:
-        (WebCore::MainResourceLoader::didReceiveData): If the response is null create a dummy response.  This is much 
+        (WebCore::MainResourceLoader::didReceiveData): If the response is null create a dummy response.  This is much
           like the FrameLoader::init() response that is used for initial document creation.
 
 2009-02-06  Geoffrey Garen  <ggaren@apple.com>
 2009-02-06  Justin Garcia  <justin.garcia@apple.com>
 
         Reviewed by Oliver Hunt.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=23800
         Header elements are not always preserved during paste
-        
+
         At paste time, don't merge out of header elements.  At copy time, be sure to include
         headers in the list of special common ancestor blocks, so that copying a paragraph or less
         of content inside a header will include the header element in the copied markup.
         Reviewed by Sam Weinig.
 
         Part III of <rdar://problem/6552272>.
-        
+
         Make redirect data available to WebKit. (I tried having WebKit track this
         data for itself, but that went wrong -- it's just too hard to figure
         out the weird corners of the WebCore loader from within another framework.)
         * loader/DocumentLoader.h: Store a flag indicating whether we created
         a global history entry. If we didn't, then we know that the next redirect
         should not link to us, since we're not in history.
-        
+
         (WebCore::DocumentLoader::clientRedirectSourceForHistory):
         (WebCore::DocumentLoader::clientRedirectDestinationForHistory):
         (WebCore::DocumentLoader::setClientRedirectSourceForHistory):
         (WebCore::FrameLoader::provisionalDocumentLoader):
         (WebCore::FrameLoader::state): Inlined a few trivial functions because
         performance mistakes give me the itches.
-        
+
         * loader/FrameLoaderClient.h: Renamed this client function to say what
         it does, instead of when it's called.
 
         Factor code that checks whether the object is rooted in
         repaint() and repaintRectangle() into a common isRooted()
         method.
-        
+
         * rendering/RenderObject.cpp:
         (WebCore::RenderObject::repaint):
         (WebCore::RenderObject::repaintRectangle):
         Reviewed by Eric Seidel
 
         Make RenderStyle::transitionForProperty() const.
-        
+
         * rendering/style/RenderStyle.cpp:
         (WebCore::RenderStyle::transitionForProperty):
         * rendering/style/RenderStyle.h:
 2009-02-06  Anders Carlsson  <andersca@apple.com>
 
         Build fix.
-        
+
         * bridge/npapi.h:
 
 2009-02-06  Darin Fisher  <darin@chromium.org>
         Reviewed by Dave Hyatt
 
         https://bugs.webkit.org/show_bug.cgi?id=23781
-        
+
         When the document is attached and detached, and when it becomes active/inactive
         (e.g. coming out or going into the page cache) we need to call
         RenderView::didMoveOnscreen()/willMoveOffscreen() to hook and unhook the composited
         layers.
-        
+
         Also, if Document::recalcStyle() does not do a layout, it needs to
         update the compositing layers.
-        
+
         * dom/Document.cpp:
         (WebCore::Document::updateRendering):
         (WebCore::Document::updateLayout):
         getAnimatedStyleForRenderer() should not create a new
         CompositeAnimation if the object isn't already animating, and
         it should just fall back to returning renderer->style().
-        
+
         * page/animation/AnimationController.cpp:
         (WebCore::AnimationControllerPrivate::getAnimatedStyleForRenderer):
 
 2009-02-05  Simon Fraser  <simon.fraser@apple.com>
 
         Fix build when ACCELERATED_COMPOSITING is turned on.
-       
+
         Give RenderLayerBacking some love after the great
         RenderBoxModelObject split.
-        
+
         * rendering/RenderLayerBacking.cpp:
         (WebCore::RenderLayerBacking::updateLayerTransform):
         (WebCore::RenderLayerBacking::updateGraphicsLayerGeometry):
 2009-02-05  Simon Fraser  <simon.fraser@apple.com>
 
         Fix build when ACCELERATED_COMPOSITING is turned on.
-        
+
         writeIndent() needs to be static to avoid warnings on some OSes,
         and remove unused CAToTransform3D() method.
 
 2009-02-05  Simon Fraser  <simon.fraser@apple.com>
 
         Fix build when ACCELERATED_COMPOSITING is turned on.
-        
+
         animationOfPropertyIsAccelerated() needs to be public because
         it is called from AnimationController.
 
 
         Add toRenderBoxModelObject() so that we can cast objects that
         can have layers.
-        
+
         Fix getComputedStyle to return the current value of style
         properties that are undergoing accelerated animations, by
         asking the AnimationController if the property has accelerated
         animations, and, if so, asking it to generate an appropriate
         style.
-        
+
         Tightened up some type safety (CSSPropertyID), and code clarity
         (replace bool with enum).
 
 2009-02-05  David Hyatt  <hyatt@apple.com>
 
         Move the m_layer member variable of RenderBox up into RenderBoxModelObject.  Refactor styleDidChange'
-        of RenderObject, RenderBoxModelObject and RenderBox to get more of the code in the correct subclass.  
-        In order to set the various RenderObject bits properly prior to the test for whether a layer is 
+        of RenderObject, RenderBoxModelObject and RenderBox to get more of the code in the correct subclass.
+        In order to set the various RenderObject bits properly prior to the test for whether a layer is
         required, I added an additional method, updateBoxModelInfoAfterStyleChange, that is responsible
         for setting all of the bits.
-        
+
         Eliminate hasStaticX/Y, staticX/Y, and setStaticX/Y from the render tree.  Move hasStaticX/Y to
         RenderStyle, and just use the corresponding layer() methods for the
         staticX/Y getters/setters.
 2009-02-05  Anders Carlsson  <andersca@apple.com>
 
         Reviewed by Kevin Decker.
-        
+
         Update to match Gecko.
-        
+
         * bridge/npapi.h:
         * plugins/npfunctions.h:
 
         closely match the spec which states that all Nodes are EventTargets) and save
         4 bytes per Node that the polymorphic multiple-inheritance was costing us.
 
-        This does the bare minimum to get things compiling, the next patch will clean 
+        This does the bare minimum to get things compiling, the next patch will clean
 
         * DerivedSources.make:
         * WebCore.base.exp:
           wait for a callback (notifyAnimationStarted()) in order to get the start time.
         * if software and accelerated animations start in the same cycle, they all need
           to use the same start time, which is that of the accelerated animations.
-          
+
         Added AnimationControllerPrivate to preserve the encapsulation under
         AnimationController.
 
         Reverse the meaning of the if check so that Android
         (dpad devices) do not scroll on focus and normal platforms do
         http://trac.webkit.org/changeset/40647
-        
+
         This was caught by George Staikos, and I wrote the patch.
 
         * dom/Element.cpp:
         Reviewed by Darin Adler.
 
         Change RenderBlock::columnGap() access permissions from protected to
-        public. ANDROID creates an array of focus rectangles for navigation 
-        with a trackball or directional pad, and needs access to the column gap 
+        public. ANDROID creates an array of focus rectangles for navigation
+        with a trackball or directional pad, and needs access to the column gap
         to implement this logic.
 
         Android has no testing harness @ webkit.org, thus no tests.
 
         Reviewed by Dave Hyatt.
 
-        Fix for <rdar://problem/6386112> Some inline-elements are not 
+        Fix for <rdar://problem/6386112> Some inline-elements are not
         becoming proper dashboard-regions
 
-        RenderInline needs its own implementation of addDashboardRegions 
-        because otherwise all of the coordinate calculations are 
-        inaccurate. 
+        RenderInline needs its own implementation of addDashboardRegions
+        because otherwise all of the coordinate calculations are
+        inaccurate.
 
         * rendering/RenderInline.cpp:
         (WebCore::RenderInline::addDashboardRegions):
         Reviewed by Dave Hyatt
 
         https://bugs.webkit.org/show_bug.cgi?id=23360
-        
+
         When using accelerated compositing, make repaints use the correct
         repaint container. Hook up the RenderLayerCompositor in RenderView,
         and add to RenderView a method that repaints both the view
         contents, and any intersecting composited layers.
-        
+
         Make enclosingCompositingLayer() a method on RenderObject, and update
         the compositor to use that. Add a helper method on the compositor to
         get the non-self compositing ancestor: ancestorCompositingLayer().
-        
+
         Call repaintRectangleInViewAndCompositedLayers() in places that don't do
         container-relative repainting: selection redraw on focus change, caret,
         and widget updates.
-        
+
         Fix a bug in RenderLayerCompositor where the m_compositingLayersNeedUpdate flag
         would get cleared for partial updates, thus preventing subsequent full updates.
-        
+
         Fix a bug in RenderLayer::ensureBacking() that made new backing every time.
 
         * WebCore.xcodeproj/project.pbxproj:
 2009-02-04  David Hyatt  <hyatt@apple.com>
 
         Fix for https://bugs.webkit.org/show_bug.cgi?id=23734, blank content on blogs.msdn.com.
-        
+
         Add debugging code that will catch the incorrect dirtying of a table section immediately if it happens
         during calcRowHeight or layoutRows.  This should make it possible to discover future bugs where table sections don't
         paint.
 2009-02-04  miggilin  <mr.diggilin@gmail.com>
 
         Reviewed by Kevin Ollivier.
-        
+
         Switch to using a ref-counted wxFont pointer for FontPlatformData to fix a crash
         with assigning to uninitialized HashTable buckets.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=23669
 
         * platform/graphics/SimpleFontData.h:
         <rdar://problem/6554092> Slow script dialog starts coming up over and over again
 
         Work around <rdar://problem/6554067> by consuming any exceptions that are
-        thrown by _web_localizedDescription        
+        thrown by _web_localizedDescription
 
         * platform/network/mac/ResourceErrorMac.mm:
         (WebCore::ResourceError::platformLazyInit):
         Fix selection repainting to do container-relative repaints. RenderView now
         repaints the selection using the containerForRepaint() for each RenderSelectionInfo.
         selectionRect() is now a wrapper for selectionRectForRepaint() with no container.
-        
+
         Pull SelectionInfo out of RenderObject.h, and BlockSelectionInfo out
         of RenderBlock.h and move them into RenderSelectionInfo.h, with some
         sharing and refactoring.
-        
+
         RenderBlock::selectionGapRectsForRepaint() is not yet container-aware.
 
         * GNUmakefile.am:
 
         https://bugs.webkit.org/show_bug.cgi?id=13490
         Implement execCommand("styleWithCSS", ...)
-        
+
         Determine whether or not to use HTML formatting tags for styling on the new
         boolean on Editor instead of whether or not the document is quirks mode.
 
 
         On Tiger, create an autorelease pool before creating the NSGraphicsContext and drain it immediately
         after the call to -[NSView displayRectIgnoringOpacity:inContext:].
-        
+
         * platform/mac/WidgetMac.mm:
         (WebCore::Widget::paint):
 
 
         Rename addLineBoxRects to absoluteRectsForRange, and do the same thing for the corresponding quads methods.  Move the RenderContainer implementations
         of these methods up to RenderObject.
-        
+
         Reviewed by Beth Dakin
 
         * bindings/objc/DOM.mm:
         (WebCore::RenderLayer::RenderLayer):
         (WebCore::RenderLayer::~RenderLayer):
         Init and clean up backing and the hasCompositingDescendant bit.
-        
+
         (WebCore::RenderLayer::compositor):
         Shortcut to get to the RenderLayerCompositor.
 
         (WebCore::RenderLayer::updateLayerPositions):
-        We need to let full repaints propagate to all compositing layers, and 
+        We need to let full repaints propagate to all compositing layers, and
         update the backing after layout here.
-        
+
         (WebCore::RenderLayer::transparentAncestor):
         (WebCore::transparencyClipBox):
         (WebCore::RenderLayer::beginTransparencyLayers):
         Account for the fact that opacity may be rendered via the compositing
         layer sometimes.
-        
+
         (WebCore::RenderLayer::addChild):
         (WebCore::RenderLayer::removeChild):
         (WebCore::RenderLayer::removeOnlyThisLayer):
         Tell the compositor when the RenderLayer hierarchy changes.
-        
+
         (WebCore::RenderLayer::scrollToOffset):
         Update layer geometry after scrolling.
-        
+
         (WebCore::RenderLayer::paintLayer):
         We short-circuit painting on composited layers, because these layers
         are painted on a callback from the compositing system.
-        
+
         (WebCore::RenderLayer::hitTestLayer):
         Only apply transforms if we are rendering them in software.
-        
+
         (WebCore::RenderLayer::localBoundingBox):
         (WebCore::RenderLayer::boundingBox):
         (WebCore::RenderLayer::absoluteBoundingBox):
         Refactor bounding box code that we can compute local, and ancestor-relative
         bounding box, as well as absolute.
-        
+
         (WebCore::RenderLayer::ensureBacking):
         (WebCore::RenderLayer::clearBacking):
         RenderLayerBacking creation and destruction.
-        
+
         (WebCore::RenderLayer::setParent):
         Tell the compositor when the RenderLayer hierarchy changes.
 
         (WebCore::RenderLayer::dirtyOverflowList):
         When the z-order and overflow lists change, we need to tell the
         compositor that the composited layers need to be rejiggered soon.
-        
+
         (WebCore::RenderLayer::updateZOrderLists):
         Whitespace cleanup.
-        
+
         (WebCore::RenderLayer::setBackingNeedsRepaint):
         (WebCore::RenderLayer::setBackingNeedsRepaintInRect):
         Dirty composited layer contents for painting.
-        
+
         (WebCore::RenderLayer::styleChanged):
         Update the layer backing after style changes.
 
         (WebCore::RenderLayer::isComposited):
         (WebCore::RenderLayer::backing):
         Accessors for testing and getting the backing for this RenderLayer.
-        
+
         (WebCore::RenderLayer::paintsWithTransparency):
         (WebCore::RenderLayer::paintsWithTransform):
         Transform and opacity can be applied via the compositing layer, or rendered
         in software. These methods tell us if we need to account for them in the
         non-compositing path.
-        
+
         (WebCore::RenderLayer::hasCompositingDescendant):
         (WebCore::RenderLayer::setHasCompositingDescendant):
         Maintain a bit to tell if this layer has composited descendants.
         Reviewed by Dave Hyatt
 
         https://bugs.webkit.org/show_bug.cgi?id=23365
-        
+
         Hook up accelerated compositing layers the native
         view system on Mac.
 
         (WebCore::Page::didMoveOnscreen):
         (WebCore::Page::willMoveOffscreen):
         * page/Page.h:
-        New methods to allow the native view system to tell the Page when it 
+        New methods to allow the native view system to tell the Page when it
         starts to be presented on-screen, and when it will be hidden.
 
         * rendering/RenderLayerCompositor.h:
 
         Reviewed by David "The Barnabas Jones" Hyatt.
 
-        Move removeChildNode, appendChildNode and insertChildNode from 
+        Move removeChildNode, appendChildNode and insertChildNode from
         RenderContainer into RenderObjectChildList.  Make moveChildNode
         static in RenderBlock, as that was the only user.
 
         Track redirects in global history.
 
         * WebCore.base.exp: Renamed some exports.
-        
+
         * bindings/js/JSDOMWindowBase.cpp:
         (WebCore::createWindow):
         (windowProtoFuncOpen):
         * loader/EmptyClients.h:
         (WebCore::EmptyFrameLoaderClient::updateGlobalHistoryForRedirectWithoutHistoryItem):
         Stubbed out a client function to keep things building.
-        
+
         * loader/FrameLoader.cpp:
         (WebCore::isBackForwardLoadType):
         (WebCore::FrameLoader::restoreDocumentState): Renamed FrameLoadTypeRedirect =>
         (WebCore::FrameLoader::continueFragmentScrollAfterNavigationPolicy):
         (WebCore::FrameLoader::continueLoadAfterNewWindowPolicy):
         (WebCore::FrameLoader::loadItem): Updated for extra parameter and rename.
-        
+
         (WebCore::FrameLoader::updateHistory*): Notify WebKit about redirect
         navigations even if they don't create new history items, so we can track
         the redirect in the existing history item.
 
         * loader/FrameLoader.h: See above.
-        
+
         * loader/FrameLoaderClient.h: New client interface used by
         FrameLoader::updateHistory* to record a redirect even if it doesn't create
         a new history item of its own.
         Reviewed by Sam Weinig.
 
         Add jsObjectForPluginElement to ScriptController, and have createScriptObjectForPluginElement call it.
-        
+
         * WebCore.LP64.exp:
         * bindings/js/ScriptController.cpp:
         (WebCore::ScriptController::createScriptObjectForPluginElement):
         Reviewed by Dan Bernstein.
 
         More plug-in cleanup.
-        
+
         * WebCore.NPAPI.exp:
         Add new symbols.
-        
+
         * bindings/objc/DOM.mm:
         * bindings/objc/DOMPrivate.h:
         Remove methods that WebKit doesn't use anymore.
         Reviewed by Dan Bernstein.
 
         Change createPlugin to take a HTMLPlugInElement, and createJavaAppletWidget to take a HTMLAppletElement.
-        
+
         * WebCore.xcodeproj/project.pbxproj:
         * loader/EmptyClients.h:
         (WebCore::EmptyFrameLoaderClient::createPlugin):
         * platform/graphics/win/GraphicsContextCairoWin.cpp: Remove
           the implementation of inTransparencyLayer.
         * platform/graphics/win/GraphicsContextWin.cpp: Move common code
-          from GraphicsContextCGWin.cpp and GraphicsContextCairoWin.cpp. 
+          from GraphicsContextCGWin.cpp and GraphicsContextCairoWin.cpp.
         (WebCore::GraphicsContext::inTransparencyLayer):
         (WebCore::GraphicsContext::setShouldIncludeChildWindows):
         (WebCore::GraphicsContext::shouldIncludeChildWindows):
 2009-01-31  David Hyatt  <hyatt@apple.com>
 
         Fix for https://bugs.webkit.org/show_bug.cgi?id=11363
-        
+
         Remove the hacks in table code around the DeleteButtonController and fix tables so that positioned children don't get wrapped in anonymous objects.
         With the removal of a non-positioned DeleteButtonController renderer, table layout can be tightened up to only lay out table sections.  Table section
         layout is tightened up to ignore non-table rows.  When a table has multiple captions, only the real one will do a layout now.  The other ones will
         Reviewed by Holger Freyther.
 
         [GTK] Logic fixes and build fixes
+
         https://bugs.webkit.org/show_bug.cgi?id=22022
 
         Catch up with the internal GeoLocation API and fix the logic of
         Reviewed by Dave Hyatt
 
         https://bugs.webkit.org/show_bug.cgi?id=23357
-        
+
         New files to support accelerated compositing at the RenderLayer
         level.
-        
+
         * WebCore.xcodeproj/project.pbxproj:
         New files added to project, and run through the sort script.
-        
+
         * rendering/RenderLayerBacking.cpp: Added.
         * rendering/RenderLayerBacking.h: Added.
         New object to store compositing-related data for a single
         RenderLayer.
-        
+
         * rendering/RenderLayerCompositor.cpp: Added.
         * rendering/RenderLayerCompositor.h: Added.
         Per-RenderView controller object for compositing hierarchy
         Reviewed by Dave Hyatt
 
         https://bugs.webkit.org/show_bug.cgi?id=23356
-        
+
         Add new files related to accelerated compositing:
-        
+
         * WebCore.xcodeproj/project.pbxproj:
         Add the files to the project. Also add ColorCG.cpp, for the
         Color(CGColorRef) constructor.
-        
+
         * platform/graphics/GraphicsLayer.cpp: Added.
         * platform/graphics/GraphicsLayer.h: Added.
         GraphicsLayer is a platform abstraction for a hardware-backed
         layer.
-        
+
         * platform/graphics/GraphicsLayerClient.h: Added.
         Client of GraphicsLayer, allowing GraphicsLayer to call out
         to WebCore code.
-        
+
         * platform/graphics/mac/GraphicsLayerCA.h: Added.
         * platform/graphics/mac/GraphicsLayerCA.mm: Added.
         Mac Core Animation implementation of GraphicsLayer.
-        
+
         * platform/graphics/mac/WebLayer.h: Added.
         * platform/graphics/mac/WebLayer.mm: Added.
         * platform/graphics/mac/WebTiledLayer.h: Added.
 2009-01-30  Chris Fleizach  <cfleizach@apple.com>
 
         Reviewed by Oliver Hunt.
-        
+
         Bug 23656: AX hierarchy for iFrames is incorrect
         https://bugs.webkit.org/show_bug.cgi?id=23656
 
 2009-01-30  Geoffrey Garen  <ggaren@apple.com>
 
         Reviewed by Darin Adler.
-        
+
         More scaffolding for redirect tracking. No behavior change yet.
 
         * bindings/js/JSHTMLFormElementCustom.cpp:
         * bridge/IdentifierRep.cpp:
         (WebCore::identifierSet):
         Returns a set of all identifiers.
-        
+
         (WebCore::IdentifierRep::isValid):
         Return whether an identifier is valid, meaning that it's present in the
         set of identifiers.
-        
+
         * bridge/IdentifierRep.h:
         (WebCore::IdentifierRep::isString):
         (WebCore::IdentifierRep::number):
         (WebCore::CachedFrame::domWindow):
 
         * loader/FrameLoader.cpp:
-        (WebCore::FrameLoader::commitProvisionalLoad): Let CachedFrame  creation handle the 
+        (WebCore::FrameLoader::commitProvisionalLoad): Let CachedFrame  creation handle the
           suspending of active DOM objects.
         (WebCore::FrameLoader::open): Let CachedFrame::restore() handle resuming those DOM objects.
         (WebCore::FrameLoader::checkLoadCompleteForThisFrame): Call frameLoadCompleted() instead
 
         Reviewed by Nikolas Zimmermann, Eric Seidel.
 
-        Just transform the pattern directly instead of transforming the context fixes this 
+        Just transform the pattern directly instead of transforming the context fixes this
         problem. Texts or strokes are no longer affected by transformations.
 
         SVG pattern transformation/BoundingBox can cause ugly stroke thickness or text positions
         Move all of the contination insertion code from RenderFlow to RenderInline.  addChildToFlow is renamed to addChildIgnoringContinuation, and it has been moved to
         RenderContainer so that it can still be called on either blocks or inlines.  The base class implementation in RenderContainer is what RenderBlock uses.  RenderInline
         subclasses it to do all the work it used to do in addChildToFlow.
-        
+
         RenderBlock's old addChildToFlow can just become addChild.  This simplification is possible because addChild was actually never being called on anonymous
         block continuations.
-        
+
         The code dealing with anonymous table parts in the old addChildWithContinuation method can now be removed as a result with no harmful side effects.  Falling through to the
         base class and ignoring a block continuation makes the right thing happen.
-        
+
         addChildWithContinuation moved to RenderInline and has been renamed to addChildToContinuation.
-        
+
         Reviewed by Eric Seidel
 
         * rendering/RenderBlock.cpp:
 2009-01-29  David Hyatt  <hyatt@apple.com>
 
         Move dirtyLinesFromChangedChild into RenderLineBoxList so that it can be shared by RenderBlock and RenderInline.
-    
+
         Reviewed by Sam Weinig
 
         * rendering/RenderBlock.h:
 2009-01-28  Geoffrey Garen  <ggaren@apple.com>
 
         Reviewed by Sam Weinig.
-        
+
         Merged FrameLoadTypeReloadAllowingStaleData with FrameLoadTypeReload.
-        
+
         Technically, selecting View->Text Encoding->[Non-Default Encoding] on
         a page with frames may now be slightly slower than it used to be. Oh well.
 
 
 2009-01-28  David Hyatt  <hyatt@apple.com>
 
-        Fix for 
+        Fix for
 
         <rdar://problem/6531287> REGRESSION: CrashTracer: [USER] 12 crashes in Safari at com.apple.WebCore • WebCore::RenderBlock::removeFloatingObject + 59
 
         https://bugs.webkit.org/show_bug.cgi?id=23576
 
         Work towards eliminating RenderFlow from the tree.
-        
+
         Move the m_continuation variable down from RenderFlow into RenderInline and RenderBlock.  Since a block can only have an inline continuation following it, the type
         of the member and methods reflect this (inlineContinuation() and m_inlineContinuation).  Since an inline can have either a block or an inline continuation following it,
         a base class of RenderBox* is used for the type of object returned (in anticipation of the removal of RenderFlow).
-        
+
         Since moving the continuation variable down into the subclasses increased the size of RenderInline and RenderBlock by 4 bytes, this patch also moves all of the bitfield
         members of RenderFlow up to RenderObject.  Since they fit within the available bits on RenderObject, this patch actually results in a net savings of 4 bytes on RenderInlines
         and RenderBlocks!
-        
+
         One bitfield member was eliminated rather than moved: m_firstLine.  This was really more of a state variable used during line layout only, so I removed it as a member
         and just passed firstLine down to various methods as needed.  Doing so uncovered some potential bugs where the first line state was not being respected when querying for
         line offsets.
 
         Continuations have been completely hidden from all files except for RenderFlow, RenderBlock and RenderInline.  All of the code that referenced continuations directly
         from base classes has been refactored to use virtual methods on RenderBlock and RenderInline instead.
-        
+
         RenderFlow still has a common addChildWithContinuation method that is used by both blocks and inlines.  Since refactoring that method will be pretty tricky, I've held
         off on that for a later patch.
 
         Reviewed by Dave Hyatt
 
         https://bugs.webkit.org/show_bug.cgi?id=23360
-        
+
         Use the new container-relative repaint methods to compute
         post-layout repaints, and repaints using RenderLayer's
         cached repaintRect() relative to that container, rather than
 
         Flip the arguments of computeRectForRepaint() from
             computeRectForRepaint(IntRect&, RenderBox* repaintContainer, bool)
-        to 
+        to
             computeRectForRepaint(RenderBox* repaintContainer, IntRect&, bool)
 
         * rendering/RenderBox.cpp:
         Reviewed by Dave Hyatt
 
         https://bugs.webkit.org/show_bug.cgi?id=23567
-        
+
         Fix crash of svg/custom/stop-crash.svg testcase because of
         infinite recursion, as I missed replacing absoluteClippedOverflowRect()
         with clippedOverflowRectForRepaint() in RenderSVGGradientStop.
-        
+
         Also fix a missed conversion in RenderTableCol, and add a couple
         of comments.
 
 
 2009-01-27  Adele Peterson  <adele@apple.com>
 
-        RS by Dave Hyatt  
+        RS by Dave Hyatt
 
         Use the textfield appearance for search fields in themeWin.css so the border/background
         styling will work correctly on these fields.
 2009-01-26  Simon Fraser  <simon.fraser@apple.com>
 
         Reviewed by David Hyatt
-        
+
         Back out r40285, because it was checked in with no bug number, no
         testcase, is rendering change that did not get thorough review,
         and broke the Mac build.
 
         -Make the recently added back/forward cache logging much better by actually
          walking the entire frame tree and indenting the resulting output.
-        -Fix a null-termination bug in HistoryItem tree logging 
+        -Fix a null-termination bug in HistoryItem tree logging
 
         * history/HistoryItem.cpp:
         (WebCore::HistoryItem::showTreeWithIndent):
 2009-01-26  Anders Carlsson  <andersca@apple.com>
 
         Fix 64-bit build.
-        
+
         * WebCore.LP64.exp:
 
 2009-01-26  Dmitry Titov  <dimich@chromium.org>
 
         Fix for <rdar://problem/6525392> REGRESSION (r40180): Scroll wheel events over text scroll the page
         instead of the ancestor overflow (23525)
-        
+
         Make sure that we always call scroll() on the nearest enclosing RenderBox.  No test case is possible
         here, since wheel scrolling is currently untestable (the offset varies from machine to machine).
 
         https://bugs.webkit.org/show_bug.cgi?id=23467
 
         Change styleAvailable functionality to be a linked list
-        rather than scanning every CompositeAnimation for 
+        rather than scanning every CompositeAnimation for
         Animations that are waiting for styleAvailable. This has
         potential for large performance improvement in cases where
         there are many animations firing at once. But it will also
 
         Bug 23509: Crash at -[WebCoreAXObject doAXNextSentenceEndTextMarkerForTextMarker:]
         https://bugs.webkit.org/show_bug.cgi?id=23509
-  
+
         Add null checking in makeRange instead of AX code
 
         * editing/VisiblePosition.cpp:
 
         Reviewed by Kevin Ollivier.
 
-        Fix drawing in situations where sub-portions of the bitmap are to be 
+        Fix drawing in situations where sub-portions of the bitmap are to be
         drawn to the screen.
 
         * platform/graphics/wx/ImageWx.cpp:
 2009-01-23  Eric Carlson  <eric.carlson@apple.com>
 
         Reviewed by Adele Peterson
-        
+
         Fix for https://bugs.webkit.org/show_bug.cgi?id=23407
 
-        Implement QTMovieWin::hasVideo so the controller on an audio-only <video> element 
+        Implement QTMovieWin::hasVideo so the controller on an audio-only <video> element
         is never hidden (as with an <audio> element).
 
         Test: media/video-controls-visible-audio-only.html
 
         Bug 23509: Crash at -[WebCoreAXObject doAXNextSentenceEndTextMarkerForTextMarker:]
         https://bugs.webkit.org/show_bug.cgi?id=23509
-  
+
         Add null checking around next/previous sentence retrieval in AX code
 
         * page/AccessibilityObject.cpp:
 2009-01-23  Anders Carlsson  <andersca@apple.com>
 
         Fix tyop.
-        
+
         * editing/markup.cpp:
         (WebCore::joinMarkups):
 
 
         * Configurations/Base.xcconfig:
         Fix GCC 4.0 build.
-        
+
         * editing/markup.cpp:
         (WebCore::joinMarkups):
         Pass a const reference to the vector.
-        
+
         * html/HTMLElement.cpp:
         * platform/text/String.cpp:
         Add comments indicating that these functions are for use in the debugger.
         Reviewed by Sam Weinig.
 
         Turn on -Wmissing-prototypes and fix the resulting warnings.
-        
+
         * Configurations/Base.xcconfig:
         * bindings/js/JSHistoryCustom.cpp:
         (WebCore::nonCachingStaticBackFunctionGetter):
         add extra space above and below the content of a cell.  This system was not confined to the table code and spilled out into all the other RenderObjects.
         The y-position of the table cell box was set to the outer edge of the cell, but the y() method of RenderBox lied and added in borderTopExtra().  height()
         also excluded the extra space, so did not accurately reflect the true size of the cell.
-        
+
         With the new system, the table cell box is completely accurate.  The extra space becomes part of the padding of the cell.  Padding has been reworked so that
         additional intrinsic padding can be added on to the specified padding from style.  Only the table code has to deal with the extra cell padding.
 
         (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
         Make CSSComputedStyleDeclaration ask for padding values that exclude the built-in intrinsic padding.  This ensures that getComputedStyle continues
         to give the right answer.
-    
+
         * dom/ContainerNode.cpp:
         (WebCore::ContainerNode::getLowerRightCorner):
         * editing/visible_units.cpp:
         * rendering/LayoutState.cpp:
         (WebCore::LayoutState::LayoutState):
         Remove borderTopExtra()/borderBottomExtra() hacks and localToAbsoluteForContent calls.
-        
+
         * rendering/RenderBlock.cpp:
         (WebCore::RenderBlock::overflowRect):
         Remove borderTopExtra()/borderBottomExtra() hacks and localToAbsoluteForContent calls.
-        
+
         (WebCore::RenderBlock::layoutBlock):
         (WebCore::RenderBlock::expandsToEncloseOverhangingFloats):
         The expandsToEncloseOverhangingFloats logic has been reworked.  Table cells and fieldsets did not properly handle floats that spilled into the bottom padding.  The
         resulting box needed to not only expand to encompass the float, but also needed to place the full bottom padding after the float.  The switch to make the extra table cell
         space into padding exposed this issue.  No extra layout test is required, since an existing table layout test exposes this issue and progresses to match Firefox with
         this change.
-    
+
         (WebCore::RenderBlock::selectionGapRects):
         (WebCore::RenderBlock::paintSelection):
         (WebCore::RenderBlock::fillSelectionGaps):
         (WebCore::RenderBlock::positionForCoordinates):
         (WebCore::RenderBlock::offsetForContents):
         Remove borderTopExtra()/borderBottomExtra() hacks and localToAbsoluteForContent calls.
-    
+
         * rendering/RenderBlock.h:
         Add the new simplified method for expanding to encompass overhanging floats.  The method has been devirtualized and made to include all of the cases (and not just a subset).
 
         (WebCore::RenderBox::absoluteToLocal):
         (WebCore::RenderBox::localToAbsoluteQuad):
         Remove borderTopExtra()/borderBottomExtra() hacks and localToAbsoluteForContent calls.
-        
+
         * rendering/RenderBox.h:
         (WebCore::RenderBox::y):
         (WebCore::RenderBox::location):
         (WebCore::RenderTableCell::paddingTop):
         (WebCore::RenderTableCell::paddingBottom):
         The new paddingTop() and paddingBottom() methods on table cells include the extra intrinsic padding.
-    
+
         (WebCore::RenderTableCell::setOverrideSize):
         When a table's override size gets altered, the intrinsic padding needs to be cleared.
-        
+
         (WebCore::RenderTableCell::absoluteClippedOverflowRect):
         (WebCore::RenderTableCell::computeAbsoluteRepaintRect):
         (WebCore::RenderTableCell::baselinePosition):
         (WebCore::RenderTableCell::paintBoxDecorations):
         (WebCore::RenderTableCell::paintMask):
         Remove borderTopExtra()/borderBottomExtra() hacks and localToAbsoluteForContent calls.
-        
+
         * rendering/RenderTableCell.h:
         (WebCore::RenderTableCell::setIntrinsicPaddingTop):
         (WebCore::RenderTableCell::setIntrinsicPaddingBottom):
         (WebCore::RenderTableCell::intrinsicPaddingTop):
         (WebCore::RenderTableCell::intrinsicPaddingBottom):
         Add new helper methods for getting/setting a cell's intrinsic padding.
-    
+
         * rendering/RenderTableSection.cpp:
         (WebCore::RenderTableSection::calcRowHeight):
         calcRowHeight has been modified to exclude the intrinsic padding when calculating the base height of rows prior to flexing.  Because a cell now includes that
         extra space, it has to be subtracted out in this method.
-        
+
         (WebCore::RenderTableSection::layoutRows):
         Modify the code that sets up the intrinsic padding so that it does a relayout if the intrinsic padding changes.   There was also an error where the baseline
         position mismatched leading to negative intrinsic padding being added in (this error exists in ToT).  The code now properly ignores cells that don't establish
         a baseline.  A number of tests progress with this change.
-        
+
         * rendering/RenderText.cpp:
         (WebCore::RenderText::addLineBoxRects):
         Remove borderTopExtra()/borderBottomExtra() hacks and localToAbsoluteForContent calls.
-        
+
         * rendering/RenderTreeAsText.cpp:
         (WebCore::operator<<):
         (WebCore::writeTextRun):
 
         * WebCore.LP64.exp:
         Add some bridge related symbols.
-        
+
         * WebCore.xcodeproj/project.pbxproj:
         Make runtime_object.h a private header.
 
         Fix for https://bugs.webkit.org/show_bug.cgi?id=23317
 
         The high CPU usage was really from repeatedly firing transitions caused
-        by a bug in the way we handle background-color animations. If animating 
+        by a bug in the way we handle background-color animations. If animating
         from a valid background color to no background color, we sometimes left
         (based on timing) the background color as transparent black, but valid
         rather than invalid, which it should be. Fixing that got rid of the
 
         Reviewed by Beth Dakin.
 
-        Bug 23443: Table accessibility should be re-enabled after fixing crash that occurs at WebCore::AccessibilityTable::isTableExposableThroughAccessibility() 
+        Bug 23443: Table accessibility should be re-enabled after fixing crash that occurs at WebCore::AccessibilityTable::isTableExposableThroughAccessibility()
         https://bugs.webkit.org/show_bug.cgi?id=23443
 
         Test: accessibility/table-modification-crash.html
 
         Fix execCommand() 'super' and 'sub' commands to add <sup> and <sub> in quirks mode, and to toggle when called twice
         https://bugs.webkit.org/show_bug.cgi?id=17733
-        
+
         Test changed: editing/execCommand/toggle-styles-expected.txt
 
         * editing/ApplyStyleCommand.cpp:
 
         Bug 23438: Provide a default value to the .in file parser
 
-        Added the default value 1 as it simplifies some of 
+        Added the default value 1 as it simplifies some of
         the format by removing some "=1" that were awkward.
 
         This was suggested by Darin Adler as par of a previous
         * bridge/c/c_class.cpp:
         * bridge/c/c_class.h:
         Get rid of CClass::name().
-        
+
         * bridge/jni/jni_class.h:
         Get rid of JavaClass::name().
-        
+
         * bridge/objc/objc_class.h:
         * bridge/objc/objc_class.mm:
         Get rid of ObjcClass::name().
-        
+
         * bridge/objc/objc_instance.h:
         Get rid of supportsSetValueOfUndefinedField.
-        
+
         * bridge/objc/objc_instance.mm:
         (ObjcInstance::setValueOfUndefinedField):
         Fold supportsSetValueOfUndefinedField into setValueOfUndefinedField.
-        
+
         * bridge/runtime.cpp:
         Get rid of getValueOfField and setValueOfField.
-        
+
         * bridge/runtime.h:
         (JSC::Bindings::Instance::setValueOfUndefinedField):
         Have this return a bool and get rid of supportsSetValueOfUndefinedField
-        
+
         * bridge/runtime_object.cpp:
         (JSC::RuntimeObjectImp::fieldGetter):
         Call Field::valueFromInstance on the field.
-        
+
         (JSC::RuntimeObjectImp::put):
         Call Field::setValueToInstance on the field.
 
 
         Build fix: remove duplicate entries in the Xcode project, and
         sort the project with sort-Xcode-project-file .
-        
+
         * WebCore.xcodeproj/project.pbxproj:
 
 2009-01-19  Simon Fraser  <simon.fraser@apple.com>
         * WebCore.xcodeproj/project.pbxproj:
         * WebCoreSources.bkl:
         Add CSSPropertyLonghand.h/cpp.
-        
+
         * css/CSSMutableStyleDeclaration.cpp:
         (WebCore::CSSMutableStyleDeclaration::removeShorthandProperty):
         Move CSSPropertyLonghand to its own file.
-        
+
         * css/CSSPropertyLonghand.cpp: Added.
         (WebCore::initShorthandMap):
         (WebCore::longhandForProperty):
         Reviewed by Kevin Decker.
 
         Allow plug-in views to return an instance directly.
-        
+
         * bindings/js/ScriptControllerMac.mm:
         (WebCore::ScriptController::createScriptInstanceForWidget):
 
 
 2009-01-18  Eric Carlson  <eric.carlson@apple.com>
 
-        Build fix for !ENABLE(VIDEO) following r40016. 
+        Build fix for !ENABLE(VIDEO) following r40016.
 
         * rendering/RenderSlider.cpp:
         (WebCore::RenderSlider::mouseEventIsInThumb):
         * WebCore.xcodeproj/project.pbxproj: add mediaControlsQT.css
 
         * css/CSSPrimitiveValueMappings.h:
-        (WebCore::CSSPrimitiveValue::CSSPrimitiveValue): add MediaTimelineContainerPart, 
+        (WebCore::CSSPrimitiveValue::CSSPrimitiveValue): add MediaTimelineContainerPart,
         CSSValueMediaCurrentTimeDisplay, and CSSValueMediaTimeRemainingDisplay
 
         * css/CSSSelector.cpp:
         (WebCore::CSSSelector::extractPseudoType): include new media controller element styles
-        * css/CSSSelector.h: 
+        * css/CSSSelector.h:
         (WebCore::CSSSelector::): Ditto.
 
         * css/CSSStyleSelector.cpp:
-        (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector):  include new media 
+        (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector):  include new media
         controller element styles
 
         * css/CSSValueKeywords.in: add keywords for new media time display controls
         * rendering/RenderMedia.h: Updated
 
         * rendering/RenderSlider.cpp:
-        (WebCore::RenderSlider::mouseEventIsInThumb): When slider is media timeline control, call 
+        (WebCore::RenderSlider::mouseEventIsInThumb): When slider is media timeline control, call
         element hitTest() method instead of assuming it is rectangular
         * rendering/RenderSlider.h: make mouseEventIsInThumb virtual
 
 2009-01-16  Steve Falkenburg  <sfalken@apple.com>
 
         <rdar://problem/6502511> Safari crashes if it's running while the desktop theme is changed.
-        
+
         This was caused by mismatched ENABLE definitions across WebCore and WebKit.
-        
+
         Several virtual methods were added to RenderTheme.h, conditionalized by ENABLE(VIDEO).
         In addition to adding ENABLE_VIDEO to WebKit, this change also adds ENABLE_DATABASE and ENABLE_ICONDATABASE
         to Windows WebCore/WebKit, and adds ENABLE_WORKERS, and several ENABLE_SVG_ flags to WebKit on Windows.
         Reviewed by Sam Weinig.
 
         Clean up the bridge code and get rid of Field::name() and Method::name().
-        
+
         * bridge/c/c_instance.cpp:
         (JSC::Bindings::CInstance::invokeMethod):
         * bridge/c/c_runtime.cpp:
         which causes an infinite loop. I fixed this by deferring the setChanged to the next
         run loop iteration. That made it not infinite loop, but it still retriggers the
         transition forever. The problem is that there is both an 'all' and specific transition
-        on 'opacity'. This tickled a bug in AnimationController which causes the opacity 
+        on 'opacity'. This tickled a bug in AnimationController which causes the opacity
         transition to get constantly cancelled and then retriggered. The problem is that
-        the specific opacity transition has a duration of 0. I got rid of the logic to 
+        the specific opacity transition has a duration of 0. I got rid of the logic to
         flush out 0 duration transitions and it is no longer constantly triggered. The
         logic to flush them was just an optimization, and you really need to keep them
         around to make the logic to override earlier animations by later ones work. And there is
 2009-01-15  Justin Garcia  <justin.garcia@apple.com>
 
         Reviewed by Oliver Hunt.
-        
+
         <rdar://problem/6444148> Styling a selection that ends in a line break can sometimes style what's after the break
 
         If the range to style ended at [node, 0] or inside node (and if for some reason the node didn't get split),
         applyInlineStyle would style node.
 
         * editing/ApplyStyleCommand.cpp:
-        (WebCore::ApplyStyleCommand::applyInlineStyle): Rename pastLast to pastEndNode, since it doesn't come from 
+        (WebCore::ApplyStyleCommand::applyInlineStyle): Rename pastLast to pastEndNode, since it doesn't come from
         Range::pastLastNode() anymore.  pastEndNode is the node after the last one that is fully selected, since
         the work done in the loop that follows should only be performed on fully selected nodes.
         * editing/InsertLineBreakCommand.cpp:
         (WebCore::InsertLineBreakCommand::doApply): Explicitly handle insertion into containers and after non-text nodes.
         Those were handled accidently before.
         * editing/htmlediting.cpp:
-        (WebCore::caretMaxOffset): In various places we call this with a container and expect to be given the number 
-        of children in that container, so I changed it to match that.  RenderBR and RenderObject::caretMaxOffset() should 
+        (WebCore::caretMaxOffset): In various places we call this with a container and expect to be given the number
+        of children in that container, so I changed it to match that.  RenderBR and RenderObject::caretMaxOffset() should
         also be eliminated since maxDeepOffset handles non-text nodes.
 
 2009-01-15  Anders Carlsson  <andersca@apple.com>
 2009-01-15  Anders Carlsson  <andersca@apple.com>
 
         Another attempt at fixing the Qt build.
-        
+
         * bridge/qt/qt_instance.cpp:
         (JSC::Bindings::QtInstance::getOwnPropertySlot):
         (JSC::Bindings::QtInstance::put):
         Reviewed by Geoffrey Garen.
 
         Make Instance::createRuntimeObject a virtual function and override it for QtInstance.
-        
+
         * bindings/js/JSPluginElementFunctions.cpp:
         (WebCore::getRuntimeObject):
         * bridge/c/c_utility.cpp:
 
         No new tests added.  In the future, we need to update DumpRenderTree to be able to run with different RenderThemes and
         we'll need a separate set of results for different themes.
-        
+
         * css/themeWin.css: Renable search fields.  Added new rules for sub-element placement.
-        
+
         * rendering/RenderTextControlSingleLine.cpp:
         (WebCore::RenderTextControlSingleLine::textBlockWidth): Consider margin set on the sub-elements when computing the
         desired width for the text block.
-        
+
         * rendering/RenderThemeWin.cpp:
         (WebCore::RenderThemeWin::supportsFocus): Treat SearchFieldPart the same as TextFieldPart.
         (WebCore::RenderThemeWin::determineState): ditto.
 2009-01-14  Dimitri Glazkov  <dglazkov@chromium.org>
 
         Reviewed by Eric Seidel.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=23321
         Upstream more bits of graphics/chromium.
-        
+
         * platform/graphics/chromium/FontPlatformDataChromiumWin.cpp: Added.
         (WebCore::FontPlatformData::FontPlatformData):
         (WebCore::FontPlatformData::operator=):
         Reviewed by Simon Hausmann.
 
         [Qt] Make sure media elements dispatch the 'loaded' event
-        
+
         We assume that when Phonon goes into paused state that we have the
         complete media file. Once we do media loading ourselves we can
         distinguish between loading the first frame and the complete media.
         Reviewed by Simon Hausmann.
 
         Change how themes adjust mediaControls.css to match html4/quicks.css
-        
+
         Instead of providing the full style sheet, the themes provide extra
         overrides to the default style defined in UserAgentStyleSheetsData.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=23210
-        
+
         Also, merge WebKitResources.qrc and WebCoreResources.qrc to speed up
         build time for the Qt port.
 
         Reviewed by Darin Adler
 
         Fix for <https://bugs.webkit.org/show_bug.cgi?id=22096>
-        Bug 22096: REGRESSION (r35879) scrolldelay is counted in seconds 
+        Bug 22096: REGRESSION (r35879) scrolldelay is counted in seconds
         instead of miliseconds
 
         Test: fast/css/webkit-marquee-speed-unit-in-quirksmode.html
 
         * css/CSSParser.cpp:
-        (WebCore::CSSParser::validUnit): treat unitless values in quirks mode 
+        (WebCore::CSSParser::validUnit): treat unitless values in quirks mode
         as miliseconds instead of seconds.
 
 2009-01-12  David Hyatt  <hyatt@apple.com>
 
         !ENABLE(SVG_FONTS) build fix. Move defaultUnitsPerEm into a non-SVG header
         so it can be used by all builds.
-        
+
         * platform/graphics/Font.h:
         * platform/graphics/SimpleFontData.cpp:
         * svg/SVGFontFaceElement.cpp:
         In RenderBox::repaintLayerRectsForImage(), the repaint rect does not have
         to be computed in absolute coordintes. Instead, we compute a repaintRect
         relative to the RenderObject, which can repaint itself.
-        
+
         * rendering/RenderBox.cpp:
         (WebCore::RenderBox::repaintLayerRectsForImage):
 
         (WebCore::FontCache::FontCache):
 
 2009-01-12  Eric Roman  <eroman@chromium.org>
+
         Reviewed by Darin Adler.
+
         Fix some bugs with Selection::appendTrailingWhitespace().
         https://bugs.webkit.org/show_bug.cgi?id=23232
+
         Test: editing/selection/doubleclick-whitespace-crash.html
+
         * editing/Selection.cpp:
         (WebCore::makeSearchRange):
         (WebCore::Selection::appendTrailingWhitespace):
         Reviewed by Oliver Hunt
 
         https://bugs.webkit.org/show_bug.cgi?id=23242
-        
+
         Fix CanvasRenderingContext2D::transform to do a pre-multiply,
         rather than a post-multiply into m_transform. This bug did not affect
         drawing, but did cause m_transform to be incorrect, which impacted
         willDraw(), and isPointInPath.
-        
+
         Test: fast/canvas/canvas-incremental-repaint-2.html
 
         * html/CanvasRenderingContext2D.cpp:
 
         Checking whether malloc can allocate memory or not. If it can't, CRASH macro is invoked
         (like in fastMalloc).
-   
+
         * bridge/npruntime.cpp:
         (_NPN_GetStringIdentifier):
         (_NPN_GetIntIdentifier):
         Test: editing/selection/extend-selection-bidi.html
 
         * rendering/RootInlineBox.cpp: (WebCore::RootInlineBox::fillLineSelectionGap):
-        Make this function  not assume a contiguous visual LTR selection range. 
+        Make this function  not assume a contiguous visual LTR selection range.
 
 2009-01-11  Dmitry Titov  <dimich@chromium.org>
 
 
         * css/CSSStyleSelector.cpp:
         (WebCore::CSSStyleSelector::styleForElement): call theme()->styleSheetForMediaControls to
-        get the media controller style sheet 
+        get the media controller style sheet
 
         * rendering/MediaControlElements.cpp:
         (WebCore::MediaControlInputElement::hitTest): Added
         Reviewed by Dan Bernstein
 
         https://bugs.webkit.org/show_bug.cgi?id=23222
-        
+
         We need to disable LayoutState when an object has transforms,
         because LayoutState is not transform-aware and therefore
         repaint rects can be computed incorrectly.
-        
+
         Test: fast/repaint/transform-disable-layoutstate.html
 
         * rendering/RenderBlock.cpp:
 
         Fix infinite recursion in clientPaddingLeft and clientPaddingRight.  This changes the code back to
         how it was before RenderTextControlSingleLine was split out from RenderTextControl.
-        
+
         No test added since this code is only exercised when trying to place a search field's recent searches popup menu.
 
         * rendering/RenderTextControlSingleLine.cpp:
 2009-01-09  Anders Carlsson  <andersca@apple.com>
 
         Fix build.
-        
+
         * WebCore.LP64.exp:
 
 2009-01-09  Dimitri Glazkov  <dglazkov@chromium.org>
         Reviewed by Simon Hausmann.
 
         Prevent qmake from generating duplicate rules for embedded stylesheets
-        
+
         The STYLESHEETS_EMBED variable used to contain only one file name, but
         now that it is a list of files we need to change it to be a dependency
         for the stylesheet generator instead of the input.
 
         Add ImageSourceSkia to platform/graphics/skia
         https://bugs.webkit.org/show_bug.cgi?id=23200
-        
+
         ImageSourceSkia is mostly a hack to support our
         ICO decoder model.  See ImageSourceSkia.h for more
         explanation.  Eventually we'd like to make our ICO
 
         Reviewed by Oliver Hunt.
 
-        Fix <rdar://problem/6467206> 
+        Fix <rdar://problem/6467206>
         Resources loaded from the memory cache do not get correctly inserted into the DocLoader resource map (22994)
-        
+
         Use CachedResourceHandle in document resource map so resources get updated correctly when using
         using cache validation conditionals.
 
 
         Reviewed by Darin Adler.
 
-        Part one of 
+        Part one of
         https://bugs.webkit.org/show_bug.cgi?id=23165
         Add support for application cache dynamic entries
 
         * loader/appcache/ApplicationCache.h: Added a list of pending dynamic entry actions, to be
         used in the near future.
 
-        * loader/appcache/DOMApplicationCache.idl: Updated for spec changes. Instead of length 
+        * loader/appcache/DOMApplicationCache.idl: Updated for spec changes. Instead of length
         attribute and item(), we now have an items attribute that returns a DOMStringList, and a
         hasItem convenience method.
 
 2009-01-07  Anders Carlsson  <andersca@apple.com>
 
         Another build fix.
-        
+
         * WebCore.LP64.exp:
 
 2008-12-16  David Hyatt  <hyatt@apple.com>
         Reviewed by Oliver Hunt.
 
         <rdar://problem/6391734> SnowLeopard: Crash doing Copy Image from context menu
-        
+
         Can't create an automated test case for specific contextual menu items yet and
         a normal Copy doesn't cause a crash.
 
                animations/simultaneous-start-transform.html
 
         Fixed https://bugs.webkit.org/show_bug.cgi?id=22870
+
         I added calls beginAnimationUpdate() and endAnimationUpdate() calls
         to AnimationController. These are called by Document at the start
-        and end of the recalcStyle cycle. Right now, I'm just using the 
+        and end of the recalcStyle cycle. Right now, I'm just using the
         beginAnimationUpdate() method to reset an animation time value.
         The first time the animation time is accessed after this reset I set
         it to the currentTime. So all animations in that cycle get the same
-        start time. 
+        start time.
 
         The test cases checked in test this, but in the case of the 'left'
         test it actually doesn't make any difference in most cases. This is
-        because values are clamped to whole pixels, so the start times would 
+        because values are clamped to whole pixels, so the start times would
         have to be pretty far off for the test to fail using the old
         currentTime() model. Still, under really heavy load, it's possible for
         the test to fail without these changes.
 
         Reviewed by Sam Weinig.
 
-        Add a way for frame loader clients to always create a PluginDocument, regardless of 
+        Add a way for frame loader clients to always create a PluginDocument, regardless of
         the real document MIME type.
-        
+
         * loader/FrameLoader.cpp:
         (WebCore::FrameLoader::begin):
         (WebCore::FrameLoader::shouldUsePlugin):
 2009-01-06  David Smith  <catfish.man@gmail.com>
 
         Reviewed by Darin Adler.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=23129
         Devirtualize Node::childNodes().
 
         Reviewed by Dave Hyatt
 
         https://bugs.webkit.org/show_bug.cgi?id=22985
-        
+
         Add an assertion that clip rects are being used when painting with the same
         rootLayer that they were computed with.
-        
+
         Fix two issues detected by the assertion:
 
         RenderLayer::updateClipRects() should not unconditionally update the clip rects
         on its parent, but stop when reaching rootLayer (just like calculateClipRects()).
-        
+
         We need to pass the temporaryClipRects flag down through reflection painting
         to handle the case of nested reflections.
-        
+
         Also use temporary clip rects in RenderTreeAsText, since that code does not
         reset the painting root for transformed layers, so cached clip rects will not
         match those used for painting.
 
         Implement 'pointer-events' for HTML content. This involved
         adding a new value 'auto' which behaves as 'visiblePainted'
-        in SVG content. 
-        
+        in SVG content.
+
         Moved the property out of the SVG CSS code and into
-        the general CSS (both parsing and RenderStyle). 
-        
+        the general CSS (both parsing and RenderStyle).
+
         Changes to the hit testing functionality of the Render tree,
-        specifically the nodeAtPoint methods. Where they used to 
+        specifically the nodeAtPoint methods. Where they used to
         test for visibility, they now use a helper function defined
         on base classes (RenderObject and InlineBox) that checks both
         visibility and pointer-events.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=11395
 
         Tests: fast/events/pointer-events-2.html
 
         Reviewed by Gavin Barraclough.
 
-        CanvasPixelArray performance is too slow 
+        CanvasPixelArray performance is too slow
         <https://bugs.webkit.org/show_bug.cgi?id=23123>
 
         Remove the WebCore CanvasPixelArray implementation and replace
 2009-01-05  Anders Carlsson  <andersca@apple.com>
 
         Build fix.
-        
+
         * plugins/PluginView.h:
         (WebCore::PluginManualLoader::~PluginManualLoader):
 
 2009-01-05  Anders Carlsson  <andersca@apple.com>
 
         Reviewed by Kevin Decker.
-        
-        Add an abstract PluginManualLoader class and make PluginView inherit from it. 
-        
+
+        Add an abstract PluginManualLoader class and make PluginView inherit from it.
+
         Add some error checking that currently exists in WebKit (but not for long!)
-        
+
         * plugins/PluginView.cpp:
         (WebCore::PluginView::didReceiveResponse):
         (WebCore::PluginView::didReceiveData):
         Reviewed by Darin Adler
 
         https://bugs.webkit.org/show_bug.cgi?id=23090
-        
+
         If an object gets a Layout hint, and the style change will result in
         the creation of a RenderLayer, then we need to repaint the old position
         of the object. This was done for transform, but we have to test opacity too.
 
         Reverse the order in the icon database main thread loop to
         write the pending icons to the database before trying
-        to read any requested icons. This ensures that a requested icon 
+        to read any requested icons. This ensures that a requested icon
         has the correct data when read.
-        
+
         * loader/icon/IconDatabase.cpp:
         (WebCore::IconDatabase::syncThreadMainLoop):
 
         https://bugs.webkit.org/show_bug.cgi?id=23025
         DOMTimer lifetime cleanup: timeoutMap methods on Document now do not delete the timer.
         Instead, all 3 places that delete timers do it directly calling 'delete' and then timer's dtor removes the ID from the timeoutMap.
-        Note that in case the context is destroyed and timers are deleted at once, the check in ~DOMTimer() prevents 
+        Note that in case the context is destroyed and timers are deleted at once, the check in ~DOMTimer() prevents
         unnecessary HashMap remove in case the Document is being destroyed.
 
         * bindings/js/DOMTimer.cpp:
         Reviewed by Darin Adler
 
         https://bugs.webkit.org/show_bug.cgi?id=23082
-        
+
         Fix GIF animation by ensuring that the signature of
         startAnimation() in the base class matches the method in BitmapImage.
-        
+
         Test: fast/backgrounds/animated-gif-as-background.html
 
         * platform/graphics/Image.h:
 2009-01-02  David Smith  <catfish.man@gmail.com>
 
         Reviewed by Darin Adler.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=22699
         Enable NodeList caching for getElementsByTagName
-        
+
         test: fast/dom/getelementsbytagnamens-mixed-namespaces.html
 
         * dom/Document.cpp:
         (WebCore::MappedAttributeHash::hash): Use WTF::stringHashingStartValue
         * dom/TagNodeList.cpp:
         (WebCore::TagNodeList::TagNodeList): Add a NodeList cache argument
-        * dom/TagNodeList.h: 
+        * dom/TagNodeList.h:
         (WebCore::TagNodeList::create): Add a NodeList cache argument
         * platform/text/StringHash.h:
         (WebCore::CaseFoldingHash::hash): Use WTF::stringHashingStartValue
 
         Reviewed by Darin Adler
 
-        Fix for https://bugs.webkit.org/show_bug.cgi?id=23066 & <rdar://problem/6028417> 
+        Fix for https://bugs.webkit.org/show_bug.cgi?id=23066 & <rdar://problem/6028417>
         Full Page Zoom: a <video> element that doesn't include width/height attribute does not scale
 
         * rendering/RenderVideo.cpp:
         Reviewed by Darin Adler
 
         Use an OwnPtr for m_chromeClient.
-        
+
         * svg/graphics/SVGImage.cpp:
         (WebCore::SVGImage::~SVGImage):
         (WebCore::SVGImage::dataChanged):
         a rect parameter for the changed rect, allowing incremental repaints.
         Fix RenderImage::imageChanged to take advantage of this changedRect to
         only repaint the changed parts of the image.
-        
+
         This also enables incremental painting for canvas-as-image, so
         add tests for that too.
 
         Reviewed by Darin Adler, Oliver Hunt
 
         https://bugs.webkit.org/show_bug.cgi?id=23065
-        
+
         Enable incremental painting of canvas. This turns incremental painting
         back on, and fixes issues with stroke width and miters, calling willDraw()
         for strokeText and fillText, and taking shadows into account.
-        
+
         Test: fast/canvas/canvas-incremental-repaint.html
 
         * html/CanvasRenderingContext2D.cpp:
         Reviewed by Darin Adler
 
         https://bugs.webkit.org/show_bug.cgi?id=23042
-        
+
         Rather than doing a repaint() inside of computeAbsoluteRepaintRect()
         when there is a reflection, compute the location of the rect inside
         the reflection and take the union of the unreflected and reflected rects.
-        
+
         Test: fast/repaint/reflection-redraw.html
 
         * rendering/RenderBox.cpp:
         Reviewed by Brady Eidson.
 
         Fix for https://bugs.webkit.org/show_bug.cgi?id=21797
-        <rdar://problem/6310682> REGRESSION: Crash in CFHTTPCookieStorageCopy beneath WebCore::cookies() when 
+        <rdar://problem/6310682> REGRESSION: Crash in CFHTTPCookieStorageCopy beneath WebCore::cookies() when
         running fast/dom/document-attribute-js-null.html and http/tests/security/cookies/create-document.html
-        
+
         Return early if the document is trying to get or set a cookie with an empty cookie url.
 
         * dom/Document.cpp:
         Prepare to add create/remove timeout methods to JSWorkerContext by moving
         timer-specific code from JSDOMWindowBase to DOMTimer.
         Moved everything JS-related from DOMTimer to ScheduledAction.
-        Now ScheduledAction is what it wanted to be all the time: a JS engine-specific 
+        Now ScheduledAction is what it wanted to be all the time: a JS engine-specific
         container for timer callback that knows how to invoke it.
         DOMTimer is not anymore JS-specific.
 
         Reviewed by Oliver Hunt.
 
         <rdar://problem/6465669> Frequent !isPurgeable() assertion in WebCore::CachedResource::addClient
-        
+
         Disallow turning resources that are being revalidated to purgable state.
-        
+
         No test, the condition is difficult to produce in DRT.
 
         * loader/CachedCSSStyleSheet.cpp:
         Reviewed by Dan Bernstein
 
         https://bugs.webkit.org/show_bug.cgi?id=22941
-        
+
         If the document element has opacity, we need to erase the view background to
         white before painting.
-        
+
         Test: fast/backgrounds/opacity-on-document-element.html
 
         * rendering/RenderView.cpp:
         Reviewed by Dan Bernstein
 
         https://bugs.webkit.org/show_bug.cgi?id=21910
-        
+
         Fix SVGImage painting by ensuring that the SVGImage resizes its FrameView correctly.
         Otherwise the FrameView is left at 0x0, and nothing paints.
-        
+
         * svg/graphics/SVGImage.cpp:
         (WebCore::SVGImage::draw):
 
         Reviewed by Kevin Ollivier.
 
         Add Context Menu support to wx bindings.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=22675
 
         * platform/ContextMenu.h:
 
         Reviewed by Darin Adler.
 
-        Temporary band-aide fix for <rdar://problem/6372481> In Gmail, a 
-        crash occurs at 
-        AccessibilityTable::isTableExposableThroughAccessibility() when 
+        Temporary band-aide fix for <rdar://problem/6372481> In Gmail, a
+        crash occurs at
+        AccessibilityTable::isTableExposableThroughAccessibility() when
         attempting to create a link in a rich text message
 
-        We need to disable Accessibility Tables until we get this fixed for 
+        We need to disable Accessibility Tables until we get this fixed for
         real to prevent rampant crashing.
 
         * page/AccessibilityTable.cpp:
         Reviewed by Darin Adler
 
         https://bugs.webkit.org/show_bug.cgi?id=22938
-        
+
         When the document element is transformed, we need to paint
         the view background to avoid unpainted areas.
-        
+
         Test: fast/transforms/transformed-document-element.html
 
         * rendering/RenderView.cpp:
           storage is disallowed, all authentication challenges are sent to
           the client.
         - let the FrameLoaderClient decide whether to use the credential
-          storage. 
+          storage.
 
         * loader/EmptyClients.h:
         (WebCore::EmptyFrameLoaderClient::shouldUseCredentialStorage): Added.
         to the FrameLoaderClient.
         * loader/FrameLoader.h: Declared shouldUseCredentialStorage().
         * loader/FrameLoaderClient.h: Declared shouldUseCredentialStorage().
-        * loader/ResourceLoader.cpp: 
+        * loader/ResourceLoader.cpp:
         (WebCore::ResourceLoader::shouldUseCredentialStorage): Added. Calls
         through to the FrameLoader.
         * loader/ResourceLoader.h:
 
         Reviewed by John Sullivan.
 
-        Add new FrameLoaderClient method to indicate the first visually 
+        Add new FrameLoaderClient method to indicate the first visually
         non-empty layout based on an heuristic.  Right now that heuristic
         is the first layout after an image, text or plugin has been added
         to the render tree, but I can imagine it becoming much smarter.
         If JavaScript is not currently executing, the handleEvent member function
         of JSAbstractEventListener should set the dynamic global object to the
         global object of the context in which the event occurred.
-        
+
         If this is not set, then JavaScriptCore will simply take the global object
         of the context where the event handler function was created, which may be
         a different frame. This will cause the popup blocker to incorrectly block
         Reviewed by Dave Hyatt
 
         https://bugs.webkit.org/show_bug.cgi?id=22570
-        
+
         Add the ability to compute clip rects independently from
-        caching them on the RenderLayer. When painting reflections, use 
+        caching them on the RenderLayer. When painting reflections, use
         such temporarily computed clipRects, otherwise the layer may cache
         clipRects which are invalid, since they have been computed with
         a rootLayer that is not the one usually used to paint.
-        
+
         Test: fast/reflections/reflection-overflow-hidden.html
 
         * rendering/RenderLayer.cpp:
 
         https://bugs.webkit.org/show_bug.cgi?id=22618
 
-        Fix MinGW QtWebKit linking problems and also make the 
+        Fix MinGW QtWebKit linking problems and also make the
         QtWebKit build system more robust.
 
         * WebCore.pro:
 
         Reviewed by Geoff Garen.
 
-        Change HTTPHeaderMap to use an AtomicString as its key.  
+        Change HTTPHeaderMap to use an AtomicString as its key.
         Shaves ~1MB off of the Mozilla Memory Test
-        No functionality difference    
+        No functionality difference
 
         * WebCore.xcodeproj/project.pbxproj:
         * inspector/InspectorController.cpp:
         Fix for https://bugs.webkit.org/show_bug.cgi?id=22871
         <rdar://problem/6417316> RenderThemeWin buttons are too short/thin
 
-        * rendering/RenderButton.cpp: (WebCore::RenderButton::setupInnerStyle): 
+        * rendering/RenderButton.cpp: (WebCore::RenderButton::setupInnerStyle):
         Check if the button's style (not the new inner style) has appearance set to determine whether we should set padding on the inner style.
 
 2008-12-15  Nikolas Zimmermann  <nikolas.zimmermann@torchmobile.com>
 
         * bindings/js/JSGeolocationCustom.cpp:
         (WebCore::createPositionOptions): Added. Extracts the enableHighAccuracy
-        and timeout fields from a vanilla JS object in order to create the 
+        and timeout fields from a vanilla JS object in order to create the
         PositionOptions object, checking for exceptions as necessary.
         (WebCore::JSGeolocation::getCurrentPosition): Use createPositionOptions
         instead of toPositionOptions.
         Reviewed by Darin Adler.
 
         When a resource is cached locally, WebKit should follow RFC 2616 "Specific end-to-end revalidation" instead of "Unspecified end-to-end revalidation"
-        https://bugs.webkit.org/show_bug.cgi?id=17998 
-        
-        - Enable conditional revalidation for reloads by default. 
+        https://bugs.webkit.org/show_bug.cgi?id=17998
+
+        - Enable conditional revalidation for reloads by default.
         - Add a parameter to FrameLoader::reload() for forcing end-to-end reload.
-        - To avoid duplicating state remove m_cachePolicy variables from FrameLoader and DocLoader. 
+        - To avoid duplicating state remove m_cachePolicy variables from FrameLoader and DocLoader.
           Instead synthezise the policy on demand.
-        
+
         This speeds up reloads and makes them use way less bandwidth.
 
         * WebCore.base.exp:
         (WebCore::DocLoader::checkForReload): Support CachePolicyRevalidate.
         (WebCore::DocLoader::requestResource):
         (WebCore::DocLoader::cachePolicy):
-        * loader/DocLoader.h: Get rid of m_cachePolicy member. 
+        * loader/DocLoader.h: Get rid of m_cachePolicy member.
         * loader/FrameLoader.cpp:
-        (WebCore::ScheduledRedirection::ScheduledRedirection): 
-            Add parameter to differentiate refresh from other types of redirects. 
+        (WebCore::ScheduledRedirection::ScheduledRedirection):
+            Add parameter to differentiate refresh from other types of redirects.
             m_cachePolicy was used for signaling this before.
         (WebCore::isBackForwardLoadType):
         (WebCore::FrameLoader::FrameLoader):
         (WebCore::FrameLoader::redirectionTimerFired):
         (WebCore::FrameLoader::canCachePage):
         (WebCore::FrameLoader::loadURL):
-        (WebCore::FrameLoader::reload): 
+        (WebCore::FrameLoader::reload):
             Differentiate between revalidation and reload.
             No need to use setHTTPHeaderField here, addExtraFieldsToRequest will set the headers.
         (WebCore::FrameLoader::transitionToCommitted):
-        (WebCore::FrameLoader::cachePolicy): Determine the cache policy based on current load type. 
+        (WebCore::FrameLoader::cachePolicy): Determine the cache policy based on current load type.
         (WebCore::FrameLoader::checkLoadCompleteForThisFrame):
         (WebCore::FrameLoader::addExtraFieldsToRequest): Set Cache-control: no-cache for FrameLoadTypeReloadFromOrigin
         (WebCore::FrameLoader::shouldScrollToAnchor):
         * loader/FrameLoader.h:
         * loader/FrameLoaderTypes.h: Add FrameLoadTypeReloadFromOrigin
         (WebCore::):
-        * loader/NavigationAction.cpp: 
+        * loader/NavigationAction.cpp:
         (WebCore::navigationType):  Support FrameLoadTypeReloadFromOrigin
         * loader/loader.cpp:
         (WebCore::Loader::Host::servePendingRequests):
         * platform/win/DragImageCGWin.cpp:
         (WebCore::deallocContext): Add a generic CGContextRef destructor.
         * platform/win/DragImageCairoWin.cpp:
-        (WebCore::deallocContext): Add a generic cairo_* destructor. 
+        (WebCore::deallocContext): Add a generic cairo_* destructor.
         (WebCore::allocImage):  New implementation to allocate a Cairo
         surface of a specified size.
         (WebCore::createCairoContextFromBitmap): New implementation to
         WebKitCSSKeyframesRule to Window object.
         This required generating constructors for the event
         interfaces.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=20560
 
         * dom/WebKitAnimationEvent.idl:
 
         Delete the previous timer-queue timer in the main thread, just prior to scheduling a new timer.
         The code previously called DeleteTimerQueueTimer in the timer callback proc.
-        
+
         The new technique simplifies the code, since we now create and delete timers on the
         same thread, and don't access the timer queue or timer handles in the callback.
         This allows us to remove some mutex use, and more importantly, it solves a race
         condition that was occuring between ChangeTimerQueueTimer and DeleteTimerQueueTimer.
-        
+
         Since the timer callback isn't passed the timer handle, we were retrieving that handle
         via a global. If the timer callback code was entered, but then a new timer was immediately
         scheduled (prior to the callback acquiring the mutex and calling DeleteTimerQueueTimer),
         there was a small window where the timer could be re-scheduled via ChangeTimerQueueTimer
         and then immediately deleted once the already running callback acquired the mutex and
         then called DeleteTimerQueueTimer. This resulted in the newly scheduled timer never firing.
-        
+
         Reviewed by Oliver Hunt.
 
         * platform/win/SharedTimerWin.cpp:
         Reviewed by Simon Hausmann.
 
         Implement ImageSource::filenameExtension() for the Qt port
-        
+
         We're using QImageReader::imageFormat().toLower() to check
         that the image format is supported, and if it is we store
         the resulting extension when creating the ImageDecoderQt.
         Reviewed by Geoff Garen
 
         Account for the size of the response and request headers as well as other overhead
-        when calculating the size a resource takes up in the cache.  Halts unbounded 
+        when calculating the size a resource takes up in the cache.  Halts unbounded
         growth in the cache.  Reduced stress test memory high water marks by > 50%.
 
-        Uses estimates gathered from the stress test to set the overhead size.  
-        A version of the patch was created that calculated most of the sizes, but it was 
+        Uses estimates gathered from the stress test to set the overhead size.
+        A version of the patch was created that calculated most of the sizes, but it was
         decided that the patch was still at a basic level an estimate.  What gains it made
         in accuracy was offset by the complexity involved in creating and updating the
         estimate.
         I had to add one more bit of code. When animation timers used to fire the animation events.
         This would always happen from the RunLoop, so any style changes that happened in the
         event handler would get picked up on the next updateRendering() call. But now the start
-        event is generated during the styleIsAvailable() call, which is in the middle of the 
+        event is generated during the styleIsAvailable() call, which is in the middle of the
         updateRendering() cycle. And calling an event handler in the middle of updateRendering()
         is not allowed and causes style changes to get missed. We already have a mechanism in
         AnimationController to defer updateRendering() calls. So I added logic to defer all
         Reviewed by Antti Koivisto
 
         <rdar://problem/6431224>
-        
+
         When updating the value of a slider, don't mark the parents
         as needing layout, because the size of the slider can never
         change. This fixes full-page repaints in some cases.
 
         Potential build fix. The forward declaration of FloatPoint should
         be inside the WebCore namespace.
-        
+
         * platform/graphics/FloatPoint3D.h:
 
 2008-12-10  Simon Fraser  <simon.fraser@apple.com>
         Cleanup FloatPoint3D: inline the getters and setters,
         fix a potential divide-by-zero in normalize(), and add
         a FloatPoint constructor.
-        
+
         * platform/graphics/FloatPoint3D.cpp:
         (WebCore::FloatPoint3D::FloatPoint3D):
         (WebCore::FloatPoint3D::normalize):
 
         Part of
         https://bugs.webkit.org/show_bug.cgi?id=22570
-        
+
         Rename methods on RenderLayer for clarity:
             clearClipRects -> clearClipRectsIncludingDescendants
             clearClipRect  -> clearClipRects
 2008-12-10  Kevin Ollivier  <kevino@theolliviers.com>
 
         wx build fix after the script call stack/frame additions.
-        
+
         * WebCoreSources.bkl:
 
 2008-12-10   Srinivasa Rao M. Hamse  <msrinirao@gmail.com>
         * bindings/js/ScriptValue.h: Added isNull and isUndefined
         * bindings/scripts/CodeGeneratorJS.pm: Add handling for
         CustomArgumentHandling attribute.
-        * inspector/InspectorController.cpp: Refactored to use 
+        * inspector/InspectorController.cpp: Refactored to use
         ScriptCallFrame and ScriptCallStack.
         (WebCore::ConsoleMessage::ConsoleMessage):
         (WebCore::InspectorController::addMessageToConsole):
 
         Fix issues which break reading inline style for -webkit-transition
         and -webkit-transform-origin.
-        
+
         Test: fast/css/transform-inline-style.html
 
         * css/CSSMutableStyleDeclaration.cpp:
-        (WebCore::CSSMutableStyleDeclaration::getPropertyValue): Add cases 
+        (WebCore::CSSMutableStyleDeclaration::getPropertyValue): Add cases
         for CSSPropertyWebkitTransformOrigin and CSSPropertyWebkitTransition
         so that these shorthand properties are returned correctly.
         * css/CSSParser.cpp:
 
         https://bugs.webkit.org/show_bug.cgi?id=22717
         Make CSS values use less memory
-        
+
         Share CSSPrimitiveValue objects for commonly used values including
         - idents
         - colors
         - small integers
-        
+
         This reduces the amount CSSPrimitiveValue instances by > 80%.
 
         * css/CSSPrimitiveValue.cpp:
         Get CSSValues off from the common StyleBase base class. They don't
         need a parent pointer or anything else there and there is no real
         reason to have them in same data structures with other CSSOM objects.
-        
-        Disabled (instead of refactoring around the lack of common base) the ability 
-        to have style declaration blocks as CSS variable values. They don't exist in 
-        the spec so I wasn't sure if they have future or not. It would not be hard to 
-        get them back. CSS variables are in any case an experimental feature and 
+
+        Disabled (instead of refactoring around the lack of common base) the ability
+        to have style declaration blocks as CSS variable values. They don't exist in
+        the spec so I wasn't sure if they have future or not. It would not be hard to
+        get them back. CSS variables are in any case an experimental feature and
         not enabled by default.
 
         * css/CSSInitialValue.h:
 
         https://bugs.webkit.org/show_bug.cgi?id=22379
         Make CSSOM use less memory
-        
+
         Use vector instead of a double linked list for properties in CSSMutableStyleDeclaration.
-        
+
         Taught setter functions to use existing slots to avoid memory moves, plus some
         other optimizations.
 
         Fix logic related to repainting when transform changes:
         If an object has a layer, and the transform changes, then we need
         to do a repaintIncludingDescendants(), not just a repaint.
-        
+
         Test: fast/repaint/transform-repaint-descendants.html
 
         * rendering/RenderObject.cpp:
         Reviewed by Dave Hyatt
 
         https://bugs.webkit.org/show_bug.cgi?id=15671
-        
+
         Fix caret rendering to behave correctly with transforms:
         * Rename caretRect() methods to localCaretRect() and
           absoluteCaretBounds() as appropriate
 
         Bug 22579: Providing a function to ScrollbarClient.h which allows us to get at the tickmarks
         without relying on high-level WebCore types, as requested by Dave Hyatt.
-        
+
         No functional changes, thus no test cases.
 
         * page/FrameView.cpp:
         Reviewed by Kevin Ollivier.
 
         Add a MIME mapping for the .htm extension to wx and GTK ports.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=22668
-        
+
         * platform/gtk/MIMETypeRegistryGtk.cpp:
         (WebCore::):
         * platform/wx/MimeTypeRegistryWx.cpp:
 
         Implement basic text paste support in wx and add notImplemented stubs
         to catch other methods.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=22667
 
         * platform/wx/PasteboardWx.cpp:
         Reviewed by Kevin Ollivier.
 
         Turn off styled controls until we can implement them properly.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=22662
 
         * platform/wx/RenderThemeWx.cpp:
         makes it unnecessary to 'serialize' them into special PausedTimeouts
         instance, so pause/resumeTimeouts implementation in JSDOMWindowBase can also be removed.
         Also, moving TimeoutMap from JSDOMWindowBase to Document matches lifetime
-        of timeouts and makes it possible to not roundtrip them via PausedTimeouts 
+        of timeouts and makes it possible to not roundtrip them via PausedTimeouts
         every time when JSDOMWindow wrapper is destroyed while the page is in the b/f cache.
 
         Timeouts are now paused with other ActiveDOMObjects:
 
         * bindings/js/DOMTimer.h:
         * bindings/js/JSDOMBinding.cpp: ActiveDOMObject can have no JS wrapper
-        (WebCore::markActiveObjectsForContext): 
+        (WebCore::markActiveObjectsForContext):
 
         * bindings/js/JSDOMWindowBase.cpp:
         (WebCore::JSDOMWindowBase::~JSDOMWindowBase):
         * inspector/JavaScriptDebugServer.h:
 
         * loader/FrameLoader.cpp:
-        (WebCore::FrameLoader::commitProvisionalLoad): 
+        (WebCore::FrameLoader::commitProvisionalLoad):
         removed clearAllTimeouts since all ActiveDOMObjects will be stopped in FrameLoader::clear();
         I don't see how the old comment can be correct - the code in the same method proceeds to invoke 'onunload'
         and then calls into client which can be external code and can cause any active object created in onunload
         Reviewed by Beth Dakin.
 
         <rdar://problem/6018653> Extra blank line when pasting paragraph in plain text
-        
+
         In SnowLeopard, Mail occasionally adds an empty, unstyled paragraph at the
         end of pasted content so that users don't get stuck with non-standard pargraph
         spacing.  This content threw off our handling of interchange newlines.  Any interchange
         newline, regardless of it's position in the incoming fragment was considered to be
-        "at the start" of the fragment, and would result in us inserting in a newline before 
-        inserted content.  This patch makes the checks for interchange newlines more strict, 
+        "at the start" of the fragment, and would result in us inserting in a newline before
+        inserted content.  This patch makes the checks for interchange newlines more strict,
         and treats interchange newlines found elsewhere as normal <br>s.
 
         * editing/ReplaceSelectionCommand.cpp:
         Reduce size of the CSSSelector by 3/8 by moving rarely used fields to a rare data
         struct. Browsing around with some instrumentation showed that ~0.1% of all selectors
         encountered had rare data.
-        
+
         This also eliminates the CSSNthSelector subclass which will make possible to store
         CSSSelectors in an array instead of a linked list for futher memory savings.
 
         Reviewed by Dave Hyatt
 
         https://bugs.webkit.org/show_bug.cgi?id=22472
-        
+
         Override absoluteClippedOverflowRect() in RenderReplaced to return a rect
         that is large enough to encompass the selection, so that the repainting of
         selected replaced elements works correctly.
-        
+
         Test: fast/repaint/selected-replaced.html
 
         * rendering/RenderReplaced.cpp:
 
         Bug 22596: Some elements don't report AXBlockQuoteLevel
         https://bugs.webkit.org/show_bug.cgi?id=22596
+
         Reviewed by John Sullivan.
 
         * page/mac/AccessibilityObjectWrapper.mm:
         Reviewed by Eric Seidel.
 
         https://bugs.webkit.org/show_bug.cgi?id=22538
-        
+
         startsWith uses find which searches through the whole string if no match is found.
         Using reverseFind with an index of 0 has the benefit of only searching for the match
         at the beginning of the string.  This may only be a small benefit in the overall program,
         but it may help in some cases when the string is big.
 
         No observable change in behavior, so no test.
+
         * platform/text/StringImpl.h:
         (WebCore::StringImpl::startsWith):
 
         Move securityOrigin() from Document and WorkerContext into ScriptExecutionContext.
 
         No observable change in behavior, so no test.
-        
+
         * dom/Document.cpp:
         (WebCore::Document::open):
         (WebCore::Document::domain):
         (WebCore::WorkerContext::WorkerContext):
         * dom/WorkerContext.h:
         These changes are for the move of securityOrigin().
-        
+
         * xml/XMLHttpRequest.cpp:
         (WebCore::XMLHttpRequest::createRequest):
         (WebCore::XMLHttpRequest::makeSimpleCrossSiteAccessRequest):
 
         Reviewed by Dan Bernstein.
 
-        Fix for https://bugs.webkit.org/show_bug.cgi?id=13736 REGRESSION 
-        (r19811): Using the down arrow in a textarea gets "stuck" at the 
+        Fix for https://bugs.webkit.org/show_bug.cgi?id=13736 REGRESSION
+        (r19811): Using the down arrow in a textarea gets "stuck" at the
         end of a wrapped line
         And corresponding: <rdar://problem/5347931>
 
-        The basic problem here is that Position::getInlineBoxAndOffset() 
-        failed to look beyond a single renderer. This patch looks for a 
-        better match beyond the first renderer when the affinity is 
-        downstream and we failed to find a "perfect" match. 
+        The basic problem here is that Position::getInlineBoxAndOffset()
+        failed to look beyond a single renderer. This patch looks for a
+        better match beyond the first renderer when the affinity is
+        downstream and we failed to find a "perfect" match.
         * dom/Position.cpp:
         (WebCore::isNonTextLeafChild):
         (WebCore::searchAheadForBetterMatch):
         (WebCore::Position::getInlineBoxAndOffset):
 
-        This is a fix I made based on code inspection. It looks like the 
-        old code here and skipped over the parent as a possible match. 
+        This is a fix I made based on code inspection. It looks like the
+        old code here and skipped over the parent as a possible match.
         * rendering/RenderObject.cpp:
         (WebCore::RenderObject::nextInPreOrderAfterChildren):
 
 
         * WebCore.vcproj/WebCore.vcproj:
           1. Add files from the wml directory to the set of windows files.
-          2. Extend include paths with new wml directory. 
+          2. Extend include paths with new wml directory.
           3. Add new autogenerated WML files to DerivedSources.
           4. Alphabetize preprocesor includes (holdover from earlier debugging).
 
 
         Revise node/selection image fix.
         Moved updateLayout call so selection rect is fetched after the layout.
-        
+
         Reviewed by Adam Roben.
 
         * page/win/FrameCGWin.cpp:
         Reviewed by Dan Bernstein
 
         https://bugs.webkit.org/show_bug.cgi?id=22581
-        
+
         Fix the painting of the caps lock indicator for transformed text inputs,
         by replacing a call to absoluteContentBox() with code that computes the
         painting rect for the input contents.
         them. So now the pointer is valid throughout the entire sequence of callback code.
 
         The testcase for https://bugs.webkit.org/show_bug.cgi?id=22052 also exhibits a
-        crash which this patch fixes. 
+        crash which this patch fixes.
 
         * page/animation/AnimationBase.cpp:
         (WebCore::AnimationBase::updateStateMachine):
 
         Fixes: https://bugs.webkit.org/show_bug.cgi?id=22549
 
-        Add <do> element support. It provides a way to bind a task element to a <template>/<card> element. 
+        Add <do> element support. It provides a way to bind a task element to a <template>/<card> element.
         Changes numerous of layout tests which contain <do> elements, as they render as buttons now.
 
         * WebCore.xcodeproj/project.pbxproj:
         upper-case glyph defined in the set of available fonts.
 
         This changes Font.cpp to check if the font being used exists before trying to apply the small-caps variant.
-        
+
         Test: fast/css/small-caps-crash.html
 
         * platform/graphics/Font.cpp:
 2008-11-25  Kevin Ollivier  <kevino@theolliviers.com>
 
         wx build fix - we don't support PurgeableBuffer on Leopard for now.
-        
+
         * platform/PurgeableBuffer.h:
 
 2008-11-25  Antti Koivisto  <antti@apple.com>
 
         Fix https://bugs.webkit.org/show_bug.cgi?id=22483
         Assertion failure in CachedResource::makePurgeable during layout tests
-        
+
         Deleting SVG image can re-enter destroyDecodedData.
 
         * loader/CachedImage.cpp:
                http/tests/misc/url-in-utf32be.html
                http/tests/misc/url-in-utf32le.html
                http/tests/misc/url-in-utf7.html
+
         * html/HTMLFormElement.cpp:
         (WebCore::HTMLFormElement::formData):
         * loader/TextResourceDecoder.cpp:
 2008-11-24  Antti Koivisto  <antti@apple.com>
 
         Reviewed by Darin Adler.
-        
+
         Fix for https://bugs.webkit.org/show_bug.cgi?id=22073
         REGRESSION(r33544): Palace in the Sky crashes WebKit
-        
+
         HTMLTokenizer::m_hasScriptsWaitingForStylesheets would still be set while
-        there were no scripts left to execute. 
-        
+        there were no scripts left to execute.
+
         If m_hasScriptsWaitingForStylesheets becomes true during script execution
         bail out from executing more scripts synchronously.
 
         https://bugs.webkit.org/show_bug.cgi?id=22214
         Keep dead resources in memory cache in purgeable memory.
         <rdar://problem/6333088>
-        
+
         OS X 10.5 has purgeable memory kernel facility that allows marking reserved memory
         areas as less important. Under memory pressure system can steal pages that have
         been marked purgeable for (hopefully) better uses. This is ideal for caches.
-        
+
         Only resources larger than 16KB will be moved to the purgeable memory.
 
         * WebCore.base.exp:
 
         Bug 22461: AccessibilityListBox::doAccessibilityHitTest() signature does not match the base class
         https://bugs.webkit.org/show_bug.cgi?id=22461
+
         Changing AccessibilityListBox to use its parent's signature for hit testing
 
         * page/AccessibilityListBox.cpp:
         http://bugs.webkit.org/show_bug.cgi?id=18703
         "Changing the 'size' property on a text input does not affect its length"
 
-        Text fields would not repaint themselves after having their "size" 
-        attributes modified.  This fix tells the object to recalculate its 
+        Text fields would not repaint themselves after having their "size"
+        attributes modified.  This fix tells the object to recalculate its
         width and repaint itself when its "size" attribute is parsed.
 
         Test: fast/js/text-field-resize.html
         Reviewed by Darin Adler
 
         Followup from changes for https://bugs.webkit.org/show_bug.cgi?id=22433
-        
+
         Make RenderView::selectionRect() private to cause compile-time
         errors if someone tries to call it.
 
         Reviewed by Dan Bernstein
 
         Via: https://bugs.webkit.org/show_bug.cgi?id=22433
-        
+
         Rename RenderView::selectionRect() to selectionBounds(), to remove
         longstanding ambiguity with the base class selectionRect() method.
         Do the same on Frame for consistency with RenderView. Assert
 
         Improve wx image drawing performance considerably when using wxGraphicsContext
         by avoiding unnecessary copies and drawing.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=22404
 
         * platform/graphics/wx/ImageWx.cpp:
         Reviewed by Kevin Ollivier.
 
         Implementation of AffineTransform::mapRect for wx.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=22401
 
         * platform/graphics/wx/AffineTransformWx.cpp:
         Reviewed by Darin Adler.
 
         <rdar://problem/5381788> Match NSTextView editing behavior at the end of hyperlink text
-        
-        Change link editing behavior to match TextEdit and MS Word when editing before and after 
-        a link (Pages has two caret positions at link boundaries, Thunderbird and FF behave like we 
+
+        Change link editing behavior to match TextEdit and MS Word when editing before and after
+        a link (Pages has two caret positions at link boundaries, Thunderbird and FF behave like we
         used to, so it's difficult to get out of link editing mode):
         When inserting before or after a link, always insert content outside of the link.  This
-        makes it impossible to get stuck in link editing mode, while making it slightly more 
-        difficult to edit link labels.  WebKit editors that care about this can add UI for editing 
+        makes it impossible to get stuck in link editing mode, while making it slightly more
+        difficult to edit link labels.  WebKit editors that care about this can add UI for editing
         link labels, like GMail and GoogleDocs have done.  We never actually had any bugs complaining
-        about how it was difficult to edit link labels at the start/end, the code was just introduced 
+        about how it was difficult to edit link labels at the start/end, the code was just introduced
         with another bug fix without much thought.
-        
-        Don't remember removed links anymore, no other editor does this and it made it 
+
+        Don't remember removed links anymore, no other editor does this and it made it
         difficult/impossible to get out of link editing mode.  This code was added to fix
-        <rdar://problem/4069359>, which is fixed instead by removing the styles from an 
+        <rdar://problem/4069359>, which is fixed instead by removing the styles from an
         enclosing anchor element from those styles that we remember when we delete content.
 
         * editing/CompositeEditCommand.cpp:
         RenderBox::absoluteClippedOverflowRect() needs to inflate the rect by
         maximalOutlineSize(), since a child might have an outline which projects
         outside the parent overflowRect().
-        
+
         We also need to ensure that maximalOutlineSize() is updated early in styleDidChange,
         so that it is valid for these repaints.
 
         Reviewed by Darin Adler.
 
         Bug 22388: Add JSInterfaceName to the .in files
-        https://bugs.webkit.org/show_bug.cgi?id=22388        
+        https://bugs.webkit.org/show_bug.cgi?id=22388
 
         Add JSInterfaceName that is similar to interfaceName but
         for the JS wrappers. JSInterfaceName is always equal to interfaceName
         unless explicitly set.
-        
+
         * dom/make_names.pl:
         * html/HTMLTagNames.in:
         * svg/svgtags.in:
         Reviewed by Dan Bernstein
 
         https://bugs.webkit.org/show_bug.cgi?id=19623
-        
+
         When an non-layer object gained a transform, it would only repaint
         the bounds of the new layer, which could result in redraw artifacts
         if the new layer was smaller. So if we're gaining a transform, we
         repaint.
-        
+
         Test: fast/repaint/change-transform.html
 
         * rendering/RenderObject.cpp:
 
 2008-11-20  Adele Peterson  <adele@apple.com>
 
-        Reviewed by Darin Adler. 
+        Reviewed by Darin Adler.
 
         Fix for <rdar://problem/6111436> Support upload progress events on Windows
 
         Reviewed by Darin Adler.
 
         <rdar://problem/2610675> Blank line that is quoted can't be deleted
-        
+
         If the caret is in an empty quoted paragraph, and either there is nothing before that
         paragraph, or what is before is unquoted, and the user presses delete, unquote that
         paragraph.
         computed values for transform-origin-x, transform-origin-y, and just
         return transform-origin instead. Return "none" for default animation-name,
         and fix the initial value.
-        
+
         Tests: animations/computed-style.html
                transforms/computed-style-origin.html
 
 2008-11-20  Justin Garcia  <justin.garcia@apple.com>
 
         Reviewed by Beth Dakin.
-        
+
         Preparation for:
         <rdar://problem/2610675> Blank line that is quoted can't be deleted
-        
+
         To fix this we need to make changes to the code that creates selections to delete
         when the user does a backward or forward delete with a caret selection.  For certain
         caret positions, we now want to remove something other than the standard "caret extended
-        backward/forward by one unit".  The problem is that there were two pieces of code 
-        responsible for doing this, one in Editor::deleteWithDirection, and another inside 
-        TypingCommand::deleteKeyPressed.  The code in deleteWithDirection is a recent 
-        addition (r19172), and adding it there caused regressions because it prevented the 
-        code in deleteKeyPressed from ever running.  The regressions were never caught because 
+        backward/forward by one unit".  The problem is that there were two pieces of code
+        responsible for doing this, one in Editor::deleteWithDirection, and another inside
+        TypingCommand::deleteKeyPressed.  The code in deleteWithDirection is a recent
+        addition (r19172), and adding it there caused regressions because it prevented the
+        code in deleteKeyPressed from ever running.  The regressions were never caught because
         JS deletion uses deleteKeyPressed while manual deletion uses deleteWithDirection.
-        
+
         This patch removes selection creation code from deleteWithDirection so that deleteKeyPressed
         can handle it.  That required moving code to handle the kill ring down into deleteKeyPressed.
-        
-        Follow up patches will fix <rdar://problem/2610675>, and attempt to eliminate the rest of the 
-        discrepancies between the behavior of JS deletions and manual deletions, so that we 
+
+        Follow up patches will fix <rdar://problem/2610675>, and attempt to eliminate the rest of the
+        discrepancies between the behavior of JS deletions and manual deletions, so that we
         have better test coverage.
 
         * editing/Editor.cpp:
         it into the coords of the part's renderer (which is the one that is painting).
         To do this we need to compute an offset relative to some container, so expose
         a method on RenderObject for that.
-        
+
         Also fix the location at which the search popup shows up to take transforms
         into account, and fix the math that is used to figure out if the search
         results button, or the cancel button should get the mouse events.
-        
+
         Test: fast/forms/search-transformed.html
 
         * rendering/RenderBox.h:
 
         https://bugs.webkit.org/show_bug.cgi?id=22379
         Make CSSOM use less memory
-        
+
         - Shrink CSSProperty by half by using bitfields.
         - Get rid of m_strictParsing field in StyleBase by moving it up to CSSStyleSheet
           and CSSMutableStyleDeclaration. This reduces size of many highly popular objects.
 
         https://bugs.webkit.org/show_bug.cgi?id=22373
         Ports busted by addition of ScriptValue.{h,cpp}
-        
+
         * plugins/PluginView.cpp:
         (WebCore::PluginView::performRequest):
 
 
         Reviewed by Justin Garcia.
 
-        Fix for <rdar://problem/5472507> Remove color property when a user 
+        Fix for <rdar://problem/5472507> Remove color property when a user
         sets color to black
 
-        This patch prevents us from inserting font nodes during the 
-        ApplyStyleCommand if they will not change the computed style of an 
+        This patch prevents us from inserting font nodes during the
+        ApplyStyleCommand if they will not change the computed style of an
         element.
 
         * editing/ApplyStyleCommand.cpp:
 2008-11-19  Simon Fraser  <simon.fraser@apple.com>
 
         Reviewed by Antti Koivisto
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=22111
-        
+
         Fix hit testing in controls on transformed video elements
         by replacing absoluteBoundingBoxRect().contains() with
         code that maps the point into local coords, taking
         Reviewed by Darin Adler
 
         https://bugs.webkit.org/show_bug.cgi?id=22348
-        
+
         Need to educate style sharing about autofill, so that style does not
         get shared between input elements that are autofilled, and those
         that are not. Setting autofill should also do a setChanged on the node.
 2008-11-17  Geoffrey Garen  <ggaren@apple.com>
 
         Reviewed by Sam Weinig.
-        
+
         Updated for JavaScriptCore renames.
 
         * WebCore.pro:
         Reviewed by Beth Dakin.
 
         <rdar://problem/4922709> Copying less than a paragraph of quoted text and pasting it doesn't retain quote level
-        
-        There was code to intentionally avoid quoting pasted content if less than a paragraph of 
+
+        There was code to intentionally avoid quoting pasted content if less than a paragraph of
         it was copied.  That was added for <rdar://problem/5006779>, but was unnecessary because
         that bug was about Paste and Match style for single paragraphs.  And quote stripping for
         Paste and Match style is handled elsewhere.
 2008-11-17  Geoffrey Garen  <ggaren@apple.com>
 
         Reviewed by Sam Weinig.
-        
+
         Updated for JavaScriptCore rename.
 
         * ForwardingHeaders/interpreter/CallFrame.h: Copied from WebCore/ForwardingHeaders/runtime/ExecState.h.
 2008-11-17  Geoffrey Garen  <ggaren@apple.com>
 
         Reviewed by Sam Weinig.
-        
+
         Updated for JavaScriptCore renames.
 
         * ForwardingHeaders/VM: Removed.
         Remove use of static C++ objects that are destroyed at exit time (destructors)
 
         Find some missing DEFINE_STATIC_LOCAL use cases.
-        
+
         * bindings/js/JSSVGPODTypeWrapper.h:
         (WebCore::PODTypeWrapperCacheInfoTraits::emptyValue):
         (WebCore::JSSVGDynamicPODTypeWrapperCache::dynamicWrapperHashMap):
         Reviewed by Dan Bernstein
 
         https://bugs.webkit.org/show_bug.cgi?id=22118
-        
-        Fix resize corner tracking in transformed elements by using 
+
+        Fix resize corner tracking in transformed elements by using
         mapping the point into local coords using absoluteToLocal,
         rather than convertToLayerCoords.
-        
+
         Test: fast/css/resize-corner-tracking-transformed.html
 
         * page/EventHandler.h:
         Reviewed by Timothy Hatcher.
 
         Initialize m_networkStateChangedFunction to 0 as otherwise the check for null will fail.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=22284
 
         * platform/network/NetworkStateNotifier.h:
 2008-11-15  Geoffrey Garen  <ggaren@apple.com>
 
         Reviewed by Sam Weinig.
-        
+
         Updated for JavaScriptCore renames.
 
         * bindings/js/JSNodeCustom.cpp:
         https://bugs.webkit.org/show_bug.cgi?id=21810
         Remove use of static C++ objects that are destroyed at exit time (destructors)
 
-        Create DEFINE_STATIC_LOCAL macro. Change static local objects to leak to avoid 
-        exit-time destructor. Update code that was changed to fix this issue that ran 
-        into a gcc bug (<rdar://problem/6354696> Codegen issue with C++ static reference 
-        in gcc build 5465). Also typdefs for template types needed to be added in some 
+        Create DEFINE_STATIC_LOCAL macro. Change static local objects to leak to avoid
+        exit-time destructor. Update code that was changed to fix this issue that ran
+        into a gcc bug (<rdar://problem/6354696> Codegen issue with C++ static reference
+        in gcc build 5465). Also typdefs for template types needed to be added in some
         cases so the type could make it through the macro successfully.
 
         Basically code of the form:
 
         https://bugs.webkit.org/show_bug.cgi?id=22264
         Need to update to latest Geolocation spec (13 November 2008)
-        
+
         Update to new error code constants and values.
 
         * page/Geolocation.cpp: Use new named ErrorCode enums
 
         Reviewed by Justin Garcia.
 
-        Fix (again) for <rdar://problem/5089327> Too much indentation when 
+        Fix (again) for <rdar://problem/5089327> Too much indentation when
         pasting quoted paragraphs
 
-        This patch goes back to Justin's original approach to fix this bug, 
-        written in revision 38273. That fix was mostly rolled out by 
-        revision 38310 because of styling issues created by the addition of 
-        extra blockquote nodes. This patch again goes back to Justin's 
-        original fix because of cases we discovered the newer fix could not 
-        possibly cover. This version of the patch minimizes the styling 
-        issues of extra blockquotes by opting into the merge start code. 
+        This patch goes back to Justin's original approach to fix this bug,
+        written in revision 38273. That fix was mostly rolled out by
+        revision 38310 because of styling issues created by the addition of
+        extra blockquote nodes. This patch again goes back to Justin's
+        original fix because of cases we discovered the newer fix could not
+        possibly cover. This version of the patch minimizes the styling
+        issues of extra blockquotes by opting into the merge start code.
 
         * editing/ReplaceSelectionCommand.cpp:
-        (WebCore::hasMatchingQuoteLevel): We want shouldMergeStart to 
-        return true when the quoting level of the end of the inserted 
-        content matches the quoting level of the end of the existing 
+        (WebCore::hasMatchingQuoteLevel): We want shouldMergeStart to
+        return true when the quoting level of the end of the inserted
+        content matches the quoting level of the end of the existing
         content.
-        (WebCore::ReplaceSelectionCommand::shouldMergeStart): Now calls 
+        (WebCore::ReplaceSelectionCommand::shouldMergeStart): Now calls
         hasMatchingQuoteLevel
         (WebCore::ReplaceSelectionCommand::doApply):
 
         Reviewed by Beth Dakin.
 
         <rdar://problem/4230923> "Make Plain Text" doesn't reset text alignment in single paragraph messages
-        
+
         When applying block styles, we would add block properties to the body element, and Mail's
         Make Plain Text feature isn't equipped to remove those.  This could have been fixed on our side,
         but this change has the advantage that it fixes the bug on Tiger, where Mail does not plan future updates.
-        
+
         We have code that puts the paragraphs that we're operating on into blocks of their own before
-        adding or removing block properties from the blocks that enclose them.  We need to run this code 
+        adding or removing block properties from the blocks that enclose them.  We need to run this code
         when the enclosing block is the body element.
-        
+
         * editing/CompositeEditCommand.cpp:
         (WebCore::CompositeEditCommand::insertNewDefaultParagraphElementAt): Added, moved code from moveParagraphContents
         to here.
 
         * platform/text/AtomicString.cpp:
         (WebCore::AtomicString::add):
-        
+
 2008-11-13  Justin Garcia  <justin.garcia@apple.com>
 
         Reviewed by Darin Adler.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=18620
         Long hang under TextIterator::advance() when loading http://www.lsvd.de/
 
         we create VisiblePositions unnecessarily.
 
         * editing/TextIterator.cpp:
-        (WebCore::TextIterator::shouldRepresentNodeOffsetZero): Don't proceed to VisiblePosition 
-        creation if m_node is unrendered or invisible.  The answers wouldn't have much meaning 
+        (WebCore::TextIterator::shouldRepresentNodeOffsetZero): Don't proceed to VisiblePosition
+        creation if m_node is unrendered or invisible.  The answers wouldn't have much meaning
         and would be wasteful.  Also fixed some comments to reflect the fact that this function
         isn't specifically about emitting a newline.
 
         it may be the bounding outline box of a transformed element, and fix it
         to respect transforms by mapping the outline box to an absolute quad and
         taking the enclosing rect.
-        
+
         RenderBox::localToAbsoluteQuad() can no longer assert that there is no
         LayoutState, but LayoutState cannot be used during quad mapping (it knows
         nothing about transforms).
-        
+
         Finally, fix a bug in RenderBox::localToAbsoluteQuad() which was getting
         borderTopExtra() from the object, rather than its container (as localToAbsolute()
         does).
-        
+
         Test: fast/repaint/transform-absolute-child.html
 
         * rendering/RenderBlock.cpp:
 
         https://bugs.webkit.org/show_bug.cgi?id=21942
         https://bugs.webkit.org/show_bug.cgi?id=18557
-        
+
         Add methods which can be used to map renderer-local rectangles
         to quads in absolute coordinates, taking transforms into account:
             localToAbsoluteQuad() converts a local rect into an absolute quad.
             collectAbsoluteLineBoxQuads() is an analogue of addLineBoxRects()
                 that works with quads.
             absoluteQuads() is an analogue of absoluteRects(), for quads.
-        
+
         Use the quad methods to fix the inspector highlight for transformed
         elements.
 
 
         https://bugs.webkit.org/show_bug.cgi?id=17840
         Patch 2
-        
+
         Code cleanup in RenderBox::computeAbsoluteRepaintRect().
 
         * rendering/RenderBox.cpp:
         (WebCore::RenderBox::computeAbsoluteRepaintRect):
 
 2008-11-12  Justin Garcia  <justin.garcia@apple.com>
+
         Reviewed by Beth Dakin.
+
         <rdar://problem/5495723> Selecting and deleting quoted text quotes other text
         <rdar://problem/4775313> Deleting lines from the bottom of a messages leaves the last blank line quoted
-        
-        We don't want to merge into a block if it will mean changing the quote level of content after deleting 
-        selections that contain a whole number paragraphs plus a line break, since it is unclear to most users 
+
+        We don't want to merge into a block if it will mean changing the quote level of content after deleting
+        selections that contain a whole number paragraphs plus a line break, since it is unclear to most users
         that such a selection actually ends at the start of the next paragraph.  Instead we want to completely
-        remove the selected paragraph(s) and all evidence of the first one's quote level.  This matches TextEdit behavior 
+        remove the selected paragraph(s) and all evidence of the first one's quote level.  This matches TextEdit behavior
         for indented paragraphs.
+
         * editing/DeleteSelectionCommand.cpp:
         (WebCore::DeleteSelectionCommand::initializePositionData): For the selections described above, do not
         try to merge after the deletion.  Instead try and prune the start block or blocks if they've been emptied
         so that we remove evidence of the deleted paragraphs' quote level.
-        (WebCore::DeleteSelectionCommand::mergeParagraphs): Try and prune the start block(s) if necessary.  Also 
-        make sure that the caret is placed correctly so that it ends up on the same line that the deleted selection 
+        (WebCore::DeleteSelectionCommand::mergeParagraphs): Try and prune the start block(s) if necessary.  Also
+        make sure that the caret is placed correctly so that it ends up on the same line that the deleted selection
         started on, instead of one higher.
         * editing/DeleteSelectionCommand.h:
         * editing/htmlediting.cpp:
 
         Reviewed by Darin Adler.
 
-        Fix for <rdar://problem/5089327> Too much indentation when pasting 
+        Fix for <rdar://problem/5089327> Too much indentation when pasting
         quoted paragraphs
 
-        This patch re-addresses pasting blockquotes into blockquotes. It 
-        backs out most of revision 38273. 38273 fixed the bug by inserting 
-        the pasted content as a sibling blockquote node to the pre-existing 
-        blockquote node. The problem with that is that by default, 
-        blockquotes have a giant margin, so visually, this can be weird. 
-        This patch instead inserts the pasted content as siblings of the 
-        the content already inside the outer blockquote, and then removes 
-        the blockquote node from the pasted content itself, so that it 
+        This patch re-addresses pasting blockquotes into blockquotes. It
+        backs out most of revision 38273. 38273 fixed the bug by inserting
+        the pasted content as a sibling blockquote node to the pre-existing
+        blockquote node. The problem with that is that by default,
+        blockquotes have a giant margin, so visually, this can be weird.
+        This patch instead inserts the pasted content as siblings of the
+        the content already inside the outer blockquote, and then removes
+        the blockquote node from the pasted content itself, so that it
         doesn't nest itself into the outer blockquote.
         * editing/ReplaceSelectionCommand.cpp:
         (WebCore::ReplaceSelectionCommand::doApply):
 
-        Move contains() from Element to Node since there is nothing 
+        Move contains() from Element to Node since there is nothing
         Element-specific about it.
         * dom/Element.cpp:
         * dom/Element.h:
 
         PatternCairo needs to invert the pattern matrix because
         of transformation from user space to pattern space.
-        Give a identity matrix to pattern for Qt and Cairo. Because the 
+        Give a identity matrix to pattern for Qt and Cairo. Because the
         context is transformed already.
 
         [CAIRO][QT] Canvas transformations applied twice to Patterns
 
         Reviewed by Beth Dakin.
 
-        <rdar://problem/4037481> REGRESSION (Mail): pasting quoted text 
+        <rdar://problem/4037481> REGRESSION (Mail): pasting quoted text
         into quoted text yields double-quoting
-        
-        Don't nest inserted content in Mail blockquotes.  Perform a 
+
+        Don't nest inserted content in Mail blockquotes.  Perform a
         BreakBlockquoteCommand if we're in
         one and insert the incoming fragment between the split blockquotes.
 
         * editing/ReplaceSelectionCommand.cpp:
-        (WebCore::ReplaceSelectionCommand::shouldMerge): Renamed from -> 
+        (WebCore::ReplaceSelectionCommand::shouldMerge): Renamed from ->
         source and to -> destination.
         (WebCore::ReplaceSelectionCommand::doApply):
 
 2008-11-08  Antti Koivisto  <antti@apple.com>
 
         Reviewed by Sam Weinig.
-        
+
         Fix https://bugs.webkit.org/show_bug.cgi?id=22141
         REGRESSION: Safari error page is not fully styled when loaded from cache
-        
+
         Reset text decoder on flush so it does not pass through the BOM when it is reused.
 
         Test: fast/encoding/css-cached-bom.html
         (WebCore::TextResourceDecoder::flush):
 
 2008-11-08  Kevin Ollivier  <kevino@theolliviers.com>
+
         Reviewed by Mark Rowe.
 
         Send URL errors to stderr rather than stdout. While debugging wx DumpRenderTree,
 
 2008-11-08  Kevin Ollivier  <kevino@theolliviers.com>
 
-        wx build fixes after addition of JSCore parser and bycompiler dirs. 
+        wx build fixes after addition of JSCore parser and bycompiler dirs.
 
         * webcore-base.bkl:
         * webcore-wx.bkl:
         Reviewed by Dan Bernstein
 
         https://bugs.webkit.org/show_bug.cgi?id=21906
-        
+
         Override addFocusRingRects() in RenderTextControl to avoid
         the RenderFlow behavior of recursing on descendent renderers.
         RenderTextControl should only ever need a simple focus rect.
-        
+
         This fixes focus ring issues with transforms on text controls.
-        
+
         Test: fast/transforms/transformed-focused-text-input.html
 
         * rendering/RenderTextControl.cpp:
         Reviewed by Dan Bernstein
 
         https://bugs.webkit.org/show_bug.cgi?id=22122
-        
+
         Use a stack-based object to simplify the pushLayoutState/popLayoutState
         code. LayoutStateMaintainer either pushes in the constructor, or allows
         an explicit push() later. Both cases require an explicit pop().
         Reviewed by Dan Bernstein.
 
         https://bugs.webkit.org/show_bug.cgi?id=22093
-        
+
         Delaying the text decoding caused regression since the decoding
-        also determines the encoding in case of @charset rule. 
-        
+        also determines the encoding in case of @charset rule.
+
         Decode immediately in data() and keep the decoded string around
         during the checkNotify().
 
         NPN_HasPropertyUPP and NPN_HasMethodUPP entries in NPNetscapeFuncs are NULL
 
         Export _NPN_HasMethod and _NPN_HasProperty.
-        
+
         * WebCore.NPAPI.exp:
 
 2008-11-06  Simon Fraser  <simon.fraser@apple.com>
         Reviewed by Antti Koivisto
 
         https://bugs.webkit.org/show_bug.cgi?id=15678
-        
+
         Fix transformed menu selects to show the popup in the correct
         location.
 
 2008-11-06  Steve Falkenburg  <sfalken@apple.com>
 
         Fix failed assert at launch caused by unintialized data member.
-        
+
         Reviewed by Maciej Stachowiak.
 
         * platform/network/win/NetworkStateNotifierWin.cpp:
         Reviewed by Dave Hyatt
 
         https://bugs.webkit.org/show_bug.cgi?id=21870
-        
+
         Implement absoluteToLocal() to convert a point from absolute
         to local coordinates, optionally taking transforms into account.
-        
+
         Use this to set offsetX/offsetY in mouse events, thus fixing
         offsetX/offsetY in events on elements with transforms.
 
 2008-11-05  Antti Koivisto  <antti@apple.com>
 
         Reviewed by Dan Bernstein.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=22093
         Don't keep decoded stylesheet data in cache
-        
+
         <rdar://problem/6343588>
 
         Don't keep decoded stylesheet string around in the cache. There are no sharing benefits and
         performance benefits are negligible (no measured PLT impact). Reduces memory consumption of
-        style sheet data in cache by 2/3 in common case.  
+        style sheet data in cache by 2/3 in common case.
 
         * loader/CachedCSSStyleSheet.cpp:
         (WebCore::CachedCSSStyleSheet::sheetText):
         Reviewed by Beth Dakin.
 
         <rdar://problem/5480736> In Mail and Gmail, copied indented text pastes with line break
-        
-        As a rule, we don't allow merges out of blockquotes.  In the bug, we are inserting a text node 
+
+        As a rule, we don't allow merges out of blockquotes.  In the bug, we are inserting a text node
         between two blockquotes.  Because the start merge moves the text node into a blockquote, when we
-        determine whether or not we should do the end merge, it incorrectly appears as though the end merge 
+        determine whether or not we should do the end merge, it incorrectly appears as though the end merge
         is merging out of a blockquote.  The fix is to determine whether or not we should do the end merge
         before we do the start merge, so that the start merge doesn't effect our decision.
 
         Reviewed by Dan Bernstein.
 
         Remove two global destructors from CoreTextController.
-        
+
         * platform/graphics/mac/CoreTextController.cpp:
         (WebCore::CoreTextController::collectCoreTextRunsForCharacters):
 
         reported by zdobersek in #webkit.
 
 2008-11-04  Darin Fisher  <darin@chromium.org>
+
         Reviewed by Anders Carlsson.
+
         No need to clobber all ResourceRequest fields in FrameLoader::reload()
         https://bugs.webkit.org/show_bug.cgi?id=21949
+
         * loader/FrameLoader.cpp:
         (WebCore::FrameLoader::reload):
 
 
         When computing the bounds of the transparency layer, we need to
         map the clipRect through the enclosing transform.
-        
+
         Test: fast/layers/opacity-transforms.html
 
         * rendering/RenderLayer.cpp:
 
         https://bugs.webkit.org/show_bug.cgi?id=22053
 
-        Added additional support needed for the NPDrawingModelCoreAnimation drawing model. 
+        Added additional support needed for the NPDrawingModelCoreAnimation drawing model.
 
         * bridge/npapi.h:
 
         WebCore Windows part of fix for <rdar://problem/5839256> FILE CONTROL: multi-file upload.
         https://bugs.webkit.org/show_bug.cgi?id=22008
 
-        * platform/FileChooser.cpp: (WebCore::FileChooser::chooseIcon): 
+        * platform/FileChooser.cpp: (WebCore::FileChooser::chooseIcon):
         Rename newIconForFile and newIconForFiles to createIconForFile and createIconForFiles.
         * platform/graphics/Icon.h: ditto.
         * platform/graphics/gtk/IconGtk.cpp:
         (WebCore::Icon::createIconForFile): ditto.
         (WebCore::Icon::createIconForFiles): Add creation of an icon for multiple files.
 
-        * rendering/RenderThemeWin.cpp: (WebCore::RenderThemeWin::paintSearchFieldResultsDecoration):        
+        * rendering/RenderThemeWin.cpp: (WebCore::RenderThemeWin::paintSearchFieldResultsDecoration):
         Improve icon creation code to match new code in Icon::createIconForFiles
-        
+
 2008-10-31  Timothy Hatcher  <timothy@apple.com>
 
         Add manual tests that check breakpoints on a blockless body of "for" loops.
         Reviewed by Sam Weinig
 
         https://bugs.webkit.org/show_bug.cgi?id=21967
-        
+
         For some platforms the GeolocationService must be suspended and resumed.
 
         * page/Geolocation.cpp:
         Reviewed by Sam Weinig
 
         https://bugs.webkit.org/show_bug.cgi?id=21966
-        
+
         The Geolocation spec was updated on 10/27/2008. This brings WebCore up to date.
         http://dev.w3.org/geo/api/spec-source.html
 
         <rdar://problem/6104369> Hitting return at the end of a quoted line creates an extraneous quoted line
 
         * editing/BreakBlockquoteCommand.cpp:
-        (WebCore::BreakBlockquoteCommand::doApply): 
-        Don't store the endingSelection() in selection, just call endingSelection() in the few places it's 
+        (WebCore::BreakBlockquoteCommand::doApply):
+        Don't store the endingSelection() in selection, just call endingSelection() in the few places it's
         needed.  This function is cheap since it just returns a reference to a Selection instead of creating one.
         Don't store an affinity.  In the one place that it was used, isLastVisiblePositionInNode(VisiblePosition(pos, affinity), topBlockquote),
         we now use visiblePos (in order to avoid VisiblePosition creation).
-        Set pos after we delete the current selection (if there is one), and be consistent about what we set 
+        Set pos after we delete the current selection (if there is one), and be consistent about what we set
         pos to.  Before, we upstream()ed it if there was a selection to delete and left it alone otherwise.  In fact...
-        ...we need to use downstream() for pos so that when a caret is at the boundary between two nodes, pos is 
-        in the first node that we want to move.  This fixes the bug, since it lets code that checks for the case 
+        ...we need to use downstream() for pos so that when a caret is at the boundary between two nodes, pos is
+        in the first node that we want to move.  This fixes the bug, since it lets code that checks for the case
         where the caret is between text and a br work correctly.
 
 2008-10-30  Yael Aharon  <yael.aharon@nokia.com>
         <rdar://problem/6104369> Hitting return at the end of a quoted line creates an extraneous quoted line
 
         * editing/BreakBlockquoteCommand.cpp:
-        (WebCore::BreakBlockquoteCommand::doApply): Added comments.  Don't need to use newStartNode.  If the 
-        startNode needs to change, change it.  Afterwords, check to make sure that it hasn't left topBlockquote.  
-        This is slightly stricter than before, where we just made sure that it still had a topBlockquote.  This 
-        doesn't really fix a bug, since we can't really get into a situation where we move to a different 
+        (WebCore::BreakBlockquoteCommand::doApply): Added comments.  Don't need to use newStartNode.  If the
+        startNode needs to change, change it.  Afterwords, check to make sure that it hasn't left topBlockquote.
+        This is slightly stricter than before, where we just made sure that it still had a topBlockquote.  This
+        doesn't really fix a bug, since we can't really get into a situation where we move to a different
         topBlockquote, but it simplifies the code.
 
 2008-10-30  Dirk Schulze  <vbs85@gmx.de>
 
         Some preparation for:
         <rdar://problem/6104369> Hitting return at the end of a quoted line creates an extraneous quoted line
-        
+
         Added an early return to avoid a level of if-nesting.  No other changes.  We probably don't
         need to rebalance whitespace before the early return but for now don't risk any change in behavior
         I'll revisit that later.
 2008-10-29  Kevin Ollivier  <kevino@theolliviers.com>
 
         wx build fixes after addition of runtime and ImageBuffer changes.
-        
+
         * platform/graphics/wx/ImageBufferData.h: Added.
         * platform/graphics/wx/ImageBufferWx.cpp:
         (WebCore::ImageBufferData::ImageBufferData):
         (WebCore::Document::~Document):
         Moved active object and MessagePort tracking up to ScriptExecutionContext, to share code
         with workers.
-        
+
         * dom/Document.h:
         (WebCore::Document::isDocument):
         (WebCore::Document::refScriptExecutionContext):
 
         * editing/Editor.cpp:
         (WebCore::Editor::ignoreSpelling): Remove misspelling markers from the word.
-        (WebCore::Editor::learnSpelling): Added a FIXME about <rdar://problem/5396072>, which 
-        will probably require a change more complicated than just marking the learned word as 
+        (WebCore::Editor::learnSpelling): Added a FIXME about <rdar://problem/5396072>, which
+        will probably require a change more complicated than just marking the learned word as
         misspelled.  I'll address it with a separate patch.
         * editing/EditorCommand.cpp:
         (WebCore::executeIgnoreSpelling): Added.
 2008-10-27  Kevin Watters  <kevinwatters@gmail.com>
 
         Reviewed by Kevin Ollivier.
-        
+
         Update the active state as well as the focused state as both need to be true
         for the caret to be drawn.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=21900
 
         * WebView.cpp:
         Reviewed by Sam Weinig.
 
         No need to call release if the connection is null.
-        
+
         * platform/network/mac/ResourceHandleMac.mm:
         (WebCore::ResourceHandle::start):
 
         Reviewed by Kevin Decker.
 
         <rdar://problem/6322650> Crash in fast/loader/simultaneous-reloads-assert.html
-        
+
         Make sure to null check the NSURLConnection object.
-        
+
         * platform/network/mac/ResourceHandleMac.mm:
         (WebCore::ResourceHandle::start):
         (WebCore::ResourceHandle::setDefersLoading):
 2008-10-25  Geoffrey Garen  <ggaren@apple.com>
 
         Not reviewed.
-        
+
         Try to fix Mac debug build on the buildbot.
 
         * platform/FileChooser.cpp:
         Reviewed by Sam Weinig.
 
         <rdar://problem/5440917> Support NPN_Construct
-                
+
         * WebCore.NPAPI.exp:
         Export _NPN_Construct.
-        
+
         * bridge/NP_jsobject.cpp:
         (_NPN_Construct):
         Implement this.
-        
+
         * bridge/c/c_instance.cpp:
         (JSC::Bindings::CInstance::supportsConstruct):
         (JSC::Bindings::CInstance::invokeConstruct):
         Have this call the appropriate NPClass method.
-        
+
         * bridge/npruntime.h:
         Add NPN_Construct.
 
         * bridge/npruntime_impl.h:
         Add _NPN_Construct.
-        
+
         * bridge/runtime.h:
         (JSC::Bindings::Instance::supportsConstruct):
         (JSC::Bindings::Instance::invokeConstruct):
         New methods.
-        
+
         * bridge/runtime_object.cpp:
         (JSC::callRuntimeConstructor):
         Call the native instance.
-        
+
         (JSC::RuntimeObjectImp::getConstructData):
         Implement this.
-        
+
         * bridge/runtime_object.h:
         Add new method declarations.
-        
+
         * plugins/gtk/PluginPackageGtk.cpp:
         (WebCore::PluginPackage::load):
         * plugins/qt/PluginPackageQt.cpp:
         This is consistent with the direction HTML5 will be going in the future.
 
         The initial implementation here will show "n files" as the text next to the control when multiple files are selected.  You can view
-        the individual files in a tooltip for now.  Improvements to this control will come later.  
+        the individual files in a tooltip for now.  Improvements to this control will come later.
 
-        Web developers will be able to access the FileList from the HTMLInputElement element, where they can get a base name and a size for each file.  
+        Web developers will be able to access the FileList from the HTMLInputElement element, where they can get a base name and a size for each file.
         These FileList Files can also be sent in an XMLHTTPRequest.
 
         * manual-tests/post-multi-file-upload.html: Added.
         but we don't want to break the behavior for existing websites that may rely on this.
         (WebCore::HTMLInputElement::setValue): ditto.
         (WebCore::HTMLInputElement::setValueFromRenderer): This is no longer used for file upload controls. setFileListFromRenderer is used instead.
-        (WebCore::HTMLInputElement::setFileListFromRenderer): Added. 
+        (WebCore::HTMLInputElement::setFileListFromRenderer): Added.
         * html/HTMLInputElement.h:
 
         * page/Chrome.cpp: (WebCore::Chrome::setToolTip): Show a tooltip with the file name list for the multi-file upload control.
         Reviewed by Sam Weinig.
 
         https://bugs.webkit.org/show_bug.cgi?id=21475
-        
+
         Provide support for the Geolocation API
-        
+
         http://dev.w3.org/geo/api/spec-source.html
 
         Test: geolocation/geolocation-not-implemented.html
         Reviewed by Darin Adler
 
         https://bugs.webkit.org/show_bug.cgi?id=21818
-        
+
         Add HashTraits for AtomicString so that AtomicString can be used as
         the key for a HashMap or HashSet.
 
 
         Add WebKitCSSKeyframeRule and WebKitCSSKeyframesRule to the
         switch in _wrapCSSRule.
-        
+
         * bindings/objc/DOMCSS.mm:
         (+[DOMCSSRule _wrapCSSRule:WebCore::]):
 
         Reviewed by Sam Weinig.
 
         https://bugs.webkit.org/show_bug.cgi?id=21475
-        
+
         Provide support for the Geolocation API
-        
+
         http://dev.w3.org/geo/api/spec-source.html
 
         Test: geolocation/geolocation-not-implemented.html
         Reviewed by Oliver Hunt.
 
         * platform/network/curl/ResourceRequest.h:  A better implementation
-          of CFURLRequest, rather than void*.  This gives better compatibility 
+          of CFURLRequest, rather than void*.  This gives better compatibility
           with the WebKit.idl interface.
         (WebCore::ResourceRequest::cfURLRequest):
 
 
         - fix https://bugs.webkit.org/show_bug.cgi?id=21809
 
-        Remove PLATFORM(MAC) wrapping USE(ATSUI). It's redundant and inhibits 
+        Remove PLATFORM(MAC) wrapping USE(ATSUI). It's redundant and inhibits
         its use for PLATFORM(CHROMIUM) in the future.
 
         * platform/graphics/SimpleFontData.h:
 2008-10-22  David Smith  <catfish.man@gmail.com>
 
         Reviewed by Anders Carlsson.
-               
+
         https://bugs.webkit.org/show_bug.cgi?id=19974
         getElementsByClassName not live enough
 
 
         <rdar://6261773> - autocomplete="off" doesn't work on Windows
 
-        Visual Studio makes a poor decision regarding the combination of enums and bitfields, such that a 
-        statement like "m_autocomplete = Off" followed by "return m_autocomplete == Off" would return "false" 
+        Visual Studio makes a poor decision regarding the combination of enums and bitfields, such that a
+        statement like "m_autocomplete = Off" followed by "return m_autocomplete == Off" would return "false"
         instead of the much more correct "true."
-        
-        In the past we have worked around this by declaring the bitfield member as an unsigned instead 
+
+        In the past we have worked around this by declaring the bitfield member as an unsigned instead
         of the enum type.
 
         For more discussion, see http://trac.webkit.org/changeset/25329
 
         Make sure that the AtomicString that identifies a set of animation keyframes
         stays live by keeping a copy in the WebKitCSSKeyframesRule.
-        
+
         Also call styleSheetChanged() when the keyframes name changes, but
         add an internal method that doesn't call styleSheetChanged() for use
         during normal stylesheet parsing.
-        
+
         Test: animations/change-keyframes-name.html
 
         * css/CSSGrammar.y:
 
         Reviewed by Darin Adler.
 
-        Fix for https://bugs.webkit.org/show_bug.cgi?id=20352 
-        REGRESSION(r31030-31055): Choosing "Print window" from frame set 
+        Fix for https://bugs.webkit.org/show_bug.cgi?id=20352
+        REGRESSION(r31030-31055): Choosing "Print window" from frame set
         prints blank page
         and corresponding: <rdar://problem/6142398>
 
-        viewHeight() and viewWidth() only return valuable information if we 
-        are not printing. Everywhere else in the code, we make sure we are 
-        not printing before we call these functions. Adding a check for 
+        viewHeight() and viewWidth() only return valuable information if we
+        are not printing. Everywhere else in the code, we make sure we are
+        not printing before we call these functions. Adding a check for
         printing here fixes this bug, and framesets print again.
 
         * rendering/RenderFrameSet.cpp:
 
         Add a comment explaining layoutDelta(), and an assertion
         that checks that delta changes were correctly paired during layout.
-        
+
         * rendering/RenderView.cpp:
         (WebCore::RenderView::layout):
         * rendering/RenderView.h:
         Fix https://bugs.webkit.org/show_bug.cgi?id=21763
         REGRESSION (r36108-r36113): Can't load picture content on wired.com galleries
         <rdar://problem/6285025
-        
+
         If we get 304 response we need to explicitly check if that completes the frame load.
-        
+
         No test case, simulating the condition is pretty difficult.
 
         * loader/loader.cpp:
         Reviewed by Mark Rowe.
 
         Work around <rdar://problem/6301728>.
-        
+
         Don't assert that loads are deferred when we're using CFNetwork on Mac or Windows.
-        
+
         * loader/MainResourceLoader.cpp:
         (WebCore::MainResourceLoader::didReceiveResponse):
         (WebCore::MainResourceLoader::didReceiveData):
 
         Reviewed by Adele Peterson.
 
-        Speculative fix for <rdar://problem/6242585> CrashTracer: [USER] 19 
+        Speculative fix for <rdar://problem/6242585> CrashTracer: [USER] 19
         crashes in Safari at com.apple.WebCore
         WebCore::ScrollView::visibleContentRect const + 153
 
-        d->m_view can be null and is null-checked at other points in the 
-        code. The logs indicate that this crash occurs when closing a 
-        window containing a PDF, so in case where we crash, the PDF's view 
+        d->m_view can be null and is null-checked at other points in the
+        code. The logs indicate that this crash occurs when closing a
+        window containing a PDF, so in case where we crash, the PDF's view
         is probably already null. So the fix is to simply add a null-check.
 
         * page/Frame.cpp:
 
         Reviewed by Jon Honeycutt
 
-        Fix a crash in Accessibility where a table section was being 
+        Fix a crash in Accessibility where a table section was being
         referenced without first checking if it was null
 
         https://bugs.webkit.org/show_bug.cgi?id=21721
         (WebCore::JavaScriptDebugServer::recompileAllJSFunctionsSoon):
         (WebCore::JavaScriptDebugServer::didAddListener):
         (WebCore::JavaScriptDebugServer::didRemoveListener):
-        * inspector/JavaScriptDebugServer.h: Exported an API for recompiling, 
+        * inspector/JavaScriptDebugServer.h: Exported an API for recompiling,
         used by the Settings object.
 
         * page/Settings.cpp:
 2008-10-17  Kevin Watters  <kevinwatters@gmail.com>
 
         Reviewed by Kevin Ollivier
-        
+
         Fix wx port's scrollbar and drawing handling after recent changes.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=21720
 
         * platform/ScrollView.h:
 
         https://bugs.webkit.org/show_bug.cgi?id=21587
         NPN_PluginThreadAsyncCall does not work properly
-        
+
         * plugins/PluginMainThreadScheduler.cpp:
         (WebCore::PluginMainThreadScheduler::dispatchCalls):
         Make sure to set m_callPending to false once the queue has been cleared.
 
         Reviewed by Sam Weinig.
 
-        Implement didSendBodyData delegate method, and use older code path when that delegate doesn't exist. 
+        Implement didSendBodyData delegate method, and use older code path when that delegate doesn't exist.
 
         * platform/network/ResourceHandle.h:
         * platform/network/mac/FormDataStreamMac.mm: (WebCore::formRead):
 
         Reviewed by Darin Adler, tweaked and landed by Beth.
 
-        * WebCore.base.exp: Exposes two functions to be able to create a 
-        selection from a point on the screen. Also exposes the new 
+        * WebCore.base.exp: Exposes two functions to be able to create a
+        selection from a point on the screen. Also exposes the new
         TextIterator::currentNode function.
         * WebCore/editing/TextIterator.cpp:
-        (TextIterator::currentNode) : New function that returns the current 
+        (TextIterator::currentNode) : New function that returns the current
         text node or NULL if there is no text node.
 
 2008-10-15  Marco Barisione  <marco.barisione@collabora.co.uk>
 2008-10-14  Tor Arne Vestbø  <tavestbo@trolltech.com>
 
         Reviewed by Simon.
-        
+
         Fix QtWebKit scrollbar painting errors
 
         * platform/qt/ScrollbarThemeQt.cpp:
         Reviewed by Simon.
 
         Use style-painter to draw scrollbars in ScrollbarThemeQt
-        
+
         This has the benefit of setting up the painter properly without
         antialiasing and saving and restoring the brush. It also ensures
         that we use the style of the QWebView over the QApplication style
 2008-10-13  Maciej Stachowiak  <mjs@apple.com>
 
         Rubber stamped by Mark Rowe.
-        
+
         - use gcc 4.2 when building with Xcode 3.1 or newer on Leopard, even though this is not the default
 
         * Configurations/DebugRelease.xcconfig:
         Links from image maps were not being included in the web area's AXLinkUIElements attribute.
         The position of image map links was being reported incorrectly and the parent-chain
         hierarchy for image map links was incorrect because image map links did not have a reference
-        to their parent image. 
+        to their parent image.
 
         Test: accessibility/document-links.html
 
         but changes \r, \n, and \r\n to a single space.
 
         Tests: editing/pasteboard/paste-multiline-text-input.html
-               
+
         * html/HTMLInputElement.cpp:
         (WebCore::HTMLInputElement::constrainValue):
 
 
         Reviewed by Alp Toker.
 
-        Add version parsing for Flash, and the PluginQuirkDontSetNullWindowHandleOnDestroy 
-        plugin quirk if Flash 10 or newer since at least in b218, setting a NULL window 
+        Add version parsing for Flash, and the PluginQuirkDontSetNullWindowHandleOnDestroy
+        plugin quirk if Flash 10 or newer since at least in b218, setting a NULL window
         handler on destroy crashes WebKit <https://bugs.webkit.org/show_bug.cgi?id=19859>
 
         * plugins/gtk/PluginPackageGtk.cpp:
         Reviewed by Darin Adler
 
         https://bugs.webkit.org/show_bug.cgi?id=21538
-        
+
         The blendFunc() for TransformOperations should never mess with the
         operation lists by replacing some ops with identity, otherwise the lists
         no longer match and transitions break.
 
         Fix re-targetting a running transition; if the target property
         changes, we do need to make a new ImplicitAnimation.
-        
+
         Test: transitions/retargetted-transition.html
 
         * page/animation/CompositeAnimation.cpp:
 
         Redesigned how animation events are sent in order to get rid of
         AnimationEventDispatcher. ImplicitAnimation and KeyframeAnimation
-        are now ref counted. While calling the event handler, I keep a 
+        are now ref counted. While calling the event handler, I keep a
         reference to this class to avoid it getting destroyed out from under me.
 
         I also moved most of the functionality of CompositeAnimation to
         Media controls should not show when element is not visible
 
         https://bugs.webkit.org/show_bug.cgi?id=21155
-        
+
         Reviewed by Adele Peterson.
 
         * html/HTMLMediaElement.cpp:
         Gradient clips were not getting cleared after filling the
         gradient, this was causing nothing to draw in Canvex
         https://bugs.webkit.org/show_bug.cgi?id=21498
-        
+
         Tests forthcoming.
 
         * WebCore.xcodeproj/project.pbxproj:
 2008-10-09  Eric Seidel  <eric@webkit.org>
 
         Reviewed by Oliver Hunt.
-        
+
         Fix transformed patterns
         https://bugs.webkit.org/show_bug.cgi?id=21498
-        
+
         Test: fast/canvas/patternfill-repeat.html
 
         * WebCore.xcodeproj/project.pbxproj:
         Reviewed by Antti Koivisto.
 
         <rdar://problem/6273887> Crash in ApplicationCacheGroup
-        
+
         Make sure to stop loading even before a cache update is in progress so that the
         manifest load will be stopped.
 
             scrollbar-corner
             scrollbar-thumb
             scrollbar-track
-        
+
         These elements will work with all the usual pseudo-classes (:hover, :active, :enabled, :disabled, etc.) and with
         the following new pseudo-classes:
             scrollbar-active
             scrollbar-forward
             scrollbar-horizontal
             scrollbar-vertical
-        
+
         Reviewed by Adele
 
         * css/CSSSelector.cpp:
 2008-10-06  Jeremy Moskovich  <jeremy@chromium.org>
 
         Reviewed by Dan Bernstein.
-        
+
         Added C++ forward declaration for the NSURLAuthenticationChallenge class
         so that the m_currentMacChallenge variable doesn't cause a
         compilation error when ResourceHandleInternal.h is included from a C++ file.
 
         https://bugs.webkit.org/show_bug.cgi?id=21416
         Add missing null checks identified by Application Verifier.
-        
+
         Reviewed by Darin Adler.
 
         * platform/win/SharedTimerWin.cpp:
 2008-10-03  Steve Falkenburg  <sfalken@apple.com>
 
         <rdar://problem/6249833> Fix default button appearance
-        
+
         Reviewed by Adele Peterson.
 
         * rendering/RenderThemeWin.cpp:
 
         SVG should support ascent and descent properties <font-face> instead of <font>!
         https://bugs.webkit.org/show_bug.cgi?id=21365
-        
+
         Tested by many many existing SVG tests.
 
         * svg/SVGFontFaceElement.cpp:
 
         The solution is to track whether we are entering or reentering the parsing
         of a script element.  We do this simply by 0 checking scriptStartLineno,
-        and resetting it after we complete parsing of each script element. 
+        and resetting it after we complete parsing of each script element.
 
         Test: http/tests/incremental/pause-in-script-element.pl
 
 2008-10-03  David Hyatt  <hyatt@apple.com>
 
         https://bugs.webkit.org/show_bug.cgi?id=21340
-        
+
         Remove "containingWindow()/setContainingWindow()" from Widget.  HostWindow covers this now.
-    
+
         Reviewed by Dan Bernstein & Darin Adler
-        
+
         * platform/ScrollView.cpp:
         (WebCore::ScrollView::addChild):
         * platform/Widget.cpp:
 
         Reviewed by Dan Bernstein.
 
-        Fix for <rdar://problem/6012018> 
+        Fix for <rdar://problem/6012018>
         https://bugs.webkit.org/show_bug.cgi?id=21335
         CrashTracer: [USER] 4959 crashes in Safari at com.apple.WebCore: WebCore::Frame::settings const + 8
 
 2008-10-03  David Hyatt  <hyatt@apple.com>
 
         Remove addToDirtyRegion.
-        
+
         Reviewed by Oliver Hunt
 
         * page/Chrome.cpp:
 
         Reviewed by Maciej Stachowiak.
 
-        - fix a CachedResource leak introduced in r37176 
+        - fix a CachedResource leak introduced in r37176
 
         Undo r37176 and instead allow pruneDeadResources() to be re-entered, but
         afterwards bail out of the outer pruneDeadResources().
 
         Fix Gtk adjustments so that they are properly checked again before creating horizontal/vertical
         scrollbars.
-        
+
         Reviewed by Oliver Hunt
 
         * platform/ScrollView.cpp:
 2008-10-02  David Hyatt  <hyatt@apple.com>
 
         https://bugs.webkit.org/show_bug.cgi?id=21328
-        
+
         Make widget invalidation more cross-platform.
-        
+
         (1) Make invalidateRect a pure virtual function on Widget.  All leaf widgets must now implement this function.
-        
+
         (2) Scrollbars now send invalidations through the ScrollbarClient.  windowClipRect on ScrollbarClient has been removed and replaced with this invalidation call.
         This allows all scrollbar invalidations to go through the render tree so that transforms and reflections will be respected.
-        
+
         (3) Plugins now have the native window invalidation code for windowed plugins.  Windowless plugins do a repaintRectangle on the plugin's renderer.
-        
+
         (4) FrameViews now do a repaintRectangle on their owner element's renderer.
 
         Reviewed by Sam Weinig
 
         Reviewed by Dave Hyatt
 
-        Fix RenderFileUploadControl::setStyle() and 
+        Fix RenderFileUploadControl::setStyle() and
         RenderMenuList::setStyle() to not touch the style; replace
         with html4.css and CSSStyleSelector changes.
 
 
         Clean up code that changes the RenderStyle passed in to
         table renderer setStyle() methods.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=21287
-        
+
         Tests: fast/table/floating-th.html
                fast/table/table-display-types-strict.html
                fast/table/table-display-types.html
         - fix a Database leak that resulted in Document leaks
 
         * storage/Database.cpp:
-        (WebCore::Database::openDatabase): Account for the fact that RefCounted 
+        (WebCore::Database::openDatabase): Account for the fact that RefCounted
         objects start out with a ref count of 1.
 
 2008-10-02  Sam Weinig  <sam@webkit.org>
 2008-10-02  David Hyatt  <hyatt@apple.com>
 
         https://bugs.webkit.org/show_bug.cgi?id=21314
-        
+
         Make scrollBackingStore cross-platform.
 
         Reviewed by Sam Weinig
 2008-10-01  David Hyatt  <hyatt@apple.com>
 
         https://bugs.webkit.org/show_bug.cgi?id=21298
-        
+
         Make updateScrollbars cross-platform.  For now a stubbed out scrollContents function is invoked to do the scrolling of the backing store.  Next patch
-        will make that cross-platform.  
-        
+        will make that cross-platform.
+
         The ScrollView now implements ScrollbarClient, which means that there was a clash of windowClipRect methods from the
         multiple inheritance.  For now I solved this by adding a Scrollbar* to the ScrollbarClient version of the method, but longer term
         windowClipRect is going to be removed from ScrollbarClient (when Widget invalidation gets rewritten).
-        
+
         Reviewed by Sam Weinig
 
         * page/FrameView.cpp:
         (WebCore::XMLHttpRequest::detachRequests):
         (WebCore::XMLHttpRequest::setPendingActivity):
         (WebCore::XMLHttpRequest::unsetPendingActivity):
-        Moved XMLHttpRequest tracking from a global map to Document. 
+        Moved XMLHttpRequest tracking from a global map to Document.
 
         * bindings/js/JSDOMBinding.cpp:
         (WebCore::wrapperSet):
 2008-10-02  Geoffrey Garen  <ggaren@apple.com>
 
         Not reviewed.
-        
+
         Try to fix Windows build.
 
         * WebCore.vcproj/WebCore.vcproj:
         (WebCore::JavaScriptDebugServer::didExecuteProgram):
         (WebCore::JavaScriptDebugServer::didReachBreakpoint):
         * page/JavaScriptDebugServer.h:
-        * page/inspector/ScriptsPanel.js: Renamed internal uses of sourceId and 
+        * page/inspector/ScriptsPanel.js: Renamed internal uses of sourceId and
         sourceIdentifier to sourceID.
 
 2008-10-01  Dan Bernstein  <mitz@apple.com>
 2008-10-01  Evan Martin  <evan@chromium.org>
 
         Reviewed by Eric Seidel.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=20669
 
         * css/makeprop.pl:
         - fix an assertion failure in http/tests/security/canvas-remote-read-svg-image.html due to re-entry into Cache::pruneDeadResources()
 
         * loader/Cache.cpp:
-        (WebCore::Cache::remove): Disable pruning temporarily during 
+        (WebCore::Cache::remove): Disable pruning temporarily during
         CachedResource deletion.
 
 2008-10-01  Dan Bernstein  <mitz@apple.com>
 2008-10-01  David Hyatt  <hyatt@apple.com>
 
         https://bugs.webkit.org/show_bug.cgi?id=21282
-        
+
         Make contentsToScreen/screenToContents cross-platform.  Only implemented by Mac/Win right now.
-        
+
         Reviewed by Adam Roben
 
         * loader/EmptyClients.h:
         Reviewed by Sam Weinig.
 
         - fix HTMLViewSourceDocument leaks by breaking a ref cycle which
-          was fixed in r17249 and re-introduced in r31435. 
+          was fixed in r17249 and re-introduced in r31435.
 
         * html/HTMLViewSourceDocument.cpp:
         (WebCore::HTMLViewSourceDocument::HTMLViewSourceDocument):
         More for https://bugs.webkit.org/show_bug.cgi?id=21122
         Autogenerate JS event listeners
 
-        - Make EventListener getter/setters names in c++ files match those in 
+        - Make EventListener getter/setters names in c++ files match those in
           JS (ie. onLoadListener -> onload).
         - Add standard way to access the frame associated with the EventTarget
           for EventListener lookup.
 
         Reviewed by Darin Adler.
 
-        Fix for https://bugs.webkit.org/show_bug.cgi?id=20396 Abort caused 
+        Fix for https://bugs.webkit.org/show_bug.cgi?id=20396 Abort caused
         by failed allocation due to invalid counter/attr
         and corresponding: <rdar://problem/6152371>
 
         * css/CSSParser.cpp:
-        (WebCore::CSSParser::parseCounterContent): The spec indicates that 
+        (WebCore::CSSParser::parseCounterContent): The spec indicates that
         only identifiers should be accepted here.
 
 2008-09-30  Kevin McCullough  <kmccullough@apple.com>
         (3) Used on Qt, but implemented incorrectly to cause a full repaint for no reason.
         (4) Used on Gtk, but implemented incorrectly to cause a full repaint for no reason.
         (5) Used on wx, but probably not needed (since wx has platform widgets like Mac).
-        
+
         There is now a paint method on HostWindow that does an immediate mode repaint of an empty rect to
         cause the display/flush to happen if needed (thus saving an extra method on ChromeClient).
         With the changes to this method, the new behavior is as follows:
         (3) Nothing happens on Qt, but only because they have not implemented immediate mode updating.
         (4) Gtk now behaves like Windows and will process updates but not do any incorrect additional invalidation.
         (5) Not used on wx
-        
+
         This method was originally added for Windows. It's not clear the display/flush is really even necessary on the other
         platforms.  At the very least stopping unnecessary full invalidations on Qt/Gtk is a good thing. :)
-        
+
         Reviewed by Sam Weinig
 
         * ChangeLog:
 2008-09-30  Dave Hyatt  <hyatt@apple.com>
 
         http://bugs.webkit.org/show_bug.cgi?id=21250
-        
+
         Rename updateContents to repaintContentRectangle and make it cross-platform by always sending
         repaints up through the ChromeClient.
 
 
         Reviewed by Anders Carlsson.
 
-        * bridge/npapi.h: Tweaked NPNVariable enum. NPNVsupportsCocoaBool,  NPNVsupportsCarbonBool are now in the 3000 range instead of 
-        2000. 
+        * bridge/npapi.h: Tweaked NPNVariable enum. NPNVsupportsCocoaBool,  NPNVsupportsCarbonBool are now in the 3000 range instead of
+        2000.
 
 2008-09-29  Dan Bernstein  <mitz@apple.com>
 
 
         Reviewed by Anders Carlsson
 
-        Fix RenderStyle leak in animation code, and assert that 
+        Fix RenderStyle leak in animation code, and assert that
         keyframe resolution in CSSStyleSelector is not going to clobber
         m_style.
 
 
         Fixed https://bugs.webkit.org/show_bug.cgi?id=20995
         Rewrite keyframe resolution to be like styleForElement()
-        
+
         Test: animations/lineheight-animation.html
 
         * css/CSSStyleSelector.cpp:
 
         Clean up fix in PropertyWrapperGetter::equals
         https://bugs.webkit.org/show_bug.cgi?id=21011
-        
+
         Test: transitions/override-transition-crash.html
 
         * page/animation/AnimationBase.cpp:
 
         fix https://bugs.webkit.org/show_bug.cgi?id=20994
         <rdar://problem/6171023> HTMLVideoElement width and height attributes are now unsigned
-        
+
         HTML5 spec says HTMLVideoElement width and height attributes should be unsigned. Convert
         all unsigned media attributes from string with toUInt() instead of toInt().
-        
+
         * html/HTMLMediaElement.cpp:
         (WebCore::HTMLMediaElement::playCount): convert from attribute with toUInt().
 
 
         There was a lot wrong with 17589, so more patches will be coming to deal with the fallout of this patch
         (which should never have landed in the first place).
-       
+
         Reviewed by Tim Hatcher, Sam Weinig
 
         * platform/win/ScrollViewWin.cpp:
         Reviewed by Antti
 
         https://bugs.webkit.org/show_bug.cgi?id=21116
-        <rdar://problem/5726325> Audio from <video> can still be heard after navigating 
+        <rdar://problem/5726325> Audio from <video> can still be heard after navigating
         back to page with <video>, then closing tab
 
         Rename willSaveToCache/didRestoreFromCache callbacks to documentWillBecomeInactive/
         The solution to this is to make sure we cache these constructors
         in the same way as all the other DOM constructors.  To achieve this
         without causing any refcount cycles it is necessary to replace the
-        refcounted document pointer in the Image, MessageChannel, Option, 
-        XMLHttpRequest, and Audio constructor objects with a reference to 
+        refcounted document pointer in the Image, MessageChannel, Option,
+        XMLHttpRequest, and Audio constructor objects with a reference to
         the document's JS wrapper.
 
         Tests: fast/dom/constructors-cached-navigate.html
 2008-09-25  David Hyatt  <hyatt@apple.com>
 
         https://bugs.webkit.org/show_bug.cgi?id=21133
-        
+
         Rename resizeContents method on ScrollView to setContentsSize (to match contentsSize()).  Make it
         cross-platform.
 
 2008-09-25  David Smith  <catfish.man@gmail.com>
 
         Reviewed by Darin Adler.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=20980
         Split off uncommonly used data from Node similar to ElementRareData
-        
+
         Saves an OwnPtr and a short on Node, as well as providing room for an isContainer bit,
         which in turn allows inlining firstChild(), lastChild(), childNodeCount(), and childNode()
         for a 5-10+% performance win on SlickSpeed and assorted speedups on other tests.
 
-        * WebCore.base.exp: 
+        * WebCore.base.exp:
         * WebCore.xcodeproj/project.pbxproj:
         * dom/ChildNodeList.cpp: Include ContainerNode to pick up its definitions for childNodeCount(), etc...
         * dom/ContainerNode.cpp: Set isContainer to true
         (WebCore::Node::containerFirstChild): ditto
         (WebCore::Node::containerLastChild): ditto
         * dom/Element.cpp: Most of the changes here are moving ElementRareData to ElementRareData.h and NodeRareData.h
-        (WebCore::Element::Element): 
+        (WebCore::Element::Element):
         (WebCore::Element::~Element):
         (WebCore::Element::rareData):
         (WebCore::Element::ensureRareData): Renamed from createRareData
 
         https://bugs.webkit.org/show_bug.cgi?id=21052
         Generalize id selector special case for querySelectorAll
-        
+
         By checking the element we get with getElementById against the selector, we can use the special case in many more circumstances.
-        Changes results on http://native.khan.mozilla.org 
+        Changes results on http://native.khan.mozilla.org
         from
             #title: 2ms
             h1#title: 55ms
             #title: 1ms
             h1#title: 2ms
             div #title: 5ms
-        
+
         * dom/Node.cpp:
         (WebCore::Node::querySelector):
         * dom/SelectorNodeList.cpp:
 2008-09-25  Eric Carlson  <eric.carlson@apple.com>
 
         Reviewed by Eric Seidel.
-        
+
         <rdar://problem/6171047> HTMLMediaElement "begin" event is now "loadstart"
         https://bugs.webkit.org/show_bug.cgi?id=21003
 
 2008-09-25  David Smith  <catfish.man@gmail.com>
 
         Reviewed by Eric Seidel
-        
+
         fix https://bugs.webkit.org/show_bug.cgi?id=21091
         Regression: querySelector matches tag names case sensitively
-        
+
         Tests: fast/dom/SelectorAPI/caseTag.html
                fast/dom/SelectorAPI/caseTagX.xhtml
 
         Reviewed by Dan Bernstein.
 
         Test: editing/spelling/inline_spelling_markers.html
-        
+
         - https://bugs.webkit.org/show_bug.cgi?id=20092
         Spelling markers positioned incorrectly in RTL text
 
 
         Fix for bug 21012.  The Aqua scrollbar was returning the wrong track rect on Windows Aqua theme.  Make
         sure to not accidentally fall into the vertical scrollbar case for horizontal scrollbars. :)
-        
+
         Reviewed by Sam Weinig
 
         * platform/mac/ScrollbarThemeMac.mm:
 
         Reviewed by Sam Weinig.
 
-        Fix for https://bugs.webkit.org/show_bug.cgi?id=21041 "Add Contact" 
+        Fix for https://bugs.webkit.org/show_bug.cgi?id=21041 "Add Contact"
         link at gmail does not support AXPress action
         and corresponding: <rdar://problem/6216178>
 
-        I fixed this bug by making AccessibilityObject::anchorElement 
+        I fixed this bug by making AccessibilityObject::anchorElement
         support ARIA links.
 
         * page/AccessibilityImageMapLink.cpp:
 
         Bug 20949: Catch repeated messages in Inspector Controller to limit
         memory usage
-        - Store the repeat count in the Console Message object, in the 
+        - Store the repeat count in the Console Message object, in the
         Inspector Controller and JS ConsoleMessage object.
 
         * page/InspectorController.cpp:
         ~2% speedup on EarleyBoyer
 
         (WebCore updates.)
-        
+
         * bindings/js/JSQuarantinedObjectWrapper.h:
         (WebCore::JSQuarantinedObjectWrapper::createStructureID):
 
 2008-09-23  Eric Seidel  <eric@webkit.org>
 
         No review, build fix only.
-        
+
         Third time's the charm, eh?  My local build is sadly still not done...
 
         * platform/graphics/AffineTransform.cpp: remove extra &
         transition end event when -webkit-transition-property: all puts wrong
         propertyName in event
         https://bugs.webkit.org/show_bug.cgi?id=20903
-        
+
         * page/animation/ImplicitAnimation.cpp:
         (WebCore::ImplicitAnimation::sendTransitionEvent):
 
         - speed up instanceof operator by replacing implementsHasInstance method with a TypeInfo flag
 
         Partial work towards <https://bugs.webkit.org/show_bug.cgi?id=20818>
-        
+
         2.2% speedup on EarleyBoyer benchmark.
 
         * bindings/js/JSQuarantinedObjectWrapper.cpp:
         Reviewed by Dave Hyatt.
 
         Based on initial work by Darin Adler.
-        
+
         - replace masqueradesAsUndefined virtual method with a flag in TypeInfo
         - use this to JIT inline code for eq_null and neq_null
         https://bugs.webkit.org/show_bug.cgi?id=20823
-        
+
         * WebCore.xcodeproj/project.pbxproj:
         * WebCore.vcproj/WebCore.vcproj:
         * bindings/js/JSCSSStyleDeclarationCustom.cpp:
 
         - introduce a TypeInfo class, for holding per-type (in the C++ class sense) date in StructureID
         https://bugs.webkit.org/show_bug.cgi?id=20981
-        
+
         * bindings/js/JSAudioConstructor.cpp:
         (WebCore::JSAudioConstructor::JSAudioConstructor):
         * bindings/js/JSCSSStyleDeclarationCustom.cpp:
 2008-09-21  Steve Falkenburg  <sfalken@apple.com>
 
         Removed unnecessary nested timer check.
-        
+
         Rubber-stamped by Dan Bernstein.
 
         * platform/win/SharedTimerWin.cpp:
 
         Improve timer resolution on WinXP.
         https://bugs.webkit.org/show_bug.cgi?id=20979
-        
+
         Removed last-chance timer. It should not be necessary.
         Change timeEndPeriod timer to fire in 300ms instead of 20ms. Calling timeBeginPeriod/timeEndPeriod too often throws off accuracy.
         Remove Vista checks. We now run the same code on both XP and Vista.
-        
+
         Call through to JSC::getCurrentUTCTimeWithMicroseconds from WebCore::currentTime.
         The code previously called GetSystemTimeAsFileTime, which is always low-resolution on XP, even within timeBeginPeriod(1).
-        
+
         Reviewed by Maciej Stachowiak.
 
         * platform/win/SharedTimerWin.cpp:
         Reviewed by Dan Bernstein.
 
         Fix memory leak.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=20505
 
         * platform/wx/wxcode/mac/carbon/fontprops.cpp:
 
 2008-09-19  David Hyatt  <hyatt@apple.com>
 
-        Add support for hit testing of all five possible scrollbar button placements. 
+        Add support for hit testing of all five possible scrollbar button placements.
 
         Reviewed by Sam Weinig
 
 
         Fix for <rdar://problem/6231308> crash in AutoTableLayout
 
-        The code assumes later on that a TableSection's grid's row vector 
+        The code assumes later on that a TableSection's grid's row vector
         will never be empty. So make 1 the minimum number of columns.
 
         * rendering/RenderTableSection.cpp:
 2008-09-19  Steve Falkenburg  <sfalken@apple.com>
 
         Roll out r36626. It is causing variance in SunSpider numbers on XP.
-        
+
         Rubber stamped by Mark Rowe.
 
         * platform/win/SharedTimerWin.cpp:
         (WebCore::CoreTextController::CoreTextController): Added a
         mayUseNaturalWritingDirection parameter.
         (WebCore::CoreTextController::collectCoreTextRunsForCharacters): Changed
-        to force the bidi embedding level whenever 
+        to force the bidi embedding level whenever
         m_mayUseNaturalWritingDirectrion is false. Since this is now a common
         case, made the typesetter options dictionaries static.
         * platform/graphics/mac/CoreTextController.h:
 
         <rdar://problem/6213171> WebKit should use new array-centric methods for AX performance
 
-        Implement a few AX API methods that will be called by AppKit, which will 
+        Implement a few AX API methods that will be called by AppKit, which will
         speed up access to accessibility objects
 
         * page/mac/AccessibilityObjectWrapper.mm:
         https://bugs.webkit.org/show_bug.cgi?id=20690
 
         Invoke WebCore::prefetchDNS() on host names that appear in
-        in the href of hyperlinks and <link rel="dns-prefetch">. This 
+        in the href of hyperlinks and <link rel="dns-prefetch">. This
         can be used to implement DNS prefetching.
 
         * WebCore.vcproj/WebCore.vcproj:
 
         Reviewed by Beth Dakin
 
-        <rdar://problem/6224222> AX: should not expose a <table> as an AXTable if ARIA 
+        <rdar://problem/6224222> AX: should not expose a <table> as an AXTable if ARIA
         role specifies otherwise
 
         If a <table> isn't an AXTable, the rows and cells should default to AccessibilityRenderObject
 2008-09-18  Steve Falkenburg  <sfalken@apple.com>
 
         Use higher-resolution timers on all variants of Windows.
-        
+
         Reviewed by Darin Adler.
 
         * platform/win/SharedTimerWin.cpp:
 
         Reviewed by Dave Hyatt.
 
-        Fix for https://bugs.webkit.org/show_bug.cgi?id=20515 Crash upon 
+        Fix for https://bugs.webkit.org/show_bug.cgi?id=20515 Crash upon
         parsing CSS: unicode-range: searchfield-cancel-buttonpt=-webkit-
         dashboard-region=
         and corresponding: <rdar://problem/6174100>
 
-        This patch makes CSSParserValue::createCSSValue handle unknown 
+        This patch makes CSSParserValue::createCSSValue handle unknown
         identifiers.
 
         * css/CSSParserValues.cpp:
-        (WebCore::CSSParserValue::createCSSValue): If we have an identifier 
-        with no id (an unknown identifier) create a CSSPrimitiveValue of 
+        (WebCore::CSSParserValue::createCSSValue): If we have an identifier
+        with no id (an unknown identifier) create a CSSPrimitiveValue of
         type CSS_PARSER_IDENTIFIER
         * css/CSSPrimitiveValue.cpp:
         (WebCore::CSSPrimitiveValue::cssText):
 
         <rdar://problem/5722310> gracefully handle too many console messages
         (20904)
-        - Keep track of the most previous message and then compare it to the 
+        - Keep track of the most previous message and then compare it to the
         subsequent messages as they come in.  If there are multiple of the same
         message create a count that indicates the current number.
 
 
         Rubber-stamped with love by David Hyatt.
 
-        Split IdentityTransformOperation, MatrixTransformOperation, RotateTransformOperation, 
+        Split IdentityTransformOperation, MatrixTransformOperation, RotateTransformOperation,
         ScaleTransformOperation, SkewTransformOperation, StyleTransformData, TransformOperation,
         TransformOperations and TranslateTransformOperation out of RenderStyle.h/cpp
 
 2008-09-17  Steve Falkenburg  <sfalken@apple.com>
 
         Add back isFrameView check to fix failed assertion during scroll bar teardown.
-        
+
         Reviewed by Dave Hyatt.
 
         * platform/Scrollbar.cpp:
         This is a better fix for: Invalid CSS code crashes Safari
         https://bugs.webkit.org/show_bug.cgi?id=20512
 
-        The spec indicates that the only valid input for a counter is a 
+        The spec indicates that the only valid input for a counter is a
         number or an identifier. So that is exactly what we allow.
 
         * css/CSSParser.cpp:
 
         Reviewed by Adele Peterson.
 
-        Patch for <rdar://problem/6133884> 
+        Patch for <rdar://problem/6133884>
         Calling window.resizeTo() on a subframe shouldn't change the window size
 
         Test: fast/dom/Window/window-resize-and-move-sub-frame.html
 
         Reviewed by Adam Roben.
 
-        Fix for https://bugs.webkit.org/show_bug.cgi?id=20512 Invalid CSS 
-        code crashes Safari 
+        Fix for https://bugs.webkit.org/show_bug.cgi?id=20512 Invalid CSS
+        code crashes Safari
         and corresponding: <rdar://problem/6173832>
 
-        Reading through the spec, it seems like a function is not valid 
-        input for a counter. So this patch checks for that and bails in the 
-        case of invalid input. 
+        Reading through the spec, it seems like a function is not valid
+        input for a counter. So this patch checks for that and bails in the
+        case of invalid input.
 
         * css/CSSParser.cpp:
         (WebCore::CSSParser::parseCounterContent):
 
         Reviewed by Antti Koivisto.
 
-        Fix <rdar://problem/6227089> 
+        Fix <rdar://problem/6227089>
         Crash in WebCore::Frame::setNeedsReapplyStyles()
-        
+
         View is null checked elsewhere too.
 
         * page/Frame.cpp:
 
         Convert Qt over to its ScrollbarTheme.  Add cross-platform support for jumping the thumb location directly
         to a pressed location on the track (this is done with the middle mouse on Qt and with Shift-Click on Windows).
-      
+
         Reviewed by Sam Weinig
 
         * platform/ScrollBar.cpp:
 2008-09-15  David Smith  <catfish.man@gmail.com>
 
         Reviewed by Sam Weinig.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=20180
-        
+
         Cache the result of parsing the an+b expression in :nth-child(an+b). Roughly a 2x speedup for :nth-child on SlickSpeed.
 
         * GNUmakefile.am: Add CSSNthSelector
 
         Completely switch Windows scrollbars over to ScrollbarTheme.  The Aqua windows scrollbar and native scrollbar
         now share most of their code.
-       
+
         Also removing mini size scrollbar support, since it would have crashed Windows had it ever been used, and it
         also is never used in our code anyway.
-       
+
         Reviewed by Sam Weinig
 
         * WebCore.vcproj/WebCore.vcproj:
 
         Lists are now exposed through AX. On the mac (in SnowLeopard) they use two new subroles
         AXContentList for <ul>, <ol> and AXDefinitionList for <dl>
-        Inside the definition list, each <dt> "term" has an accessibility description ("term"), 
+        Inside the definition list, each <dt> "term" has an accessibility description ("term"),
         as does each <dd> tag ("definition")
 
         Test: accessibility/lists.html
 
         Convert PlatformScrollbarWin to use ScrollbarTheme for hit testing.  PlatformScrollbarQt/Gtk still
         need to be converted.
-        
+
         Reviewed by Sam Weinig
 
         * platform/ScrollBar.cpp:
 
         Reviewed by Maciej.
 
-        <rdar://problem/6163988> 
+        <rdar://problem/6163988>
         https://bugs.webkit.org/show_bug.cgi?id=20457
         Canvas: createPattern crashes WebKit in WTF::RefPtr<WebCore::Image>::operator!() const + 9 with a 1D pattern
-        
+
         Check the width and height and throw an exception if any of them are 0.
-        
+
         * html/CanvasRenderingContext2D.cpp:
         (WebCore::CanvasRenderingContext2D::createPattern):
 
         Reviewed by Dave Hyatt.
 
         https://bugs.webkit.org/show_bug.cgi?id=20180
-        
+
         >2x speedup on querySelector[All] with :nth-child by removing the collectRulesOnly argument on SelectorChecker.
         With collectRulesOnly set to true (the default) an optimization is turned off, and no callers were intentionally leaving it true.
         querySelector[All] assumed that the default was probably the right way to go, and so ended up being slow.
-        
+
         * css/CSSStyleSelector.cpp:
         (WebCore::CSSStyleSelector::CSSStyleSelector):
         (WebCore::CSSStyleSelector::SelectorChecker::SelectorChecker):
 
         Refactor PlatformScrollbarWin's painting so that it is now done by ScrollbarThemeWin.  PlatformScrollbarSafari
         is still painting itself (a subsequent patch will move its painting into ScrollbarThemeSafari).
-       
+
         Reviewed by Sam Weinig
 
         * platform/ScrollBar.h:
         Add a new ScrollbarThemeComposite base class that is shared by ScrollbarThemeSafari and
         ScrollbarThemeWin.  Any scrollbar that wants to render by ScrollbarPart can subclass from this
         theme.
-        
+
         Reviewed by Sam Weinig
 
         * WebCore.vcproj/WebCore.vcproj:
 
         Remove scrollbar's setRect method.  Clients can now just call setFrameGeometry (the corresponding Widget) method
         to set the dimensions of a scrollbar.
-        
+
         Reviewed by Sam Weinig
 
         * platform/ScrollBar.h:
 2008-09-13  Dave Hyatt  <hyatt@apple.com>
 
         Remove setEnabled() from all Scrollbar classes that were just using their Widget base class methods.
-       
+
         Reviewed by Sam Weinig
 
         * platform/ScrollBar.h:
         (WebCore::FontCache::getLastResortFallbackFont): Removed an outdated
         comment (the user's preferred standard font is included in the search
         thanks to code in FontCache::getFontData()) and changed to return the
-        LastResort font.  
+        LastResort font.
 
 2008-09-13  Dave Hyatt  <hyatt@apple.com>
 
         This patch adds ScrollbarTheme to the build.  ScrollbarTheme is similar to RenderTheme (but for
         scrollbars only).  For now ScrollbarTheme just handles returning the size of scrollbars.  Subsequent
         patches will move rendering and hit testing functionality into this class.
-        
+
         Implemented ScrollbarTheme subclasses for four ports (Mac, Win, Gtk, Qt).  The wxWidgets port does not
         implement scrollbars yet, so I added a temporary link stub to keep things compiling on wx.
-       
+
         Reviewed by Sam Weinig
 
         * GNUmakefile.am:
         Move the member variables of PlatformScrollbar up into Scrollbar.  Move autoscroll timer handling
         up into scrollbar.  Make sure a bunch of cross-platform code now just uses "Scrollbar" instead of
         "PlatformScrollbar."
-        
+
         Reviewed by Sam Weinig
 
         * WebCore.vcproj/WebCore.vcproj:
 
         * page/EventHandler.cpp: (WebCore::EventHandler::hitTestResultAtPoint):
         Hit testing again on the main frame will only work if we aren't already on the main frame.  If we are already on the main frame, that will
-        cause infinite recursion.  This change checks that we're not already on the main frame before hit testing again. 
+        cause infinite recursion.  This change checks that we're not already on the main frame before hit testing again.
 
 2008-09-12  Kevin McCullough  <kmccullough@apple.com>
 
         Preparation for major scrollbar rearchitecture.  Eliminate the concept of non-widget based
         scrollbars (which we never supported anyway).  Make Scrollbar derive from Widget directly.
         Remove all the isWidget and hasPlatformScrollbars guards.
-        
+
         Reviewed by Tim Hatcher
 
         * platform/ScrollBar.h:
         Reviewed by Darin
 
         https://bugs.webkit.org/show_bug.cgi?id=20180
-        
+
         This speeds up :nth-child matching as tested by querySelectorAll by around 10% by inlining the common case of Node::renderStyle(). Many thanks to mitzpettel and othermaciej for suggestions and help.
 
         * WebCore.xcodeproj/project.pbxproj: Add NodeRenderStyle.h
 2008-09-09  Dave Hyatt  <hyatt@apple.com
 
         Resurrect PlatformScrollbarWin.
-        
+
         Reviewed by Adam Roben
 
         * WebCore.vcproj/WebCore.vcproj:
         Make the Windows classic case work again.  Overlap in slider constants and other constants broke
         the push state for buttons and checkboxes.  Make sure to check the RenderObject's appearance
         value to make sure it's really a slider thumb to resolve this conflict.
-       
+
         Reviewed by Adam Roben
 
         * rendering/RenderThemeWin.cpp:
         RenderThemeWin that does not have any Skia dependencies should provide a good basis for a potential
         merge of Google Chrome's RenderThemeWin (to help us see where the Skia stuff can be cut and to see
         what changes have been made as compared with the old RenderThemeWin).
-        
+
         Fix CSS2 system fonts to properly set both the rendering mode (so that they properly pick up GDI vs. CG
         text rendering modes) and the printer font setting (so that OS X system fonts properly pick up screen
         vs. printer advancement rounding).
-    
+
         Reviewed by Adam/Sam
 
         * css/CSSStyleSelector.cpp:
 
         Reviewed by Alexey Proskuryakov.
 
-        Try MIME charset names before trying IANA names 
+        Try MIME charset names before trying IANA names
         ( https://bugs.webkit.org/show_bug.cgi?id=17537 )
 
         With this change, shorter and more widely used names (preferred MIME
         document.inputEncoding rather than IANA names. This  helps
         fixing bug 18085 in addition to web developers who are more familiar
         with MIME names. For instance, EUC-JP, ISO-8859-X and US-ASCII will be
-        returned instead of Extended_UNIX_Code_Packed_Format_for_Japanese, 
-        ISO-8859-X:19xx, and ANSI_X3.4-1968. It also replaces IBM8xx with cp8xx. 
+        returned instead of Extended_UNIX_Code_Packed_Format_for_Japanese,
+        ISO-8859-X:19xx, and ANSI_X3.4-1968. It also replaces IBM8xx with cp8xx.
         Note that cp/IBM 8xx are extremly rare in today's web. Even if they're
         used, the former is still recognized as aliases to the latter so that
         there's very little, if any, to worry about.
         Reviewed by Maciej.
 
         <rdar://problem/5850387> Showing bookmarks reloads Google Reader instead.
-        
+
         In some cases, timeouts scheduled by a page in the onunload handler could fire even when the
         page had been unloaded and replaced by a non-HTML representation based view (such as the bookmarks view).
-        
+
         Fix this by clearing timeouts for pages that aren't cached when the provisional load is committed.
-        
+
         * bindings/js/JSDOMWindowBase.h:
         * loader/FrameLoader.cpp:
         (WebCore::FrameLoader::commitProvisionalLoad):
         (WebCore::): Added NodeMessageLevel.
         * page/Console.idl: Added console.dirxml.
         * page/inspector/Console.js: A NodeMessage creates a ElementsTreeOutline.
-        * page/inspector/ElementsPanel.js: Modified to use ElementsTreeOutline. The ElementsTreeOutline 
+        * page/inspector/ElementsPanel.js: Modified to use ElementsTreeOutline. The ElementsTreeOutline
         in the ElementsPanel has includeRootDOMNode and selectEnabled set to true.
         * page/inspector/ElementsTreeOutline.js: Added.
         (WebInspector.ElementsTreeOutline): A subclass of TreeOutline for displaying a DOM node tree.
 
         Reviewed by Dave Hyatt.
 
-        <rdar://problem/6187043> 
+        <rdar://problem/6187043>
         Don't parse full HTML user agent style sheet unless it is actually needed
-        <rdar://problem/6131889> 
+        <rdar://problem/6131889>
         WebView is significantly more expensive to create recently
-        
+
         Parsing the html4.css takes significant amount of time and memory (~50kb) on application
         startup. Some clients may never use most of the rules.
-        
-        With this patch we use simplified UA stylesheet until we hit something it can't handle. 
+
+        With this patch we use simplified UA stylesheet until we hit something it can't handle.
         This avoids full stylesheet parsing on application startup (due to empty document construction)
-        and also makes it possible for clients with very simple demands (divs and spans only) never to load 
+        and also makes it possible for clients with very simple demands (divs and spans only) never to load
         the full style.
-        
+
         It also delays view source style parsing until it is used.
 
         * css/CSSStyleSelector.cpp:
         Fix <rdar://problem/6201644>
         https://bugs.webkit.org/show_bug.cgi?id=20493
         Crash after OK in dialog box and reloading page in secure mode
-        
+
         Limited loader only fix since the general timer fix is causing problems on Windows.
 
         * loader/loader.cpp:
 2008-09-05  Antti Koivisto  <antti@apple.com>
 
         Build fixes.
-        
+
         * WebCore.xcodeproj/project.pbxproj:
         * svg/graphics/filters/SVGFEImage.cpp:
         (WebCore::FEImage::cachedImage):
         Implement HTTP 1.1 "Specific end-to-end revalidation" for WebCore memory cache. This patch does
         not yet enable it for the biggest use case, reloading. However it is good for general browsing as
         well. Doing this in WebCore level as opposed to relying on disk cache has big benefit that
-        we avoid re-decoding resources, especially images. 
-        
+        we avoid re-decoding resources, especially images.
+
         To be exact the enabled case is not actually the "Specific end-to-end revalidation" since it does not include
         CacheControl: max-age=0 header. That would be added in reload case.
-        
+
         The approach for revalidation is to kick the original resource out from the memory cache
         and create a new CachedResource that represents the revalidation request. In case
-        we get 304 back for the request we put the original resource back to the cache, update 
+        we get 304 back for the request we put the original resource back to the cache, update
         its expiration date and switch the clients registered to revalidation resource to be
         clients of the original resource.
-        
+
         All heap allocated CachedImage pointers now use CachedResourceHandle<CachedImage> (and so on) instead.
         This allows updating the handles to point to the original resource when the revalidation succeeds. It
         also acts as refcounting smart pointer.
 
         <rdar://problem/6180236> - Safari times out connections after 1 or 2 minutes
 
-        A 60-second default timeout was added in http://trac.webkit.org/changeset/17144 in an attempt 
-        to model default NSURLRequest behavior in a cross-platform manner.  
+        A 60-second default timeout was added in http://trac.webkit.org/changeset/17144 in an attempt
+        to model default NSURLRequest behavior in a cross-platform manner.
 
-        Sadly by always enforcing this 60 second timeout, WebCore was stomping over the wishes of any Webkit 
+        Sadly by always enforcing this 60 second timeout, WebCore was stomping over the wishes of any Webkit
         client that wished to enforce a much larger default timeout using NSURLRequest API.
 
         Additionally, upon reviewing what all other browsers do, it seems apparent that "no limit" is desirable
         (WebCore::ResourceRequestBase::ResourceRequestBase):  Rename the constant to "unspecifiedTimeoutInterval"
           and make it UINT_MAX so platforms that do set it have an effective "no timeout."  (Windows, for example)
         * platform/network/mac/ResourceRequestMac.mm:
-        (WebCore::ResourceRequest::doUpdatePlatformRequest):  If the timeout for this request is 
+        (WebCore::ResourceRequest::doUpdatePlatformRequest):  If the timeout for this request is
           "unspecifiedTimeoutInterval", then don't bother setting the timeout using NSURLRequest API, allowing
           WebKit applications to enforce their own default timeout.
 
 
         * rendering/RenderButton.cpp:
         (WebCore::RenderButton::setStyle): Reset the inner block's style box
-        flex to 0 to avoid getting a spurious layout hint. 
+        flex to 0 to avoid getting a spurious layout hint.
 
 2008-09-04  Kevin Ollivier  <kevino@theolliviers.com>
 
         Reviewed by Simon.
 
         Re-enable support for user stylesheets in QtWebKit
-        
+
         QtWebKit now follows the FRAME_LOADS_USER_STYLESHEET
         code path, which allows us to keep API support for
         loading user style sheets from remote URLs.
-        
+
         As part of the change UserStyleSheetLoader.cpp/h was
         moved from WebCore/loader/mac to WebCore/loader.
 
         Remove the rest of the "zombie" code from the profiler.
         - There is no longer a need for the ProfilerClient callback mechanism.
 
-        * page/Console.cpp: 
+        * page/Console.cpp:
         (WebCore::Console::Console):
         (WebCore::Console::profile):
         (WebCore::Console::profileEnd): Move the variables from the header to
         style="color:#gray;" would apply the color gray when it should not.
 
         In the bison template, "hexcolor" was defined as both "HEX maybe_space" OR "IDENT maybe_space".
-        This caused identifiers not fitting the appropriate hex format but preceded by a '#' to be 
+        This caused identifiers not fitting the appropriate hex format but preceded by a '#' to be
         interpreted as a valid color (CSSPrimitiveValue::CSS_PARSER_HEXCOLOR), when it was really just
         an ignorable token.
 
 
         AnimationController.cpp should be split into separate files
         https://bugs.webkit.org/show_bug.cgi?id=20604
-        
+
         Note: All makefiles, except WebCore.xcodeproj have been changed without testing, upon
         recommendation of Dave Hyatt.
 
         * WebCore.xcodeproj/project.pbxproj:
         * WebCoreSources.bkl:
             Build files.
-        
+
         * page/AnimationController.cpp: Removed.
         * page/AnimationController.h: Removed.
         * page/animation: Added.
 
         Fix for <rdar://problem/6181588>
 
-        This patch makes hit testing take into account the new concept of a 
-        disconnected frame, in which some of the content may not be 
-        visible. The current hit testing mechanism starts at a target frame 
-        and drills down for a HitTestResult. In some cases, drilling down 
-        will find a non-visible result. When this happens, we need to try 
-        again, starting at a higher level -- namely, starting at the main 
+        This patch makes hit testing take into account the new concept of a
+        disconnected frame, in which some of the content may not be
+        visible. The current hit testing mechanism starts at a target frame
+        and drills down for a HitTestResult. In some cases, drilling down
+        will find a non-visible result. When this happens, we need to try
+        again, starting at a higher level -- namely, starting at the main
         frame.
 
         * editing/Editor.cpp:
-        (WebCore::Editor::insideVisibleArea): New function that tests if a 
+        (WebCore::Editor::insideVisibleArea): New function that tests if a
         point is inside the visible area for a disconnected frame.
         * editing/Editor.h:
         * page/EventHandler.cpp:
 
 2008-08-29  Adele Peterson  <adele@apple.com>
 
-        Reviewed by Adam Roben. 
+        Reviewed by Adam Roben.
 
         Rename HTMLTextFieldInnerElement.h/.cpp to TextControlInnerElements.h/.cpp
 
         Fix GeneratedImage to respect Image's refcounting
         Fixing potential crashers (future if not current)
         https://bugs.webkit.org/show_bug.cgi?id=20567
-        
+
         I don't know if it's possible to make the current code
         crash, thus I've not made a test.
 
 
         Fix for https://bugs.webkit.org/show_bug.cgi?id=20525
         <rdar://problem/6169301>
-        
+
         Return the size of the movie data instead of 1000.
-        
+
         Test: media/progress-event-total.html
 
         * platform/graphics/win/QTMovieWin.cpp:
 
         Make all the 'isFoo()' methods on CSSValue const,
         and fix the subclasses.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=20561
-        
+
         * css/CSSTimingFunctionValue.h:
         * css/CSSValue.h:
         (WebCore::CSSValue::isFontValue):
         <rdar://problem/6182541>
         https://bugs.webkit.org/show_bug.cgi?id=20202
         Missing http status line from the http headers.
-        
+
         Add a status line to the header string.
-        
+
         * plugins/PluginStream.cpp:
         (WebCore::PluginStream::startStream):
 
         https://bugs.webkit.org/show_bug.cgi?id=20373
 
         Changed pattern() to canvasPattern() on CanvasStyle to match canvasGradient()
-        
+
         Made Generator (aka Gradient) RefCounted so that GraphicsContext didn't
         have to store large Gradient objects in the GraphicsContextState
-        
+
         Made Pattern RefCounted for the same reason.
-        
+
         Many updates to GraphicsContext to support easier drawing with
         Patterns and Gradients.
 
         Close a leak of PausedTimeouts if the JavaScriptDebugServer was destroyed
         with timeouts paused.
         https://bugs.webkit.org/show_bug.cgi?id=20469
-        
+
         I attempted to clean up the memory management of PausedTimeouts, I'm not
         sure the solution I came up with is "cleaner", but it's in some ways
         "safer", since it no longer uses raw pointers and manual new/delete.
-        
+
         This also now prevents CachedPage from needlessly creating Window
         objects when caching pages which didn't already have one. :)
-        
+
         I also made Chrome.cpp no longer depend on the JavaScript bindings
         (aka JSDOMWindowBase.h), since there was no real reason for it to.
 
 2008-08-27  Mario Bensi  <mbensi@pleyo.com>
 
         Reviewed by Eric Seidel.
-        
+
         Fix the data content of an image with a base64.
 
         * platform/network/curl/ResourceHandleManager.cpp:
 
         Crash after OK in dialog box and reloading page in secure mode
         https://bugs.webkit.org/show_bug.cgi?id=20493
-        
-        The new run loop spawned by a modal dialog causes a timer in the loader to run 
+
+        The new run loop spawned by a modal dialog causes a timer in the loader to run
         synchronously inside didFinishLoading() deleting "this" object.
-        
+
         Defer all WebCore timers when a modal dialog is up. They are not
         safe to execute.
 
         Reviewed by Darin Adler.
 
         Change canvas tainting logic to ask the image if it contains
-        any resources outside of its own origin. Since there is no 
+        any resources outside of its own origin. Since there is no
         way to determine if SVG images contain any resources outside its
         origin, we always assume it does.
 
         Reviewed by Mark.
 
         <rdar://problem/6149787> crash @ com.apple.JavaScriptCore: WTF::callOnMainThread + 37.
-        
-        Initialize threading in the call to WebCoreObjCFinalizeOnMainThread. We currently assume 
+
+        Initialize threading in the call to WebCoreObjCFinalizeOnMainThread. We currently assume
         (safely) that any class that needs finalization on the main thread will also need to be deallocated
         on the main thread, and calling initializeThreading from WebCoreObjCFinalizeOnMainThread instead of
         calling it from WebCoreObjCScheduleDeallocateOnMainThread seems safer.
 
         * ForwardingHeaders/VM: Added.
         * ForwardingHeaders/VM/Machine.h: Added.
-        * page/Console.cpp: Gather the line number and file information when 
+        * page/Console.cpp: Gather the line number and file information when
         profileEnd has been called, but don't use it until didFinishProfiling is
         called.  We won't need to wait once we remove the profiler "zombie" mode
         which this patch helps pave the foundation for.
 2008-08-24  Steve Falkenburg  <sfalken@apple.com>
 
         Add a "last chance" WM_TIMER to the Windows shared timer.
-        
+
         The last chance timer fires every 5 seconds to run any lost WM_TIMER based timers.
         Failure to fire a timer is fatal to the cross-platform Timer code, since it won't re-schedule
         timers if a timer with an earlier expiration is already pending. This results in no timers
         firing from that point on.
-        
+
         We lose WM_TIMER messages occasionally (in the neighborhood of 1 per hour) probably due to a
         buggy window message hook.
-        
+
         This timer will start when the first WM_TIMER is scheduled, and will
         fire every 5 seconds thereafter, causing any lost timers to be fired.
-        
+
         Found this via one of its symptoms: leaking WebFrames. The fix prevents timers from stalling
         and prevents the WebFrame leak.
-        
+
         Reviewed by Darin Adler, Geoff Garen.
 
         * platform/win/SharedTimerWin.cpp:
 
         Test: fast/events/autoscroll-nonscrollable-iframe-in-scrollable-div.html
         https://bugs.webkit.org/show_bug.cgi?id=20451
-   
+
         rdar://problem/6166435 Inspector doesn't auto scroll when selecting text (20451)
         When we climb up the rendering tree looking for a scrollable renderer, we need to be able to jump outside of an iframe.
         This way we can see if what is embedding the iframe can be scrolled even if the iframe content can't.
 
         Fix a race condition in Windows timer code.
         Timer function could end up being called with a 0 timer, leading to a Windows exception.
-        
+
         Don't post a timer message if one is already pending.
-        
+
         Reviewed by Ada Chan.
 
         * platform/win/SharedTimerWin.cpp:
 
         <rdar://problem/6153432> - Many images broken in Mail
 
-        This can be traced back to the preload scanner.  With that change, CachedResources are created a lot 
-        sooner than before and confuse the WebArchive machinery.  
+        This can be traced back to the preload scanner.  With that change, CachedResources are created a lot
+        sooner than before and confuse the WebArchive machinery.
 
-        When referencing WebArchive subresources directly through the WebKit API it is appropriate to ignore 
-        such CachedResources since they are placeholders and have not been submitted to the ResourceLoadDelegate 
+        When referencing WebArchive subresources directly through the WebKit API it is appropriate to ignore
+        such CachedResources since they are placeholders and have not been submitted to the ResourceLoadDelegate
         machinery and nothing is known about where the data will eventually come from.
 
         * loader/DocumentLoader.cpp:
-        (WebCore::DocumentLoader::subresource):  Ignore a CachedResource if its preloadResult is 
+        (WebCore::DocumentLoader::subresource):  Ignore a CachedResource if its preloadResult is
           CachedResource::PreloadReferenced.
 
 2008-08-20  Holger Hans Peter Freyther  <zecke@selfish.org>
         getComputedStyle() for webkitTransform should return a transform
         that does not have the transform origin baked into it.
         https://bugs.webkit.org/show_bug.cgi?id=20464
-        
+
         Test: fast/css/getComputedStyle-transform.html
 
         * css/CSSComputedStyleDeclaration.cpp:
 
         Reviewed and landed by Anders.
 
-        <rdar://problem/6163636> 
+        <rdar://problem/6163636>
         rename NPCocoaEvent's "event" struct to "data" (20446)
-        
+
         * bridge/npapi.h:
         (_NPCocoaEvent::):
 
 
         Reviewed by Darin Adler.
 
-        Fix for <rdar://problem/6145626> 
-        This patch fixes a number of remaining problems getting 
-        disconnected frames to work correctly with markAllMatchesForText() 
-        and findString(). Details inline. 
+        Fix for <rdar://problem/6145626>
+        This patch fixes a number of remaining problems getting
+        disconnected frames to work correctly with markAllMatchesForText()
+        and findString(). Details inline.
 
-        This was a static helper function in Frame, but this patch requires 
-        the same functionality in Editor, so I just added it as a function 
+        This was a static helper function in Frame, but this patch requires
+        the same functionality in Editor, so I just added it as a function
         on Node.
         * dom/Node.cpp:
         (WebCore::Node::isInShadowTree):
         * dom/Node.h:
 
         * editing/Editor.cpp:
-        (WebCore::Editor::insideVisibleArea): Returns false if 
+        (WebCore::Editor::insideVisibleArea): Returns false if
         excludeFromTextSearch() is true.
 
-        In a normal (non-disconnected) frame, findString returns a range of 
-        the document node if the text is not found in the frame. I changed 
-        firstVisibleRange and lastVisibleRange to match that behavior when 
+        In a normal (non-disconnected) frame, findString returns a range of
+        the document node if the text is not found in the frame. I changed
+        firstVisibleRange and lastVisibleRange to match that behavior when
         the text is not found
         (WebCore::Editor::firstVisibleRange):
         (WebCore::Editor::lastVisibleRange):
 
-        Here are the bulk of the changes in the patch. A lot of text was 
-        not being found in disconnected frames because I failed to account 
-        for all of the possible problems associated with shadow trees. That 
+        Here are the bulk of the changes in the patch. A lot of text was
+        not being found in disconnected frames because I failed to account
+        for all of the possible problems associated with shadow trees. That
         is fixed here.
         (WebCore::Editor::nextVisibleRange):
         * editing/Editor.h:
 
-        excludeFromTextSearch() is new. It allows a WebKit client to mark a 
-        frame as not-text-searchable through SPI. 
+        excludeFromTextSearch() is new. It allows a WebKit client to mark a
+        frame as not-text-searchable through SPI.
         * WebCore.base.exp:
         * page/Frame.cpp:
         (WebCore::Frame::excludeFromTextSearch):
         * page/FramePrivate.h:
 
         (WebCore::Frame::findString):
-        (WebCore::Frame::markAllMatchesForText):  I kept running into an 
-        assertion failure in paining code because of the forced paint on 
+        (WebCore::Frame::markAllMatchesForText):  I kept running into an
+        assertion failure in paining code because of the forced paint on
         empty visible rects.
 
 2008-08-20  Timothy Hatcher  <timothy@apple.com>
         Reviewed by Beth Dakin.
 
         <rdar://problem/4003764> Expose tables as AXTables
+
         Tests: accessibility/table-attributes.html
                accessibility/table-cell-spans.html
                accessibility/table-cells.html
         * rendering/RenderBox.h:
         * rendering/RenderImage.cpp:
         (WebCore::RenderImage::calcReplacedWidth):
-        (WebCore::RenderImage::calcPrefWidths): 
+        (WebCore::RenderImage::calcPrefWidths):
         * rendering/RenderImage.h:
         * rendering/RenderReplaced.cpp:
         (WebCore::RenderReplaced::calcPrefWidths):
 2008-08-17  Geoffrey Garen  <ggaren@apple.com>
 
         Reviewed by Cameron Zwarich.
-        
+
         Updated project files to XCode 3.1.
 
         * manual-tests/NPN_Invoke/NPN_Invoke.xcodeproj/project.pbxproj:
         Reviewed by Cameron Zwarich.
 
         Made room for a free word in JSCell.
-        
+
         Changed JSDOMWindowBase to store its auxiliary data in a subclass of
         JSGlobalData, so the two could share a pointer.
-        
+
         Added a bunch of ASSERTs, to help catch over-sized objects.
 
 2008-08-15  Mark Rowe  <mrowe@apple.com>
         Don't start preloading body resources before the head is complete. This prevents
         body preloads from slowing down initial display when there is limited amount
         of bandwidth available.
-        
+
         Works by queuing up found body preloads to DocLoader and only issuing them
         after document has rendering.
-        
+
         With bandwidth capped to 300kbit/s this speeds up cnn.com initial display by ~25% or 5s
         without affecting complete load time.
 
 
         Some loader performance tweaks:
         - Make stylesheets highest priority instead of scripts. We block script execution on stylesheets.
-          Especially if a stylesheet @imports other stylesheets it is important to get them to the front of the queue 
+          Especially if a stylesheet @imports other stylesheets it is important to get them to the front of the queue
           to not delay rendering.
         - Issue the first resource load for a host immediately even if the resource is low priority. TCP connection setup
           can take long time when latency is high so it is good to get started early.
 
         - fix <rdar://problem/5993323> REGRESSION (r34210): Apple.com favicon appears stretched/clipped
 
-        * platform/graphics/BitmapImage.cpp: 
+        * platform/graphics/BitmapImage.cpp:
         (WebCore::BitmapImage::BitmapImage): Added initialization of
         m_hasUniformFrameSize.
         (WebCore::BitmapImage::cacheFrame): Added code to get the size of the
         * platform/graphics/ImageSource.h: Added frameSizeAtIndex().
         * platform/graphics/cg/ImageCG.cpp:
         (WebCore::BitmapImage::draw): Changed to use currentFrameSize(). This
-        fixes the bug, which resulted from assuming that the frame being drawn 
+        fixes the bug, which resulted from assuming that the frame being drawn
         was the same size as the first frame.
         * platform/graphics/cg/ImageSourceCG.cpp:
         (WebCore::ImageSource::frameSizeAtIndex): Renamed size() to this and
         Tests: media/video-source-type-params.html
 
         * html/HTMLMediaElement.cpp:
-        (WebCore::HTMLMediaElement::pickMedia): only pass the portion before the first ';' 
+        (WebCore::HTMLMediaElement::pickMedia): only pass the portion before the first ';'
         to isSupportedMediaMIMEType()
 
 2008-08-13  Beth Dakin  <bdakin@apple.com>
 
         Fix for <rdar://problem/6141345>
 
-        This patch refines findString and markAllMatchesForText functions' 
-        interactions with disconnected frames. They no longer rely on 
-        knowing where a range is relative to the visible region and work 
+        This patch refines findString and markAllMatchesForText functions'
+        interactions with disconnected frames. They no longer rely on
+        knowing where a range is relative to the visible region and work
         with disconnected frames that contain frames.
 
         * editing/Editor.cpp:
-        (WebCore::Editor::insideVisibleArea): Now returns a bool instead of 
+        (WebCore::Editor::insideVisibleArea): Now returns a bool instead of
         the visiblity enum.
-        (WebCore::Editor::firstVisibleRange): This now returns the very 
-        first visible range in the document. It's no longer dependent on 
+        (WebCore::Editor::firstVisibleRange): This now returns the very
+        first visible range in the document. It's no longer dependent on
         searching forward.
-        (WebCore::Editor::lastVisibleRange): This now returns the very last 
-        visible range in the document. It's no longer dependent on 
+        (WebCore::Editor::lastVisibleRange): This now returns the very last
+        visible range in the document. It's no longer dependent on
         searching backwards.
-        (WebCore::Editor::nextVisibleRange): This returns the next visible 
+        (WebCore::Editor::nextVisibleRange): This returns the next visible
         range in the appropriate direction from the current range.
         * editing/Editor.h:
         * page/Frame.cpp:
 2008-08-12  Nikolas Zimmermann  <zimmermann@kde.org>
 
         Reviewed by Dave.
-    
+
         Fixes: https://bugs.webkit.org/show_bug.cgi?id=19798
         Masks are translated, and the mask images are swapped on the y-axis.
 
         - Because console messages have group levels now, newly created messages
         that do not specify the level lose their message since the number of
         arguments is wrong.
+
         * page/inspector/Console.js:
         * page/inspector/Resource.js:
 
         * html/HTMLElement.cpp:
         * page/AnimationController.cpp:
             do all the new event stuff
-            
+
         * html/HTMLAttributeNames.in:
             the onwebkitanimation* and onwebkittransitionend attrs
 
index e6f502d..e57e085 100644 (file)
@@ -1796,31 +1796,6 @@ webcoregtk_sources += \
        WebCore/platform/image-decoders/xbm/XBMImageDecoder.h \
        WebCore/platform/text/gtk/TextBreakIteratorInternalICUGtk.cpp
 
-# ----
-# curl http backend
-# ----
-if USE_CURL
-
-webcore_cppflags += \
-       -DWTF_USE_CURL=1 \
-       -I$(srcdir)/WebCore/platform/network/curl
-
-webcore_sources += \
-       WebCore/platform/network/curl/AuthenticationChallenge.h \
-       WebCore/platform/network/curl/CookieJarCurl.cpp \
-       WebCore/platform/network/curl/DNSCurl.cpp \
-       WebCore/platform/network/curl/FormDataStreamCurl.cpp \
-       WebCore/platform/network/curl/FormDataStreamCurl.h \
-       WebCore/platform/network/curl/ResourceError.h \
-       WebCore/platform/network/curl/ResourceHandleCurl.cpp \
-       WebCore/platform/network/curl/ResourceHandleManager.cpp \
-       WebCore/platform/network/curl/ResourceHandleManager.h \
-       WebCore/platform/network/curl/ResourceRequest.h \
-       WebCore/platform/network/curl/ResourceResponse.h
-
-endif # END USE_CURL
-
-if USE_SOUP
 webcore_cppflags += \
        -DWTF_USE_SOUP=1 \
        -I$(srcdir)/WebCore/platform/network/soup
@@ -1834,7 +1809,6 @@ webcore_sources += \
        WebCore/platform/network/soup/ResourceHandleSoup.cpp \
        WebCore/platform/network/soup/ResourceRequest.h \
        WebCore/platform/network/soup/ResourceResponse.h
-endif
 
 # ---
 # Freetype font backend
index 612440e..d131e01 100644 (file)
@@ -1,3 +1,14 @@
+2009-02-23  Xan Lopez  <xan@gnome.org>
+
+        Reviewed by Alexey Proskuryakov.
+
+        https://bugs.webkit.org/show_bug.cgi?id=22624
+        [SOUP][GTK] Need API to get SoupSession from WebKit.
+
+        Add soup dependency to pc file.
+
+        * webkit.pc.in:
+
 2009-02-23  Jan Michael Alonzo  <jmalonzo@webkit.org>
 
         Reviewed by Mark Rowe.
         * webkit/webkitwebframe.cpp:
         * webkit/webkitwebview.cpp:
 
-2009-02-05  Aaron Boodman <aa@chromium.org> 
+2009-02-05  Aaron Boodman <aa@chromium.org>
 
         Reviewed by Dave Hyatt.
 
 2009-02-02  Geoffrey Garen  <ggaren@apple.com>
 
         Reviewed by Sam Weinig.
-        
+
         Track redirects in global history.
 
         Keep GTK building.
 2009-02-02  Anders Carlsson  <andersca@apple.com>
 
         Build fix.
-        
+
         * WebCoreSupport/FrameLoaderClientGtk.h:
 
 2009-02-02  Anders Carlsson  <andersca@apple.com>
         Reviewed by Darin Adler.
 
         [GTK] Fix the reference counting of WebKitWebFrames
-        
+
         The ownership is the following: WebKitWebView owns a WebCore::Page.
         WebKitWebView is creating one WebKitWebFrame which will be the
         mainFrame of the WebCore::Page (having the reference on the Frame).
-        
+
         The FrameLoaderClient has the reference of the WebKitWebFrame for
         the main frame and also any other frame. This means when the
         WebCore::Frame goes away the FrameLoaderClient will go away which
         will normally remove the last reference of the WebKitWebFrame. Because
         an API user might have g_object_ref'ed the WebKitWebFrame null
         checks had to be added to WebKitWebFrame.
-        
+
         For WebCore::Frames created by the FrameLoaderClient the ownership
         will be passed down to the FrameTree, the WebKitWebFrame is not holding
         a reference to the WebCore::Frame.
-        
+
         Do not g_object_unref the mainFrame in the destructor of the
         WebKitWebFrame as this will happen from within the WebCore::Page
         destruction. Do not hold a reference to the WebCore::Frame (circle) in
         WebKitWebFrame, add null checks as the WebCore::Frame might have gone
         away. Do not keep track of the FrameLoaderClient in the private
         structures as it was mostly unusued.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=21837
 
         * WebCoreSupport/FrameLoaderClientGtk.cpp:
 
         Renderer::caretRect() is now localCaretRect(), which needs
         converting to absolute coordinates (taking transforms into account).
-        
+
         * webkit/webkitwebview.cpp:
 
 2008-12-01  Xan Lopez  <xan@gnome.org>
 2008-10-06  David Hyatt  <hyatt@apple.com>
 
         Enable viewless Mac WebKit to paint some basic pages.
-        
+
         Reviewed by Sam Weinig
 
         * WebCoreSupport/FrameLoaderClientGtk.cpp:
 2008-10-03  David Hyatt  <hyatt@apple.com>
 
         https://bugs.webkit.org/show_bug.cgi?id=21340
-        
+
         Remove "containingWindow()/setContainingWindow()" from Widget.  HostWindow covers this now.
 
         Reviewed by Dan Bernstein & Darin Adler
-        
+
         * WebCoreSupport/FrameLoaderClientGtk.cpp:
         (WebKit::FrameLoaderClient::transitionToCommittedForNewPage):
 
 2008-10-03  David Hyatt  <hyatt@apple.com>
 
          Remove addToDirtyRegion.
-        
+
         Reviewed by Oliver Hunt
 
         * WebCoreSupport/ChromeClientGtk.cpp:
 2008-10-02  David Hyatt  <hyatt@apple.com>
 
         https://bugs.webkit.org/show_bug.cgi?id=21314
-        
+
         Make scrollBackingStore cross-platform.
 
         Reviewed by Sam Weinig
 2008-10-01  David Hyatt  <hyatt@apple.com>
 
         https://bugs.webkit.org/show_bug.cgi?id=21282
-        
+
         Make contentsToScreen/screenToContents cross-platform.  Only implemented by Mac/Win right now.
-        
+
         Reviewed by Adam Roben
 
         * WebCoreSupport/ChromeClientGtk.cpp:
 2008-09-30  Dave Hyatt  <hyatt@apple.com>
 
         http://bugs.webkit.org/show_bug.cgi?id=21250
-        
+
         Rename updateContents to repaintContentRectangle and make it cross-platform by always sending
         repaints up through the ChromeClient.
 
 
         http://bugs.webkit.org/show_bug.cgi?id=16802
         [GTK] Missing gtk properties
-        
+
         Add missing properties to WebKitViewFrame and WebKitWebView.
-        
+
         * WebView/webkitprivate.h: add some useful defines for param specs
         * WebView/webkitwebframe.cpp:
         (webkit_web_frame_get_property): added
         http://bugs.webkit.org/show_bug.cgi?id=16654
         [GTK] Signal "navigation-requested" does not react correctly on
         return TRUE from callbacks
-        
+
         * WebView/webkitwebview.cpp: use our own accumulator for signals
         returning WebKitNavigationResponse. The emission will be stopped
         when any callback returns anything but
         * WebView/webkitwebview.cpp:
         * WebView/webkitwebview.h:
 
-2007-12-19  Christian Dywan  <christian@twotoasts.de> 
+2007-12-19  Christian Dywan  <christian@twotoasts.de>
 
         Reviewed by Alp Toker.
 
         Reviewed by Holger Freyther.
 
         Delete when Destroy functions are called to avoid leaks
-        
+
         This matches the Mac port.
 
         * WebCoreSupport/ChromeClientGtk.cpp:
 
         * WebView/webkitprivate.h:
         (getFrameFromView): removed
-        
+
         * WebView/webkitwebview.cpp:
         (webkit_web_view_expose_event):
         (webkit_web_view_key_press_event): use focused frame
 
         http://bugs.webkit.org/show_bug.cgi?id=15745
         [GTK] Arrow keys do not Scroll
-        
+
         * Api/webkitgtkpage.cpp:
 
         Support Up/Down/Right/Left keys to scroll. Slight hack, see FIXME for
 2007-09-05  Geoffrey Garen  <ggaren@apple.com>
 
         Reviewed by Darin Adler, Maciej Stachowiak, Mark Rowe, Tim Hatcher.
-        
-        Fixed <rdar://problem/5326009> Make non-browser WebKit clients have no 
+
+        Fixed <rdar://problem/5326009> Make non-browser WebKit clients have no
         memory cache, or a very tiny one
-        
+
         Keep the GTK build working with an empty stub.
-        
+
         * WebCoreSupport/FrameLoaderClientGtk.cpp:
         (WebKit::FrameLoaderClient::didPerformFirstNavigation):
         * WebCoreSupport/FrameLoaderClientGtk.h:
         Reviewed by Sam.
 
         <rdar://problem/5344848> IME is incorrectly used for key events when on non-editable regions
+
         EditorClient::setInputMethodState stub
 
         * WebCoreSupport/EditorClientGtk.cpp:
 2007-08-17  Anders Carlsson  <andersca@apple.com>
 
         Build fix.
-        
+
         * WebCoreSupport/FrameLoaderClientGtk.cpp:
         (WebKit::FrameLoaderClient::createPlugin):
         * WebCoreSupport/FrameLoaderClientGtk.h:
 
         Minor changes to the WebKit::EditorClient to allow removing
         of text from TextFields. Remove the selectWordBeforeMenuEvent method
-        which is not used and not within WebCore::EditorClient. 
+        which is not used and not within WebCore::EditorClient.
 
         * WebCoreSupport/EditorClientGtk.cpp:
         (WebKit::EditorClient::shouldDeleteRange):
         classes into the WebKit namespace. Change the class names to not contain Gtk.
         The file names have to contain the Gtk suffix to not clash with files in WebCore (e.g.
         bridge/EditorClient.h).
-        
+
 
         * Api/webkitgtkframe.cpp:
         * Api/webkitgtkpage.cpp:
         GtkAdjustment of the GtkLayout and reimplement the set_scroll_adjustments method and pass
         the GtkAdjustments to ScrollView.
         This makes having one GdkWindow for the complete FrameTree possible.
-        
+
 
         * gtk/Api/webkitgtkframe.cpp:
         * gtk/Api/webkitgtkpage.cpp:
         * gtk/Api/webkitgtkpage.cpp: Implement webkit_gtk_page_can_go_backward() and
         webkit_gtk_page_can_go_forward() functions.
 
-2007-07-30  Diego Escalante Urrelo  <diegoe@gnome.org> 
+2007-07-30  Diego Escalante Urrelo  <diegoe@gnome.org>
 
         Reviewed by Alp Toker.
 
index 0ce6f71..84d6115 100644 (file)
@@ -6,6 +6,6 @@ includedir=@includedir@
 Name: WebKit
 Description: Web content engine for GTK+
 Version: @VERSION@
-Requires: gtk+-2.0
+Requires: gtk+-2.0 libsoup-2.4
 Libs: -L${libdir} -lwebkit-1.0
 Cflags: -I${includedir}/webkit-1.0
index 1c7f927..b5074b4 100644 (file)
@@ -1,3 +1,14 @@
+2009-02-23  Xan Lopez  <xan@gnome.org>
+
+        Reviewed by Alexey Proskuryakov.
+
+        https://bugs.webkit.org/show_bug.cgi?id=22624
+        [SOUP][GTK] Need API to get SoupSession from WebKit.
+
+        Add soup flags now that dependency is explicit.
+
+        * GNUmakefile.am:
+
 2009-02-22  Mark Rowe  <mrowe@apple.com>
 
         Reviewed by Oliver Hunt.
@@ -10,7 +21,7 @@
 
 2009-02-13  Eric Seidel  <eric@webkit.org>
 
-        Rubber-stamped by Alexey Proskuryakov. 
+        Rubber-stamped by Alexey Proskuryakov.
 
         Add a few ignores to make-js-test-wrappers.
 
@@ -37,7 +48,7 @@
 2009-02-12  Simon Fraser  <simon.fraser@apple.com>
 
         No review.
-        
+
         Remove debugging code which was committed by mistake.
 
         * Scripts/run-webkit-tests:
@@ -47,7 +58,7 @@
         Reviewed by Adam Roben
 
         https://bugs.webkit.org/show_bug.cgi?id=23928
-        
+
         Add detection of accelerated compositing and 3d transforms,
         and add various directories to $ignoredDirectories when these
         features are off.
 2009-01-28  Geoffrey Garen  <ggaren@apple.com>
 
         Reviewed by Sam Weinig.
-        
+
         Added support for querying how many history items were created during a
         layout test.
 
 2009-01-22  Anders Carlsson  <andersca@apple.com>
 
         Fix Windows build.
-        
+
         * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.h:
 
 2009-01-22  Anders Carlsson  <andersca@apple.com>
 
         Reviewed by Sam Weinig.
-        
+
         Clean up the test plug-in code. We now always use the CG drawing model and the
         Cocoa event model. It is however possible to revert to the old Carbon event model by
         specifying forcecarbon=true in the embed/object tag.
-        
+
         * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.h:
         * DumpRenderTree/TestNetscapePlugIn.subproj/main.cpp:
         (NPP_New):
         Reviewed by Adam Roben.
 
         Set the count to the right number of elements.
-        
+
         * DumpRenderTree/TestNetscapePlugIn.subproj/TestObject.cpp:
         (testEnumerate):
 
 2009-01-11  Robert Blaut  <webkit@blaut.biz>
 
         Reviewed by Eric Seidel.
-        
+
         <https://bugs.webkit.org/show_bug.cgi?id=23134>
         Update bisect-builds for Safari 3.2 to prevent crashes
 
 2008-12-10  Brady Eidson  <beidson@apple.com>
 
         Reviewed by Darin Adler.
-        
-        Implement the new policy delegate (including navigation type and permissive mode) for DRT/win 
+
+        Implement the new policy delegate (including navigation type and permissive mode) for DRT/win
 
         * DumpRenderTree/win/DumpRenderTree.cpp:
         * DumpRenderTree/win/DumpRenderTreeWin.h:
-        
+
         * DumpRenderTree/win/LayoutTestControllerWin.cpp:
         (LayoutTestController::setCustomPolicyDelegate):
-        
+
         * DumpRenderTree/win/PolicyDelegate.cpp:
         (PolicyDelegate::PolicyDelegate):
         (PolicyDelegate::decidePolicyForNavigationAction):
 2008-12-08  Geoffrey Garen  <ggaren@apple.com>
 
         Reviewed by Oliver Hunt.
-        
+
         Added a shortcut for --jsDriver-args, which I use a lot.
 
         * Scripts/run-javascriptcore-tests:
 2008-12-05  Eric Seidel  <eric@webkit.org>
 
         Reviewed by Geoff Garen.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=22683
         Fix gtk and qt builds which depend on --qt and --gtk being removed from ARGV
         Add a new argumentsForConfiguration() function and clean up some old code to use it.
 2008-12-03  Eric Seidel  <eric@webkit.org>
 
         Build fix only, no review.
-        
+
         Remove support for build-webkit --svg-experimental.
         All of the "experimental" svg features have their own toggles anyway.
         I broke --svg-experimental in my last commit (which then broke clean builds)
 2008-12-02  Eric Seidel  <eric@webkit.org>
 
         Reviewed by David Hyatt (and Mark Rowe).
-        
+
         Move --coverage support from build-webkit to webkitdirs.pm to share it with build-jsc
         Move --coverage support out of run-javascriptcore-tests and into build-jsc
-        
+
         Finally add a buildXCodeProject function to webkitdirs.pm and move --clean support
         there from build-webkit (to allow future sharing with other build-* scripts)
-        
+
         Change run-javascriptcore-tests to expect a --jsDriver-args= argument instead of
         picking through ARGV with a blacklist of what arguments weren't jsDriver args
         this makes run-javascriptcore-tests transparently support all arguments which
         webkitdirs.pm gives it support for.
-        
+
         Make run-javascriptcore-tests actually print what commands it's running before running them.
-        
+
         Add --help support to build-jsc and run-javascriptcore-tests!
-        
+
         Make code to support --svg-experimental defaults take up half as many lines.
 
         * Scripts/build-jsc:
         this latest round of errors, at least make build-webkit --chromium
         actually "work" under CYGWIN instead of forcing users to use
         a cmd shell directly.
-        
+
         This is working around a Scons bug:
         http://scons.tigris.org/issues/show_bug.cgi?id=2266
 
 2008-11-26  Eric Seidel  <eric@webkit.org>
 
         Reviewed by Nikolas Zimmermann and Mark Rowe.
-        
+
         Add support for build-webkit --chromium
         https://bugs.webkit.org/show_bug.cgi?id=22515
         More cleanups changing isCygwin calls to isAppleWinWebKit and deploying isAppleWebKit where necessary.
         Reviewed by Darin Adler
 
         https://bugs.webkit.org/show_bug.cgi?id=22433
-        
+
         Add script that attempts to detect virtual methods
         whose signatures differ only by constness (which can
         indicate a programming error).
-        
+
         * Scripts/detect-mismatched-virtual-const: Added.
 
 2008-11-24  Zan Dobersek  <zandobersek@gmail.com>
 
         All static C++ objects that create atexit calls are gone. Update script
         to indicate how to fix the problem should a new one appear.
-        
+
         * Scripts/check-for-exit-time-destructors:
 
 2008-11-18  Alexey Proskuryakov  <ap@webkit.org>
 2008-11-17  Geoffrey Garen  <ggaren@apple.com>
 
         Not reviewed.
-        
+
         Try to fix gtk build.
 
         * DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:
 
         Remove .o files from the exclude list that have had their exit-time destructors removed.
         Handle case when there is no current symbol.
-        
+
         * Scripts/check-for-exit-time-destructors:
 
 2008-11-16  Sam Weinig  <sam@webkit.org>
         https://bugs.webkit.org/show_bug.cgi?id=21810
         Remove use of static C++ objects that are destroyed at exit time (destructors)
 
-        Create DEFINE_STATIC_LOCAL macro. Change static local objects to leak to avoid 
-        exit-time destructor. Update code that was changed to fix this issue that ran 
-        into a gcc bug (<rdar://problem/6354696> Codegen issue with C++ static reference 
-        in gcc build 5465). Also typdefs for template types needed to be added in some 
+        Create DEFINE_STATIC_LOCAL macro. Change static local objects to leak to avoid
+        exit-time destructor. Update code that was changed to fix this issue that ran
+        into a gcc bug (<rdar://problem/6354696> Codegen issue with C++ static reference
+        in gcc build 5465). Also typdefs for template types needed to be added in some
         cases so the type could make it through the macro successfully.
 
         Basically code of the form:
         DEFINE_STATIC_LOCAL(T, m, ());
 
         Remove .o files from the exclude list that have had their exit-time destructors removed.
-        
+
         * Scripts/check-for-exit-time-destructors:
 
 2008-11-14  Kevin Ollivier  <kevino@theolliviers.com>
 2008-11-06  Greg Bolsinga  <bolsinga@apple.com>
 
         Reviewed by Darin Adler.
-        
+
         Bug 21810: Remove use of static C++ objects that are destroyed at exit time (destructors)
         https://bugs.webkit.org/show_bug.cgi?id=21810
-        
+
         Remove .o files from the exclude list that have had their exit-time destructors removed.
 
         * Scripts/check-for-exit-time-destructors:
 
         Add AccessibilityObject.o to the exclude list for the
         check for global destructors.
-        
+
         * Scripts/check-for-exit-time-destructors:
 
 2008-11-04  Darin Adler  <darin@apple.com>
 
 2008-10-28  Adele Peterson  <adele@apple.com>
 
-        Reviewed by Sam Weinig. 
+        Reviewed by Sam Weinig.
 
         Specify which Localizable.strings to update since we don't always want to update the file in the same
         directory where we're searching for the strings to localize.
-        
+
         * Scripts/extract-localizable-strings:
         * Scripts/update-webkit-localizable-strings:
 
 2008-10-24  Anders Carlsson  <andersca@apple.com>
 
         Try fixing the 64-bit build.
-        
+
         * DumpRenderTree/TestNetscapePlugIn.subproj/TestObject.cpp:
         (testGetProperty):
 
 2008-10-24  Anders Carlsson  <andersca@apple.com>
 
         Reviewed by Sam Weinig.
-        
+
         <rdar://problem/5440917> Support NPN_Construct
-        
+
         * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp:
         (testCallback):
         Fix a memory leak.
-        
+
         (testConstruct):
         New test method that treats it first argument as a constructor and invokes it with the rest of the arguments.
-        
+
         (pluginInvoke):
         Handle testConstruct.
-        
+
         * DumpRenderTree/TestNetscapePlugIn.subproj/TestObject.cpp:
         (testGetProperty):
         Add objectPointer property.
-        
+
         (testEnumerate):
         Only enumerate the two first properties.
-        
+
         (testConstruct):
         Add a simple construct implementation that just returns the test object.
 
 2008-10-22  Brady Eidson  <beidson@apple.com>
 
         Reviewed by Adam Roben
-        
+
         <rdar://6261773> - autocomplete="off" doesn't work on Windows
 
         Implement LayoutTestController::elementDoesAutoCompleteForElementWithId() on Windows
-        
+
         * DumpRenderTree/win/LayoutTestControllerWin.cpp:
 
 2008-10-22  Darin Adler  <darin@apple.com>
 2008-10-17  Maciej Stachowiak  <mjs@apple.com>
 
         Reviewed by Mark Rowe.
-        
+
         - default to 1000 tests per dump tool instance again, now that CTI no longer causes
         excess different stack logs.
-        
+
         (Also fix a small bug in my last change.)
 
         * Scripts/run-webkit-tests:
 2008-10-17  Maciej Stachowiak  <mjs@apple.com>
 
         Reviewed by Mark Rowe.
-        
+
         - make sure MallocStackLogging is only on for tools we want to
         leak check, not other random stuff as well.
 
 2008-10-17  Maciej Stachowiak  <mjs@apple.com>
 
         Reviewed by Stephanie Lewis.
-        
+
         - make run-webkit-tests --leaks default to 100 tests per run instead of 1000
-        
+
         This should reduce or eliminate crashes on the buildbot due to
         running out of memory while stack logging.
 
         about whether to expect PNG dumps by having DRT always
         print two blocks terminated by #EOF, the second of which
         may be empty.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=21483
-        
+
         * DumpRenderTree/cg/PixelDumpSupportCG.cpp:
         (printPNG):
         (dumpWebViewAsPixelsAndCompareWithExpected):
 
         Fix hang when running with --pixel --reset, which occurs
         because DRT spews PNG data when the script does not expect it.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=21323
-        
+
         * Scripts/run-webkit-tests:
 
 2008-09-30  Mark Rowe  <mrowe@apple.com>
 
         Don't run pixel comparison for text-only tests.
         https://bugs.webkit.org/show_bug.cgi?id=21124
-        
+
         * DumpRenderTree/mac/DumpRenderTree.mm:
         (dump):
         * Scripts/run-webkit-tests:
         Reviewed by Simon.
 
         Changed copyright from Trolltech ASA to Nokia.
-        
+
         Nokia acquired Trolltech ASA, assets were transferred on September 26th 2008.
-        
+
 
         * DumpRenderTree/qt/DumpRenderTree.cpp:
         * DumpRenderTree/qt/jsobjects.cpp:
 2008-09-27  Anders Carlsson  <andersca@apple.com>
 
         Reviewed by Sam Weinig.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=21178
         <rdar://problem/6248651>
-        
+
         Return the result value from getURL.
-        
+
         * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp:
         (getURL):
 
 
         Reviewed by Sam Weinig.
 
-        Speculative build fix. 
+        Speculative build fix.
 
         * DumpRenderTree/win/AccessibilityUIElementWin.cpp:
         (AccessibilityUIElement::supportsPressAction):
 
         Reviewed by Sam Weinig.
 
-        This patch extends DumpRenderTree's AccessibilityController to ask 
+        This patch extends DumpRenderTree's AccessibilityController to ask
         if the focused element supports the press action.
 
         * DumpRenderTree/AccessibilityUIElement.cpp:
 
         Reviewed by Oliver Hunt.
 
-        Implement a workaround for an incorrect mime-type on machines with 
+        Implement a workaround for an incorrect mime-type on machines with
         Dashcode 2.0.1.  Dashcode is overriding the UTI type for .js files.
-        See radar <rdar://problem/6234318>.     
+        See radar <rdar://problem/6234318>.
 
         * DumpRenderTree/mac/DumpRenderTree.mm:
         (convertMIMEType):
 2008-09-16  Simon Fraser  <simon.fraser@apple.com>
 
         Reviewed by Eric Seidel
-        
-        Protect against tests that call layoutTestController.notifyDone() 
+
+        Protect against tests that call layoutTestController.notifyDone()
         more than once, which would lead to memory corruption, by nulling
         out the LayoutTestController on the JSObjectRef on destruction. Also
         add a finalize callback on the class so that if the LTC outlives
         the JSObjectRef, the JSObjectRef backpointer on the LTC can be
         nulled out.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=20875
 
         * DumpRenderTree/LayoutTestController.cpp:
         Reviewed by Stephanie Lewis.
 
         * DumpRenderTree/mac/AccessibilityUIElementMac.mm:
-        Calling -[NSValue rangeValue] on nil object was returning bogus results on PPC, 
-        but not Intel.  Add a check for nil before calling. 
+        Calling -[NSValue rangeValue] on nil object was returning bogus results on PPC,
+        but not Intel.  Add a check for nil before calling.
         (AccessibilityUIElement::rowIndexRange):
         (AccessibilityUIElement::columnIndexRange):
 
 
         Reviewed by Mark Rowe.
 
-        Fix DRT build 
+        Fix DRT build
 
         * DumpRenderTree/DumpRenderTreePrefix.h:
         * DumpRenderTree/win/DumpRenderTree.vcproj:
 2008-09-06  Steve Falkenburg  <sfalken@apple.com>
 
         Fix Windows nightlies.
-        
+
         Copy WebKit.dll alongside application so registry-free COM can find it.
         Update embedded manifest to force use registry-free COM.
 
 
 2008-09-04  Kevin Ollivier  <kevino@theolliviers.com>
 
-        wx buildbot fix. More robust handling of clean when makefiles are not built or 
+        wx buildbot fix. More robust handling of clean when makefiles are not built or
         are not completely made.
 
         * wx/build-wxwebkit:
         (AccessibilityController::~AccessibilityController):
         (AccessibilityController::focusedElement): Get the root element, and
         request its focused object.
-        (AccessibilityController::rootElement): Query Windows for the 
+        (AccessibilityController::rootElement): Query Windows for the
         accessible client object for the WebView's window.
         * DumpRenderTree/win/AccessibilityUIElementWin.cpp: Added.
         (AccessibilityUIElement::AccessibilityUIElement):
 2008-08-21  Simon Fraser  <simon.fraser@apple.com>
 
         Reviewed by Mark Rowe
-        
+
         Make build-launcher-app work when run from a git repository.
         https://bugs.webkit.org/show_bug.cgi?id=20478
 
 2008-08-17  Geoffrey Garen  <ggaren@apple.com>
 
         Reviewed by Cameron Zwarich.
-        
+
         Updated project files to XCode 3.1.
 
         * DrawTest/DrawTest.xcodeproj/project.pbxproj:
 2008-08-06  Eric Seidel  <eric@webkit.org>
 
         Reviewed by darin.
-        
+
         Fix leaks seen on build-bot by fixing memory management of AccessibilityUIElement.
         <https://bugs.webkit.org/show_bug.cgi?id=20297>
         <rdar://problem/6093153>
-        
+
         I've made AccessibilityUIElement a stack object for now.  It could be
         made RefCounted, but I figured that might be overkill for the moment.
         Essentially it's just a RefPtr itself. :)
-        
+
         I also fixed a few typos, such as AccessibilityUIElement() instead of ~AccessibilityUIElement()
         which was causing additional leaks. :)
-        
+
         I added toAXElement to remove a bunch of copy/paste code.
 
         * DumpRenderTree/AccessibilityController.h:
 2008-08-02  Kevin Ollivier  <kevino@theolliviers.com>
 
         Reviewed by Eric Seidel.
-        
+
         Scripts for building a wxWebKit installer for wxPython on Win, eventually will
         be used for nightlies.
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=20036
 
         * wx/packaging: Added.
         Reviewed by Adam Roben.
 
         Slightly stricter checking for previous change, suggested by Adam.
-        
+
         * Scripts/extract-localizable-strings:
 
 2008-07-30  Jessica Kahn  <jess@apple.com>
         Reviewed by Darin Adler.
 
         Added support for UI_STRING and UI_STRING_KEY macros with flexible prefixes.
-        
+
         * Scripts/extract-localizable-strings:
 
 2008-07-30  Anders Carlsson  <andersca@apple.com>
         Reviewed by Adam.
 
         Copy icu38* files instead of icu36* files.
-        
+
         * DumpRenderTree/win/DumpRenderTree.vcproj:
 
 2008-07-25  Jan Michael Alonzo  <jmalonzo@webkit.org>
 
         Reviewed by Sam
 
-        Add the ability to dump whether-or-not an element should have autocomplete enabled, 
+        Add the ability to dump whether-or-not an element should have autocomplete enabled,
         from the perspective of the WebKit API
 
         * DumpRenderTree/LayoutTestController.cpp:
 
         Reviewed by Beth Dakin
 
-        Add support for AXLineForIndex and AXInsertionPointIndex through the 
+        Add support for AXLineForIndex and AXInsertionPointIndex through the
         accessibility controller
 
         * DumpRenderTree/AccessibilityUIElement.cpp:
 2008-07-09  Eric Seidel  <eric@webkit.org>
 
         Reviewed by Mark Rowe.
-        
+
         Print <div> backgrounds when printing from DRT.
 
         * DumpRenderTree/mac/DumpRenderTree.mm:
 
         Add Content-Type support to DumpRenderTree and run-webkit-tests
         and move expected.txt files to expected.webarchive
-        
+
         https://bugs.webkit.org/show_bug.cgi?id=15565
 
         * DumpRenderTree/cg/ImageDiffCG.cpp:
 
         Reviewed by Anders.
 
-        This patch extends DRT accessibility tests to add the ability to 
+        This patch extends DRT accessibility tests to add the ability to
         query the intValue, minValue, and maxValue of the focused element.
 
         * DumpRenderTree/AccessibilityController.cpp:
         <rdar://5983747> Safari crashes trying to load the SilverLight plugin,
         caused by WebKit calling NPP_DestroyStream after a plug-in returns an
         error from NPP_NewStream.
-        (pluginGetProperty): 
+        (pluginGetProperty):
         (pluginSetProperty):
         (pluginAllocate):
         * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.h: Added new
 
         Add multi-button mouseevent support to DRT
         https://bugs.webkit.org/show_bug.cgi?id=15173
-        
+
         It's now possible to specify the mouse button with:
         eventSender.mouseDown(1); eventSender.mouseUp(1); etc.
 
 
         Reviewed by Sam.
 
-        Do not include AXPosition in the dump of all of the accessibility 
+        Do not include AXPosition in the dump of all of the accessibility
         attributes since it is screen-specific.
 
         * DumpRenderTree/mac/AccessibilityControllerMac.mm:
@@