Closed Bug 803049 Opened 12 years ago Closed 12 years ago

[Gallery] Previews appear and disappear

Categories

(Firefox OS Graveyard :: Gaia, defect, P1)

ARM
Gonk (Firefox OS)
defect

Tracking

(blocking-basecamp:+)

VERIFIED FIXED
blocking-basecamp +

People

(Reporter: dscravaglieri, Assigned: djf)

References

Details

(Keywords: smoketest)

Attachments

(1 file)

Changeset gecko: bbf233397549175223da3687bc5678d91685e54a gaia: bb84a2a130fd669713986390292538a9bdbd3469 Device: otoro (unagi ?) 1- Launch Gallery 2- See photos preview Expected: Photos preview Actual: Photo preview appear and disapear randomly. Looks like a repaint bug.
blocking-basecamp: --- → ?
Summary: [Gallery] → [Gallery] Previews appear and disappear
blocking-basecamp: ? → -
Please re-nom if anyone else can reproduce, we couldn't reproduce in triage.
blocking-basecamp: - → ?
blocking-basecamp: ? → +
Priority: -- → P1
Attached image screenshot (deleted) —
screenshot of missing thumbnail previews in gallery
djf, who can fix this?
Blocks: 803020
(In reply to Andrew Overholt [:overholt] from comment #5) > djf, who can fix this? I'll add this bug to my list and investigate. Either its my bug and I can fix it, or there's a platform bug, but I won't know without digging into it.
Assignee: nobody → dflanagan
Priority: P1 → --
I have a hunch that this is actually a memory-related issue. Tony, it would be great if you could apply the patch in the PR https://github.com/mozilla-b2g/gaia/pull/6034 and then run make reset-gaia and see if you can still reproduce it. (Note that the reset-gaia is required. Just make install-gaia isn't enough; it needs to blow away the gallery's existing database.)
Priority: -- → P1
Priority: P1 → P3
Guys, we're treating this as a dogfood blocker, smoketest bug. I would suggest reprioritizing this one higher, as it's easily reproducible if you have a large set of images. David, i havent been able to apply patches to my daily builds we're testing on. has this PR been merged yet into gaia nightly?
Re-prioritized as smoketest blocker.
Severity: normal → critical
Keywords: smoketest
Priority: P3 → P1
(In reply to David Flanagan [:djf] from comment #7) > I have a hunch that this is actually a memory-related issue. > > Tony, it would be great if you could apply the patch in the PR > https://github.com/mozilla-b2g/gaia/pull/6034 and then run make reset-gaia > and see if you can still reproduce it. > > (Note that the reset-gaia is required. Just make install-gaia isn't enough; > it needs to blow away the gallery's existing database.) This fix the issue for me.
So the fix I've got for this issue apparently does not fix #805114. People who can't open the app at all still will not be able to open the app. But those who can will find it usable. So that's progress, at least. I'll clean up the patch and try to get it landed. I'll also modify the patch so that it bumps the database version number and does not require a make reset-gaia.
I should note that my patch addresses this bug by using smaller thumbnail images, which is something that needs to happen anyway. But it doesn't touch whatever underlying gecko change has caused the bug. Nothing changed in gallery to cause this bug, and I suspect that something has changed in the platform.
Fixed (I think) in https://github.com/mozilla-b2g/gaia/pull/6034 That should fix the smoketest blocker. I'm reluctant to close the bug, though, because I think there is some kind of platform issue going on. If I can come up with a test case that demonstrates the platform bug, I'll close this one.
Closing this bug now because it is specific to gallery, and I've landed a gallery-specific workaround. The more general gecko regression is now in #807143
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Verified the previews display correctly on the 11/28 unagi build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: