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

Issue 618814 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Bug



Sign in to add a comment

Trying to open encryption settings when Sync is OFF is crashing the Chrome app.

Project Member Reported by jnaveen@chromium.org, Jun 9 2016

Issue description

ENVIRONMENT and STATS
Chrome version: 52.0.2743.33
Device: Ipad mini 9.3.1

REPRO STEPS
1. Sign in to Chrome.
2. Open Sync settings
3. Turn sync OFF (disable the slider button)
4. Click on "Encryption"

ACTUAL RESULTS
Chrome app is crashed

EXPECTED RESULTS
App is not crashed and nothing should happen as the encryption setting is disabled.



 
Cc: lpromero@chromium.org
Owner: bzanotti@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 2 by sheriffbot@chromium.org, Jun 10 2016

Labels: -M-52 M-53 MovedFrom-52
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -M-53 M-52
Labels: ReleaseBlock-Stable
Status: Started (was: Assigned)
https://chromereviews.googleplex.com/448937013/
Project Member

Comment 5 by bugdroid1@chromium.org, Jun 13 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/c675ef83c8dd8f3a2ccce341e716e689f13f0d90

commit c675ef83c8dd8f3a2ccce341e716e689f13f0d90
Author: bzanotti <bzanotti@google.com>
Date: Mon Jun 13 11:47:39 2016

Labels: Merge-Request-52
Status: Fixed (was: Started)

Comment 7 by tin...@google.com, Jun 13 2016

Labels: -Merge-Request-52 Merge-Approved-52 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M52 (branch: 2743)

Comment 8 by plaree@chromium.org, Jun 13 2016

Cc: jnaveen@chromium.org plaree@chromium.org
Project Member

Comment 9 by sheriffbot@chromium.org, Jun 16 2016

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 10 by bugdroid1@chromium.org, Jun 16 2016

Labels: -merge-approved-52 Merge-Merged-2743
The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/ac972a16578408e09d3bcbb68354960097c04c4f

commit ac972a16578408e09d3bcbb68354960097c04c4f
Author: Benoit Zanotti <bzanotti@google.com>
Date: Thu Jun 16 17:09:41 2016

Status: Verified (was: Fixed)
Verified on iPhone 6SPlus( 9.2.1), iPad Air(9.3.1) on 53.0.2774.0 Canary. No crashes seen upon tapping on Encryption when sync is OFF.
Verified on iPhone 6S ( 9.2.1), iPad Air(9.3.1) on 52.0.2743.50 dev. No crashes seen upon tapping on Encryption when sync is OFF.

Sign in to add a comment