2007-01-02 Eric Seidel <eric@webkit.org>
authoreseidel <eseidel@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 2 Jan 2007 17:08:50 +0000 (17:08 +0000)
committereseidel <eseidel@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 2 Jan 2007 17:08:50 +0000 (17:08 +0000)
        Reviewed by mitz.

        Implement SVGSVGElement::deselectAll()

        * ksvg2/svg/SVGAnimationElement.h:
        (WebCore::SVGAnimationElement::updateLastValueWithCurrent): add comment
        * ksvg2/svg/SVGSVGElement.cpp:
        (WebCore::SVGSVGElement::deselectAll): implement
        (WebCore::SVGSVGElement::setCurrentTime): add comment about bug
        * ksvg2/svg/SVGSVGElement.idl: remove unnecessary comment

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

WebCore/ChangeLog
WebCore/ksvg2/svg/SVGAnimationElement.h
WebCore/ksvg2/svg/SVGSVGElement.cpp
WebCore/ksvg2/svg/SVGSVGElement.idl

index 709d8bf..2c34d75 100644 (file)
@@ -1,3 +1,16 @@
+2007-01-02  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by mitz.
+
+        Implement SVGSVGElement::deselectAll()
+
+        * ksvg2/svg/SVGAnimationElement.h:
+        (WebCore::SVGAnimationElement::updateLastValueWithCurrent): add comment
+        * ksvg2/svg/SVGSVGElement.cpp:
+        (WebCore::SVGSVGElement::deselectAll): implement
+        (WebCore::SVGSVGElement::setCurrentTime): add comment about bug
+        * ksvg2/svg/SVGSVGElement.idl: remove unnecessary comment
+
 2007-01-02  Sam Weinig  <sam@webkit.org>
 
         Reviewed by Darin.
index 3fbdbba..6892e2b 100644 (file)
@@ -135,7 +135,7 @@ namespace WebCore {
         
         virtual bool updateCurrentValue(double timePercentage) = 0;
         virtual bool handleStartCondition() = 0;
-        virtual void updateLastValueWithCurrent() { };
+        virtual void updateLastValueWithCurrent() { } // See bug 12075 for explaination of why this is a bad API
         virtual void resetValues() { }
 
         ANIMATED_PROPERTY_FORWARD_DECLARATIONS(SVGExternalResourcesRequired, bool, ExternalResourcesRequired, externalResourcesRequired)
index 9de1513..6d3f5c9 100644 (file)
 #ifdef SVG_SUPPORT
 #include "SVGSVGElement.h"
 
+#include "AffineTransform.h"
 #include "CSSPropertyNames.h"
 #include "Document.h"
 #include "EventListener.h"
 #include "EventNames.h"
+#include "Frame.h"
 #include "HTMLNames.h"
-#include "TimeScheduler.h"
 #include "RenderSVGContainer.h"
 #include "SVGAngle.h"
 #include "SVGLength.h"
-#include "AffineTransform.h"
 #include "SVGNames.h"
 #include "SVGPreserveAspectRatio.h"
 #include "SVGTransform.h"
 #include "SVGZoomEvent.h"
+#include "SelectionController.h"
 #include "TextStream.h"
+#include "TimeScheduler.h"
 
 namespace WebCore {
 
@@ -265,7 +267,7 @@ bool SVGSVGElement::checkEnclosure(SVGElement* element, const FloatRect& rect)
 
 void SVGSVGElement::deselectAll()
 {
-    // FIXME: Implement me (see bug 11275)
+    document()->frame()->selectionController()->clear();
 }
 
 double SVGSVGElement::createSVGNumber()
@@ -390,7 +392,7 @@ float SVGSVGElement::getCurrentTime() const
 
 void SVGSVGElement::setCurrentTime(float /* seconds */)
 {
-    // FIXME: Implement me
+    // FIXME: Implement me, bug 12073
 }
 
 bool SVGSVGElement::hasPercentageValues() const
index 74f4ead..ab21db6 100644 (file)
@@ -83,8 +83,6 @@ module svg {
         SVGRect createSVGRect();
         SVGTransform createSVGTransform();
         SVGTransform createSVGTransformFromMatrix(in SVGMatrix matrix);
-
-        // TODO    core::Element getElementById(in core::DOMString elementId);
     };
 
 }