1190344 - Composer print/preview locks-up at the progress-pane stage

Status: RESOLVED FIXED

(Toolkit :: Printing, defect)

Description

[Ian Neal]

From Bug 1142997 comment 21
2015-08-01 Nightly Build from <http://ftp.mozilla.org/pub/mozilla.org/seamonkey/nightly/latest-comm-central-trunk/> build. Mozilla/5.0 (X11; Linux x86_64; rv:42.0) Gecko/20100101 Firefox/42.0 SeaMonkey/2.39a1 ID:20150801003001 CSet: 85f81472a34c

Composer print/preview locks-up at the progress-pane stage-
Error Console produces this eror:-

Timestamp: 03/08/15 14:45:01
Error: [Exception... "Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [nsIWebBrowserPrint.printPreview]" nsresult: "0x80070057 (NS_ERROR_ILLEGAL_VALUE)"  location: "JS frame :: chrome://global/content/browser-content.js :: enterPrintPreview :: line 466"  data: no]
Source File: chrome://global/content/browser-content.js
Line: 470

Also, I just tried the print/print-preview from the "new-message" compose / Editor pane, and got the identical error again. Works fine for received messages, so is similar to this bug if I'm wrong.

Similar error for New Message (Compose/Editor)
Timestamp: 03/08/15 15:07:50
Error: [Exception... "Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [nsIWebBrowserPrint.printPreview]" nsresult: "0x80070057 (NS_ERROR_ILLEGAL_VALUE)"  location: "JS frame :: chrome://global/content/browser-content.js :: enterPrintPreview :: line 466"  data: no]
Source File: chrome://global/content/browser-content.js
Line: 470

Comment 1

[Barry Edwin GIlmour]

Built from http://hg.mozilla.org/mozilla-central/rev/aeb85029c3b3

Comment 2

[Hartmut Figge]

Regression window:
Last good: 2015-07-09 14:28:00 PDT   c-c:410e376f4a32 m-c:95a40a305e1b
First bad: 2015-07-10 15:09:00 PDT   c-c:f20ad5d03baa m-c:d8df07a11ff7

Comment 3

[Ian Neal]

(In reply to Hartmut Figge from comment #2)
> Regression window:
> Last good: 2015-07-09 14:28:00 PDT   c-c:410e376f4a32 m-c:95a40a305e1b
> First bad: 2015-07-10 15:09:00 PDT   c-c:f20ad5d03baa m-c:d8df07a11ff7

Which is when the fix to bug 1146454 landed on m-c: https://hg.mozilla.org/mozilla-central/rev/dfd2a128d19c

Comment 4

[Rainer Bielefeld]

I have some difficulties to reproduce the problem, see <https://bugzilla.mozilla.org/show_bug.cgi?id=1190348#c3>

Does "print/preview" mean
a) 'File → Print...' and also 'File → Print Preview' 
   or
b) 'File → Print Preview' (only)?

Comment 5

[Barry Edwin GIlmour]

My bad! Should be Print and print-preview via Editor (Web-page-Composer and Message-Compose windows)

"print/preview" means;-
a) 'File → Print...' and also 'File → Print Preview'

Comment 6

[Mike Conley (:mconley) (:⚙️)]

Can somebody point me to the code that's calling into printUtils.printPreview?

Comment 7

[Hartmut Figge]

Attachment: Occurrence of printUtils.printPreview in the source of SM-Trunk

(In reply to Mike Conley (:mconley) - Needinfo me! from comment #6)
> Can somebody point me to the code that's calling into
> printUtils.printPreview?

pUpP.txt holds the result of 'grep  -ri --exclude-dir .hg  printUtils.printPreview src/' for the source of SM-Trunk. Does this help?

Comment 8

[Mike Conley (:mconley) (:⚙️)]

Attachment: MozReview Request: Bug 1190344 - Add outerWindowID read-only property to editor binding. r?ehsan

Bug 1190344 - Add outerWindowID read-only property to editor binding. r?ehsan

Comment 9

[Mike Conley (:mconley) (:⚙️)]

Can somebody with a SeaMonkey build off of the latest mozilla-central go into the mozilla/ subdirectory and run:

hg pull -r 1a0d3b1b3c5cab4a70c7c1d5efc912e284aaab2f https://reviewboard-hg.mozilla.org/gecko/

Then update to tip, and build, and see if this fixes it?

Comment 10

[Hartmut Figge]

Well for me the easiest way to test was to implement the changes manually in the omni.ja of my current SM-Trunk. And yes, it works. :)

Comment 11

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 8643149 [details]
MozReview Request: Bug 1190344 - Add outerWindowID read-only property to editor binding. r?ehsan

Bug 1190344 - Add outerWindowID read-only property to editor binding. r?ehsan

Comment 12

[(no longer active)]

Comment on attachment 8643149 [details]
MozReview Request: Bug 1190344 - Add outerWindowID read-only property to editor binding. r?ehsan

https://reviewboard.mozilla.org/r/14995/#review13479

Ship It!

Comment 13

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/402f6265001b

Comment 14

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/402f6265001b