- From: Domenic Denicola <notifications@github.com>
- Date: Mon, 20 Jan 2025 18:28:29 -0800
- To: whatwg/xhr <xhr@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
- Message-ID: <whatwg/xhr/pull/394@github.com>
This allows specifications to use ProgressEvents while not necessarily giving away the exact number of bytes, while maintaining a granularity greater than just 1 part in 100. See discussion in https://github.com/webmachinelearning/writing-assistance-apis/issues/15 for the concrete use case. - [ ] At least two implementers are interested (and none opposed): * Chromium is interested * … - [ ] [Tests](https://github.com/web-platform-tests/wpt) are written and can be reviewed and commented upon at: * I will get these written. The test is basically how synthetic `ProgressEvent` behavior changes: fractions are preserved, and negatives are now passed through as-is instead of subtracted from 2^64. - [ ] [Implementation bugs](https://github.com/whatwg/meta/blob/main/MAINTAINERS.md#handling-pull-requests) are filed: * Chromium: … * Gecko: … * WebKit: … - [ ] [MDN issue](https://github.com/whatwg/meta/blob/main/MAINTAINERS.md#handling-pull-requests) is filed: … - [x] The top of this comment includes a [clear commit message](https://github.com/whatwg/meta/blob/main/COMMITTING.md) to use. <!-- If you created this PR from a single commit, Github copied its message. Otherwise, you need to add a commit message yourself. --> (See [WHATWG Working Mode: Changes](https://whatwg.org/working-mode#changes) for more details.) --- I wanted to get this PR up to judge implementer interest. It would be a decent bit cleaner if we all agreed this was reasonable and merged the change now, instead of having to float a patch in the relevant specs and thus causing all the databases to have two definitions of `ProgressEvent`. <!-- This comment and the below content is programmatically generated. You may add a comma-separated list of anchors you'd like a direct link to below (e.g. #idl-serializers, #idl-sequence): Don't remove this comment or modify anything below this line. If you don't want a preview generated for this pull request, just replace the whole of this comment's content by "no preview" and remove what's below. --> *** <a href="https://whatpr.org/xhr/394.html" title="Last updated on Jan 21, 2025, 2:28 AM UTC (48e8e0d)">Preview</a> | <a href="https://whatpr.org/xhr/394/f2f0397...48e8e0d.html" title="Last updated on Jan 21, 2025, 2:28 AM UTC (48e8e0d)">Diff</a> You can view, comment on, or merge this pull request online at: https://github.com/whatwg/xhr/pull/394 -- Commit Summary -- * Allow doubles for ProgressEvent's loaded and total -- File Changes -- M xhr.bs (8) -- Patch Links -- https://github.com/whatwg/xhr/pull/394.patch https://github.com/whatwg/xhr/pull/394.diff -- Reply to this email directly or view it on GitHub: https://github.com/whatwg/xhr/pull/394 You are receiving this because you are subscribed to this thread. Message ID: <whatwg/xhr/pull/394@github.com>
Received on Tuesday, 21 January 2025 02:28:33 UTC