140183 - [RTM]Add an attribute to PrintSettings that allows us silent printing

Status: VERIFIED FIXED

(Core :: Printing: Output, defect)

Description

[racham]

Adding summary from one of the emails exchanged between Rodney, Dan & myself.

The problem is the Progress dialog is intended to be non-modal, but
it becomes modal when parented to a model dialog. Once the dialog is modal, 
that's it, nothing can really print.

A different solution may be to add it to the PrintSettings, we now have a call 
for indicating "silent" as in don't disable
the Print Dialog, well maybe we add a attr that says we shouldn't display the 
Progress either. (I like this one low impact easy to implement, low risk)

Without this attribute, we can't really print when the window that triggers 
print progress dialog is a modal window. ISPs can also be blocked from having 
any terms and conditions they may want user to print from the customized wizard 
pages.

Rod has already provided a patch for this. Will post that in the next update.

Comment 1

[racham]

Added keywords.

Samir & Dan for input & reviews.

Comment 2

[racham]

Attachment: patch by [e-mail]

Comment 3

[racham]

Attachment: same patch with diff options -uw

Comment 4

[Samir Gehani]

Comment on attachment 81223 [details] [diff] [review]
same patch with diff options -uw

r=sgehani

Comment 5

[racham]

r=bhuvan

Comment 6

[(not reading, please use [e-mail] instead)]

Comment on attachment 81223 [details] [diff] [review]
same patch with diff options -uw

sr=sspitzer

Comment 7

[racham]

Fix checked in on trunk.

Adding adt1.0.0 keyword.

Comment 8

[scottputterman]

We can ship Beta without this.  But this is required for rtm so adding
appropriate keywords.

Comment 9

[Kevin McCluskey (gone)]

Resolving as fixed since it has been fixed on the trunk. Please add fixed1.0.0
keyword when the patch has been checked into the branch.

Comment 10

[sujay]

Racham, please verify when you get chance...thanks!

Comment 11

[racham]

code verified.

This one still need to make it's way onto the branch. So, will add correponding
keywords then.

Comment 12

[Syd Logan]

Adding adt1.0.0+ -- please get drivers approval and land in the branch.

Comment 13

[chris hofmann]

Comment on attachment 81223 [details] [diff] [review]
same patch with diff options -uw

a=chofmann,dbaron,valeski for the 1.0 branch

Comment 14

[sujay]

Racham, if you can verify this on branch also and add "verified1.0.0"
keyword...thanks!