Skip to content

Commit cd95955

Browse files
authored
Remove unused request.quality (Dash-Industry-Forum#4726)
1 parent 7327dc8 commit cd95955

File tree

7 files changed

+4
-14
lines changed

7 files changed

+4
-14
lines changed

src/streaming/controllers/BufferController.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -305,7 +305,7 @@ function BufferController(config) {
305305
logger.info('Init fragment finished loading saving to', type + '\'s init cache');
306306
initCache.save(e.chunk);
307307
}
308-
logger.debug('Append Init fragment', type, ' with representationId:', e.chunk.representation.id, ' and quality:', e.chunk.quality, ', data size:', e.chunk.bytes.byteLength);
308+
logger.debug(`Appending init fragment for type ${type}, representationId ${e.chunk.representation.id} and bandwidth ${e.chunk.representation.bandwidth}`);
309309
_appendToBuffer(e.chunk);
310310
}
311311

@@ -324,7 +324,8 @@ function BufferController(config) {
324324
}
325325

326326
// Append init segment into buffer
327-
logger.info('Append Init fragment', type, ' with representationId:', chunk.representation.id, ' and quality:', chunk.quality, ', data size:', chunk.bytes.byteLength);
327+
logger.debug(`Appending init fragment for type ${type}, representationId ${chunk.representation.id} and bandwidth ${chunk.representation.bandwidth}`);
328+
328329
_appendToBuffer(chunk);
329330

330331
return true;

src/streaming/controllers/FragmentController.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,6 @@ function FragmentController(config) {
120120
chunk.end = chunk.start + chunk.duration;
121121
chunk.bytes = bytes;
122122
chunk.index = request.index;
123-
chunk.quality = request.quality;
124123
chunk.representation = request.representation;
125124
chunk.endFragment = endFragment;
126125

src/streaming/models/FragmentModel.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,8 +129,7 @@ function FragmentModel(config) {
129129
* @param {Object} filter The object with properties by which the method filters the requests to be returned.
130130
* the only mandatory property is state, which must be a value from
131131
* other properties should match the properties of {@link FragmentRequest}. E.g.:
132-
* getRequests({state: FragmentModel.FRAGMENT_MODEL_EXECUTED, quality: 0}) - returns
133-
* all the requests from executedRequests array where requests.quality = filter.quality
132+
* getRequests({state: FragmentModel.FRAGMENT_MODEL_EXECUTED, quality: 0})
134133
*
135134
* @returns {Array}
136135
* @memberof FragmentModel#

src/streaming/models/MetricsModel.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,6 @@ function MetricsModel(config) {
153153
mediaType: request.mediaType,
154154
type: request.type,
155155
url: request.url,
156-
quality: request.quality,
157156
serviceLocation: null,
158157
range: request.range,
159158
startDate: request.startDate,
@@ -187,7 +186,6 @@ function MetricsModel(config) {
187186
vo._tfinish = request.endDate;
188187
vo._stream = request.mediaType;
189188
vo._mediaduration = request.duration;
190-
vo._quality = request.quality;
191189
// For backward compatibility, convert response headers into string representation
192190
vo._responseHeaders = '';
193191
for (const key in response.headers) {

src/streaming/vo/DataChunk.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@ class DataChunk {
3838
constructor() {
3939
this.streamId = null;
4040
this.segmentType = null;
41-
this.quality = NaN;
4241
this.index = NaN;
4342
this.bytes = null;
4443
this.start = NaN;

src/streaming/vo/FragmentRequest.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ class FragmentRequest {
5050
this.index = NaN;
5151
this.mediaStartTime = NaN;
5252
this.mediaType = null;
53-
this.quality = NaN;
5453
this.range = null;
5554
this.representation = null;
5655
this.responseType = 'arraybuffer';

src/streaming/vo/metrics/HTTPRequest.js

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -118,11 +118,6 @@ class HTTPRequest {
118118
* @public
119119
*/
120120
this._mediaduration = null;
121-
/**
122-
* The media segment quality
123-
* @public
124-
*/
125-
this._quality = null;
126121
/**
127122
* all the response headers from request.
128123
* @public

0 commit comments

Comments
 (0)