Closed Bug 736995 Opened 12 years ago Closed 12 years ago

Sharing option is active when attempting to share files inside omni.ja! archives

Categories

(Firefox for Android Graveyard :: General, defect)

14 Branch
ARM
Android
defect
Not set
normal

Tracking

(firefox13 verified, firefox14 verified)

VERIFIED FIXED
Firefox 14
Tracking Status
firefox13 --- verified
firefox14 --- verified

People

(Reporter: AdrianT, Assigned: mbrubeck)

Details

Attachments

(2 files)

Fennec/14.0a1 2012-03-18
Device: HTC Desire (Android 2.2)

Steps to reproduce:
1. Open Nightly.
2. Load resource: or any subdirectory.
3. Open the System Menu and tap on the share button

Expected results:
The share option is grayed out for unsupported protocols like about:,file:, or resource:

Actual results:
The button is active and can be used to share the link.
bug 703630 special cased different URI schemes. According to the screenshot, you're actually attempting to share local resources found in the Android package omni.ja! archives. That is beyond edge case. This is likely WONT FIX.
Summary: Sharing option is active for the resource protocol → Sharing option is active when attempting to share files inside omni.ja! archives
The screenshot is the result of accessing "resources:". The behavior is the same accessing subdirectories using "resources:///res/" for example to access the "res" folder. According to https://litmus.mozilla.org/show_test.cgi?id=48676 the share option should be disabled.
Attached patch patchSplinter Review
trivial fix
Assignee: nobody → mbrubeck
Status: NEW → ASSIGNED
Attachment #607749 - Flags: review?(margaret.leibovic)
Attachment #607749 - Flags: review?(margaret.leibovic) → review+
https://hg.mozilla.org/mozilla-central/rev/7e6aa4bedc46
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Verified/fixed on:

Aurora Fennec 13.0a2 (2012-03-25)
Nightly Fennec 14.0a1 (2012-03-25)
Device: Samsung Nexus S
OS: Android 2.3.6
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: