This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 18233 - Wrong IDL in definition of HitRegionOptions
Summary: Wrong IDL in definition of HitRegionOptions
Status: RESOLVED FIXED
Alias: None
Product: HTML WG
Classification: Unclassified
Component: HTML Canvas 2D Context (show other bugs)
Version: unspecified
Hardware: Other other
: P2 normal
Target Milestone: ---
Assignee: Jay Munro
QA Contact: HTML WG Bugzilla archive list
URL: http://www.whatwg.org/specs/web-apps/...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-18 17:46 UTC by contributor
Modified: 2012-08-23 23:22 UTC (History)
6 users (show)

See Also:


Attachments

Description contributor 2012-07-18 17:46:35 UTC
This was was cloned from bug 17507 as part of operation convergence.
Originally filed: 2012-06-16 10:44:00 +0000

================================================================================
 #0   contributor@whatwg.org                          2012-06-16 10:44:21 +0000 
--------------------------------------------------------------------------------
Specification: http://www.whatwg.org/specs/web-apps/current-work/multipage/the-canvas-element.html
Multipage: http://www.whatwg.org/C#2dcontext
Complete: http://www.whatwg.org/c#2dcontext

Comment:
In dictionary HitRegionOptions {   Path? path = null;	DOMString id = '';

Posted from: 84.182.211.110
User agent: Mozilla/5.0 (Windows NT 6.1) AppleWebKit/536.11 (KHTML, like Gecko) Chrome/20.0.1132.34 Safari/536.11
================================================================================
 #1   Wolfgang Keller                                 2012-06-16 10:52:55 +0000 
--------------------------------------------------------------------------------
In the IDL of HitRegionOptions we have

dictionary HitRegionOptions {
  // [snip]
  DOMString id = '';
  // [snap]
};

If we look at the current version of the WebIDL draft the definition of the nonterminal DefaultValue (which is used for the part after the equal sign) we see (http://dev.w3.org/2006/webapi/WebIDL/#proddef-DefaultValue):

[16]	DefaultValue	→	ConstValue 
 | string

'' is surely not a ConstValue, since the definition of this nonterminal is (http://www.w3.org/TR/WebIDL/#prod-ConstValue):

[27]	ConstValue	→	BooleanLiteral 
 | FloatLiteral 
 | integer 
 | "null"

Thus '' have to be the terminal named "string"

This is defined as (http://dev.w3.org/2006/webapi/WebIDL/#prod-string)

string	=	"[^"]*"

In other words: it has to begin with a double quote - not a single quote.

To solve this bug we either should change the IDL of HitRegionOptions to

dictionary HitRegionOptions {
  // [snip]
  DOMString id = ""; // Note the double quotes
  // [snap]
};

or allow in the definition of WebIDL to let strings enclosed in single quotes.
================================================================================
Comment 1 Erika Doyle Navara 2012-08-15 21:36:56 UTC
Moved to HTML Canvas 2D Context.
Comment 2 Jay Munro 2012-08-18 00:40:14 UTC
This sounds reasonable.
Comment 3 Jay Munro 2012-08-23 23:22:08 UTC
    EDITOR'S RESPONSE: This is an Editor's Response to your comment. If
    you are satisfied with this response, please change the state of
    this bug to CLOSED. If you have additional information and would
    like the Editor to reconsider, please reopen this bug. If you would
    like to escalate the issue to the full HTML Working Group, please
    add the TrackerRequest keyword to this bug, and suggest title and
    text for the Tracker Issue; or you may create a Tracker Issue
    yourself, if you are able to do so. For more details, see this
    document:

       http://dev.w3.org/html5/decision-policy/decision-policy.html

    Status: Accepted
    Change Description: Fixed 
    Rationale: fixed: bf9a95ba2ad3cbba75f04304a8d5f60ef9e5500d