generic: remove support for override/unmanaged window type (part 3)

This commit is contained in:
Ivailo Monev 2015-04-20 14:10:36 +00:00
parent 30e798dbf9
commit 0bb66c0c17
11 changed files with 10 additions and 27 deletions

View file

@ -186,8 +186,6 @@ Window_data::Window_data( WId id_P )
role = windows_handler->get_window_role( id_P );
wclass = windows_handler->get_window_class( id_P );
type = kwin_info.windowType( SUPPORTED_WINDOW_TYPES_MASK );
if( type == NET::Override ) // HACK consider non-NETWM fullscreens to be normal too
type = NET::Normal;
if( type == NET::Unknown )
type = NET::Normal;
}

View file

@ -28,7 +28,7 @@ namespace KHotKeys
{
const int SUPPORTED_WINDOW_TYPES_MASK = NET::NormalMask | NET::DesktopMask | NET::DockMask
| NET::ToolbarMask | NET::MenuMask | NET::DialogMask | NET::OverrideMask | NET::TopMenuMask
| NET::ToolbarMask | NET::MenuMask | NET::DialogMask | NET::TopMenuMask
| NET::UtilityMask | NET::SplashMask;

View file

@ -156,7 +156,7 @@ void KStart::sendRule() {
}
const int SUPPORTED_WINDOW_TYPES_MASK = NET::NormalMask | NET::DesktopMask | NET::DockMask
| NET::ToolbarMask | NET::MenuMask | NET::DialogMask | NET::OverrideMask | NET::TopMenuMask
| NET::ToolbarMask | NET::MenuMask | NET::DialogMask | NET::TopMenuMask
| NET::UtilityMask | NET::SplashMask;
void KStart::windowAdded(WId w){
@ -382,8 +382,6 @@ int main( int argc, char *argv[] )
windowtype = NET::Menu;
else if ( s == "dialog" )
windowtype = NET::Dialog;
else if ( s == "override" )
windowtype = NET::Override;
else if ( s == "topmenu" )
windowtype = NET::TopMenu;
else
@ -429,9 +427,6 @@ int main( int argc, char *argv[] )
if( i.isSupported( NET::FullScreen )) {
state |= NET::FullScreen;
mask |= NET::FullScreen;
} else {
windowtype = NET::Override;
fullscreen = true;
}
}

View file

@ -286,7 +286,7 @@ void KSysTrayCmd::checkExistingWindows()
}
const int SUPPORTED_WINDOW_TYPES_MASK = NET::NormalMask | NET::DesktopMask | NET::DockMask
| NET::ToolbarMask | NET::MenuMask | NET::DialogMask | NET::OverrideMask | NET::TopMenuMask
| NET::ToolbarMask | NET::MenuMask | NET::DialogMask | NET::TopMenuMask
| NET::UtilityMask | NET::SplashMask;
void KSysTrayCmd::windowAdded(WId w)

View file

@ -620,13 +620,6 @@ void Client::detectNoBorder()
default:
abort();
}
// NET::Override is some strange beast without clear definition, usually
// just meaning "noborder", so let's treat it only as such flag, and ignore it as
// a window type otherwise (SUPPORTED_WINDOW_TYPES_MASK doesn't include it)
if (info->windowType(SUPPORTED_MANAGED_WINDOW_TYPES_MASK | NET::OverrideMask) == NET::Override) {
noborder = true;
app_noborder = true;
}
}
void Client::updateFrameExtents()

View file

@ -82,7 +82,6 @@ RootInfo *RootInfo::create()
NET::ToolbarMask |
NET::MenuMask |
NET::DialogMask |
NET::OverrideMask |
NET::UtilityMask |
NET::SplashMask |
// No compositing window types here unless we support them also as managed window types

View file

