14913 2011-11-23 17:38:09 +0000 No way to specify [OverrideBuiltins] if the getter is in a partial interface 2011-12-29 16:22:15 +0000 1 1 1 Unclassified WebAppsWG WebIDL unspecified All All CLOSED FIXED P2 normal --- 1 ayg cam annevk ian mike public-script-coord public-webapps-bugzilla oldest_to_newest 60387 0 ayg 2011-11-23 17:38:09 +0000 HTML currently has [OverrideBuiltins] partial interface Document { ... }, which is invalid because you can't have extended attributes on a partial interface. But DOM4 can't add [OverrideBuiltins] to the actual Document interface definition, because that defines no getter. Suggested resolution: allow [OverrideBuiltins] on partial interfaces, and say it has the same effect as if it was specified on the original interface. So the partial interface can add that extended attribute too, not just members. 60393 1 ian 2011-11-23 17:58:10 +0000 By just restricting it to the interface or partial interface that has the getter, we can neatly solve the problem of it being unclear which interface should have it. 61286 2 cam 2011-12-09 06:07:17 +0000 The non-localised effects of extended attributes on partial interface definitions is exactly what I was trying to avoid with the current spec restrictions. Ian's comment 1 suggestion does neatly get around that problem. (An alternative would be to write in HTML [NoInterfaceObject,OverrideBuiltins] interface DocumentProperties { getter ...; }; Document implements DocumentProperties; .) 61586 3 cam 2011-12-15 03:18:50 +0000 I've done comment 1. http://dev.w3.org/cvsweb/2006/webapi/WebIDL/Overview.xml.diff?r1=1.421;r2=1.422;f=h Aryeh, let me know if this change is satisfactory, thanks. 62109 4 ayg 2011-12-28 18:03:47 +0000 AFAICT, the grammar still doesn't allow extended attributes on partial interfaces, although the prose now does. 62117 5 cam 2011-12-29 00:24:37 +0000 You were right. I fixed this yesterday as part of the callback edit. 62135 6 ayg 2011-12-29 16:22:15 +0000 Oh, I misread. Never mind me. Updated WebIDLParser.js and idlharness.js: https://github.com/ayg/webidl.js/commit/ebed0073 http://dvcs.w3.org/hg/html/rev/c0952a1118d7