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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/printing/PrintingControllerFactory.java

Issue 184343013: Add a method to query whether printing is active. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressed code review Created 6 years, 9 months 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
« no previous file with comments | « no previous file | chrome/android/javatests/src/org/chromium/chrome/browser/printing/PrintingControllerTest.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/printing/PrintingControllerFactory.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/printing/PrintingControllerFactory.java b/chrome/android/java/src/org/chromium/chrome/browser/printing/PrintingControllerFactory.java
index 03b1198d2cda708ab164da3f336b95da28889ab2..bc9fdf1f1414d2a7c07125dd7a8217ba4d6bd4d4 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/printing/PrintingControllerFactory.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/printing/PrintingControllerFactory.java
@@ -5,13 +5,10 @@
package org.chromium.chrome.browser.printing;
import android.app.Activity;
-import android.content.Context;
-import android.print.PrintManager;
import org.chromium.base.ApiCompatibilityUtils;
import org.chromium.chrome.R;
import org.chromium.printing.PrintDocumentAdapterWrapper;
-import org.chromium.printing.PrintManagerDelegateImpl;
import org.chromium.printing.PrintingController;
import org.chromium.printing.PrintingControllerImpl;
@@ -26,11 +23,8 @@ public class PrintingControllerFactory {
String defaultJobTitle = activity.getResources().getString(R.string.menu_print);
TabPrinter.setDefaultTitle(defaultJobTitle);
- PrintManager printManager =
- (PrintManager) activity.getSystemService(Context.PRINT_SERVICE);
String errorText = activity.getResources().getString(R.string.error_printing_failed);
- return PrintingControllerImpl.create(new PrintManagerDelegateImpl(printManager),
- new PrintDocumentAdapterWrapper(), errorText);
+ return PrintingControllerImpl.create(new PrintDocumentAdapterWrapper(), errorText);
}
return null;
}
« no previous file with comments | « no previous file | chrome/android/javatests/src/org/chromium/chrome/browser/printing/PrintingControllerTest.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698