New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 10 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Oct 2011
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Restricted
  • Only users with EditIssue permission may comment.



Sign in to add a comment
link

Issue 92000: Printing with new print preview version gives excessive margins

Reported by allen.p....@gmail.com, Aug 8 2011

Issue description

Chrome Version       : 13.0.782.107 m
URLs (if applicable) :all but try this one http://tinyurl.com/3flkojf
Other browsers tested:
Add OK or FAIL after other browsers where you have tested this issue:
     Safari 5: worse -- printed part of page only 5 3/4 inches wide
  Firefox 5.0:OK - 7 1/2 inch printed page
       IE 7/8/9: OK - 7 1/2 inch printed page

What steps will reproduce the problem?
1.print a page
2.
3.

What is the expected result?
page printed to fill the page with 1/2 inch margins


What happens instead?
margins are 1" so printed page is 6.5 inches wide instead of 7 1/2.  Too small to read. 


Please provide any additional information below. Attach a screenshot if possible.

The new print preview did not work at all on this page at first but cleared up after a few days.  The margins on the initial version were 1 inch left and 3/4 inch right.  This latest version adds another bit of margin making it even smaller.

By the way, the initial bug "Printer Setup Missing" #947 I am the initial reporting party about 3 years ago.
 

Comment 1 by thestig@chromium.org, Aug 8 2011

Labels: Action-FeedbackNeeded
Does the problem happen if you print to PDF?

I assure you  bug 947  is being worked on. See  bug 67514  and  bug 69337 .

Comment 2 by allen.p....@gmail.com, Aug 8 2011

Hard to please everyone :-)

Comment 3 by allen.p....@gmail.com, Aug 8 2011

Update.  Chrome print to PDF gives 0.9 inch margin
Print to paper gives 1.0 inch margin
Print using Firefox or IE gives .5 left and .7 right margin.
Ideal would be .5 inch left and right which is the printer keepout area although given adjustable margins I would make it ever less as the printer actually can print it.  My printer is a HP Laser Jet 1200.
One Page Report.pdf
1.0 MB Download

Comment 4 by isherman@chromium.org, Aug 8 2011

Labels: Feature-Printing

Comment 5 by allen.p....@gmail.com, Aug 9 2011

I should add that the print on the paper does not match what I see in print preview.

Comment 6 by thestig@chromium.org, Aug 9 2011

What OS is it printing incorrectly on? Windows or Mac? Both? If you are on Mac, are you using Lion by any chance?

Comment 7 by allen.p....@gmail.com, Aug 9 2011

I am using Windows XP sp3.  Interesting question though.  I just printed with Windows 7 and it prints differently.  The printed page is the same size but the margins are reversed with larger left margin and smaller right margin.

Comment 8 by vandebo@chromium.org, Aug 10 2011

Blockedon: 92045
Labels: -Action-FeedbackNeeded
This may be the same root cause as  issue 92045 .  That's on my short list of things to fix, so lets see if this gets fixed to.

Comment 9 by allen.p....@gmail.com, Aug 10 2011

I found this interesting behavior.  If I print from Chrome in the normal way, I get the wide margins.  If instead I click Advanced and print from the HP driver pop-up, the margins are correct.  I have tried this with both the PCL5 and PCL6 driver.  This is a good work around for my application and may be a hint for solving it long term.

Comment 10 by s.malaca...@gmail.com, Aug 11 2011

i think it's because normal chrome print convert the page in a picture so the browser (or the printer driver) have to scale it down to fit the page. This result in a double of the margin (0,5 original + 0,5 for the scaled down).

Advence printing use a complete different way so it has not the problem.

Comment 11 by allen.p....@gmail.com, Aug 16 2011

I ran into an additional unexpected behavior this morning.  I went to print a document and selected landscape.  Hit advanced and print and it printed portrait.  To get landscape in advanced, you have to select it there even though it was selected in the chrome preview page.  What I would expect is that the landscape setting would be defaulted to the advanced driver setup.

Comment 12 by vandebo@chromium.org, Aug 16 2011

It is true that settings are not copied into the Advanced settings.  If you would like this changed, please file a new bug for it.

Comment 13 by allen.p....@gmail.com, Aug 16 2011

To tell the truth, once you get the main page working I won't use Advanced so I will leave it to you all to decide if it should be changed or not.

Comment 14 by vandebo@chromium.org, Sep 9 2011

Owner: vandebo@chromium.org
Status: Started

Comment 15 by vandebo@chromium.org, Sep 9 2011

Issue 96031 has been merged into this issue.

Comment 16 by bugdroid1@chromium.org, Sep 28 2011

Project Member
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=103177

