Closed Bug 1182220 Opened 4 years ago Closed 4 years ago

QR Code reader should only support ZXing barcode scanner

Categories

(Firefox for Android :: General, defect)

All
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 42
Tracking Status
firefox42 --- fixed
fennec 42+ ---

People

(Reporter: kbrosnan, Assigned: karim)

References

Details

Attachments

(1 file)

Google Goggles registers as a QR reader. It has several wordy dialog on first run. It parses the QR code internally and then fires off an intent based off the QR code. See video. 

https://www.youtube.com/watch?v=TcRrQHIDQkg
Blocks: 1178503
Karim - Can we add a blocklist to Fennec that ignores Google Goggles? Kevin said that all the other QR readers he tested worked fine.
Assignee: nobody → kbenhmida
tracking-fennec: ? → 42+
Summary: The Google Goggles QR reader experience is poor → QR Code reader should only support ZXing barcode scanner
Bug 1182220 - QR Code reader should only support ZXing barcode scanner. r=liuche
Attachment #8636805 - Flags: review?(liuche)
Limiting to ZXing sounds reasonable to me. That was the initial goal anyway. We can always add more QR Code readers in the future.
Comment on attachment 8636805 [details]
MozReview Request: Bug 1182220 - QR Code reader should only support ZXing barcode scanner. r=liuche

https://reviewboard.mozilla.org/r/13713/#review12449

Ship It!
Attachment #8636805 - Flags: review?(liuche) → review+
Duplicate of this bug: 1181866
Comment on attachment 8636805 [details]
MozReview Request: Bug 1182220 - QR Code reader should only support ZXing barcode scanner. r=liuche

Bug 1182220 - QR Code reader should only support ZXing barcode scanner. r=liuche
Attachment #8636805 - Flags: review+ → review?(liuche)
Attachment #8636805 - Flags: review?(liuche) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/3fd7798e4bf1
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 42
Duplicate of this bug: 1182138
Karim, if I understand it right from bug 602818, QR code scanning is new to Firefox41 on Android. Do you think we should uplift this fix to Aurora? Thanks!
Flags: needinfo?(kbenhmida)
Ritu - I think Product Management is fine with letting this ship in Fx42. No need to uplift.

Clearing NI
Flags: needinfo?(kbenhmida)
You need to log in before you can comment on or make changes to this bug.