Closed
Bug 1152354
Opened 10 years ago
Closed 10 years ago
Print only print first and last page
Categories
(Core :: Print Preview, defect)
Tracking
()
VERIFIED
FIXED
mozilla40
People
(Reporter: kinwolf, Assigned: MatsPalmgren_bugz)
References
Details
(Keywords: regression, testcase)
Attachments
(2 files)
(deleted),
text/html
|
Details | |
(deleted),
patch
|
roc
:
review+
Sylvestre
:
approval-mozilla-aurora+
Sylvestre
:
approval-mozilla-release+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 6.0; rv:37.0) Gecko/20100101 Firefox/37.0
Build ID: 20150326190726
Steps to reproduce:
Go to http://blogs.msdn.com/b/powershell/archive/2014/01/31/want-to-secure-credentials-in-windows-powershell-desired-state-configuration.aspx
Try to print the page using the menu->print icon
Actual results:
Only the first and last page appear in the preview, and only those will print
Expected results:
There is at least 5 pages to print, not 2. Even doing a "select all" and
text continued...
Even doing a select all and try to print that will give the same result. Chrome, Opera and Internet explorer doesn't have that problem, they print that page fine. The problem also happen on Windows 7. It also happened on a https page from university, but I can't provide credentials to check this one out, obviously.
Print Prewiew is wrong on my machine with FF37, but the print output (PDF Creator, XPS Writer) is OK (4 pages).
Print preview regressed:
good=2014-12-15
bad=2014-12-16
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=f14dcd1c8c0b&tochange=b836016d82b5
Comment 4•10 years ago
|
||
Pushlog: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=9cacf0449d5e&tochange=c2184fba8cc1
Triggered by: Bug 1108104
Blocks: 1108104
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: regressionwindow-wanted → regression
Comment 5•10 years ago
|
||
[Tracking Requested - why for this release]: regressed since Firefox37
status-firefox37:
--- → affected
status-firefox38:
--- → affected
status-firefox39:
--- → affected
status-firefox40:
--- → affected
status-firefox-esr31:
--- → unaffected
tracking-firefox38:
--- → ?
tracking-firefox39:
--- → ?
tracking-firefox40:
--- → ?
Assignee | ||
Updated•10 years ago
|
Comment 6•10 years ago
|
||
Tracking for 38+ since this is a recent regression.
Summary: Print only print fist and last page → Print only print first and last page
Keywords: testcase-wanted → testcase
Assignee | ||
Comment 9•10 years ago
|
||
I don't have time to work on this for a while, so I think we should
just revert the change from bug 1108104 for now since this regression
is worse than what we were trying to fix there. This patch also fixes
bug 1155230, although the problem there seems like a minor edge case.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e88253b44292
Attachment #8594737 -
Flags: review?(roc) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 10•10 years ago
|
||
Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Comment 12•10 years ago
|
||
Mats - From comment 9 I take it that you went with a backout. 38 and 39 are also affected. Can we backout from those branches as well?
Flags: needinfo?(mats)
Assignee | ||
Comment 13•10 years ago
|
||
Comment on attachment 8594737 [details] [diff] [review]
Revert the change from bug 1108104
Approval Request Comment
[Feature/regressing bug #]: bug 1108104
[User impact if declined]: missing content in Print/Preview
[Describe test coverage new/current, TreeHerder]: we have poor test coverage here, we should add the test in this bug and bug 1155230 as reftests [helpwanted]
[Risks and why]: low-risk, reverts to old behavior before bug 1108104
[String/UUID change made/needed]: none
Flags: needinfo?(mats)
Attachment #8594737 -
Flags: approval-mozilla-beta?
Attachment #8594737 -
Flags: approval-mozilla-aurora?
Updated•10 years ago
|
Comment 14•10 years ago
|
||
Comment on attachment 8594737 [details] [diff] [review]
Revert the change from bug 1108104
[Triage Comment]
38 is now in m-r!
Attachment #8594737 -
Flags: approval-mozilla-release+
Attachment #8594737 -
Flags: approval-mozilla-beta?
Attachment #8594737 -
Flags: approval-mozilla-aurora?
Attachment #8594737 -
Flags: approval-mozilla-aurora+
Comment 15•10 years ago
|
||
Should be in 38 beta 7
Comment 16•10 years ago
|
||
Comment 17•10 years ago
|
||
Comment 18•10 years ago
|
||
Updated•10 years ago
|
Flags: qe-verify+
Comment 19•10 years ago
|
||
Reproduced this issue on Firefox 37.0.2, build ID: 20150415140819.
Confirming the fix on Windows 7 64-bit, Ubuntu 14.04 32-bit and Mac OS X 10.9.5 using:
- latest Nightly, build ID: 20150426030248;
- latest Aurora, build ID: 20150426004003;
- Firefox 38 beta 8, build ID: 20150426174329.
Status: RESOLVED → VERIFIED
QA Contact: cornel.ionce
Updated•10 years ago
|
status-firefox38.0.5:
--- → fixed
Comment 22•10 years ago
|
||
Verified on 40 Nightly, Aurora 39 and Beta 38 - removing qe-verify flag.
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•