43904 2010-08-12 03:27:38 -0700 [Qt] QGWV: Canvas never updates after first frame when using GL viewport 2010-08-21 04:08:13 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) All All RESOLVED FIXED http://www.mrspeaker.net/dev/parcycle/ Qt, QtTriaged P1 Critical --- 1 benjamin webkit-qt-unassigned ariya.hidayat benjamin kling oldest_to_newest 263700 0 benjamin 2010-08-12 03:27:38 -0700 When a QGraphicsWebView is used in a GraphicsView with a QGLWidget as the viewport, the view is never updated for canvas updates. To reproduce: -start QtTestBrowser on http://www.mrspeaker.net/dev/parcycle/ -change the mode to GraphicsView -enable the GL widget as a viewport -enjoy 267513 1 65016 kling 2010-08-20 19:47:03 -0700 Created attachment 65016 Proposed patch 267516 2 benjamin 2010-08-20 19:54:29 -0700 What about other asynchronous APIs like openVG and X11? 267523 3 65016 kling 2010-08-20 20:29:41 -0700 Comment on attachment 65016 Proposed patch r-ing myself, this is a bit more hairy than I first realized.. 267556 4 kling 2010-08-21 04:08:13 -0700 This has been fixed in Qt with commit f46cb7033fc3bfc17c4ccf9125d670810322a0f5 in oslo-staging-1. The problem was that the GL texture for the QPixmap backing store of the ImageBuffer was never synced after the first time since the QPainter::begin() call is what marks it dirty (detach_no is incremented which alters the cacheKey().) 65016 2010-08-20 19:47:03 -0700 2010-08-20 20:29:41 -0700 Proposed patch bug-43904.diff text/plain 2185 kling ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg N2EwYzhlNC4uMTNmYmFlNSAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi Q29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNCBAQAorMjAxMC0wOC0yMCAgQW5kcmVhcyBLbGlu ZyAgPGFuZHJlYXMua2xpbmdAbm9raWEuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E WSAoT09QUyEpLgorCisgICAgICAgIFtRdF0gUUdXVjogQ2FudmFzIG5ldmVyIHVwZGF0ZXMgYWZ0 ZXIgZmlyc3QgZnJhbWUgd2hlbiB1c2luZyBHTCB2aWV3cG9ydAorICAgICAgICBodHRwczovL2J1 Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NDM5MDQKKworICAgICAgICAqIHBsYXRmb3Jt L2dyYXBoaWNzL3F0L0ltYWdlQnVmZmVyUXQuY3BwOgorICAgICAgICAoV2ViQ29yZTo6SW1hZ2VC dWZmZXI6OmRyYXcpOiBNYWtlIGEgY29weSBvZiB0aGUgSW1hZ2VCdWZmZXIgd2hlbiBkcmF3aW5n CisgICAgICAgIGl0IG9udG8gYW4gT3BlbkdMIHBhaW50IGRldmljZS4KKwogMjAxMC0wOC0yMCAg RGFuIEJlcm5zdGVpbiAgPG1pdHpAYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IERh dmUgS2lsemVyLgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9xdC9JbWFn ZUJ1ZmZlclF0LmNwcCBiL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvcXQvSW1hZ2VCdWZmZXJR dC5jcHAKaW5kZXggMTFjYTM3Ny4uODllNGYwMSAxMDA2NDQKLS0tIGEvV2ViQ29yZS9wbGF0Zm9y bS9ncmFwaGljcy9xdC9JbWFnZUJ1ZmZlclF0LmNwcAorKysgYi9XZWJDb3JlL3BsYXRmb3JtL2dy YXBoaWNzL3F0L0ltYWdlQnVmZmVyUXQuY3BwCkBAIC0xMTQsOCArMTE0LDIyIEBAIFBhc3NSZWZQ dHI8SW1hZ2U+IEltYWdlQnVmZmVyOjpjb3B5SW1hZ2UoKSBjb25zdAogdm9pZCBJbWFnZUJ1ZmZl cjo6ZHJhdyhHcmFwaGljc0NvbnRleHQqIGRlc3RDb250ZXh0LCBDb2xvclNwYWNlIHN0eWxlQ29s b3JTcGFjZSwgY29uc3QgRmxvYXRSZWN0JiBkZXN0UmVjdCwgY29uc3QgRmxvYXRSZWN0JiBzcmNS ZWN0LAogICAgICAgICAgICAgICAgICAgICAgICBDb21wb3NpdGVPcGVyYXRvciBvcCwgYm9vbCB1 c2VMb3dRdWFsaXR5U2NhbGUpCiB7Ci0gICAgaWYgKGRlc3RDb250ZXh0ID09IGNvbnRleHQoKSkg ewotICAgICAgICAvLyBXZSdyZSBkcmF3aW5nIGludG8gb3VyIG93biBidWZmZXIuICBJbiBvcmRl ciBmb3IgdGhpcyB0byB3b3JrLCB3ZSBuZWVkIHRvIGNvcHkgdGhlIHNvdXJjZSBidWZmZXIgZmly c3QuCisgICAgLy8gSWYgd2UncmUgZHJhd2luZyBpbnRvIG91ciBvd24gYnVmZmVyLCBhbHdheXMg ZHJhdyBhIGNvcHkgb2YgdGhlIHNvdXJjZSBidWZmZXIuCisgICAgYm9vbCBkcmF3Q29weSA9IGRl c3RDb250ZXh0ID09IGNvbnRleHQoKTsKKworICAgIGlmICghZHJhd0NvcHkpIHsKKyAgICAgICAg UVBhaW50RGV2aWNlKiBwYWludERldmljZSA9IGRlc3RDb250ZXh0LT5wbGF0Zm9ybUNvbnRleHQo KS0+ZGV2aWNlKCk7CisgICAgICAgIGlmIChwYWludERldmljZSkgeworICAgICAgICAgICAgLy8g SWYgd2UncmUgdXNpbmcgUXQncyBPcGVuR0wgcGFpbnQgZW5naW5lLCB3ZSBuZWVkIHRvIG1ha2Ug YSBjb3B5CisgICAgICAgICAgICAvLyBvZiB0aGUgaW1hZ2Ugc2luY2UgaXQgd29uJ3QgZmx1c2gg YW55dGhpbmcgdW50aWwgUVBhaW50ZXI6OmVuZCgpCisgICAgICAgICAgICAvLyB3aGljaCB3b24n dCBoYXBwZW4gZm9yIEltYWdlQnVmZmVycy4KKyAgICAgICAgICAgIFFQYWludEVuZ2luZTo6VHlw ZSB0eXBlID0gcGFpbnREZXZpY2UtPnBhaW50RW5naW5lKCktPnR5cGUoKTsKKyAgICAgICAgICAg IGlmICh0eXBlID09IFFQYWludEVuZ2luZTo6T3BlbkdMIHx8IHR5cGUgPT0gUVBhaW50RW5naW5l OjpPcGVuR0wyKQorICAgICAgICAgICAgICAgIGRyYXdDb3B5ID0gdHJ1ZTsKKyAgICAgICAgfQor ICAgIH0KKworICAgIGlmIChkcmF3Q29weSkgewogICAgICAgICBSZWZQdHI8SW1hZ2U+IGNvcHkg PSBjb3B5SW1hZ2UoKTsKICAgICAgICAgZGVzdENvbnRleHQtPmRyYXdJbWFnZShjb3B5LmdldCgp LCBEZXZpY2VDb2xvclNwYWNlLCBkZXN0UmVjdCwgc3JjUmVjdCwgb3AsIHVzZUxvd1F1YWxpdHlT Y2FsZSk7CiAgICAgfSBlbHNlCg==