Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(578)

Unified Diff: printing/android/java/src/org/chromium/printing/PrintingControllerImpl.java

Issue 740983002: Implement window.print() on Android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..eae57b688d582f0addc7d6710abf927fd53c609f 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;
@@ -169,7 +170,9 @@ public class PrintingControllerImpl implements PrintingController, PdfGenerator
@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 +205,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 +273,8 @@ public class PrintingControllerImpl implements PrintingController, PdfGenerator
return;
}
+ Log.i("DGN", "PrintingControllerImpl#onWrite");
+
// TODO(cimamoglu): Make use of CancellationSignal.
mOnWriteCallback = callback;
@@ -303,6 +310,8 @@ public class PrintingControllerImpl implements PrintingController, PdfGenerator
@Override
public void onFinish() {
+ Log.i("DGN", "PrintingControllerImpl#onFinish");
+
dgn 2014/11/20 12:26:16 Plan B is calling a nativeMethod there, that would
mLastKnownMaxPages = PrintDocumentInfo.PAGE_COUNT_UNKNOWN;
mPages = null;

Powered by Google App Engine
This is Rietveld 408576698