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

Unified Diff: chrome/browser/printing/printing_message_filter.cc

Issue 23116003: Adds PrintingContext implementation stub for Android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 4 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
Index: chrome/browser/printing/printing_message_filter.cc
diff --git a/chrome/browser/printing/printing_message_filter.cc b/chrome/browser/printing/printing_message_filter.cc
index 35b4074328de8ba4ec5e2cdce26e9c18c0654bf4..afaac36d606bda4a69c5dadee189e816789269ed 100644
--- a/chrome/browser/printing/printing_message_filter.cc
+++ b/chrome/browser/printing/printing_message_filter.cc
@@ -32,6 +32,10 @@
#include "chrome/browser/printing/print_dialog_cloud.h"
#endif
+#if defined(OS_ANDROID)
+#include "chrome/browser/printing/print_view_manager_basic.h"
+#endif
+
using content::BrowserThread;
namespace {
@@ -100,6 +104,11 @@ void PrintingMessageFilter::OverrideThreadForMessage(
message.type() == PrintHostMsg_TempFileForPrintingWritten::ID) {
*thread = BrowserThread::FILE;
}
+#elif defined(OS_ANDROID)
+ if (message.type() == PrintHostMsg_AllocateTempFileForPrinting::ID ||
+ message.type() == PrintHostMsg_TempFileForPrintingWritten::ID) {
+ *thread = BrowserThread::UI;
+ }
#endif
}
@@ -110,7 +119,7 @@ bool PrintingMessageFilter::OnMessageReceived(const IPC::Message& message,
#if defined(OS_WIN)
IPC_MESSAGE_HANDLER(PrintHostMsg_DuplicateSection, OnDuplicateSection)
#endif
-#if defined(OS_CHROMEOS)
+#if defined(OS_CHROMEOS) || defined(OS_ANDROID)
IPC_MESSAGE_HANDLER(PrintHostMsg_AllocateTempFileForPrinting,
OnAllocateTempFileForPrinting)
IPC_MESSAGE_HANDLER(PrintHostMsg_TempFileForPrintingWritten,
@@ -141,9 +150,12 @@ void PrintingMessageFilter::OnDuplicateSection(
}
#endif
-#if defined(OS_CHROMEOS)
+#if defined(OS_CHROMEOS) || defined(OS_ANDROID)
void PrintingMessageFilter::OnAllocateTempFileForPrinting(
- base::FileDescriptor* temp_file_fd, int* sequence_number) {
+ int render_view_id,
+ base::FileDescriptor* temp_file_fd,
+ int* sequence_number) {
+#if defined(OS_CHROMEOS)
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
temp_file_fd->fd = *sequence_number = -1;
temp_file_fd->auto_close = false;
@@ -166,10 +178,19 @@ void PrintingMessageFilter::OnAllocateTempFileForPrinting(
}
}
}
+#elif defined(OS_ANDROID)
+ DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+ content::WebContents* wc = GetWebContentsForRenderView(render_view_id);
+ printing::PrintViewManagerBasic* print_view_manager =
+ printing::PrintViewManagerBasic::FromWebContents(wc);
+ temp_file_fd->auto_close = false;
+ temp_file_fd->fd = print_view_manager->GetFileDescriptor().fd;
whywhat 2013/08/15 20:19:56 Check that GetFileDescritor() is not NULL?
+#endif
}
void PrintingMessageFilter::OnTempFileForPrintingWritten(int render_view_id,
int sequence_number) {
+#if defined(OS_CHROMEOS)
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
SequenceToPathMap* map = &g_printing_file_descriptor_map.Get().map;
SequenceToPathMap::iterator it = map->find(sequence_number);
@@ -185,8 +206,11 @@ void PrintingMessageFilter::OnTempFileForPrintingWritten(int render_view_id,
// Erase the entry in the map.
map->erase(it);
+#endif
}
+#endif // defined(OS_CHROMEOS) || defined(OS_ANDROID)
+#if defined(OS_CHROMEOS)
void PrintingMessageFilter::CreatePrintDialogForFile(
int render_view_id,
const base::FilePath& path) {

Powered by Google App Engine
This is Rietveld 408576698