@ -277,7 +277,7 @@ bool GroupManagerPrivate::addTask(::TaskManager::Task *task)
}
NET::WindowType type = task->info().windowType(NET::NormalMask | NET::DialogMask |
NET::OverrideMask | NET::UtilityMask);
NET::UtilityMask);
if (type == NET::Utility) {
//kDebug() << "skipping utility window" << task->name();
skip = true;

View file

@ -227,8 +227,7 @@ void TaskManager::windowAdded(WId w)
// ignore NET::Tool and other special window types
NET::WindowType wType = info.windowType(NET::NormalMask | NET::DesktopMask | NET::DockMask |
NET::ToolbarMask | NET::MenuMask | NET::DialogMask |
NET::OverrideMask | NET::TopMenuMask |
NET::UtilityMask | NET::SplashMask);
NET::TopMenuMask | NET::UtilityMask | NET::SplashMask);
if (info.transientFor() > 0) {
const WId transientFor = info.transientFor();
@ -251,7 +250,7 @@ void TaskManager::windowAdded(WId w)
}
}
if (wType != NET::Normal && wType != NET::Override && wType != NET::Unknown &&
if (wType != NET::Normal && wType != NET::Unknown &&
wType != NET::Dialog && wType != NET::Utility) {
return;
}

View file

@ -556,7 +556,7 @@ void Pager::recalculateWindowRects()
KWindowInfo info = KWindowSystem::windowInfo(window, NET::WMGeometry | NET::WMFrameExtents |
NET::WMWindowType | NET::WMDesktop |
NET::WMState | NET::XAWMState | NET::WMVisibleName);
NET::WindowType type = info.windowType(NET::NormalMask | NET::DialogMask | NET::OverrideMask |
NET::WindowType type = info.windowType(NET::NormalMask | NET::DialogMask |
NET::UtilityMask | NET::DesktopMask | NET::DockMask |
NET::TopMenuMask | NET::SplashMask | NET::ToolbarMask |
NET::MenuMask);

View file

@ -208,7 +208,7 @@ void WindowList::showMenu(bool onlyCurrentDesktop)
for (int i = 0; i < windows.count(); ++i) {
KWindowInfo window = KWindowSystem::windowInfo(windows.at(i), (NET::WMGeometry | NET::WMFrameExtents | NET::WMWindowType | NET::WMDesktop | NET::WMState | NET::XAWMState | NET::WMVisibleName));
NET::WindowType type = window.windowType(NET::NormalMask | NET::DialogMask | NET::OverrideMask | NET::UtilityMask | NET::DesktopMask | NET::DockMask | NET::TopMenuMask | NET::SplashMask | NET::ToolbarMask | NET::MenuMask);
NET::WindowType type = window.windowType(NET::NormalMask | NET::DialogMask | NET::UtilityMask | NET::DesktopMask | NET::DockMask | NET::TopMenuMask | NET::SplashMask | NET::ToolbarMask | NET::MenuMask);
if ((onlyCurrentDesktop && !window.isOnDesktop(KWindowSystem::currentDesktop())) || type == NET::Desktop || type == NET::Dock || type == NET::TopMenu || type == NET::Splash || type == NET::Menu || type == NET::Toolbar || window.hasState(NET::SkipPager)) {
windows.removeAt(i);

View file

@ -76,10 +76,9 @@ void WindowsRunner::gatherInfo()
// ignore NET::Tool and other special window types
NET::WindowType wType = info.windowType(NET::NormalMask | NET::DesktopMask | NET::DockMask |
NET::ToolbarMask | NET::MenuMask | NET::DialogMask |
NET::OverrideMask | NET::TopMenuMask |
NET::UtilityMask | NET::SplashMask);
NET::TopMenuMask | NET::UtilityMask | NET::SplashMask);
if (wType != NET::Normal && wType != NET::Override && wType != NET::Unknown &&
if (wType != NET::Normal && wType != NET::Unknown &&
wType != NET::Dialog && wType != NET::Utility) {
continue;
}