------------------------------------------------------------------------
r103177 | vandebo@chromium.org | Wed Sep 28 13:58:01 PDT 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/ppapi/native_client/src/shared/ppapi_proxy/plugin_ppp_printing_rpc_server.cc?r1=103177&r2=103176&pathrev=103177
 M http://src.chromium.org/viewvc/chrome/trunk/src/ppapi/native_client/src/shared/ppapi_proxy/ppp_rpc_server.cc?r1=103177&r2=103176&pathrev=103177
 M http://src.chromium.org/viewvc/chrome/trunk/src/ppapi/example/example.cc?r1=103177&r2=103176&pathrev=103177
 M http://src.chromium.org/viewvc/chrome/trunk/src/ppapi/cpp/dev/printing_dev.cc?r1=103177&r2=103176&pathrev=103177
 M http://src.chromium.org/viewvc/chrome/trunk/src/ppapi/c/dev/ppp_printing_dev.h?r1=103177&r2=103176&pathrev=103177
 M http://src.chromium.org/viewvc/chrome/trunk/src/ppapi/native_client/src/shared/ppapi_proxy/browser_ppp_printing.cc?r1=103177&r2=103176&pathrev=103177
 M http://src.chromium.org/viewvc/chrome/trunk/src/ppapi/cpp/dev/printing_dev.h?r1=103177&r2=103176&pathrev=103177
 M http://src.chromium.org/viewvc/chrome/trunk/src/ppapi/native_client/src/shared/ppapi_proxy/trusted/srpcgen/ppp_rpc.h?r1=103177&r2=103176&pathrev=103177
 M http://src.chromium.org/viewvc/chrome/trunk/src/ppapi/native_client/src/trusted/plugin/plugin.cc?r1=103177&r2=103176&pathrev=103177
 M http://src.chromium.org/viewvc/chrome/trunk/src/ppapi/native_client/src/shared/ppapi_proxy/ppp_rpc_client.cc?r1=103177&r2=103176&pathrev=103177
 M http://src.chromium.org/viewvc/chrome/trunk/src/ppapi/native_client/src/shared/ppapi_proxy/ppp_printing.srpc?r1=103177&r2=103176&pathrev=103177
 M http://src.chromium.org/viewvc/chrome/trunk/src/ppapi/native_client/src/shared/ppapi_proxy/untrusted/srpcgen/ppp_rpc.h?r1=103177&r2=103176&pathrev=103177

Add IsScalingDisabled and PP_PRINTOUTPUTFORMAT_EMF to pepper printing interface.

IsScalingDisabled will let us handle PDFS that don't want to be scaled.
PP_PRINTOUTPUTFORMAT_EMF is to simply the printing flow on Windows.

BUG= 67091 ,  92045 ,  91880 ,  92000 ,  92218 ,  95905 
TEST=NONE

Review URL: http://codereview.chromium.org/8041052
------------------------------------------------------------------------

Comment 17 by vandebo@chromium.org, Oct 5 2011

 Issue 99118  has been merged into this issue.

Comment 18 by jeffreyc@google.com, Oct 5 2011

Cc: jeffharris@google.com komoroske@chromium.org
+jeffharris and +komorosoke since I am told this affects Google Docs too. thanks for working on this, Steve

Comment 19 by siedeen@chromium.org, Oct 6 2011

Labels: Hotlist-ConOps

Comment 20 by vandebo@chromium.org, Oct 7 2011

Labels: -Area-Undefined Area-Internals Merge-Requested Mstone-15 ReleaseBlock-Stable
Commit 104442 is a temporary fix for this issue.  I committed it to trunk last night.  I'd like to merge it to M15, as it's a blocker for print preview on Mac, which is enabled on 15.  Happy to wait until Monday to merge if you like.

Comment 21 by kareng@google.com, Oct 10 2011

Labels: -Merge-Requested Merge-Approved

Comment 22 by kareng@google.com, Oct 14 2011

this is merged. please close it.

Comment 23 by vandebo@chromium.org, Oct 14 2011

Labels: -Merge-Approved -Mstone-15 -ReleaseBlock-Stable Merge-Merged Mstone-16 merge-merged-874
Oops, meant to do that.  There's more to be done, but not for M15, updated labels appropriately.

Comment 24 by bugdroid1@chromium.org, Oct 15 2011

Project Member
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=105688

------------------------------------------------------------------------
r105688 | vandebo@chromium.org | Sat Oct 15 15:30:48 PDT 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/printing/print_dialog_gtk.cc?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/printing/printing_context.cc?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/printing/printing_context.h?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/printing/print_dialog_gtk.h?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/printing/printing_message_filter.cc?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/printing/printing_context_win.cc?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/printing/print_job_worker.cc?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/printing/printer_query.h?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/printing/page_setup.h?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/print_web_view_helper.cc?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/printing/print_dialog_gtk_interface.h?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/printing/print_settings.cc?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/printing/print_settings.h?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/print_web_view_helper.h?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/printing/printing_context_cairo.cc?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/printing/page_setup.cc?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/print_messages.h?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/printing/printer_query.cc?r1=105688&r2=105687&pathrev=105688
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/printing/print_job_worker.h?r1=105688&r2=105687&pathrev=105688

Move margin processing code to the browser process.

It seems that this is where it is supposed to live and it was erroneously added to PrintWebViewHelper.

