143248 - API Freeze - Add ProgressListener to Print,PP (Finalize Printing APIs)

Status: VERIFIED FIXED

(Core :: Printing: Output, defect)

Description

[rods (gone)]

Need tomake a last few additions to the PRinting/PrintPreview APIs before they
are frozen

Comment 1

[rods (gone)]

Attachment: patch with new methods stubbed out

Comment 2

[Conrad Carlen (not reading bugmail)]

 interface nsIPrintProgress: nsIWebProgressListener {
@@ -64,6 +63,9 @@
 
   /* Unregister a Web Progress Listener */
   void unregisterListener(in nsIWebProgressListener listener);
+  
+  /* Indicates we are done initing */

I think this is one place we could use some more comments.

+  void doneIniting();

With that, since the API change looks needed, r=ccarlen

Comment 3

[Marc Attinasi]

Comment on attachment 82925 [details] [diff] [review]
patch with new methods stubbed out

Hmm - DoneIniting... so, 'Initing' is a word? I guess 'Initializing' has been
deprecated  ;-)

sr=attinasi

Comment 4

[rods (gone)]

Comment on attachment 82925 [details] [diff] [review]
patch with new methods stubbed out

marking patch with conrad's r=

Comment 5

[rods (gone)]

fixed

Comment 6

[rods (gone)]

Attachment: missed these

Comment 7

[rods (gone)]

Comment on attachment 83062 [details] [diff] [review]
missed these

wrong bug

Comment 8

[sujay]

Rod, can you verify this and mark verified-fixed? thanks.

Comment 9

[scottputterman]

adding adt1.0.0+.  Please get drivers approval and then check into the 1.0 branch.

Comment 10

[sujay]

marking verified...

Comment 11

[scottputterman]

changing to adt1.0.1+ for checkin to the 1.0 branch for the Mozilla1.0.1
milestone.  Please get drivers approval before checking in.

Comment 12

[Judson Valeski]

please land on the 1.0.1 branch. once there, remove the "mozilla1.0.1+" keyword,
and add the "fixed1.0.1" keyword.

Comment 13

[rods (gone)]

checked in on branch