Skip to content

When parsing a report descriptor, reset stringIndices when parsing a Main item - #167

Open
gregstoll wants to merge 2 commits into
whatwg:mainfrom
gregstoll:parse-clearStringIndices
Open

When parsing a report descriptor, reset stringIndices when parsing a Main item#167
gregstoll wants to merge 2 commits into
whatwg:mainfrom
gregstoll:parse-clearStringIndices

Conversation

@gregstoll

@gregstoll gregstoll commented Oct 28, 2026 •
edited by pr-preview Bot
Loading

Copy link
Copy Markdown
Contributor

Fixes #153

  • At least two implementers are interested (and none opposed):
    • Chromium
    • Gecko
  • Tests are written and can be reviewed and commented upon at:
    • (do not have web-platform-tests around report descriptor parsing)
  • Implementation bugs are filed:
    • Chromium: already matches Chromium behavior
    • Gecko: not implemented
    • WebKit: not implemented
  • MDN issue is filed: …
    • (MDN does not go into this level of detail)
  • The top of this comment includes a clear commit message to use.

Preview | Diff

@nondebug
nondebug self-requested a review October 30, 2026 11:35
nondebug
nondebug requested changes Oct 30, 2026
View reviewed changes
Comment thread index.bs Resolved
@gregstoll
gregstoll force-pushed the parse-clearStringIndices branch from 890d42d to b54b862 Compare October 30, 2026 13:54
@gregstoll
gregstoll requested a review from nondebug October 30, 2026 13:55

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nondebug nondebug Awaiting requested review from nondebug

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

When parsing a report descriptor, stringIndices isn't cleared after a Main item

2 participants

@gregstoll @nondebug