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

Issue 651782 link

Starred by 2 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Shadow table rendering miscalculation after content update

Reported by giovann...@gmail.com, Sep 30 2016

Issue description

UserAgent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:48.0) Gecko/20100101 Firefox/48.0

Example URL:
https://jsfiddle.net/thg2k/anom72c9/

Steps to reproduce the problem:
1. Observe shadow initial rendering
2. Click "remove last row" link
3. Click "fix up" link

What is the expected behavior?
After removing the line, the shadow should still look the same

What went wrong?
I suspect there is a miscalculation on the position of the shadow after child content update, it does not account for the table border, but it is influenced by the border collapse and the cells height.

Does it occur on multiple sites: Yes

Is it a problem with a plugin? No 

Did this work before? No 

Does this work in other browsers? Yes 

Chrome version: 53.0.2785.143 (Official Build) m (64-bit)  Channel: stable
OS Version: 6.1 (Windows 7, Windows Server 2008 R2)
Flash Version: 

Tested on Chrome 51 on linux, same problem happens.
 
scr_chrome_bug.png
12.7 KB View Download
Cc: senorblanco@chromium.org wkorman@chromium.org
Components: -Blink Blink>CSS>Filters Blink>Paint
Status: Available (was: Unconfirmed)
A cull rect problem? Or maybe a filters problem.
Components: -Blink>CSS>Filters Blink>CSS
box-shadow doesn't use CSS filters. I'm suspecting a paint invalidation problem? Or maybe a layout problem, since it looks like the area between the element and the text below is insufficient for the shadow.
Components: -Blink>CSS Blink>Layout
Labels: Needs-Bisect
I think it's a layout issue. Agree with senorblanco's reasoning.
Cc: krajshree@chromium.org
Labels: Needs-Feedback OS-Linux OS-Mac
Retested the issue on Windows 10, Ubuntu 14.04 and Mac OS 10.11.6 using reported version  #53.0.2785.143 and latest canary #55.0.2880.4 as per comment #0.

The steps followed to reproduce the issue are as follows:
----------
1. Navigated to url https://jsfiddle.net/thg2k/anom72c9/
2. Clicked "remove last row" link
3. Clicked "fix up" link
4. Observed that after removing the line, the shadow showed initial rendering as expected.

Hence, the issue is not reproducible.

Attaching screencast for the same.

Reporter@ - Could you please verify the screencast and let us know if anything missed from our side.

Thanks,
651782.mp4
522 KB View Download
Dude the bug IS clearly visible in your screen cast, *BEFORE* clicking "fix up"
Thanks for your clarification....!! I will be providing the bisect info.
Labels: -Needs-Feedback -Needs-Bisect M-55
Status: Untriaged (was: Available)
Able to reproduce the issue on MAC 10.11.6, Windows 10 and Ubuntu 14.04 using chrome reported version #53.0.2785.143 and latest canary #55.0.2880.4

This is a non regression as it is observed from M35, M45 and M50 old builds.
Hence, marking it as untriaged to get more inputs from dev team.

Thanks,
Status: Available (was: Untriaged)
Components: -Blink>Paint Blink>Paint>Invalidation

Comment 10 by xing...@intel.com, Oct 27 2016

If we change the style as this, we can see it more clear:
<style>
table.buggy {
  background-color: #ffffff;
  border-collapse: collapse;
  border: 10px solid green;
 }
tr td {
  height: 70px; 
   vertical-align: text-top;
}
</style>


And when I changed the fix_refresh as below, it also works well.
fix_refresh = function() {
  document.getElementById('refreshMe').style.height = '81px';
  document.getElementsByClassName('price')[0].style.backgroundColor  ='red';
  document.getElementsByClassName('price')[0].style.width='0px';
}

So I think the root cause is refreshMe is re-layouted when fix_refresh is  called, while price not. 

BTW, when remove border-collapse: collapse, it works well like FF.

Comment 11 by xing...@intel.com, Oct 27 2016

Hi, I am interested with this issue, and I am looking for a proper place to force the td's get layouted when its sibling's style is changed.
Components: -Blink>Layout -Blink>Paint>Invalidation Blink>Layout>Table
Components: Blink>Paint>Invalidation
Project Member

Comment 14 by sheriffbot@chromium.org, Oct 30 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

Comment 15 by e...@chromium.org, Oct 30 2017

Status: Available (was: Untriaged)
Still broken
Bug confirmed on Chrome 62.0.3202.75

Please keep this open, it is definitely not something you can ignore, as I actually ran into this while doing actual development.

Project Member

Comment 17 by sheriffbot@chromium.org, Oct 31

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.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Available (was: Untriaged)

Sign in to add a comment