27428 2014-11-25 10:58:42 +0000 [ArrayClass] should serialize as an array by default 2017-06-21 21:23:13 +0000 1 1 1 Unclassified WebAppsWG WebIDL unspecified PC All RESOLVED WONTFIX P2 normal --- 1 zcorpan cam annevk bzbarsky d mike public-script-coord tobie.langel public-webapps-bugzilla oldest_to_newest 115457 0 zcorpan 2014-11-25 10:58:42 +0000 http://heycam.github.io/webidl/#ArrayClass [[ [ArrayClass] ]] It seems currently the JSON serialization of DOMRectList is a bit ugly: http://software.hixie.ch/utilities/js/live-dom-viewer/saved/3315 Blink {"0":{"height":16,"width":1645,"left":8,"bottom":24,"right":1653,"top":8},"length":1} WebKit {"0":{},"length":1} Gecko {"0":{}} Assuming DOMRectList stays for Web compat, it would be nice to have it serialize as an array. Would it make sense for WebIDL to serialize [ArrayClass] interfaces as an array by default? (Or whatever replaces [ArrayClass].) 115459 1 annevk 2014-11-25 11:05:59 +0000 [ArrayClass] just puts Array on the prototype chain. That's it. If you want Blink's behavior you'd have to define that in the specification. 115460 2 zcorpan 2014-11-25 11:08:22 +0000 I don't want Blink's behavior (for this bug, anyway). If we assume that DOMRect should serialize as {} then I want: [{}] 115470 3 bzbarsky 2014-11-25 14:06:02 +0000 This could be done fairly easily by just having a toJSON method on the object that returns an array with the DOMRects, right? Assuming we want to do it, that is... 115471 4 d 2014-11-25 15:58:26 +0000 JSON.stringify is indeed the easy way to fix this. The alternate fix is to make DOMRectList a proper Array subclass. Browsers don't support subclassing arrays yet though, except the IE 12 preview. 128659 5 tobie.langel 2017-06-21 20:47:13 +0000 Fixed in https://github.com/heycam/webidl/commit/a505f33 128660 6 bzbarsky 2017-06-21 20:55:53 +0000 It is? I don't see any ArrayClass provisions in there... am I just missing them? 128661 7 tobie.langel 2017-06-21 21:02:06 +0000 My understanding from the above was that interfaces with the LegacyArrayClass extended attribute could just define their own JSONification strategy, e.g. by defining a toJSON operation which returns a sequence. 128662 8 bzbarsky 2017-06-21 21:21:56 +0000 That's fine with me, but it sounds more like a wontfix than a fixed. ;) 128663 9 tobie.langel 2017-06-21 21:23:13 +0000 (In reply to Boris Zbarsky from comment #8) > That's fine with me, but it sounds more like a wontfix than a fixed. ;) That's a good point.