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

Issue 781131 link

Starred by 1 user
Status: Duplicate
Merged: issue 700196
Owner:
tapted@chromium.org
Closed: Nov 2017
Cc:
bsep@chromium.org
nyerramilli@chromium.org
rbasuvula@chromium.org
ellyjo...@chromium.org
█ msrchandra@chromium.org
█ ranjitkan@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Extra padding is observed between bubble header and body.

Reported by shruti.j...@etouch.net, Nov 3 2017 
Chrome Version: 64.0.3257.0 314f01c2122124dafaf99698f5049ee7d1c08140-refs/heads/master@{#513642} (64-bit)

OS: Win(7,8,10) Linux(14.04 LTS).

Pre-condition-Enable "Material Design in the rest of the browser's native UI".

Test URL: permission.site

Steps to reproduce:
1.Launch chrome and navigate to chrome://flags and enable "Material Design in the rest of the browser's native UI"
2.Visit the above URL  click on any button example 'Notification','Location'  seen on the page and click on 'Block' in permission bubble.
3.Click on blocked 'Notification','Location' icon seen at the RHS of omnibox and observe the bubble.

Actual Result : Extra padding is observed between bubble header and body.

Expected Result : Padding between bubble header and body should be proper.



This is regression issue broken in ‘M-64’ and will soon provide the bisect info: 

Using the per-revision bisect providing the bisect results,
Good Build: 64.0.3247.0
Bad Build:  64.0.3248.0

Comment 1 by shruti.j...@etouch.net, Nov 3 2017

Labels: hasbisect-per-revision
Owner: h...@chromium.org
Status: Assigned (was: Untriaged)
Kindly refer the Bisect-info:

Good build:64.0.3247.0(Revision:510692)
Bad build: 64.0.3248.0(Revision:510988)

You are probably looking for a change made after 510841 (known good), but no later than 510842 (first known bad).
CHANGELOG URL:
The script might not always return single CL as suspect as some perf builds might get missing due to failure.
  https://chromium.googlesource.com/chromium/src/+log/3ebe7b184b8b60fb7fdf57fea1b4868f7676614e..7378f29a80837745d43076cb5b02a13587a72f30

Suspect:https://chromium.googlesource.com/chromium/src/+/7378f29a80837745d43076cb5b02a13587a72f30

@Hans Wennborg :Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.

Thank You!

Comment 2 by shruti.j...@etouch.net, Nov 3 2017

actual.mp4
816 KB View Download
actual.png
82.6 KB View Download
expected.mp4
727 KB View Download
expected1.png
83.4 KB View Download

Comment 3 by h...@chromium.org, Nov 3 2017

Owner: ----
Status: Unconfirmed (was: Assigned)
I'm pretty sure my change is unrelated (it only touched the code in an error-reporting function in Expat). Perhaps try bisecting again?

Comment 4 by ligim...@chromium.org, Nov 3 2017

Labels: -hasbisect-per-revision Needs-Bisect

Comment 5 by shruti.j...@etouch.net, Nov 7 2017

Labels: -Needs-Bisect hasbisect-per-revision
Owner: tapted@chromium.org
Status: Assigned (was: Unconfirmed)
With respect to comment#4 :

Kindly refer the updated Bisect-info:
You are probably looking for a change made after 510763 (known good), but no later than 510764 (first known bad).
CHANGELOG URL:
The script might not always return single CL as suspect as some perf builds might get missing due to failure.
  https://chromium.googlesource.com/chromium/src/+log/31d19047cf7d866cba026088f91a30ad1467cc37..9efc45358e4a44c4e4b096294e9278b786578353

Suspect:https://chromium.googlesource.com/chromium/src/+/9efc45358e4a44c4e4b096294e9278b786578353

@tapted:Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.

Thank you!

Comment 6 by tapted@chromium.org, Nov 15 2017

Cc: bsep@chromium.org ellyjo...@chromium.org
Labels: Proj-HarmonyDialogs
Mergedinto: 700196
Status: Duplicate (was: Assigned)
I think elly's working on this in  Issue 700196
►

Sign in to add a comment