Closed Bug 1225287 Opened 9 years ago Closed 9 years ago

Display port is still too large on Fennec with APZ enabled

Categories

(Core :: Panning and Zooming, defect)

Product:
Core
Component:
Panning and Zooming
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla45
Tracking Flags:
Tracking Status
firefox45 --- fixed

People

(Reporter: snorp, Assigned: snorp)

References

Details

Attachments

(1 file)

Use more reasonable APZ multipliers on mobile
1.16 KB, patch
kats
: review+
Details | Diff | Splinter Review 
While testing some APZ stuff, I found it to be pretty easy to OOM Fennec by panning/zooming a bunch on a desktop page (nytimes.com in this case). A little digging indicates that we are creating more MemoryTextureHost instances than we are freeing, and thus leaking (or retaining?) the buffers.
Upon closer inspection it looks like we're just retaining these, and it's actually another case of bug 1223479
Summary: We sometimes leak MemoryTextureHost (and maybe MemoryTextureClient) on Android → Display port is still too large on Fennec (when zoomed in)
Summary: Display port is still too large on Fennec (when zoomed in) → Display port is still too large on Fennec with APZ enabled
Attachment #8688159 - Flags: review?(bugmail.mozilla) → review+
Assignee: jnicol → snorp
Blocks: 1206872
Component: Graphics → Panning and Zooming
https://hg.mozilla.org/mozilla-central/rev/4967e0ecdb20
Status: NEW → RESOLVED
Closed: 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.

Attachment

General

Creator:
Created:
Updated:
Size: