Bug 87735 - [Chromium] [Regression] SELECT popup misplaced when there is not enough vertical space
Summary: [Chromium] [Regression] SELECT popup misplaced when there is not enough verti...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Alexander Pavlov (apavlov)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-29 06:06 PDT by Alexander Pavlov (apavlov)
Modified: 2012-05-29 06:41 PDT (History)
2 users (show)

See Also:

Attachments
Patch (2.36 KB, patch)
2012-05-29 06:20 PDT, Alexander Pavlov (apavlov) 		tkent: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Pavlov (apavlov) 2012-05-29 06:06:26 PDT 
1. Set a screen resolution with the height of 900 or less.
2. Go to http://rezina.ua/ and scroll the page so that the SELECTs "Резина > Бренд" OR "Диски > Бренд" are exactly in the middle of the page
3. Click one of these SELECTs.

A popup with options is opened in the top of the page instead of immediately above or below the SELECT element. It can also overlap the SELECT element in certain circumstances.

Upstreaming http://code.google.com/p/chromium/issues/detail?id=129562 (the bug contains a screenshot).

This regression was introduced in http://trac.webkit.org/changeset/113418
Comment 1 Alexander Pavlov (apavlov) 2012-05-29 06:20:25 PDT 
Created attachment 144538 [details]
Patch
Comment 2 Kent Tamura 2012-05-29 06:35:41 PDT 
Comment on attachment 144538 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=144538&action=review

ok

> Source/WebCore/ChangeLog:10
> +        No new tests, as popups are not testable in Chromium.

Right.

Let's consider to introduce a unit test for the rect calculation later.
Comment 3 Alexander Pavlov (apavlov) 2012-05-29 06:41:22 PDT 
Committed r118768: <http://trac.webkit.org/changeset/118768>