BUG= 67091 ,  92045 ,  91880 ,  92000 ,  92218 ,  95905 
TEST=NONE


Review URL: http://codereview.chromium.org/8201027
------------------------------------------------------------------------

Comment 25 by vandebo@chromium.org, Oct 17 2011

Status: Fixed
I committed a fix for the rest of the margins bugs today, so this should be fixed on M16+ and Canary (tonight).

Comment 26 by willhsiao@chromium.org, Oct 24 2011

Checked with version 16.0.912.9 Canary and still experienced the same off margin print preview when printing from a Google Document.

Comment 27 by vandebo@chromium.org, Oct 24 2011

Blockedon: -92045
Status: Started
If you can give some specific repro steps, I'll take a look.  Screenshots/scans of the results wouldn't hurt.

Comment 28 by willhsiao@chromium.org, Oct 24 2011

Hi, 

Repro steps: 

1. Opened a Google Documents
2. Print the Google Doc by pressing command + p 
3. Preview is already off margin. 

Screenshot attached.
Google Docs sample.pdf
25.1 KB Download

Comment 29 by vandebo@chromium.org, Oct 24 2011

You see the problem in the preview?  I think I see what you mean in the attached pdf.  I can't seem to reproduce this though.  What OS are you on?  Do you still see it on the latest canary (should be m17)?

Comment 30 by willhsiao@chromium.org, Oct 24 2011

Just updated to 17 now. The margin is showing up correctly! Thanks!

Comment 31 by vandebo@chromium.org, Oct 24 2011

Status: Fixed

Comment 32 by siedeen@chromium.org, Oct 27 2011

Per comment 25, it should be fixed on M16, but it does not seem to be fixed on Dev (16.0.912.12 dev). Attached is the screenshot. Can someone confirm?
Print Preview on 16.0.912.12 dev.png
149 KB View Download

Comment 33 by allen.p....@gmail.com, Oct 27 2011

On the latest release, 15.0.874.106 m it is completely broken.  A printout that fills just over 10 inches of the 11 inch landscape page in what used to be called advanced print (from the printer driver) fills just over 7 inches when printed from the Chrome preview page.  The preview looks correct.  The left margin is not doubled though.

Comment 34 by allen.p....@gmail.com, Oct 27 2011

Here is the page I was printing
http://l-36.com/polar_targets.php?mode=table
Print landscape.

Comment 35 by vandebo@chromium.org, Oct 28 2011

I really thought this was patched on M15, but it seems there is still a problem.  I've filled  bug 102186  to track the remaining issues.

Comment 36 by jtan@chromium.org, Oct 31 2011

A follow-up to comment #32 from Sie Deen, this was initially still happening on M16 (16.0.912.12 to be exact), but 16.0.912.15 Mac no longer exhibits the bug.

Comment 37 by allen.p....@gmail.com, Dec 19 2011

The latest version of the printer 16.0.912.63 is expanding the pages so that they do not fit on a page.  This was not the case with previous versions that shrunk the margins. Now the right margin is off the page.  For example, in printing this on comment 35 where it says "I've filled  bug 102186  to track the" the right margin cuts the word but in half.  "to track the" are off page.  The margins seem correct but you have the scale messed up.

I tried to repeat this in 18.0.975.0 but print just crashes the browser.

Comment 38 by vandebo@chromium.org, Dec 19 2011

Please file a new bug about the new issue.  What pages other than the issue tracker do you see this bug on? If you have Safari installed, does it also happen there?

Comment 39 by allen.p....@gmail.com, Dec 19 2011

When this bug was opened the example page was http://tinyurl.com/3flkojf and it printed with 1 inch borders with the page only 6.5 inches wide.  Now the left margin is close (5/8 inches) but the program is trying to print the body of the page as 8 inches wide (it was 6.5 inches) and should be under 7.5 inches.  Thus it won't fit so part of the page is not printed.  I think this is just a new face of the same bug.  

Safari is still printing the page about 5 1/2 inches wide, just as reported when I opened this bug.

Let me summarize:
Correct: Left Margin 1/2 inch.  Right margin 1/2 inch, Page width 7 1/1 inches

Initial bug:  Left margin 1 inch, Right margin 1 inch. Page width 6 1/2 inches

Now: Left margin 5/8 inch, Right margin 3/8.  Page width 8 1/8 inches (right 5/8 inch doesn't print)

Comment 40 by allen.p....@gmail.com, Dec 19 2011

I opened  bug 108098  and copied my comments 37 and 39 into it.

Comment 41 by bugdroid1@chromium.org, Oct 13 2012

Project Member
Labels: Restrict-AddIssueComment-Commit
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.

Comment 42 by bugdroid1@chromium.org, Mar 10 2013

Project Member
Labels: -Area-Internals -Feature-Printing -Mstone-16 Cr-Internals Cr-Internals-Printing M-16

Comment 43 by bugdroid1@chromium.org, Mar 13 2013

Project Member
Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue

Sign in to add a comment