652914 - Preferred page background color is printed, even when "Print Background Colors" is unchecked

Reporter

Description

•

13 years ago

STEPS TO REPRODUCE:
 1. In Firefox preferences, go to "Content" tab, and click the "colors" button.
 2. Pick some non-white background color, e.g. red
 3. Visit http://ftp.mozilla.org/pub/mozilla.org/README and do File| Print
 4. Ensure that "print background colors" is unchecked in print dialog. (on Linux, it's under the "Options" tab in that dialog.
 5. Go ahead with the print.

EXPECTED RESULTS: no background color on the printed output.
ACTUAL RESULTS:  printed output has the background color I specified in step 2

Mozilla/5.0 (X11; Linux x86_64; rv:6.0a1) Gecko/20110426 Firefox/6.0a1

This was originally reported in bug 498787 comment 7 - 9.

Daniel Holbert [:dholbert]

Reporter

Updated

•

13 years ago

OS: Linux → All

Hardware: x86_64 → All

Alice0775 White

Comment 1

•

13 years ago

Works:
http://hg.mozilla.org/mozilla-central/rev/01fa971e62ee
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b5pre) Gecko/20100827 Firefox/4.0b5pre ID:20100827190212
Fails:
http://hg.mozilla.org/mozilla-central/rev/0886ad6e6aaa
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b5pre) Gecko/20100827 Firefox/4.0b5pre ID:20100827190555
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=01fa971e62ee&tochange=0886ad6e6aaa
Triggered by;
0886ad6e6aaa	Robert O'Callahan — Bug 584193. Allow print preview to work without its own widget. r=smaug a=blocking beta5

Blocks: 584193

Daniel Holbert [:dholbert]

Reporter

Comment 2

•

13 years ago

Alice0775, you're a rockstar. :)

Just to be clear -- did you verify that this was triggered by 0886ad6e6aaa and not another cset from that push?  e.g. this one also looks suspicious:
be60f89d06ba	Robert O'Callahan — Bug 574621. Ensure that the prescontext
DefaultBackgroundColor is painted on the background of root content documents.

Daniel Holbert [:dholbert]

Reporter

Updated

•

13 years ago

Keywords: regression

Alice0775 White

Comment 3

•

13 years ago

                                Print Background (colors & images)
                                unchecked              checked
m-c hourly build 0886ad6e6aaa : user background color  user&site background color 
local build from 7804b5cf4313 : no background color    no background color
local build from be60f89d06ba : no background color    no background color

So, I guess c-set 0886ad6e6aaa triggered the issue.

Timothy Nikkel (:tnikkel)

Assignee

Updated

•

13 years ago

Assignee: nobody → tnikkel

Yeti

Comment 4

•

13 years ago

