Change re{set,fresh}_metadata to take a film and a size from the player.

5 files changed, 11 insertions, 9 deletions

diff --git a/src/lib/player_video.cc b/src/lib/player_video.cc
index 8c1b95bba..4f8acd5e2 100644
--- a/src/lib/player_video.cc
+++ b/src/lib/player_video.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2013-2018 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2013-2020 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -25,6 +25,7 @@
 #include "image_proxy.h"
 #include "j2k_image_proxy.h"
 #include "film.h"
+#include "player.h"
 #include <dcp/raw_convert.h>
 extern "C" {
 #include <libavutil/pixfmt.h>
@@ -330,7 +331,7 @@ PlayerVideo::shallow_copy () const
  *  @return true if this was possible, false if not.
  */
 bool
-PlayerVideo::reset_metadata (shared_ptr<const Film> film, dcp::Size video_container_size, dcp::Size film_frame_size)
+PlayerVideo::reset_metadata (shared_ptr<const Film> film, dcp::Size player_video_container_size)
 {
 	shared_ptr<Content> content = _content.lock();
 	if (!content || !_video_frame) {
@@ -339,8 +340,8 @@ PlayerVideo::reset_metadata (shared_ptr<const Film> film, dcp::Size video_contai
 
 	_crop = content->video->crop();
 	_fade = content->video->fade(film, _video_frame.get());
-	_inter_size = scale_for_display(content->video->scaled_size(film_frame_size), video_container_size, film_frame_size);
-	_out_size = video_container_size;
+	_inter_size = scale_for_display(content->video->scaled_size(film->frame_size()), player_video_container_size, film->frame_size());
+	_out_size = player_video_container_size;
 	_colour_conversion = content->video->colour_conversion();
 	_video_range = content->video->range();
 
diff --git a/src/lib/player_video.h b/src/lib/player_video.h
index 56cfbd100..0952eafb9 100644
--- a/src/lib/player_video.h
+++ b/src/lib/player_video.h
@@ -76,7 +76,7 @@ public:
 	void add_metadata (xmlpp::Node* node) const;
 	void write_to_socket (boost::shared_ptr<Socket> socket) const;
 
-	bool reset_metadata (boost::shared_ptr<const Film> film, dcp::Size video_container_size, dcp::Size film_frame_size);
+	bool reset_metadata (boost::shared_ptr<const Film> film, dcp::Size player_video_container_size);
 
 	bool has_j2k () const;
 	boost::shared_ptr<const dcp::Data> j2k () const;
diff --git a/src/wx/film_viewer.cc b/src/wx/film_viewer.cc
index ee0322d6f..e91cdc9a0 100644
--- a/src/wx/film_viewer.cc
+++ b/src/wx/film_viewer.cc
@@ -441,7 +441,7 @@ FilmViewer::quick_refresh ()
 	if (!_video_view || !_film || !_player) {
 		return true;
 	}
-	return _video_view->refresh_metadata (_film, _player->video_container_size(), _film->frame_size());
+	return _video_view->refresh_metadata (_film, _player->video_container_size());
 }
 
 void
diff --git a/src/wx/video_view.cc b/src/wx/video_view.cc
index aa3a469da..08b7d2912 100644
--- a/src/wx/video_view.cc
+++ b/src/wx/video_view.cc
@@ -125,14 +125,14 @@ VideoView::start ()
 }
 
 bool
-VideoView::refresh_metadata (shared_ptr<const Film> film, dcp::Size video_container_size, dcp::Size film_frame_size)
+VideoView::refresh_metadata (shared_ptr<const Film> film, dcp::Size player_video_container_size)
 {
 	pair<shared_ptr<PlayerVideo>, dcpomatic::DCPTime> pv = player_video ();
 	if (!pv.first) {
 		return false;
 	}
 
-	if (!pv.first->reset_metadata (film, video_container_size, film_frame_size)) {
+	if (!pv.first->reset_metadata(film, player_video_container_size)) {
 		return false;
 	}
 
diff --git a/src/wx/video_view.h b/src/wx/video_view.h
index 4e0b89ed7..ef5e8107d 100644
--- a/src/wx/video_view.h
+++ b/src/wx/video_view.h
@@ -33,6 +33,7 @@
 class Image;
 class wxWindow;
 class FilmViewer;
+class Player;
 class PlayerVideo;
 
 class VideoView : public ExceptionStore, public boost::noncopyable
@@ -53,7 +54,7 @@ public:
 	virtual bool display_next_frame (bool) = 0;
 
 	void clear ();
-	bool refresh_metadata (boost::shared_ptr<const Film> film, dcp::Size video_container_size, dcp::Size film_frame_size);
+	bool refresh_metadata (boost::shared_ptr<const Film> film, dcp::Size player_video_container_size);
 
 	/** Emitted from the GUI thread when our display changes in size */
 	boost::signals2::signal<void()> Sized;