Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(121)

Unified Diff: webrtc/modules/video_coding/codecs/test/videoprocessor.h

Issue 2741953002: Step #4: Run VideoProcessor integration test batch mode on task queue. (Closed)
Patch Set: Rebase. Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/video_coding/codecs/test/videoprocessor.h
diff --git a/webrtc/modules/video_coding/codecs/test/videoprocessor.h b/webrtc/modules/video_coding/codecs/test/videoprocessor.h
index 242844387a77b6eada4d42d9ca5838acd0b5e483..93482c4f476ae6cf6144f5ce47d80b76848b01b6 100644
--- a/webrtc/modules/video_coding/codecs/test/videoprocessor.h
+++ b/webrtc/modules/video_coding/codecs/test/videoprocessor.h
@@ -132,12 +132,14 @@ struct TestConfig {
// purposes.
class VideoProcessor {
public:
- virtual ~VideoProcessor() {}
+ virtual ~VideoProcessor() = default;
// Performs initial calculations about frame size, sets up callbacks etc.
// Returns false if an error has occurred, in addition to printing to stderr.
virtual bool Init() = 0;
+ virtual void DeregisterCallbacks() = 0;
+
// Processes a single frame. Returns true as long as there's more frames
// available in the source clip.
// Frame number must be an integer >= 0.
@@ -172,8 +174,9 @@ class VideoProcessorImpl : public VideoProcessor {
FrameWriter* source_frame_writer,
IvfFileWriter* encoded_frame_writer,
FrameWriter* decoded_frame_writer);
- virtual ~VideoProcessorImpl();
+ ~VideoProcessorImpl() override;
bool Init() override;
+ void DeregisterCallbacks() override;
bool ProcessFrame(int frame_number) override;
private:

Powered by Google App Engine
This is Rietveld 408576698