<?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>23987</bug_id>
          
          <creation_ts>2013-12-03 18:09:44 +0000</creation_ts>
          <short_desc>Empty path and drawSystemFocusRing</short_desc>
          <delta_ts>2014-01-20 23:46:32 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>HTML WG</product>
          <component>CR HTML Canvas 2D Context</component>
          <version>unspecified</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows NT</op_sys>
          <bug_status>CLOSED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>a11y_canvas</keywords>
          <priority>P2</priority>
          <bug_severity>minor</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Philippe Le Hegaret">plh</reporter>
          <assigned_to name="Jay Munro">jaymunro</assigned_to>
          <cc>jmann</cc>
    
    <cc>mark.sadecki+w3c</cc>
    
    <cc>mike</cc>
    
    <cc>public-html-admin</cc>
    
    <cc>public-html-wg-issue-tracking</cc>
    
    <cc>schwer</cc>
          
          <qa_contact name="HTML WG Bugzilla archive list">public-html-bugzilla</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>97067</commentid>
    <comment_count>0</comment_count>
    <who name="Philippe Le Hegaret">plh</who>
    <bug_when>2013-12-03 18:09:44 +0000</bug_when>
    <thetext>What is expected from drawSystemFocusRing if the path is empty, e.g.
[[
 var ctx = canvas.getContext(&quot;2D&quot;);
 ctx.beginPath();
 myInput.focus();
 ctx.drawSystemFocusRing(myInput);
]]</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>97078</commentid>
    <comment_count>1</comment_count>
    <who name="Jay Munro">jaymunro</who>
    <bug_when>2013-12-03 21:29:37 +0000</bug_when>
    <thetext>Reassigning to Canvas 2D Context Level 2 since drawSystemFocusRing and drawCustomFocusRing are at risk for CR.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>97130</commentid>
    <comment_count>2</comment_count>
    <who name="Rich Schwerdtfeger">schwer</who>
    <bug_when>2013-12-04 15:14:55 +0000</bug_when>
    <thetext>There is always a default path:

The context always has a current default path. There is only one current default path, it is not part of the drawing state. The current default path is a path, as described above.

I am trying to understand the error condition you are getting at.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>97132</commentid>
    <comment_count>3</comment_count>
    <who name="Philippe Le Hegaret">plh</who>
    <bug_when>2013-12-04 16:24:47 +0000</bug_when>
    <thetext>My example has an empty path. Unless I&apos;m misreading the specification, the default path doesn&apos;t enter into the equation here. beginPath() was invoked and nothing has been added into the path. You may be suggesting that, if the path is empty, then the previous intended path in the context is used.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>97157</commentid>
    <comment_count>4</comment_count>
    <who name="Jay Munro">jaymunro</who>
    <bug_when>2013-12-04 19:49:37 +0000</bug_when>
    <thetext>I jumped the gun on moving to L2, there are still ongoing discussions on the focus ring methods. Moving back to CR until discussions are complete.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>98097</commentid>
    <comment_count>5</comment_count>
    <who name="Jay Munro">jaymunro</who>
    <bug_when>2014-01-07 17:39:33 +0000</bug_when>
    <thetext>If the path was empty, there isn&apos;t a path to act on at this point, though the intent is there with beginPath(). 

The issue appears to be whether to use the last path created before beginPath() was called? 

I would expect it to just do nothing, and treat it as a bug in code if an empty path was created.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>98140</commentid>
    <comment_count>6</comment_count>
    <who name="Philippe Le Hegaret">plh</who>
    <bug_when>2014-01-08 18:31:51 +0000</bug_when>
    <thetext>&gt; I would expect it to just do nothing, and treat it as a bug in code if an
&gt; empty path was created.

That would be my expectation as well and a clarification in the specification would be appropriate imho.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>98634</commentid>
    <comment_count>7</comment_count>
    <who name="Jay Munro">jaymunro</who>
    <bug_when>2014-01-16 17:34:28 +0000</bug_when>
    <thetext>Updated the spec to include a new first step stating that if the current path had zero subpaths, then abort the steps.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>98800</commentid>
    <comment_count>8</comment_count>
    <who name="Jatinder Mann [MSFT]">jmann</who>
    <bug_when>2014-01-20 23:46:32 +0000</bug_when>
    <thetext>Based on Canvas SubGroup discussion, http://www.w3.org/2014/01/20-html-a11y-minutes.html, we have agreed to close this bug as the processing model step 1 resolves this issue.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>