Chromium Code Reviews| Index: printing/android/java/src/org/chromium/printing/PrintingControllerImpl.java |
| diff --git a/printing/android/java/src/org/chromium/printing/PrintingControllerImpl.java b/printing/android/java/src/org/chromium/printing/PrintingControllerImpl.java |
| index 16e045616abde9cf6d3c758e141c17de4e057b9a..32606130a1f51de0039563262a97a45cb3570d06 100644 |
| --- a/printing/android/java/src/org/chromium/printing/PrintingControllerImpl.java |
| +++ b/printing/android/java/src/org/chromium/printing/PrintingControllerImpl.java |
| @@ -10,6 +10,7 @@ import android.os.ParcelFileDescriptor; |
| import android.print.PageRange; |
| import android.print.PrintAttributes; |
| import android.print.PrintDocumentInfo; |
| +import android.util.Log; |
| import org.chromium.base.ThreadUtils; |
| import org.chromium.printing.PrintDocumentAdapterWrapper.PdfGenerator; |
| @@ -164,12 +165,15 @@ public class PrintingControllerImpl implements PrintingController, PdfGenerator |
| @Override |
| public void setPrintingContext(final PrintingContextInterface printingContext) { |
| + Log.i("DGN", "setPrintingContext"); |
| mPrintingContext = printingContext; |
| } |
| @Override |
| public void startPrint(final Printable printable, PrintManagerDelegate printManager) { |
| + Log.i("DGN", "PrintingControllerImpl#startPrint" + (mIsBusy ? " (BUSY)" : "")); |
| if (mIsBusy) return; |
| + |
| mIsBusy = true; |
| mPrintable = printable; |
| mPrintDocumentAdapterWrapper.print(printManager, printable.getTitle()); |
| @@ -202,6 +206,8 @@ public class PrintingControllerImpl implements PrintingController, PdfGenerator |
| CancellationSignal cancellationSignal, |
| PrintDocumentAdapterWrapper.LayoutResultCallbackWrapper callback, |
| Bundle metadata) { |
| + Log.i("DGN", "PrintingControllerImpl#onLayout"); |
| + |
| // NOTE: Chrome printing just supports one DPI, whereas Android has both vertical and |
| // horizontal. These two values are most of the time same, so we just pass one of them. |
| mDpi = newAttributes.getResolution().getHorizontalDpi(); |
| @@ -268,6 +274,8 @@ public class PrintingControllerImpl implements PrintingController, PdfGenerator |
| return; |
| } |
| + Log.i("DGN", "PrintingControllerImpl#onWrite"); |
| + |
| // TODO(cimamoglu): Make use of CancellationSignal. |
| mOnWriteCallback = callback; |
| @@ -303,6 +311,8 @@ public class PrintingControllerImpl implements PrintingController, PdfGenerator |
| @Override |
| public void onFinish() { |
| + Log.i("DGN", "PrintingControllerImpl#onFinish"); |
| + |
| mLastKnownMaxPages = PrintDocumentInfo.PAGE_COUNT_UNKNOWN; |
| mPages = null; |
| @@ -319,6 +329,7 @@ public class PrintingControllerImpl implements PrintingController, PdfGenerator |
| mPrintingContext.askUserForSettingsReply(false); |
| } |
| mPrintingContext.updatePrintingContextMap(mFileDescriptor, true); |
| + mPrintingContext.ShowSystemDialogDone(); |
|
dgn
2014/11/25 17:07:42
The call in this line is useless because the print
|
| mPrintingContext = null; |
| } |
| mPrintingState = PRINTING_STATE_FINISHED; |