Closed
Bug 1156392
Opened 10 years ago
Closed 9 years ago
Remove MOZ_ANDROID_APZ condition from ifdef in ScrollFrameHelper::WantAsyncScroll
Categories
(Core :: Layout, defect)
Tracking
()
RESOLVED
FIXED
mozilla45
Tracking | Status | |
---|---|---|
firefox45 | --- | fixed |
People
(Reporter: kats, Assigned: kats)
References
Details
Attachments
(1 file)
2.13 KB,
patch
|
tnikkel
:
review+
|
Details | Diff | Splinter Review |
See https://bugzilla.mozilla.org/show_bug.cgi?id=1146024#c7 (second paragraph) for backstory - nsGfxScrollFrame::WantAsyncScroll uses the presence of scrollbars to determine if something should be async-scrollable, and in Fennec we have the scrollbars set to display:none in content.css (because we draw them in Java). Eventually we should rip out the Java scrollbars and use the Gecko ones like we do everywhere else, at which point we can restore WantAsyncScroll to the way it was prior to bug 1146024 (i.e. remove the MOZ_ANDROID_APZ component of the ifdef).
Assignee | ||
Updated•10 years ago
|
OS: Mac OS X → Android
Hardware: x86 → All
Assignee | ||
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
Assignee | ||
Comment 2•9 years ago
|
||
Actually I'm going to repurpose this bug for the last bit of comment (removing the ifdef condition), which hasn't been done yet.
Assignee | ||
Updated•9 years ago
|
Assignee | ||
Comment 3•9 years ago
|
||
We can take out the MOZ_ANDROID_APZ condition here now that we have gecko scrollbars enabled in android-apz.
Attachment #8694242 -
Flags: review?(tnikkel)
Updated•9 years ago
|
Attachment #8694242 -
Flags: review?(tnikkel) → review+
Comment 5•9 years ago
|
||
bugherder |
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
You need to log in
before you can comment on or make changes to this bug.
Description
•