Closed Bug 1173625 Opened 10 years ago Closed 10 years ago

Audit use of Intent.parseUri

Categories

(Firefox for Android Graveyard :: General, defect)

Product:
Firefox for Android Graveyard
Component:
General
Platform:
All
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED DUPLICATE of bug 1182140

People

(Reporter: mcomella, Assigned: mcomella)

References

Details

It turns out the flags to Intent.parseUri (e.g. INTENT_SCHEME_URI) are just validators, according to the source [1]. Therefore, we're probably not parsing as correctly as we can in GeckoAppShell.getOpenURIIntent, though I don't think it hurts functionality. Additionally, I think Intent.parseUri can handle custom schemes so I wonder why we seem to be parsing and setting values on URIs ourselves in GeckoAppShell.getOpenURIIntent. [1]: http://androidxref.com/5.1.0_r1/xref/frameworks/base/core/java/android/content/Intent.java#4310
This will be fully handled with bug 1182140.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
And I think this should already be fixed on 41.
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.