New issue
Advanced search Search tips

Issue 853109 link

Starred by 3 users

Issue metadata

Status: WontFix
Owner:
Closed: Jul 16
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug-Regression



Sign in to add a comment

[Chome][V66][Android 8.0] Request to improve a bad text visibility in landscape mode

Reported by seiyon.p...@gmail.com, Jun 15 2018

Issue description

Steps to reproduce the problem:
1. Reproducible device : Samsung Galaxy S7, Samsung Galaxy A5(
2. Open up Chrome
3. Go to "naver.com"
4. Rotate a device to enter in landscape mode

), Samsung Galaxy S7(Mali-T880) and Samsung Galaxy A5(Mali-T830) 모두 

What is the expected behavior?

What went wrong?
Text visibility is bad.
The visibility of v66 is worse that one of v64.

Did this work before? Yes v64

Chrome version: 66.0.3359.181  Channel: stable
OS Version: 8.0
Flash Version: 

There is a bad visibility issue on devices with GPU "Mali-T830" and "Mali-T880".
Chrome shader precision setting change to low(lp) have a bad impact on text visibility.
 
Chrome_v64_expected behavior.png
279 KB View Download
Chrome_v66_wrong behavior.png
347 KB View Download
Labels: Needs-triage-Mobile
Cc: jbanavatu@chromium.org
Components: Blink
Labels: -Pri-2 hasbisect-per-revision Target-67 M-68 RegressedIn-66 M-69 Triaged-Mobile M-67 FoundIn-67 Target-69 Target-68 FoundIn-68 FoundIn-69 Pri-1
Owner: bbudge@chromium.org
Status: Assigned (was: Unconfirmed)
Tested the issue in Android 7.0 and able to reproduce the issue. 

Steps Followed:
1. Launch Chrome.
2. Navigated to naver.com
3. Change to landscape mode and Observed bad text visibility(Text is pixelated)

Chrome versions tested:
67.0.3396.87(Stable), 69.0.3464.0(Canary)

OS:
Android 7.0.0

Android Devices:Samsung S7

Using the per-revision bisect providing the bisect results,
Good Build - 66.0.3341.0 (534605)
Bad Build - 66.0.3342.0 (534887)

You are looking for a change made after 534605(GOOD), but before 534606(BAD).

https://chromium.googlesource.com/chromium/src/+/9d94df31c926ba5e8f02f85e88686ee1482cf560

@Bill Budge : Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to owner concerned.

Please navigate to below link for log's and screen cast --
go/chrome-androidlogs/853109

Thanks!

Comment 4 by bbudge@chromium.org, Jun 18 2018

Cc: bbudge@chromium.org
Owner: ----
Status: Untriaged (was: Assigned)
Sorry, I have no insight on the cause of this. My change only affects unittests (no changes to shipping Chrome/WebView code on any platform.)

Could you retry the bisection? Remarking as Untriaged, and needs an owner. I would suggest a GPU or Blink person.

Comment 5 by bokan@chromium.org, Jun 19 2018

Components: -UI -Blink Internals>GPU
Labels: -RegressedIn-66 RegressedIn-65
Owner: enne@chromium.org
Status: Assigned (was: Untriaged)
As per comment #4, Bisected again. Below are the details:

Using the per-revision bisect providing the bisect results,
Good Build - 66.0.3322.0 (529187)
Bad Build - 66.0.3324.0 (530027)

You are looking for a change made after 529610(GOOD), but before 529611(BAD).

https://chromium.googlesource.com/chromium/src/+/97414c7ab5cfc16a9e672de768754279e4ca5720

@Adrienne Walker : Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to owner concerned.

Please navigate to below link for log's and screen cast --
go/chrome-androidlogs/853109

Thanks!
typo @ Good and Bad builds:
Good Build - 65.0.3322.0 (529187)
Bad Build - 65.0.3324.0 (530027)

Thanks!

Comment 8 by enne@chromium.org, Jun 19 2018

Hmm, I landed a followup for a similar bug in https://chromium-review.googlesource.com/883944 which got merged back to m65.  I suppose that doesn't address this issue though if it is also having trouble.

I should be able to get an S7 by Friday and can take a look then.
Cc: ligim...@chromium.org
Labels: -M-67 ReleaseBlock-Stable ReleaseBlock-Beta
Thanks for the update. Since this is a recent regression, it would be great to have a fix for M68/ M69.
Labels: -ReleaseBlock-Beta
This will not block this week's M68 Beta.

Comment 11 by enne@chromium.org, Jun 27 2018

I cannot make this repro on ToT Chrome (r570807) on a Galaxy S7 (SM-G930F with a Mali-T880).  Attached are pictures of this looking correct.

Is there anything else I need to do to repro this?
Screenshot_20180627-132531_Chromium.jpg
497 KB View Download
Screenshot_20180627-132540_Chromium.jpg
464 KB View Download

Comment 12 by enne@chromium.org, Jun 28 2018

Just to be sure, I tried a chromium build at 530033 and could not reproduce it either on the same device.

Does anybody have any extra hints on how to reproduce this?
Not able to repro text issue being Pixelated;
Device: Samsung Galaxy A5 / NRD90M
Android Versions: 68.0.3440.40 & 69.0.3480.0

1. Open the url 'naver.com' in Chrome.
2. Rotate to landscape mode.

Only issue i see is a gray blan area on the left side of the screen.

Please find uploaded screen-record @
https://drive.google.com/corp/drive/u/0/folders/1ZrOZoRDQNn2TXI4UVSmu7NNRsifR5f7W

jbanavatu@ are you still able to reproduce this issue?
Tested on chrome version 67.0.3396.87 , 69.0.3486.0 using samsung galaxy s7 and Unable to reproduce this issue .

Thanks!
853109.png
403 KB View Download
Dear Chrome team

Please see the description and note that this issue depends on GPU.
It is still reproducible on 67.0.3396.87 using Samsung Galaxy S7 (Mali-T880) and Samsung Galaxy A5 (Mali-T830).
All models using the Mali-T880 / Mali-T830 reports this issue.

Thanks.
Somehow we are unable to reproduce this issue. Tested on 67.0.3396.87 using samsung s7 (with GPU Mali-T880).
Please refer to the screen-cast attached.

Thanks!
Please refer to the screen-cast attached.

Thanks!
853109_notrepro.mp4
5.1 MB View Download
enne@ are you looking into this issue? M68 release is next week.
No, I am not looking into this issue.  I cannot reproduce it and neither can jbanavatu.
I am going to give this one last try today and see if I can bisect and verify that the followup change I listed above addressed this issue in the past.

Unless there are more precise instructions on how to reproduce, I don't think there's much else I can do at the moment.
Status: WontFix (was: Assigned)
Bisected this manually most of the way, and found 531643 was bad, while 531659 was good.  This is why this doesn't repro on ToT or 67.

As I suspected, https://chromium-review.googlesource.com/883944 is likely why this got fixed, that I mentioned in comment #8.

Closing this bug as WontFix.
Dear Chrome team

It's reproducible in O OS not N OS.
Could you please check if it's reproducible on O OS device using GPU "Mali-T830" and "Mali-T880"?
All of my testing is on Oreo.  I think comment #22 still stands.  This was broken in 529611, and fixed in 531658.
enne@ could you let me know which version has a fix so that I will be able to test it?
What you meat by 531658 was it's fixed on 57.0.5316.58?
It should be in any version later than 65.0.3325.49.

Sign in to add a comment