I'd just like to add it still happens in Seamonkey latest nightly (Mozilla/5.0 (Windows NT 5.1; rv:7.0a2) Gecko/20110712 Firefox/7.0a2 SeaMonkey/2.4a2).
Prints out color set up as "window background" in Win control panel when
printing any text-only e-mail message (I didn't set any custom color in the SM prefs). Reproducible always. Very annoying.
Started happening since upgrade to Seamonkey 2.1 IIRC.

tracking-firefox7: --- → ?

Christopher Blizzard (:blizzard)

Comment 5

•

13 years ago

Tracking for Firefox 7.

tracking-firefox7: ? → +

Bernard Jurgiel

Comment 6

•

13 years ago

Using Thunderbird 5.0, OS is Windows Vista Home Premium.

email background color is set to a "non-white" selection (i.e. cyan).

In page Setup, under Format & Options tab - under Options, the "Print Background (colors & images)" is unchecked.  However, when email is printed, the background color (i.e. cyan) is printed.  Tried checking the box, and printing; the background color still prints.

Note: the Background color was set in Tools - Options - Display.

Wayne Mery (:wsmwk)

Comment 7

•

13 years ago

this will be more than an annoyance for corporate types (my context=emails)
=> major

Blocks: 674211

Severity: normal → major

Wayne Mery (:wsmwk)

Comment 9

•

13 years ago

it's rather sad that this regressed (if bug 584193 is the regressor) almost a year ago, and it was never caught.

Flags: in-testsuite?

Flags: in-litmus?

tom

Comment 10

•

13 years ago

Thanks for working on this. The cost of having a dark background print on every print page is devastating!

Asa Dotzler [:asa]

Comment 12

•

13 years ago

not going to track this for 7.

tracking-firefox7: + → -

Álvaro González

Comment 13

•

13 years ago

Not sure if it's relevant but I've just found out that the background-color CSS property is not being fully ignored (tested in Firefox/6.0).

Bug doesn't happen (background colour is *not* printed) if the site uses:

    - background-color: white;
    - background-color: red;

The bug is triggered (default system colour is used) if:

    - background-color: transparent;
    - background-color is not set

Wayne Mery (:wsmwk)

Updated

•

13 years ago

URL: http://ftp.mozilla.org/pub/mozilla.or... → http://getsatisfaction.com/mozilla_me...

Bernard Jurgiel

Comment 14

•

13 years ago

Still a problem with Thunderbird 6.0.1.

If the email background is not white then the background color prints no matter what print preference is set.

Timothy Nikkel (:tnikkel)

Assignee

Comment 15

•

13 years ago

Attached patch patch (obsolete) — Details — Splinter Review

In nsCSSRendering::DetermineBackgroundColor when we are asked not to draw background colors we still draw white and not the default background color. This patch basically extends that logic to the special handling of the canvas background color we have in the presshell.

Attachment #557763 - Flags: review?(roc)

Robert O'Callahan (:roc) (email my personal email if necessary)

Updated

•

13 years ago

Attachment #557763 - Flags: review?(roc) → review+

Timothy Nikkel (:tnikkel)

Assignee

Comment 16

•

13 years ago

Pushed to inbound
http://hg.mozilla.org/integration/mozilla-inbound/rev/a38f82d29d97
but it bounched because of reftest failures:
http://hg.mozilla.org/integration/mozilla-inbound/rev/dc30ee4aca21

Timothy Nikkel (:tnikkel)

Assignee

Comment 17

•

13 years ago

Attached patch patch — Details — Splinter Review

The previous patch was wrong because DetermineBackgroundColor sets aDrawBackgroundColor to false even if we are not printing or are drawing background colors and the background color is completely transparent. So the logic of the previous patch was completely wrong.

Attachment #557763 - Attachment is obsolete: true

Attachment #558839 - Flags: review?(roc)

Robert O'Callahan (:roc) (email my personal email if necessary)

Updated

•

13 years ago

Attachment #558839 - Flags: review?(roc) → review+

Timothy Nikkel (:tnikkel)

Assignee

Comment 18

•

13 years ago

http://hg.mozilla.org/integration/mozilla-inbound/rev/9b35a07f21b1

Whiteboard: [inbound]

(no longer active)

Comment 19

•

13 years ago

http://hg.mozilla.org/mozilla-central/rev/9b35a07f21b1

Status: NEW → RESOLVED

Closed: 13 years ago

Resolution: --- → FIXED

Whiteboard: [inbound]

Target Milestone: --- → mozilla9

Paul Mason

Comment 21

•

13 years ago

Well it looks like someone bug-it-up again in 6.0.2 as theckgound colour is now printing in an enmail as I type this

Paul Mason

Comment 22

•

13 years ago

So it's back to a white background and more bloody eyestrain and headaches.

IF it is not broken why fix it? 

In this case it was fixed but it is broken again.

This is getting dam anoying and looks as if some one is trying to force us to have printed background colours regardless as per his/her request some time ago

Timothy Nikkel (:tnikkel)

Assignee

Comment 23

•

13 years ago

The fix isn't in 6.0.2. You'd need to be using a nightly build dated 2011-09-09 or newer (no such build exists at this time).

Paul Mason

Comment 24

•

13 years ago

(In reply to Timothy Nikkel (:tn) from comment #23)
> The fix isn't in 6.0.2. You'd need to be using a nightly build dated
> 2011-09-09 or newer (no such build exists at this time).

Well this is what it says in The "HELP ~ About Thunderbird"  

Thunberbird
6.0.2
Thunderbird is up to date
You are currently on the release update channel

etc.

So if no build exist why is thunderbird telling me this 

'not on nightly builds

Troble shooting information sent to bugzilla-daemon@mozilla.org as well as a screen shot of the "HELP ~ About Thunderbird" as well

Paul Mason

Comment 25

•

13 years ago


  Application Basics

    Name: Thunderbird
    Version: 6.0.2
    User Agent: Mozilla/5.0 (Windows NT 6.1; rv:6.0.2) Gecko/20110902 Thunderbird/6.0.2
    Profile Directory: Open Containing Folder

              (Local drive)
    Application Build ID: 20110902221921
    Enabled Plugins: about:plugins
    Build Configuration: about:buildconfig

  Mail and News Accounts
    account1:
      INCOMING: account1, , (pop3) pop.mail.yahoo.com:110, plain, passwordCleartext
      OUTGOING: mail.btinternet.com:25, plain, passwordCleartext, true

    account2:
      INCOMING: account2, , (none) Local Folders, plain, passwordCleartext

    account3:
      INCOMING: account3, , (rss) Feeds, plain, passwordCleartext

    account5:
      INCOMING: account5, , (pop3) pop.mail.yahoo.com:110, plain, passwordCleartext
      OUTGOING: mail.btinternet.com:25, plain, passwordCleartext, true

  Extensions
    British English Dictionary, 1.19.1, true, en-GB@dictionaries.addons.mozilla.org
    Priority Switcher, 0.6.13, true, {0D2172E4-C5AE-465A-B80D-53A840275B5E}

  Modified Preferences

    Name: Value

      extensions.lastAppVersion: 6.0.2
      font.name.monospace.el: Consolas
      font.name.monospace.tr: Consolas
      font.name.monospace.x-baltic: Consolas
      font.name.monospace.x-central-euro: Consolas
      font.name.monospace.x-cyrillic: Consolas
      font.name.monospace.x-unicode: Consolas
      font.name.monospace.x-western: Cambria
      font.name.sans-serif.el: Calibri
      font.name.sans-serif.tr: Calibri
      font.name.sans-serif.x-baltic: Calibri
      font.name.sans-serif.x-central-euro: Calibri
      font.name.sans-serif.x-cyrillic: Calibri
      font.name.sans-serif.x-unicode: Calibri
      font.name.sans-serif.x-western: Times New Roman
      font.name.serif.el: Cambria
      font.name.serif.tr: Cambria
      font.name.serif.x-baltic: Cambria
      font.name.serif.x-central-euro: Cambria
      font.name.serif.x-cyrillic: Cambria
      font.name.serif.x-unicode: Cambria
      font.name.serif.x-western: Courier New
      font.size.fixed.el: 14
      font.size.fixed.tr: 14
      font.size.fixed.x-baltic: 14
      font.size.fixed.x-central-euro: 14
      font.size.fixed.x-cyrillic: 14
      font.size.fixed.x-unicode: 14
      font.size.fixed.x-western: 16
      font.size.variable.el: 17
      font.size.variable.tr: 17
      font.size.variable.x-baltic: 17
      font.size.variable.x-central-euro: 17
      font.size.variable.x-cyrillic: 17
      font.size.variable.x-unicode: 17
      mail.openMessageBehavior.version: 1
      mail.winsearch.firstRunDone: true
      network.cookie.lifetimePolicy: 2
      network.cookie.prefsMigrated: true
      places.database.lastMaintenance: 1315482482
      places.history.expiration.transient_current_max_pages: 42219
      print.print_printer: Adobe PDF
      print.printer_Adobe_PDF.print_bgcolor: false
      print.printer_Adobe_PDF.print_bgimages: false
      print.printer_Adobe_PDF.print_command:
      print.printer_Adobe_PDF.print_downloadfonts: false
      print.printer_Adobe_PDF.print_edge_bottom: 0
      print.printer_Adobe_PDF.print_edge_left: 0
      print.printer_Adobe_PDF.print_edge_right: 0
      print.printer_Adobe_PDF.print_edge_top: 0
      print.printer_Adobe_PDF.print_evenpages: true
      print.printer_Adobe_PDF.print_footercenter:
      print.printer_Adobe_PDF.print_footerleft: &PT
      print.printer_Adobe_PDF.print_footerright: &D
      print.printer_Adobe_PDF.print_headercenter:
      print.printer_Adobe_PDF.print_headerleft: &T
      print.printer_Adobe_PDF.print_headerright: &U
      print.printer_Adobe_PDF.print_in_color: true
      print.printer_Adobe_PDF.print_margin_bottom: 0.5
      print.printer_Adobe_PDF.print_margin_left: 0.5
      print.printer_Adobe_PDF.print_margin_right: 0.5
      print.printer_Adobe_PDF.print_margin_top: 0.5
      print.printer_Adobe_PDF.print_oddpages: true
      print.printer_Adobe_PDF.print_orientation: 0
      print.printer_Adobe_PDF.print_page_delay: 50
      print.printer_Adobe_PDF.print_paper_data: 9
      print.printer_Adobe_PDF.print_paper_height: 11.00
      print.printer_Adobe_PDF.print_paper_size_type: 0
      print.printer_Adobe_PDF.print_paper_size_unit: 1
      print.printer_Adobe_PDF.print_paper_width: 8.50
      print.printer_Adobe_PDF.print_reversed: false
      print.printer_Adobe_PDF.print_scaling: 1.00
      print.printer_Adobe_PDF.print_shrink_to_fit: false
      print.printer_Adobe_PDF.print_to_file: false
      print.printer_Adobe_PDF.print_unwriteable_margin_bottom: 0
      print.printer_Adobe_PDF.print_unwriteable_margin_left: 0
      print.printer_Adobe_PDF.print_unwriteable_margin_right: 0
      print.printer_Adobe_PDF.print_unwriteable_margin_top: 0
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_bgcolor: false
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_bgimages: false
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_command:
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_downloadfonts: false
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_edge_bottom: 0
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_edge_left: 0
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_edge_right: 0
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_edge_top: 0
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_evenpages: true
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_footercenter:
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_footerleft: &PT
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_footerright: &D
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_headercenter:
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_headerleft: &T
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_headerright: &U
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_in_color: true
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_margin_bottom: 0.5
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_margin_left: 0.5
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_margin_right: 0.5
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_margin_top: 0.5
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_oddpages: true
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_orientation: 0
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_pagedelay: 500
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_paper_data: 9
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_paper_height: 11.00
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_paper_size_type: 0
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_paper_size_unit: 1
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_paper_width: 8.50
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_reversed: false
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_scaling: 1.00
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_shrink_to_fit: false
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_to_file: false
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_unwriteable_margin_bottom: 0
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_unwriteable_margin_left: 0
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_unwriteable_margin_right: 0
      print.printer_MINOLTA-QMS_PagePro_1100_(PCL).print_unwriteable_margin_top: 0
      security.disable_button.openCertManager: false
      security.disable_button.openDeviceManager: false
      security.OCSP.disable_button.managecrl: false

  Graphics

    Adapter Description: Mobile Intel(R) 4 Series Express Chipset Family
    Vendor ID: 8086
    Device ID: 2a42
    Adapter RAM: Unknown
    Adapter Drivers: igdumdx32 igd10umd32
    Driver Version: 8.15.10.2302
    Driver Date: 2-11-2011
    Direct2D Enabled: true
    DirectWrite Enabled: true (6.1.7601.17563)
    WebGL Renderer: Google Inc. -- ANGLE -- OpenGL ES 2.0 (ANGLE 0.0.0.686)
    GPU Accelerated Windows: 3/3 Direct3D 10

  For some reason the Troubleshooting Information is shown the wrong printer a defaultas well the default printer should be Epson D92

Wayne Mery (:wsmwk)

Comment 26

•

13 years ago

(In reply to Paul Mason from comment #24)
> (In reply to Timothy Nikkel (:tn) from comment #23)
> > The fix isn't in 6.0.2. You'd need to be using a nightly build dated
> > 2011-09-09 or newer (no such build exists at this time).
> 
> Well this is what it says in The "HELP ~ About Thunderbird"  
> 
> Thunberbird
> 6.0.2
> Thunderbird is up to date
> You are currently on the release update channel

Paul, You've been around long enough in bugzilla that you should know that patches don't appear immediately in release builds - it may take some weeks or months for this fix to appear in a release. Please stop spamming this bug.

As for the printer and troubleshooting info, please file a new bug

Paul Mason

Comment 27

•

13 years ago

(In reply to Wayne Mery (:wsmwk) from comment #26)
> (In reply to Paul Mason from comment #24)
> (In reply to Timothy Nikkel
> (:tn) from comment #23)
> > The fix isn't in 6.0.2. You'd need to be using a
> nightly build dated
> > 2011-09-09 or newer (no such build exists at this
> time).
> 
> Well this is what it says in The "HELP ~ About Thunderbird"  
> 
> > Thunberbird
> 6.0.2
> Thunderbird is up to date
> You are currently on the
> release update channel

Paul, You've been around long enough in bugzilla
> that you should know that patches don't appear immediately in release builds
> - it may take some weeks or months for this fix to appear in a release.
> Please stop spamming this bug.

As for the printer and troubleshooting info,
> please file a new bug

Ah BUT this was fixed in 6.0.1 so who buged it up AGAIN in 6.0.2

John Liebson

Comment 28

•

13 years ago

This bug is not fixed: The problem remains with Thunderbird 7.0, running under Windows Home Premium.

Daniel Holbert [:dholbert]

Reporter

Comment 29

•

13 years ago

This is fixed in Gecko 9.0 (see the "Target Milestone" field), which means the first fixed releases will be Firefox 9.0 and Thunderbird 9.0.

John Liebson

Comment 30

•

13 years ago

In reply to Comment 29, thank you very much for clearing up my confusion: I did not realize that the, while the thread seemed to refer only to Firefox, in also covers TB. I shall modify my last comment in MozillaZine, adding your comment, to indicate that, despite my confusion, the problem is known not to have been solved as of yet, but shall be in the future.

Paul Mason

Comment 31

•

13 years ago

(In reply to John Liebson from comment #28)
> This bug is not fixed: The problem remains with Thunderbird 7.0, running
> under Windows Home Premium.

(In reply to Daniel Holbert [:dholbert] from comment #29)
> This is fixed in Gecko 9.0 (see the "Target Milestone" field), which means
> the first fixed releases will be Firefox 9.0 and Thunderbird 9.0.

And if this is not fixed by Thunderbird 9.0 do we all go back to good old windows mail or wait it out till thunberbird 2000

Paul Mason

Comment 32

•

13 years ago

(In reply to John Liebson from comment #28)
> This bug is not fixed: The problem remains with Thunderbird 7.0, running
> under Windows Home Premium.

(In reply to Daniel Holbert [:dholbert] from comment #29)
> This is fixed in Gecko 9.0 (see the "Target Milestone" field), which means
> the first fixed releases will be Firefox 9.0 and Thunderbird 9.0.

And if this is not fixed by Thunderbird 9.0 do we all go back to good old windows mail or wait it out till thunberbird 2000

bugzilla

Comment 33

•

13 years ago

As expected, this bug also occurs in the current Seamonkey 2.4.

Seamonkey 2.5 will be using Gecko 7 (correct me if I'm wrong), Gecko 9 seems far away. Which will be the first Seamonkey to use Gecko 9?
https://developer.mozilla.org/en/Gecko is a bit outdated.

K Chayka

Comment 34

•

13 years ago

(In reply to bugzilla from comment #33)
> Seamonkey 2.5 will be using Gecko 7 (correct me if I'm wrong), Gecko 9 seems
> far away. Which will be the first Seamonkey to use Gecko 9?

I don't know about Gecko 9, but the bug looks fixed in recent Seamonkey nightly builds. I'm currently using SeaMonkey/2.7a1

patch 13 years ago Timothy Nikkel (:tnikkel) 6.30 KB, patch	roc : review+	Details \| Diff \| Splinter Review
patch 13 years ago Timothy Nikkel (:tnikkel) 3.41 KB, patch	roc : review+	Details \| Diff \| Splinter Review