Closed Bug 877726 Opened 7 years ago Closed 7 years ago

Convert FrameMetrics.mScrollOffset to be a gfx::Point<CSSPixel>

Categories

(Core :: Graphics, defect)

23 Branch
x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla24

People

(Reporter: kats, Assigned: kats)

References

Details

Attachments

(1 file)

Attached patch PatchSplinter Review
Follow-up from bug 865735 - we is an incremental propagation of pixel-units information to some uses of gfx::Point, gfx::Size, and gfx::Rect.
Attachment #756040 - Flags: review?(ajones)
Assignee: nobody → bugmail.mozilla
Comment on attachment 756040 [details] [diff] [review]
Patch

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

::: layout/base/nsDisplayList.cpp
@@ +617,5 @@
>                                 const nsDisplayItem::ContainerParameters& aContainerParameters,
>                                 bool aMayHaveTouchListeners) {
>    nsPresContext* presContext = aForFrame->PresContext();
>    int32_t auPerDevPixel = presContext->AppUnitsPerDevPixel();
> +  int32_t auPerCSSPixel = nsPresContext::AppUnitsPerCSSPixel();

auPerCSSPixel is only used once so could be inlined
Attachment #756040 - Flags: review?(ajones) → review+
https://hg.mozilla.org/mozilla-central/rev/5bb24851fd4a
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
You need to log in before you can comment on or make changes to this bug.