[SVG] SVGAnimatedRect migration to new SVG property impl.

Source/core/svg/SVGZoomEvent.cpp

 /*
  * Copyright (C) 2004, 2005 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005 Rob Buis <buis@kde.org>
  * Copyright (C) 2006 Samuel Weinig <sam.weinig@gmail.com>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Library General Public License for more details.
  *
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  */
 
 #include "config.h"
+

[haraken, 2014/01/16 04:56:43]

Nit: This empty line is not needed.

 #include "core/svg/SVGZoomEvent.h"
 
 #include "core/events/ThreadLocalEventNames.h"
+#include "core/svg/SVGRectTearOff.h"
 
 namespace WebCore {
 
 SVGZoomEvent::SVGZoomEvent()
     : m_newScale(0.0f)
     , m_previousScale(0.0f)
 {
     ScriptWrappable::init(this);
 }
 
-SVGRect SVGZoomEvent::zoomRectScreen() const
+PassRefPtr<SVGRectTearOff> SVGZoomEvent::zoomRectScreen() const
 {
-    return m_zoomRectScreen;
+    RefPtr<SVGRectTearOff> rectTearOff = SVGRectTearOff::create(SVGRect::create(), 0, PropertyIsNotAnimVal);
+    rectTearOff->setIsReadOnlyProperty();

[haraken, 2014/01/16 04:56:43]

Just help me understand: Why was previous code working without setting
read-only?

[kouhei (in TOK), 2014/01/16 05:12:37]

Because it was violating the spec. ^p^

+    return rectTearOff.release();
 }
 
 float SVGZoomEvent::previousScale() const
 {
     return m_previousScale;
 }
 
 void SVGZoomEvent::setPreviousScale(float scale)
 {
     m_previousScale = scale;
@@ skipping 18 matching lines @@
 {
     return m_newTranslate;
 }
 
 const AtomicString& SVGZoomEvent::interfaceName() const
 {
     return EventNames::SVGZoomEvent;
 }
 
 } // namespace WebCore