New issue
Advanced search Search tips

Issue 813668 link

Starred by 2 users
Status: Duplicate
Merged: issue 765985
Owner:
schenney@chromium.org
Closed: Feb 2018
Cc:
vamshi.kommuri@chromium.org
vmp...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

border-radius on <iframe>s do not work on retina displays

Reported by ja...@medbridgeed.com, Feb 20 2018 
UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.167 Safari/537.36

Steps to reproduce the problem:
Here is a codepen of the issue: https://codepen.io/jimmykup/pen/VQXYvG

An iframe has a border-radius. Viewing it on a normal definition external display shows the border-radius. If I move the browser window over to the retina display of my Mac the border-radius disappears.

What is the expected behavior?
Rounded corners on an iframe.

What went wrong?
The rounded corners do not show up when viewed on a retina display.

Did this work before? Yes Chrome 63

Does this work in other browsers? Yes

Chrome version: 64.0.3282.167  Channel: stable
OS Version: OS X 10.12.6
Flash Version:

Comment 1 by krajshree@chromium.org, Feb 20 2018

Labels: Needs-Triage-M64 Needs-Bisect

Comment 2 by vamshi.kommuri@chromium.org, Feb 20 2018

Cc: vamshi.kommuri@chromium.org
Components: Blink>Paint
Labels: -Pri-2 -Needs-Bisect ReleaseBlock-Stable Triaged-ET RegressedIn-65 M-64 FoundIn-64 FoundIn-65 Target-64 OS-Linux OS-Windows Pri-1
Owner: schenney@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce the issue on reported chrome version 64.0.3282.167 using Mac 10.13.1, Ubuntu 14.04 and Windows 10. As the issue is not seen in latest beta 65.0.3325.73 and canary 66.0.3350.0, providing reverse bisect information.

Reverse Bisect Info:
=======================
Last Bad Build  : 65.0.3287.0 (Revision-522296)
First Good Build: 65.0.3288.0 (Revision-522666)

You are probably looking for a change made after 522576 (known good), but no later than 522577 (first known bad).
CHANGELOG URL:
 https://chromium.googlesource.com/chromium/src/+log/afccf64114b233fe65af7526b2f015efb2d1e7c1..88986f832c463a210324d7d9cf9b5213f7b69f63

Suspecting the same.
Review URL: https://chromium-review.googlesource.com/772779

@schenney: Could you please check and merge the fix to M-64 if it is a valid candidate. Please help in assigning it to others if not related to your change. 
Note: Adding RB-Stable, Please remove if not the case.

Thanks!

Comment 3 by schenney@chromium.org, Feb 20 2018

Mergedinto: 765985
Status: Duplicate (was: Assigned)
This has always been broken, up until the time I fixed it. The patch is complicated and not a good merge candidate for M-64 given the remaining lifetime of that release.
►

Sign in to add a comment