Unable to zoomin webpages in “books.google.com” |
||||||||
Issue descriptionApp Version : 67.0.3396.17 beta iOS Version: 10.3.3,11.3 Device: iPhone,iPad Steps to reproduce: 1. Launch chrome 2. Goto “books.google.com” 3. Search any book in Google book search page. 4. Tap on any book. 5. Try to zoomin the webpage Observed results: Unable to zoomin the page Note : works fine when web pages are in desktop site. Expected results: Books.google.com pages should be zoomed in Number of times you were able to reproduce: 5/5 Bug reproducible after clean install: Yes Bug reproducible after clearing cache and cookies: Yes Bug reproducible on Chrome Mobile on Android: NA Bug reproducible on Dolphin/Safari/Firefox: Safari : Works fine in safari Firefox : Yes Bug reproducible on the current beta channel build : Yes in M66 Bug reproducible on the current beta channel build : Yes in M67 Video : Chrome : https://drive.google.com/file/d/12iD9sF8THo3NZgbQTDyabaaSOtBnGO_Q/view?usp=sharing Safari https://drive.google.com/file/d/1mJ5dIGjei1R-z1hxsNwjTyeVcE-_zeL6/view?usp=sharing
,
Apr 25 2018
Kurt, I think you tried to fix zooming bug on books.google.com. Do you remember details? Pramod, is this reproducible with Safari User Agent?
,
Apr 25 2018
I just set a breakpoint in the viewport extraction completion block for |-applyPageDisplayState:|, and it looks like the minimum and maximum zoom scale for the books.google.com SRP are both 1.0. I'm having trouble hooking up the safari WKWebView debugger to the simulator right now, so I'm not sure what the actual HTML is that the page is using to define the viewport meta tag, but it seems likely that it's being defined in a way that's prohibiting zooming. Note that the user-scalable bool is also true in the reported web::PageViewportState: (const web::PageViewportState) $0 = { viewport_tag_present_ = true user_scalable_ = true width_ = (use_device_length_ = true, value_ = NaN) height_ = (use_device_length_ = false, value_ = NaN) minimum_zoom_scale_ = 1 maximum_zoom_scale_ = 1 initial_zoom_scale_ = 1 }
,
Apr 25 2018
Thanks Kurt. I vaguely remember that you looked at books.google.com long time ago and seen the same max zoom scale value. Pramod, would you mind filing a bug against books.google.com? Checking with Safari User Agent is still useful.
,
Apr 25 2018
Sorry, I don't remember much from any previous investigation; just going off of the viewport data scraped from the page.
,
May 10 2018
Assigning to Pramod to file a bug against books.google.com.
,
May 18 2018
As per comment#6 filed bug in buganiser : http://b/79972863 Marking Bug as External Dependency
,
Oct 19
b/79972863 is still closed. Pramod, would you mind testing this with ignores-viewport-scale-limits flag enabled?
,
Oct 26
,
Oct 26
,
Oct 26
,
Oct 26
Pramod, would you mind testing this with ignores-viewport-scale-limits flag enabled? |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by sczs@chromium.org
, Apr 25 2018Owner: eugene...@chromium.org
Status: Assigned (was: Untriaged)