Closed Bug 683150 Opened 14 years ago Closed 14 years ago

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

Categories

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

Product:
Firefox for Android Graveyard
Component:
General
Platform:
ARM
Android
Type:
defect

Tracking

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

Status:
VERIFIED FIXED
Milestone:
Firefox 9
Tracking Flags:
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)

Nightly (08/30)
45.33 KB, image/png
Details
Patch v2
3.25 KB, patch
mbrubeck
: review+
Details | Diff | Splinter Review
Honeycomb screenshot
55.33 KB, image/png
Details
Gingerbread screenshot
22.09 KB, image/png
Details
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
needs priority. still reproducing on 09162011 nightly
tracking-fennec: --- → ?
status-firefox6: --- → affected
status-firefox7: --- → affected
status-firefox8: --- → affected
Priority: -- → P2
looks like its on nightly only. not reproducing for me on aurora and beyond.
status-firefox6: affected → unaffected
status-firefox7: affected → unaffected
status-firefox8: affected → unaffected
status-firefox9: --- → affected
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 v2 — — Splinter 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+
https://hg.mozilla.org/mozilla-central/rev/df408aabbd90
Status: NEW → RESOLVED
Closed: 14 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
status-firefox9: affected → fixed
Keywords: regressionwindow-wanted
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: