evince aborts printing when window is closed during creation of print job

Bug #139907 reported by Sebastian Urban
22
Affects Status Importance Assigned to Milestone
Evince
Fix Released
Medium
evince (Ubuntu)
Fix Released
Low
Ubuntu Desktop Bugs

Bug Description

Binary package hint: evince

steps to reproduce:

1. make sure your cpu load is at 100%
2. open a large document (> 100 pages) in evince
3. print document in evince
4. close evince immediately

result: document is not printed

expected result: document is printed

Related branches

Revision history for this message
Sebastian Urban (surban) wrote :

I have created a patch for this bug.

Evince was preparing the document for printing in a background job. This job was canceled when Evince was closed.
The patch solves this by
1) not canceling the print job when the document window is closed
2) hiding instead of deleting the document window, when there are still print jobs running.
(this is needed, because some printing code depends on the window)

Furthermore I found that printing setting may be associated with the wrong print job if the user prints the document multiple times with different settings within a short period of time (i.e. when the first print job is not finished yet). This is also fixed in this patch.

Additionally I changed the print job title from "evince_print" to the document name.

This is my first patch to an open source project, so tell me about my mistakes.

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for your bug and your work. The patch changes the upstream code and would be better placed on bugzilla.gnome.org which the software hackers are using. Do you want to send your patch there for comment or do you prefer to have somebody from Ubuntu doing that?

Changed in evince:
assignee: nobody → desktop-bugs
importance: Undecided → Low
Changed in evince:
status: New → Triaged
Revision history for this message
Sebastian Urban (surban) wrote :

I will send the patch myself.

Thank you!

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks, could you add a bug watch to the upstream bug once you have opened on gnome?

Revision history for this message
Sebastian Urban (surban) wrote :

already done ;)

Changed in evince:
status: Unknown → Confirmed
Revision history for this message
Thomas Perl (thp) wrote :

What is the status of this bug? It happened to me the last few days when printing very big PDF files that needed some time to process.

Revision history for this message
susscorfa (susscorfa) wrote :

I'm wondering as well what the status is of this bug since i'm still experiencing it as well. And i even dont have very large documents (10 page artikels) or high cpu loads

Changed in evince:
status: Confirmed → Fix Released
Revision history for this message
Sebastien Bacher (seb128) wrote :

the bug has been fixed upstream now

Changed in evince:
status: Triaged → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package evince - 2.25.4-0ubuntu1

---------------
evince (2.25.4-0ubuntu1) jaunty; urgency=low

  * New upstream version:
    New Features and UI Improvements:
    - Show progress information when loading/saving remote files
      (lp: #273611)
    - Remember page setup options (lp: #224882)
    - Show a confirmation dialog when there are pending print jobs
      while closing the main window (lp: #139907)
    - Show progress information when printing (lp: #178437)
    - Add document URI to properties page
    Bug fixes:
    - Bump glib requirement to fix build with older versions
    - Fix mime type handling
    - Fix saving images to remote locations
    - Do not try to jump to the location of the find result for every match, but
      only for the first one. It makes searching really faster.
    - Fix zip mime-type in comics backend
    - Fix several memory leaks
  * debian/control.in:
    - updated build requirement

 -- Sebastien Bacher <email address hidden> Tue, 06 Jan 2009 09:56:51 +0100

Changed in evince:
status: Fix Committed → Fix Released
Changed in evince:
importance: Unknown → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.