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

Issue 766021 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug-Regression



Sign in to add a comment

regression in CSS 3D transforms

Reported by hall.ste...@gmail.com, Sep 18 2017

Issue description

Steps to reproduce the problem:
1. Open http://projects.delimited.io/experiments/d3-threejs/
2. Observe the glitchy behavior of the 3d transformed elements

What is the expected behavior?
The elements do not flash or disappear.

What went wrong?
This is a regression.  The demo has worked for years in Chrome on Android.  The demo works on desktop (mac os sierra)  in Chrome, Firefox, Safari and Brave.  It also works great on an iPad using Chrome, FireFox or Safari.  It works in FireFox on android, but the performance of the JS is terrible.  Hoping you can fix it!

Did this work before? Yes Not sure -  The regression happened in the last 6 months or so I would guess.  Just noticed it.

Does this work in other browsers? Yes

Chrome version: 60.0.3112.116  Channel: n/a
OS Version: 6.0.0;LG-H815
Flash Version: 

It does appear to be somewhat a function of the number of elements.  I setup a link with 50 chart elements for comparison...
http://projects.delimited.io/experiments/d3-threejs-50/

It does seem to help but does not eliminate the problem.
 
Capture+_2017-09-17-17-33-07.png
639 KB View Download

Comment 1 Deleted

Comment 2 Deleted

Components: Blink>Compositing>Transform3D
Cc: msrchandra@chromium.org nyerramilli@chromium.org ligim...@chromium.org sandeepkumars@chromium.org
Labels: TE-Needs-Triage TE-NeedsTriageFromMTV
Unable to reproduce the issue using #60.0.3112.116 on Android 6.0.1: SM-J710F Build/MMB29K. No flashing or glitchy behavior is observed.

Adding TE-NeedsTriageFromMTV label for triaging of the issue as we don't have #6.0.0; LG-H815 device with us.

Thanks!!
Cc: prashanthpola@chromium.org
Labels: triage-te
prashanthpola@, please check this on the LG G4.
I can reproduce this issue on a MOTOROLA MOTO X 2ND GEN as well.  Doing this via Browserstack.  

Screencast
https://screencast.com/t/yXogg8KS

Screenshot attached.
Screen Shot 2017-09-18 at 1.04.44 PM.png
913 KB View Download
LG G5 also shows the same issue.
Screen Shot 2017-09-18 at 1.17.06 PM.png
946 KB View Download
Reproduced on Samsung Galaxy S7 
Screen Shot 2017-09-18 at 1.20.03 PM.png
1.1 MB View Download
Owner: vmp...@chromium.org
Status: Assigned (was: Unconfirmed)
We are able to repro the issue on Chrome:61.0.3163.96,62.0.3202.19,63.0.3218.0 Device:LG G4(LG-H815)/MRA58K

Bisect info:
Good build:60.0.3100.0
Bad build:60.0.3101.0
Regression range: https://chromium.googlesource.com/chromium/src/+log/60.0.3100.0..60.0.3101.0?pretty=fuller&n=10000

Good commit:471875
Bad commit:471876
Culprit CL: https://chromium.googlesource.com/chromium/src/+/dc5e1afbd1f6b01e44b58ee16f4c556d2ceaa04c
Labels: -TE-NeedsTriageFromMTV -triage-te -TE-Needs-Triage hasbisect-per-revision
Labels: M-61
Project Member

Comment 12 by bugdroid1@chromium.org, Oct 4 2017

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

commit 8ec06d20a359f2a59527e309dce52df49c2d4ae5
Author: Vladimir Levin <vmpstr@chromium.org>
Date: Wed Oct 04 19:57:41 2017

cc: Update the approximate scale math to use rect bounds.

This patch changes the approximate scale math to use rect bounds instead
of a vector, which yields more accurate results.

This means that if a layer is effectively shrunk after the transformation
we won't use a larger scale which could have been the case with the
scale math.

R=chrishtr@chromium.org

Bug: 766021
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I42aea3e7b9900b2a1e5499c8c24d7264b7bd6d3f
Reviewed-on: https://chromium-review.googlesource.com/693303
Commit-Queue: vmpstr <vmpstr@chromium.org>
Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506489}
[modify] https://crrev.com/8ec06d20a359f2a59527e309dce52df49c2d4ae5/cc/base/math_util.cc
[modify] https://crrev.com/8ec06d20a359f2a59527e309dce52df49c2d4ae5/cc/layers/layer_impl_unittest.cc
[modify] https://crrev.com/8ec06d20a359f2a59527e309dce52df49c2d4ae5/third_party/WebKit/LayoutTests/platform/linux/compositing/geometry/layer-due-to-layer-children-deep-expected.png
[modify] https://crrev.com/8ec06d20a359f2a59527e309dce52df49c2d4ae5/third_party/WebKit/LayoutTests/platform/linux/compositing/geometry/layer-due-to-layer-children-expected.png
[modify] https://crrev.com/8ec06d20a359f2a59527e309dce52df49c2d4ae5/third_party/WebKit/LayoutTests/platform/linux/transforms/3d/point-mapping/3d-point-mapping-origins-expected.png
[modify] https://crrev.com/8ec06d20a359f2a59527e309dce52df49c2d4ae5/third_party/WebKit/LayoutTests/platform/mac-mac10.9/transforms/3d/point-mapping/3d-point-mapping-origins-expected.png
[modify] https://crrev.com/8ec06d20a359f2a59527e309dce52df49c2d4ae5/third_party/WebKit/LayoutTests/platform/mac/compositing/geometry/layer-due-to-layer-children-deep-expected.png
[modify] https://crrev.com/8ec06d20a359f2a59527e309dce52df49c2d4ae5/third_party/WebKit/LayoutTests/platform/mac/compositing/geometry/layer-due-to-layer-children-expected.png
[modify] https://crrev.com/8ec06d20a359f2a59527e309dce52df49c2d4ae5/third_party/WebKit/LayoutTests/platform/mac/transforms/3d/point-mapping/3d-point-mapping-origins-expected.png
[modify] https://crrev.com/8ec06d20a359f2a59527e309dce52df49c2d4ae5/third_party/WebKit/LayoutTests/platform/win/compositing/geometry/layer-due-to-layer-children-deep-expected.png
[modify] https://crrev.com/8ec06d20a359f2a59527e309dce52df49c2d4ae5/third_party/WebKit/LayoutTests/platform/win/compositing/geometry/layer-due-to-layer-children-expected.png
[modify] https://crrev.com/8ec06d20a359f2a59527e309dce52df49c2d4ae5/third_party/WebKit/LayoutTests/platform/win/transforms/3d/point-mapping/3d-point-mapping-origins-expected.png

Sign in to add a comment