DSZone: donot check if window list is empty when zone sends stackChanged signal. 35/242335/1
authorSooChan Lim <sc1.lim@samsung.com>
Tue, 25 Aug 2020 07:57:52 +0000 (16:57 +0900)
committerSung-Jin Park <sj76.park@samsung.com>
Tue, 25 Aug 2020 12:39:43 +0000 (21:39 +0900)
Change-Id: Iabadf761ef4fdc4b9ad8bbf8771f4921e16aff95

src/DSZone/DSZone.cpp

index 3973120..6c9a31e 100644 (file)
@@ -235,8 +235,9 @@ void DSZone::__prependWindowList(std::shared_ptr<DSWindow> window)
        __updateWindowOrder();
 
        std::shared_ptr<DSWindow> wTop(__windowList.front());
-       wTop->raiseToTop();
        __windowStackChangedSignal.emit(wTop);
+
+       wTop->raiseToTop();
 }
 
 void DSZone::__appendWindowList(std::shared_ptr<DSWindow> window)
@@ -248,8 +249,9 @@ void DSZone::__appendWindowList(std::shared_ptr<DSWindow> window)
        __updateWindowOrder();
 
        std::shared_ptr<DSWindow> wTop(__windowList.front());
-       wTop->raiseToTop();
        __windowStackChangedSignal.emit(wTop);
+
+       wTop->raiseToTop();
 }
 
 std::shared_ptr<DSWindow> DSZone::__findWindow(DSWaylandSurface *dswlSurface)
@@ -304,11 +306,11 @@ void DSZone::__destroyWindow(std::shared_ptr<DSWindow> window)
        __stackChanged = true;
        __updateWindowOrder();
 
-       if (!__windowList.empty()) {
-               std::shared_ptr<DSWindow> wTop(__windowList.front());
+       std::shared_ptr<DSWindow> wTop(__windowList.front());
+       __windowStackChangedSignal.emit(wTop);
+
+       if (!__windowList.empty())
                wTop->raiseToTop();
-               __windowStackChangedSignal.emit(wTop);
-       }
 }
 
 void DSZone::__updateWindowOrder(void)