Closed
Bug 865255
Opened 11 years ago
Closed 11 years ago
Apps reporting incorrect empty message when USB plugged in
Categories
(Firefox OS Graveyard :: General, defect)
Tracking
(blocking-b2g:leo+, firefox21 disabled, firefox22 disabled, firefox23+ fixed, b2g18+ fixed, b2g18-v1.0.0 wontfix, b2g18-v1.0.1 wontfix)
People
(Reporter: zcampbell, Assigned: dhylands)
References
Details
(Keywords: regression, Whiteboard: [fromAutomation][fixed-in-birch])
Attachments
(2 files)
(deleted),
patch
|
dougt
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
dhylands
:
review+
|
Details | Diff | Splinter Review |
Gecko revision=7d1df39eff1d6f96c1ebe8bdab997c0c3df769ac
Gaia revision=140a9f7a6ade455763aca5e65b2572a51c731746
BuildID = 20130423230207
Version = 18.0
The apps Camera, Music, Video and Gallery are all reporting 'No Memory Card' when they are started up with an empty SD card and USB connected.
This is not correct as if there is media on the SD card in some cases it is still accessible.
STR
1. Wipe all image media from the SD card
2. Kill all apps
3. Plug in USB cable
4. Load the gallery app - Note the 'No Memory Card' overlay message
5. Unplug USB
6. Put image media (not a camera photo) onto the SD card
7. Plug in USB
8. Load the Gallery app
9. Wait 20 seconds
10. Gallery app will show the image media
Reporter | ||
Comment 1•11 years ago
|
||
Sorry this is: b2g18/v1-train
Updated•11 years ago
|
Keywords: regression
Assignee | ||
Comment 2•11 years ago
|
||
It looks like this was borked when bug 860934 landed. I've changed this to tef+ since bug 860934 is tef+ and it wasn't supposed to regress this.
I'm going to file another bug to see if we can add a test for this (the test requires real hardware).
Blocks: 860934
blocking-b2g: leo? → tef+
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → dhylands
Reporter | ||
Comment 3•11 years ago
|
||
We have a test (camera, gallery, music, video apps) for this on real hardware and we found it ;)
But something higher up the tree would be good too if it's possible as we only follow on v1-train.
Assignee | ||
Comment 4•11 years ago
|
||
Attachment #741463 -
Flags: review?(doug.turner)
Comment 5•11 years ago
|
||
Comment on attachment 741463 [details] [diff] [review]
Send a fully-qualified path to GetSDCardStatus function - b2g18
Review of attachment 741463 [details] [diff] [review]:
-----------------------------------------------------------------
you'll need to uplift this, right?
Attachment #741463 -
Flags: review?(doug.turner)
Attachment #741463 -
Flags: review+
Attachment #741463 -
Flags: approval-mozilla-b2g18?
Assignee | ||
Comment 6•11 years ago
|
||
Patch for master (birch)
Carrying forward the r+
Attachment #741486 -
Flags: review+
Assignee | ||
Updated•11 years ago
|
Attachment #741463 -
Attachment description: Send a fully-qualified path to GetSDCardStatus function → Send a fully-qualified path to GetSDCardStatus function - b2g18
Attachment #741463 -
Flags: approval-mozilla-b2g18?
Assignee | ||
Comment 7•11 years ago
|
||
Changed from tef+ to leo+ since 1.0.1 wasn't affected by the patch.
blocking-b2g: tef+ → leo+
Assignee | ||
Comment 8•11 years ago
|
||
https://hg.mozilla.org/projects/birch/rev/e7ea557a28ef
https://hg.mozilla.org/releases/mozilla-b2g18/rev/10a24cf4357a
I think I got the flags right.
status-b2g18:
--- → fixed
status-b2g18-v1.0.0:
--- → wontfix
status-b2g18-v1.0.1:
--- → wontfix
status-firefox21:
--- → disabled
status-firefox22:
--- → disabled
status-firefox23:
--- → fixed
tracking-b2g18:
--- → ?
tracking-firefox23:
--- → ?
Whiteboard: [fromAutomation] → [fromAutomation][fixed-in-birch]
Comment 11•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 12•11 years ago
|
||
Verified fixed on v1-train builds. automation is passing. Thanks!
Status: RESOLVED → VERIFIED
Updated•11 years ago
|
Updated•11 years ago
|
Flags: in-moztrap?
Updated•11 years ago
|
Flags: in-moztrap?
Comment 15•11 years ago
|
||
Removing NI on this, as it will be covered in future testruns; a moztrap case was added.
Flags: needinfo?(jhammink)
You need to log in
before you can comment on or make changes to this bug.
Description
•