90891 2012-07-10 10:00:05 -0700 [Gtk] fast/events/keydown-function-keys.html is failing 2012-07-11 11:42:28 -0700 1 1 1 Unclassified WebKit WebKitGTK 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 zan zan eric gustavo mrobinson pnormand spena webkit.review.bot oldest_to_newest 665830 0 zan 2012-07-10 10:00:05 -0700 The test fails because when F10 key press is simulated the context menu pops up. This is a known bug with recent xkeyboard-config version, reported at GNOME Bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=661973 There's a workaround for this until that bug gets resolved, posted at Launchpad: https://bugs.launchpad.net/ubuntu/+source/gtk+3.0/+bug/937822/comments/5 This can be used, with effect, in DumpRenderTree. 665848 1 151480 zan 2012-07-10 10:12:00 -0700 Created attachment 151480 Patch 665850 2 151480 mrobinson 2012-07-10 10:13:26 -0700 Comment on attachment 151480 Patch I guess we'll need this fix for WebKitTestRunner as well... 665869 3 zan 2012-07-10 10:38:24 -0700 (In reply to comment #2) > (From update of attachment 151480 [details]) > I guess we'll need this fix for WebKitTestRunner as well... True, but WebKitTestRunner doesn't currently set up any GtkCSSProviders before testing. 665908 4 mrobinson 2012-07-10 11:40:43 -0700 (In reply to comment #3) > (In reply to comment #2) > > (From update of attachment 151480 [details] [details]) > > I guess we'll need this fix for WebKitTestRunner as well... > > True, but WebKitTestRunner doesn't currently set up any GtkCSSProviders before testing. It hasn't needed to up to this point because the GtkCSSProviders was added to make DRT match the non-native scrollbar of WebKitTestRunner. 666424 5 zan 2012-07-11 00:03:13 -0700 (In reply to comment #4) > (In reply to comment #3) > > (In reply to comment #2) > > > (From update of attachment 151480 [details] [details] [details]) > > > I guess we'll need this fix for WebKitTestRunner as well... > > > > True, but WebKitTestRunner doesn't currently set up any GtkCSSProviders before testing. > > It hasn't needed to up to this point because the GtkCSSProviders was added to make DRT match the non-native scrollbar of WebKitTestRunner. Oddly enough the test passes in WebKitTestRunner without unbinding the F10 key, so it seems there's no need for a GtkCSSProvider in WKTR. 666916 6 151480 webkit.review.bot 2012-07-11 11:42:22 -0700 Comment on attachment 151480 Patch Clearing flags on attachment: 151480 Committed r122349: <http://trac.webkit.org/changeset/122349> 666917 7 webkit.review.bot 2012-07-11 11:42:28 -0700 All reviewed patches have been landed. Closing bug. 151480 2012-07-10 10:12:00 -0700 2012-07-11 11:42:22 -0700 Patch bug-90891-20120710191159.patch text/plain 2126 zan U3VidmVyc2lvbiBSZXZpc2lvbjogMTIyMjE3CmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggNDk1YjZhOWRlMjRiNWI5ZjUxYjE3ZWI0YzU5M2NlZmNh YjY2NzcyNC4uY2UyYzRmOTU4ZDcyNDQzMjU1MDNhYzU0NmI5NmY3ZDc2NDRkMDllZiAxMDA2NDQK LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3 IEBACisyMDEyLTA3LTEwICBaYW4gRG9iZXJzZWsgIDx6YW5kb2JlcnNla0BnbWFpbC5jb20+CisK KyAgICAgICAgW0d0a10gZmFzdC9ldmVudHMva2V5ZG93bi1mdW5jdGlvbi1rZXlzLmh0bWwgaXMg ZmFpbGluZworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9 OTA4OTEKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBX b3JrIGFyb3VuZCB0aGUgY29udGV4dCBtZW51IGJlaW5nIHNob3duIG9uIEYxMCBrZXkgYmVpbmcg cHJlc3NlZCBieQorICAgICAgICB1bmJpZGluZyB0aGUga2V5IHdoZW4gcnVubmluZyB0ZXN0cyBp biBEdW1wUmVuZGVyVHJlZS4gVGhlIHByb2JsZW0KKyAgICAgICAgYXBwZWFycyB3aGVuIHVzaW5n IGEgcmVjZW50IHZlcnNpb24gb2YgdGhlIHhrZXlib2FyZC1jb25maWcgcGFja2FnZS4KKworICAg ICAgICAqIER1bXBSZW5kZXJUcmVlL2d0ay9EdW1wUmVuZGVyVHJlZS5jcHA6CisgICAgICAgIChz ZXREZWZhdWx0c1RvQ29uc2lzdGVudFN0YXRlVmFsdWVzRm9yVGVzdGluZyk6CisKIDIwMTItMDct MTAgIErDoW5vcyBCYWRpY3MgIDxqYmFkaWNzQGluZi51LXN6ZWdlZC5odT4KIAogICAgICAgICBb UXRdW05SV1RdIEVuYWJsZSBjYXNjYWRlZCBUZXN0RXhwZWN0YXRpb25zCmRpZmYgLS1naXQgYS9U b29scy9EdW1wUmVuZGVyVHJlZS9ndGsvRHVtcFJlbmRlclRyZWUuY3BwIGIvVG9vbHMvRHVtcFJl bmRlclRyZWUvZ3RrL0R1bXBSZW5kZXJUcmVlLmNwcAppbmRleCAxMjAxNzEzMGY4OWIxZGJmMWQ3 MzVhNzJhYmQyM2MxYWE4MzZhZDk2Li4wYTRkZDRhYWUzNDE3NTg5ODNiZmU3MjkxNTIxMTJjNzcz MWYxN2ViIDEwMDY0NAotLS0gYS9Ub29scy9EdW1wUmVuZGVyVHJlZS9ndGsvRHVtcFJlbmRlclRy ZWUuY3BwCisrKyBiL1Rvb2xzL0R1bXBSZW5kZXJUcmVlL2d0ay9EdW1wUmVuZGVyVHJlZS5jcHAK QEAgLTY2NCw4ICs2NjQsMTIgQEAgc3RhdGljIHZvaWQgc2V0RGVmYXVsdHNUb0NvbnNpc3RlbnRT dGF0ZVZhbHVlc0ZvclRlc3RpbmcoKQogI2Vsc2UKICAgICBHdGtDc3NQcm92aWRlciogY3NzUHJv dmlkZXIgPSBndGtfY3NzX3Byb3ZpZGVyX25ldygpOwogICAgIGd0a19jc3NfcHJvdmlkZXJfbG9h ZF9mcm9tX2RhdGEoY3NzUHJvdmlkZXIsCisgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAiQGJpbmRpbmctc2V0IE5vS2V5Ym9hcmROYXZpZ2F0aW9uIHsgICAgICAgICIKKyAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICIgICB1bmJpbmQgXCI8c2hpZnQ+RjEwXCI7 ICAgICAgICAgICAgICAgICAgIgorICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg In0gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAiCiAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAiICogeyAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICIKICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICIgICAt R3RrU2Nyb2xsZWRXaW5kb3ctc2Nyb2xsYmFyLXNwYWNpbmc6IDA7IgorICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgIiAgIGd0ay1rZXktYmluZGluZ3M6IE5vS2V5Ym9hcmROYXZp Z2F0aW9uOyAiCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAifSAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICIsCiAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAtMSwgMCk7CiAgICAgZ3RrX3N0eWxlX2NvbnRleHRfYWRkX3Byb3Zp ZGVyX2Zvcl9zY3JlZW4oZ2RrX2Rpc3BsYXlfZ2V0X2RlZmF1bHRfc2NyZWVuKGdka19kaXNwbGF5 X2dldF9kZWZhdWx0KCkpLAo=