From 14dfbfe1da056d9570c85df8b5d5bed3943d46e9 Mon Sep 17 00:00:00 2001 From: Jan-Marek Glogowski Date: Sat, 11 Jun 2022 01:49:02 +0200 Subject: Drop SalMenu::DrawMenuBar (is WIN-only) This follow-up on commit 483f3125ecd4744cbcc2f8fd3cdf243d81a601ff ("WIN drop effectively empty ImplDrawMenuBar") should probably have been included in it. Windows was the only implementation of that API, but the API is just called for a native menubar, which Windows doesn't use (VisibleMenuBar returns false). Maybe Windows SalFrame::SetMenu should also be dropped? Change-Id: Ic3600e66c91ce9d56d7f911bce5674e0a9ee833c Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135633 Tested-by: Jenkins Reviewed-by: Jan-Marek Glogowski --- vcl/source/window/menubarwindow.cxx | 3 --- 1 file changed, 3 deletions(-) (limited to 'vcl/source') diff --git a/vcl/source/window/menubarwindow.cxx b/vcl/source/window/menubarwindow.cxx index 51bad55d4cba..0109bb559503 100644 --- a/vcl/source/window/menubarwindow.cxx +++ b/vcl/source/window/menubarwindow.cxx @@ -863,10 +863,7 @@ void MenuBarWindow::Paint(vcl::RenderContext& rRenderContext, const tools::Recta // no VCL paint if native menus if (m_pMenu->ImplGetSalMenu() && m_pMenu->ImplGetSalMenu()->VisibleMenuBar()) - { - ImplGetFrame()->DrawMenuBar(); return; - } // Make sure that all actual rendering happens in one go to avoid flicker. vcl::BufferDevice pBuffer(this, rRenderContext); -- cgit