New issue
Advanced search Search tips

Issue 803343 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

[Mac] Permission bubble is not visible completely in presentation mode for RTL.

Reported by avsha...@etouch.net, Jan 18 2018

Issue description

Chrome version : 65.0.3323.0 (Official Build) 470d3a6be2f380affa8de3dc0dd1508f3eeba9f7-refs/heads/master@{#529554} 64 bit
OS : Mac(10.12.6, 10.13.1, 10.13.3)

Pre condition : Enable ‘#force-ui-direction’ and ‘#mac-rtl’ from chrome://flags page.

What steps will reproduce the problem?
1. Launch chrome with above flags enabled and navigate to https://permission.site/
2. Click on ‘Fullscreen’ button on the page (browser enters into presentation mode) and click on ‘Location’ button.
3. Observe the permission bubble. 

Actual Result : Permission bubble is not visible completely in presentation mode for RTL.

Expected Result : Complete permission bubble should be visible to the user in RTL presentation mode.

This is a non-regression issue seen from M-61 series build #61.0.3122.0, as in previous builds RTL flags were not functioning properly. 

Note : Above issue is not observed on Windows(7,8,8,1,10) & Linux(14.04 LTS) OS.
 
Actual_Permission_Bubble.mov
7.6 MB Download
Labels: Target-65 FoundIn-65 Target-64
Status: Untriaged (was: Unconfirmed)
As this being a Non-Regression issue, changing the status to Untriaged so that the issue would get addressed.

Thank You!
Owner: lgrey@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 30 2018

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

commit 5ee30f65b56ddf4e6cb2269c638596bb045f47b3
Author: Leonard Grey <lgrey@chromium.org>
Date: Tue Jan 30 20:57:12 2018

[MacRTL] Position permission bubbles correctly in content fullscreen

Bug:  803343 
Change-Id: Ida606541b5391c38bbe776ef558ae2e342b6ac2b
Reviewed-on: https://chromium-review.googlesource.com/894022
Commit-Queue: Leonard Grey <lgrey@chromium.org>
Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#533019}
[modify] https://crrev.com/5ee30f65b56ddf4e6cb2269c638596bb045f47b3/chrome/browser/ui/cocoa/bubble_anchor_helper.mm

Comment 4 by avsha...@etouch.net, Jan 31 2018

Labels: TE-Verified-M66 TE-Verified-66.0.3335.0
Update : 
Verified this issue on Mac(10.12.6, 10.13.1, 10.13.3) OS with latest Canary build #66.0.3335.0 and the issue is fixed. Kindly review an attached screencast.

Thank you!
Canary_behaviour.mov
4.9 MB View Download
Labels: ET-MUM-Reported

Comment 6 by lgrey@chromium.org, Feb 8 2018

Status: Fixed (was: Assigned)

Sign in to add a comment