Closed Bug 103160 Opened 23 years ago Closed 23 years ago

DocumentViewer::Print Interface is misleading

Categories

(Core :: Layout, defect, P1)

x86
Windows 98
defect

Tracking

()

RESOLVED FIXED
mozilla0.9.8

People

(Reporter: bernd_mozilla, Assigned: rods)

Details

The function call for DocumentViewer::Print http://lxr.mozilla.org/seamonkey/source/content/base/src/nsDocumentViewer.cpp#4256 lists as the second argument FILE *aFile, I think it should be renamed into aRegressionOutput because it looks very much that you can use this to print to file, this impression is also encouraged by the comment at http://lxr.mozilla.org/seamonkey/source/content/base/src/nsDocumentViewer.cpp#349 . But this aFile and mFilePointer are purely used for the layout regression data.
I'll fix this when I do my next check in of nsDocumentViewer.cpp
Status: NEW → ASSIGNED
Priority: -- → P1
Target Milestone: --- → mozilla0.9.8
fixed
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.