Closed Bug 683150 Opened 8 years ago Closed 8 years ago

Content area of remote desktop pane displays area of previously viewed page

Categories

(Firefox for Android Graveyard :: General, defect, P2)

ARM
Android
defect

Tracking

(firefox6 unaffected, firefox7 unaffected, firefox8 unaffected, firefox9 fixed, fennec9+)

VERIFIED FIXED
Firefox 9
Tracking Status
firefox6 --- unaffected
firefox7 --- unaffected
firefox8 --- unaffected
firefox9 --- fixed
fennec 9+ ---

People

(Reporter: aaronmt, Assigned: wesj)

References

Details

(Keywords: regression, Whiteboard: [inbound])

Attachments

(4 files, 1 obsolete file)

Attached image Nightly (08/30)
Mozilla/5.0 (Android; Linux armv7l; rv:9.0a1) Gecko/20110830 Firefox/9.0a1 Fennec/9.0a1

Currently on trunk when viewing the remote desktop pane, you will see an area of the previously viewed page (behind the Sync related dialog).

Not reproducible on Aurora (8.0a2 [08-30]) nor Beta (7.0-build2).

Regression from bug 681356?

STR:
1. Visit a website, i.e., (mozilla.com)
2. Tap the navigation-bar, tap the 'Desktop' pane
OS: Mac OS X → Android
Hardware: x86 → ARM
Related? bug 668760
Assignee: nobody → wjohnston
Duplicate of this bug: 684463
Duplicate of this bug: 683185
needs priority.  still reproducing on 09162011 nightly
tracking-fennec: --- → ?
Priority: -- → P2
looks like its on nightly only.  not reproducing for me on aurora and beyond.
tracking-fennec: ? → 9+
Attached patch Patch (obsolete) — Splinter Review
Attachment #561944 - Flags: review?(mark.finkle)
Comment on attachment 561944 [details] [diff] [review]
Patch

Froyo is not affected?
Attachment #561944 - Flags: review?(mark.finkle) → review+
Drive-by review comment: In the Froyo theme we put a background on the .autocomplete-items class instead.
Attached patch Patch v2Splinter Review
I should have looked at this harder. We're trying to show a message in this case, but the sync message pane isn't themed on honeycomb or gingerbread. This just copies over the theme code from Froyo.

I'll attach some screenshots I took on desktop or both. In both, I've forced the theme to show a message by adding an else statement if "engine" does not exist here:

http://mxr.mozilla.org/mozilla-central/source/mobile/chrome/content/bindings.xml#1412

we should show a message in that case and probably show a button(/link?) ("Set up sync", or if sync is set up but disabled "Enable sync"). I'll file a separate bug for that.
Attachment #561944 - Attachment is obsolete: true
Attachment #562065 - Flags: review?(mbrubeck)
Attached image Honeycomb screenshot
Attached image Gingerbread screenshot
Attachment #562065 - Flags: review?(mbrubeck) → review+
Duplicate of this bug: 688758
https://hg.mozilla.org/mozilla-central/rev/df408aabbd90
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 9
Mozilla/5.0 (Android; Linux armv7l; rv:9.0a1) Gecko/20110925 Firefox/9.0a1 Fennec/9.0a1
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.