64327 2011-07-11 16:48:51 -0700 HTML5 video controller in fullscreen is partly off-screen (at least on youtube) using ClickToFlash 2011-07-12 11:24:47 -0700 1 1 1 Unclassified WebKit Media 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 jer.noble jer.noble darin oldest_to_newest 435304 0 jer.noble 2011-07-11 16:48:51 -0700 HTML5 video controller in fullscreen is partly off-screen (at least on youtube) using ClickToFlash 435306 1 100384 jer.noble 2011-07-11 16:50:48 -0700 Created attachment 100384 Patch 435333 2 jer.noble 2011-07-11 17:31:03 -0700 Committed r90797: <http://trac.webkit.org/changeset/90797> 435777 3 darin 2011-07-12 11:03:09 -0700 It seems fine to make these CSS rules !important. But I don’t think this is a real fix to the underlying problem. Other CSS properties could be set, ones not listed in this stylesheet at all, and those would affect fullscreen mode and the video controller. The problem of styles on the page affecting the controller is an architectural one that needs a real solution at some point. This is one of multiple problems that stem from building the fullscreen mode for video out of the general fullscreen feature; that implementation detail should not show through. While I don’t know exactly what the solution is, I am sure there are some simple ways to isolate the fullscreen controller from other page content. 435803 4 jer.noble 2011-07-12 11:24:11 -0700 Bug #61843 and #61848 would provide a real solution for this problem by implementing the video-full-screen mode in WebKit2. (As opposed to making the element-full-screen mode simulate the behavior and side-effects of video-full-screen mode.) 435804 5 jer.noble 2011-07-12 11:24:47 -0700 For clickability: Bug #61848 100384 2011-07-11 16:50:48 -0700 2011-07-11 17:16:11 -0700 Patch bug-64327-20110711165046.patch text/plain 2712 jer.noble U3VidmVyc2lvbiBSZXZpc2lvbjogOTA3ODIKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCAyMGRiMWJhZjM4NDFmNzY0 NDFhYzUwMzg2NjM1MTIxZjJlOWViNGFhLi5kZjE1NWFiMzBhZjIwZjIyN2IxOWE2NWYyNTNiMjI4 ODRiNmNjN2UwIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvU291 cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTggQEAKKzIwMTEtMDctMTEgIEplciBO b2JsZSAgPGplci5ub2JsZUBhcHBsZS5jb20+CisKKyAgICAgICAgSFRNTDUgdmlkZW8gY29udHJv bGxlciBpbiBmdWxsc2NyZWVuIGlzIHBhcnRseSBvZmYtc2NyZWVuIChhdCBsZWFzdCBvbiB5b3V0 dWJlKSB1c2luZyBDbGlja1RvRmxhc2gKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv c2hvd19idWcuY2dpP2lkPTY0MzI3CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT ISkuCisKKyAgICAgICAgTm8gbmV3IHRlc3RzOyBubyBjaGFuZ2UgaW4gZnVuY3Rpb25hbGl0eSwg c28gY292ZXJlZCBieSBleGlzdGluZyB0ZXN0cy4KKworICAgICAgICBNYWtlIHRoZSBydWxlcyBm cm9tIGZ1bGxzY3JlZW5RdWlja1RpbWUuY3NzICFpbXBvcnRhbnQgc28gdGhleSBhcmUgbm90IG92 ZXJyaWRkZW4KKyAgICAgICAgYnkgcGFnZSBhdXRob3JzLgorCisgICAgICAgICogY3NzL2Z1bGxz Y3JlZW5RdWlja1RpbWUuY3NzOgorICAgICAgICAodmlkZW86LXdlYmtpdC1mdWxsLXNjcmVlbjo6 LXdlYmtpdC1tZWRpYS1jb250cm9scy1wYW5lbCk6CisKIDIwMTEtMDctMTEgIEJyYWRsZXkgTmVs c29uICA8YnJhZG5lbHNvbkBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgRGFy aW4gRmlzaGVyLgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvY3NzL2Z1bGxzY3JlZW5RdWlj a1RpbWUuY3NzIGIvU291cmNlL1dlYkNvcmUvY3NzL2Z1bGxzY3JlZW5RdWlja1RpbWUuY3NzCmlu ZGV4IDhmZThhZGYxOTUwNTc2ZGE1M2VlZjk5YmQ3NzEwNWI4ZWVmMGNkMDYuLmE1NTU5NzY4ZGVj OGE2OGViNDk2YzU1MDQxZDZhNTYxNDNhZDI1Y2MgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3Jl L2Nzcy9mdWxsc2NyZWVuUXVpY2tUaW1lLmNzcworKysgYi9Tb3VyY2UvV2ViQ29yZS9jc3MvZnVs bHNjcmVlblF1aWNrVGltZS5jc3MKQEAgLTI1LDE2ICsyNSwxNiBAQAogLyogYWx0ZXJuYXRlIG1l ZGlhIGNvbnRyb2xzIC0gRXh0ZW5kIGZ1bGxzY3JlZW4uY3NzICovCiAKIHZpZGVvOi13ZWJraXQt ZnVsbC1zY3JlZW46Oi13ZWJraXQtbWVkaWEtY29udHJvbHMtcGFuZWwgewotICAgIC13ZWJraXQt Ym94LWFsaWduOiBzdGFydDsKLSAgICAtd2Via2l0LWJveC1wYWNrOiBlbmQ7ICAgIAotICAgIC13 ZWJraXQtYXBwZWFyYW5jZTogbm9uZTsKLQotICAgIGJvdHRvbTogNTBweDsKLSAgICBsZWZ0OiA1 MCU7Ci0gICAgbWFyZ2luLWxlZnQ6IC0yMjBweDsKLSAgICBwYWRkaW5nOiAxMnB4IDAgMCAxMHB4 OwotICAgIHdpZHRoOiA0MzBweDsKLSAgICBoZWlnaHQ6IDQ4cHg7CisgICAgLXdlYmtpdC1ib3gt YWxpZ246IHN0YXJ0ICFpbXBvcnRhbnQ7CisgICAgLXdlYmtpdC1ib3gtcGFjazogZW5kICFpbXBv cnRhbnQ7CisgICAgLXdlYmtpdC1hcHBlYXJhbmNlOiBub25lICFpbXBvcnRhbnQ7CisKKyAgICBi b3R0b206IDUwcHggIWltcG9ydGFudDsKKyAgICBsZWZ0OiA1MCUgIWltcG9ydGFudDsKKyAgICBt YXJnaW4tbGVmdDogLTIyMHB4ICFpbXBvcnRhbnQ7CisgICAgcGFkZGluZzogMTJweCAwIDAgMTBw eCAhaW1wb3J0YW50OworICAgIHdpZHRoOiA0MzBweCAhaW1wb3J0YW50OworICAgIGhlaWdodDog NDhweCAhaW1wb3J0YW50OwogICAgIAogICAgIGJhY2tncm91bmQtaW1hZ2U6IC13ZWJraXQtZ3Jh ZGllbnQoCiAgICAgICAgIGxpbmVhciwKQEAgLTQ2LDE2ICs0NiwxNiBAQCB2aWRlbzotd2Via2l0 LWZ1bGwtc2NyZWVuOjotd2Via2l0LW1lZGlhLWNvbnRyb2xzLXBhbmVsIHsKICAgICAgICAgY29s b3Itc3RvcCgwLjM2NiwgcmdiYSgyNSwgMjUsIDI1LCAuOTcpKSwKICAgICAgICAgY29sb3Itc3Rv cCgwLjM2NiwgcmdiYSgxMiwgMTIsIDEyLCAuOTcpKSwKICAgICAgICAgY29sb3Itc3RvcCgxLCBy Z2JhKDE5LCAxOSwgMTksIC45NykpCi0gICAgKTsKKyAgICApICFpbXBvcnRhbnQ7CiAKICAgICAt d2Via2l0LWJveC1zaGFkb3c6IAogICAgICAgICBpbnNldCAwIC0xcHggMXB4IHJnYmEoMCwgMCwg MCwgMC41KSwKICAgICAgICAgaW5zZXQgMCAgMXB4IDAgMHB4ICAgcmdiYSgyNTUsIDI1NSwgMjU1 LCAwLjE1KSwKICAgICAgICAgaW5zZXQgMCAtMXB4IDAgMHB4ICAgcmdiYSgyMDIsIDIwMiwgMjAy LCAwLjA5KSwKICAgICAgICAgMCAgMCAgIDAgMXB4IHJnYmEoMCwgMCwgMCwgMC41KTsKLSAgICAt d2Via2l0LWJvcmRlci1yYWRpdXM6IDhweDsKKyAgICAtd2Via2l0LWJvcmRlci1yYWRpdXM6IDhw eCAhaW1wb3J0YW50OwogCi0gICAgLXdlYmtpdC10cmFuc2l0aW9uOiBvcGFjaXR5IDAuM3MgbGlu ZWFyOworICAgIC13ZWJraXQtdHJhbnNpdGlvbjogb3BhY2l0eSAwLjNzIGxpbmVhciAhaW1wb3J0 YW50OwogfQogCiB2aWRlbzotd2Via2l0LWFuaW1hdGluZy1mdWxsLXNjcmVlbi10cmFuc2l0aW9u Ojotd2Via2l0LW1lZGlhLWNvbnRyb2xzLXBhbmVsIHsK