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

Unified Diff: chrome/renderer/printing/chrome_print_web_view_helper_delegate.cc

Issue 858363002: OOP PDF: Override printing for MimeHandlerViewContainers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@mime-handler-api-simpler
Patch Set: fix android compile and remove the test as it hits a probably dodgy assert Created 5 years, 11 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/renderer/printing/chrome_print_web_view_helper_delegate.cc
diff --git a/chrome/renderer/printing/chrome_print_web_view_helper_delegate.cc b/chrome/renderer/printing/chrome_print_web_view_helper_delegate.cc
index 6eb3881661690cd3486cfdb8d93049a05e2a97e3..576bb15aa42a3a413a4d7fa58cf6b8cf147d15af 100644
--- a/chrome/renderer/printing/chrome_print_web_view_helper_delegate.cc
+++ b/chrome/renderer/printing/chrome_print_web_view_helper_delegate.cc
@@ -4,10 +4,13 @@
#include "chrome/renderer/printing/chrome_print_web_view_helper_delegate.h"
+#include <vector>
+
#include "base/command_line.h"
#include "chrome/common/chrome_switches.h"
#include "chrome/common/render_messages.h"
#include "chrome/renderer/prerender/prerender_helper.h"
+#include "content/public/renderer/render_frame.h"
#include "content/public/renderer/render_view.h"
#include "ipc/ipc_message.h"
#include "third_party/WebKit/public/web/WebDocument.h"
@@ -17,6 +20,7 @@
#if defined(ENABLE_EXTENSIONS)
#include "chrome/common/extensions/extension_constants.h"
#include "extensions/common/constants.h"
+#include "extensions/renderer/guest_view/mime_handler_view/mime_handler_view_container.h"
#endif // defined(ENABLE_EXTENSIONS)
ChromePrintWebViewHelperDelegate::~ChromePrintWebViewHelperDelegate(){
@@ -59,3 +63,26 @@ bool ChromePrintWebViewHelperDelegate::IsPrintPreviewEnabled() {
base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
return !command_line->HasSwitch(switches::kDisablePrintPreview);
}
+
+bool ChromePrintWebViewHelperDelegate::OverridePrint(
+ blink::WebLocalFrame* frame) {
+#if defined(ENABLE_EXTENSIONS)
+ if (!frame->document().isPluginDocument())
+ return false;
+
+ std::vector<extensions::MimeHandlerViewContainer*> mime_handlers =
+ extensions::MimeHandlerViewContainer::FromRenderFrame(
+ content::RenderFrame::FromWebFrame(frame));
+ if (!mime_handlers.empty()) {
+ // This message is handled in chrome/browser/resources/pdf/pdf.js and
+ // instructs the PDF plugin to print. This is to make window.print() on a
+ // PDF plugin document correctly print the PDF. See
+ // https://crbug.com/448720.
+ base::DictionaryValue message;
+ message.SetString("type", "print");
+ mime_handlers.front()->PostMessageFromValue(message);
+ return true;
+ }
+#endif // defined(ENABLE_EXTENSIONS)
+ return false;
+}
« no previous file with comments | « chrome/renderer/printing/chrome_print_web_view_helper_delegate.h ('k') | components/printing/renderer/print_web_view_helper.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698