New issue
Advanced search Search tips

Issue 816870 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

blurred images in simplified view

Project Member Reported by mar...@mwiacek.com, Feb 27 2018

Issue description

Device name:

From "Settings > About Chrome"
Application version:66.0.3356.0 canary
Operating system:7

URLs (if applicable):https://www.dobreprogramy.pl/Trzy-nowe-iPhoney-X-moga-sprawic-ze-to-bedzie-swietny-rok-dla-Apple,News,86417.html

Steps to reproduce:
(1)enable simplified view in settings\accessbility
(2)open url
(3)scroll a little page & click "show simplified view"

Expected result:
no blurred images

Actual result:
blurred images

 
Screenshot_20180227-120112.png
445 KB View Download

Comment 1 by mar...@mwiacek.com, Feb 27 2018

happens on various subpages from dobreprogramy.pl
Labels: Needs-triage-Mobile
Components: UI>Accessibility UI>Browser>ReaderMode
Cc: pnangunoori@chromium.org
Labels: FoundIn-66 Triaged-Mobile M-66 Target-66 FoundIn-65
Status: Untriaged (was: Unconfirmed)
Tested the issue in Android and able to reproduce the issue. Similar behavior is observed since Chrome #65.0.3325.88

Steps Followed:
1. Launched the Chrome Browser.
2. Navigate to the URL: https://www.dobreprogramy.pl/Trzy-nowe-iPhoney-X-moga-sprawic-ze-to-bedzie-swietny-rok-dla-Apple,News,86417.html
3. Scroll the page up and down.
4. Tap on any "Show simplified view" option.
5. Observed that iPhone image is displayed blurred in the same page navigated to in Step-2.

Chrome versions tested:
65.0.3325.88(Beta), 66.0.3356.0(Canary)

OS:
Android 7.0.0

Android Devices:
Samsung J7

This seems to be a Non-Regression issue as same behavior is seen since M65.  Untriaged for further input's on this issue.

Please navigate to below link for log's and video--
go/chrome-androidlogs/816870

Note: 
1. This issue is not observed in Desktop.
2. Builds before M-65 doesn't have Simplified View option. Hence Untriaged this issue.

Thanks!
Prerequisite for Comment #4:
Enable "Simplified View" from Chrome Settings >> Accessibility.
Cc: mdjones@chromium.org
Labels: -Needs-triage-Mobile android-fe-triaged
Owner: wychen@chromium.org
Status: Assigned (was: Untriaged)
+wychen@ and mdjones@ for further investigation.
 Issue 817749  has been merged into this issue.
This likely has to do with lazy-loaded images or images with the src-set attribute.
There are two <img>s in the <figure>. The first one is a thumbnail, with visibility: hidden, opacity: 0. DOM distiller naively takes the first <img> right now. We should take the first visible <img> instead.
Labels: Hotlist-GoodFirstBug
@wychen I took this up and made the changes suggested by you, but this doesn't seem to solve the problem.
https://chromium-review.googlesource.com/c/chromium/dom-distiller/+/974624
Further checking the code flow.
The <img> is loaded lazily due to which it is not visible until the page is scrolled till that point.
Adding a check to select <img> which has 'data-src' attribute, should work.

Comment 14 Deleted

Good to close this out? 
Bug is NOT fixed.

No, it cannot be closed.
@wychen @mdjones
Added a patch for this bug 
https://chromium-review.googlesource.com/c/chromium/dom-distiller/+/974624
PTAL. Thanks.

Sign in to add a comment