New issue
Advanced search Search tips

Issue 678751 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Android Webview & chrome 55..2883.91 display ebooks incorrectly on tablet

Reported by tnguyenm...@gmail.com, Jan 5 2017

Issue description

Steps to reproduce the problem:
1. download myon app 1.2.1 or 1.2.2 on google play store to tablet Android 7.x.x or Android 6.x.x
2. click on the demobook to open webview reading demo ebook without log in

What is the expected behavior?
The webview/chrome  display partly the page.

What went wrong?
The ebook doesn't display the whole page like before on tablet Android 7.x.x with chrome or 6.x.x with Android System Webview build 55.2883.91

Other ebooks got the same problem

Did this work before? Yes 54.0.2840.85

Does this work in other browsers? N/A

Chrome version: 55.0.2883.91  Channel: stable
OS Version: 7.1.1
Flash Version: Shockwave Flash 24.0 r0
 
Status: Available (was: Unconfirmed)
Looks like regression on M55, works fine on M54/54.0.2840.85

55.0.2861.0 - good
55.0.2862.0 - bad

https://chromium.googlesource.com/chromium/src/+log/55.0.2861.0..55.0.2862.0?pretty=fuller&n=10000
Components: Mobile>WebView
Owner: wangxianzhu@chromium.org
There are a lot of SVG related changes, but this one is suspicious 

https://codereview.chromium.org/2343673003

Assigned to the owner
Cc: wangxianzhu@chromium.org
Components: Internals>Skia
Labels: Needs-Bisect
Owner: ----
This should have nothing to do with SVG and blink. I believe this is caused by some skia changes. A finer grain bisect would be helpful.

tnguyenmyon@gmail.com/michaelbai@chromium.org can you upload expected and actual screen shots? I'm not sure what the problem looks like.
Status: Untriaged (was: Available)
Hi, It should display the whole picture, but display only the upleft part , see attachments
Thanks
Screenshot_2017-01-06-14-22-51.png
3.4 MB View Download
Screenshot_2017-01-06-14-32-02.png
856 KB View Download
Components: -Internals>Skia
Thanks for the screen shots. Removing Internal>Skia. Will bisect with local builds. 
Sorry, never mind #4 about SVG/blink. I mixed this bug with another one.

Sorry about not providing detail information, from dev-tools, the application uses SVG, that is why I thought it might relates to SVG viewport. 
Components: -Mobile>WebView Blink>SVG
Labels: -OS-Android -Needs-Bisect OS-All
Owner: f...@opera.com
Status: Assigned (was: Untriaged)
Reduced test case:

<svg viewBox="513 54 414 594" style="width: 768px; height: 592px; clip-path: url('#clip'); position: absolute">
  <defs>
    <clipPath id="clip">
      <rect x="513px" y="54px" width="414px" height="594px"></rect>
    </clipPath>
  </defs>
  <rect x="513px" y="54px" width="414px" height="594px" fill="yellow"></rect>
</svg>

Can reproduce on Linux.

Bisected to https://chromium.googlesource.com/chromium/src/+log/440493088d060522b1d1a35346edec1dba12fac8..9e619beca68cf73772ae4c9f35a21f9bb13d7e65.

It's because we unprefixed -webkit-clip-path. Before that, clip-path was unsupported and was ignored, and now we support it.

fs@opera.com please verify if the behavior is expected. It seems that the author expects the clip is applied in contents coordinates of the svg.

For the app to fix the issue, just remove the 'clip-path' declaration. 

svgclip1.html
365 bytes View Download

Comment 11 by f...@opera.com, Jan 7 2017

Yes, the behavior appears to be what's expected. I believe that previous to that commit, we would still apply the clip-path, but in the coordinate space defined by the viewBox, while now we apply it in the coordinate space before applying the viewBox.

Comment 12 by f...@opera.com, Jan 11 2017

Status: WontFix (was: Assigned)
Thanks, the solution of removing clip-path working. Can I know why it is "WontFix"? Just out of my curiosity

Comment 14 by f...@opera.com, Jan 12 2017

WontFix is the status for "works as intended".

Sign in to add a comment