X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Ffilm_viewer.h;h=60cde60d00139b10fb6a6546995e1eca66b50196;hb=4f0575fcb518d959e8dcf581ec8181609782b4ef;hp=55356f18815885a0218e752e9b4a29ea4d55e4d2;hpb=0a3f387f5d39da2ca38ec90a9593c1b598040dd7;p=dcpomatic.git diff --git a/src/wx/film_viewer.h b/src/wx/film_viewer.h index 55356f188..60cde60d0 100644 --- a/src/wx/film_viewer.h +++ b/src/wx/film_viewer.h @@ -27,6 +27,7 @@ #include "lib/config.h" #include "lib/player_text.h" #include "lib/timer.h" +#include "lib/signaller.h" #include #include @@ -42,7 +43,7 @@ class ClosedCaptionsDialog; /** @class FilmViewer * @brief A wx widget to view a Film. */ -class FilmViewer +class FilmViewer : public Signaller { public: FilmViewer (wxWindow *); @@ -69,7 +70,7 @@ public: void seek_by (dcpomatic::DCPTime by, bool accurate); /** @return our `playhead' position; this may not lie exactly on a frame boundary */ dcpomatic::DCPTime position () const { - return _video_position; + return _video_view->position(); } dcpomatic::DCPTime one_video_frame () const; @@ -77,6 +78,7 @@ public: bool stop (); void suspend (); void resume (); + bool playing () const { return _playing; } @@ -90,16 +92,18 @@ public: void slow_refresh (); - int dropped () const { - return _dropped; - } + dcpomatic::DCPTime time () const; + boost::optional audio_time () const; + + int dropped () const; + int gets () const; int audio_callback (void* out, unsigned int frames); #ifdef DCPOMATIC_VARIANT_SWAROOP void set_background_image (bool b) { _background_image = b; - refresh_view (); + _video_view->update (); } bool background_image () const { @@ -108,27 +112,13 @@ public: #endif StateTimer const & state_timer () const { - return _state_timer; - } - - StateTimer& state_timer () { - return _state_timer; - } - - int gets () const { - return _gets; + return _video_view->state_timer (); } /* Some accessors and utility methods that VideoView classes need */ dcp::Size out_size () const { return _out_size; } - dcp::Size inter_size () const { - return _inter_size; - } - Position inter_position () const { - return _inter_position; - } bool outline_content () const { return _outline_content; } @@ -138,10 +128,12 @@ public: boost::shared_ptr butler () const { return _butler; } - int time_until_next_frame () const; + ClosedCaptionsDialog* closed_captions_dialog () const { + return _closed_captions_dialog; + } + void finished (); boost::signals2::signal)> ImageChanged; - boost::signals2::signal PositionChanged; boost::signals2::signal Started; boost::signals2::signal Stopped; /** While playing back we reached the end of the film (emitted from GUI thread) */ @@ -151,23 +143,20 @@ public: private: - /* XXX_b: to remove */ - friend class SimpleVideoView; - void video_view_sized (); void calculate_sizes (); void player_change (ChangeType type, int, bool); void idle_handler (); - void request_idle_get (); + void request_idle_display_next_frame (); void film_change (ChangeType, Film::Property); void recreate_butler (); void config_changed (Config::Property); + void film_length_change (); + void ui_finished (); - dcpomatic::DCPTime time () const; dcpomatic::DCPTime uncorrected_time () const; Frame average_latency () const; - void refresh_view (); bool quick_refresh (); boost::shared_ptr _film; @@ -177,10 +166,6 @@ private: bool _coalesce_player_changes; std::list _pending_player_changes; - dcpomatic::DCPTime _video_position; - Position _inter_position; - dcp::Size _inter_size; - /** Size of our output (including padding if we have any) */ dcp::Size _out_size; @@ -196,13 +181,11 @@ private: mutable boost::mutex _latency_history_mutex; int _latency_history_count; - int _dropped; boost::optional _dcp_decode_reduction; ClosedCaptionsDialog* _closed_captions_dialog; bool _outline_content; - Eyes _eyes; /** true to pad the viewer panel with black, false to use the normal window background colour. */ @@ -212,9 +195,6 @@ private: bool _background_image; #endif - StateTimer _state_timer; - int _gets; - /** true if an get() is required next time we are idle */ bool _idle_get;