Closed
Bug 489224
Opened 15 years ago
Closed 15 years ago
Printing doesn't print text field values
Categories
(Core :: Printing: Output, defect, P2)
Core
Printing: Output
Tracking
()
VERIFIED
FIXED
mozilla1.9.2a1
People
(Reporter: smaug, Assigned: smaug)
References
Details
(Keywords: regression, verified1.9.1, Whiteboard: regression from 425265)
Attachments
(3 files)
12.49 KB,
patch
|
roc
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
12.41 KB,
patch
|
Details | Diff | Splinter Review | |
3.38 KB,
patch
|
Details | Diff | Splinter Review |
<input type="text"> and <textarea> are printed as empty, even if they contained some text. I guess this might be a regression from my printing changes. Will verify tomorrow.
Flags: blocking1.9.1?
Assignee | ||
Comment 1•15 years ago
|
||
Oh, I haven't tested this on 1.9.1 yet.
Assignee | ||
Comment 2•15 years ago
|
||
Yes, also in 1.9.1
Assignee | ||
Updated•15 years ago
|
Priority: -- → P2
Assignee | ||
Comment 3•15 years ago
|
||
And should be pretty easy to fix. Will write the patch tomorrow.
Comment 4•15 years ago
|
||
Same happens with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b4pre) Gecko/20090420 Shiretoko/3.5b4pre ID:20090420031111
Assignee | ||
Comment 5•15 years ago
|
||
The interesting part is the last hunk in the patch. I want to be very strict when to allow mutations. Most of the patch is just moving nsDocumentObserverForNonDynamicPresContext, so that it can use PresShell. (Form elements will need some special handling also with clone-document-for-printing approach I'm doing for trunk.)
Attachment #373887 -
Flags: superreview?(roc)
Attachment #373887 -
Flags: review?(roc)
Assignee | ||
Comment 6•15 years ago
|
||
1.9.1 will need just a bit different patch, and I'll write tests before landing. (well, if this gets review)
Keywords: regressionwindow-wanted
Updated•15 years ago
|
Flags: blocking1.9.1? → blocking1.9.1+
Assignee | ||
Comment 7•15 years ago
|
||
The only difference is the last hunk in the patch.
Comment on attachment 373887 [details] [diff] [review] for trunk This is yucky. Can't wait for document cloning!
Attachment #373887 -
Flags: superreview?(roc)
Attachment #373887 -
Flags: superreview+
Attachment #373887 -
Flags: review?(roc)
Attachment #373887 -
Flags: review+
Assignee | ||
Comment 9•15 years ago
|
||
Assignee | ||
Comment 10•15 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/44d20da17412
Status: NEW → RESOLVED
Closed: 15 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Assignee | ||
Comment 11•15 years ago
|
||
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/f062a51f5446
Keywords: fixed1.9.1
Comment 12•15 years ago
|
||
Verified fixed on trunk and 1.9.1 with: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2a1pre) Gecko/20090423 Minefield/3.6a1pre ID:20090423040020 Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b4pre) Gecko/20090423 Shiretoko/3.5b4pre ID:20090423040926
Status: RESOLVED → VERIFIED
Flags: in-testsuite? → in-testsuite+
Keywords: fixed1.9.1 → verified1.9.1
Target Milestone: --- → mozilla1.9.2a1
Updated•15 years ago
|
Flags: wanted1.9.0.x+
Flags: blocking1.9.0.12?
Whiteboard: regression from 425265
Updated•15 years ago
|
Flags: wanted1.9.0.x+
Flags: blocking1.9.0.12?
You need to log in
before you can comment on or make changes to this bug.
Description
•