Chromium Code Reviews| Index: media/base/text_track.h |
| diff --git a/media/base/text_track.h b/media/base/text_track.h |
| index 01a2ed727f90bc3ce9880796185b9ab782536c5d..a187e1f3eb67c2299ff448f5022155a59330ce19 100644 |
| --- a/media/base/text_track.h |
| +++ b/media/base/text_track.h |
| @@ -5,6 +5,7 @@ |
| #ifndef MEDIA_BASE_TEXT_TRACK_H_ |
| #define MEDIA_BASE_TEXT_TRACK_H_ |
| +#include <map> |
| #include <string> |
| #include "base/callback.h" |
| @@ -32,10 +33,32 @@ class TextTrack { |
| const std::string& settings) = 0; |
| }; |
| -typedef base::Callback<scoped_ptr<TextTrack> |
| +typedef base::Callback<void |
| + (scoped_ptr<TextTrack>)> AddTextTrackDoneCB; |
| + |
| +typedef base::Callback<void |
| (TextKind kind, |
|
acolwell GONE FROM CHROMIUM
2013/10/21 20:10:40
nit: Use const TextTrackConfig& here instead.
Matthew Heaney (Chromium)
2013/10/23 05:09:01
Done.
|
| const std::string& label, |
| - const std::string& language)> AddTextTrackCB; |
| + const std::string& language, |
| + const AddTextTrackDoneCB& done_cb)> AddTextTrackCB; |
| + |
| +class TextTrackConfig { |
| + public: |
| + TextTrackConfig(); |
| + TextTrackConfig(TextKind kind, |
| + const std::string& label, |
| + const std::string& language); |
| + |
| + // Returns true if all fields in |config| match this config. |
| + bool Matches(const TextTrackConfig& config) const; |
| + |
|
acolwell GONE FROM CHROMIUM
2013/10/21 20:10:40
Add kind(), label(), language() accessors.
Matthew Heaney (Chromium)
2013/10/23 05:09:01
Done.
|
| + private: |
| + TextKind kind_; |
| + std::string label_; |
| + std::string language_; |
| +}; |
| + |
| +typedef std::map<int, TextTrackConfig> TextTrackConfigMap; |
| } // namespace media |