New issue
Advanced search Search tips

Issue 892048 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 5
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression:Progress bar hides when mouse is hovered on volume slider in mounirlamouri.github.io .

Reported by shruti.j...@etouch.net, Oct 4

Issue description

Chrome Version: 71.0.3569.0 Revision fb87327b9e0a0a50c96e084943e5525c25f76fd1-refs/branch-heads/3569@{#1}(32/64 bit)
OS : Windows(7, 8, 8.1 ,10), Mac(10.12.6, 10.13.1, 10.14, 10.13.6), Linux(14.04 LTS)

Test URL : 
https://mounirlamouri.github.io/sandbox/media/dynamic-controls.html

What steps will reproduce the problem?
1. Launch chrome and navigate to above URL.
2. Select 'Audio' option and play audio.
3. Hover mouse on 'Volume' icon.
4. Observe the progress bar of audio when mouse is hovered on 'Volume' icon.

Actual Result :Progress bar hides when mouse is hovered on volume slider.
Expected Result : Progress bar should not hide when mouse is hovered on volume slider.

This is a regression issue broken in ‘M-71’ and below is the 'Chromium bisect' information:
Good Build : 71.0.3562.0 (Revision : 594164)
Bad Build : 71.0.3563.0 (Revision : 594642)

Chromium bisect URL:
https://chromium.googlesource.com/chromium/src/+log/65d577fb9759e31673e19d76db8b4a4e447dab39..cd031b000616f1d080e646a6fd545c03d94a369c

Suspect: r594612

steimel@ : 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.

Note:

1.Unable to provide 'per-revision' bisect as it shows "We don't have enough builds to bisect" error message.
2.Tried on other machines but still getting the same error again.
3.Hence provided suspect through 'Chromium bisect'
4.Pardon me if it is an intended change but,visually differentiating between Volume slider and progress bar is not easy hence when mouse is hovered on volume slider , progress bar gets hide.
5.Issue is also seen on Latest Canary #71.0.3570.0

Thank you..!
 
Expected_result.mp4
229 KB View Download
Actual_Result.mp4
471 KB View Download
Summary: Regression:Progress bar hides when mouse is hovered on volume slider in mounirlamouri.github.io . (was: Regression:Progress bar jumps back at it's initial position due to overlapping of volume slider with progress bar in mounirlamouri.github.io .)
Description: Show this description
Project Member

Comment 3 by bugdroid1@chromium.org, Oct 5

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

commit ab76b11dae74e3e7739e5d8b15088909c4b357cb
Author: Tommy Steimel <steimel@chromium.org>
Date: Fri Oct 05 19:34:23 2018

[Media Controls] Use shorter volume slider length

This CL updates the volume slider length to be shorter in order to fix
an issue where the slider is too long for the default audio controls
length.

Bug:  892048 
Change-Id: I545fd2395ec277125f7c6edbfbdc56aa03b2580f
Reviewed-on: https://chromium-review.googlesource.com/c/1265057
Reviewed-by: Becca Hughes <beccahughes@chromium.org>
Commit-Queue: Tommy Steimel <steimel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597244}
[modify] https://crrev.com/ab76b11dae74e3e7739e5d8b15088909c4b357cb/third_party/blink/renderer/modules/media_controls/resources/modernMediaControls.css

Status: Fixed (was: Assigned)

Sign in to add a comment