New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 640272 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 771852
Owner: ----
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , All
Pri: 3
Type: Bug

Blocked on:
issue 801384



Sign in to add a comment

SVG overflow on foreign object incorrect.

Project Member Reported by wkorman@chromium.org, Aug 23 2016

Issue description

Breakout from  http://crbug.com/598051  --

Potentially related to  http://crbug.com/640265  but does not involve zoom, so tentatively presuming it is different.

svg/overflow/overflow-on-foreignObject.svg

Layout test failing, looks incorrectly clipped.
 

Comment 1 by tkent@chromium.org, Aug 23 2016

Components: Blink>SVG
Blocking: -598051
Labels: -Pri-2 Pri-3
Status: Assigned (was: Unconfirmed)
Summary: SVG overflow on foreign object incorrect. (was: SVG overflow on foreign object incorrect with visual rect rtree.)
This test was already broken, and is now in fact improved from previous behavior. From the test:

 Verify default overflow rules on foreignObject
 You should see a 300x300 green rectangle at 100x100 and no scrollbars.

 NOTE: This is currently BROKEN in WebKit. We're clipping the viewport of <foreignObject> correctly,
       though for each positioned object a Layer is created, that has no knownledge of the
       size of the <foreignObject> viewport - as <foreignObject> doesn't create a Layer.
       So for now, you'll see an unclipped content area, just as if overflow was visible.

The test was added 7 years ago in:

https://chromium.googlesource.com/chromium/src/+/2d18ee90d8954e0111916a4e7786d13aa38ea7b8

It appears implementation of clipping for foreign objects has never been fully completed per
above comment.

This bug should not block shipping visual rect rtrees in cc. IMO we should rebaseline the test
and leave this bug open to track completing the feature work.
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 24 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c5d5747f55102dba020cbce7ee9e9c65dc4e22bf

commit c5d5747f55102dba020cbce7ee9e9c65dc4e22bf
Author: wkorman <wkorman@chromium.org>
Date: Wed Aug 24 00:51:31 2016

Rebaseline overflow-on-foreignObject.svg to reflect altered visual rect behavior.

BUG= 640272 
TBR=pdr

Review-Url: https://codereview.chromium.org/2269963004
Cr-Commit-Position: refs/heads/master@{#413913}

[modify] https://crrev.com/c5d5747f55102dba020cbce7ee9e9c65dc4e22bf/third_party/WebKit/LayoutTests/TestExpectations

Cc: wkorman@chromium.org
Owner: ----
Status: Available (was: Assigned)
Project Member

Comment 6 by sheriffbot@chromium.org, Aug 25 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Hotlist-Recharge-Cold PaintTeamTriaged-20170825 BugSource-Team
Status: Available (was: Untriaged)

Comment 8 by f...@opera.com, Jan 12 2018

Blockedon: 801384
Mergedinto: 771852
Status: Duplicate (was: Available)

Sign in to add a comment