Closed Bug 510961 Opened 10 years ago Closed 10 years ago

Remove scrollingOuterX hacks related to zooming

Categories

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

x86
Windows NT
defect
Not set

Tracking

(fennec1.0+)

RESOLVED FIXED
Future
Tracking Status
fennec 1.0+ ---

People

(Reporter: pavlov, Assigned: stechz)

References

Details

Attachments

(1 file, 1 obsolete file)

When we added zooming, we added a few hacks that need to be removed and cleaned up related to the scrollingOuterX code.
Assignee: nobody → webapps
Attached patch Refactored patch (obsolete) — Splinter Review
First crack at a refactoring.  I think the drag code is much clearer, but I'm worried about edge cases and performance regression.
Attachment #397718 - Flags: review?
Attachment #397718 - Attachment is obsolete: true
Attachment #397871 - Flags: review?
Attachment #397718 - Flags: review?
Attachment #397871 - Flags: review?(pavlov)
Attachment #397871 - Flags: review?(froystig)
Attachment #397871 - Flags: review?
Attachment #397871 - Flags: review?(froystig) → review+
Attachment #397871 - Flags: review?(pavlov) → review+
pushed: https://hg.mozilla.org/mobile-browser/rev/298d0049b2da

updated to trunk first
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Future
Component: General → Panning/Zooming
You need to log in before you can comment on or make changes to this bug.