Closed Bug 649583 Opened 9 years ago Closed 9 years ago
Panning regression on google reader in 2011-04-12 nightly
Here is my setup: - Go on http://google.com/reader/i - Subscribe to http://feeds.feedburner.com/DilbertDailyStrip - Select any of the past comic - Click zoom in Up until 2011-04-11 nightly (double checked with ftp://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/2011-04-11-05-mozilla-central-android-r7/), one could scroll the zoomed comic strip by dragging it. Since 2011-04-12 nightly (double checked with ftp://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/2011-04-12-04-mozilla-central-android-r7/), dragging the zoomed comic strip brings the right panel and doesn't scroll the comic strip.
The tinderbox build corresponding to http://hg.mozilla.org/mozilla-central/rev/b48ebf9695bb already has this problem. Unfortunately, there's no older tinderbox build on ftp.mozilla.org
This might be a regression from bug 639179
Probably iframes in general. See bug 649585 too: Steps to reproduce: 1. Go to html code tutorial: http://www.htmlcodetutorial.com/frames/_IFRAME.html 2. Click into the iframe 3. Try to scroll within the iframe
So, I did a try build off that failing nightly, with bug 639179 backed out, and... still the same problem. http://hg.mozilla.org/try/rev/d1c92a3c2989 https://email@example.com/try-mob-andrd-r7-bld/
We'd like to get a solution for this on Aurora within a week, either by backout or fix.
A few data points, since I have them: The nightly that I tested was working is 09b605eb3e0d The nightly that I tested was not working is 48d6abe0a436 The oldest tinderbox build that I tested was not working is b48ebf9695bb The latest aurora build also doesn't work. (wasn't sure of the merge/fork point)
Grah, looks like firefox decided I wanted to reset tracking-firefox5 :(
OK, somehow I missed a merge conflict and one of the files did not get patched.
Assignee: wjohnston → ben
Comment on attachment 525795 [details] [diff] [review] PAnning regression on google reader in 2011-04-12 nightly (already reviewed)
Attachment #525795 - Flags: review+
Attachment #525795 - Attachment is obsolete: true
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
This was a regression caused by an error on checkin of bug 642246. This should go to Aurora.
Did this get review from someone besides the patch author?
Yes, roc. See comments 10 and 11.
Attachment #525797 - Flags: approval-mozilla-aurora?
Mozilla /5.0 (Android;Linux armv7l;rv:5.0a2) Gecko/20110414 Firefox/5.0a2 Fennec/5.0a2 Device: LG Optimus 2X (Android 2.2) I still can't pan the iframe from http://www.htmlcodetutorial.com/frames/_IFRAME.html. Reopening the bug.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
My Dilbert test case works on the 20110414 6.0a1 nightly.
Your test case is also fixed (no surprise, really). It probably hasn't landed on Aurora yet, though.
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
Not approving this yet, we'd like to get more confidence that this doesn't hurt desktop. Can we get layout people to vouch for that, or some other feedback here?
Comment on attachment 525797 [details] [diff] [review] Panning regression on google reader in 2011-04-12 nightly We definitely want this patch for Fennec. Roc, could you weigh in? As a reminder, Roc approved the original patch, which landed but did not have the needed changes to FrameLayerBuilder.cpp. These changes allow for empty layers to be sent across process instead of culled. For Firefox, there should be no other impact, because Firefox will never have any LAYER_ACTIVE_EMPTY display items.
Attachment #525797 - Flags: feedback?(roc)
Comment on attachment 525797 [details] [diff] [review] Panning regression on google reader in 2011-04-12 nightly Right, we can be very confident this does not affect desktop.
Attachment #525797 - Flags: feedback?(roc) → feedback+
verified FIXED On builds: Mozilla/5.0 (Android; Linux armv7l; rv:6.0a1) Gecko/20110421 Firefox/6.0a1 Fennec/6.0a1 ID:20110421043148
Status: RESOLVED → VERIFIED
Attachment #525797 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Pushed to Aurora: http://hg.mozilla.org/mozilla-aurora/rev/8369b648ab94
Verified on Aurora Branch: Mozilla/5.0 (Android; Linux armv71; rv5.0a2) Gecko/20110509 Firefox/5.0a2 Fennec/5.0a2 Device: Thunderbolt OS: Android 2.2
You need to log in before you can comment on or make changes to this bug.