Issue metadata
Sign in to add a comment
|
Regression : 'Cookies set by this page' text appears weird in 'view site information' overlay.
Reported by
mni...@etouch.net,
Mar 30 2016
|
||||||||||||||||||||||
Issue descriptionChrome Version : 50.0.2661.57 31761a6af4781d5cfd8ab5673f485a256dd5fe12-refs/branch-heads/2661@{#425} (32/64-bit) OS : Windows(Win 7 aero enabled). Url : http://www.pepsicoindia.co.in/ What steps will reproduce the problem? 1. Launch chrome and navigate to above url. 2. Now click on 'view site information' icon in 'address bar' and click on cookies link and overlay will open. 3. Now 'Minimize' the browser window and again open it and observe the text 'Cookies set by this page' on top of the overlay Actual: 'Cookies set by this page' text appears weird after minimizing the browser window and again opening it. Expected: 'Cookies set by this page' text should be seen properly. This is a regression issue broken in 'M-50' and below is the manual regression and will soon update other info : Good Build: 50.0.2640.0 Bad Build: 50.0.2641.0
,
Mar 30 2016
,
Mar 30 2016
Unable to reproduce the issue on latest Chrome beta 50.0.2661.57, even the video which was provided looks ok to me. mnikam@ please check this again and let
,
Mar 31 2016
With response to comment #3 : Able to reproduce the above issue on latest Chrome beta 50.0.2661.57.Please refer the attached video. Actual result : Text appears to be chopped.(i.e 'Cookies' appears as 'Co','this' is seen as 'thi').Please refer the attached screenshot.
,
Apr 1 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/63302b696824ab46a212de5bc1a1cf80b678a88a commit 63302b696824ab46a212de5bc1a1cf80b678a88a Author: estade <estade@chromium.org> Date: Fri Apr 01 23:49:32 2016 Handle bubble title resizing (growth) by reworking title layout The easiest way to demonstrate the bug is to shrink the browser window horizontally until the title starts to wrap, then drag it larger again. Without this patch, the title doesn't unwrap. I also noticed that one of the labels was jiggling in the collected cookies view during resize; left-aligning the text fixes that. BUG= 598985 TBR=sky@chromium.org Review URL: https://codereview.chromium.org/1849703004 Cr-Commit-Position: refs/heads/master@{#384722} [modify] https://crrev.com/63302b696824ab46a212de5bc1a1cf80b678a88a/chrome/browser/ui/views/collected_cookies_views.cc [modify] https://crrev.com/63302b696824ab46a212de5bc1a1cf80b678a88a/ui/views/bubble/bubble_frame_view.cc [modify] https://crrev.com/63302b696824ab46a212de5bc1a1cf80b678a88a/ui/views/window/dialog_delegate_unittest.cc
,
Apr 4 2016
,
Apr 4 2016
Your change meets the bar and is auto-approved for M50 (branch: 2661)
,
Apr 4 2016
Please merge your change to M50 branch 2661 by 5:00 PM PST today, 04/04/16 so we can take it for this week beta. Thank you.
,
Apr 4 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5d9c9aa306d3ba4e7feeeee8fabbca79034d00ac commit 5d9c9aa306d3ba4e7feeeee8fabbca79034d00ac Author: Evan Stade <estade@chromium.org> Date: Mon Apr 04 22:16:34 2016 Handle bubble title resizing (growth) by reworking title layout The easiest way to demonstrate the bug is to shrink the browser window horizontally until the title starts to wrap, then drag it larger again. Without this patch, the title doesn't unwrap. I also noticed that one of the labels was jiggling in the collected cookies view during resize; left-aligning the text fixes that. BUG= 598985 TBR=sky@chromium.org Review URL: https://codereview.chromium.org/1849703004 Cr-Commit-Position: refs/heads/master@{#384722} (cherry picked from commit 63302b696824ab46a212de5bc1a1cf80b678a88a) Review URL: https://codereview.chromium.org/1860723003 . Cr-Commit-Position: refs/branch-heads/2661@{#484} Cr-Branched-From: ef6f6ae5e4c96622286b563658d5cd62a6cf1197-refs/heads/master@{#378081} [modify] https://crrev.com/5d9c9aa306d3ba4e7feeeee8fabbca79034d00ac/chrome/browser/ui/views/collected_cookies_views.cc [modify] https://crrev.com/5d9c9aa306d3ba4e7feeeee8fabbca79034d00ac/ui/views/bubble/bubble_frame_view.cc [modify] https://crrev.com/5d9c9aa306d3ba4e7feeeee8fabbca79034d00ac/ui/views/window/dialog_delegate_unittest.cc
,
Apr 5 2016
,
Apr 6 2016
Tested the issue on windows 7 using chrome version 50.0.2661.66.Observed "Cookies set by this page" text rendering is fine. Please find the attached screen cast for the same. Adding TE-Verified label. Thanks, |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by mni...@etouch.net
, Mar 30 2016Owner: est...@chromium.org
Status: Assigned (was: Unconfirmed)