Closed Bug 633258 Opened 9 years ago Closed 9 years ago

Regression: iframe panning is broken

Categories

(Firefox for Android Graveyard :: Panning/Zooming, defect)

defect
Not set
normal

Tracking

(fennec2.0b5+)

VERIFIED FIXED
Tracking Status
fennec 2.0b5+ ---

People

(Reporter: stechz, Assigned: mbrubeck)

References

Details

(Keywords: regression)

Attachments

(1 file)

STR:
1. Go to http://people.mozilla.org/~bstover/iframe3.html
2. Go to bottom of page and pan iframe

The iframe appears white, as if it has no content.
tracking-fennec: --- → 2.0b5+
Assignee: nobody → mbrubeck
I still see this after reverting back to http://hg.mozilla.org/mobile-browser/rev/d49b851fe2fc - so it looks like something in platform (mozilla-central) must have broken it.
I was wrong, this is a regression from bug 628799.  The first bad revision is:
http://hg.mozilla.org/mobile-browser/rev/134e1e507ca3
Status: NEW → ASSIGNED
Attached patch patchSplinter Review
updateCacheViewport need to use the scale of the root content view, regardless of which view we are scrolling.
Attachment #511856 - Flags: review?(ben)
Attachment #511856 - Flags: review?(ben) → review+
http://hg.mozilla.org/mobile-browser/rev/63fbf2c3ca08

Need some tests for this...
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
verified FIXED on build:
Mozilla/5.0 (Android; Linux armv71; rv:2.0b12pre) Gecko/20110216 Firefox/4.0b12pre Fennec/4.0b5pre


Filed a follow-up issue in bug 634636
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.