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

Issue 828925 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug

Blocked on:
issue skia:7785



Sign in to add a comment

Circle is partially filled on iMac 2017 27"

Reported by rose...@gmail.com, Apr 4 2018

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/65.0.3325.181 Safari/537.36

Steps to reproduce the problem:
1. Open test.svg
2. The circle will be partially filled in the top-right corner, as shown in the test.png image.

What is the expected behavior?
The circle should have no fill. 

What went wrong?
The circle is partially filled. 

Did this work before? N/A 

Does this work in other browsers? Yes

Chrome version: 65.0.3325.181  Channel: stable
OS Version: 
Flash Version: 

The test.svg file shows this problem on iMac 27" only. I had similar issues while using MacBook 12-inch 2016, but this particular test file doesn't create the problem on MacBook. I couldn't see anything like this using Safari.
 
test.svg
1.2 KB Download
test.png
63.8 KB View Download

Comment 1 by pdr@chromium.org, Apr 4 2018

Components: Internals>Skia
Owner: fmalita@chromium.org
Status: Untriaged (was: Unconfirmed)
Thanks for the small testcase.

I think this is a Skia bug. The testcase doesn't look minimized but it actually is: removing or moving any of the paths seems to fix this.

Florin, could you triage this?
Cc: bsalomon@chromium.org
Components: Internals>GPU>Rasterization
Status: Assigned (was: Untriaged)
@pdr were you able to repro on something other than iMac 2017 27"?

Comment 3 by pdr@chromium.org, Apr 4 2018

I'm able to reproduce on a highdpi macbook pro (MacOS 10.13.3, macbook pro 15in 2016). If I pass --disable-gpu, it does not reproduce. I tried my usual flags but couldn't easily get it to repro on linux.


Cc: fmalita@chromium.org senorblanco@chromium.org
Owner: bsalomon@chromium.org
Strange, I cannot repro on a similar config: 15" 2016 MBP 10.13.3.

Punting to Brian for further triage.

Comment 5 by pdr@chromium.org, Apr 4 2018

Attaching about:gpu in case it is needed. I'm able to repro in Chrome Dev and Chrome Canary and content shell, so it's unlikely to be related to finch/etc.
aboutgpu.htm
63.7 KB View Download
Owner: egdaniel@chromium.org
Greg is the lucky GPU wrangler.
A-ha:

My about://gpu

GPU0	VENDOR = 0x1002, DEVICE= 0x67ef
GPU1	VENDOR = 0x8086, DEVICE= 0x191b *ACTIVE*

pdr's

GPU0	VENDOR = 0x1002, DEVICE= 0x67ef *ACTIVE*
GPU1	VENDOR = 0x8086, DEVICE= 0x191b

Looks like I'm using the integrated (IntelHD?) GPU while the issue repros with the discrete card (RadeonPro).

@pdr: did you do anything special to make Chrome use the discrete GPU?
Hmm I have an AMD Radeom Pro 560 which does not repo the issue

Comment 9 by pdr@chromium.org, Apr 4 2018

Nice! That's probably it.

I have automatic graphics switching disabled (settings > energy saver > uncheck "Automatic graphics switching"). I'm also plugged into a power source.
I can now repro after forcing Chrome to use the dedicated GPU (disabled "Automatic graphics switching" in Settings).

So it seems to be a RadeonPro driver issue.  I'll see if bisecting yields anything interesting.

Radeon Pro 460:

  Chipset Model:	AMD Radeon Pro 460
  Type:	GPU
  Bus:	PCIe
  PCIe Lane Width:	x8
  VRAM (Total):	4 GB
  Vendor:	AMD (0x1002)
  Device ID:	0x67ef
  Revision ID:	0x00c0
  ROM Revision:	113-C980AF-908
  VBIOS Version:	113-C9801AU-029
  EFI Driver Version:	01.00.908
  Automatic Graphics Switching:	Supported
  gMux Version:	4.0.29 [3.2.8]
  Metal:	Supported, feature set macOS GPUFamily1 v3
  Displays:
Color LCD:
  Display Type:	Built-In Retina LCD
  Resolution:	2880 x 1800 Retina
  Framebuffer Depth:	24-Bit Color (ARGB8888)
  Main Display:	Yes
  Mirror:	Off
  Online:	Yes
  Rotation:	Supported
  Automatically Adjust Brightness:	No


Cc: egdaniel@chromium.org
Owner: fmalita@chromium.org
Owner: ericrk@chromium.org
You are probably looking for a change made after 505839 (known good), but no later than 505859 (first known bad).
CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/2214bd4a3a01c5830d7a26afb4fe20c2dd9ca134..382773265bc845437e2b2abb050768de08ebea3b

Suspecting https://chromium.googlesource.com/chromium/src/+/e6db79b65553a1d4907217f8aaa7cb589ac976c6 rather than the Skia roll, because disabling MSAA (--gpu-rasterization-msaa-sample-count=0) fixes the issue.

Eric, is MSAA supposed to be disabled for this config and for some reason Skia's MSAA status doesn't reflect that?
FYI, we narrowed this down to MSAA + tessellating path renderer and can repro in Skia.  I'll open a separate Skia issue.
Blockedon: skia:7785

Comment 15 by rose...@gmail.com, Apr 5 2018

For info, this is the program which triggered the problem:
https://rosek86.github.io/smith-app/

There is mouseover cursor, once I change cursor position green circle flickers. I was hoping to solve this by changing circle to a 'path' (two arcs) but it seems that half of one arc flickers too (red circle). Is this a separate problem?


Owner: senorblanco@chromium.org
Due to issues with dynamic switching, Chrome typically blacklists MSAA for both GPUs if either is unsupported. Skia shouldn't have these same issues, as we re-create the GrContext on GPU switch. Because of this, we were enabling MSAA on multi-GPU devices when the discrete GPU is in used where we may have previously suppressed it.

It seems like we should just fix the Skia issue (or blacklist MSAA for this device in Skia) - Chrome never intended to blacklist MSAA on this device, and we probably had this bug before my change on single-GPU AMD systems.

Assigning to senorblanco@, as I think the only thing to do here is close this out when the blocking Skia bug is fixed.
Status: Fixed (was: Assigned)
Should be fixed as of the Skia roll including Skia 531a48ed788c5fabfe21704286b54d7567f35469
aka Chrome aa14a6793133f867d63bb3154866c8ae1dfe8c6a.
Cc: sindhu.chelamcherla@chromium.org
 Issue 847080  has been merged into this issue.
Labels: Merge-Request-68
Requesting merge of Skia change https://crrev.com/ to M68. This is a 
fix for a rendering bug that has been reported against multiple sites. The fix is is simple, safe, and has been in Canary since June 1.
Project Member

Comment 20 by sheriffbot@chromium.org, Jun 22 2018

Labels: -Merge-Request-68 Hotlist-Merge-Review Merge-Review-68
This bug requires manual review: M68 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: cmasso@(Android), kariahda@(iOS), bhthompson@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Review-68 Merge-Approved-68
Approved for M68. Branch:3440
Project Member

Comment 22 by sheriffbot@chromium.org, Jun 29 2018

Cc: abdulsyed@google.com
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Approved-68
Sorry, this was merged to M68 as https://skia.googlesource.com/skia.git/+/a4f4c3f16123741b900bcd5e29305e183b020a56. Since it was a Skia merge, bugdroid didn't notice. Removed the Merge-Approved-68 label.

Sign in to add a comment