Closed Bug 742207 Opened 9 years ago Closed 9 years ago

Use the same code for determining size of SVG documents and other synthetic documents

Categories

(Firefox for Android Graveyard :: General, defect, P3)

ARM
Android
defect

Tracking

(blocking-fennec1.0 -, fennec11+)

RESOLVED FIXED
Firefox 15
Tracking Status
blocking-fennec1.0 --- -
fennec 11+ ---

People

(Reporter: mbrubeck, Assigned: mbrubeck)

References

Details

(Whiteboard: [not for 1.0])

Attachments

(1 file, 2 obsolete files)

Attached patch patch (obsolete) — Splinter Review
+++ This bug was initially created as a clone of Bug #710126 +++

Web pages that are wider than they are long can't be viewed in full. This is especially an issue with images and presumably video.

Bug 710126 fixed this for image documents but not for other synthetic documents (video, and possibly SVG).  This patch should extend the same fix to work for all synthetic document types.  I haven't tested it yet.
Thanks for doing this, I had it on my to-do list. The only question I have about this if the SVG-specific code in Tab.getPageSize needs to be taken out with this change. If SVGs render properly centered without that code then we should take it out.
Attached patch patch v2 (obsolete) — Splinter Review
This works fine, although I'm not sure if it's fixing any real problems.  Videos seem to be rendered at the correct size/scale with or without this patch.  I can't reproduce bug 712065 with or without this patch, even if I back out the fix from bug 712065.
Assignee: nobody → mbrubeck
Status: NEW → ASSIGNED
Attachment #612322 - Flags: review?(bugmail.mozilla)
By the way, I think we should wait until after we've branched for our "1.0" native release before landing this, to avoid adding any risk for this non-critical fix.
Whiteboard: [not for 1.0]
Forgot to hg qref.
Attachment #612104 - Attachment is obsolete: true
Attachment #612322 - Attachment is obsolete: true
Attachment #612322 - Flags: review?(bugmail.mozilla)
Attachment #612344 - Flags: review?(bugmail.mozilla)
Comment on attachment 612344 [details] [diff] [review]
patch v2 (for real)

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

Agreed.
Attachment #612344 - Flags: review?(bugmail.mozilla) → review+
Pushed to inbound:
https://hg.mozilla.org/integration/mozilla-inbound/rev/fb238b1ab6dc

QA note: As mentioned in comment 2, this patch does not fix any reproducible bugs; it is just code cleanup at this point and it should not have any visible effects.
Summary: video documents wider than they are long can't be fully viewed → Use the same code for determining size of SVG documents and other synthetic documents
Target Milestone: --- → Firefox 15
Just to be clear, it would be good if QA verified that it has no visible effects when loading different types of documents (HTML, SVG, any image, video, etc.)
https://hg.mozilla.org/mozilla-central/rev/fb238b1ab6dc
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.