New issue
Advanced search Search tips

Issue 826214 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Aug 2
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression:Print preview overlay appears misplaced on exiting full screen

Reported by vineetha...@etouch.net, Mar 27 2018

Issue description

Chrome Version: 67.0.3381.0 (Official Build) Revision ae7ce2710f11dd78c2f787b888d2cc19b912bf49-refs/heads/master@{#545918}(64 Bit).
OS: Mac(10.12.6,10.13.1),10.13.4(Macbook Pro Touchbar) 

What steps will reproduce the problem?
(1) Freshly launch chrome, navigate to NTP in browser full screen mode.
(2) Now press cmd + P and click on the browser full screen icon on the top LHS again to exit full screen and observe.

Actual Result: Print preview overlay appears misplaced.
Expected Result: Print preview overlay should not appear misplaced.

This is regression issue broken in ‘M-67’ and providing the bisect using per-revision bisect,
Good build: 67.0.3378.0(Revision: 544932)
Bad build: 67.0.3379.0(Revision: 545319)

You are probably looking for a change made after 545257 (known good), but no later than 545258 (first known bad).

CHANGE-LOG 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/0ace60f0bda35d4788e12314314758465ccf70f5..e09df9251bdc1f717aa76df1629a04143971a15e

Suspect:https://chromium.googlesource.com/chromium/src/+/e09df9251bdc1f717aa76df1629a04143971a15e

@ellyjones: 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. The issue is not seen Windows(7,8,8.1,10) and Linux(14.04).

Thank You!

 
ActualVideo.mov
811 KB View Download
ExpectedVideo.mov
2.6 MB View Download
Note: Issue is also seen on websites like www.youtube.com, www.cnn.com, www.facebook.com etc. Please refer the attached screencast.
ActualVideo.mov
5.7 MB View Download
Labels: ReleaseBlock-Stable
adding RBS, please change if required
It seems like the actual bug here is that the fullscreen button is not exiting fullscreen for you - in your movies the browser window does not exit fullscreen when you click it.

I can't reproduce this with 67.0.3381.0 on my 10.13.4 beta mac desktop. Which specific 10.13.4 beta are you running? I have 17E190a, although I see a new beta update is available.
I tried on 10.13.4 beta 7 as well (17E197a) and I can't repro there either. I'll try to track down a touchbar mac, but in the mean time, can you try this on a non-touchbar mac, and try with today's canary (.3381.0)?
w.r.t Comment #4, Checked the issue with Canary #67.0.3381.0, the issue is reproducible on Mac 10.13.4(17E197a)(touchbar), as well as on non-touchbar machines with Mac(10.12.6 & 10.13.1) OS.

The following are the steps to reproduce the issue,
1.Fresh launch chrome and click on fullscreen button(from the traffic light buttons) for browser.
2.Navigate to NTP or any website like www.youtube.com,www.facebook.com
3.Press CMD+P to open Print preview overlay for webpage.
4.Now click on fullscreen again(to exit the fullscreen mode of browser)
5.Observe Print preview overlay gets misplaced

Thank you
Owner: spqc...@chromium.org
Okay, I reproed locally with .3382.0 on 10.13.4 17E197a.

spqchan, can you take a look at this? The steps in #5 worked for me.
Status: Started (was: Assigned)

Comment 8 by gov...@chromium.org, Apr 25 2018

M67 Stable promotion is coming soon. Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and request a merge into the release branch ASAP. Thank you.


*** Bulk Edit ***
M67 Stable promotion is coming soon. Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and request a merge into the release branch ASAP. 

If fix is already merged to M67 and nothing else is pending, pls mark the bug as fixed. Thank you.
*** Bulk Edit ***
M67 Stable promotion is coming VERY soon. Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and request a merge into the release branch ASAP. 

If fix is already merged to M67 and nothing else is pending, pls mark the bug as fixed. Thank you.
Labels: -M-67 M-68
Labels: -Target-67 Target-68
Labels: -M-68 -Target-68 Target-69 M-69
Owner: ellyjo...@chromium.org
Status: Assigned (was: Started)
Thanks!
Labels: -Pri-1 -ReleaseBlock-Stable Pri-2
MacViews triage: Pri-1 -> Pri-2 & -RBS.
Status: WontFix (was: Assigned)
Mac triage: This works properly for me in 70.0.3510.0 on 10.13.6. WontFix.

Sign in to add a comment