Last Comment Bug 807174 - Drop-down list used inside a frameset document renders outside the frame
: Drop-down list used inside a frameset document renders outside the frame
Regressed by 575294
: regression, testcase
Product: Core
Classification: Components
Component: Layout: Form Controls (show other bugs)
: 16 Branch
: x86_64 All
-- normal (vote)
: mozilla19
Assigned To: Mats Palmgren (:mats)
: Manuela Muntean [Away]
: Jet Villegas (:jet)
: 807288 (view as bug list)
Depends on:
Blocks: CVE-2012-3984
  Show dependency treegraph
Reported: 2012-10-30 17:18 PDT by Loic
Modified: 2012-12-17 00:32 PST (History)
9 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Testcase (1.17 KB, text/html)
2012-10-30 17:18 PDT, Loic
no flags Details
Screenshot on Win 7 (25.42 KB, image/png)
2012-10-30 17:19 PDT, Loic
no flags Details
fix (1.54 KB, patch)
2012-10-30 19:51 PDT, Mats Palmgren (:mats)
roc: review+
lukasblakk+bugs: approval‑mozilla‑aurora+
lukasblakk+bugs: approval‑mozilla‑beta+
Details | Diff | Splinter Review

Description User image Loic 2012-10-30 17:18:02 PDT
1) Open the testcase
2) Click on the drop-down list displayed in the footer

With FF16+, the drop-down list is rendered outside the frame (and the Firefox window sometimes).

Mozregression range:



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:
<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>

Comment 1 User image Loic 2012-10-30 17:18:48 PDT
Created attachment 676855 [details]
Comment 2 User image Loic 2012-10-30 17:19:25 PDT
Created attachment 676856 [details]
Screenshot on Win 7
Comment 3 User image Alice0775 White 2012-10-30 17:49:37 PDT
Regression window(m-i)
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/16.0 Firefox/16.0 ID:20120622175046
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/16.0 Firefox/16.0 ID:20120622182843

In local build:
Last Good: e3676fde39a8
First Bad: 23f5c88adb8f

Triggered by:
23f5c88adb8f	Mats Palmgren — Bug 575294. part=2/5 r=smaug,roc
Comment 4 User image Mats Palmgren (:mats) 2012-10-30 19:51:29 PDT
Created attachment 676891 [details] [diff] [review]

Use GetToplevelContentDocumentPresContext() instead of the pres context
of the combobox when calculating the available space.
Comment 6 User image Alice0775 White 2012-10-31 06:34:15 PDT
*** Bug 807288 has been marked as a duplicate of this bug. ***
Comment 7 User image Ed Morley [:emorley] 2012-10-31 07:15:23 PDT
Comment 8 User image Lukas Blakk [:lsblakk] use ?needinfo 2012-10-31 12:13:07 PDT
Would consider late beta uplift nomination since this has a testcase and is easily verifiable by QA.
Comment 9 User image Mats Palmgren (:mats) 2012-11-04 01:00:38 PDT
Comment on attachment 676891 [details] [diff] [review]

[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 User image Lukas Blakk [:lsblakk] use ?needinfo 2012-11-05 12:24:05 PST
Comment on attachment 676891 [details] [diff] [review]

Please land asap so this gets into tomorrow's beta.
Comment 12 User image Manuela Muntean [Away] 2012-12-17 00:32:54 PST
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

Note You need to log in before you can comment on or make changes to this bug.