Bug 1749046 - Even when the configuration of an encoder wouldn't change because the config is identical, the decoder config must be emitted. r=chunmin

marco-c · marco-c · commit a44593a75ccc · 2024-06-06T17:28:50.000Z
This fixes a test failure in audio encoder WPTs. Differential Revision: https://phabricator.services.mozilla.com/D199542 UltraBlame original commit: 022e47d17ccaf4408863a4a51aded62a86f1bbf2
diff --git a/dom/media/webcodecs/EncoderTemplate.cpp b/dom/media/webcodecs/EncoderTemplate.cpp
@@ -5355,6 +5355,23 @@ to
 do
 return
 now
+but
+per
+spec
+the
+config
+/
+/
+must
+be
+output
+next
+time
+a
+packet
+is
+output
+.
 if
 (
 configDiff
@@ -5365,6 +5382,10 @@ Empty
 )
 )
 {
+mOutputNewDecoderConfig
+=
+true
+;
 LOG
 (
 "

Original file line number	Diff line number	Diff line change
`@@ -5355,6 +5355,23 @@ to`
`5355`	`5355`	`do`
`5356`	`5356`	`return`
`5357`	`5357`	`now`
	`5358`	`+but`
	`5359`	`+per`
	`5360`	`+spec`
	`5361`	`+the`
	`5362`	`+config`
	`5363`	`+/`
	`5364`	`+/`
	`5365`	`+must`
	`5366`	`+be`
	`5367`	`+output`
	`5368`	`+next`
	`5369`	`+time`
	`5370`	`+a`
	`5371`	`+packet`
	`5372`	`+is`
	`5373`	`+output`
	`5374`	`+.`
`5358`	`5375`	`if`
`5359`	`5376`	`(`
`5360`	`5377`	`configDiff`
`@@ -5365,6 +5382,10 @@ Empty`
`5365`	`5382`	`)`
`5366`	`5383`	`)`
`5367`	`5384`	`{`
	`5385`	`+mOutputNewDecoderConfig`
	`5386`	`+=`
	`5387`	`+true`
	`5388`	`+;`
`5368`	`5389`	`LOG`
`5369`	`5390`	`(`
`5370`	`5391`	`"`