Mirror-Tools
/
shaka-packager
mirror of https://github.com/shaka-project/shaka-packager.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Use scoped_ptr for box ownership transfer 

MP4Muxer passes ownership of ftyp and moov box to MP4Segmenter.

Change-Id: Iae7c7161560c744d8d3e7e5382450e7d056112ec
This commit is contained in:
Rintaro Kuroiwa 2014-01-08 11:56:59 -08:00
parent 1b5c3b5316
commit 0f3056b551
7 changed files with 23 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
media/mp4/mp4_general_segmenter.cc
View File

@ -16,14 +16,14 @@ namespace media {
namespace mp4 {
MP4GeneralSegmenter::MP4GeneralSegmenter(const MuxerOptions& options,
FileType* ftyp,
Movie* moov)
: MP4Segmenter(options, ftyp, moov),
scoped_ptr<FileType> ftyp,
scoped_ptr<Movie> moov)
: MP4Segmenter(options, ftyp.Pass(), moov.Pass()),
styp_(new SegmentType),
num_segments_(0) {
// Use the same brands for styp as ftyp.
styp_->major_brand = ftyp->major_brand;
styp_->compatible_brands = ftyp->compatible_brands;
styp_->major_brand = MP4Segmenter::ftyp()->major_brand;
styp_->compatible_brands = MP4Segmenter::ftyp()->compatible_brands;
}
MP4GeneralSegmenter::~MP4GeneralSegmenter() {}

4
media/mp4/mp4_general_segmenter.h
View File

@ -29,7 +29,9 @@ struct SegmentType;
class MP4GeneralSegmenter : public MP4Segmenter {
public:
// Caller transfers the ownership of |ftyp| and |moov| to this class.
MP4GeneralSegmenter(const MuxerOptions& options, FileType* ftyp, Movie* moov);
MP4GeneralSegmenter(const MuxerOptions& options,
scoped_ptr<FileType> ftyp,
scoped_ptr<Movie> moov);
~MP4GeneralSegmenter();
// MP4Segmenter implementations.

4
media/mp4/mp4_muxer.cc
View File

@ -93,10 +93,10 @@ Status MP4Muxer::Initialize() {
if (options().single_segment) {
segmenter_.reset(
new MP4VODSegmenter(options(), ftyp.release(), moov.release()));
new MP4VODSegmenter(options(), ftyp.Pass(), moov.Pass()));
} else {
segmenter_.reset(
new MP4GeneralSegmenter(options(), ftyp.release(), moov.release()));
new MP4GeneralSegmenter(options(), ftyp.Pass(), moov.Pass()));
}
return segmenter_->Initialize(encryptor_source(), streams());
}

8
media/mp4/mp4_segmenter.cc
View File

@ -26,12 +26,12 @@ namespace media {
namespace mp4 {
MP4Segmenter::MP4Segmenter(const MuxerOptions& options,
FileType* ftyp,
Movie* moov)
scoped_ptr<FileType> ftyp,
scoped_ptr<Movie> moov)
: options_(options),
fragment_buffer_(new BufferWriter()),
ftyp_(ftyp),
moov_(moov),
ftyp_(ftyp.Pass()),
moov_(moov.Pass()),
moof_(new MovieFragment()),
sidx_(new SegmentIndex()),
segment_initialized_(false),

4
media/mp4/mp4_segmenter.h
View File

@ -40,7 +40,9 @@ struct SegmentIndex;
class MP4Segmenter {
public:
// Caller transfers the ownership of |ftyp| and |moov| to this class.
MP4Segmenter(const MuxerOptions& options, FileType* ftyp, Movie* moov);
MP4Segmenter(const MuxerOptions& options,
scoped_ptr<FileType> ftyp,
scoped_ptr<Movie> moov);
virtual ~MP4Segmenter();
// Initialize the segmenter. Caller retains the ownership of

6
media/mp4/mp4_vod_segmenter.cc
View File

@ -14,9 +14,9 @@ namespace media {
namespace mp4 {
MP4VODSegmenter::MP4VODSegmenter(const MuxerOptions& options,
FileType* ftyp,
Movie* moov)
: MP4Segmenter(options, ftyp, moov) {}
scoped_ptr<FileType> ftyp,
scoped_ptr<Movie> moov)
: MP4Segmenter(options, ftyp.Pass(), moov.Pass()) {}
MP4VODSegmenter::~MP4VODSegmenter() {}
Status MP4VODSegmenter::Initialize(EncryptorSource* encryptor_source,

4
media/mp4/mp4_vod_segmenter.h
View File

@ -26,7 +26,9 @@ namespace mp4 {
class MP4VODSegmenter : public MP4Segmenter {
public:
// Caller transfers the ownership of |ftyp| and |moov| to this class.
MP4VODSegmenter(const MuxerOptions& options, FileType* ftyp, Movie* moov);
MP4VODSegmenter(const MuxerOptions& options,
scoped_ptr<FileType> ftyp,
scoped_ptr<Movie> moov);
~MP4VODSegmenter();
// MP4Segmenter implementations.