21284 2008-10-01 13:20:01 -0700 Max height affects max width 2008-10-01 13:42:52 -0700 1 1 1 Unclassified WebKit Web Inspector (Deprecated) 528+ (Nightly build) Mac OS X 10.5 RESOLVED FIXED InRadar P2 Normal --- 1 kmccullough kmccullough oldest_to_newest 93587 0 kmccullough 2008-10-01 13:20:01 -0700 I have a layout test and patch that fixes this. <rdar://problem/6192014> 93588 1 23989 kmccullough 2008-10-01 13:23:16 -0700 Created attachment 23989 Proposed patch and layout test. 93595 2 23989 mitz 2008-10-01 13:28:43 -0700 Comment on attachment 23989 Proposed patch and layout test. r=me Please add a reference to the test in the WebCore ChangeLog (see previous entries for the format). I also find that complete bugzilla URLs are more useful that just "bug #####". 93598 3 kmccullough 2008-10-01 13:42:52 -0700 Committed revision 37154. 23989 2008-10-01 13:23:16 -0700 2008-10-01 13:28:43 -0700 Proposed patch and layout test. maxWidthHeight.txt text/plain 3861 kmccullough SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiAzNzE1MSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTMgQEAKKzIwMDgtMTAtMDEgIEtldmluIE1jQ3VsbG91Z2ggIDxrbWNjdWxsb3Vn aEBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAg ICAgICAgQnVnIDIxMjg0OiBNYXggaGVpZ2h0IGFmZmVjdHMgbWF4IHdpZHRoCisgICAgICAgIC0g TG9va3MgbGlrZSBhIGN1dCBhbmQgcGFzdGUgYnVnCisKKyAgICAgICAgKiBjc3MvQ1NTQ29tcHV0 ZWRTdHlsZURlY2xhcmF0aW9uLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkNTU0NvbXB1dGVkU3R5 bGVEZWNsYXJhdGlvbjo6Z2V0UHJvcGVydHlDU1NWYWx1ZSk6CisKIDIwMDgtMTAtMDEgIERhdmlk IEh5YXR0ICA8aHlhdHRAYXBwbGUuY29tPgogCiAgICAgICAgIE1ha2Ugc2V0RnJhbWVSZWN0IG9u IFNjcm9sbFZpZXcgY3Jvc3MtcGxhdGZvcm0uCkluZGV4OiBXZWJDb3JlL2Nzcy9DU1NDb21wdXRl ZFN0eWxlRGVjbGFyYXRpb24uY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvY3NzL0NTU0NvbXB1 dGVkU3R5bGVEZWNsYXJhdGlvbi5jcHAJKHJldmlzaW9uIDM3MTE1KQorKysgV2ViQ29yZS9jc3Mv Q1NTQ29tcHV0ZWRTdHlsZURlY2xhcmF0aW9uLmNwcAkod29ya2luZyBjb3B5KQpAQCAtNzgyLDcg Kzc4Miw3IEBAIFBhc3NSZWZQdHI8Q1NTVmFsdWU+IENTU0NvbXB1dGVkU3R5bGVEZWMKICAgICAg ICAgICAgIHJldHVybiBDU1NQcmltaXRpdmVWYWx1ZTo6Y3JlYXRlKG1heEhlaWdodCk7CiAgICAg ICAgIH0KICAgICAgICAgY2FzZSBDU1NQcm9wZXJ0eU1heFdpZHRoOiB7Ci0gICAgICAgICAgICBj b25zdCBMZW5ndGgmIG1heFdpZHRoID0gc3R5bGUtPm1heEhlaWdodCgpOworICAgICAgICAgICAg Y29uc3QgTGVuZ3RoJiBtYXhXaWR0aCA9IHN0eWxlLT5tYXhXaWR0aCgpOwogICAgICAgICAgICAg aWYgKG1heFdpZHRoLmlzRml4ZWQoKSAmJiBtYXhXaWR0aC52YWx1ZSgpID09IHVuZGVmaW5lZExl bmd0aCkKICAgICAgICAgICAgICAgICByZXR1cm4gQ1NTUHJpbWl0aXZlVmFsdWU6OmNyZWF0ZUlk ZW50aWZpZXIoQ1NTVmFsdWVOb25lKTsKICAgICAgICAgICAgIHJldHVybiBDU1NQcmltaXRpdmVW YWx1ZTo6Y3JlYXRlKG1heFdpZHRoKTsKSW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJKHJldmlzaW9uIDM3MTUxKQorKysgTGF5 b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTMgQEAKKzIwMDgt MTAtMDEgIEtldmluIE1jQ3VsbG91Z2ggIDxrbWNjdWxsb3VnaEBhcHBsZS5jb20+CisKKyAgICAg ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgQnVnIDIxMjg0OiBNYXgg aGVpZ2h0IGFmZmVjdHMgbWF4IHdpZHRoCisgICAgICAgIC0gTG9va3MgbGlrZSBhIGN1dCBhbmQg cGFzdGUgYnVnCisKKyAgICAgICAgKiBmYXN0L2Nzcy9tYXgtaGVpZ2h0LWFuZC1tYXgtd2lkdGgt ZXhwZWN0ZWQudHh0OiBBZGRlZC4KKyAgICAgICAgKiBmYXN0L2Nzcy9tYXgtaGVpZ2h0LWFuZC1t YXgtd2lkdGguaHRtbDogQWRkZWQuCisKIDIwMDgtMDktMzAgIEVyaWMgQ2FybHNvbiAgPGVyaWMu Y2FybHNvbkBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgQW50dGkKSW5kZXg6IExh eW91dFRlc3RzL2Zhc3QvY3NzL21heC1oZWlnaHQtYW5kLW1heC13aWR0aC1leHBlY3RlZC50eHQK PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvZmFzdC9jc3MvbWF4LWhlaWdodC1hbmQtbWF4LXdp ZHRoLWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2Zhc3QvY3NzL21h eC1oZWlnaHQtYW5kLW1heC13aWR0aC1leHBlY3RlZC50eHQJKHJldmlzaW9uIDApCkBAIC0wLDAg KzEsNyBAQAorVGhpcyBzdWNjZWVkcyBpZiBtYXgtaGVpZ2h0IGFuZCBtYXgtd2lkdGggYXJlIG5v dCBzZXQgdG9nZXRoZXIsIGJ1dCBhcmUgZGlmZmVyZW50IHZhbHVlcy4KKworT24gc3VjY2Vzcywg eW91IHdpbGwgc2VlIGEgc2VyaWVzIG9mICJQQVNTIiBtZXNzYWdlcywgZm9sbG93ZWQgYnkgIlRF U1QgQ09NUExFVEUiLgorCisKK1BBU1MgbWF4LXdpZHRoIGlzIGluZGVwZW5kZW50IG9mIG1heC1o ZWlnaHQuCisKSW5kZXg6IExheW91dFRlc3RzL2Zhc3QvY3NzL21heC1oZWlnaHQtYW5kLW1heC13 aWR0aC5odG1sCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL2Zhc3QvY3NzL21heC1oZWlnaHQt YW5kLW1heC13aWR0aC5odG1sCShyZXZpc2lvbiAwKQorKysgTGF5b3V0VGVzdHMvZmFzdC9jc3Mv bWF4LWhlaWdodC1hbmQtbWF4LXdpZHRoLmh0bWwJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEsMzUg QEAKKzwhRE9DVFlQRSBIVE1MIFBVQkxJQyAiLS8vSUVURi8vRFREIEhUTUwvL0VOIj4KKzxodG1s PgorPGhlYWQgaWQ9Im15aGVhZCI+Cis8bGluayByZWw9InN0eWxlc2hlZXQiIGhyZWY9Ii4uL2pz L3Jlc291cmNlcy9qcy10ZXN0LXN0eWxlLmNzcyI+Cis8c2NyaXB0IHNyYz0iLi4vanMvcmVzb3Vy Y2VzL2pzLXRlc3QtcHJlLmpzIj48L3NjcmlwdD4KKzxzY3JpcHQgc3JjPSIuLi9qcy9yZXNvdXJj ZXMvanMtdGVzdC1wb3N0LWZ1bmN0aW9uLmpzIj48L3NjcmlwdD4KKzxzY3JpcHQ+CitpZiAod2lu ZG93LmxheW91dFRlc3RDb250cm9sbGVyKQorICAgIHdpbmRvdy5sYXlvdXRUZXN0Q29udHJvbGxl ci5kdW1wQXNUZXh0KCk7CisKK2Z1bmN0aW9uIGRvVGVzdCgpCit7CisgICAgZGVzY3JpcHRpb24o IlRoaXMgc3VjY2VlZHMgaWYgbWF4LWhlaWdodCBhbmQgbWF4LXdpZHRoIGFyZSBub3Qgc2V0IHRv Z2V0aGVyLCBidXQgYXJlIGRpZmZlcmVudCB2YWx1ZXMuIik7CisKKyAgICB2YXIgYm94ID0gZG9j dW1lbnQuY3JlYXRlRWxlbWVudCgiZGl2Iik7CisgICAgYm94LnN0eWxlWydtYXgtaGVpZ2h0J10g PSAiMjBweCI7CisgICAgYm94LnN0eWxlWydtYXgtd2lkdGgnXSA9ICIzMHB4IjsKKworICAgIGRv Y3VtZW50LmdldEVsZW1lbnRCeUlkKCJib2R5IikuYXBwZW5kQ2hpbGQoYm94KTsgICAgCisKKyAg ICB2YXIgc3R5bGUgPSB3aW5kb3cuZ2V0Q29tcHV0ZWRTdHlsZShib3gpOworICAgIGlmKHN0eWxl LmdldFByb3BlcnR5VmFsdWUoIm1heC1oZWlnaHQiKSA9PSBzdHlsZS5nZXRQcm9wZXJ0eVZhbHVl KCJtYXgtd2lkdGgiKSkKKyAgICAgICAgdGVzdEZhaWxlZCgibWF4LXdpZHRoIGlzIE5PVCBpbmRl cGVuZGVudCBvZiBtYXgtaGVpZ2h0LiIpOworICAgIGVsc2UKKyAgICAgICAgdGVzdFBhc3NlZCgi bWF4LXdpZHRoIGlzIGluZGVwZW5kZW50IG9mIG1heC1oZWlnaHQuIik7CisKKyAgICB3aW5kb3cu c3VjY2Vzc2Z1bGx5UGFyc2VkID0gdHJ1ZTsKK30KKzwvc2NyaXB0PgorPC9oZWFkPgorPGJvZHkg b25sb2FkPSJkb1Rlc3QoKSIgaWQ9ImJvZHkiPgorPHAgaWQ9ImRlc2NyaXB0aW9uIj48L3A+Cis8 ZGl2IGlkPSJjb25zb2xlIj48L2Rpdj4KKzwvYm9keT4KKzwvaHRtbD4K