Closed Bug 730899 Opened 8 years ago Closed 8 years ago

Maple: during scrolling we do the first scroll to the same position that we're already at

Categories

(Firefox for Android :: General, defect)

All
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 14
Tracking Status
firefox14 --- fixed

People

(Reporter: jrmuizel, Assigned: kats)

Details

Attachments

(1 file)

We call ScrollTo with the current offset. This doesn't do a lot of harm, but is sort of wasteful.
Assignee: nobody → bugmail.mozilla
Component: Graphics → General
OS: Linux → Android
Product: Core → Fennec Native
QA Contact: thebes → general
Hardware: x86_64 → All
Attached patch PatchSplinter Review
Save some cycles if we're not scrolling anywhere.
Attachment #611102 - Flags: review?(chrislord.net)
Comment on attachment 611102 [details] [diff] [review]
Patch

Review of attachment 611102 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM.
Attachment #611102 - Flags: review?(chrislord.net) → review+
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.