Regression: Paper-Icon Button is misplaced on chrome://settings/content/cookie.
Reported by
aiman.an...@etouch.net,
Apr 27 2017
|
||||||
Issue descriptionChrome Version:60.0.3082.0 (Official Build) (cohort: win_58_81)190ee07cecf53e3795197cc195b844f7cc15a9ea-refs/heads/master@{#467534}(32/64 Bit). OS: Windows(7,8,10). Steps to reproduce: 1.Launch Chrome, go to chrome://settings/content/cookies click on ADD button for Block. 2.Add any URL and observe Actual Result: Paper-Icon Button is misplaced. Expected Result: Paper-Icon Button should not be misplaced. This is regression issue broken in 'M 60' and will soon update the bisect info: Manual Bisect Info: Good Build 60.0.3081.0 Bad Build 60.0.3082.0 Note: Will update MAC and LINUX status once build is available
,
Apr 27 2017
Manual Bisect: -------------- Good Build 60.0.3081.0-(Revision-467177) Bad Build 60.0.3082.0- (Revision-467534) Per revision bisect Tool Info: ------------------------------ https://chromium.googlesource.com/chromium/src/+log/efd8268f95f1c7a212e5b6edd521761dd8a00e31..60869ccf490d8a71bdbe64bca909b80b0348d805 Review-Url: https://codereview.chromium.org/2840713002 dpapad@ Kindly take a look and please help us to reassign this issue to a right owner if not with respect to this change. Thanks.!
,
Apr 27 2017
,
Apr 27 2017
,
Apr 28 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d681cc9be589ff887ca6259e1872e36d75f2b932 commit d681cc9be589ff887ca6259e1872e36d75f2b932 Author: dpapad <dpapad@chromium.org> Date: Fri Apr 28 01:59:46 2017 MD Settings: Fix cookie exceptions flex layout regression. site_list.html was relying on flex related CSS classes defined in the deprecated iron-flex-layout.html, but it was not explicitly importing that file. When I migrated iron-flex-layout.html to the new iron-flex-layout-classes.html at crrev.com/2840713002, the UI broke and revealed the missing dependency. BUG= 715903 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2846973002 Cr-Commit-Position: refs/heads/master@{#467827} [modify] https://crrev.com/d681cc9be589ff887ca6259e1872e36d75f2b932/chrome/browser/resources/settings/site_settings/site_list.html
,
Apr 28 2017
,
May 4 2017
Tested the issue on windows 7, Mac 10.12.4, Linux Ubuntu 14.04 using chrome version#60.0.3088.3 with the steps mentioned in comment #0.Observed that the Paper-Icon Button is displayed properly. Hence adding TE-Verified labels. Please find the attached screen cast for the same. Thanks..!! |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by aiman.an...@etouch.net
, Apr 27 2017