Adds PrintingContext implementation stub for Android.

chrome/browser/printing/print_view_manager_basic.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/printing/print_view_manager_basic.h"
 
+#if defined(OS_ANDROID)
+#include "base/file_descriptor_posix.h"
+#include "chrome/common/print_messages.h"
+#include "printing/printing_context_android.h"
+#endif
+
 DEFINE_WEB_CONTENTS_USER_DATA_KEY(printing::PrintViewManagerBasic);
 
 namespace printing {
 
 PrintViewManagerBasic::PrintViewManagerBasic(content::WebContents* web_contents)
     : PrintViewManagerBase(web_contents) {
 }
 
 PrintViewManagerBasic::~PrintViewManagerBasic() {
 }
 
+#if defined(OS_ANDROID)
+void PrintViewManagerBasic::RenderProcessGone(base::TerminationStatus status) {

[Lei Zhang, 2013/08/18 09:56:26]

I think any RenderViewHost that prints will have a PrintViewManager. Even after
the printing has finished, when the tab closes, this may get called. If this
happens, please make sure that doesn't cause any weird side effects.

[Lei Zhang, 2013/08/19 21:02:15]

I didn't get a response to this, but I tested this and you'll get calls here for
every renderer that goes away. Assuming PdfWritingDone() can handle this, then
ok.

+  PrintingContextAndroid::PdfWritingDone(file_descriptor_.fd, false);
+  file_descriptor_ = base::FileDescriptor(-1, false);
+  PrintViewManagerBase::RenderProcessGone(status);
+}
+
+void PrintViewManagerBasic::OnPrintingFailed(int cookie) {
+  PrintingContextAndroid::PdfWritingDone(file_descriptor_.fd, false);
+  file_descriptor_ = base::FileDescriptor(-1, false);
+  PrintViewManagerBase::OnPrintingFailed(cookie);
+}
+
+bool PrintViewManagerBasic::OnMessageReceived(const IPC::Message& message) {
+  bool handled = true;
+  IPC_BEGIN_MESSAGE_MAP(PrintViewManagerBasic, message)
+    IPC_MESSAGE_HANDLER(PrintHostMsg_PrintingFailed, OnPrintingFailed)
+    IPC_MESSAGE_UNHANDLED(handled = false)
+  IPC_END_MESSAGE_MAP()
+
+  return handled ? true : PrintViewManagerBase::OnMessageReceived(message);

[whywhat, 2013/08/16 23:31:36]

nit: I find something like

if (handled)
  return true;
return PrintViewManagerBase::OnMessageReceived(message);

slightly more readable.

[Lei Zhang, 2013/08/18 09:56:26]

I think I suggested the ternary operator in the first place. I don't have a
strong preference.

+}
+#endif
+
 }  // namespace printing