Closed Bug 633258 Opened 12 years ago Closed 12 years ago

Regression: iframe panning is broken

Categories

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

Product:
Firefox for Android Graveyard
Component:
Panning/Zooming
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(fennec2.0b5+)

Status:
VERIFIED FIXED
Tracking Flags:
Tracking Status
fennec 2.0b5+ ---

People

(Reporter: stechz, Assigned: mbrubeck)

References

Details

(Keywords: regression)

Attachments

(1 file)

patch
1.38 KB, patch
stechz
: review+
Details | Diff | Splinter Review 
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
Blocks: 628799
Keywords: regressionwindow-wanted
Attached patch patch — — Splinter 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: 12 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.