Closed
Bug 915985
Opened 11 years ago
Closed 10 years ago
Add a ZoomConstraints class to hold the allowzoom and min/max zoom values
Categories
(Core :: Panning and Zooming, defect)
Tracking
()
People
(Reporter: kats, Assigned: kats)
References
Details
Attachments
(2 files, 1 obsolete file)
36.54 KB,
patch
|
botond
:
review+
|
Details | Diff | Splinter Review |
35.47 KB,
patch
|
Details | Diff | Splinter Review |
The arguments to AsyncPanZoomController::UpdateZoomConstraints are passed around together in a bunch of places, originating somewhere in TabChild, going through TabParent, RenderFrameParent, APZCTreeManager, and GeckoContentController. It would be nice to define a struct to hold these three values, as botond noted at https://bugzilla.mozilla.org/show_bug.cgi?id=909881#c1.
Assignee | ||
Updated•11 years ago
|
Component: Graphics: Layers → Panning and Zooming
Assignee | ||
Comment 1•10 years ago
|
||
This will simplify the B2G half of bug 941995.
Assignee: nobody → bugmail.mozilla
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•10 years ago
|
||
Pretty straightforward mechanical change. Try push to make sure I didn't run afoul of some arcane compiler: https://tbpl.mozilla.org/?tree=Try&rev=5cb774c860b1
Comment 3•10 years ago
|
||
Comment on attachment 8355673 [details] [diff] [review] Patch Review of attachment 8355673 [details] [diff] [review]: ----------------------------------------------------------------- ::: gfx/layers/ipc/AsyncPanZoomController.cpp @@ +1696,4 @@ > } > > +ZoomConstraints > +AsyncPanZoomController::GetZoomConstraints() This method can be const. (Could have been before, too.) ::: gfx/layers/ipc/GeckoContentController.h @@ +75,5 @@ > * Retrieves the last known zoom constraints for the root scrollable layer > * for this layers tree. This function should return false if there are no > * last known zoom constraints. > */ > + virtual bool GetRootZoomConstraints(ZoomConstraints* aConstraints) Let's call this aOutConstraints for consistency. ::: layout/ipc/RenderFrameParent.cpp @@ +617,3 @@ > } > > + virtual bool GetRootZoomConstraints(ZoomConstraints* aConstraints) Likewise.
Assignee | ||
Comment 4•10 years ago
|
||
Made the changes above and got a green try: https://tbpl.mozilla.org/?tree=Try&rev=0a0f729e03f9 I also added a check to make sure aOutConstraints is non-null before writing to it for consistency.
Attachment #8355673 -
Attachment is obsolete: true
Attachment #8355871 -
Flags: review?(botond)
Updated•10 years ago
|
Attachment #8355871 -
Flags: review?(botond) → review+
Assignee | ||
Comment 5•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/8812993d675b
https://hg.mozilla.org/mozilla-central/rev/8812993d675b
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Comment 7•10 years ago
|
||
Kats, is the patch on this bug appropriate as-is for Aurora/1.3?
Flags: needinfo?(bugmail.mozilla)
Assignee | ||
Comment 8•10 years ago
|
||
Yup. Here's a version that applies cleanly to the latest aurora tip (required some minor manual rebasing). If you mark the bug as 1.3+ I'm happy to uplift it.
Flags: needinfo?(bugmail.mozilla) → needinfo?(milan)
Assignee | ||
Updated•10 years ago
|
blocking-b2g: --- → 1.3?
Assignee | ||
Comment 10•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/af1d31586716
status-b2g-v1.2:
--- → wontfix
status-b2g-v1.3:
--- → fixed
status-b2g-v1.4:
--- → fixed
status-firefox27:
--- → wontfix
status-firefox28:
--- → fixed
status-firefox29:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•