Bug 103684 - [CSS Regions] border-radius on a region causes overflow to be hidden
Summary: [CSS Regions] border-radius on a region causes overflow to be hidden
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Mihnea Ovidenie
URL:
Keywords: AdobeTracked
Depends on:
Blocks: 57312
  Show dependency treegraph
 
Reported: 2012-11-29 16:17 PST by Rebecca Hauck
Modified: 2013-02-21 09:11 PST (History)
7 users (show)

See Also:


Attachments
test case to reproduce bug (3.94 KB, text/html)
2012-11-29 16:17 PST, Rebecca Hauck
no flags Details
Patch (5.09 KB, patch)
2013-01-24 07:55 PST, Mihnea Ovidenie
no flags Details | Formatted Diff | Diff
Patch (3.40 KB, patch)
2013-02-18 12:22 PST, Mihnea Ovidenie
no flags Details | Formatted Diff | Diff
Patch for landing (3.56 KB, patch)
2013-02-21 08:50 PST, Mihnea Ovidenie
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Rebecca Hauck 2012-11-29 16:17:05 PST
Found in Version 25.0.1335.0 canary

1. Open the attached file in a browser

Result: 

The overflow text is hidden but should be visible as it should inherit the default value of overflow-x and overflow-y. The computedStyle shows as visible.

Comment out the border-radius on region-3 and the overflow text will again be visible.

Just as a control, I tested this scenario without regions with just a div + border-radius and the default overflow behavior is correct in that scenario
Comment 1 Rebecca Hauck 2012-11-29 16:17:28 PST
Created attachment 176845 [details]
test case to reproduce bug
Comment 2 Mihnea Ovidenie 2013-01-24 07:55:44 PST
Created attachment 184494 [details]
Patch
Comment 3 Alexandru Chiculita 2013-01-24 09:45:36 PST
Comment on attachment 184494 [details]
Patch

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

> Source/WebCore/rendering/RenderReplaced.cpp:146
>              completelyClippedOut = true;

How does this work with the overflow of the RenderRegion? It looks like it will just not paint the overflow at all.
Comment 4 Mihnea Ovidenie 2013-02-18 12:22:27 PST
Created attachment 188929 [details]
Patch

The fix for https://bugs.webkit.org/show_bug.cgi?id=74132 also fixed this bug. I still think it deserves a test.
Comment 5 Mihnea Ovidenie 2013-02-21 08:50:26 PST
Created attachment 189539 [details]
Patch for landing
Comment 6 WebKit Review Bot 2013-02-21 09:11:14 PST
Comment on attachment 189539 [details]
Patch for landing

Clearing flags on attachment: 189539

Committed r143613: <http://trac.webkit.org/changeset/143613>
Comment 7 WebKit Review Bot 2013-02-21 09:11:17 PST
All reviewed patches have been landed.  Closing bug.