Closed Bug 1713586 Opened 3 years ago Closed 3 years ago

refactor AsyncPanZoomController::ZoomToRect a bit

Categories

(Core :: Panning and Zooming, task)

task

Tracking

()

RESOLVED FIXED
91 Branch
Tracking Status
firefox91 --- fixed

People

(Reporter: tnikkel, Assigned: tnikkel)

References

Details

Attachments

(2 files)

No description provided.
Assignee: nobody → tnikkel
Status: NEW → ASSIGNED

This is a slight behaviour change but should be more correct.

Depends on D116368

Blocks: 1713589
Pushed by tnikkel@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/684397d12b95 Move zoom out calculation in AsyncPanZoomController::ZoomToRect after code to calculate zoom in non-zoom out case so we don't need to duplicate calling endZoomToMetrics.SetZoom and calculating sizeAfterZoom. r=botond https://hg.mozilla.org/integration/autoland/rev/32e3e98ca847 Move the clamping of the target zoom value in AsyncPanZoomController::ZoomToRect after some code which can change it. r=botond
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 91 Branch
Regressions: 1738696
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: