-
Notifications
You must be signed in to change notification settings - Fork 2.6k
gpu difference between chrome and brave #6506
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Not reproducible on Ubuntu
Brave 0.70.117 and Chrome 78.0.3904.50. Viz Hit-test Surface Layer is disabled in both |
Originally when I was comparing However, Google pushed a new CR78 a ~3-4hrs ago and Perhaps we're being placed into field trails in Chrome where |
@GeetaSarvadnya to confirm, you are comparing Chrome 78 to Brave using Chromium 78, correct? |
From what I can tell, this feature is still disabled by default in C78. The change to enable it by default is here: https://chromium.googlesource.com/chromium/src/+/55112a03100d6f33ce83a84e9df89db94bcf4356 and has only landed in C79 (https://storage.googleapis.com/chromium-find-releases-static/551.html#55112a03100d6f33ce83a84e9df89db94bcf4356). I thought that the feature is then enabled in Chrome via experiments, but when I check chrome://flags this particular one is set to Default for me. There's also this curious tidbit in // In theory the following switch should have no effects, but its absence // would cause VizHitTestSurfaceLayer feature to be enabled on Chromium // builds, causing a test failure due to https://crbug.com/996865. "force-fieldtrials", So, I think the feature is properly disabled in our build, but I can't quite tell why it is enabled in Chrome. |
Good call, @mkarolin - this might be a @GeetaSarvadnya confirmed it was C78 earlier I'm curious what variations are exposed for you in Chrome 78, @GeetaSarvadnya. You can view those by visiting chrome://version in Chrome. If you see entries there, it's possible one is for VizHitTestSurfaceLayer |
Closing out as wontfix. Difference is Chrome has fields trial enabled likely changing these values for some users. |
Description
Viz Hit-test Surface Layer
is different on Chrome and Brave for the same chromium version.This has been the case for me for several versions (using macOS), but @kjozwiak sees them as the same on his macOS. I'm currently using High Sierra, he has Mojave.
Steps to Reproduce
Viz Hit-test Surface Layer
setting.Actual result:
They differ.

Using C77:
Using C78:

Expected result:
Expected them to be the same.
Reproduces how often:
easily
Brave version (brave://version info)
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
cc @bsclifton @kjozwiak
The text was updated successfully, but these errors were encountered: