822 2013-08-29 18:04:01 +0200 NewtCanvasSWT: no more rendering after detaching editor from the main EclipseRCP app window 2019-03-30 02:59:27 +0100 1 1 3 JogAmp Newt swt 2 All all RESOLVED FIXED --- critical --- 969 0 crougier pjgl bryan.osborn gouessej michael.dehoog sgothel DEFECT --- oldest_to_newest 2919 0 crougier 2013-08-29 18:04:01 +0200 Eclipse Kepler (4.3) Windows 7 and RedHat 5.2 JOGL 2.0.2 The NewtCanvasSWT is displayed in an editor of an Eclipse RCP app. When the editor is detached from the main app window to its own window, the 3D scene is no more rendered. The OpenGL context is not destroyed. The NewtCanvasSWT handle a resize event with a height value = 0, hence the window is marked as invisible. Then, other resize events with valid values are handled but the window stay in an invisible state. Originally reported at: http://forum.jogamp.org/NativeWindowException-Unable-to-lock-surface-td4029205.html gouessej said: "You're right, validateNative fails in this case. Edit.: There is still the same kind of test in NewtCanvasAWT :( " 3808 1 sgothel 2014-02-26 20:13:26 +0100 How can this bug be reproduced ? As a unit test (best), manually (bad), or not at all (worst) ? If we are not able to reproduce a bug, we may have to close it. 4523 2 michael.dehoog 2015-01-13 04:22:40 +0100 Can confirm that this is indeed an issue. To reproduce, create an Eclipse RCP application, embed a NewtCanvasSWT in a tab, and then detach that tab from the main RCP window so that a new window is created. The NewtCanvasSWT receives two resize events, one with 0,0,0,0, and then another with the correct size. To fix this issue in our application (https://github.com/GeoscienceAustralia/earthsci/issues/123), I overrode the setBounds(int,int,int,int) method of NewtCanvasSWT as follows: @Override public void setBounds(int x, int y, int width, int height) { //do not allow a size of 0,0, because NEWT window becomes invisible super.setBounds(x, y, Math.max(1, width), Math.max(1, height)); } 4524 3 gouessej 2015-01-13 09:48:32 +0100 (In reply to comment #2) > Can confirm that this is indeed an issue. To reproduce, create an Eclipse > RCP application, embed a NewtCanvasSWT in a tab, and then detach that tab > from the main RCP window so that a new window is created. The NewtCanvasSWT > receives two resize events, one with 0,0,0,0, and then another with the > correct size. > > To fix this issue in our application > (https://github.com/GeoscienceAustralia/earthsci/issues/123), I overrode the > setBounds(int,int,int,int) method of NewtCanvasSWT as follows: > > @Override > public void setBounds(int x, int y, int width, int height) > { > //do not allow a size of 0,0, because NEWT window becomes invisible > super.setBounds(x, y, Math.max(1, width), Math.max(1, height)); > } Thank you very much for sharing your findings. It confirms what Céline wrote about the resize event: http://forum.jogamp.org/NativeWindowException-Unable-to-lock-surface-tp4029205p4029700.html 4559 4 sgothel 2015-01-29 01:04:50 +0100 So how do we proceed with the solution in comment 2 ? Is there a git pull req, or a patch available ? Or shall somebody (Me, Petros) add this change ? 4561 5 gouessej 2015-01-29 10:44:40 +0100 (In reply to comment #4) > So how do we proceed with the solution in comment 2 ? > > Is there a git pull req, or a patch available ? > Or shall somebody (Me, Petros) add this change ? Is there any side effect? As I said, there is probably the same problem in NewtCanvasAWT. 4562 6 687 michael.dehoog 2015-01-29 22:29:46 +0100 Created attachment 687 Test case that reproduces the bug Overriding setBounds() is just a workaround that worked for us. It would be better to fix the bug itself. I have attached a simple example that reproduces the bug. Two canvases are created, one with an overridden setSize() and one without. When you click the "Set size to 0,0" button, both canvases disappear with a size of 0,0. When you click "Set size to 50,50", only the workaround canvas reappears. 6151 7 sgothel 2019-03-30 02:59:16 +0100 Fixed via bug 969 I assume. Otherwise we need to reopen it. 687 2015-01-29 22:29:46 +0100 2015-01-29 22:29:46 +0100 Test case that reproduces the bug NewtTest.java text/plain 5394 michael.dehoog aW1wb3J0IGphdmF4Lm1lZGlhLm9wZW5nbC5HTDI7DQppbXBvcnQgamF2YXgubWVkaWEub3Blbmds LkdMQXV0b0RyYXdhYmxlOw0KaW1wb3J0IGphdmF4Lm1lZGlhLm9wZW5nbC5HTENhcGFiaWxpdGll czsNCmltcG9ydCBqYXZheC5tZWRpYS5vcGVuZ2wuR0xFdmVudExpc3RlbmVyOw0KaW1wb3J0IGph dmF4Lm1lZGlhLm9wZW5nbC5HTFByb2ZpbGU7DQppbXBvcnQgamF2YXgubWVkaWEub3BlbmdsLmds dS5HTFU7DQoNCmltcG9ydCBvcmcuZWNsaXBzZS5zd3QuU1dUOw0KaW1wb3J0IG9yZy5lY2xpcHNl LnN3dC5ldmVudHMuU2VsZWN0aW9uQWRhcHRlcjsNCmltcG9ydCBvcmcuZWNsaXBzZS5zd3QuZXZl bnRzLlNlbGVjdGlvbkV2ZW50Ow0KaW1wb3J0IG9yZy5lY2xpcHNlLnN3dC5sYXlvdXQuUm93TGF5 b3V0Ow0KaW1wb3J0IG9yZy5lY2xpcHNlLnN3dC53aWRnZXRzLkJ1dHRvbjsNCmltcG9ydCBvcmcu ZWNsaXBzZS5zd3Qud2lkZ2V0cy5Db21wb3NpdGU7DQppbXBvcnQgb3JnLmVjbGlwc2Uuc3d0Lndp ZGdldHMuRGlzcGxheTsNCmltcG9ydCBvcmcuZWNsaXBzZS5zd3Qud2lkZ2V0cy5TaGVsbDsNCg0K aW1wb3J0IGNvbS5qb2dhbXAubmV3dC5vcGVuZ2wuR0xXaW5kb3c7DQppbXBvcnQgY29tLmpvZ2Ft cC5uZXd0LnN3dC5OZXd0Q2FudmFzU1dUOw0KaW1wb3J0IGNvbS5qb2dhbXAub3BlbmdsLnV0aWwu QW5pbWF0b3I7DQoNCnB1YmxpYyBjbGFzcyBOZXd0VGVzdA0Kew0KCXB1YmxpYyBzdGF0aWMgdm9p ZCBtYWluKFN0cmluZ1tdIGFyZ3MpDQoJew0KCQlEaXNwbGF5IGRpc3BsYXkgPSBuZXcgRGlzcGxh eSgpOw0KCQlTaGVsbCBzaGVsbCA9IG5ldyBTaGVsbChkaXNwbGF5KTsNCgkJc2hlbGwuc2V0TGF5 b3V0KG5ldyBSb3dMYXlvdXQoKSk7DQoNCgkJZmluYWwgTmV3dENhbnZhc1NXVCBjYW52YXMxID0g aW5pdChzaGVsbCwgZmFsc2UpOw0KCQlmaW5hbCBOZXd0Q2FudmFzU1dUIGNhbnZhczIgPSBpbml0 KHNoZWxsLCB0cnVlKTsNCg0KCQlCdXR0b24gYnV0dG9uMSA9IG5ldyBCdXR0b24oc2hlbGwsIFNX VC5QVVNIKTsNCgkJYnV0dG9uMS5zZXRUZXh0KCJTZXQgc2l6ZSB0byAwLDAiKTsNCgkJYnV0dG9u MS5hZGRTZWxlY3Rpb25MaXN0ZW5lcihuZXcgU2VsZWN0aW9uQWRhcHRlcigpDQoJCXsNCgkJCUBP dmVycmlkZQ0KCQkJcHVibGljIHZvaWQgd2lkZ2V0U2VsZWN0ZWQoU2VsZWN0aW9uRXZlbnQgZSkN CgkJCXsNCgkJCQljYW52YXMxLnNldFNpemUoMCwgMCk7DQoJCQkJY2FudmFzMi5zZXRTaXplKDAs IDApOw0KCQkJfQ0KCQl9KTsNCg0KCQlCdXR0b24gYnV0dG9uMiA9IG5ldyBCdXR0b24oc2hlbGws IFNXVC5QVVNIKTsNCgkJYnV0dG9uMi5zZXRUZXh0KCJTZXQgc2l6ZSB0byA1MCw1MCIpOw0KCQli dXR0b24yLmFkZFNlbGVjdGlvbkxpc3RlbmVyKG5ldyBTZWxlY3Rpb25BZGFwdGVyKCkNCgkJew0K CQkJQE92ZXJyaWRlDQoJCQlwdWJsaWMgdm9pZCB3aWRnZXRTZWxlY3RlZChTZWxlY3Rpb25FdmVu dCBlKQ0KCQkJew0KCQkJCWNhbnZhczEuc2V0U2l6ZSg1MCwgNTApOw0KCQkJCWNhbnZhczIuc2V0 U2l6ZSg1MCwgNTApOw0KCQkJfQ0KCQl9KTsNCg0KCQlzaGVsbC5wYWNrKCk7DQoJCXNoZWxsLm9w ZW4oKTsNCgkJd2hpbGUgKCFzaGVsbC5pc0Rpc3Bvc2VkKCkpDQoJCXsNCgkJCWlmICghZGlzcGxh eS5yZWFkQW5kRGlzcGF0Y2goKSkNCgkJCXsNCgkJCQlkaXNwbGF5LnNsZWVwKCk7DQoJCQl9DQoJ CX0NCgkJZGlzcGxheS5kaXNwb3NlKCk7DQoJfQ0KDQoJcHVibGljIHN0YXRpYyBOZXd0Q2FudmFz U1dUIGluaXQoQ29tcG9zaXRlIHBhcmVudCwgZmluYWwgYm9vbGVhbiBhcHBseUZpeCkNCgl7DQoJ CUdMQ2FwYWJpbGl0aWVzIGNhcHMgPSBuZXcgR0xDYXBhYmlsaXRpZXMoR0xQcm9maWxlLmdldChH TFByb2ZpbGUuR0wyKSk7DQoJCWZpbmFsIEdMV2luZG93IGdsV2luZG93ID0gR0xXaW5kb3cuY3Jl YXRlKGNhcHMpOw0KCQlOZXd0Q2FudmFzU1dUIGNhbnZhcyA9IG5ldyBOZXd0Q2FudmFzU1dUKHBh cmVudCwgU1dULk5PTkUsIGdsV2luZG93KQ0KCQl7DQoJCQlAT3ZlcnJpZGUNCgkJCXB1YmxpYyB2 b2lkIHNldFNpemUoaW50IHdpZHRoLCBpbnQgaGVpZ2h0KQ0KCQkJew0KCQkJCS8vZG8gbm90IGFs bG93IGEgc2l6ZSBvZiAwLDAsIGJlY2F1c2UgTkVXVCB3aW5kb3cgYmVjb21lcyBpbnZpc2libGUg KGh0dHBzOi8vam9nYW1wLm9yZy9idWd6aWxsYS9zaG93X2J1Zy5jZ2k/aWQ9ODIyKQ0KCQkJCWlm IChhcHBseUZpeCkNCgkJCQl7DQoJCQkJCXN1cGVyLnNldFNpemUoTWF0aC5tYXgoMSwgd2lkdGgp LCBNYXRoLm1heCgxLCBoZWlnaHQpKTsNCgkJCQl9DQoJCQkJZWxzZQ0KCQkJCXsNCgkJCQkJc3Vw ZXIuc2V0U2l6ZSh3aWR0aCwgaGVpZ2h0KTsNCgkJCQl9DQoJCQl9DQoJCX07DQoNCgkJZ2xXaW5k b3cuYWRkR0xFdmVudExpc3RlbmVyKG5ldyBNeUdsRXZlbnRMaXN0ZW5lcigpKTsNCg0KCQlBbmlt YXRvciBhbmltYXRvciA9IG5ldyBBbmltYXRvcihnbFdpbmRvdyk7DQoJCWFuaW1hdG9yLnNldFVw ZGF0ZUZQU0ZyYW1lcygxLCBudWxsKTsNCgkJYW5pbWF0b3Iuc3RhcnQoKTsNCg0KCQlyZXR1cm4g Y2FudmFzOw0KCX0NCg0KCS8qKg0KCSAqIEdMRXZlbnRMaXN0ZW5lciB3aGljaCBkcmF3cyBhIHNw aW5uaW5nIHRvcnVzLiBGcm9tDQoJICogaHR0cHM6Ly9naXRodWIuY29tL3Nnb3RoZWwNCgkgKiAv am9nbC1kZW1vcy9ibG9iL21hc3Rlci9zcmMvZGVtb3Mvc3d0L1NuaXBwZXQyMDkuamF2YS4NCgkg Ki8NCglwcml2YXRlIHN0YXRpYyBjbGFzcyBNeUdsRXZlbnRMaXN0ZW5lciBpbXBsZW1lbnRzIEdM RXZlbnRMaXN0ZW5lcg0KCXsNCgkJcHJpdmF0ZSBpbnQgcm90ID0gMDsNCg0KCQlAT3ZlcnJpZGUN CgkJcHVibGljIHZvaWQgaW5pdChHTEF1dG9EcmF3YWJsZSBkcmF3YWJsZSkNCgkJew0KCQkJR0wy IGdsID0gZHJhd2FibGUuZ2V0R0woKS5nZXRHTDIoKTsNCgkJCWdsLnNldFN3YXBJbnRlcnZhbCgx KTsNCgkJCWdsLmdsQ2xlYXJDb2xvcigxLjBmLCAxLjBmLCAxLjBmLCAxLjBmKTsNCgkJCWdsLmds Q29sb3IzZigxLjBmLCAwLjBmLCAwLjBmKTsNCgkJCWdsLmdsSGludChHTDIuR0xfUEVSU1BFQ1RJ VkVfQ09SUkVDVElPTl9ISU5ULCBHTDIuR0xfTklDRVNUKTsNCgkJCWdsLmdsQ2xlYXJEZXB0aCgx LjApOw0KCQkJZ2wuZ2xMaW5lV2lkdGgoMik7DQoJCQlnbC5nbEVuYWJsZShHTDIuR0xfREVQVEhf VEVTVCk7DQoJCX0NCg0KCQlAT3ZlcnJpZGUNCgkJcHVibGljIHZvaWQgcmVzaGFwZShHTEF1dG9E cmF3YWJsZSBkcmF3YWJsZSwgaW50IHgsIGludCB5LCBpbnQgd2lkdGgsIGludCBoZWlnaHQpDQoJ CXsNCgkJCWZsb2F0IGZBc3BlY3QgPSAoZmxvYXQpIHdpZHRoIC8gKGZsb2F0KSBoZWlnaHQ7DQoJ CQlHTDIgZ2wgPSBkcmF3YWJsZS5nZXRHTCgpLmdldEdMMigpOw0KCQkJZ2wuZ2xWaWV3cG9ydCgw LCAwLCB3aWR0aCwgaGVpZ2h0KTsNCgkJCWdsLmdsTWF0cml4TW9kZShHTDIuR0xfUFJPSkVDVElP Tik7DQoJCQlnbC5nbExvYWRJZGVudGl0eSgpOw0KCQkJR0xVIGdsdSA9IG5ldyBHTFUoKTsNCgkJ CWdsdS5nbHVQZXJzcGVjdGl2ZSg0NS4wZiwgZkFzcGVjdCwgMC41ZiwgNDAwLjBmKTsNCgkJCWds LmdsTWF0cml4TW9kZShHTDIuR0xfTU9ERUxWSUVXKTsNCgkJCWdsLmdsTG9hZElkZW50aXR5KCk7 DQoJCX0NCg0KCQlAT3ZlcnJpZGUNCgkJcHVibGljIHZvaWQgZGlzcGxheShHTEF1dG9EcmF3YWJs ZSBkcmF3YWJsZSkNCgkJew0KCQkJR0wyIGdsID0gZHJhd2FibGUuZ2V0R0woKS5nZXRHTDIoKTsN CgkJCWdsLmdsQ2xlYXIoR0wyLkdMX0NPTE9SX0JVRkZFUl9CSVQgfCBHTDIuR0xfREVQVEhfQlVG RkVSX0JJVCk7DQoJCQlnbC5nbENsZWFyQ29sb3IoLjNmLCAuNWYsIC44ZiwgMS4wZik7DQoJCQln bC5nbExvYWRJZGVudGl0eSgpOw0KCQkJZ2wuZ2xUcmFuc2xhdGVmKDAuMGYsIDAuMGYsIC0xMC4w Zik7DQoJCQlnbC5nbFJvdGF0ZWYoMC4xNWYgKiByb3QsIDIuMGYgKiByb3QsIDEwLjBmICogcm90 LCAxLjBmKTsNCgkJCWdsLmdsUm90YXRlZigwLjNmICogcm90LCAzLjBmICogcm90LCAxLjBmICog cm90LCAxLjBmKTsNCgkJCXJvdCsrOw0KCQkJZ2wuZ2xQb2x5Z29uTW9kZShHTDIuR0xfRlJPTlRf QU5EX0JBQ0ssIEdMMi5HTF9MSU5FKTsNCgkJCWdsLmdsQ29sb3IzZigwLjlmLCAwLjlmLCAwLjlm KTsNCgkJCWRyYXdUb3J1cyhnbCwgMSwgMS45ZiArICgoZmxvYXQpIE1hdGguc2luKCgwLjAwNGYg KiByb3QpKSksIDE1LCAxNSk7DQoJCX0NCg0KCQlAT3ZlcnJpZGUNCgkJcHVibGljIHZvaWQgZGlz cG9zZShHTEF1dG9EcmF3YWJsZSBkcmF3YWJsZSkNCgkJew0KCQl9DQoNCgkJcHJvdGVjdGVkIHZv aWQgZHJhd1RvcnVzKEdMMiBnbCwgZmxvYXQgciwgZmxvYXQgUiwgaW50IG5zaWRlcywgaW50IHJp bmdzKQ0KCQl7DQoJCQlmbG9hdCByaW5nRGVsdGEgPSAyLjBmICogKGZsb2F0KSBNYXRoLlBJIC8g cmluZ3M7DQoJCQlmbG9hdCBzaWRlRGVsdGEgPSAyLjBmICogKGZsb2F0KSBNYXRoLlBJIC8gbnNp ZGVzOw0KCQkJZmxvYXQgdGhldGEgPSAwLjBmLCBjb3NUaGV0YSA9IDEuMGYsIHNpblRoZXRhID0g MC4wZjsNCgkJCWZvciAoaW50IGkgPSByaW5ncyAtIDE7IGkgPj0gMDsgaS0tKQ0KCQkJew0KCQkJ CWZsb2F0IHRoZXRhMSA9IHRoZXRhICsgcmluZ0RlbHRhOw0KCQkJCWZsb2F0IGNvc1RoZXRhMSA9 IChmbG9hdCkgTWF0aC5jb3ModGhldGExKTsNCgkJCQlmbG9hdCBzaW5UaGV0YTEgPSAoZmxvYXQp IE1hdGguc2luKHRoZXRhMSk7DQoJCQkJZ2wuZ2xCZWdpbihHTDIuR0xfUVVBRF9TVFJJUCk7DQoJ CQkJZmxvYXQgcGhpID0gMC4wZjsNCgkJCQlmb3IgKGludCBqID0gbnNpZGVzOyBqID49IDA7IGot LSkNCgkJCQl7DQoJCQkJCXBoaSArPSBzaWRlRGVsdGE7DQoJCQkJCWZsb2F0IGNvc1BoaSA9IChm bG9hdCkgTWF0aC5jb3MocGhpKTsNCgkJCQkJZmxvYXQgc2luUGhpID0gKGZsb2F0KSBNYXRoLnNp bihwaGkpOw0KCQkJCQlmbG9hdCBkaXN0ID0gUiArIHIgKiBjb3NQaGk7DQoJCQkJCWdsLmdsTm9y bWFsM2YoY29zVGhldGExICogY29zUGhpLCAtc2luVGhldGExICogY29zUGhpLCBzaW5QaGkpOw0K CQkJCQlnbC5nbFZlcnRleDNmKGNvc1RoZXRhMSAqIGRpc3QsIC1zaW5UaGV0YTEgKiBkaXN0LCBy ICogc2luUGhpKTsNCgkJCQkJZ2wuZ2xOb3JtYWwzZihjb3NUaGV0YSAqIGNvc1BoaSwgLXNpblRo ZXRhICogY29zUGhpLCBzaW5QaGkpOw0KCQkJCQlnbC5nbFZlcnRleDNmKGNvc1RoZXRhICogZGlz dCwgLXNpblRoZXRhICogZGlzdCwgciAqIHNpblBoaSk7DQoJCQkJfQ0KCQkJCWdsLmdsRW5kKCk7 DQoJCQkJdGhldGEgPSB0aGV0YTE7DQoJCQkJY29zVGhldGEgPSBjb3NUaGV0YTE7DQoJCQkJc2lu VGhldGEgPSBzaW5UaGV0YTE7DQoJCQl9DQoJCX0NCgl9DQp9