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

Unified Diff: android_webview/browser/aw_print_manager.h

Issue 2426503002: Make printing work better with OOPIF. (Closed)
Patch Set: Fix build, fix some tests Created 4 years, 2 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: android_webview/browser/aw_print_manager.h
diff --git a/android_webview/browser/aw_print_manager.h b/android_webview/browser/aw_print_manager.h
index 0166896804b734b5e21a38ad5720abfdf3b792f2..71b4e2f977c299bd3a7776f9f19d75ef5b198fb4 100644
--- a/android_webview/browser/aw_print_manager.h
+++ b/android_webview/browser/aw_print_manager.h
@@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef ANDROID_WEBVIEW_BROWSER_PRINT_MANAGER_H_
-#define ANDROID_WEBVIEW_BROWSER_PRINT_MANAGER_H_
+#ifndef ANDROID_WEBVIEW_BROWSER_AW_PRINT_MANAGER_H_
+#define ANDROID_WEBVIEW_BROWSER_AW_PRINT_MANAGER_H_
#include "base/macros.h"
#include "components/printing/browser/print_manager.h"
@@ -15,7 +15,7 @@ namespace android_webview {
class AwPrintManager : public printing::PrintManager,
public content::WebContentsUserData<AwPrintManager> {
public:
- // Creates a AwPrintManager for the provided webcontents. If the
+ // Creates an AwPrintManager for the provided WebContents. If the
// AwPrintManager already exists, it is destroyed and a new one is created.
static AwPrintManager* CreateForWebContents(
content::WebContents* contents,
@@ -29,12 +29,17 @@ class AwPrintManager : public printing::PrintManager,
private:
friend class content::WebContentsUserData<AwPrintManager>;
+
AwPrintManager(content::WebContents* contents,
const printing::PrintSettings& settings,
const base::FileDescriptor& file_descriptor,
const PdfWritingDoneCallback& callback);
- bool OnMessageReceived(const IPC::Message& message) override;
+ // printing::PrintManager:
+ bool OnMessageReceived(const IPC::Message& message,
+ content::RenderFrameHost* render_frame_host) override;
+
+ // IPC Handlers
void OnGetDefaultPrintSettings(IPC::Message* reply_msg);
printing::PrintSettings settings_;
@@ -44,4 +49,4 @@ class AwPrintManager : public printing::PrintManager,
} // namespace android_webview
-#endif // ANDROID_WEBVIEW_BROWSER_PRINT_MANAGER_H_
+#endif // ANDROID_WEBVIEW_BROWSER_AW_PRINT_MANAGER_H_
« no previous file with comments | « no previous file | android_webview/browser/aw_print_manager.cc » ('j') | components/printing/common/print_messages.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698