remove check_victory in play_human_turn

we don't need these because we call check_victory at the end of the
synced_context.

I removed them because check_victory assumes that "this code is run
rarely" which is not the case if we call it there.

src/playmp_controller.cpp

@@ -228,7 +228,7 @@ void playmp_controller::play_human_turn(){
 			}
 
 			play_slice();
-			check_victory();
+			check_end_level();
 			// give a chance to the whiteboard to continue an execute_all_actions
 			resources::whiteboard->continue_execute_all();
 		} catch(const end_level_exception&) {
@@ -294,7 +294,7 @@ void playmp_controller::play_idle_loop()
 			}
 
 			play_slice();
-			check_victory();
+			check_end_level();
 		} catch(const end_level_exception&) {
 			turn_data_->send_data();
 			throw;

src/playsingle_controller.cpp

@@ -798,7 +798,7 @@ void playsingle_controller::play_human_turn() {
 	end_turn_enable(true);
 	while(!end_turn_) {
 		play_slice();
-		check_victory();
+		check_end_level();
 		gui_->draw();
 	}
 }