|
1.17 KB,
text/html
|
Details | |
|
25.42 KB,
image/png
|
Details | |
|
1.54 KB,
patch
|
roc
:
review+
lsblakk
:
approval-mozilla-aurora+
lsblakk
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
STR: 1) Open the testcase 2) Click on the drop-down list displayed in the footer Result: With FF16+, the drop-down list is rendered outside the frame (and the Firefox window sometimes). Mozregression range: m-c good=2012-06-23 bad=2012-06-24 http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=bb4b37094b9f&tochange=cb2904476d14 m-i good=2012-06-22 bad=2012-06-23 http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=a7621ccb684e&tochange=daf9c42be518 Suspected bug: Robert O'Callahan — Bug 766007. Create scrollframes for frameset documents. r=tnikkel For the record, utf-8 translation of base64 part from the testcase: <html> <body> <select name="testdropdown" style="width:150px;" > <option value="1">option 1</option> <option value="2">option 2</option> <option value="3">option 3</option> <option value="4">option 4</option> <option value="5">option 5</option> <option value="6">option 6</option> <option value="7">option 7</option> <option value="8">option 8</option> <option value="9">option 9</option> </select> </body> </html> Reporter: http://forums.mozillazine.org/viewtopic.php?f=25&t=2592587
Created attachment 676855 [details]
TestcaseCreated attachment 676856 [details] Screenshot on Win 7
|
||
Comment 3•5 years ago
|
||
Regression window(m-i) Good: http://hg.mozilla.org/integration/mozilla-inbound/rev/80b8680bda1c Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/16.0 Firefox/16.0 ID:20120622175046 Bad: http://hg.mozilla.org/integration/mozilla-inbound/rev/c35d2d3071ac Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/16.0 Firefox/16.0 ID:20120622182843 http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=80b8680bda1c&tochange=c35d2d3071ac In local build: Last Good: e3676fde39a8 First Bad: 23f5c88adb8f Triggered by: 23f5c88adb8f Mats Palmgren — Bug 575294. part=2/5 r=smaug,roc
|
|
||
Updated•5 years ago
|
||
|
(Assignee) | |
Updated•5 years ago
|
||
|
|
(Assignee) | |
Comment 4•5 years ago
|
||
Created attachment 676891 [details] [diff] [review] fix Use GetToplevelContentDocumentPresContext() instead of the pres context of the combobox when calculating the available space.
|
|
(Assignee) | |
Comment 5•5 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/1858a5369d22
|
||
Comment 7•5 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/1858a5369d22
|
||
Comment 8•5 years ago
|
||
Would consider late beta uplift nomination since this has a testcase and is easily verifiable by QA.
|
|
(Assignee) | |
Comment 9•5 years ago
|
||
Comment on attachment 676891 [details] [diff] [review] fix [Approval Request Comment] Bug caused by (feature/regressing bug #): 575294 User impact if declined: cosmetic layout error for combobox menu in an edge case, the control should still be usable without the fix Testing completed (on m-c, etc.): on m-c since 2012-10-31 Risk to taking this patch (and alternatives if risky): low risk String or UUID changes made by this patch: none
|
|
||
Comment 10•5 years ago
|
||
Comment on attachment 676891 [details] [diff] [review] fix Please land asap so this gets into tomorrow's beta.
|
|
(Assignee) | |
Comment 11•5 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/c82029001796 https://hg.mozilla.org/releases/mozilla-beta/rev/6c1314c972a4
|
||
Comment 12•5 years ago
|
||
Verified fixed on Firefox 18 beta 4. User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:18.0) Gecko/20100101 Firefox/18.0 Build ID: 20121212073002
Description
•