+static void
+note_handler (dcp::NoteType, std::string)
+{
+
+}
+
+/** Test some seeks towards the start of a DCP with awkward subtitles; see mantis #1085
+ * and a number of others. I thought this was a player seek bug but in fact it was
+ * caused by the subtitle starting just after the start of the video frame and hence
+ * being faded out.
+ */
+BOOST_AUTO_TEST_CASE (player_seek_test)
+{
+ shared_ptr<Film> film (new Film (optional<boost::filesystem::path>()));
+ shared_ptr<DCPContent> dcp (new DCPContent (film, private_data / "awkward_subs"));
+ film->examine_and_add_content (dcp, true);
+ BOOST_REQUIRE (!wait_for_jobs ());
+ dcp->only_text()->set_use (true);
+
+ shared_ptr<Player> player (new Player (film, film->playlist()));
+ player->set_fast ();
+ player->set_always_burn_open_subtitles ();
+ player->set_play_referenced ();
+
+ shared_ptr<Butler> butler (new Butler (player, film->log(), AudioMapping(), 2));
+ butler->disable_audio();
+
+ for (int i = 0; i < 10; ++i) {
+ DCPTime t = DCPTime::from_frames (i, 24);
+ butler->seek (t, true);
+ pair<shared_ptr<PlayerVideo>, DCPTime> video = butler->get_video();
+ BOOST_CHECK_EQUAL(video.second.get(), t.get());
+ write_image(video.first->image(note_handler, PlayerVideo::always_rgb, false, true), String::compose("build/test/player_seek_test_%1.png", i), "RGB");
+ /* This 0.011 is empirically chosen (hopefully) to accept changes in rendering between the reference and a test machine
+ (17.10 and 16.04 seem to anti-alias a little differently) but to reject gross errors e.g. missing fonts or missing
+ text altogether.
+ */
+ check_image(String::compose("test/data/player_seek_test_%1.png", i), String::compose("build/test/player_seek_test_%1.png", i), 0.011);
+ }
+}
+
+/** Test some more seeks towards the start of a DCP with awkward subtitles */
+BOOST_AUTO_TEST_CASE (player_seek_test2)
+{
+ shared_ptr<Film> film (new Film (optional<boost::filesystem::path>()));
+ shared_ptr<DCPContent> dcp (new DCPContent (film, private_data / "awkward_subs2"));
+ film->examine_and_add_content (dcp, true);
+ BOOST_REQUIRE (!wait_for_jobs ());
+ dcp->only_text()->set_use (true);
+
+ shared_ptr<Player> player (new Player (film, film->playlist()));
+ player->set_fast ();
+ player->set_always_burn_open_subtitles ();
+ player->set_play_referenced ();
+
+ shared_ptr<Butler> butler (new Butler (player, film->log(), AudioMapping(), 2));
+ butler->disable_audio();
+
+ butler->seek(DCPTime::from_seconds(5), true);
+
+ for (int i = 0; i < 10; ++i) {
+ DCPTime t = DCPTime::from_seconds(5) + DCPTime::from_frames (i, 24);
+ butler->seek (t, true);
+ pair<shared_ptr<PlayerVideo>, DCPTime> video = butler->get_video();
+ BOOST_CHECK_EQUAL(video.second.get(), t.get());
+ write_image(video.first->image(note_handler, PlayerVideo::always_rgb, false, true), String::compose("build/test/player_seek_test2_%1.png", i), "RGB");
+ check_image(String::compose("test/data/player_seek_test2_%1.png", i), String::compose("build/test/player_seek_test2_%1.png", i), 0.011);
+ }
+}
+
+/** Test a bug when trimmed content follows other content */
+BOOST_AUTO_TEST_CASE (player_trim_test)
+{
+ shared_ptr<Film> film = new_test_film2 ("player_trim_test");
+ shared_ptr<Content> A = content_factory(film, "test/data/flat_red.png").front();
+ film->examine_and_add_content (A);
+ BOOST_REQUIRE (!wait_for_jobs ());
+ A->video->set_length (10 * 24);
+ shared_ptr<Content> B = content_factory(film, "test/data/flat_red.png").front();
+ film->examine_and_add_content (B);
+ BOOST_REQUIRE (!wait_for_jobs ());
+ B->video->set_length (10 * 24);
+ B->set_position (DCPTime::from_seconds (10));
+ B->set_trim_start (ContentTime::from_seconds (2));
+
+ film->make_dcp ();
+ BOOST_REQUIRE (!wait_for_jobs ());
+}