perf(DASH): Release period combiner after parsing a VOD (#8443)

Author: avelad

lib/dash/dash_parser.js

@@ -852,6 +852,11 @@ shaka.dash.DashParser = class {
       // time, and we need to tell PresentationTimeline that so that it can
       // maintain consistency from here on.
       presentationTimeline.lockStartTime();
+
+      if (this.periodCombiner_ &&
+          !this.manifest_.presentationTimeline.isLive()) {
+        this.periodCombiner_.release();
+      }
     } else {
       this.manifest_.periodCount = periods.length;
       this.manifest_.gapCount = this.gapCount_;

test/dash/dash_parser_manifest_unit.js

@@ -2380,7 +2380,6 @@ describe('DashParser Manifest', () => {
     // (DASH-IF IOP v4.3 6.2.5.1.)
     const scheme = cicpScheme('TransferCharacteristics');
     const sdrValues = [1, 6, 13, 14, 15];
-    const manifestPromises = [];
     for (const value of sdrValues) {
       const manifestText = [
         '<MPD minBufferTime="PT75S">',
@@ -2404,10 +2403,9 @@ describe('DashParser Manifest', () => {
 
       fakeNetEngine.setResponseText('dummy://foo', manifestText);
 
-      manifestPromises.push(parser.start('dummy://foo', playerInterface));
-    }
-    const manifests = await Promise.all(manifestPromises);
-    for (const manifest of manifests) {
+      /** @type {shaka.extern.Manifest} */
+      // eslint-disable-next-line no-await-in-loop
+      const manifest = await parser.start('dummy://foo', playerInterface);
       expect(manifest.variants.length).toBe(1);
       const stream = manifest.variants[0].video;
       expect(stream.hdr).toBe('SDR');