Implement window.print() on Android

printing/android/java/src/org/chromium/printing/PrintingContext.java

Index: printing/android/java/src/org/chromium/printing/PrintingContext.java
diff --git a/printing/android/java/src/org/chromium/printing/PrintingContext.java b/printing/android/java/src/org/chromium/printing/PrintingContext.java
index e50f0ce8fa244e9220781c59838ac8a005eb6ac8..a5acb122f02231921333d6eb0c393da911893eb7 100644
--- a/printing/android/java/src/org/chromium/printing/PrintingContext.java
+++ b/printing/android/java/src/org/chromium/printing/PrintingContext.java
@@ -97,6 +97,17 @@ public class PrintingContext implements PrintingContextInterface {
     }
 
     @CalledByNative
+    public void showPrintDialog() {
+        ThreadUtils.assertOnUiThread();
+        if (mController != null) {

[Vitaly Buka (NO REVIEWS), 2014/12/12 08:37:17]

why do you check mController here but it's not checked in the rest of class?

[dgn, 2014/12/12 16:49:44]

When enterprise policies disable printing, mController will be null. All the
code from the class was called only from java before, through menu items that
are only available when printing is enabled. So mController was always non null
in these cases.

+            mController.startPendingPrint(this);
+        } else {
+            // Printing disabled. Notify the native side to stop waiting.
+            showSystemDialogDone();
+        }
+    }
+
+    @CalledByNative
     public static void pdfWritingDone(int fd, boolean success) {
         ThreadUtils.assertOnUiThread();
         // TODO(cimamoglu): Do something when fd == -1.
@@ -129,7 +140,14 @@ public class PrintingContext implements PrintingContextInterface {
         }
     }
 
+    @Override
+    public void showSystemDialogDone() {
+        nativeShowSystemDialogDone(mNativeObject);
+    }
+
     private native void nativeAskUserForSettingsReply(
             long nativePrintingContextAndroid,
             boolean success);
+
+    private native void nativeShowSystemDialogDone(long nativePrintingContextAndroid);
 }