Java may get incorrect page size

RESOLVED FIXED in Firefox 14

Status

()

defect
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: kats, Assigned: kats)

Tracking

unspecified
Firefox 14
All
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox14 fixed, blocking-fennec1.0 beta+)

Details

(Whiteboard: [viewport])

Attachments

(1 attachment)

There's a possibility that the page size sent from gecko -> java in the Viewport:PageSize message is at a different zoom level than that currently in java. We need to account for this. Note that the setPageSize(float,float,float) method in GeckoLayerClient that is invoked by the compositor does this too; I just forgot it needs to happen in the Viewport:PageSize handler as well. This doesn't happen very often but it's visible in the dynamic-resolution strategy and makes the scrollbars change in size incorrectly. This is not easy to fix, unfortunately, since the zoom factor associated with the page size sent from browser.js is not necessarily the tab._zoom; it is really the resolution set on the cwu.
Whiteboard: [viewport]
Posted patch PatchSplinter Review
This patch is sort of what I had in mind, but it doesn't actually fix the problem I was seeing because the messageMetrics.getZoomFactor() is the zoom level stored in the tab in browser.js, not the display resolution on the pres shell (the two are the same unless we're using DynamicResolutionStrategy).
blocking-fennec1.0: --- → ?
Would this fix bug 735539 and/or bug 740005?
No, this bug never manifest in current builds; it's only an issue for code that is not yet turned on.
Oh wait, what I just said was wrong. It could manifest in current code as well, but still wouldn't cause the bugs you refer to.
Comment on attachment 610642 [details] [diff] [review]
Patch

I figure we might as well land this since it is technically a bug in current code.
Attachment #610642 - Flags: review?(chrislord.net)
Comment on attachment 610642 [details] [diff] [review]
Patch

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

Looks good to me.
Attachment #610642 - Flags: review?(chrislord.net) → review+
https://hg.mozilla.org/mozilla-central/rev/5989e604e690
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
blocking-fennec1.0: ? → beta+
You need to log in before you can comment on or make changes to this bug.