New issue
Advanced search Search tips

Issue 662989 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Bug



Sign in to add a comment

QR Scanner does not work on first launch

Project Member Reported by jasonkliu@chromium.org, Nov 7 2016

Issue description

App Version (from "Chrome Settings > About Chrome"): 55.0.2883.33 beta, 56.0.2904.0 canary
iOS Version, Device: 
Can repro on 10.2 beta/6S, 10.1/iPad 4th gen, 10.0/iPhone 5, 10.1.1/iPhone 5SE.  Originally reported on 10.0.2/5C.

Steps to reproduce: 
1. Download and install the app
2. Launch QR via spotlight or 3D Touch

Observed behavior: 
Cannot scan codes on paper or on a screen

Expected behavior: 
Should be able to scan codes.
If you close the app via home button (force close NOT necessary) and then immediately relaunch, the scanner works again.

Frequency: 5/5
<number of times you were able to reproduce> 
 
Cc: linds...@chromium.org srikanthg@chromium.org

Comment 2 by jif@chromium.org, Nov 9 2016

Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Nov 14 2016

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

commit be62094eaacfdfebd4749ca55e013c9e657b4f3e
Author: jif <jif@google.com>
Date: Mon Nov 14 13:13:48 2016

Comment 4 by jif@chromium.org, Nov 14 2016

Status: Fixed (was: Started)
Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-55; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-55 label, otherwise remove Merge-TBD label. Thanks.
Project Member

Comment 6 by bugdroid1@chromium.org, Nov 14 2016

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

commit 452a4eed63005dc135f22548f4832c897fa9dea5
Author: lpromero <lpromero@google.com>
Date: Mon Nov 14 14:30:24 2016

Project Member

Comment 7 by bugdroid1@chromium.org, Nov 15 2016

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

commit 1dadaf4d29964b7d0278aedf37ea86059df92a10
Author: jif <jif@google.com>
Date: Tue Nov 15 18:24:22 2016

Comment 8 by jif@chromium.org, Nov 16 2016

Owner: srikanthg@chromium.org
Status: Assigned (was: Fixed)
Hi,
can you verify that this is fixed in the November 16th canary?

Comment 9 by cma...@chromium.org, Nov 16 2016

Labels: -M-55 -Merge-TBD -ReleaseBlock-Stable M-56
Adding the right Milestone to this bug
Labels: ReleaseBlock-Stable
Status: Verified (was: Assigned)
QR Scanner works fine after Fresh install and launching for the first time.
Verified on iPhone5c, iPhone6s, iPhone7, iPad Pro
iOS: 9.3.5, 10.1, 10.2 beta
Build: M56.0.2923.0 canary
Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-56; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-56 label, otherwise remove Merge-TBD label. Thanks.
'Scan QR Code' works fine when launching via Spotlight Search or 3D Touch after fresh install & launching the app.
Verified on iPhone 7(10.1.1), iPad Air 2(10.1.1) in 57.0.2927.0 canary 

Project Member

Comment 14 by sheriffbot@chromium.org, Dec 16 2016

Labels: Merge-Review-56 Hotlist-Merge-Review
This bug requires manual review: No test file found in commits.
Please contact the milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), gkihumba@(cros), bustamante@(desktop)

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

Comment 15 by sheriffbot@chromium.org, Dec 16 2016

This bug requires manual review: No test file found in commits.
Please contact the milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), gkihumba@(cros), bustamante@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-TBD -Merge-Review-56 Merge-Approved-56
Labels: -Hotlist-Merge-Review
Please Srikanth, can you check again if this fix is in M56?
Fix is in M56.
Verified on M56.0.2924.28 beta. iPhone6 plus, iOS10.1.1
QR Scanner works fine after fresh installation and Scanner launched via Spotlight.
Project Member

Comment 20 by sheriffbot@chromium.org, Dec 20 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 21 by sheriffbot@chromium.org, Dec 26 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
Labels: Merged
Jif is the merge complete here? 
Not sure why this isn't marked as merged given the above verification...
Components: UI>Browser>Mobile>QRScanner

Sign in to add a comment