Closed
Bug 1514091
Opened 5 years ago
Closed 5 years ago
[wpt-sync] Sync PR 14509 - TextDecoder: previous failure should not stick around
Categories
(Core :: Internationalization, enhancement, P4)
Core
Internationalization
Tracking
()
RESOLVED
FIXED
mozilla66
Tracking | Status | |
---|---|---|
firefox66 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 14509 into mozilla-central (this bug is closed when the sync is complete). PR: https://github.com/web-platform-tests/wpt/pull/14509 Details from upstream follow. Joshua Bell <jsbell@chromium.org> wrote: > TextDecoder: previous failure should not stick around > > When the {fatal:true} option is used, the decode() calls throw rather > than generating U+FFFD on bad data. Subsequent decode() calls should > work as the codec state should be flushed (unless {stream:true} was > specified). > > Blink was not flushing the codec state, so subsequent calls would > throw. Fix by flushing the codec state (instantiating a new codec), > and align the implementation's code flow/state more closely with the > spec. Also added a new WPT test case, based on the bug report. > > Bug: 910292 > Change-Id: Ia47b902b69a4b469341e272ed42bb6b05248c1d8 > Reviewed-on: https://chromium-review.googlesource.com/1376656 > WPT-Export-Revision: 7f2bb5205941bb8a29b21d41f67c720a02f785c3
Assignee | ||
Updated•5 years ago
|
Component: web-platform-tests → Internationalization
Product: Testing → Core
Assignee | ||
Comment 1•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=b4d6f2397063d50b26bd9f63bb48f31d937c85d5
Assignee | ||
Comment 2•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=3a4cb9dacfb0449aebbeaa4835e0364f28bcec6c
Assignee | ||
Comment 3•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=652d3eb487a7959a6a46d6a9dd3c80fa900bb504
Assignee | ||
Comment 4•5 years ago
|
||
Ran 2 tests and 72 subtests OK : 2 PASS : 72
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/mozilla-inbound/rev/8131e0336d28 [wpt PR 14509] - TextDecoder: previous failure should not stick around, a=testonly
Comment 6•5 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox66:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
You need to log in
before you can comment on or make changes to this bug.
Description
•