|
 |
a9add1 |
From b9431532ba696a39e7d35597f4083ea356b63eb3 Mon Sep 17 00:00:00 2001
|
|
|
a9add1 |
From: =?UTF-8?q?Caol=C3=A1n=20McNamara?= <caolanm@redhat.com>
|
|
|
a9add1 |
Date: Wed, 16 Dec 2015 21:30:22 +0000
|
|
|
a9add1 |
Subject: [PATCH] remove newly unused WB_NEEDSFOCUS and fragile FLOAT_FOCUSABLE
|
|
|
a9add1 |
MIME-Version: 1.0
|
|
|
a9add1 |
Content-Type: text/plain; charset=UTF-8
|
|
|
a9add1 |
Content-Transfer-Encoding: 8bit
|
|
|
a9add1 |
|
|
|
a9add1 |
Change-Id: Idce03318fbc01039a6c0638879785607970993c0
|
|
|
a9add1 |
Reviewed-on: https://gerrit.libreoffice.org/20746
|
|
|
a9add1 |
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
|
|
|
a9add1 |
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
|
a9add1 |
---
|
|
|
a9add1 |
include/tools/wintypes.hxx | 1 -
|
|
|
a9add1 |
vcl/inc/salframe.hxx | 2 -
|
|
|
a9add1 |
vcl/inc/unx/gtk/gtkframe.hxx | 3 +-
|
|
|
a9add1 |
vcl/osx/salframeview.mm | 1807 -----------------------------------------
|
|
|
a9add1 |
vcl/source/window/brdwin.cxx | 2 +-
|
|
|
a9add1 |
vcl/source/window/window.cxx | 4 +-
|
|
|
a9add1 |
vcl/unx/gtk/gtksalframe.cxx | 11 +-
|
|
|
a9add1 |
vcl/unx/gtk3/gtk3gtkframe.cxx | 9 +-
|
|
|
a9add1 |
8 files changed, 8 insertions(+), 1831 deletions(-)
|
|
|
a9add1 |
delete mode 100644 vcl/osx/salframeview.mm
|
|
|
a9add1 |
|
|
|
a9add1 |
diff --git a/include/tools/wintypes.hxx b/include/tools/wintypes.hxx
|
|
|
a9add1 |
index 897e635..3943cc4 100644
|
|
|
a9add1 |
--- a/include/tools/wintypes.hxx
|
|
|
a9add1 |
+++ b/include/tools/wintypes.hxx
|
|
|
a9add1 |
@@ -180,7 +180,6 @@ WinBits const WB_NOSHADOW = SAL_CONST_INT64(0x400000000);
|
|
|
a9add1 |
WinBits const WB_TOOLTIPWIN = SAL_CONST_INT64(0x800000000);
|
|
|
a9add1 |
WinBits const WB_OWNERDRAWDECORATION = SAL_CONST_INT64(0x2000000000);
|
|
|
a9add1 |
WinBits const WB_DEFAULTWIN = SAL_CONST_INT64(0x4000000000);
|
|
|
a9add1 |
-WinBits const WB_NEEDSFOCUS = SAL_CONST_INT64(0x1000000000);
|
|
|
a9add1 |
WinBits const WB_POPUP = SAL_CONST_INT64(0x20000000);
|
|
|
a9add1 |
|
|
|
a9add1 |
WinBits const WB_HSCROLL = WB_HORZ;
|
|
|
a9add1 |
diff --git a/vcl/inc/salframe.hxx b/vcl/inc/salframe.hxx
|
|
|
a9add1 |
index 9b3d52e..1016d3c 100644
|
|
|
a9add1 |
--- a/vcl/inc/salframe.hxx
|
|
|
a9add1 |
+++ b/vcl/inc/salframe.hxx
|
|
|
a9add1 |
@@ -76,8 +76,6 @@ struct SystemEnvData;
|
|
|
a9add1 |
#define SAL_FRAME_STYLE_SYSTEMCHILD ((sal_uLong)0x08000000)
|
|
|
a9add1 |
// floating window
|
|
|
a9add1 |
#define SAL_FRAME_STYLE_FLOAT ((sal_uLong)0x20000000)
|
|
|
a9add1 |
-// floating window that needs to be focusable
|
|
|
a9add1 |
-#define SAL_FRAME_STYLE_FLOAT_FOCUSABLE ((sal_uLong)0x04000000)
|
|
|
a9add1 |
// toolwindows should be painted with a smaller decoration
|
|
|
a9add1 |
#define SAL_FRAME_STYLE_TOOLWINDOW ((sal_uLong)0x40000000)
|
|
|
a9add1 |
// the window containing the intro bitmap, aka splashscreen
|
|
|
a9add1 |
diff --git a/vcl/inc/unx/gtk/gtkframe.hxx b/vcl/inc/unx/gtk/gtkframe.hxx
|
|
|
a9add1 |
index a55ff2c..479a41b 100644
|
|
|
a9add1 |
--- a/vcl/inc/unx/gtk/gtkframe.hxx
|
|
|
a9add1 |
+++ b/vcl/inc/unx/gtk/gtkframe.hxx
|
|
|
a9add1 |
@@ -292,8 +292,7 @@ class GtkSalFrame : public SalFrame, public X11WindowProvider
|
|
|
a9add1 |
return
|
|
|
a9add1 |
(m_nStyle & SAL_FRAME_STYLE_FLOAT) && // only a float can be floatgrab
|
|
|
a9add1 |
!(m_nStyle & SAL_FRAME_STYLE_TOOLTIP) && // tool tips are not
|
|
|
a9add1 |
- !(m_nStyle & SAL_FRAME_STYLE_OWNERDRAWDECORATION) && // toolbars are also not
|
|
|
a9add1 |
- !(m_nStyle & SAL_FRAME_STYLE_FLOAT_FOCUSABLE); // focusable floats are not
|
|
|
a9add1 |
+ !(m_nStyle & SAL_FRAME_STYLE_OWNERDRAWDECORATION); // toolbars are also not
|
|
|
a9add1 |
}
|
|
|
a9add1 |
|
|
|
a9add1 |
bool isChild( bool bPlug = true, bool bSysChild = true )
|
|
|
a9add1 |
diff --git a/vcl/source/window/brdwin.cxx b/vcl/source/window/brdwin.cxx
|
|
|
a9add1 |
index f57ab37..0e354bd 100644
|
|
|
a9add1 |
--- a/vcl/source/window/brdwin.cxx
|
|
|
a9add1 |
+++ b/vcl/source/window/brdwin.cxx
|
|
|
a9add1 |
@@ -1750,7 +1750,7 @@ void ImplBorderWindow::ImplInit( vcl::Window* pParent,
|
|
|
a9add1 |
{
|
|
|
a9add1 |
// remove all unwanted WindowBits
|
|
|
a9add1 |
WinBits nOrgStyle = nStyle;
|
|
|
a9add1 |
- WinBits nTestStyle = (WB_MOVEABLE | WB_SIZEABLE | WB_ROLLABLE | WB_PINABLE | WB_CLOSEABLE | WB_STANDALONE | WB_DIALOGCONTROL | WB_NODIALOGCONTROL | WB_SYSTEMFLOATWIN | WB_INTROWIN | WB_DEFAULTWIN | WB_TOOLTIPWIN | WB_NOSHADOW | WB_OWNERDRAWDECORATION | WB_SYSTEMCHILDWINDOW | WB_NEEDSFOCUS | WB_POPUP);
|
|
|
a9add1 |
+ WinBits nTestStyle = (WB_MOVEABLE | WB_SIZEABLE | WB_ROLLABLE | WB_PINABLE | WB_CLOSEABLE | WB_STANDALONE | WB_DIALOGCONTROL | WB_NODIALOGCONTROL | WB_SYSTEMFLOATWIN | WB_INTROWIN | WB_DEFAULTWIN | WB_TOOLTIPWIN | WB_NOSHADOW | WB_OWNERDRAWDECORATION | WB_SYSTEMCHILDWINDOW | WB_POPUP);
|
|
|
a9add1 |
if ( nTypeStyle & BORDERWINDOW_STYLE_APP )
|
|
|
a9add1 |
nTestStyle |= WB_APP;
|
|
|
a9add1 |
nStyle &= nTestStyle;
|
|
|
a9add1 |
diff --git a/vcl/source/window/window.cxx b/vcl/source/window/window.cxx
|
|
|
a9add1 |
index 808d683..87b8e93 100644
|
|
|
a9add1 |
--- a/vcl/source/window/window.cxx
|
|
|
a9add1 |
+++ b/vcl/source/window/window.cxx
|
|
|
a9add1 |
@@ -911,7 +911,7 @@ void Window::ImplInit( vcl::Window* pParent, WinBits nStyle, SystemParentData* p
|
|
|
a9add1 |
nBorderTypeStyle |= BORDERWINDOW_STYLE_FRAME;
|
|
|
a9add1 |
nStyle |= WB_BORDER;
|
|
|
a9add1 |
}
|
|
|
a9add1 |
- VclPtrInstance<ImplBorderWindow> pBorderWin( pParent, nStyle & (WB_BORDER | WB_DIALOGCONTROL | WB_NODIALOGCONTROL | WB_NEEDSFOCUS), nBorderTypeStyle );
|
|
|
a9add1 |
+ VclPtrInstance<ImplBorderWindow> pBorderWin( pParent, nStyle & (WB_BORDER | WB_DIALOGCONTROL | WB_NODIALOGCONTROL), nBorderTypeStyle );
|
|
|
a9add1 |
((vcl::Window*)pBorderWin)->mpWindowImpl->mpClientWindow = this;
|
|
|
a9add1 |
pBorderWin->GetBorder( mpWindowImpl->mnLeftBorder, mpWindowImpl->mnTopBorder, mpWindowImpl->mnRightBorder, mpWindowImpl->mnBottomBorder );
|
|
|
a9add1 |
mpWindowImpl->mpBorderWindow = pBorderWin;
|
|
|
a9add1 |
@@ -966,8 +966,6 @@ void Window::ImplInit( vcl::Window* pParent, WinBits nStyle, SystemParentData* p
|
|
|
a9add1 |
nFrameStyle = SAL_FRAME_STYLE_FLOAT;
|
|
|
a9add1 |
if( nStyle & WB_OWNERDRAWDECORATION )
|
|
|
a9add1 |
nFrameStyle |= (SAL_FRAME_STYLE_OWNERDRAWDECORATION | SAL_FRAME_STYLE_NOSHADOW);
|
|
|
a9add1 |
- if( nStyle & WB_NEEDSFOCUS )
|
|
|
a9add1 |
- nFrameStyle |= SAL_FRAME_STYLE_FLOAT_FOCUSABLE;
|
|
|
a9add1 |
}
|
|
|
a9add1 |
else if( mpWindowImpl->mbFloatWin )
|
|
|
a9add1 |
nFrameStyle |= SAL_FRAME_STYLE_TOOLWINDOW;
|
|
|
a9add1 |
diff --git a/vcl/unx/gtk/gtksalframe.cxx b/vcl/unx/gtk/gtksalframe.cxx
|
|
|
a9add1 |
index 03f25dd..bd9662c 100644
|
|
|
a9add1 |
--- a/vcl/unx/gtk/gtksalframe.cxx
|
|
|
a9add1 |
+++ b/vcl/unx/gtk/gtksalframe.cxx
|
|
|
a9add1 |
@@ -1373,8 +1373,7 @@ void GtkSalFrame::Init( SalFrame* pParent, sal_uLong nStyle )
|
|
|
a9add1 |
m_nStyle = nStyle;
|
|
|
a9add1 |
|
|
|
a9add1 |
GtkWindowType eWinType = ( (nStyle & SAL_FRAME_STYLE_FLOAT) &&
|
|
|
a9add1 |
- ! (nStyle & (SAL_FRAME_STYLE_OWNERDRAWDECORATION|
|
|
|
a9add1 |
- SAL_FRAME_STYLE_FLOAT_FOCUSABLE))
|
|
|
a9add1 |
+ ! (nStyle & SAL_FRAME_STYLE_OWNERDRAWDECORATION)
|
|
|
a9add1 |
)
|
|
|
a9add1 |
? GTK_WINDOW_POPUP : GTK_WINDOW_TOPLEVEL;
|
|
|
a9add1 |
|
|
|
a9add1 |
@@ -1419,7 +1418,7 @@ void GtkSalFrame::Init( SalFrame* pParent, sal_uLong nStyle )
|
|
|
a9add1 |
bool bDecoHandling =
|
|
|
a9add1 |
! isChild() &&
|
|
|
a9add1 |
( ! (nStyle & SAL_FRAME_STYLE_FLOAT) ||
|
|
|
a9add1 |
- (nStyle & (SAL_FRAME_STYLE_OWNERDRAWDECORATION|SAL_FRAME_STYLE_FLOAT_FOCUSABLE) ) );
|
|
|
a9add1 |
+ (nStyle & SAL_FRAME_STYLE_OWNERDRAWDECORATION ) );
|
|
|
a9add1 |
|
|
|
a9add1 |
if( bDecoHandling )
|
|
|
a9add1 |
{
|
|
|
a9add1 |
@@ -1441,10 +1440,6 @@ void GtkSalFrame::Init( SalFrame* pParent, sal_uLong nStyle )
|
|
|
a9add1 |
eType = GDK_WINDOW_TYPE_HINT_TOOLBAR;
|
|
|
a9add1 |
lcl_set_accept_focus( GTK_WINDOW(m_pWindow), false, true );
|
|
|
a9add1 |
}
|
|
|
a9add1 |
- else if( (nStyle & SAL_FRAME_STYLE_FLOAT_FOCUSABLE) )
|
|
|
a9add1 |
- {
|
|
|
a9add1 |
- eType = GDK_WINDOW_TYPE_HINT_UTILITY;
|
|
|
a9add1 |
- }
|
|
|
a9add1 |
#if !GTK_CHECK_VERSION(3,0,0)
|
|
|
a9add1 |
if( (nStyle & SAL_FRAME_STYLE_PARTIAL_FULLSCREEN )
|
|
|
a9add1 |
&& getDisplay()->getWMAdaptor()->isLegacyPartialFullscreen() )
|
|
|
a9add1 |
@@ -2770,7 +2765,7 @@ void GtkSalFrame::ToTop( sal_uInt16 nFlags )
|
|
|
a9add1 |
* to our window - which it of course won't since our input hint
|
|
|
a9add1 |
* is set to false.
|
|
|
a9add1 |
*/
|
|
|
a9add1 |
- if( (m_nStyle & (SAL_FRAME_STYLE_OWNERDRAWDECORATION|SAL_FRAME_STYLE_FLOAT_FOCUSABLE)) )
|
|
|
a9add1 |
+ if (m_nStyle & SAL_FRAME_STYLE_OWNERDRAWDECORATION)
|
|
|
a9add1 |
{
|
|
|
a9add1 |
// sad but true: this can cause an XError, we need to catch that
|
|
|
a9add1 |
// to do this we need to synchronize with the XServer
|
|
|
a9add1 |
diff --git a/vcl/unx/gtk3/gtk3gtkframe.cxx b/vcl/unx/gtk3/gtk3gtkframe.cxx
|
|
|
a9add1 |
index d41be9d..4ac3da7 100644
|
|
|
a9add1 |
--- a/vcl/unx/gtk3/gtk3gtkframe.cxx
|
|
|
a9add1 |
+++ b/vcl/unx/gtk3/gtk3gtkframe.cxx
|
|
|
a9add1 |
@@ -1121,8 +1121,7 @@ void GtkSalFrame::Init( SalFrame* pParent, sal_uLong nStyle )
|
|
|
a9add1 |
m_nStyle = nStyle;
|
|
|
a9add1 |
|
|
|
a9add1 |
GtkWindowType eWinType = ( (nStyle & SAL_FRAME_STYLE_FLOAT) &&
|
|
|
a9add1 |
- ! (nStyle & (SAL_FRAME_STYLE_OWNERDRAWDECORATION|
|
|
|
a9add1 |
- SAL_FRAME_STYLE_FLOAT_FOCUSABLE))
|
|
|
a9add1 |
+ ! (nStyle & SAL_FRAME_STYLE_OWNERDRAWDECORATION)
|
|
|
a9add1 |
)
|
|
|
a9add1 |
? GTK_WINDOW_POPUP : GTK_WINDOW_TOPLEVEL;
|
|
|
a9add1 |
|
|
|
a9add1 |
@@ -1167,7 +1166,7 @@ void GtkSalFrame::Init( SalFrame* pParent, sal_uLong nStyle )
|
|
|
a9add1 |
bool bDecoHandling =
|
|
|
a9add1 |
! isChild() &&
|
|
|
a9add1 |
( ! (nStyle & SAL_FRAME_STYLE_FLOAT) ||
|
|
|
a9add1 |
- (nStyle & (SAL_FRAME_STYLE_OWNERDRAWDECORATION|SAL_FRAME_STYLE_FLOAT_FOCUSABLE) ) );
|
|
|
a9add1 |
+ (nStyle & SAL_FRAME_STYLE_OWNERDRAWDECORATION ) );
|
|
|
a9add1 |
|
|
|
a9add1 |
if( bDecoHandling )
|
|
|
a9add1 |
{
|
|
|
a9add1 |
@@ -1190,10 +1189,6 @@ void GtkSalFrame::Init( SalFrame* pParent, sal_uLong nStyle )
|
|
|
a9add1 |
gtk_window_set_accept_focus(GTK_WINDOW(m_pWindow), false);
|
|
|
a9add1 |
gtk_window_set_decorated(GTK_WINDOW(m_pWindow), false);
|
|
|
a9add1 |
}
|
|
|
a9add1 |
- else if( (nStyle & SAL_FRAME_STYLE_FLOAT_FOCUSABLE) )
|
|
|
a9add1 |
- {
|
|
|
a9add1 |
- eType = GDK_WINDOW_TYPE_HINT_UTILITY;
|
|
|
a9add1 |
- }
|
|
|
a9add1 |
gtk_window_set_type_hint( GTK_WINDOW(m_pWindow), eType );
|
|
|
a9add1 |
gtk_window_set_gravity( GTK_WINDOW(m_pWindow), GDK_GRAVITY_STATIC );
|
|
|
a9add1 |
}
|
|
|
a9add1 |
--
|
|
|
a9add1 |
2.5.0
|
|
|
a9add1 |
|