From e744e433d5928f53e4f60505fb707642bf5d1c66 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E6=88=98=E9=A9=AC?= <chunmao.guo@bambulab.com>
Date: Thu, 18 Jan 2024 18:11:41 +0800
Subject: [PATCH] FIX: Revert "[STUDIO-4284] not set max height of liveview
 window"

This reverts commit 0312aee4d9b92e23884be8802da9801ff3b9fe93.

Reason for revert: STUDIO-5653
Jira: STUDIO-5653

Change-Id: If9d5f3e63968a0a54f9af1a2dae8f95f7f1f3f80
---
 src/slic3r/GUI/MediaPlayCtrl.cpp | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/src/slic3r/GUI/MediaPlayCtrl.cpp b/src/slic3r/GUI/MediaPlayCtrl.cpp
index 60a1ce223..086d10458 100644
--- a/src/slic3r/GUI/MediaPlayCtrl.cpp
+++ b/src/slic3r/GUI/MediaPlayCtrl.cpp
@@ -760,20 +760,20 @@ void wxMediaCtrl2::DoSetSize(int x, int y, int width, int height, int sizeFlags)
 #else
     wxMediaCtrl::DoSetSize(x, y, width, height, sizeFlags);
 #endif
-    //if (sizeFlags & wxSIZE_USE_EXISTING) return;
-    //wxSize size = GetVideoSize();
-    //if (size.GetWidth() <= 0)
-    //    size = wxSize{16, 9};
-    //int maxHeight = (width * size.GetHeight() + size.GetHeight() - 1) / size.GetWidth();
-    //if (maxHeight != GetMaxHeight()) {
-    //    // BOOST_LOG_TRIVIAL(info) << "wxMediaCtrl2::DoSetSize: width: " << width << ", height: " << height << ", maxHeight: " << maxHeight;
-    //    SetMaxSize({-1, maxHeight});
-    //    CallAfter([this] {
-    //        if (auto p = GetParent()) {
-    //            p->Layout();
-    //            p->Refresh();
-    //        }
-    //    });
-    //}
+    if (sizeFlags & wxSIZE_USE_EXISTING) return;
+    wxSize size = GetVideoSize();
+    if (size.GetWidth() <= 0)
+        size = wxSize{16, 9};
+    int maxHeight = (width * size.GetHeight() + size.GetHeight() - 1) / size.GetWidth();
+    if (maxHeight != GetMaxHeight()) {
+        // BOOST_LOG_TRIVIAL(info) << "wxMediaCtrl2::DoSetSize: width: " << width << ", height: " << height << ", maxHeight: " << maxHeight;
+        SetMaxSize({-1, maxHeight});
+        CallAfter([this] {
+            if (auto p = GetParent()) {
+                p->Layout();
+                p->Refresh();
+            }
+        });
+    }
 }