<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://www.w3.org/Bugs/Public/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4"
          urlbase="https://www.w3.org/Bugs/Public/"
          
          maintainer="sysbot+bugzilla@w3.org"
>

    <bug>
          <bug_id>22419</bug_id>
          
          <creation_ts>2013-06-20 15:37:31 +0000</creation_ts>
          <short_desc>SVGMatrix rotateFromVector exception</short_desc>
          <delta_ts>2013-06-20 17:20:06 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>SVG</product>
          <component>ECMAScript Language Binding</component>
          <version>SVG 2.0</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>Test Suite</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Erik Arvidsson">erik.arvidsson</reporter>
          <assigned_to name="Doug Schepers">schepers</assigned_to>
          <cc>dschulze</cc>
          
          <qa_contact name="SVG Public List">www-svg</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>89652</commentid>
    <comment_count>0</comment_count>
    <who name="Erik Arvidsson">erik.arvidsson</who>
    <bug_when>2013-06-20 15:37:31 +0000</bug_when>
    <thetext>http://www.w3.org/TR/SVG2/single-page.html#coords-__svg__SVGMatrix__rotateFromVector

SVGMatrix is documented to throw an InvalidAccessError. Maybe it makes more sense to throw a RangeError (since 0 is outside the valid range). http://people.mozilla.org/~jorendorff/es6-draft.html#sec-15.11.6.2

Another option is to throw a TypeError?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>89653</commentid>
    <comment_count>1</comment_count>
    <who name="Erik Arvidsson">erik.arvidsson</who>
    <bug_when>2013-06-20 16:07:52 +0000</bug_when>
    <thetext>I guess the same issue applies to skewX and skewY</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>89654</commentid>
    <comment_count>2</comment_count>
    <who name="Dirk Schulze">dschulze</who>
    <bug_when>2013-06-20 16:20:21 +0000</bug_when>
    <thetext>The idea is to replace SVGMatrix with the interface CSSMatrix https://dvcs.w3.org/hg/FXTF/raw-file/tip/matrix/index.html It still needs approval from the CSS WG though.

This would also get rid of the exception handling, since we try to avoid exceptions in general unless you do something forbidden nowadays.

Also, I assume the old exception handling is based on SVG exceptions which got replaced by general DOM exception in SVG2.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>89655</commentid>
    <comment_count>3</comment_count>
    <who name="Erik Arvidsson">erik.arvidsson</who>
    <bug_when>2013-06-20 16:36:42 +0000</bug_when>
    <thetext>CSSMatrix does not document what happens when x or y is 0.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>89656</commentid>
    <comment_count>4</comment_count>
    <who name="Dirk Schulze">dschulze</who>
    <bug_when>2013-06-20 17:16:25 +0000</bug_when>
    <thetext>(In reply to comment #3)
&gt; CSSMatrix does not document what happens when x or y is 0.

Doesn&apos;t need to. Some matrix values get undefined.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>89657</commentid>
    <comment_count>5</comment_count>
    <who name="Dirk Schulze">dschulze</who>
    <bug_when>2013-06-20 17:20:06 +0000</bug_when>
    <thetext>(In reply to comment #4)
&gt; (In reply to comment #3)
&gt; &gt; CSSMatrix does not document what happens when x or y is 0.
&gt; 
&gt; Doesn&apos;t need to. Some matrix values get undefined.

Sorry, in general depends on the math of each function. Some values can get none-finit.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>