Remove redundant API

Sine Striker

2023-12-16 1fab480dc53e910a614d5d836993adb743d59352

Remove redundant API

 src/core/contexts/abstractwindowcontext.cpp

@@ -48,21 +48,6 @@
        return true;
    }

    bool AbstractWindowContext::setHitTestVisible(const QRect &rect, bool visible) {
        Q_ASSERT(rect.isValid());
        if (!rect.isValid()) {
            return false;
        }

        if (visible) {
            m_hitTestVisibleRects.append(rect);
        } else {
            m_hitTestVisibleRects.removeAll(rect);
        }
        hitTestVisibleShapeDirty = true;
        return true;
    }

    bool AbstractWindowContext::setSystemButton(WindowAgentBase::SystemButton button,
                                                QObject *obj) {
        Q_ASSERT(obj);
@@ -89,17 +74,6 @@
        }
        m_titleBar = item;
        return true;
    }

    QRegion AbstractWindowContext::hitTestShape() const {
        if (hitTestVisibleShapeDirty) {
            hitTestVisibleShape = {};
            for (const auto &rect : m_hitTestVisibleRects) {
                hitTestVisibleShape += rect;
            }
            hitTestVisibleShapeDirty = false;
        }
        return hitTestVisibleShape;
    }

    bool AbstractWindowContext::isInSystemButtons(const QPoint &pos,
@@ -157,9 +131,6 @@
            }
        }

        if (!m_hitTestVisibleRects.isEmpty() && hitTestShape().contains(pos)) {
            return false;
        }
        return true;
    }


 src/core/contexts/abstractwindowcontext_p.h

@@ -29,7 +29,6 @@

        inline bool isHitTestVisible(const QObject *obj) const;
        bool setHitTestVisible(const QObject *obj, bool visible);
        bool setHitTestVisible(const QRect &rect, bool visible);

        inline QObject *systemButton(WindowAgentBase::SystemButton button) const;
        bool setSystemButton(WindowAgentBase::SystemButton button, QObject *obj);
@@ -37,7 +36,6 @@
        inline QObject *titleBar() const;
        bool setTitleBar(QObject *obj);

        QRegion hitTestShape() const;
        bool isInSystemButtons(const QPoint &pos, WindowAgentBase::SystemButton *button) const;
        bool isInTitleBarDraggableArea(const QPoint &pos) const;

@@ -62,14 +60,9 @@
        QWindow *m_windowHandle{};

        QSet<const QObject *> m_hitTestVisibleItems;
        QList<QRect> m_hitTestVisibleRects;

        QObject *m_titleBar{};
        std::array<QObject *, WindowAgentBase::NumSystemButton> m_systemButtons{};

        // Cached shape
        mutable bool hitTestVisibleShapeDirty{};
        mutable QRegion hitTestVisibleShape;
    };

    inline QObject *AbstractWindowContext::host() const {

 src/core/contexts/win32windowcontext.cpp

@@ -1171,7 +1171,7 @@
                const WindowPart currentWindowPart = lastHitTestResult;
                if (message == WM_NCMOUSEMOVE) {
                    if (currentWindowPart != WindowPart::ChromeButton) {
                        m_delegate->resetQtGrabbedControl();
                        m_delegate->resetQtGrabbedControl(m_host);
                        if (mouseLeaveBlocked) {
                            emulateClientAreaMessage(hWnd, message, wParam, lParam,
                                                     WM_NCMOUSELEAVE);
@@ -1233,7 +1233,7 @@
                        // window from client area, which means we will get previous window part as
                        // HTCLIENT if the mouse leaves window from client area and enters window
                        // from non-client area, but it has no bad effect.
                        m_delegate->resetQtGrabbedControl();
                        m_delegate->resetQtGrabbedControl(m_host);
                    }
                }
                break;

 src/core/kernel/nativeeventfilter_p.h

@@ -43,7 +43,7 @@
        friend class NativeEventDispatcher;
    };

    // Automatically install to QApplication at construction
    // Automatically install to QCoreApplication at construction
    class QWK_CORE_EXPORT AppNativeEventFilter : public NativeEventFilter {
    public:
        AppNativeEventFilter();

 src/quick/quickwindowagent.cpp

@@ -74,14 +74,9 @@
        return d->context->isHitTestVisible(item);
    }

    void QuickWindowAgent::setHitTestVisible_item(const QQuickItem *item, bool visible) {
    void QuickWindowAgent::setHitTestVisible(const QQuickItem *item, bool visible) {
        Q_D(QuickWindowAgent);
        d->context->setHitTestVisible(item, visible);
    }

    void QuickWindowAgent::setHitTestVisible_rect(const QRect &rect, bool visible) {
        Q_D(QuickWindowAgent);
        d->context->setHitTestVisible(rect, visible);
    }

    QuickWindowAgent::QuickWindowAgent(QuickWindowAgentPrivate &d, QObject *parent)

 src/quick/quickwindowagent.h

@@ -28,9 +28,7 @@
        Q_INVOKABLE void setSystemButton(SystemButton button, QQuickItem *item);

        Q_INVOKABLE bool isHitTestVisible(const QQuickItem *item) const;
        Q_INVOKABLE inline void setHitTestVisible(const QQuickItem *item, bool visible = true);
        Q_INVOKABLE void setHitTestVisible_item(const QQuickItem *item, bool visible = true);
        Q_INVOKABLE void setHitTestVisible_rect(const QRect &rect, bool visible = true);
        Q_INVOKABLE void setHitTestVisible(const QQuickItem *item, bool visible = true);

    Q_SIGNALS:
        void titleBarWidgetChanged(const QQuickItem *item);
@@ -39,10 +37,6 @@
    protected:
        QuickWindowAgent(QuickWindowAgentPrivate &d, QObject *parent = nullptr);
    };

    inline void QuickWindowAgent::setHitTestVisible(const QQuickItem *item, bool visible) {
        setHitTestVisible_item(item, visible);
    }

}


 src/widgets/widgetwindowagent.cpp

@@ -86,11 +86,6 @@
        d->context->setHitTestVisible(w, visible);
    }

    void WidgetWindowAgent::setHitTestVisible(const QRect &rect, bool visible) {
        Q_D(WidgetWindowAgent);
        d->context->setHitTestVisible(rect, visible);
    }

    WidgetWindowAgent::WidgetWindowAgent(WidgetWindowAgentPrivate &d, QObject *parent)
        : WindowAgentBase(d, parent) {
        d.init();

 src/widgets/widgetwindowagent.h

@@ -28,7 +28,6 @@

        bool isHitTestVisible(const QWidget *w) const;
        void setHitTestVisible(const QWidget *w, bool visible = true);
        void setHitTestVisible(const QRect &rect, bool visible = true);

    Q_SIGNALS:
        void titleBarWidgetChanged(const QWidget *w);