Unreviewed, rolling out r132702.
authorscheib@chromium.org <scheib@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 Oct 2012 23:28:46 +0000 (23:28 +0000)
committerscheib@chromium.org <scheib@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 Oct 2012 23:28:46 +0000 (23:28 +0000)
http://trac.webkit.org/changeset/132702
https://bugs.webkit.org/show_bug.cgi?id=100322

Compile error on Chromium Linux dbg builder (and others)

Source/WebCore:

* svg/SVGAnimationElement.cpp:
(WebCore::SVGAnimationElement::beginElementAt):
(WebCore::SVGAnimationElement::endElementAt):
* svg/animation/SMILTime.h:
(WebCore::SMILTime::SMILTime):
* svg/animation/SVGSMILElement.cpp:
(WebCore::SVGSMILElement::addBeginTime):
(WebCore::SVGSMILElement::addEndTime):

LayoutTests:

* svg/custom/elementTimeControl-nan-crash-expected.txt: Removed.
* svg/custom/elementTimeControl-nan-crash.html: Removed.

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

LayoutTests/ChangeLog
LayoutTests/svg/custom/elementTimeControl-nan-crash-expected.txt [deleted file]
LayoutTests/svg/custom/elementTimeControl-nan-crash.html [deleted file]
Source/WebCore/ChangeLog
Source/WebCore/svg/SVGAnimationElement.cpp
Source/WebCore/svg/animation/SMILTime.h
Source/WebCore/svg/animation/SVGSMILElement.cpp

index c65f134..b733383 100644 (file)
@@ -1,3 +1,14 @@
+2012-10-26  Vincent Scheib  <scheib@chromium.org>
+
+        Unreviewed, rolling out r132702.
+        http://trac.webkit.org/changeset/132702
+        https://bugs.webkit.org/show_bug.cgi?id=100322
+
+        Compile error on Chromium Linux dbg builder (and others)
+
+        * svg/custom/elementTimeControl-nan-crash-expected.txt: Removed.
+        * svg/custom/elementTimeControl-nan-crash.html: Removed.
+
 2012-10-26  Tony Chang  <tony@chromium.org>
 
         [chromium] Unreviewed, remove one more test that consistently passes according to the flakiness dashboard.
diff --git a/LayoutTests/svg/custom/elementTimeControl-nan-crash-expected.txt b/LayoutTests/svg/custom/elementTimeControl-nan-crash-expected.txt
deleted file mode 100644 (file)
index ef9a630..0000000
+++ /dev/null
@@ -1 +0,0 @@
-Test for WK100322: ElementTimeControl should check for invalid values. This test passes if it does not crash.
diff --git a/LayoutTests/svg/custom/elementTimeControl-nan-crash.html b/LayoutTests/svg/custom/elementTimeControl-nan-crash.html
deleted file mode 100644 (file)
index 1d5e5d6..0000000
+++ /dev/null
@@ -1,25 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-<script>
-    function crash() {
-        var animate = document.getElementById('animate');
-        var svg = document.getElementById('svg');
-        animate.endElementAt(NaN);
-        animate.beginElementAt(NaN);
-        svg.setCurrentTime(2);
-        if (window.testRunner)
-            testRunner.dumpAsText();
-    }
-</script>
-</head>
-<body onload="crash()">
-Test for WK100322: ElementTimeControl should check for invalid values. This test passes if it does not crash.
-
-<svg id="svg" width="200" height="200">
-    <rect x="0" y="0" width="100" height="100" fill="green">
-        <animate id="animate" attributeName="x" to="200" begin="3s"/>
-    </rect>
-</svg>
-</body>
-</html>
index 0ce76b6..eff436e 100644 (file)
@@ -1,3 +1,20 @@
+2012-10-26  Vincent Scheib  <scheib@chromium.org>
+
+        Unreviewed, rolling out r132702.
+        http://trac.webkit.org/changeset/132702
+        https://bugs.webkit.org/show_bug.cgi?id=100322
+
+        Compile error on Chromium Linux dbg builder (and others)
+
+        * svg/SVGAnimationElement.cpp:
+        (WebCore::SVGAnimationElement::beginElementAt):
+        (WebCore::SVGAnimationElement::endElementAt):
+        * svg/animation/SMILTime.h:
+        (WebCore::SMILTime::SMILTime):
+        * svg/animation/SVGSMILElement.cpp:
+        (WebCore::SVGSMILElement::addBeginTime):
+        (WebCore::SVGSMILElement::addEndTime):
+
 2012-10-26  Philip Rogers  <pdr@google.com>
 
         Prevent NaN offset values in ElementTimeControl.
index 1013c68..bbaadbb 100644 (file)
@@ -240,8 +240,6 @@ void SVGAnimationElement::beginElement()
 
 void SVGAnimationElement::beginElementAt(float offset)
 {
-    if (isnan(offset))
-        return;
     SMILTime elapsed = this->elapsed();
     addBeginTime(elapsed, elapsed + offset, SMILTimeWithOrigin::ScriptOrigin);
 }
@@ -253,8 +251,6 @@ void SVGAnimationElement::endElement()
 
 void SVGAnimationElement::endElementAt(float offset)
 {
-    if (isnan(offset))
-        return;
     SMILTime elapsed = this->elapsed();
     addEndTime(elapsed, elapsed + offset, SMILTimeWithOrigin::ScriptOrigin);
 }
index dd99cad..18abdae 100644 (file)
@@ -35,7 +35,7 @@ namespace WebCore {
 class SMILTime {
 public:
     SMILTime() : m_time(0) { }
-    SMILTime(double time) : m_time(time) { ASSERT(!isnan(time)); }
+    SMILTime(double time) : m_time(time) { }
     SMILTime(const SMILTime& o) : m_time(o.m_time) { }
     
     static SMILTime unresolved() { return unresolvedValue; }
index 158d461..f4deff5 100644 (file)
@@ -713,7 +713,6 @@ SMILTime SVGSMILElement::simpleDuration() const
 
 void SVGSMILElement::addBeginTime(SMILTime eventTime, SMILTime beginTime, SMILTimeWithOrigin::Origin origin)
 {
-    ASSERT(!isnan(beginTime.value()));
     m_beginTimes.append(SMILTimeWithOrigin(beginTime, origin));
     sortTimeList(m_beginTimes);
     beginListChanged(eventTime);
@@ -721,7 +720,6 @@ void SVGSMILElement::addBeginTime(SMILTime eventTime, SMILTime beginTime, SMILTi
 
 void SVGSMILElement::addEndTime(SMILTime eventTime, SMILTime endTime, SMILTimeWithOrigin::Origin origin)
 {
-    ASSERT(!isnan(endTime.value()));
     m_endTimes.append(SMILTimeWithOrigin(endTime, origin));
     sortTimeList(m_endTimes);
     endListChanged(eventTime);