Bug 1374774: P11. Ignore SPS' timing information when comparing two SPS NALs. r=gerald

They can change from one SPS to the next, causing unecessary reconstruction of the decoder.

MozReview-Commit-ID: IhCnLuzGc2i

--HG--
extra : rebase_source : ff6020c10fe9d2eaee7ee8244c92d0c1535239be
This commit is contained in:
Jean-Yves Avenard 2017-06-23 10:14:07 +02:00
parent 6b75c7d481
commit a59cb63f30
2 changed files with 5 additions and 9 deletions

View File

@ -718,11 +718,11 @@ H264::vui_parameters(BitReader& aBr, SPSData& aDest)
READUE(chroma_sample_loc_type_bottom_field, 5);
}
aDest.timing_info_present_flag = aBr.ReadBit();
if (aDest.timing_info_present_flag) {
aDest.num_units_in_tick = aBr.ReadBits(32);
aDest.time_scale = aBr.ReadBits(32);
aDest.fixed_frame_rate_flag = aBr.ReadBit();
bool timing_info_present_flag = aBr.ReadBit();
if (timing_info_present_flag) {
aBr.ReadBits(32); // num_units_in_tick
aBr.ReadBits(32); // time_scale
aBr.ReadBit(); // fixed_frame_rate_flag
}
return true;
}

View File

@ -396,10 +396,6 @@ struct SPSData
*/
uint8_t chroma_sample_loc_type_top_field;
uint8_t chroma_sample_loc_type_bottom_field;
bool timing_info_present_flag;
uint32_t num_units_in_tick;
uint32_t time_scale;
bool fixed_frame_rate_flag;
bool scaling_matrix_present;
uint8_t scaling_matrix4x4[6][16];