qtubuntu

Merge lp:~gerboland/qtubuntu/delete-qraster-crash into lp:qtubuntu

  1. delete-qraster-crash
  2. Merge into trunk
Proposed by Gerry Boland
Status: Superseded
Proposed branch: lp:~gerboland/qtubuntu/delete-qraster-crash
Merge into: lp:qtubuntu
Prerequisite: lp:~albaguirre/qtubuntu/mir-0.26.1-compat
Diff against target: 17 lines (+6/-1)
1 file modified
src/ubuntumirclient/qmirclientbackingstore.cpp (+6/-1)
To merge this branch: bzr merge lp:~gerboland/qtubuntu/delete-qraster-crash
Reviewer Review Type Date Requested Status
Unity8 CI Bot continuous-integration Approve
Ubuntu Phablet Team Pending
Review via email: mp+317848@code.launchpad.net

This proposal supersedes a proposal from 2017-02-21.

This proposal has been superseded by a proposal from 2017-02-21.

Commit message

BackingStore: make current only if no existing context current

Paraphrasing QOpenGLCompositorBackingStore: "With render-to-texture-widgets QWidget makes sure the context is made current before destroying backingstores. That is however not the case for windows with regular widgets only."

To post a comment you must log in.
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote : Posted in a previous version of this proposal

PASSED: Continuous integration, rev:372
https://unity8-jenkins.ubuntu.com/job/lp-qtubuntu-ci/185/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/4144
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/4172
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/4010
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/4010/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/4010
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/4010/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/4010
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/4010/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/4010
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/4010/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/4010
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/4010/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/4010
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/4010/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-qtubuntu-ci/185/rebuild

review: Approve (continuous-integration)
lp:~gerboland/qtubuntu/delete-qraster-crash updated
372. By Gerry Boland

BackingStore: make current only if no existing context current

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

PASSED: Continuous integration, rev:372
https://unity8-jenkins.ubuntu.com/job/lp-qtubuntu-ci/186/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/4148
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/4176
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/4014
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/4014/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/4014
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/4014/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/4014
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/4014/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/4014
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/4014/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/4014
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/4014/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/4014
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/4014/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-qtubuntu-ci/186/rebuild

review: Approve (continuous-integration)

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/ubuntumirclient/qmirclientbackingstore.cpp'
2--- src/ubuntumirclient/qmirclientbackingstore.cpp 2017-02-07 15:37:20 +0000
3+++ src/ubuntumirclient/qmirclientbackingstore.cpp 2017-02-21 13:24:07 +0000
4@@ -61,7 +61,12 @@
5
6 QMirClientBackingStore::~QMirClientBackingStore()
7 {
8- mContext->makeCurrent(window()); // needed as QOpenGLTexture destructor assumes current context
9+ // Paraphrasing QOpenGLCompositorBackingStore: "With render-to-texture-widgets QWidget makes
10+ // sure the context is made current before destroying backingstores. That is however not the
11+ // case for windows with regular widgets only.
12+ if (!QOpenGLContext::currentContext()) {
13+ mContext->makeCurrent(window());
14+ }
15 }
16
17 void QMirClientBackingStore::flush(QWindow* window, const QRegion& region, const QPoint& offset)

Subscribers

People subscribed via source and target branches