chore: Simplify inaccurateManifestTolerance when crossBoundaryStrategy is not KEEP (#8866)

Author: avelad

lib/media/streaming_engine.js

@@ -1742,8 +1742,9 @@ shaka.media.StreamingEngine = class {
       // If there's positive drift then we need to adjust the lookup time, and
       // may wind up requesting the previous segment to be safe.
       // inaccurateManifestTolerance should be 0 for low latency streaming.
-      const inaccurateTolerance = this.manifest_.sequenceMode ?
-        0 : this.config_.inaccurateManifestTolerance;
+      const inaccurateTolerance =
+          (this.manifest_.sequenceMode || this.shouldUseCrossBoundaryLogic_()) ?
+          0 : this.config_.inaccurateManifestTolerance;
       const lookupTime = Math.max(presentationTime - inaccurateTolerance, 0);
 
       shaka.log.v1(logPrefix, 'looking up segment',

lib/player.js

@@ -4594,15 +4594,6 @@ shaka.Player = class extends shaka.util.FakeEventTarget {
       delete config['streaming']['minBytesForProgressEvents'];
     }
 
-    // Enforce inaccurateManifestTolerance: 0 when using crossBoundaryStrategy
-    // different from KEEP.
-    if (config['streaming'] && 'crossBoundaryStrategy' in config['streaming']) {
-      if (config['streaming']['crossBoundaryStrategy'] !=
-          shaka.config.CrossBoundaryStrategy.KEEP) {
-        config['streaming']['inaccurateManifestTolerance'] = 0;
-      }
-    }
-
     const ret = shaka.util.PlayerConfiguration.mergeConfigObjects(
         this.config_, config, this.defaultConfig_());