New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 732062 link

Starred by 5 users

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

[Canary] bookmarks bar icons are cut off

Reported by wolfw...@gmail.com, Jun 10 2017

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/61.0.3124.10 Safari/537.36

Steps to reproduce the problem:
 In Canary, bookmarks bar icons are cut off by 1px at the bottom. They are actually positioned 1px lover than usual, and because of that bottom is cut off

What is the expected behavior?

What went wrong?
-

Did this work before? N/A 

Chrome version: 61.0.3124.10  Channel: dev
OS Version: 10.0
Flash Version:
 
fdfdfdf.png
20.0 KB View Download

Comment 1 by wolfw...@gmail.com, Jun 10 2017

Image shows Chrome stable (top) and Canary (bottom) bookmarks bar

Comment 2 by wolfw...@gmail.com, Jun 10 2017

*loWer :)

Labels: Needs-Bisect Needs-Triage-M61
Cc: kkaluri@chromium.org
Labels: -Type-Bug -Pri-2 -Needs-Bisect hasbisect-per-revision M-61 Pri-1 Type-Bug-Regression
Owner: yiyix@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce this issue on Windows 10 with chrome Stable #59.0.3071.86 and also in canary #61.0.3127.0
Issue broken in M61

Bisect Info:
===========
Good build : 61.0.3117.0  ,  Revision Range - 476123
Bad build  : 61.0.3119.0  ,  Revision Range - 476838

After executing the per-revision bisect script, i got the following CL's between good and bad build versions
===========================================
https://chromium.googlesource.com/chromium/src/+log/9939f3a8acf8565958bfc50d01ff7c0f7bd4ebe5..7073ae5bc6af913c649500e015210992f9c7111a

The suspecting Change Log is :
-----------
Review-Url: https://codereview.chromium.org/2899133004

yiyix@- Could you please look into this issue, if it's related to your change?  if not could you please help us to reassign this issue to the right owner.

Note: Issue not reproducible in Mac 10.12.5 and Ubuntu 14.04
 
*** Corrections in Comment #4 ***
 
Issue is only reproducible on Dev #61.0.3124.10 and Canary #61.0.3127.0 but not on Stable #59.0.3071.86 and Beta #60.0.3112.24

Comment 6 by yiyix@chromium.org, Jun 12 2017

Cc: msrchandra@chromium.org yiyix@chromium.org ranjitkan@chromium.org rbasuvula@chromium.org nyerramilli@chromium.org
 Issue 730570  has been merged into this issue.

Comment 7 by yiyix@chromium.org, Jun 12 2017

Status: Started (was: Assigned)
Project Member

Comment 8 by bugdroid1@chromium.org, Jun 14 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2f4c74729badb56a8a0d4c4cd18fea329f9fa552

commit 2f4c74729badb56a8a0d4c4cd18fea329f9fa552
Author: yiyix <yiyix@chromium.org>
Date: Wed Jun 14 20:09:57 2017

Fix: Icons are chopped from bottom on bookmark bar

In cl "reducing overdraw on bookmark bar", https://codereview.chromium.org/2899133004, 
the height of bookmark bar is reduced by 1 px for both detached mode and attached mode 
after removing the 1 px overlap between toolbar and bookmark bar. However, the tool 
bar and bookmark bar only overlaps when bookmark bar is in detached mode. The height 
of bookmark bar in attached mode should remain unchanged. 

TEST=BrowserView.BrowserViewLayout

Bug:  732062 
Change-Id: Id363f9cf9d3ef5a606eb3f94723b28c5b1db09cb
Reviewed-on: https://chromium-review.googlesource.com/532055
Commit-Queue: Yi Xu <yiyix@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#479482}
[modify] https://crrev.com/2f4c74729badb56a8a0d4c4cd18fea329f9fa552/chrome/browser/ui/bookmarks/bookmark_bar_constants.h
[modify] https://crrev.com/2f4c74729badb56a8a0d4c4cd18fea329f9fa552/chrome/browser/ui/views/frame/browser_view_unittest.cc

Comment 9 by yiyix@chromium.org, Jun 19 2017

Status: Fixed (was: Started)

Sign in to add a comment