New issue
Advanced search Search tips

Issue 877195 link

Starred by 0 users

Issue metadata

Status: Verified
Owner:
Closed: Aug 24
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Incognito contents are not masked properly in app switcher after rotating the device.

Project Member Reported by srikanthg@chromium.org, Aug 23

Issue description

App Version: 69.0.3497.53 beta
iOS Version: 11.4.1
Device: iPads only
URL: na

Steps to reproduce:
  1. Launch Google Chrome
  2. Open Incognito tab and navigate to any webpage
  3. Background the app
  4. Rotate the device
  5. Double tap home button to enter app switcher

Observed results: Incognito window contents are not properly masked

Expected results: Incognito tab contents should be masked properly


Number of times you were able to reproduce: 5/5
Bug reproducible after clean install: Yes
Bug reproducible after clearing cache and cookies: Yes
Bug reproducible on Chrome Mobile on Android: 
Bug reproducible on Safari/Firefox: Firefox: , Safari: 
Bug reproducible on current stable build (App Version, iOS Version): M68 NO
Bug reproducible on the current beta channel build (App Version, iOS Version): M69 Yes

Link to video/image: https://drive.google.com/file/d/1bxzwV06F6dTOeoXJ5ZDVLV7OCKWAwZ9N/view 
 
Cc: rhalavati@chromium.org
Labels: ReleaseBlock-Stable M-69 Hotlist-Privacy
Owner: rohitrao@chromium.org
Status: Assigned (was: Untriaged)
Cc: justincohen@chromium.org rohitrao@chromium.org
Owner: justincohen@chromium.org
Reassigning to Justin.
Status: Started (was: Assigned)
Cc: linds...@chromium.org kariahda@chromium.org
Project Member

Comment 6 by bugdroid1@chromium.org, Aug 24

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

commit 8e9edd108371d008b35f61ee7cac0851b53ef769
Author: Justin Cohen <justincohen@google.com>
Date: Fri Aug 24 18:57:08 2018

[ios] Correct autoresizingMasks in launch screen incognito blocker.

Bug:  877195 
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I6b94acbe327886c90b640b149099db62c17c7719
Reviewed-on: https://chromium-review.googlesource.com/1188609
Commit-Queue: Rohit Rao <rohitrao@chromium.org>
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Reviewed-by: Justin Cohen <justincohen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#585929}
[modify] https://crrev.com/8e9edd108371d008b35f61ee7cac0851b53ef769/ios/chrome/app/application_delegate/app_state.mm

Labels: Merge-Rejected-69
Status: Fixed (was: Started)
Labels: -Merge-Rejected-69 Merge-Request-69
Project Member

Comment 9 by sheriffbot@chromium.org, Aug 24

Labels: -Merge-Request-69 Merge-Review-69 Hotlist-Merge-Review
This bug requires manual review: Less than 7 days to go before AppStore submit on M69
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), kariahda@(iOS), cindyb@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Please explain why this needs to make M69 and how confident you are in the fix.
iPad incognito ui-refresh users who rotate their devices after backgrounding will see a portion of the masked web page in the OS app switcher.
It's an expectation that Incognito content is private to the user. Chrome makes an effort to mask the content from task switcher. I've tried with a few web pages (cnn.com, wikipedia), it mostly shows part of a blank page in the task switcher preview. It would be good to fix since getting privacy-related bugs fixed instills trust in the users.

Looking at the 2-liner fix (comment 6), it is assigning scalar values (mask bits) to a field in ObjC objects (UIView). This cannot do much damage (can't crash) even if the views turned out to be nil. Worst it can do is nothing.
Status: Verified (was: Fixed)
Verified on chrome canary version 70.0.3534.0 on iPad 2018 with iOS 11.4.1, following steps mentioned in comment #0.  Incognito content is completely masked.  Looks good.
Labels: -Hotlist-Merge-Review -Merge-Review-69 Merge-Approved-69
Thank you, vbarigela. Justin please merge today.
Project Member

Comment 15 by bugdroid1@chromium.org, Aug 27

Labels: -merge-approved-69 merge-merged-3497
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/003e88df8d078d219a763ca5fa1e4afcb18f0fd4

commit 003e88df8d078d219a763ca5fa1e4afcb18f0fd4
Author: Justin Cohen <justincohen@google.com>
Date: Mon Aug 27 14:09:56 2018

[ios] Correct autoresizingMasks in launch screen incognito blocker.

Bug:  877195 
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I6b94acbe327886c90b640b149099db62c17c7719
Reviewed-on: https://chromium-review.googlesource.com/1188609
Commit-Queue: Rohit Rao <rohitrao@chromium.org>
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Reviewed-by: Justin Cohen <justincohen@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#585929}(cherry picked from commit 8e9edd108371d008b35f61ee7cac0851b53ef769)
Reviewed-on: https://chromium-review.googlesource.com/1190402
Cr-Commit-Position: refs/branch-heads/3497@{#811}
Cr-Branched-From: 271eaf50594eb818c9295dc78d364aea18c82ea8-refs/heads/master@{#576753}
[modify] https://crrev.com/003e88df8d078d219a763ca5fa1e4afcb18f0fd4/ios/chrome/app/application_delegate/app_state.mm

Comment 16 Deleted

Verified on chrome beta version 69.0.3497.70 on iPad 2018 with iOS 11.4.1 and iPad Air with iOS 12 beta 10, following steps mentioned in comment #0.  Incognito content is completely masked.  Looks good
Verified on iPad Pro 12'9 iOS 11.4.1 on 70.0.3535.0 Canary.
Incognito content is completely masked. Chrome Splash screen is seen.

Sign in to add a comment