fix(anki): keep playback paused after overlay pause during timing review

- Treat overlay pause requests during an open media timing review as final: drop held resumes and the review's own playback restore
- Add cancelPlaybackResume to the media timing review runtime
- Test both paused and playing starting states
This commit is contained in:
2026-09-23 01:36:11 -07:00
parent 430120cc63
commit 33e694bc61
3 changed files with 41 additions and 9 deletions
+19 -1
View File
@@ -95,6 +95,7 @@ async function startActiveMediaTimingReview(
decisionTimeoutMs?: number;
generateWaveform?: () => Promise<number[]>;
play?: () => Promise<void>;
paused?: boolean;
} = {},
) {
const previewCalls: Array<[number, number]> = [];
@@ -107,7 +108,8 @@ async function startActiveMediaTimingReview(
getMpvClient: () => ({
connected: true,
currentVideoPath: '/video/show.mkv',
requestProperty: async (name) => (name === 'duration' ? 100 : name === 'pause' ? true : null),
requestProperty: async (name) =>
name === 'duration' ? 100 : name === 'pause' ? (options.paused ?? true) : null,
send: ({ command }) => commands.push(command),
}),
getCurrentMediaPath: () => '/video/show.mkv',
@@ -668,6 +670,22 @@ test('media timing review holds overlay resume requests until the review closes'
assert.equal(runtime.deferPlaybackResume(), false);
});
for (const paused of [true, false]) {
test(`media timing review stays paused after an overlay pause request (paused before: ${paused})`, async () => {
const { runtime, payload, pendingDecision, commands } = await startActiveMediaTimingReview({
paused,
});
assert.equal(runtime.deferPlaybackResume(), true);
runtime.cancelPlaybackResume();
runtime.resolveReview({ reviewId: payload.reviewId, decision: { action: 'use-original' } });
await pendingDecision;
assert.deepEqual(commands, [['set_property', 'pause', 'yes']]);
});
}
test('media timing review watchdog falls back when the renderer stops responding', async () => {
const { pendingDecision } = await startActiveMediaTimingReview({ decisionTimeoutMs: 0 });
+10
View File
@@ -313,6 +313,15 @@ export function createMediaTimingReviewRuntime(deps: MediaTimingReviewRuntimeDep
return true;
}
/**
* The overlay asked to pause while a review holds playback, so closing the review must
* not resume it: drops both a held overlay resume and the review's own restore.
*/
function cancelPlaybackResume(): void {
resumeDeferred = false;
if (active) active.restorePlayback = false;
}
function ensureWindow(
review: ActiveReview,
range: RemoteMediaWindowRange,
@@ -782,6 +791,7 @@ export function createMediaTimingReviewRuntime(deps: MediaTimingReviewRuntimeDep
stopPreview,
resolveReview,
deferPlaybackResume,
cancelPlaybackResume,
dispose,
};
}