Regression: Print preview appears weird for chrome://history.
Reported by
jshan...@etouch.net,
Oct 25 2016
|
||||||
Issue descriptionChrome Version: 56.0.2900.0 (Official Build) 1e4fbec3b5701d9f605e88f14dafa442be98b648-refs/heads/master@{#427219}-32/64 bit OS: Windows(7,8,10), Mac(10.10.5, 10.11.4, 10.11.5) Steps: 1. Launch Chrome and navigate to chrome://history. 2. Give Print command and observe. Actual: Print preview appears weird. i.e page is not seen properly in print preview. Expected: Print preview should appear properly for chrome://history. This is a regression issue broken in ‘M-56’, below is the Manual Regression range and will soon update bisect info. Good build : 56.0.2899.0 Bad build : 56.0.2900.0 Note: will update Linux info later
,
Oct 25 2016
Note: Above issue is reproducible on Linux OS and also observed on chrome://downloads and chrome://md-settings page.
,
Oct 25 2016
Bisect Info from issue 659013 ============================= Able to reproduce the issue on the latest 56.0.2900.0 on Mac OS 10.11.6 as well. https://chromium.googlesource.com/chromium/src/+log/67e6efd22bbef2baea0ac32fb603659f2ee93275..0c0e20baae5cfc32465e84d0e56a82ab874788e9 thestig@: Could you please take a look at this.
,
Oct 25 2016
Issue 659013 has been merged into this issue.
,
Oct 25 2016
Thanks. I'll revert.
,
Oct 25 2016
,
Oct 25 2016
,
Oct 25 2016
Note: Above issue is reproducible on win pgo build version 56.0.2900.0 (32/64 bit) and also seen on following urls: https://in.bookmyshow.com/ http://www.xda-developers.com/ http://www.espn.in/?src=com https://www.flipkart.com/ https://www.snapdeal.com/ http://gionee.co.in/smartphones/s-series/gionee-s6s? https://hangouts.google.com/ https://www.google.com/intl/en/drive/ https://plus.google.com/collections/featured https://www.google.com/photos/about/ https://www.blogger.com/about/?r=1-null_user
,
Oct 26 2016
FWIW, the actual bad commit is: https://github.com/googlei18n/sfntly/pull/60/commits/083b02b10572142d9863d945a8cf52fed2df997d
,
Oct 26 2016
tested the issue on Mac 10.11.6 using #56.0.2901.0, print preview is working fine. tested the below urls: chrome://history https://in.bookmyshow.com/ https://www.google.com/intl/en/drive/ https://plus.google.com/collections/featured https://www.google.com/photos/about/
,
Oct 26 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7ac63a352e7556a68224284af29db855b400a679 commit 7ac63a352e7556a68224284af29db855b400a679 Author: thestig <thestig@chromium.org> Date: Wed Oct 26 21:42:15 2016 Roll DEPS for sfntly 1ef790a..6e98497 6e98497 Merge pull request #61 from leizleiz/leizleiz-tablefix ebaa364 Fix breakage from commit 083b02b1. e33ba7a Merge pull request #60 from leizleiz/leizleiz-crbug 1bc53e1 Fix undefined shifts in ReadableFontData::ReadShort. d651349 Check for negative size in NameTable::NameAsBytes. 8475d2f Avoid NULL derefs inside FontHeaderTable::Builder. 1fba3b3 Fix undefined shifts in ReadableFontData::ReadLong. 083b02b Fix NULL pointer derefs in sfntly::Font::Builder. 6d1efaa Fix out of bound access in subtly sample program. cafc4c8 Merge pull request #59 from HalCanary/pronounciation 7d5169e README: pronounciation guide BUG= 638573 , 641452 , 646300 , 646347 ,654663, 655914 , 659006 TBR=behdad@chromium.org Review-Url: https://codereview.chromium.org/2452873003 Cr-Commit-Position: refs/heads/master@{#427819} [modify] https://crrev.com/7ac63a352e7556a68224284af29db855b400a679/DEPS
,
Oct 27 2016
Tested the issue on Windows 10 , Ubuntu 14.04 and Mac10.12 using chrome Dev version #56.0.2902.0 Observed that fix is working as expected. Attached screencast for reference. Adding the TE-Verified labels. |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by thestig@chromium.org
, Oct 25 2016